lodash named exports fails to import with "type": "module" in package.json

Problem
With "type": "module" in the package.json, and and named import of a function from lodash (Pure CJS, not statically analyzable by Node.js). You will get an error:
> start
> tsx index.ts

(node:10) ExperimentalWarning: Custom ESM Loaders is an experimental feature and might change at any time
(Use `node --trace-warnings ...` to show where the warning was created)
file:///home/projects/node-bixnfe/index.ts:1
var __defProp=Object.defineProperty;var __name=(target,value)=>__defProp(target,"name",{value,configurable:true});import{pad}from"lodash";function sayHello(name){console.log(pad(`Hello ${name}`,80))}__name(sayHello,"sayHello");sayHello("World");
                                                                                ^^^
SyntaxError: The requested module 'lodash' does not provide an export named 'pad'
    at ModuleJob._instantiate (https://nodebixnfe-doob.w-credentialless.staticblitz.com/blitz.09144545.js:181:1509)
    at async ModuleJob.run (https://nodebixnfe-doob.w-credentialless.staticblitz.com/blitz.09144545.js:181:2327)

Although there is also lodash-es, you still encounter packages importing lodash directly from within them, where replacing them all with lodash-es is difficult and sometimes causes issues, as those are sometimes meant to be consumed from vanilla Node.js CJS mode, which can't require ESM modules. And merging lodash into a proper hybrid package isn't happening anytime soon due to the poor maintenance status of lodash (https://github.com/lodash/lodash/issues/5107).

Expected behavior
It just works.
Minimal reproduction URL
https://stackblitz.com/edit/node-bixnfe?file=index.ts
Version
v3.12.1
Node.js version
v18.18.0
Package manager
npm
Operating system
Linux
Contributions

[ ] I plan to open a pull request for this issue
[ ] I plan to make a financial contribution to this project

Strangely, esbuild seems to work fine in this case. Ie I can do
esbuild input.ts --bundle --format=cjs --platform=node --outfile=output.cjs
node output.cjs

And it works fine. Any idea what tsx is doing differently such that we run into this issue?

Strangely, esbuild seems to work fine in this case. Ie I can do
esbuild input.ts --bundle --format=cjs --platform=node --outfile=output.cjs
node output.cjs

And it works fine. Any idea what tsx is doing differently such that we run into this issue?

Sorry, any idea why the above comment was marked as spam?

I hid it because Issues is for maintainers to triage and investigate bugs, and I prefer to keep the discussion productive towards finding a solution.
Your comment didn't particularly add anything. tsx is a different tool and with a different goal from esbuild, so I didn't see the relevance. And asking "why is this happening?" on an issue felt redundant as answering that is kind of the point of an issue.
Next time, please don't unhide a hidden comment by re-posting it. The point was to keep the thread clean.

Apologies—I was so surprised that you would mark my comment as spam that I thought maybe I had done it myself by accident 😅
Fwiw, this issue strikes me as a duplicate of https://github.com/esbuild-kit/tsx/issues/38
Best of luck with your project

Duplicate of https://github.com/privatenumber/tsx/issues/38


Change min sdk support to API - 14 (4.0)

@nasthagiri @rohan-dhamal-clarice - We had previously changed minSdk version to 16 (4.1) for exo player integration.
But as exo player is not getting integrated we will revert back to supporting minSdk version to 14 (4.0).
Please review.

:+1:

:+1:


cleanup: enhance docs ahead of full publication

Thanks for the review @dkolas! I'm gonna go ahead and merge. Once the site is fully up, we can take a fuller editing pass.


Add More Tests to azure-core-test

Description
Adds more testing to azure-core-test.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-core-test


The allowable number of md-sidenav

Hi! Thank you for your work! The question is... sidenav 2 work fine... But when you add 3 they are all hidden. There are no errors. Version of angular 2, and the latest material. Big Thanks!

example https://plnkr.co/edit/1cM80qRTOKBpNkGAOjTi?p=preview

From the documentation for sidenav

A sidenav container may contain one or two  elements. When there are two  elements, each must be placed on a different side of the container. See the section on positioning below.

Is there a way to do example 2 on the left? Tried 1 sidenav-container, all is well. But when closed, it overlaps another container, although it is hidden. Thank you for your response.

Just put *ngIf inside the left sidenav and show content depending on what you are opening.

By design, we only allow one sidenav per side, but it's a bad experience to fail silently. We should be throwing an error.

Error thrown in #3369


Angular tabs part 2 - The revenge of the tabs

Closes #2338
Follows from #2266
Adds actions and tidies up a bunch of the controller API.

Things to look for during review

[x] Make sure all references to iot or bx class prefix is using the prefix variable
[ ] (React) All major areas have a data-testid attribute. New test ids should have test written to ensure they are not changed or removed.
[ ] UI should be checked in RTL mode to ensure the proper handling of layout and text.
[ ] All strings should be translatable.
[ ] The code should pass a11y scans (The storybook a11y knob should show no violations). New components should have a11y test written.
[ ] Unit test should be written and should have a coverage of 90% or higher in all areas.
[ ] All components should be passing visual regression test. For new styles or components either a visual regression test should be written for all permutations or the base image updated.
[x] Changes or new components should either write new or update existing documentation.
[x] PR should link and close out an existing issue

The best PR name to date. Been waiting for the sequel


ERROR TWO DAYS AGO

It hasn't connected to the system for two days and it gives these errors.
Este error se originó a partir de una integración personalizada.
Logger: custom_components.mypyllant
Source: custom_components/mypyllant/init.py:179
Integration: myVAILLANT (documentation, issues)
First occurred: 12:12:06 (1 occurrences)
Last logged: 12:12:06
Unexpected error fetching myVAILLANT data: 1 validation error for Zone heating_state value is not a valid enumeration member; permitted: 'IDLE', 'HEATING_UP' (type=type_error.enum; enum_values=[<ZoneHeatingState.IDLE: 'IDLE'>, <ZoneHeatingState.HEATING_UP: 'HEATING_UP'>])
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 283, in _async_refresh
self.data = await self._async_update_data()
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/mypyllant/init.py", line 179, in _async_update_data
async for system in await self.hass.async_add_executor_job(
File "/usr/local/lib/python3.11/site-packages/myPyllant/api.py", line 227, in get_systems
system = System(
^^^^^^^
File "/usr/local/lib/python3.11/site-packages/myPyllant/models.py", line 123, in init
self.zones = [Zone(system_id=self.id, **z) for z in self._raw_zones]
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/myPyllant/models.py", line 123, in
self.zones = [Zone(system_id=self.id, **z) for z in self._raw_zones]
^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "pydantic/main.py", line 341, in pydantic.main.BaseModel.init
pydantic.error_wrappers.ValidationError: 1 validation error for Zone
heating_state
value is not a valid enumeration member; permitted: 'IDLE', 'HEATING_UP' (type=type_error.enum; enum_values=[<ZoneHeatingState.IDLE: 'IDLE'>, <ZoneHeatingState.HEATING_UP: 'HEATING_UP'>])

I have detected that the error occurs when activating the cooling floor, not the underfloor heating.
I think that some variable is not declared to show the refreshing soil

Thanks for the report, the latest release should fix this

Thanks!
El vie, 21 jul 2023, 23:46, signalkraft @.***> escribió:

Thanks for the report, the latest release should fix this
—
Reply to this email directly, view it on GitHub
https://github.com/signalkraft/myPyllant/issues/10#issuecomment-1646268472,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYY5SLSWNTPKNFBMMXV7ZPLXRL2CBANCNFSM6AAAAAAZKHDWPY
.
You are receiving this because you authored the thread.Message ID:
@.***>


Where can I found this for IOS?

Where can I found this for IOS?

:smile: I don't think someone developped it yet


Use Cabal API for determining package-id of submodules

In the early days we savagely pulled the package-id out of cabal files using a combination of Data.Text hacks in order to avoid pulling in Cabal as a dependency. Now we have Cabal as a dependency we might as well use the proper parser.
I was working with a .cabal file where our string savagery didn't work (below),  so I thought we should fix this properly.
package:
  foo
version:
  1.2.3
...

! @charleso

👍


If FTL can't start for some reason, attempting to start it again results in failure to start + icon spinning

The first start failed because PG was hosed. The second attempt failed with the following message, and the spinner just kept spinning forever. I had to restart.

Related, my attempt shows successfully built when intentionally crashing on startup:


⚠ SceneTime has degraded performance

In f863d15, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 2448 ms

Resolved: SceneTime performance has improved in c6ddd9a after 6 minutes.


standard_name and units for CMIP6_formula_terms.json

We should probably add standard_name (when available) and units attributes to each of the variables defined in CMIP6_formula_terms.json .

This duplicates a request made in https://github.com/PCMDI/cmip6-cmor-tables/issues/250, so I'll close.


chore(build): apply EmbedUntrackedSources

This change is part of the following stack:

#35 ◀

Change managed by git-spice.


Brackets will not run

Issue description
installed with nixos-rebuild test
Error at runtime:
[root@nixos:/etc/nixos]# brackets
/run/current-system/sw/bin/brackets: error while loading shared libraries: libXdamage.so.1: cannot open shared object file: No such file or directory

Technical details
[root@nixos:/drives]# nix-shell -p nix-info --run "nix-info -m"
these paths will be fetched (0.05 MiB download, 0.28 MiB unpacked):
  /nix/store/ial8a56ys2rswfq4j2abm3cl99li5b6p-bash-interactive-4.4-p23-dev
copying path '/nix/store/ial8a56ys2rswfq4j2abm3cl99li5b6p-bash-interactive-4.4-p23-dev' from 'https://cache.nixos.org'...
 - system: `"x86_64-linux"`
 - host os: `Linux 4.14.97, NixOS, 18.09.2108.8008d0bbc69 (Jellyfish)`
 - multi-user?: `no`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.1.3`
 - channels(root): `"nixos-18.09.2108.8008d0bbc69"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

It's a matter of adding the missing librar(y|ies) to bracketLibs. Do you want to try a PR?

It's a matter of adding the missing librar(y|ies) to bracketLibs. Do you want to try a PR?

Yes! I know how to do git fork/PR, but I don't know the process for including a missing library. Can you teach me? I can sign up for a chat forum/slack/mattermost/Zulip/IRC. Btw Zulip is the best.

Due to the time difference, it might be a bit difficult to find a time where we are both online but it basically boils down to:

find the relevant missing libraries (ldd on the binary is the easiest to get an overview)
find the corresponding derivation name in nixpkgs (in case of libxdamage it's xorg.libXdamage)
add that to the bracketLibs list
build and see if it runs. If not, rinse and repeat until it does.


Good artifact effects

About the PR
adds several new GOOD effects:

spawn monkeys
charge batteries
spawn money
healing chem foam
healing damage
creating a new artifact
intercom permanent effect

also buffed two old effects:

ore effect now spawns clusters
machine part effect now can spawn 1-2 parts

Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

:cl:

add: Added several new beneficial artifact effects.

a proposal: fax machine utility artifact


[Bug]: Issue in styling of navbar

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
When the left and right elements of the navbar are hovered, the black background of these elements touches the border of the navbar. It needs to be fixed by adding proper margin,
Add Screenshots / Video Clip

What browsers are you seeing the problem on?
Firefox, Chrome, Safari, Microsoft Edge
Agreements

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSoC contributor
[X] I'm a Hacktoberfest contributor
[X] I want to work on this issue [Optional]
[X] I'm willing to provide further clarification or assistance if needed.

@Tejashri-Taral Could you pls take a look and assign me the issue?

Navbar is already good


Python coding question....

I would like to compress the sgui.ProgressMeter widget down so that it has a single call.
Currently it;.s
meter = sgui.ProgressMeterCreate(....)
loop...
not_cancelled = sgui.ProgressMeterUpdate(...)
end loop
I would like to propose this instead
loop...
not_cancelled = sgui.ProgressMeterUpdate(...)
end loop
No create call.
I want the code in ProgressMeterUpdate to call the Create function is the meter is not created yet. If it is, then it just needs to update it.
How can  I know if this is the first call to a function?  Would 'nested functions' be of help as a way to save state?
I'm searching for 1-liner anywhere I can make them

I am getting help with this on StackOverflow.  The 'Correct' answer was just posted.

I was looking for the 'decorator' solution.  It turned out to be damned complicated looking.
I want to be multi-threaded in the future.
I'm multi-processed at the moment which is good enough for me.
The reason for this push of mine for 1-line solution is that even with tqdm, I find adding 2 lines of code to instrument a section is a barrier at times.
The new API call is simply sgui.ProgressBar(title, msg, currentItemNumber, totalNumberItems) and it can be placed in any loop basically.


Add support for --terragrunt-source-map as a more consistent, predictable way to override source URLs

Terragrunt's --terragrunt-source parameter, which is designed to make it easy to override source URLs in terragrunt.hcl, has a couple issues:

If you run terragrunt apply --terragrunt-source /foo/bar in folder yyy, as of now, the only thing it does is override the source URL in yyy/terragrunt.hcl to the exact value /foo/bar. However, if you run terragrunt apply-all --terragrunt-source /foo/bar in folder yyy, it overrides the source value of each child module within yyy/sub/path to something like /foo/bar/sub/path. This inconsistency can be confusing and only gets worse when working with read_terragrunt_config, where we have to parse another terragrunt.hcl and decide whether to override its source URL or not.
--terragrunt-source assumes all your modules are coming from the same root repo. You won't be able to do an apply-all if you're pulling from multiple repos (e.g., your own infra modules, Terraform Registry, Gruntwork repos, etc).

Perhaps a better solution to this is to support some sort of mapping:
terragrunt apply --terragrunt-source-map source:dest

The --terragrunt-source-map param would replace any source URL that starts with source to instead start with dest.
For example:
terragrunt apply --terragrunt-source-map github.com/org/modules.git:/local/path/to/modules

The above would replace source = "github.com/org/modules.git//xxx" with /local/path/to/modules//xxx regardless of whether you were running apply, or apply-all, or using a dependency. This would give us a consistent behavior across all commands and support mapping multiple repos by specifying the param multiple times:
terragrunt apply \
  --terragrunt-source-map github.com/org/modules.git:/local/path/to/modules \
  --terragrunt-source-map github.com/org/another-repo.git:/local/path/to/another-repo

We could also consider allowing users to specify this mapping in a file:
terragrunt apply --terragrunt-source-map mapping.hcl

Where mapping.hcl contains:
mappings = {
  "github.com/org/modules.git"      = "/local/path/to/modules"
  "github.com/org/another-repo.git" = "/local/path/to/another-repo"
}

See https://github.com/gruntwork-io/terragrunt/pull/1134#discussion_r407167726 for context for this issue.

Core feature is done, but still missing the following:

[ ] Environment variable configuration
[ ] file based configuration

Env var config PR: https://github.com/gruntwork-io/terragrunt/pull/1676

I am wondering, can this handle the immutable ref suffix if used?
https://github.com/gruntwork-io/terragrunt/issues/1627#issuecomment-847082445
in the example linked:
source = "github.com/firehawkvfx/firehawk-main.git//modules/vault?ref=v0.0.20"
source = "github.com/firehawkvfx/firehawk-main.git//modules/vault-kv-init?ref=v0.0.20"
source = "github.com/firehawkvfx/firehawk-main.git//modules/vault-policies?ref=v0.0.20"

We would want to override all these to point at another path to test before the next release.

Yes this should work with --terragrunt-source-map github.com/firehawkvfx/firehawk-main.git=/path/to/local/modules. The source map feature ignores and drops query parameters when doing the mapping.

Thank you for clarifying.

First of all this is a great feature! I've got a lot of nested modules and it is a pain to test them.
Sadly I got no luck configuring that in Windows. I tried to run like --terragrunt-source-map git::https://azuredevopsurl/_git/=C:\IaC and it fails with "Did not find any Terraform files (*.tf)".
I compared debug logs with/without the --terragrunt-source-map argument and it seems that the "WorkingDir" variable gets a non hashed value when using --terragrunt-source-map with C:\ in the path.
Have anyone being able to use this with Windows?

The source map feature ignores and drops query parameters when doing the mapping.

Are there any plans to allow this behaviour to be configurable? e.g. --preserve-query-parameters or equivalent?
My use case is that I define the paths to my modules using something like
git::ssh://git@gitlab.com/company/project.git?ref=1.0

This means a developer can check the project out and run a plan without any faffing with credentials or personal access tokens.
However, my CI runner doesn't have an SSH key, so would ideally use a path like
git::https://gitlab-ci-token:${CI_JOB_TOKEN}@gitlab.com/company/project.git?ref=1.0

I tried using TERRAFORM_SOURCE_MAP for this but unfortunately, because the query parameters are ignored, it will always point to the master branch rather than the ref specified.
Cheers!

The source map feature ignores and drops query parameters when doing the mapping.

Are there any plans to allow this behaviour to be configurable? e.g. --preserve-query-parameters or equivalent?
My use case is that I define the paths to my modules using something like
git::ssh://git@gitlab.com/company/project.git?ref=1.0

This means a developer can check the project out and run a plan without any faffing with credentials or personal access tokens.
However, my CI runner doesn't have an SSH key, so would ideally use a path like
git::https://gitlab-ci-token:${CI_JOB_TOKEN}@gitlab.com/company/project.git?ref=1.0

I tried using TERRAFORM_SOURCE_MAP for this but unfortunately, because the query parameters are ignored, it will always point to the master branch rather than the ref specified.
Cheers!

Can you try to add to your CI .gitconfig the following contents like Atlantis does
https://github.com/runatlantis/helm-charts/blob/main/charts/atlantis/values.yaml#L53 ?
I think this will solve your issue.

@angeloskaltsikis amazing! Thanks so much for the quick response, this did just the trick. I added the following as a pre-script:
git config --global<EMAIL_ADDRESS><EMAIL_ADDRESS>
And now no need to use TERRAFORM_SOURCE_MAP at all in the CI. Thanks a lot!

@angeloskaltsikis amazing! Thanks so much for the quick response, this did just the trick. I added the following as a pre-script:
git config --global<EMAIL_ADDRESS><EMAIL_ADDRESS>
And now no need to use TERRAFORM_SOURCE_MAP at all in the CI. Thanks a lot!

That's great to hear. I would be glad if you can add me in LinkedIn to discuss further about your CI and how do you use it :-)

@angeloskaltsikis amazing! Thanks so much for the quick response, this did just the trick. I added the following as a pre-script:
git config --global<EMAIL_ADDRESS><EMAIL_ADDRESS>

And now no need to use TERRAFORM_SOURCE_MAP at all in the CI. Thanks a lot!

That's great to hear. I would be glad if you can add me in LinkedIn to discuss further about your CI and how do you use it :-)

Thanks, It is works for the Main branch. But when I create another branch and run the pipeline I see the following err.
Can you pls. suggest something.
$ terragrunt init -upgrade
time=2024-02-06T10:38:06Z level=error msg=1 error occurred:
	* error downloading 'ssh://git@gitlab.bare.mycompany.org/edp/service/mynet/iam1/terraform-modules.git?ref=HEAD': /usr/bin/git exited with 128: Cloning into '/builds/edp/service/mynet/iam1/keycloak-terragrunt/environments/test_env/.terragrunt-cache/ZZZZ-u3X9KRbErgeRuqI-FaZT0/q-vjwbnLERG79bd2l12h4OVvHZA'...
remote: HTTP Basic: Access denied. The provided password or token is incorrect or your account has 2FA enabled and you must use a personal access token instead of a password. See https://gitlab.bare.pandrosion.org/help/topics/git/troubleshooting_git#error-on-git-fetch-http-basic-access-denied
fatal: Authentication failed for 'https://gitlab.bare.mycompany.org/edp/service/mynet/iam1/terraform-modules.git/'
time=20[24](https://gitlab.bare.mycompany.org/edp/service/mynet/iam1/keycloak-terragrunt/-/jobs/673296#L24)-02-06T10:38:06Z level=error msg=Unable to determine underlying exit code, so Terragrunt will exit with error code 1


add rust-gnu-1.43.1

add rust-gnu-1.43.1(latest version)
Microsoft Reviewers: Open in CodeFlow

submission

@KevinLaMS (this file)[https://static.rust-lang.org/dist/rust-1.43.1-x86_64-pc-windows-gnu.msi] is Mozilla Rustlang official binary, and static.rust-lang.org is rustlang official website domain, should be ok?

https://www.microsoft.com/en-us/wdsi/submission/cca06711-4c2a-4780-aaf1-6e04f8f33198


groovy.json.JsonException: Unable to determine the current character, it is not a string, number, array, or object FAILURE: Build failed with an exception

<EMAIL_ADDRESS>config]` finished with an exception. Error message: groovy.json.JsonException: Unable to determine the current character, it is not a string, number, array, or object

What went wrong:
A problem occurred evaluating script.

Calling [node<EMAIL_ADDRESS>config] finished with an exception. Error message: groovy.json.JsonException: Unable to determine the current character, it is not a string, number, array, or object

The current character read is 'i' with an int value of 105
Unable to determine the current character, it is not a string, number, array, or object
line number 1
index number 0
"react": "18.0.0",
"react-native": "0.69.3",

You need to upgrade RN to 0.71


rudimentary specification of short_name and long_name result.

nothing fancy, formattedAddress doesn't change according to specified options but the individual options returned work. Have not added any fancy options to change inside html document, only change in .js file.

type variable was specified in the wrong place.

I didn't realise that the commit was added automatically.

Done


feat:handle version confilct

Consider the condition that there are different version modules, Only one of them should be resolved correctly.
If  we are failed to add the peerDependencies of module A , we shouldn't handle A as a external module.

Hi @fedeoo
Thank you for your PR! :)
Is this really an issue? urls is an object, so if there is a module is required twice, it will be overwritten (Which I agree is not the best solution)
Would you mind adding some tests

@mastilver  I will add some tests soon.
I know urls is an object,  but just for a case：A project depends<EMAIL_ADDRESS>and a UI module named A, A depend on<EMAIL_ADDRESS> we will store both<EMAIL_ADDRESS>and<EMAIL_ADDRESS>to urls. This may be lead some exceptions.

No it will store only one, the latest one
The real issue is that it will use the wrong version for one of the dependencies
Can you also get rid of your code around peerDependencies
We also need to address the case when:
A@1 -> B@1
C@1 -> B@2 (peerDependency)
B@2 (because of C)
A can't be served through cdn, B and C can

A@1 is required: it's being bundled
B@1 is required by A: it's being served from the cdn
C@1 is required: it's being served from the cdn / It will use B@1 as it's exposing a global var
B@2 is required: it's being bundled / it will not be use as C is already using B@1

I'm not sure how to fix that yet, feel free to just summit failing tests if you are not sure how to resolve that

May be it's not a case, Actually we just get a warning when we install those packages.
I was wrong about the peerDependency. I will close this PR. Thanks for your remand.

There is still a case we should deal with.
When we are failed to resolve B from CDN, A and C shouldn't be served from the CDN(in most case will be failed as well).

I still think this is an issue, thank you for making me realise it #22

When we are failed to resolve B from CDN, A and C shouldn't be served from the CDN(in most case will be failed as well).

This should be dealt by module-to-cdn, there is an issue for that: https://github.com/mastilver/module-to-cdn/issues/5 ;)

@fedeoo Damn, I just realise your approach with peerDependencies was good, I will put some comments on this PR, if you want to work on that
If you don't that's fine, I will try to spend some time on it next week :)

@fedeoo fixed that on https://github.com/mastilver/modules-cdn-webpack-plugin/commit/7257b5ffd12a3213077c096f51e77f6a1742ae56

@mastilver 👍


maplocal: Tweaks for release

Removed unused content.
Removed unused API & references.
Fix help content.
Reduce logging level from error to warn.

Related to: https://github.com/zaproxy/zaproxy/issues/7940

Pushed an amend with no changes to see if the build would get happy. Things seem fine on my end, but GitHub was having issues the day this was originally opened.

Yay! Success.


random 'projects up to date' error

Randomly, I see a dialog popup saying that 'The projects are up to date.' Sometimes it happens on the method compile.
This time, there was a subsequent error that allowed me to see a little of the stack.
Where does this come from?

I think that Jade occasionally (or all of the time?) does double compiles (when there is a Warning perhaps) and the second compile will almost always signal this ... Perhaps we should signal an explicit notification and in the places where Jade is doing a second compile (if that is what is going on), we can resume without informing the user ...

bounce to @ericwinger ... @ericwinger will investigate double Jade compiles and/or try to get a stack or something more than anecdotal evidence so that we know where to look for a fix

haven't seen this in a long time


Feature Request - Add "Copy Relative Path" to Explorer

VSCode Version: 1.10
OS Version: Windows 10

Adding the correct relative path for import statements takes time. For example, if you have a lot of files in your project it takes time to determine the correct path i.e.
import { Link } from '/data-structures/sanitised-data/link';
import { Link } from './data-structures/sanitised-data/link';
import { Link } from '../data-structures/sanitised-data/link';
import { Link } from '../../data-structures/sanitised-data/link';
It would be really useful if when you right click on a file in the VS Code explorer there was the option to copy relative path next to Copy Path and Copy. Where the relative path is compared to the active window you have open at the time so you can just copy the import statement in straight away.

#12518

Oops, mark as a duplicate in that case! I did try to search for other issues but none came up, honest (https://github.com/Microsoft/vscode/issues?utf8=✓&q=Copy Relative Path)

You can close this issue yourself if you're confident it's a duplicate 👍

Thanks @adambowles
Closing as duplicate
@henryJack you can add any additional thoughts to the linked item #12518


libgcrypt fails to build on Ubuntu 14.10

This is with version 1.6.2 of libgcrypt.  Discovered when attempting to install pianobar
https://gist.github.com/38a7178239568f946cd2

I'm seeing three failed tests as well. You should report this issue upstream to http://bugs.gnupg.org
/bin/bash: line 5: 16459 Segmentation fault      (core dumped) GCRYPT_IN_REGRESSION_TEST=1 ${dir}$tst
FAIL: basic
SALSA20     /bin/bash: line 5: 16521 Segmentation fault      (core dumped) GCRYPT_IN_REGRESSION_TEST=1 ${dir}$tst
FAIL: benchmark
     STREAM enc | /bin/bash: line 5: 16528 Segmentation fault      (core dumped) GCRYPT_IN_REGRESSION_TEST=1 ${dir}$tst
FAIL: bench-slope
SKIP: hashtest-256g
======================================
3 of 24 tests failed
(1 test was not run)
Please report to http://bugs.gnupg.org

As a work around you can brew edit libgcrypt and delete the line make check.


Profile memory utilization of worker daemon

Currently, the worker daemon is set to utilize 3gb of memory. Regardless of how much memory is actually utilized, AWS will charge us for the full memory reservation. We should profile the worker daemon to investigate why this high memory limit is required. There are some python tools that can help us here. We should also dig into the logs to see what the distribution of memory usage is historically (lambda logs output this in the last line of each invocation).
Requirement for done: Either we fix the issue and reduce memory utilization or record why we need to maintain this limit of 3gb

I believe this is now more or less a canned query in Cloudwatch Logs Insights.

Running logs on cloudwatch insights indicates that the max memory used in Prod worker 1784mb and
1895mb over the last 4 weeks. @mckinsel @calvinnhieu  any complaints if I bump us down to 2000mb from 2861mb?

Sure, to save costs over the holidays? I anticipate we'll have to play around with that value when we start processing 10x bundles in the new year.

It's not necessarily just to save costs over the holidays. It's not even that much money at our current request rate. There's just no need to over provision though if we're not using it

I see, and that sounds good. Thanks for looking into this

No problem, but I suspect we'll have to crank it right back up with the first 10x bundle.


this parameter of union signatures should be the intersection of constituents’ this parameters

...but currently it’s the union. Discovered while looking at #31485.
Minimal repro:
type F1 = (this: void) => void;
type F2 = (this: number) => void;
declare var fn: F1 | F2;
fn(); // should error

Silly but understandable real-world-ish example (playground):
interface Car {
    odometer: number;
    drive(): void;
}

interface Cow {
    weight: number;
    eatGrass(): void;
    isARealCow(): boolean;
}

function carWorks(this: Car): boolean {
    this.drive();
    return this.odometer > 0;
}

function cowWorks(this: Cow): boolean {
    const prevWeight = this.weight;
    this.eatGrass();
    return this.weight > prevWeight;
}

const cow = {
    weight: 100,
    eatGrass() { this.weight++ },
    // Conditional expression creates a union type here
    isARealCow: Math.random() > 0.5 ? carWorks : cowWorks
}

// Runtime error ~50% of the time:
// TypeError: this.drive is not a function
//    at Object.carWorks [as isARealCow]
cow.isARealCow();

Since cow.isARealCow might call this.drive() or might call this.eatGrass(), the only thing that should be able to call it is something has a weight, eatGrass(), odometer, and drive()—that is, the intersection of the constitutents’ this parameter types.
TypeScript Version:  6a559e37ee0d660fcc94f086a34370e79e94b17a
Search Terms: union signatures this parameter variance intersection
Playground Link
Related Issues: #31485

Perhaps a mechanical cow of some kind would do.

:smile:
🤖🐄


wrong version on quickstar page, produce  error 'raw' RFC822 payload

on page:
https://developers.google.com/gmail/api/quickstart/nodejs
the docs say to install version 27:
npm install googleapis@27 --save
I was getting error  'raw' RFC822 payload ...'
after to uninstall and install the new version 32
npm install googleapis --save
all is working well, please correct the guide and delete "@27" part
thanks

Greetings!  We tend to track updates to developers.google.com on our other issue tracker.  I created a bug here:
https://issuetracker.google.com/112509526
You can track updates over there!  FYI @grant.


Prerendered Endpoints return corrupted Buffers

Describe the bug
I discovered this behaviour whilst making an image optimisation endpoint powered by sharp. Basically endpoints whose output are buffers end up being wrongly prerendered (whilst being correctly returned in server environments). I'm certain that this behaviour is not intended. I did find a similar issue #5088 but it seems like no one interacted with it up until now.
Reproduction
Link to repository for reproduction
src/routes/video.mp4.js is the endpoint and gets referenced in src/routes/index.svelte.
Run npm run dev to test the endpoint in an server environment (video plays fine).
Run npm run build or look into my provided build folder to view a prerendered video file at /video.mp4 which ends up being corrupted (but not empty).
My code that returns a buffer:
import { readFileSync } from "fs"

export function get() {
	return {
		body: readFileSync("src/lib/source.mp4")
}	}

Logs
No response
System Info
System:
    OS: Linux 5.18 Fedora Linux 36 (Workstation Edition)
    CPU: (4) x64 Intel(R) Core(TM) i5-7500 CPU @ 3.40GHz
    Memory: 10.57 GB / 15.49 GB
    Container: Yes
    Shell: 3.5.0 - /usr/bin/fish
  Binaries:
    Node: 16.14.0 - /usr/bin/node
    npm: 8.13.1 - /usr/local/bin/npm
  Browsers:
    Firefox: 101.0.1
  npmPackages:
    @sveltejs/adapter-static: ^1.0.0-next.29 => 1.0.0-next.34
    @sveltejs/kit: next => 1.0.0-next.355
    svelte: ^3.46.4 => 3.48.0

Severity
blocking an upgrade
Additional Information
No response

This does sound like a duplicate of #5088. The static adapter uses the same prerendering handling internally as you're using here.


DBAAS-1403 Add ReadOnly registry

This will allow us to expose to services running in VMs the ability to pull services containers from cluster docker registry but without the risk of pushing from the guest network

Are u adding another docker registry to the cluster?
On Thu, Sep 13, 2018, 23:42 ravidbro<EMAIL_ADDRESS>wrote:

@ravidbro https://github.com/ravidbro requested your review on:
Stratoscale/distribution#2
https://github.com/Stratoscale/distribution/pull/2 DBAAS-1403 Add
ReadOnly registry.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/Stratoscale/distribution/pull/2#event-1844629615, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AFb381zYjY_UtLI445DZH3wIfdAR0U5sks5uasMjgaJpZM4WoLJm
.

Yes. Another service on a different port but using the same database so you will see the same images but this port is read-only


[Feature Request] Built in chairs?

Feature Description:
Maybe a built in chairs as Purpur already has so much to do with riding things and all. Sorry for all these feature requests, I'll stop after this.

Sounds good.


Develop highway bart

What existing problem does the pull request solve and why should we include it?
Bring Bart-specific highway changes (pre re-achitected) and transit to develop
What is the testing plan?
Demonstrate the code is solid by discussing how results are verified and covered by tests

[ ] Code for this PR is covered in tests
[ ] Code passes all existing tests

Code formatting
Code should be PEP8 compliant before merging by running a package like black

[ ] Code linted

Applicable Issues
Please do not create a Pull Request without creating an issue first.
Put closes #XXXX in your comment to auto-close the issue that your PR fixes.
Issues List

closes...
closes...

Close this PR. We will not merge it as we switch to the refactor code base.


yum sync history to remove rpmdb warning

sync rpmdb to remove warnings from yum

Nice catch!
I think another consideration should be replacing the use of rpm with yum when installing packages.  That way the rpmdb doesn't get out of sync in the first place.

Thanks @hmeiland

Yep, although some of that is in the hands of magic install scripts from e.g. Intel...
From: Richard Alloway @.>
Sent: Friday, September 24, 2021 3:25 PM
To: Azure/azhpc-images @.>
Cc: Hugo Meiland @.>; Author @.>
Subject: Re: [Azure/azhpc-images] yum sync history to remove rpmdb warning (#101)
Nice catch!
I think another consideration should be replacing the use of rpm with yum when installing packages. That way the rpmdb doesn't get out of sync in the first place.

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fazhpc-images%2Fpull%2F101%23issuecomment-926622556&data=04|01|hugo.meiland%40microsoft.com|02259ab2614c458ec8c508d97f5ebe70|72f988bf86f141af91ab2d7cd011db47|1|0|637680867195118151|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=FuTmTrR0ouCm%2FwM4zl1YiPRRu46Q2Qk434vePjetYvk%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAHWFPJ54FR6YF3BF56CCEBTUDR33VANCNFSM5EVZVXOQ&data=04|01|hugo.meiland%40microsoft.com|02259ab2614c458ec8c508d97f5ebe70|72f988bf86f141af91ab2d7cd011db47|1|0|637680867195128144|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=gpm8f4TfAIVmPg4GMfuTCj6iJFB3K9u8aR0%2F296jXjc%3D&reserved=0.
Triage notifications on the go with GitHub Mobile for iOShttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fapps.apple.com%2Fapp%2Fapple-store%2Fid1477376905%3Fct%3Dnotification-email%26mt%3D8%26pt%3D524675&data=04|01|hugo.meiland%40microsoft.com|02259ab2614c458ec8c508d97f5ebe70|72f988bf86f141af91ab2d7cd011db47|1|0|637680867195128144|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=8NRu2ofkM5DJzQMzEEDIGcm91JcRxha7S2kTH%2FbsKdQ%3D&reserved=0 or Androidhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fplay.google.com%2Fstore%2Fapps%2Fdetails%3Fid%3Dcom.github.android%26referrer%3Dutm_campaign%253Dnotification-email%2526utm_medium%253Demail%2526utm_source%253Dgithub&data=04|01|hugo.meiland%40microsoft.com|02259ab2614c458ec8c508d97f5ebe70|72f988bf86f141af91ab2d7cd011db47|1|0|637680867195138140|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=QOLl%2BFKoNmvXDqiDQgvZ9%2FzNsKulRoH8VUjQVehkBeo%3D&reserved=0.


Crash in InactivityCop::check_inactivity with dead HttpSM, ATS 6.2.1

We are seeing this crash in ATS 6.2.1, with http/2 enabled:
Program terminated with signal 6, Aborted.
#0  0x00002b4d32759625 in raise () from /lib64/libc.so.6
(gdb) bt
#0  0x00002b4d32759625 in raise () from /lib64/libc.so.6
#1  0x00002b4d3275ad8d in abort () from /lib64/libc.so.6
#2  0x00002b4d2fd314f3 in ink_die_die_die (fmt=0x2b4d2fd3ddd0 "%s:%d: failed assert `%s`", ap=0x2b4d384ffa40) at ink_error.cc:43
#3  ink_fatal_va(const char *, typedef __va_list_tag __va_list_tag *) (fmt=0x2b4d2fd3ddd0 "%s:%d: failed assert `%s`", ap=0x2b4d384ffa40) at ink_error.cc:65
#4  0x00002b4d2fd3158d in ink_fatal (message_format=0x11b0 <Address 0x11b0 out of bounds>) at ink_error.cc:73
#5  0x00002b4d2fd2ef35 in _ink_assert (expression=0xffffffffffffffff <Address 0xffffffffffffffff out of bounds>, file=0x13f6 <Address 0x13f6 out of bounds>, line=6) at ink_assert.cc:37
#6  0x00000000005a3cd2 in HttpSM::main_handler (this=0x2b4d42cad580, event=105, data=0x2b4d9b996438) at HttpSM.cc:2818
#7  0x00000000007267e9 in handleEvent (event=<value optimized out>, vc=0x2b4d9b996320) at ../../iocore/eventsystem/I_Continuation.h:153
#8  read_signal_and_update (event=<value optimized out>, vc=0x2b4d9b996320) at UnixNetVConnection.cc:148
#9  0x0000000000729ddf in UnixNetVConnection::mainEvent (this=0x2b4d9b996320, event=<value optimized out>, e=<value optimized out>) at UnixNetVConnection.cc:1205
#10 0x000000000071f921 in handleEvent (this=0x2b4d34b1b440, event=<value optimized out>, e=0x2b4d35756ca0) at ../../iocore/eventsystem/I_Continuation.h:153
#11 InactivityCop::check_inactivity (this=0x2b4d34b1b440, event=<value optimized out>, e=0x2b4d35756ca0) at UnixNet.cc:102
#12 0x0000000000749f1d in handleEvent (this=0x2b4d35507000, e=0x2b4d35756ca0, calling_code=2) at I_Continuation.h:153
#13 EThread::process_event (this=0x2b4d35507000, e=0x2b4d35756ca0, calling_code=2) at UnixEThread.cc:148
#14 0x000000000074aa43 in EThread::execute (this=0x2b4d35507000) at UnixEThread.cc:230
#15 0x000000000074935a in spawn_thread_internal (a=0x2b4d34b176d0) at Thread.cc:86
#16 0x00002b4d32ac29d1 in start_thread () from /lib64/libpthread.so.0
#17 0x00002b4d3280f9dd in clone () from /lib64/libc.so.6
(gdb) f 6
#6  0x00000000005a3cd2 in HttpSM::main_handler (this=0x2b4d42cad580, event=105, data=0x2b4d9b996438) at HttpSM.cc:2818
2818	HttpSM.cc: No such file or directory.
	in HttpSM.cc
(gdb) p/x magic
$1 = 0xdeadfeed

If possible, can you test with 7.0 or 7.1x (branch) and see if it happens there too ?

@zwoop , we are seeing this issue only in our production env and our synthetic tests and internal deployments do not have the issue. Also, we are in the process of upgrading from ATS 5 to ATS 6, and so can not quickly deploy 7 to a prod env.
I will continue to get a local repro, and will update the issue if that is possible. Meanwhile, wanted to post here to see if others have run into this, or if there have been changes to this feature recently.
Thanks!

If this is still an issue please reopen.


Error using hexrotor_base with a gimbal

Hi

Using quadrotor_base works fine
Using quadrotor_base with a gimbal works fine
Using hexrotor_base works fine
Using hexrotor_base with a gimbal raise an error

I use the following world.json file, all I change is the drone model_type, and the presence of a gimbal:
{
    "world_name": "my_world",
    "drones": [
        {
            "model_type": "hexrotor_base",
            "model_name": "drone0",
            "xyz": [0.0, 0.0, 0.3],
            "rpy": [0, 0, 0],
            "flight_time": 60,
            "payload": [
		        {
                    "model_type": "gimbal_position",
                    "model_name": "gb",
                    "payload":
                    {
                        "model_type": "rgbd_camera",
                        "model_name": "rgbd_camera"
                    }
                }
	        ]
        }
    ]
}

For hexrotor_base with a gimbal, I get the following error:

Why would there be any problem if it works with a quadrotor_base?
And how do I start to debug it?
Thank you

Hi, what you mention is a known issue with aerostack2 v1.0.9. It is already solved at the main branch and new releases would include the fix.

Yes, it is the only option. We are planning to release aerostack2 in the next days


Verification Mapping

Needs to validate the carrier mapping key and subscriber id and write out a report to the output bucket with the report and the status of the report.  Look at subscriber location as an example.
https://github.com/helium/oracles/blob/341b6e151c63e852fe2a64a06125e62d9599f9b6/mobile_verifier/src/subscriber_location.rs#L123


FEATURE: MPNT-31 Update pdm.json to new schema name

Build succeeded (check pipeline).

https://checklist.intgdc.com/job/compliance/job/gdc-public-demos-oss-check/21/ : SUCCESS in 5s


Shared photo paths are incorrect on Ventura (was: Missing photos, not actually missing?)

For the last day or so, I have been unable to handle the missing photos issue.
Describe the bug
osxphotos is reporting that I am missing photos, even after... I tried:

Running --retry 3 --download-missing
Exporting and reviewing errors
Manually verifying and loading each of them
Manually verify that they in fact were not exported
Deleting them, re-uploading them
Deleting them, repairing my Apple Photos, re-uploading them
Upgrading osxphotos

This is the command I am running:
osxphotos export "/Volumes/Extreme SSD/osxphotos" --exiftool --exiftool-merge-keywords --album-keyword --retry 3 --download-missing --directory "{created.year}/" --touch-file --update --cleanup --skip-edited --report export.csv

In investigating this, I also discovered that it wasn't adding the photos to a "missing" album. That feature didn't work for these photos either which I thought was strange. It did work for some other photos which I managed to resolve; however, for these photos it just didn't work at all.
To Reproduce
I am not sure how to reproduce it on another computer without uploading my entire library.
Expected behavior
These photos are present; I can download and re-import them. I am trying to figure out how to resolve this.
Desktop (please complete the following information):
OS: macOS Ventura Version 13.1 (22C65)
% osxphotos --version
osxphotos, version 0.55.5

Additional context
Here is the verbose log:
% osxphotos export "/Volumes/Extreme SSD/osxphotos" --exiftool --exiftool-merge-keywords --album-keyword --retry 3 --download-missing --directory "{created.year}/" --touch-file --update --skip-edited --report export.csv --missing --verbose
osxphotos version 0.55.5
exiftool path: /opt/homebrew/bin/exiftool
Using last opened Photos library: /Users/michaelpetrochuk/Pictures/Photos Library.photoslibrary
Using export database /Volumes/Extreme SSD/osxphotos/.osxphotos_export.db
Processing database /Users/michaelpetrochuk/Pictures/Photos Library.photoslibrary/database/photos.db
Processing database /Users/michaelpetrochuk/Pictures/Photos Library.photoslibrary/database/Photos.sqlite
Processing database.
Database version: 6000, 8.
Processing persons in photos.
Processing detected faces in photos.
Processing albums.
Processing keywords.
Processing photo details.
Processing import sessions.
Processing additional photo details.
Processing face details.
Processing photo labels.
Processing EXIF details.
Processing computed aesthetic scores.
Processing comments and likes for shared photos.
Processing moments.
Done processing details from Photos library.
Exporting 21 photos to /Volumes/Extreme SSD/osxphotos...
Exporting IMG_4804.HEIC (6AED2C77-58FC-4104-B73F-7AC55749EB75.JPG) (1/21)
Skipping missing original photo IMG_4804.HEIC (6AED2C77-58FC-4104-B73F-7AC55749EB75)
Exporting IMG_4807.HEIC (8FF24096-4709-400F-B97C-2470B2F93DC2.JPG) (2/21)
Skipping missing original photo IMG_4807.HEIC (8FF24096-4709-400F-B97C-2470B2F93DC2)
Exporting IMG_4806.HEIC (4353809B-6C3A-4E25-9E9D-D10A6D41107D.JPG) (3/21)
Skipping missing original photo IMG_4806.HEIC (4353809B-6C3A-4E25-9E9D-D10A6D41107D)
Exporting IMG_4793.HEIC (DEF6E36C-061B-42F4-B11B-1B5B6F85E4BE.JPG) (4/21)
Skipping missing original photo IMG_4793.HEIC (DEF6E36C-061B-42F4-B11B-1B5B6F85E4BE)
Exporting IMG_4812.HEIC (62403E43-816E-44C8-A39B-58755DBB17B0.JPG) (5/21)
Skipping missing original photo IMG_4812.HEIC (62403E43-816E-44C8-A39B-58755DBB17B0)
Exporting IMG_4810.HEIC (19C5853B-9579-4FDB-B4F5-6627CE77C857.JPG) (6/21)
Skipping missing original photo IMG_4810.HEIC (19C5853B-9579-4FDB-B4F5-6627CE77C857)
Exporting IMG_4811.HEIC (7677DBF7-9C0B-4ADC-9EFD-45B47A74ED3B.JPG) (7/21)
Skipping missing original photo IMG_4811.HEIC (7677DBF7-9C0B-4ADC-9EFD-45B47A74ED3B)
Exporting IMG_4795.HEIC (023D8DD6-1660-46A4-8993-75C957C9CFE8.JPG) (8/21)
Skipping missing original photo IMG_4795.HEIC (023D8DD6-1660-46A4-8993-75C957C9CFE8)
Exporting IMG_4794.HEIC (66BF4740-048A-4081-82C8-6CCABC4A99DF.JPG) (9/21)
Skipping missing original photo IMG_4794.HEIC (66BF4740-048A-4081-82C8-6CCABC4A99DF)
Exporting IMG_4797.HEIC (20E1E1B9-875B-4200-98A8-4DDB3F2A521B.JPG) (10/21)
Skipping missing original photo IMG_4797.HEIC (20E1E1B9-875B-4200-98A8-4DDB3F2A521B)
Exporting IMG_4802.HEIC (EB2F6BB7-53E2-493F-940C-179B19FBBF33.JPG) (11/21)
Skipping missing original photo IMG_4802.HEIC (EB2F6BB7-53E2-493F-940C-179B19FBBF33)
Exporting IMG_4800.HEIC (146C32BF-A93C-44A8-A19C-DD2FA7D740BE.JPG) (12/21)
Skipping missing original photo IMG_4800.HEIC (146C32BF-A93C-44A8-A19C-DD2FA7D740BE)
Exporting IMG_4808.HEIC (6D1A0AFD-5D5B-467D-9C88-FCFDBCC0207A.JPG) (13/21)
Skipping missing original photo IMG_4808.HEIC (6D1A0AFD-5D5B-467D-9C88-FCFDBCC0207A)
Exporting IMG_4801.HEIC (F3DC9A8A-96E7-4F49-B4AF-58ED49AA22B3.JPG) (14/21)
Skipping missing original photo IMG_4801.HEIC (F3DC9A8A-96E7-4F49-B4AF-58ED49AA22B3)
Exporting IMG_4796.HEIC (8098DD87-F5E7-4EE9-A359-26CC64761F7A.JPG) (15/21)
Skipping missing original photo IMG_4796.HEIC (8098DD87-F5E7-4EE9-A359-26CC64761F7A)
Exporting IMG_4809.HEIC (AD49464C-9878-491A-896E-E69A13845F55.JPG) (16/21)
Skipping missing original photo IMG_4809.HEIC (AD49464C-9878-491A-896E-E69A13845F55)
Exporting IMG_4803.HEIC (82D45994-3D5E-4071-9090-86EF43D07D38.JPG) (17/21)
Skipping missing original photo IMG_4803.HEIC (82D45994-3D5E-4071-9090-86EF43D07D38)
Exporting IMG_4799.HEIC (3D890EF6-9A8D-4A3E-BF60-C2424899EBDD.JPG) (18/21)
Skipping missing original photo IMG_4799.HEIC (3D890EF6-9A8D-4A3E-BF60-C2424899EBDD)
Exporting IMG_4792.HEIC (0EC6E0A1-1445-4010-8022-72B60369593A.JPG) (19/21)
Skipping missing original photo IMG_4792.HEIC (0EC6E0A1-1445-4010-8022-72B60369593A)
Exporting IMG_4805.HEIC (20FDB02B-B9F8-434F-B997-434E126A7D9E.JPG) (20/21)
Skipping missing original photo IMG_4805.HEIC (20FDB02B-B9F8-434F-B997-434E126A7D9E)
Exporting IMG_4798.HEIC (A472728D-021F-4315-87AB-DC770116C911.JPG) (21/21)
Skipping missing original photo IMG_4798.HEIC (A472728D-021F-4315-87AB-DC770116C911)
Exporting 21 photos ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 100% 0:00:00
Processed: 21 photos, exported: 0, updated: 0, skipped: 0, updated EXIF data: 0, missing: 21, error: 0, touched date: 0
Elapsed time: 0:00:04
Wrote export report to export.csv

What should I do next? Maybe I should do a fresh export?

In investigating this, I also discovered that it wasn't adding the photos to a "missing" album.

Were you using the --add-missing-to-album option? That wasn't included on your command line above.
Try the following debug command:
osxphotos debug-dump --dump photos --uuid 6AED2C77-58FC-4104-B73F-7AC55749EB75 --uuid 8FF24096-4709-400F-B97C-2470B2F93DC2 > debug.txt
Then post the debug.txt here or send to me at<EMAIL_ADDRESS>(and reference this issue #883 in the subject).  Note: the debug output may contain data you consider sensitive such as location.  You may want to look at the text file first and delete any data you don't want to share.

One more bit of debug data that would be helpful:
Run the command osxphotos repl --missing
This will open an interactive osxphotos python prompt. At the prompt, type the following:
>>> for p in photos:
...     print(f"{p.uuid}, {p._path_5()}, {os.path.exists(p._path_5())}")

Type or paste the first line after the >>> then hit return and type the second line (it will ident automatically)
It should look something like this:
>>> for p in photos:
...     print(f"{p.uuid}, {p._path_5()}, {os.path.exists(p._path_5())}")
8E1D7BC9-9321-44F9-8CFB-4083F6B9232A, /Users/rhet/Downloads/IMG_2000.JPG, False
A1DD1F98-2ECD-431F-9AC9-5AFEFE2D3A5C, /Volumes/MacBook Mojave/Users/Shared/Pumpkins4.jpg, False

This will print the expected path (where the original photo should be) for each missing photo and print out True if the file actually exists on disk or False if not.

Done!! Thank you!

In investigating this, I also discovered that it wasn't adding the photos to a "missing" album.

Were you using the --add-missing-to-album option? That wasn't included on your command line above.

Yeah, sorry, I tried it another time. I can confirm I tried it because an album was created, and some files were added. Unfortunately, those files above were not.

Got the data -- thanks. These may be related to #884.  The files are HEIC format ( 'originalFilename': 'IMG_4804.HEIC') but the UTI is listed as public.jpeg and Photos thinks the file on disk is a jpeg:  'filename': '6AED2C77-58FC-4104-B73F-7AC55749EB75.JPG',
Try the following command and send the output:
file  ~/Pictures/Photos\ Library.photoslibrary/originals/6/6AED2C77-58FC-4104-B73F-7AC55749EB75*

Okay, so, I tried file but that came back with "no matches found" so then I tried find.
% file  ~/Pictures/Photos\ Library.photoslibrary/originals/6/6AED2C77-58FC-4104-B73F-7AC55749EB75*
zsh: no matches found: /Users/michaelpetrochuk/Pictures/Photos Library.photoslibrary/originals/6/6AED2C77-58FC-4104-B73F-7AC55749EB75*
% find ~/Pictures/Photos\ Library.photoslibrary/ -name "6AED2C77-58FC-4104-B73F-7AC55749EB75*"
/Users/michaelpetrochuk/Pictures/Photos Library.photoslibrary//scopes/cloudsharing/resources/derivatives/masters/6/6AED2C77-58FC-4104-B73F-7AC55749EB75_4_5005_c.jpeg
/Users/michaelpetrochuk/Pictures/Photos Library.photoslibrary//scopes/cloudsharing/data/1370202095/8C86169C-ABEB-4AC2-BD73-82DE5FF85496/6AED2C77-58FC-4104-B73F-7AC55749EB75.JPG

I also tried `find` with another image in the set. I am not sure why it says `cloudsharing`, I didn't share that photo, as far as I can recall.

```bash
% find ~/Pictures/Photos\ Library.photoslibrary/ -name "A472728D-021F-4315-87AB-DC770116C911*"
/Users/michaelpetrochuk/Pictures/Photos Library.photoslibrary//scopes/cloudsharing/resources/derivatives/masters/A/A472728D-021F-4315-87AB-DC770116C911_4_5005_c.jpeg
/Users/michaelpetrochuk/Pictures/Photos Library.photoslibrary//scopes/cloudsharing/data/1370202095/8C86169C-ABEB-4AC2-BD73-82DE5FF85496/A472728D-021F-4315-87AB-DC770116C911.JPG

I think I've figured this out. All the affected photos are shared photos in a shared iCloud album (the old-style shared album, not iCloud shared library).  The path for these photos has apparently changed -- in previous versions of macOS they were in Photos Library.photoslibrary/resources/cloudsharing/... but they're now in Photos Library.photoslibrary/scopes/cloudsharing/... and osxphotos wasn't aware of this so could not find the files. I've just recently gotten a machine with Ventura so hadn't tested this before.  Now that I know where Ventura is storing the files I can implement a fix.
This is possible related to #626 but I'll need to check on a machine running with shared iCloud albums on Monterey (which I don't have access to).

That's so fascinating! It sounds like we are close to a fix.
I am still curious how these ended up in a shared folder. These are pretty private photos that I wouldn't have any reason to share. They are also not in a shared album.
I also downloaded them and re-imported them, and it sounds like the shared state persisted. This could be connected to the earlier issue I experienced where Apple's cache persisted even after deleting and re-importing files.
I wonder if there is a way to clear the cache, so that database issues like this don't persist.

OH, I figured it out. I have photos, with the same exact name in a shared album, that was shared with me! I think I have been downloading and re-importing the wrong photos because the names were the same!

In investigating this, I also discovered that it wasn't adding the photos to a "missing" album. That feature didn't work for these photos either which I thought was strange.

Shared photos cannot be added to regular albums unless they are imported to the library,  hence these photos would not be added.

@PetrochukM I believe this should be fixed in v0.55.7.  Please upgrade (pipx upgrade osxphotos assuming you installed with pipx) and let me know if you're still experiencing the problem.

@PetrochukM have you had a chance to try the latest osxphotos and if so, are you still experiencing this issue?

Hey! I have been trying it, and I still have missing photos. I need to double-check what's happening; sorry for the slow response!

Please run the debug dump again (substituting the uuid of one of the photos still missing, which you can get from the verbose output).
osxphotos debug-dump --dump photos --uuid 6AED2C77-58FC-4104-B73F-7AC55749EB75 --uuid 8FF24096-4709-400F-B97C-2470B2F93DC2 > debug.txt

Okay, I took a look, and this is what is going on. I now have photos missing when two photos have the same filename and different file extensions. In this case, one photo is shared, and the other is not. The image which is shared, in this case, does not get exported.

Please run the debug-dump command for a couple of the photos in question. You can get the UUID from the verbose output:
For example, in this line:
Exporting IMG_4804.HEIC (6AED2C77-58FC-4104-B73F-7AC55749EB75.JPG) (1/21)
The UUID is 6AED2C77-58FC-4104-B73F-7AC55749EB75
So use that in the debug dump command:
osxphotos debug-dump --dump photos --uuid 6AED2C77-58FC-4104-B73F-7AC55749EB75 --uuid 8FF24096-4709-400F-B97C-2470B2F93DC2 > debug.txt


Get user token after social authentication

Hey there. I have a JavaScript client app where I am able to authenticate my users and send the access_token to the Django app. With dj-rest-auth I am successfully registering my users and I get this response:
{
"data":
    {
      "type":"FacebookLogin", "id":"54b777180f8303bbb63ebf91a73e9cfef47e4e4f",
      "attributes":{"key":"54b777180f8303bbb63ebf91a73e9cfef47e4e4f"}
     }
}

But to access my protected views in the backend I need to send a token back to the server like this:
Authorization: Token 9944b09199c62bcf9418ad846dd0e4bbdfc6ee4b
I couldn't figure out how to use the response above to get the user token. How to get the access_token that I need to send to the server when authenticating users with Facebook or Google provider?

I suppose that "key":"54b777180f8303bbb63ebf91a73e9cfef47e4e4f" is the token that you need to include into the header to access protected backend endpoints.


Download file with annotations

It's possible download file with annotations???

If you ask about downloading PDF with embedded highlights, the answer is no. However some useful code might be found in https://github.com/agentcooper/pdf-annotation-service.


Use the embedded-hal::digital::v2::OutputPin

Very similarly to https://github.com/jamwaffles/sh1106/pull/8, I added an error enum with two types: a communication error type and a pin error type which allows users to handle I2C or SPI communication errors as well as SPI pin setting errors.
I had to make reset fallible, though. User code will then get an unused Result warning.
No other changes are required in user code if the results were simply unwrapped.

Thanks @eldruin! I'll get this released when I'm next on a Linux machine.

Released in 0.2.5.


Why did actor i move his dimension k position in turn t?

We need to make it easier for analysts to answer this question.  See Trello card https://trello.com/c/UDkRG9V9.

PR 235 took care of this, so closing the issue.


AZ CLI: Improve handling of npm install

Tool
Name: az cli
Version: 2.0.53 (0.1.3)
Describe the bug
KuduClient.install_npm_dependencies() performs a request for Kudu to install the npm dependencies. However, npm install can take a long time and the client can timeout while waiting for the successful response from Kudu.
Expected behavior
Not have the KuduClient timeout due to npm install taking a long time. KuduClient timing out leads to an error that makes it seem like the publish failed (which it may or may not have).
[bug]

Closing via Azure/Azure-cli#8445 and Azure/Azure-cli#8711


Anndata update

Provides ability to chose custom labels names to load in batch indices, cell types, etc.

Codecov Report

Merging #475 into master will increase coverage by 0.03%.
The diff coverage is 72.72%.

@@            Coverage Diff             @@
##           master     #475      +/-   ##
==========================================
+ Coverage   89.23%   89.27%   +0.03%
==========================================
  Files          31       31
  Lines        3522     3525       +3
==========================================
+ Hits         3143     3147       +4
+ Misses        379      378       -1

Impacted Files
Coverage Δ

scvi/dataset/anndataset.py
85.41% <72.72%> (+0.97%)
:arrow_up:

scvi/inference/autotune.py
75.78% <0%> (+0.21%)
:arrow_up:

Continue to review full report at Codecov.

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


Node: Set temperature resolution

sensor.setResolution(R_DEGREES_0_0625);

Somehow did causes the node to brick. Maybe because we call it before setup().


Revert "Merge pull request #4344 from bartvde/issue-2844"

This reverts commit e5c44b37c9495faf0399eba2c33dfb5cd5cee1f4, reversing
changes made to 7f4c6e363464ed72e35d2465e0b38bf68afeea22.
Fixes #4405.
When this is reverted, we can work on an alternative solution to the OP's issue #2844. I'm envisioning a canvas that the ImageStatic source delivers to the renderer, which already has the desired aspect ratio, and is created once from the original ImageStatic image.

please merge


Removed EvaluatingCommand

Removed EvaluatingCommand, the interpreter now uses a DelayedCommand.
Added client condition to prevent calling a DelayedCommand.
Several categories renamed for consistency.
Reverted use of supplier anchor.
Added tests for more commands.

Coverage decreased (-0.02%) to 99.893% when pulling d1d16ce468536d6eddebe4e6c3cbc594f1bba94a on Async-Anchor+Checkbox into 5f163831ca312979181b5435f61e64bfc815d394 on master.

Coverage increased (+0.0001%) to 99.917% when pulling ca2513ae3b763405fcb6b3a14b4da899e0662854 on Async-Anchor+Checkbox into 5f163831ca312979181b5435f61e64bfc815d394 on master.


If I set the deepspeed_config.zero_stage: 3, how should I save my model ?

I used the following code but the saved model files missed the pytorch_model-00001-of-00003.bin
accelerator.wait_for_everyone()
unwrapped_model = accelerator.unwrap_model(model)
unwrapped_model.save_pretrained(save_dir, save_function=accelerator.save, state_dict=accelerator.get_state_dict(model))

Have you solve the problem? I get the same problem.

Have you solve the problem? I get the same problem.

I solved the problem by modifying the transformer source code. I remove the line of code that deletes model files in the save_pretrained function.

Thank you! I have solved the problem after removing the code.


Switch to CircleCI

Let's switch from Travis to Circle.
What this PR does:

Have artifacts per build / PR build stored in CircleCI, not as GitHub release, eg. https://circleci.com/gh/StefanScherer/image-builder-rpi64/3#artifacts/containers/0
Enable shellcheck again
Workflow for a release: Draft a GitHub release with a version number, CircleCI will trigger a fresh build and deploy it to GitHub.

Configuration of CircleCI

Add project to CircleCI builds
GITHUB_TOKEN secret is needed when a new (pre-)release should be deployed, but in the meantime we could just use the CircleCI artifacts to share the sd-card images while PR's are still open.

Great idea. I'm configuring the project to get build on CircleCI.


638: updated css gulp taks to include a task for compiling and copyin…

…g over style guide styles

Could you also add a sample SASS file and change the docs master template to include the new CSS along with the Techne CSS? What you have looks good and should work but I can't test it without creating my own.

done and done.


GH Collab (Ruby): Commit to an existing PR

Reduce the number of PR's created on GH Collab Repo
As a Developer
I want Modify the existing add full org member to a Terraform file functionally
so that rather than creating a new PR add the modified file to an existing PR.
Do not need to move tickets to blocked, apply labels, close and recreate new PR's when to many exist
Functional Requirements:

Modify the function where it checks if a PR exists already for the user to check if the Terraform file is in the PR, if the file is not in the PR, then commit the file to that PR.
Update the tests.
Apply change to AS Org GH Collab repo.

Acceptance Criteria:

Commits "Add user to terraform file" on existing branch rather than creating a new PR

Notes:

No longer applies. This functionality has been removed.


dashboard ui added

Task1
NavBar updated with dashboard button

Task 2
Dashboard UI created

@yatulearn I have completed the Dashboard UI, please look into it and if any change is required let me know.
I would be very happy if you merge this with level 2 , hacktoberfest, gssoc-ext lebel

your pr has conflicts kindly resolve them

@yatulearn conflicts are now resolved you can check

@yatulearn conflicts are now resolved you can check

ok

integrate the dashboard page with database/backend

@yatulearn there is no existing database in your code, How can i connect it to database

@yatulearn Just merge the PR and assign me the appropriate label (Level 2). You can assign the backend work to another contributor who might be able to assist with that part.

@yatulearn Are you active on your Repo?

@yatulearn why closed this PR without proper context, the contributor is waiting from last week to get it merge!


Fields for custom menu item do not appear when switching from one type to another

When editing a menu item and changing the type from something like a Static URL to a custom menu item the fields that have been defined in the getFields method don't appear.
Steps to reproduce:

Define a custom menu item with custom fields.
Go to the menu builder and add a new item with one of the default types (like Text or Static URL)
Edit the menu item
Change the type to the custom menu item type
Notice that the fields for the custom menu item type don't appear
Save the item, edit it again and notice that the fields now do appear

Hi! Thanks a ton for reporting this. :) There was indeed an issue with the fields not updating. I fixed it and released a new version, 5.0.1. Good luck!

Hi! Thanks a ton for reporting this. :) There was indeed an issue with the fields not updating. I fixed it and released a new version, 5.0.1. Good luck!

Thanks! Works great 👏🏻

Thanks! Works great 👏🏻


Remove some old CSS and fix selector

This PR removes some old CSS which does nothing, .tab-content has a .p-0 class for example.
It also fixes the .nav-tabs selector to only select elements in the custom nav-tabs bar, otherwise it also selects menu items in top_right_content blade file.
Maybe it's an idea to move the remaining CSS and the list.css contents to the Backstap package (https://github.com/DigitallyHappy/BackStrap/blob/master/src/scss/_custom.scss)
And use this files strictly for user overrides.

A new inspection was created.

The inspection completed: No new issues

Thank you for this @martijnb92, good catch.
These files need a clean up for sure, and yes, the idea is to move the code somewhere developer don't need to see it, and leave these files (like form.css) for developer custom rules.

Thanks guys! While testing this I stumbled unto a small bug kind of annoying bug. The border-bottom is still showing on :hover, even if the tab is already selected:

At first I thought it was caused by this PR, because I've never seen this before in Backpack. But... apparently it also shows up using the OLD css, but... in fewer cases. Which again is surprising - it's the first time I see this. Couldn't tell you what those cases are though, sometimes it works sometimes it doesn't... 😡  It looks like maybe it happens when you hover when the page isn't completely loaded... maybe?! But then sometimes it seems to happen when the page is fully loaded too... Which is... yeah... 😔
@promatik could you please take a look at this too? Since we're doing changes to this file, it's best to have all annoyances fixed in one change, so that when people publish the assets they get the best.

Good catch, that's caused by some other borders when the bottom border is hidden.
Will update this PR.

A new inspection was created.

The inspection completed: No new issues

Awesome. Thanks guys, merging!


[Bug] Return link resets filters in some case

Hello,
I dont know if its a bug or im doing sth wrong. But im having trouble with list view url with filters.
What ive develop is a button that if click it opens a new tab with the crud list view with a filter applied passing the query in the url.
Like this

If enter to the update view it show a return link to the list but on returning the filters are reseted.

Ive the persist filter option on true and this only happens if the user click on that button.
Filtering manually then going to the update view and click in the return link works well.
Any idea?

Hey @aberaldo once again. 🙃
Today was your day! 👍 Sorry it took so much time to came back here, but I am happy I finally did.
I've just submitted #4558 that address this issue and another issue I've found while working on this.
If you happen to test the PR, please let us know your feedback in the PR thread.
I am closing this for now. Thanks again for your patience 🙏
Cheers


Blank tiles at high zoom levels

At high zoom levels for many areas in the US, notably around the Smokey Mountains National Park, blank tiles are generated using the Terrain style.
This can be easily seen here on the maps.stamen.com site:
http://maps.stamen.com/terrain/#18/35.47397/-83.92101
Some tiles are correctly generated.

The issue was expired SSL certificates.  The certificates have been restored and tiles are flowing :-)

Thank you!
Where can I ask "help" type questions? (e.g. i'd like to put together a docker instance of the stack necessary to generate and server these classic terrain tiles ...)


[mechanics_io] time_of_birth broken for MoreauJeanDirectProjectionOSI

Probably just need to ensure the _M matrix is correctly sized in LinearOSNS, i.e., probably just need to reset _hasBeenUpdated at the right moment.
output:
step 502 < 2001
Traceback (most recent call last):
  File "cylindrical_stops.py", line 139, in <module>
    osi=Kernel.MoreauJeanDirectProjectionOSI,
  File "/home/sinclairs/.local/lib/python2.7/site-packages/siconos/io/mechanics_io.py", line 2544, in run
    log(simulation.computeOneStep, with_timer)()
  File "/home/sinclairs/.local/lib/python2.7/site-packages/siconos/io/mechanics_io.py", line 233, in silent
    fun(*args)
  File "/home/sinclairs/.local/lib/python2.7/site-packages/siconos/kernel.py", line 24270, in computeOneStep
    return _kernel.TimeStepping_computeOneStep(self)
Exception: Siconos Matrix Exception : SimpleMatrix::setBlock(row,col): row is out of range

relevant gdb backtrace:
#2  0x00007ffff1fcdf53 in SimpleMatrix::setBlock (this=0x14c5a20, row_min=0, col_min=0, m
=...) at /home/sinclairs/projects/siconos/kernel/src/utils/SiconosAlgebra/SimpleMatrixSet
Get.cpp:107
#3  0x00007ffff23ed31c in OSNSMatrixProjectOnConstraints::fill (this=0x14c59d0, indexSet=
..., update=false) at /home/sinclairs/projects/siconos/kernel/src/simulationTools/OSNSMat
rixProjectOnConstraints.cpp:140
#4  0x00007ffff23ebf32 in LinearOSNS::preCompute (this=0x12c5360, time=5.0200000000000005
) at /home/sinclairs/projects/siconos/kernel/src/simulationTools/LinearOSNS.cpp:739
#5  0x00007ffff23de810 in MLCP::preCompute (this=0x12c5360, time=5.0200000000000005) at /
home/sinclairs/projects/siconos/kernel/src/simulationTools/MLCP.cpp:142
#6  0x00007ffff23de881 in MLCP::compute (this=0x12c5360, time=5.0200000000000005) at /hom
e/sinclairs/projects/siconos/kernel/src/simulationTools/MLCP.cpp:153
#7  0x00007ffff2351ceb in Simulation::computeOneStepNSProblem (this=0x12c5bc0, Id=1) at /
home/sinclairs/projects/siconos/kernel/src/simulationTools/Simulation.cpp:356
#8  0x00007ffff2327633 in TimeSteppingDirectProjection::advanceToEvent (this=0x12c5bc0) a
t /home/sinclairs/projects/siconos/kernel/src/simulationTools/TimeSteppingDirectProjectio
n.cpp:209
#9  0x00007ffff23982d5 in TimeStepping::computeOneStep (this=0x12c5bc0) at /home/sinclair
s/projects/siconos/kernel/src/simulationTools/TimeStepping.cpp:363

I've been unable to reproduce this recently, even with slightly older versions. I'll close this, unless I stumble on it again and can make a repeatable test.


Remove deprecated WinGet package for October 2023 GDK

Remove deprecated WinGet package for October 2023 GDK
Microsoft Reviewers: Open in CodeFlow

URL: https://github.com/microsoft/GDK/archive/refs/tags/October_2023.zip
Status Code: 200
(Automated message - build 897)


fix #4065

Topic must exist before adding attachment (first topic.save()), but also must be saved again after adding attachments, too, else the attachment info will disappear into the void. Only topic creation was affected by this.
maybe there's a more elegant way, but I'm sure the referenced bug is here

Ahh, when seeing this code.. can you try the following: add autoupdate=False to the topic.save() two lines above and change your topic.save() into an inform_changed_data(topic)
If this works, please go this way.
(Just as a reference: Its the similar issue to #4055)
Thanks!

@FinnStutzenstein  I followed the spirit of your suggestion, please have another look if I guessed correctly.


Dynamic path parameters replacing of PathFilters

Description
Added a new method to PathFilters with a String UnaryOperator to allow the dynamic customization of replacing target value.
Motivation and Context
Resolves #1164
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have added tests to cover my changes.

That's a really nice feature! @ismail2ov Thanks for contributing it! 📣

@ismail2ov Can you take a look at the code coverage?

@ismail2ov Can you take a look at the code coverage?

@whiskeysierra Can you check where the problem is?
JaCoCo gives me 100% coverage for DynamicPathFilter

Line coverage yes, but there is an uncovered branch. If you run mvn clean verify locally and check target/site/jacoco/index.html afterwards you'll find the line.

Done

👍

🚀 Great, thank you for your contribution! 📣


SigHashType: add a method to error on non-standard hashtypes

Right now, any sighash type could be parsed without error, which matches
consensus rules. However most of them would be invalid by standardness,
so it's a bit footgun-y (even more so for pre-signed transactions
protocols for which standardness is critical).
This adds from_u32_standard(), which takes care to error if we are
passed an invalid-by-current-policy-rules SIGHASH type.
(Happy to bikeshed the method name)

We should note that this being applicable only to pre-tapscript ECDSA signatures and not for Schnorr Sigs

Yes, as these rules actually become Consensus rules with BIP341.

I think we may be mixing two things here. The sighash appears in two contexts when dealing with bitcoin transactions.

It is serialized as 4 bytes u32 when used as a signature message in the ECDSA sig algorithm
It is appended as a single byte when it is appended to the ECDSA sig to form a bitcoin signature.

I think what we want is SighashType from u8?

I think what we want is SighashType from u8?

I believe so (see #rust-bitcoin), but it's a bit of a mess on bitcoind itself:

You can't have a >1byte SIGHASH type
But it's then treated as a signed integer
It's also serialized as an int32_t in the sighash

I believe @apoelstra wanted to mimic Bitcoin-Core behaviour in 38b2cacf

Ok, i think i either responded or adressed all the review comments. Thanks everyone!
Added another documentation commit after a nice deep dive with @sanket1729 about the rationale of masking with 0x9f. This was apparently meant to be an extension mechanism in a precedent era of Bitcoin, at least from what we could get off the source code fossils :)


Example problems cannot be loaded in built JAR / JRE

After building the project to a JAR + JRE, the examples from the resources folder cannot be found, leading to a fatal exception

See https://github.com/ProvideQ/toolbox-server/actions/runs/6340698758/job/17222888044 for an example


ImageView changes aren't reflected in screenshot

Setting ImageView.setDrawable isn't reflected in the screenshot. Here's a test that reproduces the issue. The test updates the image view's drawable to a red box, but the resulting screenshot is empty. Interestingly, using imageView.drawToBitmap renders the red square correctly.
Also is it possible to change the size of the screenshot? Thanks!

Thanks. Perhaps it is because Roborazzi only sees the layout changes.
https://github.com/takahirom/roborazzi/blob/main/roborazzi/src/main/java/com/github/takahirom/roborazzi/Roborazzi.kt#L195
I'll see what I can do about it, but for now it might be a good idea to have it taken with captureRoboImage().
As for image size, I don't think there is a workaround at this time, so I will consider it as soon as possible.
If you don't mind, could you please create an issue on image size?

Using captureRoboImage works great, thanks!

Thanks for your feedback.
I'm working on resize issue 💪
https://github.com/takahirom/roborazzi/pull/13


Update docs to coincide with Github transition to visallo (3.1)

See #1011

Will redo these changes after major documentation update from @jharwig


[graphiql] [security vulnerability] GraphiQL leaking query in the URL

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
When using GraphiQL, the whole query and content updates the URL to reflect the query in the URL query parameter ?query=.

If someone develops or debugs with sensitive data like passwords, keys,... these are then effectively leaked in the:

Browsers History
Network logs (which VPN's may have access to)

Expected Behavior
GraphiQL should not reflect the query in the URL, or make it an option disabled by default.
Steps To Reproduce

Open GraphiQL
Write any query
Notice that the URL reflects the query and its contents

Environment

GraphiQL Version: 3.4.1

Anything else?
No response

@larscyl it is disabled by default, only enabled in our demos

@acao thanks for your swift response! Can you point out which option this is? I can't seem to locate it in the config API.
I'm using graphql-yoga/node, so I'll create a ticket over there thereafter.


Problem with .vdex Poco F1 (Xiaomi)

I have of the problems with these files:
Files.zip
Is possible fix this problem?
Thanks

What is the problem? I don't see any issues when decompiling/disassembling the files you've provided. Also be aware that the Vdex containers you've provided are of version 010 (probably from an Oreo ROM).

Yes is oreo 8.1 the problem is lastversion vdex 0.52? i try with old version?

I've tested against the current master. You should be ok with the latest version. What is exactly the error you get?

Sorry is problem of my script because these apk has classes.dex and classes2.dex
I have fixed.
Sorry for disturb
Thanks for your reply


Support for Box Plots?

The statistics plot options are currently pretty limited. Box plots could change that.
https://www.isixsigma.com/wp-content/uploads/2019/07/Box-Plot.gif
^Example in case you aren't familiar

@Benny121221 this is an excellent idea!
https://en.wikipedia.org/wiki/Box_plot
The PlottableBar could be modified to create a PlottableBox
https://github.com/swharden/ScottPlot/tree/master/cookbook#plot-bar-data
We could also add tools to the statistics module to turn a double arrays into the various box components. One thing to keep in mind is that many people use boxes to show quartiles, but I often use boxes to show standard error, so we should keep it flexible.

#246

I created ScottPlot.Statistics.PopulationStats()
to make it easy to get min, max, mean, median, standard deviation, standard error, etc.
https://github.com/swharden/ScottPlot/blob/ade0446036fb4d814d49d4d4aff8b938b2e9ce81/src/ScottPlot/Statistics/PopulationStats.cs#L8-L58
I created ScottPlot.Statistics.BoxAndWhisker
it is similar to OHLC. You're intended to build them and pass an array of these into Plot functions
https://github.com/swharden/ScottPlot/blob/ade0446036fb4d814d49d4d4aff8b938b2e9ce81/src/ScottPlot/Statistics/BoxAndWhisker.cs#L7-L75
You can build a BoxAndWhisker like this
https://github.com/swharden/ScottPlot/blob/ade0446036fb4d814d49d4d4aff8b938b2e9ce81/demos/WinForms/Demos/FormBoxWhisker.cs#L20-L32
You can plot BoxAndWhiskers[] like this
https://github.com/swharden/ScottPlot/blob/ade0446036fb4d814d49d4d4aff8b938b2e9ce81/demos/WinForms/Demos/FormBoxWhisker.cs#L82-L97

@Benny121221 did an awesome job with #246! I modified the interactive demo to showcase both of our plottables. I'm impressed there were no merge conflicts even though we both did the same thing at the same time 😎

Using Scott's Version
https://github.com/swharden/ScottPlot/blob/aad44762d282d24d6d7e49e74ccf9c04c3f23f40/demos/WinForms/Demos/FormBoxWhisker.cs#L58-L76
Using Benny's Version
https://github.com/swharden/ScottPlot/blob/aad44762d282d24d6d7e49e74ccf9c04c3f23f40/demos/WinForms/Demos/FormBoxWhisker.cs#L27-L42
Comparison
I think the biggest difference in our implementations is @Benny121221's does math and mine doesn't (my math was extracted to a population stats module, and even a separate box and whisker builder). Since box and whiskers aren't always quartiles (I've seen them used to show limits and stdev or stderr) there might be an advantage to keeping calculations out of the plottable 🤔
I think the final version should accept double[] values to let the user plot individual values over (or beside) the box plot.

When using boxwhisker plots to show quartiles, it is common to extend the whiskers only to 1.5 * IQR, and to plot any points outside of that as either dots or asterisks. This 1.5 * IQR rule is a simple rule for detecting outliers.
It looks like your code draws the whiskers out to the most extreme point, which is another common way of drawing these plots. I think this should probably make its way into an argument like bool plotOutliersIndividually or something like that

When using boxwhisker plots to show quartiles, it is common to extend the whiskers only to 1.5 * IQR, and to plot any points outside of that as either dots or asterisks. This 1.5 * IQR rule is a simple rule for detecting outliers.

@Benny121221 I'd really like to use the math you cane up with here:
https://github.com/swharden/ScottPlot/blob/aad44762d282d24d6d7e49e74ccf9c04c3f23f40/src/ScottPlot/plottables/PlottableBoxAndWhiskerV2.cs#L79-L90
What do you think about moving it into the population stats module?
https://github.com/swharden/ScottPlot/blob/aad44762d282d24d6d7e49e74ccf9c04c3f23f40/src/ScottPlot/Statistics/PopulationStats.cs#L8-L58

It looks like your code draws the whiskers out to the most extreme point, which is another common way of drawing these plots. I think this should probably make its way into an argument like bool plotOutliersIndividually or something like that

What about having multiple box plot builders? Something like the following function but one which returns the proper values. This would could become GetBoxAndWhisker_LimitsAndDeviation and the other one could be GetBoxAndWhisker_OutliersAndQuartiles or something.
https://github.com/swharden/ScottPlot/blob/aad44762d282d24d6d7e49e74ccf9c04c3f23f40/demos/WinForms/Demos/FormBoxWhisker.cs#L44-L56
If you want to have a go at it I'll stop making changes for a bit to avoid merge conflicts. Thanks again for all your work on this! I'm looking forward to your thoughts here

Yeah sure I'll have a crack at it

Yeah sure I'll have a crack at it

Sounds great! I look forward to seeing what you come up with!
Right now that method is in FormBoxWhisker.cs... but it seems like code that builds BoxAndWhisker objects from double[] data might be good static methods in a ScottPlot.Statistics.BoxAndWhiskerFactory static class 🤔

#247

I consolidated a lot of the discussion/code from this issue into one place. However, the styling still needs a lot of work. I intentionally have not placed user-facing styling arguments in the Plot module while I work to refactor it some more. I'll probably work on some other issues, then loop back around to this one.
One thing to note is that I created a ScottPlot.Demos module which contains plots that can be accessed from within the WinForms and WPF demos.
https://github.com/swharden/ScottPlot/blob/35e792dd25944b989dd5f6736ebd5447d100e959/src/ScottPlot/Demo/PlotTypes/BoxAndWhisker.cs#L11-L24

I'll close this issue and track progress on box plot refactoring in #315

following up, ScottPlot 4.0.25 has some new related features. Check out the cookbooks to see what's possible now:
Bar Plot
This is a relatively simple plot type. It's just for bar plots, and it does not display population details (like scatter plots of individual data points).
http://swharden.com/scottplot/cookbooks/4.0.25/#PlotTypes_Bar_Quickstart
Box and Whisker Plot
This has extensive options, including displaying as bar plots, and rendering population data points next to the box/bar plot.
http://swharden.com/scottplot/cookbooks/4.0.25/#PlotTypes_Populations_PlotPopulationSeriesUnique


Add hexo-theme-aircv theme

Change name from aircv-hexo-theme-aircv
Check List
Please read and check followings before submitting a PR.

[x] I want to publish my theme on Hexo official website.

[x] I have read the theme publishing doc.
[x] name is unique.
[x] link URL is correct.
[x] preview URL is correct.
[x] preview URL web site is rendered correctly.
[x] Add a screenshot to source/themes/screenshots.
[x] Screenshot filename is same as value of name.
[x] Screenshot size is 800 * 500.
[x] Screenshot file format is png.

[ ] I want to publish my plugin on Hexo official website.

[ ] I have read the plugin publishing doc.
[ ] name is unique.
[ ] link URL is correct.

[ ] Others (Update, fix, translation, etc...)

Languages:
[ ] en English
[ ] ko Korean
[ ] pt-br Brazilian Portuguese
[ ] ru Russian
[ ] th Thai
[ ] zh-cn simplified Chinese
[ ] zh-tw traditional Chinese

Hi Sukka.
Ok I will change for this name. At first I didn’t want to use same name, because template is a third party designer. But I got his permission to adapt It to hexo.
Question:
Could I change theme name and keep name in GitHub repository?

El 18-05-2020, a las 12:08, Sukka<EMAIL_ADDRESS>escribió:
@SukkaW requested changes on this pull request.
In source/_data/themes.yml https://github.com/hexojs/site/pull/1422#discussion_r426738407:

@@ -1,8 +1,22 @@
+- name: Hexo theme aircv
⬇ Suggested change
-- name: Hexo theme aircv
+- name: aircv
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/hexojs/site/pull/1422#pullrequestreview-413738930, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADCA4CL5LWFZSDMFTNH5DZDRSFMQDANCNFSM4M6JYZ5Q.

Could I change theme name and keep name in GitHub repository?

The name of GitHub repo doesn't matter.

Name updated from "hexo-theme-aircv" to "Aricv"


fix: no buttons for blob store links

Media links mistakenly get deocrated as buttons, resulting in a wrong floating CTA.
Test URLs:

Before: https://main--express-website--adobe.hlx.page/tw/express/create/flyer
After: https://v7-regression--express-website--adobe.hlx.page/tw/express/create/flyer?lighthouse=on

Background: the original check to exclude media links was:
      && originalHref !== $a.textContent

Since pipeline v7 media links get rewritten to relative links, so the check was changed to
      && !(linkText.startsWith('https') && linkText.includes('/media_'))

But that didn't take into account that some older pages still contain blob store links...


Writing variable values to file using a script

I wonder if it is possible to write data from a variable to a file using scripts. I tried the following command, but got an 'ERROR: illegal script command'.
pigs proc ld v0 48 fo /tmp/foo 10 sta v100 fw v100 v0 fc v100

It seems that the fw-command does not accept variables neither for the handle nor for the data.
What I tried to do in the example:

ld v0 48 - Save the value 48 (ASCII '0') in the variable v0
fo /tmp/foo 10 - create a new file
sta v100 - store the file handle in the variable v100
fw v100 v0 - write the data of v0 using the handle v100
fc v100 - close handle v100

I'm afraid (as you found) that will not work.
Not all commands work within scripts.
When I get time I will add a list of the commands which will not work to the pigpio documentation.


Adding it to the DaCy robustness test

Hi @sorenmulli and @asgerius really enjoyed reading your work. There are some great points to be taken from it. For example, inspired by your model evaluation we will try to add plank and wikiANN to the robustness tests as out of distribution datasets. I am also interested in how retraining this model in a multitask setting would perform 🧐
I saw your issue on adding daluke to the DaNLP benchmark and would very much like to add daluke to the bias and robustness evaluation in DaCy framework. Feel free to reach out if this is something you are interested in.
PS. the medium dacy model is actually the Danish BERT by botxo, not the multilingual Roberta ;)

Hello! Thank you very much for your interest in the DaLUKE project - we will be continue development and analysis of the model and are thus very excited to see this evaluation method.
To make robustness tests run, it seems you guys need to make an apply function for DaLUKE which seems to require us adding the same API capability as described in #103.
Furthermore, we found a bug in pre-training with omission of [SEP] token and are in the process of rerunning it. We will let you know when these efforts are done which might be impacted a bit by summer holiday ;-)
Let us know if more is needed and if you have any questions to the methods such as using Plank and WikiANN.
And; sorry for misrepresenting DaCy medium - we only discovered your awesome project late in the bachelor and it seems that we were a bit superficial in our reading.
We are back from vacations in August and would very much like to hear your thoughts on further work on this project such as multitask learning + general Danish NER input, so, if you are interested, we'll shoot you a mail about a possible meeting.

