initial distributed summary plot

As per #16, plus a few additions to support scatter, and a few tweaks. @slundberg - it's not finished yet, but can you give some initial feedback:

function call: I put all the arguments at the end so as to be backward compatible, but it's not as clean
do you have any idea about the sum of individual kdes not equalling the overall kde?
any comments on the visuals and changes, including code style etc.

Examples below. Note that the 2nd and 7th from bottom have only two unique values (i.e. one-hot encoding). These two don't quite seem to match the scatterplot, which makes me somewhat suspicious.
hidden_names = [str(i) for i in range(len(X_train.columns))]
summary_plot(shap_values, hidden_names, max_display=20, features=X_train.as_matrix())
summary_plot(shap_values, hidden_names, color="#cccccc", max_display=20, features=X_train.as_matrix())
summary_plot(shap_values, hidden_names, max_display=10, violin=False, features=X_train.as_matrix(), alpha=0.01)
summary_plot(shap_values, hidden_names, max_display=10, violin=False, features=X_train.as_matrix(), width=0., alpha=0.01)

Since files were refactored merging wouldn't make sense now. So I just copied the code (and fixed what seemed to be broken about it after merging into plots.py). I still think I might have broken something based on what I see in the notebook example for violins https://github.com/slundberg/shap/blob/master/notebooks/Scatter Density vs. Violin Plot Comparison.ipynb
Are you using a working version of this currently?
Thanks!

Does that mean my contributions won't be in recorded in git?
I was using a working version of this, but I think I'd developed it somewhat after the repo. If you'd like, I can submit a new PR with that code, which would hopefully fix the merge and attribution stuff too.

Great - will aim to do so this weekend.

Great!
On Thu, Apr 19, 2018 at 5:31 PM kodonnell<EMAIL_ADDRESS>wrote:

Great - will aim to do so this weekend.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/slundberg/shap/pull/17#issuecomment-382924148, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADkTxZ0xj2oZ55Y8O2iFDjb4ANvXLHcBks5tqSxxgaJpZM4Rivd4
.

@slundberg - I've pushed the tweaks. Unfortunately I don't think my original work will be recognized (as you'll show as the original committer and the net change from this PR will be minor). In future, it's probably a good idea to avoid doing the copy-and-paste trick for this reason, especially when you say that you were going to merge it. Anyway, a little discouraging, but let's leave it at that.
In these updates:

fixed bug with not supporting categorical data correctly - see change in 'sex' and 's5_binned' in plots below (I just binned the s5 variable to demonstrate this) - you can see it's "right" when comparing with dot plot
fixed bug in coloring (needed a -1 not a -0) - see change in 'sex' feature in plots below - you can see it's "right" when comparing with dot plot
added support for single color or a map - only for layered_violin
added layered_violin_max_num_bins as an option
updated the ipynb to document the options a bit more, and explain their differences. I used the diabetes dataset as it gave nicer plots and explained more features.
added the sidebar for the layered_violin
removed the warning as I think it's OK (at least to the same level as the others)

Before:

After:

Dot plot for reference:

Questions:

Can you explain the plot_type='violin' option ... I was confused by it, and I'm not sure it does anything better than the other options. For that reason, I'd propose calling the 'layered_violin' just the 'violin' plot, and call this one something else ('striped_violin'?).
Can I add an example of the layered_violin plot to the readme? I think it looks great, so it's good to put in the readme.
Happy to handle #59 in here too ...

As a general comment, the plotting code/API could probably do with a bit of tidying and documenting, and getting some easy wins (e.g. #62).

Thanks @slundberg - that's a clever solution, and thanks for bringing it up.

@kodonnell can you fix the merge conflict that came when I cleared out the old version? If I fix the merge I am not sure how the attribution will pan out and it would be a pain to undo.

can you fix the merge conflict

Done, I think.

Now merged! I'll look over the other suggestions next week.


⚠ Bofrost.lu has degraded performance

In 6849b3a, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 2366 ms

Resolved: Bofrost.lu performance has improved in 7b9dc65 after 6 minutes.


Basic types failed to be handled in object definition

After applying #115 or use mouse07410's repository having APER-capability, when processing ASN.1 file of 3GPP's S1AP (36.413) by asn1c, fatal error message occurred at RerouteNASRequest-IEs :
FATAL: Cannot find OCTET referenced by RerouteNASRequest-IEs at line 1545 in ./S1AP-PDU-Contents.asn
FATAL: Parsing ObjectSet RerouteNASRequest-IEs failed at 1545 in ./S1AP-PDU-Contents.asn

The content of RerouteNASRequest-IEs is :
RerouteNASRequest-IEs S1AP-PROTOCOL-IES ::= {
	{ ID id-eNB-UE-S1AP-ID			CRITICALITY reject	TYPE ENB-UE-S1AP-ID			PRESENCE mandatory}|
	{ ID id-MME-UE-S1AP-ID			CRITICALITY ignore	TYPE MME-UE-S1AP-ID			PRESENCE optional}|
	{ ID id-S1-Message			CRITICALITY reject	TYPE OCTET STRING			PRESENCE mandatory}|
	{ ID id-MME-Group-ID			CRITICALITY reject	TYPE MME-Group-ID			PRESENCE mandatory}|
	{ ID id-Additional-GUTI			CRITICALITY ignore	TYPE Additional-GUTI		PRESENCE optional},
	...
}

This issue comes from :

function _asn1f_parse_class_object_data does not handle field name having space within it, so only OCTECT of OCTET STRING is read.
function asn1f_lookup_symbol, called in _asn1f_assign_cell_value does not handle basic type like OCTECT STRING, INTEGER .... etc.

Part one, for the specific OCTET STRING case, can be fixed without much difficulty.
Part two, might need some more modification.

@brchiu care to submit a PR (with test-cases) for (1) to my fork? :-)
There aren't that many of multi-word types, I think: OCTET STRING, SEQUENCE OF, SET OF - anything else?

This issue is solved with better approach in pr 22 of mouse07410 repository.
A separate pull request to this main repository is to be prepared.

Issue fixed after merged pr #154 and checked ok on newest commit 6aae7c62c1883e007108e79d3123e7b17940da11 .
Issue closed.


add prometheus.

Please answer these questions before submitting pull request

Why submit this pull request?
[ ] Bug fix
[ ] New feature provided
[x] Improve performance

Coverage remained the same at 16.247% when pulling 06f19c3b0f206eab786da9cd479a588ba46fecf5 on prometheus_host_port into e3555479b4f88436dcf62c9be02db3e5a8c3c42f on master.


What is the timeline for getting to Azure.Core?

Library name and version
Microsoft.Azure.Batch 16.0
Query/Question
Microsoft.Rest.ClientRuntime
Microsoft.Rest.ClientRuntime.Azure
Our project shows both of these being deprecated, but Microsoft.Azure.Batch still depends on them.
Just wondering when this library would update so we can plan. I couldn't locate the info. Sorry if I missed it.
Microsoft.Rest.ClientRuntime (>= 2.3.20 && < 3.0.0)
Microsoft.Rest.ClientRuntime.Azure (>= 3.3.18 && < 4.0.0)
Environment
No response

Are there any updates on this issue? What is the policy regarding this kind of issues from a security perspective?


Can't install the package in Laravel 10

Describe the bug
Can't install the package in Laravel 10
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots

Desktop (please complete the following information):

OS: [e.g. iOS] MAC OS
Browser [e.g. chrome, safari]
Version [e.g. 22]

It was just released today 😂

Looking at the screenshot you provided it says that the error in the dependency package.

There is a PR for the dependency:
https://github.com/eklundkristoffer/seedster/pull/8


ctrl-R history search suddenly stopped working

Category

[ X ] fzf binary
[ ] fzf-tmux script
[ ] Key bindings
[ ] Completion
[ ] Vim
[ ] Neovim
[ X ] Etc.

OS

[ X ] Linux
[ ] Mac OS X
[ ] Windows
[ ] Windows Subsystem for Linux
[ ] Etc.

Shell

[ ] bash
[ X ] zsh
[ ] fish

Hi, my history search (ctrl-R) began malfunctioning in zsh this morning. The search works, and I can also browse the history with the arrows, but when I press ENTER, the command isn't pasted into the shell, and instead the selection just goes down one. You can also see that soon after I press ctrl-R, the '>' switches back to my shell prompt, which didn't happen before. I'm attaching a short screencast with the issue.
What I tried so far:

Upgraded fzf
Downgraded fzf
Unset PROMPT and RPROMPT
None of these worked.

% zsh --version
zsh 5.6 (x86_64-debian-linux-gnu)
% fzf --version
0.18.0 (ff95134)
Thanks!

Check if you use bash shell accidentally.

When I use bash it does work :)
On Mon, Nov 4, 2019, 21:54 Michał Nowotnik<EMAIL_ADDRESS>wrote:

Check if you use bash shell accidentally.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/junegunn/fzf/issues/1662?email_source=notifications&email_token=AAL6MP7DVXMQWGCGLEVSVDTQSB4XVA5CNFSM4IKIBBGKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDAQEUQ#issuecomment-549519954,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAL6MPYPPB5YDT7IUDWZ353QSB4XVANCNFSM4IKIBBGA
.

Try to run it with empty .zshrc. I haven't experienced any problems related to Ctrl-R fuzzy history search with fzf in zsh shell. Only in bash.
Run fzf-history-widget to check if widget is visible in your env.

Works. Thanks!
On Mon, Nov 4, 2019, 23:14 Michał Nowotnik<EMAIL_ADDRESS>wrote:

Try to run it with empty .zshrc. I haven't experienced any problems
related to Ctrl-R fuzzy history search with fzf in zsh shell. Only in bash
.
Run fzf-history-widget to check if widget is visible in your env.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/junegunn/fzf/issues/1662?email_source=notifications&email_token=AAL6MP47WZXU3CCFNM53P2LQSCGCRA5CNFSM4IKIBBGKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDAXOGI#issuecomment-549549849,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAL6MP26TPWMHPHTYG55NHLQSCGCRANCNFSM4IKIBBGA
.


is it not working anymore?

I am unable to even initiate the VintedScraper class because of the following error:
RuntimeError: Cannot fetch session cookie from https://www.vinted.co.uk, because of status code: 403 different from 200.
Is it not working anymore? I tried another library and it didn't work either with the same status code 403

I opened a pr that fixes the issue, you may want to use my fork for the moment

I have the same issue @Giglium

I opened a pr that fixes the issue, you may want to use my fork for the moment

Does it work with your cookie update pr? If you don't mind, may I ask if you were successful in using it with this small fix? Thanks!

It works for me with this fix, doesn't it work for you?

@Fedee0 I tried to run the test of your PR but they failed, we should also fix the mock, so I can merge it!
So Vinted still uses the _vinted_fr_session but the API now wants the access_token_web? Interesting...

@Fedee0 I tried to run the test of your PR but they failed, we should also fix the mock, so I can merge it!
So Vinted still uses the _vinted_fr_session but the API now wants the access_token_web? Interesting...

my fix indeed worked for me, and yes vinted still uses _vinted_fr_session although when sending requests it wants access_token_web

