Not sure if me, or they are getting hammered for the link for NPSSO

I get a HTTP 429 Too many requests error.   I do have the ps5_mqtt addon which never really did much for me and I can remove it, but not sure if that's it.  Also have two mobile phones logged into the account.  During your development of this or testing encounter the same error?

When I did get this error it was due to me not being logged into my PlayStation account in the same session when navigating to the npsso url. Are you sure you're logged in?

I made sure I was logged in, but got to thinking based on your reply, and logged out, cleared my cookies for the site, and then logged in and was able to get it to work, so it must have been a stale cookie from somewhere.

I had this issue yesterday also, seemed to be a login issue for me based on region, the link provided to login redirected me to the en-US site, once I figured that out, I switched to the en-GB site logged in at it worked instantly.

 I would like to include it in the docs for others.

@xXREDXIIIIXx, do you happen to have the link handy? I would like to include it in the docs for others.

Sure, The link I used is https://www.playstation.com/en-gb/
Needs testing but in my case I am from the UK and that is the the account login I registered with so it might just be that those from other countries just need to use their own region for the playstation.com login.
That being said... I just tested this again now, and I also hit the HTTP 429 Too many requests. so maybe I just got lucky? (but it did work the 1st time after changing the region) maybe you can check for yourself see if you also get the error to confirm?

Same strategy shared by @xXREDXIIIIXx worked for me. The steps I did:
1 - I logged out of my account
2 - Move to my region endpoint (https://www.playstation.com/pt-pt/)
3 - I logged in again
After that, the 429 error stopped and I was able to get the npsso token.

I also was logged out a couple of hours ago. Seems like it may have been something on Sony's end.
I've updated the readme to hopefully help people through some of these problems. Thanks guys!


graphql-mongodb-loader

I am using mongodb native driver. Would you mind creating graphql-mongodb-loader as well? Thanks!

I think this would go in another package
feel free to start it and we can help as we can


(doc): Initial commit for Connector Kit and edc version 0.5.x

Description
This is the initial commit/preperation für the Connector Kit which represents the future version of the edc 0.5.x

preperation for documentation 0.5.x edc
cleanup files (deleted unused files)
generated new version of the api
md linter (adapted broken files -> also outside the Connector Kit)

Information

the changes within this PR will result in the "Next" section which is the future 3.2 section
This PR closes #241

The Changelog got no updates. I think there were some version bumps. If these are not revelvant this is change request is obsolete.

Hey @LuisRickert the changes are relevant ;) changelog will be updated with another PR. This here is quite big and I just wanted to be fast :) I have to consolidate with the developers about the changes. And since the changes are on "Next" it should be OK for now. Just to get it merged.

And now it's updated :)


Example!!

I can have an example of Gamma tone gram generation written in Python?

It depends on what exactly you need. In the readme file it shows how you can do it from the command line with:
python -m gammatone FurElise.wav -d 10

All this does is call render_audio_from_file. Does that answer your question?

Thank you! But in this case in the method "render_audio_from_file" the "function" parameter is just:
-> function = gammatone.gtgram.gtgram ?
Is there any parameter that escapes me?

-> function = gammatone.gtgram.gtgram ?

This parameter is the function used to create the gammatone spectrogram, it can be either:

gammatone.gtgram.gtgram - the slower but more accurate method that uses a full gammatone filterbank
gammatone.fftweight.fft_gtgram - the faster method (but less precise) that uses weights applied to an FFT

The other parameters are the path to the .wav file and the time in seconds from the start you want to render until.


[dev] Error while install requirements on OpenShift

Expected Behavior
Actual Behavior
Obtaining pgoapi from git+https://github.com/tejado/pgoapi.git@81e786cabf027a1c8fbd1e9a07e1c11aa3d8ee8b#egg=pgoapi (from -r requirements.txt (line 1))
Updating ./src/pgoapi clone (to 81e786cabf027a1c8fbd1e9a07e1c11aa3d8ee8b)
Could not find a tag or branch '81e786cabf027a1c8fbd1e9a07e1c11aa3d8ee8b', assuming commit.
Running setup.py egg_info for package pgoapi
Traceback (most recent call last):
File "", line 16, in
File "/var/lib/openshift/579a11ed2d527124a600013d/app-deployments/2016-07-28_10-10-25.478/repo/src/pgoapi/setup.py", line 9, in
install_reqs = parse_requirements(path_req, session=False)
TypeError: parse_requirements() got an unexpected keyword argument 'session'
Complete output from command python setup.py egg_info:
Traceback (most recent call last):
File "", line 16, in
File "/var/lib/openshift/579a11ed2d527124a600013d/app-deployments/2016-07-28_10-10-25.478/repo/src/pgoapi/setup.py", line 9, in
install_reqs = parse_requirements(path_req, session=False)

TypeError: parse_requirements() got an unexpected keyword argument 'session'

Cleaning up...
Command python setup.py egg_info failed with error code 1 in /var/lib/openshift/579a11ed2d527124a600013d/app-deployments/2016-07-28_10-10-25.478/repo/src/pgoapi
Steps to Reproduce
pip install -r requirements.txt
Other Information
OS: OpenShift

pip --version?
probably old pip version, try pip install --upgrade pip first.

@kriwil yes, its works sir..... terima kasih.
upgraded pip 7 to 8


How to safely restart/pullout a triton server in prod env?

When running on prod with quite a lot payload, we need to update a model. For some resource limitation like memoery/gpu memory, we have to pull the serer out and switch model version and restart it.
How can we wait till that all requests in the server have been handled plz? is that any api like shutdown?

If you send Triton a sighup it will wait for all inflight inference requests to complete before shutting down. You can see this in the console output. Run perf_client against triton so that there are in-flight inferences and then try to stop triton... you will see it waiting for the inflight inferences to complete. By default triton will wait 30 seconds for inflight infrences to complete before forcefully exiting, but there is a command-line arg to adjust that.


Error on NodeJS v8.1.4

/home/polin/.node-gyp/8.1.4/include/node/v8config.h:329:3: note: in definition of macro ‘V8_DEPRECATED’
   declarator __attribute__((deprecated(message)))
   ^
lwip_decoder.target.mk:168: recipe for target 'Release/obj.target/lwip_decoder/src/decoder/init.o' failed
make: *** [Release/obj.target/lwip_decoder/src/decoder/init.o] Error 1
make: Leaving directory '/home/polin/.nvm/versions/node/v8.1.4/lib/node_modules/android-splash-generate/node_modules/lwip/build'
gyp ERR! build error
gyp ERR! stack Error: `make` failed with exit code: 2
gyp ERR! stack     at ChildProcess.onExit (/home/polin/.nvm/versions/node/v8.1.4/lib/node_modules/npm/node_modules/node-gyp/lib/build.js:258:23)
gyp ERR! stack     at emitTwo (events.js:125:13)
gyp ERR! stack     at ChildProcess.emit (events.js:213:7)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:197:12)
gyp ERR! System Linux 4.4.0-83-generic
gyp ERR! command "/home/polin/.nvm/versions/node/v8.1.4/bin/node" "/home/polin/.nvm/versions/node/v8.1.4/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /home/polin/.nvm/versions/node/v8.1.4/lib/node_modules/android-splash-generate/node_modules/lwip
gyp ERR! node -v v8.1.4
gyp ERR! node-gyp -v v3.6.2
gyp ERR! not ok
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>install: `node-gyp rebuild`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/polin/.npm/_logs/2017-07-21T02_02_31_091Z-debug.log
➜  ~

@paulomcnally Thank you for reporting this problem. There is an installation problem in the underlying lwip dependency. The issue is being tracking in the lwip repo here. Once that issue is resolved, we can update this package and hopefully solve the problem you reported.


receiving vague error and stack trace running amplify build in an uninitialized project

How did you install the Amplify CLI?
pnpm
If applicable, what version of Node.js are you using?
18
Amplify CLI Version
10.6.0
What operating system are you using?
mac
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
n/a
Describe the bug
Running amplify build in an uninitialized project causes vague error and stack trace looking for amplify-meta.json
➜  amplify build
🛑 Error: File at path: '/Users/josef/Documents/projects/aws-amplify/tmp/docs/amplify/backend/amplify-meta.json' does not exist
    at Function.readJson (/snapshot/repo/build/node_modules/amplify-cli-core/lib/jsonUtilities.js:40:19)
    at StateManager.getData (/snapshot/repo/build/node_modules/amplify-cli-core/lib/state-manager/stateManager.js:323:56)
    at StateManager.getMeta (/snapshot/repo/build/node_modules/amplify-cli-core/lib/state-manager/stateManager.js:42:25)
    at getAllResources (/snapshot/repo/build/node_modules/@aws-amplify/amplify-category-custom/lib/utils/dependency-management-utils.js:88:50)
    at generateDependentResourcesType (/snapshot/repo/build/node_modules/@aws-amplify/amplify-category-custom/lib/utils/build-custom-resources.js:60:198)
    at Object.run (/snapshot/repo/build/node_modules/@aws-amplify/cli-internal/lib/commands/build.js:14:76)
    at Object.executeAmplifyCommand (/snapshot/repo/build/node_modules/@aws-amplify/cli-internal/lib/index.js:179:29)
    at processTicksAndRejections (internal/process/task_queues.js:95:5)
    at async executePluginModuleCommand (/snapshot/repo/build/node_modules/@aws-amplify/cli-internal/lib/execution-manager.js:142:5)
    at async executeCommand (/snapshot/repo/build/node_modules/@aws-amplify/cli-internal/lib/execution-manager.js:40:9)
    at async Object.run (/snapshot/repo/build/node_modules/@aws-amplify/cli-internal/lib/index.js:119:5)
🛑 There was an error building the resource

Expected behavior
CLI should print a helpful message
We've detected an Amplify project but it does not appear to be initialized. Run `amplify init` to get started

Reproduction steps

clone some Amplify project
run amplify build
observe error

Project Identifier
No response
Log output

# Put your logs below this line

Additional information
No response
Before submitting, please confirm:

