Created disabled everything before date

I want to create two inputs and to work as Range:

END date picker:

Can't be future (biggest value is new Date() - now/today)
Can't be smaller then START date picker

START  date picker:

The biggest value can be today;
Can't be bigger then today

END & START - Can be the same day
How to define -infinity and +infinity? (I know there is no real infinity) I just don't want to hardcode the values 0 and 9999
Template:
<datepicker placeholder="Select date" :disabled="startDate.disabled" :value="start" name="start" v-on:selected="updateStart"></datepicker>
<datepicker placeholder="Select Date" :disabled="endDate.disabled" :value="end" name="end" v-on:selected="updateEnd"></datepicker>
Script:
startDate: {
  disabled: {
    from: date,
  }
},
endDate: {
    disabled: {
        ranges: [{
		from: new Date(0, 0, 0),
		to: new Date(0, 0, 0)
	},
	{
		from: date,
		to: new Date(9999, 11, 1) // How to define infinity?
	}]
    }
}

Methods:
updateStart(value) {
	this.start = value;
	this.endDate.disabled.ranges[0].to = this.start
},
updateEnd(value) {
	this.end = value;
	this.startDate.disabled.from = this.end;
}

Using this code this works good. But I want to created it better if its possible.
Please assist me, thank you!

highest possible date to infinity? Maybe set date from UTC timestamp with Number.MAX_SAFE_INTEGER. Or, use a custom predictor function as described in the release https://github.com/charliekassel/vuejs-datepicker/pull/322 . Note that in order to make it reactive to reflect new "disabled" values, you need some "dirty" appraoches to get it to force refresh again when necessary, as described in the post.


Icon request: Betterment

URL of the site or app (required): https://www.betterment.com/
Logo resources for the icon (optional, but recommended): https://www.betterment.com/press/

Here you go!
https://github.com/krisu5/aegis-icons/blob/master/PNG/Betterment.png
https://github.com/krisu5/aegis-icons/blob/master/PNG/Variations/Betterment bg.var.png


How to use sequential module for a frozen part and trainable part.

Hi, I am new to Mxnet, and I want to use sequential module for this specific task. Suppose I have a feature extraction network, which is fixed and takes no gradient. In other words, I use this as "grad_req='null' and is_train=False". The second network is trainable with several layers. I saw there is a argument "fixed_param_names". Does it mean the parameters only get lr=0? If yes, it will be not very sufficient, because there is no need for the frozen part to calculate gradients and so on. Any suggestions? really appreciate it.

This issue is closed due to lack of activity in the last 90 days. Feel free to ping me to reopen if this is still an active issue. Thanks!


Problems with v2.9 installation (window closes)

I follow the steps to install the software... after I define the password to encrypt the wallet it start to charge and finally the window closes, and the process closes in the Windows Task Manager.
When I open the folder %appdata%/ElectronCash there are some files but no a .exe.
Previously I uninstall my electrum software to ensure a good installation of electron cash.
Also the checksum of the downloaded exe is ok.
This is on Windows 10.

this is provided as a stand alone EXE not an installation

then what I need to do to open the software

Are you on windows?  Just double click the exe file.

I'm on windows 10. When I double click the exe file nothing happen. If I see it in the Windows Task Manager the program starts but after a few seconds it closes.

check anti virus stuff


building windows wheels fails due to a change in vcpkg

PyBaMM version - $ python -m pip show pybamm: 21.08
Python version - $ python --version: n/a

Describe the bug
The .github/workflows/build_wheels.yml workflow fails when building the windows wheel, with the following error:
---------- Running CMake for idaklu solver ----------------------------------------
    -- Building for: Visual Studio 16 2019
    -- Running vcpkg install
    Error<EMAIL_ADDRESS>does not have required feature klu

    -- Running vcpkg install - failed
    CMake Error at C:/vcpkg/scripts/buildsystems/vcpkg.cmake:827 (message):
      vcpkg install failed.  See logs for more information:
      C:\Users\runneradmin\AppData\Local\Temp\pip-req-build-_9f138ur\build\temp.win32-3.6\vcpkg-manifest-install.log
    Call Stack (most recent call first):
      C:/Program Files/CMake/share/cmake-3.21/Modules/CMakeDetermineSystem.cmake:124 (include)
      CMakeLists.txt:18 (project)

    -- Configuring incomplete, errors occurred!

To Reproduce
merge to main
Expected behaviour
pypi wheels are build and deployed to pypi

perhaps the problem is with the cache step:
Run actions/cache@v2
  with:
    path: C:\Users\runneradmin\AppData\Local\vcpkg\archives
    key: Windows-build-vckpg_binary_cache-7fbb1233e518b325804dd94a6631598e7affcb2ebdc79c1a25718b88c68ed9c3
  env:
    pythonLocation: C:\hostedtoolcache\windows\Python\3.9.6\x64
    cache-name: vckpg_binary_cache
Cache not found for input keys: Windows-build-vckpg_binary_cache-7fbb1233e518b325804dd94a6631598e7affcb2ebdc79c1a25718b88c68ed9c3

cache might not exist cause https://github.com/pybamm-team/PyBaMM/blob/main/vcpkg.json has changed

That's right - but I don't think a cache miss is a problem, a new cache would be created should the workflow be successful
https://docs.github.com/en/actions/guides/caching-dependencies-to-speed-up-workflows#using-the-cache-action

The cache action will attempt to restore a cache based on the key you provide. When the action finds a cache, the action restores the cached files to the path you configure.
If there is no exact match, the action creates a new cache entry if the job completes successfully. The new cache will use the key you provided and contains the files in the path directory.

I'm more surprised about this line though (within the CMake output)
   Error<EMAIL_ADDRESS>does not have required feature klu

The Sundials port in pybamm-team/sundials-vcpkg-registry has the klu feature. Is it now somehow not finding it? Is it because of bfed1b ?

Also, see this error https://github.com/pybamm-team/PyBaMM/runs/3566774115?check_suite_focus=true#step:9:114

@priyanshuone6 , what was the baseline addition in bfed1b? Looks like this was to fix the macosx builds, but I'm confused as I don't think macosx uses vcpkg?

The workflow was failing earlier saying that it required field 'baseline' which was missing (here)


[Enhancement] Add graphics for cluster structure and services

Add ability to see the cluster schematics,
for example if the cluster have
3 data nodes of 128GB RAM/16VCPU
2 index node - 128GB RAM/8VCPU
2 query - 32GB RAM/16VCPU
2 FTS - 32GB RAM/16VCPU
2 Analytics  - 32GB RAM/32VCPU
Present the cluster in a graphics way to easy understand which cluster is in the works

+1


404 not found

Hi guys.
Your idea is very well, but I found bug.
When I try to generate plugin via command sao template-kibana-plugin I catched error
npm ERR! 404 Registry returned 404 for GET on https://registry.npmjs.org/template-template-kibana-plugin
But when I change command to sao kibana-plugin , generator work fine.

Whoops, you're right, that's a typo. I'll get that fixed, thanks!


prevent setting random.seed in outer scope

np.random.seed(seed) in _make_n_folds(...) has a side effect setting random seed for outer scope

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot


Please treat umode +g notices as highlights or something more noticable

I think it's numeric 718. nick ident@host:is messaging you, and you have umode +g.
Currently I never notice when I get PM attepts without the person pinging me on the channel.
More about umode +g:

https://mikaela.info/english/2015/04/02/umode+g.html

Please add steps about how to reproduce this. I get notifications for all personal messages ― what's the difference here?

/mode yournick +g
be actively talking on channel or query or for some reason just don't look at the server tab.
ask someone to PM you, as you are doing the 2. you don't notice that someone wanted to PM you.