fixed the test and released the new package: https://github.com/Giglium/vinted_scraper/releases/tag/2.3.1
Thanks, @Fedee0 for your contribution!


Tone suddenly disappears, high frequencies don't work in other tabs and applications anymore

I played with the doppler tool a bit, when suddenly the once clearly audible tone disappeared and the motion tracking didn't work anymore. I didn't think much of it, but now, even after a restart, the higher frequencies on any other website and application don't work anymore. Does the tool actively block the frequency or is it likely, that my speakers are broken?

I work on an late 2013 iMac, OS X Yosemite 10.10.5, latest Chrome version. Audio and MIDI preferences are unobtrusive. Neither sudo killall coreaudiod nor resetting PRAM worked. The high frequencies are just cut away (or at least very quiet), no matter which program I use, no matter how often I restart my system. It all sounds like one muzzled the speaker with a blanket.

The tool does not actively block frequencies. I've not heard this happen before! Strage!


release: v2.4.0

see #1973

@manuelwedler you may want to cherry-pick d383a01df6e0cc5a3f8d037acfe1b35d33ce12f1 to pass the lint.


Issue/#2019 notification UI

Redo error notifications to be more helpful and include link to tech help forum.
Also change some common user errors to not be an error notification and instead a warning

New error notification

@Sheikah45 Also I request a "copy to clipboard" button for the stacktrace / message in the box.

New UI

Hoping to merge soon


Few questions

Hi, thanks for great work!

Are you planning to implement Xiaomi 2H support? Is it possible?
I think Air Purifers should automaticly switch themself to manual mode when you control them by intensity silder. When you’re in automode nothing is happening when you’re doing this.
Do you think you can implement Idle state for Air purifiers? Like in this plugin https://github.com/rgavril/homebridge-xiaomi-air-purifier3#readme ?
Do you think it is possible to make led/light accesorry to appear like this?

Jeszcze raz dzięki za super robotę!

Hello,
thanks for feedback, this is still relatively fresh and a work in progress so that helps me to improve the plugin.
To you questions:

What exact device model has the Xiaomi Air Purifier 2H? If you can tell me that then implementing it should be straight forward 😄
You are right and that makes a lot of sense, i will add it
What exactly does the "Idle" state mean? How do you know that the device is in idle state? I mean what the device does in idle state?
The led/light should already appear as a switch in the group, what is the difference to the picture above?

Dzięki 😃

2H is zhimi.airpurifier.mc2
Idle state is when The purifier fan is the lowest level when air quality is 1-7 I think.
The light appears like a big switch rather then round button and whole air purifier panel is messed up

Are you sure that the zhimi.airpurifier.mc2 is a miot device? I think that this device still uses the old miio protocol...
I will look into implementing the idle state.
I do not think that it is possible to place additional switches on a device, from where do you have the screen capture above?

I’m not sure that’s why I’m asking :)
The screenshot is from 2h plugin I’m using

On 17 Mar 2021, at 23:49, Marcin @.***> wrote:

Are you sure that the zhimi.airpurifier.mc2 is a miot device? I think that this device still uses the old miio protocol...
I will look into implementing the idle state.
I do not think that it is possible to place additional switches on a device, from where do you have the screen capture above?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Which plugin do you use? I can have a look and tell you if this device is a miot device.

https://www.npmjs.com/package/homebridge-mi-air-purifier This one

On 17 Mar 2021, at 23:58, Marcin @.***> wrote:

Which plugin do you use? I can have a look and tell you if this device is a miot device.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Unfortunately based on that link this device is a miio device which are not supported here.


InternalServerErrorException has incorrect documentation

see code
https://github.com/Sairyss/domain-driven-hexagon/blob/6fa27463b50f19d9b2050735f98c453933911871/src/libs/exceptions/exceptions.ts#L74

Thanks, fixed it


Revert "makefile fixes: setting DEPFLAGS_OS only if it hasn't been set by the user; adds O option to building Sundials"

Reverts stan-dev/math#1042

This reverts the PR that just went in. This should make Math build again.

If develop passes (I think it will now that I added more /tmp to the EC2 node) I think we should just close this.

Thanks. I agree.
(If it doesn’t and this does pass, we’ll revert and I’ll figure it out)
On Fri, Sep 28, 2018 at 10:11 AM seantalts<EMAIL_ADDRESS>wrote:

If develop passes (I think it will now that I added more /tmp to the EC2
node) I think we should just close this.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/stan-dev/math/pull/1044#issuecomment-425448176, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAZ_FxYx8y_uc-atCz6Asgy4hqyQn36Wks5ufi4BgaJpZM4W-bN-
.

Looks like develop passed. Closing this PR.


Fixed #28650 -- Add TruncWeek to truncate to monday of week