exactly if you reach out when you have finished #103 then I will make sure to add it (or if you want to be a contributor to DaCy feel free to open a PR).
No problem, I understand that a bachelor thesis can hard-pressed for time.
Feel free to shoot me a mail at kenneth.enevoldsen (at) cas.au.dk!


[Feature] Make PPO compatible with composite actions and log-probs

Stack from ghstack (oldest at bottom):

-> #2665

In this PR, I propose to let PPO have series of actions defined in the in-keys (rather than a single one) to accomodate CompositeDistributions better.
This PR requires https://github.com/pytorch/tensordict/pull/1146 and https://github.com/pytorch/tensordict/pull/1145 to be merged or checked out.
Here is a demo:
https://gist.github.com/vmoens/46175764240dcbaf311af562b9e53294
cc @matteobettini

Cool!
Just to understand a bit, how is this related to multiagent?
I see in the example that you are using different agent groups, but the feature seems to be more suited for composite single-agent actions.
In multiagent, the suggested way to do things was to create a different loss for each group. This is to avoid losses taking a list of dones, rewards, and actions and have to match them.
I think this feature for me makes sense for composite actions within a single-agent or a single marl group (avoiding taking a list of rewards and dones).

Also in the example you are using a single module to output actions for multiple groups.
I think also here the way we suggest to do things is to process different groups in different modules, so that each module can go to its loss.

