update MaterialCommunityIcons to 5.3.45

Supersedes https://github.com/oblador/react-native-vector-icons/pull/1137 and https://github.com/oblador/react-native-vector-icons/pull/1115
Credits to @connectdotz for the fromCharCode -> fromCodePoint tweak as per https://github.com/oblador/react-native-vector-icons/pull/1115#issue-362356120

@oblador: hopefully you can take a look at this, the icons were very old and a lot of them were missing.

@oblador This seems to be working properly. could you please take a look? There are several new and update icons, leading to incorrect icon rendering (or no rendering at all) when referencing from readme instructions.


Avoid allocation when matching hosts for vol lookup

This avoids a malloc, free, memcpy and a scan for end of string (for string_view construction).
This call to CacheHostMatcher::Match could happen multiple times per request when a hosting.config is used.

[approve ci autest]

Cherry-picked to v9.2.x


PR on OMNT seems to be gone

Hello CC Team,
there was a open PR from you (RkShaRkz) which seems to be gone from github now. As we still want to merge it / figure out what caused the issue the PR fixed i would like to ask if you could open it again. Would love to keep the codebase as similar as possible to make colaboration possible. We sadly still did not find a way to reproduce the issue, if you have any hints that would be very welcome.
regards Peter

I think i found most of the old PR an merged it into the https://github.com/omnt/OpenMobileNetworkToolkit/tree/rel/0.5 feel free to base further PRs on that branch until the 0.5 release

Hi - his account got suspended somehow, but we will try to sort the issue out asap. Thanks a lot for everything.


Add libudev, which Ledger Live app requires

Motivation for this change
Ledger Live app didn't work
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Fits CONTRIBUTING.md.

cc @tilpner


pfr_assemblyqc: please use the latest Nextflow version on pP

Description of the bug
For pfr users:
The nextflow version needs to be updated in the  pfr_assemblyqc, Otherwise, the following error may happen:

Failed requirement - Plugin<EMAIL_ADDRESS>requires Nextflow version >=23.10.0 (current 23.04.4)

Command used and terminal output
No response
Relevant files
No response
System information
No response

Can you please share your launch command? Which pipeline version are you using? The latest version of the pipeline (https://github.com/Plant-Food-Research-Open/assemblyqc/releases/tag/2.2.0) uses Nextflow 24.04.3 as can be seen in the pfr_assemblyqc file,
https://github.com/Plant-Food-Research-Open/assemblyqc/blob/01419251d52a057caa8fda571704ddc468be10d3/pfr_assemblyqc#L30
Is it possible that you have tried to use the old launch script with the new version of the pipeline?


In the finished_quiz function, rename the variables to help understand their purpose better.

In order to reduce ambiguity:

response has been refactored to participant_response
Additional comment for the usage of reponse is added, clarifying the previous comment asking about its usage.
questions has been refactored to quiz_questions

Rest of the variables are currently understandable enough to show their purposes.


Tooltip with appendToBody is never removed

If I create chart with tooltip plugin without appendToBody it doesn't show where my mouse is but somewhere else.
If I create chart with appendToBody it just adds the tooltips to the body on each new chartist and is not removed when chartist is removed / detached. Here I always had only two charts, but they were detached / recreated on the same page multiple times:

Do I miss some options or do not detach chart / tooltip correctly?

PR #160 once merged should fix this


chore(devenv): remove Nix Flake in favor of devenv-based environment

omnigres on  devenv via 🐳 orbstack via △ v3.26.4 via ❄  impure (devenv-shell) took 1m59s
❯ pg
psql (15.4)
Type "help" for help.

omnigres=> \c
You are now connected to database "omnigres" as user "omnigres".
omnigres=> \pset pager off
Pager usage is off.
omnigres=> select name, default_version from pg_available_extensions;
             name             | default_version
------------------------------+-----------------
 refint                       | 1.0
 postgis                      | 3.3.3
 unaccent                     | 1.1
 btree_gin                    | 1.3
 ltree                        | 1.2
 tsm_system_rows              | 1.0
 datasketches                 | 1.6.0
 temporal_tables              | 1.2.0
 plr                          | 8.4.6
 jsonb_deep_sum               | 0.0.2
 plpgsql_check                | 2.4
 adminpack                    | 2.1
 dict_xsyn                    | 1.0
 pg_repack                    | 1.4.8
 address_standardizer         | 3.3.3
 xml2                         | 1.1
 hstore                       | 1.8
 pg_visibility                | 1.2
 tsearch_extras               | 1.0
 cube                         | 1.5
 postgis_tiger_geocoder       | 3.3.3
 seg                          | 1.4
 hll                          | 2.16
 intagg                       | 1.1
 tcn                          | 1.0
 isn                          | 1.2
 tsm_system_time              | 1.0
 pgroonga_database            | 3.0.8
 lo                           | 1.1
 pgrowlocks                   | 1.2
 pg_bigm                      | 1.2
 pgroonga                     | 3.0.8
 pg_hint_plan                 | 1.4
 sslinfo                      | 1.2
 topn                         | 2.5.0
 pgstattuple                  | 1.5
 autoinc                      | 1.0
 address_standardizer_data_us | 3.3.3
 postgis_topology             | 3.3.3
 postgis_raster               | 3.3.3
 pg_freespacemap              | 1.2
 pg_net                       | 0.7.1
 pg_relusage                  | 0.0.1
 file_fdw                     | 1.0
 pg_surgery                   | 1.0
 pg_buffercache               | 1.3
 dblink                       | 1.2
 pg_stat_statements           | 1.10
 insert_username              | 1.0
 repmgr                       | 5.4
 pg_prewarm                   | 1.2
 old_snapshot                 | 1.0
 pgtap                        | 1.2.0
 pg_ed25519                   | 0.2
 earthdistance                | 1.1
 uuid-ossp                    | 1.1
 intarray                     | 1.5
 periods                      | 1.2
 pg_trgm                      | 1.6
 dict_int                     | 1.0
 amcheck                      | 1.3
 vector                       | 0.5.0
 btree_gist                   | 1.7
 pageinspect                  | 1.11
 rum                          | 1.3
 moddatetime                  | 1.0
 pg_rational                  | 0.0.2
 pg_ivm                       | 1.5
 fuzzystrmatch                | 1.1
 pgautofailover               | 2.0
 pgrouting                    | 3.5.0
 pg_partman                   | 4.7.3
 pgcrypto                     | 1.3
 pg_similarity                | 1.0
 pg_cron                      | 1.5
 tablefunc                    | 1.0
 pg_walinspect                | 1.0
 postgres_fdw                 | 1.1
 smlar                        | 1.0
 bloom                        | 1.0
 pgjwt                        | 0.2.0
 citext                       | 1.6
 plpgsql                      | 1.0
 pgaudit                      | 1.7
(84 rows)

Thank you so much for your contribution!
For my better understanding, what is the problem with flakes that devenv ultimately solves, esp. in this scenario?
I am trying to understand if this contribution is intended to help "developing Omnigres" or "developing with Omnigres"?

Thank you for such a detailed explanation of your vision! I am going ahead and merging this; let's see how it will evolve over time :)


HTTP: 403 Forbidden

Anyone know or remember why the HTTP: 403 Forbidden is showing up on screen for Twitter Search, even though the browser search is fine while executing this bash script.
Parameters used for headers XMLHttpRequest
headers = [
			('Host', "twitter.com"),
			('User-Agent', "Mozilla/5.0 (Windows NT 6.1; Win64; x64)"),
			('Accept', "application/json, text/javascript, */*; q=0.01"),
			('Accept-Language', "de,en-US;q=0.7,en;q=0.3"),
			('X-Requested-With', "XMLHttpRequest"),
			('Referer', url),
			('Connection', "keep-alive")
		]

python Exporter.py --querysearch "europe refugees" --maxtweets 1 --output new.csv

python Exporter.py --querysearch "europe refugees" --maxtweets 1 --output new.csv
Searching...

Twitter weird response. Try to see on browser:  https://twitter.com/search?f=tweets&q=%20europe%20refugees&src=typd&max_position=
Twitter weird response. Try to see on browser: https://twitter.com/search?q=%20europe%20refugees&src=typd
Unexpected error: <class 'urllib.error.HTTPError'>
Done. Output file generated "new.csv".
Traceback (most recent call last):
  File "C:\Users\burse\Documents\Project-Documentation\Perceived-Stress-Questionnaire\GetOldTweets-python\got3\manager\TweetManager.py", line 130, in getJsonReponse
    response = opener.open(url)
  File "C:\Users\burse\anaconda3\envs\meshcnn\lib\urllib\request.py", line 532, in open
    response = meth(req, response)
  File "C:\Users\burse\anaconda3\envs\meshcnn\lib\urllib\request.py", line 642, in http_response
    'http', request, response, code, msg, hdrs)
  File "C:\Users\burse\anaconda3\envs\meshcnn\lib\urllib\request.py", line 570, in error
    return self._call_chain(*args)
  File "C:\Users\burse\anaconda3\envs\meshcnn\lib\urllib\request.py", line 504, in _call_chain
    result = func(*args)
  File "C:\Users\burse\anaconda3\envs\meshcnn\lib\urllib\request.py", line 650, in http_error_default
    raise HTTPError(req.full_url, code, msg, hdrs, fp)
urllib.error.HTTPError: HTTP Error 403: Forbidden

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "Exporter.py", line 70, in main
    got.manager.TweetManager.getTweets(tweetCriteria, receiveBuffer)
  File "C:\Users\burse\Documents\Project-Documentation\Perceived-Stress-Questionnaire\GetOldTweets-python\got3\manager\TweetManager.py", line 21, in getTweets
    json = TweetManager.getJsonReponse(tweetCriteria, refreshCursor, cookieJar, proxy)
  File "C:\Users\burse\Documents\Project-Documentation\Perceived-Stress-Questionnaire\GetOldTweets-python\got3\manager\TweetManager.py", line 136, in getJsonReponse
    sys.exit()