[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

Was able to reproduce the issue. marking as bug.


cannot import name 'Unpack' from 'transformers.processing_utils'

The code was working fine till this morning but then it started throwing this error while running this line of code:
Code:
RAG = RAGMultiModalModel.from_pretrained("vidore/colpali-v1.2", verbose=1)
Error:
RuntimeError: Failed to import transformers.models.paligemma.processing_paligemma because of the following error (look up to see its traceback):
cannot import name 'Unpack' from 'transformers.processing_utils' (/usr/local/lib/python3.10/dist-packages/transformers/processing_utils.py)

This should be fixed in the most recent release (on pypi in a few minutes), likely due to an old version of transformers -- we now enforce at least 4.42


Cosmic Rays Rejection should be default

We decided to finally leave it as it is. I'm closing the issue but we can re-open it if we want to reconsider


Rename UUID.new()

Since new is a reserved keyword in JS I'd recommend renaming the method to create() or generate().
I think I'd prefer
var uuid = UUID.create();

Furthermore I suggest to get rid of newTS() and allow creating uuid's of different versions with the same command like so:
var uuidv4 = UUID.create();
var uuidv1 = UUID.create(1);

so the signature of the create method would become
UUID.create([version = 4])

I suggest to default the version to 4 since v4 UUIDs are more widely used than v1 UUIDs.

Sure we could keep the current names as aliases to support legacy code and maybe just leave them undocumented.
I'll send a suggestion as a pull request.

Sent pull request #5


openOrJoin Failure - v3.4.4

openOrJoin function not working as expected.
RTCMultipleConnection.js - v3.4.4
Open and join are working separately
Note: checkPresence not working as well.

openOrJoin in this demo is working for me:

https://rtcmulticonnection.herokuapp.com/demos/Video-Conferencing.html

checkPresence demo is also working for me:

https://rtcmulticonnection.herokuapp.com/demos/checkPresence.html

Please reopen if this issue persists.


Obfuscating dll of an asp.net mvc web application is not working.

When I obfuscate the dll of a asp.net mvc web application. It obfuscates successfully. however after obfuscating if I run the application it gives 403.14 forbidden error.

Framework like ASP.NET MVC rely on naming conventions to locate controllers and views. You have to exclude such classes and properties out of obfuscation manually.

If I obfuscate only one class and the object of this class is created in controller. Will that break the flow.

It is quite common to exclude some classes or class members, so usually there should be no issue. Besides, why do you have to obfuscate an ASP.NET MVC app? Such server side applications usually do not expose assemblies to end users and obfuscation is usually unnecessary.

The server on which the code will be is on client side.


4.8 is missing

The Release value found in the registry is 528040 for the Version 4.8.03752. I'm guessing the lower boundary for 4.8 in general would be 520000?

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

ID: 2115da38-e95e-46fc-1e2d-da88ef92c6cf
Version Independent ID: a3790e8a-0654-851b-c991-ab7382025406
Content: How to: Determine which .NET Framework versions are installed
Content Source: docs/framework/migration-guide/how-to-determine-which-versions-are-installed.md
Product: dotnet-framework
Technology: dotnet-appcompat
GitHub Login: @rpetrusha
Microsoft Alias: ronpet

We’re updating the articles for 4.8 now.
According to https://github.com/dotnet/samples/pull/789/files, the value will be 528049.

The lower boundary will be 528040.


Fix the markdown formatting of testnet.md

fix the formatting issue, fix the table and header format

I'm preparing a testnet-related documentation commit including these changes, the changes raised by #667 and some of my own edits. This will be committed later today. Thank you for the suggestions.


[html] mirror cursor: pasting a full line will copy the full line twice

Issue Type: Bug
If I copy a full line with CTRL+C (my cursor range is empty.)
Then position my cursor on top of a starting tag, it will paste the line above the opening tag and the closing tag.

   Something

Copy a line like
HELLO

Position your cursor on the opening

Press CTRL+V.
 The extra cursor is added because of html.mirrorCursorOnMatchingTag setting. You can disable it for now if you don't like it.

In tomorrow's insiders html.mirrorCursorOnMatchingTag is replaced with editor.renameOnType. The new implementation fixes this issue. You can find more details here: https://github.com/microsoft/vscode/issues/88424#issuecomment-600840619


added maxlength in redis xadd

ISSUE

redis memory was increasing continuously and there were no measures to clean it up once it was not required

FIX

added max length parameter which takes number of maximum entries in the redis stream. It will act like a FIFO queue so whenever there is message and queue is full it will evict the first entry.

Memory stats

It was taking around 200mb for 50k entries.

@sankethkini Thanks. I will update the Gorush repo.


User can't click "site thumbnail" tab while Figgy is open in another tab in Firefox

Reported by @roelmoon:

I’m encountering that weirdness with figgy and DPUL in tabs. I’m logged into both. I’m trying to look at the “site thumbnail” so I can download the tiff from Figgy but DPUL dashboard won’t let me switch from “site masthead” while in configuration/appearance. Works fine in separate windows. Using Mac/Firefox.

I was unable to replicate this bug; it seems as though it may have been resolved. Please reopen and let me know if it occurs again; I'd like to see what this looks like.


Callouts being displayed under LineString

Describe the bug
My callouts are getting displayed under the LineString.. not sure how to set the Z cords. In addition I would like to have them displayed where the user tapped... not sure how to adjust that.

To Reproduce
My source code for this:

                    var g = feature.Geometry as NetTopologySuite.Geometries.MultiLineString;
                    for(var i = 0; i < g.Count(); i++)
                    {

                        if(i == 0)
                            rgb = ColorConverters.FromHex(feature.Attributes["color"] as string);
                        else
                            rgb = ColorConverters.FromHex(feature.Attributes["color2"] as string);

                        var c = new Mapsui.Styles.Color(rgb.R, rgb.G, rgb.B);

                        var cords = g.Geometries[i];

                        var lineString = new LineString(cords.Coordinates.Select(v => SphericalMercator.FromLonLat(v.X, v.Y).ToCoordinate()).ToArray());
                        var f = new GeometryFeature { Geometry = lineString };

                        var desc = feature.Attributes.GetOptionalValue("desc");
                        var calloutStyle = CreateCalloutStyle(feature.Attributes["name"] as string, desc as string);

                        var v = new VectorStyle
                        {
                            Fill = null,
                            Outline = null,
                            Line = { Color = c, Width = 5 }
                        };

                        f.Styles.Add(calloutStyle);
                        f.Styles.Add(v);

                        features2.Add(f);
                    }

Creation of callout:
private static CalloutStyle CreateCalloutStyle(string name, string desc = null)
        {
            return new CalloutStyle
            {
                Type = string.IsNullOrWhiteSpace(desc) ? CalloutType.Single : CalloutType.Detail,
                Title = name,
                Subtitle = desc,
                SubtitleFont = { FontFamily = null, Size = 8, Italic = false, Bold = false },
                SubtitleFontColor = Mapsui.Styles.Color.Gray,
                TitleFont = { FontFamily = null, Size = 12, Italic = false, Bold = true },
                TitleFontColor = Mapsui.Styles.Color.Gray,
                MaxWidth = 140,
                RectRadius = 10,
                ShadowWidth = 4,
                Enabled = false,
                SymbolOffset = new Offset(0, SymbolStyle.DefaultHeight * 0.2f),
                Outline = new Pen(Mapsui.Styles.Color.Violet, 2)
            };
        }

Expected behavior
Should appear on top
Smartphone (please complete the following information):

Device: Emulators/SImulators
OS: iOS/Android
Version ios 16 and latest android

Additional context
Nothing that i can think of.

That is, because you add first the CalloutStyle and then the VectorStyle. Mapsui goes through the styles in the order they are added. So the CalloutStyle is drawn and then, above, the VectorStyle. Add it vice versa and it will drawn correct.

One more think. If you want to have the CalloutStyle on top, you should make the callout layer the top most layer.

I tried that as well @charlenni... I think the problem is maybe that the callout here actually belongs to the purple line... but is being drawn under it... i guess i could try to re-order things so that I add the top layers first so the callouts are near the bottom are always at the top...hmmmm
here for example in my image the callout belongs to a different one

let me try some different ordering

Ok, for that you should have a look, how the features are drawn in the draw process. You could find this here.
There is an outer loop over all layers. Inside of this, the first loop iterates through all styles the layer contains. For each of these styles, all features are drawn (if the conditions fulfilled). The second loop goes over all features and draws all the styles, that belong to this feature.
In your case, you added the styles (vector and callout) to the feature itself. With this, the first loop draws nothing (no style for the layer). The second loop draws feature 1 as vector and then as callout. After that, the second feature is drawn as vector (perhaps over callout from feature 1) and then as callout. As you see, you don't get, what you want.
So, if you want the callouts over all other features, you need a second layer: one layer for your line strings and one layer for the callouts (point features with callout style). Use a dictionary to connect your line string features with your point features, so that you could show and hide the callouts.

Thanks for the help and clarification on this! Super helpful.


[Metrics Explorer] Moving filters to dimension column from id

Using dimension column name in filters for the API. The UI internals will continue using IDs.
Using measure sql name in multiple places instead of id. The UI internals will continue using IDs.

Regarding the svelte-query/selectors/metrics-view.ts file, could we:

Use the native Svelte Query selector functionality. (It looks like the native selector functionality isn't described in the SvelteQuery docs, but it is in the ReactQuery v3 docs, so I think it's possible.)
Use the select-partial-subscriptions format recommended here: https://tkdodo.eu/blog/react-query-data-transformations#3-using-the-select-option
Colocate the selectors with the queries, as demonstrated in the link above. If the metrics-view.ts file is getting too big, perhaps we could split it into files per API call, e.g. meta.ts, timeseries.ts, totals.ts files. That way "the code that changes together, stays together".

What's the reasoning for the UI internals still using IDs not names? It looks like it adds complexity to work in IDs in one place (UI) and names in another (API).

A couple of readability suggestions:

The files in svelte-query/queries/metrics-views don't need to also include "metrics-views" in the name. It's a bit redundant and I think it'd be cleaner without. E.g. /metrics-views/metrics-views-meta.ts -> /metrics-views/meta.ts
The selectors in metrics-views-metadata.ts might be better named e.g. useMetaDimension rather than useDimensionFromMetaQuery. Then you can understand it left-to-right: "use" (svelte-query hook), "meta" (API endpoint), "dimension" (selected attribute).


Invalid color functions generated

Two of the colors that got generated for my project were invalid/didn't compile:
/// Color #FF0000 (alpha 217)
class func genFF0000 (alpha 217)() -> UIColor {
	return UIColor(named: "FF0000 (alpha 217)") ?? UIColor.white
}

/// Color #E00000 (alpha 217)
class func genE00000 (alpha 217)() -> UIColor {
	return UIColor(named: "E00000 (alpha 217)") ?? UIColor.white
}

Let me know what else would be helpful to debug

Thanks for raising that up, the generated name should probably be:
genFF0000_alpha_217
The color names are generated by proximity, so if there's two colors close to the same name, for instance:
#000000 -> Black
#000001 -> Black
Instead of the name of the color, the hex is used to avoid name collision. Also, if the alpha is less than 255, it's appended to the color name to avoid name collision.
Though, the correct should be:
FF0000 (alpha 217) on the Asset Catalog
and
genFF0000_alpha_217 on the generated file
Probably some wrong string replace : P
I will take a look, thanks again by reporting the first issue here 👍


Blacklist branches on travis

@franckcuny yup. the change is to verify the pull request pass the tests. I think we need a coverall token to report coverall reports. I am thinking of moving the coverall report to be part of the apache ci build. (https://issues.apache.org/jira/browse/INFRA-12549)

Got it. LGTM.


[SOLUTION] FAILED TO LOAD EXTENSION

Dear windows users.
When you are extracting a .zip file it's creating a new folder inside the original like this:

test.zip "unzip it"
test > test > content

Don't unload the outer folder, you have to unload the insider test folder
Follow Step by step:
1.Extract it correctly

2.Then drag that folder into the chrome extensions page

congrats!!! you did it 🥳🙌
but if you still face any problem feel free to comment

works even for mac users. i tested in brave.

unable to run in mac.

unable to run in mac.

@surbhi498 im using it on mac, it's working perfectly, can you open a new issue with screenshots and error masseges?
here: https://github.com/31b4/Leetcode-Premium-Bypass/issues/new/choose


Dialog: Focus shifts unexpectedly on opening

Description
I've made a Dialog that has 4 items you can choose from (ActionList items). When I open the dialog, it automatically highlights the last item you can choose.
There are multiple problems happening here.

There is a visual focus state when navigating by mouse
It focuses on the last ActionList item.

https://github.com/primer/react/assets/980622/f750b27c-8c1e-4e5d-925d-5f644d156f4e
Steps to reproduce

Internally go to our Primer Recipes example
Click Assigned

Version
Latest
Browser
Safari

@maximedegreve Tried to reproduce and couldn't 🤔. Tested on Safari and Chrome
https://github.com/primer/react/assets/1863771/f66f2dcc-a5c9-4171-a62b-afc54bdffb46

@siddharthkp I'm not sure why this works differently on mine here.
I've took a full window video to demonstrate my setup.
https://github.com/primer/react/assets/980622/63c5e7b4-98ee-4378-85ae-6a3d7e77c602

Figured out how to reproduce this:
MacOS → Settings → Appearance → Show scroll bars → Always

@siddharthkp this is likely related, I believe it wasn't like this before we switched the component here recently 🤔:
https://github.com/user-attachments/assets/98772be2-28bf-42f1-938b-85f0832dc671

Yep, same component. The change isn't new (July 2023) but as the Dialog gets more usage, this keeps popping up in more places
https://github.com/user-attachments/assets/653f814e-1a33-46aa-8bdf-ce12dc04def5
Notes:

top-layer / popover does not solve for this
overscroll-behavior: contain would help but only if the dialog has overflow not otherwise (like in the sidebar example)

@keithamus @mperrotti Do you have thoughts?

@keithamus is correct. This is not about the scrollbars here.

Oh, my bad. I may have mixed a different issue with this one and went on the wrong track
I tried reproducing the focus jumping and could not reproduce it in Safari or Chrome 😔

I presume there's a bug in our focus trap code which our React dialog is using.

That's possible, @keithamus were you able to reproduce it as well?

Confirmed with @maximedegreve, we can't reproduce it anymore. My guess is that this was a bug in the recipes component that has now been fixed. Can reopen if we see this or a similar bug again


feat: support v3 rest api client for edgex v3 api

What type of PR is this?
/sig iot
/kind feature
/kind enhancement
What this PR does / why we need it:
The api design of edgex has changed in v2 and v3 versions, the original v2 client does not support v3 api due to dependency libraries and api request paths, etc. In addition, the community intends to implement the v3 client based on MessageBus in the future. This PR is mainly for the preparation of message bus support.
The specific programs are as follows:

Supports edgex list/get based on message bus (openyurt side will still use the rest api for create/update/delete).
Supports edgex create/update/delete based on message bus (all will be implemented using the message bus).

It will gradually replace the original implementation based on the rest API method.
Special notes for your reviewer:
/assign @LavenderQAQ @Rui-Gan

/rerun

/rerun

/rerun


CORE-14383 - Doubled the maximum memory that a Kafka pod can use

wondering - should we merge this change in 5.0 instead and bring it in 5.1 as part of a forward merge?
Suspecting the issues we were seeing will be present in 5.0 too.

wondering - should we merge this change in 5.0 instead and bring it in 5.1 as part of a forward merge?
Suspecting the issues we were seeing will be present in 5.0 too.

I guess we can merge it in 5.0 but it will involve more scrutiny as for:

Is it fixing a critical bug?
Is it improving performance?

And only limited set of people can approve anything into 5.0 at the moment.
I would merge it to 5.1 for now. If this will be a re-occurring pattern in 5.0 we would be able to backport this change there as well.

that sounds reasonable 👍


How do you use it in a private channel?

I tried using /mermaid in a private channel, but it shows Mermaid Preview can't automatically join private channels. If it's a private channel, please invite Mermaid bot to it.. I tried adding the Mermaid bot to the channel; however, it doesn't show up on the participants list.
How do I use it in a private channel?

@lucas-santos-resolver I have that same problem. How did you end up fixing it?


Inter middleware dependency

Description of the feature
Not sure if it is meant to work this way but it looks like if I want middleware1 to be called before middleware2 (inter middleware dependency) using .with I have to chain them like this .with(middleware2).with(middleware1), i.e. middleware2 before middleware1, instead of natural middleware1 before middleware2
Something like this:
Route::new()
        .at(...)
        .with(middleware2)
        .with(middleware1)

If it is meant to be this way, maybe we should have some note in the book somehow.
Or nicer if we can make it work like this, i.e. middleware1 before middleware2, it looks more more natural.
.at(...).with(middleware1).with(middleware2)

Based on the principle of poem middleware, this is impossible to achieve. 😂

Yeah thanks Sunil, I wonder if something similar like this is easier
.at(...).with(middleware1.with(middleware2))

So that relationship between middleware1 and middleware2 is somewhat clearer. It probably looks a bit un-necessarily nested though.

If you understand the working principle of middleware, the current solution is easier to understand. 🙂

Yeah current structure seems nice and concise.
Maybe just a note about the sequence of execution of the middlewares is good enough for anyone who might be wondering.

Alright I am closing this as it isn't anything significant.
I am happy with poem middleware 🙂


Use relative man page path for user installs

tested locally.
RE: https://github.com/tobimensch/termsql/issues/13

Hmm, I had an issue with this solution a couple years back, but now it seems to work for me, who thought it could be this simple? eh :-)
Thanks man

all good cheers :+1:


Update Copy

https://next.carbondesignsystem.com/about-carbon/how-carbon-works#the-carbon-ecosystem
current
The Carbon ecosystem
Carbon is a system of systems. At the epicenter lies the core system, with elements, assets, code, design kits, and guidelines intended as a base for the widest variety of situations. These assets are open source and can be easily extended or adapted.
Teams at IBM also create local systems that depend on and extend the core system for specific use cases. For example, Carbon for IBM.com is a local system that specializes in an editorial experience, whereas Carbon for IBM Products provides assets and guidance specific to the product experience.
Some local systems are open source while others require IBM employee authentication to view.
It is recommended that you start with the core system and whichever local systems are most relevant to your team. Catalogs for components, patterns, functions, and templates allow you to explore everything Carbon has to offer, with filters to find exactly what you need.
If you’re ready to get started with Carbon, check out our guidance for designing and developing

expected
The Carbon ecosystem
Carbon is a system of systems. At the epicenter lies the core system, with elements, assets, code, design kits, and guidelines intended as a base for the widest variety of situations. These assets are open source and can be easily extended or adapted.
Teams at IBM also create local systems that depend on and extend the core system for specific use cases. For example, Carbon for IBM.com is a local system that specializes in an editorial experience, whereas Carbon for IBM Products provides assets and guidance specific to the product experience. Some local systems are open source while others require IBM employee authentication to view.
It is recommended that you start with the core system and whichever local systems are most relevant to your team. Catalogs for components, patterns, functions, and templates allow you to explore everything Carbon has to offer, with filters to find exactly what you need.
If you’re ready to get started with Carbon, check out our guidance for designing and developing
.
.
.
.
In the first paragraph, there could be mention of contributing back to the system
e.g.
.... These assets are open source and can be easily extended or adapted and also reused by other teams in the ecosystem

@aubrey-oneal @jeanservaas
The main point of this (as most of my copy edits) is the continued use of hard returns when not necessary.
The below sentence is a continuation of the preceding paragraph. It does not need to be its own sentence between two paragraphs.
e.g.
“Some local systems are open source while others require IBM employee authentication to view.”

Hi Mike, in the this case I would not take out the carriage return. In my content practice I follow several guidelines and the Content hierarchy guidelines that Tom has written does such a great job of ensuring that the content we write is scannable. i.e. Content hierarchy 1d-content-that-guides-users): Where relevant, headings, subheadings, tabs, bullet-point lists, field labels, and so on are used to break up content and to help users quickly scan the page.
We are also including more information and methods on this in the new UX writing course. For example, this page illustrates common reading patterns and discusses that, since people often scan a page, the writer should Include the most important points in the first sentence or two of any section. I deemed the point about IBM authentication important enough to warrant its own section so that it wouldn't be hidden at the end of a paragraph (burying the lede).
The return helps break up the new concept of what a local system is and who is able to access them. This is something I want to really highlight because it impacts internal and open source users very differently.
I'm definitely open to additional suggestions (for example I merged your last suggestion on taking out a return because the point I was highlighting wasn't so important as this one.)


