Migration Library cannot load migration script module

Expected Behavior
Migration Script runs provided demo code and creates content type dog with field name of type Symbol.
Actual Behavior
Script errors out with the following message:
The migration.js script could not be parsed, as it seems to contain syntax errors.

Error: Cannot find module 'migration.js'
Require stack:
- /mnt/c/dev/contentful/migration/node_modules/contentful-migration/built/bin/cli.js
- /mnt/c/dev/contentful/migration/node_modules/contentful-migration/built/index.js
- /mnt/c/dev/contentful/migration/index.js
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:976:15)
    at Function.Module._load (internal/modules/cjs/loader.js:859:27)
    at Module.require (internal/modules/cjs/loader.js:1036:19)
    at require (internal/modules/cjs/helpers.js:72:18)
    at run (/mnt/c/dev/contentful/migration/node_modules/contentful-migration/built/bin/cli.js:50:29)
    at Object.<anonymous> (/mnt/c/dev/contentful/migration/index.js:11:1)
    at Module._compile (internal/modules/cjs/loader.js:1147:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1167:10)
    at Module.load (internal/modules/cjs/loader.js:996:32)
    at Function.Module._load (internal/modules/cjs/loader.js:896:14)
    at Function.executeUserEntryPoint [as runMain] (internal/modules/run_main.js:71:12)
    at internal/main/run_main_module.js:17:47 {
  code: 'MODULE_NOT_FOUND',
  requireStack: [
    '/mnt/c/dev/contentful/migration/node_modules/contentful-migration/built/bin/cli.js',
    '/mnt/c/dev/contentful/migration/node_modules/contentful-migration/built/index.js',
    '/mnt/c/dev/contentful/migration/index.js'
  ]
}

Steps to Reproduce
Followed steps on this projects README file:

npm install contentful-migration
add index.js script with:

const { runMigration } = require('contentful-migration')

const options = {
  accessToken: '<my-content-management-api-token>',
  spaceId: '<my-space-id>',
  environmentId: '<my-environment-id>',
  yes: true,
  filePath: 'migration.js'
}

runMigration(options)
  .then(() => console.log('Migration Done!'))
  .catch((e) => console.error(e));

add migration.js script with:

module.exports = function (migration, context) {
  const dog = migration.createContentType('dog');
  const name = dog.createField('name');
  name.type('Symbol').required(true);
};

run script with: node index.js