SystemExit

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "Exporter.py", line 79, in <module>
    main(sys.argv[1:])
  File "Exporter.py", line 72, in main
    except arg:
TypeError: catching classes that do not inherit from BaseException is not allowed

I am also getting this error for accessing a user profile with the code:
tweetCriteria = got.manager.TweetCriteria().setUsername('barackobama').setMaxTweets(1)
tweets = got.manager.TweetManager.getTweets(tweetCriteria)

Error is:
An error occured during an HTTP request: HTTP Error 403: Forbidden
Try to open in browser: https://twitter.com/search?q=%20from%3Abarackobama&src=typd

Has twitter updated its scraping detection? Please help


Query object binding

@vpetrusevici check out this commit
the following works now.

// ?id=1&name=john&age=32

public class Request
{
    public int Id { get; set; }

    [FromQueryParams]
    public Person Person { get; set; }
}


AWS Storage - protection level changes the key used to retrieve our image from S3

The AWS Storage SDK allows us to download resources from our S3 Bucket.
The problem our company has is that the Storage SDK requires a protection level ('private', 'protected', 'public') and a key for retreiving a resource. When setting the protection level, the protection level itself gets prefixed to the key of the resource we're looking for in our S3 Bucket.
e.g. key: friends/hamster.png, protection level: public
what the SDK does to our Key: public/friends/hamster.png
This causes following Response from AWS
<?xml version="1.0" encoding="UTF-8"?> <Error>NoSuchKey<Message>The specified key does not exist.</Message><Key>public/friends/hamster.png</Key><RequestId>xxx</RequestId><HostId>xxx</HostId></Error>
(xxx -> censored)
After modifying the Source Code in Storage.js to offer a case to that returns no prefix if no protection level is set, the error response vanished and we're receiving now correctly the picture of our hamster.
Is it a wanted behavior that a resource in a S3 Bucket always needs to be stored in an either public, private/<..>/ or protected/<..> folder? If not, I would like to offer a pull request. If yes, I'd be really thankful to hear how to solve our problem.
If I'm completely wrong and there is another way to access a S3 Ressource without having to specify a protection level, I'd also be very thankful to hear about this approach.
Thank you a lot

Hi can you check this doc if it could resolve your problem? https://aws.github.io/aws-amplify/media/storage_guide#customize-upload-path

Hello, thank you for your response. You are right, this should solve our problem. I just noticed that we're still using AWS-Amplify Vers. 0.2.9, where this problem occurs and customPrefixes are not supported.  Guess it's time to update :)


invoke() failed in magic_wand(modified)

Based on the magic_wand project, I replaced the model trained by myself.
The model has 5 labels, and I modified the labels[label_count] of the source file. And I tried to modify kTensorArenaSize. But in the serial monitor, it shows
...............###.........#....
.............##............#....
............#..............#....
...........#...............#....
.........##...............#.....
........#.................#.....
.......#.................#......
......#..................#......
.....#..................#.......
....#...................#.......
..##...................#........
.#.....................#........
#.....................#.........
#................#####..........
#..............######.#.........
..............#.....###.........
...................#............
.............#....###...........
.............#...###............
............#.#.###.............
...........#..####..............
#...........###...##............
#.....#.###########.............
.#.###..........................
................................
................................
................................
................................
................................
................................
................................
................................
MicroAllocator: Model allocation started before finishing previously allocated model
Failed starting model allocation.

Invoke failed
................................
................................
................................
................................
..............#####.............
............##.....#............
...........#........#...........
..........#..........#..........
.........#...........#..........
.........#...........#..........
........#............#..........
........#............#..........
.....................#..........
.......#............##..........
.......#...........#............
.......#..........##..#.........
.........##.....##....#.........
........##.#...#.....#..........
..........#####......#..........
...........####.....#...........
...............#....##..........
...............#####............
................................
................................
................................
................................
................................
................................
................................
................................
................................
................................
Invoke() called after initialization failed

Invoke failed

How can I fix it?
Here is my src file.
https://github.com/Steel-Shadow/FengRu
Thanks for any reply！

I just found out that my teammates got the wrong tutorial for training the model, they used the following training tutorial.
tflite-micro/tensorflow/lite/micro/examples/magic_wand
The two repo is different.


Make agents Norm aware

When we add norms, we should also take care that we change the agent profiles  so that they have an opinion on sanctions, i.e. they incorporate sanctions in their utility function.

This is now done in a very rudimentary way, that is: HighFine = -0.1 utility, LowFine = -1 utility. This is of course ludicrious, but allowed us to quickly see if the rest worked. Now we need to make this proper.


Shorter way to select a container in the CLI

--impersonated-container-name is quite a mouthful with no short flags.
Instead of introducing a short flag for it, I'd suggest an alternative for --pod-name.
Since #293 is planned, something more general could work as a target selector.
Some ideas:
--target pod/foo-asd1234/container-name

For deployments:
--target deployment/foo

Or in case services are supported in the future:
--target svc/foo

That's an interesting approach. We will start thinking about this after the MVP stable release!


FIX ALERT DIALOG

PR https://github.com/ruby-ui/ruby_ui/pull/178 broke ALERT_DIALOG

closed in favor of https://github.com/ruby-ui/ruby_ui/pull/184


cosign: 1.7.1 -> 1.7.2

Description of changes
https://github.com/sigstore/cosign/releases/tag/v1.7.2
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[x] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review run on aarch64-darwin
1 package built:
cosign

:wave: hi @bradbeck
If you fancy adding yourself to the maintainers/maintainers-list.nix file & as a maintainer on cosign I'll close #168379 in favor of this PR :smiley:

👋🏼 Hi @06kellyjac
Sorry should have looked before opening the PR. It was worth going through the exercise though. Maybe I can jump on the maintainers next time 😉
Cheers.


View Logs needs design & implementation

Steps:

In Jump drop-down enter Pod
Go to a Pod's menu and select View Logs

Results:

Request:
Need design first then dev to implement
Example of old Rancher log modal:

White box has been implemented but the top button, filter button and the default tab is still bleeding through from the main list page

Fixed


[HOLD for payment 2023-06-21] [$1000] Chat - Page loads infinitely instead of error page when accessing invalid link

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:
Scenario 1:

Copy IOU link which the logged-in user does not have access to.
Paste it on a new tab.

Scenario 2:

Go to https://staging.new.expensify.com/
Copy a chat link.
Paste it on the browser.

Scenario 3:

Go to invalid chat link. For example, https://staging.new.expensify.com/r/1234.

Expected Result:
Scenario 1: User lands on error page because they do not have access to the link.
Scenario 2: User is able to open the chat link.
Scenario 3: User lands on error page because they do not have access to the link.
Actual Result:
Scenario 1: User lands on the page which loads infinitely instead of error page.
Scenario 2: User is able to open the chat link.
Scenario 3: User lands on the page which loads infinitely instead of error page.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/78819774/47761cae-5243-48e1-9c3c-56571c49b35a
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team / @adeel0202
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1686035561587859
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0164184e82be52ad8b
Upwork Job ID:<PHONE_NUMBER>563983360
Last Price Increase: 2023-06-07

@aldo-expensify Can you please assign me this issue as it is a regression indeed . we should assign @dukenv0307 as well.

For some reason @dukenv0307 doesn't show up in the list to assign

Merged the fix, thanks @dukenv0307 and @fedirjh !

Going to take this from April as it's a regression from my https://github.com/Expensify/App/issues/19364

Paid this out as a bonus to the linked https://github.com/Expensify/App/issues/19364

Reopening since there seem to be a pending payment

Sent the offer for this!

Paid.


V0.4 Overlapping Event Notification for Edit and Restore Commands

Part of #158

Coverage decreased (-0.4%) to 79.461% when pulling 24c835deee54a56f25a4d4d3077e896d54559be7 on V0.4_overlapping-task-notification into 769de2777b7d88a120cc62d5612c069bff40fb27 on V0.4.

Coverage decreased (-0.4%) to 79.461% when pulling 24c835deee54a56f25a4d4d3077e896d54559be7 on V0.4_overlapping-task-notification into 769de2777b7d88a120cc62d5612c069bff40fb27 on V0.4.


[Documentation] Examples code from scratch or implementation code

The stories code is not very helpful and going to the code in the main repo doesn't help either, I have some doubts about how to implement a kanban like Dnd example and the stories examples are great but I cannot fully grasp the concepts from neither from the doc option in story nor the repo code.

I completely agree. When I want to implement something like the "Sortable Multiple Containers Basic Setup" and look at the code, the only thing I see is
() => <MultipleContainers />

https://master--5fc05e08a4a65d0021ae0bf2.chromatic.com/?path=/docs/presets-sortable-multiple-containers--basic-setup
What am I missing?

same here, the art of concealment?

Yes, I agree on that.
I am trying to debug something, and it's really difficult to get a codesandbox.io example going, when the components and elements are distributed over several places, and have properties which are of no use to the current story.
I am fine with the stories, I also use storybook. But I try to keep the stories mainly self sufficient. That also enables the show code option in the docs part as well.

The docs here are probably the least helpful docs I've ever seen.

@Jiert The problem is that the storybook is just a cupple of massive components that depending on the example changes the props for the setup, or at least that's my understanding by checking the storybook code. The only option available is to navigate the examples folder try to understand the implementation navigating the code :v

The library looks like it produces beautiful output, but the amount of code necessary for the most simple sortable list is kind of astounding. I look at the Sandbox example and am overcome with feelings of "Is there any way I can avoid drag and drop entirely".

I have created a really simple repo for a sortable vertical list, which is what I was going for initially. They actually have decent sample code down in Sortable: https://docs.dndkit.com/presets/sortable
But if you try to run it, there's no styling so nothing will show on the screen. Confusing! I added some simple TailwindCSS classes to make it show up and look obvious. Hopefully this is useful to other people like me who just wanted something dead simple, but instead found sample projects in the Code Sandbox that contained many files and confusing to read.

