Add errata supporting changes to position: static

Summary:
X-link: https://github.com/facebook/react-native/pull/41130
I will use this errata to gate my changes that actually make position: static behave like the web. We have future plans to make position: relative the default again but users could still have declared certain nodes as position: static, so I think this is needed regardless.
Differential Revision: D50506915

FYI @nicoburns in case Taffy wants to support position: "static" in the future (esp https://github.com/facebook/yoga/pull/1434/commits/2f42c9d33e709e686bb508ed7058c3b0acac117a which adds fixtures).

Oo, yeah this has been requested (https://github.com/DioxusLabs/taffy/issues/212), and is definitely needed if one wants to render a "default web div".
@joevilches I'd be interested in what your implementation strategy for this is. In particular, have you thought about:

How this interacts with caching? In particular, in the incremental relayout case where one of the node's position styles has changed.
Whether there needs to be some kind of output of parent-child linkage between containing blocks and their non-direct-child contained nodes (for consumers of Yoga/Taffy)?

Also, I think some px tests would be good.

Caching is a good callout. Right now some of the refactoring is to pass containing block when recursing during layout, instead of just parent dimensions. Containing block dimensions might need to be added as cache keys, like owner size is right now.

Caching is a good callout... Containing block dimensions might need to be added as cache keys, like owner size is right now.

Yeah, that would probably work. Actually, I figured that what we call "parent size" in Taffy might be called "owner size" in Yoga precisely because for this use case. I think that the containing block dimensions could probably be used instead of the "parent size" in the cache key (bearing in mind that for non-absolutely positioned nodes the parent is the containing block). Which would be nice, to avoid bloating the cache size.

The history of “owner” is in this commit: https://github.com/facebook/yoga/commit/f0edefdbb741f22aad77551dcedef59baab7a63f
The history here is that the React Native Fabric renderer keeps a Yoga node per ShadowNode, but Fabric ShadowNodes are immutable after being laid out. https://reactnative.dev/architecture/render-pipeline
This clashes with Yoga’s existing public API, which is oriented around nodes being mutated throughout their lifetime. Fabric took a dependency on Yoga internals, to build this different model, with some of it making its way into Yoga itself. I’ve been thinking about how we can move that boundary into something public in the future.
Out of probably a couple dozen of Yoga using frameworks in our monorepo, Fabric is the only one using the private APIs. All others are now set up to only have visibility to public headers.

Hmm... a node being able to have more than one parent doesn't sound like a fantastic idea. I'm guessing this is to allow for some kind of copy-on-write semantics on the Fabric side? I'd need to look at the code, but off the top of my head it seems to me that:

Either old trees are relayouted after a newer tree is created. Which sounds like it wouldn't be great for perf due to cache thrashing, and should perhaps lead to the Yoga node being cloned too.
Or they're not, in which case I don't understand why old ShadowNodes would need to retain their references to a yoga nodes at all. The only reason I can think is to access to the computed layout. And I'm thinking maybe this could just be copied into the ShadowNode.

Long term, it would probably be good to make Yoga's algorithms generic over the node type they operate over, and make them operate directly on ShadowNodes in Fabric.

Copy-on-write is correct. Fabric treats a node owned by a different parent as one that cannot be mutated, so we lazily create new writable Yoga nodes during layout.
Layout information is copied to the ShadowNode, but original Yoga node needs to be cloned when creating new ShadowNode revision. This also clones the nodes layout cache. Then Fabric will dirty on node prop change. Shadow trees are allowed to be created on multiple threads, and multiple revisions can exist, derived from a single ancestor revision.
There are a bunch of ideas on how to refactor this eventually.

@nicoburns Yeah what Nick said is the gist of it. Basically just plan to pass down containing block information in the calculate layout functions, haven't planned much else beyond that. Caching was not on my mind initially but I will keep an eye out for it after the discussion here. I plan on getting started with these things this week, we shall see how much progress I make :)


help me about DID scanning

Hi Jacob,
i don't know how to scan DID with DoiPClient library.
I use this piece of code:
from doipclient.connectors import DoIPClientUDSConnector
from udsoncan.client import Client, Request
uds_client = Client(uds_connection)
with Client(uds_connection, request_timeout=2) as client:
try:
client.read_data_by_identifier_first([0xF190])
except NegativeResponseException as e:
print('Server refused our request for service %s with code "%s" (0x%02x)' % (e.response.service.get_name(), e.response.code_name, e.response.code))
except (InvalidResponseException, UnexpectedResponseException) as e:
print('Server sent an invalid payload : %s' % e.response.original_payload)
and when I run this script I 've this error:
[ConfigError] : Actual data identifier configuration contains no definition for data identifier 0xf190
[ConfigError] : Actual data identifier configuration contains no definition for data identifier 0xf190
For me that normal, because I've don't initiate the config dictionnary with right parameters for this record.
But my aim is to discovers services and DID in UDS so how to define something I don't know at the beginning ?
So if you can help me ?  many thanks by advance :)

This question is better suited to the udsoncan project as DID's are not DoIP related. But from what I recall there is no such service in UDS. You're only options are to:

Know the DIDs because you're the OEM and the define the config struct accordingly
Have access to a definition file like ODX which defines these things in a machine readable format
Fuzz the network and figure out from the errors what ID's are valid in any given session and what their buffer sizes are

Thanks for your answer.
I m trying something
Best regards
Le dim. 28 août 2022 à 08:37, Jacob Schaer @.***> a
écrit :

Closed #19 https://github.com/jacobschaer/python-doipclient/issues/19
as completed.
—
Reply to this email directly, view it on GitHub
https://github.com/jacobschaer/python-doipclient/issues/19#event-7272984372,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB565KH3D4SNCYMWERIYUVLV3MCL7ANCNFSM57SGO2IA
.
You are receiving this because you authored the thread.Message ID:
@.***>


Install process went fine, no errors, but the chrome extension is lot loading.

When you start elementor a temporary protractor config file is created. If you look at the log you will see something like this:
Creating protractor configuration file
Starting protractor
Running command: [node /home/andres/.nvm/v0.11.16/lib/node_modules/elementor/node_modules/protractor-elementor/lib/cli.js --elementExplorer true --debuggerServerPort 6969 /tmp/protractor-conf115316-11669-10le421]

Where /tmp/protractor-conf115316-11669-10le421 is the temporary file.
The configuration file looks like this:
exports.config = {
  seleniumAddress: 'http://localhost:4444/wd/hub',

  capabilities: {
    browserName: 'chrome',
    chromeOptions: {
      args: [
        '--load-extension=/home/andres/.nvm/v0.11.16/lib/node_modules/elementor/extension'
      ]
    }
  }
};

Can you send me the text of your configuration file? Maybe there is a space in the path and chrome cannot find the extension.

exports.config = {
//directConnect: true,
seleniumAddress: 'http://localhost:4444/wd/hub',
capabilities: {
browserName: 'chrome',
chromeOptions: {
args: [
'--load-extension=C:\Users\derrickfink\AppData\Roaming\npm\node_modules\elementor\extension'
]
}
}
};
On Thu, Apr 16, 2015 at 9:33 AM, Andres Dominguez<EMAIL_ADDRESS>wrote:

When you start elementor a temporary protractor config file is created. If
you look at the log you will see something like this:
Creating protractor configuration file
Starting protractor
Running command: [node /home/andres/.nvm/v0.11.16/lib/node_modules/elementor/node_modules/protractor-elementor/lib/cli.js --elementExplorer true --debuggerServerPort 6969 /tmp/protractor-conf115316-11669-10le421]
Where /tmp/protractor-conf115316-11669-10le421 is the temporary file.
The configuration file looks like this:
exports.config = {
seleniumAddress: 'http://localhost:4444/wd/hub',
capabilities: {
browserName: 'chrome',
chromeOptions: {
args: [
'--load-extension=/home/andres/.nvm/v0.11.16/lib/node_modules/elementor/extension'
]
}
}
};
Can you send me the text of your configuration file? Maybe there is a
space in the path and chrome cannot find the extension.
—
Reply to this email directly or view it on GitHub
https://github.com/andresdominguez/elementor/issues/10#issuecomment-93765005
.

--
Derrick Fink,
Testing and Quality Assurance
Web & Mobile Applications
Boise State University
<EMAIL_ADDRESS>

I think I found a solution. I don't have my laptop with me (using my wife's laptop). I will get it fixed soon.

I was able to manually load the extension, without changing anything in the
manifest file, by loading the unpacked extension in the developer options.
It seems to work if you do it that way.
On Thu, Apr 16, 2015 at 6:43 PM, Andres Dominguez<EMAIL_ADDRESS>wrote:

I think I found a solution. I don't have my laptop with me (using my
wife's laptop). I will get it fixed soon.
—
Reply to this email directly or view it on GitHub
https://github.com/andresdominguez/elementor/issues/10#issuecomment-93869445
.

--
Derrick Fink,
Student Web Developer
Web & Mobile Applications
Boise State University
<EMAIL_ADDRESS>

I've found but am unable to load the manifest.json file
Failed to load extension from: ~
Could not load background script 'scripts/background.js'.
I also can't find the Chrome Automation Extension.

There was a problem in the path of the file for the extension. It looked like this:
c:\path\to\extension
Now the path uses forward slash:
C:/path/to/extension
I tested it on Windows 7. Let me know if it works for you. I just pushed a new version: 1.0.8

Thanks!
I can confirm the the new build (1.0.8) is working great.
Currently running Windows 8.1


openexr_3: 3.1.2 -> 3.1.3

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/openexr/versions.
meta.description for openexr_3 is: "A high dynamic-range (HDR) image file format"
meta.homepage for openexr_3 is: "https://www.openexr.com/"
meta.changelog for openexr_3 is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
/nix/store/kdnswhz5v757w9bgc69g4ysk5fd1nv5m-openexr-3.1.3-bin/bin/exrmakepreview passed the binary check.
/nix/store/kdnswhz5v757w9bgc69g4ysk5fd1nv5m-openexr-3.1.3-bin/bin/exrmultiview passed the binary check.
/nix/store/kdnswhz5v757w9bgc69g4ysk5fd1nv5m-openexr-3.1.3-bin/bin/exrenvmap passed the binary check.
/nix/store/kdnswhz5v757w9bgc69g4ysk5fd1nv5m-openexr-3.1.3-bin/bin/exrstdattr passed the binary check.
/nix/store/kdnswhz5v757w9bgc69g4ysk5fd1nv5m-openexr-3.1.3-bin/bin/exrmaketiled passed the binary check.
/nix/store/kdnswhz5v757w9bgc69g4ysk5fd1nv5m-openexr-3.1.3-bin/bin/exrmultipart passed the binary check.
/nix/store/kdnswhz5v757w9bgc69g4ysk5fd1nv5m-openexr-3.1.3-bin/bin/exrinfo passed the binary check.
/nix/store/kdnswhz5v757w9bgc69g4ysk5fd1nv5m-openexr-3.1.3-bin/bin/exr2aces passed the binary check.
/nix/store/kdnswhz5v757w9bgc69g4ysk5fd1nv5m-openexr-3.1.3-bin/bin/exrheader passed the binary check.
1 of 9 passed binary check by having a zero exit code.
9 of 9 passed binary check by having the new version present in output.
found 3.1.3 with grep in /nix/store/kdnswhz5v757w9bgc69g4ysk5fd1nv5m-openexr-3.1.3-bin
found 3.1.3 in filename of file in /nix/store/kdnswhz5v757w9bgc69g4ysk5fd1nv5m-openexr-3.1.3-bin

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

2 package rebuild(s)