I don't have a strong feeling RE multiagent or not, the use case that was suggested to me here had a composite action space where each leaf was labelled "agent_x"
I guess that long term there's a version of this where you could have one loss for all, since tensordict now supports arithmetic ops you could perfectly do
log_prob = make_some_tensordict(...)
prev_log_prob = make_some_tensordict(...)
advantage = make_a_tensordict_or_a_tensor(...)
loss = (log_prob - prev_log_prob).exp().clamp(...) * advantage

and your loss will be a tensordict itself.
This would probably break now but I do think we could actually get this to work and simplify the code at the same time (that will require deprecating some default behaviours in CompositeDistribution in v0.9 as announed in tensordict)

This makes sense for a composite action space yes. But in your PR i see you are also allowing lists of dones and rewards.
This is a bit less trivial as it opens up to a bunch of compatibility usecases if you want to use this in MARL.
The done and reward keys might not be a one to one mapping to actions:

groups can have composite actions
reward and done could be partially or totally shared across groups

Supporting all these usecases might become a big headacke which is why I preferred to stick with one reward and done key per loss class.


refactor(events): simplify icons code

Description
refactored getIcon() to keep the code tight

Coverage remained the same at 42.316% when pulling d1bd928b9f46774e10130f32e9b578c1b5fe902c on machour:refactor/events-icons into a49b4310e13386c3f422756375f327516563e8c1 on gitpoint:master.


