Adding integrations via npm?

I don't think this used to be broken but all the integrations (e.g. vue / image) say to add them via:
npm run astro add image

But that doesn't make sense as there isn't an astro script by default in packages.json and, even if there was, it wouldn't take arguments.
I believe this used to say npx astro add image which would work.

And if you want a dependency (npx) free version:
npm exec astro add image

Thanks for this, @altano! 🙌  I'm tracking down where (and why) these changed because you're right... they were correct before. Not sure what happened there, but I'm going to get that reverted pronto.

@delucis Can I assign this one to you? It seems to be your PR here https://github.com/withastro/astro/pull/4340 where this change was made. I can see this was an intentional change.

Turns out I am mostly wrong.

It looks like a newly created astro project does include an astro npm script in package.json.
And, if that script exists, npm run astro add image seems to do the right thing. I'm very confused on this last point: npm scripts didn't always take arguments (unless you use --)? It's still documented to require --. Did npm change how script arguments work? I can't find any announcements or documentation. Consider me confused.

Anyway, npm exec might be the safest bet since it doesn't rely on the package script existing. Or just leave it as-is and assume I'm probably the only person who screwed up their package.json and removed the astro script and this isn't super important.

Thanks again for bringing this to our attention, @altano! It sparked some discussion, and we ultimately opted to go for what had a lower chance of not working for people. (I think a good choice.)


Must restart ghost to take affect from change/edit theme

I use Ghost Blog and I see it very useful and fast, but when I custom my theme, I must restart ghost and it take more than 60 seconds to take effects. I want Ghost to add a feature to fix this troubling thing.

Hi,
I think that's because you're running Ghost in production mode, which uses cache(I guess?) to improve the speed.
In production mode, you don't have to restart Ghost, to made the changes effective. Just go to the admin panel, active another theme, then switch back. In that way the theme will be reloaded.
Or you can run Ghost in development mode to disable cache.
Hope it helps.

I just noticed this issue too. It is a bit frustrating to have to restart npm everytime you make a change to a theme, is there a better way?

@paulmaunders sorry to hear you've also run into this bug, there's an open issue for it here https://github.com/TryGhost/Ghost/issues/7812
For now, if you are developing a theme I would suggest keeping web inspector open with the "disable cache option checked.

I've got reply from Ghost Team. Hope it can help you.

Hi,
I think that's because you're running Ghost in production mode, which uses cache(I guess?) to improve the speed.
In production mode, you don't have to restart Ghost, to made the changes effective. Just go to the admin panel, active another theme, then switch back. In that way the theme will be reloaded.
Or you can run Ghost in development mode to disable cache.
Hope it helps.

@yabeow
I had made that reply. (And then I deleted it cuz I thought I was wrong.)

I've got reply from Ghost Team.

I'm not a Ghost developer. 😅

That does not seem to work.
I run ghost in development.
I don't cache with the dev tools open and I almost always have them open.
I call 'ghost buster' in the local ghost install.
I stop and start ghost. I also tried to restart ghost.
The changes made in the page.hbs template of the default casper theme are not coming through.

@dbauszus-glx if you need support please use https://forum.ghost.org


Update grafana slicing dashboard

I'm, as a slicing promoter, show slicing demo using grafana dashboards where monitoring parameters are shown per slice separatly.
Acceptance criteria:

[x] Every monitoring parameter is shown per slice if possible
[x] Dynamical panel per slice supported
[ ] Updated dashboard reviewed
[ ] Source code of dashboard merged in repo
[ ] Dashboard deployed in K8s demo stand

Row options

Title
S-NSSAI $snssai
Repeat for
snssai

Panel SMF query A : ues_active and on(job) fivegs_smffunction_sm_sessionnbr{snssai="$snssai"}
filter for SMF related to $snssai only
refer to https://stackoverflow.com/questions/63660751/nesting-query-in-promql#63667762

Selecting all instances with a particular label can be done with the and operator, which will return the left hand side if the labels on the right hand side has matching labels.

dashboard looks like:

Update open5gs_sessions.json  https://github.com/edgecomllc/eupf/pull/442


add security incident response process

add security incident response process to website - copying over from sig-security

@svrnm I can remove the javascript files. That formatter did those changes automatically

Fix everything. Thanks for the feedback and suggestions @svrnm @chalin. Should be ready to go

Fix everything. Thanks for the feedback and suggestions @svrnm @chalin. Should be ready to go

Thanks!
Can a second person from @open-telemetry/sig-security-maintainers approve the PR as well? Then we can merge it.

@cartersocha this looks good overall. If you can address the bits from @jpkrohling then I think we can merge this.

done @cartermp


feat(plugin): add ICP token creation support

Add plugin-icp to enable token creation on Pickpump platform
Implement createToken functionality for Internet Computer Protocol (ICP)
Support ICP token standard and specifications
Enable seamless integration with Pickpump's token creation flow

BREAKING CHANGE: None

this is fucking rad, will review later

Also feel free to fix the PR Title check, add in a a screen grab of successful token creation, and consider an integration test to simulate the entire token creation flow using mocks or a test canister to ensure the plugin works end-to-end. Also can we update the plugin associated READMEs so it is easier to leverage. Thanks so much! So sorry for all the comments! You are a badass and did some great work here!

Thank u for ur review, I will fix the issues. @monilpat

Oh yeah


Automated Package Upgrade

:tada: This PR is included in version 12.4.92 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


added fix and tests

Added fix by trimming left slash for blob name
Added tests which upload file with absolute path

Resolves #14
Potential further issues:

uploading file with path ./myname
uploading file with path ../dir/name

New build and integration test on the branch were successful.


[HUDI-4694] Print testcase running time for CI jobs

Change Logs
Add a CI task for each UT/FT task to print top 100 long-running testcases.
Impact
Risk level: none
Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

@xushiyan Great to add the running time reports!


Document secrets handling in Marathon

Marathon 1.3 has added secrets to environment variables.
The AppDefinition supports defining secret env vars and secrets.
In order to make secrets work, you need to write a plugin to render secret values.
This is a plugin extension that needs to be documented as part of the plugin extension hooks.

Hey @aquamatthias , interested in getting these docs.  Is there a draft anywhere I could take a look at?

@scottjpack busy times - sorry no doc yet.
The gist of it:

You can define an app definition like this

{
  "id": "/foo",
  "user": "root",
  "secrets": {
    "secret1": { "source": "/path/in/secret/store/eg/db/password" }
  }
  "env": {
    "PASSWORD": { "secret": "secret1" }
  },
}

It defines secrets with a source and assigns a name (secret1 in this example)
You can reference this secret as environment variable.

You have to define a plugin, which can resolve secrets and sets those values as environment variables.
See: https://github.com/mesosphere/marathon/blob/master/plugin-interface/src/main/scala/mesosphere/marathon/plugin/task/RunSpecTaskProcessor.scala#L11

Whenever a task is started, this plugin is called to set the correct env vars.

Hope this helps as a quick start.

Is the corresponding GUI shown at https://docs.mesosphere.com/1.8/administration/secrets/config-service/ (step 8) in a non-public Marathon fork, a non-public Marathon plugin or something else altogether?

@petr-k this link points to DC/OS which has its own UI and secrets implementation. There is currently no GUI support in Marathon.

@aquamatthias Should secrets handling be considered a beta functionality as it relies on plugins?

@koshelev That is intended since there can be different secrets provider. Marathon provides an API for secrets, but no implementation. The whole functionality has a feature flag, so you can adjust if secrets are allowed or not.

@sascala can you add a section to secrets handling to our gh-pages site?

@aquamatthias Of course

Note: This issue has been migrated to https://jira.mesosphere.com/browse/MARATHON-2432. For more information see https://groups.google.com/forum/#!topic/marathon-framework/khtvf-ifnp8.


[FEATURE REQUEST] Better support for parameterized tests

Gathering feedback from https://github.com/Pante/elementary/issues/141#issuecomment-1275979361, adding a @LabelSource (tentative name) as a source for parameterized tests will reduce the ceremony involved in writing parameterized tests.

I like it (though I might be biased 😁)


Error during Resize-Image when making windows 10 image

I'm trying to create a windows 10 openstack image. During the Resize-Image step, I am getting a "Not Enough Available Capacity" error. Do you have any suggestions about how to get around this?
Thanks
Applying image
[==========================100.0%==========================]
The operation completed successfully.
8/21/2019 6:06:25 AM - Create BCDBoot Config for Windows 10 Pro
The operation completed successfully.
The operation completed successfully.
The operation completed successfully.
8/21/2019 6:06:28 AM - BCDBoot config has been created.
8/21/2019 6:06:30 AM - Cloud image generation finished.
8/21/2019 6:06:30 AM - Creating VM WindowsOnlineImage-Sysprep1135807869 attached to ExternalSwitch
8/21/2019 6:06:32 AM - Starting WindowsOnlineImage-Sysprep1135807869
8/21/2019 6:06:37 AM - Waiting for WindowsOnlineImage-Sysprep1135807869 to finish sysprep.
8/21/2019 6:18:42 AM - Shrinking VHD to minimum size
8/21/2019 6:18:43 AM - Initial VHD size is: 80 GB
8/21/2019 6:18:50 AM - Current partition size: 79.998046875 GB
8/21/2019 6:18:50 AM - New partition size: 12 GB
8/21/2019 6:18:50 AM - Size increased:<PHONE_NUMBER>8
WARNING: Not enough available capacity
Activity ID: {9e74ec16-5791-0000-5877-759e9157d501}
8/21/2019 6:18:56 AM - Size increased:<PHONE_NUMBER>8
WARNING: Not enough available capacity
Activity ID: {9e74ec16-5791-0000-5877-759e9157d501}
8/21/2019 6:19:02 AM - Size increased:<PHONE_NUMBER>8
WARNING: Not enough available capacity
Activity ID: {9e74ec16-5791-0000-5877-759e9157d501}
8/21/2019 6:19:08 AM - Size increased:<PHONE_NUMBER>8
8/21/2019 6:19:11 AM - Not enough available capacity
Activity ID: {9e74ec16-5791-0000-5877-759e9157d501}
Resize-Partition : Not enough available capacity
Activity ID: {9e74ec16-5791-0000-5877-759e9157d501}
At C:\Users\Administrator\projects\windows-openstack-imaging-tools\WinImageBuilder.psm1:994 char:17
+ ...             Resize-Partition -DriveLetter $Drive -Size $sizeIncreased ...
+                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (StorageWMI:ROOT/Microsoft/.../MSFT_Partition) [Resize-Partition], CimException
    + FullyQualifiedErrorId : StorageWMI 40000,Resize-Partition

Hello,
Could you give us some details about your workflow? Did you use the New-WindowsOnlineImage or New-WindowsCloudImage? How does your config file look like?

Hello,
You have hit a weird PowerShell issue for the Resize-Partition, as the minimum partition size does not coincide with the minimum resizeable partition size. As your image file has already been generated, you can unmount it manually and run "qemu-img.exe convert -O qcow2 image.vhdx image.qcow2". You will find qemu-img.exe in the bin folder.
To try to fix the code, you can increase the number of retries of the resize or increase the minimum buffer for the resize.
https://github.com/cloudbase/windows-openstack-imaging-tools/blob/master/WinImageBuilder.psm1#L990
or
https://github.com/cloudbase/windows-openstack-imaging-tools/blob/master/WinImageBuilder.psm1#L963
By default, the number of retries is 4.

