Huge load times for ecommerce site when having many products and categories

Found a bug? Please fill out the sections below.
Issue Summary
Oscar is having huge load times when a site has a lot of products and categories .
Steps to Reproduce
For example the page I am currently handling has at least 80k products and 500 categories and it slows down the whole page . Even homepage where no product is shown ( yes the navigation bar and what is shown is optimized that's why the load time is not as huge as in ProductCategoryView ) .
Any other relevant information. For example, why do you consider this a bug and what did you expect to happen instead?
Technical details

Python version: 3.8.6.
Django version: Django==3.2.7.
Oscar version: django-oscar==3.1.
Deployed using : nginx and uwsgi

It's not that the issue is fixed . Still trying to understand what's wrong . Meanwhile I've done some optimizations to my website , still slow . The optimizations were using some DJANGO REST API endpoints for navbar category set and URL's . I 've tried connecting to a remote database for more "" physical power "" ( More CPU , RAM dedicated  to Queries only ) still slow . I repeat the amount of products that my site contains is 81k products ( with variants ) and around 800 Categories. Yes wherever was appropriated I did use queries such as  select_related and prefetch_related. The question still remains the same . What is the maximum capacity for products and categories that oscar can handle

Hi @iSolorak, the numbers you mention (80k products and 500 categories) are quite a lot if you combine them together.
I've a few suggestions though;
Cache the header / navigation menu
Rendering the navigation menu with so many categories, is not ideal and should preferably be cached. This can simply be done by the Django template cache mechanism. In our sites, we cache for like 10/15 minutes and then after that the first user that comes on the site will have a slight delay but then it's cached again for 10/15 minutes.
You could also cache this with no time limit and remove the cache key once changes are made to a category, though I suggest the first one.
Use elasticsearch
We use elasticsearch on our webshops and I think this would help you as well. We've got a site with around a million products and it runs smooth and the main reason is ElasticSearch.
There's a package for django-oscar: https://github.com/django-oscar/django-oscar-elasticsearch
** Implement caching **
If possibly, you can implement some sort of caching. Category pages (normally) don't change very often, same with detail pages of products. Though, I don't know much about this, so I'm not sure what to recommend. We use varnish, but that's quite some work to get right.
Also, I'm curious on what kind of database your webshop runs?

YOu should be very very careful with the category_tree templatetag, only use it once on each page and render only to the depth you need. And make sure not to access item.category when looping through the items or you will make additional queries!

.

.

I am using postgresql

Also I do not understand why the issue was closed . Still having the same problem and I am eager to learn why it gets slower when having many products . Should I use more than one database ? Should I use remote database . I 've been searching for a while there is now answer anywhere .

@specialunderwear  @joeyjurjens


Revert "Marks Linux_android image_list_jit_reported_duration to be flaky"

Reverts flutter/flutter#93805. Based on how flaky this has been recently.
Fixes https://github.com/flutter/flutter/issues/93804

I believe we can close this PR. Added some details about the failure being caused by a single test bed consistently failing this test.

@godofredoc Do you mean we should keep the test marked "flaky"? This PR will remove the flaky flag. (It needs to be rebased, though.)

aaah, I got to the PR from the bug link and I thought it was going to mark it flaky. This test is not flaky anymore and we can remove the flaky property.

This needs a rebase, or just closing it and clicking the revert button on the original PR again.


[read-only]: cannot apply workspace edit for read-only workspace provider

In the latest Insiders, when trying to apply workspace edits to a read-only workspace, the applied edits have no effect. Works fine in the read-write mode. This breaks the co-editing scenarios for VS Live Share extension.

VSCode Version: 1.31.0-insider  <2018-12-18T06:07:13.414Z>
OS Version: Mas OSx 10.13.6

Steps to Reproduce:

Register a read-only workspace:

vscode.workspace.registerFileSystemProvider(
  'test-authority',
  /* your file system provider */,
  { isCaseSensitive: true, isReadonly: true }
);

Apply workspace edit(s):

const edits = /* edits */;
const workspaceEdit = new vscode.WorkspaceEdit();
workspaceEdit.set(uri, edits);
await vscode.workspace.applyEdit(workspaceEdit);

Observe that the edits are not applied with read-only file system provider.

Does this issue occur when all extensions are disabled?: Yes

Alternative repro:

Install VS Live Share extension from the marketplace on the latest Insiders(1.31.0-insiders).
Open a folder and click Start read-only collaboration session... in the Live Share viewlet.
Open another instance of the VSCode insiders and join the session.
On the host side, open a file and make some edits.
Notice that the edits don't get applied on the guest side.

cc @IlyaBiryukov

cc @jrieken, @isidorn This seems to be the same breaking change that was reverted in #57032 regressing again.

@isidorn sorry, missed your reply. Yes, it is still reproducible in the current Insiders(1.32.0-insider) and Stable(1.31.0). This completely breaks our read-only scenarios for VS Live Share. Working on the repro for you.

@isidorn there is the extension with the repro: fsprovider-sample.zip

Lunch the extension and run MemFS: Workspace Setup in command palette.
Run MemFS: Create Files in command palette.
Run MemFS: Start Bash.

The extension will try to insert a current timestamp string into beginning of the file.yaml in the root with interval of 1s. It logs the same message to the debug console for testing purposes. Notice that if the file.yaml is open in background, the edits come through, but if user makes the file active, the edits no longer applied.

Thanks for repro steps, reopening and assinging to february.

Ok I see that you reopened the issue, closing this one


All Stan processes finished, but it is still working

fit.check
returns
"All Stan processes finished."
fit.stop
returns
Stan is already finished.
but
fit.peek
gives
"Stan is still working. You can either:

Use the peek method to get partial samples
Come back later, or
Attach a listener to the StanFit object."

how is that possible ?

Ah, the peek method is quite untested. I suspect the message is incorrect. I imagine you actually have the correct number of samples in your StanFit output?

I can't tell because when I run
sim = extract(fit);
it tells me the Stan is still working

Even if I just type
fit
I get the same message

I have the same problem after 1 chain crashed early (possibly due to memory issues) but the other two chains finished.


Users can manually configure the XML file location and naming.

Users can manually configure the XML file location and naming.
用户可以手动配置xml文件位置与命名
andulir:
  scan-package: org.andulir.controller
  file:
    path-setting: root # If not filled, it will be manually set
    path:              # Optional, fill in the absolute path
    filename:          # Optional

目前，XML 文件只能在根目录以 aTest.xml 文件名显示，需要提供配置选项，以便灵活配置文件目录。
创建了一个配置Bean XmlFileProperty ，可以在 application.yml 中配置 xml 文件的目录与文件名格式。
Currently, the XML file can only be displayed in the root directory with the filename aTest.xml. Configuration options need to be provided to allow flexible configuration of the file directory. A configuration bean called XmlFileProperty has been created, which allows the directory and filename format of the XML file to be configured in application.yml.


Form dialogs: LG for RepromptHelp and MissingPrompt are the same for properties that are not enums

Describe the bug
For properties that are defined as types other than a list, the LG for RepromptHelp and MissingPrompt are the same, which causes the bot to ask the message twice in a row when an invalid answer is given.
Version
1.3.0
Browser

[x] Electron distribution
[ ] Chrome
[ ] Safari
[ ] Firefox
[ ] Edge

OS

[x] macOS
[ ] Windows
[ ] Ubuntu

To Reproduce
Steps to reproduce the behavior:

Create a form dialog (create a schema and generate the dialog)
Start the bot in Emulator and give an invalid answer. The bot will respond with both the RepromptHelp and MIssingPrompt, which are the same text:

It looks like RepromptHelp and MissingPrompt are the same for property types datetime, geography, and integer:

but not for type enum:

Expected behavior
The bot should not say the same message twice in a row it looks like an error, and it might be confusing to debug: the user might not realize that two different LG templates are firing. Maybe change the MissingPrompt for the non-enum types to "what value would you like for ?" This would align with the prompts for enums

@coldplaying42 how was this resolved?

@coldplaying42 how was this resolved?


Fixed statusbar error when focusing to webview

Hi Waldo,
I have seen that your extension is subscribed to vscode.workspace.onDidOpenTextDocument event. This subscription causes errors when there is no active TextEditor.
1. A Webview is focused
First case is no big issue as it happens rarely, but still it should be fixed. :)
2. No open document
Second case, however, can easily become a real performance-killer when another extension decides to read up all AL files via VSCode api, thus emitting onDidOpenTextDocument event like 6000 times for a BaseApp.

The fix
My proposed solution would be to move your statusbar update code from workspace.onDidOpenTextDocument to window.onDidChangeActiveTextEditor with some additional error checks to avoid unexpected exceptions.
It would be a big help for me, as AL Studio is currently crashing during activation phase when AL XML Documentation is present, indirectly overloading VSCode by an event subscription in your extension. Simple times indeed :)
Thanks,
Marton

thank you so much!  I will merge these in asap!

Thank you! In the meantime, I have also added a pull request to AL XML Doc in order to get to the bottom of the root cause as well. :)
https://github.com/365businessdev/vscode-alxmldocumentation/pull/34


Refactor convert to tablet sink to tablet writer

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run buildall

run buildall

run buildall

run buildall

run buildall


Attribute morphology consolidation

STYLE: Documentation and removal of code hidden by macros
This PR consolidates documentation between parent and child classes - previously the child
classes had copies of the parent documentation.
The PAMI macro has been removed. Originally there were two versions
of the code from consecutive publications of the algorithm. The
non PAMI code was incorrect and retained for historical purposes only.
PERF: Used stl::stable_sort instead of standard sort of a class containing
both offset and pixel intensity. This slightly reduces RAM use.
Changes suggested as a result of:
@zivy SimpleITK/SimpleITK#1656
PR Checklist

[ ] No API changes were made (or the changes have been approved)
[ ] No major design changes were made (or the changes have been approved)
[ ] Updated API documentation (or API not changed)