https://code.djangoproject.com/ticket/28650
(haven't gotten the tests up and running locally, but hopefully this works)

buildbot, test on oracle.


chore: price feeder imports umee/v2.0.0

Description

update price-feeder to umee/v2

Author Checklist
All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.
I have...

[ ] included the correct type prefix in the PR title
[ ] added appropriate labels to the PR
[ ] added ! to the type prefix if API or client breaking change
[ ] targeted the correct branch (see PR Targeting)
[ ] provided a link to the relevant issue or specification
[ ] added a changelog entry to CHANGELOG.md
[ ] included comments for documenting Go code
[ ] updated the relevant documentation or specification
[ ] reviewed "Files changed" and left comments if necessary
[ ] confirmed all CI checks have passed

Reviewers Checklist
All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.
I have...

[ ] confirmed the correct type prefix in the PR title
[ ] confirmed ! in the type prefix if API or client breaking change
[ ] confirmed all author checklist items have been addressed
[ ] reviewed state machine logic
[ ] reviewed API design and naming
[ ] reviewed documentation is accurate
[ ] reviewed tests and test coverage
[ ] manually tested (if applicable)

Codecov Report

Merging #766 (85d2155) into main (a176fb2) will decrease coverage by 0.42%.
The diff coverage is 0.00%.

:exclamation: Current head 85d2155 differs from pull request most recent head 1eb9d2f. Consider uploading reports for the commit 1eb9d2f to get more accurate results

@@            Coverage Diff             @@
##             main     #766      +/-   ##
==========================================
- Coverage   48.03%   47.60%   -0.43%
==========================================
  Files          64       64
  Lines        9329     9413      +84
==========================================
  Hits         4481     4481
- Misses       4581     4665      +84
  Partials      267      267

Impacted Files
Coverage Δ

ante/fee.go
93.61% <ø> (ø)

ante/spam_prevention.go
61.76% <ø> (ø)

x/ibctransfer/keeper/keeper.go
77.92% <ø> (ø)

x/leverage/abci.go
0.00% <ø> (ø)

x/leverage/client/cli/proposal.go
72.72% <ø> (ø)

x/leverage/client/cli/query.go
0.00% <ø> (ø)

x/leverage/client/cli/tx.go
0.00% <ø> (ø)

x/leverage/client/tests/suite.go
100.00% <ø> (ø)

x/leverage/client/tests/util.go
87.69% <ø> (ø)

x/leverage/genesis.go
50.00% <ø> (ø)

... and 34 more

Finished a local test of importing price feeder at this commit, and confirmed no errors appear with the non-v2 import path. Merging now.


[chore][ci] use docker/metadata-action for standardizing tabs/labels

Oh why did you close this? This seemed like a great PR

Just ran out of time before going on vacation, I'll likely take another look after I get back (if nobody else beats me to it)

Oh LOL alright then! I can work on this tomorrow if you arent able to (or just dont want to)


[backport] ROCm: Fix bugs and test suites to make ROCm/HIP happy - Part 1

Backport of #3823

[automatic post] Jenkins, test this please.

Jenkins CI test (for commit f7834573223531b5efb815fe3c117e3d1c9803f9, target branch v8) succeeded!

LGTM.

Thanks!! 🙂


Bug: Desktop app windows 11, notifications won't turn off, sound keep coming.

Even after switching off notifications, sounds and notifications keep coming users report

fixed in #958 and #965


Parameterise the job regenerators

Closes: #1994
Signed-off-by: Morgan Davies<EMAIL_ADDRESS>

Generators have already been updated to account for this change Job regenerators have already been updated to account for this change

Generators have been automatically kicked off https://ci.adoptopenjdk.net/job/build-scripts/job/utils/

JDK generator has finished with no issue https://ci.adoptopenjdk.net/job/build-scripts/job/utils/job/pipeline_jobs_generator_jdk/165/console


Testbuilds for Release-Testing (everyone can participate!)

Please press the "Watch" button below if you want to get notified of test-builds for upcoming releases.
I will post updated links from time to time here. Feel free to try them and report any issues you find. Just make sure you note in the report that you are using a testbuild (and which one).

IntelliJ IDEA 13 (and products based on it):
http://urswolfer.com/gerrit-intellij-plugin/testbuilds/20150308/gerrit-intellij-plugin-13.zip
Latest IntelliJ IDEA 14 (and products based on it):
http://urswolfer.com/gerrit-intellij-plugin/testbuilds/20150308/gerrit-intellij-plugin-14.zip
Latest IntelliJ IDEA 14.1 EAP (and products based on it):
http://urswolfer.com/gerrit-intellij-plugin/testbuilds/20150308/gerrit-intellij-plugin-14.1.zip
Build for IntelliJ IDEA 12.x build (and any products based on it):
http://urswolfer.com/gerrit-intellij-plugin/testbuilds/20150308/gerrit-intellij-plugin-12.zip
Changes:

dynamically show label columns according to server support ("Verify", "Core-Review", ...)
IntelliJ 14: add support for multi-repository-projects in push dialog (limitation: it does not work cleanly with setting "Push to Gerrit by default" activated - in this case all repos are selected on dialog show), #139
fix COMMIT_MSG review for first commit in a new Git repository, #122
fix reviews with comments for Gerrit >= 2.10, #151
many minor fixes and improvements

Good news!
From now on, you can get updates for test builds (release candidates) automatically in your IDE:

Copy the following URL: https://plugins.jetbrains.com/plugins/rc/7272
Use the copied URL as the Custom Plugin Repository
Reload the list of plugins
Search for the 'Gerrit' plugin in the plugin manager and install it

I have published a new release candidate which contains new features:

add support for setting custom review labels when supported by server (other than Code-Review and Verified)
add support for publish / delete draft changes
add column "status" to change list
reviewer and owner filter: allow filtering by username
disable actions when not permitted
fix status-filter option "All"
many minor fixes and improvements
Note: Updates for IntelliJ versions older than 14 are no longer provided


isFetchingNextPage is not always true in the next render cycle after fetchNextPage is called

Describe the bug
isFetchingNextPage should be set to true after fetchNextPage is called to prevent fetching the same page in the next render cycle.  As a result, sometimes we are fetching the same page twice or event three times.
Your minimal, reproducible example
CodeSandbox
Steps to reproduce

Open CodeSandbox
Open "Preview console"
Refresh the page
Scroll down (fast)
You should see 2 fetched 100 - 200. That means that page with items from 100 to 200 was fetched twice

Note: this issue is less reproducible if define useInfiniteQuery and useVirtualizer in the same component (example: VirtualListWithBranches)
Expected behavior
isFetchingNextPage should block fetchNextPage method in the next render cycle if the next page is currently fetching
if (!isFetchingNextPage) {
  fetchNextPage();
}

How often does this bug happen?
Sometimes
Screenshots or Videos
https://github.com/TanStack/query/assets/4423716/dffc4263-b272-4671-a6c7-19137bd7a044
Platform

OS: macOS
Browser: Chrome

Tanstack Query adapter
react-query
TanStack Query version
v5.17.9
TypeScript version
v5.2.2
Additional context
This issue is easier reproducible with a complex UI (more components, useInfiniteQuery props are passing through several components). In this case, you can easily fetch even three identical pages with slow scrolling.

I can't reproduce this in the sandbox and your video doesn't play for me ...

@TkDodo, try opening the video in a different browser. The example is reproducible, but it may require several attempts to see the issue.

interesting. so I added another log that just imperatively reads queryClient.isFetching() right before calling fetchNextPage, and we can really see that it's already at 1 (which means there is a fetch going on) while the fetchNextPage variable is still at false:
 invoking fetchNextPage(), isFetching is 0
  fetchBranches()
 isFetchingNextPage: ❌
 invoking fetchNextPage(), isFetching is 1
  fetchBranches()
4 isFetchingNextPage: ✅
2 fetched 200 - 300

This could be because we batch updates to observers together to avoid frequent renders, but in this case, where every minimal scroll seems to trigger a re-render, this could interfer.
We allow overriding how we batch things in the latest v5 version with notifyManager.setScheduler
I tried setting it to not batch at all with:
notifyManager.setScheduler((cb) => cb());

and I haven't been able to reproduce the issue since.
You can also try requestAnimationFrame or queueMicrotask to see if that yields better results for you over our default settings.

Thanks, @TkDodo. Batching with setTimeout(callback, 0) as a default setting was the main reason why isFetchingNextPage wasn't updated in the next render cycle. cb = cb() works in 100% of cases 🎉, and requestAnimationFrame and queueMicrotask also work almost perfectly (I only saw one parallel request among 200 pages).
Would it be possible to apply this setting to just one query, not the entire project? If yes, how can it be done? If no, is it safe to use the requestAnimationFrame or queueMicrotask option for the entire project?
P.S. Also, it would be good to mention it here: TanStack Infinite Scroll

Would it be possible to apply this setting to just one query

not possible, because it's about batching updates together, and that doesn't happen on a per-query level.

is it safe to use the requestAnimationFrame or queueMicrotask option for the entire project?

absolutely - any batching mechanism is okay. Some might result in more re-renders than others. We might switch default in a future major, but that's not decided.


Update error500.html.twig

Useless, already included with {{encore_entry_link_tags ('css / app')}}

@20uf thanks for this contribution! In normal circumstances you would be right and this would be a mistake ... however, this is a temporary ugly solution to fix a very tricky error related to Encore. It's explained here: https://github.com/symfony/demo/issues/910
This is not fixed yet, so I'm afraid we can't merge this yet and that's why I'm closing it.


Attempt to fix some URL bugs

Description
I made a quick attempt to fix #447, but it's not working yet. Output looks like:
{% set name = "pyospackage" %}
{% set version = "0.1.10" %}

package:
  name: {{ name|lower }}
  version: {{ version }}

source:
  url: https://pypi.io/packages/source/{{ name[0] }}/{{ name }}/pyospackage-{{ version }}.tar.gz
  sha256: b16fa74b9d3659609bde00fd6f1f7973ea15a47e41fdbd911f0666250feb95d9

build:
  noarch: python
  script: {{ PYTHON }} -m pip install . -vv --no-deps --no-build-isolation
  number: 0

requirements:
  host:
    - python >=3.9
    - hatchling
    - pip
  run:
    - python >=3.9

test:
  imports:
    - pyospackage
  commands:
    - pip check
  requires:
    - pip

about:
  home: https://pypi.org/project/pyospackage/
  summary: A package that adds numbers together
  doc_url: https://github.com/pyopensci/pyospackage
 # readme
  dev_url: https://github.com/pyopensci/pyospackage
 # readme
  license: MIT
 # readme
  license_file: LICENSE  # readme

extra:
  recipe-maintainers:
    - maresb

I'm not sure where those # readme comments are coming from, and I'm not sure why the doc_url gets clobbered during the rendering. (It's correct when about_section is defined.)

yeah, tricky, it comes from the pypi json payload
Grayskull also tries to fetch info from there, for some reason is putting a breakline after the end of the URL
this is part of the json response from pypi for that package:
"project_url": "https://pypi.org/project/pyospackage/",
    "project_urls": {
      "Documentation": "https://github.com/pyopensci/pyospackage#readme",
      "Homepage": "https://pypi.org/project/pyospackage/",
      "Issues": "https://github.com/pyopensci/pyospackage/issues",
      "Source": "https://github.com/pyopensci/pyospackage"
    },

can you see that #readme at the end of Documentation? it is coming from there. I hope that helps, if you need more help with that please let me know :)

Woah, thanks a lot @marcelotrevisani for the advice and quick response! Not sure when I'll be able to make a second pass at this. Looks like maybe something needs to be quoted/escaped?

Woah, thanks a lot @marcelotrevisani for the advice and quick response! Not sure when I'll be able to make a second pass at this. Looks like maybe something needs to be quoted/escaped?

Yeah, probably it does need some escaping. I can try to take a look later to give you a hint about that

@maresb keep in mind that a lot of the code in play here is part of souschef

@msarahan should we submit a bug report to souchef repository as well? i'm just watching this. it would be nice to fix this because otherwise users will get that linting error when they submit their recipe and need to figure out how to fix things.

I think that there are two stacked bugs here.
One bug has to do with the metadata getting passed around without being accidentally dropped. The other has to do with the # in the URL being interpreted as a comment.
I actually think that I fixed the bug with the metadata being dropped, but unmasking the bug due to the # in @lwasser's URL.
@lwasser, as a temporary workaround could you try removing the #... part? I'm thinking that it may be easier to proceed with one bug at a time.

Restarting the CI due to the connection error...

It has seemed like @maresb has this covered. Ben, if you need my help, please ping me. Otherwise, I'm leaving it in your capable hands and cheering you on.

ok great! thanks @msarahan  !!
@maresb can you remind me - what link are you referring to with the commented out element? i'm happy to test against this branch but i'm unclear about what you have identified that i can modify. many thanks! ❇

@lwasser it's your docs link: https://github.com/pyopensci/pyospackage#readme. I think the current PR would fix the recipe if your docs link were instead simply https://github.com/pyopensci/pyospackage.

@lwasser it's your docs link: https://github.com/pyopensci/pyospackage#readme. I think the current PR would fix the recipe if your docs link were instead simply https://github.com/pyopensci/pyospackage.

Thank you. i see that now. @maresb in testing this can i run grayskull against a sdist locally so i can skip releasing another version to pypi? i keep digging through the docs but don't see that option (may be user error)?

You can, but you'll need to adjust the source section.
From README.md:
You can also generate a recipe from a local sdist archive:
grayskull pypi ./pytest-5.3.5.tar.gz

Note that such a recipe isn't really portable as it will depend on the local path of the sdist file. It can be useful if you want to automatically generate a conda package.

Thank you everyone @msarahan i missed that in the readme. i was checking the docs. As a side note would folks here be open to a documentation pr from me that had the get started info in the readme? it was confusing to me as i saw docs and assumed that info would be there rather than in the readme! I suppose the alternative coming with less maintenance would be to just include the readme in the docs! just a thought :)
@maresb Ok i was not understanding the issue and now i see it. There is no reason to have a # in a url as you point out. when i remove that my metadata render perfectly (from your branch) with grayskull running in editable mode on my computer.
The metadata / recipe now look like this:
{% set name = "pyospackage" %}
{% set version = "0.1.10" %}

package:
  name: {{ name|lower }}
  version: {{ version }}

source:
  url: file:///Users/leahawasser/Documents/GitHub/pyos/pyospackage/dist/pyospackage-0.1.10.tar.gz
  sha256: 1db1ceb1d769a081c2945d235016ddb6dfe1b555c68014704c978b198b841ded

build:
  noarch: python
  script: {{ PYTHON }} -m pip install . -vv --no-deps --no-build-isolation
  number: 0

requirements:
  host:
    - python >=3.9
    - hatchling
    - pip
  run:
    - python >=3.9

test:
  imports:
    - pyospackage
  commands:
    - pip check
  requires:
    - pip

about:
  home: https://pypi.org/project/pyospackage/
  summary: A package that adds numbers together
  license: MIT
  license_file: LICENSE

extra:
  recipe-maintainers:
    - lwasser

i wonder if this pr could be merged as i am not seeing anything else broken here in the output. thank you all for walking me through what was going on. i think i misunderstood above what "breaking things" related to.

I'm glad that this is sorted out, but that is still a bug that I can fix it later, because it should be able to handle the # fine. It seems something related to the parsing and ruamel.yaml
I have been quite busy these days with my day work and personal life that I didn't have much time to look into it, but I hope to fix this soon :)
Thank you everybody for putting efforts into this.
@maresb , would you like to change the status of this PR to "ready for review"? or are you going to work more in this front?


Improve parsing of "sizes" property

The "sizes" property on an icon in the Web App Manifest is a space-delimited list of sizes. In addition to the standard 'nxn' format, "any" is a valid size.

Thanks! I'll do a release soon, I want to add the CSS variable support I mentioned in another bug first.


nltk_Data/punkt

I tried to download punkt using
Import nltk
nltk.download('punkt')
getting the common error :

import nltk
nltk.download('punkt')
[nltk_data] Error loading punkt: <urlopen error [WinError 10060] A
[nltk_data]     connection attempt failed because the connected party
[nltk_data]     did not properly respond after a period of time, or
[nltk_data]     established connection failed because connected host
[nltk_data]     has failed to respond>
False

pls help me out with the following error

ASAP

Try doing from terminal instead
python
>> import nltk
>> nltk.download('punkt')


Move TLS logging to verbosity level 5

Patch for https://github.com/NLnetLabs/nsd/issues/347

Thanks for the change patch! The other commits for #347 change a couple other log messages for TCP and TLS to verbosity level 5 as well.


NuGet: Strange Namespace and no *.lib generated

Hi
I have installed the NuGet package (v1.5.2) to an .NET 4.6.1 library Project. There are two issues:

The DllExportAttribute resides in a very strange Namespace: "D3F00FF1770DED978EC774BA389F2DC901F4.B00000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000.C00000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000.D00000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000.E00000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000.F00000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000"

When I build the Project (x64) no *.lib is generated which can be linked by my C++ lib

You have to unload the project, and open it again (or simply restart Visual Studio).
This is a known issue, and has been discussed previously. Also see #14.

@ArgusMagnus

ddNS feature that should update this sequence after installation.
You should define property $(DllExportGenExpLib = true) to activate this feature because it disabled by default.

Probably you have some errors when install our package, because you should see this:

Or you should define this manually like:
<DllExportNamespace>System.Runtime.InteropServices</DllExportNamespace>
<DllExportDDNSCecil>true</DllExportDDNSCecil>
<DllExportOrdinalsBase>1</DllExportOrdinalsBase>
<DllExportGenExpLib>true</DllExportGenExpLib>
<DllExportOurILAsm>false</DllExportOurILAsm>
...

Thus, for clarify please provide detailed log when you install this - for VS IDE:

Tools - Options - Project and Solutions - Build and Run - MSBuild project build verbosity

For msbuild.exe:

/v:diag

That Dialog never showed up.
msbuild_log.txt

That Dialog never showed up.

seems you have problem with installing, thus please also provide log when you install this package (see from Output).
Current package uses init.ps1 & install.ps1 scripts by nuget
It can be related for #36, but I haven't seen any similar problem for both new VS versions:

VisualStudio.15.Release/15.2.0+26430.4
VisualStudio.15.Preview/15.3.0-Preview+26507.0

that is, the install.ps1 has been obsolete already for VS 14.0 (at least, as I remember) and recommended \build\... targets instead (as for my other packages e.g.: \build\native\ or \build\net\).

issue is fixed by adding the above elements ... <Import Project="...

yes, this line should be automatically added after install.ps1 or \build\xxx\xxx.targets. This is required for DllExport anyway.

Cannot find lib.exe

The new VS 2017 contains major changes with its environment (+hMSBuild), so I know what's going on <_< thanks for notice.
Temporarily try to redefine this:
<installationPath>\<Version>\<Edition>\VC\Tools\MSVC\<Version>\bin\<HostTarget>\<Arch>
for example:
<DllExportLibToolPath>$(DevEnvDir)..\..\VC\Tools\MSVC\14.11.25301\bin\HostX86\x86</DllExportLibToolPath>

For all: I've plan to consider this project to the end of this month or early (as soon as possible for my time)

Sorry for responding so late, I only work once a week on the project I use DllExport.
Package Manager Output:
Successfully installed 'DllExport 1.5.2' to MyProject
Executing script file 'C:\Users\automation\.nuget\packages\dllexport\1.5.2\tools\init.ps1'...
The names of some imported commands from the module 'DllExportCmdLets' include unapproved verbs that might make them less discoverable. To find the commands with unapproved verbs, run the Import-Module command again with the Verbose parameter. For a list of approved verbs, type Get-Verb.
Executing nuget actions took 2.51 sec
Time Elapsed: 00:00:02.7435282
========== Finished ==========

@ArgusMagnus
yes, yes, only init.ps1 that related for #36
thanks for information
by the way, it still works for me (VisualStudio.15.Release/15.2.0+26430.6):
....
Executing script file 'D:\tmp\2017\ConsoleApp1\packages\DllExport.1.5.2\tools\init.ps1'...
The names of some imported commands from the module 'DllExportCmdLets' include unapproved verbs that might make them less discoverable. To find the commands with unapproved verbs, run the Import-Module command again with the Verbose parameter. For a list of approved verbs, type Get-Verb.
Executing script file 'D:\tmp\2017\ConsoleApp1\packages\DllExport.1.5.2\tools\install.ps1'...
set new namespace(Cecil: True): (ConsoleApp1) - (D:\tmp\2017\ConsoleApp1\packages\DllExport.1.5.2\lib\net20\DllExport.dll)
....

so, what your build version of VS15 for clarify ? (Help - About - first 3 lines)

Microsoft Visual Studio Enterprise 2017
Version 15.2 (26430.6) Release
VisualStudio.15.Release/15.2.0+26430.6

thanks,
anyway I've plan to change this obsolete variant soon :) +PackageReference
for today, please configure this manually (see above) if anyone else have this problem.

Ok, thanks

I closed this issue because of new Wizard for v1.6+ versions. You already can test it via our beta releases:

Latest manager: https://3F.github.io/DllExport/releases/latest/manager/
Please note again - you shouldn't use standard nuget clients anymore: https://www.youtube.com/watch?v=okPThdWDZMM

If you still have a some related problems, please create new issue for new planned Wizard.


Traffic overage costs

Please consider documenting cost of traffic overages for each provider.
Also worth noting differences in cost between inbound/outbound traffic (Linode inbound is free, for example).

Thanks! I'll look into that! I'll try to put all the information that I find about it.

(Hello there, I don't think this is worth creating a new issue so I'll post it here)

I have serious doubts about the OVH’s and Scaleway’s unlimited traffic, seems more marketing strategy than real to me (joe di castro).

Traffic is really unmetered, french hosting providers never billed for traffic (to my knowledge at least). You're limited by your bandwidth, but not charged for it / throttled after some consumption level.
Disclaimer: none, I don't work for an hosting provider, I'm just an happy customer

Thanks @PunKeel I'll change that because you are probably the third person to tell me the same (the others in HN), and even gave me a reasonable explanation why probably they do that. I quote that explanation by RX14 here:

Scaleway are owned by one of the largest ISPs in France. Bandwidth is typically brought symmetric, but home traffic is nearly entirely download. It's my assumption that they use their server business to offset their bandwidth costs for their underutilized outbound links.

@PunKeel I fixed that in 103aac18b0547c55ecf1a78c80fac6a57a24fba3
Thanks!


replace boxstream with secretstream

so far i see that sodium-secretstream is throwing an error pull failed, which i believe corresponds to this line in sodium-native. but not sure why.

okay, i think the tests pass now. :relaxed:

Really sweet, thanks!


Document other output line types of generate.py

Added mention of D-, T-, A- and E- output line types. Missing I-.
Before submitting

[ ] Was this discussed/approved via a Github issue? (no need for typos, doc improvements)
[x] Did you read the contributor guideline?
[x] Did you make sure to update the docs?
[ ] Did you write any new necessary tests?

What does this PR do?
Fixes #2499.
PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃 : 🙂

Merged in adbd89fd4be9e68100bf9a4ba9eed1e7fb2e4040


Document how to install?

Is this available on any package manager? It would be much easier to get started with

It doesn't really do much, and my priorities have shifted - purty might be a better idea.


Postgres Dollar-Quoted String Constants

The use of  Dollar-Quoted String Constants causes crash
For example:
COMMENT ON TABLE comment_type IS $$This is a comment for a table.$$;

Expected behavior
What did you expect to happen?
Actual behavior
The command  sql-formatter CommentType.json.sql --fix crashed due to Dollar quotes.
See https://www.postgresql.org/docs/current/sql-syntax-lexical.html#SQL-SYNTAX-CONSTANTS
Usage

How are you calling / using the library?  sql-formatter CommentType.json.sql --fix
What SQL language(s) does this apply to? Postgres
Which SQL Formatter version are you using? 12.1.3

Just specify the correct dialect with --language postgresql option.

Thanks, yes, --language postgresql made it work.


Update mergebot config for labels.

High Level Description

PR creators and Owners can ask mergebot add label to a PR.

Related Issues

DCOS-14379 - Delegate Issue label application via mergebot

Checklist for all PR's

[x] Read the DC/OS contributing guidelines
[x] Followed relevant code rules Rules for Packages and Systemd

@lingmann , @BenWhitehead , @spahl  - Please review and let's include this in the next train.
Demo here: https://github.com/mesosphere-mergebot/mergebot-test-dcos/pull/27 ( Notice Work In Progress was added by default and then I asked the bot to update the label).


[SPARK-7517] Rename 'unsafe' to 'managedmemory'

This patch renames the unsafe package to managedmemory and updates all relevant SparkConf configurations and class names to reflect this new naming convention.
I think that the new names are less scary to users and more informative to readers. For example, ManagedMemoryRow seems like a better clearer than UnsafeRow.  Similarly, ManagedMemoryShuffleManager seems more descriptive than UnsafeShuffleManager.
This patch should be merged into master and branch-1.4 so that we can rename the configurations before they appear in any public release.

/cc @rxin @marmbrus

Merged build triggered.

Merged build started.

Merged build triggered.

Merged build started.

Test build #32346 has started for   PR 6043 at commit 14bcfb2.

Test build #32346 has finished for   PR 6043 at commit 14bcfb2.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32346/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32345/
Test FAILed.

Merged build triggered.

Merged build started.

Test build #32347 has started for   PR 6043 at commit 56679fd.

(Test comment to test a feature on spark-prs).

Test build #32347 has finished for   PR 6043 at commit 56679fd.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32347/
Test FAILed.

Merged build triggered.

Merged build started.

Test build #32350 has started for   PR 6043 at commit acd36f6.

Not to spend too much time deciding the color of this bikeshed... but what do you think about just unmanaged instead?  With the current name there is a lot of managedmemory.memory which is kind of redundant.  Also, a managed language is usually one that has garbage collection and in this case that is what we are avoiding.

Test build #32350 has finished for   PR 6043 at commit acd36f6.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32350/
Test PASSed.

The thing that matters the most here is the config variable naming. Internal names we can change easily. Also we should keep in mind that in the future, likely everything will be "managed", so maybe there isn't much point to say managed vs unmanaged in the future.

What about having the configs be under managedMemory but having the internal package name be managed? That's my personal preference.

+1 to patricks suggestion

Sounds good; I'll update this patch in a little bit.

Per some offline discussion, we're going to leave this at the current name unsafe for now.  The package name doesn't really matter that much because it's not user-facing; we only have to worry about the names of user-facing configs, which we'll address in other patches.  Therefore, I'll close this PR for now.


Add Bloom Filter

Adds a 4-bit Bloom filter. Offers ~2% false positives with 8 bits of
size per entry.

Very cool! Where are we planning to use it?

Uses include prefilter of checking for admission for SSD cache, approx. count of distinct files, maybe hash join prefilter if that is actually useful.
From: Pedro Pedreira @.>
Sent: Monday, October 4, 2021 10:34 AM
To: facebookincubator/velox @.>
Cc: oerling @.>; Author @.>
Subject: Re: [facebookincubator/velox] Add Bloom Filter (#370)
Very cool! Where are we planning to use it?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/facebookincubator/velox/pull/370#issuecomment-933702653 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AKPPPT7LUBZ3WONLVEILSN3UFHQRDANCNFSM5FJT3ZAA .
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675  or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub .


dfn panels show up in "reader view"

Test https://streams.spec.whatwg.org with the bookmarklet from https://www.readability.com/bookmarklets and you will see the problem. Also visible in Firefox's reader mode (although getting Streams to go into reader mode is not easy right now for unknown reasons I'm still working through).
EDIT: you can get reader view to trigger if you go to https://streams.spec.whatwg.org/index.html. Firefox has a heuristic that disallows "homepages" from triggering reader view.

I can't figure out how to do anything about this. ;_;  There's extremely minimal documentation on the various reader modes, so I can't tell what, if any, methods exist to force something to hide.  (I did find a little bit of guidelines for the Readability bookmarklet, but they appear to be DAMNABLE LIES.)
I'll go ahead and close this, because afaict it's hopeless, but I'll gladly reopen if anyone can figure out how to do it.

If you can move the content around a bit, and use <aside>s, I think that works. A simple patch didn't work because of the <p> problem you mention. IIRC the original dfn.js did not insert things in the same location you do (they were just children of body I think), so maybe look at that?

Yeah, original dfn.js has a single element as a child of body, and fills it in on the fly and positions it when you click.  I suppose that's not too bad; I think I avoided it because I was originally planning on doing a no-JS implementation (with the checkbox hack, yay!) but that didn't end up working out.  I'll try it, at least.

Here's another example of where putting it as the sibling of the <dfn> works out poorly: https://drafts.csswg.org/cssom/#parse-a-list-of-css-page-selectors

Yup, definitely gonna change that.

Okay, all the dfn-panels are now ASIDEs at the end of the document. I think they'll be omitted in reader views now.

Confirmed works! Rebuilt streams, and https://streams.spec.whatwg.org/index.html Firefox reader view now looks quite nice.

:+1:
On May 17, 2016 18:11, "Domenic Denicola"<EMAIL_ADDRESS>wrote:

Confirmed works! Rebuilt streams, and
https://streams.spec.whatwg.org/index.html Firefox reader view now looks
quite nice.
—
You are receiving this because you modified the open/close state.
Reply to this email directly or view it on GitHub
https://github.com/tabatkins/bikeshed/issues/656#issuecomment-219899118


Validate backwards-facing step

Compare to results from Boujo & Gallaire (2015)

[x] Secondary upper recirculation zone appears near Re=272 (x_u ~ 8.2)
[x] Separation and reattachment points at Re=600: (x_lr, x_us, x_ur) = (11.82, 9.34, 20.59)
[ ] 3D stability analysis with periodic spanwise direction: (Re_c, beta_c) = (715, 0.88)

Currently the separation and reattachment points are significantly off - for instance, x_lr = 4.12 and the upper recirculation zone appears above Re=500.  Need to check definitions of Reynolds number, inlet velocity, etc.

Confirmed the separation/reattachment points are fine if using the length scale convention in the paper (the inlet height, not outlet).  Stability analysis will take a bit more work, but I think this is okay for the time being.


Update README w/ note about GH plans supporting attestations

Updates the readme with a note outlining which GH plans support artifact attestations.

Needshelp@bdehamer

I was a cloud plan just they locked me out of "master333-ui-web"

That's my company account

Updates the readme with a note outlining which GH plans support artifact attestations.

-git request -pull > > Updates the readme with a note outlining which GH plans support artifact attestations.

`> -git request -pull > > Updates the readme with a note outlining which GH plans support artifact attestations.

`


How the ordinals-api should configure link hooks

I use the relevant information below to start the chain hook only to see the structure, but not to start port monitoring.
If I want to use ordinals-api, chainhook doesn't have listening port, I can't configure ordinals-api env.ts correctly
chainhook service start --predicate-path=./ordinals.json --config-path=./config.toml
[storage]
driver = "memory"
redis_uri = ""
cache_path = "./data2"
[chainhooks]
max_stacks_registrations = 500
max_bitcoin_registrations = 500
[network]
mode = "mainnet"
bitcoind_rpc_url = ""
bitcoind_rpc_username = ""
bitcoind_rpc_password = ""
stacks_node_rpc_url = "https://stacks-node-api.mainnet.stacks.co/"
{
"chain": "bitcoin",
"uuid": "818725a4-555d-46db-a60c-213a506b37ba",
"name": "inscription_feed",
"version": 1,
"networks": {
"mainnet": {
"start_block": 767430,
"end_block": 793083,
"if_this": {
"scope": "ordinals_protocol",
"operation": "inscription_feed"
},
"then_that": {
"file_append": {
"path": "ordinals.txt"
}
}
}
}
}

@lgalabru, I guess this is now handled using Ordhook in the documentation here. https://github.com/hirosystems/ordhook#run-ordhook-as-a-service-for-streaming-blocks. Please correct me if I am wrong.

Closing this issue @pupudaye feel free to re-open if you think the documentation can be improved.


Async loading of scripts with a global callback

Resolves https://github.com/Jasig/cas/issues/734
This pull does the following:

Loads javascript references asynchronously
Attempts to call a global callback fucntion jqueryReady when defined, to signal the CAS UI that it is now safe to execute custom JS code based on jquery.

I'm not a Javascript specialist, but I think you should take a look at: http://headjs.com/. It seems to do the job.

Cool, thank you. I'll review and see if there is applicability.

Thank you for the suggestion. head.js worked beautifully with less code :) updated pull

One more cool framework! :+1:


[4pt] Display rockets - Lists render

Use useSelector() Redux Hook to select the state slices and render lists of rockets in corresponding routes. i.e.:

// get rockets data from the store
const rockets = useSelector(state => state.rockets);

You can style the whole application "by hand" or you could use React Bootstrap, a UI library that could speed up the process. This is a popular library and working with its components would be good practice.
Render a list of rockets (as per design). For the image of a rocket use the first image in the array of flickr_images.

Link to the PR: https://github.com/SaveryIV/Space-X-Project/pull/30

The rendering is working well. Continue.


Generate SBOM for go dependency

Summary

Resolves #341
Use Cases

Checklist

[ ] I have viewed, signed, and submitted the Contributor License Agreement.
[ ] I have linked issue(s) that this PR should close using keywords or the Github UI (See docs)
[ ] I have added an integration test, if necessary.
[ ] I have reviewed the styleguide for guidance on my code quality.
[ ] I'm happy with the commit history on this PR (I have rebased/squashed as needed).

Labeling as semver:minor because changes are backward compatible even though the buildpack API version has been upgraded.


Remove Rust toolchain question

I see no value in having a question for the desired Rust toolchain when generating the project, we could do like in esp-template and assume the toolchain based on the selected target (nigthly for esp32c3, esp otherwise)

That's a good point, seems like a good idea!


Synergy issue

Hi
I've been using https://symless.com/synergy on dwm, currently i've moved to hyprland, but it seems it does not working.
Is any one had the same issue?

@piotrzarycki this is a synergy issue, not a Hyprland one
please see https://github.com/symless/synergy-core/issues/4090

ok, thanks for info


for sepecific device , can i get keypress event , and block that device keystrokes typing on my forms just winform App only.  i dont care if it writes to notepad or not.

for sepecific device ,
can i get keypress event , and block that device typing  keystrokes on my forms . ?
i dont care if it writes to notepad/other external apps or not.

https://github.com/stevemesser/rawinput
You can use this: https://www.codeproject.com/Articles/17123/Using-Raw-Input-from-C-to-handle-multiple-keyboard
(It seems work on win7, but not in win10 or win11)

And you can use this tool to block the keyboard keys. (Work with RawInputProcessor together.)

And you can use this tool to block the keyboard keys. (Work with RawInputProcessor together.) https://github.com/natevoci/KeyboardRedirector https://nate.deepcreek.org.au/dev/keyboardredirector.rails
http://www.oblita.com/interception.html

You can see this: https://github.com/danch15/RawInputProcessor/
修复Windows 10及以上系统无法拦截按键的问题。增加WinForm的单独项目实现
Fix a problem where Windows 10 and above systems cannot capture the keyboard. Add a separate project implementation for WinForm.


Adding README.md for each module

Adding a short brief REAEME.md in src/slam/, src/object-recognition and src/person-tracking/ dir.
This is required by npmjs.com.

Drafting README for each npm module


bug on insert row between existing row

My website which use jexcel suddenly can't insert new row between existing rows. Below is a sample code to demonstrate the bug. When the "INSERT ROW BEFORE SECOND ROW" button is clicked, the second row's number is changed but no new row is showed before it.
<!DOCTYPE html>

    <meta charset="UTF-8">
    <title></title>

<link rel="stylesheet" href="https://bossanova.uk/jsuites/v2/jsuites.css" type="text/css" />
<link rel="stylesheet" href="https://bossanova.uk/jexcel/v3/jexcel.css" type="text/css" />

<button id="add_row">INSERT ROW BEFORE SECOND ROW</button>

Screenshot showing before and after executing the insertion :

I hope this could be resolved soon. In the meantime I plan to use previous version before this issue could be resolved. This library is such a lifesaver and I could only express many gratitude for the maintainer. Thank you very much!

It has been fixed


Twitter API Stopper working, status shows 410 gone

Twitter API Stopper working, status shows 410 gone
Its reported from last 5-10 days from one of our client

Seems it stopped working Nov 20th. [Here] (https://blog.twitter.com/2015/hard-decisions-for-a-sustainable-platform) is an official response regarding why the API was shut down.

The twitter API has changed and now even with newer APIs you can't get the tweet count, and Twitter most likely won't change that. Right now you can use http://opensharecount.com/ that is configured in the latests versions. You need to register on opensharecount though.

Hi, That stuff is just gorgeous, you  have to read about it  more at http://choice.pinkypc.us/e6yprho
tadakute


Roll Engine from 189cdc7df7ae to 74aa54964f39 (2 revisions)

https://github.com/flutter/engine/compare/189cdc7df7ae...74aa54964f39
2022-04-27<EMAIL_ADDRESS>Roll Skia from 850de30a19f3 to be726c106958 (1 revision) (flutter/engine#32951)
2022-04-27<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from V9WZPjuRB... to 0BdXJKVKf... (flutter/engine#32950)
Also rolling transitive DEPS:
fuchsia/sdk/core/mac-amd64 from V9WZPjuRB6Mu to 0BdXJKVKfjhQ
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Google login not working

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem. Check browser console for errors.

Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]
or copy-paste a link from https://www.whatsmybrowser.org/

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Thank you @glittle.
I hope I'll find time to fix this in next few days.

Fixed.


[🐛 Bug]: Sentry bloats up bundle size when building with npx @cloudflare/next-on-pages

next-on-pages environment related information
System:
Platform: darwin
CPU: arm64 Apple M3 Pro
Memory: 18 GB
Shell: /bin/zsh
Package Manager Used: npm (10.5.0)
Relevant Packages:
@cloudflare/next-on-pages: 1.13.2
next: 14.2.5
@sentry/nextjs: 8.26.0
Description
When installing Sentry in any Next.js project, it massively bloats the bundle size, going way beyond the limit of 25MB for normal projects. For example, in my project, it increases the bundle size almost by a factor of 10, reaching 70MB. This, of course, is unreasonable, and I suspect that it has something to do with the deduplication that Cloudflare performs during the build, but I'm not really sure.
I think this issue is somehow related to this GitHub issue, which is caused by the vercel/otel package.
Overall, it prevents the use of Cloudflare Pages for Next.js in production at some scale, as there seems to be no working option to use an observability tool. We are overall really satisfied with hosting on Cloudflare, but this might force us to switch soon.
Reproduction
Steps to reproduce:

Clone a next.js project (I will use https://github.com/vercel/ai-chatbot as example)
Create a build using npx @cloudflare/next-on-pages
Check the bundle size in the folder .vercel/output/static/_worker.js (for the above example I arrive at 4.1 MB)
Install Sentry and jump through the setup (npx @sentry/wizard@latest -i nextjs)
Build and check the bundle size again (for the above example I arrive at 15.8 MB)

Pages Deployment Method
None
Pages Deployment ID
No response
Additional Information
No response
Would you like to help?

[X] Would you like to help fixing this bug?

Unfortunately, this issue is not new. 32 days ago, a thread in Sentry's Discord channel was created about this issue. We hope this issue gains more attraction since we also want to integrate Sentry into our project, but we can't.

@jantiegges Thank you for the issue 🙂
(Sorry for the late reply 🙇 )
I've just tried your reproduction steps and it doesn't seem like I am getting the > 15MB worker 😕
I can actually normally deploy the worker: https://0deed5a4.next-on-p-sentry.pages.dev/
Could you try again? Any chance you could share a minimal repro (a hello world app ideally) that presents this error?
(@cjxe as well, I'd really appreciate a minimal reproduction if we could get one 🙏)

@dario-piotrowicz Thanks for picking this up, still highly interested in getting this fixed.
I also created a vanilla next.js app, which did not seem to show the issue at first. However, when I added more dummy routes the bundle size of the version with sentry started to bloat up. So the issue seems to be somehow related to the number of routes.
Here is a repository which contains 6 versions (1, 10, 50 routes w/ and w/o sentry) and these are the resulting build sizes:

# Routes
Build Size (w/o Sentry)
Build Size (w/ Sentry)

1
821 KB
929 KB

10
1.3 MB
7.1 MB

50
3.1 MB
31.5 MB

Hope that helps in pinpointing the issue. Lmk if you need any further help!

@jantiegges do you only see bloated build when using next-on-pages or do you also see when you just run next build?
Do you see it if you try using @opennextjs/cloudflare?

@irvinebroque when running just next build I get the following numbers (size of .next/static):

# Routes
Build Size (w/o Sentry)
Build Size (w/ Sentry)

1
836 KB
5.4 MB

10
827 KB
5.4 MB

50
856 KB
5.4 MB

seems a bit weird but I double checked. Was just quickly trying out @opennextjs/cloudflare but got a build error and don't have the time rn to dive in. Might try it later today, but so far we cannot switch to it anyway, because I think the manual cache revalidation is not implemented yet. Interesting project tho

@dario-piotrowicz Thanks for picking this up, still highly interested in getting this fixed.
I also created a vanilla next.js app, which did not seem to show the issue at first. However, when I added more dummy routes the bundle size of the version with sentry started to bloat up. So the issue seems to be somehow related to the number of routes.
Here is a repository which contains 6 versions (1, 10, 50 routes w/ and w/o sentry) and these are the resulting build sizes:
Routes	Build Size (w/o Sentry)	Build Size (w/ Sentry)
1	821 KB	929 KB
10	1.3 MB	7.1 MB
50	3.1 MB	31.5 MB
Hope that helps in pinpointing the issue. Lmk if you need any further help!

Thanks for the reproduction. Was super helpful. It looks like Sentry changes the AST node we need to find for deduping webpack chunks.
Can you try out the prerelease in https://github.com/cloudflare/next-on-pages/pull/886#issuecomment-2385697903 and let me know how you get on @jantiegges?

@james-elicx Amazing, it fixed it, glad the reproduction helped. I also tested it in our large production application, and it works well there too. That said, I'm starting to get a bit worried since we just hit the 20 MB bundle size, which brings us close to cloudflares 25 MB Limit for pages. I feel like this limit is pretty low for production-level Next.js apps, but anyway, I'm really happy this issue was resolved. Thanks a lot!


Port and host written to console on server starting (with cli)

Closed here https://github.com/divshot/superstatic/commit/ddc066dcd8a044fa5e801858be004f9f0b64f008


Regular expressions material needs filling in

Request type

Correction or update

Details
Sorry it's my first time here so I hope I've followed correct procedure.
I have relied heavily on the content of this section of the documentation for a couple of years, as it gave one of the clearest and most succinct explanations of what is a complex topic for many.
One of it's strengths was that everything was on a single page, so when trying to interpret a complex regular expression it was easy to scroll up and down to get the meaning of different components.
I appreciate greatly what a fantastic resource this is and the tremendous effort all contributors make.  I would be really really grateful if you could revert to the single page for this subject. Although it is quite long, that is the most helpful format for people who need to keep referencing  this topic who are maybe less experienced, like me.

Hi @duncan-crane,
Thanks for your thoughts here.  One of my counterparts at Google started doing this splitting work, on the understanding that the page was getting really long, and each section needs even more information added before it can be counted as complete, so it would just get longer.
I will give him a ping to see when he might complete the work. I think maybe more signposts as to what the pages contain would make things easier to navigate, but the consensus was that generally the page was getting too unwieldy and needed splitting.

@chrisdavidmills / @Elchi3 is there any update about this work (last significant edits are from Q1 2019)? There are still empty sections/headings left.
I agree that the page was getting quite long and needed splitting. However, I prefer 1 long page rather than several "drafts" lasting for months (esp. for en-US / fr parity maintenance).
Depending on the plan, I'm willing to lend a hand and help fixing this.

@SphinxKnight yeah, looks like this isn't getting done ;-(
If you had some time to spend on this, I would be really grateful. I can probably chip in too. Let's make some plans!

Referencing resource: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Guide/Regular_Expressions

For my first task on this ticket, I've gone over the entire main page, and given it a good tidy up:
https://wiki.developer.mozilla.org/en-US/docs/Web/JavaScript/Guide/Regular_Expressions

Remaining tasks:

https://wiki.developer.mozilla.org/en-US/docs/Web/JavaScript/Guide/Regular_Expressions/Assertions: Mainly just needs the Compatibility data, Specifications, and See also sections filled in, but we could also do with updating the table entries to be the same as what I did in the original article.
https://wiki.developer.mozilla.org/en-US/docs/Web/JavaScript/Guide/Regular_Expressions/Boundaries: same as the above really
https://wiki.developer.mozilla.org/en-US/docs/Web/JavaScript/Guide/Regular_Expressions/Character_Classes: same, but also needs examples filling in
https://wiki.developer.mozilla.org/en-US/docs/Web/JavaScript/Guide/Regular_Expressions/Groups_and_Ranges: same, plus also needs examples.
https://wiki.developer.mozilla.org/en-US/docs/Web/JavaScript/Guide/Regular_Expressions/Quantifiers: same, also needs examples
https://wiki.developer.mozilla.org/en-US/docs/Web/JavaScript/Guide/Regular_Expressions/Unicode_Property_Escapes: this is in a somewhat different format, and we need to think about what to do about it.

Update: I have gone through all 6 sub-pages, and added the following sections:

Specifications
Browser compatibility
See also

Still to be done:

Update the javascript regexp browser-compat-data (javascript.builtins.RegExp) to include the structures I've indicated on the different pages.
Decide whether Assertions and Boundaries should really just be one page, given that the spec basically groups them together as one thing.
Add Examples to "Character classes", "Groups and Ranges", "Quantifiers", and "Unicode property escapes"
Get all the other sections of "Unicode property escapes" filled in, as well as deciding if the page structure is suitable.

Also:

Remove the draft macro when the pages are done.
Check in with @Elchi3 to see if all this sounds ok, when he is back. For example, we might want to move the subpages below the regexp constructor page instead, as it is a bit strange having these reference pages hung off of a JS guide page.

@SphinxKnight are you still interested in contributing to this one?

@chrisdavidmills Yes I am :s I'll add examples to the pages and will review them (tonight / tomorrow)

@SphinxKnight super ;-)
So if you work on examples, I'll get to work on updating the Regexp browser-compat-data.

Browser compat data updated: https://github.com/mdn/browser-compat-data/pull/5350

* Decide whether Assertions and Boundaries should really just be one page, given that the spec basically groups them together as one thing.

I'm enclined to merge them as well (writing examples using both of them feels natural but clunky if this is documented in two distinct pages)

I've added a few examples across the page and I concur with the Unicode property escape issue. I think providing a few examples and the concepts should be enough then linking to the spec(s) (tc39 + Unicode) for a thorougher list
(unless we can use a <details> element on the page and dump a loooong table in it :s)

I've added a few examples across most of the pages and I concur with the Unicode property escape issue. I think providing a few examples and the concepts should be enough then linking to the spec(s) (tc39 + Unicode) for a thorougher list

(unless we can use a <details> element on the page and dump a loooong table in it :s)

Thanks for the awesome work!
Yeah, I think deleting the long list and linking to the spec sounds like a good idea. Are you happy to handle that, or would you rather I do it? Happy either way.

I'm inclined to merge them as well (writing examples using both of them feels natural but clunky if this is documented in two distinct pages)

Cool, I'm gonna merge 'em!

Yeah, I think deleting the long list and linking to the spec sounds like a good idea. Are you happy to handle that, or would you rather I do it?

I'll do it while adding examples :) (still no emoji on kuma for the examples tho :p)

I'll do it while adding examples :) (still no emoji on kuma for the examples tho :p)

Yay, thanks!
For emoji examples, I think we could get around the restriction by putting an example on https://github.com/mdn/js-examples/ for now, and embedding it using the EmbedGHLiveSample macro?

Cool, I'm gonna merge 'em!

This is now done. The assertions page now includes all the information that was on the boundaries page, and the latter now redirects to the former.

back from holiday, I'll be tackling those before the end of the week :)

@SphinxKnight superb, thank you! And a happy new year to you too; hope you had fun.

Happy new year too @chrisdavidmills \m/ Looking forward to 2020 :)
Fun and rest, aka good holidays :)

After rewriting https://wiki.developer.mozilla.org/en-US/docs/Web/JavaScript/Guide/Regular_Expressions/Unicode_Property_Escapes, a quick attempt at guessing what's left:

[ ] Insert interactive examples in the pages once they will have been merged
[ ] Revamp https://wiki.developer.mozilla.org/en-US/docs/Web/JavaScript/Guide/Regular_Expressions to reduce the amount of duplicated content
[ ] Check String pages to fix existing links or insert new ones
[ ] Check RegExp pages to fix existing links or insert new ones

Great progress here, thanks @SphinxKnight !
I've removed the draft banner from all the pages, except https://wiki.developer.mozilla.org/en-US/docs/Web/JavaScript/Guide/Regular_Expressions/Quantifiers. I think they are all looking good and functional for now, but I wondered if the Quantifiers page still felt a bit light on examples?

@chrisdavidmills I went across those pages and ones about RegExp and String and added a few links here and there for cross-references. I also checked the compat data displaying thread. Let me know if some more work is needed before closing this :)
@duncan-crane sorry for the flood of messages you were (maybe) getting about this lately. As this is (I think) coming to a (good?) end, could you tell us if it is better now?


Info for ERROR 29381

Hi,
I try to sync 2 jira projects with jira-sync but I have an error that I can not understand.

Is a very generic error!
Can you help me understand what I'm wrong?

Hi Stefano,
Could you provide a full stacktrace and re-run application with --debug?

Thanks!
Your log helped me a lot. You can see there a JiraResourceNotFoundException. The stacktrace indicates that there is a problem with the source filter settings:
de.cronn.jira.sync.projects[XXXX].sourceFilterId=<filterId>

Please double check that <filterId> points to the existing filter in your source Jira.
BTW. We need to improve our exceptions to be more descriptive ;)