I have created a really simple repo for a sortable vertical list, which is what I was going for initially. They actually have decent sample code down in Sortable: https://docs.dndkit.com/presets/sortable
But if you try to run it, there's no styling so nothing will show on the screen. Confusing! I added some simple TailwindCSS classes to make it show up and look obvious. Hopefully this is useful to other people like me who just wanted something dead simple, but instead found sample projects in the Code Sandbox that contained many files and confusing to read.

This is for single sortable list. Could you also try to setup a Kanban?

I completely agree. When I want to implement something like the "Sortable Multiple Containers Basic Setup" and look at the code, the only thing I see is
() => <MultipleContainers />

https://master--5fc05e08a4a65d0021ae0bf2.chromatic.com/?path=/docs/presets-sortable-multiple-containers--basic-setup
What am I missing?

Here is a link to the Component being used in the storybook.
https://github.com/clauderic/dnd-kit/blob/master/stories/2 - Presets/Sortable/MultipleContainers.tsx

+1 Would also love to see simple sandboxes, much easier to learn / reproduce than trying to dig through the repo

This library has one of the worst documentation I've ever seen. So many things are hidden and not documented.


BLIP2 fine-tuning on custom LLM+dataset

Hi!
I want to extend BLIP2 capabilities to another language. I have a pre-trained LLM (T5 family) and a dataset with image captions. Could you please help me understand my next steps to train the model? Do I need to perform pre-training, or can I use pre-trained ViT along with my T5 model and do fine-tuning?

Hey,
Looking to do a similar project. Did you ever found an answer to your question? Would love to get some clarification.

嘿，
想做一个类似的项目。您找到问题的答案了吗？希望得到一些澄清。

Hello!
I am also working on fine-tuning the image capture task using my own dataset, BLIP2. Have you succeeded

Working on the same, did anyone have any luck?

any progress?

Can you share code?


SWARM-578: Introduce FileSystemLayout abstraction

Supports the IDE:run cases by providing abstraction to resolve gradle and maven specific file systems locations.

@bobmcwhirter Here's the abstraction we talked about the other day. It's only used for the IDE:run cases.

Would it make sense to move the FS layout classes from /bootstrap to /container?
there's no guarantee that the bootstrap module will be loaded in anything other than an uber jar situation, as container module only has an optional dep on bootstrap module. And every fraction definitely has a dependency on the container module.

Container does have a maven dep on bootstrap so it should transitive
around.
The module dep is optional only because the bootstrap module doesn't have a
real module.xml and that freaks out the fraction plugin. It's always
available.
--
Sent from my iPhone
On Aug 4, 2016, at 5:56 AM, Ken Finnigan<EMAIL_ADDRESS>wrote:
Would it make sense to move the FS layout classes from /bootstrap to
/container?
there's no guarantee that the bootstrap module will be loaded in anything
other than an uber jar situation, as container module only has an optional
dep on bootstrap module. And every fraction definitely has a dependency on
the container module.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/wildfly-swarm/wildfly-swarm/pull/60#issuecomment-237508404,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAA-T6BXivU790v5RFnhvefMCS7cGTiyks5qcbdngaJpZM4Jcbyx
.

I can't really remember why I moved it there.But I agree, /container seems to be more sensible.

retest this please

Comments from Reviewable


Address Dev review 2/2

Changes

[x] Addressed more of the comments done in #68. Please checks commits for each change.

Codecov Report

Merging #79 (e49d0c9) into main (d212c06) will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##            main     #79   +/-   ##
=====================================
  Coverage   5.97%   5.97%
=====================================
  Files          5       5
  Lines        184     184
=====================================
  Hits          11      11
  Misses       169     169
  Partials       4       4

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d212c06...e49d0c9. Read the comment docs.

Hi! Any news on this?

Codecov Report

Merging #79 (65f887f) into main (6dc67b6) will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##            main     #79   +/-   ##
=====================================
  Coverage   5.97%   5.97%
=====================================
  Files          5       5
  Lines        184     184
=====================================
  Hits          11      11
  Misses       169     169
  Partials       4       4

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6dc67b6...65f887f. Read the comment docs.

@DHaussermann Are you find testing this as part of the release process ?


Support BTreeIterator going backwards

BTreeIterator is iterating over b-tree. So it would be handy to have .prev() method which would go backwards on the tree.

We can contribute implementation, just need confirmation this is a welcome change to the ParityDb API :slightly_smiling_face:
We found ParityDb is a good fit for some other blockchain-related use cases outside of the state database, but reverse iteration is hard requirement there.

No objections if you want to make PR.
It should be noted however that BTree columns are not well optimized for storing a lot of data. These were introduced as a temporary solution to store some additional small values in polkadot. Hash indexes should be used whenever possible.

We'll store 8 to 32 byte values, so should be fine

PR: https://github.com/paritytech/parity-db/issues/72


Add ability to test payload functions without setting them

I think it would make sense to be able to test payload functions before actually committing them to
the handler.  This makes testing a bit safer and one can do it while the handler is still processing stuff using the old functions (I do not know if this is an existing usecase).
So instead of
write --> set --> test --+
  ^                      |
  |                      |
  +----------------------+

We can have
write --> test -- (ok) --> set
   ^       |
   |       |
   +-------+

We could pass the functions to be tested as separate field in TestPayloadFunctionsReq.
Are there reasons not to do this? We could also have both flows at the same time (eg. TestPayloadFunctions(appEUI, Payload) and TestPayloadFunctionsOffline(appEUI, Payload, Functions)).
I've been looking at the handler code and can make a PR if need be.

Yes, good idea. I don't see a real use in having both. Let's keep it plain and simple. Make a PR, otherwise assign this issue to me


unauthorized transaction

there is an unauthorized transaction from my wallet. olease check
f0b99a182fb4c4f860358f1be94fe3174e7b14e79dbc068b75282c3588f199aa

@rafeek88 Talk to Cardano Support on the Live support page so that your issue can be looked into.

Don't do that! BarrDev is a scammer. The live support page (forum.cardaNNo.com) is a scam. In order to get help you must put ADA's in your account. Yeah right. They will disappear to their account.


Update compelling example in README.md for v6.x

No one true C# style anymore? :)

No one true C# style anymore? :)

Alas, I have not been instructed in the ways of the One True C# style.
I've updated the pull request to fix the issues. I admit that there are several changes that involve the Color class that make this less readable (i.e. .FromArgb and x.G), but now you can copy and paste this example into a .cs file and it will compile.

I think it looks good now. I leave it to @ghuntley to merge as he assigned it to himself.


The output of "istioctl ps" cmd show the cluster(k8s) name which a pod belongs

Please provide a description of this PR:
fix #36290

Hi @miaoyq. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

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

/ok-to-test

@miaoyq: Cannot trigger testing until a trusted user reviews the PR and leaves an /ok-to-test message.

In response to this:

/ok-to-test

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

/ok-to-test

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

Test name
Commit
Details
Required
Rerun command

release-notes_istio
ed4c589d3739f003c71d7610deee165399ae6c62
link
true
/test release-notes_istio

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

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

Test name
Commit
Details
Required
Rerun command

release-notes_istio
ed4c589d3739f003c71d7610deee165399ae6c62
link
true
/test release-notes_istio

unit-tests_istio
ed4c589d3739f003c71d7610deee165399ae6c62
link
true
/test unit-tests_istio

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

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

Test name
Commit
Details
Required
Rerun command

release-notes_istio
ed4c589d3739f003c71d7610deee165399ae6c62
link
true
/test release-notes_istio

unit-tests_istio
ed4c589d3739f003c71d7610deee165399ae6c62
link
true
/test unit-tests_istio

integ-security-k8s-tests_istio
ed4c589d3739f003c71d7610deee165399ae6c62
link
true
/test integ-security-k8s-tests_istio

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

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

Test name
Commit
Details
Required
Rerun command

unit-tests_istio
ed4c589d3739f003c71d7610deee165399ae6c62
link
true
/test unit-tests_istio

integ-security-k8s-tests_istio
ed4c589d3739f003c71d7610deee165399ae6c62
link
true
/test integ-security-k8s-tests_istio

release-notes_istio
0307c8406f36f8de76093cbf396fdf7ede6b71dc
link
true
/test release-notes_istio

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

/test release-notes_istio

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

Test name
Commit
Details
Required
Rerun command

release-notes_istio
0307c8406f36f8de76093cbf396fdf7ede6b71dc
link
true
/test release-notes_istio

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

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

Test name
Commit
Details
Required
Rerun command

release-notes_istio
1d962c435a92fe12f2ba6db4fbdc21dbca3c9b10
link
true
/test release-notes_istio

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

@hzxuzhonghu Thanks for review :-).

/cc @istio/wg-user-experience-maintainers
istio/wg-user-experience-maintainers

@howardjohn Could you take a look the PR  :-)

cc  @therealmitchconnors  :-)

@esnible Could you please take a look at the PR, Thx :-)

Looking at istio.io, I don't see any of the places where istioctl ps is used with example output that is tested. Thus, I don't think any istio.io tests will fail once this is merged, but several pages will have old output without the CLUSTER name.

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

Test name
Commit
Details
Required
Rerun command

unit-tests_istio
a0a5e30ed7cc881879afe859c8bace201517adda
link
unknown
/test unit-tests_istio

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

/test unit-tests_istio

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

Test name
Commit
Details
Required
Rerun command

unit-tests_istio
a0a5e30ed7cc881879afe859c8bace201517adda
link
true
/test unit-tests_istio

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

/test unit-tests_istio


boards: arm: nxp: mimxrt1024_evk: enable dma driver

Enable the dma driver on the mimxrt1024_evk board.
Tested with:

tests/drivers/dma/chan_blen_transfer
tests/drivers/dma/chan_link_transfer
tests/drivers/dma/loop_transfer

Signed-off-by: Henrik Brix Andersen<EMAIL_ADDRESS>

@henrikbrixandersen, can you address the conflicts?

@henrikbrixandersen, can you address the conflicts?

@dleach02 Done.


[SPARK-15762][SQL] Cache Metadata & StructType hashCodes; use singleton Metadata.empty

We should cache Metadata.hashCode and use a singleton for Metadata.empty because calculating metadata hashCodes appears to be a bottleneck for certain workloads.
We should also cache StructType.hashCode.
In an optimizer stress-test benchmark run by @ericl, these hashCode calls accounted for roughly 40% of the total CPU time and this bottleneck was completely eliminated by the caching added by this patch.

Test build #59971 has started for PR 13504 at commit 8061600.

Test build #59974 has started for PR 13504 at commit 92c6c69.

Can you put some data about performance improvements on the pr description?

Test build #59971 has finished for PR 13504 at commit 8061600.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/59971/
Test PASSed.

Merged build finished. Test PASSed.

Test build #59981 has started for PR 13504 at commit a3a6898.

Test build #59974 has finished for PR 13504 at commit 92c6c69.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

@rxin, I don't have the specific performance numbers handy but in an optimizer stress-test benchmark run by @ericl, these hashCode calls accounted for roughly 40% of the total CPU time and this bottleneck was completely eliminated by the caching added by this patch.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/59974/
Test PASSed.

hash code profile

Test build #59981 has finished for PR 13504 at commit a3a6898.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/59981/
Test PASSed.

LGTM other than that small comment.

Merging to master and 2.0.


nixos/redmine: add database.createLocally option

Motivation for this change
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

@GrahamcOfBorg test redmine.v3-mysql redmine.v4-mysql redmine.v4-pgsql

machine# [  502.934336] systemd[1]: redmine.service: Start-pre operation timed out. Terminating.

:sad:

@GrahamcOfBorg test redmine.v4-mysql

This pull request has been mentioned on Nix community. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review-may-2019/3032/25

Thanks @Infinisil!


Allow some mobs to walk slowly

About the PR

for a lot of mobs that have walkspeed the same as sprintspeed i reduced walkspeed
for details check files changed
Why / Balance

i want to walk
Technical details

Media

Requirements

