Network status indicator goes read

Open the tn 122 explorer in your browser and leave it open for a while. https://stage-dash.spacemesh.io/
After some time, the network status indicator changes from green to red.
Refresh the page - it goes green again.
Expected: network status to stay green while the network is alive and there's no error from the backend

fixed

It is actually a good time to see this issue for tn 122 which has a stuck verified layer but the color in the explore is orange - it should be red so please have a look and fix...

actually yes, my mistake, forgot to update stage, currently - done


Make shift laser gun

About the PR
Adds the construction graph for makeshift laser
Screenshots

Changelog

:cl:

add: Added makeshift laser gun!

The techfab circuit board should be mentioned in the PR description and changelog.

I made a dumbass suggestion and removed it. Apparently GitHub still keeps the little thingy. Oops.

👀 what's up with those files changed

i believe there might be a forking issue, causing files from your previous pr to be on this one. They should be removed before this is merged.

does this even work anymore now that laser cells arent ejectable

i dont really think this should be in regardless design wise

Do you have a different recipe in mind, or just remove the makeshift laser?

Do you have a different recipe in mind, or just remove the makeshift laser?

I think mirror doesn’t want mining mains mass producing laser guns.
Imo if it stays maybe require a battery, actual laser and some metal as the graph doesn’t make a ton of sense right now. Then salvage will have to get lucky on a ship (don’t think any have lasers) or cooperate with science.


Brewery explanation

I made a .md file to explain the brewery plugin and all the recipes for new players. The file would go under the "Getting Started" tab.

Thanks so much! I might move this to a future "Plugins" category.


Update formatPatternMap to correctly interpret %u

Solves https://github.com/goccy/go-zetasqlite/issues/125
Summary of problem:
When using something like FORMAT_TIMESTAMP('%u', sale_happened_at) it currently returns the week number of the current year (e.g. 43).
According to the BigQuery docs this should return the number of the week day instead (starting at 0 for Sunday).
(possible) cause
It looks like this is simply calling the wrong function.
Changes made in this PR:
I created a new function to return the number and added a test.
Thank you for considering this merge!

Thank you! Updated, please re-review ❤

Due to lack of response, this PR is closed


New spaces are delayed in status bar

When creating a new space, there is a noticeable delay of up to around 2 seconds before the space appears in the yabai status bar. The actual delay is quite variable.
This issue can be seen by just typing the command:
yabai -m space --create

If I use the example skhd config and focus the window as well, this issue does not persist.

I'm not going to bother changing this. The statusbar is for all purposes temporary and will be removed in some future version of yabai.


Empty list of results

Hi,
thank you very much for this tutorial!
I'm trying to build a small node app that allows me to fetch me calendar items of the day to help me taking notes and wanted to use this project as a foundation.
The Auth and other setup seems to work fine - my username is displayed and I get a 200 from the GET /calendar but there seems to be no results coming back - so at least the there are no events displayed...
I've tried to print out the the result of the query but unfortunately I'm not 100% sure how...
Is it possible that I need to pass in more information to the query - like the name of the desired calendar or something like it?
GET /calendar 200 177.840 ms - 3168   express:router dispatching GET /stylesheets/style.css +35ms   express:router query  : /stylesheets/style.css +0ms   express:router expressInit  : /stylesheets/style.css +0ms   express:router session  : /stylesheets/style.css +0ms   express:router <anonymous>  : /stylesheets/style.css +0ms   express:router <anonymous>  : /stylesheets/style.css +0ms   express:router logger  : /stylesheets/style.css +1ms   express:router jsonParser  : /stylesheets/style.css +0ms   express:router urlencodedParser  : /stylesheets/style.css +0ms   express:router cookieParser  : /stylesheets/style.css +0ms   express:router serveStatic  : /stylesheets/style.css +0ms GET /stylesheets/style.css 304 0.727 ms - -   express:router dispatching GET /images/no-profile-photo.png +1ms   express:router query  : /images/no-profile-photo.png +1ms   express:router expressInit  : /images/no-profile-photo.png +0ms   express:router session  : /images/no-profile-photo.png +0ms   express:router <anonymous>  : /images/no-profile-photo.png +0ms   express:router <anonymous>  : /images/no-profile-photo.png +0ms   express:router logger  : /images/no-profile-photo.png +0ms   express:router jsonParser  : /images/no-profile-photo.png +0ms   express:router urlencodedParser  : /images/no-profile-photo.png +0ms   express:router cookieParser  : /images/no-profile-photo.png +0ms   express:router serveStatic  : /images/no-profile-photo.png +0ms GET /images/no-profile-photo.png 304 0.700 ms - -
-- this are just the logs that the call to the /calendar api seems to work...
Every help is highly appreciated!
Thanks.
Peter

The sample defaults to getting any events for the current week on the default calendar. If you view your calendar on outlook.office.com, are there events for the current week?

Thanks for your answer - and yes there are tons of meetings

Hmm. You may be running into an issue with the time zone setting on your mailbox that others have reported. Can you change this line:
https://github.com/microsoftgraph/msgraph-training-nodeexpressapp/blob/08cc363e577b41dde4f6a72ad465439af20f4c3a/demo/graph-tutorial/routes/auth.js#L56
to this:
timeZone: user.mailboxSettings.timeZone || 'UTC'

Let me know if that improves the situation at all.

@jasonjoh - thanks - I've changed the line but there are still no events visisble...
I just tested the creation of a new event and this works perfectly fine - also the timezone is correct - could you give me a hint where in the code I could debug if there are really no events fetched? Maybe there is an issue with the displaying of the events?
Thanks so much!!!

You could set a breakpoint at this line and see what's in events:
https://github.com/microsoftgraph/msgraph-training-nodeexpressapp/blob/08cc363e577b41dde4f6a72ad465439af20f4c3a/demo/graph-tutorial/graph.js#L36
It would also be useful to see what the values of timeZone, start, and end are, and use those to replicate the same Graph request using Graph Explorer. The format for the request would be:
GET https://graph.microsoft.com/v1.0/me/calendarview?startdatetime=start&enddatetime=end

https://github.com/microsoftgraph/msgraph-training-nodeexpressapp/blob/08cc363e577b41dde4f6a72ad465439af20f4c3a/demo/graph-tutorial/graph.js#L25-L27