Context
I'm trying to use the migration library to build a script that allows me to merge changes done in one environment into the production environment. This is nothing fancy and it's really what the library was made for. It's frustating to be stopped at the very first step of this project which is running the basic demo code provided.
The scripts don't seem to have any syntax errors (run several linters both on my IDE and manually).
the migration script is in the same folder as the index script so it should be findable (I've tried ./ and / to the name just in case but the result is the same).
I've also tried different versions of NodeJS from 8 to 14 in case there was any issues with breaking chances. The result is the same as above.
At this point I'm out of solutions as to what may cause the issue and hence this issue.
Environment

Node Version: v14.3.0
Package Manager Version: 6.14.5
Operating System:  Linux KORIOLIS 4.4.0-18362-Microsoft #836-Microsoft Mon May 05 16:04:00 PST 2020 x86_64 x86_64 x86_64 GNU/Linux
(Windows 10 running WSL Ubuntu distro)
Package Version: 1.1.3

So after digging around the library source code figured out that a workaround for this issue is to pass an absolute filepath instead of relative. So using path it would be something like:
filePath: path.join(__dirname, 'migration.js')

If this is the general case you can possibly warn in the documentation that filePath needs an absolute path?


Implement Dynamic Favicon Theming and Optimize SVG for Light Mode

Purpose:
This pull request introduces dynamic favicon theming to enhance user experience by automatically adjusting to the system's light or dark mode. It also optimizes SVG assets for better visibility in light mode.
Changes:
1. FaviconUpdate Component:

Switches favicon based on light/dark mode.
Uses MediaQueryList to detect and respond to color scheme changes.
Adds event listeners to update the favicon dynamically.

2. Light Mode SVG Asset:

Adds a new SVG asset optimized for light mode with black stroke colors for better visibility.

3. App Integration:

Integrates FaviconUpdate component into App.tsx for dynamic favicon theming.

4. SVG Styling Adjustments:

Adjusts SVG color and fill-opacity for improved visibility in light mode.

Testing:

Verified favicon switches correctly between light and dark modes.
Checked SVG visibility in both light and dark modes.

Good work! Did not know the SVG wasn't working as intended on light mode. Your fix is good, However you added functionality for changing the tab icon conditionally. This is not part of my vision for this app


Adds build script for building xcframework

Adds a script and modifications required to build this project as an XCFramework

Thank you so much!


[Workers] Remove Workers notification

It appears the feature no longer exists

I wasn't able to work off of this branch due to the fork. I've created a new PR: https://github.com/cloudflare/cloudflare-docs/pull/8863

Closing in favor of https://github.com/cloudflare/cloudflare-docs/pull/8863


[BUG] don't work on iOS 12 and 11

Describe the bug
Same code tested on iOS: 14 (work), 13 (work), 12 (DON'T work), 11 (DON'T work)
HTTP Requests do not work on iOS 12 and 11
To Reproduce
Steps to reproduce the behavior:
Start you App under iOS 11 or 12 and make HTTP Request
Expected behavior
HTTP Requests work on all iOS versions

LOG:
2021-09-22 14:21:32.260648+0200 App[60074:1373830] TIC Read Status [2:0x600002309d40]: 1:57
2021-09-22 14:21:32.261020+0200 App[60074:1373830] TIC Read Status [2:0x600002309d40]: 1:57
2021-09-22 14:21:32.261200+0200 App[60074:1373830] TIC Read Status [2:0x600002309d40]: 1:57
2021-09-22 14:21:32.266288+0200 App[60074:1373830] [BoringSSL] nw_protocol_boringssl_get_output_frames(1301) [C2.1:2][0x7fb2144162b0] get output frames failed, state 8196
2021-09-22 14:21:32.266403+0200 App[60074:1373830] [BoringSSL] nw_protocol_boringssl_get_output_frames(1301) [C2.1:2][0x7fb2144162b0] get output frames failed, state 8196
2021-09-22 14:21:32.267213+0200 App[60074:1373830] TIC Read Status [2:0x0]: 1:57
2021-09-22 14:21:32.267346+0200 App[60074:1373830] TIC Read Status [2:0x0]: 1:57
2021-09-22 14:21:32.267534+0200 App[60074:1373830] TIC Read Status [2:0x0]: 1:57
2021-09-22 14:21:32.267605+0200 App[60074:1373830] TIC Read Status [2:0x0]: 1:57

This plugin uses APIs from iOS and WebKit behaviour that are not available on older iOS versions unfortunately. There is probably no way to make this plugin work on older iOS versions.

If this plugin just don't change the URL in the method window.WebviewProxy.convertProxyUrl on iOS 12, 11..., it will work.

This is a good idea. I created a new issue for this. Pull requests welcome.


No more monkey patching

I've added some information debugger-xml needs from byebug to remove monkey-patching from debugger-xml.
I've made a small change in var.rb which will not allow us to remove monkey-patching completely because we do provide additional sub-commands and replace couple methods (see https://github.com/astashov/debugger-xml/blob/master/lib/byebug/commands/variables.rb)  Any ideas how to improve that is welcome.
Also debugger-xml monkey-patches several commands to make them "unsupported" (e.g. https://github.com/astashov/debugger-xml/blob/master/lib/byebug/commands/help.rb)
Again any ideas how to fix this w/o monkey-patching are welcome ;)

Thanks Oleg. I just added one inline comment.
Regarding removing monkeypatches. I've addressed the first issue with subcommands, I'll push the changes soon. I'm still thinking of the best way to support disabling commands. Maybe a whitelist of supported commands in the printer?

I'll push the changes soon. I'm still thinking of the best way to support disabling commands. Maybe a whitelist of supported commands in the printer?

Well, I'd prefer to have some API in Byebug class for this.  E.g. modifiable list of commands or something similar.  Imho, printer is a wrong place for filtering commands.

Yep, I agree. But the idea of a whitelist works?

But the idea of a whitelist works?

do you mean if it is ok to be able to whitelist/blacklist some commands?
If so, then yes, I think it is.

do you mean if it is ok to be able to whitelist/blacklist some commands?
If so, then yes, I think it is.

Sorry I didn't answer this. Yes, I meant that plugins should provided a whitelist of the core byebug commands they want enabled.
I don't currently have time to work on this, just wanted to let you know in case you are waiting for me to move this forward.
Regards.

Sorry I didn't answer this. Yes, I meant that plugins should provided a whitelist of the core byebug commands they want enabled.

Current approach debugger-xml uses is more like black-listing some commands most likely because most of commands you provide are useful for vim integration (for IDE we do not need to blacklist anything because user has not access to the debugger's commands ;)
Thus if/when you will have time for this I'd vote for blacklisting.

Got it, makes sense.


[FIX] miniupnpc compilation issue

This fixes the compilation issue with various miniupnpc version across
various systems. The patch is adapted from what was used in the Monero
project here: https://github.com/monero-project/monero/pull/422/files

A fix has been merged at last 👍


aci: perform ExtractTar in chroot

This is a bit hackish but performs the extraction in a chrooted
thread after unshare(CLONE_FS).
Go doesn't provide us with enough control over the OS threads to do
this elegantly, see comments in the code.
Fixes #904

/cc @sgotti thoughts/ideas on this?

I can't think of a better golang approach for an in process chroot than the @vcaputo one. This is going to leak n thread (the number of images counting also the dependencies) and has all the possible problems described in the patch.
An external process approach looks, IMHO, more clean and less hackish (I've a little prototype quite ready that uses and additional unix pipes to pass the FilesMap to extract, and stdin for the tar data).
There's the problem that having another binary introduces the need to find its location during tests or when packaged (and naming it with a non colliding name). Perhaps an approach where we can execute the same rkt binary passing as argv[0] a command like "chroottarextract" will be better? (like done by various shells and something similar is done by docker pkg/reexec).
P.S. Using chroot needs the tests to be executed with a user having CAP_SYS_CHROOT or setting the capability on the executable. Maybe someone (like me) is executing the tests with a normal user...

Capturing OOB discussion: Go Is Not Great, so @vcaputo is going to pursue the fork/exec solution.

@jonboulle @vcaputo I made something this evening to test my proposed idea. I pushed it now (also to check if the functional tests works with it). Feel free to discard it if you're already working on it and have different ideas! Thanks!

I have to say this approach is interesting.

@sgotti I've commented on our PR, we'll just iterate on yours, it's fairly similar to what I had..


Code example doesn't work

I copied the code example from the repo, and it doesn't work for me.
First of all there's a typo, there's a comma missing at the end of the line with lambdasSourcePath.
After fixing that, I get the error:
Argument of type '{ serviceName: string; openApiSpec: string; lambdasSourcePath: string; integrations: ({ operationId: string; handler: string; } | { operationId: string; handler: string; timeoutSeconds: number; memorySize: number; env: { ...; }; })[]; }' is not assignable to parameter of type 'HttpApiProps'.
Object literal may only specify known properties, and 'serviceName' does not exist in type 'HttpApiProps'.ts(2345)
Node version: 16.18.1
cdk version:  2.50.0

Thanks @anebz for your feedback.
I'm gonna look into it ASAP and provide a fix. 🔨
Daniel

Hey @anebz ! I've included more examples in the README for how to:

define an API + 1 lambda for each route
enable custom domain for API
configure '*'CORS
configure lambda customer authoriser


Move the UI window style to a top to bottom layout

This is part of {wip} side project i had to make the ui more
user friendly when run on mobile i.e in termux or other linux mobile environtments
that can show our qt ui.

This seems very undesirable on desktops... Shouldn't it be an option, or at least based on screen ratio?

I fiddled around with long v3 Tor and i2p addresses and they where truncated way to much and even on desktop, i guess that could make sense to have this layout for ppl, to cherrypick on top of there compile of core if testing v3 or other things when you want multiple nodes side by side.
And since it make no sense to use the pre-builds, your are right, its an option, for sovereign multiplatform coiners.

be an option, or at least based on screen ratio?

i.e../configure --gui-top-bottom-design ?

I was thinking a checkbox in the GUI settings at runtime.

I was thinking a checkbox in the GUI settings at runtime.

even better, but since the ui is fixed and the headers are created at compile time.
I am not quite sure how to do that? Two layouts ? ping @hebasto

I was thinking a checkbox in the GUI settings at runtime.

even better, but since the ui is fixed and the headers are created at compile time.
I am not quite sure how to do that? Two layouts ? ping @hebasto

Probably building layouts dynamically rather using *.ui forms is a way, no?

Or modifying the default layout in code should also be possible.

I believe that the over all interface should maintain a "Landscape" layout - This approach seems to be forcing a "Portrait" layout...which isn't optimal.
Either way - optimizing the use of the empty space may guide layout decisions...
For example:

Notice how much empty space can be utilized in other ways...
By formatting the layout of the text - this extra space can be used by the peer detail widget...
Some simple changes could be in the NodeID column - this could be simplified to ID
I don't think sorting by NodeID is particularly useful and could be disabled for better display of the label.
Also removing the slashes from the User Agent display would look better..

The layout should depend on the screensize ratio (or on the window-size ratio). I agree that Core should also have an option to run on rather portrait mode screens and devices.
Ideally, the design-implementation can handle run-time window size changes and switches from landscape to top-down mode. Similar to how webpages have a floating design.
If too complicated, I agree with @luke-jr about the checkbox-in-settings approach.
A compile option seems not ideal to me.

Agree with @luke-jr - this should not be hardcoded but instead based on screen ratio - or better still, user-overridable.

The flowing GUI could be implemented in https://github.com/bitcoin-core/gui-qml/blob/main/src/qml/README.md
Closing this for now.


Ctrl + W Conflict With Nano Editor in Bash/WSL (not an issue in normal WSL or CMD)

Nano is a popular text editor in the bash/Linux environment but this uses Ctrl + W (for where) to initiate a search (same command Windows uses for closing the active window/tab) so it is a bit unexpected when this command closes the session down leaving the open file locked for editing.
Should Ctrl + W close the tab or should it be something else?
So this is sort of an issue although not exactly a bug as this functionality is obviously intended but it just conflicts with the Nano command and the standard CMD and WSL windows instead treat Ctrl + W as input so do not have this issue/conflict.

I think our goal is to fix this with #537

Indeed. We'll close this one in favor of rebindable hotkeys.

Ok thanks fair enough but I think maybe it would be good if the default mapping was to leave this key combination unmapped so it functions the same as CMD and WSL do.


v-memo/v-once child components rerendering

Vue version
3.3.0+
Link to minimal reproduction
https://play.vuejs.org/#eNq1Vslu2zAQ/RVCPThBIilFenIdI22QQwo0DZr2VPWgiGNbCReBpBwXhv+9Q1KbZVtwgcYXi7O+N0MOuQ4+FUW0LCEYBxOdqbwwRIMpC8JSMb9KAqOTYJqInBdSGbImCmZkQ2ZKcjJCt1FHVWq4kaUwjT6KM7sefWyMHlIFaFCr/dKm79jcy0cmWxu/rG0SkUmhbTYX+pyUBU0NYMqrJv/JKanNOHB5R1GHsE8uThMxiT1JpIQLA7xg6O5WhExovpx+RReS0zFZr2v3zWYSW1XHymV3Nu5ry6QxCsOw70iWoQ2Khf3lg//G8lqCBihxqavIk6pU8bQOMRCgKpkVNP6V7Ch/YMBtsm6AvdQOExsksSfnDrAhEsf4vzkxKTLYJWalh6hseTRUnHQPeitvAHdd/xFwHWoI96H0A8D/B+Kn0hgpyHXG8uwFO+UPL/bppz/FzmkSezOfqe+Ch/tqWh3Ms7PW1XU4px1ne9ibAx6cB368hDwtomctBQ68tc2QVAqccwjcSqwMx41dJ8HCmEKP4zijAt0osHypIgEmFgWPr9EsVog55xBSya8vow/RRZiyYpFG75G5Nl11BJqHT0q+alAYLAnOO+liFC5BhdgfCgrUsel7bvsg9Ex2YFgUm0RssEhG4+Cc5fNeiTLJi5yB+laYHAfrVqlSxuTrFyczqoSGU7aA7GWP/FmvPLcHBQ5Zpw4mVXMwXn37eA8r/G6UXNKSVW05oPwOWrLSYvRmn0tBEXbHzqG9cx3PxfyHvl0ZELomZYG6ajh715ebAeot3MvoslPF9l477lq9WeSMNjeeW/UuxeHbtXcr7t6GAxcfnjGX0H3iolaRd7BK8dPb2F/nVFe27flyceKsCrR98Bo+x74xKMxyAXbq6N5bI8Ed3Ogmrg8VypNCSTwp6M31fEy0Udhesjkdk1T8wa5UA+FwEbQdciLlgICqmIgKPVC+Q8nVITIaGQ28jGxc3xhft/YZAivnNCtFZndVt1lIyvq1KjffvMKHkwwiJucnoww3M9ARxttWeHTLlJXgdFbbSHBo+o1q/xS2QYnd15Tfxu3T69jOvd0m3dp83WZs/gJN0q6q
Steps to reproduce
Click on the update count button.  The children components are re-evaluated even when enclosed in a v-memo/v-once tree.
There is an update memo id button to cause the v-memo evaluation to change.
What is expected?
Child components are not re-evaluated unless the memo key has changed.
The count updating should not re-trigger child components to update, the whole subtree should be static.
Only updating the v-memo dependency should trigger re-evaluation of child components.
What is actually happening?
Child component rerendering when using outside state, with v-once and v-memo.
When clicking update count, the children components are re-rendering even when there is a parent v-memo/v-once applied.
System Info
No response
Any additional comments?
No response

The parent does respect the v-memo. But since all of the child components that you nested under these v-memos have a reactive dependency on the count ref that the button in the parent changes, they update themselves

@LinusBorg thanks for the reply. That makes sense, i think this may be a gotcha that the docs could explore and explain.
It makes sense from Vue perspective to have components update themselves and i see the benefit it brings, however the documentation seems to alude to that it memoizes the vnode tree.

All updates for this  and its children will be skipped. In fact, even the Virtual DOM VNode creation will also be skipped since the memoized copy of the sub-tree can be reused.

This reads to me that anything in the below tree is memoized, and as such component trees are included. Perhaps we could note some more information on the above to make it clear that child components referencing reactive dependencies will update themselves. Solutions being to pass this state down so the memoization layer is affective.

Only now realized that this is you, Blake - hi! :D
Yeah, I see how this could be read in the way you did. I guess we are too deeply aware that components always work and update independently of one another for us to notice this reading.
The docs team would be happy to add a note I think, over in /vuejs/docs

Hey! 😅 Yes its moi
Ill take a look at doing this tomorrow.


Sieve New

Thank you for your contribution. Please provide the details requested below.
ISSUE NUMBER
#194
SHORT DESCRIPTION
Algorithm for Sieve of Eratosthenes.
##TESTING
Input any number 'n' and it will generate all the number from 2 to 'n' which are prime.

you need to resolve the conflicts in the contributors.md file @Ankitr19

@prateekiiest ...I have resolved the conflict.

All the checks have passed. :-)

And yes Prateek, I want to point out at one thing, I have written that
Bellman-Ford algorithm, but in spite of that there are zero commits, please
see to it.
https://mailtrack.io/ Sent with Mailtrack
https://chrome.google.com/webstore/detail/mailtrack-for-gmail-inbox/ndnaehgpjlnokgebbaldlmgkapkpjkkb?utm_source=gmail&utm_medium=signature&utm_campaign=signaturevirality
On Thu, Dec 14, 2017 at 3:04 PM, Prateek Chanda<EMAIL_ADDRESS>wrote:

you need to resolve the conflicts in the contributors.md file @Ankitr19
https://github.com/ankitr19
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/codeIIEST/Algorithms/pull/203#issuecomment-351657487,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AXfyubJYjtRJV_fZ3H4mmqqi-ygsV19Eks5tAOu-gaJpZM4RBqoi
.

--
Ankit Raj
Electronics and Communication Engineering
NIT Agartala
8414846511

@prateekiiest , everything looks fine but the only thing is that this PR has more than one algorithm present in it which violates one of the rules on CONTRIBUTIONS.md

Yes,  I don't know why,  when I added new algo in my repo,  and then after
the commit when I clicked on new pull request,  it got appended the sieve's
pull request. So please,  if you can manage this time and merge it. I will
take care from next time.
Ankit Raj,
B. Tech,  Electronics and Communication  Engineering
NIT Agartala
8414846511
On 14-Dec-2017 7:38 pm, "Rahul Arulkumaran"<EMAIL_ADDRESS>wrote:

@prateekiiest https://github.com/prateekiiest , everything looks fine
but the only thing is that this PR has more than one algorithm present in
it which violates one of the rules on CONTRIBUTIONS.md
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/codeIIEST/Algorithms/pull/203#issuecomment-351720392,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AXfyuX1qzSA-1lLX2arXetHjX1UdCjVwks5tASvJgaJpZM4RBqoi
.

Thank you :-)
https://mailtrack.io/ Sent with Mailtrack
https://chrome.google.com/webstore/detail/mailtrack-for-gmail-inbox/ndnaehgpjlnokgebbaldlmgkapkpjkkb?utm_source=gmail&utm_medium=signature&utm_campaign=signaturevirality
On Thu, Dec 14, 2017 at 7:51 PM, Prateek Chanda<EMAIL_ADDRESS>wrote:

Merged #203 https://github.com/codeIIEST/Algorithms/pull/203.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/codeIIEST/Algorithms/pull/203#event-1387771227, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AXfyubxBmSa_QnimLSTdfWIibrPx9kNcks5tAS7VgaJpZM4RBqoi
.

--
Ankit Raj
Electronics and Communication Engineering
NIT Agartala
8414846511


IntelliJ plugin doesn't respect max line length setting.

The IntelliJ plugin does not provide an option to consume max_line_length as FormattingOptions.maxWidth in UiFormatter. Thus we cannot synchronize the IDE with the overridden setting in our build script:
spotless {
    kotlin {
        ktfmt("0.40").kotlinlangStyle().configure { optionsConfiguration ->
            optionsConfiguration.setMaxWidth(120)
        }
    }
}

KtfmtConfigurable.form should provide an option to respect the Hard wrap at setting in Preferences > Editor > Code Style or max_line_length option in .editorconfig.

Definitely interested to see if this is something that would be considered. The discrepancy between IDE and linter is one of the main issues my team has with ktlint (our current style tool). We're considering ktfmt to replace it but I can't convince others to change our max line width to 100 (as required by ktfmt). Spotless gets around this but then we have the issue of the IDE plugin clashing with build script tooling.

This would be very useful.

@mateherber-jt, oh so the issue persist even at 0.50? Interesting...
Long shot from my end, as I don't really know much about how Spotless and IJ formatting works, but have you tried using .editorconfig? I wonder if that's the missing piece of the puzzle

As far as I understand from working on the plugin and looking into the ktfmt step for spotless, both use ktfmt as a direct dependency, but configure it in two different locations (IJ settings vs gradle file).
I think the easiest way to implement this would be allowing to set that value directly in the config form - if that is something that should be configurable. Not sure how much effort it would require to implement reading the max line length value from an .editorconfig file or from the IJ code style settings. Also, I'm unsure if that would actually be sensible. The less places you have to go to configure ktfmt, the better.

I'm running into this same issue. It seems like the Intellij plugin uses 100 for max_line_limit regardless of the .editorconfig.

Fixed in #503

Should be fixed by https://github.com/facebook/ktfmt/pull/503


Strange background flux around transients

Hi,
Getting nice results with your code - however I have a strange issue when recovering transient flux - the background around the transient is around 4x times less than the image.
I'm using the code snippet from the tutorial. Setting Beta=True gives these results, without it I get strange residuals of the host galaxy. Any helps would be much appreciated.
result = subtract(     ref=ref,     new=sci,     smooth_psf=False,     fitted_psf=True,     align=False,     iterative=False,     beta=True,     shift=False)

Hi! This looks strange. It seems related to the background when the code measures the PSF. As if the PSF would tend to sit on a negative background or similar...
What is the mask ourtput?

Hi, can you give me more details? What is the mask output? I am not actively working on the code for a long time, but would be nice to hear more about this problem.
Is it possible to get some reproducible case? can you tar images + script? Thanks

Hello Bruno. Thanks for the reply. Similar to Astro-Sean (previous message), I'm using the code from the tutorial:
result = subtract( ref=ref, new=sci, smooth_psf=True, fitted_psf=True, align=False, iterative=False, beta=True, shift=False). I tried different subtraction options but similar results. Here the result with Scorr_diff from NGC 3556:

Most of the time, I get dark square with a bright star inside, at center. Here the mask result:

Here a copy of the Science and Reference files:
NGC 3556_Reference.zip
Thanks again for your help.


security(weave): bump pillow to latest

Description
Bumping pillow to address CVE-2023-4863

I have read the CLA Document and I hereby sign the CLA

recheck


🐛 [Bug] Not all open Tables are properly shown

🐛 Bug

Not all open Tables are properly shown
You cannot see your own open Table in the Tablelist.

this was solved in iteration 4


Their is no logo in the signup page

Their is no logo in the signup page
Screenshot:

Please assign this issue to me @khushi-joshi-05

And please add the gssoc labels to it @khushi-joshi-05

yeah I finished the work @sunny0625

and you assigned this issue to me @sunny0625

This issue is already solved.
Try to contribute to different issues.


Add play, pause, restart tray features - fixes #19

I haven't done any testing on Windows yet, but I wanted to throw up the PR to get some feedback. I'll update with any changes necessary to accommodate Windows shortly. Let me know what you think of the approach!
P.S. - There's a small bug where the first song doesn't update the tray tooltip. This appears to be because the event listeners in the preload script are added after the first song is triggered by the web app (so it never propagates to the tray). I'm still investigating how to get around that, but let me know if you have any thoughts.

Hey thanks so much for your hard work on this @zdoc01
It works flawlessly! 👍
However, I'm in a dilemma here as #23 fixes this and also the guy claimed the issue so, I have to merge that.
However #19 doesn't cover the Play Next and Play Previous button so if you could work on that then it would be awesome. 😄

Btw please join - https://gitter.im/ongaku-desktop/Lobby# for further discussions about the project. 😄

@anshumanv - np! I had already started looking into these changes and didn't realize another PR had been open. Sorry for the confusion!

No worries!
Looking forward to more contributions from you! :smile:


Output error: doesn't have a size known at compile-time

running against babbage.cddl we get:
 |                     for (key_bytes, key, value) in key_order {
 |                          ^^^^^^^^^ doesn't have a size known at compile-time

in one spot (TransactionBody::withdrawals). All other of those key_order iterations compile fine.

This seems to have been fixed at some point.


Rolling updates with Swarm plugin stop after the Leader is recreated

I have a Swarm with 3 managers and 3 workers. All managers run the swarm flavor plugin with two committed groups: workers and managers. All nodes, but the leader, were created by Infrakit to scale both groups.
I'm then trying to update the groups with a change that requires all the nodes to be recreated (FTR the managers will keep their disk tough). This triggers a rolling update both on workers and managers. If the first manager to be recreated is the leader, then the rolling update stops and is not carried on by the next elected leader.
My Swarm ends up with only two nodes updated.
This is with the Swarm flavour plugin < 0.3.0. I couldn't test this scenario with the latest Swarm plugin because I hit another bug with it.
cc @chungers

I can confirm the same behaviour with latest Swarm Flavour plugin

Yes I am able to reproduce this too.
I think this is a case that was not covered (a bug).  Previously we had considered only the nodes instances where infrakit is not running -- ie. the worker nodes.  In order to prevent run-away, or unstoppable updates (which can be an update gone wrong and the operator wants to stop everything), we had made the decision that if the group controller is stopped at any point during an update, that group controller will not resume updating when it's started back up.  This seemed a reasonable solution because if the operator had a chance to fix the problem and wants the update to continue, she would just do a commit again.
Now the same logic is applied to managers and things aren't so great.  Because the leader group controller is the one that matters, if it gets interrupted before an update of all the managers is completed, the update will not continue without a manual commit again...  So this applies when the leader node itself is killed in order to provision a new instance and another node picks up as the leader (on failing over) --  it is going to not do anything, because it thinks it has been interrupted for some reason, possibly because the update had gone wrong. This would explain what you are observing.
This is a problem as long as we assume update is a blocking process that needs to run to completion, and not something that can be picked up intermittently unless the user explicitly cancels the update operation.
The solution here IMHO is to reverse the policy -- make the update an inherently a nonblocking process and cancelable.  The default behavior would then be to continue the update when the group controller is restarted unless there's signal from user to cancel the update.  This also means that we'd have to make sure the signal to cancel an update (if issued by user) is persisted so that if a leader comes back online it would honor the user intent and halts the update.

Ideas on implementing this:

On a commit, compute a SHA for each group config.  The group plugin stores this SHA in memory and begins the update.
As the update begins, check in the persistent store if there's a CANCEL that's keyed by the group and the SHA.  Do this for each mutation, before a terminate and create new instance.  Proceed if and only if there is no CANCEL.
When the group controller is restarted (or failed over to a new node), continue the update to completion (as long as there are nodes whose config SHA don't match the computed SHA), and as long as there are no CANCEL for that SHA.
Add a new CLI to cancel an update.   This will write a record of CANCEL keyed by the current group config SHA.
A CANCEL only affects a specific commit/update because it's associated to the SHA of the config being updated.  When the config changes, the update (to the new SHA) will start.


[backport] Improve handling of FlexCI test runs

Backport of #5814 by @kmaehashi

[automatic post] Jenkins, test this please.

Jenkins CI test (for commit 40c04f53126b995ffcc75eeec949807cccae37c1, target branch v9) succeeded!


zpay32: add FuzzEncode inputs

Increases coverage of the new blinded path decoding logic.
coverage.tar.gz

Added a few more inputs that increase coverage slightly for blinded path decoding.
coverage.tar.gz

I think we could have a fuzz test for the blinded path code separate from FuzzEncode


pull request for e2e test at 02/05/2017 09:56:09

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


[Python] Automatically generate stubs

Motivation and Context
We want to automatically generate stubs in the codegen diff to ensure they can be reviewed and have a simple way to generate and include the stubs inside the Maturin wheel.
Description
The Python example has been moved to the examples folder and refactored. The refactoring
ensures the script stubgen.py is included in the codegeneration of the SDK crate. The script is later used to generate stubs automatically during testing and can be used by customers to add their own stubs before the Maturin build

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Closes: #2489


Sending the application to the TV

I follow the steps at this address, but somehow I could not send the application to my TV.
URL: https://github.com/flutter-tizen/flutter-tizen/blob/master/doc/get-started.md
1)I created the app
$ flutter-tizen create myapp $ cd myapp
2)I am sending the application.
flutter-tizen run
Output in terminal:
Taha-MacBook-Pro:myapp tahayasinkoksal$ flutter-tizen run
Launching tizen/flutter/generated_main.dart on Tizen UE75BU8000UXTK in debug mode...
The ttt profile is used for signing.
Building a Tizen application in debug mode...                      16,7s
✓ Built build/tizen/tpk/com.example.myapp-1.0.0.tpk (23.5MB).
Installing build/tizen/tpk/com.example.myapp-1.0.0.tpk...          32,4s
Installing TPK failed:
WARNING: Your data are to be sent over an unencrypted connection and could be read by others.
pushed    com.example.myapp-1.0.0.tpk	  0%	      0MB	    0KB/spushed    com.example.myapp-1.0.0.tpk	  7%	      1MB
1509KB/spushed    com.example.myapp-1.0.0.tpk	 11%	      2MB	  947KB/spushed    com.example.myapp-1.0.0.tpk	 15%	      3MB
957KB/spushed    com.example.myapp-1.0.0.tpk	 19%	      4MB	  914KB/spushed    com.example.myapp-1.0.0.tpk	 22%	      5MB
896KB/spushed    com.example.myapp-1.0.0.tpk	 27%	      6MB	  907KB/spushed    com.example.myapp-1.0.0.tpk	 30%	      7MB
856KB/spushed    com.example.myapp-1.0.0.tpk	 34%	      8MB	  856KB/spushed    com.example.myapp-1.0.0.tpk	 36%	      8MB
543KB/spushed    com.example.myapp-1.0.0.tpk	 38%	      9MB	  659KB/spushed    com.example.myapp-1.0.0.tpk	 43%	     10MB
863KB/spushed    com.example.myapp-1.0.0.tpk	 46%	     10MB	  884KB/spushed    com.example.myapp-1.0.0.tpk	 50%	     11MB
911KB/spushed    com.example.myapp-1.0.0.tpk	 53%	     12MB	  920KB/spushed    com.example.myapp-1.0.0.tpk	 57%	     13MB
857KB/spushed    com.example.myapp-1.0.0.tpk	 61%	     14MB	  895KB/spushed    com.example.myapp-1.0.0.tpk	 65%	     15MB
904KB/spushed    com.example.myapp-1.0.0.tpk	 68%	     16MB	  918KB/spushed    com.example.myapp-1.0.0.tpk	 72%	     17MB
897KB/spushed    com.example.myapp-1.0.0.tpk	 75%	     17MB	  862KB/spushed    com.example.myapp-1.0.0.tpk	 80%	     18MB
881KB/spushed    com.example.myapp-1.0.0.tpk	 83%	     19MB	  902KB/spushed    com.example.myapp-1.0.0.tpk	 87%	     20MB
843KB/spushed    com.example.myapp-1.0.0.tpk	 90%	     21MB	  865KB/spushed    com.example.myapp-1.0.0.tpk	 94%	     22MB
932KB/spushed    com.example.myapp-1.0.0.tpk	 97%	     23MB	  887KB/spushed    com.example.myapp-1.0.0.tpk	100%	     23MB
887KB/s
1 file(s) pushed. 0 file(s) skipped.
/Users/tahayasinkoksal/Desktop/tvAppDeneme/myapp/build/tizen/tpk/com.example.myapp-1.0.0.tpk   869KB/s (24689734 bytes in
27.717s)
install com.example.myapp
package_path /home/owner/share/tmp/sdk_tools/com.example.myapp-1.0.0.tpk
app_id[com.example.myapp] install start
app_id[com.example.myapp] installing[10]
app_id[com.example.myapp] installing[13]
app_id[com.example.myapp] installing[16]
app_id[com.example.myapp] installing[18]
app_id[com.example.myapp] installing[21]
app_id[com.example.myapp] installing[24]
app_id[com.example.myapp] installing[27]
app_id[com.example.myapp] install failed[118, -12], reason: Check certificate error :
spend time for wascmd is [4428]ms
Error launching application on Tizen UE75BU8000UXTK.
(base) Taha-MacBook-Pro:myapp tahayasinkoksal$

Hi. Did you create and activate a "Samsung" certificate profile in Certificate Manager? The device must be connected to your PC when you create a distributor certificate.

Hi. Did you create and activate a "Samsung" certificate profile in Certificate Manager? The device must be connected to your PC when you create a distributor certificate.

Thanks for the answer but I created it before and it looks like this. Did I create it wrong?
I'm using Apple M1, could it be a problem with the computer?

Hmm. I'm not sure how you created the certificate profile but it looks like something went wrong. The verifier part ("Verified by" in the screenshot) of the author certificate should be either Tizen or Samsung, but not your name. Please make sure you have installed the "Samsung Certificate Extension" in Package Manager and follow the steps in Creating Certificates to create a valid Samsung certificate profile. Use the "Public" level privilege unless you are a registered partner.


Layout broken with RTL languages

See https://matrix.to/#/!DgvjtOljKujDBrxyHk:matrix.org/$148000080747320GOTMn:matrix.org. Cause turns out to be using the phone in an RTL langauge.
Moved from https://github.com/vector-im/vector-web/issues/2639 (accidentally filed on vector-web).

duplicate of https://github.com/vector-im/vector-android/issues/749


NZB file generation

Apparently I forgot that newsmangler generated a .nzb file. GoPostStuff should probably do that too.
Process:

[ ] Modify Spawner to include "generate article metadata" step between "find files" and "start article generator".
[ ] After metadata is generated, use that to build a .NZB and add it to the files slice

Depends on GH-9 for consistent Message-ID generation.

F4n4t already make a fork with nzb creation see: https://github.com/F4n4t/GoPostStuff

thanks a bunch twiddern for linking that fork, it's exactly what I needed. I tested it and it works nicely.

I would like revive this request.
Although there is a fork for this addition. It has at least one bug in writing the nzb to disk and there is no way to add issues to that fork.


How measure visibility of a group of elements using Vissense.js

I have more than 10 videos in my page. (videos are very short, each about 5 seconds)
I want when 0.75 percent of each video get visible,  that video plays.
so I used below code:
	var timer;
	var vids = document.getElementsByTagName('video');
	$(window).scroll(function() {
		if(timer) {
			window.clearTimeout(timer);
		}
		timer = window.setTimeout(function() {
			for( var i = 0; i < vids.length; i++ ){
				if (VisSense(vids.item(i)).percentage() > 0.75) {
					vids.item(i).play();
				} else {
					vids.item(i).pause();
				}
			}
		}, 200);
	});

the above code works perfectly when there is just a single video in viewport.
but when there is more than a video in viewport, the code not works as expected and none of the videos get played.
What I need is:
Each video has visibility more than 0.75 percent must be played.
For i.e, if there are 3 videos in viewport and 2 of theme has visibility more than 0.75 percent so both videos must be played.
any help is appreciated
thanks

Here there. Sorry for the late reply.
Your code only checks the visiblity of your element once.
You'd need to monitor the visiblity over time and act as soon as the percentage is greater than 75 percent.
I think the code you are looking for is actually in the readme:
var video = $('#video');
var visibility = VisSense(video[0], { fullyvisible: 0.75 });

var visibility_monitor = visibility.monitor({
  fullyvisible: function() {
    video.play();
  },
  hidden: function() {
    video.stop();
  }
}).start();

Hope this can guide you into the right direction. Reopen if necessary!


:memo: add user

Add antoinesoetewey.com to the list of congo users

Thanks for using Congo! I'll merge this in.


fix(deps): update module github.com/tubenhirn/dagger-ci-modules/v5 to v5.3.2

This PR contains the following updates:

Package
Type
Update
Change

github.com/tubenhirn/dagger-ci-modules/v5
require
patch
v5.3.1 -> v5.3.2

Release Notes

tubenhirn/dagger-ci-modules (github.com/tubenhirn/dagger-ci-modules/v5)
v5.3.2
Compare Source
Bug Fixes

deps: update module github.com/tubenhirn/dagger-ci-modules/v5 to v5.3.1 (4eabb20)

Miscellaneous Chores

deps: update renovate/renovate docker tag to v37.77.1 (176eae6)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

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


[BUG] data timestamp consistency violated with eCAL.

steps
mkdir experiment
# t1
./install/performance_test/lib/performance_test/perf_test -c ECAL --msg Array1k -p 0 -s 1
# t2
./install/performance_test/lib/performance_test/perf_test -c ECAL --msg Array1k -p 1 -s 0

actual
terminate called after throwing an instance of 'std::runtime_error'
  what():  Data consistency violated. Received sample with not strictly older timestamp. Time diff: -5739 Data Time:<PHONE_NUMBER>58649
Aborted (core dumped)

comment
qos function of eCAL is abnormal.
EDIT:
aditional info
Simply comment the qos settings for CSubscriber and CPublisher will clear NOT CLEAR this fault.
// m_publisher->SetQOS(writerQos);
// m_subscriber->SetQOS(ReaderQos);
m_publisher->ShmSetBufferCount(1);

will clear this fault
But this issue needs to be fixed.

Should be fixed in the latest versions too.

Should be fixed in the latest versions too.

@rex-schilasky  it still appears in the latest version #4

Okay, I will check this today more carefully.

Okay, I will check this today more carefully.

this problem disapear in automatic tests, but apear in handy test using scripts.
I would like to close this now.


CPU Time doesn't show full stacktrace, where time is lost

I remember earlier, that CPU Time showed for each part of code how much time is lost.
I am working with Django==2.0.7 and django-debug-toolbar==2.0
Now I get just this:

Is this something that has been change, or I did something wrong?
Thanks

Would be spectacular to see what's up here. I haven't used the toolbar like this until recently,  but if that worked it would be a great fix.

@milutinke-crowdbotics Any chance you could determine which version of DDT contained the missing functionality you are referencing?

@jheld @milutinke-crowdbotics Are you two thinking of the ProfilingPanel?

Possibly though that one also doesn't seem to do anything (other than maybe slow the request).


Roll engine 101b27da7a1b..8a7ae957ff77 (15 commits)

https://github.com/flutter/engine/compare/101b27da7a1b...8a7ae957ff77
git log 101b27da7a1b..8a7ae957ff77 --no-merges --oneline
8a7ae957f Undeprecated BigInteger support, but document what it actually does. (flutter/engine#6903)
17789249d Offset.fromDirection and Size.aspectRatio (flutter/engine#6805)
9ae82173a Roll src/third_party/skia 8b78d70b9a10..d4962e7e07c0 (1 commits) (flutter/engine#7225)
a987b1700 Roll src/third_party/skia a1bded9a4f28..8b78d70b9a10 (1 commits) (flutter/engine#7224)
ac0599357 Roll src/third_party/skia 16d00eeef7d1..a1bded9a4f28 (2 commits) (flutter/engine#7223)
91bb1f8f4 Roll src/third_party/skia f391d0f771c6..16d00eeef7d1 (4 commits) (flutter/engine#7222)
6c40f84a5 [vulkan] Fix Fuchsia build
4f148edf0 Roll src/third_party/skia f6206f91b3c1..f391d0f771c6 (4 commits) (flutter/engine#7220)
1bc7ccf60 Support real fonts in 'flutter test' (flutter/engine#6913)
5244cf429 Roll src/third_party/skia 59c9f1595ecd..f6206f91b3c1 (1 commits) (flutter/engine#7219)
0f10d3f99 Roll src/third_party/skia 42e7a7ed6511..59c9f1595ecd (3 commits) (flutter/engine#7218)
167642182 Roll src/third_party/skia 21a7be0741ac..42e7a7ed6511 (1 commits) (flutter/engine#7217)
1a2714bbe Adds force cursor support (flutter/engine#6945)
b50e3491c Roll src/third_party/skia cb6b4bde0c2e..21a7be0741ac (3 commits) (flutter/engine#7216)
826f34264 [Fuchsia] Depend on libtrace when that is what's really meant (flutter/engine#7214)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff (liyuqian@google.com), and stop
the roller if necessary.

@liyuqian : New roll has been created by https://autoroll.skia.org/r/flutter-engine-flutter-autoroll

Trybots failed. These were the failed builds: https://cirrus-ci.com/task/5071582231789568

Commit queue failed; closing this roll.


DeployDemoServer for non intershop repos

PR Type

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build-related changes
[ x] CI-related changes
[ ] Documentation content changes
[ ] Application / infrastructure changes
[ ] Other:
What Is the Current Behavior?

Issue Number: Closes #
What Is the New Behavior?
Does this PR Introduce a Breaking Change?

[ ] Yes
[ ] No
Other Information

I think, this should not be working, as the needed Azure AppService credentials are not shared to forked repositories.

Yes @dhhyi, I had expected that as well. Could this be made available (at least technically) to specific contributors to make the demo server deployment work for them?

Could this be made available (at least technically) to specific contributors to make the demo server deployment work for them?

That would mean exposing the docker registry credentials and also the service principal credentials. I think I can safely respond with "NO!" 😄
You can ask for an invite as contributor to the project and commit directly to it instead of your own fork.
Any other thoughts @jometzner @shauke @rkarl-ish ?

I found the condition as well but I also read about the GitHub secrets not being shared to the fork pull requests. So as @dhhyi already mentioned this would not solve the problem. So we need to think about inviting the developers as direct contributors or we need to copy branches and pull requests in our repository that need a review server.

It is quite simple:

You want demo server capabilities in your fork? Please, built a workflow that builds and deploys your artifacts in your environment.
You want to use our demo environment? Kindly ask us to become a contributor to this project.


Prepare release 0.5.3

As per subject.
Updates changelogs and bumps version.
(normally I'd do this with a direct push to avoid many PRs, but this repository doesn't allow that, so here we are)

As this only updates changelogs, I'm merging with a green CI.


Fix a small markdown format error

Fix a small markdown format error in examples/post-processing/readme.md

Thanks for contributing!


Feature/9215 booster notifications dev menu extension

Description
Add Dev menu extension for booster notification
functionalities:

select a health certified person
download the rules
Show the downloaded booster rules
Show the passed rule for this person if it existed
show last download date
clear the cached last download date
clear the cached rules
clear cached passed rule
Also show errors in alert if any was encountered

Link to Jira
https://jira-ibs.wbs.net.sap/browse/EXPOSUREAPP-9215
Screenshots
https://user-images.githubusercontent.com/15270737/132816079-96db4c8d-6c6d-4fad-ac94-91080cb5dd7c.MP4

A small note: we download the rules and try to match with all health certified persons, and we show if there is any error, for example in the attached video we had two persons, when we tried to download the rules we get a match for Robert and an error for Erika because Erika is not eligible a for a booster vaccination


.NET 7: Code sample for Bind using TryParse in MVC and API Controllers

Bind using TryParse in MVC and API Controllers
Add sample code to this folder

@fiyazbinhasan

on it!


Adjusted NAME loader to recognise circular longitudes.

Small tweak to make extracted data plot with no gap at the meridian.

Thanks @pp-mo :smile:


Feature Request: Helm Chart for Target Allocator

Description
The Target Allocator (TA) is a sub-project of the OpenTelemetry Operator. While it is commonly used alongside the Operator, it is also capable of running as a standalone application without requiring the Operator.
To enhance user adoption and simplify deployment, I propose creating a Helm chart for the Target Allocator. This would allow users to easily deploy TA in Kubernetes environments without the need for the OpenTelemetry Operator.

Motivation

Ease of Deployment: A Helm chart will streamline the deployment process for users who prefer not to rely on the OpenTelemetry Operator.
Wider Adoption: By providing a standalone Helm chart, more users can integrate TA into their Kubernetes workflows.
Flexibility: Users can deploy TA independently or as part of a custom setup that does not include the Operator.

Proposed Solution

Develop a Helm chart for the Target Allocator with:

Configurable parameters (e.g., replicas, resource requests/limits, environment variables).
Support for popular configurations such as custom service discovery.
Optionally, integrate with existing observability stacks, e.g., Prometheus and OpenTelemetry.

Expected Outcome

Users will be able to deploy the Target Allocator using a single Helm command:helm install target-allocator <chart-repo>

@nicolastakashi this is a duplicate issue with https://github.com/open-telemetry/opentelemetry-helm-charts/issues/334... TL;DR I tried to make this work but it was reallyyyyy hard to get helm to do the replacement that the operator does to convert the prometheus config given for the TA... if we were to limit the scope of the chart to deploy a TA with its own jobs that it communicates to the collector that would be okay, otherwise this is challenging...

Closing in favor of https://github.com/open-telemetry/opentelemetry-helm-charts/issues/334, @nicolastakashi feel free to keep the conversation going in that issue.

My idea was only about deploy TA and not configure prometheus receiver @jaronoff97


Incorrect git clone command in README

git clone git@github.com:wxs77577/rest-admin.git command not working as expected, error is throwed:
Permission denied (publickey). fatal: Could not read from remote repository.
But this is working ok:
git clone https://github.com/wxs77577/rest-admin.git
Please update README.

please check this post:
https://stackoverflow.com/questions/12940626/github-error-message-permission-denied-publickey


Enable CI on Windows hosts with Appveyor

Enable integration with the Appveyor CI infrastructure (www.appveyor.com).
After this PR is merged, any push to the repository will trigger common, Android and Windows unit tests on Windows host. Results will be visible at https://ci.appveyor.com/project/crosswalk-project/crosswalk-app-tools.
Test runs will be triggered on PR submission as well.
Related to (but doesn't completely fix) XWALK-5889

gogogo


refactor: Make as_requests_kwargs and as_werkzeug_kwargs reuse the same code for headers handling.

It will also make them return the User-Agent header value.

Codecov Report

Merging #776 into master will increase coverage by 0.07%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #776      +/-   ##
==========================================
+ Coverage   99.18%   99.26%   +0.07%
==========================================
  Files          59       59
  Lines        3798     3795       -3
  Branches      648      645       -3
==========================================
  Hits         3767     3767
+ Misses         15       13       -2
+ Partials       16       15       -1

Flag
Coverage Δ

#unittests
99.26% <100.00%> (+0.07%)
:arrow_up:

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

Impacted Files
Coverage Δ

src/schemathesis/models.py
96.98% <100.00%> (-0.29%)
:arrow_down:

src/schemathesis/specs/graphql/schemas.py
92.00% <100.00%> (+3.76%)
:arrow_up:

src/schemathesis/extra/pytest_plugin.py
100.00% <0.00%> (+1.62%)
: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 edb99f0...4c7db4a. Read the comment docs.


Add Blog post in Articles and other media section

Hi there 👋,
This pull request adds a link to a blog post in the PyO3 README under the "Articles and other media" section. The blog post, titled "How we extended the River stats module with Rust using PyO3," details our experience in utilizing PyO3 to improve the performance of the River library's stats module.

GitHub Packages is having issues, let's try land this later.

bors retry

It's cool, thanks. I really like your visualization on how the extension looks like from Python:
https://boring-guy.sh/img/river-rust/visual_rust_stats.png

It's cool, thanks. I really like your visualization on how the extension looks like from Python: https://boring-guy.sh/img/river-rust/visual_rust_stats.png

Thank you! 🤗


python3Packages.losant-rest: 1.16.0 -> 1.16.1

Description of changes
Update to latest upstream release 1.16.1

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

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

2 packages built:

python310Packages.losant-rest
python39Packages.losant-rest


useResize not working in production

Describe the bug
UseResize works normally in the development environment, but does not work in production.
I am using version 7.2.2.
Steps To Reproduce
After deploying the application (create-react-app), go to the table and try to resize the width of any column. get the following error:

code of table -> https://codesandbox.io/s/damp-browser-ww2p5?file=/src/Table.js

I think your code sandbox is missing some file.

Needs a better repro.


Add new dependency

With this dependency works fine on Raspberry Pi Zero

I can confirm that this depency pyasn1 is indeed missing to make it work on a relatively fresh install of Gentoo.


Open json data with correct MIME type

The links to json metadata are pointing to:
https://raw.githubusercontent.com/GFDRR/rdl-jkan/rdl-0.2/_datasets/json/
This cause them to be opened as normal text file instead of using parser/beautifier.
Instead, all json links should point to:
https://jkan.riskdatalibrary.org/datasets/json/

FIxed by commit 8244d2a.


CONTROLLER: Intern Post

Two things:

Split ACCEPTANCE and CONTROLLER into two different PRs
Post a screenshot from Postman/Swagger that shows the results


feat: support all etcd client configs

What this PR does / why we need it:
support all etcd client configs
Specified Reviewers:
/assign @sfwn

/approve


Fix LAMMPS pair potential

Fixes #135

The doc builds are failing, I think because RTD is using a newer version of sphinx than we do. It may not be possible to actually pin the sphinx version on RTD, so I will updated our requirements and see if I can reproduce & fix.


Fix Python test auth

tested: https://github.com/apache/beam/actions/runs/9894928466
Please add a meaningful description for your change here

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

This change replaced the deprecated setup-gcloud@v0 to auth@v1 for the following tests:
python test (fixed): https://github.com/apache/beam/actions/runs/9894928466
java test (already passing): https://github.com/apache/beam/actions/runs/9895714192
typescript test (permared): https://github.com/apache/beam/actions/runs/9895711562
build and version runner docker image (already passing): https://github.com/apache/beam/actions/runs/9895728848
This fixes current breakage on python test. Also tried to fix build_wheels and cleanUpGcpResource, not able to get them to work so reverted

The reason build_wheel not work but python test worked was because the latter is running on self hosted runner
auth@v1 works for dataflow runner authentication but seems not effective for gsutil auth

R: @ahmedabu98


New X-MAGICAL + MuJoCo environments 👀

Hi Kevin,
I hope you're doing well! I've recently been writing a version of the X-MAGICAL environments in MuJoCo for fun, and was wondering if you'd be interested in adding those environments to this repo?
I was going to run some experiments with XIRL and GraphIRL for fun, and think it could be cool to offer these to the community.
I've attached a video of my MuJoCo shortstick environment so far (WIP):
https://github.com/kevinzakka/x-magical/assets/50164418/3f719b12-4a6f-4983-b496-e101789a92e3

I made a Light-mode version of the environment which more closely resembles the 2D X-MAGICAL env:
https://github.com/kevinzakka/x-magical/assets/50164418/06328878-5dd6-442e-a9f2-b544ced7af38

Hi @jonzamora, great to see your excitement for MuJoCo and cross-embodiment imitation learning :)!
I haven't maintained this repo and honestly have no clue if everything still installs and works correctly (in theory it should?). So I see 2 possible options:

Add your environments here and make sure things are up-to-date.
Create a new repo to host your MuJoCo-version of X-MAGICAL.

Hi @kevinzakka, thanks for getting back to me about this! How does this plan sound:

I'll finish up the environments for each embodiment in MuJoCo
I'll benchmark XIRL and GraphIRL (was planning to do this anyways)
We can make a V2 release of x-magical environment which introduces the MuJoCo-based envs

If this sounds good to you, let me know and i'm happy to lead this effort.

SGTM!


The configuration file 'appSettings.json' was not found ASP.NET core

I am running an ASP.NET core application.   My current settings file being used is appsettings.Development.json and configuration for the rest of my application is loading just fine from there.
However I am getting the message The configuration file 'appSettings.json' was not found
/Users/richardcollette/code/projectNameRoot/service/Dotted.Project.Name/bin/Debug/netcoreapp2.0/appSettings.json
It is however in the folder /Users/richardcollette/code/projectNameRoot/service/Dotted.Project.Name/bin/Debug/netcoreapp2.0/publish
Note the additional folder path of publish.
So two problems.  It's not picking up the current executable path and second, it's not picking up the Development version.
I saw closed issue for this but this is ASP.net and also I have the concern about it not picking up the correct environment appsettings file.
If the rest of my application is picking up the configuration correctly, it would seem that perhaps the property accessor being used is too low level?
Is there a way to load my own list of settings (GetSection?) and pass them in to a feature FlagConfigurer?

https://github.com/jason-roberts/FeatureToggle/blob/ca9bb30a2582f2ea30045e62c812980d8f84e53d/src/FeatureToggle.Shared/Internal/AppSettingsProvider.cs#L38

I'm running on Mac but the application is running in a case insensitive file system so I don't think it's related to the casing.
I'm primarily developing with ASP.NET and specifying a file name explicitly this way is generally not necessary.   Is there another standard way of getting the settings without having to specify a file name?
Program.cs
public class Program
    {
        public static void Main(string[] args)
        {
            BuildWebHost(args).Run();
        }

        private static IWebHost BuildWebHost(string[] args)
        {
            var config = new ConfigurationBuilder()
                .SetBasePath(Directory.GetCurrentDirectory())
                .AddJsonFile("hosting.json", true)
                .AddEnvironmentVariables("ASPNETCORE_")
                .AddCommandLine(args)
                .Build();

            return WebHost.CreateDefaultBuilder(args)
                .ConfigureLogging((context, builder) => { builder.ClearProviders(); })
                .UseConfiguration(config)
                .PreferHostingUrls(true)
                .UseStartup<Startup>()
                .Build();
        }
    }

Startup.cs
public class Startup
    {
        public Startup(IConfiguration configuration)
        {
            Configuration = configuration;
            ConfigurationSection section = Configuration.GetSection("SectionName");
           //etc.
        }

        private IConfiguration Configuration { get; }

        // This method gets called by the runtime. Use this method to add services to the container.
        public void ConfigureServices(IServiceCollection services)
        {
         ...
        }

        // This method gets called by the runtime. Use this method to configure the HTTP request pipeline.
        public void Configure(IApplicationBuilder app,
                              IHostingEnvironment env)
        {
...
        }
}

This is happening for me on Linux.  ASP.NET's built in configuration resolution expects appsettings.json (in the decompiled source) but the this library expects appSettings.json.  So at least on linux, the casing is problem, especially if you develop on a case insensitive file system, like those on Windows, but deploy to Unix.

I'm getting this error when I'm doing F5 debugging of my ASP.NET Core app in Visual Studio. I've checked the directory and there is no appSetting.json there. But should there be during local running?

Try running dotnet run inside .csproj dir. The Directory.GetCurrentDirectory() and .SetBasePath(builderContext.HostingEnvironment.ContentRootPath paths are apparently relative to the folder where the command is run, instead of where the Program.cs file is located at.
I usually use vscode for .NET Core, so can't tell much about how to tackle that on VS.

I am running an ASP.NET core application. My current settings file being used is appsettings.Development.json and configuration for the rest of my application is loading just fine from there.
However I am getting the message The configuration file 'appSettings.json' was not found /Users/richardcollette/code/projectNameRoot/service/Dotted.Project.Name/bin/Debug/netcoreapp2.0/appSettings.json
It is however in the folder /Users/richardcollette/code/projectNameRoot/service/Dotted.Project.Name/bin/Debug/netcoreapp2.0/publish
Note the additional folder path of publish.
So two problems. It's not picking up the current executable path and second, it's not picking up the Development version.
I saw closed issue for this but this is ASP.net and also I have the concern about it not picking up the correct environment appsettings file.
If the rest of my application is picking up the configuration correctly, it would seem that perhaps the property accessor being used is too low level?
Is there a way to load my own list of settings (GetSection?) and pass them in to a feature FlagConfigurer?

I was having the same error and I am using only the terminal, I just added the following group in my .csproj and it worked
<ItemGroup>
	  <None Update="appsettings.json">
	    <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
	  </None>
</ItemGroup>

*.csproj
<Project Sdk="Microsoft.NET.Sdk">

  <PropertyGroup>
    <OutputType>Exe</OutputType>
    <TargetFramework>net7.0</TargetFramework>
    <ImplicitUsings>enable</ImplicitUsings>
    <Nullable>enable</Nullable>
  </PropertyGroup>

  <ItemGroup>
    <PackageReference Include="Microsoft.EntityFrameworkCore.Design" Version="7.0.7">
      <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
      <PrivateAssets>all</PrivateAssets>
    </PackageReference>
    <PackageReference Include="Microsoft.Extensions.Configuration.Json" Version="8.0.0-preview.5.23280.8" />
    <PackageReference Include="Npgsql.EntityFrameworkCore.PostgreSQL" Version="7.0.4" />
  </ItemGroup>

 <ItemGroup>
	  <None Update="appsettings.json">
	    <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
	  </None>
	</ItemGroup>
</Project>

Even after a long time it's hard to find the solution, hope this helps someone somehow


Enabled ClickOnce Support and waiting for the result

Just now i enabled this ClickOnce Support feature  on the chromium-based edge browser v
Version 83.0.478.44 (Official Build) (64-bit).
i want to verify this feature, how that works in reality.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: c8101f32-2f63-f2dd-06a4-ba3ae23786c6
Version Independent ID: f6010158-dd84-09c4-4d8b-d74ae12d6d18
Content: ClickOnce and DirectInvoke in Microsoft Edge
Content Source: edgeenterprise/edge-learn-more-co-di.md
Product: microsoft-edge
GitHub Login: @dan-wesley
Microsoft Alias: kele

Clean up of old issues.  To verify the behavior, you need to navigate to a site using ClickOnce.


Shorthand for "current yaml path" in templating

A very common pattern with this approach is to have an entry in the template that resolves to the value at the same YAML path in the document being compared. This is repetitive and therefore a potential source of error, particularly with longer paths or more complex documents.
Example:
For the document:
spec:
  template:
    metadata:
      name: ThisNameCanBeAnything

the current template would have to look like:
spec:
  template:
    metadata:
      name: {{ .spec.template.metadata.name }}

If there were to be a shorthand for this operation it could look like:
spec:
  template:
    metadata:
      name: {{ anything() }}

Consider this a discussion of:

is it possible and worthwhile to include a shorthand for "replace this with the value from the same yaml path"?
is this worth implementing only for trivial cases (i.e. only when the full path already exists in the template)?

My initial thought is that this would be too complex if all cases are supported.
In order to reliably figure out the yaml path from the template the rest of the template would have to have been rendered, necessitating a two-stage rendering process and/or some form of lazy evaluation in the helper. This may not be possible where loops and conditionals are concerned without invasive changes in the templating engine.
However, I do think this would be a considerable improvement to readability and usability of the templates if this is possible, even if only for the cases where embedded logic does not affect the yaml path.

too complex if all cases are supported

Yep agreed. Unless you're talking about specifically the name? i.e a template function that would map to  .spec.template.metadata.name --> getname() implemented with go (.eg see /compare/funcmap.go).
But generally we should try to be a little prudent when comes to adding custom functions in the code base and when we do...it should super general purpose IMO (of course exception apply but hopefully not many)
And tbh .spec.template.metadata.name and  it's just cleaner to understand + for user...one less thing to remember and decide whether or not to use getname() when initially creating the references with the template rules?

I think using the merges would make this no longer required as you could just leave it out completely.

/lifecycle frozen

The new regex inline diff functionality may make this easier to represent in templates:
spec:
  template:
    metadata:
      name: ".*"

The additional cost of this approach is that you would need to mark that field as a regex in the metadata.yaml:
apiVersion: v2
parts:
- name: ExamplePart
  components:
  - name: Example
    allOf:
    - path: that-template.yaml
      config:
        perField:
        - pathToKey: spec.template.metadata.name
          inlineDiffFunc: regex


In nodesDescription only merge tokens if they occur consecutively in the source code

This was motiviated by the following discussion, where the issue becomes more apparent: https://github.com/apple/swift-syntax/pull/1643#discussion_r1187871863.

@swift-ci Please test

@swift-ci Please test

@swift-ci Please test macOS


weexpack 0.2.0 window下无法打包

weexpack0.2.0不是已经支持window平台了吗，但是window下面没有open命令，run android时报错
{ Error: Command failed: open ./start
这个怎么破

thanks for your question, will fix it soon...

fixed ，just try 0.2.1!

we'll close this issue for now. if 0.2.1didn't solve your problem, pls feel free to submit another. thanks


Allow to multiple info on the same line

Hello,
I'm trying to use this to make a few monster cards, one thing that I noticed that could help a bit was the possibility to put "AC", "Hit Points" and "Speed" on the same line but I wasn't able to do so without breaking the card structure.
Stay classy!

This is a fairly specific request. Could you work around using either of the following?
property | | AC: 10, HP: 60, Speed: 30ft
property | Properties | AC 10, HP 60, Speed 30ft

Well, that works! Thanks!


chore: RAC Autocomplete audit for release

Closes
✅ Pull Request Checklist:

[ ] Included link to corresponding React Spectrum GitHub Issue.
[ ] Added/updated unit tests and storybook for this change (for new code or code which already has tests).
[ ] Filled out test instructions.
[ ] Updated documentation (if it already exists for this component).
[ ] Looked at the Accessibility Practices for this feature - Aria Practices

📝 Test Instructions:

🧢 Your Project:
RSP

totally tiny nit I just noticed when testing, can be fixed later: there seems like there is a slight delay sometimes between when the filtered list updates and when the first item is focused (can be seen in the virtualized list most easily). we may need a useUpdateLayoutEffect or something, not sure. Maybe I am seeing things 😂

@devongovett I'm not quite sure I'm seeing the delay you are mentioning 😅. I tried the virtualized story and a couple of the other ones, was there a particular sequence of interactions/browsers/etc?

Can't wait for this to be released! Y'all are amazing! 🙌🏻


Support key repeat with hardware keyboards

We're certainly not reporting these as key-repeats, so there's probably some issue here.

It looks like the UIViewController pressesBegan, pressesChanged, pressesEnded callbacks don't give us any key-repeat information. We therefore need to implement this ourselves. Pushing this out to Future for consideration there.


README: Swap naming and links Tardigrade by Storj DCS

The README was not update after we rebrand Tardigrade to Storj DCS.
This commit only does the README update but it doesn't make any change
in the reference, identifiers, etc. in the sources nor the plugin name
from Tardigrade to Storj DCS.

I think that I was wrong that Jenkins was broken, I close this PR and if Jenkins needs some changes, I will open a new one.


Update Adobe PDF viewer

migration notes here (one .js file change): https://developer.adobe.com/document-services/docs/overview/pdf-embed-api/modernviewer/

Updated it as part of the boolean issue #591


Update prompb to have metric metadata

Signed-off-by: yeya24<EMAIL_ADDRESS>

[ ] I added CHANGELOG entry for this change.
[ ] Change is not relevant to the end user.

Changes

Update the prompb proto to have the new MetricMetadata part so that Receiver can handle and parse the remote write requests with MetricMetadata.
Right now the MetricMetadata requests are just discarded. We can handle this after the splitting pr is done.
Verification

Hey @yeya24, is this still relevant? If so is there any chance to wrap this up soon?

@kakkoyun It is not relevant at this point. We need this when we want to have metadata in the receiver. But the first thing to do is to fix the E2E test if we update Prometheus version.


Calico - facing issues with multiple etcd nodes

"ETCD_AUTHORITY": "<IP_ADDRESS>:2379,<IP_ADDRESS>:2379,<IP_ADDRESS>:2379"
}
2016-02-04 17:02:47,679 27792 [kube-system/kube-dns-v9-viogj] INFO Starting Calico CNI plugin execution
2016-02-04 17:02:47,679 27792 [kube-system/kube-dns-v9-viogj] ERROR Unhandled Exception killed plugin
I'd like ETCD_AUTHORITY to point to multiple etcd nodes. Currently it fails with the following error. Can you take a look please?
Traceback (most recent call last):
File "", line 585, in main
File "", line 47, in init
File "/code/build/calico/out00-PYZ.pyz/pycalico.datastore", line 182, in init
ValueError: invalid literal for int() with base 10: '2379,<IP_ADDRESS>:2379,<IP_ADDRESS>:2379'
2016-02-04 17:02:47,679 27792 [kube-system/kube-dns-v9-viogj] ERROR CNI Error:
{
"msg": "Unhandled Exception killed plugin",
"cniVersion": "0.1.0",
"code": 100,
"details": null
}

@madhurranjan Calico doesn't support specifying multiple etcd servers.  We recommend using a local etcd proxy on each host as per these instructions.  The proxy is a lightweight process that only serves to redirect Calico's requests to a live member of the cluster.


Add support for multi languages

Thank you for this addition. This will help certainly!
However i have one question:
The current solution with language works via the 'magic' method but not if you would create the Country Object as you would do in the current README.
https://github.com/spatie/holidays/blob/faf22be7976c1f56505ba4aa69447db81c212db9/README.md?plain=1#L119-L148
This might be because https://github.com/spatie/holidays/blob/faf22be7976c1f56505ba4aa69447db81c212db9/src/Countries/Country.php#L44
does not accept named parameters. Are you open to accept named parameter in that method and drop ...func_get_args() as well?
Or is the construction of holidays via the magic method the standard?

I'm not sure I understand what you mean. Could you add a PR with a failing test?
I've included a test on how to use this with named params:
https://github.com/spatie/holidays/pull/177/files#diff-198ea683244cb0b1bcd6bc0093bf6a397a011ee9dcbbaca1e15917f6acbf54bcR96


context_environment_variable: handle context deletion

Adds state removal logic for context environment variables. When the parent context is no longer accessible, the state for a variable should be removed and not refreshed.
This was already done for contexts themselves, but not their variables. Additionally, The API has either changed or the original 404 detection was never working. In reality, when performing a GET request to a context you owned but just deleted, you get a 403 response. Same for any random context ID. Adding StatusForbidden ensures that logic works.
An acceptance test triggers the deletion of the context outside of Terraform and ensures that refreshes continue to work. If the fix is removed, this test fails:
=== RUN   TestAccCircleCIContextEnvironmentVariable_deleted_context
    testing.go:705: Step 0 error: errors during follow-up refresh:

        Error: You don't have permission to view that context.

    testing.go:766: Error destroying resource! WARNING: Dangling resources
        may exist. The full state and error is shown below.

        Error: errors during refresh: You don't have permission to view that context.

        State: <nil>

Full output:
make testacc TESTARGS='-run TestAccCircleCIContextEnvironmentVariable'
TF_ACC=1 go test ./... -v -run TestAccCircleCIContextEnvironmentVariable -timeout 120m
?   	github.com/mrolla/terraform-provider-circleci	[no test files]
=== RUN   TestAccCircleCIContextEnvironmentVariable_basic
--- PASS: TestAccCircleCIContextEnvironmentVariable_basic (2.68s)
=== RUN   TestAccCircleCIContextEnvironmentVariable_update
--- PASS: TestAccCircleCIContextEnvironmentVariable_update (3.56s)
=== RUN   TestAccCircleCIContextEnvironmentVariable_deleted_context
--- PASS: TestAccCircleCIContextEnvironmentVariable_deleted_context (3.65s)
=== RUN   TestAccCircleCIContextEnvironmentVariable_import
--- PASS: TestAccCircleCIContextEnvironmentVariable_import (2.53s)
=== RUN   TestAccCircleCIContextEnvironmentVariable_import_name
--- PASS: TestAccCircleCIContextEnvironmentVariable_import_name (2.44s)
PASS
ok  	github.com/mrolla/terraform-provider-circleci/circleci	15.190s
?   	github.com/mrolla/terraform-provider-circleci/circleci/client	[no test files]
?   	github.com/mrolla/terraform-provider-circleci/circleci/client/rest	[no test files]

Closes #66

Reminder @mrolla 🙂


[Static if] replace overload constraints with static if (iteration.d)

Split up of #5145

And just for the record, the last bit is what I meant in various forum posts that user-facing overloads should have sig constraints that reflect intent rather than implementation: the intent of each is that it would accept anything that either supports foreach or has a range API.  So those should be what constitute the sig constraints of each.
There may be, however, some ranges that satisfy these constraints but which aren't handled by the current implementation; these should be caught by the final else clause in the static if inside each overload body, with a helpful error message explaining to the user why it didn't work ("support for range with characteristics XYZ isn't implemented yet", or "cannot call each with range with characteristics XYZ because PQR").

Jenkins doesn't build and clicking the link goes to
503 Service Unavailable

No server is available to handle this request.

I came to see label: as a near antipattern - it's too nonlocal and makes code difficult to understand and review without context. @wilzbach how about a rule that prevents at least the use of private: and public: in phobos?

I would point out that aside from the fact that druntime and Phobos use private: and public: in quite a few places, the trend for handling attributes seems to be going in the opposite direction of what you're suggesting. Increasingly, we seem to be ending up with attributes being applied with : or braces rather than applied to each symbol individually. And of course, using braces has pretty much the same problems that using : does. I don't know how easy it would be to catch either.
Now, I tend to agree that mass-applying attributes is a problem, but to be honest, I think that using public: and  private: causes far less of a problem than mass-applying other attributes, particularly since it makes a lot of sense to be grouping public and private stuff that way, whereas which of the other attributes apply to functions generally has nothing to do with how you would organize them in the file.
Regardless, for better or worse, the current trend seems to be headed in the opposite direction of avoiding : and braces with attributes - I think mostly because folks don't want to have to keep typing them. So, if we want to make a policy change, we should keep that in mind.

@wilzbach I noticed that this (nonsensical) example from issue 15459 doesn't compile any more, do you think it indicates a regression? I couldn't come up with a meaningful variation that would manifest as a regression.
import std.stdio;
import std.algorithm;

char somefunc(char c)
{
    return c;
}

void main()
{
    stdin
        .byLine
        .each!(map!somefunc);
}

do you think it indicates a regression? I couldn't come up with a meaningful variation that would manifest as a regression.

Hmm, auto-decoding has been a thing in Phobos for a long, long time - so no I don't think it's a regression. On the contrary, now at least you get a compiler error.
With 2.073.1
bar.o:bar.d:function _D3std9algorithm9iteration30__T3mapS19_D3bar8somefuncFaZaZ11__T3mapTAaZ3mapFNaNbNiNfAaZS3std9algorithm9iteration39__T9MapResultS19_D3bar8somefuncFaZaTAaZ9MapResult: error: undefined reference to '_D3std9algorithm9iteration39__T9MapResultS19_D3bar8somefuncFaZaTAaZ9MapResult6__ctorMFNaNbNcNiNfAaZS3std9algorithm9iteration39__T9MapResultS19_D3bar8somefuncFaZaTAaZ9MapResult'
collect2: error: ld returned 1 exit status
Error: linker exited with status 1

With 2.074.0
/usr/include/dlang/dmd/std/algorithm/iteration.d(890):        std.algorithm.iteration.each!(map).each(Range)(Range r) if (!isForeachIterable!Range && (isRangeIterable!Range || __traits(compiles, typeof(r.front).length)))
/usr/include/dlang/dmd/std/algorithm/iteration.d(925):        std.algorithm.iteration.each!(map).each(Iterable)(auto ref Iterable r) if (isForeachIterable!Iterable || __traits(compiles, Parameters!(Parameters!(r.opApply))))

Right, but is the compiler error fixing (correctly replacing) the link error, or just making it unreproducible with that example?


[Feature] Checkbox/CheckboxGroup: Add heading role and list position

Description of Change

This PR contains accessibility improvements as suggested by @brea11y.

Adds heading accessibility role to Header when provided to Checkbox or Checkbox group
Add convenience hook useA11yListPosition to return listPosition translation when provided position and total length

Updated SegmentedControl to use this hook

Adds accessibilityValue to Checkbox items within Checkbox group to allow screen reader to read out position within list

Testing Packages

Screenshots/Video

https://github.com/user-attachments/assets/d2766d7a-6758-422f-b5c4-4dd51228d46d
Testing

[x] Tested on iOS
[x] Tested on Android
[x] Tested on Web

PR Checklist
Code reviewer validation:

General

[x] PR is linked to ticket(s)
[x] PR has changelog label applied if it's to be included in the changelog
[x] Acceptance criteria:

All satisfied or
Documented reason for not being performed or
Split to separate ticket and ticket is linked by relevant AC(s)

[ ] Above PR sections adequately filled out
[x] If any breaking changes, in accordance with the pre-1.0.0 versioning guidelines: a CU ticket has been created for the VA Mobile App detailing necessary adjustments with the package version that will be published by this ticket

Code

[ ] Tests are included if appropriate (or split to separate ticket)
[x] New functions have proper TSDoc annotations

Publish

If changes warrant a new version per the versioning guidelines and the PR is approved and ready to merge:

[ ] Merge main into branch
[ ] Merge branch to main
[ ] Verify that Check Component Integrations workflow ran successfully
[ ] Publish new version after merging

Ready for another review @TimRoe:

Updated list position hook to accept index and add + 1 to calculate position
Alphabetized exports in accessibility.ts
Updated components to use hook inline instead of declaring separately
Updated Checkbox and SegmentedControl to use aria-valuetext for list position

Deleted useA11yListPosition hook and reverted to using listPosition translation directly since we're now using aria-valuetext
Added unit tests to check for heading role and accessibility value
Fix bug where Talkback wasn't reading a11y labels for Label and Description but combining aria-labels and setting on Pressable


ceph-config: remove redundant condition on a block

The error[1] doesn't look relevant (although I think it was on some other job for some other PR), triggering the job again.

jenkins test centos-non_container-all_daemons

jenkins test nautilus-centos-non_container-all_daemons

@guits, the job that failed in pipeline passed


The failure of nightly build

For example, we always fail when building on different platforms.
https://github.com/eclipse-uprotocol/up-transport-zenoh-rust/actions/runs/11379141927/job/31656065596
It's weird that it will succeed if we don't use Swatinem/rust-cache@v2
https://github.com/eclipse-uprotocol/up-transport-zenoh-rust/blob/f13821a314ff25876a58fd52e9c204248fc6218f/.github/workflows/x-build.yaml#L58
@sophokles73 Have you faced this kind of issue before?

No, I haven't. But it seems that your test_nightly branch actually succeeds, right?

No, I haven't. But it seems that your test_nightly branch actually succeeds, right?

Yes, it works either I have cargo clean or remove<EMAIL_ADDRESS>

Might be related. Will give it a try
https://github.com/eclipse-zenoh/zenoh/pull/1542

Hmm.... after updating Zenoh, the issue disappeared. Not sure what happened.

Since Zenoh has been updated to the latest version, let's close the issue for now.
https://github.com/eclipse-uprotocol/up-transport-zenoh-rust/actions/runs/11512988311


test script fails because ShiftFasta command is missing

I'm trying to run the short reads MicrobialGenomePipeline wdl with the supplied inputs.json file with the command:
miniwdl run -i GATK-for-Microbes/wdl/shortReads/inputs.json GATK-for-Microbes/wdl/shortReads/MicrobialGenomePipeline.wdl
but I get an error on the shift fasta step. I tried using the inputs.json file exactly as provided in this repo and also with the two lines that override the default GATK jar file removed and the error I get is the same. In both cases, the error I get is "A USER ERROR has occurred: 'ShiftFasta' is not a valid command".
I've attached the stderr.txt file that's output by call-ShiftReference when I run the script using the inputs.json exactly as provided - stderr.txt
Do I need to update the docker image provided here or the version of GATK that's being pointed to by the inputs.json? Thanks for your help!

I'm having what might be another versioning issue. The --microbial-mode option used for filtering in the shortReads/MicrobialGenomePipeline.wdl doesn't exist in the version of GATK included in the default docker, in the version of GATK selected in the example inputs.json file, or in the latest docker image at us.gcr.io/broad-gatk/gatk:<IP_ADDRESS> (although documentation here suggests that is should). Is there a different version of GATK that I should be using in order to have access to this option? Thanks!


Update Request.php to add LINK/UNLINK HTTP methods

LINK and UNLINK need to be added to allowed methods.

Coverage remained the same at 97.954% when pulling 448234d4c5b2566cb451a6d8b96245e27d5295cb on amreladawy:patch-1 into c138554cb16a92156db787143ab1f266dece7c7f on slimphp:3.x.

Sorry, but a far better solution is to remove the list of validMethods altogether and change filterMethod() to check for a regex of the form: /^[[:alnum:]!#$%&\'*\/+\-.^_|~]+$/Iand only throw theInvalidMethodExceptionif the$method` doesn't match that regex.


jdorn/json-editor is deprecated

Just realized when clicking on your readme link that the underlying jdorn/json-editor repository has been deprecated, and updates are now done on json-editor/json-editor.
Have you updated the static files with those from that new repo ?
Also, wouldn't it be simpler to use their cdn instead of copying the file ?
<script<EMAIL_ADDRESS>

@Fandekasp thanks, I will update static files in the repo. Also I can make optional support for CDN.

json-editor has been updated to latest in master

done


Disallow both hexa and bech32 vanity at the same time

Vanity requirements for hexadecimal and bech32 format cannot be together since they are related (both representations of the same public key). For that reason a new check was put in place to prevent that.
Refer to #15

LGTM!


🐛  Cannot log out of app and log in as another user

Cc: @adamcogan

Hi
Describe the Bug

To Reproduce
Steps to reproduce the behavior:

Log into app
Click on Logout
Close app
Re-open app
Click 'log in'

Expected Behaviour

I can see the log in interface
Actual Behaviour
Logs in with the last logged in user
Tasks

[ ] Investigate
[ ] Fix

More Information

Environment

Device: iPhone 13
OS: iOS

Screenshots
https://github.com/SSWConsulting/SSW.Rewards.Mobile/assets/600044/e8cce8fd-a579-400a-91a9-cf26f61a8a96
Thanks!

👍 - I had the same issues.
Environment was the same (iPhone 13)

https://github.com/SSWConsulting/SSW.Rewards.Mobile/assets/141381253/1f5ea442-eded-4dc2-ab2f-c14ec49018fd

Thanks for reporting this @wicksipedia and @BenDunkerleySSW .
The issue was fixed, closing the issue.


autofix creates long lines

if (thisIsAReallyLongNameOhYesItIs
    && thisIsAReallyLongNameOhYesItIs2
    && thisIsAReallyLongNameOhYesItIs3
    && thisIsAReallyLongNameOhYesItIs4) {
  alert(1)
}

Run:
jscs --preset=google -x test.js
Expected:
jscs will put the operator on the previous line
Actual:
jscs merges all the lines together, creating one really long line and creating a new warning
if (thisIsAReallyLongNameOhYesItIs && thisIsAReallyLongNameOhYesItIs2 && thisIsAReallyLongNameOhYesItIs3 && thisIsAReallyLongNameOhYesItIs4) {
  alert(1)
}

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/11643355-autofix-creates-long-lines?utm_campaign=plugin&utm_content=tracker%2F281640&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F281640&utm_medium=issues&utm_source=github).

google preset has this enabled rule - which requires && to be on the same line.
Not sure if this incorrect, but it certainly could done better

would you accept a pr that just moved the operator instead of the whole line?

yeah


Redstone Integrator texture is the same on all sides.

As in the title, there is no obvious "front" side of the redstone integrator.
I see two ways to remedy this:

The "preferable" way: Remove left, front, etc, directions and switch to cardinal directions, north, east, south, west, up, and down.
The easier way: Add obvious sides to the texture.

Obviously, I'd prefer the first method because it it will bring this to 1:1 parity with Plethora's redstone integrator.

The redstone integrator already supports cardinal directions and the directions left, front, ...
And this is the wrong repository.


Viewing errors when publishing compiled contracts

I published https://kovan.etherscan.io/tx/0xd9435083f0f92efe30ee82639c198814a81a5c5ba2c5145b4b679dcad1833b8c through Remix
It said there was an error, didn't say what.
How do I view the error?

@quantumproducer  hi. and everything was ok in remix side? are you able to debug it there?

@quantumproducer This would be great if you can provide more details about issue

closing because of inactivity.


enc: resize output buffer size

re-alloc a new bigger output buffer for the encoded picture size is
larger than the default.
to fix issue #657
Signed-off-by: wudping<EMAIL_ADDRESS>

please help to review.

updated according to the suggestions, please help to review.


[UI] Tooltips have cut-off text in some cases

Version: master-head (v2.3) (8/16/19)
What kind of request is this (question/bug/enhancement/feature request):
Bug
Steps to reproduce (least amount of steps as possible):

Create a CT. Hover over some of the override switches to get tooltips. Several areas show that the text is cut off.

Result:

I think we need to tweak styling -- if I had to guess this is happening due to the method we do wrapping or something.
Other examples:

Other details that may be helpful:
This is probably happening in other areas of the Rancher UI as well; I just haven't found any additional areas as of yet. The change we make should be global ideally so we can fix this on all pages!

@davidnuzik - I just removed the key from the text.  So now it just says "Allow user override?"

I think this makes sense. We don't allow custom overrides anymore and so that isn't really needed anymore.

Version: master-head (v2.3) (8/21/19)
Now all the help text for the override switches just says "Allow user override?"


cme not reading files passed as -H

Describe the bug
Crackmapexec is passing _io.TextIOWrapper instead of string to os.path.exists(ntlm_hash).
To Reproduce
Steps to reproduce the behavior:
ls -l
total 16
-rw-r--r-- 1 florian florian 7392 Dec 11 17:52 hashes.txt
-rw-r--r-- 1 florian florian 4105 Dec 11 17:51 users.txt

cme smb -u users.txt -H hashes.txt <IP_ADDRESS>
ProxyChains-3.1 (http://proxychains.sf.net)
Traceback (most recent call last):
  File "/home/florian/.local/bin/cme", line 8, in <module>
    sys.exit(main())
  File "/home/florian/.local/pipx/venvs/crackmapexec/lib/python3.8/site-packages/cme/crackmapexec.py", line 83, in main
    if os.path.exists(ntlm_hash):
  File "/usr/lib/python3.8/genericpath.py", line 19, in exists
    os.stat(path)
TypeError: stat: path should be string, bytes, os.PathLike or integer, not _io.TextIOWrapper

Expected behavior
Crackmapexec should read the contents of hashes.txt
Crackmapexec info

OS: Linux parrot 5.8.0-1parrot1-amd64 #1 SMP Debian 5.8.7-1parrot1 (2020-09-14) x86_64 GNU/Linux
Version of CME:  5.1.1dev
Installed from apt or using latest release ? Installed from pipx

Try with full path of the file.

Now you know :)
cme <proto> <ip> -<options>

Sorry, how is this fixed? At least change the help message to reflect your last message, my command was straight out of it with the positional parameter of the target at the end, and didn't work, does that seem like normal behavior to you?


No show icon

Hi. I use your card and i have a problem no show me icon...

.

Thanks. Fix in new version.


Benchmark results are off by 3 orders of magnitude

Describe the bug
Bencmark result are wrong by many orders of magnitue. The operations/sec might be operations/millisec or something else.
Reproduction
import { bench, describe } from 'vitest';

describe('Deserialize', async () => {
  bench('1ops', async () => {
    await new Promise((res) => setTimeout(res, 1000));
  });
  bench('2ops', async () => {
    await new Promise((res) => setTimeout(res, 500));
  });
  bench('10ops', async () => {
    await new Promise((res) => setTimeout(res, 100));
  });
  bench('100ops', async () => {
    await new Promise((res) => setTimeout(res, 10));
  });
});

Result:
 ✓ test/test.bench.ts (4) 15126ms
   ✓ Deserialize (4) 15123ms
     · 1ops    0.0010 ops/sec ±0.76% (10 samples) slowest
     · 2ops    0.0020 ops/sec ±1.08% (10 samples)
     · 10ops   0.0092 ops/sec ±2.31% (10 samples)
     · 100ops  0.0641 ops/sec ±1.77% (33 samples) fastest```

### System Info

```shell
System:
    OS: Windows 10 10.0.19043
    CPU: (8) x64 Intel(R) Core(TM) i7-9700K CPU @ 3.60GHz
    Memory: 6.47 GB / 15.93 GB
  Binaries:
    Node: 16.13.2 - C:\Program Files\nodejs\node.EXE
    npm: 8.1.2 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Chrome: 105.0.5195.127
    Edge: Spartan (44.19041.1266.0), Chromium (105.0.1343.53)
    Internet Explorer: 11.0.19041.1566
  npmPackages:
    @vitejs/plugin-vue: ^3.0.3 => 3.1.0
    vite: ^3.0.8 => 3.1.3
    vitest: ^0.23.4 => 0.23.4

Used Package Manager
npm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

These issues are solved in the last version of Tinybench.


tags HO, WA should be enclosed to show full tag length

It should be made clear that tags are four characters, with padded spaces.

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

ID: 662c6f54-f90c-52ed-dca2-0f0a0a49fee5
Version Independent ID: 6aa33924-d45e-60c3-a9fd-e317ff662993
Content: Language system tags - Typography
Content Source: typographydocs/opentype/spec/languagetags.md
Product: typography
GitHub Login: @PeterCon
Microsoft Alias: PeterCon

Fixed for next version.