[Docs] Kill eslint preloader

[ ] PR has tests / docs demo, and is linted.
[x] Commit and PR titles begin with [ComponentName], and are in imperative form: "[Component] Fix leaky abstraction".
[x] Description explains the issue / use-case resolved, and auto-closes the related issue(s) (http://tr.im/vFqem).

I accidentally included this change in a previous PR, and it was received with much tribulation!
It didn't seem appropriate to merge it as part of an unrelated change, so here it is for real this time, in it's own, very special PR. :tada: :balloon:

:100: Should we also remove the npm dependency?

Done! Didn't realise there was one, and surprised it's in the main package.json when it's only used by the docs site! :confused:

@mbrookes Thanks!


Set Default simulation time to 5-10 seconds

The Default end time for the simulation tool in the toolbar is 1000 seconds, which is probably a bit much 🤣

Could we change this to 5 seconds? 10 second?
My guess is the default value is set here, so it seems like an easy change that I am more than happy to make.

Great,  please do.
On Mon, Jul 22, 2019, 8:29 AM jimmyDunne<EMAIL_ADDRESS>wrote:

The Default end time for the simulation tool in the toolbar is 1000
seconds, which is probably a bit much 🤣
[image: Screen Shot 2019-07-22 at 11 25 37 AM]
https://user-images.githubusercontent.com/5296507/61644863-82f7b880-ac73-11e9-8bdf-bdfca3bb7779.png
Could we change this to 5 seconds? 10 second?
My guess is the default value is set here
https://github.com/opensim-org/opensim-gui/blob/9c84113658939e42e734c9c3aab3649de38002c2/Gui/opensim/tracking/src/org/opensim/tracking/tools/Bundle.properties,
so it seems like an easy change that I am more than happy to make.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/opensim-org/opensim-gui/issues/1127?email_source=notifications&email_token=AA6JY4B5GDD26TS2MBTQCITQAXG4BA5CNFSM4IFZ3YKKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HAVLNOQ,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA6JY4FB5KIH5YPL45DLG4DQAXG4BANCNFSM4IFZ3YKA
.

@jimmyDunne this can be closed I think.


Comparison of bools in structs fails under certain circumstances

So, I ran into a confusing thing earlier, where one line of code in one of my scripts wasn't working as expected. I investigated, and here's some test code that demonstrates the bug:
scriptname boolTestB

struct testStruct
	bool b
	int i
	float f
	ObjectReference O
endStruct

Function testBoolStruct() Global
	testStruct structA = new testStruct
	testStruct structB = new testStruct

	structA.b = false
	structA.i = 0
	structA.f = 0
	structA.O = Game.GetForm(0x14) as ObjectReference ; PlayerRef

	structB.b = true
	structB.i = 1
	structB.f = 1
	structB.O = Game.GetForm(0x22B94) as ObjectReference ; some arbitrary rock somewhere

	bool bFalse = false
	bool bTrue = true

	int i0 = 0
	int i1 = 1

	float f0 = 0
	float f1 = 1

	ObjectReference PlayerRef = Game.GetForm(0x14) as ObjectReference
	ObjectReference Rock = Game.GetForm(0x22B94) as ObjectReference

	Debug.Trace("##############################")
	; these work properly
	Debug.Trace("structA: " + structA.b)
	Debug.Trace("structB: " + structB.b)
	Debug.Trace("structA.b || structB.b: " + (structA.b || structB.b))
	Debug.Trace("structB.b || structA.b: " + (structB.b || structA.b))
	Debug.Trace("false || (bTrue != bFalse): " + (false || (bTrue != bFalse)))
	Debug.Trace("false || (i0 != i1): " + (false || (i0 != i1)))
	Debug.Trace("false || (f0 != f1): " + (false || (f0 != f1)))
	Debug.Trace("false || (PlayerRef != Rock): " + (false || (PlayerRef != Rock)))
	Debug.Trace("false || (structA.i != structB.i): " + (false || (structA.i != structB.i)))
	Debug.Trace("false || (structA.f != structB.f): " + (false || (structA.f != structB.f)))
	Debug.Trace("false || (structA.O != structB.O): " + (false || (structA.O != structB.O)))

	; this one doesn't, with Caprica
	Debug.Trace("false || (structA.b != structB.b): " + (false || (structA.b != structB.b)))

	Debug.Trace("##############################")
EndFunction

I compiled two separate scripts, identical except that one was compiled with the Bethesda compiler, and the other with Caprica (boolTestB and boolTestC respectively), and here's the log output, again in the same order:

[04/17/2018 - 07:46:57PM] ##############################
[04/17/2018 - 07:46:57PM] structA: False
[04/17/2018 - 07:46:57PM] structB: True
[04/17/2018 - 07:46:57PM] structA.b || structB.b: True
[04/17/2018 - 07:46:57PM] structB.b || structA.b: True
[04/17/2018 - 07:46:57PM] false || (bTrue != bFalse): True
[04/17/2018 - 07:46:57PM] false || (i0 != i1): True
[04/17/2018 - 07:46:57PM] false || (f0 != f1): True
[04/17/2018 - 07:46:57PM] false || (PlayerRef != Rock): True
[04/17/2018 - 07:46:57PM] false || (structA.i != structB.i): True
[04/17/2018 - 07:46:57PM] false || (structA.f != structB.f): True
[04/17/2018 - 07:46:57PM] false || (structA.O != structB.O): True
[04/17/2018 - 07:46:57PM] false || (structA.b != structB.b): True
[04/17/2018 - 07:46:57PM] ##############################

[04/17/2018 - 07:47:00PM] ##############################
[04/17/2018 - 07:47:00PM] structA: False
[04/17/2018 - 07:47:00PM] structB: True
[04/17/2018 - 07:47:00PM] structA.b || structB.b: True
[04/17/2018 - 07:47:00PM] structB.b || structA.b: True
[04/17/2018 - 07:47:00PM] false || (bTrue != bFalse): True
[04/17/2018 - 07:47:00PM] false || (i0 != i1): True
[04/17/2018 - 07:47:00PM] false || (f0 != f1): True
[04/17/2018 - 07:47:00PM] false || (PlayerRef != Rock): True
[04/17/2018 - 07:47:00PM] false || (structA.i != structB.i): True
[04/17/2018 - 07:47:00PM] false || (structA.f != structB.f): True
[04/17/2018 - 07:47:00PM] false || (structA.O != structB.O): True
[04/17/2018 - 07:47:00PM] false || (structA.b != structB.b): False
[04/17/2018 - 07:47:00PM] ##############################

Also, here's the assembly after running each through Bethesda's disassembler:
						JumpT ::temp15 _label9                                   ;@line 51
						StructGet ::temp13 structA b                             ;@line 51
						StructGet ::temp2 structB b                              ;@line 51
						CompareEQ ::temp12 ::temp13 ::temp2                      ;@line 51
						Not ::temp12 ::temp12                                    ;@line 51
						Cast ::temp15 ::temp12                                   ;@line 51
					_label9:
						Cast ::temp10 ::temp15                                   ;@line 51
						StrCat ::temp11 "false || (structA.b != structB.b): " ::temp10  ;@line 51
						CallStatic debug Trace ::nonevar ::temp11 0              ;@line 51

						JumpT ::temp3 _label9                                    ;@line 51
						StructGet ::temp3 structA b                              ;@line 51
						StructGet ::temp3 structB b                              ;@line 51
						CompareEQ ::temp3 ::temp3 ::temp3                        ;@line 51
						Not ::temp3 ::temp3                                      ;@line 51
						Assign ::temp3 ::temp3                                   ;@line 51
					_label9:
						Cast ::temp4 ::temp3                                     ;@line 51
						StrCat ::temp4 "false || (structA.b != structB.b): " ::temp4  ;@line 51
						CallStatic debug Trace ::nonevar ::temp4 0               ;@line 51

It appears that Caprica gets really confused and starts assigning everything to ::temp3 for some reason. This also makes Champollion fail miserably on the Caprica script, it just gives up when it gets to that section. This was compiled with the now 2-year-old v0.2.0 release binary from the Nexus, without the -O flag since I know it's borked in that version.
Also, here's a link to an archive with the sources, compiled scripts, Caprica .psc output (for boolTestC), and disassembled .psc outputs for both, in case it helps any.
This is easy enough to work around, but really confusing if you run into it like I did. Any ideas?

Some extra weirdness. Commenting out some of the previous calls to Debug.trace results in what looks like proper output (I have not ran it)
Function testBoolStruct() Global
	testStruct structA = new testStruct
	testStruct structB = new testStruct

	structA.b = false
	structA.i = 0
	structA.f = 0
	structA.O = Game.GetForm(0x14) as ObjectReference ; PlayerRef

	structB.b = true
	structB.i = 1
	structB.f = 1
	structB.O = Game.GetForm(0x22B94) as ObjectReference ; some arbitrary rock somewhere

	bool bFalse = false
	bool bTrue = true

	int i0 = 0
	int i1 = 1

	float f0 = 0
	float f1 = 1

	ObjectReference PlayerRef = Game.GetForm(0x14) as ObjectReference
	ObjectReference Rock = Game.GetForm(0x22B94) as ObjectReference

	Debug.Trace("##############################")
	; these work properly
	Debug.Trace("structA: " + structA.b)
	Debug.Trace("structB: " + structB.b)
;	Debug.Trace("structA.b || structB.b: " + (structA.b || structB.b))
;	Debug.Trace("structB.b || structA.b: " + (structB.b || structA.b))
;	Debug.Trace("false || (bTrue != bFalse): " + (false || (bTrue != bFalse)))
;	Debug.Trace("false || (i0 != i1): " + (false || (i0 != i1)))
;	Debug.Trace("false || (f0 != f1): " + (false || (f0 != f1)))
;	Debug.Trace("false || (PlayerRef != Rock): " + (false || (PlayerRef != Rock)))
;	Debug.Trace("false || (structA.i != structB.i): " + (false || (structA.i != structB.i)))
;	Debug.Trace("false || (structA.f != structB.f): " + (false || (structA.f != structB.f)))
;	Debug.Trace("false || (structA.O != structB.O): " + (false || (structA.O != structB.O)))

	; this one doesn't, with Caprica
	Debug.Trace("false || (structA.b != structB.b): " + (false || (structA.b != structB.b)))

	Debug.Trace("##############################")
EndFunction

The assembly:
            ASSIGN ::temp2 False
            JUMPT ::temp2 label0
            STRUCTGET ::temp2 structA b
            STRUCTGET ::temp4 structB b
            COMPAREEQ ::temp4 ::temp2 ::temp4
            NOT ::temp4 ::temp4
            ASSIGN ::temp2 ::temp4
            label0:
            CAST ::temp3 ::temp2
            STRCAT ::temp3 "false || (structA.b != structB.b): " ::temp3
            CALLSTATIC debug Trace ::nonevar ::temp3 0

The two STRUCTGET instructions store to different temporaries.


jade-bootstrap can't understand express return variables

+dropdown("select type",[
{text: "by month", url: 'javascript:change_chart(#{chart},'month')'},
{text: "by year", url: 'javascript:change_chart(#{chart},'year')'}
])
this is my snippet, i want to render the page with variables that server gives me. but not matter how, i couldn't make it.

@yongdi Do you have a sample repo so that i can look into the issue in detail.
and also it will help to nail down the root cause...

Archive.zip
according to our company's policy, I can't give all the project. 2 jade views affixed.  the node server responses like:
res.render('data-table', {                 cols: cols,                 tableName: req.params.table,                 chart: req.params.table,                 show_date_search: show_date_search             });
Till now, everything is ok, including table & charts, except this issue. thank you for helping me


KAFKA-15663, KAFKA-15794: Telemetry reporter and request handling (KIP-714)

Implements ClientTelemetryReporter which manages the lifecycle for client metrics collection. The reporter also defines  TelemetrySender which will be used by Network clients to send API calls to broker.
Details about the handling of request and response are covered in KIP-714.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Build for the PR depends on PR: https://github.com/apache/kafka/pull/14620

Hi @apoorvmittal10 - I left some questions while making the first pass of the PR. They are all aesthetics.  Will follow up with more questions. Thanks!

Hi @apoorvmittal10 - I left some questions while making the first pass of the PR. They are all aesthetics. Will follow up with more questions. Thanks!

Thanks @philipnee , I have addressed the comments. Can you please re-review.

Thanks for review @AndrewJSchofield, @philipnee. I have addressed the feedback.

Hi @mjsax, thanks a lot for the review. I have addressed the feedback, please let me know if I need to further address anything.
I have updated the PR with other merges and will wait for build for successful completion now.

@mjsax I have also moved ClientTelemetryUtils to internals package in this PR as well, the class is new and have changes in both PRs.

@mjsax I have added the comment as suggested.

Hi @apoorvmittal10 thanks for writing this PR - also sorry for the delay but I made the first pass and left some stylistic comments.

Hi @apoorvmittal10 thanks for writing this PR - also sorry for the delay but I made the first pass and left some stylistic comments.

Hi @philipnee Thanks for the review. I have addressed the comments.

Hi @apoorvmittal10 Thanks for the time to address my comments - I made another pass and added some more comments, mostly nit-picking.  After this I think we are good to go.

Hi @apoorvmittal10 Thanks for the time to address my comments - I made another pass and added some more comments, mostly nit-picking. After this I think we are good to go.

Hi @philipnee thanks for the re-review. I have addressed the feedback.

Another build succeeded with flaky tests.


Add Incremental Review /review -i

Use /review -i to review only new commits that hasn't been reviewed since the last pr-agent review, instead of full review to the entire PR (from the first commit).
How is works:
The pr-agent will search last review published by the PR-Agent and review all the subsequent commits that hasn't been reviewed yet ( * commits where commit_date > last_review_date)

@hussam789 /review -i

@hussam789 /review -i

Preparing review...

@hussam789 /review -i

Preparing review...

Incremental PR Review

⏮ Review for commits since previous PR-Agent review: Starting from commit https://api.github.com/repos/Codium-ai/pr-agent/pulls/94/commits/5f36f0d75332f7be956cb79fdf620a6f5dd1bf04

PR Analysis

🎯 Main theme: Adding incremental review functionality to the PR-Agent
📌 Type of PR: Enhancement
🧪 Relevant tests added: No
✨ Focused PR: Yes, the PR is focused as it introduces a new feature of incremental review. The title and description clearly explain the purpose of the PR and the changes in the code are aligned with this purpose.
🔒 Security concerns: No, the PR does not introduce any apparent security issues. The changes are primarily related to the logic of the PR review process and do not involve any sensitive data handling or operations that could be exploited for malicious purposes.

PR Feedback

💡 General PR suggestions: The PR is well-structured and the code changes are logically organized. However, it would be beneficial to include tests for the new functionality to ensure it works as expected and does not introduce regressions. Additionally, it would be helpful to update the documentation to explain how to use the new incremental review feature.

How to use

Tag me in a comment '@hussam789' and add one of the following commands:
/review - Request a review of the latest update to the PR.
/describe - Modify the PR title and description based on the contents of the PR.
/improve - Suggest improvements to the code in the PR. These will be provided as pull request comments, ready to commit.
/ask  - Pose a question about the PR.

@hussam789 /ask do I have enough documentation

@hussam789 /ask do i have enough documentation

@hussam789 /review

@hussam789 /review -i

Preparing review...

@CodiumAI-Agent /improve

Preparing review...


feat: add C implementation for stats/base/dists/t/median

Resolves #3875.
Description

What is the purpose of this pull request?

This pull request:

Adds a C implementation of the t median.
Includes benchmarks for the C implementation.
Provides examples demonstrating the usage of the t median in C.

Related Issues

Does this pull request have any related issues?

This pull request:

resolves #3875

Questions

Any questions for reviewers of this pull request?

No.
Other

Any other information relevant to this pull request? This may include screenshots, references, and/or implementation notes.

No.
Checklist

Please ensure the following tasks are completed before submitting this pull request.

[x] Read, understood, and followed the contributing guidelines.

@stdlib-js/reviewers

Please refer this .


Add in an !!/unwatch command

We are pretty liberal with keywords we watch, therefore we need to remove keyword all the time.
Would be nice to have a command like !!/unwatch {keyword} or even !!/unblacklist
We already have this for users (!!/rmblu)

already implemented in helios

Just to point out that the !!/rmblu process is way different to what would be needed to make an !!/unwatch command.
The blacklisted users list is stored locally per instance, while the watch and blacklists are stored on GitHub.

Done


fix docs: Wrong description in get-started-ip-addresses.md

fix docs: Wrong description in get-started-ip-addresses.md
Description

Related issues/PRs

Todos

[ ] Tests
[x] Documentation
[ ] Release note

Release Note

None required

 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.

Deploy preview for calico ready!
Built with commit 74a4f92759d108290c32c3bdda3c1f4d6eda9b52
https://deploy-preview-3091--calico.netlify.com

Thanks @zgfh !


fix(EVM): Fix SSTORE warmth check

What ❔

Why ❔

Checklist

[ ] PR title corresponds to the body of PR (we generate changelog entries from PRs).
[ ] Tests for the changes have been added / updated.
[ ] Documentation comments have been added / updated.

Closed in favor of https://github.com/matter-labs/era-contracts/pull/1134


File validation issues

This issue will be used to collect all info related to the verification data for 3.0.0-Preview 9 release:
Build to use: daily build from https://github.com/dotnet/core-sdk#installers-and-binaries
If you plan to create any Azure resources specifically for the verification work, please create those in the 3.0-p9-validation resource group, in the ASP.NET Core Test subscription.
Make sure to create the issues in aspnet/AspNetCore repo and apply the proper area label and Validation label. Put the issue in 3.0.0-preview9 milestone.
The project to use to assign issues to:
After you create the issues, add them to the https://github.com/aspnet/AspNetCore/projects/4 project and unassign yourself from this issue.

Filed https://github.com/aspnet/AspNetCore/issues/13199

Is this about validation work our team does or something else?

https://github.com/aspnet/AspNetCore/issues/13254
https://github.com/aspnet/AspNetCore/issues/13255
https://github.com/aspnet/AspNetCore/issues/13256
https://github.com/aspnet/AspNetCore/issues/13257

@dougbu This is about our team's validation.

https://github.com/aspnet/AspNetCore/issues/13178
https://github.com/aspnet/AspNetCore/issues/13076
https://github.com/aspnet/AspNetCore/issues/13074
https://github.com/aspnet/AspNetCore/issues/13073
https://github.com/aspnet/AspNetCore/issues/13072
https://github.com/aspnet/AspNetCore/issues/13071

https://github.com/aspnet/AspNetCore/issues/13282

Unassigning @SteveSandersonMS since he's out most of validation week.

My validation is done.

https://github.com/aspnet/AspNetCore/issues/13334
https://github.com/aspnet/AspNetCore/issues/13335
https://github.com/aspnet/AspNetCore/issues/13336
https://github.com/aspnet/AspNetCore/issues/13337

https://github.com/aspnet/AspNetCore/issues/13387


Fix operator target

What this PR does / why we need it:
Removing the whole suffix after Sig so we set the correct provider.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:
NONE

/cc @acardace @dhiller

/retest


bug: high performance on collection-repeat with defined objects

As topic as topic. Why 4 called times per row for getting only first name? Object-defined isn't friendly for directive collection-repeat?
Link: http://codepen.io/machei/full/dogOro/

Greetings @machei!
My sensors indicate that you need to update your issue through our custom issue form. We are now requiring all issues to be submitted this way, to ensure that we have all of the information necessary to fix them as quickly as possible.
Click Here To Update Your Issue
I will have no choice but to close this issue if it is not resubmitted through the form. Please fill out the rest of the form, so that I may use my friendly robot powers to assist you.
Thank you!

The example you are pointing to is from beta9. Please update the codepen to the latest and check to see if the issue is still there.

My codepen has been updated to the latest version 1.0.1. So 3 replies per one first name - seems that is more double than prevouis. Not good.
http://codepen.io/machei/full/dogOro/

The error is just with the way data is being returned in that codepen, not with collection repeat.
http://codepen.io/mhartington/pen/XbyjyR

U are wrong with code but nothing isnt relating to object-defined. Need fix for improviving performance, should be only one called access to variable in purpose to reading value.
http://codepen.io/machei/full/aOQmMV/ -> Please, replaced object-defined method with your codepen and look at console.
Please. reopen issue.

Hello all! I will be closing this for now as i am not seeing any performance issues on any of my devices, including my old Moto X 2014 and Nexus 7 2013. Thanks for using Ionic!


Passing null value for command line arguments from IntelliJ for HDInsight

Hello,
I am trying to pass null arguments for command line like '' but its not taking. May I know how to pass the null value ? I hope there will be a way.

Could you anyone update me how can I puss null value ? like ''


Support for large input file

Stackoverflow Posts.xml is 109Gb, when running it in memory it ends up crashing.
This PR adds a disk-backed memory using lmdb to support storage and access to such large data.

Without the support of the disk, with a fast disk and several Gb of memory and GPus, parsing stackoverflow (over 100G uncompressed file) took:
Parsing stackoverflow XML file: 59171902it [3:58:53, 4128.21it/s]


git submodule failed for ondc-mmi-service. ondc-mmi-service not properly configured in the sandbox

lenovo@lenovo-ThinkPad-T470-W10DG:~/code/ondc/ondc-sandbox$ git submodule update --recursive
Cloning into '/home/lenovo/code/ondc/ondc-sandbox/ondc-mmi-service'...
ERROR: Repository not found.
fatal: Could not read from remote repository.
Please make sure you have the correct access rights
and the repository exists.
fatal: clone of 'git@github.com:dataorchestration/ondc-mmi-server.git' into submodule path '/home/lenovo/code/ondc/ondc-sandbox/ondc-mmi-service' failed
Failed to clone 'ondc-mmi-service'. Retry scheduled
Cloning into '/home/lenovo/code/ondc/ondc-sandbox/ondc-mmi-service'...
ERROR: Repository not found.
fatal: Could not read from remote repository.
Please make sure you have the correct access rights
and the repository exists.
fatal: clone of 'git@github.com:dataorchestration/ondc-mmi-server.git' into submodule path '/home/lenovo/code/ondc/ondc-sandbox/ondc-mmi-service' failed
Failed to clone 'ondc-mmi-service' a second time, aborting
lenovo@lenovo-ThinkPad-T470-W10DG:~/code/ondc/ondc-sandbox

+1


Pipelines samples

Thanks @davidfowl and @Youssef1313.
I've added some more changes and fixes. Good to merge?


Fund 5 Draft Proposal

This is the genesis Issue for this project (sample data)

[ ] Write Proposal
[ ] Submit Proposal

Issue and Project Board unused


about z-axis

hello!
My problem is that when I use AWR1642 to collect data, the value of z[objectNum] is always being 0. Could you please tell me what I can do to solve the problem?
Hope for your reply! Thanks!

Hello,
If you are using the normal AWR1642 BOOST, it only measures the 2D position, that is why the Z value is 0. You will have to use another board like the AWR1843 for the 3D point cloud.
Regards,
Ibai

Oh, I see. I really appreciate for your help!


Display "Node" rather than "Node.js" when selecting Node.js runtimes

OS: All
Build Version: 20191010.14
Repro Steps:

F1 --> Type and select "Azure App Service: Create New Web App... (Advanced)".
Enter a valid name --> Select a resource group --> Select Linux OS --> Check the runtimes of Node.js.

Expect:
Display "Node.js xxx".
Actual:
Display "Node xxx".

The display property returned from the Azure API is now "Node" instead of "Node.js".
@fiveisprime is keeping this new display name OK? We can insert the ".js" if needed

Node without the .js is fine with me. Going to close this since there's no action required.


Ability to limit concurrency?

This looks like a fantastic tool! Very interested in it. I'm trying to see if there's a way to limit concurrency while using this tool. For example, I'd like to make hundreds or thousands of API calls, each of which takes about 10 seconds, but I'd like to limit the load on the server and have at most about 20 calls running at a time. Is that something I could do with this tool? I couldn't find anything in the tests or documentation so I'm guessing no, but thought I'd check. If no, is that a reasonable feature request or would that use case mean I should choose a different tool?

Functionality like that is usually left up to the application. I found a nice example on SO which I've adapted below into a handy decorator (that works with unsync).
def limit(n):
    def wrapper_gen(func):
        semaphore = asyncio.Semaphore(n, loop = unsync.loop)
        @wraps(func)
        async def limiter(*args, **kwargs):
            async with semaphore:
                return await func(*args, **kwargs)
        return limiter
    return wrapper_gen

@unsync
@limit(2)
async def request():
    await asyncio.sleep(1)
    print("Done")

requests = [request() for i in range(10)]
print([request.result() for request in requests])

Something like this can be done with or without unsync, but IMO it's a bit easier with the single event loop. Thanks for the question!

@alex-sherman Could you please reopen ?
The decorator doesn't work anymore in Py 3.10 as Semaphore's loop has been deprecated.
But mainly, it doesn't work with a sync function returning something e.g. the json response from requests.get(). I think that's what the OP wanted to know.
One could (try to) implement aiohttp but then no need for unsync I believe.

I'm not sure what I could do with reopening this, my point in closing it was that it can be accomplished outside of unsync. The solution I gave would work with any normal async method.
Since the deprecation of the loop parameter you can just use the unsync event loop with asyncio.set_event_loop(unsync.loop). The request method can also return a value. I updated my solution with these changes.

Thx @alex-sherman.
Semaphore didn't work out for me. I ended up using the backoff library - together  with unsync - as the API I was targeting returned a specific message when over-the-limit. Not really what the OP was after as it doesn't limit the rate per se and rather retry but worked for my use case.


Add the ability for users to comment on the documentation.

Justin said he would work on this.
We spoke about this at Orchard Harvest 2014.
We talked about using Disqus: http://disqus.com/
We talked about using Discourse http://www.discourse.org/

The last time I heard this talked about in the weekly meeting it was said that the docs are easy enough to edit if you have something to contribute so the comments aren't required.
Can this be closed?


Comp/guided tour

Hello tem,
This PR contains the implementation of a brand new component, the guided tour.
Here is some details about the feature

browsing throughout website pages following a scenario
spotlight wanted areas

things to improve

can't spotlight an element that is contain in a component that come from the design-system (ex: dropdown)
does not work well in the design-system website

It's not the intent to merge this PR in develop currently, more update will occur over time (through sub PR)

:tada: This PR is included in version 1.3.0-rc.1 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Simulated tap gesture not working inside InAppWebView on iOS

[x] I have read the Getting Started section
[x] I have already searched for the same problem

Environment

Technology
Version

Flutter version
3.7.5

Plugin version
5.7.2+3

Android version

iOS version
16.3

macOS version

Xcode version
14.1

Device information: Iphone SE 2
Description
I am trying to simulate a tap event inside the webview, with the following code:
Future<void> tap() async {
    setState(() {
      hideButton = true;
    });

    await Future.delayed(const Duration(milliseconds: 200));

    GestureBinding.instance.handlePointerEvent(PointerDownEvent(position: Offset(coordX, coordY + appBarAndStatusBarHeight)));
    GestureBinding.instance.handlePointerEvent(PointerUpEvent(position: Offset(coordX, coordY + appBarAndStatusBarHeight)));

    setState(() {
      hideButton = false;
    });
  }

And I am using the following options in my webview:

  initialOptions: InAppWebViewGroupOptions(
      crossPlatform: InAppWebViewOptions(
        transparentBackground: true,
        allowFileAccessFromFileURLs: true,
        useShouldOverrideUrlLoading: true,
        allowUniversalAccessFromFileURLs: true,
        javaScriptEnabled: true,
        javaScriptCanOpenWindowsAutomatically: false,
        preferredContentMode: UserPreferredContentMode.MOBILE,
      ),
      ios: IOSInAppWebViewOptions(
        allowsInlineMediaPlayback: true,
        allowsPictureInPictureMediaPlayback: false,
        allowsAirPlayForMediaPlayback: false,
      ),
      android: AndroidInAppWebViewOptions(
        preventFullScreen: true,
        useHybridComposition: false,
      ),
    ),

It works fine on Android, but on iOS it doesn't trigger the tap event, but it does on a button or any other touchable widgets.
Expected behavior:
Trigger the tap event in the webview.
Current behavior:
Tap event is not recognized by the webivew's content.
Steps to reproduce
I tried with a draggable Positioned widget, which I can drag over buttons in the webview, so I can trigger the button's functionality.
Unfortunately, I can't share the whole code, because it is my company's property, but I can answer questions, if there will be any.

Same problem here, I want to call tap() on my InAppWebview during tests. It works well on Android but not with iOS.


[Bug] Edit registration saves new registration but old one is still displayed

Editing the registration and making the registrationSummoner contract call saves a new registration but the old one is still being displayed.
As some of the edit fields deal with smart contract data, they cannot really be edited and it makes sense to consider it as a new registration, but we need to have a way to hide the old registrations to prevent duplicates on Explorer page.
One way to solve this would be to fetch the registrationId which is unique per registration and not per DAO, and add it to hide addresses on the frontend.
we would need to think of a design that is more equipped to handle things at scale when there would be more registrations and edits as a simple list filter of registrationId's would slow down the page load and cause bad UX.

The better solution is not to hide it exactly but to allow to show up somewhere in the interface, so you can see the old one if you want.

Can group new+old registrations by unique tuple (name, contract address, network).

An alternate solution, this can be easily solved by adding a timestamp filed to the registration instance and having a filter which If the DAO Name is the same, displays the latest registration instance based on the timestamp
However, as the DAO Name isn't a unique identifier, one could register a new instance with the same DAO Name

Can group new+old registrations by the unique tuple (name, contact address, network).

This solution fails when the contract address is updated.

For now, When someone edits a registration Instance, they could submit this form https://forms.gle/GieH87a7LGrFRUnd6 for us to hide it. or They could create a PR  for a change in this file to include their registration ID
As this scenario is rare, this would serve as a fallback solution.


config parameter assignment to classes

Read in the config in one shot, check it for any issues (ie parameters that don't exist), and assign config parameters to classes ASAP.  If this means we need a config class, make one.  As is, a series of dictionaries/subdictionaries of configurations are passed around that are difficult to trace and never verified.  Should be done after/in conjunction with #74 .

Feel free to view the HyperTrace code for an example of this, but I believe you have experience with this already. I would also instead consider PEP 589, which provides strongly typed dictionaries (link). The types can be built-in or custom. As we discussed, one thing we have to be careful of is to not modify the parameters object as it passes through the processing pipeline. A Parameters class is only necessary when it requires member functions, iterators, generators, etc. This comes at the cost of added complexity. I still feel it would be ideal, if that functionality is not needed, to immediately unpack a strictly typed dictionary upon initialization. This way, there is some built-in validation and each parameter is visible and not hidden within a Parameters class instance, easily exportable to a JSON file. Dictionary unpacking is also super simple with **kwargs in Python, with each parameter subsequently passed though functions explicitly (not hidden within a class instance or self). This will make it much easier to follow the program logic. The only reason I used a parameters class in my case is that member functions and iterators were needed in that case. Again, I strongly support the additional of parameter value validation in a new validation module, which we do as well. This will take away a lot of the mystery in debugging, as currently the parameter values are not checked at all and there is overuse of implicit calls to self that mutates an entire tree of complex object states. Adding yet another class could make this more onerous. In other words, my vote is for a simple, serializable, immutable parameters object and explicit passing of parameters to functions to ease debugging. Dictionaries are also faster than classes, though they may use slightly more memory. The new strictly typed dictionaries solve most of the aforementioned problems.


The CommandAPI doesn't safeguard against paths with duplicate node names. This leads to client crashes

CommandAPI version
8.5.0
Minecraft version
1.19
Are you shading the CommandAPI?
No
What I did

Register the following command:new CommandAPICommand("broken1")
    .withArguments(new StringArgument("key"))
    .withArguments(new IntegerArgument("a"))
    .withArguments(new StringArgument("key"))
    .executesPlayer((player, objects) -> {
        System.out.println(objects[0] + ", " + objects[1] + ", " + objects[2]);
    })
    .register();

Type /broken1 str 123 str

What actually happened
Minecraft's client (the vanilla client, not the server) crashes and closes.
What should have happened
The command should be runnable
Server logs and CommandAPI config
No response
Other
This appears to be the case when there is a "gap" between arguments. The following (with two consecutive arguments with the same key) doesn't cause a crash:
new CommandAPICommand("broken")
    .withArguments(new StringArgument("key"))
    .withArguments(new StringArgument("key"))
    .executesPlayer((player, objects) -> {
        System.out.println(objects[0] + ", " + objects[1]);
    })
    .register();

Fixed in 8.5.1.


Add requestToken endpoints

For password reset and 3pid adding

As it stands, I'm not happy with the descriptions here, simply because we have so many API endpoints which have very similar looking paths, do very similar things, and the subtle differences between them are not emphasised (either with bold or more prose).
I'm not familiar with these API calls so I'm not drawing on prior knowledge. From this documentation alone, I am highly confused as to what I should be doing. If this is because the proxied endpoints have got the core documentation and these are just "helper" methods effectively then I would like to see links in these descriptions to the relevant documentation. However, from a brief skim of http://matrix.org/speculator/spec/HEAD/identity_service/unstable.html#creating-a-session I'm sure not sure what should be happening.

ok, think that's done evrything now: ptal

LGTM!


[UCAI] Add enhancements to docstring parsing for function creation from python callable

PR Checklist

[X] A description of the changes is added to the description of this PR.
[ ] If there is a related issue, make sure it is linked to this PR.
[ ] If you've fixed a bug or added code that should be tested, add tests!
[ ] If you've added or modified a feature, documentation in docs is updated

Description of changes
This PR includes enhancements for handling non-standard Google-style docstrings, common typos, and the handling of missing descriptions within argument key entries within a docstring.
Additionally, added comprehensive test coverage for the validation of parsing functionality and the extraction of both type declarations within callable signatures and the descriptions within python docstrings.
Documentation guide regarding the features contained within this PR will be filed in a followup.

@serena-ruan @dennyglee this one is ready for a look!

@dennyglee we're all set to merge this one!


Fix 'flicker' during window presentation caused by early-out criteria.

Related discussion can be found on PR #5, which caused this bug as a result of trying to fix a different bug.
The intent here is to avoid the too-early resize event generated from having setFrameTopLeft move a window between screens, but to keep subsequent events that correctly resize the MacVim frame by the time the window animates.

@jpetrie Thank you

@jpetrie Thank you


Windows 10: Unexpected error unsupported signal SIGQUIT

As the title shows, I am receiving an error when trying to run on Windows 10:
Unexpected error unsupported signal SIGQUIT
Any ideas? Serverengine is getting used by Sneakers (ruby gem), and Serverengine is throwing this error.

I found a similar error occurred in this build as well on appveyor: https://ci.appveyor.com/project/fluent/fluentd/build/2345/job/mgy36d6xqqlye4bt


New point icons and border options

New icons have been added to the point annotation tool:

You can now add a coloured border to points and change its width:

Tweaks have been made to the UI and how the highlight shows when you go to edit an annotation. The highlight now fits the clicked-on annotation better and scales in size. This change has also been made to highlighting text annotations.

Finally, all applicable instances of let in the annotation files have been updated to const.
Once merged, push to staging for further testing.
Closes #202

Looks good from a brief review of the code side and I look forward to seeing the innovative uses people come up with for the lighting bolt icon. I'll let Rob do the proper review as I believe he is scheduling in the releases.


Change REST API for test #11

Checked output of changelogs in #11
https://github.com/arktwin/arktwin/actions/runs/9753919978/job/26919956348


Document workflow for vsportal

Download sequences and metadata from VirusSeq Portal using Download All link (API pending).
Decompress gzipped tar file, e.g., tar -zvxf virusseq.tar.gz
Run Pangolin on FASTA file (conda activate pangolin; pangolin [FASTA]) - writes to lineage_report.csv unless we specify another output file stem.
Convert data:% python3 convert.py virusseq-consensus-export-all-2021-06-30T17:03:29.727965Z.fasta \
lineage_report.csv virusseq-consensus-export-all-2021-06-30T17:03:29.727965Z.tsv \
-o virusseq-2021-06-30.json.xz --delimiter '\t' --name isolate \
--coldate 'sample collection date' \
--set-region 'North America' --country 'geo_loc_name (country)' \
--accession "specimen collector sample ID"

Process data: python3 process.py virusseq-2021-06-30.json.xz

Problem with resulting clusters.json file:
[{"nodes": {
  "None": [
    ["2020-10-10", "None", "Canada", "North America", "L00298195", "hCoV-19/Canada/Qc-L00298195/2020"],
    ["2020-10-10", "None", "Canada", "North America", "L00298203", "hCoV-19/Canada/Qc-L00298203/2020"]
  ]},
"edges": [
  ["None", "None", 1.74, 0.57],
  ["None", "None", 1.04, null],
  ["None", "None", 0.91, null],
  ["None", "None", 2.21, null],
  ["None", "None", 1.92, null],
  ["None", "None", 0.91, null],

Fixed in commit e3991ed

Ran on orolo:
python3 process.py virusseq-2021-06-30.json.xz --ft2bin fasttree2-mp -np 6

Note -np argument means we don't need a machine file.
TODO: if user sets neither -np nor --machinefile, then pipeline should run in serial mode.

wget https://data.nextstrain.org/files/ncov/open/metadata.tsv.gz
wget https://data.nextstrain.org/files/ncov/open/sequences.fasta.xz
conda activate pangolin
pangolin --update  # actually there was a major revision
pangolin sequences.fasta.xz --threads 8

Forgot I need to write region and country to labels now, because these are not necessarily embedded in sequence names.

Current data: clusters.2021-07-21T23:22:29.json
Front-end errors:

Error: <rect> attribute width: Expected length, "NaN". (59 times)
Error: <text> attribute x: Expected length, "NaN". (59 times)

beadplot.js:674 Uncaught TypeError: Cannot read property 'replace' of undefined
  at SVGLineElement.<anonymous> (beadplot.js:674)
  at SVGLineElement.<anonymous> (d3.js:1209)
  at Selection.selection_each [as each] (d3.js:1176)
  at Selection.selection_attr [as attr] (d3.js:1233)
  at redraw (beadplot.js:673)
  at beadplot (beadplot.js:875)
  at Object.<anonymous> (covizu.js:163)
  at fire (jquery.js:3496)
  at Object.fireWith [as resolveWith] (jquery.js:3626)
  at done (jquery.js:9786)

drawtree.js:277 Uncaught TypeError: mutations is not iterable
  at mutations_to_string (drawtree.js:277)
  at mouseover (drawtree.js:323)
  at SVGTextElement.<anonymous> (drawtree.js:415)
  at SVGTextElement.<anonymous> (d3.js:1526)

Fixed first three problems.  Still getting Uncaught TypeError: mutations is not iterable.
Probably this field is not populated in vsportal branch.

Processing newer VirusSeq data:
(pangolin) art@orolo:~/git/covizu$ python3 convert.py virusseq-consensus-export-all-2021-07-21T15\:40\:59.370802Z.fasta lineage_report.csv virusseq-consensus-export-all-2021-07-21T15\:40\:59.370802Z.tsv -o virusseq-2021-07-21.json.xz --delimiter '\t' --name 'fasta header name' --coldate 'sample collection date' --set-region 'North America' --country 'geo_loc_name (country)' --accession "specimen collector sample ID"

(pangolin) art@orolo:~/git/covizu$ python3 process.py virusseq-2021-07-21.json.xz --ft2bin fasttree2-mp -np 8
🏄 [0:00:01.624509] aligned 0 records
🏄 [0:00:01.639270] aligned 1000 records
🏄 [0:00:03.310331] aligned 2000 records
🏄 [0:00:04.935787] aligned 3000 records
🏄 [0:00:06.442471] aligned 4000 records
🏄 [0:00:06.457993] aligned 5000 records
🏄 [0:00:08.193083] aligned 6000 records
🏄 [0:00:09.712305] aligned 7000 records
🏄 [0:00:11.282963] aligned 8000 records
🏄 [0:00:13.051069] aligned 9000 records
🏄 [0:00:13.206514] filtered 3008 problematic features
🏄 [0:00:13.206572]          3975 genomes with excess missing sites
🏄 [0:00:13.206580]          2219 genomes with excess divergence

This is a substantial fraction of genomes discarded with excess divergence, better check

Weird error:
🏄 [0:01:43.084084] start MPI on minor lineages
Traceback (most recent call last):
  File "covizu/clustering.py", line 13, in <module>
    from covizu.utils.progress_utils import Callback
ModuleNotFoundError: No module named 'covizu'
...
Traceback (most recent call last):
  File "process.py", line 172, in <module>
    analyze_feed(handle, args, callback=cb.callback)
  File "process.py", line 131, in analyze_feed
    result = make_beadplots(by_lineage, args, callback, t0=t0.timestamp())
  File "/home/art/git/covizu/covizu/utils/batch_utils.py", line 117, in make_beadplots
    subprocess.check_call(mpirun+cmd)
  File "/home/art/miniconda3/envs/pangolin/lib/python3.8/subprocess.py", line 364, in check_call
    raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['mpirun', '-np', '8', 'python3', 'covizu/clustering.py', 'data/by_lineage.json', 'minor_lineages.txt', '--mode', 'flat', '--max-variants', '5000', '--nboot', '100', '--outdir', 'data', '--timestamp', '1626979494.000991']' returned non-zero exit status 1.


Refactor to work like hashid-rails

Currently, we require a column in the database to store the prefix ID. Using a HashID + Prefix would allow us to skip that entirely and mask the database ID. This would be much easier to use and more user friendly.
https://github.com/jcypret/hashid-rails

Looks like there a PR (that didn't get merged) to basically add this functionality to HashID (if you didn't want a separate gem): https://github.com/jcypret/hashid-rails/pull/62

Nice, wonder why it was closed.

Released as 1.1


Fix compactions

@Gouthamve I switched to starting with the smallest range. This should guarantee us that the first pick is the one we want instead of correcting afterwards.
Switched from recursive to iterative approach to – recursion is pretty but a for loop seemed easier to grasp in this particular case.
Currently have a prombench running to verify that everything works.

This is a much, much more readable version! I over complicated last time. LGTM!

Prombench run so far. Looks good aside from known spikes.

Hmm, 15+ scenario? So we only pick: p = parts[0] and hence we are not looking beyond the first partition. Which could me there could be smaller blocks in further parts that are only picked for a laaarge block size.
Can you try by iterating through the parts?

Actually no sorry for the noise, we will still pick the smaller blocks because parts won't have blocks larger than the interval.
While this could cause some extra blocks, 15+ is waaay too much. What are block-ranges being used?

Argh, just forgot a for loop. PTAL.

Can we add a test case for the part where missing the loop breaks it? Other than that nit, this LGTM

Test case for that is already in there.
On Fri, Aug 4, 2017 at 12:05 PM Goutham Veeramachaneni <
<EMAIL_ADDRESS>wrote:

Can we add a test case for the part where missing the loop breaks it?
Other than that nit, this LGTM
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/prometheus/tsdb/pull/115#issuecomment-320211058, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AEuA8q3EyJPSXVuBXevsk7P9DATyRUEvks5sUuz6gaJpZM4Osstw
.


License

Hello @newagebegins
which license has your pacman implementation? Because without a license your code is almost like closed source and nobody can legally use it or modify it. Maybe this can help: http://choosealicense.com/
Thanks for your time and this nice pacman game!

Hello @zauni
I'm glad that you liked the game! Added LICENSE.txt for MIT license.

Thank you very much!


fix the keystone package location

this pull request https://github.com/kubernetes/kubernetes/pull/23240 changed the keystone authenticator location.
this PR aims to fix that.

I'll merge this in, the plugin location looks correct near as I can tell :)


V0.2 cli visible

Description
Add maro grass mode and visible backend support
Linked issue(s)/Pull request(s)

issue_number

Type of Change

[ ] Non-breaking bug fix
[ ] Breaking bug fix
[X] New feature
[ ] Test
[ ] Doc update
[ ] Docker update

Related Component

[ ] Simulation toolkit
[ ] RL toolkit
[X] Distributed toolkit

Has Been Tested

OS:

[ ] Windows
[ ] Mac OS
[X] Linux

Python version:

[X] 3.6
[ ] 3.7

Key information snapshot(s):

Needs Follow Up Actions

[ ] New release package
[ ] New docker image

Checklist

[ ] Add/update the related comments
[ ] Add/update the related tests
[ ] Add/update the related documentations
[ ] Update the dependent downstream modules usage

Codecov Report

Merging #277 (1744f9e) into v0.2 (94af638) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             v0.2     #277      +/-   ##
==========================================
- Coverage   64.02%   64.01%   -0.01%
==========================================
  Files         110      110
  Lines        5634     5633       -1
==========================================
- Hits         3607     3606       -1
  Misses       2027     2027

Flag
Coverage Δ

unittests
64.01% <ø> (-0.01%)
:arrow_down:

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

Impacted Files
Coverage Δ

maro/rl/exploration/epsilon_greedy_explorer.py
47.05% <ø> (-2.95%)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 94af638...1744f9e. Read the comment docs.


tortoisehg: 3.7.3 -> 3.8.1

Things done

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

[X] NixOS
[ ] OS X
[ ] Linux

[ ] 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/)
[ ] Fits CONTRIBUTING.md.

mercurial was updated to 3.8.1 and breaked TortoiseHg. This is fix. Tested against nixpkgs-unstable

looks OK to me


A big bug about conflict between different version of @thatopen

Describe the bug 📝
I'm looking for a version that runs most stably without conflicts.

@thatopen/fragments

Normally I declare a variable as follows

 private _groupSerializer = new FRAGS.Serializer();
this._groupSerializer.version = 1;

When i do
const buffer = this._groupSerializer.export(group);

It thrown error

2. I use the templates.
"dependencies": {
    "@preact/signals-react": "^2.0.2",
    "@thatopen/components": "^2.0.19",
    "@thatopen/components-front": "^2.0.19",
    "@thatopen/fragments": "^2.0.7",
    "@thatopen/ui": "^2.0.8",
    "@thatopen/ui-obc": "^2.0.12",
    "jszip": "3.10.1",
    "three": "0.160.1",
    "web-ifc": "0.0.55"
  }

I did something for tiler
  private wasm = {
    path<EMAIL_ADDRESS>    absolute: true,
    logLevel: WEBIFC.LogLevel.LOG_LEVEL_OFF as WEBIFC.LogLevel | undefined,
  } as const;

  private excludedCategories = new Set([
    WEBIFC.IFCSPACE,
    WEBIFC.IFCREINFORCINGBAR,
    WEBIFC.IFCOPENINGELEMENT,
  ]);
async streamIfc(buffer: Uint8Array) {
    /* ==========  IfcGeometryTiler  ========== */
    const tiler = this.components.get(OBC.IfcGeometryTiler);
    tiler.settings.wasm = this.wasm;
    tiler.settings.autoSetWasm = false;
    tiler.settings.webIfc = this.webIfc;
    tiler.settings.excludedCategories = this.excludedCategories;
    tiler.settings.minGeometrySize = 10;
    tiler.settings.minAssetsSize = 1000;
    tiler.onAssetStreamed.reset();
    tiler.onGeometryStreamed.reset();
    tiler.onIfcLoaded.reset();

    let assets: OBC.StreamedAsset[] = [];
    let geometries: OBC.StreamedGeometries;
    let groupBuffer: Uint8Array;
    let geometryFilesCount = 0;
    const onSuccess = async () => {
      const customIfcStreamer = this.components.get(CustomIfcStreamer);
      if (!customIfcStreamer) return;

      if (
        assets.length === 0 ||
        geometries === undefined ||
        groupBuffer === undefined
      )
        return;
      const settings = {assets, geometries} as OBF.StreamLoaderSettings;
      await customIfcStreamer.load(settings, groupBuffer, true);
    };

    const streamGeometry = (
      data: OBC.StreamedGeometries,
      buffer: Uint8Array
    ) => {
      const geometryFile = `small.ifc-processed-geometries-${geometryFilesCount}`;
      if (geometries === undefined) geometries = {};
      if (!this.streamedBufferFiles) this.streamedBufferFiles = {};
      for (const id in data) {
        if (!geometries[id]) geometries[id] = {...data[id], geometryFile};
      }
      if (!this.streamedBufferFiles[geometryFile])
        this.streamedBufferFiles[geometryFile] = buffer;
      geometryFilesCount++;
    };

    tiler.onAssetStreamed.add(async (assetItems: OBC.StreamedAsset[]) => {
      assets = [...assets, ...assetItems];
      await onSuccess();
    });

    tiler.onGeometryStreamed.add(
      async ({
        data,
        buffer,
      }: {
        data: OBC.StreamedGeometries;
        buffer: Uint8Array;
      }) => {
        streamGeometry(data, buffer);
        await onSuccess();
      }
    );

    tiler.onIfcLoaded.add(async (group: Uint8Array) => {
      groupBuffer = group;
      await onSuccess();
    });
    await tiler.streamFromBuffer(buffer);
  }

Then i customized this class. and load
/**
   * Loads a new fragment group into the scene using streaming.
   *
   * @param settings - The settings for the new fragment group.
   * @param coordinate - Whether to federate this model with the rest.
   * @param properties - Optional properties for the new fragment group.
   * @returns The newly loaded fragment group.
   */
  async load(
    settings: OBF.StreamLoaderSettings,
    groupBuffer: Uint8Array,
    coordinate: boolean
  ) {
    const {assets, geometries} = settings;
    const fragments = this.components.get(OBC.FragmentsManager);
    const group = fragments.load(groupBuffer, {coordinate});
    this.world.scene.three.add(group);
    const {opaque, transparent} = group.geometryIDs;
    for (const [geometryID, key] of opaque) {
      const fragID = group.keyFragments.get(key);
      if (fragID === undefined) {
        throw new Error("Malformed fragments group!");
      }
      this.fragIDData.set(fragID, [group, geometryID, new Set()]);
    }
    for (const [geometryID, key] of transparent) {
      const fragID = group.keyFragments.get(key);
      if (fragID === undefined) {
        throw new Error("Malformed fragments group!");
      }
      this.fragIDData.set(fragID, [group, Math.abs(geometryID), new Set()]);
    }
....

It did not work. So i try to debug at
const {opaque, transparent} = group.geometryIDs;
console.log(group.geometryIDs);

This is result

Then i tried to customized this class (i mean i copied exactly). and debug at this.
And this is result :

After that i tried to change for wasmmodule url :
  private wasm = {
    path<EMAIL_ADDRESS>    absolute: true,
    logLevel: WEBIFC.LogLevel.LOG_LEVEL_OFF as WEBIFC.LogLevel | undefined,
  } as const;

this result for<EMAIL_ADDRESS>
this result for<EMAIL_ADDRESS>
this result for<EMAIL_ADDRESS>
3<EMAIL_ADDRESS>So i downgrated to older version
 "openbim-components": "^1.5.0",
  "bim-fragment": "^1.5.0",
  "web-ifc": "^0.0.53"

Every thing worked. this file for testing.
Reproduction ▶
No response
Steps to reproduce 🔢
No response
System Info 💻
Chrome
Win10

Used Package Manager 📦
npm
Error Trace/Logs 📃
No response
Validations ✅

[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a repository issue and not a framework-specific issue. For example, if it's a THREE.js related bug, it should likely be reported to mrdoob/threejs instead.
[X] Check that this is a concrete bug. For Q&A join our Community.
[X] The provided reproduction is a minimal reproducible example of the bug.

@RyugaRyuzaki I think you are mixing things up. The original bug (changing the version of the serializer) is the problem. I'll change the name of the issue.

@RyugaRyuzaki I think you are mixing things up. The original bug (changing the version of the serializer) is the problem. I'll change the name of the issue.

@agviegas i also mean when we change @web-ifc version the output also change as i show the results.

@RyugaRyuzaki you have to create one issue per problem. Otherwise it's really hard to track things down. I've solved the serializer error in<EMAIL_ADDRESS>If you have an issue with web-ifc versioning in the libraries, please create a separate issue.


Hover over quick pick

I had the mouse over the explorer item and opened quick pick and then the hover popped over the quick pick

Version: 1.87.0-insider (user setup)
Commit: 019f4d1419fbc8219a181fab7892ebccf7ee29a2
Date: 2024-02-27T23:37:17.140Z
Electron: 27.3.2
ElectronBuildId: 26836302
Chromium: 118.0.5993.159
Node.js: 18.17.1
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22635

I understand why this might seem like a bug, but I'm not sure having the hover underneath the quick pick is a good idea. There are might be scenarios where showing the hover underneath the quick-pick might seem like a bug.
@Tyriar what do you think?


Support container start and stop

Currently it looks like you can really only:

Create a container
Delete a container
Restart a container

Seems like being able to start/stop containers would be very useful. I don't want to have to search for a container and create a new one every time I want to start and use it.
I often use docker to try out some new app/version, or to spin up a dev environment when I need it. However I want to be able to stop a container and restart it later - not recreate it. Stopped containers should be visible in the container list in the UI (left sidebar in current UI).

Hi there! Right now stopped containers are visible (if you run docker stop <container>) they'll grey out. That said there's currently no way to stop them via the interface - added a label.
  Would be very welcome to have one!
+1

+1 for a stop button as well - love the app btw ;-)

+2
1 for each start and stop ;)

+1

+1000
Guys this is a severe UX flaw... I've spent 20 minutes in the GUI looking for the stop button before starting to google the solution... who doesn't immediately look for stop/play buttons? xD Hope you fix it ASAP

@Telemakhos thanks for the note!!

+1

+1

+1

+1

+1

+1

Quick update, work in progress here: #433

We have added the container stop button. Thanks!


fix: header tooltips bug #1606

This fixes #1606
https://user-images.githubusercontent.com/26791103/103573617-4e021480-4ead-11eb-83bd-c29a2132b5ab.mov

@leandroercoli
thanks pr
Does dragging work normally?

@leandroercoli
thanks pr
Does dragging work normally?

@wdh2100 yes, since dragging is being controlled by useDrag, this line of code was not affecting that functionality, but only closing the tooltip when the column's name was pressed (even if the intention was not to drag the column).
As shown on the video posted, draggable columns are working with this fix.

@wdh2100 yes, since dragging is being controlled by useDrag, this line of code was not affecting that functionality, but only closing the tooltip when the column's name was pressed (even if the intention was not to drag the column).
As shown on the video posted, draggable columns are working with this fix.

@leandroercoli
LGTM 👍

@leandroercoli
LGTM 👍


fix foreground message to not imply that debug mode is on

Right now, running with -f or --foreground prints:
Staying open for debugging [pid: 251]

Which is kinda confusing and I was worried if some debug functionality was exposed.
This PR will print the above message only if -D or --debug is given.

Something like Running in foreground or so.
@psychon I have changed the message to Running in foreground. It will also always right the Pidfile.


Docker Container?

Any plan on creating a docker container?

as much as I believe author of this repo has got docker support. still if you are looking for something easy pickings than what is available here. please check https://github.com/The-Bike-Rentals/messaging-api, it has actions written to publish docker to dockerhub  https://hub.docker.com/repository/docker/hiteshdutt/whatsapp-api/general.
it also has deployment to AKS deployment script added.


Pss integration tests

@vandot Hi! Once this is approved and merged, we may need to release a new version, so I can run this test in the infrastructure tests.


dot is valid for username

Currently Drone doesn't allow (.) in username but thats valid (at least in gitlab)
example: https://gitlab.com/hadi.farnoud

 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.

@bradrydzewski I guess this would be safe to merge?

Any update?

I'm also waiting on this guys @tboerger @bradrydzewski

Thanks for you patience everyone and special thanks to @okian for the fix and tests. 🎆