2 x86_64-linux rebuild(s)
2 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
docbookrx
openexr_3

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/kdnswhz5v757w9bgc69g4ysk5fd1nv5m-openexr-3.1.3-bin \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A openexr_3 https://github.com/r-ryantm/nixpkgs/archive/9b938a0ce07639ed861e9c02e9c90edd07b5ede7.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/kdnswhz5v757w9bgc69g4ysk5fd1nv5m-openexr-3.1.3-bin
ls -la /nix/store/kdnswhz5v757w9bgc69g4ysk5fd1nv5m-openexr-3.1.3-bin/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

openexr_3

Maintainer pings
cc @paperdigits for testing.

Change looks good to me

Change looks good to me


Add missing code-style rules

Contributes to #28791.
Adds:

IDE0150
IDE0170
IDE0180

Renames option for IDE0039.

Thanks Bill and Youssef!


Fix data: ../../data/shuffling//Fiona/2023/10/01/8b78cae.csv column 4

Failure (???): ../../data/shuffling//Fiona/2023/10/01/8b78cae.csv column 4 elements are 1:52
1:52 not equal to sort(as.integer(d[, c])).
30/52 mismatches (average diff: 1)
[15] 15 - 16 == -1
[16] 16 - 17 == -1
[17] 17 - 18 == -1
[18] 18 - 19 == -1
[19] 19 - 20 == -1
[20] 20 - 21 == -1
[21] 21 - 22 == -1
[22] 22 - 23 == -1
[23] 23 - 24 == -1

Fixed in c4c6e92


Can't load the list

I tried running it this morning and it ran flawlessly, everything was fine, but when I tried downloading the next episode, the list was entirely 0/0,
This was the error: curl: (35) OpenSSL SSL_connect: Connection reset by peer in connection to animepahe.com:443
Pls help 🥺

Hi @karmakarmeghdip, from the error message you posted, it looks like a connection issue from server side. Maybe try it later? Hope it will work.


Small fixes

Avg response time
coverage on master

Summary of changes

Prevent Makefiles from getting into release archives (advanced users only)
Added News section to README w/ content about trying OpenCoarrays online
Don't include gasnet dir in release archives

Rationale for changes
Save users from themselves & allow users to try OpenCoarrays in the cloud w/ zero setup/config etc
Additional info and certifications
This pull request (PR) is a:

[x] Bug fix
[x] Feature addition
[ ] Other, Please describe:

I certify that

[x] I reviewed and followed the contributing guidelines, including
- Increasing test coverage for all feature-addition PRs
- Increasing test coverage for all bug-fix PRs for which there
does not already exist a related test that failed before the PR
- At least maintaining test coverage for all other PRs
- Ensuring that all tests pass when run locally
- Naming PR  to indicate work in progress (WIP) and to attach the PR
to the appropriate bug report or feature request issue
- White space (no trailing white space or white space errors may
be introduced)
- Commenting code where it is non-obvious and non-trivial
- Logically atomic, self consistent and coherent commits
- Commit message content
- Waiting 24 hours before self-approving the PR to give another
OpenCoarrays developer a chance to review my proposed code

LGTM
documentation changes only.


v14 alpha does not work with SSR

This is because of the following line accessing "window" on import:
https://github.com/streamich/react-use/blob/v14.0/src/useMeasure.ts#L50

Thanks, fixed!


[material-ui][Autocomplete] Fix regression in freeSolo Autocomplete to render custom Popper/Paper when options are empty

Fixes #44048
Regression from #41300
Ensure custom Popper and Paper components render in freeSolo mode and when there are no options.
We also maintain the fix from #41300.

Before fix: https://codesandbox.io/p/sandbox/material-ui-cra-ts-forked-4nxwj7
After fix: https://codesandbox.io/p/sandbox/material-ui-cra-ts-forked-qgg2rr

This looks too much like we are trying to fix the issue the user reported. We should either always render the popup when there are no options, or never render it. It shouldn't depend on whether the user provided a custom paper. I may have a custom paper but I don't want it to appear if there are no options. It almost looks like we should provide a prop for this, in case people truly need these two different use-cases.

This looks too much like we are trying to fix the issue the specific user had. We should either always render the popup when there are no options, or never render it. It shouldn't depend on whether the user provided a custom paper. I may have a custom paper but I don't want it to appear if there are no options. It almost looks like we should provide a prop for this, in case people truly need these two different use-cases.
It strange that our reaction to the issues is:

the paper is render when there is an empty array -> let's not render it
I have a custom paper I want it always to render -> let's render it if there is custom paper (the fact that there is custom paper, has nothing to do with the rendering or not of the popper)

If there is a use-case where people want to have the thing render regardless of whether the options array is empty, to me this is a new feature and we should have a prop to configure this. If we solved it that way the first time, we wouldn't have had regression, as we could have set up the prop default value to be aligned with the behavior we had before.

You're right. Initially, #40843 looked like a bug since custom styles on the Popper were showing up like a border. I think the best approach is to avoid rendering the Popper if there are no options. But now I'm unsure how to proceed with #44048.

I may have a custom paper but I don't want it to appear if there are no options.

Good point. I missed this scenario.

@mnajdova

This looks too much like we are trying to fix the issue the specific user had.

as the person who brought up the original issue, i agree that this may not be the ideal fix for the regression as it adds a new conditional for displaying the poppper in the renderer rather than rectifying the internal state in the autocomplete hook which, as i mentioned in my original bug report, is currently incorrect. as it is now with this regression in the code if the popper is not being displayed because of no items being present the autocomplete hook's open state will still "think" the popper is open even though it's not being displayed.

If there is a use-case where people want to have the thing render regardless of whether the options array is empty, to me this is a new feature and we should have a prop to configure this. If we solved it that way the first time, we wouldn't have had regression, as we could have set up the prop default value to be aligned with the behavior we had before.

i could be onboard with a new prop if it gets us what we had before. the only problem i have with that is technically this regression was a breaking change of behavior in the 5.x release and if you're adding the functionality back as a new feature i imagine it would only show up in 6.x. if that were the case, it seems like a big ask to ask users that want the behavior we already had in 5.x (before the regression) to upgrade to the latest major version to get that behavior back.
@ZeeshanTamboli
i think the ideal fix is refactoring the fix in https://github.com/mui/material-ui/issues/44048 to not change the renderer output manually but instead fix the autocomplete hook's internal state of "open" to correctly take the option count into consideration. ito resolve the regression it should also consider custom paper/poppers but how it decides to consider them could be based on a prop, as @mnajdova suggested. in the 5.x release i think it'd make sense for it to default to allowing the autocomplete to be "open" if there is a custom popper with no options as that was the behavior that existed before the regression. in 6.x i think you should be free to do whatever you deem appropriate as that is when breaking changes should be introduced, but defaulting to not displaying with no options but a prop allowing the custom paper to be displayed is acceptable i think.

@mnajdova @aarongarciah Are there plans for a v5 release? I can revert #41300 for v5 and, for v6, fix the open state and add the new prop.

My recommendation would be:

revert back to the original behavior for both v5 and v6
add prop for overriding the open behavior when there are no options available (the default should be whatever we had previously as a behavior)

This should solve both issues without disrupting people with introducing breaking changes.

@ZeeshanTamboli - are you still working on a fix for this?

@ibu-tv Are you using v5 or v6?


PSShouldProcess Rule and module specific adaption

Scenario
One of the default rules checks, whether a command that supports should process, actually uses ...
$PSCmdlet.ShouldProcess( ... )