Thank you for responding so quickly. I increased the retry count in WinImageBuilder.psm1 and was able to get around this issue.
Thanks again.

@icksa I will make a patch to increase the retry count to a higher value so that this issue does not happen anymore.

@icksa  I added also a binary search resize retry as sometimes not even 20 or 30 retries are not enough on Windows 10 updated. https://github.com/cloudbase/windows-openstack-imaging-tools/pull/294


feat: add LumberjackLogFactory

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: CONTRIBUTING.md#commit
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?
Using LumberjackLogBuilder requires passing LumberjackTimeService to maintain testability. This can be cumbersome.

Issue Number: N/A
What is the new behavior?
LumberjackLogFactory removes the need to having to inject and pass LumberjackTimeService.
Does this PR introduce a breaking change?
[ ] Yes
[x] No

Other information

I missed a test file for the factory.
Indeed. I wanted your feedback on this concept first 🙂

I missed a test file for the factory.
Indeed. I wanted your feedback on this concept first 🙂

I added inline docs. Let's add docs in README in the docs review branch after merging.

I added inline docs. Let's add docs in README in the docs review branch after merging.


waypoint: init at 0.1.5

Motivation for this change
Init waypoint at 0.1.5
Based on @winpat's pr #100994
I'm happy for any of my modifications to be coppied over & delete this branch. I'll put a PR in upstream so we can probably use make bin as is.
Resolves the injection issue issue

Removed the dontPatchELF stuff because none of it seems to actually be passed through by buildGoModule and I'm not comfortable changing it.
Things done
I ran the init and the build but it could do with more testing.

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

[X] NixOS (x86_64)
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[X] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

Great job! I closed my PR.
Should be able to test this tomorrow.

LGTM. All the issues I had with my original expressions are resolved.

It's looking like -static isn't going to be added upstream.
I've tried a bunch of things to get it working without it but no luck so far.
I'm also happy to add back in the git stuff for setting the version if you'd like

I'm also happy to add back in the git stuff for setting the version if you'd like

That sounds like a good idea.

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

1 package built:

waypoint

@winpat actually it looks like fetchgit with leaveDotGit isn't a great idea in its current state #100498 https://github.com/NixOS/nixpkgs/pull/64319#issuecomment-729837272
TIL

Also it doesn't look like there's going to be much progress investigating the issue so I'm happy with it as is & I don't want to keep you and others waiting by leaving this in review if it's working well with everything we've tried so far

/marvin opt-in
/status needs_reviewer

Added the extra flags to reduce size as requested.
Results:
before:
103.4 MB - waypoint
 45.4 MB - waypoint-entrypoint

after:
 90.7 MB - waypoint
 40.3 MB - waypoint-entrypoint

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

1 package built:

waypoint

Result of nixpkgs-review pr 103754 run on x86_64-darwin 1


DF-698: Add covering date filter

Ticket URL: https://national-archives.atlassian.net/browse/DF-698
Supersedes #991
About these changes
Adds the ability to filter record results by 'Covering date'
How to check these changes
Where possible, provide guidance to help your reviewer
Before assigning to reviewer, please make sure you have

[x] Checked things thoroughly before handing over to reviewer.
[x] Checked PR title starts with ticket number as per project conventions to help us keep track of changes.
[x] Ensured that PR includes only commits relevant to the ticket.
[x] Waited for all CI jobs to pass before requesting a review.
[x] Added/updated tests and documentation where relevant.

Merging PR guidance
Follow docs\developer-guide\contributing.md
Deployment guidance
Follow docs\infra\environments.md

@TNA-Allan
A couple of questions for you:

Scenario 1
Issue: Filter label still remains

Could you elaborate a little here? I haven't touched the 'opening date' filter, so not sure what you mean.

Scenario 3
Issue: Sort by filter shows - whereas there are no results

I would recommend raising this as a separate ticket in Jira. It's completely unrelated to these changes, and it's not something I intend to fix here.

Scenario 1
Issue: Filter label still remains

Could you elaborate a little here? I haven't touched the 'opening date' filter, so not sure what you mean.

This was the link   instead.
Please ignore, not a issue.

@TNA-Allan This work shouldn't really have been merged yet. There are still a couple of outstanding issues RE hidden fields. I'll submit a follow-up PR instead


Updated isNumber check in SqlResult.ToString method

Empty string was treated as number previously.
How it was:
var query = new Query("table").Select("col").Where("col", "");
new Compiler().Compile(query).ToString();
// will return `SELECT col FROM table WHERE col = `
// but should return `SELECT col FROM table WHERE col = ''`

Thanks, could you add a unit test for this case ?

@ahmad-moussawi yes, added

@Happi-cat thanks merged 👍


Webhook: add e2e tests

Signed-off-by: Martin Kennelly<EMAIL_ADDRESS>

/LGTM lets wait for #76 to land 1st

/lgtm
lets wait for #76 to land 1st

/approve


Add description for code_generator_toml

Why is this change needed?
Add description for code_generator_toml.
(Japanese)
code_generator_tomlの説明を追加しました。
(@chaemon さんに書いていただいた方がいいかも?)

Codecov Report

Merging #244 (67a2245) into stable (882b831) will decrease coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           stable     #244      +/-   ##
==========================================
- Coverage   89.73%   89.69%   -0.05%
==========================================
  Files          62       62
  Lines        2612     2600      -12
==========================================
- Hits         2344     2332      -12
  Misses        268      268

Impacted Files
Coverage Δ

atcodertools/common/judgetype.py
87.71% <0.00%> (-0.81%)
:arrow_down:

atcodertools/tools/models/metadata.py
91.17% <0.00%> (-0.50%)
:arrow_down:

atcodertools/client/models/contest.py
95.45% <0.00%> (-0.20%)
:arrow_down:

atcodertools/client/models/problem_content.py
95.23% <0.00%> (-0.12%)
:arrow_down:

atcodertools/client/models/problem.py
100.00% <0.00%> (ø)

...tcodertools/fmtprediction/models/variable_token.py
100.00% <0.00%> (ø)

...s/fmtprediction/models/format_prediction_result.py
100.00% <0.00%> (ø)

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 882b831...67a2245. Read the comment docs.

ありがとうございます。マージします。が


Use 2.0.1 version of itsdangerous for demo client

#137

Hi @Fronox,
Thanks for the PR, it is fixed in the current release. I will close this one.
Kind regards,


Incorrect emoji widths with conhost and openconsole

Windows Terminal version
1.20.24050601
Windows build number
10.0.19045.0
Other Software
Far manager v3 - easy to test by creating files with needed names and opening Far in different terminals, but the behavior is not specific to Far.
Steps to reproduce
print string / create file with emoji in it:
test1 ♻.txt
test2 🧩.txt
test3 ⭐.txt

In Windows Terminal
Windows Terminal + pwsh:

Plz, disregard the ^C. Note:

♻ size is like half-width or so - .txt is printed inside it
🧩 size looks to be correct
⭐ size looks to be correct, but note that the cursor is rendered between x and t, when it's actually at the end (-1 symbol cursor position rendering). Also note that black&white version is rendered, not colored one, like for other emojis.

with Far:

Everything looks good, but ♻, which is half-width or so and b&w image for star.
In conhost
Far using default Windows 10 console:

It's not very clear from the screenshot, but it's easy to repeat my test using latest stable Far manager v3 and creating files with the names above.
Notes for screenshot:

♻ looks to be rendered correctly (considering no emoji support)
🧩 looks to be rendered correctly, but the cursor shown over test2 file is 1 symbol shorter
⭐ vertical border is moved 1 symbol to the left

OpenConsole
Far in openconsole directly:

Same as with WT - ♻ symbol looks to be half-width or so.

NOTE: I've tested mainly with Consolas font, but I've tried a few other, like Fira Code, IBM Plex, Cascadia Mono - behavior seems to be consistent between fonts.
NOTE: to fast-insert emojis - Win+; and start typing recycle for ♻, puzzle for 🧩, star for ⭐.

Expected Behavior
Consistent spacing and rendering of emojis.
Actual Behavior
Inconsistent spacing, cursor positioning, etc.

I apologize for the late response. The behavior is unfortunately intentional, because for most glyphs the widths are determined by the Unicode standard and not by the font. The standard is this one: https://www.unicode.org/reports/tr11/
If the font contains glyphs that are wider than they should be, they'll show up like this (overlapped).
Your glyphs are:

♻ https://codepoints.net/U+267b
🧩 https://codepoints.net/U+1f9e9
⭐ https://codepoints.net/U+2b50

Check out the "East Asian Width" for each of these. The first one is "neutral", which basically means "1 column", and the other two are "wide" (= 2 columns). We use the official text renderers from Windows (GDI and Direct2D) in their default configuration. The reason they behave differently is because GDI has Segoe UI Symbol (= black and white glyphs) higher up in the font fallback priority list than Segoe UI Emoji (= colored glyphs), and for Direct2D it's the other way around. Unfortunately neither font adheres to UAX #11.

@lhecker thank you for clarification. I understand there are technical issues. But from end-user point of view - that's a bug for sure.

Check out the "East Asian Width" for each of these. The first one is "neutral", which basically means "1 column", and the other two are "wide" (= 2 columns).

That might be a good reason for 2 different behaviors, but does not explain 3. All 3 symbols behave differently (1 overlaps, 1 renders normally, 1 shifts the cursor).
Finally, if that cannot be fixed in windows terminal directly due to reliance on some other product (renderer in this case) - I'd really appreciate if the team takes ownership of the problem and opens required tickets, as dev team is the only entity who really knows what's happening underneath. End user really expects a product feature of correct rendering from console app.

There's more to this - let me respond to your issue description one by one:

Windows Terminal + pwsh:
♻ size is like half-width or so - .txt is printed inside it