@N-Dekker I am seeing some local new[]/deletes[] here that would be good candidates for make_unique_* improvements. Am I correct that you are not applying your improvements to the Review module?

Am I correct that you are not applying your improvements to the Review module?

Thanks for asking @blowekamp  I see now, Module_ITKReview is OFF in my local CMake configuration! So yes, indeed, I did not include the Review module! Would you like to do it, or shall I take a look?

Maybe turn on a few more non-default modules, such as ITKFEMRegistration?

@N-Dekker You are likely more efficient and know what to look for with your experience in making this change. Thank you.


Make Frontinus skip the tutorial if you have peanuts

https://github.com/michaelskyba/savo/commit/c6aa5e2cf6ce5470e3f96390b2a75c0b9a0f3abf


Are pub(crate) members intentional?

Currently some of the elements have members with their visibility set to pub(crate) which makes it impossible to write external formatters. Is this intentional?

Oh well, I see. Yeah that needs to be changed.


To go riak ping

Branched from #2365 - recommend merging that first.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR/1797/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR/1807/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR/1849/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/1903/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/2047/
Test PASSed.


TAB not working

VSCode Version: Code - Insiders 1.15.0-insider
OS Version: WINDOWS 10 x64

Steps to Reproduce:
Open an HTML file or create and save a new one
Type any tag, for example div
Hit "tab" -- it should display
Same thing with Emmet shortcuts, and in SCSS files as well.
Nothing happens. This used to work, I believe it worked in yesterday's Insider build.

Reproduces without extensions: Yes

@jpfrontbr In the Insiders for 1.15,we have set emmet.useNewEmmet to true
Meaning the new emmet model is in effect where tab is no longer bound to the emmet expand abbreviation command.
Read https://code.visualstudio.com/updates/v1_13#_emmet-abbreviation-expansion-in-suggestion-list and https://code.visualstudio.com/updates/v1_14#_emmet-abbreviation-improvements to know more on this.


No matching resource found in the API for the given request

I publish api according to the wso2 restful api.
Setps:
1、create my api with " /api/am/publisher/v0.14/apis"
2、publish my api with "/api/am/publisher/v0.14/apis/change-lifecycle"
3、subscribe api to application with "/api/am/store/v0.14/subscriptions"
I got the response when I invoke api in store-ui.The response is:
{"fault":{"code":900906,"message":"No matching resource found in the API for the given request","description":"Access failure for API: /giteaapi.v2/p0ilyf/1.1.1, version: 1.1.1 status: (900906) - No matching resource found in the API for the given request. Check the API documentation and add a proper REST resource path to the invocation URL"}}
How can I solve this problem?

I had the same problem.

Even I am facing the same problem. As a workaround, EDIT the API from /publisher and go to Manage section.
Just click on save button present down and then try to invoke the API. It should work.
But I guess, this is a bug in 2.6.0. I have also reported the same in git. Lets see if we get any response on this.

Seems like the resources are not created. Can you send the payload you used to create the API please?

Hi,
Thanks for your email.
Attached is the payload.
On Mon, Oct 14, 2019 at 2:15 PM Chamila Adhikarinayake <
<EMAIL_ADDRESS>wrote:

Seems like the resources are not created. Can you send the payload you
used to create the API please?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/wso2/product-apim/issues/5487?email_source=notifications&email_token=AKW6QEY6ENOKP3XFRVBFZLDQORPH3A5CNFSM4IUGUSDKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBEM6EI#issuecomment-541642513,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKW6QE5LE7LKBYVINDIU4W3QORPH3ANCNFSM4IUGUSDA
.

@chamilaadhi: am I missing anything in this request or its a valid scenario?

hi, sorry but i couldn't find any attachment with the payload in this git issue. Can you attach it again please?

APICreate_payload.json.txt
@chamilaadhi Please find it attached. I have changed the extension from .json to .txt

@chamilaadhi I have sent you an email with the payload, please chick your email.

@chamilaadhi did you get chance to check on this issue?

Hi all,
Found the problem here. "code":900906 related issue is coming due to authorization related issues as well (no resource found with valid authorization criteria). When API is created from API Manager publisher UI, it sets custom elements to the swagger for authorization (x-auth-type) and for throttling (x-throttling-tier). In your swagger definition (apiDefinition element) in your payload, these are missing. You will have to add following to each resource to get it working
  x-auth-type: "Application & Application User"
  x-throttling-tier: Unlimited

When you go to Manage tab and save, what happens is, it sets these missing elements to the swagger. Thats why it works after saving. (You could compare the swagger definition before and after save to see above missing element)
Hope you could get it working after adding this. I have attached the updated swagger file for your reference
swagger-updted.json.txt

@chamilaadhi Thanks for your response.
Yes, by adding this I am able to create an API and invoking does not give any more errors. However, one thing I want to add here, that by adding this we are actually changing the swagger file of the API and it reflects also in the swagger file when we pull it.
So, ideally this should not be the case? Any way to handle it?

@chamilaadhi It worked

@chamarasilva : Could you please guide on the above question.

I'm also interested, is there a way to make it work without having to modify the swagger file? This used to work fine in 2.1.0.


Embedded link with slash within anchor text does not work.

For example, try opening a test
'..\directory\myfile.cpp]' is not correct.
This is due to the regex driving link discovery in SdkUIUtilities.EmbeddedLinkPattern. The  group will halt on encountering a backslash.  See the ^\\\] segment below:
        // Embedded link format: [link text](n|uri) where n is a non-negative integer, or uri is an absolute URL
        private const string EmbeddedLinkPattern = @"\[(?<text>[^\\\]]+)\]\((?<target>[^)]+)\)";

@EasyRhinoMSFT may have some insight into why this exists or maybe there is a clue in the unit tess

With this test file:
Bug117.sarif.txt
... the link now renders (and functions) properly:


strange behaviour - ha os 10.3 - rpi 3

this add-on works when first install it but doesn't after reboot, i have to restart service to connect client to hostapd wifi. it is set to start at boot but doesn't seems like it is running even though add-on info has stop and restart options, and no cpu % activity. after every reboot or ha restart i have to go back to add-on and restart. any way to fix this?
s6-rc: info: service legacy-cont-init successfully started
s6-rc: info: service legacy-services: starting
s6-rc: info: service legacy-services successfully started
Starting Hass.io Access Point Addon
Setup interface:
Add to /etc/network/interfaces: iface wlan0 inet static
Run command: nmcli dev set wlan0 managed no
Run command: ip link set wlan0 down
Add to /etc/network/interfaces: address <IP_ADDRESS>
Add to /etc/network/interfaces: netmask <IP_ADDRESS>
Add to /etc/network/interfaces: broadcast <IP_ADDRESS>
Run command: ip link set wlan0 up
Setup hostapd:
Add to hostapd.conf: ssid=homeFI
Add to hostapd.conf: wpa_passphrase=********
Add to hostapd.conf: channel=6
Add to hostapd.conf: ignore_broadcast_ssid=0
Add to hostapd.conf: macaddr_acl=0
Add to hostapd.conf: interface=wlan0
DHCP enabled. Setup dnsmasq:
Add to dnsmasq.conf: dhcp-range=<IP_ADDRESS>,<IP_ADDRESS>,12h
Add to dnsmasq.conf: interface=wlan0
Add DNS: dhcp-option=6,<IP_ADDRESS>
Starting dnsmasq daemon
Starting hostapd daemon
rfkill: Cannot open RFKILL control device
wlan0: interface state UNINITIALIZED->ENABLED
wlan0: AP-ENABLED
update
it seems to an issue with the integrated wifi chip on the rpi3 board. tested with usb rtl8812 chip with no issue.

Hi, does this still occur in the newest version with updated HASSOS?


inscription_feed error!

Prerequisites

[X] I have searched for duplicate or closed issues
[X] I understand the code of conduct
[X] I have read the contributing guidelines

Describe the issue
chainhook-node  | {"msg":"Trigger http://<IP_ADDRESS>:3099/chainhook/inscription_feed failed with status 500 Internal Server Error","level":"WARN","ts":"2023-07-12T02:50:31.904950704Z"}
chainhook-node  | {"msg":"unable to send request after several retries","level":"ERRO","ts":"2023-07-12T02:50:31.904977795Z"}
chainhook-node  | {"msg":"Initial ingestion failing","level":"ERRO","ts":"2023-07-12T02:50:31.909487451Z"}
chainhook-node  | {"msg":"Trigger http://<IP_ADDRESS>:3099/chainhook/inscription_feed failed with status 500 Internal Server Error","level":"WARN","ts":"2023-07-12T02:50:31.932226621Z"}
What operating system(s) are you seeing the problem on?
Linux
What browser(s) are you seeing the problem on?
No response
What version are you using?
chainhook 0.16.0

@pupudaye I've moved your issue to our new repo https://github.com/hirosystems/hord which is dedicated to ordinals.
I invite you to follow the steps from the README, this new tool is way easier to approach and use, while providing the same features, from an ordinal point of view.
Feel free to re-open issues if you're having some troubles getting started with hord.


Add support for scalar item assignment by Series

Adds a check in DataFrame.__setitem__ for when key is a series or series-like; currently uses df.where, as would be done if key was a dataframe or dataframe-like.

[x] Closes #8190
[ ] Tests added / passed
[ ] Passes black dask / flake8 dask / isort dask

Thanks @charlesbluca and @jcrist! Merging :)


Add max props to limit the number of selected values for selectionStore, Selection, MultiSelect, MultiSelectField, and MultiSelectMenu

Current dependencies on/for this PR:

main

PR #231   👈

This stack of pull requests is managed by Graphite.


Fix on_train_batch_end signature and call in ProgressBarBase example

What does this PR do?
This PR fixes the docs example usage for ProgressBarBase. Specifically, on_train_batch_end takes more arguments than shown.
Does your PR introduce any breaking changes? If yes, please list them.
None
Before submitting