Now, this is a great rule in principle, but in the framework module I maintain (https://psframework.org) I've got a cmdlet that replaces that method call (for readability and mocking reasons).
My current solution is to disable the specific script analyzer rule (PSShouldProcess) in my automatic tests. Which works well enough for my purposes.
But it would be more awesome if it were to detect the command and equate it with the method call (cause I'd actually appreciate having that check enabled, if it were practical).
Creating my own rule wouldn't be a problem, so my own projects would be covered. But since I'm trying to distribute this to others - with increasing success - distributing the custom rule to their tests would be ... inconvenient outside of PSScriptAnalyzer.
Questions
What's the policy on changes to rules that are module specific?
If I write that update, would it even be considered?

One can suppress a violation a per cmdlet basis, see docs and example here, I am not sure if you missed this feature or if I misunderstood the problem. Please note also related issue #945
There is no policy on rule change that are module specific, there are issues open e.g. what to do about Pester's Gherkin aliases, at the end everything is up for discussion and this change for the better

Heya, alas it appears to be misunderstood or ill explained :)
I'm fully aware of the ability to work around this limitations. For my own modules, I'm totally cool with the current status quo.
My desire however is to optimize to user experience of developers that build their tools on the PSFramework. At the moment they are forced to do either blanket or targeted exclusions, which I first have to explain to them (And I really want them to be able to just go and have fun with the framework without bothering them with those kinds of details). The cmdlet in question is not equivalent to the scenario discussed in #945 , where they rely on preference variable inheritance.
The only way I can see to make this work as desired would be to adjust the original rule to correctly identify the command and accept it as fulfilling its examples.

@FriedrichWeinmann does the proxy function support ShouldProcess?
If I remember correctly all rule checks is that either

$PSCmdlet.ShouldProcess is called

Any command that also supports ShouldProcess is called

So as long as your proxy command supports ShouldProcess and is resolvable, that should be enough to "fix" the warning.

hm, the cmdlet does not do so yet.
But if I understand #945 correctly that may not be a behavior I want to rely on remaining unchanged


Newtonsoft.JSON not detected in the vulnerability report with --hints option enabled

Issue details
Newtonsoft.JSON is not getting detected under the evidence section in the ODC html vulnerability report with the --hints option enabled when running the tool.
we have Newtonsoft.Json package with version 9.0.1 in the project.assets.json file which is vulnerable to CVE https://nvd.nist.gov/vuln/detail/CVE-2024-21907
Please find attached screenshots related to command run with the --hints file.
The hints file used has an entry to Newtonsoft.Json as a fix from issue #6789
Version of dependency-check used
Dependency-Check Core version 10.0.4
While running the tool it also throws this warning -> [WARN] Unable to determine Package-URL identifiers for 1 dependencies

Let us know if you need any other info.

can you provide the Evidence section from the HTML report?

Hi Jeremy, the issue is Newtonsoft.Json package is not getting detected under Dependencies (all) section and hence no evidence section populated for the same.

The assembly analyzer requires that you scan the DLLs - not the project files.

Hi Jeremy, we tried running the dotnet publish command before running dependency-check, it still did not detect the Newtonsoft.Json 9.0.1 version from project.assets.json, instead it detected "Newtonsoft.Json": "11.0.2" version from a newly added directory (created after running dotnet publish) under /bin -> /Debug/netcoreapp2.0/publish/Newtonsoft.Json.dll


4.9.2

@rultor release tag is 4.9.2

@rultor release tag is 4.9.2

@portlek OK, I will release it now. Please check the progress here

@rultor release tag is 4.9.2

@portlek Done! FYI, the full log is here (took me 9min)


[Draft] Add support for Azure storage container resources

Opening draft PR to get feedback. This PR adds support for:

Listing Azure storage container resources in a workspace
Generating SAS tokens for a storage container (via terra resource resolve …)
Creating and deleting controlled storage container resources

Not yet supported:

Creating a workspace with an Azure cloud context
Other types of Azure controlled resources (such as VM)
Authenticating with B2C

I tested locally against some Azure workspaces in broad-dev and everything works well. There aren't unit tests yet because there is no way to create a workspace with an Azure cloud context -- I can work on that next.

Thanks for the feedback @melissachang @jmczerk! I'll do another pass and re-request review when ready.


ci: trigger a build of downstream morpho-kit

Should fullfil the requirements outlined in NSETM-915 by triggering a
build of morpho-kit whenever a push to the main branch of MorphIO
occurs.
Needs to have the corresponding token set, still.

Closing. I think using a webhook would probably be simpler.


Tons of GUI related mappings + reorganization

Many things mapped, though the most notable change is a reorganization of gui related packages:

gui: Gui, GuiScreen, GuiContainer and GuiYesNo, the base / important / reusable classes

hud: Ingame overlay elements (GuiInGame, chat, scoreboard, ...)
widget: GuiButton and all other GUI elements
screen: Base directory for all GuiScreen classes

ingame: All GUIs relating to ingame / in-world stuff (books, signs, command blocks, ...)

container: Container classes

menu: All menu related GUIs, including the ingame menu

singleplayer

new

multiplayer
settings

I hope net.minecraft.client.gui.screen.menu.singleplayer.new is not too long of a package name. I figured it'd be more beneficial to organize the massive amount of menus instead of having them all collect in a single package. That means no impl package though :cry:
Currently, buttons is the name of the field holding all GUI widgets / elements. It should be renamed, even though everything currently seems to be building on the GuiButton class (which is a little frustrating).
Additionally, I was thinking maybe Gui is not the right term for something that seems to be a drawable 2D element (a button is just a GUI element, not a whole GUI), so perhaps it would make sense to find a more fitting name. For me, Render implies 3D and Draw is 2D, so maybe Drawable?
By the way, @Thog, Enigma could benefit from the ability to rename a whole package :wink:

Additionally, I was thinking maybe Gui is not the right term for something that seems to be a drawable 2D element (a button is just a GUI element, not a whole GUI), so perhaps it would make sense to find a more fitting name. For me, Render implies 3D and Draw is 2D, so maybe Drawable?

Rename Gui to Drawable, GuiScreen to Gui and HUD GUIs to something else? Count me in! :D
The cutoff for repackaging should be at screen/container (ingame sounds weird? maybe container?) and screen/menu levels, in my view.

Going to do the renaming to Drawable in another PR, since that'll affect a lot of field / method names too. As for reorganizing the GUI packages, please :+1: if you're okay with the following proposal or respond with criticism.

gui

hud
widget
screen

ingame
container
menu

The menu GuiScreens would be named differently, such as GuiSettingsAudio, GuiServerAdd and GuiNewWorldCustomize.


Compression

Add server-side compression, possibly with zlib.

Cannot be implemented. Encrypted data is almost random which makes it incompressible.


CIPD auth not working on macOS 13 flutter-devicelab-mac-x

Is there an existing issue for this?

[X] I have searched the existing infra issues

Type of Request
bug
Infrastructure Environment
Devicelab
What is happening?
flutter-devicelab-mac-11 is failing to download Xcode due to CIPD auth error
https://luci-milo.appspot.com/ui/p/flutter/builders/staging/Mac_arm64_ios flavors_test_ios_xcode_debug/306/overview
failed to resolve infra_internal/ios/xcode/mac@14c18 (line 1): prefix "infra_internal/ios/xcode/mac" doesn't exist or<EMAIL_ADDRESS>is not allowed to see it, run `cipd auth-login` to login or relogin
[E 2023-11-07 14:28:01] original error: exit status 1

Steps to reproduce
No response
Expected results
I expect Xcode to download and install

https://chromium-swarm.appspot.com/bot?id=flutter-devicelab-mac-10 is having same problem

All staging bots are having this problem.
Turns out the flutter-staging-builder lacks permissions to infra_internal/ios/xcode/mac. The try and prod pool have permissions, though. We've requested permission be given.

Permission has been given. And issue is fixed.


more css styling changes

added logo in navbar
changed padding so navbar doesnt break on resize
split about section in 2 parts
added 2 "sidebar" backgrounds on about section1 together with the text
gave margin and padding in about section 2 & changed background color: #f6f6f6;
reconstructed the whole devcash section into a structured card

Those changes sound good to me. I will merge now with the note that we need a new PR to address issues I've noticed with mobile compatibility.


feature request: Render emoji at larger size when it is the only content of a message.

here's how discord does it for example:

Related to #12857.

@zulipbot add "area: emoji"

here's how discord does it for example:

@YashRE42 I and my team would like to contribute to it. Please assign the work to us.

@dushyantgupta72 feel free to work on this.

Hi! I'm a first time contributor, is this an issue I can work on?

@zulipbot claim

Hey , @YashRE42 . Is there any threshold you would like to suggest regarding the height and width of emoji ?

Discord does 48x48, which looks fine. (their small version being 22x22, our current size being 20x20).

Cool , Working on it :)

Bumping this as it seems an interesting suggestion.

Bumping this as it seems an interesting suggestion.

Related discussion on CZO.
Note on the details for this feature:

In the screenshot in the issue, which is showing Discord's behavior, a message consisting of two emoji has them made bigger in the same way as if it's just one. I think that's the behavior I've seen in other implementations of this feature, too.

If you make a message out of like 100 emoji, those probably shouldn't be magnified, so maybe there should be some limit: the larger emoji apply only when the message consists entirely of up to N emoji, for some N like 5 or 10.

I'm marking this issue as "help wanted". The next step here is to rebase and address the feedback on the open PR (#16792).

@zulipbot claim

@zulipbot abandon

@zulipbot claim


Relative rediects per RFC7231 as the new default

In #747 @polymorphm pojnted out that according to RFC7231 (which obsoletes RFC2616) relative URLs in the Location header are allowed.
Current behavior: Bottle tries to turn relative URLs in bottle.redirect() calls into absolute URLs by joining with the request.url string. See https://github.com/bottlepy/bottle/blob/master/bottle.py#L2445
New behavior: Bottle just passes the provided URI as is. No URL-joining will be performed.
This behavior change should have no effect on well behaved clients. Relative URLs in Location headers are supported by all client libraries I know of because it is a common (previously non-standard but now allowed) practice for years now.
It is still a behavior change with no obvious upgrade path. This is why I opened this issue. If you can think of a situation where this change breaks an existing application, or introduce a security risk, please speak up.

What effect would this have on cross-domain redirects?

It had entered for the 0.12+ or just 0.13?

What effect would this have on cross-domain redirects?

@shakna-israel , like this (?) --
bottle.redirect('//my-other-domain.com/my-page/123')

or something another?

@polymorphm, That sounds good. Sorry for weighing in, just developing a CORS app at the moment, and it tends to be the kind of thing that breaks when trying to match URI compliance.

@shakna-israel , please , try test this changing in your CORS-apps
diff --git a/bottle.py b/bottle.py
index 535ddd3..359ceaf 100644
--- a/bottle.py
+++ b/bottle.py
@@ -2418,7 +2418,7 @@ def redirect(url, code=None):
     res = response.copy(cls=HTTPResponse)
     res.status = code
     res.body = ""
-    res.set_header('Location', urljoin(request.url, url))
+    res.set_header('Location', url)
     raise res

I think this test-result -- is will be important for all

@polymorphm, it seems to work absolutely fine.

The solution proposed by @polymorphm meets our case.

I think this should be linked to the following related issues: #748 and (the much older) #293
In my case, I have an application deployed on Apache with mod_wsgi at a subdirectory (e.g. example.com/myapp/). From my virtualhost config:
WSGIScriptAlias /myapp /var/www/myapp/app.wsgi
# ...

And in myapp.py:
@route('/<dirname>')
def bare_dir(dirname):
    redirect('/%s/' % dirname)

Even though the /<dirname> path matches example.com/myapp/abc, the browser is redirected to example.com/abc/. I need to use the following:
@route('/<dirname>')
def bare_dir(dirname):
    redirect('%s/' % dirname)  # no leading slash

I'm not an expert on web standards, so maybe this is the intended usage? But I expected a /url/ redirect to go to the application root and not the server root (as described in #293).

The server/app root problem is a very common and annoying thing. Unfortunately, there is no reliable way to know about the 'application root' other than configuration. A proxy could redirect arbitrary urls to the application, rewriting the URL in any way desirable. There is SCRIPT_PATH but that has its own issues.
Currently bottle resolves relative URLs against something it thinks is the server root. It uses request.url which is reconstructed out of different CGI parameters and breaks easily for proxied applications. So, by removing the automatic urljoin(), we would not change behavior for most apps expect for broken proxy configurations, which is acceptable I think.
Magically prefixing redirect paths with SCRIPT_PATH or a configurable 'application root' would however break existing applications, so that is not an option. You could define your own patched redirect() function in two or three lines of code it you really want this behavior.
I tent do (finally) just pull this simple change, and if we find out later that the missing urljoin breaks reasonable existing applications, we could still make it configurable.


fix example and hoc

Coverage increased (+0.5%) to 89.547% when pulling 26d760a6c37c432abc687d0291607383f8a45963 on feature/example-and-hoc into e7974ca9ec0c149f6db58141ce761a210b0a01fd on master.


Bluetooth: Mesh: Rename adv relay to adv simultaneous

Since notice that simultaneous advertising is not only used
by relay message, provision over pb-adv can also be used.
so it was changed to a more general name.
Signed-off-by: Lingao Meng<EMAIL_ADDRESS>

@LingaoM are you planning to do anything with this PR?

@LingaoM are you planning to do anything with this PR?

@LingaoM any updates on this?


We are logging newDot messages in Pusher event logs

Problem
Pusher logs are logging the entire event including the message being sent to users so anyone searching the logs can potentially see private messages.
Example from my expensifail account: https://www.expensify.com/_devportal/tools/logSearch/#query=request_id:("7cb42e2dcce32f88-LAX")+AND+timestamp:[2023-05-22T08:54:53.964Z+TO+2023-05-22T10:54:53.964Z]&index=logs_expensify-022447
Solution
Redact the message inside the event as we do in any other instance before storing the log.

Oh I meant to open this in our private repo. Too late I guess, I have already created a PR for it.

This has been deployed already, not sure why it was never closed.


export clean datasets step 1, 2 and 3 + intermediary tables

I am working on cleaning up questions 2.10 to 2.14. I've got the right dummy columns for each of these corrected with the excel files. However, I'm unsure what to do with 'none' and 'Other' answers.
All 'other' answers have been corrected with the excel files. So in which case could there still be a 1 in the Q2.10_Other dummy column?
(example below for question 2.10 but the same applies for the other questions)
My decision rule for Q2.10_Other=1: I assume that an unclassified 'Other' answer is only possible if the answer that was written could not be classified in any of the value types. In practice, this means that there are very few papers with other value types except for the one that were listed in the corrected version.
My decision rule for Q2.10_none=1: All Q2.10 dummy columns (excluding Q2.10_none) are zero
I'm hoping this is clear but I would understand if it isn't. I can explain more on Thursday!

hi Raisa
there could be 'real' others: things that are a modality of the variable
not offered in the choices.
so your decision rules seem correct!
Sander
On Tue, Dec 1, 2020 at 9:30 PM raisa_carmen<EMAIL_ADDRESS>wrote:

I am working on cleaning up questions 2.10 to 2.14. I've got the right
dummy columns for each of these corrected with the excel files. However,
I'm unsure what to do with 'none' and 'Other' answers.
All 'other' answers have been corrected with the excel files. So in which
case could there still be a 1 in the Q2.10_Other dummy column?
(example below for question 2.10 but the same applies for the other
questions)
My decision rule for Q2.10_Other=1: I assume that an unclassified 'Other'
answer is only possible if the answer that was written could not be
classified in any of the value types. In practice, this means that there
are very few papers with other value types except for the one that were
listed in the corrected version.
My decision rule for Q2.10_none=1: All Q2.10 dummy columns (excluding
Q2.10_none) are zero
I'm hoping this is clear but I would understand if it isn't. I can explain
more on Thursday!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/RCinbo/IPBESValueAssessment/issues/31#issuecomment-736800874,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AREM7L5CZ4T7ZZWS43HWCODSSVG5PANCNFSM4T3HMX3A
.