My filterId is "allopenissues"

Hi @ldziedziul , I have created a custom filter.
Now the error is different (I think):

I see. We've used custom filters so far which have always numeric id, allopenissues is a special case. A custom filter should resolve it.
Please send the log file with the full stacktrace again. This time without --debug

I think the problem is here:

but I don't have this field (I think) in the destination screen.
sorry but I don't know how to setting parameters in application.properties.
Can you explain the parameters of application.properties file ?

I'm guessing without a stacktrace but currently we allways fetch the following fields using filters: summary,status,issuetype,description,priority,project,resolution,labels,versions,fixVersions,assignee,updated,comment
It looks like that for some reason you don't have versions and fixVersions fields which, we thought, are built-in (https://confluence.atlassian.com/adminjiraserver071/issue-fields-and-statuses-802592413.html).
Could you try to add a "1.0" version on both side to see if it helps?
You don't have to add the following lines to application.properties:
de.cronn.jira.sync.projects[XXX].versionMapping....

and
de.cronn.jira.sync.projects[XXX].versionsToIgnore=Undefined

Version mapping is optional.
Background:
These lines are used if you have different versions in both jira servers, e.g.
de.cronn.jira.sync.projects[XXX].versionMapping[10.0]=10

means that you have version 10.0 in the source server and it has the same meaning as version 10  on the target server. If you don't configure versionMapping it will be omitted.