Thx @jasonjoh
this are the values:
end: '"2022-01-23T00:00:00+01:00"'
start: '"2022-01-16T00:00:00+01:00"'
timeZone: "W. Europe Standard Time"
events is undefined...
If I run the query at the Graph Explorer - I get a 200 back with a JSON that contains my information, but there is no events object...
It looks like this:
{<EMAIL_ADDRESS>"https://graph.microsoft.com/v1.0/$metadata#users/$entity",     "businessPhones": [         "+..."     ],     "displayName": "Krauß, Peter",     "givenName": "Peter",     "jobTitle": null,     "mail": "...,     "mobilePhone": null,     "officeLocation": "...",     "preferredLanguage": null,     "surname": "Krauß",     "userPrincipalName": "...",     "id": "..." }
I replaced the values, whcih are correct, with ...

I did a GET for /me/calendarview but if I try this once I get this error - see scrren -
Malformed JSON body - Review the request body and fix any malformed JSON. SyntaxError: JSON.parse: unexpected character at line 1 column 1 of the JSON data

If I run the query once again I get the result that I published here - see screen

The query doesn't go in the Request body field. It needs to be in the text box to the left of the Run query button.

@jasonjoh - sorry for the stupid mistake...
Turns out that this is a permission problem whcih can be only fixed by an admin from my org - which is unlikely to happen...
So I'm closing this here - thanks again for your help!


There is a touch event that returns information?

Is there an event that returns information about a certain part of model, including the name or index of the animation, by clicking on the this part?
Roughly speaking. There is a shelf with drawers. By clicking on one of the drawers, need to turn on its animation.
3D-viewer (kotlin)

Thanks for the answer! Yes, I understand that it is necessary to use the API on View. But can't you direct me a little deeper?

The PickingResult will return the Entity that was clicked, any other information needs to be queried from that. For instance, FilamentAsset::getName(Entity) will give you the name of that Entity if it came from a glTF.


bug Basic Javascript 30

Challenge http://www.freecodecamp.com/challenges/waypoint-generate-random-whole-numbers-with-javascript has an issue. Please describe how to reproduce it, and include links to screenshots if possible.
I typed in return(Math.floor(Math.random()*10));
without the *10, to figure out exactly what Math.floor does  (I'm still not sure) and I still cleared the waypoint, Eventhough that's not what you're supposed to do.

@Rudeclis thanks for posting this issue!
Note that your issue is a duplicate:
https://github.com/FreeCodeCamp/FreeCodeCamp/issues/2256

Please review the Guidelines for Contributing, thank you!.


bug: username collision on registration when remix-auth providers are used

Describe the bug
Right now a user can register and change their username to be an email that they don't own, and that would prevent a regular registration with that email.
Problem:

Current implementation populates username field with the email value we get from remix-auth providers.
This creates the problem described in the scenario

Scenario:

umut creates an account with an email<EMAIL_ADDRESS>umut goes to account settings and changes his username to<EMAIL_ADDRESS>but keeps his email the same<EMAIL_ADDRESS>can tries to create an account with an email<EMAIL_ADDRESS>we return an error saying that we have a user with that username.

Expected behavior
We need to find a way to not use emails for default values for usernames :D

We could try to store both the username and email as NAME@PROVIDER, this would ensure that both the fields are unique for each provider, and since we know the type of all the possible providers, it should not be hard to query these fields afterwards.

We could try adding randomly generated numbers after the username if the username is not available.
For example:

umut creates an account with an email<EMAIL_ADDRESS>umut goes to account settings and changes his username to<EMAIL_ADDRESS>but - - keeps his email the same<EMAIL_ADDRESS>can tries to create an account with an email<EMAIL_ADDRESS>can's newly created account has the name cansirin123 (or some other random value)


Leon test 25 November

Background

Proposed user journey

Approach

Questions / Assumptions

Acceptance Criteria

[ ] Enter text
[ ] Enter text
[ ] Enter text
[ ] Enter text

Reference
How to write good user stories

Test ticket

close in ZH not GHP


chore(main): release 0.1.25

:robot: I have created a release beep boop
0.1.25 (2024-09-10)
Features

Option to delete the base cluster topology (#16) (a49ab71)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/kurtosis-tech/kardinal-kontrol/releases/tag/0.1.25 :sunflower:


Make small tweaks to JavaScript time series tutorial

Changed the title from JavaScript to Node.js because the latter is more specific, we refer to the JavaScript client as Node.js here (https://crate.io/docs/crate/clients-tools/en/latest/), and because this tutorial is mainly targeting backend developers.  Not a big deal, but I thought it might sound more accurate!
Then I also made various small tweaks and fixes.

woo thanks!! looks great!
I have two comments:

for changes to the text that is also used by the other tutorials, can you also make the corresponding edits to those files? I'd like to keep the tutorials as similar as possible (down to word choice) and I figure that any changes to the shared text should be made atomically

the file itself should probably be changed from javascript.rst to node.rst
to accompany this rename, we will have to add a redirect to our nginx config so we're not breaking any existing links. I can show you how to do that!


cc tactic regression

We can't recover cc here in bernoulliPowerSeries_mul_exp_sub_one on Mathlib.NumberTheory.Bernoulli:
A : Type u_1
inst✝¹ : CommRing A
inst✝ : Algebra ℚ A
n : ℕ
hfact : ∀ (m : ℕ), ↑m ! ≠ 0
hite2 : (if n.succ = 0 then 1 else 0) = 0
x : ℕ × ℕ
h : x.1 + x.2 = n.succ
⊢ (↑x.2! * (↑x.2 + 1) = (↑x.2 + 1) * ↑x.2! ∨ (x.1 + x.2)! = 0) ∨ bernoulli x.1 = 0

As a porter of cc, I'm trying to fix this regression.

I found another error while investigating this error: #13369

In conclusion, this is not a regression but a bug from Lean 3.
⊢ (↑x.2! * (↑x.2 + 1) = (↑x.2 + 1) * ↑x.2! ∨ (x.1 + x.2)! = 0) ∨ bernoulli x.1 = 0
                   ^            ^

The OfNat ℚ 1 instances in these two 1s is different in Lean 4 but the same in Lean 3 (not OfNat ℚ 1 but has_one ℚ).
cc tactic can unify the def-eq instances usually, but it treats any OfNat.ofNat n as atomic symbols so it doesn't unify these two 1s.
This issues occurs in Lean 3 too:
example : @has_one.one ℚ rat.has_one = @has_one.one ℚ (add_monoid_with_one.to_has_one _) :=
begin
  cc,
/-
cc tactic failed
state:
⊢ 1 = 1
-/
end


Missing trading day in BVMF

For the BVMF calendar:
Due to the covid pandemic, a regularly observed holiday was moved to another date in Brazil. Later this holiday (Day of Black Awareness on November 20) became an actual trading on BVMF.
Therefore 20-Nov-2020 was a trading day but this information is not yet reflected in the package.

Hi @zecaclasher - nice catch, thanks for reporting it! Any chance you have a source you can share (any sort of news article or something similar) that has the 2020 observance date of Day of Black Awareness for BVMF?

https://www.b3.com.br/en_us/solutions/platforms/puma-trading-system/for-members-and-traders/trading-calendar/holidays/
Change reported from the exchange itself


📚 Create tutorial regarding SW setup for real drones

See question https://github.com/ctu-mrs/mrs_uav_system/discussions/57 and points after building own platform here https://github.com/ctu-mrs/mrs_uav_system/issues/39

WIP: https://ctu-mrs.github.io/docs/hardware/post_installation.html


[Feature] Make Peering, Network, and Storage Fabrics Optional

Appreciate the vision and code!
OTOH, it seems that for many use cases (at least for me), only the offloading feature is need and that the peering/network and storage fabric layers are mostly overhead. e.g., the IPAM of cooperative clusters could be planned ahead and that there exist more secure and efficient peering and network solutions without exposing public endpoints that need to be managed by Liqo. Data gravity is IMO a niche (anti-cloud) feature that's arguably not really aligned with the vision of moving workloads easily across clusters.
IMO, the control plane should focus on offloading and make it robust, which many could use immediately without the rest of the opinionated "features". I'd be delighted if it's already possible to install/deploy the offloading control plane only.
Thanks in advance for your advice/considerations!

Hi @vicaya!
Thanks for your feedback! We are working toward better modularity to allow users to use single liqo features (as the offloading) without the need for the others.
Currently, you can turn off the networking by setting the values with helm accordingly:
networking:
  # -- Use the default Liqo network manager.
  internal: false
  # -- Reflect pod IPs and EnpointSlices to the remote clusters.
  reflectIPs: false

The first flag turns off the liqo network, while the second turns off the reflection of pod IPs and EndpointSlices to remote clusters; you can set them based on your needs.
The only requirement is to be able to reach the authentication endpoint for the first peering (this will likely removed in future releases), and the remote API Server.
I would appreciate suggestions about missing features or improvements in the offloading module. Thanks in advance!

It is now possible with the liqo modules, see docs


Download Tickets as PDF

Describe the feature you'd like
There should be an option to download ticket in pdf format
Would you like to work on the issue?

[x] Yes

@iamareebjamal any thoughts on this

Go ahead


Page breaks when [mention] is supplied with an array which has null  value

If array supplied to [mention] has a null value, then the page breaks.
Can we handle this issue ?

I'm not able to reproduce this. Please provide an example of the code that is causing the issue.


refactor(docs): better styling for docs

✅ Checklist

[x] I have followed every step in the contributing guide (updated 2022-10-06).
[x] The PR title follows the convention we established conventional-commit
[x] I performed a functional test on my final commit

Changelog
[WIP]

changed the icons for copy button
reduced border radius for code blocks
added little bottom margin to h3 elements
border color for code blocks (inspired by astro docs)
darker code block bg-color
darker blockquote color for trpc
smaller font sizes

Screenshots

added bg color to nav elements and reduced font sizes

before:

after:

changed bg-color for code blocks and added better icons, reduced border-radius and added some fancy border

before:

after:

changed bg-color of blockquote in trpc and also made the name non-italic, removed unnecessary margin

before:

after:

💯

don't merge!

don't merge!

make it a draft

When you are ready for review, could you give some before/after screenshots of the specific things you changed? With styling PRs its sometimes hard to figure out what is actually different.

When you are ready for review, could you give some before/after screenshots of the specific things you changed? With styling PRs its sometimes hard to figure out what is actually different.

sure

@c-ehrlich i have updated the pr with some images, you may have a look

got most of the things done, will be leaving the codeblock theme as rose-pine for now and will add our own custom theme later

Looks like we can merge now. Did you have some font suggestions?

also, maybe add line numbers to code blocks 🤔

eh who cares

also, maybe add line numbers to code blocks 🤔

eh who cares

yea agreed, doesn't really matter for our code blocks

alright

btw need opinions on this, I dont really like scrollbars floating like this

It's fine imo

anything else we need in this pr or can this get merged now?

You've missed styling of the Theo quote:


Nuxt Build Error on VERCEL [error] [nuxt] [request error] [unhandled] [500] [nuxt] instance unavailable nuxt

Environment

Operating System: Darwin
Node Version:     v16.17.0
Nuxt Version:     3.6.1
Nitro Version:    2.5.2
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      modules, gtag, vite, linkChecker, experimental, nitro, robots, i18n, image, extends, runtimeConfig, css, build, app
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Build Modules:    -

Reproduction
The error is on Vercel, after the recent version of 3.6.1
All build processes are broken now

But on 'yarn build' local, all things runs fine
Describe the bug
The error is on Vercel, after the recent version of 3.6.1
All build processes are broken now

But on 'yarn build' local, all things runs fine
Additional context
No response
Logs
[success] [nitro] Generated public .vercel/output/static
[info] [nitro] Initializing prerenderer
[info] [nitro] Prerendering 2 initial routes with crawler
[log] [nitro]   ├─ /robots.txt (59ms)
[error] [nuxt] [request error] [unhandled] [500] [nuxt] instance unavailable

Seems like an i18n issue from the stacktrace. Are you sure the versions of the i18n module are the same?
In any case, would you provide a reproduction? 🙏

In my case, I called useAsyncData() twice in the setup hook to handle paging and it threw an instance unavailable exception. This was fine in 3.5.3 and earlier, but in 3.6.x it's still happening.
 ERROR  [nuxt] instance unavailable                                                           10:21:53 PM

  at useNuxtApp (.nuxt/prerender/chunks/app/server.mjs:212:13)
  at useAsyncData (.nuxt/prerender/chunks/app/_nuxt/query-82504ba2.mjs:26:16)
  at usePaginatedContent (.nuxt/prerender/chunks/app/_nuxt/usePaginatedContent-7ad2c523.mjs:310:54)     // my custom composable
  at runNextTicks (node:internal/process/task_queues:60:5)
  at process.processImmediate (node:internal/timers:442:9)
  at async setup (.nuxt/prerender/chunks/app/_nuxt/index-c4430770.mjs:83:146)


[Package Request]:

Package Requested

[ ] I would like someone else to build the manifest.
[X] I would like help so I can submit the manifest.
[ ] I have performed a search and couldn't find this package.
[ ] I think there is a new version available and I have provided the URL.

Please provide the following information
Publisher: PCGen Project
Publisher Url:  https://pcgen.org
Package Name: PCGen
Description: Tabletop character generator
Package Version: 6.08.00
Installer URL: https://github.com/PCGen/pcgen/releases/download/6.08.00RC8/pcgen-6.08.00RC8_win_install.exe
Current Package Identifier (for new versions): NA
I am getting validation errors in my YAML files, but I cant figure out what is failing. I am getting
Manifest Error: The multi file manifest is incomplete. A multi file manifest must contain at least version, installer and defaultLocale manifest. File: PCGen.PCGen.yaml
I have all 3 YAML files generated using the YAMLCreate script, but no dice. (converted to txt for upload)
PCGen.PCGen.installer.txt
PCGen.PCGen.locale.en-US.txt
PCGen.PCGen.txt

I smell a custom edit
  Dependencies:
  - OpenJDK


Kong 1.4.2 Ubuntu ARM64 image not working

Summary
Image 1.4.2-ubuntu for arm64 is not working.
Steps To Reproduce

docker run --rm -it --entrypoint=/bin/bash kong:ubuntu
kong version

/usr/local/openresty/nginx/sbin/nginx: 1: /usr/local/openresty/nginx/sbin/nginx: Syntax error: Unterminated quoted string
/usr/local/openresty/nginx/sbin/nginx: 1: /usr/local/openresty/nginx/sbin/nginx: ELF: not found

I encountered the same problem！
The nginx in kong:1.4.2-ubuntu for arm64 is x86_64：
file /usr/local/openresty/nginx/sbin/nginx
nginx: ELF 64-bit LSB executable, x86-64, version 1 (SYSV), dynamically linked (uses shared libs), for GNU/Linux 2.6.32, BuildID[sha1]=4c5a740327d6544393602efa8652f744af990a49, not stripped

@carnei-ro   Does it support arm64?

I also need to verify kong's support for arm64. Does the official support mirroring and theory?

@iver3on yes, kong works with x86 and arm64 (the last one is quite new. The 1.4.0 ubuntu image works)

created docker image leandrocarneiro/kong:1.4.2-ubuntu => arm64

Sorry about the mixup / thanks for bringing it to our attention.
The Kong 1.4.2 bintray asset has been fixed. Until the official docker image is updated one can build it locally if necessary:
uname -a
Linux docker-machine-arm64-ubuntu 4.15.0-1043-aws #45-Ubuntu SMP Mon Jun 24 14:08:49 UTC 2019 aarch64 aarch64 aarch64 GNU/Linux

git clone --branch 1.4.2 https://github.com/Kong/docker-kong.git
cd docker-kong/ubuntu/
docker build -t kong:1.4.2 .
docker run -it --rm kong:1.4.2 kong version
1.4.2

Alternatively the one I built is available here ( hutchic/kong:1.4.2-arm )

@lyndon160 thanks, we're looking into fixing the images!

The bintray asset has been updated to be correct
The docker-kong repository is updated including a naive test https://github.com/Kong/docker-kong/tree/1.4.3
Official kong image is pending https://github.com/docker-library/official-images/pull/7400

Closing this since more recent images are now working properly :+1:


WICKET-6688 add RFC support (to avoid unsafe eval)

Initial commit addressing https://issues.apache.org/jira/browse/WICKET-6688.
Please have a look if it makes sense. If it is I can add documentation and do a cleanup.

I went through comments.

(did a force push)

Hi! What's up with this one? Are there any alternative approaches to get rid of eval I can consider?

I'll take another look.

I've taken a second look and I'm all in favor of pursuing this improvement.
#appendJavaScript() and #appendRemoteFunctionCall() could be unified into a single method:
target.appendJavaScriptCall("Window.eval", "Wicket.Ajax.ajax{...}");
target.appendJavaScriptCall("Wicket.Ajax.RFC.testFunc1", "something2", 1);

But more importantly, in this current state of the proposal any Ajax refresh of any component breaks immediately due to blocking of eval(). Either we have to put much more effort into this and adapt most of Wicket's JS or this change is worthless.

I'm ready to contribute a substantial amount of time into resolving this issue.
My original concept was:

add the RPC/RFC support
create a new breed of CSP safe fetch based behaviors
deprecate existing behaviors and let them die slowly over the years.

Then we can put this pull-request on-hold until these fetch-based behaviors are available.
Until then eval() will be needed anyways, so there's no need to use this new RFC invocation.

Well, the whole point of this PR was that ajax could be used with strict CSP if proper behaviours used. (Old ajax behaviours using evals won't obviously work)
Let me merge back the master and see how it works together with nonce stuff.

Hi @svenmeier !
Now I see what do you mean.
		final Label c1 = new Label("c1", LambdaModel.of(this::getCounter1)) {

			@Override
			public void renderHead(IHeaderResponse response)
			{
				super.renderHead(response);
				response.render(OnDomReadyHeaderItem.forScript("console.log('c1 rendered')"));
			}
		};

The code above will add
<evaluate>
(function(){console.log('c1 rendered')})();
</evaluate>

to every response when target.add(c1) is called
This could be worked around if the JS is registered in advance with some name resolved from component name, however something like this:
		final Label c1 = new Label("c1", LambdaModel.of(this::getCounter1)) {

			@Override
			public void renderHead(IHeaderResponse response)
			{
				super.renderHead(response);
				if (Math.random() > 0.5)
				{
					response.render(OnDomReadyHeaderItem.forScript("console.log('c1 rendered more')"));
				} else {
					response.render(OnDomReadyHeaderItem.forScript("console.log('c1 rendered less')"));
				}
			}
		};

is not going to work. In this case both possible actions should be registered in advance.
What do you think?

Yes, this concerns any JS that is added during an ajax call.
If a component has an attached AjaxBehavior, setting it visible is sufficient to trigger an JS evaluation - via AjaxCallListeners this might contain a plethora of JS snippets.
How would you 'register' all these?
Until each an every of these cases (in Wicket's code and all users' code) are migrated, you'll need <evaluate> and thus nobody will use this RFC feature.

@svenmeier please have a look at the proof of concept in my last commit.
Generally the issue is in header item being evaluated instead of added to the head, so what I did is made all header contributions in ajax to render these header items, also I added nonce support to the JS.
Essentially this can be used to replace all evals (as a legacy support feature). I don't think this approach is optimal, and ideally components should keep their JS in a JS file similar to what we do with properties and html file, these JS's will be rendered as header items and later used with some JS initialisation callback, or maybe with standard function.
Do you think it's a right direction?
cc @martin-g @solomax

IMO we can't force user to use this or that technique
We only can propose sort of best-practices
Short inline JS is already being used everywhere (including multiple wicket based libraries)
I believe remove or reduce eval usage is good thing

@andruhon
It's not clear who will clean-up all those header items. Can we garantee, thas this doesn't impact the browser performance (e.g. garbage collection)?
If this solution works, why bother with this RFC at all, and just use header items all the time?

It's not clear who will clean-up all those header items.
The header item wipes itself on the last line https://github.com/apache/wicket/pull/378/files#diff-a7f583254a200944d085608b18d15244R2153

Can we garantee, thas this doesn't impact the browser performance (e.g. garbage collection)?
If this solution works, why bother with this RFC at all, and just use header items all the time?
The approach with header items is not particularly slow, however it's aimed to HEADER ITEMS. The direct call to predefined code is about 10 times faster than adding a new node in chrome. Please have a look at the exercise below:

    window.myNameSpace = {};
    window.myNameSpace.doSomething = function (){
      document.getElementById("myTestContainer").innerHTML = Date.now();
    };

    var cycles = 10000;
    var withDomStart = Date.now();
    for(var i = 0; i < cycles; i++) {
      var myScript = document.createElement("script");
      myScript.id = "myScr"+i;
      myScript.innerHTML = 'window.myNameSpace.doSomething(); document.getElementById("'+myScript.id+'").remove();';
      myScript.nonce = "4AEemGb0xJptoIGFP3Nd";
      document.head.appendChild(myScript);
    }
    var withDomTime = Date.now() - withDomStart;
    console.log("Execute with dom total, ms", withDomTime);
    console.log("Execute with dom average, ms", withDomTime/cycles);

    var withoutDomStart = Date.now();
    for(var i = 0; i < cycles; i++) {
      window.myNameSpace.doSomething();
    }
    var withoutDomTime = Date.now() - withoutDomStart;
    console.log("Execute without dom total, ms", withoutDomTime);
    console.log("Execute without dom average, ms", withoutDomTime/cycles);

The output is:
Execute with dom total, ms 786
Execute with dom average, ms 0.0786
Execute without dom total, ms 87
Execute without dom average, ms 0.0087

The difference is a way smaller if the function also creates and removes nodes, which usually happens when we add a new element onto the page. However, there must be a tidy way to call the function without touching the DOM at all (rfc/rpc).

would it be smarter if we reuse the <script> element and replace only its body ? would the browser re-evaluate the new body ? I haven't tested yet

@martin-g I thought about using the same JS. This could work if we set an id to this script tag. The question is how we going to identify them? Component id wouldn't work, because component could have several different header items added in renderHead. Any ideas?

Thanks Andrew!
Actually the header items performance isn't so bad if we compare it to eval():
window.myNameSpace = {};
    window.myNameSpace.doSomething = function (){
      document.getElementById("myTestContainer").innerHTML = Date.now();
    };

    var cycles = 10000;
    var withDomStart = Date.now();
    for(var i = 0; i < cycles; i++) {
      var myScript = document.createElement("script");
      myScript.id = "myScr"+i;
      myScript.innerHTML = 'window.myNameSpace.doSomething(); document.getElementById("'+myScript.id+'").remove();';
      myScript.nonce = "4AEemGb0xJptoIGFP3Nd";
      document.head.appendChild(myScript);
    }
    var withDomTime = Date.now() - withDomStart;
    console.log("Execute with dom total, ms", withDomTime);
    console.log("Execute with dom average, ms", withDomTime/cycles);

    var withoutDomStart = Date.now();
    for(var i = 0; i < cycles; i++) {
      window.eval("window.myNameSpace.doSomething();");
    }
    var withoutDomTime = Date.now() - withoutDomStart;
    console.log("Execute without dom total, ms", withoutDomTime);
    console.log("Execute without dom average, ms", withoutDomTime/cycles);

    var withoutEvalStart = Date.now();
    for(var i = 0; i < cycles; i++) {
      window.myNameSpace.doSomething();
    }
    var withoutEvalTime = Date.now() - withoutEvalStart;
    console.log("Execute without eval total, ms", withoutEvalTime);
    console.log("Execute without eval average, ms", withoutEvalTime/cycles);

The output:
Execute with dom total, ms 2102
Execute with dom average, ms 0.2102
Execute without dom total, ms 1176
Execute without dom average, ms 0.1176
Execute without eval total, ms 49
Execute without eval average, ms

console.time can be used for benchmarking :)
https://developer.mozilla.org/en-US/docs/Web/API/Console/time

BTW just stumbled on https://api.jquery.com/jQuery.globalEval/ - note that it supports a nonce option.
That function just forwards to https://github.com/jquery/jquery/blob/master/src/core/DOMEval.js, interesting how jQuery removes the item immediately.

Ah, yes. Век живи — век учись (а умрешь дураком).
I think we can use jQuery.globalEval in wicket-ajax-jquery then. I'll push tidied code next week then.
Have a nice weekend, everyone!

Век живи — век учись (а умрешь дураком)

well said! :-)


Can't use 'local' database through mongos' on server

System information:

Operating system (distribution) and version: Windows 11
DBeaver version: Version <IP_ADDRESS>206131757

Connection specification:

Driver name: C:\Program Files\DBeaverEE\plugins\org.jkiss.bundle.mongodb_4.5.1\lib\mongodb-driver-core-4.5.1.jar
Do you use tunnels or proxies (SSH, SOCKS, etc)? No

Describe the problem you're observing:
As a follow up to the new driver being implemented from issue: https://github.com/dbeaver/dbeaver/issues/16168#issuecomment-1098847982
Now when trying to connect to a MongoDB Atlas server instance the following error is received:
Error connecting to MongoDB instance [localhost]
  Command failed with error 20 (IllegalOperation): 'Can't use 'local' database through mongos' on server. The full response is {"ok": 0, "errmsg": "Can't use 'local' database through mongos", "code": 20, "codeName": "IllegalOperation"}

Hello, @Loizzus. Thank you for bug report.
Does this issue appear only in a newer DBeaver versions (since 22.0.5)?

Hi @E1izabeth,
Yes this is a new issue since 22.0.5, however I have never managed to get it working so that doesn't necessarily mean much.

@emironovDB  Here you go: dbeaver-debug.log

Fixed. Now we use database name from URL instead of local.

Verified


[Feature] Include the re:@user when using the QuoteDisable option

Dropping the quotes has cleaned things up quite a bit (thanks again), but telegram users have a tendency to respond to messages that are hours old so including the username of the person they're responding to would help with context.
Bonus points if you can have it replace the name of the bridge with the actual user who sent the text.

Not going to make this an extra option.
Instead I'm thinking about a QuoteFormat option, so you can specify the quote stuff like you want.

Working great, thanks again!
Any chance of getting it to replace the bridge name with the username when quoting something from the bridge?

Can't NickFormat do that for you ?
Or could you give an example of the issue?

The bridge's name is being used instead of the username of the person who sent the message.

Should I open a new feature request for this last bit, it's still nearly impossible to tell who a telegram user is responding to when they quote a message from the bridge.


BPjM Filter API

Hi @jan1993, @michischeid,
I am currently watching for BPjM filter APIs or services that give me access to any content. As far as I can see there is no way to consume the BPjM content directly: http://www.bundespruefstelle.de/bpjm/Aufgaben/Listenfuehrung/bpjm-modul.html
However there is an API from Google called "Safe Browsing" which seems to deliver what we need. The API is open source and the usage of the services are free of charge. But we need a developer key in order to use the service. I don't mind to use my account for developing and testing but for a long-term usage I prefer an official CAS account. How do you feel about this?

BTW... here is the link to the Google API: https://developers.google.com/safe-browsing/

@Monti93  I don't mind either. But for me it is just a project. I don't know if really would like to invest that effort. But feel free to do so.


[New game]: Word Detective

🎮 Game Request
"Word Detective" is an engaging and challenging word puzzle game. The objective is to guess the hidden word based on a given hint. Here's how the game works:

Letter Boxes: You'll have a series of letter boxes where you can type your guesses for the hidden word.
Hint Provided: A hint will be given to help you figure out the word. Use this clue wisely to make your guesses.
Incorrect Letters: Each time you guess a letter incorrectly, it will be added to the 'Incorrect Letters' row, making it easier to see which letters you've already tried.

Point down the features
Game points:
Letter Boxes: You'll have a series of letter boxes where you can type your guesses for the hidden word.
Hint Provided: A hint will be given to help you figure out the word. Use this clue wisely to make your guesses.
Incorrect Letters: Each time you guess a letter incorrectly, it will be added to the 'Incorrect Letters' row, making it easier to see which letters you've already tried.
Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @MuraliDharan7 , You can't have another issue before completing the previous one 😀
you already have the following 1 open issues 👀 ! :
#4352

/assign

Hey @MuraliDharan7 !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹


Dynops refactor

Some op() functions were very long and complex, so I tried to move part of them to private methods.
Also extracted some common parts like formatting of name when logging.

Parts of this request have been considered and (optionally) included during rewrites and updates. Thanks!


Codesystem/$lookup returns no results for CodeSystems without properties

Repro
Create flat CodeSystem, with no hiearchy and no code properties. Call CodeSystem/$lookup on a valid code
Expected Behavior
Code and display name are returned
Actual Behavior
"Not Found" exception
Root Cause
I think the root cause is the use of inner join rather than left join here:
https://github.com/medplum/medplum/blob/14789cdc2d0ff6a40ffb523b48279c888fea7a45/packages/server/src/fhir/operations/codesystemlookup.ts#L49-L54

Incidentally, this was already found and fixed in #4148

@mattwiller - close if done

Fixed in https://github.com/medplum/medplum/pull/4148


Implement query metadata convention for identifying Halin queries

https://trello.com/c/L5lCCjIq/68-convention-for-query-metadata

This is about adding standardized metadata to transactions that Halin sends to the server so that by looking at TX metadata it will always be obvious what was generated by Halin.

Note -- query metadata is only supported in Neo4j 3.5.x, so 3.4.* instances won't have query metadata sent at all.


Client Allocation Request for: Legend Corp

Client Allocation Questions
Core Information

Name: Legend Corp
Website / Social Media: https://aiqicha.baidu.com/company_detail_18439124798299
DataCap Requested: 5TiB
Addresses to be Notarized: f13vgza2trn2oagqyn6khveoysosl2kgtpicetbxa
Notary Requested: philippbanhardt

Hey @kidcdf - thanks for your request. Can you kindly add more information / context here on:

yourself , e.g. include links to social media / news mentions etc
your history working on IPFS / Filecoin, e.g. notable contributions to the community, type of engagement shown over time etc
type, size and source of the data you (and by extension other Discover participants) are looking to store on Filecoin, e.g. public data vs private data etc
your relationship with the data, especially with regard to copyright
how you intend to store the data on Filecoin, e.g. redundancy, location / miner preferences, etc

Thanks

Hey @kidcdf - thanks for your request. Can you kindly add more information / context here on:

yourself , e.g. include links to social media / news mentions etc
your history working on IPFS / Filecoin, e.g. notable contributions to the community, type of engagement shown over time etc
type, size and source of the data you (and by extension other Discover participants) are looking to store on Filecoin, e.g. public data vs private data etc
your relationship with the data, especially with regard to copyright
how you intend to store the data on Filecoin, e.g. redundancy, location / miner preferences, etc

Thanks


Crafting Serpentine Helm

🚀 Feature Proposal
Add an option in crafting that lets the player craft a serpentine helm from a serpentine visage. Currently, the helmet is not obtainable.
Describe the feature you're suggesting here, give lots of information and describe it as best you can. Please check that the feature hasn't already been suggested first, by searching the issues in this repository.

The claim that the serpentine helm (uncharged) is not craftable is demonstrably false. In the future, please ensure that the issue you’re submitting is correct.

And if you were referring to charging the serpentine helm then a) that’s not crafting and b) you can refer to issues #750 or #802


Added JWST M106 to Constellations and Legacy

Showed as a tutorial so I hope this worked!

I had to add a small change because there were two "place" entries for the new image, for some reason — I think because of the dry-run step. Other than that, looks good!


etcdctlv3: implement ping command

/cc @mqliang @heyitsanthony @gyuho
ping pings every endpoint.
It can generate 3 outputs:

cannot connect to the member through endpoint

connected to the member, but member failed to commit any proposals

connected to the member, and member committed a proposal

ping may be confusing (e.g., I can ping the server, why can't I ping through etcdctl?), but I can't think of a better name (best I've got is cping for consensus ping)...

@colhom @jonboulle @philips any opinion on the naming? cluster heath is proved to be confusing when we are actually checking the health of each member. (as long as one member can commit proposal, the cluster is OK)

What about member-health?

I have the same questions as @jonboulle, isn't this what member-health is?

@jonboulle @philips This is for v3. Cluster health is confusing actually: see #3852

Understanding the use case perhaps "test-connectivity"

Or test-endpoint-connectivity

@philips @heyitsanthony test-endpoints-liveness? or just liveness?

But it's more than liveness, you're also checking whether the member can propose/commit.. I still don't understand why member-health isn't appropriate here?

@jonboulle
This command tries to propose through every endpoint user gives. So it is different from member. One member might have multiple endpoints and the endpoint user gives might not cover all members. If we make endpoint == member, there will be confusion.
We do not want to resolve the member list through the given endpoints either. This is proven to cause confusion. Say a user gives us three endpoints A, B, C. We successfully resolved member list and got the member url A, E, F. Then we will not try B, C. User then will get confused. I asked this command to try A, B, C! Why do you try A, E, F.

But it's more than liveness, you're also checking whether the member can propose/commit..

liveness == making progress in distributed system (https://en.wikipedia.org/wiki/Liveness). It is not very accurate though.

@xiang90 it seems like liveness a property of the entire system, not the node/process?
Why not endpoint-health?

+1
On Tue, Mar 22, 2016 at 5:54 PM Anthony Romano<EMAIL_ADDRESS>wrote:

@xiang90 https://github.com/xiang90 it seems like liveness a property
of the entire system, not the node/process?
Why not endpoint-health?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/coreos/etcd/pull/4812#issuecomment-199903485

@heyitsanthony @gyuho PTAL.

lgtm.

lgtm


Questions  About Reward Model

Hi thx so much about this brilliant project which benefits me a lot!! But I still have some questions about the Reward Model, I would be appreciated if you could provide me some illustration!!
Question 1:  Why doesn't the  Reward Model (OpenRLHF/Llama-3-8b-rm-mixture) have a value head?
I tried to load the reward model directly:
model_name_or_path = "OpenRLHF/Llama-3-8b-rm-mixture"
model = AutoModel.from_pretrained(model_name_or_path, torch_dtype=torch.bfloat16 )

and get the model structure as follows:
LlamaModel(
  (embed_tokens): Embedding(128256, 4096)
  (layers): ModuleList(
    (0-31): 32 x LlamaDecoderLayer(
      (self_attn): LlamaSdpaAttention(
        (q_proj): Linear(in_features=4096, out_features=4096, bias=False)
        (k_proj): Linear(in_features=4096, out_features=1024, bias=False)
        (v_proj): Linear(in_features=4096, out_features=1024, bias=False)
        (o_proj): Linear(in_features=4096, out_features=4096, bias=False)
        (rotary_emb): LlamaRotaryEmbedding()
      )
      (mlp): LlamaMLP(
        (gate_proj): Linear(in_features=4096, out_features=14336, bias=False)
        (up_proj): Linear(in_features=4096, out_features=14336, bias=False)
        (down_proj): Linear(in_features=14336, out_features=4096, bias=False)
        (act_fn): SiLU()
      )
      (input_layernorm): LlamaRMSNorm((4096,), eps=1e-05)
      (post_attention_layernorm): LlamaRMSNorm((4096,), eps=1e-05)
    )
  )
  (norm): LlamaRMSNorm((4096,), eps=1e-05)
  (rotary_emb): LlamaRotaryEmbedding()
)

I expected to get something like (v_head): Linear(in_features=..., out_features=1... bias=False) just as I found in other opensource reward model like internlm/internlm2-1_8b-reward etc., could you give some hints about this question?
Question2:  As for reward model, do we initialize the value head every time no matter whether we are training it or just using it?
I found that in train_ppo.py, we get reward model by _get_reward_model.
But when I dived into this function, I found in the __init__that:
# https://github.com/OpenRLHF/OpenRLHF/blob/main/openrlhf/models/model.py#L155
setattr(self, value_head_prefix, nn.Linear(config.hidden_size, 1, bias=False))

By using setattr, it seems that we would use nn.Linear(config.hidden_size, 1, bias=False) to replace  the reward model's value_head no matter the reward model(OpenRLHF/Llama-3-8b-rm-mixture) has a value_head or not. But in PPO training we just want to use the reward model directly without refining its structure.  I'm really confused about this and don't know if I make something wrong here.

OpenRLHF/Llama-3-8b-rm-mixture has a value_head, just print the state_dict

OpenRLHF/Llama-3-8b-rm-mixture has a value_head, just print the state_dict

oh yes,  i just read the source codes of Transformer .from_pretrained method, and I know where the probs are:

First, I should not use something like AutoModel to load RM directly, I should use cls_class from _get_reward_model, that solves my Question 1.
Second, As for  _get_reward_model, it always initialize a value_head first, and then when we use .from_pretrained to load model, if the model itself has value_head, it would replace the initialized value_head, otherwise we just use the value_head we initilaize, thus for separating RM inference and Training, that solves my Question 2.

Thx for your patience and reply!


More or less a pixel randomly occured when wifi is connecting.

Wifi is connecting,according my  investigation,when esp32 collects data from camera ,more or less a pixel occured randomly.It lead to data of the line shift.So this line will be wrong.If this issue cann't be avoided,what measures should be taken on the software?

It looks to me that the same issue is posted on esp32.com forum . I think that forum is more appropriate place to discuss it since esp32-cam-demo does not have any Wi-Fi functionality implemented.


Delete fail netcore

context.Table1.Where(t => t.CreatedDate.Date == DateTime.Today).Delete();
It delete all record from Table1. LOL

Hello @giangcoi48k ,
Which provider are you using? Is it possible to provide us the SQL statement executed?
Best Regards,
Jonathan


Number of needed tensors to be processed

Dear Clair team
It would be great if Clair prints the number of needed tensors to be processed, maybe like this Processed 5000 out of 1000000 tensors. This can help me to know how much time it needs to finish.
Delay 7 seconds before starting variant calling ...
[INFO] Using 3 CPU threads
INFO:tensorflow:Restoring parameters from /Volumes/sina2tb/model
Restoring parameters from /Volumes/sina2tb/model
Calling variants ...
Processed 1000 tensors
Processed 2000 tensors
Processed 3000 tensors
Processed 4000 tensors
Processed 5000 tensors

Regards,
Sina

Thanks Sina for the suggestion. As Clair supports streamlined processing, meaning that Clair will start processing the already prepared tensors before all tensors are generated, thus Clair doesn't know how many tensors beforehand.

I see. Thanks for your kind response.
I reach to the idea that one can check the last line of VCF file being generated and compare the position of last called variant to the length of chromosome (or the chr compared to number of all chrs in bam file) in order to have a rough sense of the needed time.
The reason that I asked question was experiencing several hours of run although previously it takes minutes to finish.  I found my mistake that I used different reference genome version (hg19)  as the input of Clair rather than the aligned reads (hg18).
Thanks anyway :)


Split task to copy OSGi bundles to bundlesPath from createOsgiRuntime

It might be useful to run a task to just copy all runtime bundles to a specific directory, without creating a full runtime (with system libs, the system bundle, run scripts etc).
This can be easily done by moving some functionality from the createOsgiRuntime task into a new task, say createOsgiBundlesDir.

Fixed in the 1.5.3 release.


Update the demo data

Some fields are incorrect (DOI in 9xxx fields).
The metadata should be fixed (cleaned according to HOWTOMARC specifications).

Find the demodata (in invenio-records ?, maybe in tests), clean the incorrect fields (use dojson to perform identity test - use code from Sami)

Test function: https://github.com/inveniosoftware/dojson/pull/76/files#diff-09f983a1db5a7aca6102f2e2a260c88cR24

(TODO assign to Leo)

Do you mean run this and compare the result with the original?
dojson -l marcxml -d marcxml do marc21 do to_marc21 < in.xml > out.xml

After analyzing  authority.xml with that command, I found that:
Fields that are defined in marc21 but not in to_marc21:

[ ] 148
[ ] 150
[ ] 371
[ ] 372
[ ] 430
[ ] 450
[ ] 643
[ ] 670
[ ] 678
[ ] 680

Fields completely missed:

[ ] 690
[ ] 913
[ ] 914
[ ] 920
[ ] 961
[ ] 970
[ ] 980
[ ] FFT

Be careful with the authority records, they use a different set of rules
https://github.com/inveniosoftware/dojson/blob/master/dojson/contrib/marc21/model.py#L17

I tried with:
dojson -l marcxml -d marcxml do marc21_authority do to_marc21 < authority.xml > /tmp/output.xml

But, actually, remains only these tags in output.xml:

003
005
024
035
270
856

This is because you are mixing models, the input you have marc21_authority and in the output to_marc21, which is meant for bibliographic marc and not for authority.
These two they should go in pairs and I am afraid there is no to_marc21_authority for now.

I tried the same analysis on bibliographic.xml and I found this list of missing fields:

269
590
595
690
691
693
694
695
697
859
901
909
916
960
961
962
963
964
970
980
999
FFT

WDYT @egabancho

@hachreak When you find a field that is missing, you can check whether it is part of the specification here: https://www.loc.gov/marc/bibliographic/. All the 9xx fields for example are not part of the specification and can be removed.

@SamiHiltunen is right, in general  59X, 69X and 9XX are custom/local fields and they should not be considered here.
I don't think any of the fields you listed are standard fields :wink:

@egabancho There is a small mention of 9xx here:
https://www.loc.gov/marc/bibliographic/bd9xx.html
should we map some of the 9xx, like:
980 -> 800 (Series Personal Name)
or should we leave them out completely ?

I think we should leave them out completely, we are using them with a totally different meaning

:+1:

Ok, I made the clean of bibliographic.xml.
Only 2 things:

sometimes the order is not respected (but I think we need only wait @SamiHiltunen PR)
in the field 773 the t subfield disappears. @egabancho any idea why? because it seems to be defined in marc21 and to_marc21.

@hachreak It might have to do something with the incomplete field_map from the mappings. I have checked the PR https://github.com/inveniosoftware/dojson/pull/76 from @SamiHiltunen and with the code from his pull request it works fine (he is adding field t to field_map: https://github.com/inveniosoftware/dojson/pull/76/files#diff-9603caabcde9c9896ee926f6d3a8f915R574 so maybe that's why).

@switowski can we close this issue? or maybe ask to sami to automatically close on merge #76

Sure, closing as there is nothing else to be done here.


Elishaokon13 patch 1

Fixes Issue

Changes proposed

Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] My change requires changes to the documentation.
[x] I have updated the documentation accordingly.
[x] All new and existing tests passed.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

Hi @dan-mba Thank you for that observation.
I have fixed it now


AuthorizationContext => AuthorizationHandlerContext

AuthorizationContext in the code sample should be AuthorizationHandlerContext

Hi @jerriep, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, DNFBOT;

@blowdart  please review.

Oops, I missed one? Sorry. LGTM

@jerriep  Thanks!

@RickAndMSFT and @blowdart You may just want to review the "Accessing MVC Request Context In Handlers" section at the bottom of the document.
There is also a mix of AuthorizationFilterContext and AuthorizationContext being used in that section. I did not bother to figure out what the correct usage should be in that case.


Suggestion: Alternate Data Streams

SQL Server should be updated to support alternate data streams, which allows alternate datasets to be returned if enabled, otherwise not sent.  The common use for alternate streams would be debugging, QA, and logging.  Developers could seed debugging queries directly into the code, enabling and disabling without the need to recompile or distribute.
When enabled, query results also include metadata to indicate which stream the results belong to, where the stream is an INT.    Clients are responsible for interpreting results as appropriate.
A connection option should enable this feature, as well as a per-stream enabling.  e.g. ;USE ALTSREAM.  Specific streams also be enabled: e.g. USE ALTER STREAM:1,4,10.  Stream 0 would be permeated enabled.
Within a session, data sent to streams other than 0 should be able to be suppressed.  e.g. SET ALTSTREAM OFF, or on a stream by stream basis: SET ALTSTREAM 2 OFF.  Metadata on the stream would still be included, but only data for stream 0 should be sent. The ability to toggle this state would be permissions-based.  By default, if the feature is enabled and the connecting user has permissions it would be to ON, otherwise OFF.
CRUD statements would be extended to support the selection of the stream, defaulting to 0 when not supplied.   The EXECUTE statement would also be extended to support the selection of the default stream during the execution of the procedure. One possible way may be through the use of a query hint/execute option: e.g. USE STREAM 2.   The ability to not execute a statement when alternate streams are not available should exist:  USE STREAM 2 ONLY WHEN AVAILABLE.  Any stream hints/options would be ignored when outputting to a table (of any sort).

@grahamlower -- thank you for your considered and detailed feedback.  Please add these suggestions here:

SQL Server Feedback

I am closing this issue now.  You are welcome to @ mention me for any followup.  We hope to hear from you again.

@WilliamAntonRohm I'd love to, but it seems there has been an update recently and it redirects to a generic Azure Feedback page (https://azure.microsoft.com/en-us/feedback/) with no obvious way to actually give feedback or suggestions.  Any pointers?
As an aside: your comment suggests this is not the preferred place for feedback and suggestions, so I want to point out that  https://docs.microsoft.com/en-us/sql/sql-server/sql-server-get-help?view=sql-server-ver15 links here to share "an idea, a suggestion, feedback, anything!"  It may be worth following up on that if there is such a preference.

@grahamlower -- thank you for your follow-up & researches.  I'm looking into the current suggestion link.

@grahamlower -- thank you for your patience.  There's a new place for SQL feedback:

SQL forum

I am closing this issue now.  You are welcome to @ mention me for any followup.  We hope to hear from you again.


[ports/nuklear] Upgrade to latest nuklear (12/05/2022)

Describe the pull request
Upgrade port

What does your PR fix?

Upgrades the port

Which triplets are supported/not supported? Have you updated the CI baseline?

All

Does your PR follow the maintainer guide?

Yes
If you are still working on the PR, open it as a Draft: https://github.blog/2019-02-14-introducing-draft-pull-requests/

Thanks for this PR!
However, we prefer to not take large nonessential formatting changes like this -- imagine if another user had a differently-configured autoformatter and every update the file was thrashing back and forth between different formats!
Please apply the following patch, then this PR LGTM:
diff --git a/ports/nuklear/CMakeLists.txt b/ports/nuklear/CMakeLists.txt
index 89e344165..eba91e1fd 100644
--- a/ports/nuklear/CMakeLists.txt
+++ b/ports/nuklear/CMakeLists.txt
@@ -6,33 +6,29 @@ option(INSTALL_DEMO "Install the demo code" OFF)

 add_library(nuklear INTERFACE)

-include(GNUInstallDirs)
-target_include_directories(nuklear INTERFACE "$<INSTALL_INTERFACE:${CMAKE_INSTALL_INCLUDEDIR}/nuklear>")
+target_include_directories(nuklear INTERFACE $<INSTALL_INTERFACE:include/nuklear>)
 target_compile_definitions(nuklear INTERFACE NK_IMPLEMENTATION)

 # Installation
 install(
     TARGETS nuklear
     EXPORT unofficial-nuklear
-    ARCHIVE DESTINATION "${CMAKE_INSTALL_LIBDIR}"
-    LIBRARY DESTINATION "${CMAKE_INSTALL_LIBDIR}"
-    RUNTIME DESTINATION "${CMAKE_INSTALL_BINDIR}"
+    RUNTIME DESTINATION bin
+    LIBRARY DESTINATION lib
+    ARCHIVE DESTINATION lib
 )

-install(FILES "${CMAKE_CURRENT_LIST_DIR}/nuklear.h"
-        DESTINATION "${CMAKE_INSTALL_INCLUDEDIR}/nuklear")
+install(FILES "${CMAKE_CURRENT_LIST_DIR}/nuklear.h" DESTINATION "include/nuklear")

-install(EXPORT unofficial-nuklear
-        FILE "unofficial-nuklear-config.cmake"
-        DESTINATION "${CMAKE_INSTALL_DATAROOTDIR}/unofficial-nuklear")
+install(EXPORT unofficial-nuklear FILE unofficial-nuklear-config.cmake DESTINATION share/unofficial-nuklear)

 if (INSTALL_EXAMPLE)
-    install(DIRECTORY "${CMAKE_CURRENT_LIST_DIR}/example"
-            DESTINATION "${CMAKE_INSTALL_DATAROOTDIR}/nuklear")
+    install(DIRECTORY "${CMAKE_CURRENT_LIST_DIR}/example" DESTINATION share/nuklear)
+
 endif()

 if (INSTALL_DEMO)
-    install(DIRECTORY "${CMAKE_CURRENT_LIST_DIR}/demo"
-            DESTINATION "${CMAKE_INSTALL_DATAROOTDIR}/nuklear")
+    install(DIRECTORY "${CMAKE_CURRENT_LIST_DIR}/demo" DESTINATION share/nuklear)
+
 endif()
diff --git a/ports/nuklear/portfile.cmake b/ports/nuklear/portfile.cmake
index 4ca2095c1..f30f54d0f 100644
--- a/ports/nuklear/portfile.cmake
+++ b/ports/nuklear/portfile.cmake
@@ -1,13 +1,12 @@
 vcpkg_from_github(
     OUT_SOURCE_PATH SOURCE_PATH
-    REPO            Immediate-Mode-UI/Nuklear
-    REF             0a77211844458673446aa99b8a0ae1090a04e413 # 2022-05-12
-    SHA512          57ad5578c534d19ce6dd5e9b8c7b725da6e7f379a5289daed6b7128a1671dc5c368175d1e780a7fcb3405fad989e3bc50a8a921d62301e64e58014e351845cae
-    HEAD_REF        master
+    REPO Immediate-Mode-UI/Nuklear
+    REF 0a77211844458673446aa99b8a0ae1090a04e413 # 2022-05-12
+    SHA512 57ad5578c534d19ce6dd5e9b8c7b725da6e7f379a5289daed6b7128a1671dc5c368175d1e780a7fcb3405fad989e3bc50a8a921d62301e64e58014e351845cae
+    HEAD_REF master
 )

-file(COPY "${CURRENT_PORT_DIR}/CMakeLists.txt"
-     DESTINATION "${SOURCE_PATH}")
+file(COPY "${CURRENT_PORT_DIR}/CMakeLists.txt" DESTINATION "${SOURCE_PATH}")

 vcpkg_check_features(OUT_FEATURE_OPTIONS FEATURE_OPTIONS
     FEATURES
@@ -16,8 +15,8 @@ vcpkg_check_features(OUT_FEATURE_OPTIONS FEATURE_OPTIONS
 )

 vcpkg_cmake_configure(
-    SOURCE_PATH "${SOURCE_PATH}"
-    OPTIONS     ${FEATURE_OPTIONS}
+    SOURCE_PATH ${SOURCE_PATH}
+    OPTIONS ${FEATURE_OPTIONS}
 )

 vcpkg_cmake_install()
@@ -27,6 +26,4 @@ vcpkg_cmake_config_fixup(PACKAGE_NAME unofficial-nuklear)

 file(REMOVE_RECURSE "${CURRENT_PACKAGES_DIR}/debug")

-file(INSTALL "${SOURCE_PATH}/src/LICENSE"
-     DESTINATION "${CURRENT_PACKAGES_DIR}/share/${PORT}"
-     RENAME copyright)
+file(INSTALL "${SOURCE_PATH}/src/LICENSE" DESTINATION "${CURRENT_PACKAGES_DIR}/share/${PORT}" RENAME copyright)
diff --git a/ports/nuklear/vcpkg.json b/ports/nuklear/vcpkg.json
index 25261916e..1e48043bf 100644
--- a/ports/nuklear/vcpkg.json
+++ b/ports/nuklear/vcpkg.json
@@ -1,9 +1,9 @@
 {
   "name": "nuklear",
-  "version-date": "2022-05-12",
+  "version-date": "2021-03-18",
+  "port-version": 1,
   "description": "This is a minimal state immediate mode graphical user interface toolkit written in ANSI C and licensed under public domain",
   "homepage": "https://github.com/Immediate-Mode-UI/Nuklear",
-  "license": "Unlicense OR MIT",
   "dependencies": [
     {
       "name": "vcpkg-cmake",
diff --git a/versions/baseline.json b/versions/baseline.json
index 3c94378e7..87bf58967 100644
--- a/versions/baseline.json
+++ b/versions/baseline.json
@@ -4921,8 +4921,8 @@
       "port-version": 0
     },
     "nuklear": {
-      "baseline": "2022-05-12",
-      "port-version": 0
+      "baseline": "2021-03-18",
+      "port-version": 1
     },
     "numactl": {
       "baseline": "2.0.14",
diff --git a/versions/n-/nuklear.json b/versions/n-/nuklear.json
index 9286743e6..b0784005d 100644
--- a/versions/n-/nuklear.json
+++ b/versions/n-/nuklear.json
@@ -6,7 +6,7 @@
       "port-version": 0
     },
     {
-      "git-tree": "9c243e51fbb0d9fa4562a3e4a8eba9a7fb2e0cdd",
+      "git-tree": "d0f86ceb1fe400a4e2cdd6ff76d82dfe539a0203",
       "version-date": "2021-03-18",
       "port-version": 1
     },

Wait what are all those non-whitespace changes?

Sorry, I had some errors in that patch as well as perhaps it didn't fully post correctly. I've pushed a branch instead:
https://github.com/ras0219-msft/vcpkg/commit/5858af159a63fb9f88d68182221e67e3ceef4d4e.
In addition to reverting the whitespace changes, I've also reverted many of the changes to the CMakeLists.txt. While your changes would be following best practices for an external project CMake buildsystem, we intend replacement buildsystems inside vcpkg to be as direct and concrete as possible -- that means there's no need for GNUInstallDirs etc because vcpkg has a fixed install format.
Hopefully your upstream PR (https://github.com/Immediate-Mode-UI/Nuklear/pull/462) gets merged, we can delete this entire file, and never look back :)
git fetch https://github.com/ras0219-msft/vcpkg dev/roschuma/pr24829
git merge FETCH_HEAD

@SamuelMarks, could you please apply the patch?

Pinging @SamuelMarks for response.

Hmm, I thought I applied the patch, just double checked:
/vcpkg (nuklear-12_05_2022-upgrade) $ git fetch https://github.com/ras0219-msft/vcpkg dev/roschuma/pr24829
From https://github.com/ras0219-msft/vcpkg
 * branch                dev/roschuma/pr24829 -> FETCH_HEAD
/vcpkg (nuklear-12_05_2022-upgrade) $ git pull
Already up to date.
/vcpkg (nuklear-12_05_2022-upgrade) $ git merge FETCH_HEAD
Already up to date.
/vcpkg (nuklear-12_05_2022-upgrade) $ git push
Everything up-to-date

Even triple-checked:
/vcpkg (nuklear-12_05_2022-upgrade) $ git merge 5858af159a63fb9f88d68182221e67e3ceef4d4e
Already up to date.

Hmm, I thought I applied the patch, just double checked:
/vcpkg (nuklear-12_05_2022-upgrade) $ git fetch https://github.com/ras0219-msft/vcpkg dev/roschuma/pr24829
From https://github.com/ras0219-msft/vcpkg
 * branch                dev/roschuma/pr24829 -> FETCH_HEAD
/vcpkg (nuklear-12_05_2022-upgrade) $ git pull
Already up to date.
/vcpkg (nuklear-12_05_2022-upgrade) $ git merge FETCH_HEAD
Already up to date.
/vcpkg (nuklear-12_05_2022-upgrade) $ git push
Everything up-to-date

Even triple-checked:
/vcpkg (nuklear-12_05_2022-upgrade) $ git merge 5858af159a63fb9f88d68182221e67e3ceef4d4e
Already up to date.


Add documentation on how to run locally with the updater flag.

Obsolete since we are moving away from importing assessments from Git.


PluggableMap class removed from OL7

The PluggableMap class has been removed in Openlayers 7. But ol-layerswitcher v4.0.0 is still trying to import it. Is it possible to replace it with the regular Map class?

Good catch, I guess that makes the types incompatible with ol v7.
Would you be happy submitting a pull request?


Git commit with two separate messages

How can I commit with two separate messages?
According to this post I must use two -m to commit message and close issue on GitLab.
Or how can I commit on VSCode and close issue on GitLab?

Just type the message. Then press Enter twice. Then type the description. Then press Ctrl+Enter.


(GH-2698) Support interpreters configuration when running scripts

This adds support for running scripts using the interpreters transport
configuration. This feature is gated by the script_interpreter future
flag.
!feature

Support interpreters configuration when running scripts
(#2698)
Bolt now supports the interpreters transport configuration when
running scripts. To enable this feature, set the
future.script_interpreter configuration to true.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Oh dang does need a r-r-r-r-rebase tho


chore: add pull request template

What
I'd like to make this repo easier to manage by creating a template for pull requests.
The documentation for creating templates on GitHub can be found here: https://docs.github.com/en/communities/using-templates-to-encourage-useful-issues-and-pull-requests/creating-a-pull-request-template-for-your-repository.
What to include in the template
Here is what I would like to find in the template:

a description section
does this solve an issue? If so, reference the issue, or describe it
some checkboxes to make stating

[ ] I understand that my contributions needs to pass the checks
[ ] If I create new functions / methods, I documented them and add type hints
[ ] If I modified already existing code, I updated the documentation accordingly

additional contents: images, error output, ...

Hey @jeertmans I would like to work on this issue. Can you assign it to me. Thanks 😄

Hi @ahmedsilat44! Sure, go on ;-)
Do not hesitate to ask questions if you have any!

If you want to add more issue templates I would like to work on that too. Let me know if you do or if u you want to give someone else the chance to contribute 😄

@ahmedsilat44, I actually realized I wrote « Feature Request » as a title but the contents of this issue is related to creating a « Pull Request » template.
Anyway, I am very happy with #39 ans I will merge it soon :)
If you want, you can create an issue template (bug report), see #35 or you can also now create a « Pull Request » template, see https://docs.github.com/en/communities/using-templates-to-encourage-useful-issues-and-pull-requests/creating-a-pull-request-template-for-your-repository

Nice, thanks @ahmedsilat44!


Bounding Box (BoundsControl) handle scaling for the large objects

Describe the bug
*Reported by Julian Jimenez<EMAIL_ADDRESS>Bounds Control's affordances do not scale up with the object size. Instancing an object with a big scale makes it very difficult to manipulate since Bounds Control's affordances are too small.

Suggested behavior update:
HoloLens2's shell shows properly scaled up handles with grabbable size when the object becomes very big.
You can try this by launching an app with 3D icon (e.g. Tips app) and scale it up with handles.

We appreciate your feedback and thank you for reporting this issue.
Microsoft Mixed Reality Toolkit version 2 (MRTK2) is currently in limited support. This means that Microsoft is only fixing high priority security issues. Unfortunately, this issue does not meet the necessary priority and will be closed. If you strongly feel that this issue deserves more attention, please open a new issue and explain why it is important.
Microsoft recommends that all new HoloLens 2 Unity applications use MRTK3 instead of MRTK2.
Please note that MRTK3 was released in August 2023. It features an all-new architecture for developing rich mixed reality experiences and has a minimum requirement of Unity 2021.3 LTS. For more information about MRTK3, please visit https://www.mixedrealitytoolkit.org.
Thank you for your continued support of the Mixed Reality Toolkit!


Access to keychain?

Why does this build need access to the keychain?
Hardware Overview:
Model Name:	MacBook Pro
Model Identifier:	MacBookPro12,1
Processor Name:	Intel Core i7
Processor Speed:	3.1 GHz
Number of Processors:	1
Total Number of Cores:	2
L2 Cache (per Core):	256 KB
L3 Cache:	4 MB
Memory:	16 GB
Boot ROM Version:	MBP121.0167.B24
SMC Version (system):	2.28f7

I don't change anything related to that. Chromium uses the keychain to store passwords from websites. I don't know if there's any way to disable that, but your best bet is a GN flag.
Let me know if something I said doesn't make sense.


Deprecate and remove server specific thrift max message size properties

In #3737 I realized that:

The client code uses a max message size of Integer.MAX_VALUE
The server code may use a specific property for max message size, that falls back to the general max message size property if it doesn't exist. However, since the server specific properties have default values, they exist in the configuration.
The max message size is enforced on READ and thrift allows you to write data larger than max message size. So, if a client sends a large mutation to the server, then it will fail on the server. Likewise, if the server sends a large mutation to the client, it will blow up on the client.

The user configuration for this is too confusing, I think it would be good to remove all of the server specific max message size properties and just use one with a large default value. We should consider whether or not the client should continue to use Integer.MAX_VALUE or whether it should use this new single property value so that the client and server are the same.

There might be downsides, but I'm leaning towards just making them accept up to Integer.MAX_VALUE and get rid of all of the config for this. I think the constraint might be hurting more than helping in any way.


examples/updates.go fix

fix examples/updates.go
resolve #40

@shyaminayesh I'm really sorry for forgetting this, would you mind rebasing it on master?

@Jguer I did pull latest changes from upstream and updated the PR.

Thanks @shyaminayesh , sorry for such a long wait


fixfix replace_column_data semantic

What problem does this PR solve?
Issue Number: close #xxx
Related PR: #xxx
Problem Summary:
replace_column_data is only for column which is not variable_length column like column_vector...
set is_variable_length = true such as column_string|array|map|struct
should not impl this function, when in agg situation for replace_if_not_null : we should use insert_from!
Release note
None
Check List (For Author)

Test

[x] Regression test
[ ] Unit Test
[ ] Manual test (add detailed scripts or steps below)
[ ] No need to test or manual test. Explain why:

[ ] This is a refactor/code format and no logic has been changed.
[ ] Previous test can cover this change.
[ ] No code files have been changed.
[ ] Other reason

Behavior changed:

[x] No.
[ ] Yes.

Does this need documentation?

[x] No.
[ ] Yes.

Check List (For Reviewer who merge this PR)

[ ] Confirm the release note
[ ] Confirm test cases
[ ] Confirm document
[ ] Add branch pick label

run buildall

run buildall

do not do it in branch-2.0


Remove vertex indices from Meshes

Appear to not be used or necessary anywhere. Also strictly increasing all the time.
@alexmillane

When we wrote the meshing code for OpenChisel, the meshes were originally being used in an opengl ES renderer on the device. When doing this its efficient to have a flat array of floats as the vertex positions / colors indexed by shorts. When we cut out the opengl renderer for the open source version and just started visualizing through RVIZ (which doesn't support indexing), we just passed around vertexes with the assumption that the index buffer is just increasing.

With the merge of #79 the vertex indices are no longer always just an increasing list. They now need to be kept so that mesh connectivity information can be made available.

Closing as they are now needed


Add an example for Node.js

This example is based off the FastAPI example with some differences:

The state example is using an ephemeral keypair (used to send message) instead of a counter
The "post a message on Aleph" and "retrieve http post data" are merged into a single example
There is no "cache"

The goal of the state example is to mute data in a file on disk inside a mutable volume. Why change its logic ?

How can we build and package this application ?

Not reviewed at all, this was a spam comment by a spam account @NostalgicGareth. He goes and spam a lot of projects like this.

Can we get confirmation that this still works as intended?
We talked about how we should support more languages/frameworks in the VMs and also to improve examples/docs for deploying these. Merging this PR should help with that.


[frontend] feat: add edit button

Also made some aesthetic changes to the question list

Thanks for working on this! This is just a minor nit but I personally think it might be better to remove the colors on the buttons in the question list as it feels a little too overwhelming (too much cognitive overload, attention goes to the button column instead of the 'more important' data)- this might be a personal preference though, so feel free to disagree! :)

No color
Color

Thanks for working on this! This is just a minor nit but I personally think it might be better to remove the colors on the buttons in the question list as it feels a little too overwhelming (too much cognitive overload, attention goes to the button column instead of the 'more important' data)- this might be a personal preference though, so feel free to disagree! :)
No color	Color

we shd also consider if the delete button (a bin icon) is beside, it’s abit too many things in the table. should we make the edit in the individual question page or it’s okay to have three buttons in each row

Changed button colours to gray, except arrow which is blue

@chloeelim ^what do you think abt the colors 🤔


Restore program options for RViz

closes #223
This pull request restores the program options that can currently be used.
In particular, the Ogre output is now hidden by default and can be enabled via the corresponding option.

@wjwwood, the instability of the Windows build seems to be unrelated to our changes.

New CI after rebase:

Linux
Linux-aarch64
macOS
Windows

New CI after rebase:

Linux
Linux-aarch64
macOS
Windows

@wjwwood, thank you for the comments. We implemented your requested changes and rebased on the new master.
New CI:

Linux
Linux-aarch64
macOS
Windows

I restarted CI for you (it looks like a disconnect on our end):

MacOS:


List plugin on Gatsby's website

Community plugins needs the "gatsby-plugin" keyword to show up in the search.
See https://github.com/gatsbyjs/gatsby/issues/4394#issuecomment-371659310

Thank you very much, Frank!


Retrieve commit hash

Would it be fruitful to return the commit of files that we've downloaded?
I understand I can manually make a timestamp of when I grab the files, however, I'd like the commit hash of the files I've just retrieved.
I would periodically check against the hash (rather than redownload everything) to ensure I am up-to-date.

I like this idea. I don't pay much attention to the Github API anymore, do you know of an easy way to achieve this?

https://api.github.com/repos/:owner/:repo/commits/master
.sha is the commit hash.
I can see either making another call .getCommit() or returning it from the callback.


bn_nist: fix strict aliasing problem (1.1.1 backport)

As of clang-14 the strict aliasing is causing code to magically disappear.
By explicitly inlining the code, the aliasing problem evaporates.
Fixes #18225
Backport of #18258 to 1.1.1.

This pull request is ready to merge

Merged to 1.1.1. Thank you.


glibc: fix cross builds on Darwin

When cross-compiling glibc, the postInstall attribute will use the build-platform version of localedef to generate locale information. This does not work when cross-compiling glibc on Darwin machines, because evaluating buildPackages.glibc will correctly lead to an unsupported platform error. Darwin machines cannot host glibc and thus cannot run the binaries.
My suggested fix ensures that the postInstall snippet which uses buildPackages.glibc is only included, when the build platform is Linux. On other platforms, the locale information will not be generated. This appears to be no problem in my cross compilation tool chain.
Things done

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

[ ] NixOS
[x] macOS
[ ] other Linux distributions

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

Should be ok to do so that for now. Eventually however we want to depend on LC_LANG=C.UTF-8 in more and more places, also we probably will start at the build time in which case the macOS stdenv would provide the locale. Long-term it might be possible to compile a cross-platform localedef as a proper fix: http://cgit.openembedded.org/openembedded-core/tree/meta/recipes-core/glibc/cross-localedef-native_2.29.bb?h=master

Thanks for your thoughts on the future goals, I will keep this in mind for later cross-toolchain work.
Also, I’ll just leave this here as a reference for later: macOS natively ships with a /usr/bin/localedef tool. Its arguments are not fully compatible with glibc’s version, but from the manpage, it appears to fulfil a similar purpose. But it may be the better option to properly cross-compile glibc’s own localedef.

This should be merged in release-19.03 as well; see #

Cheryy picked in 3924b344254350e4bee875a3353c1093e325743f


[3pt] PR: Docker, python packages and jdk updates

This PR rolls up a bunch of other PR's and python packages requests including:

Issue 1208  Bump OpenJDK from 17.0.8 to 17.0.10 (via updating to JDK 21.0.3)
PR 1207 - Dependabot bump certifi from 2023.7.22 to 2024.7.4
PR 1192 - Dependabot Bump urllib3 from 1.26.18 to 1.26.19
Updates required from ongoing PR 1206 - Probabilistic Flood Inundation Mapping. These updates make it easier for that branch/task to continue forward and staying in sync with dev. This triggered a few other packages that needed to be updated.

Other tasks included are:

Removing the now heavily obsolete unit_test system, including the package pytest. This included some changes to the CONTRIBUTING.md document.
Clean of a couple of packages no longer in use: pluggy and iniconfig
Removal of a deprecated file named config/aws_s3_put_fim3_hydrovis_whitelist.lst
Removed duration stamps around a few parts in fim_post_processing.sh
Fixes and updates to linting files. e.g. pyproject.toml. (line length was not working correctly)

Changes

Dockerfile, Pipfile, Pipfile.lock: as describe above for python package changes
.gitignore, CONTRIBUTING.md: File changes related to removing the unit_test system.
fim_post_processing.sh: noted above.
pyproject.toml: fixes and updates for linting

Removals

unit_tests folder and all related files under it. Appx 25 to 30 files removed.

Testing

A number of tests were done with various combinations using the new docker file and build testing.
Inside the new docker container, a fim pipeline run against 12040101 was done as well as running alpha testing on it.
The new open JDK updates  (WILL BE) tested in OWP servers to ensure it now passes vulnerability tests.

Deployment Plan (For developer use)
How does the changes affect the product?

[ ] Code only?
[ ] Require new or adjusted data inputs? Does it have start, end and duration code (in UTC)?
[ ] If new or updated data sets, has the FIM code been updated and tested with the new/adjusted data (subset is fine, but must be a subset of the new data)?
[ ] Require new pre-clip set?
[x] Has new or updated python packages?
[x] If applicable, has a deployment plan be created with the deployment person/team?

Issuer Checklist (For developer use)
You may update this checklist before and/or after creating the PR. If you're unsure about any of them, please ask, we're here to help! These items are what we are going to look for before merging your code.

[x] Informative and human-readable title, using the format: [_pt] PR: <description>
[x] Links are provided if this PR resolves an issue, or depends on another other PR
[x] If submitting a PR to the dev branch (the default branch), you have a descriptive Feature Branch name using the format: dev-<description-of-change> (e.g. dev-revise-levee-masking)
[ ] Changes are limited to a single goal (no scope creep)
[ ] (pending)  The feature branch you're submitting as a PR is up to date (merged) with the latest dev branch
[x] pre-commit hooks were run locally
[x] Any change in functionality is tested
[ ] n/a Passes all unit tests locally (inside interactive Docker container, at /foss_fim/, run: pytest unit_tests/)
[x] New functions are documented (with a description, list of inputs, and expected output)
[x] Placeholder code is flagged / future todos are captured in comments
[x] CHANGELOG updated with template version number, e.g. 4.x.x.x
[ ] Reviewers requested
[x] Add yourself as an assignee in the PR  as well as the FIM Technical Lead

Merge Checklist (For Technical Lead use only)

[ ] Update CHANGELOG with latest version number and merge date
[ ] Update the Citation.cff file to reflect the latest version number in the CHANGELOG
[ ] If applicable, update README with major alterations

@mluck When you test this PR, keep this image. I have another small PR coming that I would like tested against this image so both can be merged at once. (if that is ok)

I built the image from Dockerfile and ran fim_pipeline.sh and synthesize_test_cases.py. All scripts completed successfully and CSI was identical to <IP_ADDRESS>. However, there were a number of messages, copied below.
It looks like Whitebox is still trying to download the executable, unless these are just being printed to the screen.
Creating AGREE DEM using 70 meter buffer 03100204 0
Downloading WhiteboxTools pre-compiled binary for first time use ...
Downloading WhiteboxTools binary from https://www.whiteboxgeo.com/WBT_Linux/WhiteboxTools_linux_amd64.zip
Decompressing WhiteboxTools_linux_amd64.zip ...
WhiteboxTools package directory: /usr/local/lib/python3.10/dist-packages/whitebox
Downloading testdata ...

A warning:
root@4fc02f9ac92c:/# /foss_fim/tools/synthesize_test_cases.py -c DEV -jh 1 -jb 6 -vr -vg -o -b nws -v dev-depbot-urllib
/usr/local/lib/python3.10/dist-packages/dask/dataframe/__init__.py:49: FutureWarning:
Dask dataframe query planning is disabled because dask-expr is not installed.

You can install it with `pip install dask[dataframe]` or `conda install dask`.
This will raise in a future version.

  warnings.warn(msg, FutureWarning)
================================

A lot of these warnings:
/foss_fim/tools/overlapping_inundation.py:399: RuntimeWarning: invalid value encountered in cast
  nan_tile = np.array([np.nan]).astype(dtype)[0]

A couple of these warnings:
/usr/local/lib/python3.10/dist-packages/gval/comparison/agreement.py:134: FutureWarning: It is recommended to use 'rio.write_crs()' instead. 'rio.set_crs()' will likelybe removed in a future release.
  agreement_map.rio.set_crs(crs, inplace=True)

I built the image from Dockerfile and ran fim_pipeline.sh and synthesize_test_cases.py. All scripts completed successfully and CSI was identical to v<IP_ADDRESS>. However, there were a number of messages, copied below.
It looks like Whitebox is still trying to download the executable, unless these are just being printed to the screen.
Creating AGREE DEM using 70 meter buffer 03100204 0
Downloading WhiteboxTools pre-compiled binary for first time use ...
Downloading WhiteboxTools binary from https://www.whiteboxgeo.com/WBT_Linux/WhiteboxTools_linux_amd64.zip
Decompressing WhiteboxTools_linux_amd64.zip ...
WhiteboxTools package directory: /usr/local/lib/python3.10/dist-packages/whitebox
Downloading testdata ...

A warning:
root@4fc02f9ac92c:/# /foss_fim/tools/synthesize_test_cases.py -c DEV -jh 1 -jb 6 -vr -vg -o -b nws -v dev-depbot-urllib
/usr/local/lib/python3.10/dist-packages/dask/dataframe/__init__.py:49: FutureWarning:
Dask dataframe query planning is disabled because dask-expr is not installed.

You can install it with `pip install dask[dataframe]` or `conda install dask`.
This will raise in a future version.

  warnings.warn(msg, FutureWarning)
================================

A lot of these warnings:
/foss_fim/tools/overlapping_inundation.py:399: RuntimeWarning: invalid value encountered in cast
  nan_tile = np.array([np.nan]).astype(dtype)[0]

A couple of these warnings:
/usr/local/lib/python3.10/dist-packages/gval/comparison/agreement.py:134: FutureWarning: It is recommended to use 'rio.write_crs()' instead. 'rio.set_crs()' will likelybe removed in a future release.
  agreement_map.rio.set_crs(crs, inplace=True)

for Whitebox:  yes.. we will have to defer that one for now. There is a lot to that one. I updated the Issue card 1209  for the details so we can come back to it.

Dask framework: The fix is easy but requires some discussion. I would prefer to leave it for now.

Yes.. there are a wild amount of new warnings;
a) Numpy: We are going to see them absolutey everywhere and a huge amount
b) Synthesize_test_cases.py:  Issue 1228. I am going to have this fixed and merged with this PR as it creates so many warnings onscreen that we can't monitor any real issues.
c) There are a very, very large number of other warnings mostly fitting into just a few fixes that already exist prior to this PR. All similar in nature to the numpy warning. See 1201.
b) I am recommending taking the next step of upgrading other key packages like panda, rasterio, fiona, etc, see issue 1227. We can consider fixing more of the warnings at that point.

But.. there are so many warnings, we may want to cleanup some of the current warnings (pre this release) and new warning post this release), only in the name of sheer volume,
I am open to suggestions.
The biggest driver on getting this out, is that both ITSG and depbot are wanting this merge sooner than later.