Crash and sound doesn't play since 0.17.1

Since version 0.17.1, my app crashes on android when playing sound with audiocache.
I use a singleton for audio and I have duckAudio enabled.
In my code, the crash happens at :
player.play(sound, mode: PlayerMode.LOW_LATENCY);
In audio_cache.dart, the crash happens at :
await player.play(
url,
volume: volume,
respectSilence: isNotification ?? respectSilence,
stayAwake: stayAwake,
recordingActive: recordingActive,
duckAudio: duckAudio ?? this.duckAudio,
);
E/AudioplayersPlugin(10602): Unexpected error!
E/AudioplayersPlugin(10602): java.lang.UnsupportedOperationException: Operation is not supported for read-only collection
E/AudioplayersPlugin(10602): 	at kotlin.collections.EmptyMap.put(Unknown Source:4)
E/AudioplayersPlugin(10602): 	at java.util.Collections$SynchronizedMap.put(Collections.java:2632)
E/AudioplayersPlugin(10602): 	at xyz.luan.audioplayers.WrappedSoundPool.setUrl(WrappedSoundPool.kt:247)
E/AudioplayersPlugin(10602): 	at xyz.luan.audioplayers.AudioplayersPlugin.handleMethodCall(AudioplayersPlugin.kt:62)
E/AudioplayersPlugin(10602): 	at xyz.luan.audioplayers.AudioplayersPlugin.onMethodCall(AudioplayersPlugin.kt:37)
E/AudioplayersPlugin(10602): 	at io.flutter.plugin.common.MethodChannel$IncomingMethodCallHandler.onMessage(MethodChannel.java:233)
E/AudioplayersPlugin(10602): 	at io.flutter.embedding.engine.dart.DartMessenger.handleMessageFromDart(DartMessenger.java:85)
E/AudioplayersPlugin(10602): 	at io.flutter.embedding.engine.FlutterJNI.handlePlatformMessage(FlutterJNI.java:692)
E/AudioplayersPlugin(10602): 	at android.os.MessageQueue.nativePollOnce(Native Method)
E/AudioplayersPlugin(10602): 	at android.os.MessageQueue.next(MessageQueue.java:325)
E/AudioplayersPlugin(10602): 	at android.os.Looper.loop(Looper.java:142)
E/AudioplayersPlugin(10602): 	at android.app.ActivityThread.main(ActivityThread.java:6541)
E/AudioplayersPlugin(10602): 	at java.lang.reflect.Method.invoke(Native Method)
E/AudioplayersPlugin(10602): 	at com.android.internal.os.Zygote$MethodAndArgsCaller.run(Zygote.java:240)
E/AudioplayersPlugin(10602): 	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:767)
With 0.17.0, I have no problem.
Regards,

Facing the same issue. Any progress?

Facing the same issue. Any progress?

I'm seeing the same in 0.17.1 and 0.17.2.  Several other similar reports on sites like StackOverflow when searching the error message and audioplayers in Google.
See also #717 and #721

I'm seeing the same in 0.17.1 and 0.17.2.  Several other similar reports on sites like StackOverflow when searching the error message and audioplayers in Google.
See also #717 and #721

Try setting the fixed player argument of audiocache to an AudioPlayer instance.

Try setting the fixed player argument of audiocache to an AudioPlayer instance.

Same here, here is a bit longer log output.
E/AudioAttributes(10407): Invalid stream type -2147483648 for AudioAttributes
W/SoundPool::StreamManager(10407): StreamMap: requested 100 streams, clamping to 32
E/AudioplayersPlugin(10407): Unexpected error!
E/AudioplayersPlugin(10407): java.lang.UnsupportedOperationException: Operation is not supported for read-only collection
E/AudioplayersPlugin(10407): 	at kotlin.collections.EmptyMap.put(Unknown Source:4)
E/AudioplayersPlugin(10407): 	at java.util.Collections$SynchronizedMap.put(Collections.java:2639)
E/AudioplayersPlugin(10407): 	at xyz.luan.audioplayers.WrappedSoundPool.setUrl(WrappedSoundPool.kt:247)
E/AudioplayersPlugin(10407): 	at xyz.luan.audioplayers.AudioplayersPlugin.handleMethodCall(AudioplayersPlugin.kt:62)
E/AudioplayersPlugin(10407): 	at xyz.luan.audioplayers.AudioplayersPlugin.onMethodCall(AudioplayersPlugin.kt:37)
E/AudioplayersPlugin(10407): 	at io.flutter.plugin.common.MethodChannel$IncomingMethodCallHandler.onMessage(MethodChannel.java:233)
E/AudioplayersPlugin(10407): 	at io.flutter.embedding.engine.dart.DartMessenger.handleMessageFromDart(DartMessenger.java:85)
E/AudioplayersPlugin(10407): 	at io.flutter.embedding.engine.FlutterJNI.handlePlatformMessage(FlutterJNI.java:738)
E/AudioplayersPlugin(10407): 	at android.os.MessageQueue.nativePollOnce(Native Method)
E/AudioplayersPlugin(10407): 	at android.os.MessageQueue.next(MessageQueue.java:335)
E/AudioplayersPlugin(10407): 	at android.os.Looper.loop(Looper.java:183)
E/AudioplayersPlugin(10407): 	at android.app.ActivityThread.main(ActivityThread.java:7656)
E/AudioplayersPlugin(10407): 	at java.lang.reflect.Method.invoke(Native Method)
E/AudioplayersPlugin(10407): 	at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:592)
E/AudioplayersPlugin(10407): 	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:947)
E/flutter (10407): [ERROR:flutter/lib/ui/ui_dart_state.cc(186)] Unhandled Exception: PlatformException(Unexpected error!, Operation is not supported for read-only collection, java.lang.UnsupportedOperationException: Operation is not supported for read-only collection
E/flutter (10407): 	at kotlin.collections.EmptyMap.put(Unknown Source:4)
E/flutter (10407): 	at java.util.Collections$SynchronizedMap.put(Collections.java:2639)
E/flutter (10407): 	at xyz.luan.audioplayers.WrappedSoundPool.setUrl(WrappedSoundPool.kt:247)
E/flutter (10407): 	at xyz.luan.audioplayers.AudioplayersPlugin.handleMethodCall(AudioplayersPlugin.kt:62)
E/flutter (10407): 	at xyz.luan.audioplayers.AudioplayersPlugin.onMethodCall(AudioplayersPlugin.kt:37)
E/flutter (10407): 	at io.flutter.plugin.common.MethodChannel$IncomingMethodCallHandler.onMessage(MethodChannel.java:233)
E/flutter (10407): 	at io.flutter.embedding.engine.dart.DartMessenger.handleMessageFromDart(DartMessenger.java:85)
E/flutter (10407): 	at io.flutter.embedding.engine.FlutterJNI.handlePlatformMessage(FlutterJNI.java:738)
E/flutter (10407): 	at android.os.MessageQueue.nativePollOnce(Native Method)
E/flutter (10407): 	at android.os.MessageQueue.next(MessageQueue.java:335)
E/flutter (10407): 	at android.os.Looper.loop(Looper.java:183)
E/flutter (10407): 	at android.app.ActivityThread.main(ActivityThread.java:7656)
E/flutter (10407): 	at java.lang.reflect.Method.invoke(Native Method)
E/flutter (10407): 	at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:592)
E/flutter (10407): 	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:947)
E/flutter (10407): , null)
E/flutter (10407): #0      StandardMethodCodec.decodeEnvelope
 package:flutter/…/services/message_codecs.dart
E/flutter (10407): #1      MethodChannel._invokeMethod
 package:flutter/…/services/platform_channel.dart
E/flutter (10407): <asynchronous suspension>
E/flutter (10407): #2      AudioPlayer._invokeMethod.<anonymous closure> (package:audioplayers/audioplayers.dart)
 package:audioplayers/audioplayers.dart
E/flutter (10407): <asynchronous suspension>
E/flutter (10407):

Same here, here is a bit longer log output.
E/AudioAttributes(10407): Invalid stream type -2147483648 for AudioAttributes
W/SoundPool::StreamManager(10407): StreamMap: requested 100 streams, clamping to 32
E/AudioplayersPlugin(10407): Unexpected error!
E/AudioplayersPlugin(10407): java.lang.UnsupportedOperationException: Operation is not supported for read-only collection
E/AudioplayersPlugin(10407): 	at kotlin.collections.EmptyMap.put(Unknown Source:4)
E/AudioplayersPlugin(10407): 	at java.util.Collections$SynchronizedMap.put(Collections.java:2639)
E/AudioplayersPlugin(10407): 	at xyz.luan.audioplayers.WrappedSoundPool.setUrl(WrappedSoundPool.kt:247)
E/AudioplayersPlugin(10407): 	at xyz.luan.audioplayers.AudioplayersPlugin.handleMethodCall(AudioplayersPlugin.kt:62)
E/AudioplayersPlugin(10407): 	at xyz.luan.audioplayers.AudioplayersPlugin.onMethodCall(AudioplayersPlugin.kt:37)
E/AudioplayersPlugin(10407): 	at io.flutter.plugin.common.MethodChannel$IncomingMethodCallHandler.onMessage(MethodChannel.java:233)
E/AudioplayersPlugin(10407): 	at io.flutter.embedding.engine.dart.DartMessenger.handleMessageFromDart(DartMessenger.java:85)
E/AudioplayersPlugin(10407): 	at io.flutter.embedding.engine.FlutterJNI.handlePlatformMessage(FlutterJNI.java:738)
E/AudioplayersPlugin(10407): 	at android.os.MessageQueue.nativePollOnce(Native Method)
E/AudioplayersPlugin(10407): 	at android.os.MessageQueue.next(MessageQueue.java:335)
E/AudioplayersPlugin(10407): 	at android.os.Looper.loop(Looper.java:183)
E/AudioplayersPlugin(10407): 	at android.app.ActivityThread.main(ActivityThread.java:7656)
E/AudioplayersPlugin(10407): 	at java.lang.reflect.Method.invoke(Native Method)
E/AudioplayersPlugin(10407): 	at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:592)
E/AudioplayersPlugin(10407): 	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:947)
E/flutter (10407): [ERROR:flutter/lib/ui/ui_dart_state.cc(186)] Unhandled Exception: PlatformException(Unexpected error!, Operation is not supported for read-only collection, java.lang.UnsupportedOperationException: Operation is not supported for read-only collection
E/flutter (10407): 	at kotlin.collections.EmptyMap.put(Unknown Source:4)
E/flutter (10407): 	at java.util.Collections$SynchronizedMap.put(Collections.java:2639)
E/flutter (10407): 	at xyz.luan.audioplayers.WrappedSoundPool.setUrl(WrappedSoundPool.kt:247)
E/flutter (10407): 	at xyz.luan.audioplayers.AudioplayersPlugin.handleMethodCall(AudioplayersPlugin.kt:62)
E/flutter (10407): 	at xyz.luan.audioplayers.AudioplayersPlugin.onMethodCall(AudioplayersPlugin.kt:37)
E/flutter (10407): 	at io.flutter.plugin.common.MethodChannel$IncomingMethodCallHandler.onMessage(MethodChannel.java:233)
E/flutter (10407): 	at io.flutter.embedding.engine.dart.DartMessenger.handleMessageFromDart(DartMessenger.java:85)
E/flutter (10407): 	at io.flutter.embedding.engine.FlutterJNI.handlePlatformMessage(FlutterJNI.java:738)
E/flutter (10407): 	at android.os.MessageQueue.nativePollOnce(Native Method)
E/flutter (10407): 	at android.os.MessageQueue.next(MessageQueue.java:335)
E/flutter (10407): 	at android.os.Looper.loop(Looper.java:183)
E/flutter (10407): 	at android.app.ActivityThread.main(ActivityThread.java:7656)
E/flutter (10407): 	at java.lang.reflect.Method.invoke(Native Method)
E/flutter (10407): 	at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:592)
E/flutter (10407): 	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:947)
E/flutter (10407): , null)
E/flutter (10407): #0      StandardMethodCodec.decodeEnvelope
 package:flutter/…/services/message_codecs.dart