I have added Version 1.0 on both side but the result is the same.
log2.txt
application.properties.txt
Execute with this command line:
./build/libs/jira-sync-1.0.jar --spring.config.location=file:/Users/teopost/dev/jira-sync/config/application.properties
Some other info: The destination project is "Service Desk" type

Hi. I've pushed the fix, please try now.

Thank you. Now work!


Task103/update dbnsfp readme

Description
Same as #450
dbNSFP release a new version and we are updating to the new version for release/102. This PR updates the README file location for the new dbNSFP file.
Use case
Users check the README file to see which data columns they can collect.
Benefits
NA
Possible Drawbacks
NA
Testing
I tested that the links work on a farm node.
Changelog
Not needed

Coverage decreased (-0.003%) to 44.456% when pulling 2993fcd54d03a32f05248e744d3301ea2a7808ee on at7:task103/update_dbnsfp_readme into 97c49e637e52abc2d3664ff4868aadd4260b9d5d on Ensembl:master.


Fix bugs in integ test deprecation logger scraping

This commit fixes a number of bugs in the infrastructure used to detect deprecation log messages in build outputs.

Rich consoles messed with the detection since we can get weird console control characters and progress bars before, after, and in between deprecation log messages. In these cases, we just disable deprecation log checks
When the integ test specifies a gradleVesrionOverride, we did not properly normalize documented deprecation URLs. They are now properly normalized to take into account the overridden gradle version.
When info or debug logs are enabled, deprecation messages are prfixed with an SLF4J log prefix. We strip this when checking for deprecation messages in a line
When a daemon crashes, we print the last ~20 lines of the daemon log. This can include a deprecation, causing the infrastructure to think we emitted the deprecation once and not twice. We strip any daemon log outputs dumps from the deprecation message scraping.