reset: adjust "lost all my devices" button

Issue: Y2K-161
CC @keybase/y2ksquad
CC @cecileboucheron

I think one line-break is enough no?

Also: copmuter

fixed comments from danny and cecile.


artalk: init at 2.8.5

Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

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

1 package built:

artalk

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

1 package built:

artalk

@ofborg test artalk

@ofborg test artalk

@ofborg test artalk

Due to the frontend of Artalk is built by pnpm which is not reproducible enough for nix right now, I’d like to request a separate frontend release from upstream.

@ofborg test artalk


Robocopy based download related fixes

Robocopy based download related fixes

@SunnyKumarR,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

is this same as #1008 ?

Yes it is same. Tried it to see whether test failure was due to temporary issue. Closing it now.


update the Larmor code with evolution plot of the frequency

Feature added in this commit.

Feature added in this commit.


CMD ERROR: Access violation: Read [0000000000000030] in C:/projects/kyty/source/emulator/src/Loader/RuntimeLinker.cpp:221

have you found a fix?

Nope. I hope Inori gets back to work...

Could not find suitable device in C:/projects/kyty/source/emulator/src/Graphics/Window.cpp:2198


[Question] Bastion Host in API response with SKU of Premium? This shouldn't be possible, is this a valid SKU?

API Spec link
https://github.com/Azure/azure-rest-api-specs/blob/main/specification/network/resource-manager/Microsoft.Network/stable/2021-03-01/bastionHost.json
API Spec version
2021-03-01
Question/Query
When I query Bastion Host using the Resource Graph API I am seeing an SKU of Premium on a Bastion Host. This SKU shouldn't be possible. From all docs and specs I can only find "Developer, Standard and Basic". This resource looks very similar to the "Standard" resource tier. Can anyone shed any light on why the SKU would be "Premium" here?
Environment
No response

