Don't apply settings on version 0.11.1121.0

Environment
Windows 10 Pro Insider Preview 19613.1000 (Japanese)
Windows Terminal 0.11.1121.0
Windows build number: [run `[Environment]::OSVersion` for powershell, or `ver` for cmd]
Windows Terminal version (if applicable):

Any other software?

Steps to reproduce
Launch the Windows Terminal

Expected behavior
Launch the Windows Terminal applying the settings.

Actual behavior
Launch the Windows Terminal with default settings.

/dup #5458


Folder name not being overridden with mockPrompt

I have a prompt such as:
  prompts: {
    name: {
      message: 'What is the name of the new package',
      default: ':folderName:',

And a test such as:
test('defaults', async t => {
  const stream = await sao.mockPrompt(template, {
    name: 'Foo Bar Baz'
  });
  t.snapshot(stream.fileList, 'generated files');
  const content = stream.fileContents('README.md');
  t.snapshot(content, 'content of README.md');
});

However the value of name in the test is always output (the default :folderName value) instead of Foo Bar Baz.
I think this is a bug with mockPrompt not taking in default values specified in the options.

non issue


Preview freeze while capturing image at 60fps

JCA version
1.0.0-rc01
Device
Pixel 7 pro
Steps to reproduce

Set 60 FPS from settings (might need app restart to take effect)
Capture an image

Expected
No preview freeze/lag for capture
Observed
Preview froze (stuck at same frame) during the capture
Remarks
Not sure, but probably related to a known CameraX issue of not copying repeating request FPS to non-repeating request. This can cause a session recreation due to default still capture template value being 30 FPS which can potentially have these kinds of effects.
0:com.google.jetpackcamera  f208:222357385634699ns:             REQ: output stream ids: 1
0:com.google.jetpackcamera  f208:222357385634699ns:             REQ:android.control.captureIntent: [VIDEO_SNAPSHOT]
0:com.google.jetpackcamera  f208:222357385634699ns:             REQ:android.control.aeTargetFpsRange: [30 30 ]
0:com.google.jetpackcamera  f208:222357488990754ns:                            RES:android.control.captureIntent: [VIDEO_SNAPSHOT]
0:com.google.jetpackcamera  f208:222357488990754ns:                            RES:android.control.awbState: [SEARCHING]
0:com.google.jetpackcamera  f208:222357488990754ns:                            RES:android.control.afRegions: [1714 1353 2366 1719 0 ]
0:com.google.jetpackcamera  f208:222357488990754ns:                            RES:android.control.aeTargetFpsRange: [30 30 ]
0:com.google.jetpackcamera  f209:222357734020686ns:             REQ: output stream ids: 0
0:com.google.jetpackcamera  f209:222357734020686ns:             REQ:android.control.captureIntent: [VIDEO_RECORD]
0:com.google.jetpackcamera  f209:222357734020686ns:             REQ:android.control.aeTargetFpsRange: [60 60 ]
0:com.google.jetpackcamera  f209:222357848951487ns:                            RES:android.control.captureIntent: [VIDEO_RECORD]

Recording
https://github.com/google/jetpack-camera-app/assets/34229687/77ead22b-a2d2-45e2-9f33-26455d8467ec

This has been fixed in CameraX side in https://android-review.googlesource.com/c/platform/frameworks/support/+/3061644
The issue should be fixed in JCA when the snapshot version is updated (tested manually with CameraX change ported to JCA).

Verified this is fixed with the latest build of JCA. Thanks!


Add compiler reference

Add high level documentation for the architecture of the compiler. This documentation should provide a high-level overview of the compiler, the phases of compilation, and the data structures used throughout the compiler's C source.
It may be worth waiting until the compiler is stabilized before beginning this document as the language and compiler are still undergoing rapid changes.

Although the compiler is mostly stabilized at this point, I am closing this issue as a WONTFIX since discussion on the compiler implementation would probably be better served by individual blog posts outside of this repo.


Double elimination playoff losers bracket spoilers are shown when BYEs are present

https://discordapp.com/channels/200267155479068672/230800342579806208/575759772389867521
the hiding spoilers thing isnt quite working right in a double elim bracket:
https://heroeslounge.gg/tournament/nut-cup
You can see the teams seeded into the lower bracket after loosing a game if they're first matched against a bye :NotLikeSloth:

Is this still a bug? I checked the Nut Cup and it appears to have been fixed? @rogusdev

Hmm I must not have tagged it on the PR for it: https://github.com/Fabian-Sommer/HeroesLounge/pull/112


Add 0.3 Tagged Hashes

adds 0.3 notebook for tagged hashes
removes introductory tagged hash content from 2.3
adds reference to 0.3 in README

Addresses https://github.com/bitcoinops/taproot-workshop/issues/91

Thanks for doing this, @bitschmidty. I've made a couple of minor changes. Let me know what you think.


Add information on browser support to documentation

Ideally this could be achieved using Can I use?. As of right now, the documentation includes a data field to specify used CSS features. If this was filled thoroughly for all objects this could be used to auto generate browser compatibility charts per object.

Can I use provides it's data for free on github > https://github.com/Fyrd/caniuse/blob/master/data.json

Can I use provides it's data for free on github > https://github.com/Fyrd/caniuse/blob/master/data.json


Who's Cooking

Assign who's cooking to days of the week when planning meals out.

Branch: https://github.com/amplitudesolutions/mealtime/tree/WhosCooking

Done feature!

Has been merged into Master.


FIX platform version windows

Codecov Report

Merging #392 (c36ba30) into main (1ecd1c9) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #392   +/-   ##
=======================================
  Coverage   56.09%   56.09%
=======================================
  Files          42       42
  Lines        2519     2519
  Branches      476      476
=======================================
  Hits         1413     1413
  Misses       1006     1006
  Partials      100      100

Regarding the problem with the length of the terminal messages as well as the colors,
It's a platform-related issue, namely Windows, as the ANSI API is not enabled by default in it hence must be done manually. I did that by following this answer https://superuser.com/questions/413073/windows-console-with-ansi-colors-handling#:~:text=Fortunately%2C the global,explicitly disable ANSI
P.S: This comment is for future related issues.

Could you try to call this at module init in terminal_output.py?
import ctypes

kernel32 = ctypes.windll.kernel32
kernel32.SetConsoleMode(kernel32.GetStdHandle(-11), 7)

@tomMoral,
Just tried that on my local machine, after I disabled ANSI, and it solve the problem.
If it doesn't break the code for other platforms (e.g. Linux), I shall make a pull request.

(this would break on Linux as ctypes.windll.kernel32 is only defined on windwos.


Add apm_config.connection_limit in config_template.yaml

What does this PR do?
Add apm_config.connection_limit in config_template.yaml
Motivation
apm_config.connection_limit is not in config_template.yaml, even the public doc explain it.
https://docs.datadoghq.com/tracing/troubleshooting/agent_rate_limits/#max-connection-limit
Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided if the qa/skip-qa label is not applied.
[ ] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the config template has been updated.

@jtappa @djmitche Could you merge this PR instead of me since I don't have permission to do it?


Build gradle

Hello Please see the screenshot and let me know how to fix and sort out these issues and how to install if I do follow the wrong path.

CMD errors
C:\Users\Joban>flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, v1.9.1+hotfix.5, on Microsoft Windows [Version 10.0.18362.418], locale en-US)

[!] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
    X Android licenses not accepted.  To resolve this, run: flutter doctor --android-licenses
[√] Android Studio (version 3.5)
[!] Connected device
    ! No devices available

! Doctor found issues in 2 categories.

Hey @KanwarpartapSinghBimrah did you do what the log suggests?
Android licenses not accepted.  To resolve this, run: flutter doctor --android-license

@miquelbeltran I know but wanna future solution before it.

Hi @KanwarpartapSinghBimrah
are you still experiencing the issue after accepting Android's licence?
(thank you @miquelbeltran)
If so can you please describe the issue in detail
and provide your updated flutter doctor -v
thank you

Everything is fine but when copy-paste some code from github it provide some errors and Also I got same problem in build.gradle file. See above

Hi again @KanwarpartapSinghBimrah
does the problem described above persist
after trying the proposed solution?

@iapicca hi
yes again I got an error which already persists in the screenshot above

Hi again @KanwarpartapSinghBimrah
can you please provide your  update flutter doctor -v
and your and your flutter run --verbose/flutter build --verbose ?
Also, to better address the issue, would be helpful
if you could post a self contained app on github
or the steps to reproduce it.
Please refrain
from posting pictures if not necessary.
Thank you

Please @iapicca You can see the screenshot. It was clicked by me in sequences. 4 screenshot serial wise

Hi again @KanwarpartapSinghBimrah
please refrain from posting pictures
could you please post a self contained app on github
or the steps to reproduce it.
Thank you

Please let me know now what I can do

If the issue is the installation itself
I'd recommend a fresh install following our documentaion.
If with a fresh install the problem still persist would be helpful
if you could post a self contained app on github.
Thank you

okay thank I will do that again. If I got an error I will let you know. But I think now should not be.

Hi @KanwarpartapSinghBimrah
given your last message I feel safe to close this issue,
if you disagree please write in the comments
and I will reopen it.
Thank you


⚠ Bofrost.at has degraded performance

In 3a29ed4, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1342 ms

Resolved: Bofrost.at performance has improved in ad9d8bd after 13 minutes.


Broken link to CONTRIBUTORS.md File

CONTRIBUTORS.md File missing but linked in invitation template.
It seems the CONTRIBUTORS.md file is missing.
The link directed to https://github.com/chryz-hub/chryz-hub.github.io/blob/master/CONTRIBUTORS.md

Thanks, @SpiffGreen.
This is the active https://github.com/chryz-hub/chryz-hub.github.io/blob/master/CONTRIBUTING.md
But you can go ahead to make a pull request on this.

Thanks 👍🏽


Testnet readme

Completed the testnet readme to show the right set of commands, making onboarding new nodes easier.

Small extra addition needed


VSCode forbids suspending to RAM in Linux

Expected result

Linux machine suspends correctly when VSCode is opened

Actual result

Code prevents computer from suspending

Debug info

$ uname -a
Linux machine 5.10.0-8-amd64 #1 SMP Debian 5.10.46-3 (2021-07-28) x86_64 GNU/Linux

$ journalctl -xef`

Ago 10 23:50:13 machine kernel: Freezing of tasks failed after 20.008 seconds (1 tasks refusing to freeze, wq_busy=0):
Ago 10 23:50:13 machine kernel: task:code            state:D stack:    0 pid: 7606 ppid:  7396 flags:0x00004004
Ago 10 23:50:13 machine kernel: Call Trace:
Ago 10 23:50:13 machine kernel:  __schedule+0x282/0x870
Ago 10 23:50:13 machine kernel:  schedule+0x46/0xb0
Ago 10 23:50:13 machine kernel:  request_wait_answer+0xa1/0x220 [fuse]
Ago 10 23:50:13 machine kernel:  ? add_wait_queue_exclusive+0x70/0x70
Ago 10 23:50:13 machine kernel:  fuse_simple_request+0x1a1/0x310 [fuse]
Ago 10 23:50:13 machine kernel:  fuse_dentry_revalidate+0x17a/0x380 [fuse]
Ago 10 23:50:13 machine kernel:  ? d_splice_alias+0x8b/0x290
Ago 10 23:50:13 machine kernel:  lookup_fast+0xd7/0x150
Ago 10 23:50:13 machine kernel:  walk_component+0x41/0x1d0
Ago 10 23:50:13 machine kernel:  link_path_walk.part.0+0x223/0x370
Ago 10 23:50:13 machine kernel:  ? path_init+0x2bd/0x3e0
Ago 10 23:50:13 machine kernel:  path_lookupat+0x43/0x1c0
Ago 10 23:50:13 machine kernel:  filename_lookup+0xaa/0x1b0
Ago 10 23:50:13 machine kernel:  ? getname_flags.part.0+0x45/0x1a0
Ago 10 23:50:13 machine kernel:  do_faccessat+0x7c/0x260
Ago 10 23:50:13 machine kernel:  do_syscall_64+0x33/0x80
Ago 10 23:50:13 machine kernel:  entry_SYSCALL_64_after_hwframe+0x44/0xa9
Ago 10 23:50:13 machine kernel: RIP: 0033:0x7ff1abaa6ff7
Ago 10 23:50:13 machine kernel: RSP: 002b:00007ffc7a8b01a8 EFLAGS: 00000206 ORIG_RAX:<PHONE_NUMBER>000015
Ago 10 23:50:13 machine kernel: RAX: ffffffffffffffda RBX: 00007ffc7a8b2510 RCX: 00007ff1abaa6ff7
Ago 10 23:50:13 machine kernel: RDX: 00007ffc7a8b2660 RSI:<PHONE_NUMBER>000000 RDI: 00007ffc7a8b2660
Ago 10 23:50:13 machine kernel: RBP: 00007ffc7a8b23d0 R08:<PHONE_NUMBER>000000 R09: 00007ffc7a8b2460
Ago 10 23:50:13 machine kernel: R10:<PHONE_NUMBER>000002 R11:<PHONE_NUMBER>000206 R12: 00007ffc7a8b2450
Ago 10 23:50:13 machine kernel: R13: 00007ff1aaac7a00 R14: 000055a73a8d2810 R15:<PHONE_NUMBER>00000c
Ago 10 23:50:13 machine kernel:

After closing code I was still unable to suspend. Then I checked the running processes
$ ps aux | grep code
user      7606  0.1  0.5 8676440 169300 ?      Sl   Ago08   6:01 /usr/share/code/code /home/user/.vscode/extensions/ms-python.vscode-pylance-2021.8.0/dist/server.bundle.js --cancellationReceive=file:0dc4623216c22d77816f870a75b8a01b38c34a68dc --node-ipc --clientProcessId=7396

So it looks like the pylance extension, which is still running after closing the window is the culprit. I'll update the title.

/extPython


RSS feed issue

RSS feed used to work properly. But lately, it stops working properly and return the same results for popular and newest feed. The results for all category, all time, and popular feed is not correct.
Newest
http://backend.deviantart.com/rss.xml?type=deviation&q=sort%3Atime&offset=0
Popular
http://backend.deviantart.com/rss.xml?type=deviation&q=boost%3Apopular&offset=0

Please check again, it should be fixed now

Most of all categories, popular feeds are still seem to be broken.
The following feeds seem to return the same results:
All time: http://backend.deviantart.com/rss.xml?type=deviation&q=boost%3Apopular&offset=0
24 hours: http://backend.deviantart.com/rss.xml?type=deviation&q=boost%3Apopular+max_age%3A24h&offset=0
3 days: http://backend.deviantart.com/rss.xml?type=deviation&q=boost%3Apopular+max_age%3A72h&offset=0
1 week: http://backend.deviantart.com/rss.xml?type=deviation&q=boost%3Apopular+max_age%3A168h&offset=0
1 month: http://backend.deviantart.com/rss.xml?type=deviation&q=boost%3Apopular+max_age%3A720h&offset=0
The only exception is 8 hours: http://backend.deviantart.com/rss.xml?type=deviation&q=boost%3Apopular+max_age%3A8h&offset=0


.hide-for-large class has overflow issue on small screen.

How can we reproduce this bug?
.hide-for-large class has overlfow issue on small media for email templates.
Add class to an item and resize the media to small. Overflow issue is obvious. (It happens in the demo as well http://foundation.zurb.com/emails/docs/visibility.html)
Item with class should display
Item with class displays with issues.
I was able to correct this locally by resetting the max-height back to inherit. There maybe a better way to solve this but following the css order, max-height is set to 0 but never adjusted on small detection.
@media only screen and (max-width: [width]px) {
table.body table.container .hide-for-large {
display: block !important;
width: auto !important;
overflow: visible !important;
max-height: inherit; } }
Give us a link to a CodePen or JSFiddle that recreates the issue.

CodePen with Foundation 6 and MotionUI
CodePen with Foundation 6, Flexbox grid and MotionUI

@jwfranks I believe this belongs on the Foundation for Emails repo. You should open a new issue over at https://github.com/zurb/foundation-emails/issues instead of here. Thanks!


IOF Email filtering

Is it possible to introduce an iof.EmailFilter() function ?
It doesn't introduce a lot of code to add, possible regex : /^\w+([.-]?\w+)@\w+([.-]?\w+)(.\w{2,3})+$/ and reduce a lot the code to put into APIDescription :eyes:

Fixed in upstream https://github.com/7PH/io-filter/issues/6

Published: You can use iof.EmailFilter() since<EMAIL_ADDRESS>


Use template file to generate inputs.json files in tests.

To manage the files without duplication.

https://github.com/GargGroup/BioDivGenomics/actions/runs/3221016165/jobs/5268696145#step:2:175

/usr/bin/git lfs fetch origin refs/remotes/pull/30/merge
fetch: Fetching reference refs/remotes/pull/30/merge
batch response: This repository is over its data quota. Account responsible for LFS bandwidth should purchase more data packs to restore access.

https://github.com/nabla-c0d3/nassl/issues/17

OK. I stopped using Git LFS.


Remove target/ and schema/ directories after polar compile

target/ and schema/ dir locations depend on Cargo.toml.
locate dirs and remove them.
only remove once checking artifacts dir has .wasm and schema .json files.

This was done intentionally because doing this will not be good in terms of time taken to recompile contracts

Got it, and I'll add there 2 dirs to .gitignore.


How to extract time from an uuid v1?

I found this: https://github.com/krassif/node-uuid/commit/a9aac0e56f6c78b80454e9686e2eebcbbf724d45
From here: https://stackoverflow.com/questions/17571100/how-to-extract-timestamp-from-uuid-v1-timeuuid-using-javascript
But since node-uuid (the npm module) is deprecated: https://www.npmjs.com/package/node-uuid what can I use?
It looks to me that this feature should be included in this library.

https://www.npmjs.com/package/uuid-time works.  Just be aware that the value it returns (Unix epoch, +/1 msec) is imprecise. (v1 uuids are precise to +/- 0.01 μsec)
See also https://github.com/kelektiv/node-uuid/pull/49

Thanks!

Can this be included in this library? (would offer TS support also)

Given there has not been a single request for this feature in the past 3 years and given there’s a dedicated npm module for this use case, I think it’s very unlikely that we‘ll support this feature in this library.

~10-15 people have shown interest in this feature in the last decade (including comments on #49).  The uuid-time module that spun out of that gets ~900 downloads/week, compared to uuid's current traffic of 67M/week.
In addition to the lack of demand, there's also the matter of deciding what the "right" API should be.  I'm not convinced there's good solution here, due to poor choice of epoch used in RFC4122 (gregorian epoch, 100-nsec units).  uuid-time returns a JS timestamp value (unix epoch, msec units), but drops sub-msec resolution in the process.  uuid uses msec and nsec values internally and (regrettably?) exposes those in the API.  But those are weird and ad-hoc.   Really, the best datatype for is probably BigInt, but BigInts are cumbersome to work with.  (E.g. new Date(1234n) throws) and, I suspect, probably aren't what most people need.
Oh, and there's a new proposal in the works - https://github.com/uuid6/uuid6-ietf-draft - that adds seems bent on adding another epoch to the mix. (Unix, 1-nanosecond units).
I guess the TL;DR: is that I [still] don't see a solution popular and "right" enough to warrant taking this.


[Fiat currency] Not updated in Fee rate box and getting reset to CAD on app re-start

App version: v0.1.97/Mainnet (001325b147e0b5a29debbbc1207ae99e5b08ed03)

[ ] Network fee popup is CAD always.
[ ] On re-start, for few seconds it's shows my selected fiat in home page (/wallets) and then it resets to CAD.

@saiy2k @i5hi fixed last
works on my side

Yes. Second point is fixed now.

First also fixed.


Disable Revolutionary Gamemode

About the PR
Disable Revolutionary Gamemode until it can become something better.
Why / Balance
Revolutionary mode is by far the worst gamemode to not only play, but also moderate. There's too many rule issues around this mode- such as metagaming and what is ok and what's not ok to call for a Revolution amongst the station. By the time a revolution could be called out legally it's likely too late. In order for a revolution to safely come to a halt within a good time limit it sadly has to be metagamed. As for it being awful to play; people can easily snitch out the head revs or other revs after being de-converted. Along with that, people are always complaining in LOOC or OOC about the rules around this mode claiming "you metagamed". Another issue I've seen is the flashes for Revs. They struggle to get more unless they're Sci.
Requirements

[x] I have read and I am following the Pull Request Guidelines. I understand that not doing so may get my pr closed at maintainer’s discretion
[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

:cl:

Removed Revolutionaries gamemode pending rework.

Previous discussion:
https://github.com/space-wizards/space-station-14/pull/25704
https://github.com/space-wizards/space-station-14/pull/24420

Also not commenting on if it should be removed or not, but instead of removing the game rule remove it from the secret gamemode in secret_weights.yml. That way the tests won't fail and admins will still be able to run it if they want to.

Ah you're completely right, thanks


Add xc7a100T Linux/Litex example and update README

I also changed any mention of "arty_50" to "arty_35" since the "50" reference might confuse new users.

@tcal-x -- Could you rebase this and then we can merge? Thanks!

@tcal-x -- Could you rebase this and then we can merge? Thanks!

@kgugala suggested that I wait until the IDELAYCTRL fix lands, at which point my changes can be greatly simplified.   That makes sense to me.

This PR is superseded by PR #50 and should be abandoned.


Test HTTP against individual GG nodes in api-status

Changes

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

Current dependencies on/for this PR:

main

PR #453

PR #454

PR #455   👈

This stack of pull requests is managed by Graphite.


K8s helm extending configs v2

Squashed k8s helm extending configs

Looks good to me, and actually, just made my day easier too ;-) Thanks @dsever :)


feat(EntityChanges): created handler and manager and required helper …

…methods for entity change tracking

[x] Base implementation
[x] Helpers in universe
[x] Tests
[ ] Add config that would enable this as a default for all entities loaded by the Universe

https://app.clickup.com/t/24327122/TECH-5123

tests are failing due to the copy

@eljefedelrodeodeljefe  I have changed the design for the eager reaction let me know do u think
Relevant files: https://github.com/c-commerce/charles-browser-sdk/pull/262/files#diff-af2f32ddd9b2dbfb65cf235b2e205974c63c83d730f46f0a63cd94ab9d365b8c
customHandler Param


⚠ Processors has degraded performance

In cf0d13a, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 124 ms

Resolved: Processors performance has improved in 5df3471.


help: Clean up beta app installation instructions.

Converts the beta app installation instructions into our standard tabbed instructions format.
Fixes #28885.
Screenshots and screen captures:

https://chat.zulip.org/help/desktop-app-install-guide#install-a-beta-release

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability

Communicate decisions, questions, and potential concerns.

[x] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] End-to-end functionality of buttons, interactions and flows.
[x] Corner cases, error conditions, and easily imagined bugs.

@alya This PR is ready for review.
@zulipbot add "help center review"

@alya Thanks for the feedback! I updated the PR + screenshots.

Works for me!

lgtm aside from #29008 (comment).

I updated the PR  to address this feedback.
@zulipbot add "integration review"
@zulipbot remove "help center review"

@timabbott I changed the formatting of all the command instructions in the documentation to fenced code blocks.

Merged, thanks @drrosa!


Create more detailed error responses when submitting batch jobs

NOTE: this PR is a continuation of the work/PR that our summer 2023 MLH fellow/intern Raaj Patel had created. Unfortunately, his internship ended and he has returned to university before that PR (https://github.com/armadaproject/armada/pull/2623) was fully finished, approved, and merged. The only change to this code since he departed I have made is to fix a conflict in go.mod and go.sum.
Following is his description in the original PR:
What type of PR is this?
This PR adds functionality to JobSubmitResponse when an error has occured. Previously the only thing returned was the error. The changes made in this PR allow the code to do 2 things:

Build a collection of errors, if mulitple jobs error for the same return, these changes will return them at the same time to the user. Rather than them having to reattempt submissions over and over again with 1 fix each time.
Return more detailed errors to the user by utilizing the JobSubmitResponse Struct

Which issue(s) this PR fixes:
Fixed https://github.com/armadaproject/armada/issues/1748
Original Description from issue 1748 in JIRA, opened by @d80tb7 :

When we reject a job (for example because it can't be scheduled on any node) the whole grpc call fails.
There is an error message that is returned, but users have said that this doesn't give enough infomation
about what was valid. Instead they ask:

is it possible for these errors to be returned on the Error field on the JobResponseItem that we're given?

Screenshots from Raaj Patel, showing the logging output before and after this change.

Submitted a bad sleep yaml job containing two jobs, both have invalid podSpecs as the requested and limit resources both contain "memory: 0"


How to clean up all watchers

Since this doesn't appear to maintain any global state (which is good), there's no obvious way, if any, to clear all watchers in a single instruction AFAIK.
My scenario:
While the user is logged in, I don't want to destroy "item component" watchers (say a list), when the "item component" is $destroyed, i.e. user navigates away. However I do want to clear them on logout.
Any clues?

Add a decorator on $firebaseArray/Object that stores any instances created in an array. Add a decorator on $firebaseAuth.unauth() that iterates the array and calls $destroy on the instances.


proposal: gitupload as a kubernetes custom resource

ProblemStatement: As an automation engineer, I want to upload specific kubernetes resources as well as pod logs from a running kubernetes cluster to some github repo. This helps me to refer to these resources long after the kubernetes cluster is gone.

Draft Specs:
kind: GitUpload
spec:
  resources:
  - kind:
    apiVersion:
    selector:
      matchLabels:
      matchLabelExpressions:
  logs:
    podSelector:
      matchLabels:
      matchLabelExpressions:
      matchContainers:
      matchContainerExpressions:


docs: adds more details to use, description. and examples to oc-mirro…

…r commands and subcommands
Signed-off-by: Jennifer Power<EMAIL_ADDRESS>Description
This PR adds a more detailed description and more examples for the oc-mirror root command. It makes better use of the Use field when defining the commands to document required arguments.
Closes #426
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[X] This change requires a documentation update

How Has This Been Tested?
Documentation update
Checklist:

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have made corresponding changes to the documentation
[X] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

@bergerhoffer Thanks for the excellent feedback! Changes have been incorporated.

/lgtm


change z-index of shadow in marker

Hello,
it seems like the z-index for the shadow of a marker is not changed when defining a zIndexOffset. This behaviour does make sense. But I'd like to override it for one special marker. Is this easily possible? I couldn't find anything about that in the docs.

You can do marker._shadow.style.zIndex = ... for now.


Style guide

Adds a new route /style-guide to the app so we can view all of our visual components together.
Makes some minor changes to styles and removes a debugger; statement from the session_monitor.js file.
Example guide:

@bengolder is this what you had in mind? This is pretty basic info. Let me know if you think there should be anything else added.


Reading synced file problem

Hi everyone,
I have a problem using this plugin when the page load content on IOS (Android is ok).
Without apparent reason, some rows in the js code disappear when the content load in page.
This is an example:

Content returned from get in the browser is this:

If I uninstall the cordova-hot-code-push plugin everything is fine.
Checking the file in the www on  cordova-hot-code-push.plugin in the simulator the content seems ok.
Thank you for any support,
Daniele

Hi,
Plugin does not modify files in any way. It only moves them from one directory to another. And that's it... So that's the very weird thing you see in here.
Have you tried to put all correct files in www folder inside the project, and on the server side - don't deploy anything at all so the update would fail? But the plugin should just copy www folder from bundle to the external storage. And your js file should be the same.
If that works - try to deploy on the server the same www folder with the same chcp.json config. Nothing should change.
Then update something minor in www folder on the server side (i.e. add spaces in some html file), rebuild configs and let the plugin update your app. After that - check, what happened.
But again - plugin does not touch the content of the files in any way.


JSWorm ransomware notes and extensions added.

Added ransomware extensions and ransomware notes for various variants of JSWorm. Below are the references and descriptions for the same.
Reference - https://securelist.com/evolution-of-jsworm-ransomware/102428/?web_view=true
JSWorm 4.0.3
Note File: JSWORM-DECRYPT.txt
Extension<EMAIL_ADDRESS>Came out - July 2019
Specific Reference - https://www.pcrisk.com/removal-guides/14388-jsworm-ransomware
Nemty 1.4
Note File: NEMTY_[random_string]-DECRYPT.txt
Extension: .NEMTY_<…>_,
Came out - August 2019
Specific Reference - https://www.pcrisk.com/removal-guides/17259-nemty-2-6-ransomware
Nefilim
Note File: NEFILIM-DECRYPT.txt
Extension: .NEFILIM
Came out - March 2020
Specific Reference - https://www.pcrisk.com/removal-guides/17463-nephilim-ransomware
Offwhite
Note File: scam.jpg, OFFWHITE-MANUAL.txt
Extension: .OFFWHITE
Came out - April 2020
Specific Reference - https://www.pcrisk.com/removal-guides/17728-offwhite-ransomware
Telegram
Note File: TELEGRAM-RECOVER.txt
Extension: .TELEGRAM
Came out - June 2020
Specific Reference - https://www.pcrisk.com/removal-guides/18132-telegram-ransomware
Fusion
Note File: FUSION-README.txt
Extension: .FUSION
Came out - November 2020
Specific Reference - https://www.pcrisk.com/removal-guides/19332-fusion-ransomware
Milihpen
Note File: MILIHPEN-INSTRUCT.txt
Extension: .MILIHPEN
Came out - January 2021
Specific Reference - https://www.pcrisk.com/removal-guides/20055-milihpen-ransomware
Gangbang
Note File: GANGBANG-NOTE.txt
Extension: .GANGBANG
Came out - February 2021
Specific Reference - https://www.pcrisk.com/removal-guides/20497-gangbang-ransomware

Thank you @VatsalJagani for updating the lookups and your contribution


Polars 20.6 breaks strings module - but requirement map to latest polars version

When calling the notebook
df2.select(
    pl.col("sen").str.to_lowercase().str2.tokenize(stem=True).explode().unique()
)

thread '<unnamed>' panicked at crates/polars-lazy/src/physical_plan/exotic.rs:39:65:
called `Result::unwrap()` on an `Err` value: ComputeError(ErrString("cannot call plugin\n\nThis Polars' version has a different 'binary/string' layout. Please compile with latest 'pyo3-polars'\n\nError originated just after this operation:\nDF [\"\"]; PROJECT */1 COLUMNS; SELECTION: \"None\""))
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

i fixed forcing in the pyproject.toml
dependencies = [
    "polars == 0.20.5",
]

Also
num.py was not registering some methods throwing error in the notebook that have forward type hints using np. without importing numpy i solved by adding import numpy as np and recompiled I can run the full notebook (also from the graph branch) - will test the graphs methods soon

Should I wait to test out the graphs method on the next refactor? Or is it useful if I play around the current more_graph branch today?

Better wait I think. A lot of exciting stuff coming up. E.g shortest path with unequal distance cost per edge, parallel execution of graph queries, etc. Meanwhile I really appreciate you reading the source code like you ate doing. So keep the questions coming. Thank you

Fyi, current main is the release candidate. I want to wait for a few days before release in case I want to make any additional changes or I find any other bugs. But all functionality should be more or less final.


Network Error : ETIMEDOUT

After running the meteor-down for around 90 seconds, I get this error:
Network error: wss://XXX:443/websocket: connect ETIMEDOUT

Has anyone ran into anything similar ?

it seems like a network error. Also possible if the load testing server is having CPU issues.
If it got enough load.

Did you find out a workaround ?


Contributing guide for nbsp in "Perl 6" / "Perl 5"?

Went in to drop a handy one-liner that converts "Perl 6" / "Perl 5" with spaces into non-breaking space version, but didn't find where this stuff is even mentioned in our contributing guides.
perl -C -pi -e 's/Perl (6|5)/Perl\x{A0}$1/g'  SOME-FILE

Added something to the STYLEGUIDE.md, I believe this could be useful to prevent others do my very same error. If that sounds ok, please close the issue, otherwise comment out and I will keep working on it.
I'm also thinking if we should include some kind of automatic non-breaking script within the htmlify.p6 one (or before in the build chain) to fix these kinds of mistakes.

👍 Looks great. Thanks!


⚠ Spryr has degraded performance

In 49bb92f, Spryr (https://spryr.com) experienced degraded performance:

HTTP code: 200
Response time: 5483 ms

Resolved: Spryr performance has improved in fef8210 after 9 minutes.


[Feature]: Add prefix option for commit messages

Description
Hi @di-sukharev,
Thanks for this wonderful tool.
It would be great to have the option to define a custom prefix message for the commits.
Suggested Solution
Something like:
custom-message: JIRA#
custom-message: AB#
custom-message: GI#

Alternatives

https://github.com/di-sukharev/opencommit/issues/107
https://github.com/di-sukharev/opencommit/issues/59
https://github.com/di-sukharev/opencommit/issues/45

Additional Context
This will allow us to pass the commit message validator in our CI pipelines. For e.g. in our team, we use Azure Board for defining commit messages.
AB#0 feat(back): add new backend module

Since this tool is already using the conventional commits standard, would be great to have an option to include our specific project prefix in the tool.

that's smart, would you like to make a PR?

@amyu98  sounds really powerful.

@dtcMLOps Ill give it a shot

@dtcMLOps @di-sukharev
https://github.com/di-sukharev/opencommit/pull/160
First time contributing to open-source, let me know what you think 😅


Laravel schema blueprint default values do not convert PHP Enums to It's primitive scalar type

Laravel Version
10.10.1
PHP Version
8.1.12
Database Driver & Version
No response
Description
I'm using the new PHP 8 enums in my repo, and i noticed that laravel already provides compatibility with them in the majority of cases, queries, eloquent models (with casts), etc. So i'm not sure if this is something that was missed...
But i noticed that it throws an error if used in migrations when defining default values for columns, see the code:
Schema::table('users', function (Blueprint $table) {
   $table->string('role')->default(RoleEnum::CUSTOMER);
});

Is that intended? This could solved with something like this in framework's  \Illuminate\Database\Schema\Grammars\Grammar.php
  /**
   * Format a value so that it can be used in "default" clauses.
   *
   * @param  mixed  $value
   * @return string
   */
  protected function getDefaultValue($value)
  {
      if ($value instanceof Expression) {
          return $this->getValue($value);
      }

      //here
      if ($value instanceof \BackedEnum){
          return "'".$value->value."'";
      }

      return is_bool($value)
                  ? "'".(int) $value."'"
                  : "'".(string) $value."'";
  }

I could open a PR for that if its appropriate.
Steps To Reproduce
-> Create a migration
-> Any column's default  method does not accept plain Enum values.

It seems it's not. Interesting PR discussion: https://github.com/laravel/framework/pull/45654

This isn't supported sorry and we have no plans to do this.


Bias term using NCD?

Hey!
I've been playing around with this method and I've noticed something interesting.
When calculating the NCD you use
ncd = (Cx1x2 - min(Cx1, Cx2)) / max(Cx1,Cx2)

And Cx1x2, Cx1 and Cx2 are computed using gzip.compress
The thing is, that the output of gzip.compress contains not only the compressed data but metadata on it.
This includes signature, compression method, flags, modification time, crc32 and more. Depends on the configuration this is at least 20 bytes that I think should not be a part of the NCD calculation as they don't represent any actual information of the string that was compressed.
On my WSL (Ubuntu 22.04) the default behavior is 20 bytes of metadata.
To be clear, essentially what happens is this:
ncd = (Cx1x2 + metadata_size - min(Cx1,Cx2) - metadata_size) / (max(Cx1, Cx2) + metadata_size)

Basically all the distances are calculated with a bias term in the denominator of +20, as the metadata size is nullified in the numerator.
This means that all samples receive a bias towards being closer to each other, and not in the same way for each two samples that the NCD is calculated for.
I've tested various values for metadata_size by adding a constant to the denominator on the AG News dataset.
Note that I specifically used majority vote for accuracy with different K's so the results are not the same as the paper's
When the denominator constant is 0 the value is the same as in the original implementation of NCD, -20 is how I suspect it should be.

I also tested this with increments of 20 up to 200

It seems that for AG News the higher the denominator constant the lower the accuracy.
Does this make sense or am I missing something?

hi @Maimonator, this is an interesting finding! The metadata occupying the denominator is larger than I thought.
It seems removing the bias increases the accuracy a bit? (Maybe needs to confirm by using e.g., -40 as the bias term and see if the accuracy decreases?)

hi @Maimonator, this is an interesting finding! The metadata occupying the denominator is larger than I thought. It seems removing the bias increases the accuracy a bit? (Maybe needs to confirm by using e.g., -40 as the bias term and see if the accuracy decreases?)

Yes I I think that removing the bias increases the accuracy but I wouldn't use anything higher than -20 as you increase the risk of zero division or getting negative values which as I understand the paper, the idea is just to normalize the distance to be between 0 and 1.
Also it's important to note that this has a larger impact on shorter strings .

Yeah, you are right - subtracting more than 20 may increase the chance of zero division.
This bias term will have more impact on shorter strings but I'm curious whether this "relatively higher" impact on the shorter string affects the preference for the length of the training sample.

Yeah, you are right - subtracting more than 20 may increase the chance of zero division. This bias term will have more impact on shorter strings but I'm curious whether this "relatively higher" impact on the shorter string affects the preference for the length of the training sample.

I'll try and find a dataset with class imbalance where I think the impact could be prominent in false negatives/false positives.
Maybe I'll do it later today 🤗


BLD: Use versioneer to pull version from git tag

Instead of having to update the code along with the git tag/release.
Be aware _version.py, versioneer.py, .gitattributes, and MANIFEST.in were all added/updated by versioneer itself, not manually by me.

LGTM


Release notes?

I noticed several new releases last month, I like that! Are there any release notes to find out what got changed/fixed/added?

I'm planning to setup a website with this type of information, primarily the last update was:
http://www.michaelridland.com/xamarin/announcing-freshmvvm-1-0-1-release/

Site is great idea but It would be also nice to have titles of changes on nuget, I believe :)


cleanup dead containers

I've been experiencing intermittent problems with app containers removal. Old containers would end up "dead" after cleanup. This should take care of it.

@michaelshobbs ?

Seems safe enough. @kdomanski can you confirm this works for you when you have dead containers?

@michaelshobbs Affirmative.

:+1:


:sparkles: Add contributors challenge 16

This PR adds various contributors to challenge 16. Still to be done:

Decide on a winner and add emoji 🏆  next to their name

From all of us on the Cloud Advocacy team at Microsoft, thank you for creating this pull request!
Have a Happy Holiday season!


Add initial react-loadable libdefs

https://github.com/jamiebuilds/react-loadable

Looks great, thanks!

Hi there, I think this PR might have broken my build. I get all sorts of errors, in particular:
Error: src/app/App.js:206
206:       CHUNKS_TO_PRELOAD.forEach(chunk => chunk.preload())
                                                    ^^^^^^^ property `preload`. Property not found in
206:       CHUNKS_TO_PRELOAD.forEach(chunk => chunk.preload())
                                              ^^^^^ object type

Error: src/app/App.js:206
206:       CHUNKS_TO_PRELOAD.forEach(chunk => chunk.preload())
                                                    ^^^^^^^ property `preload`. Property not found in
206:       CHUNKS_TO_PRELOAD.forEach(chunk => chunk.preload())
                                              ^^^^^ statics of React$Component

Looking at the code it doesn't look like the preload method is defined.

Hi @Gpx, changing module.exports in the following way might help
declare module.exports: <P>(opts: {
  loader(): Promise<LoaderResult<React$ComponentType<P>>>,
  loading: React$ComponentType<*>,
  delay?: number,
  timeout?: number,
  render?: (r: LoaderResult<React$ComponentType<P>>, p: P) => React$Node,
  webpack?: () => Array<string>,
  modules?: Array<string>
}) => React$ComponentType<P> & { preload(): void }

I didn't use .preload internally, so initial libdefs missing some functionality.

Thanks, @maxmalov. How can I test it? Can I just edit the file in my flow-typed lib?

Yes, this is the fastest way to get it working

Ok, this fixes the problem with preload but I still have issues with react-router. It looks like flow doesn't understand that Loadable objects can be used as react components.

I made a repo in case you want to check the error https://github.com/Gpx/loadable

Let's try this one:
declare class LoadableComponent<P> extends React$Component<P> {
  static preload(): void
}

declare module.exports: <P>(opts: {
  loader(): Promise<LoaderResult<React$ComponentType<P>>>,
  loading: React$ComponentType<*>,
  delay?: number,
  timeout?: number,
  render?: (r: LoaderResult<React$ComponentType<P>>, p: P) => React$Node,
  webpack?: () => Array<string>,
  modules?: Array<string>
}) => Class<LoadableComponent<P>>;

Seems to be working!

Nice, I'll try to revisit and fix these libdefs in the near future


CSMark Accuracy is not reset after each benchmark run

So, the CSMark Accuracy Level used is not reset after each benchmark run and causes the benchmark times to be increased as well as adding inconsistencies.
This will be fixed in 0.13.1

Fixed in 0.13.1


build: support space between coookie delimiter

Coverage remained the same at 93.548% when pulling dc77cec1383c7913987dd47d87d6a647d43e607b on next into 5d408ecfb02e3a3ecafc54d0dace3e8bb74ff928 on master.


[api-docs] document multi embed update

A new commit was made into the api-docs repo. This issue was created automatically. Need to review if we made any change to the library for this commit.
https://github.com/discord/discord-api-docs/commit/fd6501898c0827a04e7ab2eb49190f3fa9871028

ncn


FBX/GLB animation won't play when press PLAY to test the project. Cannot read property 'frame' of null

If there some specific export settings for animation to start to play?
Error Cannot read property 'frame' of null
This Error shows up if you clone Nunustudio from GitHub to localhost. WebEditor - works.
In the same time, StarWars example works on localhost without a problem even with deleted JS script

For example, regular Mixamo FBX
Jumping Down.zip
Or inside of the project already
RobotJumpingProjectNunu.zip
I tested on StarWars example, even you delete the JS script, the animation still play, even you re-save it without it.
Tried to use same script as in StarWards example, didn't work...
Not sure what to do.
Version
0.98.28
Platform

Web Version [X]
Windows [X]
Linux [X]

Hello
I will have to take a look into the file and figure out what is actually happening.
Thanks a lot
Cheers


fix: add HuggingFacePipeline model_id parameter

PR title: "Fixes issue with model parameter not getting initialized correctly when passing transformers pipeline"

Package: "langchain_huggingface"

Issue: #25915

Dependencies: None

The current version of the code contains default parameter of model_id in HuggingFacePipeline class. If the user passes a transformers pipeline object, ideally the default value should get overridden, but it doesn't. This PR fixes it.
Cheers!

@eyurtsev I have added the root initializer as you had suggested. Please review the changes.

Hello @eyurtsev can you review the changes?

Hi @NotShrirang, i re-merged the code against the updated master branch (which uses pydantic 2)
The original code did not look correct in terms of initialization logic since it was not handling the defaults properly -- i rewrote the code to match the description in the PR.
Given that the original PR had errors in it, could you please add a unit test before we can merge this PR that tests that initialization is working as expected.


fix perf issues

Leveraging qdrant index.
Can't really test this locally though without going back to my local-api branch, but is straightforward, will see if this fixes the Alan issue.
@spolu good to merge?

Awesome :+1:


Rebalance API mk X

Primarily changes the rebalance API to something which should be uncontroversial at this point.
Testing uncovered a problem (left as a todo to fix) - exceptions thrown in a librdkafka callback don't propagate through librdafka as i'd expect - they can't be caught (yet cause the program to terminate). This could be solved via additional bookkeeping but we can fix this in a future PR.
Also, a number of other smaller things:

Consumer.Position now operates only on a single TopicPartition, not a collection. This (A) actually does what the name says (B) matches the java API and (C) is no more difficult to use on a collection (because of LINQ) than the method that takes a collection whilst at the same time being easier to use for a single partition.
Similar changes to StoreOffsets (now StoreOffset). Similar reasons, except additionally, there is already an overload of StoreOffset which takes a ConsumeResult. Having both messys up the top level intellisense.
Change Offset.Invalid to Offset.Unset. There are two reasons. (A) The word Invalid is less appropriate in most of the places this constant is used. (B) The word Invalid is also a bit alarming - it risks users incorrectly inferring there is an issue when in fact there is not.
Fixed a mistake in the AssignPastEnd which has by fluke been passing 'till now due to semi-deterministic ordering of test execution.

Testing uncovered a problem (left as a todo to fix) - exceptions thrown in a librdkafka callback don't propagate through librdafka as i'd expect - they can't be caught (yet cause the program to terminate).
This could be solved via additional bookkeeping but we can fix this in a future PR.

If this the case where rd_kafka_poll() is called, triggers a callback, the managed code throws an exception, the callback and poll returns, but the exception is not properly propagated?
What we do in Python is have a bit of wrapping the callback  trampoline that checks if the managed code raised an exception, and if so save it on a callstate struct that is later from the code that called rd_kafka_poll() and then re-raises the exception, sort of.


InvalidCharacterError: Failed to execute 'createElement' on 'Document': The tag name provided ('/static/media/arrow-right.3e83082a.svg') is not a valid name.

when i try to run the DateRangePickerWrapper i get this error, has somebody else faced the same error? thanks

Do you mean the example? are you talking about in our storybook, or in your own site? In which browser?

managed to solve it (i think) (at least compiles now) by doing:
import React from "react";
const RightArrow = props =>

export default RightArrow;
on the RightArrow.jsx
i took the example as you have it on "DateRangePickerWrapper" and put it on a new page on my site.
However, what it only displays is the start and end date without the actual calendar even when i press on them. Does it have to do with the "focused" or "focusedInput" values? because the focused is what makes the calendar to show on the "SingleDatePicker" but on the range it is the focusedInput, if am not mistaken? and i have seen the comment on README saying: " the calendar is only visible if focusedInput is defined" . so even after initialising focusedInput the calendar does not appear

thats the only thing i get.

thats how i solved it, it looks like i cannot post the  here so took a screenshot

@ljharb just to add , in order to make the "DateRangePickerWrapper" compile i copied your src files needed to make "DateRangePickerWrapper" compile, to my src file

@Stathismi Thank you
I've fixed this problem with inline SVG imports.


Support for MPI

https://github.com/ggerganov/ggml/pull/340
https://github.com/ggerganov/llama.cpp/pull/2099
The project hasn't been updated for a month, so I'll find some work for someone to do

I would like to pick this up continuing parallelly from ggerganov/llama.cpp#2164
I believe rwkv might be more optimized for the use case I am on currently so if you can boil it a bit to generic steps I would need to take to complete it, I'll try to pick it up


Spacing / Styling of events less than ideal, especially regarding metadata terms

The relationship of the metadata to the events pages is not very good, and also titles for all the metadata are lacking. It should say Geographical focus: before geographical terms Thematic focus: before thematic terms and Related LandVoc Terms before related terms.
As, the positioning is really questionable and confusing, and the styling overall needs some attention.
https://landportal.org/event/2018/01/land-tenure-and-property-rights-mooc-30

hi @realneilsorensen
I'm closing this issue because it seems that this already has been changed in the site. Please re-open the ticket if there is still changes to be done. Thanks.


kubectl get triggerauthentication should show information about scaledobjects

kubectl get triggerauthentication should show information about scaledobjects where it is being used

As per @zroubalik :

It is not possible with current implementation. It would require some logic in the operator, to watch ScaledObjects/ScaledJobs and TriggerAuths and then map triggerauth usages and store them probably into triggerauth.Status. Then we will be able to print them here.

Hi all,  I am trying to solve this issue. Here is my proposal:

Add a Status variable in triggerauthentication and ScalingRules variables in Status for saving related ScaledObject/ScaledJob's name.
In reconciled of ScaledObject/ScaledJob controller, get triggerauthentication from ScaledObject/ScaledJob's authenticationref and update status of triggerauthentication.
The status will be string format as "kubectl get" still not supports array type so I will combine all ScaledObject/ScaledJobs' names into a string.

And here are my concerns:
I find that SO/SJ and triggerauthentication are not watch with each other. We may need to add logic in both SO/SJ and triggerauthentication. Or should we update the watch logic in SO/SJ to watch triggerauthentication's update? I prefer to add updating logic in both SO/SJ and triggerauthentication. What do you think? @tomkerkhove @JorTurFer @zroubalik

Any thoughts @JorTurFer @zroubalik?
I think the above approach is fine, except that we only need a "scaling rules" section.

I missed this issue 😨
Honestly, I won't do it because the information will not be usable from kubectl if there are 2 or more usages. I mean, we can update the .status for listing all the places where is used, but displaying them as part of kubectl output (even though I see the value) will finish with not usable behavior IMO.
As a user, I'd like this feature if the same TriggerAuthentication is used more than once, so the expected length of that field is large. Said this, I'd agree with having the information as part of the TriggerAuthentication .status and not displaying the info as part of kubectl, adding too much information makes the cli not usable IMO.
The question is, does having those values in status and not displaying them make sense?
I'd like to know what @zroubalik thinks too

(although I believe "status" is the wrong name for this though)

No no, I meant in .status section (like .metadata or .spec), something like
status:
  usages:
    - namespace/so-name
    - namespace/sj-name
    - namespace...

adding it to the output is just a simple effort that has big impact to operators/end-users.

I agree, but if the information is unstructured, it's not usable. operators/end-users can use jsonpath for extracting the information that they want if we have already set in the triggerauthentication.
Just to clarify, I'm not against adding the information to the triggerauthentication, I'm not sure about displaying it as part of the output.
As an example, we use a ClusterTriggerAuthentication in at least 10 places like these ScaledJobs:
- prd-mypoints-calculatepoints-subscriber-job/mypoints-calculatepoints-subscriber-job
- prd-mypoints-calculatepoints-publisher-job/mypoints-calculatepoints-publisher-job

If you print them as part of kubectl output, both will be displayed in a single line, which is not usable at all taking into account that kubectl displays a table with the info.
But I agree that having the info available to be queried using kubectl get ... -o jsonpath could be useful

The open PR has nice example of what it should like look IMO:

I think this is super valuable but curious about your thoughts why this is not ideal? (although I think we are aligned but talking about different things)

let me pull that code and show you what I mean

This is how it looks with one ScaledObject:

It looks well and useful, but this is how it looks with 3 ScaledObject:

Basically, the table is split into 2 lines, making the info not readable. In this case , I have 3 ScaledObjects for the example, but we in case of ClusterTriggerAuthentications, this could grow untill the moon, making the command literally not usable.
Due to this, I'd not expose the info as part of the command output but using -o jsonpath.
This approach also has another important point and it's that we can use arrays and treat each so/sj as an element in the array, not needing to concatenate them to use a string porperty, keeping to the users the option to join them or use them line by line

This is how it looks with one ScaledObject:  It looks well and useful, but this is how it looks with 3 ScaledObject:

Frankly, I do not have a problem with this, it is better than what we have today and only a workaround until Kubernetes has proper support. This should, however, require -o wide flag and not be done by default.

Due to this, I'd not expose the info as part of the command output but using -o jsonpath.

Why one or the other if we can do both?

If we add it only with -o wide I don't have any objection, but I prefer to not set it in the default output

I totally agree with you! @SpiritZhou Would you mind aligning with this please?

Sure. Will update it soon.


Examples in documentation outdated

All the examples in README.md contain this line:
final HolidayManager holidayManager = HolidayManager.getInstance(HolidayCalendar.GERMANY);

However, the method with the matching signature was deprecated for some time and eventually got removed in commit https://github.com/focus-shift/jollyday/commit/6dcddf0e8d705c9208f4fce09f60f295ab220ffe.

All the examples in README.md contain this line:
final HolidayManager holidayManager = HolidayManager.getInstance(HolidayCalendar.GERMANY);

However, the method with the matching signature was deprecated for some time and eventually got removed in commit 6dcddf0.

Thanks for your Issue. I will take a look and improve the readme.


[bug]: error TS7016: Could not find a declaration file for module '@strapi/design-system'.

What version of strapi-design-system are you using?
Npm version: 9
Node.js version: 18
Strapi Design System version: 1.13.0 | 1.13.1
Strapi version: 4.15.4
What's Wrong?
error TS7016: Could not find a declaration file for module '@strapi/design-system'.

Does it not support typescript yet?
To Reproduce
package.json:
{
  "devDependencies": {
    "@strapi/typescript-utils": "4.15.4",
    "typescript": "5.2.2"
  },
  "dependencies": {
    "@strapi/design-system": "1.13.0"
  }
}

tsconfig.json:
{
  "extends": "@strapi/typescript-utils/tsconfigs/admin"
}

index.tsx:
import { Layout } from '@strapi/design-system';

After creating the above files, run the following command
npm install && npx tsc --noEmit

Expected Behaviour
no errors occur

Types won't be published till V2

Why close this issue? Plugins using typescript now don't work anymore. Building the plugin works and doesn't produce any error. Building Strapi with the plugins will produce the errors statet above.


Low-Resolution YT Downloads

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm asking a question and not reporting a bug or requesting a feature
[X] I've looked through the README
[X] I've verified that I have updated yt-dlp to nightly or master (update instructions)
[X] I've searched known issues and the bugtracker for similar questions including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Please make sure the question is worded well enough to be understood
YT-DLP has suddenly started downloading YouTube videos in 360p, when higher resolutions are available. This is without any change in my local configuration at all.
Could Google have updated its algorithm to something that is somehow confounding YT-DLP?
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
C:\Users\Work\AppData\Roaming\yt-dlg>yt-dlp.exe -vU --newline -i -o "D:\Dev\Projects\Learning\Milan Jovanović\Handling Side Effects In Functional Code With Railway-Oriented Programming\%(title)s.%(ext)s" -f mp4 vGkgsduwnc4
[debug] Command-line config: ['-vU', '--newline', '-i', '-o', 'D:\\Dev\\Projects\\Learning\\Milan Jovanović\\Handling Side Effects In Functional Code With Railway-Oriented Programming\\%(title)s.%(ext)s', '-f', 'mp4', 'vGkgsduwnc4']
[debug] Encodings: locale cp1252, fs utf-8, pref cp1252, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>from yt-dlp/yt-dlp [12b248ce6] (win_exe)
[debug] Python 3.8.10 (CPython AMD64 64bit) - Windows-10-10.0.22631-SP0 (OpenSSL 1.1.1k  25 Mar 2021)
[debug] exe versions: ffmpeg 5.1.2-full_build-www.gyan.dev (setts), ffprobe 5.1.2-full_build-www.gyan.dev
[debug] Optional libraries: Cryptodome-3.20.0, brotli-1.1.0, certifi-2024.02.02, curl_cffi-0.5.10, mutagen-1.47.0, requests-2.32.2, sqlite3-3.35.5, urllib3-2.2.1, websockets-12.0
[debug] Proxy map: {}
[debug] Request Handlers: urllib, requests, websockets, curl_cffi
[debug] Loaded 1820 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Latest version<EMAIL_ADDRESS>from yt-dlp/yt-dlp
yt-dlp is up to date<EMAIL_ADDRESS>from yt-dlp/yt-dlp)
[youtube] Extracting URL: vGkgsduwnc4
[youtube] vGkgsduwnc4: Downloading webpage
[youtube] vGkgsduwnc4: Downloading ios player API JSON
[debug] Loading youtube-nsig.84314bef from cache
[debug] [youtube] Decrypted nsig ulKM4W9dLC5fd-0jmu => tG5u8D_GDPmYMg
[debug] Loading youtube-nsig.84314bef from cache
[debug] [youtube] Decrypted nsig UuRyY0GgKeIXGnoQe_ => taCg8YqpjUK_Gg
[youtube] vGkgsduwnc4: Downloading m3u8 information
[debug] Sort order given by extractor: quality, res, fps, hdr:12, source, vcodec:vp9.2, channels, acodec, lang, proto
[debug] Formats sorted by: hasvid, ie_pref, quality, res, fps, hdr:12(7), source, vcodec:vp9.2(10), channels, acodec, lang, proto, size, br, asr, vext, aext, hasaud, id
[info] vGkgsduwnc4: Downloading 1 format(s): 18
[debug] Invoking http downloader on "https://rr5---sn-bjp-5oje.googlevideo.com/videoplayback?expire=1719027708&ei=nPN1ZuGkDsPesfIPypig8Ao&ip=<IP_ADDRESS>&id=o-AL4uoXzzaGOryqEGZV0iUwcowJdjx3EN1Jkpj_-S48q9&itag=18&source=youtube&requiressl=yes&xpc=EgVo2aDSNQ%3D%3D&mh=12&mm=31%2C29&mn=sn-bjp-5oje%2Csn-o097znzr&ms=au%2Crdu&mv=m&mvi=5&pl=21&initcwndbps=1147500&bui=AbKP-1Mlh6vKWWLN--Bsaa5f_ybx6IoWRR6R8I-WUEhcBr2CjxUg-slyzlsHd1NiUT362okbXNdOG2T2&spc=UWF9f7r3Gu6_Tp9b_13667T-I1m9s1wzFUiGFDxIbXbhzu2JA87hm6hVSmB-&vprv=1&svpuc=1&mime=video%2Fmp4&ns=QBVYJqe5kG267ULvcx1POxYQ&rqh=1&cnr=14&ratebypass=yes&dur=889.649&lmt=1697432488643738&mt=1719005820&fvip=5&c=WEB&sefc=1&txp=6308224&n=tG5u8D_GDPmYMg&sparams=expire%2Cei%2Cip%2Cid%2Citag%2Csource%2Crequiressl%2Cxpc%2Cbui%2Cspc%2Cvprv%2Csvpuc%2Cmime%2Cns%2Crqh%2Ccnr%2Cratebypass%2Cdur%2Clmt&sig=AJfQdSswRQIhAL-gSYtClLsWGLookY7hUuf5C4izJdDOX8Gau-mwpe89AiBCB-IPw5XhArx_n6U3XPvlqufYX1IOuO9jBWzV7g8d5A%3D%3D&lsparams=mh%2Cmm%2Cmn%2Cms%2Cmv%2Cmvi%2Cpl%2Cinitcwndbps&lsig=AHlkHjAwRAIgbGPlZ2Em3ffVZ8HULthsISpMA3mOzfN86ukBEhzB5eICIC5szyfhJqDgtCJpUr1Ns9vlmC8IJ92qRO5qpsk9Kpn_"
[debug] File locking is not supported. Proceeding without locking
[download] Destination: D:\Dev\Projects\Learning\Milan Jovanović\Handling Side Effects In Functional Code With Railway-Oriented Programming\Handling Side Effects In Functional Code With Railway-Oriented Programming.mp4
[download]   0.0% of   24.97MiB at   86.99KiB/s ETA 04:53
[download]   0.0% of   24.97MiB at  248.72KiB/s ETA 01:42
[download]   0.0% of   24.97MiB at  554.08KiB/s ETA 00:46
[download]   0.1% of   24.97MiB at    1.11MiB/s ETA 00:22
[download]   0.1% of   24.97MiB at    2.18MiB/s ETA 00:11
[download]   0.2% of   24.97MiB at    2.58MiB/s ETA 00:09
[download]   0.5% of   24.97MiB at    3.46MiB/s ETA 00:07
[download]   1.0% of   24.97MiB at    4.11MiB/s ETA 00:06
[download]   2.0% of   24.97MiB at    3.73MiB/s ETA 00:06
[download]   4.0% of   24.97MiB at    6.16MiB/s ETA 00:03
[download]   8.0% of   24.97MiB at   10.11MiB/s ETA 00:02
[download]  16.0% of   24.97MiB at   16.95MiB/s ETA 00:01
[download]  32.0% of   24.97MiB at   23.99MiB/s ETA 00:00
[download]  38.5% of   24.97MiB at   26.23MiB/s ETA 00:00
[download]  38.5% of   24.97MiB at  Unknown B/s ETA Unknown
[download]  38.5% of   24.97MiB at    1.54MiB/s ETA 00:10
[download]  38.5% of   24.97MiB at    2.35MiB/s ETA 00:06
[download]  38.5% of   24.97MiB at    2.99MiB/s ETA 00:05
[download]  38.6% of   24.97MiB at    5.13MiB/s ETA 00:02
[download]  38.7% of   24.97MiB at    4.58MiB/s ETA 00:03
[download]  39.0% of   24.97MiB at    3.37MiB/s ETA 00:04
[download]  39.5% of   24.97MiB at    4.39MiB/s ETA 00:03
[download]  40.5% of   24.97MiB at    6.01MiB/s ETA 00:02
[download]  42.5% of   24.97MiB at    9.29MiB/s ETA 00:01
[download]  46.5% of   24.97MiB at   14.44MiB/s ETA 00:00
[download]  54.5% of   24.97MiB at   20.94MiB/s ETA 00:00
[download]  70.5% of   24.97MiB at   26.64MiB/s ETA 00:00
[download]  77.5% of   24.97MiB at   28.44MiB/s ETA 00:00
[download]  77.5% of   24.97MiB at  Unknown B/s ETA Unknown
[download]  77.5% of   24.97MiB at 1002.54KiB/s ETA 00:05
[download]  77.5% of   24.97MiB at    1.37MiB/s ETA 00:04
[download]  77.5% of   24.97MiB at    1.63MiB/s ETA 00:03
[download]  77.6% of   24.97MiB at    3.03MiB/s ETA 00:01
[download]  77.7% of   24.97MiB at    2.93MiB/s ETA 00:01
[download]  78.0% of   24.97MiB at    3.39MiB/s ETA 00:01
[download]  78.5% of   24.97MiB at    4.69MiB/s ETA 00:01
[download]  79.5% of   24.97MiB at    7.10MiB/s ETA 00:00
[download]  81.5% of   24.97MiB at   10.43MiB/s ETA 00:00
[download]  85.5% of   24.97MiB at   15.94MiB/s ETA 00:00
[download]  93.5% of   24.97MiB at   23.72MiB/s ETA 00:00
[download] 100.0% of   24.97MiB at   28.95MiB/s ETA 00:00
[download] 100% of   24.97MiB in 00:00:01 at 20.01MiB/s

If you pass -f mp4 then yt-dlp is going to look for the "best" single mp4 format (i.e. it will not try to merge separate video and audio formats). Youtube is doing their best to make format 22 unavailable for most videos, which was the previously available 720p pre-merged format.
The solution is to remove -f mp4 from your command and replace it with -S vcodec:h264,res,acodec:aac

That works, thank you.


RabbitMq message getting lost

Environment :
spring-boot-starter-amqp : 2.1.4
Platform : Windows 10
Java : 11
I am using RabbitTemplate.convertAndSend , it does not throw any exception or error but message is not published to the exchange (direct exchange in this case). There is no error in rabbitMq server log as well. Any Help would be appreciated.
Sample metrics ::
No of message sent ------------------>Found in Queue
25--------------------------- ----------->19
100-------------------------------------->69
100--------------------------------------->80
Note: Metrics is just to show that, there is no pattern for number of missing messages.

You need to show your code and configuration (queues, exchanges, bindings).

I had two different message producer sending notification to their respective exchange queue(bound with routing key) while having a single rabbitTemplate instantiated. Both the producer were setting the default exchange using rabbitTemplate.setExchange(), hence leading to default exchange inconsistency.
Before ,
Producer 1,
rabbitTemplate.setExchange("EXCH1");
rabbitTemplate.setRoutingKey(routingKey1);
rabbitTemplate.convertAndSend(notificationObj);
Producer 2,
rabbitTemplate.setExchange("EXCH2");
rabbitTemplate.setRoutingKey(routingKey2);
rabbitTemplate.convertAndSend(notificationObj);
Now on, directly using rabbitTemplate.convertAndSend(exchangeName, routingKey, resourceNotificationTypeObj) and as expected, it resolved the issue.
Appreciate your quick reply.


Authorization token not being added to request headers

Describe the bug
When attempting to query a GraphQL API I have locally with OAuth2 Implicit credentials, the Authorization header is not being added to the request. I am able to get the Acces Token to populate and if I manually add an Authorization header with that token (prefixed with "Bearer ") it works just fine. However, this requires needing to copy and paste the token into the headers section. From the documentation, it looks like the request should automatically add the header to my request without this additional step.
From my about:
Version: Insomnia Core 2020.4.2
Release date: 10/21/2020
OS: Windows_NT x64 10.0.19041
Electron: 9.1.1
Node: 12.14.1
V8: <IP_ADDRESS>-electron.0
Architecture: x64
node-libcurl: libcurl/7.69.1-DEV OpenSSL/1.1.1d zlib/1.2.11 WinIDN libssh2/1.9.0_DEV nghttp2/1.40.0
To Reproduce

Create new request, set auth type to "OAuth2" and Grant type to "Implicit". Fill out AuthorizationURL, Client Id, Redirect URL, Response Type = Access Token, Scope, and State (empty) as values.
Optionally set Header Prefix (I tried with it empty and with "Bearer") and Audience (wasn't needed for me).
Click Fetch Tokens and log in to your OAuth2 Provider. The Access Token should populate.
Ensure the "Header" tab does not include an "Authorization" header
Send the request.
Note that, if the endpoint required authentication, it fails.
Check the Timeline tab on the response - note that the Request header information does NOT include Authorization.

Expected behavior
I expect when the Authorization tab is properly able to create a token, or has already received one, that that Authorization header will be added to the outbound request when sent.
Screenshots

Desktop (please complete the following information):

OS: Windows 10 Enterprise Version 2004 (19041.572)
Installation Method: Insomnia.Core-2020.4.2.exe downloaded from https://insomnia.rest/download/#windows
App Version v2020.4.2

Additional context
I've had other developers at my company have the same issue with Insomnia on their machines, and they also had to copy and paste the token into the headers. I've also tried Authorization Code and it's the same - the tab can retrieve a token, but it is not added to the request at all. I also copied the token and tried using Bearer with the same results.

Same issue here when using Oauth 1.0 with the here.com token request. Replicated it in postman all I had to do was select was "Add Authorization header to" and choose "Request Headers" and it worked. I can't find any option in Insomnia to do the same thing.

OAuth2 headers are now being added automatically for me in Insomnia Core 2020.5.2.

OAuth2 headers are now being added automatically for me in Insomnia Core 2020.5.2.

Hey
I also have this problem.  I have to manually add an Authorization header and copy my token into it (adding Bearer before)
I didn't always have this issue it seems to sometimes happens - but when it happens it stays like that.

Same issue here when using Oauth 1.0

For me this is still isn't working in the latest version:
Version: Insomnia 9.3.3
Build date: 31/07/2024
OS: Windows_NT x64 10.0.19045
Electron: 30.2.0
Node: 20.15.0
Node ABI: 123
V8: <IP_ADDRESS>-electron.0
Architecture: x64

@ops-neodynamics-io ”works on my machine”


Fixing bwc tests

Description
https://github.com/opensearch-project/OpenSearch/pull/15530/ has some version checks which needs to be forward ported
Related Issues
Resolves #[Issue number to be closed when this PR is merged]

Check List

[ ] Functionality includes testing.
[ ] API changes companion pull request created, if applicable.
[ ] Public documentation issue/PR created, if applicable.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

https://github.com/opensearch-project/OpenSearch/pull/15572 - already a open PR is present for same , so closed this


V next yarong 417 812 merge v next

I merged vNext with vNext_yarong_417_812. I then tested both desktop (which passes with V8, not with SM, but those are known issues) as well as doing a full mobile build (which appears to have succeeded). But there are enough changes here that I wanted @larryonoff and @andrew-aladev to take a look before I checked it in.

This change is

PR is added to the queue for testing as 1. task. (8a10bf1)

Test (Success) 84389264 build is completed (8a10bf1)
See https://github.com/ThaliTester/TestResults/tree/843892648a10bf1_V_next_yarong_417_812_merge_v_next_yaronyg/ for the logs

Test 843892648a10bf1(8a10bf1) has failed
See https://github.com/ThaliTester/TestResults/tree/843892648a10bf1_V_next_yarong_417_812_merge_v_next_yaronyg/ for the fail logs

@yaronyg check please this log  and search for My device.
You will see an error
Error!: Error when loading file /data/data/com.test.thalitest/files/www/jxcore/bv_tests/testLocalSeqManagerCoordinated.js: Error: Cannot find module '../../thalilogger'

that (and some other bugs) I fixed in #936. I highly recommend to merge #936 into #928 and only after that work with vNext.

PR is added to the queue for testing as 2. task. (11e1d40)

@yaronyg I check this branch and already fixed the post-merge issue that I've found. I'm not sure that @andrew-aladev needs to check it, because all merged changes are mine.
As far as I understand this branch won't build, because there should be changes that @artemjackson did in his branch that fixes #vNext_yarong_417_812 branch.

Test (Fail) 84389264 build is completed (11e1d40)
See https://github.com/ThaliTester/TestResults/tree/8438926411e1d40_V_next_yarong_417_812_merge_v_next_yaronyg/ for the logs

We will try to do this merge later we first want to get #928 working


Add docker-compose support

#81

目前 playground 依赖的 NEXT_PUBLIC_REQUEST_URL 环境变量只在编译时有效，修改后需要重新 docker compose build
next框架不太熟悉

I had a brief discussion with the other engineers yesterday. We believe that most web developers should already be familiar with how node and nvm work, which is why we’re not using docker compose build.
However, @Lensual, your point is still valid for developers who aren’t as familiar with web development tech stack . I’ll mark the issue with the question label and see what the other engineers have to say about it.

Something worth mentioning: we are at the AdventureX 2024 Hangzhou Hackathon. It’s clear that many developers aren’t familiar with the challenges of using Node.js and nvm, especially when dealing with AI, where Python and its environment are predominant.
This makes me think that having a docker-compose file is the right approach. It solves a lot of pain when getting the entire framework up and running.

@Lensual
docker-compose build astra_playground 会报错, 不能构建出镜像
1.playground/.env.example 后续这个会移除
2..env.sample 这个建议不这样用, docker-compose.yml 改成 docker-compose.yml.example 类似, 使用的时候拷贝一个命名为 docker-compose.yml, 并修改环境变量值

@Lensual We have concluded that having docker-compose is the right thing to do for improving DX. @sunshinexcode will raise a PR for the change. Feel free to review it.

PR: https://github.com/rte-design/ASTRA.ai/pull/123

@Lensual take a look, if #123 is something you intended to do in the first place? Also from your perspective, which is someone who doesn't know the web development very well, do you think having this docker-compose is gonna boost the EX overall?

@Lensual take a look, if #123 is something you intended to do in the first place? Also from your perspective, which is someone who doesn't know the web development very well, do you think having this docker-compose is gonna boost the EX overall?

Yes, it looks good.


bump max series to 120

rather than limiting to 20... this makes it 120.  or we could remove it

#31


for loops for bigint is very slow

in Nim 2.0
import bigints

var x:BigInt = 1.initBigInt

for i in 1 .. 99999:
    x *= i.initBigInt
echo x

takes about 19 seconds in my computer, but Python 3.10 and Julia 1.6.7 only takes less than 2 seconds...

Please share the Python and Julia code.
This huge time is due to the initialisations (initBigInt) inside the for loop.
Note that you should prefer a dedicated function for factorial computation, a direct for loop has an absurdly high binary complexity.
To lower the binary complexity, you may have a look at https://forum.nim-lang.org/t/8967.
I do not know about Julia, but Python has a native support for multiprecision integers with fast simple multiplication integers (Karatsuba).
I have begun to implement it, but I do not have the time to finish my pull request. Feel free to look at #95, if you are eager to contribute to this library.
If you really need to compute factorials fast and efficiently for your application, consider using e.g. Pari/GP or Sagemath or any good formal computer library.
This computation factorial(99_999) should not take more than 20ms when well parallelized.

Please share the Python and Julia code.
This huge time is due to the initialisations (initBigInt) inside the for loop. Note that you should prefer a dedicated function for factorial computation, a direct for loop has an absurdly high binary complexity. To lower the binary complexity, you may have a look at https://forum.nim-lang.org/t/8967.
I do not know about Julia, but Python has a native support for multiprecision integers with fast simple multiplication integers (Karatsuba). I have begun to implement it, but I do not have the time to finish my pull request. Feel free to look at #95, if you are eager to contribute to this library.
If you really need to compute factorials fast and efficiently for your application, consider using e.g. Pari/GP or Sagemath or any good formal computer library. This computation factorial(99_999) should not take more than 20ms when well parallelized.

Thank you very much! I am just a beginner of Nim. So, this is a toy test. I didn't optimize the julia codes or python codes.
let
    x::BigInt = 1

    @time for i in 1:99999
        x *= i
    end
    println
end

x = 1
for i in range(1,100000):
    x = x * i
print(x)

I tried to use the bignum, but always errors...so, thank you again!

Note that when compiling with the -d:danger flag, timings on my machine went from 23 seconds to 8 seconds.
I tried to remove initBigInt calls, but timings were similar:
import bigints

var x:BigInt = 1.initBigInt
var counter = 1.initBigInt

for i in 1 .. 99998:
  x *= counter
  counter.inc
x *= counter
echo x

Note that when compiling with the -d:danger flag, timings on my machine went from 23 seconds to 8 seconds. I tried to remove initBigInt calls, but timings were similar:
import bigints

var x:BigInt = 1.initBigInt
var counter = 1.initBigInt

for i in 1 .. 99998:
  x *= counter
  counter.inc
x *= counter
echo x

Great! I think bigints or Nim need more optimizations...I will return to check it in future!


Remove hardcoded links from the website. Use route names.

https://github.com/indianracingcommunity/RLIProject/blob/2f3bb79adc68586ec16694b7a5a4bf8bc5770212/resources/views/layouts/app.blade.php#L409


Install docs update

updates the install section to match Bryhn-Bjolgerud changes in azure_ad_config_guide. Ignore this pull request if it's not necessary

The README is only "wrong" because it's settings for ADFS and not Azure AD, so I'll close this one. Thanks for all the docs you guys have contributed with :tada:


Request data is form-encoded instead of passed as json payload

The query
POST /naavre-communicator/external-service

{
  "query": {
    "method": "POST",
    "url": "https://naavre-dev.minikube.test/NaaVRE-containerizer-service/containerize",
    "headers": {},
    "data": {
      "something": "random"
    }
  }
}

results in
POST https://naavre-dev.minikube.test/NaaVRE-containerizer-service/containerize?something=random
Authentication: Bearer ...

but we want
POST https://naavre-dev.minikube.test/NaaVRE-containerizer-service/containerize
Authentication: Bearer ...

{
  "something": "random"
}

This is because we use requests.request(..., data=data) at https://github.com/NaaVRE/NaaVRE-communicator-jupyterlab/blob/3d54814e9d5e8de908fc3c079a18c2d999e96e72/NaaVRE_communicator_jupyterlab/handlers.py#L77
Because data is a dict, it is url-encoded:

data – the body to attach to the request. If a dictionary or list of tuples [(key, value)] is provided, form-encoding will take place.
requests.Request documentation

We should use json instead.


stage xPro tracking log

What are the relevant tickets?

https://github.com/mitodl/ol-data-platform/issues/765
Description (What does it do?)

This adds stg__mitxpro__openedx__tracking_logs__user_activity model from xPro tracking logs. It

extracts course_id and other common metadata from the context field. context has additional fields based on different events that can be extracted per event type base.
normalizes the time field as this value has inconsistent date formats due to log collector changes
filters out any blank username/user_id, or server exception events as those are not useful

This model is materialized as incremental due to the size
How can this be tested?

dbt build on stg__mitxpro__openedx__tracking_logs__user_activity, tests should pass

This can't be merged until https://github.com/mitodl/ol-data-platform/issues/771 is resolved for duplicate data in raw table

I've handled de-dupe for https://github.com/mitodl/ol-data-platform/issues/771 in this PR, and tests are passed


BgpNextHop array - ip split by ,

BgpNextHop appears to spit the IP into an array
"BgpNextHop": [
192,
168,
3,
132
],
is this expected? I would have thought we should have:
"BgpNextHop": [
<IP_ADDRESS>,
],

Hello @mooneym17
I will need a bit more information: what version, what CLI arguments are you passing?

@ispgn
sure, nothing special, just running the container as so:
podman run --net host -d --name goflow2  goflow2:v1.3.4   -listen "netflow://:4738" -metrics.addr ":8091" -reuseport -transport "kafka"  -transport.kafka.brokers "<IP_ADDRESS>:9092" -transport.kafka.hashing -workers 1

Will give it a try. Thank you for the report :)
Would you be able to test out v2.0.0?
This may handle better formatting

hi ispgn, with version 2.0.0 , BGP nexthop appears to show as hex, ie: "bgp_next_hop":"5167df41"

Hi @mooneym17,
Sorry for the delay.
In the mapping.yaml for v2, could you add the following:
formatter:
  render:
    bgp_next_hop: ip

I realize this is listed here but it's a bit hidden.
This said, as bgp_next_hop is a supported field, I will force the default renderer.

Great. That works.


[WIP] Change runtime channel for worker message passing

Purpose
$subject
Fixes #
Approach

Describe how you are implementing the solutions along with the design details.

Samples

Provide high-level details about the samples related to this feature.

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Closing the required changes are added with PR https://github.com/ballerina-platform/ballerina-lang/pull/41754


Elixir errors noisy

One thing we are struggling with is that Sentry is getting events for every single error , which is not ideal due to the "let it fail" aspect of elixir application development. Most of these errors are perfectly acceptable "failures" that we don't need to know about. The problem is that between ~10 elixir applications we're exceeding our quota really fast each month.
I don't know if this is something that can be addressed here or if we just need to use the tool differently (for example, catch and explicitly send only the important errors, perhaps?).
It would be nice if there was a way to add errors to an ignore list so that they wouldn't even be sent to Sentry. If this concern/question should go somewhere else, feel free to close it up!

@jedeleh thanks for opening an issue 🙂
I have a couple ideas on how improve on this.  There is support for filtering, described here: https://hexdocs.pm/sentry/Sentry.html#module-filtering-exceptions
Could you elaborate on the acceptable failures?  Are these processes crashing, Plug errors, or something else?

I think the thing we see the most, are db connection issues during application deploy. The specific error message is one we never see outside those brief moments when the database is down during a quick deploy, but we end up with hundreds of error events each time it happens. The link you provided (to filtering) is a pretty nice solution. I hadn't seen that. At the moment we're using the before_send_event/1 callback (and fingerprinting) but I really like the filtering approach better.
thanks for your quick response :)

Awesome.  I have been meaning to add more documentation around the EventFilter Behaviour, and plan to take care of that this week.
Were there other specific errors or categories of errors that are noisy?


Probably a bug

Hi,
It is probably a bug or something else is wrong.
When you start playing audio and then hit loop button, audio will start over when it finishes playing, but if you, while playing, will navigate through timeline for at least one time, it will not start over again, you will have to press play button again and wait till end.

Can you share a URL where I can check this please?

I tested this using mediaelement-plugin repository and the latest version of the MEJS plugin. I couldn't reproduce that error. Tested on Chrome, FF and Safari


Adding label to transparent qr makes it opaque again

If i configure like
$qrCode = new QrCode('https://www.google.com');
$qrCode->setSize(1000);
$qrCode->setMargin(50);
$qrCode->setWriterByName('png');
$qrCode->setEncoding('UTF-8');
$qrCode->setErrorCorrectionLevel(ErrorCorrectionLevel::HIGH());
$qrCode->setBackgroundColor(['r' => 255, 'g' => 255, 'b' => 255, 'a' => 127]);
$qrCode->setValidateResult(false);
$qrCode->setRoundBlockSize(true, QrCode::ROUND_BLOCK_SIZE_MODE_MARGIN);
header('Content-Type: ' . $qrCode->getContentType());
echo $qrCode->writeString();
I get a nice QR with a transparent background!
But when I add:
$qrCode->setLabel('Testtest', 72,storage_path().'/app/fonts/Alata-Regular.ttf', LabelAlignment::CENTER());
The transparent background turns opaque again.

What is the a parameter in $qrCode->setBackgroundColor(['r' => 255, 'g' => 255, 'b' => 255, 'a' => 127]);  ???
I read through all the documentation and can't find it.  I assume its a transparency value of some sort.   Can it also be applied to the setForegroundColor parameter as well?

Hi @rolinger, "a" stands for alpha and can be applied to the foreground color as well ;)

@MelchiorKokernoot this should be fixed.


Editing in comments should not bring up completions

This behavior is new in master branch.
If that was added for VSCode, then we should have it optional so that VS can opt out of it.

Haha, whoops. This is just a bug

Seems to still repro in VSC, see https://github.com/Microsoft/vscode-python/issues/1775.


rotation manipulator

hi
In addition to moving, we can also rotate.
Used the rotation manipulator

I'm not sure I understand your issue. Can you explain a bit more?


File organization

From yobi9's Discord comment on 3/10/21:

File organization: atm, all the files are dumped at the root of the repo: python files, and documentation images. Some folders cleanup would be nice.

Hi @timotheeg !  I made some progress on this issue today.  For fun, I decided to finally sit down and try writing a "proper" Config class bc I noticed it was the first TODO that you'd commented in the code 🙂.  I've coded in Python for a while now but I've (shamefully 🙄) always essentially done it the way that you did it bc it was always my own side project "weekend-code-warrior" thing 😅.  Definitely worked, but very brittle...  God help the poor soul who tried accidentally introduced a bogus key into my config.json 😬...
So here's my first attempt:
https://github.com/r002/tetris_ctm_summary/blob/d42f61b09c50e71c73545bd0ed3e676aa19946c0/main/config.py
You can run its "test harness" from root with $ python -m main.config
So with this implementation thus far, I've handled two cases:
1. If config.json doesn't exist, it defaults to the hardcoded values:

2. If there's a bogus key in config.json, throw an error:

So that's progress.
But I'm running to a problem... if someone specifies a config key of the wrong type, the Python code just accepts it:
Eg. "print_score_difference": 123123 in config.json yields:

Nooooooo.... Curse you, dynamically-typed-language!!!  😤😤😤
What is the easiest way to catch this?  Maybe my mind is just too fried but I want to make
https://github.com/r002/tetris_ctm_summary/blob/d42f61b09c50e71c73545bd0ed3e676aa19946c0/main/config.py#L46
fail when it tries to populate my Config dataclass with a field of the wrong type.  (And preferably with a one-liner; would like to avoid explicitly writing boilerplate if condition type checks in Config  if at all possible.)  Any suggestions?
Thank you! 🙏🙏🙏

PS.  Copy/pasting this musing from the Discord.  Contemplating the best "expected behavior" of how the config loading should work:

Also, taking a step back:  With the config values:  In the spirit of "explicit is better than implicit"-- if a config value is missing (or malformed) in config.json, is it better to:

Exit the program entirely with error message.

Reject config.json entirely (with error message), load all config values from a default set, and continue running the program.

Load well-defined values from config.json but fall back on defaults for any specific missing (or malformed) config values (with error message).

My instinct is (1) or (2) but not (3).  Thoughts...? :thinking:


Feature request

Hi Andy,
This looks great!  Would it be possible to add tab navigation for iOS and drawer navigation for Android in future releases?

yes, there are actually both drawer & tab experiences in the UI kit, so will build them when I get to those portions.


Refactored client wrappers

Now wrapped clients will derive from OpenstackClientWrapper, which
provides client caching, and call retrying.
Adapted NovaClientWrapper and IronicClientWrapper to use
OpenstackClientWrapper.
Updated tests to reflect the new reality.
Updated novaclient requirement to use 2.22.0.

Quick comment on my thinking here: With the knowledge that I need to pull in the glance client, I wanted a common base to wrap clients with. I didn't feel comfortable pulling in shade. Not quite the functionality I wanted, and this route lets me only pull in the clients I want, as well as pin client requirements.

comments inline, otherwise lgtm


proxy config works for some urls but not others, illogical and inexplicable behavior

my proxy.conf.json file:
{
  "/v1": {
    "target": "https://example.com/v1",
    "secure": true,
    "pathRewrite": {
      "^/v1": ""
    },
    "changeOrigin": true
  }
}

path that works:
/v1/api/controller/endpoint

path that does not work:
/v1/controller/endpoint

Given that I am proxying all urls that start with /v1 I would expect both of the aforementioned paths to work however that is not the case, /v1/controller/endpoint returns 404 for some reason. I tried creating another entry in the proxy.conf.json with a completely arbitrary value like for example instead of /v1 I used /cat and then my url looked like /cat/controller/endpoint but that did not work either... I find this completely illogical, I thought that if we have that /v1 proxy key it would mean that any url that starts with /v1 will be proxied, therefore it would be easy to just prefix whatever we need with /v1 and be done with it, but nooooooo, it doesn't work and it is driving me crazy.
Both of the endpoints have been tested through Swagger and Postman and they both work, just unable to proxy to them from the Angular.
The actual paths exposed from the back end look like this:
/v1/api/users/count
/v1/auth/token

Angular CLI: 12.2.5
Node: 14.15.4
Package Manager: npm 6.14.10
OS: win32 x64

Angular: 12.2.5
... animations, cdk, cli, common, compiler, compiler-cli, core
... forms, material, platform-browser, platform-browser-dynamic
... router

Package                         Version
---------------------------------------------------------
@angular-devkit/architect       0.1202.5
@angular-devkit/build-angular   12.2.5
@angular-devkit/core            12.2.5
@angular-devkit/schematics      12.2.5
@schematics/angular             12.2.5
rxjs                            6.6.7
typescript                      4.3.5

I'm sorry, but this issue is not caused by Angular CLI.
The CLI uses webpack-dev-server as a development server which in turn uses http-proxy-middleware
Can you try the simplify the configuration to the below?
{
  "/v1": {
    "target": "https://example.com",
    "secure": true,
    "changeOrigin": true
  }
}

If the problem persists please contact the author(s) of http-proxy-middleware project or file an issue on their issue tracker.

nope, that doesn't help... well thanks anyway, cheers


Add StarOSTablet serialization and tiny refactor

What type of PR is this：

[ ] bug
[x] feature
[ ] enhancement
[ ] others

Problem Summary(Required) ：

Use json object to serialization.
Add getRowCount interface in Tablet.

[FE PR Coverage check]
:heart_eyes: pass : 37 / 41 (90.24%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/http/rest/RowCountAction.java
0
1
00.00%

:large_blue_circle:
com/starrocks/catalog/TabletStatMgr.java
0
1
00.00%

:large_blue_circle:
com/starrocks/catalog/LocalTablet.java
7
9
77.78%

:large_blue_circle:
com/starrocks/sql/optimizer/rule/transformation/LimitPruneTabletsRule.java
1
1
100.00%

:large_blue_circle:
com/starrocks/catalog/StarOSTablet.java
28
28
100.00%

:large_blue_circle:
com/starrocks/catalog/OlapTable.java
1
1
100.00%

run starrocks_fe_unittest

run starrocks_fe_unittest

[FE PR Coverage check]
:heart_eyes: pass : 22 / 26 (84.62%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/http/rest/RowCountAction.java
0
1
00.00%

:large_blue_circle:
com/starrocks/catalog/TabletStatMgr.java
0
1
00.00%

:large_blue_circle:
com/starrocks/catalog/LocalTablet.java
7
9
77.78%

:large_blue_circle:
com/starrocks/sql/optimizer/rule/transformation/LimitPruneTabletsRule.java
1
1
100.00%

:large_blue_circle:
com/starrocks/catalog/StarOSTablet.java
13
13
100.00%

:large_blue_circle:
com/starrocks/catalog/OlapTable.java
1
1
100.00%


[Do not merge] Sample Djinni Outputs

The sample generated Djinni outputs.
Djinni sources:

Account
AuthorityType
Credential
CredentialType
ErrorCodes
IStorageManager
JsonKeys
JsonValues
OperationStatus
ReadAccountResponse
ReadAccountsResponse
StatusType

@shoatman For your perusal. Not planning to merge.

@iambmelt Thanks Brian.  Looks similar to the last version I saw generated.

Objects will likely change in the future: unsafe to take a dependency on these implementations.


How to test the model?

Hello
I am able to train using your code but how do I test the model to see the results and visualize them? I don't see the script for that.

The training scripts also evaluate the model at the end of very epoch.  You can also use the flag --visdom to enable logging to visdom which will generate graphs.
There isn't a dedicated script for evaluating the models or visualizing results.

Hello @erikwijmans  ,
For what reasons scripts under tests are created !! If for evaluate the model,how I can use them!
thanks in advance,
Raouf

The scripts in tests are integration tests of the code base to make sure everything is still working after changes.
There currently aren't any dedicated evaluation scripts.  If you'd like to make one and contribute it, I'd be more than happy to accept that PR.

hello @erikwijmans ,
how to use pytest to test the code base(scripts in tests) .what are the steps to follow?
Thanks in advance,

The https://github.com/erikwijmans/Pointnet2_PyTorch/blob/master/tox.ini shows the commands and dependencies needed for the tests.  If you'd just like to run them, you can install tox via pip install tox and then simply executing tox will run all the tests.


How can i add another country data??

I have installed and imported single country data and it is running successfully. Now i want to add another country data. please help me.

You can merge PBF files.
Instructions here

The above is correct and is the easiest method to import multiple countries. However, I don't see how you could use the update function for this file since it is original and may contain multiple regions.
Maybe you can use the new importing tool from latest nominatim's version.
You'd have to use this script inside init.sh and replace the line sudo -u nominatim ./src/build/utils/setup.php --osm-file $OSMFILE --all --osm2pgsql-cache 8000 2>&1 --threads $THREADS && \ with the script from utils/import_multiple_regions.sh
Also either change the parameter OSMFILE to add more files or change the nominatim script enabling you to download them before you init the database by editing it and importing it back into docker's container.
I think there's a potential upgrade to be made to this docker project, hopefully someone could take advantage of that functionality