It so happens that none of these cases came up when checking for deprecations. However, if we do come across a build that encounters one of these cases and also emits deprecations, we would have problems.
Reviewing cheatsheet
Before merging the PR, comments starting with

❌ ❓must be fixed
🤔 💅 should be fixed
💭 may be fixed
🎉 celebrate happy things

@bot-gradle test this without pts

I've triggered the following builds with parameters: -DenablePredictiveTestSelection=false for you. Click here to see all build failures.

PullRequestFeedback build


See https://github.com/ZeusWPI/Gandalf/


[115] Issue #115 - August 9, 2018

To contribute to this issue, simply leave a comment here. Please also review our contributing guidelines.
The current draft for this issue in _drafts/. If you want to contribute directly, feel free to open a pull request.

@BasThomas should that go in #402 instead?

No; it is scheduled for review August 6 through 12 :)

Ah 👍

https://twitter.com/aciidb0mb3r/status/1024035974722945024

SwiftPM can now use llbuild as a library, thanks to @dhartbit! This will enable great new features and enhancements in the future.

https://github.com/apple/swift-package-manager/pull/1537


[Event Request] codeunit 453 "Job Queue - Enqueue" - OnJobQueueEntryOnBeforeJobQueueEntryModify

Hello, would it be possible to add OnJobQueueEntryOnBeforeJobQueueEntryModify event to codeunit 453 "Job Queue - Enqueue", like this:
    local procedure EnqueueJobQueueEntry(var JobQueueEntry: Record "Job Queue Entry")