The half-width is intentional due to UAX #11. The reason why ".txt" is printed inside it is: "This is the same treatment given by iTerm2 and Terminal.app on macOS (one of which I believe was the first terminal emulator to support emoji(?).)" (See https://github.com/microsoft/terminal/issues/16852)
In other words, terminal applications expect us to render emojis with an overlap like that, because the first terminals that supported emojis did so.
However, if you expected the ♻ glyph (recycling symbol) to show up as an emoji then you should print ♻ instead (recycling emoji). The difference between the two is that ♻ is U+267B and is called an "unqualified emoji". Unqualified means that it's unspecified how the glyph gets rendered. ♻ on the other hand is U+267B U+FE0F, which is a "fully-qualified emoji" thanks to the U+FE0F variation selector. This means it always gets rendered as something that's 2 cells wide and is colored if the renderer supports color (conhost and GDI don't).
If you want narrow emojis to show up as wide emojis you should always use U+FE0F. You can find a list of all emojis with their U+FE0F counterparts here: https://unicode.org/Public/emoji/latest/emoji-test.txt
Support for U+FE0F was still somewhat buggy until recently. That was fixed by #16916.

⭐ size looks to be correct, but note that the cursor is rendered between x and t, when it's actually at the end (-1 symbol cursor position rendering).

Most emojis don't work inside the pwsh prompt line. I'm not entirely sure why this one in particular is broken, but these problems are tracked here: https://github.com/PowerShell/PSReadLine/issues/1329
I've been hearing whispers that PSReadLine will add support grapheme clusters in the near future too, which would solve all of these issues all at once.

Also note that black&white version is rendered, not colored one, like for other emojis.

It seems that this is a bug in DirectWrite on Windows 10. We could try and fix that!
It also occurs with GDI in conhost, but I'm not entirely sure whether we can fix that easily.

Far using default Windows 10 console: [...]

The Windows 10 console predates significant parts of our emoji support and most of the bug fixes. Our request to backport them has been denied unfortunately, so we can't do anything about that. We do ship Windows Terminal on Windows 10 however which works correctly (as per iTerm 2's and Terminal.app's emoji behavior).

Far in openconsole directly: [...]

FYI OpenConsole is basically the latest conhost version. That's why it behaves like Windows Terminal. conhost on the latest Windows 11 version will also behave like that. But it still uses the GDI renderer so that's why it's black & white.

The only thing that's not intentional is how ⭐ looks like a tiny star in Windows Terminal on Windows 10 and in conhost in general.
Regarding Windows Terminal, I suspect that the font fallback table in the Windows 10 version of DirectWrite simply didn't include that codepoint in the list. Fixing that is non-trivial because the font fallback list is gigantic. We could patch that one codepoint inside our font-mapping code, but I'd not particularly like adding such bodgy compatibility code. I'd be open to do that though if we decide that we do want to see ⭐ fixed.
Regarding conhost, custom font fallback with Uniscribe works like this: https://learn.microsoft.com/en-us/windows/win32/intl/using-font-fallback#assign-a-fallback-font
However, we'd have to be extra careful because applications may actually rely on the fallback to Segoe UI Symbol and so it would either need a setting, or we'd have to make it so that we detect fully-qualified emojis automatically and only use Segoe UI Emoji for those that are. We currently don't have a Unicode database that includes this information, because we're trying to reduce the binary size. We could rely on the ICU database for this though, but even then, it'd be a very difficult fix.

We've discussed this internally earlier this week and we've decided that we don't want to fix the ⭐ issue at the moment, unless more people are interested in seeing this fixed. This is mostly because fixing this is likely to be non-trivial right now.
The other issues are - for better or worse - unfortunately working as intended, as explained above. As such I'll be closing this issue for now. If you feel like anything I said is incorrect, please let me know!


Replace argument type with @Source to determine source/parent

Annotation way will be more precise.
leniency model: argument type
I think argumentType way may lead mismatch. There will be various parameter types when spring-graphql support custom HandlerMethodArgumentResolver.
This way also require strict restriction of code: SourceMethodArgumentResolver must be put after other HandlerMethodArgumentResolver.
strictness model: @Source
Developers will easily know whether argument is source/parent by @Source, they don't have to look at the implementations of HandlerMethodArgumentResolver to determine which one supports the parameter.
And there is no need to take SourceMethodArgumentResolver as fallback.

Thanks for the proposal, and I admit there is something appealing about having an @Source annotation that explicitly marks the method parameter, and it's something we did consider.
The pause that I have is that the annotation does not serve any real purpose. We can work it out just fine without it, without any ambiguity that we've come across so far. In terms of developers wondering, well if someone new wants to find out how to get a hold of the parent, the answer is just declare it. I think that's as simple as it gets and you don't need to know more.
From the opposite perspective as well, if we had @Source, it would be legitimate to ask, should it be required and if so why? I don't think there is a good reason to require it, so that leads back again to having some controller methods with it and some without it. This reminds of @ModelAttribute in Spring MVC where you could have it or not, but the point there is that when you do add it, it can serve a purpose. There would be no such case here.

about mismatch(match incorrectly)
With @Source, developers can easily avoid mismatching the 'parent/source' in their custom MethodArgumentResolver.
Conversely, if the parent/source and parameter parsed by custom MethodArgumentResolver share many characteristics, parent/source maybe easy to mismatch——That's extreme scene, maybe we can ignore.
about readability of strictness model

Developers will easily know whether argument is source/parent by @Source, they don't have to look at the implementations of HandlerMethodArgumentResolver to determine which one supports the parameter.

Consider the @Controller method：
@SchemaMapping
public X  test(A a, B b, C c){
      // do something and return
}

Maybe developer must read source code about custom HandlerMethodArgumentResolver and the DataFetcher of parent for determining which one is parent/source(and which one will be resolved by customized HandlerMethodArgumentResolver).
All the strategies about resolving parameter in @SchemaMapping are marked with concrete class types(DataFetchingEnvironment,GraphQLContext ,Principal,) or annotation(@Argument,@Arguments), except for source.

The same thing occur in @BatchMapping((List<V>)). And developer may want to define list-type-argument on batch method,  the code below will  have problem, that's another issue discussed in #324.
https://github.com/spring-projects/spring-graphql/blob/b2b1e2d506aba402fb07bf7d3b149f439c7f729e/spring-graphql/src/main/java/org/springframework/graphql/data/method/annotation/support/AnnotatedControllerConfigurer.java#L312

Thanks for elaborating. For now, I'm closing this but we can always reconsider when it becomes an actual issue.


Use docker to run, QR code address prompt "unrecognized QR code"

[question]
Use docker to run, QR code address prompt "unrecognized QR code"
docker run -ti --name=mybot --rm -v="$(pwd)":/bot zixia/wechaty mybot.js

bingo! thx!


Adds more tests

This PR forks all the redis tests that were running on the docker-library/official-images.
I have added these as the external tests while we're using the bashbrew scripts to build, run, test and push it.

@inetol  @jrwren would you be interested in reviewing this PR?

@yosifkit @tianon would you please take a look at this PR?

@inetol  @jrwren would you be interested in reviewing this PR?

I'm not sure why you're copying all the tests, especially without changes; the bashbrew GitHub action will run the tests in config.sh in addition to the upstream DOI tests (which you can also reference in your config).
You can also explicitly set GITHUB_REPOSITORY to control the image name (https://github.com/tianon/docker-qemu/blob/72cd4ebd938b1628a5e6d0abbd2ceac1590aef88/.github/workflows/ci.yml#L34).
You might also want to switch from @HEAD to the latest tagged release of bashbrew so you're not alpha testing our changes like we prefer to. 😄

The reason I am doing this is, in future we might be moving away and we might want to modify the tests and add new ones.
I thought that it would have been better to keep the tests and the code in the same repo for maintainability.
WDYT @tianon, does it makes sense?


Add admiral/common/scripts/x86_64/Ubuntu_16.04/remote/installVault.sh

https://github.com/Shippable/pm/issues/10169

Completed.


Some browsers on the mobile side can not be used,such as  Some browsers on the mobile side can not use uc browsers etc...

After using this plugin, the browser on the computer side is good, but some browsers on the phone side are not available. Opera, chrome, firefox are all available but some of China's native browsers are not available. Look forward to you can solve this problem

Rendering effects is just like uc browser. Can not switch, nor render the correct language. Because in the mobile browser, I temporarily no way to find out the reason

my project use the requirejs + vue combination, when i use this plugin, the page is nothing but white screen directly.

The same problem. Tested on iPhone 4s.
If you have mac, you can add custom user agent  in Safari to reproduce this bug:
Mozilla/5.0 (iPhone; CPU iPhone OS 5_1 like Mac OS X) AppleWebKit/534.46 (KHTML, like Gecko) Version/5.1 Mobile/9B179 Safari/7534.48.3
No errors on console. Empty page.

Could be the issue referenced here https://github.com/requirejs/requirejs/issues/1234

Hi again. Thank you for your react reply. I reproduced bug. There are my
steps to reproduce it.

I created new vue project via vue-cli (vue init webpack-simple)

Then to connect from my iphone i added line in dev script
(<IP_ADDRESS> - my local IP):
"scripts": {
"dev": "cross-env NODE_ENV=development webpack-dev-server --inline --port
8080 --host <IP_ADDRESS> --content-base .",
"build": "cross-env NODE_ENV=production webpack --progress --hide-modules"
}

In main.js file i added vuex and vuex-i18n

import Vue from 'vue';
import Vuex from 'vuex';
import vuexI18n from 'vuex-i18n';
import App from './App.vue';
Vue.use(Vuex)
const store = new Vuex.Store();
Vue.use(vuexI18n.plugin, store);
const translationsEn = {
"content": "This is some {type} content"
};
const translationsDe = {
"My nice title": "Ein schöner Titel",
"content": "Dies ist ein toller Inhalt"
};
Vue.i18n.add('en', translationsEn);
Vue.i18n.add('de', translationsDe);
Vue.i18n.set('en');
new Vue({
store,
el: '#app',
render: h => h(App)
})

Result is blank page

I hope this will help us to find any solution to make vuex-i18n capability
with old safari versions :)
2018-02-27 16:25 GMT+02:00 Ramon Saccilotto<EMAIL_ADDRESS>
Could be the issue referenced here requirejs/requirejs#1234
https://github.com/requirejs/requirejs/issues/1234
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/dkfbasel/vuex-i18n/issues/68#issuecomment-368894476,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AalAJfxHuHrlbWS8AOKj6t-ac7Q9yGw-ks5tZBA-gaJpZM4Rx0e4
.

Hi again. Thank you for your react reply. I reproduced bug. There are my
steps to reproduce it.

I created new vue project via vue-cli (vue init webpack-simple)
Then to connect from my iphone i added line in dev script
(<IP_ADDRESS> - my local IP):

"scripts": {
"dev": "cross-env NODE_ENV=development webpack-dev-server --inline --port
8080 --host <IP_ADDRESS> --content-base .",
"build": "cross-env NODE_ENV=production webpack --progress --hide-modules"
}

In main.js file i added vuex and vuex-i18n

import Vue from 'vue';
import Vuex from 'vuex';
import vuexI18n from 'vuex-i18n';
import App from './App.vue';

Vue.use(Vuex)

const store = new Vuex.Store();
Vue.use(vuexI18n.plugin, store);

const translationsEn = {
  "content": "This is some {type} content"
};

const translationsDe = {
  "My nice title": "Ein schöner Titel",
  "content": "Dies ist ein toller Inhalt"
};

Vue.i18n.add('en', translationsEn);
Vue.i18n.add('de', translationsDe);

Vue.i18n.set('en');

new Vue({
store,
el: '#app',
render: h => h(App)
})

Result is blank page

I hope this will help us to find any solution to make vuex-i18n capability
with old safari versions :)

Hi all.
Is not a vuexI18n problem but a vuex problem in general of the UC browser.
Vuex uses promises but the UC browser does not support promise at default.
See the issue in the vuex github repo.
Using a polyfill should solve your problem.
https://cdn.polyfill.io/v2/polyfill.min.js?flags=gated&unknown=polyfill
For the UC browser you also have to set additional polyfill params:
flags=gated to avoid override of native implementations
unknown=polyfill to use polyfill for unrecognized user-agents what in this case the UC browser is.
Here is an example running on my Android test environment using the latest version of the UC browser:
<!DOCTYPE html>

	<meta charset="utf-8">
	<title>vuex-i18n</title>