[N/A] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[x] Did you read the contributor guideline, Pull Request section?
[X] Did you make sure your PR does only one thing, instead of bundling different changes together?
[X] Did you make sure to update the documentation with your changes? (if necessary)
[X] Did you write any new necessary tests? (not for typos and docs)
[X] Did you verify new and existing tests pass locally with your changes?
[X] Did you list all the breaking changes introduced by this pull request?
[X] Did you update the CHANGELOG? (not for typos, docs, test updates, or internal minor changes/refactorings)

PR review
Anyone in the community is welcome to review the PR.
Before you start reviewing make sure you have read Review guidelines. In short, see the following bullet-list:

[ ] Is this pull request ready for review? (if not, please submit in draft mode)
[ ] Check that all items from Before submitting are resolved
[ ] Make sure the title is self-explanatory and the description concisely explains the PR
[ ] Add labels and milestones (and optionally projects) to the PR so it can be classified

Did you have fun?
Make sure you had fun coding 🙃

Hello @stfwn! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file pytorch_lightning/callbacks/progress.py:

Line 81:121: E501 line too long (129 > 120 characters)


Remove KubeClient from webhookParameters

We're retrieving the client from the MultiCluster object now.
This is a leftover from https://github.com/istio/istio/pull/49336 and https://github.com/istio/istio/pull/49689
Please provide a description of this PR:

@dgn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-telemetry-mc_istio
ebfeb8e3869c385ddf072dc91ac3add7fb25b330
link
true
/test integ-telemetry-mc

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


Preserve spaces on resize

PR fixes bug when spaces will be deleted on resize

re-approved; i fixed the merge conflict (metainfo.xml) and did a pico code cleanup


Fix attribute parsing

This PR adds tests for attribute parsing and fixes bugs found. It introduces new AttrError type that now returned by Attributes::next() and contains all possible errors when attributes is parsed.
Also it introduces a new type Attr which is by design is borrowed-only. In version 0.24 I plan to change Attributes iterator and return that new type.
@dralley, it would be great to run performance tests and see the results and maybe make optimizations like that you've done in #3

Sure, I'll try to review in the next few days (when time allows)

Is this an exhaustive list of the various issues with the previous parsing algorithm?
    events::attributes::html::duplicated::with_check::double_quoted
    events::attributes::html::duplicated::with_check::key_only
    events::attributes::html::duplicated::with_check::single_quoted
    events::attributes::html::duplicated::with_check::unquoted
    events::attributes::html::single::missed_value
    events::attributes::html::sparsed::double_quoted
    events::attributes::html::sparsed::key_contains_invalid
    events::attributes::html::sparsed::key_only
    events::attributes::html::sparsed::key_start_invalid
    events::attributes::html::sparsed::missed_value
    events::attributes::html::sparsed::single_quoted
    events::attributes::html::sparsed::unquoted
    events::attributes::xml::duplicated::with_check::double_quoted
    events::attributes::xml::duplicated::with_check::key_only
    events::attributes::xml::duplicated::with_check::single_quoted
    events::attributes::xml::duplicated::with_check::unquoted
    events::attributes::xml::duplicated::without_check::key_only
    events::attributes::xml::duplicated::without_check::unquoted
    events::attributes::xml::first::key_only
    events::attributes::xml::first::missed_value
    events::attributes::xml::first::unquoted
    events::attributes::xml::single::key_only
    events::attributes::xml::single::missed_value
    events::attributes::xml::sparsed::missed_value

I assume this is an exhaustive list of (tests demonstrating) the various issues with the previous parsing algorithm?

Yes, it is. If you checkout this commit you will see what exactly errors present. Particularly, in some cases key contains spaces, in many cases after returning Err next next() call returns None and in some returns next attribute.
The attributes parser itself is very rough and do not follow specification. It doesn't check for illegal symbols in a key and unquoted value, entities does not expanded (but that is fine at that level -- you anyway needs an entity manager to get a list of defined entities). For now I left that as "by design" -- correctness of each attribute should be checked after getting it. But I have plans to try to replace manually written parser with one generated by grammar using rust-peg with more robust parsing.
If you want to work on attribute normalization, some my minds: first, you need to introduce an entity manager. That manager should collect entities defined in a document and allow user to define external entities. Collecting entities requires, in turn, DTD parser. There already exist crates for that, I did not look deeply. May be we can use some of them.
Then normalization would require reference to the entity manager. No need to store it inside an attribute, it is better to create a dedicated type for a well-formed attribute that would created by normalization. In fact, that PR already did that differentiation: Attr by design contains borrowed unprocessed data to the input, and created only by the parser, but Attribute could be created by the user and have own data.
Introducing entity manager and doing attribute normalization could be done in different PRs, because entity manager should be used everywhere

@Mingun I'm going through commit-by-commit so this isn't a final review yet.  I'll continue in a few hours.
The code looks great so far, I only have a few minor grammatical / wording nitpicks.

Ok. Yeah, grammar should be checked because I'm not a native speaker :) I tried, but native speaker always can write it better

@dralley, tell me when you finish with review. I'll plan to make a 0.23 release after merging this.

@Mingun Barring the two docstrings that need to be fixed, let's call this approved - I'd like to experiment for myself to either better understand the problems you've discussed or figure out a way around them - but since that will take some time I won't block the PR on it.  The tests pass at least.

@dralley, thanks for your work on reviewing this PR!


feat: add video reader

based on https://github.com/gcanat/video_reader-rs

Does it work with gpu?

any reason not to use https://pypi.org/project/PyNvVideoCodec/ ?

for CPU : why not good'ol OpenCV since under the hood `video_reader-rs leveraged ffmpeg as well

https://github.com/gcanat/video_reader-rs?tab=readme-ov-file#rocket-performance-comparison
https://gitlab.com/jfolz/simplejpeg/blob/master/README.rst:

OpenCV is gigantic, only accepts Numpy arrays as input, and returns images as BGR instead of RGB.


Generate database schema when class is extends with abstract class

Currently when class is extended with abstract class the database schema is not generated with doctrine:schema:update. This commit fix that.

Looks fine for me (at least it will allow not to specify all the classes in hierarchy), but pretty please do fix unit test for newer versions (see travis).

It's work with PHPUnit 4.0.0 but not with PHPUnit 4.6.4 I don't know why :(

Can you take a look on the build? https://travis-ci.org/simplethings/EntityAudit/jobs/59362013 ? It's not a phpunit issues, it just breaks things with 1:M somehow...

Yes I looked. But even if I try without my change the unit test don't pass...
It's not with my little change :( I don't know why

Well, currently the master branch tests will fail.
There are several errors of this kind:
Entity 'SimpleThings\EntityAudit\Tests\AnimalAudit' has to be part of the discriminator map of 'SimpleThings\EntityAudit\Tests\AnimalAudit' to be properly mapped in the inheritance hierarchy. Alternatively you can make 'SimpleThings\EntityAudit\Tests\AnimalAudit' an abstract class to avoid this exception from occurring.
Also 2 errors with already existing tables:
SQLSTATE[42S01]: Base table or view already exists: 1050 Table 'escapedcolumnsentity' already exists' while executing DDL: CREATE TABLE EscapedColumnsEntity (id INT AUTO_INCREMENT NOT NULL, lft INT NOT NULL, left INT NOT NULL, PRIMARY KEY(id)) DEFAULT CHARACTER SET utf8 COLLATE utf8_unicode_ci ENGINE = InnoDB
and more

It work's !
Unit test suite don't work with doctrine/orm >= 2.5.
I fixed version on composer.json

Hmm this conflicts with the already merged https://github.com/simplethings/EntityAudit/pull/126

true, doctrine/orm 2.5 was released 21 days ago

doctrine/orm ~2.2 update all version before branch 3
2015-04-23 15:31 GMT+02:00 sebastianreloaded<EMAIL_ADDRESS>
true, doctrine/orm 2.5 was released 21 days ago
—
Reply to this email directly or view it on GitHub
https://github.com/simplethings/EntityAudit/pull/127#issuecomment-95586003
.

jep, got that ;)

Guys, it's cool, but we have to support backwards compatibility. I can not accept PR that rises doctrine to 2.4 when symfony is bundled against ~2.2. Can you find another fix?

We can't force the version to 2.4 because we would save backwards compatibility in feature releases. We need a different approach.

..and future compatibility too, so please remove <2.5 and stick to ~2.2.

Yes but your unit test suite don't work with actual last version 2.5...
2015-04-24 9:23 UTC+02:00, Andrew Tch<EMAIL_ADDRESS>
..and future compatibility too, so please remove <2.5 and stick to ~2.2.

Reply to this email directly or view it on GitHub:
https://github.com/simplethings/EntityAudit/pull/127#issuecomment-95832588

Then we have to fix it, because symfony works fine with 2.5 afaik.

i have restart travis for current master: https://travis-ci.org/simplethings/EntityAudit/builds/56563337
so we can see what is broken

But the last pr in master already breaks symfony compatibility, which has doctrine/dbal < 2.5. Then this should be reverted too.

they also removed doctrine/orm 2.5 support: https://github.com/symfony/symfony-standard/commit/acd2c335957a0bc2377367c0547a50f4eb05f1a2

Please create a new PR if the error still exists, thanks!


session.loadAll sorts by SortOrder specified instead of by ids, fixes #302

To be merged to 2.1.x as well


Mips: Error relocating /usr/bin/netbird: __nanosleep_time64: symbol not found

Using the latest netbird_0.8.2-1_mips_24kc.ipk

Looks like out of memory problem.

@a1ad can you share the output of:
cat /etc/openwrt_release

DISTRIB_ID='OpenWrt'
DISTRIB_RELEASE='21.02.3'
DISTRIB_REVISION='r16554-1d4dea6d4f'
DISTRIB_TARGET='ath79/generic'
DISTRIB_ARCH='mips_24kc'
DISTRIB_DESCRIPTION='OpenWrt 21.02.3 r16554-1d4dea6d4f'
DISTRIB_TAINTS=''

