Fix reflection warning

I added a (set! *warn-on-reflection* true) in the deps.edn test profile.
There is one remaining warning in a dependency of martian:

Reflection warning, tripod/context.cljc:12:52 - reference to field getMessage can't be resolved.

Pull request by borkdude tackling this problem: https://github.com/frankiesardo/tripod/pull/8
Since the PR is still open, he solved it using a patch: https://github.com/oliyh/martian/blob/732be670cd26d2a401b3830867589e33817e7fec/core/src/martian/core.cljc#L13
Test results:
 openai-clojure git:(main) ✗ clojure -M:test -m cognitect.test-runner
true

Running tests in #{"test"}
Reflection warning, tripod/context.cljc:12:52 - reference to field getMessage can't be resolved.

Testing wkok.openai-clojure.api-integration-test

Testing wkok.openai-clojure.api-test

WARNING - passing impl as first argument is deprecated, use options instead

WARNING - passing impl as first argument is deprecated, use options instead

Testing wkok.openai-clojure.azure-test

Testing wkok.openai-clojure.core-test

WARNING - passing impl as first argument is deprecated, use options instead

WARNING - passing impl as first argument is deprecated, use options instead

Testing wkok.openai-clojure.openai-test

Testing wkok.openai-clojure.sse-test

Ran 16 tests containing 93 assertions.
0 failures, 0 errors.

Thanks!


Tealium fails in Strict DI Mode

When running the AngularJS in production, the tealium integration fails in Strict DI Mode.
See: https://code.angularjs.org/1.5.5/docs/guide/production

@theDustRoom Thanks for pointing this out..  This AngularJS rookie just learned something new.  Let me know if recent updates look good.  I've also added the following the sample index.html file.
<body ng-app="App" ng-strict-di>


Adding discount

Hi,
I was just wondering if that would be possible to add a PayPalItem whose price is less than zero to give discount to customers.
At the moment when I do that, it just cancels the payment without even prompting for PayPal/Card Details.

No is supposed that you need to charge a valid price not a 0 or less values in that case you need to handle that logic on your app side code

Thanks!
I figured that out but I asked that question because a lot of times on PayPal receipts generated on other website I have seen the discount value (a negative number).
But thanks for your response and keep up the good work!


fix: add gray 10 background to color svg images

Closes #270
Adds gray10 background color to transparent svgs on color overview page (bottom two images, test mobile/desktop)

I'm seeing a thin white line in the bottom image. The other one looks good. Not sure if it's a big deal or not. It's very subtle, almost missed it at first.

@jnm2377 👀  I'll take a look

this is weird. there is no space in the svg, trying to figure out why the whitespace is showing

@vpicone @jeanservaas had me add that white background to images a couple of weeks ago, we have a bunch of transparent images on the Carbon site that need the white background. So until someone has the time to go and re-save all of those out, we're stuck with these rules.

Ok I lied about the top img not having a white line 😅 Idk how I missed it yesterday. It's so subtle, you kinda have to zoom in to see it properly.
The bottom image is good now, though!!


Slideshow

Discussed in https://github.com/photobooth-app/photobooth-app/discussions/122

Originally posted by laguinch November  4, 2023
Hy, is it possible to make a slideshow from the galley ? Thx

Do you already have an approach in mind?
From the top of my head, I'd probably add a new page for that (or is that what standalone is meant for?) displaying the galleryimagedetail component.
Further brainstorming:

Auto start slideshow after timeout on main page
Auto start slideshow after timeout on gallery page
button to start slideshow in gallery page
when clicking the slideshow, return to main page (maybe prompt "Stop the slideshow?")
configurable slideshow mode (random, in-order)
configurable timeouts (timeout to start, time per image)
probably fade transition between images
hide buttons (i.e. no toolbar in slideshow mode)
no qr code in slideshow mode
handle carousel button presses and swipes not as "cancel the slide show" intent
probably exclude videos and gifs from the slideshow?
config stop of the slideshow (after time X, when all images were shown, repeat forever)

Let me know what you think about it :) unfortunately I won't have time to work on it anyways in the upcoming days.

There is a very simple standalone layout already implemented, which could be extended.

https://github.com/photobooth-app/photobooth-frontend/blob/main/src/layouts/StandaloneLayout.vue
https://photobooth-app.org/reference/foldersandurls/#special-urls
quasar also has some components that could be reused or at least give inspiration:
https://quasar.dev/vue-components/carousel#controls
https://quasar.dev/vue-components/carousel#infinite-and-autoplay

I would implement all these additions in the separate pages and only after a timeout forward the router there.
The separate pages are also great if multiple devices are used. You can point additional devices to a specific standalone Url and it would play random slides permanently. … or it could display a gallery where user can choose for printing.

See https://github.com/photobooth-app/photobooth-frontend/pull/9.

Can be closed, I guess ✌


NDRS-966: Fix retrieval of booking block hashes for eras initialized after restart.

https://casperlabs.atlassian.net/browse/NDRS-966
Fixes retrieval of booking block hashes for eras initialized after the restart.

Sorry for the merge conflict! :grimacing:

bors merge

bors merge


profiles/minimal: reduce size

Description of changes
Minimize size netboot and iso images.
Splitted this PR - https://github.com/NixOS/nixpkgs/pull/170460
cc @SuperSandro2000
Things done

Built on platform(s)

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

[x] 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

[ ] 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

[ ] Fits CONTRIBUTING.md.

Rebased PR.
Removing commit 'nixos/installer/netboot: use makeInitrdNG' - https://github.com/NixOS/nixpkgs/pull/176796
cc @ajs124 @Lassulus @lheckemann

@SuperSandro2000 virtualisation.containers does not use nixos/modules/profiles/minimal.nix.

Didn't not found mysself any problems with environment.noXlibs.

This should have a release notes entry, since it may require changes to people's configurations.

This should have a release notes entry, since it may require changes to people's configurations.

Added release notes.

Rebased PR.
What is needed to get this PR merged? :)

rebased the pr on master(git rebase --onto master HEAD~7)
and built the iso (nix build -f ./nixos/release-combined.nix  nixos.iso_minimal.x86_64-linux)
but the size seems to be larger?
now
du -shc --apparent-size $(readlink -f result/iso/nixos-minimal-22.11pre56789.gfedcba-x86_64-linux.iso)
3.8G	/nix/store/w6aalmkc9w9mvmlmvcc4a6567h5ncrqg-nixos-minimal-22.11pre56789.gfedcba-x86_64-linux.iso/iso/nixos-minimal-22.11pre56789.gfedcba-x86_64-linux.iso
3.8G	total

before
$ du -shc --apparent-size $(readlink -f result/iso/nixos-minimal-22.11pre56789.gfedcba-x86_64-linux.iso)
2.2G	/nix/store/6lby9i5kx22qqgpin23klwpbf6nn6vxm-nixos-minimal-22.11pre56789.gfedcba-x86_64-linux.iso/iso/nixos-minimal-22.11pre56789.gfedcba-x86_64-linux.iso
2.2G	total

@Artturin strange, I have such a result, and the sizes are same:
Result with PR:
du -shc --apparent-size $(readlink -f result/iso/nixos-minimal-22.11pre56789.gfedcba-x86_64-linux.iso)
1,9G    /nix/store/1pj7g3cldsxbkgi1y50vkbvjj4hnllc1-nixos-minimal-22.11pre56789.gfedcba-x86_64-linux.iso/iso/nixos-minimal-22.11pre56789.gfedcba-x86_64-linux.iso
1,9G    total

Result without PR (git checkout HEAD~7):
du -shc --apparent-size $(readlink -f result/iso/nixos-minimal-22.11pre56789.gfedcba-x86_64-linux.iso)
1,9G    /nix/store/sx3lc13lsl5z7ly4z4x9cl79samphbmy-nixos-minimal-22.11pre56789.gfedcba-x86_64-linux.iso/iso/nixos-minimal-22.11pre56789.gfedcba-x86_64-linux.iso
1,9G    total

From somewhere, are added nixpkgs sources with git history to image:
du -d 1 -hc  /nix/.ro-store/5q1kw0svyl0j3p1jwc0yv20x1sa63bw5-source/
1.2G    /nix/.ro-store/5q1kw0svyl0j3p1jwc0yv20x1sa63bw5-source/.git
53K     /nix/.ro-store/5q1kw0svyl0j3p1jwc0yv20x1sa63bw5-source/.github
788K    /nix/.ro-store/5q1kw0svyl0j3p1jwc0yv20x1sa63bw5-source/doc
479K    /nix/.ro-store/5q1kw0svyl0j3p1jwc0yv20x1sa63bw5-source/lib
563K    /nix/.ro-store/5q1kw0svyl0j3p1jwc0yv20x1sa63bw5-source/maintainers
13M     /nix/.ro-store/5q1kw0svyl0j3p1jwc0yv20x1sa63bw5-source/nixos
109M    /nix/.ro-store/5q1kw0svyl0j3p1jwc0yv20x1sa63bw5-source/pkgs
1.3G    /nix/.ro-store/5q1kw0svyl0j3p1jwc0yv20x1sa63bw5-source/
1.3G    total

Checking without PR https://github.com/NixOS/nixpkgs/pull/189113
With this PR:
778M    /nix/store/g01d3rz12z3qwjs3bmwd8q5skhngadn8-nixos-minimal-22.11pre56789.gfedcba-x86_64-linux.iso/iso/nixos-minimal-22.11pre56789.gfedcba-x86_64-linux.iso
778M    total

Without this PR:
du -shc --apparent-size $(readlink -f result/iso/nixos-minimal-22.11pre56789.gfedcba-x86_64-linux.iso)
779M    /nix/store/0fbgh67sslf960l05pklwcj6pzf31yn0-nixos-minimal-22.11pre56789.gfedcba-x86_64-linux.iso/iso/nixos-minimal-22.11pre56789.gfedcba-x86_64-linux.iso
779M    total

сс @MatthewCroughan @ck3d

@Izorkin If you run nix store optimise I think it might eliminate that difference.

@MatthewCroughan this command will not help for iso image.

Fix conflicts.

Can you move the release notes to 23.05 ? Think it's good to merge now.

Can you move the release notes to 23.05 ? Think it's good to merge now.

Updated.

From somewhere, are added nixpkgs sources with git history to image:
du -d 1 -hc  /nix/.ro-store/5q1kw0svyl0j3p1jwc0yv20x1sa63bw5-source/
1.2G    /nix/.ro-store/5q1kw0svyl0j3p1jwc0yv20x1sa63bw5-source/.git
53K     /nix/.ro-store/5q1kw0svyl0j3p1jwc0yv20x1sa63bw5-source/.github
788K    /nix/.ro-store/5q1kw0svyl0j3p1jwc0yv20x1sa63bw5-source/doc
479K    /nix/.ro-store/5q1kw0svyl0j3p1jwc0yv20x1sa63bw5-source/lib
563K    /nix/.ro-store/5q1kw0svyl0j3p1jwc0yv20x1sa63bw5-source/maintainers
13M     /nix/.ro-store/5q1kw0svyl0j3p1jwc0yv20x1sa63bw5-source/nixos
109M    /nix/.ro-store/5q1kw0svyl0j3p1jwc0yv20x1sa63bw5-source/pkgs
1.3G    /nix/.ro-store/5q1kw0svyl0j3p1jwc0yv20x1sa63bw5-source/
1.3G    total

Will fix it
https://github.com/NixOS/nixpkgs/pull/204178

Resolve conflicts.

Resolve conflicts.
Who can merge PR?

Thanks!

Note that the discussion about disabling logrotate in LXC containers is still open.

Note that the discussion about disabling logrotate in LXC containers is still open.

I will try to test the lxc containers a little later.

Note that the discussion about disabling logrotate in LXC containers is still open.

We should fix this regression ASAP because it could fill up the disk completely if things go really south.

We should fix this regression ASAP because it could fill up the disk completely if things go really south.

Created PR - https://github.com/NixOS/nixpkgs/pull/205346


Fix issues for initializing the environment

Added a couple of Batch scripts for easy Windows installs
Fixed a missing dependency in pyproject.toml
Fixed a folder not being created on first run

Cancelling the PR as the process was not correct


Database bootstrap sql file name (S3 object key should be changed)

I am not sure we can call this bug, but I am pretty sure this badly influences on user experience. Whatever the bootstrap name is, SaaS boost uploads the file and name it bootstrap.sql (as a object key in S3)
https://github.com/awslabs/aws-saas-boost/blob/798371c40b032e097d51ac9e57292af78bc201c4/services/settings-service/src/main/java/com/amazon/aws/partners/saasfactory/saasboost/SettingsService.java#L604
When I updated the app config by changing the sql file, submitted and came back to the page, the file name shown in the page was not what I had just uploaded. So Ithough the update was not applied and tried to keep updating that..
Then I realized the update was applied when I manually downloaded file from S3 and checked the contents. The logic works but it confuses the user.
Reproduction Steps

First, set the sql file in app config page.
Second, update the sql file in app config page. File name won't change..
What did you expect to happen?

The file name should be shown as what the user uploaded
What actually happened?

bootstrap.sql file was shown in the page.
Environment

AWS Region  :
AWS SaaS Boost Version :
Workload OS  (Linux or Windows) :

Other

This is :bug: Bug Report

Hi @rnlduaeo , as you mention this is not a bug. But we will consider this as a feature request. Thanks for the report!


Why permute (0,3,1,2) is required?

Hi ninetf135246, I am not able to understand why you are permuting the input tensor in line #142 of train_Sony.py. Please help me to understand your code.
https://github.com/ninetf135246/pytorch-Learning-to-See-in-the-Dark/blob/8af3e309d6947b1addf52765869190ec1e6e2ded/train_Sony.py#L142

Thank you


adjust python test tolerances

Expanded test margins for relative accuracy python tests by about 50%, for all provided variants of quantiles sketches.
The original margins are generally ok, but we're using random data in the tests (which also function as documentation/tutorials) and don't have a way to make the structures deterministic even for testing. As a result, we end up with the expected real-world "failures" due to low probability events showing up eventually. We have enough independent tests that it's not surprising that one of them is outside the stated tolerance every so often.
The non-quantiles sketches do not have this issue and are unchanged.

Pull Request Test Coverage Report for Build 991414770

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 92.76%

Totals

Change from base Build 991138921:
0.0%

Covered Lines:
2178

Relevant Lines:
2348

💛  - Coveralls

Pull Request Test Coverage Report for Build 991414770
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 97.927%

Totals

Change from base Build 991138921:
0.0%

Covered Lines:
7415

Relevant Lines:
7572

💛  - Coveralls


System Apps section not working (iOS 10.3.1, g0blin)

The System App section does not work for me on 10.3.1 with g0blin jailbreak. All other sections are sorted, but the System App section is not there and no system apps are sorted anywhere..

Does this still occur on version 4.0.4 (or whatever the latest version is when you read this)?

Can you enable debug mode and upload the log somewhere…? I am so confused at how this appears to be something that's actually specific to the jailbreak and not iOS…

Sure
https://www.dropbox.com/s/3dvqmsgvucp1w0e/net.angelxwind.preferenceorganizer2.zip?dl=0
On Sun, Jul 22, 2018 at 4:05 PM angelXwind<EMAIL_ADDRESS>wrote:

Can you enable debug mode and upload the log somewhere…? I am so
confused at how this appears to be something that's actually specific to
the jailbreak and not iOS…
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/angelXwind/PreferenceOrganizer2/issues/39#issuecomment-406896675,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AfY4usPF7NsHicQx9FbtiRwBJSi0oq8oks5uJOkjgaJpZM4SR0ni
.

This is a literal 3-year-old issue at this point, but I was not able to reproduce this at least on iOS 10.2.1 (which is the only iOS 10 version I have access to).
What's strange to me is that the change that occurred between 4.0.3 and 4.0.4 was in commit e19459891f7e0357fe7e2694b0d78b8c8ad07f4c, and no changes in there should be able to cause something like this…


refactor(projectrc): extract base64 to string

Current dependencies on/for this PR:

main

PR #30   👈

This stack of pull requests is managed by Graphite.

Merge activity

Dec 9, 1:23 AM: @@luxass started a stack merge that includes this pull request via Graphite.


Make npc work with kubenet+cloudprovider networking

From @bboreham on October 8, 2016 21:16
Eg on AWS native network.
Copied from original issue: weaveworks/weave-npc#18

I wonder whether this would work in an IPv6 network.  I just had a quick look at the npc code and it seems to treat every address as a string, so perhaps it would work.


SP-1468 - Patch vulnerabilities #patch

Codecov Report
Patch and project coverage have no change.

Comparison is base (331351e) 72.93% compared to head (ba51904) 72.93%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #297   +/-   ##
=======================================
  Coverage   72.93%   72.93%
=======================================
  Files          88       88
  Lines        3503     3503
=======================================
  Hits         2555     2555
  Misses        680      680
  Partials      268      268

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 in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Require all parameters to be explicitly declared

Using 'with' statements.
Rather than attempting to infer them from the usage in the template
itself. This has been suggested by @lpil and @michallepicki in order to
make it easier to import modules and use them within the templates and
avoid the engine attempting to infer the module name as a parameter or
generally failing in other ways.
There is an error if you declare the same thing twice using 'with'
statements.
This does mean that users need to explicitly declare all the parameters
name and declare types for String and StringBuilder values. We might be
able to remove the {[]} syntax for StringBuilder injection as we'll
already have that information from the 'with' statements.

Looking great! Is the type required or can it be omitted?

Thanks for checking it out.
The type is currently required. Seems like the most explicit thing to do and that's the motivation behind the change.
(I see your edit now :))


add more non-human genetic diseases

addresses #7271

@sabrinatoro could you please review this PR for me at your convenience?
Thank you!

Thank you @sabrinatoro. Your point is noted and I will document our decision to keep the taxon as Bos taurus as long as the NCBIGene points to Bos taurus, although it is possible that the condition could be reported in a different species of Bos. If the NCBIGene points to a different Bos, I will report that species for in_taxon. Thank you for catching this!


How to Count Distribution Points?

I'd like to count points for a hand's distribution. Default would use the traditional system (1,2,3 for doubleton, singleton, void). I don't see any existing method for doing so. Surely this is a common need. What am I missing? I created some functions as a work-around.
def dist_points_suit(suit, points=[3,2,1]+[0]*10):
    return points[len(suit)]

def dist_points(hand):
    return sum([dist_points_suit(suit) for suit in [hand.clubs, hand.diamonds, hand.hearts, hand.spades]])

def constraints(deal):
    points = hcp(deal.west)+dist_points(deal.west)-dist_points_suit(deal.west.spades)
    return points >= 13 and points <= 15 and len(deal.west.spades) <= 4 and len(deal.west.hearts) >= 5 and len(deal.west.diamonds) < 5 and len(deal.west.clubs) < 5

`

Agreed, this should be included in the evaluate module. I think your approach for shortage points is good, and then in the evaluate namespace there can be various definitions for different standard ways of calculating distribution, e.g something like:
dp_shortage = [3,2,1] + [0]*10 # Counting shortages
dp_length = [0]*4 + list(range(9)) # Counting long suits
dp_mixed = [s+l for s,l in zip(dp_shortage, dp_length)] # For calculating total points

def dist_points(o: Union[SuitHolding, Hand], scale):
    if isinstance(o, SuitHolding):
        return scale[len(o)]
    else:
        return sum(dist_points(o[s]) for s in Denom.suits())

def total_points(hand: Hand, hcp_scale, dist_scale):
    return hcp(hand, hcp_scale) + dist_points(hand, dist_scale)

Thoughts?

Issues:

My example mislead you. There's 14 different suit lengths (0-13) so [3,2,1]+[0]*11 and [0]*5 + list(range(9)).
Short suit distribution points are naively counted [3,2,1] before a trump fit is agreed upon and [5,3,1] after.
Short suit dist points should not add in trump suit or arguably partner's long suit. Compensate in constraints() or pass an arg?
Total points are naively hcp+dist_points but not so for K, QX, JX.

Yes of course :P. Best in the actual package file to explicitly write out the list anyway.
Can be fixed by shortage_points_trump and shortage_points_notrump scale?
An optional exclude parameter which accepts a suit/list of suits would be good as this is common enough
This one is harder and I have often had lengthy discussions with various partners about exactly how much these holdings are worth. It can often depend on the bidding too -- Qx in a suit partner has bid is more useful than a side suit. There is the endplay.evaluate.cccc function which is based on the original Kaplan evaluation (without the Rubens modification) method which takes into account all of these sorts of things and a myriad of other consideration too and in my opinion provides the most accurate calculation of hand strength, so perhaps the total_points function is not actually necessary over a simple estimation of how the high card points and distribution combine.

shortage_points_notrump perhaps should be named shortage_points_trump_fit?
Provide multiple hand evaluation methods which make API use simpler; those commonly used (explainable), advanced (Kaplan), eventually neural net weights (probably most accurate), roll-your-own.

I'll make the names nice and unambiguous. I think for now, I will add in the dist_points function and a few common scales but definitely with the optional parameter so that a custom one can be defined. I'll put in the 'naïve' total points function for now but I'd like to keep the issue open to discuss a few different implementations as hand evaluation methods are definitely useful. I have a flight tomorrow so I'll put it together then and push it in the afternoon.

All good. Bon voyage.

Implementation is pushed adding dist_points and total_points. Actions is running the test suite now and then (assuming all green) will push to pypi.
The compromise I have come up with for now is a protect_honours flag for total_points, which discounts honours which would drop if the opponents played a suit from the top, so any K, Qx, Jxx. This also includes the queen in e.g. KQ which is not ideal but is simpler to implement.
Another option is to have a exclude_holding parameter which contains a list of holdings which shouldn't count towards total points. The trouble with this is twofold: (a) some holdings may just be worth less, not 0 and (b) this requires some way to represent a generic small card. Its possible to add a Rank.Rx entry to the enum --- internally each suit holding is represented by a 16-bit integer where each bit represents a different rank, so there are three spare bits to play around with. However, this might cause some unintuitive behaviour. Another option would be to have an extra parameter smallest_significant_card or something like that, and the function would treat all cards below this as equal.
Either way, it might just be easier to define ones own my_total_points function on a per-usecase basis rather than have a standard total_points function which tries to solve every possible situation. I am happy to expand the evaluate module with more functions if you come up with any evaluation methods you think might be useful.


fix(react-instantsearch): update deps + run code

I couldn't immediately get the range component working again, so I commented it out. @mthuret, maybe you can take a second look?

I will have a look, thanks @Haroenv :)


Override language file not found

I have created a da-DK.json, and put it in the lang/lang-country-overrides folder, however I get this error:
file_get_contents(/Users/rabol/code/web/test/vendor/stefro/laravel-lang-country/src/LangCountryData/da-DK.json): Failed to open stream: No such file or directory

missing '/'
private function getDataFromFile(?string $lang_country): array
    {
        if ($lang_country === null) {
            throw new Exception('The lang_country session is not set');
        }

        if (file_exists(lang_path('lang-country-overrides') . '/' . $lang_country . '.json')) {
            $resource = lang_path('lang-country-overrides') . '/' .$lang_country . '.json';
        } else {
            $resource = __DIR__ . '/LangCountryData/' . $lang_country . '.json';
        }

        return json_decode(file_get_contents($resource), true);
    }
    ```