...
              if SavedStatus = JobQueueEntry.Status::"On Hold with Inactivity Timeout" then
                 JobQueueEntry.Status := SavedStatus
             else
                 JobQueueEntry.Status := JobQueueEntry.Status::Ready;
>             OnJobQueueEntryOnBeforeJobQueueEntryModify(JobQueueEntry);//New Event
             JobQueueEntry.Modify();
         end;

         OnAfterEnqueueJobQueueEntry(JobQueueEntry);

     [IntegrationEvent(false, false)]
     local procedure OnJobQueueEntryOnBeforeJobQueueEntryModify(var JobQueueEntry: Record "Job Queue Entry")
     begin
     end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 10053.


A more "standard" build

I saw you produced a binary for Windows with some variation of the Mingw theme, but I was curious how the Visual Studio toolchain would fair. It looks like there are bits of syntax and constants that are not part of the C++ spec, so I inserted into geometry.h
`#ifndef M_PI
#define M_PI 3.14159265358979323846
#endif
define M_1_PIf32	 (0.318309886183790671537767526745028724) /* 1/pi */`
That went OK for a while but I borked hard with:
`
C:\Repos\SoftwareRenderer\include\shader.h(286): warning C4305: 'initializing': truncation from 'double' to 'float'
C:\Repos\SoftwareRenderer\include\shader.h(304): warning C4244: 'initializing': conversion from 'double' to 'float', possible loss of data
C:\Repos\SoftwareRenderer\include\shader.h(314): warning C4244: '=': conversion from 'double' to 'float', possible loss of data
C:\Repos\SoftwareRenderer\include\shader.h(381): error C2131: expression did not evaluate to a constant
C:\Repos\SoftwareRenderer\include\shader.h(378): note: failure was caused by a read of a variable outside its lifetime
C:\Repos\SoftwareRenderer\include\shader.h(378): note: see usage of 'this'
C:\Repos\SoftwareRenderer\include\shader.h(382): error C2131: expression did not evaluate to a constant
C:\Repos\SoftwareRenderer\include\shader.h(378): note: failure was caused by a read of a variable outside its lifetime
C:\Repos\SoftwareRenderer\include\shader.h(378): note: see usage of 'this'
C:\Repos\SoftwareRenderer\include\shader.h(383): error C2131: expression did not evaluate to a constant
C:\Repos\SoftwareRenderer\include\shader.h(378): note: failure was caused by a read of a variable outside its lifetime
C:\Repos\SoftwareRenderer\include\shader.h(378): note: see usage of 'this'
C:\Repos\SoftwareRenderer\include\shader.h(386): error C2131: expression did not evaluate to a constant
C:\Repos\SoftwareRenderer\include\shader.h(378): note: failure was caused by a read of a variable outside its lifetime
C:\Repos\SoftwareRenderer\include\shader.h(378): note: see usage of 'this'
C:\Repos\SoftwareRenderer\include\shader.h(387): error C2131: expression did not evaluate to a constant
C:\Repos\SoftwareRenderer\include\shader.h(378): note: failure was caused by a read of a variable outside its lifetime
C:\Repos\SoftwareRenderer\include\shader.h(378): note: see usage of 'this'
C:\Repos\SoftwareRenderer\include\shader.h(388): error C2131: expression did not evaluate to a constant
C:\Repos\SoftwareRenderer\include\shader.h(378): note: failure was caused by a read of a variable outside its lifetime
C:\Repos\SoftwareRenderer\include\shader.h(378): note: see usage of 'this'
C:\Repos\SoftwareRenderer\include\shader.h(389): error C2131: expression did not evaluate to a constant
C:\Repos\SoftwareRenderer\include\shader.h(378): note: failure was caused by a read of a variable outside its lifetime
C:\Repos\SoftwareRenderer\include\shader.h(378): note: see usage of 'this'
C:\Repos\SoftwareRenderer\include\shader.h(390): error C2131: expression did not evaluate to a constant
C:\Repos\SoftwareRenderer\include\shader.h(378): note: failure was caused by a read of a variable outside its lifetime
C:\Repos\SoftwareRenderer\include\shader.h(378): note: see usage of 'this'
C:\Repos\SoftwareRenderer\include\shader.h(391): error C2131: expression did not evaluate to a constant
C:\Repos\SoftwareRenderer\include\shader.h(378): note: failure was caused by a read of a variable outside its lifetime
C:\Repos\SoftwareRenderer\include\shader.h(378): note: see usage of 'this'
C:\Repos\SoftwareRenderer\include\shader.h(392): error C2131: expression did not evaluate to a constant
C:\Repos\SoftwareRenderer\include\shader.h(378): note: failure was caused by a read of a variable outside its lifetime
C:\Repos\SoftwareRenderer\include\shader.h(378): note: see usage of 'this'
C:\Repos\SoftwareRenderer\include\shader.h(393): error C2131: expression did not evaluate to a constant
C:\Repos\SoftwareRenderer\include\shader.h(378): note: failure was caused by a read of a variable outside its lifetime
C:\Repos\SoftwareRenderer\include\shader.h(378): note: see usage of 'this'
C:\Repos\SoftwareRenderer\include\shader.h(399): error C3863: array type 'Vector3f [maxLights]' is not assignable
C:\Repos\SoftwareRenderer\include\shader.h(405): error C3001: 'simd': expected an OpenMP directive name
C:\Repos\SoftwareRenderer\include\shader.h(407): error C3863: array type 'Vector3f [maxLights]' is not assignable
C:\Repos\SoftwareRenderer\include\shader.h(409): error C3863: array type 'float [maxLights]' is not assignable
C:\Repos\SoftwareRenderer\include\shader.h(414): error C3863: array type 'float [maxLights]' is not assignable
C:\Repos\SoftwareRenderer\include\shader.h(415): error C3863: array type 'float [maxLights]' is not assignable
C:\Repos\SoftwareRenderer\include\shader.h(417): error C3863: array type 'Vector3f [maxLights]' is not assignable
C:\Repos\SoftwareRenderer\include\shader.h(418): error C3863: array type 'float [maxLights]' is not assignable
C:\Repos\SoftwareRenderer\include\shader.h(419): error C3863: array type 'Vector3f [maxLights]' is not assignable
C:\Repos\SoftwareRenderer\include\shader.h(422): error C3863: array type 'Vector3f [maxLights]' is not assignable
C:\Repos\SoftwareRenderer\include\shader.h(425): warning C4305: 'argument': truncation from 'double' to 'const T'
`
Unfortunately I don't know at all what happens around line 378 but here it is
` const int maxLights = numLights;
    //Fresnel, normal distribution function and geometry occlusion
    Vector3f F[maxLights];`

Hey! I apologize for the delayed response. There's a windows branch for the project that fixes all of these issues. Let me know if that works better for you. Cheers!