message.title: {{ $t('message.title') }}

You also need to make sure, that only es5 javascript syntax is used in these older browsers.


Fix resource_quota_sync deepEqual bug

Problem(#24502)
Using reflect.DeepEqual to compare resource.Quantity can lead to the compare results are not expected. This problem is particularly acute when setting CPU related quotas. Code Line 1 Code Line 2
Solution
Using k8s.io/apimachinery/pkg/api/equality Sementic Ignore formatting, only care that numeric value stayed the same. References kubernetes quotaController code

@prachidamle @dramich @loganhz  already added a unit test.


fix: translate non-localized parts to zh-TW

Checklist

There is no reference issue for this pull request

What kind of change does this PR introduce?
There were non-localized parts that needs to be translated to zh-TW

I am closing this PR since the commits of other fix has been queued with the current one, sorry for the inconvenience caused!


dv pages doesn't find all files

I want to create a property that allows me to select a link from among the files in the folder. To do this, I use the construct in the Dataview query field
dv.pages('"Foldername"')
As a result, I only get .md files, But I need a list of all files (.pdf, office, etc) in this folder.
Please help!

there's a new "Media" field type for it. Does it suit your needs?

Could the Media field be enhanced to allow filtering by folder? For instance, displaying all files from a specific folder like "Resources," regardless of their extensions or media types.
Here's the structure of the vault:
Resources
  Canvas
  Drawio
  Excalidraw

It would be helpful if media property field could list all media from the "Resources" folder, allowing us to select and attach them. Nowadays, it just shows Drawio files.
Another suggestion is to implement filtering for these media files using DV. For instance, allow/filter all media from the "Resources" folder where the file names match the name of the note or other criteria using dv.

Could the Media field be enhanced to allow filtering by folder? For instance, displaying all files from a specific folder like "Resources," regardless of their extensions or media types.
Here's the structure of the vault:
Resources
  Canvas
  Drawio
  Excalidraw

It would be helpful if media property field could list all media from the "Resources" folder, allowing us to select and attach them. Nowadays, it just shows Drawio files.
Another suggestion is to implement filtering for these media files using DV. For instance, allow/filter all media from the "Resources" folder where the file names match the name of the note or other criteria using dv.

If anyone needs something similar, here's how I managed to do it
this.app.vault.getAllLoadedFiles().filter(
file => file.path.startsWith("Resources")  && [ ".canvas", ".drawio", ".excalidraw", "excalidraw.md" ].some(
ext => file.path.endsWith(ext))
).map(file => "[[" + file.path + "]]"
)


Fix data race in vtgate unit test.

Changed sync2.Atomic* types to make sure they cannot be used without going through the atomic interface.
@sougou @guoliang100 @alainjobart
I suggest to review commit by commit and not using the "Files changed" tab.

This is great. Definitely will make the unit tests less flaky.
(I submitted a one-line fix to a race condition I saw, should be the same fix as yours, so the merge should be easy, sorry about that).
LGTM

LGTM


IDA Free 7.6.210526 - Exporting second database fails

Hello,
I have tried this on both macOS and Windows and keep running into the same exact issue.
I am currently using IDA Free 7.6 to perform my analysis on a mach0 file.
The analysis completes find, however whenever I try to perform a BinDiff it always reports back:
"Error while diffing: Exporting the secondary database failed. Is it opened in another instance?
Please close all other IDA instances and try again."
Is this a known issue when using the free version of IDA?
Is there a fix for this ???
On a related note:
I tried running BinExport on each database and then ran the BinDiff app to perform a diff.
It completes, however, the "Single Function Diff Views" reports 0. And provides no diff information.
It does show a pie chart of the percentage of differences.
Note: I tried this on windows examing a windows exe and had the exact same issue.
Thank You,
Frank

Hi there,
Thanks for reporting this issue. Note that we do not test with the free version of IDA and I don't even know whether plugins are officially supported in that version, or whether they "half-work".
I need to test this myself to confirm.
As for the UI, the way "Single Function Diff Views" behaves is by design -- this feature is intended for saving single flow graph diffs from different IDBs. It is also badly implemented and thus only rarely used/useful.
Instead, after you have performed the diff from the UI, load and expand it by double-clicking it. Then you should be able to see "Matched Functions" and unmatched functions, as well as "Call Graph".
Christian


when disable write typetree, am.LoadAssetsFileFromBundle returns null

If I use the code below to build bundles
BuildPipeline.BuildAssetBundles(dir, BuildAssetBundleOptions.DisableWriteTypeTree, BuildTarget.WebGL);

and use the code below to read a bundle. am.LoadAssetsFileFromBundle always returns null.
            var am = new AssetsManager();

            am.LoadClassPackage("classdata.tpk");

            var bun = am.LoadBundleFile("file/bg_11zhuan1.bundle");
            var inst = am.LoadAssetsFileFromBundle(bun, bun.file.GetFileName(0));

If I use BuildAssetBundleOptions.None instead of  BuildAssetBundleOptions.DisableWriteTypeTree to build bundles, am.LoadAssetsFileFromBundle doesn't return null.
What should I do I, if I want to use BuildAssetBundleOptions.DisableWriteTypeTree?

You can use this bundle to test.

You can use this bundle to test.

The code that checks whether a file is an assets file expects the version string to not contain any special characters. In your case, you have a custom engine string ending with \n2 (common for custom engine versions) that causes this check to fail. Since this is fairly common, I don't mind adding this to the regex.

It works,Thanks very much!


Implement NEP-413: signMessage method

Story
As a developer of application using the Near protocol, I want to use the NEP-413 proposal the methods it describes so I can sign piece of data for the purposes of authentication.

NEP-413: https://github.com/near/NEPs/pull/413

Acceptance Criteria

[ ] The signMessage method is implemented according to the NEP-413 proposal.
[ ] There is no gas fees nor transaction being made costing the user any Near.
[ ] Mobile ok
[ ] Tested

Unhappy Path

Invalid parameters
The user declines the signing of the message
The account has no full-access key (Locked Account) and thus unable to perform the signing?

+1 on this Issue

+1 also

is the signMessage method available or not yet ?


VTOL: Only on Main OUT problem

Hello,
I rewire and reconfigure now the VTOL in order to have all the servos and motors connected to the main out. Here is the custom confiog.txt
set MIXER vtol_quad_x
set PWM_OUT 12345678
set PWM_RATE 400
set PWM_DISARMED 900
set PWM_MIN 1100
set PWM_MAX 1900
set MIXER_AUX vtol_delta
set PWM_AUX_RATE 400
set PWM_AUX_OUT 1234
set PWM_AUX_DISARMED 1500
set PWM_AUX_MIN 1000
set PWM_AUX_MAX 2000
Here is custom extras.txt
pwm disarmed -c 12 -p 1500
pwm min -c 12 -p 1000
pwm max -c 12 -p 2000
pwm disarmed -c 34 -p 900
pwm min -c 34 -p 1000
pwm max -c 34 -p 2000
pwm failsafe -c 1 -p 1500
pwm failsafe -c 2 -p 1500
pwm failsafe -c 3 -p 1100
pwm failsafe -c 4 -p 900
pwm terminatefail on
The custom mixer vtol_quad_x.main.mix
VTOL quad X mixer for PX4FMU
#=============================
Elevon mixers
Three scalers total (output, roll, pitch).
On the assumption that the two elevon servos are physically reversed, the pitch
input is inverted between the two servos.
M: 2
O:      10000  10000      0 -10000  10000
S: 1 0   7500   7500      0 -10000  10000
S: 1 1   6500   6500      0 -10000  10000
M: 2
O:       10000   10000    0 -10000  10000
S: 1 0    7500    7500    0 -10000  10000
S: 1 1   -6500   -6500    0 -10000  10000
Throttle mixer
M: 1
O:      10000  10000      0 -10000  10000
S: 1 3      0  20000 -10000 -10000  10000
Z:
R: 4x 10000 10000 10000 0
And the custom AUX mixer vtol_delta.aux.mix
Z:
Z:
Z:
Z:
Z:
Z:
So on the bench test, when I flip the switch to FW, the 5th pusher motor starts spinning with transition throttle. When transition airspeed is reached, the pusher motor shuts down immediately despite I'm in manual and with 50% throttle. The servos go to their min position and lock without control from the RC. When I flip back to quad the quad starts but the pusher motor start spinning on idle speed. I cannot stop the pusher motor until i disarm with the button.
Any ideas?

When I reversed the order it worked perfect!!
VTOL quad X mixer for PX4FMU
#=============================
R: 4x 10000 10000 10000 0
Elevon mixers
Three scalers total (output, roll, pitch).
On the assumption that the two elevon servos are physically reversed, the pitch
input is inverted between the two servos.
M: 2
O:      10000  10000      0 -10000  10000
S: 1 0   7500   7500      0 -10000  10000
S: 1 1   6500   6500      0 -10000  10000
M: 2
O:       10000   10000    0 -10000  10000
S: 1 0    7500    7500    0 -10000  10000
S: 1 1   -6500   -6500    0 -10000  10000
Throttle mixer
M: 1
O:      10000  10000      0 -10000  10000
S: 1 3      0  20000 -10000 -10000  10000
Z:


FI: Fixed URN:UVCI to capital letters. Added "CERTIFICATE" to json

Fix merge conflicts

Will update the pull request with fixed merge conflicts


HttpRequest.FromTcpClient

how to build from System.Net.Sockets.TcpClient

This method was removed long ago; please don't use it.  For a better reference on building HTTP from an incoming TCP request, look at https://github.com/jchristn/httpserverlite


Initialize the project

Start the project with the framework needed and configuring the details of the project

Completed the issue! ✅


UP Sender Transform detects schema circular dependencies in schema configuration

User Story:
As developer, I want the UP sender transform to detect when there is a schema circular dependency, so I can fix the issue.
Description/Use Case
The UP sender transform configuration allows for child schemas (a schema that includes another schema).  There is the possibility that someone could include a schema that was already included at a higher level which will result in an infinite loop.  We need to detect any circular dependency, so the pipeline is more robust.
Risks/Impacts/Considerations

Need to make sure to detect circular dependencies between child and parents and not confuse it with a reuse of a schema.

Dev Notes:

Child schemas could be reused within a schema, so care must be taken to not remove this feature and only detect circular dependencies.

Acceptance Criteria

The UP sender transform throws an error (e.g. SchemaException) if it detects a schema circular dependency.

Work is already done, moving to product review to demo it.


Datetime refactor

Description
Refactors the useDateTimeDiff method into a hook with a single parameter, the reference date.
Issue / Notion doc (if applicable)
closes https://github.com/decent-dao/fractal-interface/issues/906
Screenshots (if applicable)

Should all timestamps - like this one below - show "time ago" instead of the actual time if it was recent enough? The one below was less than 5 minutes ago, but still shows the timestamp

Should all timestamps - like this one below - show "time ago" instead of the actual time if it was recent enough? The one below was less than 5 minutes ago, but still shows the timestamp

hmm, yeah this should have no impact on that, since it wasn't using this method, but we could show the "time ago" there as well, and go to a date if it's past a certain time

Ok, yeah if we didn't intend for this to affect those timestamps, then this LGTM


Pygmy Mailhog is not available to PHP containers on mac

The GovCMS team made change to the scaffolded docker stack to attach the PHP container to the amazeeio network that Pygmy creates, this was done to resolve issues for some users where they could not test mail configuration from Drupal.
It appears on mac (or potentially some other docker configurations) that mailhog is listening on host.docker.internal instead of the internal IP (<IP_ADDRESS> 1025).
https://github.com/govCMS/scaffold/issues/94

This may be due to Docker network configuration. I have also encountered similar problems when using Docker for development, especially when switching between different operating systems.
On macOS, host.decker.internal is a special DNS name that resolves to the IP address of the host machine. This may cause network connectivity issues in some cases, especially when containers attempt to access services through internal IP addresses (such as <IP_ADDRESS>).
Here are two solutions:

Replace Docker, as far as I know, Servbay is more suitable for Mac.

Use host.decker.internal:
Ensure that your PHP container uses host.decker.internal to access Mailhog. You can set the Mailhog address to host.decker.internal: 1025 in the PHP configuration file or code.
$mailhogHost = 'host.docker.internal';
$mailhogPort = 1025;
Update Docker Compose file:
Ensure that the PHP container and Mailhog container are on the same network in the docker-compose.yml file. If you are using a custom network, you can configure it as follows:
version: '3.8'
services:
php:
image: your-php-image
networks:

my-network
mailhog:
image: mailhog/mailhog
ports:
"1025:1025"
networks:
my-network
networks:
my-network:
driver: bridge
Check Pygmy configuration:
Ensure that Pygmy is properly configured and started. In some cases, it may be necessary to restart the Pygmy service to apply the new configuration:
pygmy up
Manually add host.decker.internal:
In some cases, you may need to manually add host.decker.internal to the/etc/hosts file. You can achieve this through the extra_ hosts configuration option when starting a Docker container:
services:
php:
image: your-php-image
extra_hosts:
"host.docker.internal:host-gateway"

On mac, you can also add an environment variable to the php&cli containers that will auto-configure whilst we wait for #1073 to come in.
SSMTP_MAILHUB: host.docker.internal:1025
The extra_hosts method will be useful in linux later as an alternative to the <IP_ADDRESS> default too.

I've been using the SSMTP_MAILHUB: host.docker.internal:1025 override in my docker-compose.yml since April 2023 (more than a year) and it works really well.


Emoji in article titles are not rendered correctly

Probably best to strip emoji from article titles.

May be because of database charset.

Seems to be solved now.

Seems to be solved now.


feat: index.html Clarify how to re-add media folder after running tool

Small addition to explain where to put the tweet media folder in the archiver output.

Thanks Andy!


Bring across the git version extraction from g3log to support windows as well

I could do a pull request but it is probably quicker for you to just copy paste that part

Thanks for the nudge. Yes. I'll do that

@KjellKod Maybe it is not worth doing.
I think the cmake build needs a few fixes for this to build fine in windows.
It is pretty hard as it is now to specify the headers of g3logger as well as the include path of boost.
My current hack from command line is by specifying the below definitions which are not accesible from gui

-DBoost_INCLUDE_DIR=../path_to_/boost-headers/
-DCMAKE_PREFIX_PATH=../path_to_/g3log-static/

However if you link dynamically to boost in windows you also need

DBOOST_ALL_DYN_LINK
so that you link to the import lib filename of  boost_* rather than the libboost_* lib

I will be able to probably send you a pull request next week.

The attached file tries to have as minimal changes as possible and should cause no issues or changes to *NIX.
You can have a look at it but made small changes to:

add version extraction from g3log
add G3_LOGGER_INCLUDE_PATH variable for the header path of g3logger
add G3_LOGGER_LIB_PATH variable to specify search path for g3logger
add separate BOOST_SEARCH_PATH variable to specify search path for g3logger
add BOOST_INCLUDE to specify boost include path
add BOOST_ALL_DYN_LINK to support dynamic linking to boost, off by default

CMakeLists.txt

Interesting.
In OSX and Linux boost is found automatically.
I think It's time to put up cloud CI for g3sinks as well.

I wish that was the case in Windows land !
I think I had some success in the past by setting the Boost_Root or BOOST_DIR manually.
I could it give it a try again but I think it still depends on how the package is sorted.
E.g. some use nuget packages so having the ability to specify the options manually is always good.


Add stress parsing functionality

Fixes ondracka/electronic-parsers#2

This is looking good, just few nitpicks and please add some tests (we should test at least three cases, one where we calculate the stress tensor, one where we calculated it but stdout is not there and one where we don't calculate it). I'm not sure if the second one is there, but in theory we could just run the good case, than remove the stdout and test run it again.

Tests added.

Thanks Jitka, I'll add few more comments and send this upstream, you can take on another issue.


HLA Typing Issue

Hi,
I am running into an issue with the HLA typing that I cannot figure out. The program fails with the message:
Traceback (most recent call last):
File "bin/optitype_ext.py", line 31, in
with open(input_optitype_result) as f:
IOError: [Errno 2] No such file or directory: '/root/data/hlatyping/2021_01_08_05_28_37/2021_01_08_05_28_37_result.tsv'
hla type process done.
And when I look at the log file for the hla typing it says:
/root/source/seqan-seqan-v2.4.0/include/seqan/basic/basic_exception.h:363 FAILED!  (Uncaught exception of type std::bad_alloc: std::bad_alloc)
From my research it looks like it is possibly a memory problem but I have expanded the volume I am working on twice with no change in the error. Do you have any estimates of how much space running the entire docker image requires?
Thank you!
-Elizabeth

Hi, ElizabethBorden,
Sorry for the delay. The HLA typing procedure in the pTuneos is performed by Optitype, which uses razers3 for reads mapping and extracting. As razers3 puts all mapped reads into memory, it requires large memory(>200G) if the sequencing depth of your data is deep (maybe >200X).
This issue could be handle by calling the HLA type using other tools such as POLYSOLVELR, xHLA, SOAP-HLA, hla-genotyper, HLA-HD, Kourima and, HLA-VBSeq by yourself. When you get the four-digital HLA type of your sample, you can provide it in the configure file field "hla_str". Then pTuneos would omit this procedure when running the whole pipeline.
Hope it helps!
Best,
Chi

Hi, ElizabethBorden,
Sorry for the delay. The HLA typing procedure in the pTuneos is performed by Optitype, which uses razers3 for reads mapping and extracting. As razers3 puts all mapped reads into memory, it requires large memory(>200G) if the sequencing depth of your data is deep (maybe >200X).
This issue could be handle by calling the HLA type using other tools such as POLYSOLVELR, xHLA, SOAP-HLA, hla-genotyper, HLA-HD, Kourima and, HLA-VBSeq by yourself. When you get the four-digital HLA type of your sample, you can provide it in the configure file field "hla_str". Then pTuneos would omit this procedure when running the whole pipeline.
Hope it helps!
Best,
Chi

That is very helpful! Thank you for your reply and I will try that!

That is very helpful! Thank you for your reply and I will try that!


No rows affected after UPDATE command

This is still a little bit mysterious issue, so investigation is in progress. Here is related soci-users thread get_affected_rows() after UPDATE.
The problem is leaking in SOCI/soci/builds/2986243 as:
 /home/travis/builds/SOCI/soci/src/backends/sqlite3/test/test-sqlite3.cpp:290:
    void test6(): Assertion `st1.get_affected_rows() == 10' failed.

I've briefly tracked down the problem and it looks that for UPDATE SQL command, the relevant sequence is this:

satement_impl::execute(false) executing equivalent of this:

int num = 0;
statement_backend::exec_fetch_result res = backEnd_->execute(num);

sqlite3_statement_backend::execute(number=0)
sqlite3_statement_backend::load_rowset(totalRows=0) in sqlite3/statement.cpp:87
totalRows leads to empty dataCache_.resize(totalRows); in sqlite3/statement.cpp:100
For empty dataCache_, no SQL command is executed
Result: UPDATE SQL with prepared statement has no effect

The bug is definitely fixable, but what puzzles me is

has it never worked?
if it worked, when it's got broken,

since the implementation of execute/load_rowset hasn't changed since the beginning of time:

core/statement.cpp
sqlite3/statement.cpp

As I haven't touched many places in SOCI for ages, before I try to fix
something, I'd rather prefer to understand if/when it's got broken.

We have test_get_affected_rows which checks effect of UPDATE SQL and exercises the SQLite backend.
I assume this has been fixed.


Hi Team - Announcement

I think it's better if we have a task listed on the project and assign someone who will spearheading the project. anyone? :)

I am interested.


Test under Node.js 4.2 (LTS)

@gaearon Do you think testing io.js on Travis brings any value, now that they've merged back into Node.js? Should we just test 4.2? Should we add 0.12 (and maybe 0.10) as they are still relatively popular?

Yes, this sounds like a good plan.
I don't have any preference here so do what you deem reasonable.


Only the first text is previewed correctly for a list of encrypted texts.

Hi, @meld-cp.
I found a preview issue with "In-place Encryption".
Only the first text is previewed correctly for a list of encrypted texts.
Source view

Preview view

I hope this issue is fixed.
Thank you.

The same situation is for tables:


Make a clear separation of public and internal message packages

Right now all of RM's ROS messages are defined by packages inside of rmf_internal_msgs. These message packages contain many internal messages that are unstable (i.e. their definitions may change at any time to accommodate implementation/design changes). This creates problems for packages downstream of RMF that want to integrate using ROS messages. There are currently no explicitly public, stable messages available for them to tap into.
To fix this issue, we should think very carefully about what the exact inputs and output of an RMF core system should be, and then create explicit, stable public API messages for downstream systems to tap into.

Closing as rmf_api_msgs was introduced since.


Compression is likely not working.

I've been playing around with the compression algorithms in hdf5 and observed that the Blosc compression algorithms basically do nothing. I suspect this is a bug on your end (though also possible on my end)
Here's my evaluation, hope it helps =]

See #273 for a potential duplicate

Great, I'll not worry about it for now then and hope a fix turns up in time. Thanks for the help!

You are very welcome to make a PR exposing blosc_sys::blosc_set_blocksize as hdf5::filters::blosc_set_blocksize and another adding features blosc-lz4 which enables `blosc-sys/lz4 etc.!


numericality greaterThan is always required

Hello,
I my case price field is optional, but if it's filled, then it should be a positive integer.
How do I configure the schema?

var constraints = {
  price: {
    numericality: {
      onlyInteger: true,
      greaterThan: 0
    }
  }
};

validate({}, constraints);
// => undefined

validate({price: 0}, constraints);
// => {price: ["Price must be greater than 0"]}

validate({price: 1.2}, constraints);
// => {price: ["Price must be an integer"]}

Nicklas, my config is:
price:
	numericality:
		greaterThan: 0
		onlyInteger: true
		notGreaterThan: "Положительное число"

I see 2 problems.

I get must be greater than 0 exception for an empty field
notGreaterThan option is replaced with default error message

When I enter positive integer, I see no error message.
Version is 0.4.0

You cannot set a greater than only message like that. The validators only accepts "message", the other ones are when setting default messages.
I don't really understand problem #1, the validator will accept all integers above 0 and reject all values that aren't empty.
On Thu, Feb 12, 2015 at 12:29 PM, droganov<EMAIL_ADDRESS>wrote:

Nicklas, my config is:
price:
	numericality:
		greaterThan: 0
		onlyInteger: true
		notGreaterThan: "Положительное число"

I see 2 problems.

I get must be greater than 0 exception for an empty field
notGreaterThan option is replaced with default error message
When I enter positive integer, I see no error message.
Version is 0.4.0

Reply to this email directly or view it on GitHub:
https://github.com/ansman/validate.js/issues/31#issuecomment-74056469

Ok, I can reconfigure messages, but as for validation,  I expect validator to accept empty value, since the field is not required.

But it does accept empty values as in my example? Or do you have an example of where it doesn't work?

No. it doesn't accept empty values. Here is my config:
https://github.com/droganov/Sdelkino.com-statics/blob/master/dev/blocks/form-classy/form-classy.coffee#L29
Rendered page is here: http://www.sd-statics.supremedesign.ru/publish.html

Ah, the problem is that it doesn't accept empty strings. I'll make a fix for this.

Great, thank you!


Trying Mill & not sure why I am not getting

Hi:
Trying to use your wonderful tool for mill
  import mill.modules.Assembly
  import coursier.maven.MavenRepository

  //object cor_poc extends ScalaModule {
  object ai_io extends ScalaModule {
+   def name = "ai_io_grab_data"
    def scalaVersion = "2.12.10"
+   def sparkInstanceCount = 5
    def sparkAwsRegion = "us-east-1"
    def sparkS3JarFolder = "s3://hn-sandbox/emr_jarlys/"

    def repositories = super.repositories ++ Seq(
      MavenRepository("http://dl.bintray.com/spark-packages/maven"),
      MavenRepository("https://mvnrepository.com/artifact/org.apache.spark/spark-yarn"),
      MavenRepository("https://mvnrepository.com/artifact/org.apache.hadoop/hadoop-aws"),
      MavenRepository("https://mvnrepository.com/artifact/com.amazonaws/aws-java-sdk")
    ),
     MavenRepository("https://mvnrepository.com/artifact")
     // net.pishen/sbt-lighter")
   )

(snip)...

    def ivyDeps = Agg(
      ivy"org.apache.spark::spark-core:3.0.1",
      ivy"org.apache.spark::spark-sql:3.0.1",
      ivy"org.apache.spark::spark-yarn:3.0.1",
      ivy"io.delta::delta-core:0.7.0",
      ivy"org.apache.hadoop:hadoop-aws:2.7.7",
     ivy"com.amazonaws:aws-java-sdk:1.7.4",
     ivy"net.pishen:sbt-lighter:1.2.0")

I suspect
MavenRepository("https://mvnrepository.com/artifact")
or
MavenRepository("https://mvnrepository.com/artifact/net.pishen/sbt-lighter")
is not right places to go. Can you tell me where I need point to?
BTW: Also tried       ivy"net.pishen::sbt-lighter:1.2.0"
Also saw that publishMavenStyle := false in your sbt file. Darn~
Kind regards,
1 targets failed
ai_io.resolvedIvyDeps
Resolution failed for 1 modules:
--------------------------------------------
  net.pishen:sbt-lighter:1.2.0
        not found: /home/syoon/.ivy2/local/net.pishen/sbt-lighter/1.2.0/ivys/ivy.xml
        not found: https://repo1.maven.org/maven2/net/pishen/sbt-lighter/1.2.0/sbt-lighter-1.2.0.pom
        not found: http://dl.bintray.com/spark-packages/maven/net/pishen/sbt-lighter/1.2.0/sbt-lighter-1.2.0.pom
        not found: https://mvnrepository.com/artifact/org.apache.spark/spark-yarn/net/pishen/sbt-lighter/1.2.0/sbt-lighter-1.2.0.pom
        not found: https://mvnrepository.com/artifact/org.apache.hadoop/hadoop-aws/net/pishen/sbt-lighter/1.2.0/sbt-lighter-1.2.0.pom
        not found: https://mvnrepository.com/artifact/com.amazonaws/aws-java-sdk/net/pishen/sbt-lighter/1.2.0/sbt-lighter-1.2.0.pom
        not found: https://mvnrepository.com/artifact/net/pishen/sbt-lighter/1.2.0/sbt-lighter-1.2.0.pom

Haven't used mill before. But maybe you can try
https://dl.bintray.com/sbt/sbt-plugin-releases/


Input to enable ECR login and select Account ID.

Context
Input to enable ECR login and select Account ID.
DOD

[ ] Possibility to enable ECR login.

▶ Action Inputs

Field
Mandatory
Default Value
Observation

AWS_ECR_LOGIN
NO
false
Enable screen login to use ECR images in CONTAINER_URL.

AWS_ECR_ACCOUNT_ID
NO
""
If the image repository is in another account.

Solution in the issue: https://github.com/stack-spot/idp-reusable-workflows/issues/71


Estimated camera parameters

Discussed in https://github.com/Arthur151/ROMP/discussions/447

Originally posted by jamesyoung0623 June  1, 2023
Hello,
Thank you for the great work!
I want to ask how to construct the intrinsic and extrinsic parameters from the data in saved npz files.
There are items like cam, global_orient, center_preds, center_confs, and cam_trans but I don't know how to construct the intrinsic and extrinsic parameters with these data.
Can someone help?
Thanks in advance.

Discussed in #447
Originally posted by jamesyoung0623 June 1, 2023 Hello, Thank you for the great work! I want to ask how to construct the intrinsic and extrinsic parameters from the data in saved npz files. There are items like cam, global_orient, center_preds, center_confs, and cam_trans but I don't know how to construct the intrinsic and extrinsic parameters with these data. Can someone help? Thanks in advance.

I believe this comment can help you https://github.com/Arthur151/ROMP/issues/421#issue-1600146589

Thanks a lot!


Include test vectors for BLS bindings

We should include test vectors from:

Test vectors from IRTF draft on the Optimal Ate Pairing
Test vectors created internally, using the zkcrypto library

Included test vectors for zkcrypto in #408 . Test vectors from the IRTF draft for BLS signatures are being help up because their test vectors seem to be invalid. I notified them, and here is there response I got from one of the draft authors:
I am still waiting to receive from the co-author who created the
generation script of the test vectors listed in I-D.
On the other hand, I have checked by the library mcl and confirmed
that the results by mcl are equal to yours.

https://github.com/herumi/mcl

I'm sorry, but from this situation, we will review the I-D for
possible errors in the test vectors in the I-D.

When the IRTF draft is updated, we will include valid test vectors from there. Until then, closing this as fixed.


Verify if mkdir container can be safely removed

DevWorkspace controller from POC stage run mkdir containers as init containers but that's not going to help solve files permissions issues if other init containers mounts some subfolders, because (as Che used to do it) subfolders must be initialized before they are mounted.
But since it works somehow with the current approach, and Angel heard that it might be resolved on k8s side(we don't have a good referene), WE MUST make sure it works on K8s/OpenShift clusters and remove mkdir init container at all,
OR such permissions issue still exists on some k8s/openshift clusters - we should rework it in the proper way, where subfolders are inited from a separate pod before they are mounted into devworkspace pod.
For more see https://github.com/eclipse/che/blob/master/assembly/assembly-wsmaster-war/src/main/webapp/WEB-INF/classes/che/che.properties#L338

I'm not sure if it can be removed safely. Despite the fact that currently, DevWorkspace Operator has kind of no-op implementation and we don't have any issues on crc or minikube users still reports issues, like

k8s 1.17 via eks as well
https://github.com/kubernetes/kubernetes/issues/41638#issuecomment-680666823

Che (and I suppose CRW) has it enabled by default:
github.com/eclipse/che/...che.properties#che.infra.kubernetes.pvc.precreate_subpaths
We can drop it for now since the current implementation does not work as it's supposed to work but I think we should return back to it later, and implement a pluggable mechanism that would pre-create subpaths in the right way, with a dedicated pod (not init container). But that should be smarter that Che has currently and it does not need to be run on each workspace start. Volumes should be analyzed and precreated subpaths should be stored somewhere in PVC annotations to avoid unneeded runs.

It's proven that the current implementation with initContainer is not able to handle permissions issues in the proper way, so this functionality is cleaned up https://github.com/devfile/devworkspace-operator/pull/211
Init subpaths should be reimplemented in the proper way withing https://github.com/devfile/devworkspace-operator/issues/214


A puzzled

When learning the Redux source code, I was confused by variable currentListeners and nextListeners in createStore function.
I think nextListeners variable is unnecessary, beacause currentListeners will always equal to nextListeners when dispatch the action. In addition, only nextListeners shallow copy could happenned in subscribe function.
Could you please tell me why set two arrray here.thx.

Looks like the code was added in commit c031c0a8d900e0e95a4915ecc0f96c6fe2d6e92b .  Per Dan's commit message:

Delay copying listeners until is necessary
We can avoid excessive slice() calls by only copying the listeners if they are modified during dispatch()


Added DrizzlePac Align Mosaics notebook

The DrizzlePac Align Mosaics notebook is ready to be added to the spacetelescope notebook repository.

Re: the most recent failure: the build log indicates “No output has been received in the last 10m0s, this potentially indicates a stalled build or something wrong with the build itself.”   That means one of two things

some long (>10 min) calculation is required, which is a no-go for these notebooks (and we have to figure out an alternate plan)
something is getting run that’s waiting for user input, which will of course never finish

To test this you can run in locally and see if some part is really slow (option 1) and if not, my money is on ginga doing something.  The thing is, ginga is only used there for the zscale algorithm, which is already in astropy anyway: http://docs.astropy.org/en/stable/api/astropy.visualization.ZScaleInterval.html - if you can just switch to using that, maybe problem solved?

Hey @eteq the rebase failed because spacetelescope and my branch both had changes to the travis.yml file. I'm not sure how to resolve that, but my newest commit removed the ginga dependency completely from the notebook.
Also, I ran the cell locally, and the longest cell run time was ~3mins, so I don't know why it would be failing with a 10 minute limit.

@samanthalh - the conflict was because there was a change made to the dependency list in a different notebook. A rebase just adding in ccdproc would have done the job, but because github makes it easy I just did an in-browser conflict resolution, and we shall see if that is all that's needed to get this working.

In Step 7, perhaps move the text showing how to call astrodrizzle with crrejection into the cell below but leave it commented out.  It looks a bit strange with the current formatting as text.
'To further improve cosmic-ray rejection in the chip gap, AstroDrizzle may alternatively be run with all steps turned on, as shown below.'
#astrodrizzle.AstroDrizzle('*flc.fits', output='f657n', preserve=False, clean=False, build=False, context=False, resetbits=0, skymethod='match', driz_sep_bits='64,32', combine_type='minmed', final_bits='64,32', final_wcs=True, final_scale=0.04, final_pixfrac=1.0, final_rot=-35, final_ra=274.721587, final_dec=-13.841549, final_outnx=8000, final_outny=8400)


Insert custom local system resources

This modifies some of the traits on Local<T> causing it to only initialize T using FromResources if the resource T doesn't yet exist on the system. This allows having a Local<T> on a system which you then initialize later using whatever value of T you'd like.
In addition, this adds some convenience functions to AppBuilder that allow easily adding a custom T value to a given system.

I think the "only set Local if it hasn't been initialized" part of this is good to go, but I think the convenience functions are a bit too "hard coded" as a system could have more than one local resource.
I think I'd rather have an api like this:
app.add_system(something.system())
  .add_system_local(something.name(), SomeResource)

Retrieving a system id/name without a system instance is something thats still being discussed. We'll want whatever api we use for "adding system locals" to line up with other apis like "adding explicit manual dependencies".
Can we scope this pr to just the "only set Local if it hasn't been initialized"  for now? Then we can spec out the "address system by name and/or id without re-initializing the system" design elsewhere.

I specifically made sure to do those two in separate commits so the convenience functions could be shelved if they aren't as agreed upon. That said, I'm not entirely sure what you mean by the "Retrieving a system id/name without a system instance" part. As far as I understand what you mean, this doesn't require that?
Personally, I know that if these conveniences aren't implemented, I'll be making my own convenience trait that effectively does this:
app.init_system(system.system(), |resources| {
    let system = system.system();
    resources.insert_local(system.id(), local_resource_to_init_with);
    system
})

Did you mean that it's presently not possible (or in debate for how it should be possible) to call Resources::insert_local with a system that's not been "initialized" in some sense yet? In that case, the above convenience function wouldn't work either.

I mean that right now if you dont have a direct reference to Box<dyn System> then you cannot get the name or the id. You're right to say that a builder like the above would work. But rather than adding "yet another" set of AppBuilder system registration variants, I think id rather build a separate api that can stand alone / compose with all of the other existing app builder system functions.
If in the short term you can build your own helpers, I think thats preferable while we work out what the final api looks like.

I understand, makes sense. I'm going to get to removing that commit soon and then this can hopefully get merged

Rebased and removed the appbuilder changes. Should be all ready to merge now!


azure: Cannot build the podvm image

After following the build instructions from here when I run make image the packer fails with mandatory fields as follows:
$ make image
...
packer build ./ubuntu/
Error: Unset variable "plan_product"

A used variable must be set or have a default value; see
https://packer.io/docs/templates/hcl_templates/syntax for details.

Error: Unset variable "plan_publisher"

A used variable must be set or have a default value; see
https://packer.io/docs/templates/hcl_templates/syntax for details.

Error: Unset variable "plan_name"

A used variable must be set or have a default value; see
https://packer.io/docs/templates/hcl_templates/syntax for details.

make: *** [Makefile:26: podvm-0927a70-amd64] Error 1

problematic source code: https://github.com/confidential-containers/cloud-api-adaptor/blob/65ca44b149ea429b7d0ebbff9b288c35b8536633/azure/image/ubuntu/variables.pkr.hcl#L73-L83

Even if I go beyond the current restrictions by providing garbage values this fails as follows:
$ export PKR_VAR_plan_name="foo"
$ export PKR_VAR_plan_product="foo"
$ export PKR_VAR_plan_publisher="foo
$ PODVM_DISTRO=ubuntu make image

==> peer-pod-ubuntu.azure-arm.ubuntu:  -> DeploymentName    : 'pkrdppc967gim99'
==> peer-pod-ubuntu.azure-arm.ubuntu: Deploying deployment template ...
==> peer-pod-ubuntu.azure-arm.ubuntu:  -> ResourceGroupName : 'suraj-caa-cvm-4'
==> peer-pod-ubuntu.azure-arm.ubuntu:  -> DeploymentName    : 'pkrdppc967gim99'
==> peer-pod-ubuntu.azure-arm.ubuntu: ERROR: -> MarketplacePurchaseEligibilityFailed : Marketplace purchase eligibilty check returned errors. See inner errors for details.
==> peer-pod-ubuntu.azure-arm.ubuntu: ERROR:   -> BadRequest : The offer with Offer ID 'foo' was not found. Try one of these: 1- Make sure the offer name is written correctly. 2- If this offer was recently created, allow up to 30 minutes for it to be available for purchase. 3- The offer may have been removed from the marketplace. Look for similar offers from this publisher here: 'https://azuremarketplace.microsoft.com/en-us/marketplace/apps?page=1%26search=foo%20foo'. Publisher ID: 'foo', Offer ID: 'foo', Correlation ID '83a5d4e8-3664-466d-8f50-1c820851123a'.
==> peer-pod-ubuntu.azure-arm.ubuntu:
==> peer-pod-ubuntu.azure-arm.ubuntu: resources.DeploymentsClient#CreateOrUpdate: Failure sending request: StatusCode=400 -- Original Error: Code="MarketplacePurchaseEligibilityFailed" Message="Marketplace purchase eligibilty check returned errors. See inner errors for details. " Details=[{"code":"BadRequest","message":"The offer with Offer ID 'foo' was not found. Try one of these: 1- Make sure the offer name is written correctly. 2- If this offer was recently created, allow up to 30 minutes for it to be available for purchase. 3- The offer may have been removed from the marketplace. Look for similar offers from this publisher here: 'https://azuremarketplace.microsoft.com/en-us/marketplace/apps?page=1%26search=foo%20foo'. Publisher ID: 'foo', Offer ID: 'foo', Correlation ID '83a5d4e8-3664-466d-8f50-1c820851123a'."}]
==> peer-pod-ubuntu.azure-arm.ubuntu:
==> peer-pod-ubuntu.azure-arm.ubuntu: Deleting individual resources ...
==> peer-pod-ubuntu.azure-arm.ubuntu: Error deleting resource. Please delete manually.
==> peer-pod-ubuntu.azure-arm.ubuntu:
==> peer-pod-ubuntu.azure-arm.ubuntu: Name: suraj-caa-cvm-4
==> peer-pod-ubuntu.azure-arm.ubuntu: Error: resources.DeploymentOperationsClient#List: Failure responding to request: StatusCode=404 -- Original Error: autorest/azure: Service returned an error. Status=404 Code="DeploymentNotFound" Message="Deployment 'pkrdppc967gim99' could not be found."
==> peer-pod-ubuntu.azure-arm.ubuntu: resources.DeploymentOperationsClient#List: Failure responding to request: StatusCode=404 -- Original Error: autorest/azure: Service returned an error. Status=404 Code="DeploymentNotFound" Message="Deployment 'pkrdppc967gim99' could not be found."
==> peer-pod-ubuntu.azure-arm.ubuntu: Error deleting resource. Please delete manually.
==> peer-pod-ubuntu.azure-arm.ubuntu:
==> peer-pod-ubuntu.azure-arm.ubuntu: Name: suraj-caa-cvm-4
==> peer-pod-ubuntu.azure-arm.ubuntu: Error: resources.DeploymentOperationsClient#List: Failure responding to request: StatusCode=404 -- Original Error: autorest/azure: Service returned an error. Status=404 Code="DeploymentNotFound" Message="Deployment 'pkrdppc967gim99' could not be found."
==> peer-pod-ubuntu.azure-arm.ubuntu: resources.DeploymentOperationsClient#List: Failure responding to request: StatusCode=404 -- Original Error: autorest/azure: Service returned an error. Status=404 Code="DeploymentNotFound" Message="Deployment 'pkrdppc967gim99' could not be found."
==> peer-pod-ubuntu.azure-arm.ubuntu: Failed to find temporary OS disk on VM.  Please delete manually.
==> peer-pod-ubuntu.azure-arm.ubuntu:
==> peer-pod-ubuntu.azure-arm.ubuntu: VM Name: pkrvmpc967gim99
==> peer-pod-ubuntu.azure-arm.ubuntu: Error: resources.DeploymentOperationsClient#List: Failure responding to request: StatusCode=404 -- Original Error: autorest/azure: Service returned an error. Status=404 Code="DeploymentNotFound" Message="Deployment 'pkrdppc967gim99' could not be found."
==> peer-pod-ubuntu.azure-arm.ubuntu: Removing the created Deployment object: 'pkrdppc967gim99'
==> peer-pod-ubuntu.azure-arm.ubuntu:
==> peer-pod-ubuntu.azure-arm.ubuntu: The resource group was not created by Packer, not deleting ...
Build 'peer-pod-ubuntu.azure-arm.ubuntu' errored after 7 seconds 391 milliseconds: resources.DeploymentsClient#CreateOrUpdate: Failure sending request: StatusCode=400 -- Original Error: Code="MarketplacePurchaseEligibilityFailed" Message="Marketplace purchase eligibilty check returned errors. See inner errors for details. " Details=[{"code":"BadRequest","message":"The offer with Offer ID 'foo' was not found. Try one of these: 1- Make sure the offer name is written correctly. 2- If this offer was recently created, allow up to 30 minutes for it to be available for purchase. 3- The offer may have been removed from the marketplace. Look for similar offers from this publisher here: 'https://azuremarketplace.microsoft.com/en-us/marketplace/apps?page=1%26search=foo%20foo'. Publisher ID: 'foo', Offer ID: 'foo', Correlation ID '83a5d4e8-3664-466d-8f50-1c820851123a'."}]

==> Wait completed after 7 seconds 392 milliseconds

==> Some builds didn't complete successfully and had errors:
--> peer-pod-ubuntu.azure-arm.ubuntu: resources.DeploymentsClient#CreateOrUpdate: Failure sending request: StatusCode=400 -- Original Error: Code="MarketplacePurchaseEligibilityFailed" Message="Marketplace purchase eligibilty check returned errors. See inner errors for details. " Details=[{"code":"BadRequest","message":"The offer with Offer ID 'foo' was not found. Try one of these: 1- Make sure the offer name is written correctly. 2- If this offer was recently created, allow up to 30 minutes for it to be available for purchase. 3- The offer may have been removed from the marketplace. Look for similar offers from this publisher here: 'https://azuremarketplace.microsoft.com/en-us/marketplace/apps?page=1%26search=foo%20foo'. Publisher ID: 'foo', Offer ID: 'foo', Correlation ID '83a5d4e8-3664-466d-8f50-1c820851123a'."}]

==> Builds finished but no artifacts were created.
make: *** [Makefile:14: podvm-627a14c-amd64] Error 1

Specifically look at:
BadRequest : The offer with Offer ID 'foo' was not found. Try one of these: 1- Make sure the offer name is written correctly. 2- If this offer was recently created, allow up to 30 minutes for it to be available for purchase. 3- The offer may have been removed from the marketplace. Look for similar offers from this publisher here: 'https://azuremarketplace.microsoft.com/en-us/marketplace/apps?page=1%26search=foo%20foo'. Publisher ID: 'foo', Offer ID: 'foo', Correlation ID '83a5d4e8-3664-466d-8f50-1c820851123a'..

Did you try with empty "" values ?
Anyways I'll check it today

Just having it empty works. Didn't face the issue in my environment because it was already set to empty string and in docker based build also it's the same.

I'll raise a PR to set the default values to empty string


Add Object Id Support for Managed Identity.

Closes https://github.com/Azure/azure-sdk-for-java-pr/issues/1424

Bumps to latest msal.
Adds support to let user specify object Id when using Managed Identity credential.

API change check
APIView has identified API level changes in this PR and created following API reviews.
com.azure:azure-identity

/check-enforcer override


Create dataset loader for Lio and the Central Flores languages

Dataloader name: lio_and_central_flores/lio_and_central_flores.py
DataCatalogue: http://seacrowd.github.io/seacrowd-catalogue/card.html?lio_and_central_flores

Dataset
lio_and_central_flores

Description
This dataset is a collection of language resources of Li'o, Ende, Nage, and So'a which are collected in Ende, Flores, Eastern Nusa Tenggara. This dataset is the dataset from the research MA thesis by Alexander Elias. Title: Lio and the Central Flores languages

Subsets
Lio Collection

Languages
end, nxe, ssq, ljl, eng

Tasks
Automatic Speech Recognition, Machine Translation

License
Unknown (unknown)

Homepage
https://archive.mpi.nl/tla/islandora/search/alexander elias?type=dismax&islandora_solr_search_navigation=0&f[0]=cmd.Contributor%3A"Alexander\ Elias"

HF URL
-

Paper URL
https://studenttheses.universiteitleiden.nl/handle/1887/69452

#self-assign

Hi!
I have a question regarding this dataset. Would you mind that should we separate the task data loaders within this dataset for the sake of simplicity?: Speeh Recognition and Machine Translation. If not, could you please share a reference that has implemented two or more tasks in a  single data loader?
Thanks!

Hi! I have a question regarding this dataset. Would you mind that should we separate the task data loaders within this dataset for the sake of simplicity?: Speeh Recognition and Machine Translation. If not, could you please share a reference that has implemented two or more tasks in a single data loader? Thanks!

Hi @joanitolopo, thank you for taking on this dataloader. Could we have multiple subsets instead of multiple dataloaders?
seacrowd subsets

lio_and_central_flores_asr_{lang}_seacrowd_sptext for all of the SEA languages
lio_and_central_flores_mt_{lang}_seacrowd_sptext for all of the SEA languages

source subsets

lio_and_central_flores_asr_{lang}_source for all of the SEA languages
lio_and_central_flores_mt_{lang}_source for all of the SEA languages

HI @holylovenia.

Could we have multiple subsets instead of multiple dataloaders?
I assumed that we have 16 configs for each language because there are four languages and two task.

For seacrowd subsets, i used lio_and_central_flores_asr_{lang}_seacrowd_sptext for ASR task and lio_and_central_flores_asr_{lang}_seacrowd_t2t for MT task. Am i right? Thankyou

HI @holylovenia.

Could we have multiple subsets instead of multiple dataloaders?

I assumed that we have 16 configs for each language because there are four languages and two task.
For seacrowd subsets, i used lio_and_central_flores_asr_{lang}_seacrowd_sptext for ASR task and lio_and_central_flores_mt_{lang}_seacrowd_t2t for MT task. Am i right? Thankyou

Yes. For the MT, could you please use lio_and_central_flores_mt_eng_{lang}_seacrowd_t2t instead of lio_and_central_flores_mt_{lang}_seacrowd_t2t? Just for clarity's sake.

Adding top-priority and bonus+ labels because we would need this for the experiments.

Hi @joanitolopo, may I know if you need any help with the dataloader?

Hi @holylovenia, I had a discussion with Joanito earlier, and it seems like it is almost impossible to create a useful ASR dataset from this data because the video (audio) is impossible to align because there is no clear timestamp, the audio is noisy, and even sometimes repetitive.
Nonetheless, I think we can keep the machine translation task, as there are source-to-english sentence pairs provided in the transcription file.

Hi @holylovenia, I had a discussion with @joanitolopo earlier, and it seems like it is almost impossible to create a useful ASR dataset from this data because the video (audio) is impossible to align because there is no clear timestamp, the audio is noisy, and even sometimes repetitive.
Nonetheless, I think we can keep the machine translation task, as there are source-to-english sentence pairs provided in the transcription file.

Noted, thanks @joanitolopo @SamuelCahyawijaya! But I'll keep the datasheet as-is with ASR and MT tasks since the dataset provides the resources needed for these tasks—albeit with additional postprocessing steps.


ndk/looper: Add remove_fd() method

We've previously only been able to unregister file descriptors by using a callback and returning false from it to be unregistered.  Add the missing method that allows users to unregister their file descriptors from the looper for any reason.
There has always been a "concern" about leaking the Box inside add_fd_with_callback(), but this is already very easy by never returning false from the callback or registering a different callback / event on the same fd, so this new function doesn't really expose that.

Maybe for the issue with leaking the Box there could be some way of having add_fd_with_callback return some form of handle that could then be used to remove that callback, which could also make sure to drop the Box.

Perhaps we could, since a ForeignLooper can call remove_fd and is Send/Sync and probably also easily clonable.
Alternatively we could keep a HashMap of sorts mapping fd to Box. Then we can also catch overwrites when the same fd in add_fd(_with_callback)(). Not sure what to do when users close fds and get the same fd back on subsequent open() etc, it's up to them if they forgot to remove_fd it.

Anyway I tried to not mention (in the docs) the fact that it might be hard to trigger the callback just to return false currently (maybe via EPOLLHUP) to clean up the Box. But let's mention it here anyway.

Regardless, leaking memory isn't unsafe/UB (these APIs playing with random fds that we have no control over are, maybe they should've been marked unsafe to that effect), so I'll just merge this now I think and revisit this once we have a concrete user with this problem and maybe some more time / interest in thinking up the right solution.

I intentionally didn't bring that up. Many parts of the ndk are trivial to get raw pointers from (or otherwise) and process "things" outside of our Rust wrapper, and if we have to concern ourselves with that it becomes much harder to build a fully sound wrapper. This is anyway similar to the user e.g. closing the fd while it is still recorded inside the Looper, as mentioned before.


Add make a pr guide for contributors

This PR introduces a new readme file for contributors which they can use for reference for making Pull requests.

@21vidushi @Sampada067 @Yash080902 @Suryansh-1 PTAL and let me know your thoughts on it, Thanks.

Thanks @Yash080902, I am merging this PR for now. Also @21vidushi @Sampada067 @suryansh-1 If there's anything which you guys want us to edit or suggest feel free to do so by commenting on this thread.


[No QA] update PR checklist

Details

This updates the PR checklist for both Contributors and Reviewers based on the recent changes to this doc.
slack convo
cc @aldo-expensify
Fixed Issues

N/A
Tests

N/A
QA Steps

N/A
Screenshots

N/A

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by @MariaHCD in version: 1.1.46-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by @timszot in version: 1.1.46-3 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅


fix performance createListenerCollection

createListenerCollection has a bad performance for more listeners.
In my implementation, performance is much better
before:

after:

Can you provide some numbers on how this performs better?

Can you provide some numbers on how this performs better?
I added screenshots to make it clear.

OK, I cleaned things up a bit.
We used the previous current/next form copied from Redux core because we wanted to ensure adding a subscriber while dispatching wouldn't trigger an edge case. However, since this code is nested inside of Subscription anyways, we actually don't need these guarantees.
Thanks for the PR!

I'm sorry, I made a mistake. This PR must be revert. Because it led to the wrong order of calling parent/child selectors.😟


Typo

https://github.com/PlayVoice/so-vits-svc-5.0/blob/a18ebd448da9ef3be19109f39fcd55870a56042d/README.md?plain=1#L105
For this line it should be svc

Also, I might add, you need to add resampy to the requirements file too.


Rename iOS Flutter.xcframework directory ios-arm64_armv7 -> ios-arm64

Must be merged in the same engine roll that removes armv7 from the engine artifacts.  Do not merge.
Tool half of https://github.com/flutter/flutter/issues/97342
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Superceded by https://github.com/flutter/flutter/pull/97341 and https://github.com/flutter/flutter/pull/101592


[Snyk] Upgrade jakarta.xml.bind:jakarta.xml.bind-api from 2.3.3 to 4.0.2

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade jakarta.xml.bind:jakarta.xml.bind-api from 2.3.3 to 4.0.2.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

Warning: This is a major version upgrade, and may be a breaking change.

The recommended version is 11 versions ahead of your current version.
The recommended version was released 23 days ago, on 2024-02-19.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

closing in favor of #613


The "https://bower.herokuapp.com/packages/search/XY" file could not be downloaded (HTTP/1.1 404 Not Found)

Hi,
we are using this plugin shortly and from today I'm not able to update or install any package becouse of 404 error:
[Composer\Downloader\TransportException]
The "https://bower.herokuapp.com/packages/search/responsivefilemanager" file could not be downloaded (HTTP/1.1 404 Not Found)
Is there some way how to prevent search on heroku? It's totally useless as we have specified our url for repository...
Our simplified composer.json:
	"require": {
		...
		"bower-asset/ResponsiveFilemanager": "dev-master"
	},
	"extra": {
     		"asset-installer-paths": {
     			"bower-asset-library": "public/components"
     		},
		"asset-repositories": [ {
			"type": "bower-git",
			"url": "git://github.com/jkocour/ResponsiveFilemanager",
			"name": "bower-asset/ResponsiveFilemanager"
		},

The same error. Looks like bower search engine case sensitive again.

Today it works again.
But it do not change anythink that I do not understand why it is searched on bower sites if I have specified repository url and why is not 404 status handled correctly (it should continue with resolving). But I'm not sure, is it problem of this plugin or composer itself?

@norbe Because it's the standard behavior of the VCS Drivers.


Better outage notification emails

We simply customize like this when outage happens:
var error = JSON.parse(JSON.stringify(outage.error));
so, here we need to put some useful information when ping is failed, like this:
var error = {"code":"xxx", "message":err.message};
callback(error, "failed", "failed", time);

But this feature seems should be done in the plugin and ping-service projects?


Crash build if Android tests exist out of the app module

Task/Issue URL: https://app.asana.com/0/488551667048375/1202931331947206/f
Description
Steps to test this PR
Feature 1

[ ]
[ ]

UI changes

Before
After

!(Upload before screenshot)
(Upload after screenshot)

Current dependencies on/for this PR:

develop

PR #2260   👈

This comment was auto-generated by Graphite.


Images of Tilemap Pattern do not match sometimes to their corresponding Pattern

Godot version
4.0-beta15
System information
Windows 10, Vulkan (Mobile Mode), RTX 3090 nvidia v.528.02, Ryzen 9 5900X
Issue description
I could not find any related issue so I opened an issue:
Images of tileset pattern do not always match to the corresponding tileset pattern in the bottum menu bars: "Tileset" and "Tilemap"
here you can see the bug in action:
https://user-images.githubusercontent.com/86411785/214831628-4d4be827-cb59-4d48-a44b-a9d0e2e9411b.mp4
Steps to reproduce

Create Tilemap and Tileset
Create some different Pattern
Toggle "Tilemap" or "Tileset" in the bottom menu bar

Minimal reproduction project
Godot4_Beta15_Bug_TilemapPattern_NotCorrectImageOrder.zip

Looks related to #70825 but not for terrains


why output >0.5 belongs to road?

Hi,I am learning your code and have some questions. when you do  segmentation, you think the pixel vaue which over 0.5(output_im > 0.5), why?

I think it just a threshold, you can change it with your own result

OK,Thanks!