@dmartin-r7  will you be able to reach out with your bastion details through a support ticket. We would like to check it further. Thanks


Unknown layer type: Warping

I got the latest version caffe and set WITH_PYTHON_LAYER := 1 in makefile.config. However, I still get the following error:
===
F0325 16:49:50.756420  6840 layer_factory.hpp:81] Check failed: registry.count(type) == 1 (0 vs. 1) Unknown layer type: Warping (known types: AbsVal, Accuracy, ArgMax, BNLL, BatchNorm, BatchReindex, Bias, Clip, Concat, ContrastiveLoss, Convolution, Crop, Data, Deconvolution, Dropout, DummyData, ELU, Eltwise, Embed, EuclideanLoss, Exp, Filter, Flatten, HDF5Data, HDF5Output, HingeLoss, Im2col, ImageData, InfogainLoss, InnerProduct, Input, LRN, LSTM, LSTMUnit, Log, MVN, MemoryData, MultinomialLogisticLoss, PReLU, Parameter, Pooling, Power, Python, RNN, ReLU, Reduction, Reshape, SPP, Scale, Sigmoid, SigmoidCrossEntropyLoss, Silence, Slice, Softmax, SoftmaxWithLoss, Split, Swish, TanH, Threshold, Tile, WindowData)
===
Is there anything I missed?