Standardize folder format for benchmarks (Move benchmark-specific READMEs, util scripts, metrics, etc. to appropriate folders)

As we onboard more benchmarks and evaluation datasets, I think it is worth reorganizing the Testbed folder structure. A 1st pass proposed layout would be:
./testbed/
./testbed/README.md
./testbed/testbed  <-- Convert `run_scenarios.py` to a module and move it and other global stuff like `Includes` here
./testbed/scenarios <-- Keep as-is

Then for each Benchmark within scenarios, (using GAIA as an example):
./testbed/scenarios/GAIA/README.md         <-- Readme for running GAIA
./testbed/scenarios/GAIA/Templates             <-- As-is templates for the scenarios
./testbed/scenarios/GAIA/Scripts                  <-- GAIA related scripts (download, collate, etc.)
./testbed/scenarios/GAIA/Tasks                     <-- JSONL files describing the tasks (created dynamically when download script run)
./testbed/scenarios/GAIA/Results                  <-- Results folder produced by run_scenarios.py (created dynamically with run_scenarios)
./testbed/scenarios/GAIA/TaskResources      <-- Files referenced by tasks (created dynamically when download script is run)

This pattern would be repeated for HumanEval, MATH, etc.
Moving forward, we might also provide a Docker file and etc. perhaps in the root of each Benchmark as warranted.
@LeoLjl @kevin666aa  You have worked with the Testbed. What do you think of this?
I am willing to do the legwork to make this happen. I am not asking you to modify your contributions, but am asking if it would be ok for me to do so,

Where should on put requirements.txt for a dataset? I think it should also be put in corresponding scenarios.
./testbed/scenarios/GAIA/Scripts includes py files right? Just a personal preference, I would put these files in/testbed/scenarios/GAIA/ directly if there are not too many.
Another idea:
Remove TaskResources and add:
./testbed/scenarios/GAIA/Tasks/JSONL
./testbed/scenarios/GAIA/Tasks/Downloads

Yeah requirements will get moved too. I'm trying to decide the best place.
Technically if you put a requirements.txt file in a template folder, it will be copied over with that scenario, allowing. You can also even template it and have control per task instance. Either might be a better option going forward (I'm sort of learning as I go here)

Completed in #1048


Add token computation function

What this PR does:
Add a token computation function for the cases where num_token is low we won't rely on automatic token allocation.
This is adapted from an existing script in the DSE codebase; I've focused on producing identical results, even though some parts of the algorithm are perplexing (see inline comments).
Which issue(s) this PR fixes:
N/A
Checklist

[x] Changes manually tested
[x] Automated Tests added/updated
[ ] Documentation added/updated
[x] CHANGELOG.md updated (not required for documentation PRs)
[x] CLA Signed:  DataStax CLA

Codecov Report

Merging #713 (f7d68f7) into main (141fbf6) will increase coverage by 0.28%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #713      +/-   ##
==========================================
+ Coverage   53.38%   53.66%   +0.28%
==========================================
  Files          80       81       +1
  Lines        8216     8260      +44
==========================================
+ Hits         4386     4433      +47
+ Misses       3382     3380       -2
+ Partials      448      447       -1

Impacted Files
Coverage Δ

pkg/utils/tokens.go
100.00% <100.00%> (ø)

controllers/replication/secret_controller.go
65.43% <0.00%> (+1.00%)
:arrow_up:

@olim7t do you mind adding at least the issue URL in the changelog so that we can merge this? The rest can wait imo.

Rebased and addressed some of the feedback.

For what it's worth, there's also a token generation utility in open-source Cassandra. It plugs directly into the Cassandra internals (see GenerateTokens.java), so extracting the logic would not be as straightforward.
A few differences in the expected input:

it only handles one datacenter, there's no explanation on how you would extrapolate to multiple DCs.
the replication factor must be provided, and it does influence the results (I've checked). I don't really understand why.

On a single DC it generates different results than the DSE script.

@olim7t all is good I fixed the changelog and merged this. Thanks!


Some SED-ML variable names appear not to be supported

Including this additional SED-ML in the attached example
<dataGenerator id="Mcmin" name="Mcmin">
      <math xmlns="http://www.w3.org/1998/Math/MathML">
        <ci> Mcmin </ci>
      </math>
      <listOfVariables>
        <variable id="Mcmin" name="Mcmin" target="/sbml:sbml/sbml:model/sbml:listOfParameters/sbml:parameter[@id='Mcmin']" taskReference="BIOMD0000000297_sim"/>
      </listOfVariables>
    </dataGenerator>

<dataSet id="Mcmin" label="Mcmin" dataReference="Mcmin"/>

Causes this error:
Traceback (most recent call last):
  File "/usr/local/bin/tellurium", line 11, in <module>
    load_entry_point('Biosimulations-tellurium==2.4.1', 'console_scripts', 'tellurium')()
  File "/usr/local/lib/python3.6/dist-packages/Biosimulations_tellurium/__main__.py", line 51, in main
    app.run()
  File "/usr/local/lib/python3.6/dist-packages/cement/core/foundation.py", line 916, in run
    return_val = self.controller._dispatch()
  File "/usr/local/lib/python3.6/dist-packages/cement/ext/ext_argparse.py", line 808, in _dispatch
    return func()
  File "/usr/local/lib/python3.6/dist-packages/Biosimulations_tellurium/__main__.py", line 36, in _default
    exec_combine_archive(args.archive, args.out_dir)
  File "/usr/local/lib/python3.6/dist-packages/Biosimulations_tellurium/core.py", line 72, in exec_combine_archive
    factory.executePython()
  File "/usr/local/lib/python3.6/dist-packages/tellurium/sedml/tesedml.py", line 505, in executePython
    exec(compile(code, filename, 'exec'), symbols)
  File "/tmp/te-generated-sedml.py", line 202, in <module>
    Mcmin = __var__Mcnp.nanmin
NameError: name '__var__Mcnp' is not defined

Reference: BIOMD0000000650_sim_2.omex

The above example comes from BIOMD0000000297


Get invalid fields in OnValidationError method.

Pass a list of invalid fields as a method parameter that is annotated with @OnValidationError.

Done! (58eed4d1)


feat(release): Tweak version computation for release context

What does this PR do?
Pass a new argument to filter out the custom tags that can conflict with version computation

Motivation
Improve stability of release jobs. The automatic release PR creation wanted to create a version 7.56.0. This is because some custom tags like 7.55.0-installer* can be generated after creation of rc version and the script would want to create a new minor version instead of a release-candidate

Additional Notes
I'm not proud of this fix, if you have a better proposal I'd like it!
This PR also removes mention of agent6 on the create RC PR

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

I'm not proud of this fix, if you have a better proposal I'd like it!

Could you explain exactly what went wrong with the current version?

I'm not proud of this fix, if you have a better proposal I'd like it!

Could you explain exactly what went wrong with the current version?

@FlorentClarret Currently the issue is related to the custom build, that produces tags like 7.55.0-installer-* in the middle of the releasing process, that could make the script believe we should do a 7.56 instead of a 7.55.0-rc.2.
I've added this precise example in the PR description

/merge


An implementation for field.Field

Change Overview
Linked-list based implementation.
The approach attempts to prevent/limit the mutability of the fields outside the field package.
Also, in the package the contents of linkedField struct are not modified after the struct has been created.

Pull request type
Please check the type of change your PR introduces:

[x] Work in Progress
[x] Feature

Issues

#273

Test Plan

[x] Unit test

Minimum number of exported functions.
We may need to add other functions/helpers down the road to support other use cases for logging and errors.


Events are not emitted when colors are changed manually

🐛 Bug
When the face color or the edge color of a points layer is changed from the layer control, the corresponding events are not emitted.
To Reproduce
Steps to reproduce the behavior:

Add a points layer.
Run viewer.layers[0].events.face_color.connect(lambda x: print(x))
Change the face color in the layer control.

Expected behavior
Events should be emitted as other properties.
Environment

Please copy and paste the information at napari info option in help menubar here:

napari: 0.4.17
Platform: Windows-10-10.0.22621-SP0
Python: 3.9.16 | packaged by conda-forge | (main, Feb 1 2023, 21:28:38) [MSC v.1929 64 bit (AMD64)]
Qt: 5.15.2
PyQt5: 5.15.9
NumPy: 1.23.5
SciPy: 1.9.1
Dask: 2023.3.2
VisPy: 0.11.0
magicgui: 0.7.2
superqt: unknown
in-n-out: 0.1.7
app-model: 0.1.4
npe2: 0.6.2

OpenGL:
- GL version: 4.6.0 - Build <IP_ADDRESS>21
- MAX_TEXTURE_SIZE: 16384

Screens:
- screen 1: resolution 1368x912, scale 2.0

Any other relevant information:

Additional context

This is a combination of two things:

A bit of an inconsistency with the points layer control. Those values actually afftect the current_face_color and current_size, and not the face_color and size. There's been discussion about changing this, but there's also good reasons for this behaviour.
Even using the correct events, the events are not fired because the emitter is (wrongly) blocked. This is a bug! I just made a PR for this at #5722.


Exception when no dumps file

[ERROR] Returning error [Errno 2] No such file or directory: 'dumps/uri2rank_dump'
Traceback:
Traceback (most recent call last):
File "/usr/local/lib/python3.6/dist-packages/flask/app.py", line 1813, in full_dispatch_request
rv = self.dispatch_request()
File "/usr/local/lib/python3.6/dist-packages/flask/app.py", line 1799, in dispatch_request
return self.view_functionsrule.endpoint
File "/SBOLExplorer/flask/explorer.py", line 120, in sparql_search_endpoint
response = jsonify(search.search(sparql_query, utils.get_uri2rank(), utils.get_clusters()))
File "/SBOLExplorer/flask/utils.py", line 114, in get_uri2rank
uri2rank = deserialize(uri2rank_filename)
File "/SBOLExplorer/flask/utils.py", line 126, in deserialize
f = open(filename, 'rb')
FileNotFoundError: [Errno 2] No such file or directory: 'dumps/uri2rank_dump'

@cjmyers See #60

Should be fixed in the PR.


Landscape: "PPA" term entry

Write an explanation of PPAs and how they relate to Landscape. Your entry shouldn’t simply describe what a PPA is; the goal is to explain how PPAs relate to Landscape. For example, PPAs are used in the Landscape quickstart installation and client installation.
You should provide the entry as a Markdown file and put it in this file path: landscape/explanation/glossary/ppa.md. If any directories are missing, add them so your file path matches the one provided.
Prerequisites
A general understanding of how Landscape works and experience using and installing Landscape.
However, you don’t need any prior experience if you’re willing to explore the Landscape Docs and spend time learning about Landscape.
See also: Diátaxis - Explanation and Contribute to our documentation.

Can I be assigned to this @YanisaHS?

 Just linking it here as well in case you find it useful.
Let us know if you have any more questions! Otherwise, I'm looking forward to seeing your PR. (We'll also help you through the review process - your PR doesn't have to be perfect!) 😃

Hi @YanisaHS! :) Sorry for the radio silence. I'm still sifting through LandScape's documentation, so it's going to take me a while to create my Pull Request.

@CBID2 Sounds good. Thanks for the update! :smiley: