cc-wrapper: Cleanup of Nix

Motivation for this change
I cleaned up the installation script a bit. This is a prereq for me splitting out the ld wrappers into a binutils-wrapper.
One of the changes was using set -u but that will be simpler and cleaner if https://github.com/NixOS/nixpkgs/pull/28057 is accepted. This also depends on https://github.com/NixOS/nixpkgs/pull/28556
This is most easily reviewed commit-by-commit.
Things done

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

[x] NixOS
[x] macOS
[ ] Linux

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

Fixed Linux's allowedRequisites so both builds stdenv passed.

I'm interested in merging this within the next few hours---I've gone any manual checked everything I was concerned about, as documented in the comments above.


Revert back to using CUDA_HOME in cuda enabled images

Checklist before submitting

[x] Did you read the contributor guide?
[ ] Did you update any affected documentation?
[ ] Did you write any tests to validate this change?

Description
Fixes # (issue).
Review process to land

All tests and other checks must succeed.
At least one maintainer must review and approve.
If any  maintainer requests changes, they must be addressed.

Codecov Report

Merging #84 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #84   +/-   ##
=======================================
  Coverage   96.20%   96.20%
=======================================
  Files           8        8
  Lines         579      579
=======================================
  Hits          557      557
  Misses         22       22

Flag
Coverage Δ

#unittests
96.20% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

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


Fix admin filtering

Filtering by organisation wasn't working at all.
Searching was doing an exact match so was very hard to use.
I've copied the rspec features style from https://github.com/alphagov/manuals-frontend/pull/76. I hope to rewrite all of the features to this style shortly.
Fixes https://github.com/alphagov/hmrc-contacts/issues/171

Thanks :hamburger:


time for calculated query answers needs to reflect actual data

When influxdb is used to select an individual result, the time associated with it is returned. e.g., if I do:
SELECT LAST(x) FROM table WHERE ... time > now() - 1d ....
then influxdb will give me output with the timestamp set to the last row from "table".
However if I do a query like this:
SELECT MAX(x) - MIN(x) FROM table WHERE ... time > now() - 1d ....
then influxdb uses the value of "now() - 1d" as the timestamp for the result "MAX(x) - MIN(x)". This is not useful as it cannot be aligned with any other timestamps in influxdb.
Ideally the timestamp for "MAX(x) - MIN(x)" would come from the actual timestamp of either the row for MAX(x) or MIN(x), not "now() - 1d"/"now()".

Actually, maybe I've got that wrong if all of the selectors return a timestamp for "now() - 1d".
If I do:
SELECT x FROM table WHERE time < now() - 1d
then the row with the smallest timestamp should have a timestamp that matches this:
SELECT FIRST(x) FROM table WHERE time < now() - 1d
Similarly there should be a single row that has a matching timestamp in this data set:
SELECT x FROM table WHERE time > now() - 1d
and this:
SELECT LAST(x) FROM table WHERE time > now() - 1d
And again I should be able to match exactly one row (using timestamps) for these two queries (if we assume that both complete before a new row in table is inserted):
SELECT * FROM table WHERE time > now() - 1d
SELECT MAX(x) - MIN(x) FROM table WHERE time > now() - 1d
Since there is no row "MAX(x) - MIN(x) for influxdb to copy the time from, I suppose there are two schools of thought here - does the query return the "now() - 1d" time in the timestamp or does it return the timestamp of one of the rows in the table that it uses to calculate the answer from. I'd argue that the former is not useful and that it is the latter that is desired.
In the former case, there's an assumption being made by the consumer (and influxdb) of the data that the data returned for a timestamp other than which it was received is correct however influxdb kind of lies here. Why is it lieing? When a selector is not used, the timestamp returned with data reflects that actually associated with the metric it is storing (no lie) whereas when a selector is used, the timestamp no longer reflects the original metric but that of the query and influxdb pretends (or lies) that the value it returns was actually the correct value for that time.
There's one caveat to that - if I've asked influxdb to invent points in a series by supplying "fill()" in a query then I would also be happy for influxdb to invent a timestamp to go along with that data point.
e.g.:
SELECT MAX(x) - MIN(x) FROM table WHERE time > now() - 1d GROUP BY fill(previous)

won't fix or don't know how to fix?


add retries to openshift_aws_group to avoid failing due to aws api rate limiting.

@stenwt gave :+1: in #2803


MA - Last time The Residence at Five Corners reported

MA is trying to see when the last time The Residence at Five Corners reporting via reportstream. The sender has told MADPH that they have been submitting results recently but MA is unable to find them.

@Adrian-Brewster the last I see of data we got from them is May 6th. I'll ask SR to check on their side too.

Emma confirmed that's the last time they entered a test into SR too.

@sliu1000 do you know if Adrian ever followed up with Mass on this? I can't find an email in the shared inbox with this specific ticket. I see an email thread from June, but doesn't look like he followed up there. LMK if I can/should just respond there.

@sarahnw Hi Sarah, I do not see any follow-up response from Adrian on this.

@sarahnw will email them cc: @BerniXiongA6

Responded. Closing.


Henders: Blacklist visit4supplements.com

Henders requests the blacklist of the website visit4supplements.com. See the Metasmoke search here

!!/approve


ScrollUP and Scrolldown events fired twice.

hi,
i observe that when i am using range and input both in my project ScrollUP and Scroll Down event fired twice. so whenever value changes multiple of +2 or -2.

Hey, this one got lost in my mail, i'll take a look at this right away :)

Turns out, the support for scroll events is a small directive that is shipped in both datetime-range and datetime-inputs projects simultaneously. I guess this issue never came up before because people have not used those components on the same page. :)
There are two ways to fix this, either we integrate datetime-range together with other datetime inputs in a single library or take the scroll events out and make it a reusable utility.
I'm considering both options right now and will get back to you with an update shortly :)

Okay, I decided to go with the second option and made scroll directives available as a separate library that both datetime-range and datetime-inputs can use. This update is available in the latest release, let me know if this works for you :)
v0.2.1

working fine.


Add x and y labels to bar, line and stacked area charts

We need to think how this could be extracted so that we don't duplicate code.

I'd be interested in taking this on

Great!! Go for it!
You could use the same code you added to the step chart. I would only set a fixed offset on the X and Y axis for now. Later we could open those options if people need them!
Thanks a lot!!

Let's close this for now. Not many people seem interested.


Encoding not recognized

Hi,
When I right click a schema and click "import" and select an .sql file to import, I get the following error.

Hi @Nardusgc,
i also noticed this issue that is related to the mysql2 client library and utf8mb3 charset.
I'm wating for an update that fix this problem.
A workaround to this issue is remove SET NAMES utf8mb3; from the sql file you are importing.

Same issue, thanks @Fabio286 for the workaround!


Update screenshots of command-line and showcase on the front page

At the moment, main 2 screenshots on the front page are outdated. We need to make a new screenshots using current look of our CLI tool and showcase page.
Acceptance criteria

[x] When I arrive at front page I can see current/live examples of CLI tool and showcase page

FIXED in https://github.com/datahq/frontend/commit/1a6a19669633f09b5696d9fe949eb1639a9c1ebb


Shrouded march: remove icon size docs

Icon sizes are always inline, unless directly overridden

:rocket: PR was released in v0.19.0 :rocket:


Phpmyadmin image creating/starting optional

Is your feature request related to a problem? Please describe.
For my local environment on my Mac I have SequelPro, other people use HeidiSQL or MySQL Workbench for their DB connections. Imho there is no need to start phpmyadmin by default all the time.
Describe the solution you'd like
I would like to have the option in the YAML file to activate phpmyadmin. By default, it's deactivated.
Describe alternatives you've considered
No alternatives. :)

I'd be interested in why the phpmyadmin container is annoying. Is it just the problem of having something running that you don't use? Or are you running into resource constraints? (I also never use it and often forget it's there, would be interested to know how many people do use it.)
You can easily get rid of most of the weight of the phpmyadmin container by just swapping out the docker image for "busybox". In your .ddev/ create docker-compose.phpmyadmin.yaml with these contents:
version: '3'

services:
  dba:
    image: "busybox"

I do think it's possible we should just make phpmyadmin be an add-on.

Cool, that workaround is a good option.
I want to have less docker images and prevent to have docker images I don't use. That's why the phpmyadmin image is a thorn in my side.
But I'll take your solution. 👍


[IntelliJ] Inconsistent colours for numeric literals

Refer to the image below. int literals are highlighted in blue while float literals are black (and decimal too). Shouldn't these be highlighted in blue as well?

@pubudu91 Yes you are correct. It seems to be an issue with the semantic highlighting with the recent grammar changes introduced. Thanks for reporting!

Update: I tried with the latest plugin (1.2.0) and this was not reproduced for me when I was trying the same sample, as below.

@pubudu91 Could you also please confirm whether this is resolved with the latest plugin, so that we can close this?

closing since not reproducible.


USe GitHub release tags

Could you please use GitHub release tags when publishing a new release?
That's just the recommend way to do it and makes it possible to find past (or the most current) release much quicker.

I'm a bit forgetful, so I can't promise anything, but will try to keep it in mind. I've added one for the latest, 0.4.0.


Pass user_ctx in _bulk_get

This fixes _bulk_get for _users db and probably others I don't know

+1

chttpd_db_bulk_get_test eunit failure

I hope it also will fix  _metadata db along with  _users db

@adv4000 CouchDB doesn't currently do anything with the _metadata DB, what are you expecting to find there?

Issue reported on couchdbbackup tool: https://github.com/cloudant/couchbackup/issues/253
Looks like _bulk_get not working with databases named _users and _metadata


Cleanup - removed temp thumbnail files

There where a few binary temporary thumbnail files on the main branch. Those temporary files were created by a local OSX or Windows environment and shouldn't make it onto the main branch. To prevent the main branch from those files in the future I created a .gitignore file where those files are listed.

Much appreciated @dnltsk 👏


CI testing for "Rename CMake project to gz"

Running CI for these PRs together:

https://github.com/ignitionrobotics/ign-cmake/pull/244
https://github.com/ignitionrobotics/ign-utils/pull/57
https://github.com/ignitionrobotics/ign-math/pull/420

I pushed a fix to your branch in https://github.com/ignitionrobotics/ign-cmake/commit/e44d9dc0bfd2c715238247d6465a7447cb8b0089, which may help. I'll restart some workflows to see if it helps

I pushed a fix to your branch in ignitionrobotics/ign-cmake@e44d9dc, which may help. I'll restart some workflows to see if it helps

didn't help

Finally, compiling dependencies from source is passing


onChange throws an error, but is not included in the emit

      if (error) {
        this.$emit("error")
      }

As a result, attempts to handle errors in the component only receive undefined in all cases.
I think this should be
      if (error) {
        this.$emit("error", error)
      }

@MichaelPaulukonis
Thank you very much for your issue, already fixed in the latest version.


Typo in Clean up Resources - Note

"If you are immediately proceeding to the next tutorial to learn how to run ETL operations using Hadoop on HDInsight, you may want to keep the cluster running. This is becuase in the tutorial you have to create a Hadoop cluster again. However, if you are not going through the next tutorial right away, you must delete the cluster now."
Misspelled "because"
Should be
"If you are immediately proceeding to the next tutorial to learn how to run ETL operations using Hadoop on HDInsight, you may want to keep the cluster running. This is because in the tutorial you have to create a Hadoop cluster again. However, if you are not going through the next tutorial right away, you must delete the cluster now."

Document details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 6593c2a0-76bc-47e0-519e-d653ab435fdd
Version Independent ID: 17c83e2b-3fe4-97ad-1bce-923dd3e900e6
Content: Quickstart: Get started with Apache Hadoop and Apache Hive using the Azure portal - Azure HDInsight
Content Source: articles/hdinsight/hadoop/apache-hadoop-linux-create-cluster-get-started-portal.md
Service: hdinsight
GitHub Login: @hrasheed-msft
Microsoft Alias: hrasheed

@Lavoiec  Thanks for pointing out the issue, I've addressed this issue in below PR: https://github.com/MicrosoftDocs/azure-docs-pr/pull/67579
The changes should be live EOD or tomorrow. We will now close this issue. In future, if you see any minor bug fixes in the doc, feel free to click on the edit button and submit a PR to update. we love to take community contributions.


GitHub Fork and Fork Names

Hi! Just noticed that some forks have not been created yet or not named, and wanted to put a reminder here for us to fork from the UBC-MDS repo and rename our forks.
To rename just:

Go to your fork
Settings
rename it "group29-yourname"
press "rename"

Thank you!

done!


update this?

Hello
This page was helpful to me, thank you! However it seems to be written for an older version of the SDK -- it still calls CosmosDb "DocumentDB". I know the latest CosmosDb v4 SDK has a different API for bulk inserts, can the example be updated to use that?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: f942c792-655b-4174-a427-644108abd6d5
Version Independent ID: 4009eeb3-ee41-ab49-d0d0-4269d678a0bc
Content: Use bulk executor Java library in Azure Cosmos DB to perform bulk import and update operations
Content Source: articles/cosmos-db/sql/bulk-executor-java.md
Service: cosmos-db
Sub-service: cosmosdb-sql
GitHub Login: @abinav2307
Microsoft Alias: abramees

@benhsu Thank you for your feedback. We are reviewing this and will get back to you.

Thanks @benhsu for your feedback!
I think this article has been updated per your feedback two months ago, but no-one ever closed this issue. I'm seeing updates addressing the issues you described so I will go ahead and close this issue.
Thanks again
#please-close


Page title should include the package name

Currently, when browsing into a package page, the HTML page title is always Construct Hub. It should include the package name (e.g. cdk-lambda-bash - ConstructHub) or event just the package name (cdk-lambda-bash).

Definitely something we will want to include when we tackle the work for setting up SEO and page specific meta


Support caching the Git repo

For each build, the Git repo is re-cloned. I suggest that in the event that the repo is already cloned, do a git reset --hard && git pull to save time and bandwidth.
The plugin should save the clone parameters to a file to ensure that they haven't changed (in which case a clone should be used instead).

Wait, why do you think that the repo is cloned on each build? It will only happen if you'll delete the directory of you git dependency. Otherwise the plugin checks if current version is changed, and then does fetch & checkout if needed.

Closing this one for now. Feel free to re-open if you beleive there is an issue.


Bring Google Tag Manager custom HTML tags into cfgov-refresh

Google Tag Manager (GTM) introduces JS code to the site without code review and tests.
GTM also includes youtube code that may be the source of [ghe]/CFGOV/platform/issues/166.
This PR brings the existing code from GTM to this repo so that we can improve its quality and test coverage. This will run the scripts through webpack and make them available on the site at the static/apps/analytics-gtm/ URL, which will allow GTM to lazy load the scripts by utilizing the global jsl() function we already use to load other scripts.
Additions

GTM custom HTML tags from Google Tag Manager dashboard.

Changes

Moves javascript_loader jinja block to the head above the analytics code imports.

Testing

JS should load on the site as before.
Edit unprocessed/apps/analytics-gtm/bah-check-rates-listeners.js to add console.log( 'OAHRCAnalytics loaded!' ); on line 2.
Run gulp build.
Add this:

<script>
   jsl(['/static/apps/analytics-gtm/js/bah-check-rates-listeners.js']);
</script>

To line 150 of base-common.html
5. Verify you see OAHRCAnalytics loaded! in the dev console when loading a page. Note: there will be an error too, unless you're viewing on the appropriate OAH page. This doesn't matter though, we just want to know that it loads.
Notes

After this goes to production the custom HTML tags in GTM should be updated with jsl(['/static/apps/analytics-gtm/js/[script to load].js']);, where [script to load] is the custom HTML tag to load.

Note that edits to the actual scripts in unprocessed/apps/analytics-gtm/js/ shouldn't happen in this PR, though feel free to comment on things you see. Those scripts are not put live in this PR and are duplicates of what is currently delivered to the site. When they go live and the GTM copy is removed we can then refine them here.

This is really good; one of my favorite PRs. How does this affect apps which aren't in cfgov-refresh?

How does this affect apps which aren't in cfgov-refresh?

That's a good question. So what about this:

Take the inline jsl script and move it to it's own unminified JS file, which is run through webpack and minified like everything else.

Take the minified standalone jsl script created above and include it inline in the base template (like is done with modernizr).

In GTM have a script that tries to load the analytics script ('/static/apps/analytics-gtm/js/bah-check-rates-listeners.js' or whatever) locally, and if not found (if the page is not under consumerfinance.gov), have it load the jsl script from a consumerfinance.gov absolute URL and then use the loaded jsl script to load the analytics script from an absolute consumerfinance.gov URL as well.

jsl(['/static/apps/analytics-gtm/js/bah-check-rates-listeners.js'])

This is great!
As a note, I think the main apps with custom analytics outside cfgov-refresh are (parts of) Paying for College and Before You Claim.

I'm going to merge this and then test in GTM next deploy and address options for https://github.com/cfpb/cfgov-refresh/pull/3919#issuecomment-375379969


Move planning scene utilities into separate repository

The planning scenes are a requirement for grasplan due to MoveIt!. To continue the clean up of this repository, I would propose to create another repository with all your utilities to create new planning scenes (would mainly include the rqt_gui from @oscar-lima). I could imagine something like planning_scene_helpers.

As discussed in person (Julian and I) we will delete the Grasplan GUI that allows to modify yaml planning scenes and go with the approach of editing planning scenes using RViz MotionPlanning MoveIt plugin, see issue: https://github.com/DFKI-NI/grasplan/issues/26


ADDED ISSUE_TEMPLATE

ISSUE_TEMPLATE has been added.

@tharunc Add yourselves under the README as a contributor.
Also If Possible look for ways to automate this process & u can make a new PR for that !

@tharunc Add yourselves under the README as a contributor.
Also If Possible look for ways to automate this process & u can make a new PR for that !

@all-contributors please add @tharunc  for Documentation

@all-contributors please add @tharunc  for Documentation


Fixes  'aad siteclassification set' command.  Closes #2677

Closes #2677
This PR updates endpoint of aad siteclassification set command from beta/settings endpoint to v1.0/groupSettings endpoint.

Thank you! We'll review it asap.

Merged manually. Thank you, and welcome to the team! Great to have you with us! 👏


When I change state by "setState", the editor always refresh

monaco-editor version: 0.14.3
**Browser:**chrome
OS: windows
Steps or JS usage snippet reproducing the issue:
I use contextMenu in 'react-contexify' to show information about the specified row by changing the state.So I use setState func to toggle the state "show".But every time I change state, the editor freshes obviously, especially when there is code coloring.
The larger the current line number, the more obvious the refresh.

I found a terrible mistake.. sorry..


Provided models from download link does not work

Hi, can you re-check the models you provided by the download link? I downloaded the first one but it cannot be unzipped.

Hi, the checkpoint can be load using torch.load. No need to unzip.

I see. Yeah, torch.load works, thanks for the reply.


Analyzer Feedback from IntelliJ

Not able to see FlatButton properities
Version information

IDEA AI-202.76<IP_ADDRESS>22048
2.12.2
AI-202.76<IP_ADDRESS>22048, JRE 11.0.8+10-b944.6842174x64 JetBrains s.r.o, OS Windows 10(amd64) v10.0 , screens 1920x1080

Thanks for filing an issue. Please explain in more detail the issue you are facing. How do you reproduce the issue? What is the expected behavior? What is the actual behavior?


ARROW-5979: [FlightRPC] Expose opaque (de)serialization of protocol types

Motivation: make it easier for non-Flight services to consume/produce Flight types. For instance, a REST service may want to produce FlightInfo or Ticket objects for a Flight client. This gives us an (opaque) serialization of a few protocol types that can be deserialized and then used as normal.
I've noticed some of the Java APIs accidentally expose conversions to/from Protobuf types; if it's worth it, I can go back and make those private.
Personal Travis: https://travis-ci.com/lihalite/arrow/builds/119836432

Let's try to keep all protobuf types hidden (by fixing or avoid displaying). For example, you are exposing a new protobuf based exception in this patch. Can we use something more generic like IOException?

Good point, I'll fix both of those!

Updated and rebased.

Codecov Report

Merging #4914 into master will increase coverage by 1.64%.
The diff coverage is 89.76%.

@@            Coverage Diff             @@
##           master    #4914      +/-   ##
==========================================
+ Coverage   87.45%    89.1%   +1.64%
==========================================
  Files         994      717     -277
  Lines      140389   100381   -40008
  Branches     1418        0    -1418
==========================================
- Hits       122776    89441   -33335
+ Misses      17251    10940    -6311
+ Partials      362        0     -362

Impacted Files
Coverage Δ

python/pyarrow/tests/test_flight.py
71.26% <100%> (+0.92%)
:arrow_up:

cpp/src/arrow/flight/flight-test.cc
97.94% <100%> (+0.14%)
:arrow_up:

cpp/src/arrow/flight/types.h
100% <100%> (ø)
:arrow_up:

python/pyarrow/_flight.pyx
55.07% <79.41%> (+5.37%)
:arrow_up:

cpp/src/arrow/flight/types.cc
90.15% <84.61%> (-2.24%)
:arrow_down:

go/arrow/ipc/writer.go

js/src/util/fn.ts

go/arrow/memory/memory_avx2_amd64.go

rust/datafusion/src/execution/filter.rs

rust/arrow/src/csv/writer.rs

... and 277 more

Continue to review full report at Codecov.

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

Thanks!


Missing markets drop down.

Just installed the last version and the markets drop down is missing for import data.

@virgiltu how you solve this issue.


[watchindianporn] Fix parser

Before submitting a pull request make sure you have:

[x] At least skimmed through adding new extractor tutorial and youtube-dl coding conventions sections
[x] Searched the bugtracker for similar pull requests

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[x] Bug fix
[ ] Improvement
[ ] New extractor
[ ] New feature

Description of your pull request and other information
Fix #13411. Note that uploader, upload_info and comment_count  infos are no more present in the service.

Updated

I'm not sure if the last regex is correct. Look forward to your answer @dstftw

Not correct.

I hope I understood

No you don't. <a[^<] you should look for anything but closing angle bracket, not opening.

Oh, I swear, I didn't see it. I thought the mistake was at the end. If the rest is right correct I fix it and squash commits

Fixed @dstftw


How to restore a session after delete?

I use the python sdk delete_memory function to delete the memory, but then when I try to use the same session id to store messages, it shows up error like  API error: {'status_code': 400, 'message': 'storage error: failed to put messages (original error: storage error: session xxxx is deleted (original error: ))\n'}. Also I do not see a specific function within zepclient that handles session creation

@wukao1985 - when you delete a session, Zep will 'soft delete' the session and associated conversations. The server periodically (based on purge interval) remove records that have been soft-deleted. You can control the purge interval in config.yaml. Until the session is purged, you will not be able to re-use the same sessionId.

@wukao1985 For more on Zep's approach to data deletion, see: https://docs.getzep.com/deployment/data/

Perfect. I am still actively developing, so need hard deletion to happen faster, setting up it as 2 mins for now should be ok now.

umm.. I did more testing, seems after setting 2min in the environment variable in Render, the purge did not happen ... how can I further troubleshoot? or is there any workaround that I can make a call to do hard delete myself?

@wukao1985 You reopened this issue. Did you get the data purge working?


Godot's .NET math types are way slower than System.Numerics

Godot version
4.0 beta 6
System information
Win 11, Ryzen 5 5600H, 32GB RAM
Issue description
Godot's integrated math library for .NET 6 is considerably slower than standard .NET System.Numerics.

Steps to reproduce
Do pretty much any Vector ops and compare it against System.Numerics. See included project for a small benchmark.
Minimal reproduction project
VectorBenchmark.zip

System.Numerics uses intrinsics so it's expected it would be faster.
We could probably use SIMD to speed things up a bit, note that SIMD is also not implemented on the C++ side of the engine, there is a proposal for that https://github.com/godotengine/godot-proposals/issues/4563 but it likely won't affect C# since we reimplement the math in order to avoid marshaling.
If you really need the performance, there's nothing wrong about converting your inputs into System.Numerics types to perform the math operations and then convert back to Godot types to return the output.

I'm taking a look at this as part of #79395.
I have a suite of benchmarks that compares:

Godot.VectorX 4.1
Godot.VectorX PR impl
System.Numerics.VectorX
System.Runtime.Intrinsics.Vector128
Godot.VectorX wrapping System.Numerics.VectorX
Godot.VectorX wrapping System.Runtime.Intrinsics.Vector128
Hand-rolled System.Runtime.Intrinsics.Vector128

Summary:

The difference between net6 and net8 is gigantic and using SnVec (net6) is not always the best solution.
Net8 is still in preview (6 right now), in some case the codegen is "strange".
By fully using xmm registers, stack spill is less of a problem but it still there and sometimes it leads to unexpected regressions.
The PR i'm working on is almost useless with net8, the jit is a lot smarter now.
Most of the problems caused by the odd size of Vec3 (96bits) are solved with net8.

@tannergooding I'll contact you if these problems are not solved with next preview (or RC). Thanks.
Results: (example ported to C#, 512x512px) https://github.com/godotengine/godot/blob/725beaa7afce294f38d4aca1fa287249021b6bf8/core/io/image.cpp#L3646-L3692

Namespace
Method
Runtime
Mean
Error
Ratio
Code Size

Godot_Old
BumpToNormal
.NET 6.0
4.750 ms
NA
1.00
735 B

Godot_New
BumpToNormal
.NET 6.0
3.593 ms
NA
0.76
500 B

SnVec_Struct
BumpToNormal
.NET 6.0
4.143 ms
NA
0.87
675 B

SnVec
BumpToNormal
.NET 6.0
3.858 ms
NA
0.81
663 B

Vec128_Struct
BumpToNormal
.NET 6.0
2.886 ms
NA
0.61
580 B

Vec128
BumpToNormal
.NET 6.0
1.936 ms
NA
0.41
503 B

Godot_Old
BumpToNormal
.NET 8.0
3.597 ms
NA
0.76
482 B

Godot_New
BumpToNormal
.NET 8.0
3.569 ms
NA
0.75
586 B

SnVec_Struct
BumpToNormal
.NET 8.0
2.907 ms
NA
0.61
564 B

SnVec
BumpToNormal
.NET 8.0
2.495 ms
NA
0.53
556 B

Vec128_Struct
BumpToNormal
.NET 8.0
2.181 ms
NA
0.46
484 B

Vec128
BumpToNormal
.NET 8.0
1.871 ms
NA
0.39
470 B

I'll contact you if these problems are not solved with next preview (or RC). Thanks.

Sounds good, thanks! You can notably get nightly builds here as well: https://github.com/dotnet/installer#installers-and-binaries
Also glad to see the improvements done in .NET 8 are paying off. Did quite a bit of refactoring and improvements, particularly to Quaternion, Plane, and Matrix4x4. There's more still to land, but .NET 8 will be in a lot better shape than previous releases when it comes out in November.

^ .NET 8 is out now.
Guess I will use .NET vectors for non-interfacing code for now, but some kind of parity would be nice for Godot Vectors.

Maybe some implicit cast from Numerics would be nice

@tannergooding If you're ever bored, you might take a look at System.Numerics.Vector3.Cross(). It's the only method that we're able to get noticeably faster performance from a custom function by converting to Vector128 and back. I believe the Numerics version is still only using scalar instructions.

If you're ever bored, you might take a look at System.Numerics.Vector3.Cross(). It's the only method that we're able to get noticeably faster performance from a custom function by converting to Vector128 and back. I believe the Numerics version is still only using scalar instructions.

This was done in https://github.com/dotnet/runtime/pull/103527, as was several other improvements including to matrix multiplication. It should be available in .NET 9 Preview 6.

Maybe some implicit cast from Numerics would be nice

Agree, is there a separate issue/discussion tracking that? I haven't found one yet, but there's a lot of different terms to search for. I guess that'd go in the proposals repo?
I was just doing some work to keep my game logic in an isolated C# project and was going to leverage the System.Numerics Vector3 to store my resulting data. It'd be nice if I could just directly assign it to the Node3D transforms and such within my game project layer.
I guess the question would be if implicit operators would be wanted, someone could run into issues with perf if they're not aware it's creating/converting behind the scenes, but an explicit conversion operator and/or extension method ToGodotVector() would be nice:
public static class VectorExtensionMethods
{
    public static Godot.Vector2 ToGodotVector(this System.Numerics.Vector2 v) => new(v.X, v.Y);

    public static Godot.Vector3 ToGodotVector(this System.Numerics.Vector3 v) => new(v.X, v.Y, v.Z);
}


JQuery doesn't get injected inside a React Native WebView

Bug description:
I'm trying to remove the header from a WebView using the injectedJavaScript() function.
If I write my function in pure JavaScript, it executes:
document.getElementsByClassName('site-header')[0].innerHTML="";
However, if I write my function in JQuery, it doesn't execute:
$('.site-header').remove();
If I execute the JQUERY directly on https://www.ourworldindata.org/grapher/prevalence-of-undernourishment, it does indeed work.
To Reproduce:
https://snack.expo.io/@alishehzad/jquery-not-executing-on-webview
Expected behavior:
The header should be removed.
Environment:

Tested on: Android, Android Emulator, iOS Simulator
react-native version: 0.61
react-native-webview version: 8.1.1

Note: It was working fine just yesterday. No idea what happened today!

This is happening to me as well


Add OpenDrainIO pin state (with InputPin capability)

As already reported in https://github.com/nrf-rs/nrf-hal/issues/339, there is currently no way to create an I/O pin in nrf-hal.
There are several reasons why having this would be useful. My personal usecase is the DHT22/AM2302 sensor. Controlling it requires a single-wire pulled-up open drain IO.
Sadly, this means that all libraries that can interface with it require InputPin + OutputPin.
Solution
There is no inherent reason why nrf chips shouldn't be able to implement this. The input buffer is always available and allows reading back the pin values during every pin configuration. Although for energy saving reasons, the input buffer is currently disabled during the OpenDrain state.
My proposal is to add an OpenDrainIO state that does not disable the input buffer and implements InputPin.

Open Questions: How should IoPin behave? Should we switch back and forth between OpenDrain and OpenDrainIO? Or should we just stay in OpenDrainIO? What's the usecase where a library would use IoPin? Would that usecase benefit from switching the input buffer on and off, or would that rather be a performance hit?
In my case, the DHT22 sensor libraries require InputPin + OutputPin. When would they use IoPin instead? What's the benefit of IoPin over InputPin + OutputPin?

Turns out other people have doubts as well. Therefore IoPin was removed from embedded-hal for now. I'll remove it from this PR as well.

Possible solution if we want to re-introduce IoPin at some point:

Implement IoPin<Self, Self> for Pin<Output<OpenDrainIO>>
Switch to passive output state when transitioning to Read

would require tracking if we are in 0Normal1Disabled or 0Disabled1Normal.

Will write tests for this as well, as soon as I have an nrf52840-DK.

I wonder if this should just be the behavior of OpenDrain pins in general? I agree with https://github.com/nrf-rs/nrf-hal/pull/340#issuecomment-917206892 that it's a footgun for PushPull pins, but for OpenDrain, not so much, since those are legitimate input pins.

I wonder if this should just be the behavior of OpenDrain pins in general?

To my knowledge this was an intentional decision, because having the input buffer connected to a floating pin can cause substantially increased power drain.

Okay, that sounds good!
bors r+

@jonas-schievink Added tests in #402


Add miniforge to version parsing

Codecov Report

Merging #15702 (ffffa93) into main (532e0ba) will decrease coverage by 0%.
The diff coverage is 100%.

@@          Coverage Diff           @@
##            main   #15702   +/-   ##
======================================
- Coverage     64%      64%   -1%
======================================
  Files        557      557
  Lines      26867    26815   -52
  Branches    3911     3901   -10
======================================
- Hits       17403    17367   -36
+ Misses      8730     8714   -16
  Partials     734      734

Impacted Files
Coverage Δ

...nments/discovery/locators/services/pyenvLocator.ts
69% <100%> (+<1%)
:arrow_up:

src/client/common/experiments/service.ts
83% <0%> (-5%)
:arrow_down:

src/client/activation/jedi/languageServerProxy.ts
20% <0%> (-2%)
:arrow_down:

src/client/terminals/codeExecution/helper.ts
13% <0%> (-2%)
:arrow_down:

...t/jupyter/languageserver/notebookConcatDocument.ts
5% <0%> (-1%)
:arrow_down:

...client/jupyter/languageserver/notebookConverter.ts
1% <0%> (-1%)
:arrow_down:

src/client/common/types.ts
100% <0%> (ø)

src/client/common/serviceRegistry.ts
100% <0%> (ø)

src/client/common/application/types.ts
100% <0%> (ø)

src/client/common/experiments/groups.ts
100% <0%> (ø)

... and 8 more


Difficult to test (hardcoded values)

Values for CONJUR_ACCOUNT and CONJUR_APPLIANCE_URL are retrieved from the system properties.
These should be within its own class so I can setup a Config object containing this information and init the Conjur class with this Config object. This will allow me to have more granular control over my tests.

This can be closed since these values can be retrieved from system or environment.


add link to gist for ubuntu-based distro install

if you want to install ROS and DBW on Linux Mint or other Ubuntu-based distros, you need to adapt the install scripts. wrote and linked a gist for that

Hi @wAuner,
Thanks for suggesting an update to the Readme.md file. Unfortunately, there is a conflict with your proposed changes.
So, we will not be able to accept your changes. We would suggest removing the conflicts.
Thank you


confusing error: CUDA_ERROR_INVALID_PTX

I have two computer.
Computer A has  two 24G memory nvidia gpu (RTX 6000).
Computer B only has a 2000M memory nvidia gpu (MX330).
The two computer all use same cuda-9.0 , same tvm version, same gcc version,  same system ubuntu18.04 and same code.
There are two scene,
scene 1,
I export a tvm model on  computer A.
Then i run the model on computer B.
TVM report error "Check failed: ret == 0 (-1 vs. 0) : CUDAError: cuModuleLoadData(&(module_[device_id]), data_.c_str()) failed with error: CUDA_ERROR_INVALID_PTX".
scene2,
I export a tvm model on computer A.
Then i run the model on computer B.
It runs correctly.
So, what is the reason about the above error?
It is about gpu memory ?

Please ask questions on the forum. I don't see the difference in your case 1/2.


support detecting zc.buildout directories as another kind of virtual environment

Currently vscode will scan multiple paths to try and detect virtualenv and other similar tools to discover extrapaths to include for intellisense, and which interpreter is likely to be used for debugging. It doesn't find and detect where zc.buildout is used in place of a virtualenv. zc.buildout is a popular tool in the zope and plone communities.
A previous request was made but closed by mistake - https://github.com/microsoft/vscode-python/issues/6992
A related request is in https://github.com/microsoft/vscode-python/issues/1479 to change how vscode detects virtualenvs.
To detect a zc.buildout in the current workspace look for a .installed.cfg in the root of the workspace. Buildout is somewhat different from virtualenv in that it allows many environments at once so it doesn't support a single interpreter or a single working set of python paths. There are two ways to solve this. Either have some configuration to pick which working set to use (pycharm does this), or to assume you can combine all the working sets and just pick the first interpreter. I'd propose the later as in almost all cases this is how build is used. Having an automatic detection and configuration is a nicer user experience.
.installed.cfg will contain entries like
[instance]
__buildout_installed__ = /Users/home/Projects/pretaform/bin/runzope
        /Users/home/Projects/pretaform/bin/zopectl
        /Users/home/Projects/pretaform/bin/zpasswd
__buildout_signature__ = zc.recipe.egg-2.0.5-py2.7.egg zc.buildout-dd444aa64a9523549c6de5651e44280e setuptools-dd444aa64a9523549c6de5651e44280e ...
_b = /Users/home/Projects/pretaform/bin
_d = /Users/home/Projects/pretaform/develop-eggs
_e = /Users/home/Projects/download-cache/eggs
...

__buildout_installed__ refers to scripts it installed or other files it created.
__buildout_signature__ includes  what eggs were downloaded and used
_e is where the eggs were downloaded to.
_d this is where additional python paths could be
A script is python that will look like
#!/Users/home/Projects/pretaform/bin/python2.7

import sys
sys.path[0:0] = [
  '/Users/home/Projects/collective/Plomino2/src',
  '/Users/home/Projects/collective/collective.trustedimports',
  '/Users/home/Projects/download-cache/eggs/pystache-0.5.4-py2.7.egg',
...
  ]

import ipdb.__main__

if __name__ == '__main__':
    sys.exit(ipdb.__main__.main())

You can see there are two ways to get the working set. Either by finding and parsing the installed scripts. Or understanding the data inside .installed.cfg. However it seems finding the interpreter path can only be done by parsing the installed scripts.
I would propose

if .installed.cfg is found assume it's a buildout dir. Use this as priority over virtualenv as sometimes both are used
Get a list of potential scripts from all the __buildout_installed__ entries
parse each script for interpreter and working set. ignore if it doesn't have both.
combine the set of working sets into one big set of python paths. Include this automatically in the extrapaths used for intellisense etc.

Thank you for the suggestion! We have marked this issue as "needs decision" to make sure we have a conversation about your idea. We plan to leave this feature request open for at least a month to see how many 👍 votes the opening comment gets to help us make our decision.

Thanks for the suggestion! We talked about it with the team and we have unfortunately decided we will not be moving forward with this idea. We think there isn't an enough widespread need for this to warrant the maintenance cost for the feature.


Rootless Podman container cannot resolve DNS after enabling systemd-resolved

Describe the bug
As the title mentions, a rootless podman container is unable to resolve DNS after enabling systemd-resolved via the services.resolved module.
Since home-manager does not have a module for rootless podman containers, I am manually creating a systemd service to start a Gitea container at boot (source). This Gitea container handles my personal Gitea instance where I have a push mirror to my GitHub repositories. I push to Gitea which in turn pushes to GitHub.
With services.resolved.enable = true; in my configuration.nix file, I get the following error in the container's logs:
2024/01/20 19:04:25 ...irror/mirror_push.go:164:func1() [E] Error pushing /data/git/repositories/thefossguy/dotfiles.git mirror[4] remote remote_mirror_XXXXXXXXXX: push failed: exit status 128 - fatal: unable to access 'https://github.com/thefossguy/dotfiles/': Could not resolve host: github.com
 - fatal: unable to access 'https://github.com/thefossguy/dotfiles/': Could not resolve host: github.com
 - fatal: unable to access 'https://github.com/thefossguy/dotfiles/': Could not resolve host: github.com

2024/01/20 19:04:25 ...irror/mirror_push.go:112:SyncPushMirror() [E] SyncPushMirror [mirror: 4][repo: <Repository 5:thefossguy/dotfiles>]: push failed: exit status 128 - fatal: unable to access 'https://github.com/thefossguy/dotfiles/': Could not resolve host: github.com
 - fatal: unable to access 'https://github.com/thefossguy/dotfiles/': Could not resolve host: github.com
 - fatal: unable to access 'https://github.com/thefossguy/dotfiles/': Could not resolve host: github.com

Since aardvark-dns is the DNS server that the upstream (Podman) documentation recommends, I also tried putting aardvark-dns in the PATH environment for the systemd user service like so:
-          Environment = [ "PODMAN_SYSTEMD_UNIT=%n" ];
+          Environment = [
+            "PODMAN_SYSTEMD_UNIT=%n"
+            "PATH=\"${pkgs.aardvark-dns}/bin\""
+          ];

But this doesn't seem to work either.
Steps To Reproduce

Ensure that services.resolved is disabled/unset.
Use my nix file to create a systemd service for the container. (Or just run apache/httpd instead of copying unnecessary lines of code.)
Enter container using podman exec -it gitea-govinda sh.
From the container's shell, running nslookup github.com should give you a familiar, positive-feeling output.
Enable services.resolved in the NixOS configuration and reboot (haven't tried with a nixos-rebuild switch).
Enter container using podman exec -it gitea-govinda sh.
From the container's shell, running nslookup github.com would result in some sort of error. It says ;; connection timed out; no servers could be reached on my end.

Expected behavior
I expect the DNS lookup to work since the distros (RHEL and Fedora) where Podman is the first class citizen does use systemd-resolved and DNS lookup does work there.
Screenshots
N/A
Additional context
N/A
Notify maintainers
Pinging the last three people that modified this module: @benaryorg @RaitoBezarius @roberth
Metadata
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"aarch64-linux"`
 - host os: `Linux 6.1.73, NixOS, 23.11 (Tapir), 23.11.3326.d2003f2223cb`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.18.1`
 - channels(root): `"nixos-23.11"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

Add a :+1: reaction to issues you find important.

Hi there, I don't do container alas, I am afraid I cannot help.

That's not an issue. Do you have any idea how to "debug" this? I assumed adding aardvark-dns to the PATH might prevent it but as I mentioned, it doesn't work. Comparing two NixOS versions with systemd-resolved enabled and disabled (with nvd) show me that openresolv gets added when systemd-resolved is disabled.
$ nvd diff /nix/var/nix/profiles/system-3{4,5}-link
<<< /nix/var/nix/profiles/system-34-link
>>> /nix/var/nix/profiles/system-35-link
Selection state changes:
[C+]  #1  openresolv  3.13.2
Added packages:
[A.]  #1  X-Restart-Triggers-resolvconf    <none>
[A.]  #2  unit-network-setup.service       <none>
[A.]  #3  unit-resolvconf.service          <none>
[A.]  #4  unit-script-network-setup-start  <none>
Removed packages:
[R.]  #1  X-Restart-Triggers-systemd-resolved  <none>
[R.]  #2  etc-systemd-resolved.conf            <none>
[R.]  #3  unit-systemd-resolved.service        <none>
Closure size: 1382 -> 1383 (25 paths added, 24 paths removed, delta +1, disk usage +3.0KiB).

@thefossguy what does /etc/resolv.conf point to inside the container?
Without further testing I'd assume that either in that specific setting (or maybe just the general default) the host resolv.conf gets copied into the container verbatim which would break DNS resolution if the container has its own network namespace (i.e. anything but --net=host) since it cannot reach the host's localhost (since it has its own).
Looking at podman-run(1) it says this:

Several files will be automatically created within the container. These include /etc/hosts, /etc/hostname, and /etc/resolv.conf to manage networking.  These will be based on the host’s version of the files, though they can be customized with options (for example,  --dns  will  override  the  host’s  DNS  servers  in the created resolv.conf).

Which would point the same direction.
Both DNS and networking in general are a very complex topic in any kind of infrastructure that is not Link-Layer bound, which is the case for qemu using slirp and containers of any variety (LXC, OCI, nspawn).
As a rule of thumb all virtualization software has knobs to turn and twist for these cases and to find a solution that fits your use-case usually hinges more on the virtualization software than the host system, so it's often more viable to check with the requirements pertaining to network on the virtualization side before looking at the host.
For instance in this case if you're using something on top of podman (such as a form of *-compose) then that solution probably has some orchestration abilities to aid with this, otherwise podman itself has the above mentioned options to tweak it manually on invocation. Any solution offering both the host-local caching and/or proxying so desired in mobile scenarios will likely require rebinding (ideally with appropriate ACLs or firewalling since DNS is known to be used for amplification attacks) so as to bind on a local-scoped address (e.g. switching from ::1/<IP_ADDRESS> to ULA/RFC 1918 addresses) and deploying a resolv.conf which remains valid even when copied into the container. Or you could run your container in the host network namespace if you don't require separation on that level.

Use my nix file to create a systemd service for the container. (Or just run apache/httpd instead of copying unnecessary lines of code.)

I see you are specifying a network in your nix configuration however as far as I can see that network is not managed with Nix (or am I mistaken?).
My quick tests on a Nix machine with resolved yield proper DNS resolution using a podman provided RFC1918 address when run without arguments and when creating a network with default parameters (which sets the DNSEnabled flag to true by default, see also the above mentioned man-page, which lists this command to check: podman network inspect -f  {{.DNSEnabled}} <name>).

@thefossguy what does /etc/resolv.conf point to inside the container?

$ cat /etc/resolv.conf
# This is /run/systemd/resolve/stub-resolv.conf managed by man:systemd-resolved(8).
# Do not edit.
#
# This file might be symlinked as /etc/resolv.conf. If you're looking at
# /etc/resolv.conf and seeing this text, you have followed the symlink.
#
# This is a dynamic resolv.conf file for connecting local clients to the
# internal DNS stub resolver of systemd-resolved. This file lists all
# configured search domains.
#
# Run "resolvectl status" to see details about the uplink DNS servers
# currently in use.
#
# Third party programs should typically not access this file directly, but only
# through the symlink at /etc/resolv.conf. To manage man:resolv.conf(5) in a
# different way, replace this symlink by a static file or a different symlink.
#
# See man:systemd-resolved.service(8) for details about the supported modes of
# operation for /etc/resolv.conf.

nameserver <IP_ADDRESS>
options edns0 trust-ad
search .

$ podman exec -it gitea-govinda sh
/ # nslookup github.com
;; connection timed out; no servers could be reached

/ # nslookup google.com
;; connection timed out; no servers could be reached

/ # ls -l /etc/resolv.conf
-rw-r--r--    1 root     root            39 Jan 21 11:33 /etc/resolv.conf
/ # cat /etc/resolv.conf
search dns.podman
nameserver <IP_ADDRESS>

Use my nix file to create a systemd service for the container. (Or just run apache/httpd instead of copying unnecessary lines of code.)

I see you are specifying a network in your nix configuration however as far as I can see that network is not managed with Nix (or am I mistaken?).

Yes, the networks get setup by an "init" service which calls this shell script. This "init" script sets up a network if one doesn't exist already. (The network name--the entire systemd user service to be honest--is influenced purely by whatever podman-compose systemd create-unit -f docker-compose.yaml resulted in when I was on RHEL 9.)

My quick tests on a Nix machine with resolved yield proper DNS resolution using a podman provided RFC1918 address when run without arguments and when creating a network with default parameters (which sets the DNSEnabled flag to true by default, see also the above mentioned man-page, which lists this command to check: podman network inspect -f {{.DNSEnabled}} <name>).

I created a test VM with the same config (and enabled systemd-resolved in it) and it works there. But still fails on the physical hardware (Raspberry Pi 4B with PFTF's EFI).
Following is the output for what you asked:
$ podman network inspect -f {{.DNSEnabled}} containers_default
true

@thefossguy okay, I stand corrected, this is an actual issue within podman then, with this configuration it should work as far as I can tell.
As you're on the regular release channel you should be getting virtually identical packages to me (on a flake-based setup), though verification would be nice, could you send the output of readlink /nix/var/nix/profiles/per-user/root/channels/nixos as well as the path of podman (e.g. mine is: /nix/store/mbmy1201f8lnsai9fj0mw0jijfi0zs44-podman-4.7.2) just so we're sure we're using the same version.
If we are on the same version then adding aardvark-dns to your path won't be necessary since it's already pulled in via podman indirectly (nix run nixpkgs#nix-tree -- /run/current-system should show […] ⇒ podman-4.7.2 ⇒ podman-helper-binary-wrapper ⇒ { netavark-1.7.0, aardvark-dns-1.8.0 } to be somewhere in there.
You can check whether aardvark-dns is being used by looking in your host-side process list when a container is running, when using --network to run a container there should be an aardvark-dns process.
I tried stracing this on my end, here are some snippets:
# getting the initial request (this was `apt update` running in a debian container)
[pid 16373] 15:29:28.456799 [  45] recvfrom(9, "\270Y\1\0\0\1\0\0\0\0\0\0\3deb\6debian\3org\0\0\34\0\1", 4096, 0, {sa_family=AF_INET, sin_port=htons(58803), sin_addr=inet_addr("<IP_ADDRESS>")}, [128 => 16]) = 32
# creating an outbound socket
[pid 16382] 15:29:28.457109 [  41] socket(AF_INET, SOCK_DGRAM|SOCK_CLOEXEC|SOCK_NONBLOCK, IPPROTO_IP) = 10
# and binding to all IPv4s
[pid 16382] 15:29:28.457199 [  49] bind(10, {sa_family=AF_INET, sin_port=htons(53033), sin_addr=inet_addr("<IP_ADDRESS>")}, 16) = 0
# sending the request onwards
sendto(10, "\224\204\1\0\0\1\0\0\0\0\0\0\3deb\6debian\3org\0\0\1\0\1", 32, MSG_NOSIGNAL, {sa_family=AF_INET, sin_port=htons(53), sin_addr=inet_addr("<IP_ADDRESS>")}, 16) = 32

The <IP_ADDRESS> is not actually in use on my machine (the machine's got two interfaces, one purely IPv6 sitting in a /64, one a single routed /32 IPv4).
<IP_ADDRESS> is what slirp usually uses, which you might also know from running qemu with user networking.
When stracing the slirp4netns process, I also see the request showing up as a read(), followed by this:
openat(AT_FDCWD, "/etc/resolv.conf", O_RDONLY) = -1 ENOENT (No such file or directory)

On a hunch I decided to remove the /etc/resolv.conf symlink and replace it with a file with the exact same content (note: do not use in production, this is just for testing: (cat /etc/resolv.conf && sudo rm /etc/resolv.conf) | sudo sponge /etc/resolv.conf).
The openat() now succeeds and is followed by this:
sendto(3, "\377r\1\0\0\1\0\0\0\0\0\0\3deb\6debian\3org\0\0\34\0\1", 32, 0, {sa_family=AF_INET, sin_port=htons(53), sin_addr=inet_addr("<IP_ADDRESS>")}, 16)
34625

It seems that the culprit here is that the symlink is copied rather than the underlying file which causes the slirp side of things to break.
However this continues one step further.
With resolved on NixOS the /etc/resolv.conf symlink doesn't just point at /run/systemd/resolve/stub-resolv.conf, No, instead it points at /etc/static/resolv.conf and from there points to the one in /run.
Since /etc/static however is also a symlink to the nix store, this is where things probably break.
You see, if /etc/resolv.conf is a file it works, if /etc/resolv.conf is a symlink directly to /run it also works, yet as soon as /etc/static and thereby /nix is involved it does not work anymore.
Now if you look at the slirp documentation I linked above you'll find that there's an interesting --enable-sandbox, one which the slirp4netns process is actually using on my machine:

(since v0.4.0) enter the user namespace and create a new mount namespace where only /etc and /run are mounted from the host.

This sandbox seems to be 100% hardcoded to those directories.
I do not see a lot of solutions here to be honest.
The obvious one is disabling the sandbox (which should be possible for a user in the podman configs, I haven't checked), however that's disabling a meaningful sandbox.
The other one would be to literally patch /nix/store into those mounts on the NixOS side of things.
We could also report this to slirp4netns upstream and see if there's anything there to be done about this, ideally adding a flag for additional directories in the sandbox.
And then there'd be the last one, which is definitely actionable on the NixOS side of things, but I'm not sure if it's feasible, and that's to make the symlink at /etc/resolv.conf point directly at /run instead of via the static symlink part, however that mechanism exists for a reason.

Wow, okay, I should've just checked the issues in slirp4netns for a split second there: https://github.com/rootless-containers/slirp4netns/issues/333

@benaryorg, please use my NixOS config + installer script to create a VM of your own and test it (if you are willing to do so). Please follow these steps to install NixOS with my config in a VM:

Create the VM's disk with a minimum size of 64 GB.
From the live ISO, do a git clone https://github.com/thefossguy/prathams-nixos && cd prathams-nixos && git checkout de8bae6faab3e4cba523f8eb41656bcb12f9fc8a.
Apply this patch to my NixOS config repo, so things like unnecessary containers for testing aren't created (Gitea depends on Caddy and Caddy will fail it it can't find valid certificates! This patch removes those containers.) and the pratham user has the password asdf.
Start the installer script with this command: sudo ./install.sh <drive> reddish virt 2>&1 | tee install.log. ("syntax" for the installer script is in the README. Please make sure that you use the hostname reddish or the podman services will not be enabled.)
Once the installer finishes, reboot and login as pratham with the password asdf. (Enjoy sudo without password, disabled with the aforementioned patch.)
The Gitea server will start after podman-init.service exits cleanly. If podman-init.service hasn't automatically started on the first boot, just reboot. This is a minor issue with user services managed by home-manager.

(Warning: This is a very opinionated NixOS config so there will be some weird things like the first launch of Neovim will install the plugins and the sort. Sorry about that.)
With the checked-out commit, systemd-resolved is disabled. So, with the first generation, DNS resolution will work form the container(s). After adding the line services.resolved.enable = true; to the configuration.nix file and rebooting into the new generation, DNS resolution fails. I tested the patches from my end and hopefully they replicate on your side too. :)

Wow, okay, I should've just checked the issues in slirp4netns for a split second there. An upstream bug report exists (https://github.com/rootless-containers/slirp4netns/issues/333) and this is most likely a duplicate for #231191, which contains a workaround.

Okay this did end up working! :tada:
So yeah, for whatever reason, /etc/resolv.conf is not a symlink to anything with systemd-resolved disabled (openresolv) but it is a link to /run/systemd/resolve/stub-resolv.conf with systemd-resolved enabled.
No need to replicate my NixOS config. This is resolved. Thank you @benaryorg


Frmt type needed

type Frmt= {Bold: bool;
               Italic: bool;
               Strike: bool;
               InlineCode: bool}

I need Frmt to include styling info in ParsedObj, so that the HTML generator can know what to style.

In master, I have replaced them by different types completely. I think this is a better type definition, if you make it recursive in the ParsedObj. This is because InlinedCode cannot be true while all the others are true, and Italic can have bold inside of it or vice versa.
Are you ok with that?

I think that for now though, we should just assume that we can't have bold and italic at the same time and use the token types that I have set in master. We can fix it later when we bring everything together later.


All items should be SVG

If all items are svg, we can use more complex "cursor-pointer" options. See this jsfiddle
We also can drop shadow on demand (while moving for instance).

What about the playing card images?

Not needed for now. Close it.


[RFE] rancher-monitoring 102.0.0+up40.1.2 and higher: Documentation for command line installation / required fields values.yaml

Is your feature request related to a problem? Please describe.

Starting with Rancher 2.7.3 and rancher-monitoring version 102.0.0+up40.1.2 the clusterId must be set to "local" in  helm values for the local cluster, so that the Grafana ingress is working as intended. e.g.
global:
  cattle:
    clusterId: local

This is automatically added, when you install the helm chart on the local Rancher cluster via Rancher Cluster Tools.
But we have fully automated the Rancher cluster installation via Ansible and so we missed this setting when initially upgrading to the new chart version.
Describe the solution you'd like

I have not found any documentation, how to install then rancher-monitoring via command line. e.g.

https://github.com/rancher/charts/tree/dev-v2.7/charts/rancher-monitoring/102.0.1%2Bup40.1.2 is still the upsteam README.md from kube-prometheus-stack
https://github.com/rancher/charts/tree/dev-v2.7/charts/rancher-monitoring has no README.md yet at all.

Can you please add somewhere the settings which are needed for the manual installation.
Describe alternatives you've considered

The helm install/upgrade could fail, if the clusterId is not set.
For this the helm chart would need to be adapted.
Additional context

--

/reopen


Confict with certain namespace in v-for of vue

Bug Report

🔎 Search Terms

key words:

vue

v-for

route

namespace

code intelligence in Vs Code

🕗 Version & Regression Information

This is a namespace confict between Typescript and vue which is not a big problem.
I found it accidently, so i'm confused as well.
This is the behavior in every version I tried, and I reviewed the FAQ for entries about 10 minutes
I was unable to test this on prior versions because I am new to Typescript

⏯ Playground Link

[None playground link available for it matters nothing]
💻 Code

// route is a preserved keyword in Typescript somehow, but here, it's just a name of parameter
<div v-for="(route,index) in routeList">{{ route }} </div>

🙁 Actual behavior
TypeScript alerts me in Vs Code that
Could not find name 'route'. Did you mean 'router'?ts(2570)
Array element destructuring pattern expected.
🙂 Expected behavior

No alert.
In actual, everything works properly in browser even if it's error in Vs Code, which reminds me that this may be a misunderstanding, if you click Quick fix,  the parameter 'route' become 'router' as Typescript suggested, the error will disappear, then press Crtl + Z and 'router' turns back to 'route',  the error which showed before will disappear as well.

@Ryuuki6602 For problems with Vue, it is most probable that the issue comes from the Vue language server that compiles whatever Vue syntax into TypeScript. So you should go there.
Also, for TypeScript related issues with frameworks, it is best to specify the version of the framework you are using, as well as relevant package versions because web development is not pleasant to work with

I don't see anything TS-related here. Unbound identifiers are unbound identifiers?

This is the behavior in every version I tried, and I reviewed the FAQ for entries about 10 minutes

I don’t think this is how you’re meant to fill this field…


Typography suggestions

📝 Description
Created Typistry, an AI Agent which provides fonts and typography suggestions to user by specifying details about their project.
💣 Is this a breaking change (Yes/No):

[ ] Yes
[x] No

🚧 How to migrate?
Don't need.
📝 Additional Information
Closes #2347.

Dear user,
we appreciate your contribution. We have noticed that you just wrote some introduction in the prompt but didn't ask GPT to do anything. You can try to make GPT do certain things according to your requirements. There are many examples available on illa.ai for reference.


TalkBack  focus at incorrect location (V46)

When the user goes from Step 5 to Step 6 at on-boarding screens with TalkBack on, "Turn on COVID-19 Exposure Notification" pops up. When the user selects Cancel OR Turn on, the focus goes to Done at the bottom right the first time they come to this. When they go Back and come back to this page again, then it goes to top like it should. But sometimes it goes to Done. Tried is more than 5 times after reinstalling the app.
Android 10 on OnePlus 7
Android 8.1.0 is fine

Samsung Galaxy J3 Prime
Android version 7.0
App version: 1.0 (46)
Similar issue on Android 7, the focus goes to "Done" button not to the top of the screen.

I am unable to re-create this issue; I have tested this via Android 7 & 8 as well as on the iPhone X. It has worked as intended every time I have tested it with the latest build.

Related to this https://github.com/cds-snc/covid-shield-mobile/issues/722

@jelen-HC
Samsung Galaxy J3 Prime
Android version 7.0
Tested in v50 with Talkback on and the behaviour mentioned above seems to be inconsistent. Some times the focus stays on "Done" and some times it moves to "Step 5 of 5". Noticed there is slight lag in focus moving to top.

Samsung Galaxy J3 Prime
Android version 7.0
Samsung Galaxy S5
Android version 6.0
App version: 1.0 (53)
With Talkback on, after going through onboarding flow initially pressing the "Next" button, when users go back and forth between screens pressing the "Back" and "Next" buttons, the focus remains on the buttons instead of moving to the top of screens.
Expected result
The focus should move to the top of screens.
Screenshots

@JuliannaR can you confirm if this is still and issue?


Support for Python 3.7.x

Hey @jshwi! I came across this tool because of a comment of yours on StackOverflow. I wanted to use your tool for a small project that I'm currently working on, but poetry cannot resolve the dependencies because my tool is built for Python 3.7.2+, and yours for 3.8+.
Is there a reason for not to support Python 3.7? If so, is there anything I can do to help add support for it?

Hey @rcisterna, just looked into this, there is a bit of a dependency hell going on with lowering the min python version to 3.7.
3.7 is end of life in 11 months, I'll have to look into this some more to confirm whether this would be a worthwhile change.

Hey @jshwi! Thanks for your answers!
You are right about the EOL of 3.7.x, I should have checked that first. Anyway, I'll take a quick look at this as well, but if it requires too much effort I don't think it's worth it for just eleven months.

I'll close @rcisterna, I won't get round to supporting 3.7 personally, but this will continue to work with >=3.8.
Thanks for the interest, if you ever got round to it I'd be happy to review a PR


DODatabaseCluster writeConnectionSecretToRef won't create secret

What happened?
How can we reproduce it?
After the provider configuration, I just applied the following spec:
apiVersion: database.do.crossplane.io/v1alpha1
kind: DODatabaseCluster
metadata:
  name: backstage-io
spec:
  forProvider:
    engine: pg
    version: "13"
    numNodes: 1
    size: db-s-1vcpu-1gb
    region: nyc1
    tags:
      - "crossplane"
  providerConfigRef:
    name: digitalocean
  writeConnectionSecretToRef:
    name: backstage-db-conn
    namespace: backstage-io

Which created the DO DatabaseCluster correctly, but doesn't created the secret with the resource credentials.
I checked and the resource was ready and synced and the namespace exists.
What environment did it happen in?
CrossPlane: 1.6.3
DoProvider: v0.1.0

Hello, thanks for creating this! Seems like it was glanced over when creating these resources. I'll get right on adding this!

Hello, following the documentation I can still see this error with the latest helm chart v1.10.1
---
apiVersion: database.do.crossplane.io/v1alpha1
kind: DODatabaseCluster
metadata:
  name: redis-test-to-be-deleted
spec:
  forProvider:
    engine: redis
    version: "7"
    numNodes: 1
    size: db-s-1vcpu-1gb
    region: fra1
    tags:
      - "from-crossplane"
      - "redis"
  writeConnectionSecretToRef:
    namespace: crossplane-system
    name: redis-test
  providerConfigRef:
    name: do-provider-config

I can see the database is created in the DO UI, and the resource is Sync & Ready, but no secret redis-test is created in any namespace.


NUTCH-2378 ChildFirst plugin classloader

(contributed by Jurian Broertjes)

Nice

Yes, definitely. Solves some of the most ugliest bugs (eg. NUTCH-2316) and allows to clean up the plugin dependencies.


Add test to assert exception message

Related to #281

@joneslee85 In the original implementation of #281, @Moratorius used concrete examples of exceptions that Sequel can raise. They carry on useful messages for developers.
There is no value for them to see "Sequel::Error" without having a clue on what to do.
This PR should solve this problem. Can you please try to raise realistic exceptions and check if the returning message is the same? Thank you. :+1:
See https://github.com/Acornsgrow/model/commit/1db52737f0a3c00964f73ba7e94e1f7433b9c6f8#diff-1ad76a79f04e29c81971a6206b63b136R85

@joneslee85 But we aren't testing against Sequel::CheckConstraintViolation and others exceptions. That was the point of this cleanup :wink:

@joneslee85 Sweet, thank you very much :+1:


Unable to export all Journeys to a specfic folder

frodo -v
A new version of frodo is available.
Please run 'frodo -v' for more details.
You seem to be running the NPM package
Installed versions:
cli: v0.19.4
lib: v0.17.6
node: v18.13.0
Newer version(s) available
cli (npm): v0.21.1
Describe the issue
Tried this command
frodo journey export -D frodo/clapman -A clapman
and it puts it under root dir

Hi @rclapman , a similar request was placed a while ago but has not yet been implemented: rockcarver/frodo-lib#26

Fixed in frodo-cli v2.0.0-23 and later.


can you please share image dataset

can you please share image data set link or files to me. i need you dataset immediateliy


Enable recursive CTEs in ZetaSQL

This PR enables language feature FEATURE_V_1_3_WITH_RECURSIVE. It is needed to support recursive CTEs.

This PR enables language feature FEATURE_V_1_3_WITH_RECURSIVE. It is needed to support recursive CTEs.

What is CTE?

This PR enables language feature FEATURE_V_1_3_WITH_RECURSIVE. It is needed to support recursive CTEs.

What is CTE?

Common Table Expressions.


Hash helper {{#form-for model as |f|}}

The future is here! :fireworks:
Please, don't merge this PR!!
This is a test using the new hash helper available on Ember 2.3. As you can see, it is very simple to support the new syntax.
{{#form-for model as |f|}}
  {{f.input "firstName"}}
  {{f.input "lastName"}}
   <input type="submit">
 {{/form-for}}

For the record, there is an issue related to positional params on Ember 2.13: https://github.com/emberjs/ember.js/issues/12762
We get this error:
Assertion Failed: You cannot specify both a positional param (at position 0)
and the hash argument `propertyName`.

@diogomafra as long as the pre-existing API is not affected then I'm :+1: on this for 1-0-stable. I would prefer to maintain semver compatibility and not break backwards compatibility. It seems from the change in the test this might not be the case?
FWIW, this syntax is the direction I've had in mind for about two years for easy-form 2.0.

@bcardarella It's possible to support both APIs. I just changed the test to demonstrate that the scoped helper is working.
To finish this PR we need:

to fix this issue https://github.com/emberjs/ember.js/issues/12762
make it work with previous versions of Ember that don't have the hash helper
test the new api
update the documentation

I'm going to keep this PR here until Ember 2.3 lands, with the issue fixed.

"make it work with previous versions of Ember that don't have the hash helper"
this might help https://github.com/cibernox/ember-hash-helper-polyfill

@jcope2013 this is very nice. Thank you for pointing this addon.

@diogomafra: I think this is not what is causing your problem. I think your problem is in these lines. You can see that knownProperties are copied to the options causing the positional param to already be there.

Sorry, I thought I was writing last comment in emberjs/ember.js#12762
The first this refers to that issue.
Sorry again.

@Serabe the test that is failing does not use this handlebar's keyword, but it has a similar code: https://github.com/diogomafra/ember-easy-form/blob/hash-helper/addon/components/internal-input-for.js#L96
In that code, all "known properties" are removed, "propertyName" among them, so, it's not including this value twice.
I will try to create a failing test on Ember's repo to demonstrate the problem. Then we can be sure it's on Ember and it will be easier to fix.
Thank you!

If you are in the Ember community Slack, ping me to take a deeper look at this, please.

@diogomafra I updated the other issue, but I think this test will pass in 2.3 beta 4 / 2.3.0. There was a fix for the same issue added a day after beta 3 was released.

@Serabe using the canary build instead of beta, it works. Thank you very much for your help.

You're welcome! :+1:

Closing in favor of #229


Bluetooth: Controller: ISO: Retry sync to BIG after incorrect broadcast code only provides a single SDU

Describe the bug
If an application provides the incorrect broadcast code when syncing to an encrypted BIG, then retrying with the correct code correctly syncs to the BIG but only a single SDU per BIS is received.
To Reproduce
Steps to reproduce the behavior:

Checkout https://github.com/zephyrproject-rtos/zephyr/pull/77642
Build and run https://github.com/zephyrproject-rtos/zephyr/blob/main/tests/bsim/bluetooth/audio/test_scripts/bap_broadcast_audio_encrypted.sh
See error only a single SDU received and the test fails

Expected behavior
Expect that providing the correct code after a sync failure will work. Resyncing to a BIG generally works if both attempts work.
Impact
Seems to be an irreversible state from the host, so quite significant.
Logs and console output
d_01: @00:00:22.825207  [00:00:22.825,195] <dbg> bt_iso: hci_le_big_sync_established: BIG[0] 0x825b620 sync established, status 0x00
d_01: @00:00:22.825207  [00:00:22.825,195] <dbg> bt_iso: bt_iso_connected: 0x825b2a0
d_01: @00:00:22.825207  [00:00:22.825,195] <dbg> bt_iso: bt_iso_chan_set_state_debug: chan 0x825ff40 iso 0x825b2a0 connecting -> connected
d_01: @00:00:22.825207  [00:00:22.825,195] <dbg> bt_bap_broadcast_sink: broadcast_sink_iso_connected: stream 0x82535c0
d_01: @00:00:22.825207  [00:00:22.825,195] <dbg> bt_bap_broadcast_sink: broadcast_sink_set_ep_state: ep 0x8262600 id 0x00 qos-configured -> streaming
d_01: @00:00:22.825207  Stream 0x82535c0 started
d_01: @00:00:22.825207  Could not get octets per frame: -61
d_01: @00:00:22.825207  [00:00:22.825,195] <dbg> bt_iso: bt_iso_connected: 0x825b37c
d_01: @00:00:22.825207  [00:00:22.825,195] <dbg> bt_iso: bt_iso_chan_set_state_debug: chan 0x825ffd0 iso 0x825b37c connecting -> connected
d_01: @00:00:22.825207  [00:00:22.825,195] <dbg> bt_bap_broadcast_sink: broadcast_sink_iso_connected: stream 0x8253604
d_01: @00:00:22.825207  [00:00:22.825,195] <dbg> bt_bap_broadcast_sink: broadcast_sink_set_ep_state: ep 0x826276c id 0x00 qos-configured -> streaming
d_01: @00:00:22.825207  Stream 0x8253604 started
d_01: @00:00:22.825207  Could not get octets per frame: -61
d_01: @00:00:22.825207  Waiting for data
d_01: @00:00:22.837789  [0]: Incoming audio on stream 0x82535c0 len 30 and ts 22830022
d_01: @00:00:22.837789  [0]: Incoming audio on stream 0x8253604 len 30 and ts 22830022
d_00: @00:00:23.107790  Sent with seq_num 2300
d_00: @00:00:23.107790  Sent with seq_num 2300
d_00: @00:00:24.107790  Sent with seq_num 2400
d_00: @00:00:24.107790  Sent with seq_num 2400
d_00: @00:00:25.107790  Sent with seq_num 2500
d_00: @00:00:25.107790  Sent with seq_num 2500
d_00: @00:00:26.107790  Sent with seq_num 2600

Environment (please complete the following information):

Commit SHA or Version used: eaf92a1494cc0e245f14f22ad1d1d42f363a07b3

Additional context
N/A

This seems to have been fixed at some point - At least the above PR is now passing. Closing the issue


1072 support running tidy with compilation db

This PR adds an option to Clang Tidy settings to specify path to directory where compile_commands.json file is located.  It is passed to clang-tidy.exe as -p option instead of compilation flags built by clang power tools.
This allows to use Clang Power Tools with projects generated by Unreal Engine.
This should resolve issue #1072

Thank you @AGTomasz for your contribution


Notification on chat

This looks like an application after my own heart! Great job.
It seems like you have a lot of infrastructure in place already, so I thought to ask you if you could extend the application to also give a notification upon new chat messages for user(s)? It's functionality that I've always wanted, but no one seems to have made it (except Chatty, which is a full featured chat/stream management app).

That is very much doable as twitch exposes chat over websocket or irc endpoints, and it is documented here: https://dev.twitch.tv/docs/chat/irc/
But I don't see how implementing that in application like TwitchNotify can be done in a sane way. You would not want to receive each chat message as notification - that would be too crazy to manage. So it would mean creating separate gui to view messages? I'm not sure I want to change TwitchNotify too much for this.
You can easily run any IRC client & connect to all the streamers you want, and get all the messages that way or even send messages back in a chat. All that without even opening browser.

Good info.
My use case is as follows:
I use OBS to stream on Twitch and have the chat docked to the side in OBS. To check chat I just quickly Alt-Tab to OBS and back to whatever I was doing. But problem is that I'd much rather be notified when there is a message for me to read. I have low chat traffic, so it's a catch-22: Either check often and do so in vain many times, or check seldom and miss the little traffic that does occur. A little notification would be perfect. A GUI is not what I need--I'd use Chatty or something like it if that was so.
Although I don't personally face any craziness to manage, a single, simple checkbox in the tray icon to enable/disable chat notifications* would be enough to deal with a spike in chat traffic leading to notification spam.
"On/off, and check elsewhere."

A small extension of this idea could be an additional timed option to disable chat notifications for 5/10/15 minutes to deal with overwhelming spikes.


Can no longer build drift: Could not resolve annotation for [various]

We can no longer compile via build_runner on Flutter 3.7.3, drift 2.5.0 and drift_dev 2.5.2.
flutter cleans, dependency upgrades and various Flutter and dependency downgrades don't appear to have any effect.
We have no idea what the issue is or how to go about fixing it. Can you help?
build.yaml:
targets:
  $default:
    builders:
      drift_dev:
        options:
          mutable_classes: true

flutter pub run build_runner build:
line 1, column 7124 of package:app/ios/.symlinks/plugins/webview_flutter_wkwebview/pigeons/web_kit.dart: Could not resolve annotation for `abstract class WKWebsiteDataStoreHostApi`.
  ╷
1 │ @HostApi(dartHostTestHandler: 'TestWKWebsiteDataStoreHostApi')
  │ ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  ╵
[SEVERE] drift_dev on lib/ios/.symlinks/plugins/shared_preferences_foundation/pigeons/messages.dart (cached):

line 1, column 204 of package:app/ios/.symlinks/plugins/shared_preferences_foundation/pigeons/messages.dart: Could not resolve annotation for `abstract class UserDefaultsApi`.
  ╷
1 │ ┌ @ConfigurePigeon(PigeonOptions(
2 │ │   dartOut: 'lib/messages.g.dart',
3 │ │   dartTestOut: 'test/test_api.g.dart',
4 │ │   swiftOut: 'darwin/Classes/messages.g.swift',
5 │ │   copyrightHeader: 'pigeons/copyright_header.txt',
6 │ └ ))
  ╵
[SEVERE] drift_dev on lib/ios/.symlinks/plugins/video_player_avfoundation/pigeons/messages.dart (cached):

line 1, column 204 of package:app/ios/.symlinks/plugins/video_player_avfoundation/pigeons/messages.dart: Could not resolve annotation for `class TextureMessage`.
   ╷
1  │ ┌ @ConfigurePigeon(PigeonOptions(
2  │ │   dartOut: 'lib/src/messages.g.dart',
3  │ │   dartTestOut: 'test/test_api.dart',
4  │ │   objcHeaderOut: 'ios/Classes/messages.g.h',
5  │ │   objcSourceOut: 'ios/Classes/messages.g.m',
6  │ │   objcOptions: ObjcOptions(
7  │ │     prefix: 'FLT',
8  │ │   ),
9  │ │   copyrightHeader: 'pigeons/copyright.txt',
10 │ └ ))
   ╵
[SEVERE] drift_dev on lib/ios/.symlinks/plugins/path_provider_foundation/pigeons/messages.dart (cached):

line 1, column 542 of package:app/ios/.symlinks/plugins/path_provider_foundation/pigeons/messages.dart: Could not resolve annotation for `abstract class PathProviderApi`.
  ╷
1 │ @HostApi(dartHostTestHandler: 'TestPathProviderApi')
  │ ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  ╵
[SEVERE] Failed after 451ms
pub finished with exit code 1

Is there supposed to be a hidden ios/.symlink folder in lib/?
You could probably work around this issue by adding this to your build.yaml:
targets:
  $default:
    sources:
       exclude:
         - "lib/ios/.symlinks/**"
    builders:
      drift_dev:
        options:
          mutable_classes: true

This error probably comes from the fact that other packages are using annotations not in your local package graph used by build_runner because their sources were blindly copied into lib/. Still, it would be interesting to know where that comes from.

@simolus3 Interesting. It looks like this is a side effect of running certain Flutter commands while in the ios folder. Just deleting that folder just fixed it. Thanks a ton!


Create New Issue & New branch for new PR and Follow the Rules in README.md for #hactoberfest-accepted Label

I wish to contribute. Kindly assign me this.

Create New Issue & New branch for you to be assigned @saku1331

@HarshwardhanPatil07 Please assign me this issue

@ankitkumar748846 Create new issue and then i will assign you!

Hey @HarshwardhanPatil07
I have found a new issue in your Readme.md file
Please assign me this task, I will make your Readme.md file more attractive and error free

create new issue @Shubham23011 Make sure you star the repo as per Guideline for hactoberfest this repo!

Ok @HarshwardhanPatil07 , I have star the repo, can you now assign this task to me.
It will be a great opportunity for me and will help me in growing.

@HarshwardhanPatil07 I have created a new issue as already said in my previous comment.

@Shubham23011 there are no new issues
please create whatever you want by clicking new issue button!

Thanks @HarshwardhanPatil07

@Shubham23011 Follow code of conduct by Star the Repo!

please assign me new issue in your repo it will be my first projecct of open source

Hey @HarshwardhanPatil07  can You assign me in this issue and I will try to make your readme most unique ?

@HarshwardhanPatil07 it will be really helpful if you'll assign me this issue as my first open source contribution.

@Kirti-Pant you can directly send valid PR to get merged!
#hactoberfest

@priyanka0921 changes in readme not allowed in this repo!

Please assign this to me

@Pritee-gupta-jpg changes in this issue not allowed in this repo!

pls assign this issue to me @HarshwardhanPatil07 , with hacktoberfest label.

@HarshwardhanPatil07 can you add hacktoberfest2024 label for this issue?


Filter crashes when the input schema contains a map

Describe the bug
DataFusion v33 panics & returns an error in filter when the input schema contains a map because ScalarValue doesn't support maps.
Just making a simple memory table with a map field and querying it with a simple where clause produces this error when creating a physical plan:
Context("EnforceDistribution", NotImplemented("Can't create a scalar from data_type \"Map(Field { name: \"entries\", data_type: Struct([Field { name: \"key\", data_type: Int64, nullable: false, dict_id: 0, dict_is_ordered: false, metadata: {} }, Field { name: \"value\", data_type: Int64, nullable: true, dict_id: 0, dict_is_ordered: false, metadata: {} }]), nullable: false, dict_id: 0, dict_is_ordered: false, metadata: {} }, false)\""))

Creating an explain plan panics in the Display implementation
thread 'main' panicked at /rustc/cc66ad468955717ab92600c770da8c1601a4ff33/library/alloc/src/string.rs:2460:14:
a Display implementation returned an error unexpectedly: Error
stack backtrace:
   0: rust_begin_unwind
             at /rustc/cc66ad468955717ab92600c770da8c1601a4ff33/library/std/src/panicking.rs:595:5
   1: core::panicking::panic_fmt
             at /rustc/cc66ad468955717ab92600c770da8c1601a4ff33/library/core/src/panicking.rs:67:14
   2: core::result::unwrap_failed
             at /rustc/cc66ad468955717ab92600c770da8c1601a4ff33/library/core/src/result.rs:1652:5
   3: core::result::Result<T,E>::expect
             at /rustc/cc66ad468955717ab92600c770da8c1601a4ff33/library/core/src/result.rs:1034:23
   4: <T as alloc::string::ToString>::to_string
             at /rustc/cc66ad468955717ab92600c770da8c1601a4ff33/library/alloc/src/string.rs:2459:9
   5: datafusion_physical_plan::display::DisplayableExecutionPlan::to_stringified
             at /home/helgi/.cargo/registry/src/index.crates.io-6f17d22bba15001f/datafusion-physical-plan-33.0.0/src/display.rs:206:41
   6: datafusion::physical_planner::DefaultPhysicalPlanner::handle_explain::{{closure}}
             at /home/helgi/.cargo/registry/src/index.crates.io-6f17d22bba15001f/datafusion-33.0.0/src/physical_planner.rs:1897:29
   7: <datafusion::physical_planner::DefaultPhysicalPlanner as datafusion::physical_planner::PhysicalPlanner>::create_physical_plan::{{closure}}
             at /home/helgi/.cargo/registry/src/index.crates.io-6f17d22bba15001f/datafusion-33.0.0/src/physical_planner.rs:445:64

To Reproduce
Here's a simple repro
use std::sync::Arc;

use datafusion::{
    arrow::datatypes::{DataType, Field, Schema},
    datasource::MemTable,
    prelude::SessionContext,
};

#[tokio::main]
async fn main() {
    let ctx = SessionContext::new();

    let key = Field::new("key", DataType::Int64, false);
    let value = Field::new("value", DataType::Int64, true);
    let map_field = Field::new("entries", DataType::Struct(vec![key, value].into()), false);
    let fields = vec![
        Field::new("a", DataType::Int64, true),
        Field::new("map", DataType::Map(map_field.into(), false), true),
    ];
    let schema = Schema::new(fields);

    let memory_table = MemTable::try_new(schema.into(), vec![vec![]]).unwrap();

    ctx.register_table("tbl", Arc::new(memory_table)).unwrap();

    // This panics in the display implementation, removing the explain causes
    // this to return the not implemented error described above instead.
    ctx.sql("explain select * from tbl where a > 0;")
        .await
        .unwrap()
        .collect()
        .await
        .unwrap();
}

I have a more complicated query within a more complex system that panics
in the equivalence_properties function. I don't have a small repro. I'll work on making a small repro for it and then I'll update the issue.
Expected behavior
This seems to be a regression as this worked in earlier releases (specifically v32).
Additional context
No response

If this is still relevant, I would like to take a look at this?

If this is still relevant, I would like to take a look at this?

THanks @razeghi71  -- the first step is probably to see if the reproducer on this ticket still panics

This problem has been introduced in https://github.com/apache/arrow-datafusion/commit/c9330bc09e583734aed721a208a0684118b170f1 which changed the default empty value for lower and upper bound from ScalarValue::Null to ScalarValue::try_from(field.data_type()). which is not implemented for map, to fix it we have to either implement it for Map or fallback on Null value. I have done these two in two draft PRs I have above.
In the first one I tried to implement the empty value for map as described in the description of Map. and in the second one I fall back to Null. Would be great to get some feedback to see what you folks think? And what is the good approach here?

This problem has been introduced in c9330bc which changed the default empty value for lower and upper bound from ScalarValue::Null to ScalarValue::try_from(field.data_type()). which is not implemented for map, to fix it we have to either implement it for Map or fallback on Null value. I have done these two in two draft PRs I have above.

I think implementing it for Map is what makes the most sense to me
However, I think this may be a non trivial change as it will require implementing a ScalarValue::Map variant which does not see to exist
https://github.com/apache/arrow-datafusion/blob/7f312c8a1d82b19f03c640e4a5d7d6437b2ee835/datafusion/common/src/scalar.rs#L102
l I think ScalarValue::Map can follow the model of ScalarValue::List (as Maps are just lists of structs, as I undertand):
enum ScalarValue {
  ...
  /// one element array
  Map(ArrayRef)
}

In the first one I tried to implement the empty value for map as described in the description of Map which returns a List of key values as a struct called entries.


Correction for raw affiliation University of Montpellier, AgroParisTech,CIRAD, CNRS, INRAE, TETIS,Montpellier,France

Correction needed for raw affiliation University of Montpellier, AgroParisTech,CIRAD, CNRS, INRAE, TETIS,Montpellier,France
raw_affiliation_name: University of Montpellier, AgroParisTech,CIRAD, CNRS, INRAE, TETIS,Montpellier,France
new_rors: 05kpkpg04;02kbmgc12;051escj72;02feahw73;003vg9w96;0458hw939
previous_rors: 05kpkpg04;02kbmgc12;051escj72;02feahw73;003vg9w96
works_examples: W4401808517
contact: 807ea33e7ec8095fc0371afb88b83c20:6c8e78e90535970ce6e72e609edd04254650 @ inrae.fr

This issue was accepted and ingested by the OpenAlex team on 2024-12-19. The new affiliations should be visible within the next 7 days.


Add "Placeholder" withdraw option to the Menu Entry Swapper's "Bank W…

Add "Placeholder" withdraw option to the Menu Entry Swapper's "Bank Withdraw Shift-Click" menu.

Why does the comment say choose "Withdraw-all" twice?


Add karma-jasmine with travis

#39

Add karma & jasmine to execute test under the test/unit folder
Add screenshot testing to test examples

@davidmanzanares I reverted the changes so please move this forward

I'm seeing this:
$ yarn test:acceptance
yarn run v1.3.2
$ mocha test/acceptance/index.test.js --timeout 15000

  Screenshot tests:

   ┌───────────────────────────────────────────────────┐
   │                                                   │
   │   Serving!                                        │
   │                                                   │
   │   - Local:            http://localhost:5555       │
   │   - On Your Network:  http://<IP_ADDRESS>:5555   │
   │                                                   │
   │   Copied local address to clipboard!              │
   │                                                   │
   └───────────────────────────────────────────────────┘

    1) multi-layer
    2) single-layer
    3) cielab
    4) default-all
    5) float
    6) floatMul
    7) hsv
    8) now
    9) opacity
    10) rampNumericAuto
    11) rgba

  0 passing (8s)
  11 failing

  1) Screenshot tests:
       multi-layer:
     Error: Error: Failed to navigate: http://localhost:5000/example/basic/multi-layer.html
      at Object.test (node_modules/exquisite-sst/src/exquisite.js:22:15)
      at <anonymous>
      at process._tickCallback (internal/process/next_tick.js:188:7)

  2) Screenshot tests:
       single-layer:
     Error: Error: Failed to navigate: http://localhost:5000/example/basic/single-layer.html
      at Object.test (node_modules/exquisite-sst/src/exquisite.js:22:15)
      at <anonymous>
      at process._tickCallback (internal/process/next_tick.js:188:7)

  3) Screenshot tests:
       cielab:
     Error: Error: Failed to navigate: http://localhost:5000/example/styling/cielab.html
      at Object.test (node_modules/exquisite-sst/src/exquisite.js:22:15)
      at <anonymous>
      at process._tickCallback (internal/process/next_tick.js:188:7)

  4) Screenshot tests:
       default-all:
     Error: Error: Failed to navigate: http://localhost:5000/example/styling/default-all.html
      at Object.test (node_modules/exquisite-sst/src/exquisite.js:22:15)
      at <anonymous>
      at process._tickCallback (internal/process/next_tick.js:188:7)

  5) Screenshot tests:
       float:
     Error: Error: Failed to navigate: http://localhost:5000/example/styling/float.html
      at Object.test (node_modules/exquisite-sst/src/exquisite.js:22:15)
      at <anonymous>
      at process._tickCallback (internal/process/next_tick.js:188:7)

  6) Screenshot tests:
       floatMul:
     Error: Error: Failed to navigate: http://localhost:5000/example/styling/floatMul.html
      at Object.test (node_modules/exquisite-sst/src/exquisite.js:22:15)
      at <anonymous>
      at process._tickCallback (internal/process/next_tick.js:188:7)

  7) Screenshot tests:
       hsv:
     Error: Error: Failed to navigate: http://localhost:5000/example/styling/hsv.html
      at Object.test (node_modules/exquisite-sst/src/exquisite.js:22:15)
      at <anonymous>
      at process._tickCallback (internal/process/next_tick.js:188:7)

  8) Screenshot tests:
       now:
     Error: Error: Failed to navigate: http://localhost:5000/example/styling/now.html
      at Object.test (node_modules/exquisite-sst/src/exquisite.js:22:15)
      at <anonymous>
      at process._tickCallback (internal/process/next_tick.js:188:7)

  9) Screenshot tests:
       opacity:
     Error: Error: Failed to navigate: http://localhost:5000/example/styling/opacity.html
      at Object.test (node_modules/exquisite-sst/src/exquisite.js:22:15)
      at <anonymous>
      at process._tickCallback (internal/process/next_tick.js:188:7)

  10) Screenshot tests:
       rampNumericAuto:
     Error: Error: Failed to navigate: http://localhost:5000/example/styling/rampNumericAuto.html
      at Object.test (node_modules/exquisite-sst/src/exquisite.js:22:15)
      at <anonymous>
      at process._tickCallback (internal/process/next_tick.js:188:7)

  11) Screenshot tests:
       rgba:
     Error: Error: Failed to navigate: http://localhost:5000/example/styling/rgba.html
      at Object.test (node_modules/exquisite-sst/src/exquisite.js:22:15)
      at <anonymous>
      at process._tickCallback (internal/process/next_tick.js:188:7)

error Command failed with exit code 11.

A chrome window is launched, but it closes after just a few miliseconds

Problems I see with this PR:

It requires Node.js >= 7.6.0. We need to run this in Node.js 6 LTS if possible.
It is really difficult to see what changed when a test fails. I would expect to have a diff image for that purpose.

Regarding CI
After a quick research it looks like is going to be hard to configure travis to use GPU testing but I saw this:
https://circleci.com/docs/2.0/gpu/
It looks like circleci allows GPU builders, should we give it a try? cc @rochoa


Changes Version of 'active_model_serializers' Dependency

At present, spree_wombat depends on the 0.9.x branch of active_model_serializers. Development along this branch appears to be defunct, with the rails-api team opting to work on an 0.10.x branch based off the 0.8.x branch. Phew, confusing!
Given the roadmap for AMS, I propose that it makes more sense to depend on the 0.8.x branch than the 0.9.x branch. This PR changes the version of active_model_serializers on which spree_wombat depends, and small changes to the serializers to keep the test suite green. My PR is targets the 2-4-stable branch—please let me know if this maps to your contribution guidelines.
Looking forward to comments/feedback. Thanks!

I think these changes are okay, but I'm not sure we should back port them.

+1 for moving this to master only given everything we need is still working on 2-4-stable

I had based my changes on 2-4-stable, because I'm working on a storefront that uses 0.8.x. I will rebase to master for this PR, and maintain my own fork.

Thanks @DanielWright

I'm closing this, as I can't change the branch being targetted.


use-the-odata-connector-for-sap-solutions

Please use the form below, leaving the prefilled data to help us. Thank you.
Page link: use-the-odata-connector-for-sap-solutions
Document link: use-sap-odata-connector.md
My Issue/Suggestion
products instead of customers :
In the microflow, make the return value of the microflow List of Products. This is so that you can call the microflow as a data source in a page.

Thanks for spotting this error.
I have corrected it in the commit https://github.com/mendix/docs/commit/f09c9190be30ed4bfdcbf70b95eaa9cb9735abf0
Thanks for taking the time to contact us - we rely on customers to tell us about errors they see and make the documentation of Mendix better for everyone.
Yours
Mark van Ments.


Virtualbox VM GUI remains black

I've played a bit around with Packer and Virtualbox, but while vagrant ssh works as expected, the GUI remains black.
My Vagrantfile looks like this
Vagrant.configure("2") do |config|
  config.vm.box = "centos7-desktop"
  config.vm.box_version = "0"
  config.vm.hostname = "centos-dev"
  config.vm.synced_folder '.', '/vagrant', type: :virtualbox

  config.vm.provider :virtualbox do |vb|
    vb.name = "centos-dev"
    vb.gui = true
    vb.cpus = 2
    vb.memory = 4096
  end

  #config.vm.provision "shell", inline: <<-SHELL
  #	sudo yum -y groupinstall "GNOME Desktop"
  #	sudo systemctl set-default graphical.target
  #	sudo systemctl start graphical.target
  #SHELL
end

I am experiencing this as well, seems to be when centos was upgrad to 7.5.  Interestingly while packer is building the GUI works fine, it's when I vagrant up with a new build there is an issue.