You have to change the proto file and add the warping layer before recompiling the caffe.

Appreciate it. I will try it.


Dependabot Email re: add issue to project failure when filing a ticket in dependabot-core?

Is there an existing issue for this?

[X] I have searched the existing issues

Package ecosystem
n/a
Package manager version
n/a
Language version
n/a
Manifest location and content before the Dependabot update
n/a
dependabot.yml content
n/a
Updated dependency
n/a
What you expected to see, versus what you actually saw
I filed a ticket in this repo for support  on dependabot but got a failure email, linking to https://github.com/dependabot/dependabot-core/actions/runs/9664272755
I don't think it's related to me, should this email/failure even go to bug filers? anything I need to do?

Native package manager behavior
No response
Images of the diff or a link to the PR, issue, or logs
No response
Smallest manifest that reproduces the issue
No response

Got same failure email when filing this issue

Thanks for reporting that, it should be fixed now!


Wrong indentation def__str__, page 54

@leifos
Page 54
def str has too much indentation.
You need to deindent the block of
def str
Otherwise after the population script you can see that name of categories are showing up as objects.
Cheers.

Thanks!


[Improvement] reading excel file from base64 should allow ignoring data uri information

When reading a base64 file, it would be nice if XLSX would ignore the leading data URI parameters.
So it can treat
data:text/csv;base64,TmFtZXMNCkhhZmV6DQpTYW0NCg==
the same way it treats
TmFtZXMNCkhhZmV6DQpTYW0NCg==
const workBook = XLSX.read(sheetInBase64, { type: 'base64' });
// workBook should be the same whether `sheetInBase64` starts with a dataURI scheme or without it.