On WeeChat issue about the same subject (https://github.com/weechat/weechat/issues/515) I was asked for specification and on #IRCv3 dx linked me to https://github.com/grawity/irc-docs/blob/master/misc/irc-documentation-jilles/reference/modeg.txt in case you happen to need it.


Quick Action Menu Feature Flag Removed

After development is over and feature is QA'd and we are ready for the menus to be included by default, remove feature flag and logic surrounding it.

@csossi reference the PR description in #7795 for details on this feature!

Nothing Selected

Insert Media - the tooltip sez "Insert Background Media", but this just insert media on canvas (not on background)

Background Image Selected

7795 indicates:

but options on canvas are uctually Change Background Colour, Insert Background Media, and Insert Text
note also that Insert Background Media just inserts media (foreground)

Foreground Image

Selecting Replace Media isn't replacing the selected media. it's just adding additional media
Reset element  clears the animation - question: should it clear any link entered as well?

Video

Selecting Replace Media isn't replacing the selected media. it's just adding additional media

Nothing Selected

Insert Media - the tooltip sez "Insert Background Media", but this just insert media on canvas (not on background)

This is intentional,  the design directions are to take the user to the media panel so that they can select what media they want, they still have to drag and drop as usual.

Background Image Selected

7795 indicates:

but options on canvas are uctually Change Background Colour, Insert Background Media, and Insert Text
note also that Insert Background Media just inserts media (foreground)

It sounds like you didn't actually have the background media layer selected?

Foreground Image

Selecting Replace Media isn't replacing the selected media. it's just adding additional media
Reset element  clears the animation - question: should it clear any link entered as well?

Same here for the "replace media" -  it just takes you to the media panel that existing media is from (if you have a 3p media image in use it will take you to that tab, whereas if you have a local media item it'll take you to that tab).
For reset, good clarification. For now it will not clear the link. It will clear the following: animation, overlay, border radius, border, and opacity

Video

Selecting Replace Media isn't replacing the selected media. it's just adding additional media

Same thing here for the media replacement.
sounds like maybe the copy is misleading. We'll see what happens for that in bugbash.

Verified in QA


How to get last torrent activity?

First off, great project you have here. I really like it so far :)
Is it somehow possible to access the last activity time of a torrent?
The reason why I'm asking is that I remove torrents based on their last activity time when the space is running low.
In qBittorrent, this information was accessible through the API, but perhaps it's stored somewhere else and has nothing to do with libtorrent.
It would be nice to have this feature here, or maybe you have a better idea for a cleanup script.

Hi! Happy you like it 😃 I've opened a PR to add last_download and last_upload to the response of the torrents.list JSONRPC method. Both values behave the same, they represent the number of seconds since the last activity of that type (download, upload). Or -1 if we don't know.
Does that help?

Wow that was fast :smile:
Thank you this is exactly what I was looking for.

Awesome! There's some issues with BuildJet and Docker rate limits right now - I'll merge the PR as soon as I can make it green, then after that it will build a beta version you can use. I'll ping when it's all done 😃 Feel free to hop into our Discord if you need any help as well!

I've merged the PR and the stand-alone binaries are available here - https://github.com/porla/porla/actions/runs/5735946537. If you are using Docker, the beta-latest tag has also been updated so you can pull the latest version of that!


Product version of Styled-Components when render with Rendertron

https://stackoverflow.com/questions/56394337/issue-on-product-version-of-styled-components-when-render-with-rendertron
have very simple sample app which build Create React App + Styled-Components to prove this issue. But I have real big application which I am facing this issue which I am going to explain it below.
I would like to pre-render this app with Rendertron for SEO/GoogleBots and etc. But the problem is when I build PRODUCTION version of React App which use Styled-Components . all the style will be missing on static version which Rendertron produced, but from other side if I try the same workflow with dev-server of app , everything looks fine .
So far I know there is different on PROD version and DEV version of my application when I render it with Rendertron . But I am not sure what cause this issue and how I can fix this issue .
I am looking for solution or idea which can help me to solve this issue .
Here is my sample code which I peppered for test .
https://github.com/AJ-7885/test-styled-component-with-rendertron
Here is screen shot from different version of Rendered version by Rendertron base on PROD or DEV version of the same application .

This doesn't seem to be CRA specific problem.  Sorry we couldn't be more of a help.


Improve Mesh Documentation

Objective
This PR continues https://github.com/bevyengine/bevy/pull/8885
It aims to improve the Mesh documentation in the following ways:

Put everything at the "top level" instead of the "impl".
Explain better what is a Mesh, how it can be created, and that it can be edited.
Explain it can be used with a Material, and mention StandardMaterial, PbrBundle, ColorMaterial, and ColorMesh2dBundle since those cover most cases
Mention the glTF/Bevy vocabulary discrepancy for "Mesh"
Add an image for the example
Various nitpicky modifications

Note

The image I added is 90.3ko which I think is small enough?
Since rustdoc doesn't allow cross-reference not in dependencies of a subcrate yet, I have a lot of backtick references that are not links :(
Since rustdoc doesn't allow linking to code in the crate (?) I put link to github directly.
Since rustdoc doesn't allow embed images in doc yet, maybe soon, I had to put only a link to the image. I don't think it's worth adding embed_doc_image as a dependency for this.

Thank you for your review!
I tried to fix stuff from your comments in the last commit.
About the "flipped UV" and use of "normal" those weren't my sentences to begin with but I agree it's clearer that way!

Couple of nits. Personally, this feels like it's touching a bit too many things, like I'm not sure the AssetServer or StandardMaterial should really be mentioned here but this really isn't targeted at someone like me so I'm not sure what the best solution is.

I think people might want to use a Mesh in Bevy, have no idea how to, seach "Mesh Bevy", and either find this doc, or Bevy's cheatbook, in the first case I think it's good to have some references to them at least.
Also knowing that the "UV" and "Normals" are important to put in a mesh for StandardMaterial to work properly is also important imo.
Tell me if it still needs improvement.


[BUG] [Remote Store] Previous cluster UUID is messed up during recovery of QL after publish

Describe the bug

Ensure previous cluster UUID is restored properly after auto restore

Duplicate of https://github.com/opensearch-project/OpenSearch/issues/10841


Bubble instead of throwing on connection errors so application can im…

Bubble instead of throwing on connection errors so application can implement retry pattern.

@robconery @xivSolutions @dmfay Beside the conflicts that must be resolved because the code in the PR is now outdated anything thats blocking this PR?

I need to think about it a bit more. Not being able to connect should throw... and I just want a little time to think this through as it doesn't directly address the problem, it's essentially a hack to allow your situation to work (and I'm sure others might have this situation as well).
I want to address this, but I'm wavering a bit on whether bubbling a connection error is the right way.

For whatever it's worth I am pro-bubbling in general and would prefer to be able to decide what to do with a connection error without any extra try/catch rigmarole on top of the existing error checks in my callbacks.

What happened w/ this pr? It is not clear to me. We have the same use case.

We merged #257, you can click it right there above your comment. We're now passing defaults and bubbling errors...


TuneableConsistencyScatter Cassandra tests

https://docs.shotover.io/shotover-blog/docs/transforms.html#tuneableconsistencyscatter
Docs demonstrate TuneableConsistencyScatter can be used with Cassandra but a quick look at the code it seems to only support Redis? Need to add some tests for this.

TuneableConsistencyScatter was removed in https://github.com/shotover/shotover-proxy/pull/1577


debugger: operate on prebuilt binaries with exec command

This command skips the compile step and uses the named binary directly.
We add a new command exec that accepts a prebuilt binary name and skips the compile step.
Closes #6

This was faster than I expected, I hope you didn't go into much trouble understanding the code. I'll test it later today and merge it. thank you for the effort

Can you describe how did you test this feature? because I tried the following:
gh pr checkout 7
go install .
debugger exec `which reflex`

where reflex is a go program I installed with go install https://github.com/cespare/reflex
but when I try to put a breakpoint for example I get this error in the terminal
^C2024/02/24 23:17:26 ERR could not find file<EMAIL_ADDRESS>2024/02/24 23:17:30 ERR could not find file<EMAIL_ADDRESS>2024/02/24 23:17:33 debugger needs a command: `run`, `exec` or `test`

That exact command works for me.
debugger exec $(which reflex)

Hmmm. There is a bug where during a restart we try to rebuild. Need to tell it not to do that.
However, setting breakpoints works for me.


Import of collection to Postman ends with error "format not recognized"

When I import collection generated by Graphman v1.2.0 to Postman (version 10.8.0 and version 9.31.0) it fails with error message Error while importing: format not recognized
Beginning of generated collection:
{
	"postmanCollection": {
		"info": {
			"name": "rori-editor.backend.doeg8.praha.tmapy.cz-GraphMan",
			"schema": "https://schema.getpostman.com/json/collection/v2.1.0/collection.json"
		},
		"item": [
			{
				"name": "Queries",
				"item": [
					{
						"name": "mapApp",
						"request": {
							"method": "POST",
							"header": [],
							"body": {
								"mode": "graphql",
								"graphql": {
									"query": "query mapApp($id: GlobalID!) {\n  mapApp(id: $id) {\n    __typename\n    id\n    slug\n    title\n  }\n}",
									"variables": "{\n\t\"id\": null\n}"
								}
							},
...

Any hint how to fix it?

Older version of Graphman (1.1.4) generate collection that I can import without errors.

Oh, that shouldn't be the case. I would like to fix this asap but it's hard without more context. I'll check the diffs between 1.1.4 & 1.2.0 to check what could be the cause. In the meantime, if you have any more information I'll be glad to get through it.

Just released v1.2.1 which fixes a bug (and introduces a new flag). It works well on several APIs I tried on. Could you check if this has fixed your issue?

Thanks for fix. It woks great now.


Decoding error on get_vehicle_status API call

Deze fout is ontstaan door een aangepaste integratie.

Logger: homeassistant
Source: custom_components/kia_connect/KiaConnectApi.py:116
Integration: Kia Connected Services (documentation, issues)
First occurred: 14 februari 2024 om 16:34:31 (11 occurrences)
Last logged: 07:48:32

Error doing job: Task exception was never retrieved
Traceback (most recent call last):
  File "/config/custom_components/kia_connect/__init__.py", line 61, in update
    await hass.data[DOMAIN][config_entry.entry_id][KIA_CONNECT_VEHICLE].update_status()
  File "/config/custom_components/kia_connect/KiaConnectVehicle.py", line 48, in update_status
    self.data = await self.kia_api.get_vehicle_status(self.vehicle_id)
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/config/custom_components/kia_connect/KiaConnectApi.py", line 116, in get_vehicle_status
    api_response = await response.json()
                   ^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/aiohttp/client_reqrep.py", line 1166, in json
    raise ContentTypeError(
aiohttp.client_exceptions.ContentTypeError: 0, message='Attempt to decode JSON with unexpected mimetype: ', url=URL('https://mijnkia.nl/api/vehicle/1357850/connected-status')

It seems that sometimes an error occurs which kinda spams my logging. I think that sometimes either the returned API data is either incorrect/missing data or that the response itself gets timed-out or is different than expected.
Depending on the effort to fix this issue, a simple "Try/Except" could already filter out the error logging while not changing (much) of the script flow.

Agreed, this error should be handled by try/except. I'll have a look at it.


ERROR Error: Uncaught (in promise): ReferenceError: EXIF is not defined

When I'm processing images, I'm getting the following error:
ERROR Error: Uncaught (in promise): ReferenceError: EXIF is not defined
This was working previously, however I'm not sure when it broke - it's definitely been working in Angular 6.x, but I may not have tested it since 6.0.x - I can't be sure.
I've noticed there was a thread on the ng2-img-max repo regarding this issue towards the end of 2017, and I'm wondering if the fix there was applied to this library as well.
I'm currently using Angular 6.1.2, and ng-img-tools 1.1.4
I tried installing exif-js: 2.3.0, but that had no impact.

Please try with 1.1.5, should fix the problem!


Add kotlin-gradle-plugin to buildscript in project build.gradle

Description
Fixes the error many developers will encounter when using this library in their project, which is explained in the README:

If you get the error Plugin with id 'kotlin-android' not found while building on Android, go to android/build.gradle and add classpath("org.jetbrains.kotlin:kotlin-gradle-plugin:1.5.0") inside your dependencies block. Please change the plugin version as per your needs.

This PR also removes the above paragraph from the README as it's no longer needed.
It does this by adding the dependency in the buildscript in the project build.gradle. We use a similar approach in Expo modules, for example in expo-gl, and you will find the same pattern in other libraries too.
Reviewers’ hat-rack :tophat:
I don't think this applies to tophatting as there is no change to runtime behavior. Here's how you can reproduce the issue and test the fix:
npx react-native init myapp
cd myapp
yarn add @shopify/flash-list
yarn android # notice the build will fail with the error mentioned above
# now copy the diff from this pr over to<EMAIL_ADDRESS>yarn android # notice the build will succeed

Screenshots or videos (if needed)

Checklist

[x] I have added a changelog entry following the Keep a Changelog guidelines.

Hey @brentvatne 👋
Thanks a lot for the PR, I've checked it locally and it does, indeed, fix the issue ❤


relax connection IO trait bounds

I can't tell if this was intentional. hyper 0.14.x didn't have this requirement and as far as I can tell it's not actually needed.

For context, I believe that this is here for https://github.com/hyperium/hyper/blob/4d0c184f935adf93ed05b86ec3a16e432aa81706/src/server/conn/http1.rs#L180.
But the Connection impl itself, and Dispatch does not require this bound, so it's safe to remove this from impl Future.

Thank you! @ehaydenr


Add EVENT privilege for MySQL

EVENT privilege is required for Magento 2 for example.
So I have to manually grant it in command line.
Is it possible to add this privilege to Dbeaver?
[Magento\Setup\Exception]
Database user does not have enough privileges. Please make sure SELECT, INSERT, UPDATE, DELETE, CREA
TE, DROP, REFERENCES, INDEX, ALTER, CREATE TEMPORARY TABLES, LOCK TABLES, EXECUTE, CREATE VIEW, SHOW
VIEW, CREATE ROUTINE, ALTER ROUTINE, EVENT, TRIGGER privileges are granted to table 'db'.

In MySQL user editor:

Thanks for adding this feature, but you could get even more specific.
It can be done like this:
GRANT EVENT ON db.* TO 'dbuser'@'%';
FLUSH PRIVILEGES;


Add memoization

Memoization (http://en.wikipedia.org/wiki/Memoization) is a method for caching the results of deterministic functions that are called repeatedly. This has natural uses in macroecology where we are often doing the same thing to many different communities, that often share the same values for things like S, N, etc. and therefore this is a valuable tool that we should implement once and then reuse via a decorator.
There is one example here (http://wiki.python.org/moin/PythonDecoratorLibrary#Memoize), which we might be able to just use though the license is unclear.

Memoization is now built in to Python 3:
https://docs.python.org/3/library/functools.html


Synced up history of JSON schemas from Ghost core

PR is a placeholder for easy revert. It's an up to date version of initial work done in https://github.com/TryGhost/Ghost-SDK/pull/345, because there were changes in Ghost's core schemas.

changes landed in master as https://github.com/TryGhost/Ghost-SDK/compare/d6e5e926639d73cae1182d649c5e8aa44b5c58b7...48dc73a38e1144291447583775329ddee29b0009


GH-36092: [C++] Simplify concurrency in as-of-join node

What changes are included in this PR?
The key hasher invalidation and memo-store time updating are moved to the processing thread.
Are these changes tested?
Yes, by existing tests.
Are there any user-facing changes?
No.

Closes: #36092

cc @westonpace, @icexelloss

cc @westonpace, @icexelloss

This makes sense to me. Let me verify I understand:
Before, on InputReceived, we would update the memo for an input based on the new batch.
Now, the only thing that really happens on InputReceived, is the batch is placed into the appropriate input queue. The update to the memo table happens when teh processing thread runs, as it is processing batches for an input.

Yes exactly

@rtpsw Looks like you added another atomic variable here:
https://github.com/apache/arrow/pull/35874/files#r1218191706
Can you fix that one too?

@rtpsw Looks like you added another atomic variable here: https://github.com/apache/arrow/pull/35874/files#r1218191706
Can you fix that one too?

Nvm looks like you fixed this

This makes sense to me. Let me verify I understand:
Before, on InputReceived, we would update the memo for an input based on the new batch.
Now, the only thing that really happens on InputReceived, is the batch is placed into the appropriate input queue. The update to the memo table happens when teh processing thread runs, as it is processing batches for an input.

As @icexelloss said, this is correct. However, there is another difference to note. The pre-PR code invalidates the key hasher and updates the memo-store time given any batch, whereas the post-PR code does so given all but the first batch (after batches_processed_ is incremented). This is fine because the key hasher is initialized as invalid and the memo-store time is initialized to the lowest value.


Allow for vulnerability to be a list

The spec doesn't require a single identifying scheme (which is good) but this raises a potential usability concern, as vulnerability is singular.
Take openSUSE-SU-2020:0051-1.  Let's say I patch my way out of it and produce a
{
      "vulnerability": "openSUSE-SU-2020:0051-1",
      "products": [...],
      "status": "not_affected",
      "justification": "component_not_present",
      "impact_statement": "The vulnerable code was removed with a custom patch"
}

This vulnerability can be referenced in lots of other ways, including:

CVE-2019-17571
GHSA-2qrg-x229-3v8q
openSUSE-SU-2020:0051-1
SNYK-RHEL7-LOG4J-1472071
DSA-4686-1
USN-4495
DLA-2065-1

As per the current spec, you'd need multiple statements, each duplicating products, status, justification and impact_statement.
{
      "vulnerability": "openSUSE-SU-2020:0051-1",
      "products": [...],
      "status": "not_affected",
      "justification": "component_not_present",
      "impact_statement": "The vulnerable code was removed with a custom patch"
},
{
      "vulnerability": "GHSA-2qrg-x229-3v8q",
      "products": [...],
      "status": "not_affected",
      "justification": "vulnerable_code_not_present",
      "impact_statement": "The vulnerable code was removed with a custom patch"
}
...

Or rely on the consumer to reconcile. That reconciliation is relatively easy in theory, at least where a CVE already exists, but in practice it's still a pain given the breadth of the ecosystem.
Another use case might be where multiple vulnerabilities relate to a single package, and that package isn't present. eg. component_not_present. Again, in that case multiple near-identical statements could be made, but it may be useful to permit it as one statement as well.
Given the above, I'd like to discuss whether vulnerability should instead by vulnerabilities and allow for a list of vulnerability identifiers?

I agree with the motivation here. 👍  I brought this up in the CISA meetings, but at the time, the group consensus was to keep statements to single vulnerability identifiers.
I'm curious for others' thoughts on this!
cc: @puerco

Hey @garethr thanks for the suggestion, this is something we had antcipated in the past, so let me give a little bit of context:

I don't think we should have a list of vulnerabilities. The VEX community voted on the structure of the statement and decided to not do that.
That said, we used to have a references field exactly for this (you can see it here in the old repo). We removed it to simplify the format at launch, always knowing we could bring it back if we needed it, so this may be it.

I agree with the motivation here. +1 I brought this up in the CISA meetings, but at the time, the group consensus was to keep statements to single vulnerability identifiers.

I agree that in the CISA meetings it was decided that you can have only one vulnerability per statement. BUT that doesn't mean that a vulnerability can't have multiple identifiers. Now you just have to make sure, nobody abuses the "multiple ids" to sneak in multiple vulnerabilities...

I agree that in the CISA meetings it was decided that you can have only one vulnerability per VEX statement. BUT that doesn't mean that a vulnerability can't have multiple identifiers.

(Emphasis added).
Agreed. This is the challenge. Agree on no to multiple vulnerabilities, but it's one vulnerability with multiple identifiers that is the issue.
@puerco that would at least allow for support of this. From a client point of view though, which one you choose as the vulnerability and which are added as references is arbitrary. So in effect it's really just a flat list. I'm be +1 on bringing that back. It's a small addition to the spec that could reduce the verbosity of resulting documents.
It does make the use case clearer from a creator perspective though, which could help to address the "Now you just have to make sure, nobody abuses the "multiple ids" to sneak in multiple vulnerabilities..." issue raised by @tschmidtb51.

Mmmh I agree, we should allow for the other references.
Keeping with the spirit of linked data, perhaps we should just have the references as a list and make the identifier an IRI. For example:
{
      "vulnerability": {
           "@id":"https://nvd.nist.gov/vuln/detail/CVE-2019-17571",
            "references": [
                "CVE-2019-17571",
                "GHSA-2qrg-x229-3v8q",
                "openSUSE-SU-2020:0051-1",
                "SNYK-RHEL7-LOG4J-1472071",
                "DSA-4686-1",
                "USN-4495",
                "DLA-2065-1",
            ]
      },
      "products": ["pkg:..."],
      "status": "not_affected",
      "justification": "component_not_present",
      "impact_statement": "The vulnerable code was removed with a custom patch"
},

This would allow for a more compact document when the IRI can be resolved to a resource that has all the data about the vulnerability, such as an "SBOM" (a security document in SPDX or CDX) or an advisory:
{
      "vulnerability": { "@id":"https://nvd.nist.gov/vuln/detail/CVE-2019-17571" }
      "products": ["pkg:..."],
      "status": "not_affected",
      "justification": "component_not_present",
      "impact_statement": "The vulnerable code was removed with a custom patch"
},

I like this direction. It may also be worth checking out how the OSV schema thinks of "IDs" and "aliases" of vulnerabilities — their way of capturing multiple references to the same underlying vulnerability. https://ossf.github.io/osv-schema/#aliases-field

Enhancement proposal OPEV-0015 was merged and with the release of spec v0.2.0, the vulnerability field can now support multiple vulnerability aliases :tada:


Feature Request: ScummVM + Touch Controls

I'd love support for ScummVM (https://www.scummvm.org/) if that's possible as well as touch controls to emulate using a mouse on mobile devices.
Thanks!

Yeah, absolutely! That is one I thought about early on and intend to add support down the road.
Thanks for submitting it!


first draft of CONTRIBUTING.MD

resolves #https://github.com/dbt-labs/core-team/issues/33
Description
A first draft of the possible CONTRIUBTING.MD file for the Adapter Repos
Checklist

[x] I have signed the CLA
[ ] I have run this code in development and it appears to resolve the stated issue
[ ] This PR includes tests, or tests are not required/relevant for this PR
[ ] I have updated the CHANGELOG.md and added information about my change to the "dbt-bigquery next" section.

Would love any advice on format, if information seems clear, if we need all the same sexctions from the dbt-core Contrubting.md file etc. once we get this one done should be just a template for all the others which will make finishing assignment quick so we can take very discerning eyes with this one.


Tuya Generic LSPA09 Firmware 1.1.15 Dump

Dump of  Tuya Generic LSPA09. Firmware version 1.1.15
readResult_BK7231N_QIO_LSPA9_105_2023-26-9-19-31-46.zip

This firmware is already on the known patched firmware list


Document trimming options

This is intended to be a reference of the supported options for trimming. I put it next to the existing description of trimmed self-contained deployments (https://docs.microsoft.com/en-us/dotnet/core/deploying/trim-self-contained). I'd be happy to put it elsewhere if there is a better place for this kind of technical documentation.
Fixes https://github.com/dotnet/runtime/issues/41280

The PR #20278 is also editing the file you have here, with what I think are the same changes.

Thanks - they're different changes, but we're editing the same line - I'll fix the conflict when that gets merged.

The other PR is merged


Transmutation missile

By Graugger
Converts any blocks made with wool or leather into animals.

We could take this concept a bit further by making a ton of magic based missiles that mutate blocks into other objects. For example, we could turn stone into dirt, gravel into glass, water into lava.... etc.

How would magic fit? The mod itself seems based around sience and technology. How would you explain magics involvment?

We have endermen and dragons in minecraft.....

Thats minecraft, not icbm.

Note from Graugger and morton - Monsters should change colors before mutating into other entities.
Example: Zombie turns purple before turning into an enderman


subctl gather - discovery.k8s.io/v1beta1 EndpointSlice is deprecated in v1.21+

When running in a deployment that has service discovery enabled, this warning is produced by the API:
Warning: discovery.k8s.io/v1beta1 EndpointSlice is deprecated in v1.21+, unavailable in v1.25+; use discovery.k8s.io/v1 EndpointSlice
We should switch to using discovery.k8s.io/v1 once we drop support for K8s 1.20

What happened:
What you expected to happen:
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:
Environment:

Diagnose information (use subctl diagnose all):
Gather information (use subctl gather)
Cloud provider or hardware configuration:
Install tools:
Others:

Depends on https://github.com/submariner-io/lighthouse/issues/576

@skitt @vthapar can you comment on how urgent is this one is for 0.13? If we want to get it in, this needs to happen this week.

This shouldn't be needed anymore. gather.go is already using the right version now. Probably whosoever moved code over fixed it.
https://github.com/submariner-io/subctl/blob/devel/internal/gather/servicediscovery.go#L24


DPE-2959 async replication

Issue
Need to have async replication
Solution
Use async replication library (owned by vm charm).
Implementation as described by this workflow
How-to draft in the discourse page
TODO:

~minimal unit test fixes~
~discourse doc page~
~full usage of data-platform secrets~
test endpoint reachability
~fence write traffic for old primary~

Integration tests and unit tests on a fup PR

As discussed in private: it is not possible to re-join the cluster after recreate-cluster. The juju stuck on:
Unit    Workload  Agent  Address       Ports  Message
db2/0*  blocked   idle   <IP_ADDRESS>         User data found, aborting async replication setup
db2/1   waiting   idle   <IP_ADDRESS>         waiting replica cluster be configured
db2/2   waiting   idle   <IP_ADDRESS>         waiting replica cluster be configured

STR:

https://discourse.charmhub.io/t/charmed-mysql-k8s-async-replication/12904
After juju run -m az2 db2/leader recreate-cluster it is not possible to re-join db2 to az1.db1 because az2 has the previos data which is impossible to wipe.

Proposal:

the blocked cluster after  juju remove-relation -m az2 async-primary db2 can be re-joined back using juju relate -m az2 async-primary db2 (the local az2 data will be wiped)
if the cluster re-create, it cannot re-join, it must be destroyed and re-deployed. Safety first.

@taurus-forever

2. if the cluster re-create, it cannot re-join, it must be destroyed and re-deployed. Safety first.

Should we make this parametrized for the action?

@taurus-forever

2. if the cluster re-create, it cannot re-join, it must be destroyed and re-deployed. Safety first.

Should we make this parametrized for the action?

IMHO, not now. It is a dangerous action. If you want to re-attach the re-created cluster:
juju remove-application mysql && juju remove-storage ... && juju deploy mysql
(c) Safety first.
P.S. we can implement such action later after the proper spec review by PMs.

For the history, revision 128 from edge/arepl fails:
unit-db2-0: 14:13:57 DEBUG unit.db2/0.juju-log async-replica:3: Syncing credentials from primary cluster
unit-db2-0: 14:13:57 ERROR unit.db2/0.juju-log async-replica:3: Uncaught exception while in charm code:
Traceback (most recent call last):
  File "/var/lib/juju/agents/unit-db2-0/charm/venv/ops/model.py", line 3019, in _run
    result = subprocess.run(args, **kwargs)  # type: ignore
  File "/usr/lib/python3.10/subprocess.py", line 526, in run
    raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command '('/var/lib/juju/tools/unit-db2-0/secret-get', 'secret:cnpfe3nmp25c77tiud60', '--format=json')' returned non-zero exit status 1.

For the history, revision 128 from edge/arepl fails:
unit-db2-0: 14:13:57 DEBUG unit.db2/0.juju-log async-replica:3: Syncing credentials from primary cluster
unit-db2-0: 14:13:57 ERROR unit.db2/0.juju-log async-replica:3: Uncaught exception while in charm code:
Traceback (most recent call last):
  File "/var/lib/juju/agents/unit-db2-0/charm/venv/ops/model.py", line 3019, in _run
    result = subprocess.run(args, **kwargs)  # type: ignore
  File "/usr/lib/python3.10/subprocess.py", line 526, in run
    raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command '('/var/lib/juju/tools/unit-db2-0/secret-get', 'secret:cnpfe3nmp25c77tiud60', '--format=json')' returned non-zero exit status 1.

Updated revision r129 with fixes


test: add tests

Codecov Report

Merging #6 into master will increase coverage by 18.82%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master       #6       +/-   ##
===========================================
+ Coverage   30.12%   48.95%   +18.82%
===========================================
  Files           5        5
  Lines         239      239
===========================================
+ Hits           72      117       +45
+ Misses        155      103       -52
- Partials       12       19        +7

Impacted Files
Coverage Δ

fagott/tester.go
38.46% <0%> (+38.46%)
:arrow_up:

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 4ebf3f6...8f95f06. Read the comment docs.


Drag analysis

Summary
This PR makes the Drag analysis less prone to fit failures by improving the initial guesses.
Details and comments
The initial guesses for the fit amplitudes are adjusted as well as for the base of the fit function. This addresses #685 . This PR also changes the internals of the drag analysis: the three frequencies are replaced with a single frequency which is multiplied by the number of drag gate repetitions. This is done by introducing reps as a fixed parameter. Furthermore, the frequency is estimated based on the fastest oscillation. Furthermore this PR introduces several tests of challenging data:

@wshanks The recent changes db9ab19 should have significantly improved the fitting. I also added more tests.


runtime: SIGILL in runtime.asmcgocall

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
1.8rc2: Illegal instruction
1.7.4: go version go1.7.4 linux/arm
What operating system and processor architecture are you using (go env)?
GOARCH="arm"
GOBIN=""
GOEXE=""
GOHOSTARCH="arm"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/hanno/go"
GORACE=""
GOROOT="/usr/local/go"
GOTOOLDIR="/usr/local/go/pkg/tool/linux_arm"
CC="gcc"
GOGCCFLAGS="-fPIC -marm -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build286957571=/tmp/go-build"
CXX="g++"
CGO_ENABLED="1"

output above from 1.7.4,  1.8rc2 just gets a SIGILL
What did you do?
Install go binary from https://storage.googleapis.com/golang/go1.8rc2.linux-armv6l.tar.gz on a Raspberry Pi 1 w/ raspbian, upgraded to latest userland.
No LSB modules are available.
Distributor ID: Debian
Description:    Debian GNU/Linux 7.11 (wheezy)
Release:        7.11
Codename:       wheezy
$ uname -a
Linux stone-garden-islet 3.10.21+ #604 PREEMPT Mon Dec 2 20:17:09 GMT 2013 armv6l GNU/Linux
$ cat /proc/cpuinfo
processor       : 0
model name      : ARMv6-compatible processor rev 7 (v6l)
BogoMIPS        : 2.00
Features        : swp half thumb fastmult vfp edsp java tls
CPU implementer : 0x41
CPU architecture: 7
CPU variant     : 0x0
CPU part        : 0xb76
CPU revision    : 7

Hardware        : BCM2708
Revision        : 000e
Serial          : 000000003a17768c

What did you expect to see?
The go version output
What did you see instead?
go binary getting a SIGILL
$ gdb --args /usr/local/go/bin/go version
GNU gdb (GDB) 7.4.1-debian
Copyright (C) 2012 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.  Type "show copying"
and "show warranty" for details.
This GDB was configured as "arm-linux-gnueabihf".
For bug reporting instructions, please see:
<http://www.gnu.org/software/gdb/bugs/>...
Reading symbols from /usr/local/go-1.8rc2/bin/go...done.
Loading Go Runtime support.
(gdb) b *'runtime.asmcgocall'
Breakpoint 1 at 0x62e54: file /usr/local/go/src/runtime/asm_arm.s, line 514.
(gdb) r
Starting program: /usr/local/go-1.8rc2/bin/go version
[Thread debugging using libthread_db enabled]
Using host libthread_db library "/lib/arm-linux-gnueabihf/libthread_db.so.1".

Breakpoint 1, runtime.asmcgocall () at /usr/local/go/src/runtime/asm_arm.s:514
514     TEXT ·asmcgocall(SB),NOSPLIT,$0-12
(gdb) info registers
r0             0xbefff744<PHONE_NUMBER>
r1             0x108005a0       276825504
r2             0x108262b8       276980408
r3             0x0      0
r4             0x3b5cf0 3890416
r5             0xd0     208
r6             0xb6e3c000<PHONE_NUMBER>
r7             0x810    2064
r8             0x400a50 4196944
r9             0x34     52
r10            0x57e828 5761064
r11            0x576a98 5728920
r12            0x0      0
sp             0xbefff728       0xbefff728
lr             0x3fa1c  260636
pc             0x62e54  0x62e54 <runtime.asmcgocall>
cpsr           0x20000010       536870928
(gdb) c
Continuing.

Program received signal SIGILL, Illegal instruction.
0x00011384 in _cgo_try_pthread_create ()
(gdb) info registers
r0             0xbefff5c4<PHONE_NUMBER>
r1             0xbefff5cc<PHONE_NUMBER>
r2             0x11460  70752
r3             0x592008 5840904
r4             0x0      0
r5             0xbefff5cc<PHONE_NUMBER>
r6             0xbefff670<PHONE_NUMBER>
r7             0x592008 5840904
r8             0x57ec38 5762104
r9             0x34     52
r10            0x57e828 5761064
r11            0x576a98 5728920
r12            0x108005a0       276825504
sp             0xbefff590       0xbefff590
lr             0x11534  70964
pc             0x11384  0x11384 <_cgo_try_pthread_create+8>
cpsr           0x60000010<PHONE_NUMBER>
(gdb) bt
#0  0x00011384 in _cgo_try_pthread_create ()
#1  0x00011534 in _cgo_sys_thread_start ()
#2  0x00062eac in runtime.asmcgocall () at /usr/local/go/src/runtime/asm_arm.s:543
#3  0x003b9430 in type.* ()
#4  0x003b9430 in type.* ()
Backtrace stopped: previous frame identical to this frame (corrupt stack?)
(gdb) ^D
$

Note: gofmt runs fine, godoc also gets a SIGILL

Possibly related: #18694. cc @minux @cherrymui

I can confirm that, on my Raspberry Pi 1. I wonder whether the release binary is really compiled to ARMv6, especially the C part (since gofmt runs ok, which is a pure Go binary).

@cherrymui, can you inspect the binaries from the download URL and check?
Otherwise, you can see how they're built in x/build/cmd/release
/cc @broady

@bradfitz, I just tried the binary from https://storage.googleapis.com/golang/go1.8rc2.linux-armv6l.tar.gz, got SIGILL on the go binary and godoc, while gofmt is ok. SIGILL happened in _cgo_try_pthread_create, that's why I wonder that's the C compiler.

It seems cmd/release just runs all.bash on the linux-arm builder, and I didn't see anywhere in the build procedure it sets gcc flags. So I assume it just builds for its native architecture, which is ARMv7.
It seems it has been like that for long. I wonder how it worked before, maybe we are just lucky that it happened to not generate new instructions? FWIW, the fault is in _cgo_try_pthread_create which is new in 1.8 (but the function itself looks ok).

Which gcc flags should be set? And by what, ... should cmd/go do it?

I wonder setting GOARM=6 should imply setting gcc flags -march=armv6, maybe by cmd/go?
cc @minux

Perhaps we could just add these two environment variable before running
make.bash?
CGO_CFLAGS="-march=armv6"
CGO_LDFLAGS="-march=armv6"
If it helps, let's not change cmd/go this late, as I'd like to figure out a
complete story for ISA variants in Go 1.9.

@minux, SGTM.
Could you send a CL?

@minux SGTM, if it works.

I send CL https://go-review.googlesource.com/c/35501/. It works on my raspberry pi.


(fix) scroll event not listened in Electron

What kind of change does this PR introduce? (check at least one)

[x] Bugfix
[ ] Feature
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[ ] Yes
[x] No

If yes, please describe the impact and migration path for existing applications:
The PR fulfills these requirements:

[x] When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)
[x] All tests are passing

If adding a new feature, the PR's description includes:

[ ] A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Other information:
I found it's unable to listen 'scroll' event using this.$refs[...].addEventListener(...) in mounted() when I am using Electron 11.2.3 and Vue 2.6.12.
So I moved the function to a function called containerScroll(e) and use @scroll to listen the scroll event, it finally works

Thanks! Will test that soon


if use option_name, help argument not work

if use option_name in configuration, help argument not work
Kirki::add_config( 'mc', array(
  'capability'  => 'edit_theme_options',
  'option_type' => 'option',
  'option_name' => 'mc'
) );

Kirki::add_field( '', array(
    'type'        => 'checkbox',
    'settings'     => 'checkbox_demo',
    'label'       => __( 'This is the label', 'kirki' ),
    'help'        => __( 'This is some extra help text.', 'kirki' ),
    'section'     => 'my_section',
    'default'     => '1',
    'priority'    => 10,
) );

if don't use option_name help argument work, but
Undefined index: default in \kirki\includes\class-kirki.php on line 157
Kirki::add_config( 'mc', array(
  'capability'  => 'edit_theme_options',
  'option_type' => 'option'
) );

line 157: $value = get_option( $field_id, self::$fields[ $field_id ]['default'] );
& Undefined index: default in \kirki\includes\class-kirki.php on line 102
Kirki::add_config( 'mc', array(
  'capability'  => 'edit_theme_options',
  'option_type' => 'theme_mod'
) );

line 102: $value = get_theme_mod( $field_id, self::$fields[ $field_id ]['default'] );

That's weird... DO you think you could give me access to your theme so that I may try to replicate this and see what goes on exactly?
Thanks,
Ari.

@aristath here is twentyfifteen theme like my settings theme

Anyone ?

@mahdi-alavi hey there!
I just tested this with the latest commits in the master branch and it looks like this has been resolved.
I'll go ahead and close this ticket, if however you can still replicate it after you pull the latest commits then please feel free to reopen this and provide as many info as possible. :+1:


Unable to access Container App that is inside a VNet integrated Container Environment

Please provide us with the following information:

This issue is a: (mark with an x)

[ ] bug report -> please search issues before submitting
[ ] documentation issue or request
[ ] regression (a behavior that used to work and stopped in a new release)

Issue description
Hi, I am unable to access using http a container that is inside a VNet integrated container environment.
When i add vnet integration, looks like a load balancer is auto created and the envoy proxy is registering TCP ports 80 and 443.
Does this mean all the requests go through envoy proxy? In that case what do i need to add to the URL to access one of the container apps inside the container environment using http?
Steps to reproduce

..
..

Expected behavior [What you expected to happen.]
Actual behavior [What actually happened.]
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Ex. Did this issue occur in the CLI or the Portal?

In your situation, when you enable VNet integration for Azure Container Apps, traffic flows through the Azure-managed Envoy proxy, which handles routing to your container apps. Here’s a detailed explanation of how ingress works with a VNet and what you need to consider:
1. Ingress Through Envoy Proxy:
When you enable ingress (whether internal or external), Azure Container Apps uses Envoy as a proxy to route the traffic:

External Ingress: Your container is publicly exposed, and traffic flows through Envoy.
Internal Ingress (VNet): The container is only accessible within your VNet.

By default, Envoy is responsible for managing all incoming HTTP (80) and HTTPS (443) traffic. It terminates the HTTP request at the proxy level and routes it to the appropriate container app.
2. Accessing Your App via HTTP:
If you’ve enabled internal ingress (private endpoint within a VNet), the container app is only accessible from within your VNet. To access it:

Ensure your DNS resolution inside the VNet can resolve the domain of your container app.
The URL format should still be in the form of http://<app_name>.<environment_name>.azurecontainerapps.io, but this will only work from within the VNet.
Ensure that the required network security rules are in place to allow traffic on ports 80 and 443 within the VNet.

3. Important Network Configurations:

Private DNS: If the app is exposed internally, ensure that the VNet includes proper DNS resolution to resolve the container app’s internal hostname. You may need to configure a private DNS zone or use Azure DNS to resolve the internal URLs.
Subnet Rules: The VNet must have adequate subnet rules allowing traffic from your internal resources to the container environment.
Firewall and NSG: Make sure your Network Security Group (NSG) or firewall isn’t blocking the traffic to your container app.

4. Accessing Your App (Internal Ingress):

If you are using internal ingress, you’ll need to ensure that the requests are sent from within the same VNet or a peered VNet.
You would use the URL associated with your Container App’s environment. If DNS resolution is correctly set up, you could access the app using:

http://<container_app_name>.<environment_name>.azurecontainerapps.io (or custom domain if configured).

5. Debugging Access Issues:

DNS Resolution: Ensure that the domain of your container app resolves correctly inside the VNet. You can use tools like nslookup or dig from within your VNet to check this.
Network Security: Check the Network Security Groups (NSG) associated with the subnets in your VNet. Ensure rules are allowing the appropriate inbound and outbound traffic on ports 80 and 443.
App Logs: Check the logs for your container apps and the Envoy proxy to ensure traffic is reaching the app.

Summary:
Yes, all your HTTP/HTTPS requests go through the Envoy proxy when using ingress in Azure Container Apps. If you are trying to access the app using a URL, you need to ensure that the DNS and network settings inside your VNet are properly configured to allow internal resolution. You'll use the same standard URL for your app (http://<app_name>.<environment_name>.azurecontainerapps.io), but it will only work from inside the VNet or a peered network.
If you want external access, you will need to enable external ingress and configure the necessary security settings for public exposure.


Calling stochasticswap pass directly fails

Environment

Qiskit Terra version: {'qiskit-terra': '0.19.2', 'qiskit-aer': '0.10.3', 'qiskit-ignis': '0.7.0', 'qiskit-ibmq-provider': '0.18.3', 'qiskit-aqua': None, 'qiskit': '0.34.2', 'qiskit-nature': None, 'qiskit-finance': None, 'qiskit-optimization': None, 'qiskit-machine-learning': None}
Python version:
Operating system:

What is happening?
A simple modification to a doc example from https://qiskit.org/documentation/tutorials/circuits_advanced/04_transpiler_passes_and_passmanager.html#Different-Variants-of-the-Same-Pass
from qiskit import QuantumCircuit
from qiskit.transpiler import CouplingMap
from qiskit.transpiler.passes import StochasticSwap
from qiskit.test.mock import FakeBoeblingen
backend = FakeBoeblingen()

coupling = backend.configuration().coupling_map

circuit = QuantumCircuit(7)
circuit.h(3)
circuit.cx(0, 6)
circuit.cx(6, 0)
circuit.cx(0, 1)
circuit.cx(3, 1)
circuit.cx(3, 0)

coupling_map = CouplingMap(couplinglist=coupling)

ss = StochasticSwap(coupling_map=coupling_map)
pass_manager = PassManager(ss)
stochastic_circ = pass_manager.run(circuit)

yields
File "/opt/miniconda3/envs/qiskit/lib/python3.10/site-packages/qiskit/transpiler/passes/routing/stochastic_swap.py", line 337, in _mapper
    raise TranspilerError(
TranspilerError: 'swap mapper failed: layer 0, sublayer 1'

How can we reproduce the issue?
Run above
What should happen?
It should work.  It works if calling from transpile with routing_method='stochastic'
Any suggestions?
No response

This appears to have been fixed on main already for me, probably as a result of #7658 - could you check locally to see if the problem persists for you on main?

Yes.  This now works on main.  Although still curious why it worked via transpile but not directly.

My guess is that it worked with transpile() because transpile() will run a layout pass (which by default will be DenseLayout) prior to running stochastic swap. With the layout applied the input dag to stochastic swap will have ancilla qubits for all the idle physical qubits and also a different order based on the layout. I expect that all makes it easier for it to find a valid mapping. As to why it's fixed on main I'm actually just assuming it's random chance because we're using a different rng with the rust re-implementation in #7658

So the backend in question has bi-direction coupling for qubits 0->4.  But then this circuit fails:
ghz = QuantumCircuit(5)
ghz.h(0)
ghz.cx(0,range(1,5))

If I pass a restricted coupling map:
coupling = [[0,1], [1,0], [1,2], [2,1], [2,3], [3,2], [3,4], [4,3]]

which is contained inside of the original then it works.  So it appears that the swap mapper requires the circuit to be of the same width as the device given by the coupling map.  Indeed the exact same circuit but over 20q works.  We should probably throw a better error if passing a circuit to a swap mapper where that is not the case.

I am not sure how rust changes things here.

Functionally there is no difference between the rust and cython implementations. The rust just runs the trials in parallel instead of serially and is using a different rng (each thread has a thread local PCG64MCG rng and the seeding is different)

Looks like StochasticSwap originally used to reject all DAGs/layouts that had a mismatch in the number of qubits, but #2598 relaxed the constraint, and I don't see a particular reason in that PR why.  Perhaps smaller DAGs worked at that point, and our tests never caught the point at which it stopped working.


bug: Unknown Hint: Core(Core(EvalCircuit

Current behavior:
Hint fails with:
Unknown Hint: Core(Core(EvalCircuit { n_add_mods: Deref(CellRef { register: AP, offset: -6 }), add_mod_builtin: Deref(CellRef { register: FP, offset: -6 }), n_mul_mods: Deref(CellRef { register: AP, offset: -4 }), mul_mod_builtin: Deref(CellRef { register: FP, offset: -5 }) }))
This happens presumably any time mul_mod is called. The problem is that this is not handled in cairo-vm.
Expected behavior:
Hint should succeed.
Steps to reproduce:
Run prove_block against affected blocks such as 160035, 160045, 160068, 160074, 160084, or 160093

I started by handling the hint in cairo-vm: https://github.com/Moonsong-Labs/cairo-vm/pull/44
But this led to an error shortly later: Unknown memory cell at address 13:6
It turns out there is some logic missing from cairo-vm's fill_memory as compared to cairo-lang's. This comment (although unclear no me) suggests what is wrong:
# Note that we can't read 'n' here because sierra expects this function to compute it.

https://github.com/starkware-libs/cairo-lang/blob/4ea4fe8e167845a3402ae2ea0a8b6004aad18dd5/src/starkware/cairo/lang/builtins/modulo/mod_builtin_runner.py#L373

A naive attempt to avoid reading n, as suggested in the comment mentioned above, isn't sufficient. It turns out there is a lot of extra code to calculate this value instead of reading it.
I began implementing this, but it turned into quite a rabbit hole, as there seems to be a lot more code omitted in the cairo-vm implementation. Even worse, a lot of the code that is omitted is actually dead code, so it's not obvious what exactly actually needed among the missing code.

The implementation from cairo seems to work :tada:  We're now passing the following blocks:
160035, 160045, 160068, 160074, 160084


[qt5] Add -j to make to parallelize on Linux and MacOS

I think there is a VCPKG variable for concurrency. VCPKG_CONCURRENCY or similar. Should be visible with --debug in the cmake call if I am not mistaken. Currently on mobile so I cannot check it.

There is a VCPKG_CONCURRENCY, see https://github.com/microsoft/vcpkg/blob/3a1e8e01f57ab54559a588cb4cd9442a2f27b6c8/ports/ace/portfile.cmake#L237 where we use it work regular make

Since you are already touching concurrency settings:
https://github.com/microsoft/vcpkg/blob/0ab1a9e1c64a3968631b59647e771beda7d1f256/ports/qt5-base/cmake/install_qt.cmake#L6-L26
could also be removed and replaced with VCPKG_CONCURRENCY


Manage Cookies Issues

Expected Behavior
https://user-images.githubusercontent.com/1693892/178826834-33347d28-b28c-4938-9d6c-3f4fbf69d697.mov
Actual Behavior
Hello Insomnia,
In the Manage Cookies modal If initially you add some text into the filter input and then you click on button Add Cookie or Actions dropdown and then click Add Cookie you will see the following things:

The newly added cookie(s) they don't appear right after adding. it confuse a little bit because users think that something went wrong and some times may clicks multiple times on Add cookie button in order just to be sure. (Improvement)
If you totally remove the text from filter cookies input the newly added cookie(s) will appear, but as you can see the name of this cookie(s) is/are the filter value we added previously. (Bug)
If you try to delete the cookie you have to click twice the delete button, one for deletion and one for confirmation, which is fine for me but again I think from UX reasons maybe an confirmation alert will be more useful or if the button itself was changing and saying something like confirm instead of icon with bigger boundaries will be more clear for the users the confirmation stage. (Improvement)

Reproduction Steps

Open Insomnia App
Click on Cookies button
Type something in filter input
Click on Add Cookie button
Remove value from filter input
See issues

Is there an existing issue for this?

[X] I have searched the issue tracker for this problem.

Additional Information
No response
Insomnia Version
2022.5.0-beta.2
What operating system are you using?
macOS
Operating System Version
macOS Monterey
Installation method
download from insomnia.rest
Last Known Working Insomnia version
No response

The bug is there, am also frustrated due to it

new cookies are not replacing old one for URL, you need to delete it and refetch new cookies again


Add unsupported browser page to catch attempted IE logins

Add a new page to show Internet Explorer users when they try to log into SimpleReport.
Issue: https://app.zenhub.com/workspaces/simplereport-growth--engagement-605b43a6ceb92c000f86279a/issues/cdcgov/prime-simplereport/664

When I tried to test this by using Firefox, it brought me to this link: https://hhs-prime.okta.com/oauth2/default/v1/authorize?client_id=0oa6fl5n8ARasjheq4h6&redirect_uri=https://test.simplereport.gov/app&response_type=token+id_token&scope=openid%20simple_report%20simple_report_test&nonce=thisisnotsafe&state=thisisbogus is there another way I should test that it works?

Good catch! It wasn't resolving the URI correctly, I've updated that. Should work on all non-IE browsers now.


Add delete button

Songs are impossible to delete
I want a delete button to delete my tracks.

Done!


Distributed Training not working after the latest commits

Hey,
After pulling the latest commit, it appears that Distributed Training stopped working.
we are getting this error:
Expected object of backend CUDA but got backend CPU for argument #2 'target' in call to _thnn_nll_loss_forward
we checked the device type of self.pred on after_pred() and it appears that the self.pred is on the cpu and not on cuda
the latest commit that distributed training worked is around January 13'th.
maybe it's related to this commit: https://github.com/fastai/fastai2/commit/2c947668a6defeced761ce37b69be27504d72e2a

Should be fixed by #85, please reopen with a reproducible example if the issue persists.


Remove sub() from gas() in SemaphoreVerifier.sol

There are lines in SemaphoreVerifier.sol that make gas() calls. GAS opcode is restricted in validateUserOp (along with some other OPCODES). GAS is only allowed if immediately followed by CALL (and similar) opcodes. Semaphore does staticcall(sub(gas(), ...) and there is a SUB in between.
So we need some other way to take the gas cost in consideration.

related: #345
refer also to this blog post (other challenges section):
https://saleel.xyz/blog/zk-account-abstraction/

@cedoor this is the issue related to #345.

@jimmychu0807 thanks 👍🏽

@saleel currently what is the simplest way to setup a test environment to trigger the issue that validateUserOp() calls SemaphoreVerifier.sol and fails due to having a sub call before gas call (without moving heaven and earth 🙂)?
If this can't be done, I can't reliably know that I have solved this issue.
Does this mean I should get back to setting up the smart-account-module?

Yea, you would need to run the bundler and try to do a userOp with semaphore proof as the verifier. Basically, get the tests in https://github.com/saleel/semaphore-wallet pass with the original Semaphore contract (currently it uses the custom ones I have over there)

@cedoor replying to your https://github.com/semaphore-protocol/semaphore/issues/345#issuecomment-2429083407
To fix this issue, I want to be able to reproduce the error by running the test cases in https://github.com/saleel/semaphore-wallet . But the bundler component needed in the e2e test has updated since then and I have a hard time making all components connect together to trigger the error.
The opcode restriction is outlined at [OP-012]: https://eips.ethereum.org/EIPS/eip-7562#opcode-rules
An alternative is just that I make line from:
success := staticcall(sub(gas(), 2000), 7, mIn, 96, mIn, 64)

to
success := staticcall(gas(), 7, mIn, 96, mIn, 64)

will that cause any security implication?

@jimmychu0807 @cedoor
Let me add some comments and explain.
Here, sub(gas(), 2000) ensures that 2000 gas units are reserved for the remaining execution after the staticcall. This is a common practice to prevent the called function from consuming all available gas, which could potentially leave insufficient gas for subsequent operations.
In the proposed change, success := staticcall(gas(), 7, mIn, 96, mIn, 64), the staticcall will have access to all remaining gas, potentially consuming the entire gas stipend allocated for the transaction.
Typically, reserving gas helps prevent/reduce reentrancy attacks by ensuring that certain operations can always complete. However, in this specific context:

The staticcall targets precompiled contracts (addresses 6 and 7), which are trusted and do not contain reentrant logic (as far I can see).
And, static calls do not allow state modifications, further reducing the risk of reentrancy.
The functions following the staticcall in this context are minimal and primarily involve memory operations and checks. In most cases, they do not require substantial gas.

I suggest to proceed with the change. Replacing sub(gas(), 2000) with gas() is unlikely to introduce security vulnerabilities in the current context.
However, if future modifications introduce more gas-intensive operations after the staticcall, not reserving gas could lead to unexpected failures.
Few more additional points:

monitor the gas consumption of the verifyProof function after making this change
test the modified function with various inputs and gas limits to ensure it behaves correctly under different conditions

@thogiti thank you for the explanation! With your comment, I take a deeper in the code. And I agree with what you said.

L#575: this is calling ethereum precompile at address 0x7, elliptic curve scalar multiplication. Yes, it is a pure/view function.

L#585: This is elliptic curve point addition.

L#665: This is doing a ecc point pairing, and is pretty much the end of the function.

Let me make the change and check the difference in gas usage. Thank again for the helpful input!

@jimmychu0807
I agree that staticcall with gas() call directly is pretty safe, given it does not allow state modifications.
that being said, if we really want to preserve 2000 gas, have we tried to use gasleft() instead of gas()?
gas() is a low-level EVM opcode, while gasleft() is a higher level Solidity built-in function. Using gasleft() allows the compiler to handle the restriction compliance.

@0xDatapunk thanks for the feedback. The issue is that we are in the yul assembly. If we call gasleft() we have to interlace yul and regular solidity code together. I am not sure if this worths the hassle.

can u call gasleft() inside of assembly?

Thanks for all the feedback!
PR: https://github.com/semaphore-protocol/semaphore/pull/883
@jimmychu0807 Please, check the previous comment https://github.com/semaphore-protocol/semaphore/issues/871#issuecomment-2432856752. The PR should include some other tests to be sure the function doesn't fail.

Another approach could be hardcoding the gas sent to the precompile. The cost is dynamic based on the bit structure of the input so we could hardcode it to the upper bound of gas needed.

Cool, this should also work!

@jimmychu0807 do you minding experimenting calling gasleft() inside of assembly, and let me know what happens? thanks

A compiler error:
DeclarationError: Function "gasleft" not found.
  --> contracts/base/SemaphoreVerifier.sol:62:43:
   |
62 |                 success := staticcall(sub(gasleft(), 2000), 7, mIn, 96, mIn, 64)
   |                                           ^^^^^^^

oops, then have to do something like:
   uint256 gasToUse = gasleft() - 2000;
   assembly {
       let gasForCall := gasToUse
       success := staticcall(gasForCall, 7, mIn, 96, mIn, 64)
   }

which u don't like.

the opcode rules disallow using gas like this (gasleft should compile to a gas call)

I take another (deeper) look at the SemaphoreVerifier.sol.
First on L#89 calling g1_mulAccC(), it is passing:

_pVk: 64 bytes containing the 4th and 5th entries of vkPoints.
the 6th (192/32) entry of vkPoints
the 7th (224/32) entry of vkPoints
the 0th entry (0) of pubSignals.

Inside g1_mulAccC() function, L#62 will always NOT fail. This is because the first two parameters, x and y, are valid constants included the contracts, vkPoints, with s (the pubSignal input from user) can be any 32-byte value.
Then pR used in L#69 and L#70, which is the _pVk above, are coming from vkPoints.
This logic applies to three others g1_mulAccC() calls in L#91, L#98, L#105.
Hmm... what I want to say is it may not be that straightforward to make L#62 and L#72 to revert.
Let me ponder more on this.

to fix your trouble check this solution click
maybe this will solve your problem.

Inside g1_mulAccC() function, L#62 will always NOT fail. This is because the first two parameters, x and y, are valid constants included the contracts, vkPoints, with s (the pubSignal input from user) can be any 32-byte value.

I looked at this a bit and it seems like you're right. If the codepath cannot revert then i think it's okay to not test the revert. e.g. we just remove the sub call and add a comment saying a revert is unreachable.
On a somewhat related note this looks like a good opportunity for optimization. We could remove a bunch of mload and add calls. This should happen in a separate issue though.

please refer to the updated #883.


autoplay for vimeo modal

hello everyone,
I just wanna know if it's possible to apply an autoplay for vimeo modal. Can you help me?
Tks,
Edy

Can you provider a jsfiddle of the code you are using?

Hello,
This is the page: http://gilproducoes.com.br/2016/social/trabalhos-aniversarios-15-anos.html
When you click on the thumb, open the modal with vimeo video without autoplay. Can you help to solve that?

Try:
$.embedly.defaults = {
  key:':your-key',
  query: {
    maxwidth:768,
    autoplay: 1
  },
  display: $.noop
};

@edycastro1
Creating a reduced test case is the best way to debug problems and report issues. Read http://css-tricks.com/reduced-test-cases/ on why they’re so great.

A reduced test case clearly demonstrates the bug or issue.
It contains the bare minimum HTML, CSS, and JavaScript required to demonstrate the bug.
A link to your production site is not a reduced test case.

Creating a reduced test case is the best way to get your issue addressed. They help you point out the problem. They help us debug the problem. They help others understand the problem.

@screeley Thanks a lot. This tip works, look: http://gilproducoes.com.br/2016/social/trabalhos-aniversarios-15-anos.html
But, now I have another problem. When I close the modal, video keeps playing. Any idea about what happen?

Sadly no, a reduced test case would help.


Blank root "GET /" for Heroku deployment

On a local deployment, this angular seed works just fine: I can see localhost:9000/, sign-in, etc. However, deploying to Heroku with the "Deploy on Heroku" button gives a blank page: Not Found for 'GET /'
The system is deployed correctly, and the logs show Node.Js, a Yeoman sanity check, and all bower components being installed.
A call to /user gives "Not Authenticated" indicating the JSON APIs are up and running. To me this indicates a "routes" problem, perhaps with Yeoman.
GET         /                                com.tuplejump.playYeoman.Yeoman.index
->          /                                yeoman.Routes

I've tried adding a "grunt build" to the package.json scripts, which runs grunt, but still gives me a blank page.
"scripts": {
    "postinstall": "cd ui && npm install && bower install && grunt build"
  }

Any suggestions? Again, works fine locally, and the Heroku logs don't give me any errors.

This seems to be the case for your master deployment as well... http://play-silhouette-angular-seed.herokuapp.com/

I'm aware of this issue and I think it has to do with https://github.com/tuplejump/play-yeoman/issues/95

Fixed with the workaround posted in https://github.com/tuplejump/play-yeoman/issues/95#issuecomment-233626696


[Bug]: Empty Stats > No Data on dark mode

Prerequisites

[X] I have searched the backlog for duplicate or closed feature requests

Your test device
NA
OS Version
NA
App version
v0.18 - b369
Describe the issue
The image does has 2 issues:

in dark mode: the image is not
in any mode

the style of this illustration is not the same as the others that are used in First use, user cleared or error
the data illustration is using other branded app, and as it will be used inside an orange app it should only express the no data for this app.

Expected Behavior
change the image to work in dark and light mode and change the illustration to express not data available in the same style as the other illustrations

Another illustration will be provided by @mccart77


 Previous attempts at fixing the nondeterministic test failures:
https://github.com/alphagov/publisher/pull/401
https://github.com/alphagov/publisher/pull/402

:'(


EpubJS CFI saving chapter not page

book.js
var saveBookPageBeforeClose = async function(){
    if (book_rendition) {
        let location = book_rendition.currentLocation(); // this
        let cfiString = location.start.cfi;// this
        var books_json = await getBooksFromJson();
        await changeValueInJsonBook(books_json, epubCodeSearch, "lastPageOpened", cfiString)
    }
}

fixed with https://github.com/mignaway/EpubReader/commit/1aedaaa748eb75527cca21100bc9f5bba69ced47


Stormfront + sources

Fixes all bugs found in stormfront on trello.

Why didn't you just use
 import static tcgwars.logic.effect.Source.*?

intellij did auto import source and I was getting issues with POKEPOWER and POKEBODY defaulting to effect types instead of sources.


Make the File object a more formal definition and explore type enforcement throughout the code base with TypeScript or other tool

Added flow types.  Yay!


Complete review of docs to clean up any TODOs and broken links

Also remove Using Macroeco section, as not critical given that reference is present.

Updated the tutorial_macroeco.rst  in commits 3bac540 and 682a6c6. Has examples and figures now.

Added explanation of MecoDesktop output to getting started.  We might want more output explanation in the other tutorials.

o_ring and comm_grid don't have tutorials. @jkitzes, not trying to pass the buck, but I figured you knew these better so I left them for you.

Not to eliminate the buck entirely (?), but do they need tutorials? I figured if we show a few cases that give the overall framework, we shouldn't need an explicit tutorial for every function - what do you think?

Sounds fine to me. Closing...


Investigate using nanoserde instead of serde

Description
As part of the effort of trying to reduce the build time we might consider replacing serde by nanoserde.
https://github.com/not-fl3/nanoserde

I would like to tackle this issue

Hey @abdelhamidbakhta after conducting some research on nanoserde and other alternatives like miniserde these are the findings :
NanoSerde
It was updated last 3 months ago, the API for the serialize and deserialize are different from serde (doesn't return errors on serialize), and doesn't include an impl_fixed_hash_serde! macro for example. There are multiple packages that we build that still use serde underneath (sp-runtime, tracing-subscriber, sp-core, etc) so removing it means that we will still be building it.
Miniserde
It's JSON only, and has worst performance (between 1.5 to 2 times) in some use cases (as stated on the README).
TL;DR
Given this constraints our opinion is that we shouldn't move forward with this.

Hey @abdelhamidbakhta after conducting some research on nanoserde and other alternatives like miniserde these are the findings :
NanoSerde
It was updated last 3 months ago, the API for the serialize and deserialize are different from serde (doesn't return errors on serialize), and doesn't include an impl_fixed_hash_serde! macro for example. There are multiple packages that we build that still use serde underneath (sp-runtime, tracing-subscriber, sp-core, etc) so removing it means that we will still be building it.
Miniserde
It's JSON only, and has worst performance (between 1.5 to 2 times) in some use cases (as stated on the README).
TL;DR
Given this constraints our opinion is that we shouldn't move forward with this.

Ok closing the issue then, thanks


Footnote and other minor fixes in SSA research page

FYI @jwal1024, some issues created because footnote markers can't have spaces, and some other small typos.

The sub-bullets within the numbered list within the quote within the footnote don't render right...this might be a JupyterBook/Markdown edge case haha. I think I tried all the potential combinations.


Fixed regression bug in validateTxid()

Pull Request Test Coverage Report for Build 2678

0 of 0   changed or added relevant lines in 0 files are covered.
244 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.6%) to 66.273%

Files with Coverage Reduction
New Missed Lines
%

dist/routes/v2/slp.js
244
43.53%

Totals

Change from base Build 2674:
0.6%

Covered Lines:
2375

Relevant Lines:
3383

💛  - Coveralls


Use Latest OPENCV BASE VERSION [IGNORE INTERMEDIATE BUILDS]

What this PR does / why we need it:
Follow up to https://github.com/project-akri/akri/pull/422
If applicable:

[x] version has been updated appropriately (./version.sh)
[x] all commits pass the DCO bot check by being signed off -- see the failing DCO check for instructions on how to retroactively sign commits

Need to wait for this OPENCV base build action to succeed before re-running the tests


make pubsub work with new gax

LGTM after the build succeeds

Changes Unknown when pulling 0559a75fc80317f79d39334b2f3a8ce2896e3d15 on pongad:flow-opt into ** on GoogleCloudPlatform:pubsub-hp**.


Consider wearing of face masks in implementation

corona-warn-app/cwa-app-android#973  Avoid duplicates

[x ] This enhancement request has not already been raised before
[ ] Enhancement request is specific for Android only, for general issues / questions that apply to iOS and Android please raise them in CWA-Wishlist
[ ] If you are proposing a new feature, please do so in CWA-Wishlist

Current Implementation
Wearing of face masks is not considered in implementation, especially not in transmission risk calculation and threshold calculation.
Suggested Enhancement
Take in account if I wear a face mask (community mask or FFP2) or not by providing the ability (optional) to input this to the app via a radio button. The threshold for warning messages should use this input as additional parameter.
Expected Benefits
The number of high risk messages might be reduced as both parties (infected person and person close to him/her) might benefit from enhanced shelter.

Dear @tangomanni,
Thank you for your contribution. Your proposal is quite similar to https://github.com/corona-warn-app/cwa-wishlist/issues/251. Therefore I close this issue here and would like to ask you to continue any further discussion on this topic there, please. Thank you again!
Best wishes,
DS
Corona-Warn-App Open Source Team

Duplicate of https://github.com/corona-warn-app/cwa-wishlist/issues/251


Fixes error from accessing a None type object.

Fixes error from accessing a None type object by moving outstanding_action initialization outside an if statement.

Have you tested a tactical report in Slack where there the error condition exists?

Have you tested a tactical report in Slack where the error condition exists?

Yes. I've replicated the issue and tested this bugfix


Featherjs-client socket.io in Nuxt ssr mode

I use feather-client whith socket.io in nuxt ssr render.
My ssr nuxt logic:
user request income => plugin check cookie => getToken=> authenticate feather-client
Issues
Every user request income nuxt server => nuxt server create socket.io to backend => connect socket.io don't close after nuxt render.  It is unstable
I try disconnect socket in nuxt hook. But nuxt hook run in global, not in standbox every request income user.
video Issues
https://youtu.be/1RsnOp05ARI
your project
https://github.com/nvthuong1996/front-end
I'm not good at English. If I use the word incorrectly I'm sorry

My recommendation is to use feathers-rest on the Nuxt server.  I've never been able to get a socketio connection reliably working in any SSR framework.  This is also the only scenario where I recommend using a hybrid approach on the client.  If you configure feathers-rest on the client, then listen to socket events separately (since feathers-client doesn't support two simultaneous transports), you'll probably have an easier time getting things working and a more reliable setup.

I'm closing this now that we have a full Nuxt example in the docs.  Thanks!


[FEATURE] - Line Chart with Colored Filled Regions (instead of a single color only)

Expected Behavior
Most color options in chart.js accept either a single color or an array of colors to control the corresponding element.  For example, you can configure a line chart to have different colored points using the pointBackgroundColor option and passing in an array.
However, this same concept is currently not supported for the line chart fill color (e.g. backgroundColor)
Current Behavior
The new behavior would allow the line chart backgroundColor option to accept an array of colors to produce colored fill areas.  Here is an example.
Possible Solution
Based on the tagged v2.5.0 source, logic could be added to Chart.elements.Line.draw() to build a canvas linear gradient based on the position of each point and use this as the ctx.fillStyle instead of a single color.
Here is a codepen example where I have added the logic in at runtime.
Context
I needed this capability in some charts used within my company.  I figured if I needed it then others might need it as well.
Environment

Chart.js version: v2.5.0
Browser name and version: All
Link to your project: http://codepen.io/jordanwillis/pen/BWxErp

@jordanwillis I like this idea and the LinearGradient is the best way to do this.
I know that some developers at the BBC built https://github.com/bbc/Chart.Bands.js/ which automates the process of gradient creation to change the line colour above a certain data value. Maybe this would fit in there too? It's a bit different in implementation in that it does not change the line drawing but instead just assigns the gradient as the line fill (which should already work).

@etimberg I actually came across the plugin that you mentioned as well, but it seemed fundamentally different to me in that it's purpose is to configure a horizontal gradient (around a certain y-axis value).  A.k.a. it adds a new concept that does not currently exist anywhere within chart.js (thus it requires a whole set of it's own configuration options).  Also, it seemed you guys were already working on something similar in the chart.js core based on #4008 (plugin.filler.js)
In my case, I was thinking it might make sense to allow the line chart backgroundColor option to accept an array of colors since all other color option properties (e.g. line - pointBorderColor, pointBackgroundColor, pointHoverBackgroundColor, pointHoverBorderColor; bar chart - backgroundColor, borderColor, hoverBackgroundColor, hoverBorderColor) accept either a single color or an array of colors.
It just so happens that the only way to accept backgroundColor as an array, is to use a vertical LinearGradient (easier approach) or change the way the fill is drawn (e.g. drawing enclosed polygon boundaries between each datapoint...obviously the more complicated approach).
So long story short (thanks for sticking with me so far!), I wanted to first check to see if you were interested in pulling this directly into the core since it seems to be in line with all other chart color options.
Otherwise, I'd be more than happy to publish as a standalone plugin that would overwrite the line chart backgroundColor type.

@jordanwillis sorry for the super late reply on this. I'd be happy to consider an improvement here. As you noticed, we refactored some of the line fill code into an internal plugin so this would probably fit in OK there.
I think allowing backgroundColor to be an array would be a good way to handle this. In terms of implementation strategy, I'm not sure which is the easiest given the code we have now.
@simonbrunel thoughts?

@etimberg Sounds good!  Let me review the latest line fill code changes and get back with you on an implementation approach.

https://github.com/bbc/Chart.Bands.js/ Doesn't work with chartjs version 2.6.

What ever happened with this request?

Hallo.
Very nice job.
I have just implemented this in my app but the only thing that goes wrong is that you are actually overriding the LINE type chart, so when i want to use a LINE chart ( the regular one ) , I cant. It will throw an exception  because it looks for an array of colors.

Hello,
Thanks for your help!
For a large line chart time, i.e a year chart, the line below needs to be changed from:
var colorStopPosition = roundNumber((point._model.x - minX) / (maxX - minX), 2);
to:
var colorStopPosition = roundNumber((point._model.x - minX) / (maxX - minX), 3);

Added a sample file showing how to use gradients as a fill in #6666. This can be used to implement this case


Improved navigation and updated README.

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Example for two way communication with a background goroutine

This is to request a new example showing two way communication between bubbletea and a background process (running as a goroutine.)  There are already two examples (send-msg and realtime) which demonstrate one way communication with bubbletea, however it would seem to be a common use case to need two way communication.
For example, the background process may do some work and need to be able to push updates into the model so that the results can be displayed in the interface.  But it also makes sense that the background process might want to change its behavior based on the user interacting with the interface, meaning it also needs to also be able to pull information from the model.  It would be great to have an example showing this two way communication, a background goroutine that changes its behavior based on flags in the model, as well as the model being updated based on work done in the background goroutine.

Great idea. This would be really useful (and fun to make).

I've submitted a PR that includes an example addressing the requested use case. I would greatly appreciate your insights and feedback!


wrap calls to klr in quotes

Fixes issue if there is a space in the users home path

Nice. Thanks!


can not luanch the gallery

Describe the bug

after publish the app with win-x64 rid, i can not launch the app.
Steps to reproduce the bug

Steps to reproduce the behavior:

download the winui gallery sample project
publish with win-x64 rid
launch the app in file explorer

Expected behavior

launch the app
Actual behavior

nothing happens
Screenshots

Application: WinUIGallery.DesktopWap.exe
CoreCLR Version: 6.0.522.21309
.NET Version: 6.0.5
Description: The process was terminated due to an unhandled exception.
Exception Info: System.DllNotFoundException: Unable to load DLL 'Microsoft.ui.xaml.dll' or one of its dependencies: 找不到指定的模块。 (0x8007007E)
   at AppUIBasics.Program.XamlCheckProcessRequirements()
   at AppUIBasics.Program.Main(String[] args) in C:\Users\User\Desktop\WinUIGallery\obj\WinUIGallery.DesktopWap\x64\Debug\net6.0-windows10.0.18362.0\win10-x64\App.g.i.cs:line 28

Version Info

latest
Gallery version (found on Settings page):

latest

他们没测试好就发出来吗？

我感觉是的，我工作负载是根据官方文档勾选的，所以环境应该是不缺的，完全新下载的代码，全新的编译，都跑不通就不是我的问题了

Thank you @liesauer for filing this issue. I recently ran into the same issue and here is what I did to fix it:

Unload both projects in the solution
Clean solution
Rebuild solution

This worked for me so I'll close this issue but feel free to tag me in this thread if the error persists.

Sorry for the delayed response. It seems like you are missing the WinUI library. Can you check that you have the latest Microsoft.WindowsAppSDK nuget package?

i can comfirm that i have Microsoft.WindowsAppSDK package. it's easy and 100% reproducible.

a fresh system(Windows Sandbox in my case)
download latest VS and install all required workloads and components following this article: https://learn.microsoft.com/en-us/windows/apps/windows-app-sdk/set-up-your-development-environment?tabs=vs-2022-17-1-a%2Cvs-2022-17-1-b#required-workloads-and-components
download and install latest wasdk runtime
clone WinUI-Gallery
launch WinUIGallery.DesktopWap
crash

the key problem is VS can load the installed wasdk runtime dlls from some kind of mysterious location, and missing some of the runtime dlls when you build the application and the application seems have no this magical ability to load the Microsoft.ui.xaml.dll.

Clearing the microsoft.ui.xaml folder in the .nuget folder should resolve this issue.


--separation sendEmail logic to another method .

--separation sendEmail logic to another method .
--separation saveEmailTraces logic to another method .
--separation create SmtpClient to another method .
--separation create MailMessage to another method .
--separation create EmailTraces to another method .
--change accessibility public methods to private .
--add EmailMessage Dto .

I love your changes, but please remove region preprocessor directives that you added.

Thank you for your attention. The changes have been made.

Thank you. The changes have been made.


[ODS-6178] Remove Admin App, Cleanup Extra Review Comments

Package versions need to be updated after changes done in thicket related to this PR

@simpat-jesus can you help me identify which package versions need to be updated? In that PR I just see extensions.

@stephenfuqua here are the updated packages

Thank you, @simpat-jesus, I'll update all of those. I never would have expected those packages to be updated by the PR in the Extensions repository.


go-ethereum: 1.14.6 -> 1.14.7

Description of changes
https://github.com/ethereum/go-ethereum/releases/tag/v1.14.7
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[x] sandbox = true

[x] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 326432 run on aarch64-darwin 1

3 packages built:

go-ethereum
go-ethereum.clef
go-ethereum.geth

Result of nixpkgs-review pr 326432 run on x86_64-linux 1

3 packages built:

go-ethereum
go-ethereum.clef
go-ethereum.geth


fix: return PermissionDenied for resource permission errors in GCP

If a user doesn't have permission to list resources and their labels (like buckets and topics) the error is wrapped when returned, so it won't be picked up as a PermissionDenied error (see here). It shouldn't be a problem for now since every function currently has list permissions, just something to think about.


Add how-to for creating infra and IAM resources separately

What this PR does / why we need it:
Adds documentation for creating infra and IAM resources separately from
cluster creation with the CLI. We currently only document how to create clusters
all at once.
Checklist

[x] Subject and description added to both, commit and PR.
[ ] Relevant issues have been referenced.
[x] This change includes docs.
[ ] This change includes unit tests.

sweet, thanks!
/lgtm


Clicking upon cards in AgroBlogs should redirect to the respective blog.

Hi @Suchitra-Sahoo
Under AgroBlogs, title of blog should be clicked to open the blog. Clicking on anywhere in the card won't open/redirect to the blog.
Can I enhance this by making the whole card clickable ?
This will also improve user experience of website.

Please assign this issue to me.
I am a contributor in GSSOC 2024.
I would like to contribute to this project.

@RanjanaShenoy assigned. You may start working on it.

Thank you @Suchitra-Sahoo.
Is there any discord channel or any other communication channel? Please add me to that. Please provide me your discord username. I would be happy to discuss with you incase of doubts.


Question about how insert works

I am reading your source code.
It is very cleverly designed and interesting. Great work!
I have one question. In your implementation of insert, the expression dist_and_fingerprint <= rhs.m_dist_and_fingerprint appears.
This works the same as the general Robin Hood hashing when dist is different. However, when dist is equal, it may be inserted before the intended position, depending on the value of fingerprint.
Is there any benefit to this behavior other than simplifying the implementation?
I understand that this works correctly.
For insert it is dist_and_fingerprint <= rhs.m_dist_and_fingerprint, for find it is dist_and_fingerprint > rhs.m_dist_and_fingerprint, which is symmetrical.

Hi! The benefit is that I don't have to check each element when distance is equal, I only have to check until the fingerprint is too large. So when you have e.g. 10 elements that hash to the same bucket and I want to find an element that hashes to the same bucket but is not there, on average I only have to check 5 of them (until dist_and_fingerprint is too large). Without that sorting, I would have to compare all 10 fingerprints.
As far as I know no robin hood hashing except my implementations use this behavior.

I see! I understand it well.
I think there is a slight performance penalty to insert by inserting the element at a slightly more forward position than intended.
But if find is faster, I think it's a good trade-off.
At least in my application, the performance of find is more important.
Thank you for your clear answer!


feat : token details form

issue: #242

please kindly review this work

@ugur-eren please check again

@Jemiiah linting issues are still present in the frontend, also please revise the design of the form so it matches the application itself.

@ugur-eren is there any way to see the page because we don’t have a deployed token yet is there any work, I can use to see page as I do the design

You can temporarily disable the is owner check and see your form. You can use any launched token, for example PAL 0x049201f03a0f0a9e70e28dcd74cbf44931174dbe3cc4b2ff488898339959e559


Custom query option

Summary
Implement the option to specify a 'query' instead of a 'table'
Description

Add optional parent trait than can be either a table name or a query
Parse one or the other on read
On write, error if query is specified and table is not
Test this using a join and aggregation

Related Issue
VER-76082
Additional Reviewers
@jonathanl-bq
@NerdLogic

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Feat 32 code ocean registration

Closes #32
Closes #38
Allows user to optionally define the raw_data_source folder using a command line argument since that option is changed more often than the other configs.

I added some basic unit tests for the post request to code ocean, but we may want to figure out a more robust method for testing out that process.