Same error for netbird_0.9.7-1_mipsel_24kc.ipk
Device is https://openwrt.org/toh/xiaomi/xiaomi_redmi_router_ac2100
# cat /etc/openwrt_release
DISTRIB_ID='OpenWrt'
DISTRIB_RELEASE='21.02.1'
DISTRIB_REVISION='r16325-88151b8303'
DISTRIB_TARGET='ramips/mt7621'
DISTRIB_ARCH='mipsel_24kc'
DISTRIB_DESCRIPTION='OpenWrt 21.02.1 r16325-88151b8303'
DISTRIB_TAINTS=''

My local build with this go command (not upx-ed) works:
CGO_ENABLED=0 GOOS=linux GOARCH=mipsle GOMIPS="softfloat" go build -ldflags "-s -w -X 'github.com/netbirdio/netbird/client/system.version=$TAG'" -o netbird .

Same error with v0.16.0 and v0.15.3
root@Teltonika-RUTX50:~/3rd_party_opkg# cat /etc/openwrt_release
DISTRIB_ID='OpenWrt'
DISTRIB_RELEASE='21.02.0'
DISTRIB_REVISION='r16279-5cc0535800'
DISTRIB_TARGET='ipq40xx/generic'
DISTRIB_ARCH='arm_cortex-a7_neon-vfpv4'
DISTRIB_DESCRIPTION='OpenWrt 21.02.0 r16279-5cc0535800'
DISTRIB_TAINTS='no-all busybox'

Testing netbird
root@Teltonika-RUTX50:~/3rd_party_opkg# netbird
Error relocating /usr/bin/netbird: __nanosleep_time64: symbol not found


Real-time stream

Required Info

Camera Model
D435i

Firmware Version
Open RealSense Viewer

Operating System & Version
Ubuntu 18.04

Kernel Version (Linux Only)
5.4.0-72-generic

Platform
PC

SDK Version
l2.0

Language
Python

Segment
{Robot/Smartphone/VR/AR/others }

Issue Description
Hello all,
I am looking for a real-time stream solution for realsense D435i camera. I would like to use real-time infrared and RGB stream. Can you suggest any repository written in C/C++ or Python?
Also I want to save the stream in mp4 format for other processes.

Hi @DuyguSerbes   I am not aware of an example of streams being saved in mp4 format, though a RealSense user shared a Python script for saving the streams in .avi video format.
https://github.com/IntelRealSense/librealsense/issues/2731#issuecomment-529938267
If you require mp4 format, building librealsense with the V4L2 backend should enable raw camera data to be accessed by standard Linux tools.
https://github.com/IntelRealSense/librealsense/issues/6841#issuecomment-660859774
In regard to real-time streaming: the link below provides scripting for aligning depth, color and infrared.  Depth is necessary to be included in that situation because the SDK needs for depth to be aligned to another type of stream, so you cannot have just RGB and infrared alignment, for example.
https://github.com/IntelRealSense/librealsense/issues/5093
If you prefer for the streams to be displayed as individual panels instead of in alignment, a RealSense user in the link below shared a modified version of the opencv_viewer_example Python example program that includes infrared.
https://github.com/IntelRealSense/librealsense/issues/3363
Conceivably, if you only wish RGB and infrared to be displayed then you could remove the depth stream code from the script, since alignment is not involved in this particular application.

@DuyguSerbes if you prefer python another alternative is ffmpeg-python https://github.com/kkroening/ffmpeg-python
There is no repository with RealSense code, but the library is well documented. With these settings I am usually able to get a 720p 30fps stream going on a Raspberry Pi 4. This also uses the Raspberry Pi's hardware encoder, which is much more efficient than OpenCV.
process = (
    ffmpeg
    .input('pipe:', format='rawvideo', pix_fmt='yuv420p', s=str(width) + 'x' + str(height))
    .output('pipe:', format='h264', vcodec='h264_omx', framerate=framerate, video_bitrate='10M')
    .global_args('-an')
    .run_async(pipe_stdin=True, pipe_stdout=True)
)

You can use OpenCV to convert an RGB8 color frame to yuv420p. Unfortunately RealSense's YUYV and yuv420p are different formats.
You can feed the color frame to ffmpeg with process.stdin.write(coloryuv).
You can have ffmpeg write straight to disk, but if you are writing to an external USB drive I have had issues where ffmpeg blocks the USB bus and the RealSense camera drops frames. If you pull the encoded bytes yourself with something like data = process.stdout.read(16384), then you can write that data to disk or stream it somewhere.
This will technically be "h264" and not "mp4", but you can transcode the stream without recompressing it.
(
	ffmpeg
	.input(filePath, framerate=int(framerate))
	.output(filePath.replace(".h264", ".mp4"), c='copy')
	.global_args('-loglevel', 'error')
	.global_args('-y')
	.run()
)

Also if you want more detail than H264 and can deal with a higher data rate, motion JPEG might be interesting.
This library https://pypi.org/project/PyTurboJPEG/ takes about 10ms to compress a 848x480 RGB image and 20ms to compress a 1280x720 RGB image on a Raspberry Pi. A desktop computer should be able to do much better.

Case closed due to no further comments received.


Assessment questions should be linked to content

This is a big one, but a huge benefit for the student.
Assessment questions should have a way to link to resources or activities.  This way when the student is reviewing their assessment results they can follow the links back to the content of the questions they got wrong.
This will need a design session to plan correctly.

This is way out for now.. we need to bring assessments back in the mix first before this happens.  Keeping it around so I don't forget.

Legacy.. Closing
Use cases / direction / design to be re-evaluated


lsq method only

In this merge, we used only scipy.optimize.leastsq method to fit the model to the synthetic lightcurves instead of using scipy.optimize.minimize method.

lsq method only


Bump gvisor-tap-vsock image to latest version

It is now using DHCP to configure the network card.

I need to also bump the vendoring on crc side at the same time.

/unhold
/retest
It is working perfectly now!

I think same would be require for 4.8 branch
/cherry-pick 4.8


Tor/SOCKS5 proxy support

Hi, I'm adding optional Tor/SOCKS5 proxy support to stellar_flutter_sdk and would like to contribute this feature back upstream.
I'm approaching it by focusing on the HttpClient in https://github.com/Soneso/stellar_flutter_sdk/blob/33c12d7a3f0ef3486707dea5884ac6cd4459bbb9/lib/src/stellar_sdk.dart
Do you have any suggestions or requirements for merging such functionality?

This week I have so far submitted PRs upstream for

Solana: https://github.com/espresso-cash/espresso-cash-public/pull/1386
Tezos: https://github.com/moneytrackio/tezart/pull/103

and will use something akin to either of these approaches here.  Please let me know if you'd like it done in any particular way :)

Sounds good, thank you! Looking forward to your PR!

Done in this release 1.7.8


TO DO & Review

[x] Follow consistent naming convention for functions and variables
[x] MMTF fetch, parse, write APIs
[x] Ensure Type safety where needed
[x] Verify if Host to BigEndian(Base.hton) and BigEndian to Host(Base.ntoh) is sufficient on all systems.
[x] Setup CI Build
[x] Add proper comments to the code
[x] Write Test cases
[x] Should we allow custom fields to be decoded and encoded? No - breaks the MMTF spec and other packages don't provide.
[ ] What to do for ncsOperatorList field(not decoded/encoded in MMTF python version). Looking for PDB with ncsOperatorList. Most of them have empty array for ncsOperatorList
[ ] Possible code improvements for better readability and performance
[ ] Update README file with instructions and examples
[ ] Publish the package

So I had a look over the code as you asked, it looks good so far. The things that I spotted are:

Often Julia packages tend to have a single file which defines the module and includes all the other files, I think that could be useful here to aid readability. So you could move all code out from MMTF.jl to another file and do all the includes in MMTF.jl.
MMTFDict could be its own struct, containing a single Dict{String, Any}. This means that down the line people could write functions with f(d::MMTFDict) and know they are getting an MMTF dictionary. In order to make standard dictionary functions work you would have to define getindex, setindex!, keys, values, haskey etc, an example of this can be seen for MMCIFDict at https://github.com/BioJulia/BioStructures.jl/blob/master/src/mmcif.jl#L40-L65. This is up for discussion of course, there are pros and cons to this approach.
[] produces Any[], when in many cases you want an array of a particular type to ensure type stability. This affects the default values in decodedata, codec.jl lines 8/29/54/70, and also some lines in converters.jl.
MMTF.jl line 40: use haskey(input_data, field), and anywhere else you check keys.
Putting docstrings above funtions means they are linked to the function and can be accessed with ?.
codec.jl line 158 and 183: should these throw an ArgumentError? It depends if you want the program to continue after this failure.
utils.jl line 1 and 5: if T=Number then Number can just go in place of T, do you mean T<:Number? In fact, Number includes Complex, you might want Real instead. Also the where typically comes outside the closing bracket.

I agree with the above list of tasks. In particular it would be good to get the test cases written, to check that the expected output is the same as the actual output.

@jgreener64 Thanks for the review and insight! I will look into them and make the necessary the changes. Will let you know once it's done.

@jgreener64
I tried implementing MMTFDict as a struct. But it appears many of the Base functionalities are lacking. Say for example
for (key,value) in mmtfdict
     println(key,value)
end

This is not possible. It appears I have to extend the Base.start and other functions as well. Wouldn't it be better to keep it as Dict alone in this package? So user wanting to use this package alone can play with the data with all Julia available base functions for Dict.
Maybe when integrating with the BioStructures.jl we can declare an MMTFDict struct as a wrapper for our Dict?
I think it's better to have this package similar to mmtf-java, mmtf-python and mmtf-javascript. Like a general package for reading MMTF files.
Maybe we can move the package to BioJulia.

Okay fair enough, this makes sense. The Dict can be kept as is.
Yes, when the package is ready we should move it to BioJulia. Perhaps it makes sense to keep this as a package separate from BioStructures.jl (i.e. BioJulia/MMTFL.jl), and you can be the maintainer of it? BioStructures.jl can then import and wrap it. This suits the modular nature of Julia packages, and keeps it similar to mmtf-python etc packages.
Let me know what you think about this.

@jgreener64 Sure. That would be grateful. I can be the maintainer of the package and would be happy to have the package under BioJulia. Looking forward to contributing even more in the future. Thank you.

@jgreener64 I have written the test cases and everything seems to be working good! Except I had this issue https://github.com/joels94/MMTF.jl/issues/2. Kindly review the code once again and test the package. Let me know if you have an insight on https://github.com/joels94/MMTF.jl/issues/2 and any code improvements. Thank you

Nice one, looks good. The only comments I have now are:

Having tried parsemmtf I was getting an error and couldn't work out what it was. Then I noticed that parsemmtf wants a zipped MMTF file (e.g. "4HHB.mmtf.gz") and doesn't work on an unzipped one, at least on Linux/Mac. See some more on this at https://github.com/joels94/MMTF.jl/issues/2. Basically my opinion is that parsemmtf and writemmtf should read/write unzipped MMTF files as default, and fetchmmtf can do the unzipping after download as required. Or you could look for .gz extensions and unzip in this case. Let me know what you think.
api.jl lines 14-38 I think these [] should have types where appropriate to ensure type safety of the dict. If they are always overwritten below then there is no need to define them here, they can just be added to the dictionary below.
api.jl line 44 I think this should throw an ArgumentError to allow users to catch it safely if required.

Actually, one more thing. I think parsemmtf/writemmtf should be able to take a filename (as currently) or an open file stream, so users can use them with files they have opened themselves. So for example you could do something similar to:
function parsemmtf(filepath::AbstractString)
    open(filepath) do f
        return parsemmtf(f)
    end
end

function parsemmtf(f::IO)
    return decodedata(unpack(f))
end

Nice one, I think this is ready to be moved over to BioJulia org. I'll get you added to the org, then you can make a new repo there and commit this code. The README can be set up similar to the other BioJulia packages and then you can tag and release.
There is one consideration, which is the package name. Perhaps mmtf.jl (lower case)? This matches some of the MMTF packages for other languages, and I'll also admit it makes it easier to integrate into BioStructures.jl - there is a name clash between the package and one of the BioStructures.jl types currently.
However I am willing to go with your feeling on this as it is your package of course.

@jgreener64 Nice to know. I am okay with the package name mmtf.jl. I just thought MMTF.jl will match Julia packages nameconvention (Eg. EzXML.jl, JSON.jl, etc). Since there is clash between BioStructures.jl types, we can make it into mmtf.jl

Thanks for being open to the change. However I asked some Julia people on Slack and it seems in Julia v0.7 I will be able to import the package into BioStructures.jl without name clash by importing specific functions.


Fix wrong test input data for dropping leading spaces

A test case in textformatter lacked the leading spaces it was supposed to test
for.

Coverage increased (+0.3%) to 30.598% when pulling 892078b7c6e5a94fd4f8ecc5c72a7a1a4c6e7363 on tsipinakis:test-drop-whitespace into 3241e10f3955fce9f48a6031799952fca1f50cc7 on akrennmair:master.

Coverage increased (+0.3%) to 30.598% when pulling 892078b7c6e5a94fd4f8ecc5c72a7a1a4c6e7363 on tsipinakis:test-drop-whitespace into 3241e10f3955fce9f48a6031799952fca1f50cc7 on akrennmair:master.

Coverage increased (+0.3%) to 30.598% when pulling 892078b7c6e5a94fd4f8ecc5c72a7a1a4c6e7363 on tsipinakis:test-drop-whitespace into 3241e10f3955fce9f48a6031799952fca1f50cc7 on akrennmair:master.

Coverage increased (+0.3%) to 30.598% when pulling 892078b7c6e5a94fd4f8ecc5c72a7a1a4c6e7363 on tsipinakis:test-drop-whitespace into 3241e10f3955fce9f48a6031799952fca1f50cc7 on akrennmair:master.

Coverage increased (+0.005%) to 30.42% when pulling e89b0e2804f64ab895347cc7a3c296fc15808a9f on tsipinakis:test-drop-whitespace into 1fa890123580556846e57dd2b8343faed1c7de96 on akrennmair:master.

For the record: me and @tsipinakis discussed this PR on IRC last night. The main takeaway is that test description could be worded better. What I meant when I wrote it is that after wrapping, lines don't start with space; i.e. if space doesn't fit onto a line and is going to be put at the beginning of a new line, it should just be ignored. The test case actually demonstrated that, but suboptimal wording confused people (including me—it's been only 3 months since I wrote it but on the first glance, I decided that the test is useless).
Yesterday I also voiced an idea: if first line of wrapped text ends up consisting solely of whitespace, it should be dropped. @tsipinakis kindly updated the code and the test to reflect this behaviour, but after some sleep I no longer believe it's really necessary ._.
A situation where whitespace takes up a whole line is extremely unlikely unless we're working on an extremely narrow screen (think a dozen columns wide). I don't believe it makes sense to complicate our code to make a marginal improvement in such an unlikely scenario.
So I'd like to change the scope of this PR to updating the test description. @tsipinakis, do you agree? Do you have any ideas about better wording for the test?

Agreed.
Perhaps

When wrapping, ignore leading whitespace wrapped to the next line from the end of the previous one

would be a better description for the current test.

Don't think we need "When wrapping" part at all, we're mentioning the function name (wrap_line) already. The rest of it seems a bit too complicated to me. Maybe "ignore whitespace that's going to be wrapped onto the next line"?
Or maybe we should use your wording and just write an extensive comment like one that I wrote in the previous post, explaining what naive function will do and how wrap_line intelligently deviates?

ignore whitespace that's going to be wrapped onto the next line

Indeed sounds a lot better. To avoid any confusion a comment with an example would be a good idea either way.

So… you'll update the PR, right? :)

Coverage remained the same at 30.401% when pulling 289e054adcfedd2067d4ba7bfb750922d212d049 on tsipinakis:test-drop-whitespace into 765b053d7ed535244cfd093de04c6911611a4b86 on akrennmair:master.

That looks perfect, thank you!


Fix long taps for mobile web.

@brockwhittaker will probably be fixing this.
For my iPhone 7, it has become recently more difficult to reply to messages, because either I don't know how to do a proper long tap or the timing/geometry is just a little off.

@zulipbot claim


got api table in markdown working again (README)

centered headers as well

I went ahead and merged PR because it was only README changes and it's public facing so I want to clean up that API table


chore(deps): Update Python dependencies

I see some major version upgrades, in particular, wagtail-modeladmin.

For wagtail-modeladmin, the changelog says that the breaking changes are dropping old versions of Django and Wagtail, but the versions in use here are still supported.
However, wagtail-modeladmin is deprecated, and the recommended upgrade path is to use snippets instead. There is only one use in the codebase, which should be an easy conversion to a snippet:
https://github.com/ietf-tools/www/blob/b222992b2541417d91947fb87a7f13a3895d73c5/ietf/utils/wagtail_hooks.py#L31-L42

Thanks @mgax and @microamp. I'll address that.

I've created a #484 to address the wagtail-modeladmin issue.


Problem with Bundle Install

ruby 1.9.2p136 on x86_64 linux CentOS with rails 3.1.1
muster > bundle install
WARNING:  ArgumentError: Illformed requirement ["Syck::DefaultKey:0x0000000d577080 3.0.10"]
-- encoding: utf-8 --
Gem::Specification.new do |s|
  s.name = %q{json_builder}
  s.version = "3.0.3"

  s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
  s.authors = ["Garrett Bjerkhoel"]
  s.date = %q{2011-12-02 00:00:00.000000000Z}
  s.description = %q{Rails provides an excellent XML Builder by default to build RSS and ATOM feeds, but nothing to help you build complex and custom JSON data structures. The standard to_json works well, but can get very verbose when you need full control of what is generated. JSON Builder hopes to solve that problem.}
  s.email =<EMAIL_ADDRESS>  s.files = [".gitignore", "Gemfile", "Gemfile.lock", "MIT-LICENSE", "README.md", "Rakefile", "json_builder.gemspec", "lib/blankslate.rb", "lib/json_builder.rb", "lib/json_builder/compiler.rb", "lib/json_builder/elements.rb", "lib/json_builder/member.rb", "lib/json_builder/template.rb", "lib/json_builder/value.rb", "lib/json_builder/version.rb", "spec/benchmarks/builder.rb", "spec/dummy/Rakefile", "spec/dummy/app/controllers/application_controller.rb", "spec/dummy/app/controllers/users_controller.rb", "spec/dummy/app/helpers/application_helper.rb", "spec/dummy/app/helpers/users_helper.rb", "spec/dummy/app/models/user.rb", "spec/dummy/app/views/layouts/application.html.erb", "spec/dummy/app/views/users/index.json_builder", "spec/dummy/config.ru", "spec/dummy/config/application.rb", "spec/dummy/config/boot.rb", "spec/dummy/config/database.yml", "spec/dummy/config/environment.rb", "spec/dummy/config/environments/development.rb", "spec/dummy/config/environments/production.rb", "spec/dummy/config/environments/test.rb", "spec/dummy/config/initializers/backtrace_silencers.rb", "spec/dummy/config/initializers/inflections.rb", "spec/dummy/config/initializers/mime_types.rb", "spec/dummy/config/initializers/secret_token.rb", "spec/dummy/config/initializers/session_store.rb", "spec/dummy/config/locales/en.yml", "spec/dummy/config/routes.rb", "spec/dummy/db/development.sqlite3", "spec/dummy/db/migrate/20111127061428_create_users.rb", "spec/dummy/db/schema.rb", "spec/dummy/db/test.sqlite3", "spec/dummy/log/production.log", "spec/dummy/log/server.log", "spec/dummy/public/404.html", "spec/dummy/public/422.html", "spec/dummy/public/500.html", "spec/dummy/public/favicon.ico", "spec/dummy/public/javascripts/application.js", "spec/dummy/public/javascripts/controls.js", "spec/dummy/public/javascripts/dragdrop.js", "spec/dummy/public/javascripts/effects.js", "spec/dummy/public/javascripts/prototype.js", "spec/dummy/public/javascripts/rails.js", "spec/dummy/public/stylesheets/.gitkeep", "spec/dummy/script/rails", "spec/dummy/spec/controllers/users_controller_spec.rb", "spec/integration/navigation_spec.rb", "spec/json_builder_spec.rb", "spec/spec_helper.rb", "spec/support/json_builder.rb", "spec/unit/json_builder_spec.rb"]
  s.require_paths = ["lib"]
  s.rubygems_version = %q{1.3.7}
  s.summary = %q{Rails provides an excellent XML Builder by default to build RSS and ATOM feeds, but nothing to help you build complex and custom JSON data structures. The standard to_json works well, but can get very verbose when you need full control of what is generated. JSON Builder hopes to solve that problem.}
  s.test_files = ["spec/benchmarks/builder.rb", "spec/dummy/Rakefile", "spec/dummy/app/controllers/application_controller.rb", "spec/dummy/app/controllers/users_controller.rb", "spec/dummy/app/helpers/application_helper.rb", "spec/dummy/app/helpers/users_helper.rb", "spec/dummy/app/models/user.rb", "spec/dummy/app/views/layouts/application.html.erb", "spec/dummy/app/views/users/index.json_builder", "spec/dummy/config.ru", "spec/dummy/config/application.rb", "spec/dummy/config/boot.rb", "spec/dummy/config/database.yml", "spec/dummy/config/environment.rb", "spec/dummy/config/environments/development.rb", "spec/dummy/config/environments/production.rb", "spec/dummy/config/environments/test.rb", "spec/dummy/config/initializers/backtrace_silencers.rb", "spec/dummy/config/initializers/inflections.rb", "spec/dummy/config/initializers/mime_types.rb", "spec/dummy/config/initializers/secret_token.rb", "spec/dummy/config/initializers/session_store.rb", "spec/dummy/config/locales/en.yml", "spec/dummy/config/routes.rb", "spec/dummy/db/development.sqlite3", "spec/dummy/db/migrate/20111127061428_create_users.rb", "spec/dummy/db/schema.rb", "spec/dummy/db/test.sqlite3", "spec/dummy/log/production.log", "spec/dummy/log/server.log", "spec/dummy/public/404.html", "spec/dummy/public/422.html", "spec/dummy/public/500.html", "spec/dummy/public/favicon.ico", "spec/dummy/public/javascripts/application.js", "spec/dummy/public/javascripts/controls.js", "spec/dummy/public/javascripts/dragdrop.js", "spec/dummy/public/javascripts/effects.js", "spec/dummy/public/javascripts/prototype.js", "spec/dummy/public/javascripts/rails.js", "spec/dummy/public/stylesheets/.gitkeep", "spec/dummy/script/rails", "spec/dummy/spec/controllers/users_controller_spec.rb", "spec/integration/navigation_spec.rb", "spec/json_builder_spec.rb", "spec/spec_helper.rb", "spec/support/json_builder.rb", "spec/unit/json_builder_spec.rb"]

  if s.respond_to? :specification_version then
    current_version = Gem::Specification::CURRENT_SPECIFICATION_VERSION
    s.specification_version = 3

    if Gem::Version.new(Gem::VERSION) >= Gem::Version.new('1.2.0') then
      s.add_runtime_dependency(%q<activesupport>, [">= 2.0.0"])
      s.add_development_dependency(%q<rspec-rails>, [">= 2.0.0.beta"])
      s.add_development_dependency(%q<rails>, ["#<Syck::DefaultKey:0x0000000d577080> 3.0.10"])
      s.add_development_dependency(%q<sqlite3>, [">= 0"])
      s.add_development_dependency(%q<capybara>, [">= 0.4.0"])
    else
      s.add_dependency(%q<activesupport>, [">= 2.0.0"])
      s.add_dependency(%q<rspec-rails>, [">= 2.0.0.beta"])
      s.add_dependency(%q<rails>, ["#<Syck::DefaultKey:0x0000000d577080> 3.0.10"])
      s.add_dependency(%q<sqlite3>, [">= 0"])
      s.add_dependency(%q<capybara>, [">= 0.4.0"])
    end
  else
    s.add_dependency(%q<activesupport>, [">= 2.0.0"])
    s.add_dependency(%q<rspec-rails>, [">= 2.0.0.beta"])
    s.add_dependency(%q<rails>, ["#<Syck::DefaultKey:0x0000000d577080> 3.0.10"])
    s.add_dependency(%q<sqlite3>, [">= 0"])
    s.add_dependency(%q<capybara>, [">= 0.4.0"])
  end
end
WARNING:  Invalid .gemspec format in<EMAIL_ADDRESS>

s.add_dependency(%q, ["#Syck::DefaultKey:0x0000001e896cf0 3.0.10"])
These lines are incorrect.  Replace the #Syck:DefaultKey:XXXX with an = sign and it works fine.

I downloaded and extracted the gem from Rubygems and my gemspec has the ["= 3.0.10"] rather than Syck::DefaultKey.
I also tried deleting the gem and re-bundling with no issue.
dewski ~/Work/sites/clientend
$ gem uninstall json_builder

Select gem to uninstall:
 1. json_builder-3.0.1
 2. json_builder-3.0.3
 3. All versions
> 3
Successfully uninstalled json_builder-3.0.1
Successfully uninstalled json_builder-3.0.3
dewski ~/Work/sites/clientend
$ ruby -v
ruby 1.9.2p290 (2011-07-09 revision 32553) [x86_64-darwin11.1.0]
dewski ~/Work/sites/clientend
$ bundle install
Fetching gem metadata from http://rubygems.org/......
Fetching gem metadata from http://rubygems.org/..
Using rake (<IP_ADDRESS>)
Using multi_json (1.0.4)
Using activesupport (3.1.3)
Using builder (3.0.0)
Using i18n (0.6.0)
Using activemodel (3.1.3)
Using erubis (2.7.0)
Using rack (1.3.5)
Using rack-cache (1.1)
Using rack-mount (0.8.3)
Using rack-test (0.6.1)
Using hike (1.2.1)
Using tilt (1.3.3)
Using sprockets (2.0.3)
Using actionpack (3.1.3)
Using mime-types (1.17.2)
Using polyglot (0.3.3)
Using treetop (1.4.10)
Using mail (2.3.0)
Using actionmailer (3.1.3)
Using arel (2.2.1)
Using tzinfo (0.3.31)
Using activerecord (3.1.3)
Using activeresource (3.1.3)
Using ansi (1.4.1)
Using bcrypt-ruby (3.0.1)
Using bson (1.5.1)
Using bson_ext (1.5.1)
Using bundler (1.1.rc)
Using highline (1.6.8)
Using net-ssh (2.2.1)
Using net-scp (1.0.4)
Using net-sftp (2.0.5)
Using net-ssh-gateway (1.1.0)
Using capistrano (2.9.0)
Using carrierwave (0.5.8)
Using coffee-script-source (1.1.3)
Using execjs (1.2.9)
Using coffee-script (2.2.0)
Using rack-ssl (1.3.2)
Using json (1.6.3)
Using rdoc (3.11)
Using thor (0.14.6)
Using railties (3.1.3)
Using coffee-rails (3.1.1)
Using orm_adapter (0.0.5)
Using warden (1.1.0)
Using devise (1.5.2)
Using jquery-rails (1.0.19)
Installing json_builder (3.0.3)
Using rails (3.1.3)
Using kaminari (0.12.4)
Using mongo (1.5.1)
Using mongoid (2.3.4)
Using stringex (1.3.0)
Using mongoid_slug (0.8.3)
Using sass (3.1.11)
Using sass-rails (3.1.5)
Using turn (0.8.2)
Using uglifier (1.1.0)
Your bundle is complete! Use `bundle show [gemname]` to see where a bundled gem is installed.

I opened up the downloaded gemspec within RVM and this is the contents:
$:.push File.expand_path("../lib", __FILE__)
require "json_builder/version"

Gem::Specification.new do |s|
  s.name = "json_builder"
  s.version = JSONBuilder::VERSION
  s.summary = "Rails provides an excellent XML Builder by default to build RSS and ATOM feeds, but nothing to help you build complex and custom JSON data structures. The standard to_json works well, but can get very verbose when you need full control of what is generated. JSON Builder hopes to solve that problem."
  s.description = "Rails provides an excellent XML Builder by default to build RSS and ATOM feeds, but nothing to help you build complex and custom JSON data structures. The standard to_json works well, but can get very verbose when you need full control of what is generated. JSON Builder hopes to solve that problem."
  s.authors     = ["Garrett Bjerkhoel"]
  s.email       =<EMAIL_ADDRESS>  s.platform    = Gem::Platform::RUBY

  s.files         = `git ls-files`.split("\n")
  s.test_files    = `git ls-files -- {test,spec,features}/*`.split("\n")
  s.executables   = `git ls-files -- bin/*`.split("\n").map{ |f| File.basename(f) }
  s.require_paths = ["lib"]

  s.add_dependency "activesupport", ">= 2.0.0"
  s.add_development_dependency "rspec-rails", ">= 2.0.0.beta"
  s.add_development_dependency "rails", "3.0.10"
  s.add_development_dependency "sqlite3"
  s.add_development_dependency "capybara", ">= 0.4.0"
end

muster > gem uninstall json_builder
Successfully uninstalled json_builder-3.0.3
muster > gem install json_builder
Successfully installed json_builder-3.0.3
1 gem installed
Installing ri documentation for json_builder-3.0.3...
Installing RDoc documentation for json_builder-3.0.3...

ruby -v
ruby 1.9.2p136 (2010-12-25 revision 30365) [x86_64-linux]
bundle install
WARNING:  #<ArgumentError: Illformed requirement ["#Syck::DefaultKey:0x00000011197fd8 3.0.10"]>
Using rake (<IP_ADDRESS>)
Using multi_json (1.0.3)
Using activesupport (3.1.1)
Using builder (3.0.0)
Using i18n (0.6.0)
Using activemodel (3.1.1)
Using erubis (2.7.0)
Using rack (1.3.4)
Using rack-cache (1.1)
Using rack-mount (0.8.3)
Using rack-test (0.6.1)
Using hike (1.2.1)
Using tilt (1.3.3)
Using sprockets (2.0.2)
Using actionpack (3.1.1)
Using mime-types (1.16)
Using polyglot (0.3.2)
Using treetop (1.4.10)
Using mail (2.3.0)
Using actionmailer (3.1.1)
Using arel (2.2.1)
Using tzinfo (0.3.30)
Using activerecord (3.1.1)
Using activeresource (3.1.1)
Using bundler (1.0.18)
Using rack-ssl (1.3.2)
Using json (1.6.1)
Using rdoc (3.10)
Using thor (0.14.6)
Using railties (3.1.1)
Using rails (3.1.1)
Using acts_as_audited (2.0.0)
Using posix-spawn (0.3.6)
Using albino (1.3.3)
Using ancestry (1.2.4)
Using archive-tar-minitar (0.5.2)
Using auto_strip_attributes (1.1)
Using awesome_nested_set (2.0.2)
Using bcrypt-ruby (3.0.1)
Using cancan (1.6.7)
Using highline (1.6.2)
Using net-ssh (2.2.1)
Using net-scp (1.0.4)
Using net-sftp (2.0.5)
Using net-ssh-gateway (1.1.0)
Using capistrano (2.9.0)
Using cocaine (0.2.0)
Using columnize (0.3.4)
Using orm_adapter (0.0.5)
Using warden (1.0.6)
Using devise (1.4.8)
Using net-ldap (0.2.2)
Using devise_ldap_authenticatable (0.4.9)
Using diff-lcs (1.1.3)
Using event-calendar (2.3.3)
Using formtastic (2.0.2)
Using github-markup (0.5.3)
Using grit (2.4.1)
Using mustache (0.99.4)
Using nokogiri (1.5.0)
Using redcarpet (1.17.2)
Using sanitize (2.0.3)
Using rack-protection (1.1.4)
Using sinatra (1.3.1)
Using gollum (1.3.1)
Using jquery-rails (1.0.14)
Using json_builder (3.0.3)
Using ruby_core_source (0.1.5)
Using linecache19 (0.5.12)
Using madeline (0.1.8)
Using mysql (2.8.1)
Using paperclip (2.4.3)
Using rspec-core (2.6.4)
Using rspec-expectations (2.6.0)
Using rspec-mocks (2.6.0)
Using rspec (2.6.0)
Using rspec-rails (2.6.1)
Using ruby-debug-base19 (0.11.25)
Using ruby-debug19 (0.11.6)
Using rubyzip (0.9.4)
Using settingslogic (2.0.6)
Using simple_navigation (1.4.5)
Using will_paginate (3.0.2)
WARNING:  Invalid .gemspec format in<EMAIL_ADDRESS>
I opened the gemspec and it showed the same Syck keys for the rails add_dependencies.  I changed those to '=' and it works fine.  My rvm is 1.2.6.  Let me know if I can be of any further assistance.

same issue here.  i'm unable to run the gem without tweaking the gemspec.
env:
RVM
REE 1.8.7

Same problem?
Installing json_builder (3.0.3) Invalid gemspec in [/Users/.../.rvm/gems/ruby-1.8.7-p299/specifications/json_builder-3.0.3.gemspec]: Illformed requirement ["#YAML::Syck::DefaultKey:0x457a6f8 3.0.10"]

Seem to have fixed it for me. I was able to bundle install without issue. Thanks!

It seems this fixed it. If it still continues for you, please re-open this or report it again.


[Feature] Add a test system for noqx

A test system will check cases automatically.

One trade-off is that the puzz.link url cannot be parsed without the browser. So the puzz.link loading scheme may be reverted :(


webrtc ascii video

for video, we can totally do it, but I wouldn't want it to be on by default.
also I was thinking it would be cool to save bandwidth by converting video to ascii (e.g. http://blog.nobel-joergensen.com/2011/11/12/creating-real-time-video-ascii-art-using-kickjs-and-webgl/) and sending the ascii along, rendering it in the corner of the screen. that way you can see if the other person is around, but it's low res enough that it isn't uncomfortable

I've been thinking about this too https://github.com/ideashare/ideas/issues/13
https://asciinema.org is able to capture terminal sessions, and play them back - not sure about live streaming though / asked them about atom.io support here asciinema/asciinema#125

what about https://github.com/merodriguezblanco/ascii-video or VLC's ASCII output

@aerth nice, thats pretty straightforward https://github.com/merodriguezblanco/ascii-video/blob/master/public/script.js#L40-L58


Cache user display info on client

not all usages of Avatar are refactored to use the id, but most are
resolves MAN-1407

resolved by 1c52cba1d6d2342d025820c75f2744c5f1e17f1f


Any plans to support PIXI.js v4?

I tried to use pixi-tiled in PIXI.js v4, and seems it works good :+1:
but, Is there any official plans to support in PIXI.js v4 ?

I'm confused, if it works with v4 already then what needs doing to support
it?
On Sat, 7 Jan 2017, 08:35 gaccob<EMAIL_ADDRESS>wrote:

I tried to use pixi-tiled in PIXI.js v4, and seems it works good 👍
but, Is there any official plans to support in PIXI.js v4 ?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/beeglebug/pixi-tiled/issues/14, or mute the thread
https://github.com/notifications/unsubscribe-auth/AA6QMH67dbrlrn90XPchNDACVA_pI-Pxks5rP05cgaJpZM4LdXoQ
.

Looks like the OP was just asking if v4 is officially supported. README.md suggests it pixi-tiled only supports up to v3

When I get a chance I'll do a quick test and bump the version in the readme
if it works OK. Thanks for the heads up!
On Sun, 15 Jan 2017, 19:14 Allain Lalonde<EMAIL_ADDRESS>wrote:

Looks like the OP was just asking if v4 is officially supported. README.md
suggests it pixi-tiled only supports up to v3
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/beeglebug/pixi-tiled/issues/14#issuecomment-272716627,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA6QMEehZ448BtlOMVRvdUPZong1hMmgks5rSnAhgaJpZM4LdXoQ
.


Add ui tests for CLI

part of https://github.com/pyroscope-io/pyroscope-rs/issues/92
Use trycmd to test the UI of CLI.

We will need you ti sign CLA before we merge it. We will add CLA assistant to this repo soon.


Zoom buttons normal size

Eliminate btn-sm from zooming pallette... bigger buttons for touch... since it is collapsable size is less of an issue.

Zoom box has been redesigned.


Where did session.virtualenv_dirname go?

I.e. what is the equivalent feature in the new implementation?

Duplicate of https://github.com/theacodes/nox/issues/96, it's a missing feature but we at least have a rough idea of how to bring it back.


Cannot convert from ChessChallange.API.Board to ChessChallenge.Chess.Board

I got this error message
1>C:\Users\jesse\Downloads\Chess-Challenge-main\Chess-Challenge-main\Chess-Challenge\src\My Bot\MyBot.cs(59,37,59,42): error CS1503: Argument 1: cannot convert from 'ChessChallenge.API.Board' to 'ChessChallenge.Chess.Board'
While I'm only using
using ChessChallenge.API; using System.Collections.Generic; using System.Linq;
How do I make it not convert? I'm very new to C#

I think it got triggered because this line:
Board boardCopy = new Board(board);
Where the latest board is the board that is included with the think function


slightly confused about where the speed up came from

Hi!
After reading your paper, I am not very clear how the speed up came from.
From my understanding, traditional cost volume approach build cost volume of size BXCXDXHXW,
but in your approach you compute the dot product between two features, which results in BXDXHXW .
Is this where the speed up came from?
Thank you for your reply!

Cost aggregation on a 4D cost volume is significant faster than a 5D counterpart, you can check the Complexity Analysis part in Section 4.4 of our paper.


Request for AES128_CBC Encryption Help

I am currently working on implementing AES128_CBC encryption on an STM32 embedded system. I have encountered some difficulties with incorporating salt and iteration counts into the encryption process.
I would greatly appreciate it if you could provide some guidance or suggestions on how to correctly implement AES128_CBC encryption with salt and iteration counts with or without OpenSSL library on an STM32 embedded system.
Thank you very much for your time and assistance. I look forward to hearing from you soon.

This is not a bug in OpenSSL, and this issue should be converted to a discussion (I don't have rights to do that).
In that discussion, it will probably be mentioned that AES-CBC is a mode of symmetric encryption; these generally do not have salt or iteration counts. Salt and iteration count together are most commonly found in PBKDF2, which uses hash functions, not symmetric encryption.
Perhaps in a discussion you could elaborate on your requirements beyond "incorporate salt and iteration count with AES128-CBC" - for example, why you think this would be necessary


Python container to give LSP in Neovim

Explore the following ideas:

First try to get it to work using VScode DEV Containers.
To use Neovim exrc and per-project lua files to specify the language server container to hook into.
Ideal world would be to have something like VSCode Containers.
See if perhaps installing the development tools (Neovim, config files, etc) in the container might be better or not. Since container CPU and memory might be thinner the UX can drop perhaps?

Explore the following ideas:

To use Neovim exrc and per-project lua files to specify the language server container to hook into.
Ideal world would be to have something like VSCode Containers.
See if perhaps installing the development tools (Neovim, config files, etc) in the container might be better or not. Since container CPU and memory might be thinner the UX can drop perhaps?

For now this works perfectly: https://github.com/eulersson/anesowa/blob/main/docs/5-configuring-your-ide.md


Configurable username and password for DBO

I'm currently working on a Docker image containing Nginx and HHVM in oder to quickly set up this server. It uses some shell scripts to replace the API key in the configuration file and I thought about doing the same with the PDO connection string.
However, from what I can tell, the PDO constructor requires the username and password to the database to be separate parameters, and these are not provided to the constructor in db.php, so there is currently no (sane) way for me make them configurable by the end user through environmental variables.
Can you add support for those in the inc/config.php?

Good catch - I've only used SQLite with this project so I never needed to specify a username or password. Are you planning on using a different database system?

I thought MariaDB/MySQL would be a good idea. Makes things a bit easier when it comes to manually deleting packages and such things, I guess. :)

I added support for this in 25a6b82fc026fb880ecb684d94db9bd9114d1b98. However, note that you may need to do schema changes to get it working in MySQL. Notably:

MySQL does not have CREATE INDEX IF NOT EXISTS to create an index only if it doesn't already exist
PHP doesn't support multiple queries in one database call if not using MySQLnd, so the exec in DB::createTables would need to be split into multiple execs
I don't think MySQL allows TEXT columns to be indexed, so the PackageId column in packages may need to be changed to VARCHAR. SQLite has no difference between VARCHAR and TEXT so this is not an issue in SQLite.

Currently the code to create the database tables if they don't exist runs on every request, as that was the easiest way to do it (and allows easy iteration when debugging, you can just delete the SQLite file to get a fresh database on your next request). It might be worth only doing it when an installer script is ran. The installer script could run different DDL statements depending on the database system being used.


TodoContainer.tsx TypeScript Error

I have this issue when I run 'Gulp Serve':
src\webparts\todo\components\TodoContainer\TodoContainer.tsx(45,29): error TS2349: Cannot invoke an expression whose type lacks a call signature

Hi @WagnerGit , this error is due to an incorrect version of the "immutability-helper"  npm package.
A quick fix is to update the package.json file and set the version like this (without the caret):
"immutability-helper": "2.2.0"
Then run a npm i and you're good to go! I will make a PR to fix this issue.


Support OCI based Helm

Is your feature request related to a problem? Please describe.
OCI based registries are not supported when pulling Helm chart dependencies
Additional context
Connected to https://github.com/cyclops-ui/cyclops/issues/75

OCI charts are now supported for dependency charts.
Root OCI charts are yet to be supported

Available from v0.0.1-alpha.9


Add new dimensions to sundeck qtag.

account - the Sundeck account identifier
user - the Sundeck user identifier
service_name - the Sundeck service which executed the query
service_operation - the kind of call the Sundeck service was executing

@rymurr

It appears I don't have karma to merge, @rymurr. Could you click the green button or grant me some karma, please?


Update 220-database-polyfills.mdx

fix minor typo
Describe this PR

Changes

What issue does this fix?

Any other relevant information

Thanks very much for spotting this typo, @jadghadry! Sorry it's a little delayed, but it's now been merged.


[1.x] Support Data Streams in OpenSearch (#690)

Backport of https://github.com/opensearch-project/OpenSearch/pull/690
This commit adds support for data streams by adding a DataStreamFieldMapper, and making timestamp
field name configurable. Backwards compatibility is supported.
Signed-off-by: Ketan Verma<EMAIL_ADDRESS>

start gradle check


Error(s) in loading state_dict for CRNN:

I downloaded and tested the pretrained densenet model, but it showed the error messages.
Error(s) in loading state_dict for CRNN:
Unexpected key(s) in state_dict: "features.1.denselayer1.norm1.num_batches_tracked", ...

I guess you load the pretrained densenet121 model with the scripts python ./main.py --arch densenet121 --resume densenet121_pretrained.pth --test-only? The pretrained model just contain the  model.state_dict(), which is not the resume checkpoint of the script main.py(It also contain optimizer.state_dict()). One way to use the pretrained model refer to the demo.ipynb notebook, or modify main.py to load the model with parameter model.load_state_dict(torch.load(PATH), strict=False).


Import the global notifier and adapter from realm-js-private

The first step for https://github.com/realm/realm-js-private/issues/462 and https://github.com/realm/realm-dotnet-server/issues/4.
This is just the six cpp/hpp files sliced out of realm-js-private and imported here with zero changes as reviewing those changes would be awkward. The next step after this will be to add build system support so that these files are at least compiled and to move the JS/node-specific portions of the GN and adapter to realm-js.

We need to move https://github.com/realm/realm-js/blob/master/vendor/json.hpp too.


bump gradle wrapper version to 2.12

Coverage remained the same at 99.559% when pulling 706e79f7b4cb7af107dceec4245b8fe6f2ec72f6 on jmcampanini:jc/gradle-wrapper-version into 17fa57aad642243d4838c35fe178ba88d377cd46 on srs:master.

Thanks for the pull request.

:smile: thank YOU for the plugin!


First Commit

This is the first PR which contains source code for the CISCO contribution of Apache Opentelemetry module.

LGTM for initial commit. Would be good to have build and tests onboarded to github actions at some time later.

@lalitb Could you please merge the PR on my behalf. Thanks


Avoid saving files on github repo by default

Currently data/ folder has two files mnist.csv(100 lines, ~2MB) and MNIST-1k.csv(1k lines, ~20mb)  saved in Github by default
This can/should be avoided for faster installation(saving time and space).
Download these datasets only if env_setup.sh is executed for tests and development.
Since data is already saved on Github  commit history, rewriting git commit history could be one way to purge this from git storage but extremely risky.

using git-lfs or converting csv to binary to use less space could be other alternatives.

Resolved


Fix saving option only using days argument

Signed-off-by: Thibaud CANALE<EMAIL_ADDRESS>Description
The saving feature was unable to save content without the none default --days argument.
Now it should be able to store content with --start/--end and for the last day without --days argument, as described in the help section of arguments.
Checklist

[Y] I have reviewed the contributing guidelines.
[N] I have included unit tests for my changes or additions.
[N] I have successfully run make test-docker with my changes.
[N] I have updated the documentation.
[Y] I have updated the changelog.

Questions or Comments
I was unable to test my commit because I met some issue to run the Docker test (I am behind a corporate proxy, I fails to fetch content from the WAN) or to execute directly the Python script (some circular imports issues).
About the documentation, this is a fix so there is nothing to update.
If someone could test for me, my tries were unsuccessful:
python3.9 elastalert/test_rule.py --config content/config/my_config.yaml --save-json content/data/my_data.json content/rules/my_rule.yaml

As you might have noticed, I store the timestamp in the class instead of querying for each call, this allows to keep a steady differential between days.
Best regards,

Yes, the changes look good. I only had the one code review concern about the timeframe param being required. Let me know your thoughts on that.

Yes, the changes look good. I only had the one code review concern about the timeframe param being required. Let me know your thoughts on that.

I posted a comment explaining some details, pointing out timeframe isn’t required, now waiting for your review.
Thanks again for your review.

Thanks for the explanation; I see what you're doing now.


Broken Log message

When detaching camera sensors I see on the console a lot of warning/error messages like the one posted below, suggesting that logger might not be called appropriately?
--- Logging error ---
Traceback (most recent call last):
  File "C:\Users\gambitemp\AppData\Local\Programs\Python\Python37\lib\logging\__init__.py", line 1025, in emit
    msg = self.format(record)
  File "C:\Users\gambitemp\AppData\Local\Programs\Python\Python37\lib\logging\__init__.py", line 869, in format
    return fmt.format(record)
  File "C:\Users\gambitemp\AppData\Local\Programs\Python\Python37\lib\logging\__init__.py", line 608, in format
    record.message = record.getMessage()
  File "C:\Users\gambitemp\AppData\Local\Programs\Python\Python37\lib\logging\__init__.py", line 369, in getMessage
    msg = msg % self.args
TypeError: not all arguments converted during string formatting
Call stack:
....
  File "Y:\sbst-tool-competition\.venv\lib\site-packages\beamngpy\sensors.py", line 576, in detach
    f'{self.annotation_handle}')

Fixed in BeamNGpy v1.22 (https://pypi.org/project/beamngpy/1.22/).


Metadata extraction pipeline

Port it from https://github.com/ARK-Builders/arklib-android

If we make this pipeline in Rust, we'll not only be able to ensure consistency between platforms, but also render previews and thumbnails on powerful server and allow light devices to just sync them over the network.
See:

https://github.com/ARK-Builders/ark-cli/issues/5


[HOLD] CLOUDSTACK-9042: remove unused edithosts.sh

As discovered in CLOUDSTACK-9042, this script contains some issues but
since it was replaced since 4.6, we remove it on master.
Also see #1045.
/cc @remibergsma

@resmo: that file is used by HyperV. I have refactored it and removed the files that have to be removed. Have a look at PR #1084
It was difficult to find because the HyperV Resource had the files all hard-coded.
The change I made is like:
final String command = String.format("%s%s %s", "/root/", VRScripts.DHCP, args);

Cheers,
Wilder

Hi @resmo
Could you please close this PR? The PR #1084 was already merged. As I mentioned above, the edithosts.txt is used by hyperv.
Cheers,
Wilder


add short description of import and connect

I'm new to supersqlite after seeing it today on hacker news.
I thought the import and connect descriptions would be helpful for those like me who were not yet familiar, to clarify how to drop it in as a replacement.


Camera monitors don't work.

Description

it won't allow you to view different cameras
Reproduction

open up monitor.
click on camera.
camera instantly disconnects.
infinite static.
Additional context

seems to work now i watched singulose last night

this got fixed at some point