I would like to create a PR with that fix, but the tests file seem to be automatically generated, and when I run npm test I simply get bits/some_file.js in the log, with no output telling me whether the tests passed or failed.

https://docs.sheetjs.com/docs/miscellany/contributing#os-specific-setup and https://docs.sheetjs.com/docs/miscellany/testing should cover how to get started.  If this does not work, please leave a note and we'll try to redo it.
To run the tests manually, use make test.  Under the hood it actually runs mocha -R spec -t 30000.  The smaller test can be run with env WTF=1 make test_misc
RFC2397 covers the full set of data URI options.  Should the parser handle data that is not base64-encoded?

Thanks. I can see that we have .ts and .js files in the bits directories.
Having both can always bring up the question as to which one we should modify.
Is there a reason to not only have the .ts files and compile the .js ones out of them?
Also, a lot of the code feels to be automatically generated (one letter variables), with unusual styling, which is quite limiting in terms of contribution. Is there a reason behind that?

(answering the question posed in the last comment)
This is a very old project.  The first commits were in 2012, around the time that TypeScript was publicly released.
One of the primary goals is preserving compatibility with a wide array of JavaScript engines and runtimes.  In 2022, PhotoShop and other Adobe products ship with ExtendScript, a runtime based on ES3 that is incompatible with modern JavaScript.  In 2022, NetSuite still uses Rhino, a JS engine based on ES3.  We still test against IE10 to ensure compatibility.
This is not a goal shared by the rest of the community.  Countless projects that worked 6 months ago with modern tooling have broken in subtle ways because of drastic API changes and deep dependencies not properly pinning versions.
With regards to the short variable names, we've more or less had to abandon tooling for Photoshop.  There are a number of tracking issues like https://github.com/mishoo/UglifyJS/issues/1144#issuecomment-227015448 and https://github.com/mishoo/UglifyJS/issues/1930#issuecomment-301225514 where other ecosystem developers decided that PhotoShop is not an important target.  We were forced to adapt by opting for short variable names.
.
The main scripts are built with a cat step to ensure that we have complete control over the final scripts.  Every line in xlsx.extendscript.js maps to a line in the source that we can directly observe and edit.
So when we explored modernizing the codebase, we wanted to incrementally shift over to TypeScript.  esbuild presented a pathway to replacing individual files in the bits folder.  For base64 the sequence is

run make 04_base64.js from the modules directory.  That creates modules/04_base64.js

run make from the base directory.  This copies modules/04_base64.js to bits/04_base64.js, inserting the file in the correct spot of the output.

We hope to eventually port everything over, but there is quite a bit of prototype manipulation and other tricks that need to be re-worked.
.
The actual docs site is built from https://github.com/SheetJS/docs.sheetjs.com/blob/master/docz/docs/09-miscellany/04-testing.md and https://github.com/SheetJS/docs.sheetjs.com/blob/master/docz/docs/09-miscellany/05-contributing.md -- please let us know how to improve those (and you can send a PR)

I understand that having to support older environments can make the development experience a bit challenging.
I'm hoping I'll find some time to fix the challenges I faced while submitting a simple fix, and incrementally put such a nice project in a nicer state.
Thanks a lot for all the hard work. 👍


Port the build system to Robo

Coverage remained the same at 69.43% when pulling 981b8cea381043f7774f8d78b2d9d8f1e0d4dff1 on feature/build_system into 90294546ca51aac5bc079337602de0aa095f466c on develop.