do not show unread messages banner

resolves:  https://github.com/WideChat/Rocket.Chat/issues/478

@Shailesh351 or @Kailash0311 can one of you guys take a look at this PR please?  It's very small, and I just took a stab at doing it.  Let me know if this is an ok way to do it, or if there is a better method.  Thanks.

@Shailesh351 cool.....  thanks.


Create LICENSE

Added in https://github.com/jspm/nodelibs-tls/commit/309759ed050a9db5c6ba0ac6e4f7ab21d446a196.


Proposal on passing configuration

To avoid computing manifest, canvas, and annotation URLs by manipulating window.location attributes, there should be explicit ways to pass this information to the Mirador viewer config:

The existing m and c search params are a good existing method

This could be extended with an a parameter for passing the annotation URL

Relying on HTML data- attributes, on the root element with the id under which Mirador is mounted, e.g.:

<div
  id="m3-dist"
  data-manifest="https://zenodo.org/api/iiif/record:1234/manifest"
  data-canvas="https://zenodo.org/api/iiif/record:1234/canvas/page-01.png"
  data-annotations="https://zenodo.org/api/records/1234/media-files/annotations.json/content"
</div>

Another alternative would be to export JS function for wrapping the initialization, to also for passing config overrides, e.g.:

function initMirador({ id, manifest, canvas, annotations, config }) {
  const config = {
    id: id,
    windows: [{
      manifestId: manifest,
      canvasId: canvas,
    }],
    // ...default config...

    // allow also to override default config
    ...(config || {}),
  };

  Mirador.viewer(config);
}