@rabol I've just made a fresh install of Laravel Jetstream ans pulled in the Laravel LangCountry package. I've added an override language as shown below and was also successful by switching to the da-DK language.

By the looks of your error message, It seems like it was not able to find the override file in the lang/lang-country-overrides dir, so it will try to load the file from the package LangCountryData . Can you check your lang/lang-country-overrides path? Also, maybe you're using a custom lang_path()?

Fixed in https://github.com/stefro/laravel-lang-country/pull/46


[Feature][python] Find a good way to change process definition name in python api

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
Separate from #6407, We still DO NOT a good way to update project or process_definition_name, it would create new workflow when their changed, maybe we should find a good way to change those binding name
Use case
No response
Related issues
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

A good mechanism to update process definition: For now DS design base on UI define workflow, which we modify database model step by step, is a quick different from define or update by code. We have to consider who to update it. Cause we should not ask user define process definition code or tasks code in the python API, I currently think maybe we should delete all tasks in process definition and then create new. It's still could not handle when users change project name and process definition, but we would try to fix it later

Should be a part of #8517


edit and insert entery by index

please add function for Edit and Insert Entry by index
like array
List.insert(index,Entry)
List.edit(index,Entry)

I try this yesterday. If the index exists in entry list, edit/update is ok (in most cases, it will update the Y value of the corresponding entry).But insert seems a little conflict, do this means entries in the list can't have the same interval in X value(before insert X values is [1.0, 2.0, 3.0 ...], after is [1.0, 1.5, 2.0, 3.0 ...]), is this you want?

yes

done. you can see realtime demo for update and dynamic demo's comment for insert


Server mode not refreshing

So, i´ve recently started using VEDA.
I can´t seem to be able to refresh the shader onces is been launched to the server.
This is my code, as you can see is almost as the example :
`/*
  {"server":3000}
*/

precision mediump float;
uniform float time;
uniform vec2 mouse;
uniform vec2 resolution;

void main(void) {
  vec2 uv = gl_FragCoord.xy / resolution.xy;

  gl_FragColor = vec4(0.0,0.0,0.9, 1.0);
}
`

The server opens one time. and then it can´t be refreshed.
I´ve already tryed in : opera, chrome, firefox with same result
If i changed the server address then it start to broadcast the shader to other server adress, the shader appears right.
It seems like it can´t refresh the server once is created.
Is there any way to force server update? Im not an expert on node.js but it is supposed to auto-update once the shader is change right?
Thanks for reading.
Look forward to hearing from you.

Fixed in v2.11.4. Thank you for reporting! @Aurelian24 😸


Is it support group with valueGetter?

have you duplicated this issue:
https://github.com/ceolter/angular-grid/issues/114


Extra 'standardised' chain name field

Is your feature request related to a problem? Please describe.
Onboard, used in frontend for wallet interaction, has an archived dependency with hardcoded chain names: 'mainnet', 'ropsten', 'rinkeby', 'kovan' and 'goerli'. This has the knock-on effect that Onboard's networkName init option must match one of these names when the corresponding chain is being interacted with.
chainName.toLowerCase(), from the chains endpoint, is currently being used as the networkName (and also for the localStorage key) in frontend. With the change from 'Mainnet' to 'Ethereum', this originally broke the Onboard integration of 'mainnet' support from the above list. As a quickfix, we hardcoded 'mainnet' back in.
If we were to change a chainName in the future and it not correspond to the hardcoded names, it would break the Onboard integration again.
Describe the solution you'd like
The addition of a 'standardised' chain name field that we can reference in the Onboard configuration. It would be best to match the ethereum-lists/chains format as closely as possible.
Suggested added field names are as follows:

onboardName
network
uniqueNetworkSlug

Breaking change field names:

chainName => displayName, adding name/network

Describe alternatives you've considered
A local lookup for the hardcoded network names. This, however, goes against the desired mantra of remaining as agnostic as possible.

Decision:

Add chainNetwork that should correspond to the network field from https://github.com/ethereum-lists

@iamacook, did you ask onboard.js if they're looking to change that?

@mikheevm, it looks as they are moving towards accepting custom chain configuration in their next big version. If that's the case, we'd no longer require this in frontend.
I'll get in touch with them to confirm.

Decision:

Add chainNetwork that should correspond to the network field from https://github.com/ethereum-lists

@rmeissner It seems that the field was deprecated https://github.com/ethereum-lists/chains/commit/62ad98d60e7fba1ce66514bbb52c430b032eaf33

Blocknative got back to me with the following:

Regarding the network name, that is defined for each network within the configuration and Onboard no longer keeps an internal map of networks. So that should solve the issue you are having!

We could remain with our current implementation and when onboard v2 is released, migrate to that and remove our hotfix logic/enable hardware wallets on the networks that aren't in the currently hardcoded list.

Blocknative got back to me with the following:

Regarding the network name, that is defined for each network within the configuration and Onboard no longer keeps an internal map of networks. So that should solve the issue you are having!

We could remain with our current implementation and when onboard v2 is released, migrate to that and remove our hotfix logic/enable hardware wallets on the networks that aren't in the currently hardcoded list.

Which list/map will be used to validate the network name then?

Which list/map will be used to validate the network name then?

The next version doesn't require a network name but just a label but you make a valid point. We rely on the network name to save configuration data locally. Please disregard my previous comment.

The next version doesn't require a network name but just a label but you make a valid point. We rely on the network name to save configuration data locally. Please disregard my previous comment.

I wonder if the label that it is mentioned refers to the name field under https://github.com/ethereum-lists/chains/blob/master/_data/chains/eip155-1.json 🤔

@gnosis/safe-web If onboard.js just requires a chainId and the label is optional (can be used for UI purposed) then I'd close the issue as this is already provided by the Safe Client Gateway. What do you think?

@iamacook I may miss something but the last time I fixed the issue it wasn't onboard.js but ledger/trezor libraries that required a certain name to work

This is related to the current version of onboard as the network name is passed onto wallets. We are currently hardcoding 'mainnet' as it's currently 'Ethereum' on the CGW.
We also use network names in our localStorage keys to save information. However, it would definitely make more sense to use the network id there. We would first have to migrate everything though. @katspaugh, what do you think?

For LS keys, the numeric chainId totally makes sense, yes.

I still think having an id-like, but human-readable, chain name is useful going forward. As per Richard's comment above:

Add chainNetwork that should correspond to the network field from https://github.com/ethereum-lists

For LS keys, the numeric chainId totally makes sense, yes.

I have created an issue https://github.com/gnosis/safe-react/issues/3300 for the migration to use network ids instead.

I would say that the chain id should be used whenever possible. It doesn't need to be human readable. To have a human readable form we already have a name for that chain (which can change – eg.: xDai to Gnosis Chain).
Having a unique field tied to another unique field for the same object can raise more validation questions. 100 : xDAI vs 100 : Gnosis Chain.

I still think having an id-like, but human-readable, chain name is useful going forward. As per Richard's comment above:

Add chainNetwork that should correspond to the network field from https://github.com/ethereum-lists

This fields was deprecated: https://github.com/gnosis/safe-client-gateway/issues/747#issuecomment-1011055188

@gnosis/safe-web any update on this issue?

Update as in, whether you can close it? No, we still need it.

Update as in, whether you can close it? No, we still need it.

So what is the current status? Does onboard.js work with chain ids at the moment?

Onboard works, but the issue still remains with mainnet being hardcoded somewhere. We have also hardcoded a workaround for it though. Their upcoming version could fix this, but that is just an assumption as @mikheevm said it could even be HW wallet related. There is also no earmarked release date for it either.
Safe Apps have also encountered an issue with the renaming of xDai to Gnosis Chain that would likely benefit from this too. cc @dasanra

Their upcoming version could fix this, but that is just an assumption as @mikheevm said it could even be HW wallet related. There is also no earmarked release date for it either.

Ok so we don't know at the moment if this is an Hardware Wallet issue or, if it is, if there's an upcoming version of onboard.js to fix this.
Thanks for clarifying.
To reiterate on the following: the network field was removed from the ethereum list repo.
We already have a unique numeric field to identify the chain and we also have a display name that is not unique to be displayed no the interfaces.
I'd see value in adding a new unique field IFF there's no way around integration with required 3rd parties (ie.: a requirement of a non-standard unique field that identifies a chain besides its id).

We are currently coping with the issues we encountered, albeit with questionable code.
We do use the names for our localStorage keys but we intend to migrate those to using the chainId at some point. I currently cannot see any benefit in adding a new field for the time being.

Safe Apps have also encountered an issue with the renaming of xDai to Gnosis Chain that would likely benefit from this too. cc @dasanra

Safe Apps problem isn't related. We used to pass the chain name along with the Safe info but removed it in later versions. With the rename, the apps that relied on old names stopped working

I guess we can close this then. Wdyt @katspaugh?

Closing, thanks for the details 👍


React has detected a change in the order of Hooks called by EmotionCssPropInternal

At first run, without any change, when I navigate over one of the 3 section (About, Project or Writing) I will receive this error from Browser Console:
index.js:2177 Warning: React has detected a change in the order of Hooks called by EmotionCssPropInternal. This will lead to bugs and errors if not fixed. For more information, read the Rules of Hooks: https://fb.me/rules-of-hooks

   Previous render            Next render
   ------------------------------------------------------
1. useContext                 useContext
2. undefined                  useContext
   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

    in EmotionCssPropInternal (created by InView)
    in InView (created by Reveal)
    in Reveal (created by Slide)
    in Slide (at Section.tsx:39)
    in Header (at About.tsx:16)
    in div (created by Section__SectionContainer)
    in Section__SectionContainer (at Section.tsx:22)
    in section (created by Section)
    in Section (at Section.tsx:20)
    in Container (at About.tsx:15)
    in About (at pages/index.tsx:14)
    in ScrollingProvider (at Layout.tsx:38)
    in Ge (at Layout.tsx:36)
    in main (at Layout.tsx:35)
    in Layout (at pages/index.tsx:11)
    in IndexPage (created by HotExportedIndexPage)
    in AppContainer (created by HotExportedIndexPage)
    in HotExportedIndexPage (created by PageRenderer)
    in PageRenderer (at query-result-store.js:90)
    in PageQueryStore (at root.js:58)
    in RouteHandler (at root.js:80)
    in div (created by FocusHandlerImpl)
    in FocusHandlerImpl (created by Context.Consumer)
    in FocusHandler (created by RouterImpl)
    in RouterImpl (created by Context.Consumer)
    in Location (created by Context.Consumer)
    in Router (at root.js:75)
    in ScrollHandler (at root.js:71)
    in RouteUpdates (at root.js:70)
    in EnsureResources (at root.js:68)
    in LocationHandler (at root.js:126)
    in LocationProvider (created by Context.Consumer)
    in Location (at root.js:125)
    in Root (at root.js:134)
    in StaticQueryStore (at root.js:150)
    in ConditionalFastRefreshOverlay (at root.js:149)
    in _default (at app.js:163)

This is a known issue with one of the dependencies, react-awesome-reveal:
https://github.com/dennismorello/react-awesome-reveal/issues/57
...which in turn is due to one of its dependencies.

when changing to "react-awesome-reveal": "^3.4.0",  doesn't work.
Do I need to delete all animations?

Keep in mind that it's a warning and not an error. For me it's more of an annoyance than anything.

Hello, I would recommend keeping an eye on the issue and wait for the maintainers of the library to address the issue. I think downgrading it's not a good approach in this situation, because as @aaronsarnat said this is a development warning, and it won't make the app crash :)


collapsible cards for each email account, static loading, and vue enhanced

The initial list of email accounts that the user owns.
Implemented using bulma cards and bulma-collapsible.
If there is only one it should be is-active
Just display the email account data. Initial page content will be statically generated by django. Django will also supply json data for all the email accounts.
In the app.js for the page it will create the Vue app and replace all the static data with the data from the json embedded in the page.

completed a little while ago


Is it possible to modify "data"?

Is it possible to modify the data or answers object after prompts?
Here is the general idea:

A generator asks the user to select one of the Mongoose Schema models from the directory - this part works.
I require the schema, then parse out the schema paths - this also works.
I'd like to modify the data object that is passed to my jade templates so that I can do something like: for each schema type, add an input field to the view.

yes, this can be accomplished via an custom action function. this would allow you to execute logic that could add additional properties to the answers object before your other actions happen.

Many thanks - that did the trick!

is it possible to do it in the prompt ?
For example, I would like the user to be able to create a directory and the file names can either have the directory name or another one.
here is what I would like to do

ask for the directory path  -> "where do you want to place the component ? (ex:foo/bar)"
ask for the file names -> "What is the comoponent name ?"

and in the 2. I would like to be able to have a predefined value of "bar"

found it, you have to use default and you can have access to already defined answers.

good find 👍


Cherry pick fixes

prepare for a docs update - can you please confirm that these are all fixes to the existing release's docs, and that I've not missed one?
There are 3 PR's that I'm not sure about, but didn't look like current release:

#1665
#1695
#1721

@RichardScothern  @dmp42 @dmcgowan @thaJeztah

LGTM (but would like a check from the maintainers here)

If possible, I would like to have #1665 in the upcoming release of docker distribution so we can sync our internal usage of the registry with open source official releases.

@andrewhsu this PR is only to update the documentation with fixes that were merged in master

I see...how can I determine if/when the #1665 feature will go into an official release?

If it's merged in "master", it will be in the next release; releases are branched of the master branch

Commits are correct. #1721 is a correction and is also OK.
#1665, #1695 will be in the next release.

miss-reading @RichardScothern 's comment as a LGTM :)


small interface change

i cant accept this but i have better idea
you can change it to this
req.notify(exampleNotification()).to({ email<EMAIL_ADDRESS>})


url input not working

I installed a voiceping router and changed the ip address. When I drove this address in your application, it says that: Failed to enter, please try again later

@Timon2023 can you paste the URL you used? And perhaps also with the screenshots?

"When I drove this address in your application"
Does this mean you test your self-hosted voiceping router with https://voiceping-router-test.netlify.app/ ?

Through this address ws://<IP_ADDRESS>:3000 I can go to https://voiceping-router-test.netlify.app/ but I can’t go to this address through your application on android, although they are connected locally. Another question is how to change the server address on the server side? Instead of ws://<IP_ADDRESS>:3000 addresses do for example ws://x.x.x.x:3000 ?

Hi. I also have the issue with url.
I installed the voice router on WSL Ubuntu, propagated the port 3000 to the win host, opened the port 3000 on firewall. The web app https://voiceping-router-test.netlify.app/ works well with ws://localhost:3000, but does not work with the IP ws://192.168.1.xx:3000 - the application is not able to connect to the voice router. Same for VPS SDK Demo app for Android - the smartphone located in the same local network but not able to connect to the voice router. But the smartphone able to connect to the default page page http://192.168.1.xx:3000 - Welcome to VoicePing Router 1.0.0.
I guess the WebSocket should work through the http, so the TCP:3000 forwarding on Firewall should be enough. Can you suggest something?
Also, can you suggest how to enable encryption? Is it possible to enable https on nginx reverse proxy, and forward the decrypted traffic to the nodejs voice router app?
A lot of thanks.

can't you helpme?


Adding "reverse geocoding restricted by type" feature

As explained in the gmaps api doc https://developers.google.com/maps/documentation/geocoding/intro#reverse-restricted
reverse method need 2 extra parameters, result_type dictionary and location_type dictionary.
Should be a very quick fix
https://github.com/swistakm/python-gmaps/blob/master/src/gmaps/geocoding.py#L39

fixed by #8


[DO NOT DELETE OR MERGE] Nightly build

See pytorch/pytorch#26817 for what this does
FB only: Chronos job page (click "Run Now" to manually trigger a nightly)

Disabling nightly builds until this is fixed.

Re-opening this

Note: This job is being run adhoc behind the curtains and used to facilitate manual push


⚠ ClamAV has degraded performance

In d4dadb1, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 41 ms

Resolved: ClamAV performance has improved in b9cbf3d.


Swift 5 compatibility

Projects that make use of this pod aren't able to upgrade to swift 5 because URITemplate isn't swift 5 compatible.

+1

Please guys do you need help to achieve this ? (a PR seems to be open with a swift 5 compatible code)

Please fix this, I cannot run my unit tests.

https://github.com/kylef/URITemplate.swift/releases/tag/3.0.0


How to contribute to project?

Hello contributors. I'm a data enginneer who use spark hive and  pandas frequently. When I saw this project, I was very excited. Because it greatly simplifies my work. I want to contribute something to this project. Is there someone guided me how to contribute to the project? And the content of the contribution?

Welcome @yuwillrun.
The Contributing Guide should answer most of your questions. If you are looking for features to work on, you can start here. If you have found something, you should open an issue for your feature where you can discuss its feasibility with other contributors before actually starting to code.

Yup, closing this.

Ok, thanks your generous reply.


Frontend: add support for GATs

https://blog.rust-lang.org/2022/10/28/gats-stabilization.html

Related:

https://github.com/hacspec/hax/pull/970

Still relevant


arrow-cpp: fix for zstd 1.3.6+

Motivation for this change
Workaround facebook/zstd#1385
Things done

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

[ ] NixOS
[x] 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)
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build arrow-cpp python2Packages.pyarrow python3Packages.pyarrow

Success on aarch64-linux (full log)
Attempted: arrow-cpp
Partial log (click to expand)
these derivations will be built:
  /nix/store/mav58d6xlzg0588lwpa7pri7ra2ypps4-arrow-cpp-0.11.0.drv
waiting for locks or build slots...
/nix/store/ynnd3lx3fzfn5kza1rv8yx8ns6bpg4mf-arrow-cpp-0.11.0

Success on x86_64-linux (full log)
Attempted: arrow-cpp
Partial log (click to expand)
57: /build/apache-arrow-0.11.0/cpp/build/src/parquet/util
56/56 Test #57: parquet-memory-test ................   Passed    0.05 sec

100% tests passed, 0 tests failed out of 56

Label Time Summary:
parquet     =  14.70 sec*proc (18 tests)
unittest    = 110.12 sec*proc (56 tests)

Total Test time (real) = 110.15 sec

Failure on aarch64-linux (full log)
Attempted: arrow-cpp, python2Packages.pyarrow, python3Packages.pyarrow
Partial log (click to expand)
  /nix/store/v79j930zcqf9nygp01zl0zjb53mqa969-python2.7-pandas-0.23.4/lib/python2.7/site-packages/pandas/core/internals.py:312: FutureWarning: Using a non-tuple sequence for multidimensional indexing is deprecated; use `arr[tuple(seq)]` instead of `arr[seq]`. In the future this will be interpreted as an array index, `arr[np.array(seq)]`, which will result either in an error or a different result.
    return self.values[slicer]
  /nix/store/v79j930zcqf9nygp01zl0zjb53mqa969-python2.7-pandas-0.23.4/lib/python2.7/site-packages/pandas/core/indexes/datetimelike.py:428: FutureWarning: Using a non-tuple sequence for multidimensional indexing is deprecated; use `arr[tuple(seq)]` instead of `arr[seq]`. In the future this will be interpreted as an array index, `arr[np.array(seq)]`, which will result either in an error or a different result.
    result = getitem(key)

-- Docs: https://docs.pytest.org/en/latest/warnings.html
 5 failed, 16896 passed, 1030 skipped, 11887 deselected, 37 xfailed, 2 xpassed, 6 warnings in 593.95 seconds
builder for '/nix/store/sfgg7h8zc274x7n3c4v97bjgsv4djrbz-python2.7-pandas-0.23.4.drv' failed with exit code 1
cannot build derivation '/nix/store/ag148r8bd68b3913285z0s2pny1xlafg-python2.7-pyarrow-0.11.0.drv': 1 dependencies couldn't be built
error: build of '/nix/store/0b1ld60bs98i9pk6dn9r0hicknbxwr0b-python3.6-pyarrow-0.11.0.drv', '/nix/store/ag148r8bd68b3913285z0s2pny1xlafg-python2.7-pyarrow-0.11.0.drv' failed

Success on x86_64-darwin (full log)
Attempted: arrow-cpp
Partial log (click to expand)
57: /private/tmp/nix-build-arrow-cpp-0.11.0.drv-0/apache-arrow-0.11.0/cpp/build/src/parquet/util
56/56 Test #57: parquet-memory-test ................   Passed    0.14 sec

100% tests passed, 0 tests failed out of 56

Label Time Summary:
parquet     = 111.44 sec*proc (18 tests)
unittest    = 884.66 sec*proc (56 tests)

Total Test time (real) = 899.70 sec

Success on x86_64-linux (full log)
Attempted: arrow-cpp, python2Packages.pyarrow, python3Packages.pyarrow
Partial log (click to expand)
Moving generated C++ source _parquet.cpp to build path /build/apache-arrow-0.11.0/python/pyarrow/_parquet.cpp
Moving built C-extension release/_parquet.cpython-36m-x86_64-linux-gnu.so to build path /build/apache-arrow-0.11.0/python/pyarrow/_parquet.cpython-36m-x86_64-linux-gnu.so
release/_orc.cpython-36m-x86_64-linux-gnu.so
Cython module _orc failure permitted
release/_plasma.cpython-36m-x86_64-linux-gnu.so
Cython module _plasma failure permitted
pytestcachePhase
/nix/store/jwmryl6i1qrksnizml0jx7ldxxy59j17-arrow-cpp-0.11.0
/nix/store/xm2qlasz4gl2y0bn0m0rzbd4v0qri2am-python2.7-pyarrow-0.11.0
/nix/store/zz6cs2cnnfyw452qqrkcppkvcvp45446-python3.6-pyarrow-0.11.0

Failure on x86_64-darwin (full log)
Attempted: arrow-cpp, python2Packages.pyarrow, python3Packages.pyarrow
Partial log (click to expand)
    suites = [self.loadTestsFromName(name, module) for name in names]
  File "/nix/store/0mha803yql9q7b6h93m8940z3sjbhiy8-python3-3.6.6/lib/python3.6/unittest/loader.py", line 219, in <listcomp>
    suites = [self.loadTestsFromName(name, module) for name in names]
  File "/nix/store/0mha803yql9q7b6h93m8940z3sjbhiy8-python3-3.6.6/lib/python3.6/unittest/loader.py", line 153, in loadTestsFromName
    module = __import__(module_name)
  File "/private/tmp/nix-build-python3.6-pyarrow-0.11.0.drv-0/apache-arrow-0.11.0/python/pyarrow/__init__.py", line 45, in <module>
    import pyarrow.compat as compat
AttributeError: module 'pyarrow' has no attribute 'compat'
builder for '/nix/store/z5zqhsq3x866w83awxynrpvpvcx2gabm-python3.6-pyarrow-0.11.0.drv' failed with exit code 1
error: build of '/nix/store/z5zqhsq3x866w83awxynrpvpvcx2gabm-python3.6-pyarrow-0.11.0.drv', '/nix/store/z9g3spxg9vvgn69lbn2lpxixf2gz1z4c-python2.7-pyarrow-0.11.0.drv' failed

LGTM. The pyarrow failures exist without this PR.

@veprbl In the future could you also report such issues upstream? Thanks in advance.


Could not find or load main class baston.vertigoboost.Main

Error: Could not find or load main class baston.vertigoboost.Main
Caused by: java.lang.NoClassDefFoundError: javafx.application.Application

JavaFX was deprecated as of Java 9, use Java 8


Mounting host's root file system inside container security issue

https://github.com/k8snetworkplumbingwg/sriov-network-operator/blob/bc2291ef9b2f2629986c2cd0368bd611dbed40fa/bindata/manifests/daemon/daemonset.yaml#L99
This YAML is configuring an SRIOV container to mount the hosts's entire root file system inside the container.  This is against basic containerisation best practices and can lead to escape and privilege escalation.  Such misconfigurations have been exploited in a number of well known security incidents (Doki, Hildegard, Peirates, Siloscape and TeamTNT).
The solution would be to not mount the entire host root file system in the container and only mount the sub directories specifically required, basically the principle of least privilege.
Consider using the strace Linux utility to help identify the specific files and directories are are needed by your processes, and then mount only the most granular required.  Also, avoid shifting the mount point down one level to one of the root's directories.  I.e., do not blanket mount /dev, /proc, run, etc. or any 1st level directory.
I would consider a CVE for this.

Hi @huornlmj thanks for the update be aware that we mount this as we need the all file system in the right order so we can run chroot from inside the container

Can you explain what is being chrooted?  Is it the entire host fs?

yes so we can run systemd and other host configuration
take a look on in the code for chroot function and see what we do after that command I think it will be the best to understand if we can do that in a different way :)

Why chroot inside a container though and not just mount what is needed?  I would ask the question as to why the host's docker socket is mounted inside the container?  As well as the hosts' /etc/shadow file, the host's setuid executable binaries (unmount, mount etc.) a full bash shell environment, the hosts' /tmp/ world-writable directory etc..  I'm naming just a small few items that are used by attackers to escape from a container and compromise the host.  The principle of granting nothing and only adding what is needed is not being followed here.

Hello,
Allow me to take the opportunity at addressing the concerns
raised here, specifically regarding the use of hostPath. First off,
thanks for raising this issue. However, it's crucial to make a clear
distinction here: we view this issue as a design limitation which in
hindsight we more clearly recognize as being a flaw.
A design flaw refers to architectural decisions that might come with
disadvantages or potential for improvement. In contrast, a vulnerability
implies the existence of an exploitable issue. The use of hostPath in
our design was a conscious decision, and while it has its limitations,
it does not provide a direct path for exploitation. Hence, it may not be
accurately represented by CVSS scores, which are typically used to
quantify vulnerabilities, not design choices.
It's worth mentioning that design flaws like this one could lead to
potential exploits being less bounded. We understand the associated
risks and agree that improvements could be made. However, given the
immediate needs at hand, we are prioritizing more urgent matters and
plan to address this sort of clean-up in a future release during a
longer timeframe.
To the Intel team and other community members, we warmly welcome your expertise
and collaboration in addressing this issue. Your contributions and
pull requests could significantly enhance the security of our platform.
We greatly appreciate your input and look forward to constructive
discussions in this space.
BR,
Yuval


Force Torque data stored in imu vector

This looks like a copy-paste mistake. force torque data are store in the imu_sensor_data_ when sim_ft_sensors_ exists.
https://github.com/ros-controls/gazebo_ros2_control/blob/master/gazebo_ros2_control/src/gazebo_system.cpp#L560
I can provide a PR unless there is a reason for this.

Yep, this look like an error. Happy to review the PR

Hello @ahcorde and @christophfroehlich. I would like to work on this issue


Make Import Wallet screen's buttons and text at the bottom move up and down when keyboard is show/hidden

4 tabs (Seed, Keystore JSON, etc)
Remember to test shorter devices. It might not fit for 5s. If it doesn't, just do a check with ScreenChecker() and retain the current behavior. Hopefully works for 6s?

@hboon these seems like done #1993, #1990, #1989

@hboon when u will have time could u please check #2048, #1993, #1990, #1989, #2079, 1849, #2048 if we can close them? and after that i will do rebase?

@vladyslav-iosdev on it.


As Long As You Love Me (feat. Emelie Cyréus) by Chad Cooper unsupported.

Check if issue with swaglyrics or whether song lyrics unavailable on Genius.

stripper -> Chad-Cooper-As-Long-As-You-Love-Me

https://genius.com/Chad-cooper-and-robaer-as-long-as-you-love-me-lyrics


Grand project rename

Rename the project from senior-project-experiment to beeswax.

@pixel5919 Could you review this?

FYI the build status button on the README is broken and will remain that way until it is merged.


virtual-chinrest does not support large viewing distances

Hello! We have integrated the jspsych-virtual-chinrest plugin, and it works nicely with good UX for most common cases. However when the viewing distance is a bit larger (e.g. 80cm) the little red ball does not go left-enough to allow it to traverse the blind spot. (I realize this is a shortcoming of the underlying virtual-chinrest library as well.)
It would be great if the plugin could support these larger distances - or if there is a maximum distance it is able to support, if it could provide instead a minimal UX to invite users to get closer to their screen.

Thanks, this is a great point. I agree that, if there's no response during the first blindspot detection attempt, ideally the plugin should include the option to show a message about getting closer to the screen, and maybe repeat the procedure. If we did it that way, I think we'd need to add parameters for the duration of each blindspot attempt and the number of times that the procedure should be repeated.


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target_921ad5c5-3629-45bd-ba98-3504d6e1ae54/pull/1 is merged into live branch.
Auto archive content to https://github.com/OPS-E2E-PPE/docs-archive-test-target_921ad5c5-3629-45bd-ba98-3504d6e1ae54.git

Docs Build status updates of commit 8fd0dfb:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error]

Cannot sync git repo to specified commit because branch Archive_master_2019-10-22-03-03-21 has been deleted or has been force pushed: fatal: Couldn't find remote ref Archive_master_2019-10-22-03-03-21

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Repeated and constant releasing of unexpected Roomba instances

Current Behavior
Lots of unexpected roomba instances clogging logs. Continues occurring for both of my Roombas and my Braava
Versions

Homebridge: 1.4.0
Node:       16.14.0
Plugin:       1.3.1

Homebridge Log / Command Output
[15/03/2022, 11:28:00] [Braava Jet m6] Releasing an unexpected Roomba instance
[15/03/2022, 11:29:01] [Braava Jet m6] Releasing an unexpected Roomba instance
[15/03/2022, 11:30:02] [Braava Jet m6] Releasing an unexpected Roomba instance
[15/03/2022, 11:31:04] [Braava Jet m6] Releasing an unexpected Roomba instance
[15/03/2022, 11:32:06] [Braava Jet m6] Releasing an unexpected Roomba instance
[15/03/2022, 11:32:49] [Roomba i7+] Releasing an unexpected Roomba instance
[15/03/2022, 11:33:08] [Braava Jet m6] Releasing an unexpected Roomba instance

Same issue here, but only on Roomba m6

Same issue on 3 Devices (m6, i7, 980)

I'm having the same issue as well. I have also noticed that the green light on the Roomba turns off for a few seconds and then back on around the time this error occurs.

Same here on my 960. It worked flawlessly before. Now it won't start on first turn on. Have to do it twice.  Thank you

Same here with Braava M6.

@arminf82 please take a look here: https://github.com/karlvr/homebridge-roomba2/issues/119
both for a possible fix and honestly homebridge is history home-assistant is much better, robust and compatible.

@jdgorelick do you have anything else trying to connect to your Roomba(s)? Connection Timeouts generally mean either the IP is wrong, or something else is holding the connection to your Roomba.

I am currently working to reduce the amount of logging from the plug-in. Tracked under #113


Sounds

More default sounds to give users a choice. Because not many people would like to search for a sound which is good for them and upload it to the app. A wider selection makes things easier.

I agree. I want Focus Keeper to have a rich default library, but to achieve this, I need to find good sounds under an open license, purchase them, or rely on community members to share their sounds under an open license.
I'm going to close this issue since it's not tech-related. As the community grows, we'll find a way to make the app sound great.


merge aggressive digital papillary adenocarcinoma

close #7185

redid this PR: https://github.com/monarch-initiative/mondo/pull/7450


bump cargo-project dep

with this change cargo-call-stack will pick up .cargo/config.toml (with extension)

bors r+


[speech-to-text]  error: { [Error: not found: git] code: 'ENOGIT'}

I have downloaded and tried to "npm install" it's failing with message,
error: { [Error: not found: git] code: 'ENOGIT'}
Do I need to Git 2.8.1 installer for windows 7?
Below error in command line is given,

It has been resolved by doing "npm install -g<EMAIL_ADDRESS>But after that when I do "npm run build" it throws error :
npm ERR! Windows_NT 6.1.7601
npm ERR! argv "C:\Program Files\nodejs\node.exe" "C:\Program Files\nodejs\
node_modules\npm\bin\npm-cli.js" "run" "build"
npm ERR! node v5.10.1
npm ERR! npm  v3.8.3
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>build: npm run build
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build script 'npm run
build'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the SpeechToTextBrowserSt
arterApp package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     npm run build
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs SpeechToTextBrowserStarterApp
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls SpeechToTextBrowserStarterApp
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     C:\Prasanta\Official\IBM\IDEA\NBA\speech-to-text-nodejs-master\npm-
debug.log
Log is attached herewith.
npm-debug.txt

Hi @prasanta303
Can you start over fresh (delete the node_modules directory and npm-debug.log, run npm install) and then give me the complete logs from the command line and the new npm-debug.log if it gets recreated? (The second error overwrote the log for the first one, which is what I need to see.)

The issue is now resolved. So closing it.
After doing as you said, did npm start and from browser, opened http://localhost:3000/ and it runs properly.


the min version of the js file isn't the same as the non minified version  <- is more recent n/t

Yeah, the min version doesn't support js:Promise...


Orderby Issues

I'm trying to sort a set of results along with pagination i am programming.  seems like there is an issue with using  "where.startsWithIgnoreCase.fofset.limit"
this.gallons = await this.$store.state.db.gallons.where('post_title').startsWithIgnoreCase(this.pagination.searchText.trim()).offset(((newVal.page-1)*newVal.per_page)).limit(newVal.per_page).toArray();
I've tried every combination I could to order by the post title, it just says orderBy is not a function.

That might need a feature coming in 4.0 https://dexie.org/roadmap/dexie4.0#richer-queries


Diseasedatasources

Please see the additions in datasource.txt for the disease databases; would like these to be part of the new release.

already applied to the series2.3 branch.


chat is empty after upgrading react version to ^17.0.2

Issue Description

chat is empty after upgrading react version to 17.0.2

Steps to Reproduce / Code Snippets

download this example project https://github.com/johniak/react-web-gifted-chat/tree/master/example
upgrade react app version to ^17.0.2

Expected Results

should be able to see chat

Additional Information

Nodejs version: v15.14.0
React version: 17.0.2
react-web-gifted-chat version: 0.6.7
Browser(s) (Chrome, etc.): Chrome

Same issue

Out of curiosity, what version of react were you upgrading from? Was just giving this a spin myself.

@kaolin 16 -> 17

Same issue I'm facing,  any update?

Hello guys did you manage to find a solution for this ? I have the exact same issue and I am using react 18.2


fix for smtplib multiple recipients

fixes #19
According to smtplib documents
SMTP.sendmail(from_addr, to_addrs, msg[, mail_options, rcpt_options])
Send mail. The required arguments are an RFC 822 from-address string, a list of RFC 822 to-address strings (a bare string will be treated as a list with 1 address), and a message string. The caller may pass a list of ESMTP options (such as 8bitmime) to be used in MAIL FROM commands as mail_options. ESMTP options (such as DSN commands) that should be used with all RCPT commands can be passed as rcpt_options. (If you need to use different ESMTP options to different recipients you have to use the low-level methods such as mail(), rcpt() and data() to send the message.)
smtp_recipients needs to be a list, but the To header must be a string.  Tested working on my install with this PR.

Coverage remained the same at 100.0% when pulling 7cc5796264bb2ab4de6d1c5683e2db20c5cf27b8 on mbassett:master into 0286e391f465a34399cc8358508bf9f6de80bdfa on TerbiumLabs:master.


canfind

Hi, I'm running node 6.9.1 and having quite a lot of problems with it.
First, let me tell that gulp, gulp-babel, babel-core dependencies are missing from package.json.
After you install it can configure your connections.js file to use this adapter, sails lift gives you the following error.

info: Could not find module: ejs in path: /Users/a212472385/Desktop/sky/datacollect/node_modules
error: A hook (orm) failed to load!
error: Error: There was an error attempting to load waterline-sqlite3.
(Sails attempted to load this adapter because it is referenced by a datastore (sqlitedb) in this app's configuration)
Please ensure that waterline-sqlite3 is a valid Sails/Waterline adapter, and that it is installed in this app.
Error details:
at constructError (/usr/local/lib/node_modules/sails/node_modules/sails-hook-orm/lib/construct-error.js:57:13)
at loadAdapterFromAppDependencies (/usr/local/lib/node_modules/sails/node_modules/sails-hook-orm/lib/load-adapter-from-app-dependencies.js:67:13)
at _eachRelevantDatastoreIdentity (/usr/local/lib/node_modules/sails/node_modules/sails-hook-orm/lib/initialize.js:141:56)
at arrayEach (/usr/local/lib/node_modules/sails/node_modules/lodash/index.js:1289:13)
at Function.<anonymous> (/usr/local/lib/node_modules/sails/node_modules/lodash/index.js:3345:13)
at Array.async.auto._attemptToLoadUnrecognizedAdapters (/usr/local/lib/node_modules/sails/node_modules/sails-hook-orm/lib/initialize.js:130:11)
at listener (/usr/local/lib/node_modules/sails/node_modules/sails-hook-orm/node_modules/async/lib/async.js:605:42)
at /usr/local/lib/node_modules/sails/node_modules/sails-hook-orm/node_modules/async/lib/async.js:544:17
at _arrayEach (/usr/local/lib/node_modules/sails/node_modules/sails-hook-orm/node_modules/async/lib/async.js:85:13)
at Immediate.taskComplete (/usr/local/lib/node_modules/sails/node_modules/sails-hook-orm/node_modules/async/lib/async.js:543:13)
at runCallback (timers.js:637:20)
at tryOnImmediate (timers.js:610:5)
at processImmediate [as _immediateCallback] (timers.js:582:5)

I don't know if it's indeed the name of the adapter or something else. But currently I can not get this working and need your help.
Best,
Onat

I had this same problem, if fixed by changing the connection configurations file under config/connections.js and i just put
sqlitedb: {       adapter: 'sqlite3',       filename: './dev-db.sqlite',       debug: false     }
That seem's to work, but im using knexJs to handle the migrations so i think something brake because when i lift my app and go to postman to test it, only shows me
{     "code": "E_UNKNOWN",     "message": "Encountered an unexpected error",     "data": {         "raw": {}     } }
But i think, if your are not using knexJs that will work.


Remove the specific commands as 2nd level headers.

Also fixing broken anchor references

Thanks for catching that! 327538d


Upgrade dependencies

Upgrade all dependencies in package.json
Upgrade tests to Jest 0.19.x
Upgrade to Koa 2 (release version - requires Node 7.6+).
Upgrade flow type defs

Coverage increased (+4.9%) to 82.301% when pulling fcb96ad6985d6d5281c4b3ebd168d95e7ddad374 on upgrade-dependencies into 9301d5caa29cb411368d7dcb6b7bbf92a9f138e7 on master.

Coverage increased (+4.9%) to 82.301% when pulling 4f30165e107907f22e899dc6ff956555d4332bb3 on upgrade-dependencies into 9301d5caa29cb411368d7dcb6b7bbf92a9f138e7 on master.

Coverage increased (+1.8%) to 79.202% when pulling 3571b35191edeb81cc17b20bbc9e4099d2ddc6fd on upgrade-dependencies into 9301d5caa29cb411368d7dcb6b7bbf92a9f138e7 on master.

Coverage increased (+1.8%) to 79.202% when pulling d89cb6b65e897c43e997f3b205f2767e23bdf3bf on upgrade-dependencies into 9301d5caa29cb411368d7dcb6b7bbf92a9f138e7 on master.


[SPARK-22878] [CORE] Count totalDroppedEvents for LiveListenerBus

What changes were proposed in this pull request?
Count total dropped events from all queues' numDroppedEvents for LiveListenerBus and log the dropped events periodically for LiveListenerBus like AsyncEventQueue do.
How was this patch tested?
Added.

Can one of the admins verify this patch?

cc: @maropu

@gatorsmile @HyukjinKwon @dongjoon-hyun Can any one of you have a look at this pr, Thanks!

ok to test

Test build #85407 has started for PR 20056 at commit cd395dc.

Test build #85407 has finished for PR 20056 at commit cd395dc.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

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

Merged build finished. Test FAILed.

please retest it. Thanks.

Test build #85408 has started for PR 20056 at commit 4fef226.

Test build #85408 has finished for PR 20056 at commit 4fef226.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Just fix and push a commit. It will retest it.

Test build #85415 has started for PR 20056 at commit 34cfb46.

It seems something wrong while testing 'ReplayListenerSuite':
[info] ReplayListenerSuite:
[info] - Simple replay (97 milliseconds)
[info] - Replay compressed inprogress log file succeeding on partial read (48 milliseconds)
[info] - Replay incompatible event log (46 milliseconds)
[info] - End-to-end replay (3 seconds, 754 milliseconds)
Exception in thread "dispatcher-event-loop-18" java.lang.OutOfMemoryError: GC overhead limit exceeded
Exception in thread "dispatcher-event-loop-2" java.lang.OutOfMemoryError: GC overhead limit exceeded
Exception in thread "dispatcher-event-loop-1" java.lang.OutOfMemoryError: GC overhead limit exceeded
Exception in thread "ScalaTest-dispatcher" java.lang.OutOfMemoryError: GC overhead limit exceeded
Internal error when running tests: java.lang.OutOfMemoryError: GC overhead limit exceeded
Exception in thread "Thread-83" java.io.EOFException
	at java.io.ObjectInputStream$BlockDataInputStream.peekByte(ObjectInputStream.java:2608)
	at java.io.ObjectInputStream.readObject0(ObjectInputStream.java:1319)
	at java.io.ObjectInputStream.defaultReadFields(ObjectInputStream.java:2000)
	at java.io.ObjectInputStream.readSerialData(ObjectInputStream.java:1924)
	at java.io.ObjectInputStream.readOrdinaryObject(ObjectInputStream.java:1801)
	at java.io.ObjectInputStream.readObject0(ObjectInputStream.java:1351)
	at java.io.ObjectInputStream.defaultReadFields(ObjectInputStream.java:2000)
	at java.io.ObjectInputStream.readSerialData(ObjectInputStream.java:1924)
	at java.io.ObjectInputStream.readOrdinaryObject(ObjectInputStream.java:1801)
	at java.io.ObjectInputStream.readObject0(ObjectInputStream.java:1351)
	at java.io.ObjectInputStream.defaultReadFields(ObjectInputStream.java:2000)
	at java.io.ObjectInputStream.readSerialData(ObjectInputStream.java:1924)
	at java.io.ObjectInputStream.readOrdinaryObject(ObjectInputStream.java:1801)
	at java.io.ObjectInputStream.readObject0(ObjectInputStream.java:1351)
	at java.io.ObjectInputStream.readObject(ObjectInputStream.java:371)
	at org.scalatest.tools.Framework$ScalaTestRunner$Skeleton$1$React.react(Framework.scala:809)
	at org.scalatest.tools.Framework$ScalaTestRunner$Skeleton$1.run(Framework.scala:798)
	at java.lang.Thread.run(Thread.java:745)

And build process is blocking here for more than two hours.
Would you please have a look at this problem? @HyukjinKwon

This test passed in your local env?

@maropu Actually, I didn't modify this unit test ever. And my unit test locate in SparkListenerSuite haven'been started according to the "Console Output".

Test build #85415 has finished for PR 20056 at commit 34cfb46.

This patch fails from timeout after a configured wait of `250m`.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Probably, you better pass all the tests by yourself to save committers' bandwidth.

i faced an issue about LiveListenerBus.
We start HiveThirftServer and execute some sql with beeline. we found some job hung for the reasone "Dropping SparkListenerEvent ...".
So we increase the spark.scheduler.listenerbus.eventqueue.size form 10000 to 1000000. After that, the job was not hung but we found the job still in processing very slowly which displayed in WebUI even though all tasks has finished. Thanks!
@Ngone51 @cloud-fan

Hi, @zuotingbing , can you file a JIRA ticket for the issue and given more details please ? We can discuss more at there.

Hi @Ngone51 ,  this issue ( Add separate listener queues to LiveListenerBus) seems has fixed it. Thank you all the same.


TextView with title cannot be edited

I realized that if I add a title to the TextView it doesn't work - I cannot edit the content of it.
Example:
VStack {
    TextView(text: $textContent) // works fine
    TextView("write here", text: $textContent) // doesn't work
}

This is now fixed courtesy of @levous!


Deploy contracts release 4 to staging

Closed by
txHash: 0x5d0fbe737164a220d98fd4333872e6b014e97e6dad049d450ef09b15b63d0ae4
Sending Transaction: proposeTx... done
ProposalQueued:
proposalId: 11
proposer: 0x4e3D385eCdEE402DA395a3b18575B05Cc5e8Ff21
transactionCount: 5
deposit:<PHONE_NUMBER>00000000000
timestamp:<PHONE_NUMBER>


Redirect TIMIT download from LDC

LDC data is protected under US copyright laws and under various legal agreements between the Linguistic Data Consortium/the University of Pennsylvania and data providers which prohibit redistribution of that data by anyone other than LDC. Similarly, LDC's membership agreements, non-member user agreement and various corpus-specific license agreements specifically state that users cannot publish, retransmit, disclose, copy, reproduce or redistribute LDC databases to others outside their organizations.
LDC explicitly asked us to remove the download script for the TIMIT dataset. In this PR I remove all means to download the dataset, and redirect users to download the data from https://catalog.ldc.upenn.edu/LDC93S1

CI is failing because some tags are outdated, but they're fixed in #4067

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

We may do a release pretty soon (today ?), let me know if it's fine to include it in the new release

Fine to include this change!


[WIP] feat: Move ADRs to this repo

This PR is already fixing too many things. But since we are also changing the content I also have some things I want to change


fix: remove hardcode cloudfront link

Fixing PR https://github.com/2023M8T2-Inteli/grupo4/issues/168
Removed cloudfront hardcode links

N/A


Release Google.Cloud.Security.PublicCA.V1Beta1 version 1.0.0-beta04

Changes in this release:
New features

Change netstandard2.1 target to netstandard2.0 (commit 82bea85)

Created release for Google.Cloud.Security.PublicCA.V1Beta1-1.0.0-beta04


SASS to CSS

Plans to add the SASS support anyday soon? Noticed the item in the "backlog".

Would need this too!


Separate output file for each source

I think it would be nice that Sourcery will create separate output files for each source wenn the output parameter is a folder.
Example
sourcery source/ templates/ output/

├─ source
│   ├── Person.swift
│   └── Car.swift
├─ templates
│   ├── Coding.stencil
│   └── Description.stencil
└─ output
    ├── Person+Coding.swift
    ├── Person+Description.swift
    ├── Car+Coding.swift
    └── Car+Description.swift

The reason we either generate a single file, or file per template is that if we are going to support the proposed format it will require more user maintenance, when you want to add another type to conform to automatic generation you'll need to remember to manually add new generate file to project.

I think it's a good thing to have too. Noticed that while working on LinuxMain template. Linked tests suites have separate manifest files for each test module, also generated. We instead can only put everything in one file. It's not a big deal of course as you don't have to maintain generated code much, but still it hurts code readability especially for large code bases.
The problem with creating smth like Person+Coding.swift is that we don't have guaranty that each scanned file has just a single type, so what name we should use?
When I was thinking about that I was thinking about using some comments, similar to annotations in the template itself, like :
// output: {{ file.path }}/{{ file.name }}+Coding.generated.swift - will tell to create file next to input file in "single-file" mode
// output: {{ output.path}}/Coding/{{ file.name }}+Coding.generated.swift - will tell to create file in output/Coding/ path
(the same for Swift templates just with different syntax)
Also this will require something similar as #45 - to provide list of types defined in a current file.

@ilyapuchka : The problem with creating smth like Person+Coding.swift is that we don't have guaranty that each scanned file has just a single type, so what name we should use?

One possibility is to create separate files based on the source.
An alternative way can be to create separate files for each type.
I think based on the source will be make more sense, because it is the structure the developer has chosen.

if anything then we'll do it per (original definition file, because a type can be split across multiple files) file, it will already cause us to have 3 different output modes (single file, per template, per file x per template).
@ilyapuchka I don't think we should complicate templates with options like that, at least until we reach 1.0

How about an PathResolver protocol where you ship a DefaultPathResolver and then it is up to the user to create a specialized class for a different pattern / output modes

Any news about the generation per source file or that option has been forgotten ?

@muescha I'd be game if you wanted to generalize this in some kind of form like that so that maintenance is not hard


Getting an intermittent error while using SDK's RetryHydrate

I am working on a table where I want to retry the list hydrate function for certain errors. Hence, I have defined the RetryConfig in table's ListConfig
List: &plugin.ListConfig{
	Hydrate: tableDNSRecordList,
	RetryConfig: &plugin.RetryConfig{
		ShouldRetryErrorFunc: shouldRetryError(),
	},
	KeyColumns: plugin.KeyColumnSlice{
		{Name: "domain", Require: plugin.Required, Operators: []string{"="}},
		{Name: "type", Require: plugin.Optional, Operators: []string{"="}},
		{Name: "dns_server", Require: plugin.Optional, Operators: []string{"="}, CacheMatch: "exact"},
	},
},

func shouldRetryError() plugin.ErrorPredicateWithContext {
	return func(ctx context.Context, d *plugin.QueryData, h *plugin.HydrateData, err error) bool {
		if err != nil {
			panic(d.Table.Name)
			return true
		}
		return false
	}
}

Note: For the test, it returns true for any error
While running the below query (see attachment) multiple times
test.sql.zip

For a successful run

➜  test cat test.sql | steampipe query
+---------------+--------+--------------------------------------------------+
| resource      | status | reason                                           |
+---------------+--------+--------------------------------------------------+
| github.com    | ok     | github.com name servers have no CNAME record.    |
| microsoft.com | ok     | microsoft.com name servers have no CNAME record. |
+---------------+--------+--------------------------------------------------+

Time: 1915.2ms. Rows fetched: 53. Hydrate calls: 0.

If an error occurs (returns panic, since I added the panic in shouldRetryError function)

➜  test cat test.sql | steampipe query
Warning: executeQueries: query 1 of 1 failed: ERROR: rpc error: code = Internal desc = list call tableDNSRecordList failed with panic net_dns_record (SQLSTATE HV000)

For the same query, sometimes it returns the error (usually it should return the panic that I have added)

➜  test cat test.sql | steampipe query
Warning: executeQueries: query 1 of 1 failed: ERROR: rpc error: code = Unknown desc = read udp <IP_ADDRESS>:54954-><IP_ADDRESS>:53: i/o timeout (SQLSTATE HV000)

Steps to reproduce:

Use branch test-retry in Net plugin
Clone Net Insights repo, and run steampipe check benchmark.dns_best_practices

Here is the SQL query that I'm using
with domain_list as (
    select distinct domain from net_dns_record where domain in ('github.com', 'microsoft.com') order by domain
  ),
  domain_ns_records as (
    select domain, type, target from net_dns_record where domain in (select domain from domain_list) and type = 'NS' order by domain
  ),
  ns_ips as (
    select domain, ip, type, target, host(ip) as ip_text from net_dns_record where domain in (select target from domain_ns_records) and type = 'A' order by domain
  ),
  ns_record_with_ip as (
    select
      domain_ns_records.domain,
      domain_ns_records.target as name_server,
      host(ns_ips.ip) as ip_text
    from
      domain_ns_records
      left join ns_ips on domain_ns_records.target = ns_ips.domain
    where
      domain_ns_records.type = 'NS'
      and ns_ips.ip is not null
    order by domain_ns_records.target
  ),
  ns_record_with_record_count_stats as (
    select
      domain,
      name_server,
      (select count(*) from net_dns_record where domain = ns_record_with_ip.domain and dns_server = ns_record_with_ip.ip_text and type = 'CNAME') as cname_record_count,
      (select count(*) from net_dns_record where domain = ns_record_with_ip.domain and dns_server = ns_record_with_ip.ip_text and type not in ('CNAME')) as non_cname_record_count
    from
      ns_record_with_ip
  ),
  ns_record_with_cname_other as (
    select distinct domain from ns_record_with_record_count_stats where cname_record_count > 0 and non_cname_record_count > 0 order by domain
  )
  select
    domain_list.domain as resource,
    case
      when ns_record_with_cname_other is null then 'ok'
      else 'alarm'
    end as status,
    case
      when ns_record_with_cname_other is null then domain_list.domain || ' name servers have no CNAME record.'
      else domain_list.domain || ' name servers have CNAME record along with NS (or any other) records.'
    end as reason
  from
    domain_list
    left join ns_record_with_cname_other on domain_list.domain = ns_record_with_cname_other.domain;


Use of kotlin.Result in JetNews

I was looking at the code of JetNews and I saw the use of a custom class Result instead of the native solution kotlin.Result.
https://github.com/android/compose-samples/blob/7132d1483676d07874f56d2bd18bd07d3212dd84/JetNews/app/src/main/java/com/example/jetnews/data/Result.kt#L22-L25
From my understanding, it is always better to use native solution (faster performance/support). Plus, the
kotlin.Result comes with a lot of cool function like runCatching.
Why do you prefer using this custom class ? Is the official kotlin.Result bugged and not recommended to use in an Android project ?
It would be really interresting to have guidance on it in the documentation, if the use of kotlin.Result is not recommanded.
Like in this article https://developer.android.com/jetpack/guide/ui-layer/events#other-use-cases, where it is not recommanded to use Kotlin Channels to send actions from the ViewModel to the view.
Thanks a lot.

The shortest answer is that I believe that code was written well before kotlin.Result was stable, so I wouldn't read into that as a recommendation against it. If you like how kotlin.Result works, go ahead and use it! I will point out one "gotcha" with runCatching since you mentioned it, which is that it will catch CancellationExceptions and therefore can mess up structured concurrency if you're not careful: https://github.com/Kotlin/kotlinx.coroutines/issues/1814


feat: pipeline管理工具集成 --story=119892878

 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.cheynechen 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.


Not working in .NET Core v2.0 project

I'm trying to create a .zip file using MSBuild following this example in a .csproj targetting netcoreapp2.0.
Since MSBuild on .NET Core does not support CodeTaskFactory, I tried modifying the task by using RoslynCodeTaskFactory in the following way:
<UsingTask TaskName="Zip" TaskFactory="CodeTaskFactory" AssemblyFile="$(RoslynCodeTaskFactory)">
    <ParameterGroup>
      <OutputFilename ParameterType="System.String" Required="true" />
      <Files ParameterType="Microsoft.Build.Framework.ITaskItem[]" Required="true" />
    </ParameterGroup>
    <Task>
      <Reference Include="System.IO.Compression" />
      <Using Namespace="System.IO.Compression" />

        <![CDATA[
      try
      {
        using (Stream zipStream = new FileStream(Path.GetFullPath(OutputFilename), FileMode.Create, FileAccess.Write))
        using (ZipArchive archive = new ZipArchive(zipStream, ZipArchiveMode.Create))
        {
            foreach (ITaskItem fileItem in Files)
            {
                string filename = fileItem.ItemSpec;
                using (Stream fileStream = new FileStream(filename, FileMode.Open, FileAccess.Read))
                using (Stream fileStreamInZip = archive.CreateEntry(new FileInfo(filename).Name).Open())
                    fileStream.CopyTo(fileStreamInZip);
            }
        }
        return true;
      }
      catch (Exception ex)
      {
        Log.LogErrorFromException(ex);
        return false;
      }
    ]]>

    </Task>
  </UsingTask>

However, I'm getting the following error:
The value "" resulting from evaluating the value of "$(RoslynCodeTaskFactory)" from attribute  "AssemblyFile" of element  <UsingTask> is not valid.
Am i doing something wrong?

The $(RoslynCodeTaskFactory) property is set by an import inside the NuGet package.  You must include a <PackageReference /> in your project and do a restore.   This should create a file named obj\MyProject.g.nuget.props which contains an import to the task factory file that creates that property.  Verify you have things set up correctly.

I've managed to reproduce the issue, keeping it as simple as possible, in this repo.

Okay sorry you're just missing a condition on your <UsingTask />.  Before packages have been restored, the package isn't imported and $(RoslynCodeTaskFactory) isn't set yet.
Add this condition: https://github.com/jeffkl/RoslynCodeTaskFactory/blob/master/src/Samples/Directory.Build.targets#L8
<Project Sdk="Microsoft.NET.Sdk.Web">

  <PropertyGroup>
    <TargetFramework>netcoreapp2.0</TargetFramework>
  </PropertyGroup>

  <ItemGroup>
    <Folder Include="wwwroot\" />
  </ItemGroup>

  <ItemGroup>
    <PackageReference Include="Microsoft.AspNetCore.All" Version="2.0.3" />
    <PackageReference Include="RoslynCodeTaskFactory" Version="1.1.7" />
  </ItemGroup>

  <Target Name="Bug_Reproduction" AfterTargets="Build">
    <Sample />
  </Target>

  <UsingTask TaskName="Sample"
        TaskFactory="CodeTaskFactory"
        AssemblyFile="$(RoslynCodeTaskFactory)"
+       Condition=" '$(RestoreSuccess)' == 'true' ">
    <Task>

        <![CDATA[
       ;
      ]]>

    </Task>
  </UsingTask>

</Project>

I have the condition in my samples but not in the main readme.  I'll update that!

mm that makes sense, thanks!
In fact I added the condition at some point of my trials, but later removed it since it didn't make it fully work (I was trying out from command line).
It works like a charm when compiling/publishing from VS.
However, I am not being able to make tasks work from command line (neither directly, with dotnet msbuild /t:my_task or after a dotnet build or dotnet build --force --no-incremental).
I'm getting the following error message (translated):
 "CodeTaskFactory" task generator couldn't be loaded from assembly file "<Route to netstandard1.5\RoslynCodeTaskFactory.dll>" Task generator must return a value for "TaskType" property.
Now you mentioned it, I suppose it has to do with package importing/restoring, but no real idea.
Any clues?


Python completely freezes when uses Generator(Iterable) and itertools.repeat

Initial Checks

[X] I confirm that I'm using Pydantic V2

Description
Hi!
Python freezes and not responding on Ctrl^C
How to fix this?
Thank you
Example Code
import itertools
from collections.abc import Generator, Iterable
from pydantic import BaseModel

class M(BaseModel):
    x: list[int] | Generator[int]
    # x: list[int] | Iterable[int]  # same result

m = M(x=[1])  # -> ok
m = M(x=itertools.repeat(5))  # -> freeze

Python, Pydantic & OS Version
pydantic version: 2.7.0
        pydantic-core version: 2.18.1
          pydantic-core build: profile=release pgo=false
                 install path: /Users/.../.pyenv/versions/3.11.5/lib/python3.11/site-packages/pydantic
               python version: 3.11.5 (main, Sep 16 2023, 09:45:16) [Clang 12.0.5 (clang-1<IP_ADDRESS>)]
                     platform: macOS-11.7.10-x86_64-i386-64bit
             related packages: typing_extensions-4.11.0 typing_extensions-4.8.0 pyright-1.1.332 pydantic-settings-2.2.1
                       commit: unknown

And Any with model_dump_json not works also
import itertools
from typing import Any
from pydantic import BaseModel

class M(BaseModel):
    x: Any

m = M(x=[1])
m.model_dump()  # ok
m.model_dump_json()  # ok

m = M(x=itertools.repeat(5))
m.model_dump()  # ok
m.model_dump_json()  # -> freeze

repeat(5) is an infinite iterator, what behavior are you expecting when doing m = M(x=itertools.repeat(5))? Pydantic could implement an iteration limit to avoid infinite loops but this is probably the responsibility of the developer.

@Viicos i try to define field that accept both list[int] and itertools.repeat and .model_dump returns original value without modification or unwraping infinity itertools.repeat
This example works(not freezing)
class M(BaseModel):
    x: Iterable[int] | list[int]

but for M(x=[5]).model_dump() returns SerializationIterator
{'x': SerializationIterator(index=0, iterator=ValidatorIterator(index=0, schema=Some(Int(IntValidator { strict: false }))))}

instead of [5]
Alternative version works for list[int] but freezing for itertools.repeat
class M(BaseModel):
    x: list[int] | Iterable[int]

And it would be perfect to get exception from Pydantic instead of freeze (if it is possible)
Thank you.


changed gqlint to lint the file on disk

gqlint gathers a config file which searches the directory
which contains the file being linted. This config file cannot
be found without linting the file on disk

Where are the tests? Have you added tests? Have you updated the tests? Read the
comment above and the documentation referenced in it first. Write tests!
Seriously, read :help ale-development and write tests.

ALE should be able to lint files in-place. Does this get rid of that functionality? What happens if the file is not saved but significant changes are made?

For this linter, yes it does forego linting the files in place because I dont foresee any other way to use a project specific config file.
This linter searches the directory that contains the file, the directory the command was run from, and the home directory for the config file
Is it possible to cd into the buffer dir and still run the lint on the buffer file? If that is possible then I think we can maintain the in-place linting

Is it possible to cd into the buffer dir and still run the lint on the buffer file? If that is possible then I think we can maintain the in-place linting

There was another linter that had a similar issue. I do not recall which one. It is possible, but I don't remember how.

Ill play with it a little bit and see if I can get it to work in place
On Tue, Dec 18, 2018, 2:58 PM Ryan<EMAIL_ADDRESS>wrote:

Is it possible to cd into the buffer dir and still run the lint on the
buffer file? If that is possible then I think we can maintain the in-place
linting
There was another linter that had a similar issue. I do not recall which
one. It is possible, but I don't remember how.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/w0rp/ale/pull/2153#issuecomment-448106353, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AI_NQLX2eNWho3pbm123_qz1ZmDyATvPks5u6IQagaJpZM4ZXfh5
.

Try using BufferCdString and '%t' together.

I got it to work and added a test as well.

Cheers! :beers:

Commit message says the wrong thing, but meh.


In Rust 2018, forbid bulid scripts which don't print rerun-if-chaged

The build scripts can use rerun-if-changed and rerun-if-env-changed variables to control when it's considered out-of-date.
The problem is that if a build script does not use either of these variables, it is rerun if anything is changed. And this is known to cause hard to diagnose rebuilds. This is a historical behavior, so we can't change it.
However, for Rust 2018 it makes sense to switch to one of the following behaviors:

error if build.rs didn't print rerun-if-changed
by default, assume rerun-if-changed=build.rs. That is, rerun only if build script itself changes.

I think option 2 would make more sense. You clearly need to rebuild the build script if the build script changes, but in the 2018 edition, we can assume that's the only file we care about unless the build script tells us otherwise.

Agree, however, the potential problem here is that might become a surprising silent change if you upgrade from 2015 to 2018.

Yeah that is true. You'd have to opt-in via your Cargo.toml to the edition I guess, but you may not be aware that this change is part of that?
I wonder if there are any other changes we want to make to build scripts for the edition that would make them break more obviously at the same time.

I wonder if there are any other changes we want to make to build scripts for the edition that would make them break more obviously at the same time.

Add another level of namespacing to the flags to so that cargo can add new flags without clobbering user defined ones
# old way
cargo:rustc-link-lib=static=foo
cargo:rustc-link-search=native=/path/to/foo
cargo:rustc-cfg=foo
cargo:rustc-env=FOO=bar
cargo:root=/path/to/foo
cargo:libdir=/path/to/foo/lib
cargo:include=/path/to/foo/include

# new way
cargo:rustc:link-lib=static=foo
cargo:rustc:link-search=native=/path/to/foo
cargo:rustc:cfg=foo
cargo:rustc:env=FOO=bar
cargo:self:root=/path/to/foo
cargo:self:libdir=/path/to/foo/lib
cargo:self:include=/path/to/foo/include

:duck:

Instead, what we really need to do is to add a flag to cargo build, which explains why Cargo rebuilds stuff. In particular, we can be pretty vocal about "rerunning build script because no rerun-if-changed is specified".

Was this implemented? It would be great for debugging spurious rebuilds.

@tomjakubowski AFAIK, not really, you might have some luck running cargo with CARGO_LOCK=info and looking for these log messages: https://github.com/rust-lang/cargo/blob/b8448d7c4177a27a4b14201b208ed7d2b61f39df/src/cargo/core/compiler/fingerprint.rs#L1045-L1049