encryption minor refactoring and query decryption

minor refactoring on moving encryption files ( com.azure.cosmos.encryption)
wired up deserialization on query with tests for decryption on query

/azp run java - cosmos - tests


Issues setting default filters fixed, further testing and refactoring

Coverage decreased (-13.5%) to 70.604% when pulling 71e724a6ad26b1a5b9ff1950e10f2013f97b272f on develop into f46dc73c206d2eb983486804077202b6f58eb580 on master.


Cleanup Thailand's localization file and in-line code comments

Proposed change
This standardizes Thailand's localization file by removing extra comments from running make l10n and standardizing in-line comments for codes in Thailand.py file as per @arkid15r 's styling recommendation mentioned in #988 .
Type of change

[ ] New country holidays support (thank you!)
[ ] Supported country holidays update (calendar discrepancy fix, localization)
[x] Existing code/tests/processes improvement (best practice, refactoring, optimization)
[ ] Dependency upgrade (version update)
[ ] Bugfix (non-breaking change which fixes an issue)
[ ] Breaking change (a code change causing existing functionality to break)
[ ] New feature (adds functionality to python-holidays in general)

Checklist

[x] I've followed the contributing guidelines
[x] This PR is filed against beta branch of the repository
[x] This PR doesn't contain any merge conflicts
[x] The code style looks good (make pre-commit)
[x] I've added tests to verify that the new code works and all tests pass locally (make test)
[x] The related documentation has been added/updated (check off the box for free if no updates is required)

Coverage: 100.0%. Remained the same when pulling 73ba4c5ae11f93f1c26f9f5b34f7cdb84eece506 on PPsyrius:th_loc_cleanup into fa8b9441bd99cb27be4e3d9ccd698ac7a67d1679 on dr-prodigy:beta.


Disable Eslasticsearch

Can I disable elasticsearch resouce on Umbrella? We already use an external.

Yes, this is possible. See the Multi-Server Setup part of the documentation about how to configure this. Basically, you'll update /etc/api-umbrella/api-umbrella.yml with something like:
services:
  - general_db
  # log_db should *not* be enabled if you want to rely on an external Elasticsearch cluster.
  # - log_db
  - router
  - web
elasticsearch:
  hosts:
    - http://your-elasticsearch-server1.example.com:9200
    - http://your-elasticsearch-server2.example.com:9200
    - http://your-elasticsearch-server3.example.com:9200

If you're external Elasticsearch instance is version 2 or greater, than you'll also want to look at External Database Usage (you'll need to add an extra config setting for Elasticsearch 2 compatibility).


create BDD .feature file

IBM Cloud toolchain: Delivery Pipeline deployed products to dev, including commit 3b9fb5e84fa1121a29f542c4dd9831b8edb141c9

IBM Cloud toolchain: Delivery Pipeline deployed products to prod, including commit 3b9fb5e84fa1121a29f542c4dd9831b8edb141c9


getgrav.org/blog RSS Feed not functional

I've added the Grav RSS Feed into my thunderbird. It can read all existing entries, and it is a valid xml it seems. However I did not receive new entries for the 1.7 News. I removed the Feed and added it again. Now the post about grav premium also did not pop up. All other feeds work as expected. I think it is also in your interest to provide the users with updates, especially about grav premium ;-)

I ran the feed through the RSS Validator to see what’s wrong.
Seems like the language is missing:

line 8, column 18: language must be an ISO-639 language code: [help]

       <description>Grav Development Blog</description>
       <language></language>
       <lastBuildDate>Mon, 07 Dec 2020 11:49:00 +0000</lastBuildDate>

It is still broken

Fixed the feed plugin (in develop).  And updated getgrav.org to use it.  Passes validation with just 'recommendation' to not use relative URLs although they are fully supported.


Music Player time stamp

Music player time stamp is adding the number of seconds during the song instead of changing it to minutes once it reachs 60 secs.

happens in simulator, not browser


Support for bulk indexing

Would it be possible to add support for bulk indexing documents, as described in the Elasticsearch Bulk API docs? Thanks in advance!

yes, actually that should be pretty easy, I will add support as soon as I find time. :)
for now consider the following workaround:
    payload = create_bulk_payload(ordered_indexes)

    elastic_url = ShopApi.config :elastic_url
    index_name = ShopApi.config :elastic_index_name

    url = "#{elastic_url}/#{index_name}/_bulk"

    Elastix.HTTP.post(url, payload)

where payload would be a concatenated string of json indexing payloads, in bettween and ending with an \n

the final line of data must end with a newline character \n.

Awesome, thanks--that worked great! Feel free to close this if you want, or keep it open until the support is added

Closed thanks to @FabienHenon https://github.com/werbitzky/elastix/pull/23


Key Vault - Keys Sample Issue

Section link1, link2:

Reason:
Values didn't match for KeyRotationPolicyAction.rotate equals 'Rotate', which causes:
AssertionError: The specified action should exist in the key rotation policy
Suggestion1&2:
Update code to 'rotate'
Suggestion3:
Update code to 'notify'
@rohit-ganguly, @lmazuel, @mario-guerra, @heaths, @mccoyp for notification.

This issue was caused by a service regression that's being mitigated now -- ICM with details: https://portal.microsofticm.com/imp/v3/incidents/details/397947956/home
There was also a PR in JS to account for the change, but this has been closed since the service is correcting itself: https://github.com/Azure/azure-sdk-for-js/pull/26217
I'll close this issue since the problem should go away once service changes are fully deployed. Thank you @ChenxiJiang333 for your attention to this!


Support for Gazebo materials

🎉 New feature
Closes https://github.com/gazebosim/gz-sim/issues/10
Summary

Ported the OgreMaterialParser from the Ogre community, also used in gzweb to add minimal parsing capabilities of Gazebo materials. This would make migration from Gazebo Classic to Gazebo Sim a little bit easier.
Limitations:

Only parse solid colors
Custom material scripts are not supported

Task list:

[x] Install gazebo.material at gz-sim/media/
[x] Add warning with documentation page linked
[x] Write integration test for the parser
[x] Check for a solution to handle dependent materials

Test it

Run Gazebo from the source directory
gz sim src/sdformat/test/integration/model/double_pendulum.sdf -v 3
Checklist

[x] Signed all commits for DCO
[x] Added tests
[x] Added example and/or tutorial
[x] Updated documentation (as needed)
[x] Updated migration guide (as needed)
[x] Consider updating Python bindings (if the library has them)
[ ] codecheck passed (See contributing)
[ ] All tests passed (See test coverage)
[ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

Tested different Gazebo/* material names and seems to work:

I also tried Gazebo/Invalid - It rendered the link black which is probably expected but maybe add a warning that the material name is not recognized?


Fixed errors in ELK and SimpliVity backups playbooks.

When running the playbooks on HPE SimpliVity setup we found these issues in the dev playbooks.

hi, I am the product manager for this solution, can you tell me what the issue is?  thx

Hi,
We've run the playbooks on SimpliVity demo / testing hardware that HPE has lent us, and found some errors / bugs.
This pull request fixes two issues we found a solution for.

Hi @asgeirn,
First off, thanks for submitting this pull request. We are hoping to make this a collaborative effort between HPE R&D and the community. We absolutely appreciate the effort.
To introduce myself, my name is Jason Shiplett, the product owner for the Express Containers solution on HPE SimpliVity. I'm more or less the engineering counterpart to Ka Wai who replied earlier.
Given the nascent status of this project, we're not very well-equipped to handle PRs at this point. Testing the changes included will be manual, and we just don't have the bandwidth to get it done and complete the next revision of the solution on time.
Moving forward, I would like to confine these types of interactions and feedback to issues, even if you have a fix in mind. Please recreate this with a full problem description and the suggested changes as an issue. Once that's done, I'll create a defect story and prioritize it within the team's backlog. We won't be able to test it immediately, but we will get to it some time in the near future.
In the meantime, feel free to reach out if you need anything else. You can get me at jason dot shiplett at hpe dot com.
Thanks!
Jason


Django variable completion regression

We've had to remove some code due to changes in web tools, and this is breaking this scenario:
{{ messag

Press CTRL+SPACE at the end, and it won't auto complete. That's because the trigger point is now after a space is added, and we don't find any completions for that location.
The removal of TemplateCompletionCommandHandler is what is causing this.

This now works with the latest web tools.


Read and write data API

Make functions like factory_data_load_and_split and factory_write_results completely independent from database connections or local folders where data are read from/pushed to.
It may make sense to keep the option of writing to disk for factory_write_results though.

When this is done, remember to update the Sphinx documentation, i.e. delete notes about API.

Having discussed with @ChrisBeeley today, it turns out this issue is not necessary after all. @ChrisBeeley says it is perfectly fine to leave our MySQL read/write commands in the functions- external users who decide to run the pipeline themselves will be experienced with Python and SQL, so all they would need to do would be to replace/modify the lines of code that currently pull from/write data to our database. The rest of the users would be simply passing a CSV file instead, and getting the results tables in CSV format.
All that left is to update a couple of lines in the Sphinx documentation. No need to keep this issue open.


swap search modal components for doorway-ui

Pull Request Template
Issue Overview
This PR addresses refactor/restyle search modal. This PR does no restyling; it only clones the logic, using the existing components in doorway-ui.

[ ] This change addresses the issue in full
[x] This change addresses only certain aspects of the issue
[ ] This change is a dependency for another issue
[ ] This change has a dependency from another issue

Description
Please include a summary of the change and which issue(s) is addressed. Please also include relevant motivation and context. List any dependencies that are required for this change.
How Can This Be Tested/Reviewed?
Provide instructions so we can review.
Describe the tests that you ran to verify your changes. Please also list any relevant details for your test configuration.
Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have added QA notes to the issue with applicable URLs
[ ] I have performed a self-review of my own code
[ ] I have reviewed the changes in a desktop view
[ ] I have reviewed the changes in a mobile view
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules
[ ] I have assigned reviewers
[ ] I have run yarn generate:client and/or created a migration if I made backend changes that require them
[ ] My commit message(s) is/are polished, and any breaking changes are indicated in the message and are well-described
[ ] Commits made across packages purposefully have the same commit message/version change, else are separated into different commits

Reviewer Notes:
Steps to review a PR:

Read and understand the issue, and ensure the author has added QA notes
Review the code itself from a style point of view
Pull the changes down locally and test that the acceptance criteria is met
Also review the acceptance criteria on the Netlify deploy preview (noting that these do not yet include any backend changes made in the PR)
Either explicitly ask a clarifying question, request changes, or approve the PR if there are small remaining changes but the PR is otherwise good to go

On Merge:
If you have one commit and message, squash. If you need each message to be applied, rebase and merge.

Isaac this is getting there!!! Great progress!!!  I built & tested it and functionality seems to be working well. I'll be able to dive into the code a bit later, but wanted to point out that we're missing Rent minimum, right now. Should be a max & min.

Isaac this is getting there!!! Great progress!!! I built & tested it and functionality seems to be working well. I'll be able to dive into the code a bit later, but wanted to point out that we're missing Rent minimum, right now. Should be a max & min.

added min rent field

I noticed that "San Francisco" is always set even if only one other county is checked.

This actually results in some results being returned in my local instance.

I don't want to block on any of this since the core functionality is mostly there and we're running low on time, but I think it's worth addressing this and a couple other things I called out in other comments.

fixed SF showing up


Waypoints

Hi,
your library is very convenient and intuitive to use.
My question is, when are you going to add the waypoint option ?
I'd like very much to use your library in my project, but without this i can't.

I had this exact issue. However, make sure to use version 1.0.5:
compile 'com.akexorcist:googledirectionlibrary:1.0.5'


Mike's Feedback before merging

This will house my feedback given that there can't be a PR to the main organization. Will be updated as I review.
Licensing:
a little suspect is https://github.com/ghodss/yaml -- which is MIT w/ Go Authors copyright attached. We are using gopkg.in/yaml.v2 in edgex-go, what does the ghodss/yaml pkg do that the one we are using doesnt would be my question.
The rest of the go.mod dependencies are already in use with EdgeX -- so didnt concern myself too much with them.

Dependency
LICENSE

github.com/BurntSushi/toml v0.3.1
MIT

github.com/edgexfoundry/go-mod-core-contracts v0.1.3
Apache-2.0

github.com/ghodss/yaml v1.0.0
MIT-ish

github.com/pelletier/go-toml v1.2.0
MIT

github.com/spf13/afero v1.1.2
Apache-2.0

github.com/spf13/cobra v0.0.5
Apache-2.0

github.com/spf13/viper v1.3.2
Apache-2.0

github.com/stretchr/testify v1.3.0
MIT

Would like to have replace removed:
https://github.com/edgexfoundry-holding/edgex-cli/blob/40e914d7f567781df4c71cea0a0dd6ca029a088f/go.mod#L14
Code Review:

NOTE: I'm using Windows 10 and Nightly Build of docker-compose for testing.

The path doesn't look correct for getting version? Shouldn't it be /api/version?
https://github.com/edgexfoundry-holding/edgex-cli/blob/40e914d7f567781df4c71cea0a0dd6ca029a088f/pkg/client.go#L184
running .\edgex-cli.exe version yields the following output:
edgex-cli version: 0.0.1
version:  404 page not found

General Feedback:

https://github.com/edgexfoundry-holding/edgex-cli/blob/40e914d7f567781df4c71cea0a0dd6ca029a088f/cmd/status/status.go#L62
Any REST Request path should leverage the routes defined in go-mod-core-contracts. Will help with maintaibility: https://github.com/edgexfoundry/go-mod-core-contracts/blob/master/clients/constants.go

Ideally these ports should be defined in a config (with these defaults set) and leveraged everywhere through the application.
https://github.com/edgexfoundry-holding/edgex-cli/blob/40e914d7f567781df4c71cea0a0dd6ca029a088f/cmd/status/status.go#L48
This is especially needed for docker deployments since it is common to change the port exposed to the host to avoid conflicts with existing software on machines.

Any reference to Export Services/Client Registration should be removed as it has been removed for Geneva. We will probably have to figure out best way to manage (if needed) App Service Configurable
https://github.com/edgexfoundry-holding/edgex-cli/blob/40e914d7f567781df4c71cea0a0dd6ca029a088f/cmd/status/status.go#L55

The path doesn't look correct for getting version? Shouldn't it be /api/version? Also should leverage go-mod-core-contracts constant as mentioned above.
https://github.com/edgexfoundry-holding/edgex-cli/blob/40e914d7f567781df4c71cea0a0dd6ca029a088f/pkg/client.go#L184
running .\edgex-cli.exe version yields the following output:
edgex-cli version: 0.0.1
version:  404 page not found

The comment in client.go says it will print version for all services, but i only see the above output -- not clear as to what version should be reported.

Pushed shouldn't read 50 Years -- should be either true/false or blank when not pushed and the date it was pushed if its filled in.
PS C:\Development\intel\edgex-cli> .\edgex-cli.exe -v  reading list
Reading ID                              Name    Device                          Origin                  Value                   Created Modified        Pushed
2ef0fead-0f63-4630-9ed5-ef58d7d1f9eb    Bool    Random-Boolean-Device<PHONE_NUMBER>041088000     false                   2 weeks 2 weeks         50 years
29d00f63-07b1-4fed-92dd-257dcb1cd3d6    Bool    Random-Boolean-Device<PHONE_NUMBER>011768100     true                    2 weeks 2 weeks         50 years

list subcommand doesn't seem to work without -v or -- no-pager on Windows:
PS C:\Development\intel\edgex-cli> .\edgex-cli.exe device list
write |1: file already closed
PS C:\Development\intel\edgex-cli> .\edgex-cli.exe deviceservice list
PS C:\Development\intel\edgex-cli> .\edgex-cli.exe event list
PS C:\Development\intel\edgex-cli> .\edgex-cli.exe reading list
PS C:\Development\intel\edgex-cli> .\edgex-cli.exe addressable list
write |1: file already closed
PS C:\Development\intel\edgex-cli> .\edgex-cli.exe addressable list -v
HEADER
Content-Type : [application/json]
Date : [Tue, 21 Jan 2020 19:51:14 GMT]
Content-Length : [354]
BODY
[{"created":1576005060115,"modified":1576005060115,"origin":1576005060115,"id":"bf65d2ac-6d29-474e-bf3d-8d1cc57c637c","name":"device-virtual","protocol":"HTTP","method":"POST","address":"edgex-device-virtual","port":49990,"path":"/api/v1/callback","baseURL":"http://edgex-device-virtual:49990","url":"http://edgex-device-virtual:49990/api/v1/callback"}]

PS C:\Development\intel\edgex-cli> .\edgex-cli.exe addressable list --no-pager
ID                                      Name            Protocol
bf65d2ac-6d29-474e-bf3d-8d1cc57c637c    device-virtual  HTTP
PS C:\Development\intel\edgex-cli>

command reading list with -v doesn't print HTTP response per the command documentation. See above comments -- reading list -v prints a table, while addressable list -v prints an HTTP Response (as expected).

Why does notification command not have a subcommand for list?

Unsure why 24H is in invalid frequency -- its the the default in EdgeX.
PS C:\Development\intel\edgex-cli> .\edgex-cli.exe interval list
http://localhost:48085/api/v1/interval
invalid Interval Frequency 24h
PS C:\Development\intel\edgex-cli>

HTTP Response:
[{"Timestamps":{"created":1576005058483,"modified":1576005058483},"id":"e4895868-65e6-4346-8be0-74342e557213","name":"midnight","start":"20180101T000000","frequency":"24h"}]

 .\edgex-cli.exe device add
Add Devices:

Indication that I am missing flags, or in this case a file is not provided/or found should be indicated.

https://github.com/edgexfoundry-holding/edgex-cli/blob/40e914d7f567781df4c71cea0a0dd6ca029a088f/cmd/status/status.go#L62
Any REST Request path should leverage the routes defined in go-mod-core-contracts. Will help with maintainability: https://github.com/edgexfoundry/go-mod-core-contracts/blob/master/clients/constants.go.
Also, can the clients in the go-mod-core-contracts be used instead of having to recreate the REST request?

Still think theres a little more work to be done here:

All the concerns here have been addressed by fixing dedicated issues. See issues list here:
https://github.com/edgexfoundry-holding/edgex-cli/issues?q=is%3Aissue+Mike+is%3Aclosed


[DOC]: sphinx fails to document a type alias for dict

How would you describe the priority of this documentation request
Critical (currently preventing usage)
Please provide a link or source to the relevant docs
https://github.com/nv-morpheus/Morpheus/blob/branch-24.10/python/morpheus/morpheus/utils/type_aliases.py
Describe the problems in the documentation
Attempting to document this alias:
TensorMapType = dict[str, NDArrayType]

Yields this error, which appears to be a variant of https://github.com/sphinx-doc/sphinx/issues/8493
sphinx.errors.SphinxWarning: /home/dagardner/work/morpheus/python/morpheus/morpheus/utils/type_aliases.py:docstring of builtins.dict:20:autosummary: failed to import clear.
Possible hints:
* ValueError: not enough values to unpack (expected 2, got 1)
* ModuleNotFoundError: No module named 'clear'
* KeyError: 'clear'
* AttributeError: module 'morpheus.utils.type_aliases' has no attribute 'clear'
* ModuleNotFoundError: No module named 'morpheus.utils.type_aliases.clear'; 'morpheus.utils.type_aliases' is not a package

Warning, treated as error:
/home/dagardner/work/morpheus/python/morpheus/morpheus/utils/type_aliases.py:docstring of builtins.dict:20:autosummary: failed to import clear.
Possible hints:
* ValueError: not enough values to unpack (expected 2, got 1)
* ModuleNotFoundError: No module named 'clear'
* KeyError: 'clear'
* AttributeError: module 'morpheus.utils.type_aliases' has no attribute 'clear'
* ModuleNotFoundError: No module named 'morpheus.utils.type_aliases.clear'; 'morpheus.utils.type_aliases' is not a package
ninja: build stopped: subcommand failed.

Changing the alias to use typing.Dict avoids the issue:
TensorMapType = typing.Dict[str, NDArrayType]

(Optional) Propose a correction
No response
Code of Conduct

[x] I agree to follow this project's Code of Conduct
[x] I have searched the open documentation issues and have found no duplicates for this bug report

I was able to repro the issue outside of Morpheus without any of our code in as simple of a repro as possible (https://github.com/dagardner-nv/repro_1956).
Through trial/error I was able to isolate the issue to the numpydoc extension, and opened up:
https://github.com/numpy/numpydoc/issues/587


Qlik SSE is not running in April 2020 release

Function
'PyTools.sklearn_Setup'
Describe the bug
'PyTools.sklearn_Setup' is not a registered SSE function. Ensure the plugin is running and try restarting Qlik Engine.
Full error message from Python
Provide full stack trace for the error.
If using Qlik-Py-Start you can provide screenshots from the terminal. Make sure to go all the way up to the start of the errors.
Alternatively, you can provide the latest log files from ../qlik-py-tools/qlik-py-env/core/logs.
Error message in Qlik
[Result-Setup]:
LOAD
model_name,
result,
timestamp
EXTENSION PyTools.sklearn_Setup(MODEL_INIT{Model_Name, EstimatorArgs, ScalerArgs, ExecutionArgs})
Expression or load script used in Qlik
Provide a screenshot of the expression or load script where the SSE is being used in Qlik.
Environment:

OS [e.g. Windows Server, Linux]
windows 10

Qlik Sense Enterprise/ Desktop/ Kubernetes/ QlikView
Qlik sense desktop

SSE Version
SSE 7

Additional context
Add any other context about the problem here.

Hi @ahmedkhancgt , the error indicates that the Qlik Sense Engine has not detected the SSE. You'll need to check through the installation notes from step 4 onwards and see if something was missed.
For Qlik Sense Desktop take a look at the solution for issue #63.


Block domain stellar5x.com

Domain: stellar5x.com
Requested by @sysmanalex

Scam, Fraud, Phishing.

@sysmanalex These have previously been flagged. Are there ones that were missed?


CentOS 7 Fails to Update Without DKMS Installed

System Information:

OSX: 10.11.1
Vagrant: 1.7.4
VBGuest: 0.11.0

Using a relatively stock CentOS 7 image, vbguest fails to update:
$ vagrant vbguest centos-7
Got different reports about installed GuestAdditions version:
Virtualbox on your host claims:   4.3.14
VBoxService inside the vm claims: 5.0.8
Going on, assuming VBoxService is correct...
GuestAdditions seems to be installed (5.0.8) correctly, but not running.

The result being that shared folders are not mounted correctly.  However, if I ssh to the vm and run yum install dkms, and then run again, it does work:
$ vagrant vbguest centos-7
GuestAdditions versions on your host (5.0.8) and guest (4.3.14) do not match.
Loaded plugins: fastestmirror
Loading mirror speeds from cached hostfile
 * base: mirrors.tummy.com
 * epel: reflector.westga.edu
 * extras: repos.dfw.quadranet.com
 * updates: mirror-centos.hostingswift.com
Package kernel-devel-3.10.0-229.20.1.el7.x86_64 already installed and latest version
Package gcc-4.8.3-9.el7.x86_64 already installed and latest version
Package 1:make-3.82-21.el7.x86_64 already installed and latest version
Package 4:perl-5.16.3-285.el7.x86_64 already installed and latest version
Package bzip2-1.0.6-12.el7.x86_64 already installed and latest version
Nothing to do
Copy iso file /Applications/VirtualBox.app/Contents/MacOS/VBoxGuestAdditions.iso into the box /tmp/VBoxGuestAdditions.iso
mount: /dev/loop0 is write-protected, mounting read-only
Installing Virtualbox Guest Additions 5.0.8 - guest version is 4.3.14
Verifying archive integrity... All good.
Uncompressing VirtualBox 5.0.8 Guest Additions for Linux............
VirtualBox Guest Additions installer
Removing installed version 5.0.8 of VirtualBox Guest Additions...
Removing existing VirtualBox DKMS kernel modules[  OK  ]
Removing existing VirtualBox non-DKMS kernel modules[  OK  ]
Copying additional installer modules ...
Installing additional modules ...
Removing existing VirtualBox DKMS kernel modules[  OK  ]
Removing existing VirtualBox non-DKMS kernel modules[  OK  ]
Building the VirtualBox Guest Additions kernel modules
[  OK  ]
Doing non-kernel setup of the Guest Additions[  OK  ]
Starting the VirtualBox Guest Additions [  OK  ]
Installing the Window System drivers
Could not find the X.Org or XFree86 Window System, skipping.
An error occurred during installation of VirtualBox Guest Additions 5.0.8. Some functionality may not work as intended.
In most cases it is OK that the "Window System drivers" installation failed.

Less noisy running again:
$ vagrant vbguest centos-7
GuestAdditions 5.0.8 running --- OK.

Seems like it doesn't like not having DKMS, but also doesn't make that obvious.

To followup on this, I also found that after installing dkms I also had to rmmod vboxguest before attempting to upgrade guest additions again.


Various minor linting tweaks

I suggest looking at each commit individually so that they're simper to read.

Thanks Hugo!


Upgrade Astro to 3.0.0 and update other devDependencies

Take note of some of the breaking changes and whether they apply: https://docs.astro.build/en/guides/upgrade-to/v3/

Hi!
Here a PR for the upgraded version: #76

Solved in #76, thanks @saulopaiva!


[elixir] feat: add athena partition for each load

In this PR, we add another step to the ingestion job. After the glue job completes successfully, we'll add the partition to Athena, so the results can be queryable right away.

is there monitoring to make sure we're keeping up in dev and in prod?

No. But taking a task to implement something. Oban stores all jobs in the database, so we'd have to query periodically and get the number of jobs queued. Or maybe we can use the new "Admin" app to stat it. Added a task here: https://app.asana.com/0/1205795733867671/1206275920295822


Recommended Resources to use HUNTRESS / multiple thread usage leads to halt of HUNTRESS ?

Hi Farid,
I am currently making heavy use of your implementation of the HUNTRESS algorithm.
tl.huntress
Do you have some guidance on the resources to allocate to HUNTRESS?
I am having some trouble running HUNTRESS.
When using a single thread HUNTRESS runs fine for small cell-mutation matrices (200 cells 10 mutations).
I tried upping the threads to 8 and ram to 2 GB in the light of larger mutation matrices  (1000 cells, 50 mutations), yet it appears that even for the small runs (200 cells 10 mutations) multiple threads bring trouble - that I don't understand.
HUNTRESS seems to start up fine with the usual first line of output:
running HUNTRESS with alpha=1e-06, beta=0.1, n_threads=8
but just does not progress after that for more than 3 hrs, which is way more than the runtime the supplementary paper of HUNTRESS suggests.
Surprisingly, I cannot find a high CPU usage related to it. When multiple threads are run no significant resources are used. Tried multiple machines and setups.
Is there a way to check if HUNTRESS is making progress -  A debug mode/log ?
Thanks in advance! Best Regards,
Gordon

Great, will give trisicell a try !  - So it is a known issue with multi-threading / processing ?
Thanks for the prompt and super helpful support !
Gordon

No, there is no issue with HUNTRESS. I tried to clean up the code in the scphylo implementation but turned out to be not good. I have to revert it back to the original code.
Sure, anytime. I'll keep you updated.

Hi Gordon,
The bug in scphylo has been fixed, and version 0.0.4 is now available for you to update. You can use HUNTRESS without any issues. Please let me know if there is still any issues. Thank you!

Great, thank you - updated to v0.0.4 now. Multiprocessing seems to work now.  Apologies, for my wording before - I of course just meant the implementation of HUNTRESS not the HUNTRESS Algorithm.
Do you have any guidance on resources and runtimes? mutations m=15 and cells n=250 runs fine - near instant, but already cells n=300 takes 15+ min (haven't) seen it finish, but also sometimes many more cells work. I am a little puzzled about the runtime varying. Is there a way to show if progress is happening?

Yes, that's what I expect as well. OK, good idea - will try the original code then.

Hi Gordon,
Thank you for the update. Did you also try the code in  https://github.com/PASSIONLab/HUNTRESS ? Do you get the same behaviour?


Observable memoryleak fix

@GianlucaGuarini - I think there is something wrong with this branch. I'm not 100% sure why you have other peoples commits in here but my guess is that you merged in master which generated a merge commit then rebased on top of master again later. Is it possible to cherry pick+squash your commits and push a clean branch with just your fixes?

Ok I will make another pull request just adding the memory test and my fix to the observable. I need to pull request on the dev branch  right?

@tipiirai It doesn't work for the same reason why the object comparison (object1 ==  object2) in javascript is not possible http://stackoverflow.com/questions/201183/how-to-determine-equality-for-two-javascript-objects

@GianlucaGuarini - You don't need to make a new PR. You can just re-use this one which will preserve the conversation, history and linking. If you're not sure here is one way you could do that:
cd <your fork of riotjs>
git checkout -b observable-memory-leak-fix

# Cherry Pick the 3 commits that are yours
git cherry-pick  b3ac8e7 a695555 b143d67

# Push the new branch to origin at the location of the original
git push origin observable-memory-leak-fix:<your PR branch name - github marked it as unknown for some reason> --force

{ foo: 1} == { foo: 1 } will not work because they are different objects, but fn == fn will work if the functions are the same. Try on console:
var foo = function() {}
foo == foo // true

on the observable we are comparing against the same function and I doubt this is the root cause for the memory leak.

To check the fix it's really simple.

$ git clone git@github.com:GianlucaGuarini/riotjs.git gian_riot
$ cd gian_riot && npm install && make perf

Check the rult of the tests in your terminal with my fix and without

You are damn right! Not still sure why the function comparison fails.
Also impressed to see that jsdom worked on the fly. Thank you! Eagerly waiting for the pull request!

I told ya.. I am working right now I will be able to do that sometime during the evening

Yeah - you told me :) Thank you again.


[Array] Add a solution to Pascal's Triangle & Chores

Hi, @soapyigu , here's what this PR do:

[Arrray] Add a solution to Pascal's Triangle
fix add binary build issue
fix reverse vowels build issue


Verify untrusted inputs in client code

As partially addressed in https://github.com/liveblocks/liveblocks/pull/161, we should verify all untrusted inputs coming in from the outside, like all WebSocket messages that we receive, before using them. Not doing so has the risk that our TypeScript annotations (aka assumptions) don't match the values at runtime.
For large object structures like the ServerMessage type, for example, these validation conditionals can become pretty unwieldy if you use simple if-statements for them, but we might be able to use a validation library for this. This is exactly the use case that I wrote https://github.com/nvie/decoders for.
Are we open to using it, or vendoring-in a subset of it? (Since it's fully tree-shakable, it would only add the minimum amount of overhead to the bundle.) Would love to discuss!

I believe that validating WebSocket messages from our server is not necessary as they're coming from a trusted source. It would impact the performance of a hot path and would likely get in our way when making the WS protocol evolve (backward compatibility).
What do you think now that you have more context on this? @nvie

Agreed, we can close this one 👍


Lose redundant tag on unary calls

Fixes #1446
Affects #899

We should probably announce this prior to merging.

It's the same semantics, yes

Ok, if there's no semantics change, then LGTM from me as well.

Green, LGTM, let's merge?


recently processed tracks not showing up on main display

the tracks are labeled processed, they have points which appear to be credited to the leaderboard totals, but they do not show up in purple on the main display.
the display used to be nearly instant; for some time now there has been a lag of a several days, but these tracks were uploaded more than a week ago and do not yet appear. this is a real usability problem as i use the map display in deciding if i want to take any alternate routes to improve coverage.
example: http://openstreetcam.org/details/3088794/1/track-info

we are facing the same problem in Libya
https://openstreetcam.org/user/aal-ldyn-bn-saayd
https://openstreetcam.org/user/mohamedkenan
https://openstreetcam.org/user/osama-elhaddad
https://openstreetcam.org/user/abdalla-altaboni
https://openstreetcam.org/user/salah-alseleni
https://openstreetcam.org/user/ahmed-elgehani
https://openstreetcam.org/user/saad-abu-matary
https://openstreetcam.org/user/noredeen-imto
https://openstreetcam.org/user/elhusadi

Yeah the purple tracks are not processing but if you go to openstreetmap and have it show images they are there.
As for the sign detections that hasn’t worked for over a year now I believe so I’m starting to think it’s not intended to work.

@rwelty1889 , we've fixed the showing up issue for http://openstreetcam.org/details/3088794/1/track-info. Could you please confirm and close the ticket? Thanks.

@JiudanZheng That's for one track though? The data is not processing for any new tracks.

@joeybab3 , checked the tracks posted by you. There were 3 tracks on Oct 27 and one track on Oct 25 under processing. Depending on different images, processing may take a couple of days.

@JiudanZheng Yeah I checked and one track I took on the 23rd has gone in so it seems to be processing again. Most of my tracks from the last month or so are not on the main map but still "processed" in some regard.

@JiudanZheng could you please check my 33 tracks not yet shown up on the map
https://openstreetcam.org/user/mohamedkenan
https://openstreetcam.org/user/aal-ldyn-bn-saayd
regards

@ALFURJANI ,I checked track 3186038 and 3186042 under Mohamedkenan. Both tracks could be seen on the main display. Also, I checked track 3192806 and 3192798. Both could be seen on the main display as well. Do you have a specific track that could show up? Thanks

@ALFURJANI , we have done some processing for the tracks you mentioned? Could you please check if those tracks can display on the map? Also, not sure if you raised one issue about the display issue on the app user profile? Could you please check as well? Thanks!

@rwelty1889 , I could see track 3088794 on the main display. Could you confirm this from your side? If everything is ok, could we close this ticket? I would close this ticket by this Fri. Thanks.

Closing the ticket as we resolved this from our side, feel free to reopen if you found it still exists


'JavaScript heap out of memory' error is thrown when using with webpack

I am not sure if this is a bug in material-ui but I am facing this issue after upgrading the material-ui packages.
I am using material-ui in a Electron-JS app, that I am developing that uses webpack for bundling. It was working fine.
After upgrading '@material-ui/core' from '1.4.0'  to '3.1.0' and '@material-ui/icons' from '1.1.0' to '3.0.1', the 'JavaScript heap out of memory' error is being shown.
Basically I am using webpack-dev-server to serve the app. It compiles and serves the app correctly at first. After I make some changes to the source code, the webpack process exists after throwing the following error.
<--- Last few GCs --->

[532:0x105001400]   769315 ms: Mark-sweep 1279.2 (1445.1) -> 1279.2 (1445.6) MB, 2962.5 / 0.0 ms  allocation failure GC in old space requested
[532:0x105001400]   771546 ms: Mark-sweep 1279.2 (1445.6) -> 1279.2 (1426.6) MB, 2231.4 / 0.0 ms  last resort GC in old space requested
[532:0x105001400]   773522 ms: Mark-sweep 1279.2 (1426.6) -> 1279.2 (1422.6) MB, 1975.3 / 0.0 ms  last resort GC in old space requested

<--- JS stacktrace --->

==== JS stack trace =========================================

Security context: 0x2b9521d25879 <JSObject>
    1: /* anonymous */ [/Users/user/project/node_modules/webpack-sources/node_modules/source-map/lib/source-node.js:~342] [pc=0x3df64b77cf1c](this=0x2b95fe18c209 <JSGlobal Object>,chunk=0x2b959b11fad1 <String[61]\:   var warnBadVendoredStyleName = function (name, getStack) {\n>,original=0x2b9561c08f89 <Object map = 0x2b95cd89dab1>)
    2: SourceNode_walk [/Users/user...

FATAL ERROR: CALL_AND_RETRY_LAST Allocation failed - JavaScript heap out of memory
 1: node::Abort() [/usr/local/opt/node@8/bin/node]
 2: node::FatalException(v8::Isolate*, v8::Local<v8::Value>, v8::Local<v8::Message>) [/usr/local/opt/node@8/bin/node]
 3: v8::Utils::ReportOOMFailure(char const*, bool) [/usr/local/opt/node@8/bin/node]
 4: v8::internal::V8::FatalProcessOutOfMemory(char const*, bool) [/usr/local/opt/node@8/bin/node]
 5: v8::internal::Factory::NewUninitializedFixedArray(int) [/usr/local/opt/node@8/bin/node]
 6: v8::internal::(anonymous namespace)::ElementsAccessorBase<v8::internal::(anonymous namespace)::FastPackedObjectElementsAccessor, v8::internal::(anonymous namespace)::ElementsKindTraits<(v8::internal::ElementsKind)2> >::ConvertElementsWithCapacity(v8::internal::Handle<v8::internal::JSObject>, v8::internal::Handle<v8::internal::FixedArrayBase>, v8::internal::ElementsKind, unsigned int, unsigned int, unsigned int, int) [/usr/local/opt/node@8/bin/node]
 7: v8::internal::(anonymous namespace)::ElementsAccessorBase<v8::internal::(anonymous namespace)::FastPackedObjectElementsAccessor, v8::internal::(anonymous namespace)::ElementsKindTraits<(v8::internal::ElementsKind)2> >::GrowCapacity(v8::internal::Handle<v8::internal::JSObject>, unsigned int) [/usr/local/opt/node@8/bin/node]
 8: v8::internal::Runtime_GrowArrayElements(int, v8::internal::Object**, v8::internal::Isolate*) [/usr/local/opt/node@8/bin/node]
 9: 0x3df64ab842fd

[x] This is not a v0.x issue.
[ ] I have searched the issues of this repository and believe that this is not a duplicate.
This is probably a duplicate of https://github.com/mui-org/material-ui/issues/11151 . I am filing a new issue since that is closed.
https://github.com/mui-org/material-ui/issues/5390 is also perhaps related.

Expected Behavior

Webpack should not throw the above error.
Current Behavior

Steps to Reproduce

Link:

Context

Your Environment

Tech
Version

Material-UI
v3.1.0

React
v16.3.0

Browser
Chrome v59.0.3071.115

TypeScript
v2.9.2

etc.

Without a reproduction, we can't do much.


File upload is broken with Nextjs

Describe the bug
File upload with yoga and nextjs works fine if its app router, but in pages legacy router it does not work as expected ... while using bodyParser: false it results with 500 error saying
TypeError: RequestInit: duplex option is required when sending a body.
    at new Request (node:internal/deps/undici/undici:7368:19)

and if I remove bodyParser:false I get a file response that is missing most of its built-in functions like stream and arrayBuffer
Your Example Website or App
none
Steps to Reproduce the Bug or Issue
1- Work with yoga (latest version) example and nextjs 14
2- Toggle bodyParser true/false
Expected behavior
for upload to work as expected
Screenshots or Videos
No response
Platform

OS: macOS
NodeJS: v18.18.0
@graphql-yoga/* version(s): 5.1.1
Nextjs: 14.1.0

Additional context
No response

Hi @mstfash and thank you for the report
Sorry but I'm not adding a lot here but just labeling it according to our new Contribution Guide and issue flow.
It seems already got into stage 1 thanks to your reproduction! Thank you for that!
Now in order to advance to stage 2 we'll need a failing test, would be great if someone could help progress the issues through the stages.
Thank you and sorry that this comment is not a complete solution (yet).

Any status? I can't upload file via mutation in nestjs.

Not yet but as @EmrysMyrddin said above, a reproduction or a PR with a failing test would be better for us or anyone else(who wants to contribute) in order to proceed faster.


Unexpected version

When I try to launch any of the streams I get an error message saying "Unexpected version check output". I have no clue where output is, since stdout gives me nothing.
Livestreamer-twitch-gui version: 0.11.1
NW.js: 0.12.3
OS: Arch Linux
Livestreamer: 1.12.2

The GUI will always validate livestreamer first, before launching any streams. That's where the error is coming from.
What's the output when running livestreamer --version --no-version-check on your machine??? Needs to be livestreamer 1.12.2, or else this error will be shown. See here...
How did you install livestreamer? I've never seen a different output before, so this error is indeed a bit unexpected :confused:.

I don't know what's going on anymore. My livestreamer just broke completely and keeps outputting same error about urllib, it worked before I installed livestreamer-twitch-gui. Close it for now, I need to sort out that first, if it appears again after I fix livestreamer, I'll let you know.

The AUR package has just listed livestreamer as a dependency, so this error should be unrelated...
Good luck fixing the error :four_leaf_clover:

Okay it was livestreamer issue, now everything is fixed. Sorry for disturbing you!


Assertion error when calling cyw43_arch_init() in FreeRTOS

When I try to call cyw43_arch_init() in a FreeRTOS task I get the following error printed to the serial:
assertion "get_core_num() == async_context_core_num(cyw43_async_context)" failed: file "......../pico-sdk/src/rp2_common/pico_cyw43_driver/cyw43_driver.c", line 53, function: cyw43_irq_init

I have the code on my repo here, however it is in a bit of a mess so here's a cleaned up version of the main file:
#include "FreeRTOS.h"
#include <task.h>
#include "pico/cyw43_arch.h"
#include <stdio.h>
#include "pico/stdlib.h"

void wifiConnectTask(void *pvParameters) {
    if (cyw43_arch_init()) {
        printf("failed to initialise\n");
        return;
    }
    cyw43_arch_enable_sta_mode();
    printf("Connecting to Wi-Fi...\n");
    if (cyw43_arch_wifi_connect_timeout_ms(WIFI_SSID, WIFI_PASSWORD, CYW43_AUTH_WPA2_AES_PSK, 30000)) {
        printf("failed to connect.\n");
        exit(1);
    } else {
        printf("Connected.\n");
    }
    while (true) {
        vTaskDelay(500);
    }
}

int main() {
    stdio_init_all();

    TaskHandle_t wifiHandle;

    xTaskCreate(wifiConnectTask, "wifiConnect", configMINIMAL_STACK_SIZE, NULL, 1, &wifiHandle);

    // Note: tried with and without this line, no difference
    vTaskCoreAffinitySet(wifiHandle, 1);

    vTaskStartScheduler();

    while (true)
        vTaskDelay(500);

    cyw43_arch_deinit();
    return 0;
}

Possibly related to #1478?
In addition, I've tried this with both the master and develop branches of the SDK; it is broken with both of them.

The only workaround I've found so far is commenting out the assert statements.

Is this with the latest FreeRTOS where they have merged the smp branch into main... might be worth trying before that happened (it is possible something is broken as a result of the FreeRTOS merge)

Thanks for the code. I'll have a look. It would be useful to know the top commit of Freertos you're using.

I can reproduce the assert with the freertos commit you're using 37678b06568fda7aaff090b77e8e34d6ab29ede4, going back to the smp branch and it works ok. So as @kilograham says, it looks like something has changed.

The problem is that they renamed configNUM_CORES to configNUMBER_OF_CORES, you either need to add #define configNUM_CORES configNUMBER_OF_CORES to the freertos config file or do a search and replace in the pico-sdk, until we can fix this.

Thanks, it works after adding the #define to the config. I'm on the develop branch anyway due to #737 (has been fixed in #1488) and should get the fix when @kilograham approves the merge.


Reading frame from CSV with string column in the middle

The following little DaphneDSL script reads a frame from a CSV file:
example.daphne:
print(readFrame("data.csv"));

data.csv:
111,22.2,hello,333,44.4,555

data.csv.meta:
{
    "numRows": 1,
    "numCols": 6,
    "schema": [
        {"label": "a", "valueType": "si64"},
        {"label": "b", "valueType": "f64"},
        {"label": "c", "valueType": "str"},
        {"label": "d", "valueType": "si64"},
        {"label": "e", "valueType": "f64"},
        {"label": "f", "valueType": "si64"}
    ]
}

The script can be executed by bin/daphne example.daphne.
Expected output:
Frame(1x6, [a:int64_t, b:double, c:std::string, d:int64_t, e:double, f:int64_t])
111 22.2 hello 333 44.4 555

Actual output:
Frame(1x6, [a:int64_t, b:double, c:std::string, d:int64_t, e:double, f:int64_t])
111 22.2 hello 0 333 44

Possible reason:
The problem seems to be related to the return value pos of the function setCString() in src/runtime/local/io/utils.h. While the return value is used as the position of the next column in ReadCsvFile<Frame>::apply(), it's actually a position relative to the current column. The existing test cases don't seem to trigger this case, because they have string column either as the first column (where relative and absolute positions are the same) or last column (where there is anyway no next column) in a CSV file. The case is complicated by the fact that the position must start from 0 again if the string cell actually spans multiple lines.

This bug is currently preventing us from reading the lineorder table of the Star Schema Benchmark.

Hi, I will work on this issue.

Thanks! Please go ahead.


JerseyClientBuilder not working with resteasy-client

If you use JerseyClientBuilder with resteasy-client dependency:
<dependency>
    <groupId>org.jboss.resteasy</groupId>
    <artifactId>resteasy-client</artifactId>
    <version>3.5.1.Final</version>
</dependency>

occurs an Exception:
javax.ws.rs.ProcessingException: java.lang.ClassCastException: org.jboss.resteasy.client.jaxrs.internal.ClientRequestContextImpl cannot be cast to org.glassfish.jersey.client.ClientRequest

To reproduce you can put the maven dependency of resteasy-client on dropwizard-example and use like that:
javax.ws.rs.client.Client client = new JerseyClientBuilder(environment)
        .using(configuration.getJerseyClientConfiguration())
        .build(getName());

client.target("http://www.google.com.br").request().get();

@aneto-core Dropwizard is an opinionated framework which only support Jersey as a JAX-RS implementation. RESTeasy (or any other JAX-RS implementation) is not and will not be supported by Dropwizard.

This makes me sad as someone who is using Dropwizard with Keycloak, which is hardcoded to use RESTEasy in their client.
Note that Dropwizard supports both Apache HttpClient and Jersey Client, at least, according to the documentation.


The activesupport gem is a runtime dependency

not just a development dependency, because of the use of String#blank?
and Array.wrap in MailInterceptor::Interceptor.

Looks good to me.
@vipulnsward please take a look.

Thanks @billhorsman !


Download button duplicating item in unordered list

Describe the bug

Clicking on the download button duplicates the item in the drop down.
To Reproduce

Use version 3.0.1 of Vitessce

Click on Vitessce download icon and two duplicate items appear in the drop down.

Data sharing

Expected behavior

Clicking on download drop down reveals unique available downloadable data.
Screenshots

Configuration

View configuration is 1.1.21.
Environment:

npm version: 6.14.16
Node version: 14.19.0
Browser: Firefox
Browser version: 115.0.3

Hi @Dert1129, thanks for reporting this issue!
Could you try again with the new release of Vitessce? The latest releases (v3.1.0+) include #1609, which should fix this issue.


Disable add related in read-only case

[x] joeferner
[ ] mwizeman joeybrk372 jharwig sfeng88

Both keyboard command alt+r and context menu
Points of Regression: Graph "Add Related" in readonly case
CHANGELOG
Fixed: Disable Add Related items in a read-only case

LGTM


command-line option for visualisation?

Dear all
It appears that the command-line version of BGCViz is limited to only the conversion of BGC-tool outputs to corresponding csv files that can be subsequently uploaded to BGCViz website. Please correct me if I am wrong, but I cannot find instructions on how to generate an output visual, based on command line, similar to what can be achieved from the BGCViz website. Is this an inherent limitation of the command line way of running BGCViz? Thanks
Marc

Hello Marc,
Unfortunately, BGCViz output charts cannot be produced from command-line version. We plan include some basic functionality regarding this issue in the future updates.
I will leave this issue open for future updates


Add support for retries

When an event failed to be sent the SDK should retry sending it. There should be a way to configure how many times to retry before giving up, the backoff policy (eg linear, exponential, etc..) and associated parameters (eg. backoff delay)

I can give shot at this one.

Here my current thoughts on how to implement this:

add a new retry client (NewRetried) for wrapping clients that don't support retrying natively.
add ContextWithRetry(max, backoff)
update default client to use NewRetried

The sarama client supports retries natively but only support configuration at the client level, not at the producer level. In that case ContextWithRetry will be ignored.
I haven't looked at the other protocols but I'm guessing the general idea is the same.
@n3wscott @slinkydeveloper comments?

I think this is an interesting option for http. I would implement it in this method: https://github.com/cloudevents/sdk-go/blob/master/v2/protocol/http/protocol.go#L102
you could make a little helper class that replaces:
	resp, err := p.Client.Do(req)
	if err != nil {
		return nil, protocol.NewReceipt(false, "%w", err)
	}

	var result protocol.Result
	if resp.StatusCode/100 == 2 {
		result = protocol.ResultACK
	} else {
		result = protocol.ResultNACK
	}

And normal it would just try once.
And if you find ContextWithRetry(max, backoff), implement a retry and backoff sender.
The last thing I would do is make a small report about how many times it took to send in the result.
I would extend the HTTPResult as HTTPRetriesResult that lets you report things like how many retries and how long it took and the response code of the final try (plus wrap in ACK or NACK)

helper class or method.

I guess we can start with HTTP and generalize later on.


exception=AttributeError("'list' object has no attribute 'close'")

/app/commands/btt_commands.py:142> exception=AttributeError("'list' object has no attribute 'close'")>
2023-01-03T05:52:08.157893+00:00 app[worker.1]: Traceback (most recent call last):
2023-01-03T05:52:08.157894+00:00 app[worker.1]:   File "/app/commands/btt_commands.py", line 278, in _btt_wr_list
2023-01-03T05:52:08.157894+00:00 app[worker.1]:     cur.close()
2023-01-03T05:52:08.157895+00:00 app[worker.1]: AttributeError: 'list' object has no attribute 'close'
Does not cause the bot to break but really should clean this up

The way to do this would be to label vars properly and close the connection when you dont need it


"Robeasca" is listed in the drop-down menu

Description:
"Robeasca" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Robeasca" in the search bar.

Expected results
The location is listed in the drop-down menu as "Comuna Robeasca, BUZĂU".


Optional verify parameter for requests package

Some projects(including that I'm currently working in) require interaction on servers with self signed certificate for testing purposes.

Perhaps it would be nice to give more descriptive name to the argument in JSONRPCDaemon constructor. Something like verify_ssl_certs, or the opposite, no_check_certificate (similar to wget option)?
Nevertheless, it's a good idea and we should have it.

Merged, thanks!


Single-line comments don't work

Background
Prettier automatically converts presenterm's slide title format into a level-2 heading, so it's necessary to add a <!-- prettier-ignore --> comment.
Slide Heading
---

becomes
## Slide Heading

I saw the fix in #23, but, in my opinion, single-line comments should be expanded to allow anything inside. It's much more disruptive to have errors when you write a single-line comment than to accidentally write a comment command.
Alternatives
You could just write
<!--
prettier-ignore
-->
Slide Heading
---

but this seems like a pointless limitation.

than to accidentally write a comment command

The issue is not accidentally writing a comment command but trying to write one, making a mistake, and the command not doing anything because the error was ignored. The problem here is I don't want comment-command parsing to be best effort: if you write a comment command the app needs to ensure it's a valid one. e.g. if you write paus or end_side or column_layout [3, 4] this must fail. Otherwise you write your presentation thinking it will be displayed in one way and then you'll either not notice or you will be confused as to why the command isn't working because there's no messages anywhere indicating so.
I'd argue, rather than having to fight against prettier every time you write a slide heading, maybe it's worth disabling it globally? It seems like more of a hassle to have to tag every single slide title with a prettier-ignore (whether it's a single line or multi line) than just disabling it altogether. I'd also argue prettier is doing the wrong thing (and it seems like they have an open issue about this).
I'm open to other solutions though. e.g. there could be a list of allowed single line comment commands in ~/.config/presenterm/config.yaml or something. The only requirement is that if your intent is to write a comment command, then it must be a valid one.

I'd also argue prettier is doing the wrong thing

100% agree!

if you write a comment command the app needs to ensure it's a valid one.

Also agree, but completely replacing single-line comments with comment commands loses an important (at least to me) part of writing markdown. I often write a sentence or paragraph
IMO, there needs to be a distinction between comments and comment commands. Maybe a prefix?
Something like
<!-- cmd:end_slide -->

This way, you can still write whatever comments you want (with the exception of ones starting with cmd:), and invalid commands fail (cmd:end_side).
Obviously this would be a huge breaking change, so there's probably a better way of doing this.

Yeah maybe there should have been a prefix to disambiguate, but that makes commands even longer. Just like you'd be okay with something like that, other people (#65) want even shorter comment-commands so it's hard to keep everyone happy.
As you said, you can use multi-line comments to workaround this issue.
Solutions we can implement here:

Let you have a list of comments that you want to skip when parsing commands. This would work for the prettier case but not for any sentence you want in a single line so it's not great.
Let you configure a prefix for commands. By default we still do slide_end but you can define a command_prefix in ~/.config/presenterm/config.yaml where you can set an arbitrary string and only if the single-line comment-command starts with that, it is considered as such. Would this work?

The command_prefix option seems to be exactly what I'm looking for! The only problem I could see is if you are collaborating with someone who doesn't have the same prefix/no prefix. In that case, you would have to change your global settings to work on that project, then change them back to work on anything else.
Maybe a per-project setting could be added in the YAML frontmatter too?

The command_prefix option seems to be exactly what I'm looking for!

Awesome! I'll work on that next.

Maybe a per-project setting could be added in the YAML frontmatter too?

Yep, I was thinking of this as well. I'll do that too. There's barely anything in the config file yet so it hasn't turned up to be an issue but it will eventually.

This should be fixed in master. I'm in the process of adding docs as part of the next release but for now:


Clean up application data model

Thanks for taking a look!

Looks like 2611f4937689d612e1d86f14581534924ade1cda snuck into this PR. Can we roll that one into this PR for expediency?

Scratch that, I accidentally just submitted it. Luckily, I didn't approve and github is tracking the files I've looked at thus far, so I'll add more comments tomorrow!

This is great! I find this set up much easier to understand than the previous one. Can't wait to get this into master!
Just left a couple of small comments, otherwise, looks good!

Glad you find it clearer! We're getting there!

I caught a build error that slipped in in previous pulls rolls eyes. We need unit tests!
I slipped it in in this review (9f2165e61580b2232f35533385b55c3e8ecd8f17) if you don't mind restamping.
In the future we should create unrelated fixes in a separate PR. Thanks!!

I caught a build error that slipped in in previous pulls rolls eyes. We need unit tests!
I slipped it in in this review (9f2165e) if you don't mind restamping.
In the future we should create unrelated fixes in a separate PR. Thanks!!

Ah interesting, sorry for missing that. I even pulled this branch into one of my own and it built just fine! I must've gotten it before the culprit commit?
It's tricky to constantly verify builds, especially while new commits roll in. Is there some kind of automated check we can setup through github? Similar to the CLA check on PRs? Do we need to integrate with jenkins or trarvis or some such?

It was actually from a previous PR but I somehow missed it. Unit tests on
commit or pusb would have caught it because it would have failed to
compile. Thanks for letting me sneak this one in - sorry for setting a bad
example!
On Wed, Jul 17, 2019, 7:15 AM Scott Olsen<EMAIL_ADDRESS>wrote:

@scolsen approved this pull request.
👍
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/google/ground-android/pull/152?email_source=notifications&email_token=AABXVUVF22KORXWTPL2OWVLP74SPTA5CNFSM4ID3IDWKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOB6XBO3I#pullrequestreview-263067501,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABXVUW35LIB3E3LHXSUBTDP74SPTANCNFSM4ID3IDWA
.


Run using python and convert nested json to csv

Hi, I want to know how to use Json2Csv for Python.
Also, I have a nested json of tweets. I wish to rename the columns while converting it to csv. Again I am trying to get the retweeted tweets into the same column as the main tweets. But it shows only 0 or 1.

See this comment for usage with python https://github.com/zemirco/json2csv/issues/145#issuecomment-250990585


Livereload 404s

Hi! I'm having some weird issue where I get 404s when modifying files.
I have a basic site:
npm init -y
yarn add next
mkdir pages
cat >pages/index.js <<EOL
import React from 'react'
export default () =>
Hello world!

EOL
jq '.scripts.dev="next"' package.json | sponge package.json
npm run dev

Next starts and initial pageload looks fine. However, when I modify index.js in any way, I get a 404. Looking at devtools I see;
next-dev.bundle.js:32334 GET http://localhost:3000/index.json 404 (Not Found)
Any idea what could be causing this? Could it be a global npm package or something?
OS: Ubuntu 16.04
Node: v6.9.2
Next: v1.2.3

I have this exact same problem. It seems to be a pattern. I make an edit, get this 404, make another and it comes good, make another and I get the 404, make another and it comes good, so on and so forth.
 Every second save results in 404. I am working on Linux (Arch Linux x64), and I have noticed that if I save and exit VIM (:wq), everything is fine (!), but if I just save (:w), I am getting this weird 404.

What?? :see_no_evil: I can confirm I'm seeing the same behaviour as @frol regarding totally closing vim (:wq) - if I do that it works as expected. Could it be related to *.swp files then? Is there a .nextignore file we can add? :stuck_out_tongue_closed_eyes:

@asbjornenge could you try using the latest beta build? npm install next@beta. I've just tried using vim with hot reloading, but for me everything is updating correctly. Even after totally closing, opening again and using :w and :wq combinations.

It seems to be working fine now (tested on next@2.0.0-beta.18)! Thank you!

I'm gonna close now.
@asbjornenge is this is still there please re-open.
(Make sure to test with the latest beta)

Can confirm that it now works with the latest @beta :grin: :rocket: :tada:


Need the Api Configuration

Hi, Alpay in your documentation there is API_BASE_URL and Other Api
How did you do that? Can you explain

He did that by creating separate backend for the email and the service status services. I think
@HeyFatimah

Can you give me idea like how can i do that using mongoDB

I Used Mongo DB Api on this but the email does not work
(https://portfolio-site-nine-beta.vercel.app/)
Can you check in your local?

@HeyFatimah I used a service status iframe code. Check mine and u will know. Also we can disable the translation switch in the navbar

I dont want to disble the translation switch i just want to use the contact form

Hey guys, the api base url is only for development purposes and points to the local url on render.com, where the backend was deployed.
So it's for the backend. The backend is small server builds with node js +express+ nodemailer.
You don't need mongodb for that purpose.
The service api points betterstack.com. You can sign up for free and builds your own uptime monitor for your services.
Actually I don't use react anymore for my website. It's only based on next.js and SSR+nodemailer.
I hope this clarify this a little bit.
Kind regards.


Domain activation

Before opening, please confirm:

[X] I have checked the custom domains troubleshooting guide.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.

App Id
d1n5yymf5qivnm
Region
us-east-2
Step
Domain activation
Status
Failed
Issue or question
My domain can't be activated.
I am using route53 for faster deployment but it continues to say that the CNAME does not match the one provided by amplify. But Amplify is supposed to modify the CNAME for me and, as I could see in Route 53, the CNAME did match the current cloudfront distribution domain, so I don't know what is the problem here if everything is perfectly configured.
Error message
Something went wrong. Please refer to our troubleshooting guide below to diagnose the issue. After fixing the issue, choose retry below. If retrying fails, please create an issue in our GitHub issue tracker.
Error message: One or more of the CNAMEs you provided are already associated with a different resource.
Additional information
No response

Is there any way to know what is the different resource associated?

We are also having the same issue using AWS Amplify, the CNAME records do get created but an error appears at the validation stage suggesting that the CNAME is already associated to another resource.
As a workaround I seem to be able to use a different subdomain, however, when trying with the root domain we keep getting this error message despite having removed the records before retrying.

Is there any way to know what is the different resource associated?

We are working on surfacing this information in the console to make it easier for customers to view associated resources.
Looks like your domain is resolving now app*.g****.w***

I have a similar issue and the website has been down for a week now.

I don't have any cloudfront setup. So I can't even troubleshoot as described in this document https://docs.aws.amazon.com/amplify/latest/userguide/custom-domain-troubleshoot-guide.html
I really like AWS Amplify when it works but I can't even explain to anyone why the site is down .

@reyts I had to check at Cloudfront if the domain was assigned to any distribution, no matter if the cname is pointing to the cloudfront distribution of the Amplify deploy.
Just unassign the domain of the other cloudfront distribution or delete the distribution and it will work.

If you don't have cloudfront set up, check another resources such as elastic beanstalk, API Gateway, etc.


Can't remove displayID once set

Prerelease 3:
sbol-10204 says that a displayID is optional, but once I set one I cannot remove it. If I delete the contents of the blank, I get an sbol-10204 error telling me it's optional and I've done it wrong.

It is optional in the specification, but it is required in SBOLDesigner.  Therefore, we should get ahead of the validation error message and report that displayId is required.

The validation error message is now: "The displayId must not be empty."


OnScanResult doesn't fire when resuming XF App

Hello,
I'm using more or less the "usage" example from the XF site (https://components.xamarin.com/gettingstarted/zxing.net.mobile.forms). This works fine until I go to another app and come back. When I come back the camera is on and focussing on barcodes but the OnScanResult event doesn't fire.
This is on Android 7 (HTC 10), I don't have a UWP or iOS device to try it on. Latest versions of everything (XF, VS, nugets, etc)
I'm fairly new to XF so maybe I am missing something?
Thanks

OK, this seems to be a debug build/VS issue. I removed the app from my phone, made an APK in release mode and installed that...works fine.
Still be interesting to know if this is a known issue/a workaround as it makes debugging a little difficult, but not a major problem.

Hard to say without a good repro but I did fix a similar bug on Android in the latest release.


[fud] Missing minimum version for networkx

I was working through the getting started guide and ran into the following error: fud seems to depend on the python package networkx, specifically the method all_simple_edge_paths. It's hard to tell from the changelog but it seems the method was introduced sometime between 2.4 (my version) and 2.6 (current).
calyx % fud e examples/tutorial/language-tutorial-iterate.futil \
  -s verilog.data examples/tutorial/data.json \
  --to dat -v

Produces the following output.
Traceback (most recent call last):
  File "/usr/local/anaconda3/bin/fud", line 8, in <module>
    sys.exit(main())
  File "/usr/local/anaconda3/lib/python3.7/site-packages/fud/main.py", line 295, in main
    exec.run_fud(args, cfg)
  File "/usr/local/anaconda3/lib/python3.7/site-packages/fud/exec.py", line 67, in run_fud
    path = construct_path(args, config, args.through)
  File "/usr/local/anaconda3/lib/python3.7/site-packages/fud/exec.py", line 45, in construct_path
    path = config.REGISTRY.make_path(source, target, through)
  File "/usr/local/anaconda3/lib/python3.7/site-packages/fud/registry.py", line 46, in make_path
    all_paths = list(nx.all_simple_edge_paths(self.graph, start, dest))
AttributeError: module 'networkx' has no attribute 'all_simple_edge_paths'

This can be fixed as needed by manually upgrading (pip install networkx --upgrade), but adding an explicit dependency on "networkx^2.6" should solve this for any future installs with old versions of networkx present.

Hm, there is a dependency on the networkx package and I assumed it'd install the right one: https://github.com/cucapra/calyx/blob/master/fud/pyproject.toml#L21
@ohxh do you want to take a shot at fixing the issue and opening a PR?

Indeed; a version constraint here would be awesome, if you don't mind throwing together a quick PR, @ohxh:
https://github.com/cucapra/calyx/blob/84e906e523569a75800e40ec15661ee759593c4d/fud/pyproject.toml#L21
(FWIW, the way pip works is that it doesn't install anything if the dependency is already satisfied on the system—as it would be if you happen to have manually, separately installed the same package. So we should declare the minimum version we require.)

@rachitnigam @sampsyo I'm just getting involved in this project and can't branch / PR in the repo. I'm talking with @EclecticGriffin on Thursday and I'll revisit after that. Probably the rest of these deps should get minimum versions too

Cool! FWIW, it's also totally cool to submit a PR from your own fork (as opposed to a branch in this repo):
https://docs.github.com/en/github/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/creating-a-pull-request-from-a-fork


Fix compiler issues in the Swift example project

Looks like the FZAccordionTableViewDelegate was switched to use nullable headers but the Swift example project wasn't updated to consume those changes.

Looks good. Thank you!


update setup.cfg pytest settings

removes a pytest warning on my laptop using v3.0.2

Looks like Travis is now failing, though. Is Travis using an older version of pytest?

I guess so.

I can live with the warning. We can still do this when travis updates.


[Sticked] Top Feature Requests

Demographics (advertising features)
Automatic campaign tracking

Hi,
I was just wondering how the demographics feature was progressing? I have been using another Google Analytics plugin, but now our client wants to be able to use IDFA and rather having to write my own plugin I was hoping to use this one.

Not sure if this is an existing feature, but I wasn't able to find any way of implementing it. We currently don't get any os or device info when running on Mac (this may be the same on standalone windows builds). The actual reporting of stats on mac works perfectly, however.


[ISSUE #6785] Isolate the remoteChannel by group

Which Issue(s) This PR Fixes

Fixes #6785
Brief Description

How Did You Test This Change?

Codecov Report

Merging #6786 (bde54e3) into develop (9dfd4a9) will increase coverage by 0.05%.
The diff coverage is 100.00%.

@@              Coverage Diff              @@
##             develop    #6786      +/-   ##
=============================================
+ Coverage      42.84%   42.89%   +0.05%
- Complexity      8976     8987      +11
=============================================
  Files           1108     1104       -4
  Lines          78444    78376      -68
  Branches       10212    10203       -9
=============================================
+ Hits           33607    33620      +13
+ Misses         40617    40542      -75
+ Partials        4220     4214       -6

Impacted Files
Coverage Δ

...client/consumer/store/RemoteBrokerOffsetStore.java
68.75% <100.00%> (-0.28%)
:arrow_down:

...etmq/proxy/service/sysmessage/HeartbeatSyncer.java
68.04% <100.00%> (ø)

... and 23 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


If updating HPA fails due to conflict, try again up to 3 times.

This is a common concurrency case - when doing get, update, put on a resource
which may be updated elsewhere, one has to either use synchronization mechanisms
or retry.
I don't believe that k8s has a synchronization mechanism to prevent against
the HPA metrics checking from updating the status at exactly the same time
that ss is trying to update the min/max replicas.
Since this new code is retrying, there is a risk in colliding with another
ss cron entry as well, and there are synchronization mechanisms (mutex)
to help ensure that the currently scheduling thing (which got there first)
gets to retry, before the new one comes in.
Fixes #28.

The bulk of this change is from mockgen to create the myriad of mocks required to test the scheduledScalerHpaCronAdd function.
Should you need to update, these were generated using
go generate ./...

@ericuldall - can you take a look at this one as well? It's a little more exhaustive than the last one (to add mocks for a unit test) but any panic can cause issues with the CRD, so removing them, also fixes #24 as well.

Fixes #24
Fixes #28

If there's any chance you'd be able to hook up https://travis-ci.org/ for this to at least run tests, that would be helpful 😄

@ericuldall - can you take a look at this one as well? It's a little more exhaustive than the last one (to add mocks for a unit test) but any panic can cause issues with the CRD, so removing them, also fixes #24 as well.

Fixes #24
Fixes #28

If there's any chance you'd be able to hook up https://travis-ci.org/ for this to at least run tests, that would be helpful 😄

I'll create a new issue to get CI running the tests.
Also, what do you think about making that retry number configurable? I'm thinking at the very least it should be globally configurable at the scheduled scaler controller but there may be value in allowing each scheduled scaler definition to have a custom retry configuration based on how important it is to the end user. Thoughts?

That makes sense; I'm thinking for HPA, you only need 1 retry as HPA only fires every 15s the chances and observed collisions are very rare, but we have seen them.  I can make the config default to 1 retry and be settable


Spotify stopped working

Blocking spclient.wg.spotify.com causes Spotify to no longer work on any device
https://github.com/dfederlein/dnsbl-dfed/blob/master/spotifyads.txt#L31
See also: https://www.reddit.com/r/pihole/comments/cbt7dp/spclientwgspotifycom/

Usually I add these to the whitelist for pfblockerng (ie: These aren't meant for pihole, for that I'd use https://dbl.oisd.nl/ )


Hide Next button

it's possible to hide next button ?

You can use overlayBuilder parameter to define custom overlay and hide the next button.
             IntroStepBuilder(               order: 4,               text: 'Hello this is my intro 4',               padding: EdgeInsets.zero,               overlayBuilder: (params)=>ColoredBox(color: Colors.black.withOpacity(0.3)),               builder: (context, key) => SizedBox(height: 30,width: 30,)             )

Please check the sample code of the project for detailed usage instructions.


Remove ChannelID unique constraint

Multiple runs can not share the same channel ID until the unique constraint is removed. Make a migration to remove the constraint.

Fixed by #1549


Lost and found Section

Include a dedicated section where people can report lost or found pets. This can serve as a valuable resource for reuniting lost pets with their owners.
@akshitagupta15june kindly allow to work on this idea as GSoC'23 Contributor

it is not currently needed for website


add readline support

fork and go mod replace upstream readline with slingamn/readline, branch experimental
Windows support is passable
readline must be enabled explicitly with --readline

No support for history files as yet
make minimal compiles readline out

Attempts to improve readline beyond these rudimentary improvements: https://github.com/slingamn/readline/commits/experimental have stalled. I'm probably going to merge this in its current form, then release irc-go, then release ircdog.


Why it is so slow

I start training with the dev code on the dataset of lego, multi-scale, 3 3090 GPU, and one epoch takes over 4 or 5 hours, wtf?

Here's log:
python train.py --out_dir out-lego --data_path ./data-blender/lego --dataset_name multi_blender exp_name lego num_gpus 3 train.num_work 48
GPU available: True, used: True
TPU available: False, using: 0 TPU cores
IPU available: False, using: 0 IPUs
initializing distributed: GLOBAL_RANK: 0, MEMBER: 1/3
initializing distributed: GLOBAL_RANK: 2, MEMBER: 3/3
initializing distributed: GLOBAL_RANK: 1, MEMBER: 2/3
distributed_backend=nccl
All distributed processes registered. Starting with 3 processes
LOCAL_RANK: 2 - CUDA_VISIBLE_DEVICES: [0,1,2]
LOCAL_RANK: 0 - CUDA_VISIBLE_DEVICES: [0,1,2]
LOCAL_RANK: 1 - CUDA_VISIBLE_DEVICES: [0,1,2]
Validation sanity check: 0it [00:00, ?it/s]/root/anaconda3/envs/mipnerf/lib/python3.9/site-packages/pytorch_lightning/trainer/data_loading.py:110: UserWarning: The dataloader, val_dataloader 0, does not have many workers which may be a bottleneck. Consider increasing the value of the num_workers argument(try 48 which is the number of cpus on this machine) in theDataLoader` init to improve performance.
rank_zero_warn(
Epoch 0:   1%|▍                                        | 99/9240 [02:36<4:00:32,  1.58s/it, loss=0.296, v_num=2, train/psnr=10.80]

I tried branch master, it gives:
python train.py --out_dir result --data_path data/lego --dataset_name multi_blender exp_name lego num_gpus 3
GPU available: True, used: True
TPU available: False, using: 0 TPU cores
IPU available: False, using: 0 IPUs
initializing distributed: GLOBAL_RANK: 0, MEMBER: 1/3
initializing distributed: GLOBAL_RANK: 2, MEMBER: 3/3
initializing distributed: GLOBAL_RANK: 1, MEMBER: 2/3
distributed_backend=nccl
All distributed processes registered. Starting with 3 processes
LOCAL_RANK: 2 - CUDA_VISIBLE_DEVICES: [0,1,2]
LOCAL_RANK: 1 - CUDA_VISIBLE_DEVICES: [0,1,2]
LOCAL_RANK: 0 - CUDA_VISIBLE_DEVICES: [0,1,2]
Validation sanity check: 0it [00:00, ?it/s]/root/anaconda3/envs/mipnerf/lib/python3.9/site-packages/pytorch_lightning/trainer/data_loading.py:110: UserWarning: The dataloader, val_dataloader 0, does not have many workers which may be a bottleneck. Consider increasing the value of the num_workers argument(try 48 which is the number of cpus on this machine) in theDataLoader` init to improve performance.
rank_zero_warn(
Epoch 0:   0%|▏                                        | 34/9236 [00:55<4:09:20,  1.63s/it, loss=0.413, v_num=1, train/psnr=9.210]

And for single GPU for branch master, it gives:
python train.py --out_dir result --data_path data/lego --dataset_name multi_blender exp_name lego
GPU available: True, used: True
TPU available: False, using: 0 TPU cores
IPU available: False, using: 0 IPUs
LOCAL_RANK: 0 - CUDA_VISIBLE_DEVICES: [0,1,2]
Validation sanity check: 0it [00:00, ?it/s]/root/anaconda3/envs/mipnerf/lib/python3.9/site-packages/pytorch_lightning/trainer/data_loading.py:110: UserWarning: The dataloader, val_dataloader 0, does not have many workers which may be a bottleneck. Consider increasing the value of the num_workers argument(try 48 which is the number of cpus on this machine) in theDataLoader` init to improve performance.
rank_zero_warn(
Epoch 0:   0%|                                        | 58/27682 [00:24<3:18:03,  2.32it/s, loss=0.368, v_num=2, train/psnr=9.820]

conclusion： single gpu give a much faster speed of training？

results: after almost 20 hours, 8 epochs on 3090, it achieve pnsr around 34-35, and it seems some problem on my GPUs, sorry about that.


Update hugie to follow v2 Inference Endpoint

Updates hugie to use the new v2 specification from Hugging Face.
In particular:

Uses datamodel-code-generator to generate the models_v2.py
Subclasses the new Endpoint class to add a from_json classmethod.
Replaces InferenceEndpointConfig class with the new EndpointV2 class.
Adds Makefile to automate the creation of hugie/models_v2.py.

Next step is to fix failing tests.


Migrate splitRe to native function

Migrate splitRe to native function path.sep
Remove unused variable
Update to two last versions of Node in .travis.yml

Please rebase so as to avoid merge commits - if you check the "allow edits" box, I can take care of it for you.

Done!

@nucliweb there's still a merge commit - you'll need to rebase on top of latest master on the command line to remove it.

I checked "Allow edits from maintainers"

@nucliweb thanks, it's all rebased - but there's a legitimate test failure on node 0.6 now.

🤔 We want support to version 0.6?

@nucliweb yes, this package supports 0.6 and will continue to do so until there's a good reason to actively drop support for it - and a minor refactor isn't one.

@ljharb So we can not merge this PR 😓

@nucliweb not without a fix for 0.6 - overall though I do prefer the use of path.sep, so I'd love to see this be able to get in.

This will need to be rebased after #113 goes in.


Superside

⛔ The following links are not accessible, but they are required:

Job Listing URL: https://careers.superside.com/creatives-illustration – is not accessible

Careers Links

URL
Verified
URL
Verified

https://careers.superside.com/creative-generative-ai
✅
https://careers.superside.com/senior-product-marketing-manager
✅


Linodes List View - Add LinodeID and plan info to view

At a glance in Linode list view, display the original linodeID which we use to identify and bill for Linodes. Even if customers change the name of the Linode continue to display the initial LinodeID. This keeps parity with the way we display the Hostname for our NodeBalancers. With NBs, customers are able to rename them but we still show the original hostname without allowing them to edit it.

Also, display a brief summary of the plan details on the same line in the list view.

ADD - original LinodeID to Linode > Settings > Display that cannot be edited or renamed.
https://cloud.linode.com/linodes/centos-block1/settings

We should display the ID# at minimum in ALL Edit dialog boxes.

That seems less useful to me than where we currently display ids: on label hover.
Furthermore, the majority of users won't care what the id is. The classic Manager doesn't show ids at all. It's most helpful IMO to developers trying to write scripts and use the Manager as a helper.


How do I integrate links with inertia Link component

Inertia uses its own Link component: https://inertiajs.com/links
What is the best way to integrate all flowbite-svelte links with this Link component from inertia?

You can use action prop.

You can use action prop.

Does this work for all components like navbar links etc too?

At the moment only Link and SidebarItem component. I can add it to NavLi component if you wish.


ci: Add test for Elasticsearch

This patch adds an integration test for booting an Elasticsearch
cluster, and then creating, seeding, and querying an index as done in
the Kibana guide (https://github.com/quilt/kibana#interacting-with-the-cluster).

Hmm, not sure why quilt-bot thinks I need to review this ... I'd be happy to review it if you want me to Kevin, but I don't think its necessary.

@ejj, no need for you to review. This PR needs to be merged before this one can be merged, though: https://github.com/quilt/elasticsearch/pull/1.


use tmpnam instead of tmpnam_s for test routine on unix

Yes there is a hypothetical security issue with tmpnam but we are testing a routine which takes a file path to open, so there is no nice way around this.

closes #10


feat: Modular Character of a locally compact group

Definition of the modular character map, and proof that it is a homomorphism.

[x] depends on: #10383
[x] depends on: #10401

I have some doubts about this PR which I hope you can help in resolving:

How to define (and name) properly the function form of this, before the proof of it being a homomorphism.
Whether to add a version of the map that does not depend on the given Haar measure.
A technical problem - to_additive fails on modularCharacter. I am not sure what to do with this.

Thanks!

@atarnoam What's the current status with the PR?
Github doesn't always present the history in the correct order, so I'm not sure about this; but it looks like the PR didn't actually change at all between Sebastien tagging it as "awaiting author", and you removing his tag and setting it back to "awaiting review". Is it possible that you made changes to the code in a local branch on your machine, but forgot to push them to github, or something like that?

@atarnoam Are you planning to continue working on this PR? If not, please feel free to close it, or label it with please-adopt.

@atarnoam, it looks like the PR hasn't moved since we last pinged you. We're going to close this for now, just for the sake of cleaning up our PR queue. But please do feel free to either re-open the PR if you'd like to continue working on it, or come across to the PR reviews zulip stream to ask for advice or request that we look at this again.


2 IpField controls bound to same IpAddress string results in lost focus

Hi!
I am making an app that allows users to change settings for another piece of software.
We provide a section at the top of "most commonly changed settings" that we know will get modified often, however we then have different sections on the page for specific areas. An IP Address is one that will get changed often so we have 2 IpFields added in our xaml, one in the common section, and one in the "Communication" section.
Both of these IpField controls are bound to the same property in the viewmodel called IpAddress.
The bug is that when I am typing into one of the IpFields, as soon as I enter a character, the IpField I was typing in loses focus, and then both IpFields have the single character that I typed in them, but I cannot type further without clicking into the control again.
My relevant xaml looks something like this and any advice you might have for this is super helpful:
<UserControl d:DataContext="{d:DesignInstance Type=MyViewModel}">
   <Expander Name="CommonSettings">
       <IpField IpAddress={Binding IpAddress}/>
   </Expander

   <!-- This expander is a section specifically for communication related controls -->
   <Expander Name="CommunicationSettings">
         <!-- Notice I am binding to the same property as above on my view model-->
         <IpField IpAddress={Binding IpAddress}/>

         <!-- more controls related to communication -->
   </Expander

   <Expander Name="SomeMoreSettings">
       <!-- more controls -->
   </Expander
   <!--
           Note that there are MANY expanders here, and the common section above is meant to put a lot of the common settings
           but that each individual section also has the settings as well
    -->
</UserControl>

Hi!
I change your code to :
<ip:IpField IpAddress="{Binding IpAddress , Mode=TwoWay}" />
and work
I fork this repository and update code and sample
check this on :
https://github.com/omidmohajers/IPUserControls

HI @zanderman112, I Will Create multiple binding sample very soon like this :

Hi @zanderman112
check this for data binding sample :
https://github.com/omidmohajers/ip-changer


Entry point and script are maybe redundant?

@acordonez and @paullric, we're all set on conda-forge, as you may have seen me mention. I can do the following:
conda config --add channels conda-forge
conda config --set channel_priority strict
conda create -y -n test -c conda-forge/label/cmec_dev python=3.9 cmec_driver
conda activate test
which cmec_driver.py
which cmec-driver
cmec-driver --help
cmec_driver.py --help

It's a little confusing maybe having both the cmec-driver entry point and the cmec_driver.py script. Would it make sense just to have the entry point?
Also, let me know when you want to do another alpha or beta release and I can help with the process on conda-forge.

@xylar Thanks for checking the install! It would make sense just to have the entry point - I assume I just remove "scripts=['cmec_driver/cmec_driver.py']" from setup.py next release?

Yes, exactly.  That should do it. The file will still be around to be used as a module, it just won't be in a user's execution path.


Fix Id for MenuItem

It would be very helpful if a fixed ID could also be passed as a parameter to MenuItem. Maintaining the numeric index can be cumbersome when you want to build a dynamic menu tree, especially if the number of children changes depending on the situation. With a fixed ID, you could, for example, pass a string that is unique for each node.

Could be a duplicate of #14. There I'm asking for a more generic payload there and MenuItem returned itself. This could be using as fixed id as well. But your suggestion might be easier to implement.


useWindowSize does not work on App Router (nextjs)

What is the current behavior?
Gives an error:
app-index.js:32 Warning: Prop width did not match. Server: "Infinity" Client: "1290"
This is probably due to app router running the code on server side even with "use client" specified.
Steps to reproduce it and if possible a minimal demo of the problem. Your bug will get fixed much faster if we can run your code and it doesn't have extra dependencies other than react-use. Paste the link to your JSFiddle or CodeSandbox example below:

install nextjs 14
use app router
create a component

"use client";

import useWindowSize from "react-use/lib/useWindowSize";

const StepSubmittedWidget = () => {
  const { width, height } = useWindowSize();

return (<></>)
}

export default StepSubmittedWidget;

What is the expected behavior?
Gives the browser's window sizes
A little about versions:

OS:  MacOS
Browser (vendor and version):  Chrome
React: 18.2.0
react-use: 17.4.2
Did this worked in the previous package version? No

Running into the same issue here :x

I don't think this is an issue, if you are doing Server Side Rendering for that component, it won't have a screen size to fetch.
Don't use a server component, try to use a client component to fix this issue
See:
https://nextjs.org/docs/app/building-your-application/rendering/server-components
https://nextjs.org/docs/app/building-your-application/rendering/client-components


Suggestion for the documentation

Hey guys, can I suggest adding a "how to use it" example to the documentation? Specifically the need to include the method in the link stumped me for a bit, hehe. :p Might help someone else to have it noted explicitly. Thanks for the great gem!

I've added some more info. What do you think?

Closing for now :)


inprove view of code example of nginx proxy

indent all code in example, only some was before
remove double code end

reason: the code on https://rancher.com/docs/rancher/v2.x/en/installation/k8s-install/create-nodes-lb/nginx/ looks strange.


Add theme: Better UniExtBtn

Add theme: Better UniExtBtn
This PR adds a new theme to the theme library.
Theme Details

Name: Better UniExtBtn
Description: change unified extensions button to zen browser icon
Homepage:
Author: @lindongbin

new image

new image

Thanks!


Which version of InternVL does  TensorRT-llm 1.5 support ?

From TensorRT-LLM code

The image preprocess is internvl 1.2 version.
Form InternVL 1.5/2.0 the image preprocess  like this:

So, does TensorRT-llm 1.5(the lastest version) support InternVL 2.0 model ?

Hi @spacegrass the latest code already declared the internvl2 supporting. Please refer to
https://github.com/NVIDIA/TensorRT-LLM/tree/main/examples/multimodal#internvl2

Hi @spacegrass the latest code already declared the internvl2 supporting. Please refer to
https://github.com/NVIDIA/TensorRT-LLM/tree/main/examples/multimodal#internvl2

--- We have use the script to convert hf-model 2 trt-llm model, I find the image process code in the tensorrt-llm main & 0.15.0 branch is the InternVL 1.2 version. the processed image shape is [1, 3, 448, 448], bute the InternVL 2.0 add dynamic split so the default shape is [3, 3, 448, 448]


Laravel 5.6

Hi,
Great work!  How can I integrate this into Laravel framework ?
Regards

This project is a pure front-end project, and Laravel is also very friendly to vue. The community already has a lot of complete solutions, so I don’t think there is any big problem with integration.

Maybe element-in-laravel-starter this will help you


When entering the function, it gives an error and the bot closes

Tell me what I'm doing wrong, I seem to have installed all the libraries, Google has the latest version.

Hello. A new update was just pushed. Please check it and let me know if everything's working good.


Canvas Optimizations

Offscreen Canvas
Layering

https://developer.mozilla.org/en-US/docs/Web/API/Canvas_API/Tutorial/Optimizing_canvas

Benchmarking

RequestAnimationFrame
Limit redraws

https://developer.ibm.com/tutorials/wa-canvashtml5layering/

https://web.dev/canvas-performance/
https://gist.github.com/jaredwilli/5469626
https://developer.chrome.com/blog/offscreen-canvas/

Dragging and drawing can be cleaner + more efficient with multiple canvas.

Cache merged text scene when dragging.


Assigning a Node to Array[Node] in the editor has no effect.

Godot version
4.0.dev (3ae61590e1)
System information
Arch Linux, Vulkan, RX 550
Issue description
When using the following:
@export var links_to: Array[Node] = []

...and attempting to assign a node on first array item, dialog shows, but after selecting the node it does not actually assign anything (no effect).
Main workaround is to keep using arrays of typeArray[NodePath], however I have personally decided to hard code a list of links as variables, since I only need 3 at most.
@export var links_to_node_1: Node3D
@export var links_to_node_2: Node3D
@export var links_to_node_3: Node3D

Steps to reproduce
This script on any Node:
extends Node

@export var links_to: Array[Node] = []

Minimal reproduction project
No response

Duplicate of #62916


Bump harness and bouncycastle versions

Co-Authored-By: Philip Stutz<EMAIL_ADDRESS>

Will be part of a follow-up PR.


CDPS-946: Using new generic-service chart to perform preprod refresh

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 78.51%. Comparing base (aaa3205) to head (44e3dc3).

Additional details and impacted files
@@            Coverage Diff            @@
##               main      #93   +/-   ##
=========================================
  Coverage     78.51%   78.51%
  Complexity      560      560
=========================================
  Files            93       93
  Lines          1801     1801
  Branches        160      160
=========================================
  Hits           1414     1414
  Misses          295      295
  Partials         92       92

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Support GPU arrays as input to inplace_predict when using gpu_predictor

I'd like to use the gpu_predictor to use the GPU for predicting. My current understanding is that using predict on a DMatrix will return the result in CPU memory, causing a potentially unnecessary device to host transfer. inplace_predict can accept a numpy array as input when using the CPU predictor, but cannot accept either a CPU or GPU array as input when using the gpu_predictor.
import xgboost as xgb
import cupy as cp

X = cp.arange(100).reshape(50, 2)
y = cp.arange(50)

dmat = xgb.DMatrix(X, y)
bst = xgb.train({"tree_method": "gpu_hist"}, dmat, 100)
bst.set_param({'predictor': 'gpu_predictor'})
type(bst.predict(dmat)) # numpy array
bst.inplace_predict(X)
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
<ipython-input-19-bc4e13033d42> in <module>
      9 bst.set_param({'predictor': 'gpu_predictor'})
     10 type(bst.predict(dmat))
---> 11 bst.inplace_predict(X)

/raid/nicholasb/miniconda3/envs/rapids-0.20/lib/python3.8/site-packages/xgboost/core.py in inplace_predict(self, data, iteration_range, predict_type, missing, validate_features, base_margin, strict_shape)
   1909             return _prediction_output(shape, dims, preds, True)
   1910
-> 1911         raise TypeError(
   1912             "Data type:" + str(type(data)) + " not supported by inplace prediction."
   1913         )

TypeError: Data type:<class 'cupy._core.core.ndarray'> not supported by inplace prediction.

conda list | grep "xgboost\|cupy\|numpy"
cupy                      9.0.0            py38hc350bd8_1    conda-forge
libxgboost                1.4.0dev.rapidsai0.20      cuda11.0_0    rapidsai-nightly
numpy                     1.20.2           py38h9894fe3_0    conda-forge
py-xgboost                1.4.0dev.rapidsai0.20  cuda11.0py38_0    rapidsai-nightly
rapids-xgboost            0.20.0a210507   cuda11.0_py38_g24174d4_329    rapidsai-nightly
xgboost                   1.4.0dev.rapidsai0.20  cuda11.0py38_0    rapidsai-nightly

@beckernick Failure is not expected, since we have code specifically to handle cuPy arrays in inplace_predict():
https://github.com/dmlc/xgboost/blob/2a9979e256bbead2f364ac8698a890ba9a1e004a/python-package/xgboost/core.py#L1945-L1966
We will investigate.

Thanks Philip! I'm happy to help triage as well

It's fixed in latest 1.4 branch.  This is caused by updated cupy.

@trivialfis Ah didn't notice it was a new addition after the 1.4.0 release. So cuPy 9.0.0 isn't supported by XGBoost 1.4.0 (+ cuML 0.19)?

No.  But 1.4.2 should be fine.

Perfect, thanks all!