E/flutter (10407): #1      MethodChannel._invokeMethod
 package:flutter/…/services/platform_channel.dart
E/flutter (10407): <asynchronous suspension>
E/flutter (10407): #2      AudioPlayer._invokeMethod.<anonymous closure> (package:audioplayers/audioplayers.dart)
 package:audioplayers/audioplayers.dart
E/flutter (10407): <asynchronous suspension>
E/flutter (10407):

"mode: PlayerMode.LOW_LATENCY" => error
"mode: PlayerMode.MEDIA_PLAYER" => none error

"mode: PlayerMode.LOW_LATENCY" => error
"mode: PlayerMode.MEDIA_PLAYER" => none error

"mode: PlayerMode.LOW_LATENCY" => error
"mode: PlayerMode.MEDIA_PLAYER" => none error

@minhtrids Thank you for the suggestion. I tried to switch to PlayerMode.MEDIA_PLAYER, the aforementioned crash has disappeared but another one popped up. The sound is still not working whilst everything works fine on iOS and even macOS builds.
E/MediaPlayerNative(11964): error (1, -2147483648)
E/MediaPlayer(11964): Error (1,-2147483648)

"mode: PlayerMode.LOW_LATENCY" => error
"mode: PlayerMode.MEDIA_PLAYER" => none error

@minhtrids Thank you for the suggestion. I tried to switch to PlayerMode.MEDIA_PLAYER, the aforementioned crash has disappeared but another one popped up. The sound is still not working whilst everything works fine on iOS and even macOS builds.
E/MediaPlayerNative(11964): error (1, -2147483648)
E/MediaPlayer(11964): Error (1,-2147483648)

(0.17.0 + mode: PlayerMode.LOW_LATENCY) => no more errors with AudioCache on Android
I can't try it on another plataform, but I hope it helps someone else.
Without low latency, I get an error after play an audio file 32 times.
E/MediaPlayerNative(17357): error (1, -19)
E/MediaPlayer(17357): Error (1,-19)
E/MediaPlayerNative(17357): pause called in state 0, mPlayer(0xb95e2390)
E/MediaPlayerNative(17357): error (-38, 0)
E/MediaPlayerNative(17357): Attempt to perform seekTo in wrong state: mPlayer=0xb95e2390, mCurrentState=0
E/MediaPlayerNative(17357): error (-38, 0)
E/MediaPlayer(17357): Error (-38,0)
E/MediaPlayer(17357): Error (-38,0)

Closing this due to reports of this being fixed on newer versions. Feel free to open a new issue in case of this still happening


Change to getClass() from instanceof

This prevents our breaking the symmetrical property of euqals through
extension.

@eljobe What's the status on this PR?

@bmuschko, I have asked twice on the build-tool slack channel for a reviewer, but nobody has volunteered. It is awaiting a reviewer. Are you interested?

@eljobe Happy to review. Likely won't happen until the Summit though.

@eljobe Can you provide more background information on your statement above? Do we observe incorrect behavior right now? If yes, how does the behavior manifest?

We do NOT observe incorrect behavior. For more background, you can read this summary:
https://groups.google.com/a/gradle.com/d/msg/build-tool-team/CyNegX5XL-I/er1WdxYvAwAJ
What I'm trying to do is prevent us from introducing a common bug when equals() is implemented with instanceof. Namely, someone will come along and subclass one of these classes, override the equals() method and instantly break the symmetrical property of the equals() contract.
So, there is no incorrect behavior now, but as a best practice, we should cleanup any equals() implementations which are using instanceof.  I'll add some EqualsVerifier tests for the classes which implement equals() and make sure they pass.

@bmuschko, I've added EqualsVerifer checks which actually helped catch some minor issues with the implementations of equals() on the types I was cleaning up. This is ready for further review.

@bmuschko ping!?

@eljobe I'll look at it today.


SOLPS-ITER impurity density plotting issue

Hi,
I'm encountering an issue with Aurora when trying to plot the density of nitrogen charge state (say, N⁺) from a SOLPS-ITER run using plot2d_b2:
import numpy as np
import matplotlib.pyplot as plt
import aurora

case = aurora.solps_case(
    b2fstate_path="./b2fstate",
    b2fgmtry_path="./b2fgmtry",
)

fig, axis = plt.subplots()

# B2 fluid species are D, D+, N, N+ -> plot the third species, i.e. N+
data = case.data("na")[3]
case.plot2d_b2(data, ax=axis, scale="log", lb = 1e13, ub = 1e19, label="Density [$m^{-3}$]")
plt.show(block=True)

I attached two figures, one which shows the issue where one row of cells is missing from the PFR and instead extends into the main plasma. The other shows how I would expect the density distribution to appear (from solpspy).
plotting_issue_with_aurora
reference_plot_from_solpspy
I also attached the code and the b2-files I used:
aurora_plotting_issue.zip
I'm using aurora(fusion) 2.2.0.
Br,
Roni

Ah, sorry to hear that there seems to be a bug... I thought that all those had been solved by now.
Unfortunately I don't have capacity to look into this now, but @Reksoatr may be able to have a look?

I'll take a look. Is this problem occurring only when plotting the density contour of an impurity, like nitrogen, or when plotting any 2d quantity?

The issue seems to appears only when I'm plotting a density from the case.data("na")-array, but  not when I plot e.g the electron density from case.data("ne") or the electron temperature from case.data("te").


[Linux, macOS] Improve nvm installer

Description
~New tool~, ~Bug fixing~, or Improvement?
Current nvm installer scripts, both on Linux and macOS, are requiring manual updates on every new nvm release. This change is to make it more automated, and to keep up with the latest releases of nvm. It also fixes a small redirect problem in macOS script. URL was pointing to the old repository, and now it's the correct one. I understand you don't accept any external contributions to macOS images, if you prefer, I can split the changes and create a PR for Linux, and an issue for macOS.
Related issue: N/A
Check list

[ ] Related issue / work item is attached
[ ] Tests are written (if applicable)
[ ] Documentation is updated (if applicable)
[ ] Changes are tested and related VM images are successfully generated

/azp run ubuntu1604, ubuntu1804, ubuntu2004

==> azure-arm: /tmp/script_1983.sh: line 36: session-manager-plugin: command not found

Not sure what's the problem here. I can see that it is in some script files to install from deb, and also via brew. But in the build logs, it's not available when it's being called.
Also not sure if I should try to fix it in this PR, would be nice to have some pointers.
Cheers

@gokaygurcan , Thank you for contribution. This change will be included to next image rollout (apx 1-2 weeks)

Wooho 🎉
Thank you for accepting it.


chore: rename SubstrateProcessor

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/camel-quarkus-pr/295/


Convert HeroCards to BlockKit

Fixes #5406
Big thanks to @martinlarosa for doing the initial work on this.
Description
Translates Hero Cards to Block Kit cards. Prior to this PR, all Hero Cards are converted to static images, so buttons don't work.
Potential Issues
This specifies a pretty specific format that matches Hero Cards closely, but won't be a 1:1.
Testing

@mdrichardson I just tested this change in my custom slack adapter and my slack messages still work great!


Added support for JWT auth

A simple change to enable JWT authentication in lieu of username/password.

Coverage decreased (-0.4%) to 89.461% when pulling 7cffd14072a3ca77361c7c1439769c86ce3149b4 on jwt-auth into d4f96e57a3a0f1ac3fd1525c8f785d3782813e82 on master.


Cache key expiration time

Hi, thanks for great work with Next-drupal!
I’m implementing Redis caching for our project and was wondering could it be possible to have support for setting cache key expiration time? Is the any cache related enhancements coming in the near future?

Hi.
We have started working on cache implementation. You can provide a custom cache via the cache option.
import { DrupalClient, DataCache } from "next-drupal"
import Redis from "ioredis"

const redis = new Redis(process.env.REDIS_URL)

export const redisCache: DataCache = {
  async set(key, value) {
    return await redis.set(key, value)
  },

  async get(key) {
    return await redis.get(key)
  },
}

export const drupal = new DrupalClient(
  process.env.NEXT_PUBLIC_DRUPAL_BASE_URL,
  {
    cache: redisCache,
  }
)

See also https://github.com/chapter-three/next-drupal/tree/main/examples/example-custom-cache
Can you elaborate the cache key expiration time please?

I believe I’m familiar with the code above. Now it seems for me that e.g. getMenu caches the menu only on build time. I would like to be able to cache menu also in run time because it can be that content editors change the menu. Using Redis cache expiration time would be enough to keep it updated frequently.
With ioredis it could be done like this:
// All arguments are passed directly to the redis server, // so technically ioredis supports all Redis commands. // The format is: redis[SOME_REDIS_COMMAND_IN_LOWERCASE](ARGUMENTS_ARE_JOINED_INTO_COMMAND_STRING) // so the following statement is equivalent to the CLI: redis> SET mykey hello EX 10 redis.set("mykey", "hello", "EX", 10);
If you have better ideas I'm more that happy to know!

I think it depends how you're handling the menus. If getMenu is called in getStaticProps, the only way to update the menu at runtime would be using on-demand ISR to revalidate the page.
This does not work as expected though because a menu expands several (if not all) pages on the site, so you end up having to revalidate everything.
The way we're handling this right now (for sites with menus that change all the time) is using client-side rendering for menu.

The page is built at build time (SSG)
On load, the menu component makes a request to /api/menus/[name].ts
/api/menus/[name].ts fetches and caches the menu tree on Redis.
We use useQuery from react-query to cache the menu in the browser to prevent hitting the API on route change.

This way the getMenu call is cached on Redis and we invalidate the cache only when it changes on Drupal.
An example api/menus/[name].ts would look like this:
export default async function handler(req, res) {
  const name = req.query.name

  // 1. Check redis for cached menu

  // 2. Return if found.

  // 3. Otherwise fetch the menu from Drupal.
  const { tree } = await drupal.getMenu(req.query.name)

  // 4. Cache it.

  // 5. Return.
}

Thanks for the info and ideas! This far I was just using ISR but now I started to think about implementing on-demand ISR. It is still experimental but will it be not experimental anymore in the future?

Next.js 12.2 announced stable revalidation API. We're working on the Drupal end now to make it stable in the next release.


preview posts links are linking to refresh

For some reason, the preview posts links on content.refresh are linking to refresh instead of content.refresh. View an example here: /policy-compliance/amicus/briefs/

High priority because this is related to the linking issues we need to get sorted out ASAP so we can effectively test our pages.

I'll handle the fix on this. Here is where the bug is. It's passing page to get the URL instead of post. Trivial.

This is fixed


[HUDI-4146][RFC-55] for Improve Hive/Meta sync class design and hierachies

Tips

Thank you very much for contributing to Apache Hudi.
Please review https://hudi.apache.org/contribute/how-to-contribute before opening a pull request.

What is the purpose of the pull request
(For example: This pull request adds quick-start document.)
Brief change log
(for example:)

Modify AnnotationLocation checkstyle rule in checkstyle.xml

Verify this pull request
(Please pick either of the following options)
This pull request is a trivial rework / code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end.
Added HoodieClientWriteTest to verify the change.
Manually verified the change by running a job locally.

Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Use spark, directly call spark.sharedState.externalCatalog api to sync metadata? avoid creating hms client