[X] I have read and I am following the Pull Request Guidelines. I understand that not doing so may get my pr closed at maintainer’s discretion
[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

:cl:

tweak: Reduced walk speed of some small mobs (mice, cockroaches, bees etc.)

can you let other species like mothroaches and cockroaches and stuff as well if they don't have it?

can you let other species like mothroaches and cockroaches and stuff as well if they don't have it?

yep, added

Do they still trigger mousetraps when walking? See #30593

Do they still trigger mousetraps when walking? See #30593

yes they do

Then it got my approval.

Honestly, it should be bumped down so they don't. It's a fair tradeoff to be super slow or risk the snap. But a sneaky mouse should be able to avoid the traps

Honestly, it should be bumped down so they don't. It's a fair tradeoff to be super slow or risk the snap. But a sneaky mouse should be able to avoid the traps

well... i think it's out of scope of this pr

Honestly, it should be bumped down so they don't. It's a fair tradeoff to be super slow or risk the snap. But a sneaky mouse should be able to avoid the traps

well... i think it's out of scope of this pr

It's really just a matter of making it like 1.9 instead of 2.5, but your PR your call 🤷♂Just my suggestion.

бля чё это за говно ебаное


docs: note about usage with NestJS for lifecycle subscriber

Added a note since other variants didn't work in NestJS
Ref: https://github.com/mikro-orm/nestjs/issues/14

@B4nan hey, I found that this whole thing with constructor is needed only if there are more than 1 subscriber classes with the same name. In other cases it's enough to just
// --- module.ts ---

@Module({
  providers: [SomeSubscriber]
})
export class SomeModule { /* ... */ }

// --- subscriber.ts ---
@Subscriber()
export class SomeSubscriber implements EventSubscriber { /* ... */ }


Update workflows

Main improvements:

Manage GitHub actions updates with dependabot
Update all actions to latest versions
Merge GoReleaser workflows into one
Add groups to release changelog (goreleaser config)
Add workflow to cleanup PR caches
Use go with version from go.mod file

@bonddim please fix DCO issue.

@bonddim awesome work. thanks so much.

@mumoshu WDYT?


Another Question

So it works after using the example, but unlike other libs, when you use a command the log doesnt pops up which says something like "User used /hello" thingy, is it a discord bug or do I have to enable something?

   includeSource: true
   });```

@Shelly7w7 Following what @TheCuteFoxxy posted:
For acknowledgements: ctx.acknowledge({ includeSource: true });
For messages: ctx.send('text', { includeSource: true })
For uto-acknowledgements (on soon-to-be 1.3.0 release): Set autoAcknowledgeSource: true in creator options

@Shelly7w7 Following what @TheCuteFoxxy posted:
For acknowledgements: ctx.acknowledge({ includeSource: true });
For messages: ctx.send('text', { includeSource: true })
For uto-acknowledgements (on soon-to-be 1.3.0 release): Set autoAcknowledgeSource: true in creator options


Wrong translation to Italian

Describe the bug
Wrong translation on Italian
To Reproduce
Steps to reproduce the behavior:

Go to 'SETTINGS'
Click on 'APPEARANCE'
Take a look on screenshot

Expected behavior
Just correct the language
Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

OS Version:  Windows 10 20H2 19042.685
App version: v1.0.0

Additional context
Add any other context about the problem here. Does this problem occur again after restarting the app?
Log file
Please upload the log file here so that we can understand your issue better. You can access it from Settings->About->Open log location. You can also find the log file manually by following this guide.

Ciao grazie del feedback.
Could you confirm that the app version you're using is v1.0.0?

Ciao grazie del feedback.
Could you confirm that the app version you're using is v1.0.0?

Confirmed v 1.0.0

Confirmed v 1.0.0

@squalo80 Would you like to try fixing this?

@squalo80 Would you like to try fixing this?

@yaichenbaum Yesterday I tried to check the file Files.it-IT.xlf but it is already translated there.
I can try, but my experience on coding isn't at this level.

@yaichenbaum Yesterday I tried to check the file Files.it-IT.xlf but it is already translated there.
I can try, but my experience on coding isn't at this level.

@squalo80 It's very possible that this issue was already fixed and will be included in the next update. Please reopen this issue if you can still reproduce it when v1.1 rolls out.

@squalo80 It's very possible that this issue was already fixed and will be included in the next update. Please reopen this issue if you can still reproduce it when v1.1 rolls out.


netlify构建错误

这是我第一次提交问题，如果有不对的，还请原谅
操作：直接点击了页面的netlify一键部署，自动部署时提示
具体提示如下：
1:30:20 PM: Deploy site
1:30:20 PM: ────────────────────────────────────────────────────────────────
1:30:20 PM:
1:30:20 PM:
1:30:20 PM: Configuration error
1:30:20 PM: ────────────────────────────────────────────────────────────────
1:30:20 PM:
1:30:20 PM:   Error message
1:30:20 PM:   Deploy did not succeed: Build cancelled, found file owned by root
1:30:20 PM:
1:30:20 PM:   Resolved config
1:30:20 PM:   build:
1:30:20 PM:     command: npm install
1:30:20 PM:     commandOrigin: config
1:30:20 PM:     publish: /opt/build/repo
1:30:20 PM:     publishOrigin: ui
1:30:20 PM: Build failed due to a user error: Build script returned non-zero exit code: 2
1:30:20 PM: Failing build: Failed to build site
1:30:20 PM: Failed during stage 'building site': Build script returned non-zero exit code: 2 (https://ntl.fyi/exit-code-2)

你需要提供更多信息，比如截图自己更详细的日志，netlify这个平台我目前没有去测试，只是根据文档适配了调用，所以我倒是预料到可能会出问题，不过等了几星期才有人反馈🤣

@stankyoztc 现在你可以通过下载 #19 的代码并上传至netlify尝试构建。 或者修改netlify.toml这个文件与 #19 相同再尝试

@stankyoztc 现在你可以通过下载 #19 的代码并上传至netlify尝试构建。 或者修改netlify.toml这个文件与 #19 相同再尝试

好的，现在是这样的提示建议：
Diagnosis: The build failure is due to a configuration error where the specified publish directory '/opt/build/repo/server.js' is not a directory as expected.
Solution: To resolve this issue, you need to ensure that the directory specified in the Netlify configuration file () is a valid directory. Update the setting to point to the correct directory that contains the files to be deployed. Verify that it is a directory path, not a file path like 'server.js'.
In this case, based on the build log, the directory should be set to a directory containing the built files, not to a specific file. Check the project structure and adjust the path accordingly in the configuration file.
[build]
  command = "npm install"
  publish = "public" # Specify the directory where the built site files are located

日志如下
2:40:46 PM: build-image version: ecdc8b770f4a0193xxxxxxxx681813a4 (focal)
2:40:46 PM: buildbot version: 244d5322c7eb13029c1d65f78393f886c2e07fdb
2:40:46 PM: Fetching cached dependencies
2:40:46 PM: Failed to fetch cache, continuing with build
2:40:46 PM: Starting to prepare the repo for build
2:40:47 PM: No cached dependencies found. Cloning fresh repo
2:40:47 PM: git clone --filter=blob:none https://github.com/stankyoztc/DeepLX-Serverless
2:40:47 PM: Preparing Git Reference refs/heads/main
2:40:49 PM: Starting to install dependencies
2:40:49 PM: Python version set to 3.8
2:40:49 PM: Attempting Ruby version 2.7.2, read from environment
2:40:50 PM: Using Ruby version 2.7.2
2:40:51 PM: Started restoring cached go cache
2:40:51 PM: Finished restoring cached go cache
2:40:52 PM: go version go1.19.13 linux/amd64
2:40:53 PM: Using PHP version 8.0
2:40:56 PM: v18.20.4 is already installed.
2:40:56 PM: Now using node v18.20.4 (npm v10.7.0)
2:40:56 PM: Enabling Node.js Corepack
2:40:56 PM: Started restoring cached build plugins
2:40:56 PM: Finished restoring cached build plugins
2:40:56 PM: Started restoring cached corepack dependencies
2:40:56 PM: Finished restoring cached corepack dependencies
2:40:56 PM: No npm workspaces detected
2:40:56 PM: Started restoring cached node modules
2:40:56 PM: Finished restoring cached node modules
2:40:56 PM: Installing npm packages using npm version 10.7.0
2:40:59 PM: added 99 packages, and audited 100 packages in 3s
2:40:59 PM: 21 packages are looking for funding
2:40:59 PM:   run npm fund for details
2:40:59 PM: found 0 vulnerabilities
2:40:59 PM: npm packages installed
2:41:00 PM: Successfully installed dependencies
2:41:00 PM: Starting build script
2:41:01 PM: Detected 0 framework(s)
2:41:01 PM: Section completed: initializing
2:41:03 PM:
2:41:03 PM: Netlify Build
2:41:03 PM: ────────────────────────────────────────────────────────────────
2:41:03 PM:
2:41:03 PM: ❯ Version
2:41:03 PM:   @netlify/build 29.54.3
2:41:03 PM:
2:41:03 PM: ❯ Flags
2:41:03 PM:   accountId: 66f190xxxxxxcd946ed3
2:41:03 PM:   baseRelDir: true
2:41:03 PM:   buildId: 66f501dba5424e733xxxxxx
2:41:03 PM:   deployId: 66f501dba5424e733762xxxxx
2:41:03 PM:
2:41:03 PM: ❯ Current directory
2:41:03 PM:   /opt/build/repo
2:41:03 PM:
2:41:03 PM: ❯ Config file
2:41:03 PM:   /opt/build/repo/netlify.toml
2:41:03 PM:
2:41:03 PM: ❯ Context
2:41:03 PM:   production
2:41:03 PM:
2:41:03 PM: build.command from netlify.toml
2:41:03 PM: ────────────────────────────────────────────────────────────────
2:41:03 PM:
2:41:03 PM: $ npm install
2:41:04 PM: Failed during stage 'building site': Build script returned non-zero exit code: 2 (https://ntl.fyi/exit-code-2)
2:41:04 PM: up to date, audited 100 packages in 577ms
2:41:04 PM: 21 packages are looking for funding
2:41:04 PM:   run npm fund for details
2:41:04 PM: found 0 vulnerabilities
2:41:04 PM:
2:41:04 PM: (build.command completed in 687ms)
2:41:04 PM:
2:41:04 PM: Deploy site
2:41:04 PM: ────────────────────────────────────────────────────────────────
2:41:04 PM:
2:41:04 PM: Section completed: deploying
2:41:04 PM:
2:41:04 PM: Configuration error
2:41:04 PM: ────────────────────────────────────────────────────────────────
2:41:04 PM:
2:41:04 PM:   Error message
2:41:04 PM:   Deploy did not succeed: Deploy directory 'server.js' is not a directory
2:41:04 PM:
2:41:04 PM:   Resolved config
2:41:04 PM:   build:
2:41:04 PM:     command: npm install
2:41:04 PM:     commandOrigin: config
2:41:04 PM:     publish: /opt/build/repo/server.js
2:41:04 PM:     publishOrigin: config
2:41:04 PM: Build failed due to a user error: Build script returned non-zero exit code: 2
2:41:04 PM: Failing build: Failed to build site
2:41:05 PM: Finished processing build request in 18.835s

@stankyoztc 再试一下，我觉得现在绝对不可能再失败了，除非还有什么刁钻限制

@stankyoztc 再试一下，我觉得现在绝对不可能再失败了，除非还有什么刁钻限制

现在是初始化阶段提示log：
5:03:46 PM: build-image version: ecdc8b770f4a0193fd3f258c1X(focal)
5:03:46 PM: buildbot version: 244d5322c7eb13029c1X
5:03:47 PM: Fetching cached dependencies
5:03:47 PM: Failed to fetch cache, continuing with build
5:03:47 PM: Starting to prepare the repo for build
5:03:47 PM: No cached dependencies found. Cloning fresh repo
5:03:47 PM: git clone --filter=blob:none https://github.com/stankyoztc/DeepLX-Serverless
5:03:47 PM: Preparing Git Reference refs/heads/main
5:03:48 PM: Failed during stage 'Reading and parsing configuration files':
{
"siteInfo": {
"id": "166df24b-8a98-4f7c-b277-a423c32X86",
"account_id": "66f190cXa64eXd946ed3"
},
"integrations": [],
"accounts": [],
"addons": [],
"env": {},
"configPath": "/opt/build/repo/netlify.toml",
"redirectsPath": "/opt/build/repo/_redirects",
"headersPath": "/opt/build/repo/_headers",
"buildDir": "/opt/build/repo",
"repositoryRoot": "/opt/build/repo",
"config": {
"functions": {
"*": {}
},
"plugins": [],
"functionsDirectory": "/opt/build/repo/functions",
"functionsDirectoryOrigin": "config",
"build": {
"environment": {
"BRANCH": "main",
"CONTEXT": "production",
"DEPLOY_PRIME_URL": "https://main--nXl.netlify.app/",
"DEPLOY_URL": "https://66f5X38380008019908--nXl.netlify.app/",
"GO_VERSION": "1.19.x",
"SITE_ID": "166df24b-8a98-4f7c-b277-a42X98c86",
"SITE_NAME": "nXl",
"URL": "https://nXl.netlify.app/",
"PORT": 6119
},
"publish": "/opt/build/repo",
"publishOrigin": "config",
"processing": {
"css": {},
"html": {},
"images": {},
"js": {}
},
"services": {},
"command": "npm install",
"commandOrigin": "config",
"functions": "/opt/build/repo/functions"
},
"headers": [],
"redirects": []
},
"context": "production",
"branch": "main"
}
: json: cannot unmarshal number into Go struct field BuildConfig.Config.build.environment of type string
5:03:48 PM: Failing build: Failed to parse configuration

我在netlify注册账号后尝试部署，发现了很多问题，而且在我深入了解文档后发现好像并没有完整支持node，也就无法使用大部分node模块，甚至是常用模块，这方面我可能需要重构代码，或是为netlify重写一份……
大概率我是不会这么干的，要么放弃支持，要么重写逻辑，我总得选一个，好吧其实我完全可以把这事搁置，等以后慢慢重构，反正目前我写了半天发现根本不兼容，看文档也是发现根本不用node模块，或者换一种大力出奇迹的方式去兼容netlify，目前我还没有头绪，所以可能会直接搁置。


location update not working (Canada)

Thanks for the library! All the examples seem to work (region: NCI/canada), except for the get-leaf-location example, which returns a  error -5256: [MyCarFinder] API call error
Is this possibly due to the API not available in Canada? Also, just for curiosity's sake, do any other regions allow you to find the location of the vehicle?

Same for me in Germany. I figured out, it is because of the missing gdc_user_id. This is empty in my case:
Traceback (most recent call last):
File "get-leaf-location.py", line 31, in
result_key = l.request_location()
File "/usr/local/lib/python2.7/dist-packages/pycarwings2/pycarwings2.py", line 415, in request_location
"UserId": self.session.gdc_user_id, # this userid is the 'gdc' userid
File "/usr/local/lib/python2.7/dist-packages/pycarwings2/pycarwings2.py", line 101, in _request_with_retry
ret = self._request(endpoint, params)
File "/usr/local/lib/python2.7/dist-packages/pycarwings2/pycarwings2.py", line 141, in _request
raise CarwingsError
pycarwings2.pycarwings2.CarwingsError

Same for me in Ireland.  I have a late 2015 Tekna, but I can't see what the gdc_user_id and password would be for?
Is there someone for whom this does work who could say whether gdc_user_id & password in their response matches the login userID and password, or are they something completely different, and if they are, where can we get them?

get-leaf-location.py for me, based in the UK using my fork of pycarwings2
POST /api_v181217_NE/gdc/UserLoginRequest.php
returns ....  "gdcUserId":"","gdcPassword":"",  ....
it does return "dcmId":"xxxxxxxxxx".


Make fields optional

Make the following fields optional
a. Email
b. GitName
c. Phone
d. TeleHandle

This is achieved by allowing empty strings in the construction of the above. This also allows removal of attributes in EditCommand simply by specifying e/, g/, p/, h/ respectively.

Add some parser utilities

Codecov Report
Base: 63.46% // Head: 63.88% // Increases project coverage by +0.41% :tada:

Coverage data is based on head (7b33592) compared to base (3d63ea7).
Patch coverage: 81.66% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##             master      #90      +/-   ##
============================================
+ Coverage     63.46%   63.88%   +0.41%
- Complexity      579      597      +18
============================================
  Files           101      101
  Lines          2105     2143      +38
  Branches        252      259       +7
============================================
+ Hits           1336     1369      +33
  Misses          638      638
- Partials        131      136       +5

Impacted Files
Coverage Δ

.../java/seedu/studmap/logic/commands/AddCommand.java
100.00% <ø> (ø)

src/main/java/seedu/studmap/ui/StudentCard.java
0.00% <0.00%> (ø)

...main/java/seedu/studmap/model/student/GitName.java
73.33% <85.71%> (+3.33%)
:arrow_up:

...n/java/seedu/studmap/model/student/TeleHandle.java
73.33% <85.71%> (+3.33%)
:arrow_up:

...a/seedu/studmap/logic/parser/AddCommandParser.java
90.62% <86.36%> (-9.38%)
:arrow_down:

...c/main/java/seedu/studmap/model/student/Email.java
86.66% <100.00%> (+6.66%)
:arrow_up:

...c/main/java/seedu/studmap/model/student/Phone.java
86.66% <100.00%> (+6.66%)
:arrow_up:

.../java/seedu/studmap/model/student/StudentData.java
67.24% <100.00%> (+2.42%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


[Easy] Include Solver Name in Simulation Failure

Based on several alerts of the form:
ERROR solver::driver: settlement simulation failed right before submission AND for block 12527827 which was current when liquidity was fetched:

It would be nice to know which solver is failing (given that many of them are rounding errors from HTTP solver)

I'm always for adding more information in the alerts!
Note that the solver name should be there in the following log (if you still need to figure it out), but I agree that it adds an unnecessary extra step when debugging:


Podman sysext

Podman sysext
Initial work on getting a podman sysext to work, see flatcar/Flatcar#112. For now just includes the basic ebuilds required to build the sysext.
A few minor changes to the infrastructure were made, it's now possible to pass the debug flag to emerge when building packages and a sysext can now be built based on multiple packages. The latter was useful as I wanted to include passt for pasta rootless networking in the podman sysext.
Next I intend to look into what a mangle script should include, particularly based on the prior work on podman sysexts based on the docker builder.
This is very much my first time working with this repo, as such I am not familiar with the context. The ebuilds were pulled directly from gentoo. For most, masking on amd64 was disabled and were otherwise kept as-is. For the podman ebuild in particular, gpgme was added as a BDEPEND, as otherwise building the binary package would fail. There appear to be scripts responsible for updating the ebuilds, how does one integrate with those?
How to use
Inside the SDK container, on a host with a kernel that is not too recent(otherwise the ZFS sysext won't build):
./build_packages
./build_image sysext

Testing done
Next to none. This is very much a draft.

[ ] Changelog entries added in the respective changelog/ directory (user-facing change, bug fix, security fix, update)
[ ] Inspected CI output for image differences: /boot and /usr size, packages, list files for any missing binaries, kernel modules, config files, kernel modules, etc.

Thanks, for the config or service files we might need to have them be under /usr if that isn't already the case. If /usr is not usable for config files, as is the case for /etc/subuid, /etc/subgid, we would need to create them on load with a helper service. See build_library/sysext_mangle_*.

Alright, latest draft takes care of the mangle script. I still want to do more some more testing but not today.
The mangle script in its current form sets up the UID/GID mapping via systemd-tmpfiles which seemed like the neatest solution, alongside the required config files. It also ignores podman* bridge networks, sets up dbus which podman wants for cgroup management, and cleans up extraneous files(only required for development etc.).

Regarding the BDEPEND issue for podman, that is arguably an upstream issue, filed that as a bug.
For this PR I used Gentoo unstable versions, is that acceptable? Or should I revert to using only the currently stable versions? I also may need to unmask arm64 separately, I can't test on that myself.

Seems to all be working nicely now. I went through and tested what I can think of, running podman containers in rootful and rootless modes works, so do quadlet and pasta networking.

My mistake it seems I committed the removal of gpgme from the BDEPEND from my testing. I'll fix that in a bit.

Looks great, you can ignore the test failure here, fix in https://github.com/flatcar/mantle/pull/530 (The other two are failing due to the GitHub Action setup).

Squashed and changelog added! Commit message is hopefully in the right format.


(Optional) Setup pre-commit framework

pre-commit is a framework for working with git hooks. It might replace our custom hooks (which may be more buggy)

Unfortunately, pre-commit framework does not seem to be an option:

it does not provide pytest hook (they argue that tests are usually slow)
it runs all the hooks even on files that are not being committed (e.g. untracked pytest tests that may fail)

Our git hooks are just much better :smile: Most importantly, tools in the hooks are applied only to the committed files and changes to be committed. Unstaged changes and untracked files and stashed before running git hooks.


Upgrade realtime protocol from version 1.2 to 2.0: 'no-connection-serial'

includes:

add channel reattaching and presence re-entering
document connection recovery properly and explicitly
remove RTN15f which was wrong (implied you should only republish if a resume was successful) and redundant to RTN19a

This is a port of https://github.com/ably/docs/pull/1526 , several discussions have already taken place on that PR.

Two more changes, per discussion this morning and on slack:

Spec: Add RTP17h -- myMembers is keyed by clientId only
Spec: set channel.properties.channelSerial from ATTACHED as well


Null check operator used on a null value

What
-

@teolemon Could it be because you have no more products to be completed?
In off-dart tests, I noticed that sometimes you get a result count of null instead of 0.

By construction, we have hundreds of thousands. So it might be that the query timed out ?
Are you able to reproduce using All products to be completed?

By construction, we have hundreds of thousands. So it might be that the query timed out ? Are you able to reproduce using All products to be completed?

Indeed, I thought it was "your" products to be completed - in that case an empty list would have been possible (but not very realistic).
I have to double-check. I noticed that sometimes the server didn't return count - anyway the problem is on the server side.
"Our" problem would be say it instead of that "null" error, but that wouldn't help the user anyway.
I'll have a look when I can, but I don't think it's a priority and anyway I'm currently busy with the 10k download.

In fact this is what the server returns: {"page":1,"page_size":25,"products":[],"skip":0}. Which is definitely not correct.
We could say in Smoothie that if count == null (or if there's no count), we should display "server error" instead.
Which does not explain why we get no count, but that's a problem for off-dart (or the server).

For the record there's a test in off-dart: api_get_to_be_completed_products_test.dart.
Which checks the number of products to be completed in France, Italy and Spain.
Sometimes it works for Italy (231822 products to be completed) and Spain (310985).
But for France we always get the Null check operator used on a null value, as there's no count.
Last year (2022-07-06) we expected:

910148 product to be completed for France
171488 for Italy
272194 for Spain

My hunch: the server chokes on too much data for France (and in Smoothie - not in off-dart tests - you have to sort them too).

@g123k

I suppose we should have rephrased the issue as: how come there are no result for "all products to be completed"?


Why am I unable to uninstall it?

I click the button but nothing happens

After you uninstall and restart all atom windows do you see it in the installed package list?

All good. I didn't restart it. Thanks!
On Tue, Jul 12, 2016 at 7:19 PM, Tarn Burton<EMAIL_ADDRESS>wrote:

After you uninstall and restart all atom windows do you see it in the
installed package list?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/yitzchak/linter-spell/issues/17#issuecomment-232210790,
or mute the thread
https://github.com/notifications/unsubscribe/ASuZRC8FM3J_5_jRLzqsgmOUAPF3h1n-ks5qVCELgaJpZM4JK6d9
.

It's probably still a bug, but at least your install is clean. Thanks for reporting!

This looks like a bug in Atom <1.9 per atom/atom#11464


Rewrite import paths in all packages that share the same dependency

If several packages require the same dependency during vendor add only one of the packages will have its imports overwritten.
This happens because when adding dependencies the imports rewrite is skipped if package files exist locally. https://github.com/kardianos/vendor/blob/master/rewrite/command.go#L233
To reproduce, here is an example: https://gist.github.com/mariash/6a48e89f18b9dd7a73a8
bosh-utils and main package share the same dependency github.com/pivotal-golang/clock. Only bosh-utils imports will be overwritten. Main package import for github.com/pivotal-golang/clock will remain the same.

Fixed in 783925603a9f2febeef8c9e4242946ca64ec506b


[Event Request] Table 36 - "Sales Header" - OnBeforeSalesLinesExist

Dear ALAppExtensions team,
through this issue we would like to request the addition of an event publisher OnBeforeSalesLinesExist in Table 36 "Sales Header" with the following signature:
[IntegrationEvent(false, false)]
local procedure OnBeforeSalesLinesExist(var SalesHeader: Record "Sales Header"; var IsHandled: Boolean)
begin
end;

which would be used in procedure SalesLinesExist() as follows:
procedure SalesLinesExist(): Boolean
var
	// >>>> New >>>>
	IsHandled: Boolean;
	// <<<< New <<<<
begin
	// >>>> New >>>>
	IsHandled := false;
	OnBeforeSalesLinesExist(Rec, IsHandled);
	if IsHandled then
		exit;
	// <<<< New <<<<
	SalesLine.Reset();
	SalesLine.SetRange("Document Type", "Document Type");
	SalesLine.SetRange("Document No.", "No.");
	exit(not SalesLine.IsEmpty());
end;

Thanks in advance!

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 10806.


Menu Item rendering as  and not  resulting in no animations.

Steps
Imported CSS at root app.js file:
import 'semantic-ui-css/semantic.min.css';
(also tried to import it directly via CDN into the HTML template header, same result).
Attempted to compile:
import React from 'react';
import { Menu, Segment } from 'semantic-ui-react';

export default class FullMenu extends React.Component {
render() {
        return (

                <Menu>
                    <Menu.Item name='home'>Home</Menu.Item>
                    Editorials
                </Menu>

        );
    };
}

Expected Result

    Home
    Editorials

Both the Home and the Editorials are responsive and have CSS animations/transitions when hovering.
Actual Result

Home

    Editorials

Only the Editorials is responsive and has CSS animations/transitions when hovering.
Version
Currently using:
"semantic-ui-css": "^2.2.12"
"semantic-ui-react": "^0.76.0"
"react": "^16.1.0"
"react-dom": "^16.1.0"
"webpack": "^3.8.1"
"babel-cli": "^6.26.0",
"babel-core": "^6.26.0"
"babel-jest": "^21.2.0"
"babel-loader": "^7.1.2"
"babel-plugin-transform-decorators-legacy": "^1.3.4"
"babel-plugin-transform-object-rest-spread": "^6.26.0"
"babel-preset-env": "^1.6.1"
"babel-preset-react": "^6.24.1"
Testcase
Unable to provide. Not sure if just my environment setup perhaps?

Read the implementation library, this behaviour seems to be desirable.
The <Menu.Item> tag will translate to  if it has no onClick attribute otherwise it will translate into an  tag (with its corresponding animations etc.).
If the animation is desired for a no-onClick tag, simply create an empty anonymous function for onClick={() => {}}.
Closing issue as it is invalid.

@dropbeardan Please don't create functions in your render method.  It reduces React's ability to do performance updates.  A new function is created every time, so props comparisons always fail.
Note, that you can control what element the component should render as.  This works for all components in SUIR:
<Menu.Item as='a' />
<Menu.Item as='div' />
<Menu.Item as={MySpecialItem} mySpecialProp />

In the third example, mySpecialProp is passed to MySpecialItem.  All unhandled Menu.Item props are passed to the component it is rendering as.  This is also true for all SUIR components.
Cheers!

Cheers!


Test.ChainProducerState: add FlexibleContexts

This fixes an issue on ghc 9.4

I checked that the package builds with ghc-9.4 and ghc-9.6 without this change.


Fix wrong filepath in init/processmonitor.conf

Fix wrong filepath "/opt/rc_process_monitor/rc_process_monitor.sh" to "/opt/processmonitor/processmonitor.sh".
After applying this change, I became possible to start processmonitor service.

Thanks, staybuzz.
LGTM.


Change getting arrays intersection

Fix getting array intersection.
How bug was found:
arrayA = [5509, 5, 1068, 3563, 5061, 458], arrayB = [1788, 5509, 5, 1068, 1754, 5061, 458]
expected : intersection = [5509, 5, 1068, 5061, 458]
current :  intersection = [1068, 5061, 5509]

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Danil Pavlov seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


c0isc_demo: fix min/max demo on singlenode cluster

c0isc_demo does not return any result for min/max on a singlenode cluster:
$ ./c0isc_demo max
c0isc_demo success

Solution: fix bug at minmax_output_prepare().

Depends on https://github.com/Seagate/m0client-sample-apps/pull/9.

Depends on https://github.com/Seagate/m0client-sample-apps/pull/9.


FAQ page has lots of empty space on right

The graphic seems to not be resizing properly:

which results in a bunch of empty space on the right:

Tested on Chrome + Nexus 5P

Confirmed, coding bug, will take a look at this one.


rosegarden: Use wrapQtAppsHook

Motivation for this change
Fixes this startup failure:
qt.qpa.plugin: Could not find the Qt platform plugin "xcb" in ""
This application failed to start because no Qt platform plugin could be initialized. Reinstalling the application may fix this problem.

Aborted (core dumped)

Fixes #76639.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @lebastr

Thank you!


Fix for the TEMPLATE_* backwards compatibility for 1.8

?: (1_8.W001) The standalone TEMPLATE_* settings were deprecated in Django
1.8 and the TEMPLATES dictionary takes precedence. You must put the values
of the following settings into your default TEMPLATES dict: TEMPLATE_DEBUG.

https://docs.djangoproject.com/en/1.8/ref/checks/#backwards-compatibility
https://docs.djangoproject.com/en/1.8/ref/templates/upgrading/#the-templates-settings

According to the release notes for upgrading templates to Django 1.8 -
If it sets TEMPLATE_DEBUG to a value that differs from DEBUG, include that value under the 'debug' key in 'OPTIONS'.
So removal of TEMPLATE_DEBUG should suffice.

Thank you!


CONTRIBUTING.md: Fix error in scripted test command

Can one of the admins verify this patch?

:+1:


Query/Aggregate Builders no longer allow incremental results / always cache

Q
A

Version
2.0.2

Support Question
In ODM 1.3 I was able to build and run queries with large result sets. This allowed me to process incoming data incrementally, by yielding data in generator.
In 2.0 with new Builders I've noticed that both builders wrap Cursor with CachingIterator, which causes high memory usage in such cases. I've hit my memory limit (obviously didn't want to raise it, I had it relatively high in the first place) and had to work around it - I'm getting pipeline built and passed directly into aggregate method of Collection.
$builder = $this->dm->createAggregationBuilder($collectionName);

$collection = $this->dm->getClient()->selectCollection('xxx', $collectionName);
yield from $collection->aggregate($builder->getPipeline(), [
    'allowDiskUse' => true,
    'cursor' => true,
]);

I'd like to be able to select which class caches the results, maybe discarding the ability to rewind/count when I know I won't be using them?

Ah, I see. We originally made the decision to wrap the MongoDB cursor in a CachingIterator instance to allow rewinding over the result set. I think adding an explicit way to disable the CachingIterator functionality should be possible for query and aggregation builders:
$builder
    ->cache(false) // true is the default
    ->getQuery()
    ->getIterator(); // Would no longer return a `CachingIterator`

For the aggregation builder I'd have to think about where to put this option. I'm a bit vary of putting it on the root level (like we do in query builder) since we only have aggregation pipeline stages there. On the other hand, aggregation options in execute might be wrong as well, as those options are all passed to the underlying aggregate method.
If you want to give implementing this a shot, please feel free to reach out on Slack or here to discuss specifics.

FWIW I'd name the method rewindable or something alike. Cache might imply we finally implemented 2nd level cache for queries :)

Good point. Let's not call it cache ;)

If you want to give implementing this a shot, please feel free to reach out on Slack or here to discuss specifics.

Gladly, however I've still got another issue I've reported here that I promised to take care of, so I don't want to promise more :) I'll try though. I'll say so here if I start doing anything about it :smile:

I've looked into both HydratingIterator and CachingIterator and their behavior is a little different. By this I mean HydratingIterator does not cache it's results and does not allow re-iterating over the results. I've confirmed this by adding a test to HydratingIteratorTest:
public function testRewindAfterPartialIteration()
{
    $iterator = new HydratingIterator($this->getTraversable([1, 2, 3]), $this->uow, $this->dm->getClassMetadata(User::class));

    $iterator->rewind();
    $this->assertTrue($iterator->valid());

    $iterator->next();
    $this->expectException(Throwable::class);
    $this->expectExceptionMessage('Cannot rewind a generator that was already run');
    iterator_to_array($iterator);
}

which passes. This is in contrast to a similar test that is present for CachingIterator:
https://github.com/doctrine/mongodb-odm/blob/3b8241869d8540c91eb7a604575ea09205ecc5fd/tests/Doctrine/ODM/MongoDB/Tests/Functional/Iterator/CachingIteratorTest.php#L14-L25
I believe that if we are to allow control over whether the returned iterator caches his results or not should be consistent between these two.
Therefore, I'm wondering if it wouldn't be "better" to make the iterator remember the original Query object instead and re-run query on rewind, leaving eventual wrapping with caching iterator to end-user? What was the reasoning behind introducing this internal cache? What is it used for?

I've looked into both HydratingIterator and CachingIterator and their behavior is a little different. By this I mean HydratingIterator does not cache it's results and does not allow re-iterating over the results. [...]
Therefore, I'm wondering if it wouldn't be "better" to make the iterator remember the original Query object instead and re-run query on rewind, leaving eventual wrapping with caching iterator to end-user? What was the reasoning behind introducing this internal cache? What is it used for?

This is intentional. For a bit of context, let me elaborate on cursor in MongoDB. Server-side cursors are forward-only: once you have received a batch, there is no going back to the previous batch. Once the cursor is exhausted, i.e. the last batch of data has been sent, the cursor is closed on the server. In the legacy ext-mongo extension, this was hidden away in the MongoCursor class. When you called rewind on the cursor, it ran the command. This means that if you iterate a MongoCursor twice, the second iteration might get you different results. Similarly, calling count on a MongoCursor didn't count the number of results, but rather sent a count command to the server. Again, this could return a different number of results than what you would see by iterating on the cursor.
When creating the current ext-mongodb extension, this behaviour was changed to be consistent to the server. The MongoDB\Driver\Cursor class is forward-only and cannot be rewound or counted. This was done to remove the issues I mentioned in the paragraph above.
When we look at this from an ODM context, we expect that in some cases, users may want to iterate cursors multiple times or count the number of results returned from the cursor. To allow for this, we added the CachingIterator class that iterates the underlying cursor during the first iteration, then iterates the cached results on the subsequent iteration. When running count on such a cursor or when rewinding a partially iterated cursor, the underlying cursor object is exhausted and all results stored in memory.
The reason the HydratingIterator does not show this behaviour is because we don't want to run hydration multiple times when rewinding the iterator. Thus, we cache hydrated results for performance reasons, at the expense of a higher memory footprint. This could technically be changed by wrapping the underlying cursor in a CachingIterator, then wrapping that in a HydratingIterator to save a little bit of memory at the expense of higher CPU usage due to the additional hydration calls.
I understand your use-case, and the goal is to create a feature to allow users more control over how their data is cached. One option would be to indeed create a cursor class that stores the original query/aggregation object and re-runs this when the cursor is rewound. The tricky part is making this behaviour explicit and understandable for users. This includes properly documenting the API as well as the performance implications of disabling caching on the cursor. However, I'm not sure if this is an API we want to introduce here, but I'll be discussing this with @malarzm and @jmikola at SymfonyCon later this week.

@alcaeus Thanks for the detailed explanation :+1: It perfectly answers my doubts regarding the reasoning of the change between previous major version.
I've read the usage of HydratingIterator wrong. I thought that it is returned immediately. I did not notice that practically always it is wrapped with CachingIterator. I expected that the memory issue would occur only when not hydrating results, which is not the case.
I would say then - if I can voice my opinion on this matter - that it is reasonable to add rewindable method (and option) to normal and aggregation queries (builders) and return iterators without wrapping them in CachingIterator when it is true. Otherwise it is almost impossible to iterate over large result sets, with hydration or not.

+1 to this issue. As workaround I've implemented helper function, which unwrap caching iterator via reflection.
class DoctrineUtil
{
    public static function iterate($query)
    {
        if ($query instanceof \Doctrine\ODM\MongoDB\Query\Builder) {
            $query = $query->getQuery();
        }
        if ($query instanceof \Doctrine\ODM\MongoDB\Aggregation\Builder) {
            $query = $query->getQuery();
        }
        if (!$query instanceof \Doctrine\ODM\MongoDB\Query\Query) {
            throw new \InvalidArgumentException(sprintf('Expected Query or Builder, given "%s"', get_class($query)));
        }

        $iterator = $query->execute();

        if ($iterator instanceof \Doctrine\ODM\MongoDB\Iterator\CachingIterator) {
            $ref = new \ReflectionObject($iterator);
            $itemsProp = $ref->getProperty('items');
            $itemsProp->setAccessible(true);

            yield from $itemsProp->getValue($iterator);

            $iteratorProp = $ref->getProperty('iterator');
            $iteratorProp->setAccessible(true);
            yield from $iteratorProp->getValue($iterator);
        }

        return $iterator;
    }
}

Fixed in #2116.


Problem loading job classes with autoloading

If you pass a string value in for the class name, the Module.const_defined? will not return true if the constant has not yet been loaded.  In a Rails environment this can be a bummer due to the lazy loading in Rails.  To address this, we can instead attempt to load the const and if a NameError occurs, we'll just return nil as the original code would have done.
I ran into this when using the #push_bulk feature of Sidekiq.  Jobs created using that method did not get created properly as status workers if the class name was a string and not yet loaded.

Hi and thanks for the PR! Would you be able to write one or more tests proving the new code works as expected? Hopefully those tests can fail against master but pass on your branch.
Earliest I'd be able to write those myself would probably be early next week, but if you could provide them, that would be super helpful.

Hey @kenaniah - sorry for the delay.  I'll see what I can do to get some additional tests in place this week.

@kenaniah - just an update that I have not forgotten about this - I just have not been able to get to it yet.  Hoping I can in the next few days.

@noiseunion I think you forgot about this 😛

@jkeen - yes I did forget!!! 😬 Sorry about that.  I will make some time this week to write some tests.

UPDATE - I have started trying to write tests for this.  However, in all honesty, I have not yet figured out how best to do so.  Somehow I have to for lazy loading of class within the test and I honestly am not really sure how best to do that at the moment.
I am definitely open to any suggestions 😄


An AnchorableBehaviour which is not using AnchorGroups is attachable to any Anchor.

If an AnchorableBehaviour (AB) is not using an AnchorGroup, it can attach to any Anchor in the scene.
If the Anchor property is intended to be used so that ABs only attach to that specific anchor (which I assumed), then the method IsValidAnchor() should check Equality I think:
public bool IsValidAnchor(Anchor anchor) {
  if (this.anchorGroup != null) {
    return this.anchorGroup.Contains(anchor);
  }
  else {
    return anchor.Equals(this.anchor);
  }
}

The Anchor Tooltip "The current anchor of this AnchorableBehaviour." however makes it sound like thats actually the intended behaviour and the Anchor property only shows whichever Anchor this AB is currently attached to. Based on how you intend the usage, I think the Tooltip should be revised to make that a bit clearer.

Oh, i just saw that the AnchorGroup tooltip clears things up a lot.
In that case, I think it might be good to also grey out the Anchor property, since my first expectation was that I should probably assign something to that property.
But then you couldn't use the "Move Object to Anchor" button anymore, so I don't know which would be the best way to go from here.

I agree, the Anchor property could clarify the role of AnchorGroups so that this functionality is more clear.
It's useful to have an Anchor attached to an AB's anchor property (1) at edit-time, for the reason you described, and (2) at run-time even if the AB isn't attached, so you can call AttachToAnchor() and send the AB back into its anchor without having to spawn the AB in its anchor.
I also think the use-case where there's only one valid Anchor for an object is probably reasonably common, but the fact that you have to construct a whole single-anchor AnchorGroup and assign it to the AB is a little cumbersome for such a simple constraint. I'll think more about this before I mark this as a to-do. Thoughts @Amarcolina? One option is just adding a toggle switch on the inspector...

I haven't really used the Anchorable system, so I couldn't really say D:

Closing here. Tracked as UNITY-745


Update to arrow 30.0.1

Which issue does this PR close?

Closes #.
Rationale for this change

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

Practically speaking this is blocked on https://github.com/calebzulawski/target-features/issues/1 it makes working on this crate completely unworkable

https://github.com/apache/arrow-datafusion/pull/4821 contains the necessary changes to make the benchmarks pass

I'm working on it :smile: There be shenanigans

Thank you @tustvold


No cursor change for DND

Testing #23329
MacOS
When I press Alt the cursor does not change - I can drag and drop as well as copy.  However the cursor does not change.
Here are my settings (sorry a very ugly theme):
// Place your settings in this file to overwrite the default settings
{
	//"editor.wrappingColumn": 0,
	"editor.renderIndentGuides": true,
	"editor.renderWhitespace": "all",
	"files.hotExit": "onExitAndWindowClose",
	"files.autoSave": "afterDelay",
	"editor.wordWrap": "on",
	"typescript.referencesCodeLens.enabled": true,
	"debug.inlineValues": true,
	"editor.tabSize": 4,
	"editor.snippetSuggestions": "top",
	"editor.dragAndDrop": true,
	"workbench.iconTheme": "vs-seti",
	"workbench.colorTheme": "Custom Theme",
	"workbench.customColorTheme": {
		"extends": "Default Light+",
		"type": "dark",
		"colors": {
			"activityBarBackground": "#888888",
			"tabsContainerBackground": "#EEEEEE",
			"titleBarActiveBackground": "#EEEEEE",
			"statusBarBackground": "#AAAAAA",
			"statusBarForeground": "#555555",
			"statusBarInfoItemBackground": "#666666",
			"activeTabBackground": "#EEEEEE",
			"activeTabInactiveGroupForeground": "#EEEEEE",
			"inactiveTabBackground": "#AAAAAA"
		} // http://colormind.io/
	}
}

cc @roblourens as he was on the books for MacOS testing and may be able to repro.

Dark theme css override leads to this problem.


Feat: Added a puzzle UI

Created the puzzle UI for the puzzle question type

I have commented the dynamic display of question type to directly display the puzzle UI to test the functionality. If there are unexpected behavior.
PS: the answer it will accept only 4 answers


Review high-level upload/download APIs

I see the following.
func (bb *BlockBlobClient) UploadBuffer()
func (bb *BlockBlobClient) UploadFile()

func (b *BlobClient) DownloadToBuffer()
func (b *BlobClient) DownloadToFile()

Why do we have separate APIs for this?  Seems like we should upload from an io.Reader and download to an io.Writer which covers both cases.
If there are legit cases where we need them that's fine, just want to make sure we enumerate the differences.

Fixed in upcoming<EMAIL_ADDRESS>


Unsigned Integers - Serialisation and Deserialisation

To interact with contract functions that utilize U256 and U512 signatures, support is needed for the serialization and deserialization of these data types.

#600 PR to resolve the issue

Bounds Checking: Ensure that the integer value is within the valid range for unsigned integers.
Endianness: Be mindful of endianness when serializing and deserializing binary data.
Data Format: Choose the appropriate format based on your storage or transmission needs.


ci: add dependabot to update github actions

The CI system fails for some unrelated reason...

I think this PR should be non-controversial, and since binderhub is a repo where we have got behind a lot on maintenance and review of PRs etc I'll go for a self-merge.


Make some NetworkClient and NetworkServer functions not internal for fully custom NetworkManager

Please explain the suggested feature in detail.
I'm trying to write my own NetworkManager from the ground up without inheriting from built in NetworkManager.  In trying to do this i ran into a blocker pretty fast where i need to call NetworkClient.ConnectHost() but it is marked as internal.
Walking through the code paths NetworkManager is using for NetworkClient and NetworkServer it does seem like to beable to do what i'm trying to do there is a handful of methods that are marked internal that would need to become public
NetworkClient

Update
ConnectHost
ConnectLocalServer
DisconnectLocalServer

NetworkServer

ActivateHostScene,

How exactly does this keep you from releasing your game right now?
It doesn't i'm just trying to write a NetworkManager from the ground up because i don't like the way some of the built in NetworkManager works and I want to change the core functionality but still reuse the NetworkClient and NetworkServer classes and their scope is preventing this.
Can you suggest a possible solution?
Convert the above methods to be public
Additional context
Add any other context or screenshots about the feature request here.

This came up in a discussion in Discord. The idea being that 'you should be able to do everything without NetMan'. I think one of the few exceptions to that is running in Host mode. I am not sure if just changing the visibility level alone will resolve the issue. Will need some testing to make sure everything works. But in general I like the thought behind this change.

@uweenukr I went through the current NetMan in 16.1.1 and checked every instance of NetworkServer and NetworkClient used to see if the functions were public or internal and these were the only ones not public - while there were a handful there were public already.

Sorry I meant I dont think changing the visibility on its own would break anything. But all of the tests and examples currently would not cover this use case. So we would need to come up with some to make sure it functions properly.

@uweenukr does that preclude this from getting merged.  I would prefer to not diverge from master for too long but my intention right now is to have these changes in my project so i can continue to build out my fully custom network manager.  If this means you want me to also supply some examples for the docs from my exploration and end result i'm happy to but i'd like to stay a close to master as possible without having a long running branch that gets out of date.  So just wanting to make sure we could get this merged in and that the examples can come later after i've done the exploration and building of a fully custom network manager.  Let me know thanks!

Nah I don't think I need to make you make examples :P

Solved by #2040


HWKAPM-771 Fix issue where client 'service name' is being incorrectly…

… associated with the invoked service node

@pavolloffay Could you review on Monday please?
Reviewer notes:

This bug occurred due to changing from using FragmentCompletionTime events to the NodeDetails - as FragmentCompletionTime distinguished between 'client' fragments that just included a Producer for a URL from the server fragment that had the Consumer.
When using just the NodeDetails, we need to perform individual queries based on Producer or !Producer, depending upon whether getting information about a 'client' node or service nodes. Due to this not being distinguished, the service name was being associated with the wrong node in the service dependency diagram.

BTW When running integration tests locally for this fix, I got a failure on the OpenTracing polyglot instrumentation framework test. It didn't seem related, but was happening each time, so I tried it against master and it is now also failing there. Is it possibly related to a recent release of the JS provider?

retest this please

@pavolloffay ready for re-review.

@pavolloffay Thanks!