Hi Alex,
Thanks for this - the window.location kludge should definietly be considered a placeholder.
I think the data-attributes is probably the tidiest soloution, although I'd like to keep the URL params for some legacy things (despite the risk of the URL getting too long). But in my view your proposed data-attribute solution should be the "primary" configuration method, as we can use it flexibly to pass additional config variables (and in some cases maybe even a whole Mirador config).
Look forward to talking more about this later!
(pinging @peter-cornwell)


Hygieia dashboard to show jenkins multiple job build result

As of now am able to see single job build results in "build per day" dashboard. But how to configure hygieia dashboard to show multiple build jobs here.

The idea is to create a set of team/component dashboard where a component has a code repo, a build job etc. And then create a product dashboard where the component dashboards can be added. The product dashboard then will show the pipeline (code -> build -> deploy) of all components in 1 single view.

Am okey to close this issue.

Closed per comment above.


Have two prefixes when Skywalking oap delete expired data

Please answer these questions before submitting your issue.

Why do you submit this issue?
[ ] Question or discussion
[√ ] Bug
[ ] Requirement
[ ] Feature or performance improvement

Question
Why do I get two prefixes when Skywalking oap delete expired data?

Bug

Which version of SkyWalking, OS and JRE?
SkyWalking v6.2.0

Which company or project?

What happen?
Skywalking oap log error:

config/application.yml:

index in es:

Requirement or improvement

Please describe about your requirements or improvement suggestions.

Fixed in 6.3.0