Use spark, directly call spark.sharedState.externalCatalog api to sync metadata? avoid creating hms client

this is not only used for spark, we need also to support Flink, KafkaConnect, and running the tools independently


What layout view is used in the example screenshots?

I dont understand which layout view is used to achieve a similar look to the examples.
If I use something like mason how do you get the separators to cross between the 2 stacks?
Thanks

Hi! This was indeed something that was missing in the documentation, you can now find it here:
https://github.com/Clooos/Bubble-Card#full-example

Ace thanks 😀


Google Closure compiler support is almost perfect except for this one line.

Hey, pthom.
Great work with hello imgui. I use it at work for developing user interfaces with scientific instrumentation. Recently I have been using it as a web interface. When i compile with emscripten it generates perfect code except that it is too big to fit on space constrained embedded devices and takes too long to load on slower computers and phones. When compiled the application code takes about 25 to 30 MB of space on the web. which is a lot even though it is acceptable. I have managed to reduce the size down to 2.6 MB which is amazing its at the level to compete with a proper web app built with React or a similar framework. Although its almost perfect i had to remove some things for the Google Closure compiler to work. like in the abstract runner file I removed support for getting the devices pixel ratio here:
float AbstractRunner::ImGuiDefaultFontGlobalScale()
{
    float fontSizeIncreaseFactor = 1.f;

#ifdef __EMSCRIPTEN__
    // Query the brower to ask for devicePixelRatio
    double windowDevicePixelRatio = EM_ASM_DOUBLE({
        // scale = window.devicePixelRatio;
        // return scale;
        return 1;
    });
    printf("window.devicePixelRatio=%lf\n", windowDevicePixelRatio);

    fontSizeIncreaseFactor = windowDevicePixelRatio;
#endif

...

What I think is happening is that the compiler needs to export this as a function to assembly to run in the browser but the closure compiler will optimize the definition of scale out to reduce memory consumption so the compiler throws a error saying scale isnt defined. or something along those lines. I will try and fix this.

ok I fixed it, I didnt read the documentation on EM_ASM_DOUBLE i was assuming it was c code but no its JS so the fix is adding the keyword var to define the scale variable.
I will make pull request.

It seems as I have an older version of hellow imgui and this has been moved to the fuction
void AbstractRunner::SetupDpiAwareParams()
...

Hello Abinash,
Thanks for contacting me!

When i compile with emscripten it generates perfect code except that it is too big to fit on space constrained embedded devices and takes too long to load on slower computers and phones. When compiled the application code takes about 25 to 30 MB of space on the web. which is a lot even though it is acceptable. I have managed to reduce the size down to 2.6 MB which is amazing

I'm very interested in this! Could you share more info about what you did to achieve this. If it works, this could be added to the FAQ.
One note however: I noticed that the build instructions for Emscripten did not include an advice to compile in release mode, which can lead to much smaller files. I updated this.
Would you mind to share how smaller the files become between a release build and when using the Google Closer compiler on this release build? If there is a sizeable reduction in the size of the files, I would be very interesting in having more information on how you obtained this result.

I will make pull request.

Thanks!

Yes of course i can show the difference let me cook up an example build tommorow for you.

On my side, I would be interested if it would be possible to reduce the size of the demo for Dear ImGui bundle (which is based on Hello ImGui).
At the moment, the sizes are as follow:
   1.8K Mar 12 09:31 build_ems_release/bin/demo_imgui_bundle.html
   348K Mar 12 09:31 build_ems_release/bin/demo_imgui_bundle.js
   5.4M Mar 12 09:31 build_ems_release/bin/demo_imgui_bundle.wasm
   9.8M Mar 12 09:31 build_ems_release/bin/demo_imgui_bundle.data

There are lots of assets, so the size of the data file is heavy (but this is normal, since it is a fully featured demo).

I am currently trying to compile imgui bundle with the closure compiler but it seems to be stuck at the imguial_term.cpp file. maybe you can compile on your end and see if you have different results.
I added this line to the very top of the cmake file after the project declaration.
add_link_options(-Os --closure 1)

it keeps terminating it self mostlikey due to running out of memory

Yes, it may happen at link time when running compilation in parallel, the link uses a lot of memory with emscripten.
Are you using make -j or ninja. If so, you could reduce the number of concurrent build with make -j 2 or even make -j 1
The build works ok with add_link_options(-Os --closure 1) on my side.

yes i was using make -j i will re do single core
What is the file size on your end?

I'm sorry, I answered to quickly add_link_options(-Os --closure 1)was not active when I compiled.
On my side, I cannot compile with those options; it fails for HelloImGui and ImGui Bundle with the message:
building:ERROR: Closure compiler run failed:

building:ERROR: /var/folders/w1/w77kvvl9613022ksfxj7v8xh0000gn/T/emscripten_temp_pzslygg1/hello_imgui_demodocking.jso1.js:1010:9: ERROR - [JSC_UNDEFINED_VARIABLE] variable allocate is undeclared
  1010|   return allocate(intArrayFromString(reply), "i8", ALLOC_NORMAL);
                 ^^^^^^^^

/var/folders/w1/w77kvvl9613022ksfxj7v8xh0000gn/T/emscripten_temp_pzslygg1/hello_imgui_demodocking.jso1.js:1010:51: ERROR - [JSC_UNDEFINED_VARIABLE] variable ALLOC_NORMAL is undeclared
  1010|   return allocate(intArrayFromString(reply), "i8", ALLOC_NORMAL);

I'm running the latest emscripten version, on macOS

So the compiler continues now that I only use 1 core but I have encountered another issue:
`building:ERROR: Closure compiler run failed:
building:ERROR: /tmp/emscripten_temp_kbh54yia/demo_imgui_bundle.js.pgrow.jso2.js:1942:9: ERROR - [JSC_UNDEFINED_VARIABLE] variable allocate is undeclared
1942|   return allocate(intArrayFromString(reply), "i8", ALLOC_NORMAL);
^^^^^^^^
/tmp/emscripten_temp_kbh54yia/demo_imgui_bundle.js.pgrow.jso2.js:1942:51: ERROR - [JSC_UNDEFINED_VARIABLE] variable ALLOC_NORMAL is undeclared
1942|   return allocate(intArrayFromString(reply), "i8", ALLOC_NORMAL);
^^^^^^^^^^^^
2 error(s), 0 warning(s)`
There still seem to be some JS code that is undefined, But i am not sure where this is

Oh ahah
I was about to say the same thing

These errors seem to be from one of the external modules which uses malloc most likely. I have not encountered this with implot so it cant bethat. are you aware of any external lib that using dynamic memory allocation?

At my work I only use hello imgui and implot with emscripten websockets. so I know for sure hello imgui on its own works

I do have the same error when building hello imgui. Strange...

hmm I am currently on a previous version of hello imgui btw commit a687d09 https://github.com/pthom/hello_imgui/commit/a687d090fb073d741c3f0f0b2ad211c21099ed86
This is the commit i am on currently

I have the same error with https://github.com/pthom/hello_imgui/commit/a687d090fb073d741c3f0f0b2ad211c21099ed86

ah
I think i see the problem by default memory growth is enabled i am compiling with it off in my project. Try it with that.  Here is a demo https://github.com/abinash18/imgui-reduced-size

It still fails on my side, when compiling Hello ImGui (master branch)
I added some more var keyword here
Now, I have this error:
 make hello_imgui_demodocking VERBOSE=1

 ...

✗ cd /Users/pascal/dvp/OpenSource/ImGuiWork/_Bundle/hello_imgui_vcpkg/build/emscripten/src/hello_imgui_demos/hello_imgui_demodocking && /opt/homebrew/Cellar/cmake/3.27.7/bin/cmake -E cmake_link_script CMakeFiles/hello_imgui_demodocking.dir/link.txt --verbose=1
/Users/pascal/emsdk/upstream/emscripten/em++ -O3 -DNDEBUG -Os --closure 1 --preload-file /Users/pascal/dvp/OpenSource/ImGuiWork/_Bundle/hello_imgui_vcpkg/build/emscripten/src/hello_imgui_demos/hello_imgui_demodocking/tmp/common_assets@/ --preload-file /Users/pascal/dvp/OpenSource/ImGuiWork/_Bundle/hello_imgui_vcpkg/src/hello_imgui_demos/hello_imgui_demodocking/assets@/ --shell-file /Users/pascal/dvp/OpenSource/ImGuiWork/_Bundle/hello_imgui_vcpkg/build/emscripten/src/hello_imgui_demos/hello_imgui_demodocking/tmp/shell.emscripten_hello_imgui_demodockin
g.html -sUSE_SDL=2 -sINITIAL_MEMORY=125829120 -s USE_SDL=2 -sMAX_WEBGL_VERSION=2 @CMakeFiles/hello_imgui_demodocking.dir/objects1.rsp -o ../../../bin/hello_imgui_demodocking.html @CMakeFiles/hello_imgui_demodocking.dir/linkLibs.rsp

/Users/pascal/emsdk/upstream/emscripten/em++ -O3 -DNDEBUG -Os --closure 1 --preload-file /Users/pascal/dvp/OpenSource/ImGuiWork/_Bundle/hello_imgui_vcpkg/build/emscripten/src/hello_imgui_demos/hello_imgui_demodocking/tmp/common_assets@/ --preload-file /Users/pascal/dvp/OpenSource/ImGuiWork/_Bundle/hello_imgui_vcpkg/src/hello_imgui_demos/hello_imgui_demodocking/assets@/ --shell-file /Users/pascal/dvp/OpenSource/ImGuiWork/_Bundle/hello_imgui_vcpkg/build/emscripten/src/hello_imgui_demos/hello_imgui_demodocking/tmp/shell.emscripten_hello_imgui_demodocking.html -sUSE_SDL=2 -sINITIAL_MEMORY=125829120 -s USE_SDL=2 -sMAX_WEBGL_VERSION=2 @CMakeFiles/hello_imgui_demodocking.dir/objects1.rsp -o ../../../bin/hello_imgui_demodocking.html @CMakeFiles/hello_imgui_demodocking.dir/linkLibs.rsp
building:ERROR: Closure compiler run failed:

building:ERROR: /var/folders/w1/w77kvvl9613022ksfxj7v8xh0000gn/T/emscripten_temp_zigp8hz9/hello_imgui_demodocking.jso4.js:30:2837: ERROR - [JSC_UNDEFINED_VARIABLE] variable allocate is undeclared

/var/folders/w1/w77kvvl9613022ksfxj7v8xh0000gn/T/emscripten_temp_zigp8hz9/hello_imgui_demodocking.jso4.js:30:2877: ERROR - [JSC_UNDEFINED_VARIABLE] variable ALLOC_NORMAL is undeclared

...
324687:$0=>{var str=UTF8ToString($0)+"\n\n"+"Abort/Retry/Ignore/AlwaysIgnore? [ariA] :";var reply=window.prompt(str,"i");if(reply===null){reply="i"}return allocate(intArrayFromString(reply),"i8",ALLOC_NORMAL)}

And the code related to this error seems to come from SDL_assert.c

aha here is something useful to me: https://web.dev/articles/emscripten-embedding-js-snippets

I might have figured out why i could compile and you could not,

Here is my CMake if i remove -flto -fno-rtti I get the same errors as you.

Hi,
Thanks for the info. Based on my tests, only the link option -flto is required.
Here are the size gains based on my tests:
Standard emscripten release compil:
5.0M bin_std/hello_imgui_demodocking.data
266K bin_std/hello_imgui_demodocking.js
2.2M bin_std/hello_imgui_demodocking.wasm

Adding -Os (optimize for size) as compile and link options leads to:
5.0M bin_os/hello_imgui_demodocking.data
265K bin_os/hello_imgui_demodocking.js
1.8M bin_os/hello_imgui_demodocking.wasm

Adding --closure 1 and. -fltoas link options leads to:
5.0M bin_lto/hello_imgui_demodocking.data
126K bin_lto/hello_imgui_demodocking.js
1.8M bin_lto/hello_imgui_demodocking.wasm


Renamed some fields in Grant schema

Renamed the following:

funder_id to funder
principal_investigator_id to principal_investigator
awardee_organization_id to awardee_organization

Thanks! Sorry I forgot to look at this earlier.


German date formatting corrupt

When using German locale, created_at and updated_at come across in corrupt format, e.g.:
t 04, 2016 17:30

Platform:

rails_admin 49aafbb
rails_admin-i18n 1.10.2
rails 4.2.6

@ain solved meanwhile?
if not, time.formats.long is the key if i am correct. what does
I18n.locale = :de
DateTime.now.in_time_zone.strftime(I18n.t('time.formats.long'))

give you in return?

Closed by https://github.com/sferik/rails_admin/commit/01e8d5fc8ec94e68af6fdbd80759a751cd83f74a .


when I try to clone the repo I get an error

error: unable to create symlink .github/LICENSE.md: File name too long
error: unable to create symlink .github/README.md: File name too long
any steps on how to get around this?
edit: I was able to get it to clone with these settings
git config --global core.symlinks false
git config --global core.longpaths true

hey,

I was able to get it to clone with these settings

yeah this was the solution I used
However, I've removed the symlinks now and have a script which manually syncs the github files, hopefully this solves the issue https://github.com/eliasm307/storybook-addon-deep-controls/commit/3e53116b0c04f2a83c6746fcb8a9889bb36d5fc3
I'll close this issue but let me know if you still have problems


update readme for asset support

closes https://github.com/markdalgleish/static-site-generator-webpack-plugin/issues/32 and  https://github.com/markdalgleish/static-site-generator-webpack-plugin/issues/70

Coverage remained the same at 100.0% when pulling 3206cad6e5009d136856185d7c239eb742c37159 on thewillhuang:master into 35c9aa29e5c225ae7bec4d8430cb120553933d8c on markdalgleish:master.

:+1:


ImportError: No module named dface.core.utils

how to do so?

Follow the Readme.md file. After git cloning the repository, you need to run "export PYTHONPATH=$PYTHONPATH:{your local DFace root path}" in your terminal.


[WebUI] Auto-minimize after key injection into Balabolka

Without this feature, Balabolka goes to the foreground and the WebUI becomes hard to reach.

The current thinking is that we will not focus onto Balabolka after key injection, because the user might want to type more in the WebUI. If the user wants to go to Balabolka, they can gaze-click Balabolka to go there. We may also provide a button for the user to go directly to Balabolka.

So this is "wontfix"


tram not loading to needs review - stuck in queue

After entering a URL and title and clicking submit nothing else happens.
All jobs remain in the queue.
PS C:\tram> py.exe .\tram.py
C:\tram\tram.py:22: DeprecationWarning: "@coroutine" decorator is deprecated since Python 3.8, use "async def" instead
async def background_tasks(taxii_local='online', build=False, json_file=None):
C:\tram\tram.py:46: DeprecationWarning: "@coroutine" decorator is deprecated since Python 3.8, use "async def" instead
async def init(host, port):
INFO:root:Welcome to TRAM
DEBUG:asyncio:Using proactor: IocpProactor
INFO:root:[] Found punkt
INFO:root:[] Found stopwords
INFO:root:server starting: <IP_ADDRESS>:9999
INFO:aiohttp.access:<IP_ADDRESS> [07/Apr/2021:06:44:08 +0000] "GET / HTTP/1.1" 200 12053 "http://localhost:9999/?" "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/89.0.4389.114 Safari/537.36"
INFO:aiohttp.access:<IP_ADDRESS> [07/Apr/2021:06:44:09 +0000] "GET /theme/images/favicon-blue-m.ico HTTP/1.1" 200 215 "http://localhost:9999/?" "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/89.0.4389.114 Safari/537.36"

As an update on this, I debugged the code to see what's happening. I experienced errors in the ML service class that weren't appearing in the console logs (I would only get them when I debugged in an IDE: I put a breakpoint on a line, when the breakpoint was hit I ran the line as a watch-expression and then saw the errors). The missing printed errors here are most likely an asyncio/background tasks thing.
I haven't fixed it for me yet (but if you're curious, you can see what kind of fixes I had to make so far here Chockity-Pok/tram@e22de62). My success with that commit is that in rest_svc.py, I'm able to finish this line which I was initially unable to do. I'm going to continue debugging the rest of the process in which an article moves out of the queue.
It sounds like from #67, @timoliciouz got to analyse a few reports. It could be an environment thing where versions of dependencies need to be specific ones. Still, some changes I made in my referenced commit fix positional-argument errors, missing 'awaits', KeyErrors and IndexErrors which would not be affected by dependencies.

As an update on this, I debugged the code to see what's happening. I experienced errors in the ML service class that weren't appearing in the console logs (I would only get them when I debugged in an IDE: I put a breakpoint on a line, when the breakpoint was hit I ran the line as a watch-expression and then saw the errors). The missing printed errors here are most likely an asyncio/background tasks thing.
I haven't fixed it for me yet but I came across positional-argument errors, missing 'awaits', KeyErrors and IndexErrors in ml_svc.py (the ML service class). It started with pickle.load(<models-pickle-file>)  raising a ModuleNotFoundError where an sklearn module was missing: it must be me having different dependency versions. My fixes so far is that in rest_svc.py, I'm able to finish this line which I was initially unable to do. I'm going to continue debugging the rest of the process in which an article moves out of the queue.
It sounds like from #67, @timoliciouz got to analyse a few reports. This adds to my theory it's an environment thing where versions of dependencies need to be specific ones.

I haven't tested it but it looks like #52 is a fix :) My changes are in #61 if it helps too


specs: two reserved bytes

Closes https://github.com/celestiaorg/celestia-app/issues/883

Thanks for reviewing Matt! Will merge by end of next week to give @adlerjohn time to review.

Will merge tomorrow if @adlerjohn has no more feedback


Update zoom related feature documentation in tabs

tabs.getZoomSettings() returns:
mode: "automatic"
scope: !privacy.resistFingerprinting && browser.zoom.siteSpecific
defaultZoomFactor: 1 - This is not the correct behaviour. It should return the (global default zoom level)/100.

tabs.onZoomChange() returns:
mode: "automatic"
scope: !privacy.resistFingerprinting && browser.zoom.siteSpecific
defaultZoomFactor: 1 - This is not the correct behaviour. It should return the (global default zoom level)/100.

"per-tab" description:
The existing wording appears to be copied from Chrome.
The sentence "Also, per-tab zoom changes are reset on navigation; navigating a tab will always load pages with their per-origin zoom factors." is not true for Firefox. The zoom level persists across page loads and navigation.

The "per-tab" description has been updated on tabs.ZoomSettingsScope and noted on Chrome incompatibilities.
After clarification the comments on tabs.getZoomSettings() and tabs.onZoomChange() relate to "defaultZoomFactor: "1" is always returned. This is no longer the correct behaviour, because Bug 1590485 (fixed in Firefox 73.0) now allows the user to change the Default Zoom level in Tools > Options." which is already consistent with the documentation.


TimeOut Error

Hi Everbody;
I am trying to read tags from Allen Bradley Micro 850 PLC with node-red-contrib-cip-ethernet-ip node
but I am taking this error : Error connecting to PLC: Error: TIMEOUT occurred while reading Controller Props. How Can I solve this problem.

Thats interesting.  Can you post a snippet of your code?

This package doesn’t support Micro800 processors yet, does it?

see issue #4, you are correct. It seems like once connected messaging is released as a first class feature in the ts implementation, we will be capable of supporting it but we cant as of this moment.  It is definitely something to keep on our radar though. Sorry @Ramo38


Add itemlist

Codecov Report
Base: 72.15% // Head: 0.00% // Decreases project coverage by -72.15% :warning:

Coverage data is based on head (e951082) compared to base (38e6810).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #35       +/-   ##
============================================
- Coverage     72.15%   0.00%   -72.16%
============================================
  Files            70      77        +7
  Lines          1232    1292       +60
  Branches        125     121        -4
============================================
- Hits            889       0      -889
- Misses          311    1292      +981
+ Partials         32       0       -32

Impacted Files
Coverage Δ

src/main/java/seedu/waddle/AppParameters.java
0.00% <ø> (ø)

src/main/java/seedu/waddle/Main.java
0.00% <ø> (ø)

src/main/java/seedu/waddle/MainApp.java
0.00% <0.00%> (ø)

...rc/main/java/seedu/waddle/commons/core/Config.java
0.00% <ø> (ø)

...in/java/seedu/waddle/commons/core/GuiSettings.java
0.00% <ø> (ø)

...ain/java/seedu/waddle/commons/core/LogsCenter.java
0.00% <ø> (ø)

.../main/java/seedu/waddle/commons/core/Messages.java
0.00% <0.00%> (ø)

...c/main/java/seedu/waddle/commons/core/Version.java
0.00% <ø> (ø)

...in/java/seedu/waddle/commons/core/index/Index.java
0.00% <ø> (ø)

...le/commons/exceptions/DataConversionException.java
0.00% <ø> (ø)

... and 105 more

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

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


erro on add with select dropdown relashionship

Laravel Version:5.4

Description:
I have a select dropdown with a problem
my relashionship is
{"relationship":
{"key":"id",
"label":"nome",
"page_slug": "admin/tipo-servicos"}
}
my \APP\TipoServico
public function tipo_sub_servicos_id()
{
	return $this->hasMany(\App\TipoSubServico::class);
}

my \APP\TipoSubServico
public function tipo_servicos_id()
{
	return $this->belongsTo(\App\TipoServico::class,'tipo_servicos_id','id');
}

and shows this error on add a new
Undefined property: App\TipoSubServico::$tipo_servicos_id (View: C:\xampp\htdocs\voyager\vendor\tcg\voyager\resources\views\formfields\select_dropdown.blade.php) (View: C:\xampp\htdocs\voyager\vendor\tcg\voyager\resources\views\formfields\select_dropdown.blade.php)

After https://github.com/the-control-group/voyager/pull/638 they changed from snake_case to  camelCase
Hope it helps

i had same problem, and i have changed id to int and any_id to varchar it worked in my case

Labels: possible bug
Close (solution posted)


Inconsistent list-item rendering accross platforms

What version of Hugo are you using?

Hugo Static Site Generator v0.61.0/extended windows/amd64 BuildDate: unknown

Does this issue reproduce with the latest release?
Yes

With Hugo/Linux on remote (and Goldmark),

*

is interpreted as a list-item:

Whereas with Hugo/Windows locally it is displaying as is (like it was with Blackfriday):

*

I think the correct behavior is the Windows/local one, according to the CommonMark Spec:

... and M is a list marker of width W followed by 1 ≤ N ≤ 4 spaces...

It seems to me this is a bug/inconsistency, but I am not sure if it is Hugo or Goldmark related, any insight?
Note: here in GitHub .md files preview, the behavior seems to be the Linux/remote one.
(my repo where this happens)

Keep ?


Support data sources containing cloud-optimized geotiffs

Some data sources may point to stores of "Cloud-Optimized" GeoTIFFs that support range requests. Reads of arbitrary portions of these data sources are possible.
For these data sources, it would be ideal to avoid downloading the entire GeoTIFF. Instead, the user should be able to specify that only needed portions of the GeoTIFF should be read. However, this is different from our current ingestion process, where we read entire items; we could represent each block of the GeoTIFF as a different item, but this is far from ideal.
Changes:

Implement a specialized tile store for cloud-optimized GeoTIFFs. The tile store says that the item is already present if it is present in the data source. The tile store reads and re-projects the data on the fly when requested.
Adjust tile store API so that tile store is responsible for keeping track of the different projections. Currently, the projection is simply encoded in a layer, and the tile store is not actually aware of it.

I assigned myself to start the work on this for XyzTiles, but right now to add COG I'm not sure there's really a good use case, most of the data out there is not actually COG.

Planetary Computer datasets might be a good use case if we start using that.


gBZZ transfer from 0.5.3 node cheque book to 0.6.x node cheque book

we found a tx that one 0.5.3 node transfer gBZZ to one 0.6.x node.
this is the cheque transfer tx
https://goerli.etherscan.io/tx/0x9f304a2bdfcdbf9a7b060d4cdd2afcdf56bc1b80b8c2f7ac22524ceecb5b047f
this is the sender cheque book
https://goerli.etherscan.io/token/0x2ac3c1d3e24b45c6c310534bc2dd84b5ed576335?a=0xac419be76edf25ff046ff6661151c99d471b0228
deployed in 0.5.3 contract https://goerli.etherscan.io/tx/0x243853bdb01d65a564d3d0e352a6e765626789fcc27f96a0e2e5258663c861d0
this is the receiver cheque book
https://goerli.etherscan.io/token/0x2ac3c1d3e24b45c6c310534bc2dd84b5ed576335?a=0xcd1605b580384659acf1e2911f0db1df46addc69
deployed in 0.6.x contract https://goerli.etherscan.io/tx/0xd5649c629c8d9d0e9b07f5eaa633e22453f1d83c13e7c32c7806ba68ca518a77
but the sender is a 0.5.3 node
and the receiver is a 0.6.x node
why will the transfer happened?

Hi there,
Thanks for trying out Bee. Support questions please to our Discord (referenced on our website).
Best,

this is a bug that some node can hijack to get illegal 0.6.x cheque


[Scenario Explorer] Filter also downloaded scenarios

I do not know whether it is useful, but it seems convenient for me to have a filter on downloaded scenarios.
Originally posted by @Ssid113 in https://github.com/NPBruce/valkyrie/issues/1012#issuecomment-458535952

I agree too

Would you agree that this one should be for 2.3 ?
Or do you think this is a must have in 2.2 ?

It isn't our call, I think. It'd be good to have it ASAP but if I remember correctly, you'd need to make some changes regarding the offline scenario list. So if you think you won't manage to do that know, it'll wait until 2.3.

It's still good to know what you guys think about it.
This one would not change anything on offline mode (all scenarios are available in offline)
You have to consider that we need all translations for this too.
Text could be "Only show downloaded scenarios [V]"

PL: Pokaż wyłącznie pobrane scenariusze [V]

GERMAN: "Nur heruntergeladene Szenarien anzeigen [V]"

RUS: Показывать только загруженные сценарии [V]

Question: is this filter now going to show:

only scenario's that are downloaded? And if Yes, does that include scenario's that have an update available?
only scenarios that are available for download? And if Yes, does that include scenario's that have an update available?
can do 1 & 2?


feat(MarkerManager): set an opacity to marker.

@TSHiYK Looking good! 👍

@TSHiYK thanks! can you change the commit message from fix to feat? Than we can merge it! good job.

@SebastianM Thanks for reply. I fixed the commit message.

Sorry, found one more little thing. When the default is 1, we can merge it.

@SebastianM Done!

Any way to animate this opacity other than setInterval?


set_id method do not change anchor id on elementos node

This bug shows when creating other node with same id of another node that was created before
To fix just in line 3673 on set_id method put
objetivo.a_attr.id = id

Duplicate #2238


MEM53-CPP:  False positive due to flow through realloc

Affected rules

MEM53-CPP

Description
In https://github.com/github/codeql/pull/14637 we added taint-flow through the indirection of the pointer passed to realloc to the indirection of the result. That is, flow through the following example:
int* p = ...;
*p = tainted_value;
int* q = (int*)realloc(p, 1024);
sink(*p);

this relies on the new taint-tracking library to distinguish between the result of realloc(...), and the result of what realloc(...) points to. Since the old AST-based taint-tracking library cannot do this this results in a FP in the testcases for MEM53-CPP (that we accepted on the next branch here: https://github.com/github/codeql-coding-standards/pull/419)
The query already tries to rule out realloc cases by excluding them in the definition of the taint-tracking configuration's isSource, but to get this query back to not reporting a FP here a barrier on realloc would have to be inserted.
As @jketema points out the affected test is actually really sketchy since there’s no guarantee that memory allocated with new can safely be realloc'ed. So maybe this scenario should be thought about more carefully by someone on your team.

Thanks for the report! I recently merged this change from next into main, and modified the query to apply realloc as a barrier, which makes the behaviour consistent with how the query worked before https://github.com/github/codeql/pull/14637.
I believe the current test case for realloc should use malloc for the allocation initially, before using placement new. Then the test case wouldn't trigger undefined behaviour.
In addition, we should consider whether we want to cover any of the realloc cases under this rule. I think a common cases would be an array of objects that is being expanded. Do we then expect to see a constructor call for each new instance added in this case, and flag otherwise?


Enable write core feature for compute pr2

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.ResourceManager.Compute


NOOB : Eve App

OK, I'm a bit new so maybe a basic question but...
I have two sets of Permanent Outdoor Lights - at the moment
2 x H705A and 2 x H705B
I added the devices to Homekit and I see them...

For me, none of the color selector work - but I can turn the lights on an off...
Further, what is the "EVE" app?
{EDIT] Found it: https://homekitnews.com/2020/04/13/creating-an-automation-in-the-eve-app-vs-the-home-app/

Hi @coryabooth
I think first we get the basics sorted, with the home app working first, can you do the following, for each model you have:

use the govee app to turn the brightness to 100%
check the home app to see which brightness % it reports?

Regarding the brightness...
If I set the brightness to 26% on the App.
[2/12/2023, 10:23:55 AM] [Govee] [Lights - North] current state [on].
[2/12/2023, 10:23:55 AM] [Govee] [Lights - North] current scene [33050aff000000000000000000000000000000c3] [op:aa].
[2/12/2023, 10:23:55 AM] [Govee] [Lights - North] new scene code: [qkAAbBsAAAAAAAAAAAAAAAAAAJ0=] [aa40006c1b00000000000000000000000000009d] [op:aa].
[2/12/2023, 10:24:06 AM] [Govee] [Lights - North] [AWS] receiving update {"source":"AWS","brightness":26,"result":1,"proType":2,"sku":"H705A","device":"7E:E3:CB:38:32:31:56:1A","softVersion":"1.00.10","wifiSoftVersion":"2.00.20","cmd":"brightness","type":0,"transaction":"a_1676219049750","pactType":1,"pactCode":1,"state":{"brightness":26,"result":1}}.
[2/12/2023, 10:24:06 AM] [Govee] [Lights - North] current brightness [10%].
If I set the brightness back too 100%
[2/12/2023, 10:24:42 AM] [Govee] [Lights - North] new scene code: [qkAAbBsAAAAAAAAAAAAAAAAAAJ0=] [aa40006c1b00000000000000000000000000009d] [op:aa].
[2/12/2023, 10:25:25 AM] [Govee] [Lights - North] [AWS] receiving update {"source":"AWS","brightness":100,"result":1,"proType":2,"sku":"H705A","device":"7E:E3:CB:38:32:31:56:1A","softVersion":"1.00.10","wifiSoftVersion":"2.00.20","cmd":"brightness","type":0,"transaction":"v_1676219129288559","pactType":1,"pactCode":1,"state":{"brightness":100,"result":1}}.
[2/12/2023, 10:25:25 AM] [Govee] [Lights - North] current brightness [39%].
[2/12/2023, 10:25:42 AM] [Govee] [Lights - North] [AWS] receiving update {"source":"AWS","onOff":1,"brightness":100,"color":{"r":255,"g":255,"b":0},"colorTemInKelvin":0,"mode":10,"result":1,"op":{"command":["qgUK/wAAAAAAAAAAAAAAAAAAAFo=","qhEAHg8PAAAAAAAAAAAAAAAAAKU=","qhL/ZAAAgAoAAAAAAAAAAAAAAKk=","qiP/AAAAgAAAAIAAAACAAAAAgHY=","qkAAbBsAAAAAAAAAAAAAAAAAAJ0="]},"proType":2,"sku":"H705A","device":"7E:E3:CB:38:32:31:56:1A","softVersion":"1.00.10","wifiSoftVersion":"2.00.20","cmd":"status","type":0,"transaction":"y_1676219146175156","pactType":1,"pactCode":1,"state":{"onOff":1,"brightness":100,"color":{"r":255,"g":255,"b":0},"colorTemInKelvin":0,"mode":10,"result":1}}.


build failing in local environment because of version locking

Issue type
Other
Problem description
After the addition of gradle version locks in code repository by some other developer in my organization, now the build is failing in my machine:
/c/git/nextgenRepository/myProject (dev)
$ ../gradlew assemble
...
> Task :myProject:compileJava FAILED
:myProject:compileJava took 30ms
Task timings:

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':myProject:compileJava'.
> Could not resolve all files for configuration ':myProject:compileClasspath'.
   > Did not resolve 'com.google.guava:guava-parent:32.1.2-jre' which is part of the dependency lock state
BUILD FAILED in 1s
23 actionable tasks: 1 executed, 22 up-to-date

When I compared dependencies of myProject from my machine (which has issue) and that from another developer's machine (where it's working fine), found the following difference:

working.txt is where the build (../gradlew assemble) is passing. "com.google.guava:guava:11.0.2" is fetching guava-parent.
issue.txt is where the build (../gradlew assemble) is failing with the aforementioned error. The same version of the library "com.google.guava:guava:11.0.2" isn't fetching guava-parent.

When we check the dependencies on maven: https://mvnrepository.com/artifact/com.google.guava/guava/32.1.2-jre : indeed, it doesn't have dependencies on "guava-parent". Details on maven site are consistent with the dependencies in my machine.

Dependencies listed in https://repo1.maven.org/maven2/com/google/guava/guava/32.1.2-jre/guava-32.1.2-jre.pom :

Then how come it's different in everyone else's machine? It's working fine in everyone else's machine. Caching somewhere? How to resolve this issue in my machine?
Context (optional)
Builds are failing in my local. As a workaround, I've disabled the version locking in my machine with the following statement in build.gradle:
lockMode = LockMode.LENIENT

$ ./gradlew --version

------------------------------------------------------------
Gradle 6.9.2
------------------------------------------------------------

Build time:   2021-12-21 20:18:38 UTC
Revision:     5d94aa68c0fdbe443838bb977080e3b9f273e889

Kotlin:       1.4.20
Groovy:       2.5.12
Ant:          Apache Ant(TM) version 1.10.9 compiled on September 27 2020
JVM:          <IP_ADDRESS> (Amazon.com Inc. <IP_ADDRESS>+9-LTS)
OS:           Windows 11 10.0 amd64

Sorry that you're having trouble with Gradle!
Your issue appears to be a question about how to use Gradle. Your question might be answered in Gradle documentation or on our community Slack or forums. Find links to these resources on help.gradle.org.
If you think you have come across a Gradle bug, please file a new issue describing the problem and how to reproduce it.
If you think Gradle is missing a feature, please file a new issue describing your use case for the feature.

Alright, I've created a post on forum: https://discuss.gradle.org/t/build-failing-in-local-environment-because-of-version-locking/47173 .