Error during using internal API for model resources

I created a generic resource and three different model resources in my local development environment as the following figure:

After that, I tested the internal API to get the objects for each resource type with these steps:
[1] $  docker exec -ti hydroshare_hydroshare_1 python manage.py shell
[2] For Generic Resource, inside the shell I wrote the following:
 In [1]: from hs_core.models import GenericResource
 In [2]: GenericResource.objects.all()
Out[2]: [<GenericResource: Generic resource creation>]

 The output was as expected which is the created Generic Resource: Generic resource creation.

[3] I did the same for the Model Instance resource (Figure below) and I got unexpected results. All the four created resource show up as Model instance resource.

[4] I did the same trial but this time for the Model Program Resource (Figure below) and also all the resources were listed as Model Program resource.

@pkdash suggested to change line #98 in hs_modelinstance/models.py from "objects = ResourceManager()" to "objects = ResourceManager(‘ModelInstanceResource’)." I did this solution which solved the problem. Also I did the same solution for Model Program resource and SWAT Model Instance resource and they work as expected.

@mohamedmorsy  the "line 98" you mentioned is in which branch?
@pkdash Does it mean all other apps should do the same changes in resource  class def? Do we need to redo migration after this change? I am wondering is this related to the edit permission problem we worked yesterday?
Thanks

@zhiyuli What I mentioned is just an exmaple to solve the problem for the Model Instance resource type (on develop branch). Based on my conversation with Calvin on HipChat, He will take care of this bug ASAP.

I have attached a PR that should address the issues. Please review @mohamedmorsy and test.

The RefTimeSeries res type class does not follow the naming convention, which will cause problem (see #635). It should be ending with "resource".  Also, the RefTS class def in dev does not fit new base model.
In the RefTS new design (branch big-merge-502-HISTimeSeries-new-design) I will fix them. So technically the RefTS in dev cannot work.

@zhiyuli so does that mean this should still merge or that it cannot?

+1

@ironfroggy
+1
It  can be merged
P.S. Do you have some time looking into #636? Pabitra and Hong were working on it yesterday and we need your help. Pabitra already reproduced this error on his local VM (see hipchat). Thanks a lot

@mjstealey Just making sure we are still holding this merge until after migration.

@mjstealey Please disregard my previous comment, this should be included before the migration.

+1, to be merged before migration for release 1.6


Incorporating Modified ALZ Modules - Pipeline failed after following the example

Let us know the feedback or general question
Followed all the steps below:
Example: Steps to follow for ALZ-Bicep Logging, Automation & Sentinel Module
Copy logging.bicep module file from upstream-releases directory to config\custom-modules directory
Modify the copied module file in custom-modules directory as needed. If new parameters are added to the module, you will need to update the relevant parameter file in the config\custom-parameters directory as well.
Update the config\custom-modules\logging.bicep file with the following comment at the top of the file:
// This module has been modified from the upstream-releases version
Update the pipeline-scripts\Deploy-ALZLoggingAndSentinel.ps1 file and change the TemplateFile variable to point to the modified module file location as shown below:
[Parameter()]
[String]$TemplateFile = "config\custom-modules\logging.bicep",
In order to trigger new deployments when subsequent changes are made, add the new module file path to the path-based filter workflow trigger in the ALZ-Bicep-1-Core workflow file as shown below:
on:
push:
paths:
- "config/custom-modules/logging.bicep"

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Hi @jamesyoung007, the error looks to be related to the customerUsageAttributes module that is referenced at a particular location in relation to the logging module location.  This module is only used by us to track telemetry data.
To get around this error, you can copy that module into your custom-module folder and then adjust the path within your logging module, which would be line 249 for the default location. You could also just remove the module reference entirely if you are already opting out as defined by the parTelemetryOptOut parameter.

I'll also adjust the wording that for the Accelerator documentation to call this out as something to look out for when moving a module to the custom-modules directory.

Sounds good, thanks for the heads up!
On Wed, Mar 13, 2024 at 1:50 PM Zach Trocinski @.***>
wrote:

I'll also adjust the wording that for the Accelerator documentation to
call this out as something to look out for when moving a module to the
custom-modules directory.
—
Reply to this email directly, view it on GitHub
https://github.com/Azure/ALZ-Bicep/issues/737#issuecomment-1992878023,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFJAQY4HW6PASLH55W5TITTYX6PF7AVCNFSM6AAAAABEL2EQFCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOJSHA3TQMBSGM
.
You are receiving this because you were mentioned.Message ID:
@.***>


[BUG] "" Is not processed

Describe the bug
When using \ to use a link alias, the "" doesn't get processed and the link gets split to two columns.
To Reproduce
Steps to reproduce the behavior:

In a table, [[enter a link\|with an alias]]
Open the markdown table editor
Two columns show up, split at |

Expected behavior
There should be only 1 column, with the alias displayed properly.
Screenshots
Error/multiple columns:

What it normally looks like:

Desktop (please complete the following information):

OS: Windows
Version 10

Does this also happen in the markdown editor as well?
I have a table as follows

If I go to view mode, the column is broken exactly like how it happens in the plugin's editor.

Same in GitHub markdown preview mode as well

So, it looks like we are honoring the general markdown parsing logic.  Fixing it in the plugin alone will solve the issue?

This is the state of the latest code. Looks good?

Don't be sorry! Your input is really helpful :)

Thank you very much @ganesshkumar I'm happy to know my comment helped !!!!!!!

@shunnkou. I have fixed the issue and made a new release (0.1.1).  Also, the plugin is available in the community plugin list. You can download it from there directly 🎉


Create Bit.Platform.Website project

this closes #1890

@ysmoradi
plz re-review this PR after new changes

PING !


Add support for reading files from S3

Support for this via the "External File" data source option should be pretty easy and extremely helpful for those of us using AWS EMR for deployment. I attempted to implement this myself, but couldn't figure out how to make sure the hadoop-aws library was included in the local-mode runtime:
https://github.com/s22s/seahorse/commit/909d37852975d88a205d4c3ee98f769e4a3430d9
So I couldn't test the implementation, at least not in local/development mode.
FWIW: The following schemes are supported by  org.apache.hadoop:hadoop-aws:

Scheme
Service Provider

s3
org.apache.hadoop.fs.s3.S3FileSystem

s3a
org.apache.hadoop.fs.s3native.NativeS3FileSystem

s3n
org.apache.hadoop.fs.s3a.S3AFileSystem

cc: @mteldridge @mobsy74

@metasim some info:

hadoop 2.7 is shipped with hadoop-aws version supporting s3, s3a and s3n
current hadoop-aws only supports s3a
credentials would be passed in core_site.xml (as described here https://www.cloudera.com/documentation/enterprise/5-14-x/topics/cm_s3_clients.html). This means some extra external configuration, but also means that credentials would be protected
only one s3 account per seahorse is available

@jaroslaw-osmanski Thanks for the additional, helpful info. All that sounds good to us, and we're comfortable with those constraints/requirements.

Proof of concept:
https://github.com/deepsense-ai/seahorse/pull/92


Allow extensions to handle saves of unmodified files

TL;DR - please could we have an on-save event that fires under the same situations that VS Code invokes the on-save formatter.

There's a difference between how VS Code handles format-on-save versus what extensions are able to do with their own actions on-save, because the save handlers don't fire if the user explicitly saves a file that has been auto-saved, yet VS Code does run the formatter.
For example:

enable autoSave: afterDelay
edit a file - and note that VS Code will suppress the call to the formatter when it auto saves (this makes sense)
press Save/Cmd+S and notice that the formatter runs

If you want to implement something in an extension (let's say when the user modifies pubspec.yaml, you want to run pub get to fetch the packages, but you don't want to do it on auto-saves as the file may be incomplete), it fails - because the last step above (pressing Save/Cmd+S) does not invoke onWillSave/onDidSave.
This means we have to have clumsy workarounds - currently for Dart we start a 10second timer after an auto-save, and keep pushing it back on subsequent saves. This means we will eventually fetch packages when we think the user has finished modifying the file, but it's kinda hit-and-miss (sometimes it'll fire while they've paused to think, and sometimes they have to wait up to 10 seconds for it to do what they need).

Another place this would help - "Hot Reload on Save". Currently if you have auto-save turned on, we will hot reload on every auto-save. If we do it only for manual saves, then a user who has auto-save on will have to hit save before the auto-save delay in order for it to work, otherwise they will get no Hot Reload on Ctrl+S.

it fails - because the last step above (pressing Save/Cmd+S) does not invoke onWillSave/

No the onWillSave will always fire, check this. @DanTup please provide a minimal sample extension that demonstrates what you are claiming

Hmm, that's not the behaviour I'm seeing. I've got handlers that print out in these two events:
context.subscriptions.push(workspace.onDidSaveTextDocument((td) => {
	console.log(`did save ${fsPath(td.uri)}`);
}));
context.subscriptions.push(workspace.onWillSaveTextDocument((e) => {
	console.log(`will save ${fsPath(e.document.uri)}: ${TextDocumentSaveReason[e.reason]}`);
}));

And then in my document formatter I also log:
public async provideDocumentFormattingEdits(document: TextDocument, options: FormattingOptions, token: CancellationToken): Promise<TextEdit[]> {
		console.log(`Running document formatter for ${fsPath(document.uri)}`);

Here's what I see logged:
// Type something then pause
will save /Users/dantup/Dev/Google/flutter_web/examples/spinning_square/lib/main.dart: AfterDelay
did save /Users/dantup/Dev/Google/flutter_web/examples/spinning_square/lib/main.dart

// Wait 10 seconds, then press Cmd+S
Running document formatter for /Users/dantup/Dev/Google/flutter_web/examples/spinning_square/lib/main.dart

Neither onWillSave of onDidSave fired when I pressed Cmd+S at the end (but the formatter ran).

Oh, hangon - I see what's happening. If the formatter throws, then the onWillSave does not fire. Is this intended, or should it run even if a formatter throws?

That said, there's still an issue here - we can't start anything in onWillSave because the document hasn't saved yet. We need to run after the file is committed. Is there a good way to do that other than just setTimeout with some small delay we think is long enough?
Thanks!

If the formatter throws, then the onWillSave does not fire. Is this intended, or should it run even if a formatter throws?

Not sure, it seems to have been like that ever since. Tho, I think the formatter shouldn't throw in the presence of syntax errors but just not do anything.

we can't start anything in onWillSave because the document hasn't saved yet

Well, if the document doesn't change, e.g if the document doesn't get modified during will-save, then we don't save it and therefore no did-save fires. TBH I think that behaviour is sound and I am not sure if listen for saving is good for your use-case. Why don't you listen on modification and check if a file is valid before doing any follow-up work?

Tho, I think the formatter shouldn't throw in the presence of syntax errors but just not do anything.

Yeah, I agree with that. I think our handling is bad. But still, I think if there was a legit error during formatting though, I wouldn't expect my willSave handler not to fire. I think maybe that should be changed (or at least, the behaviour documented - especially if a formatter from another extension could cause it - I don't know if that's the case).

Well, if the document doesn't change, e.g if the document doesn't get modified during will-save, then we don't save it and therefore no did-save fires. TBH I think that behaviour is sound

In some ways it sounds reasonable, but the naming of onWillSave also sounds like it will save :)

Why don't you listen on modification and check if a file is valid before doing any follow-up work?

It would still feel really weird performing either of these actions on auto-save, even if we abort then the file isn't valid. For ex., if in my dependencies I'm adding:
my_package: 1.2.3

If I pause briefly after typing 1.2, the file is still valid, and we'll start fetching the packages needlessly, then when I type .3 we'll cancel it and start it again. If you're typing at just the right (wrong) speed, we'll be spawning and killing processes. Really we want an explicit user action to trigger it (Cmd+S is most natural).
Hot Reload is similar - if I've enabled hot-reload-on-save and also VS Code's autoSave, I probably don't want auto-saves to perform a hot reload (because I'm likely to pause while typing code - which may be valid - to think, and although hot reload is fast, it's distracting and I might have wanted to keep the device in the state it was in while I type my changes).
In my head, it seems like an obvious event to have, but when I try to describe it sounds rather specific. I guess it it's most applicable to when you want to do something on-save, but that work is expensive or destructive, so you want only user-initiated events.
How about if onWillSave handlers could register a callback to run after the save (with an option for whether it should run even if there are no changes)? That could also improve the weird niggle that if you want to do something in onDidSave but want to know the reason for the save, you have to also hook onWillSave and then stash the reason in a variable. I think the idea of being able to do something after a save but also knowing whether it was manual or auto is reasonable? :)
I see there's already a waitUntil on TextDocumentWillSaveEvent, maybe there could be something similar?
context.subscriptions.push(vs.workspace.onWillSaveTextDocument((e) => {
	e.afterSave(() => {
		if (e.reason === vs.TextDocumentSaveReason.Manual) {
			// Do something here
		}
	}); // + some flag to run even if there were no changes made?
}));

If I pause briefly after typing 1.2, the file is still valid, and we'll start fetching the packages needlessly, then when I type .3 we'll cancel it and start it again. If you're typing at just the right (wrong) speed, we'll be spawning and killing processes.

You can never avoid that with async operations. What would you do if the user presses Cmd+S repeatedly? With or without changes? You will always debounce and cancel/discard work. In your sample, type 1.2, save, type .3 without save. I think what matters here is that the operation is async, not how/when you trigger it.
For less lightweight operations, esp. those that sync from state kept in files, you need onWill/DidSave (file watching might even be better) but I still think everything is there. Why would you kick off hot reload when the file wasn't changed (e.g. no didSave-event)? The state on disk is as before, no follow up work. What will hot reload change when nothing in the file has changed?
It seems what you want is to piggyback on the cmd+s shortcut, independent of save or not. Kinda like listening to a command being executed so that another things can run...

You can never avoid that with async operations

I can't avoid having to cancel and re-run, sure - but that doesn't mean I have to keep firing while the user is typing - that's a bad experience (esp. when there's a visible progress indicator on screen while the process runs).
As a I user, I want the operation to happen when I press Cmd+S. Always. Never on auto-save, but always on Cmd+S. Cmd+S is my shortcut for "save this file (if it needs it) and run anything that is appropriate to do when I save this file". Pressing Cmd+S on an undirty file is sort-of a way of saying "I want you to do the on-save things, even though this file isn't modified". That VS code already runs the formatter in this case shows that there are good reasons for it :-)

Why would you kick off hot reload when the file wasn't changed (e.g. no didSave-event)? The state on disk is as before, no follow up work. What will hot reload change when nothing in the file has changed?

It's exactly the same as formatting. Why doesn't VS Code auto-format on an auto-save? Because it'd be (really!) annoying. Yet it does then format when I press Cmd+S even though there are no changes.
My action is basically the same - something that shouldn't be auto-triggered, only from a user-action - however it's important that it runs after the file is committed (unlike for formatting, which can run on the in-memory file buffer).
As a workaround today, we just debounce in the auto-save handler - however there's no good value to sleep for. We currently have it set for 10 seconds, but for auto-save users, that means this happens:

User makes an edit
VS code auto-saves - we start a 10 second timer
User pressed Cmd+S - nothing happens because onDidSave didn't fire
10 seconds later, action runs

This is painful for the auto-save users. We can reduce the timer from 10s, but that makes it more annoying if the user is pausing while typing. There's not perfect value to set. Maybe we can hook onWillSave, see if it's manual, and there's an ongoing timer, then cancel the timer and run immediately, but it's starting to seem complicated/fragile.

It seems what you want is to piggyback on the cmd+s shortcut, independent of save or not. Kinda like listening to a command being executed so that another things can run...

Yeah, pretty much exactly this - but it's important that it always fires even when there was no real save, and that we don't have to keep track ourselves of what the save keybinding is if the user has changed it (or uses File -> Save, etc.). Again, this seems to be exactly what formatting does, except for it's ok to run pre-save because it doesn't use the file off disk for its actions.
There is also the code actions that run on save, but I've not tested when they fire, mainly because I don't think there's a good way for an extension to register them without contributing a setting that would overwrite the ones the user has configured themselves.

Yet it does then format when I press Cmd+S even though there are no changes

Before invoking format we don't know that there are no changes. After we know and then we won't make changes on disk unless the file has been modified.
I am closing this as dupe https://github.com/Microsoft/vscode/issues/1431 (and/or https://github.com/Microsoft/vscode/pull/72345) because you aren't interested in saving but in press+s being pressed

I am closing this as dupe #1431 (and/or #72345) because you aren't interested in saving but in press+s being pressed

Seems reasonable - but just to be clear, would this support things like File->Save too? Does that just fire the same save command?
Thanks!


Can't parse Geth node's ENR

I find it can't parse Geth node's ENR like this:
enr:-Je4QIdx4eWpaemy4ECbJU2ZsXvzX3XsiF10QOjm-87XBkoaJdEq3qjijY7fMkhP1SZRUeqHodSczUCuj01eWWzQ2xc7g2V0aMfGhOAp6ZGAgmlkgnY0gmlwhLecb4iJc2VjcDI1NmsxoQPnve6T4T8AWndeYCCi-zJncpl_W_SR8Wky_rUI8MxR24N0Y3CCdmCDdWRwgnZg

I get Error Custom("List decode fail")
But It can be parsed well by Geth.

Hey!
Sorry, I missed this issue.
The ENR given here looks like it has not been RLP encoded correctly. Can you give me the contents of the ENR and/or how it was created?
I've not seen other issues with geth so far, so this is strange.

I guess the above ENR is similar to the ENR as obtained from the discv4-dns-lists repo (e.g. https://github.com/ethereum/discv4-dns-lists/blob/master/all.mainnet.ethdisco.net/nodes.json) - Which would likewise be a decode fail (testing through enr-cli).

Awesome thanks, I'll look into this.

So this appears to be that geth is using extra lists in some of their ENR keys. This should be a straight forward fix. I'll write out another issue and we should have this corrected very soon.
I'll close this issue once it's merged


Memory error bij nieuwe library, <IP_ADDRESS>11223210536

MemoryError: Unable to allocate 29.5 MiB for an array with shape (2339, 1653) and data type float64

Hi Jolanda,
I will look into that issue.
Maybe there are some possibilities how I could reduce the memory
consumption a bit...
BR,
Many
Jolanda @.***> schrieb am Do., 24. Feb. 2022, 11:19:

Reopened #38
https://github.com/manykarim/robotframework-doctestlibrary/issues/38.
—
Reply to this email directly, view it on GitHub
https://github.com/manykarim/robotframework-doctestlibrary/issues/38#event-6131121098,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOTUE3E3TOMCYV7HDDFUTBDU4YA3ZANCNFSM5PG6CQ3Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you are subscribed to this thread.Message
ID:
<manykarim/robotframework-doctestlibrary/issue/38/issue_event/6131121098@
github.com>


Feature request: Globed menu in level/pause menu

It would be very convenient to be able to access the Globed menu and server levels without having to back out all the way to the main menu first.

Added the server levels button in the level menu in the v0.3.1 release, though the positioning might be changed in the future


Kysely + Objection.js

Is there any intent of switching from Knex.js to Kysely in the future on Objection.js?
Nice work by the way.

Thanks! Probably not, for the same reason I'm not contributing this to knex. It would mean a significant rewrite and API change.

@koskimas You are totally right. Keep up the good work.

to that point, @koskimas curious if youd build objectionjs or a new orm from the start?

I am curious as well :) maybe it will be part of this package directly :)

Will there be a new ORM like Objection?

@kasvith you can try https://github.com/seeden/kysely-orm

@seeden it seems really nice

drizzle orm also feels like objection but it is fully type-safe

tbh drizzle feels like prisma, where objection was much more like a wrapper around a query builder. we still use it in production

I also use Objection in production and it's great. But seeing that Objection has slowed down development and its lacking type safety, moving forward I'm looking for alternatives. What I love about objection is that it allows customisable querying of deep table relations into a JSON structure without too much magic (withGraphFetched). And I can still include raw sql or use knex alongside to run more complex raw queries. IMO drizzle also feels like a thin layer over sql similar to knex/kysely, but it has the added benefit of being able to easily specify and query relations into JSON objects, kind of like Objection. Prisma feels like a higher and heavier layer which abstracts away differences between SQL flavours, and it may generate an arbitrary number of sql queries based on your query. For me the biggest issues with drizzle are:

it's quite new and unstable (0.x)
it needs "lateral join" support which only the newest mysql/postgresql versions support and many hosted DB services like aws aurora and planescale do not support

i wish objection had continuous development :( ngl apart from typesafety its the best ORM even today


Document Antrea Prometheus metrics

Prometheus integration offers a list of metrics on both controller and agent components.
The metrics should be enlisted in a reference document with a short description for each metric.

@ksamoray  Understanding of network policy/addressGroups/appliedToGroups related metrics at controller is probably dependent on following documentation that is referred in ovs-pipeline doc:

The Antrea implementation of K8s Network Policy, including the communication channel between the Controller and Agents, and how a Network Policy is mapped to OVS flows at each Node, will be described in details in a separate document.

@ksamoray Understanding of network policy/addressGroups/appliedToGroups related metrics at controller is probably dependent on following documentation that is referred in ovs-pipeline doc:

The Antrea implementation of K8s Network Policy, including the communication channel between the Controller and Agents, and how a Network Policy is mapped to OVS flows at each Node, will be described in details in a separate document.

I have started working on such a document, but i will take a bot of time to do correctly.


Error in Running test_LA.py

Hi,
I tried to run the test_LA.py file and I got the following error:
FileNotFoundError: [Errno 2] No such file or directory: '../model/DTC_16labels/best_model.pth'
I wonder if you can help me.
Thanks

Hi,
You can set your own checkpoint or file path.
Best,
Xiangde.


Empty other warden scopes upon login

To finally fully disable "multilogin" and related errors, as we don't support multiple accounts.

Found how to do it, inside theWarden::Manager.after_set_user, do:
  auth.env['rack.session'].each { |key|
    key.start_with?("warden.user") && !key.end_with?(".session")
  }.each { |warden_key|
    # kill session/scope
  }


uf2 won't flash

When I drop chipicopwn.uf2 onto my Pico, nothing happens - it doesn't flash and reboot. Works fine with my own uf2s or any other uf2 I've tested with to confirm. No difference if I build it from source or use the one from this repo under /build

same here. Maybe they have fixed the exploit with a new firmware


add more options to handle errors in service registration

[ ] Current option 'Try Again' -> Should retry the registration API request. Ensure the state in URL is correctly handled
[ ] Add new option 'Upload New File' -> Should have the old behavior of 'Try Again'.

I'll take this


CMake support

Could CMake support be considered, so that you can just plug the tool into a codebase with FetchContent and it'll give you the handy explanations automatically?

I hadn't seen that but you could do this:
cmake 2> >(cwhy)

The above sends stderr to cwhy, which now does nothing if it receives no input.

That worked for the CLI! Any way to do the same in an IDE setup where cmake is invoked for you?

No doubt it is doable, but I imagine it varies considerably per IDE (and depends on where you want the results to appear). That said, we'd love to see integration into something like Visual Studio.

Closing this as CWhy does not really work at the level where FetchContent would be appropriate. Our current CMake support is pretty good IMO.
% cmake -DCMAKE_CXX_COMPILER=`cwhy --llm=gpt-4 --wrapper --wrapper-compiler=clang++` ...


Implement ToSql/FromSql for f32

This just seems to be an oversight.

https://github.com/rusqlite/rusqlite/issues/209#issuecomment-274346049

Implementing FromSql for f32 would force us to provide a FromSql that was losing precision from the values we get from SQLite.

Yeah that's exactly the behaviour I'd expect. Also that only applies to FromSql, not ToSql. Maybe this task should be split up.

Ok, thank you!

Ok, thank you!


Get WS2801 working

Is there any possibility to get a WS2801 strip correctly working?

Duplicate of #47

Solved in recent edit where we removed the custom ws2812 library, thanks @d-rez for flagging this.


Fix bad eslint change.

Fixes #5528, which was caused by an eslint change in #5483.
/cc @mramato @omh1280

This wasn't released, so no mention needed in CHANGES.md.
@moneimne This will be a small but important merge conflict to bring into your branch.

Thanks Ed! Could you explain how my change broke it? What's the difference between var binary and var binary=undefined?

The difference is several lines above:
for (var id in textures) {

The code is executed multiple times, and old values were surviving!

Ah, gotcha. Thanks

Thanks @emackey!

Thanks @emackey, sorry I missed this when I merged #5483.
The ship has sailed, but I'm surprised this wasn't caught by a unit test.

@mramato We could still add a unit test, but keep in mind that model code is already refactored for the glTF 2.0 effort, and these lines in particular don't exist in this form on @moneimne's 2.0 branch.


Specify which keys are duplicate and fix front-end bug

Fixes #95

@AlmightyYakob can we delete this branch?

@AlmightyYakob can we delete this branch?

Sure thing


Add WIKIBASE_CONCEPT_URI

Without this either wdqs import or wdqs-updater doesn't work.
Basically simplified version of https://github.com/wmde/wikibase-docker/pull/74 made against the latest version.

@Nikerabbit is there a phabricator ticket for this so that I can add it to the correct board?

I didn't create a task for this specifically. It's related to https://phabricator.wikimedia.org/T263427 though where I was told https:// in CONCEPT_URI is not supported.
Would you like me to create a task for this?

Not sure what's up with the tests. Is that something caused by the patch?

Not sure what's up with the tests. Is that something caused by the patch?

I believe https://github.com/wmde/wikibase-docker/blob/master/wikibase/1.35/.travis/build-deploy.sh will always "fail" for pull requests (which aren't supposed to immediately push to docker hub). The travis setup builds docker images, but doesn't run them and/or any tests.
I'm sure there are Phabricator tickets asking for better test coverage. If so, this is the moment to shut them down with a link to the Travis CI end-of-life announcement and, in passing, lord it over them what a waste of time it would have been.

This should be created as a feature request against the new WMDE maintained docker images.
https://phabricator.wikimedia.org/project/view/5063/


Fix SurfaceTool crash with invalid bone/weight count

Fixes #53191.
Supersedes #53201 where the base branch was deleted.

Thanks!


Incabable mod-Conjuring

Mod Name
Conjuring
Mod Homepage
https://www.curseforge.com/minecraft/mc-mods/conjuring
Describe the bug
Error: java.lang.RuntimeException: Failed to initialized owo shader program
Steps to reproduce
1.download conjuring mod on forge 47.1.3
2.download the two fabric mods sinytra connector and forge fabric api
3.crashes on launch
...
Logs
https://pastebin.com/hHS41Rat
Additional context
No response

Incompatible*

Did install the owo lib?

This is invalid the mod doesnt crash


How do I download the skeleton data on the server, because baidu disk can not get the download link

Hi, about the Kinetics skeleton (7.5gb) skeleton data, you said you can download directly from GoogleDrive or BaiduYun. But if I need a link on the server to download the skeleton data, but I found baidu disk can not get the download link address, what should I do. My sincere thanks

I can enter Baidu disk from the link on the official website and download it， this link is publicly accessible.Can not access Baidu  disk in foreign countries？


Correctly generate a Temp dir for TDBlobStore

What
Correctly generate a temporary directory for TDBlobStore.
How
Use NSTemporyDirectory() to get a temp dir path.
Testing
No new tests, encrypted blob store tests now pass on Jenkins with this change.
Reviewers
reviewer @ricellis
Notes
The main reason for the change is documented here: http://nshipster.com/nstemporarydirectory/
Long story: the current way we are generating tmp dirs is incorrect, the current way is designed for moving files around.

+1

Although the travis push build job "errored" the test did run successfully. According to the log  attached, so merging.
travis-log.txt


New instructions for connecting Flatpak Wine/Proton games to native Discord RPC clients

Hey, absolutely awesome project! I love how you've created a service that runs in the background and bridges the Windows pipe to the UNIX pipe. :)
On that note, the entire section about Flatpak apps is using a deeply broken, unfixable method of linking to the host socket.
Here are the resources that describe the best way of linking between native Discord clients and your Flatpak games:

https://github.com/Arcitec/discord-flatpak-rpc-bridge
Additional information at the arRPC project, which is an open-source Discord RPC implementation: https://github.com/OpenAsar/arrpc/issues/122

Installing that bridge is all you have to do. The Flatpak apps (such as Heroic, Steam, Lutris) must of course also be correctly configured to use the RPC sockets, as described here, but that's a job for the Flatpak app maintainers.

Hey, absolutely awesome project! I love how you've created a service that runs in the background and bridges the Windows pipe to the UNIX pipe. :)

Thank you!

On that note, the entire section about Flatpak apps is using a deeply broken, unfixable method of linking to the host socket.
Installing that bridge is all you have to do. The Flatpak apps (such as Heroic, Steam, Lutris) must of course also be correctly configured to use the RPC sockets, as described here, but that's a job for the Flatpak app maintainers.

Hmm, looks like a good idea. I will have to test myself when I got more free time and maybe automate it?

@EnderIce2 Feel free to use the project for anything you want, or even just tell people to install it if you detect that the Flatpak RPC socket is missing. It requires sudo to install the service, and must be installed via the host (not inside the Flatpak environment), so that's a problem for automating installation. I don't think you'll be able to automate it from the Wine side.
discord-flatpak-rpc-bridge is however 100% complete and won't need any changes, so what you see there is the finished project.
Systemd sets up a listening socket at the Flatpak RPC path, then as soon as a Flatpak tries to use the Discord RPC socket, systemd launches the proxy service and gives ownership of the listening socket to the proxy process, bridging the Flatpak socket to the real native Discord RPC socket.
I wrote it in a way that people just ./install.sh a single time per computer, and it will be immediately installed system-wide for all users, with no need to think about it anymore. It'll just exist in the background as a socket trigger, and as soon as some Flatpak uses Discord RPC, the lightweight bridge/proxy starts up. Things just work automatically after that.
Flatpaks will not need any changes to their manifest/permissions. They just have to be configured by the Flatpak author to use the standardized xdg-run/app/com.discordapp.Discord:create permission.


Saving the dashboard in Grafana

User story

As a developer
I want / need to save the dashboard in Grafana
So that I do not have to import json files or rebuild the dashboard

Acceptance criteria

The changes should be stored in the repository

Definition of done (DoD)

Every user can use the same Grafana configuration

Feature DoD

Feature has been fully implemented
All acceptance criteria were met
All tests are passing
No known errors
Documentation updated
Bill of materials updated

Sprint Release DoD

Feature has been fully implemented
Feature has been merged into the mainline
All acceptance criteria were met
Product owner approved features
All tests are passing
Peer code review passed
Developers agreed to release
Documentation updated
Bill of materials updated

done


Playlist creation can fail when using family plan accounts

I have an app that allows for creation of playlists and have some users reporting errors that they are receiving this message when attempting to create a playlist (I'm showing the error.localizedDescription in an alert)
You cannot create a playlist for another user (403)

I've chatted with some users and every case that this happens is for people with premium family plan accounts (Spotify Duo or 4-person plans). It appears that either their access token or URI is for the main family member's account while the other is the actual account. Some kind of mismatch resulting in this error. Have you seen anything like this before?

You cannot create a playlist for another user (403)

It appears that either their access token or URI is for the main family member's account while the other is the actual account.

I have not encountered this error before, but this is exactly what I would expect. The user that the access token was obtained on behalf of must be the same user whose URI is passed in to the method for creating a playlist. In other words, you should forget about the existence of family plan accounts entirely in this context. Family plan accounts are separate just like any other accounts are separate for the purposes of using the Web API.
It's common for end users with multiple accounts to be confused about which account the access token was obtained on behalf of. If you are authorizing using the Authorization Code Flow, you can set the showDialog parameter of the AuthorizationCodeFlowManager.makeAuthorizationURL(redirectURI:showDialog:state:scopes:) method to false so that users who are already signed in to one account in their browser have a chance to sign in to another account before proceeding with the rest of the authorization process.
To verify which user the access token was issued for, use the SpotifyAPI.currentUserProfile() method. The user URI returned by this method must be the same URI passed in to the method for creating a playlist.

It seems there's still a way that the URI from currentUserProfile is different than the one the access token is granted for.

I think what's more likely is that there's a bug in the Spotify class in which Spotify.currentUser is not always kept up to date with the currently authorized user—as opposed to there being a problem with the web API itself.
Notice that retrieveCurrentUser, which is called by Spotify.authorizationManagerDidChange(), only updates Spotify.currentUser if this property is nil (unless the onlyIfNil parameter is set to false).  And Spotify.currentUser is only set to nil after SpotifyAPI.authorizationManager.deauthorize() is called in Spotify. authorizationManagerDidDeauthorize.
Therefore, if you authorize for one user, and then authorize for another without calling SpotifyAPI.authorizationManager.deauthorize() first, Spotify.currentUser will still be set to the first user you authorized for.
In my app, it is not possible to authorize for another user without first tapping the Logout button, which does call SpotifyAPI.authorizationManager.deauthorize(), thereby setting Spotify.currentUser to nil, so this bug cannot occur.
If you're still convinced that the problem is with the web API itself, try changing the call to self.retrieveCurrentUser() in  Spotify.authorizationManagerDidChange() to self.retrieveCurrentUser(onlyIfNil: false) so that Spotify.currentUser is updated anytime the authorization information changes. This will lead to redundant calls to SpotifyAPI.currentUserProfile() when merely the access token is refreshed (because this also causes Spotify.authorizationManagerDidChange() to be called).

I also have a logout button that calls daeuthorize. I can make the change to call currentUserProfile() when creating a playlist and then see if that fixes the issue for these users. I'll report back in 2-3 days

You may simply want to call currentUserProfile() directly before creating the playlist to confirm this is not an issue with the web API.

Yeah thats the change I'm going to make. Its working for me, but I don't have a repro of these shared family accounts to try with

I put out an app update and 2 users let me know the issue is still happening for them. I'm calling currentUserProfile() directly before createPlaylist() and passing in the URI. Do you think its possible for it to be an issue on Spotify's end? I don't think this should be scopes related.

It's certainly possible. I've encountered bugs in the web API before. I would recommend that you install logging facilities in your app so that you can know for sure what's going on. Also, more specific details would be useful. I have a family plan, so I can try to reproduce the bug. Can you send me a link to your app?

https://apps.apple.com/us/app/lineupsupply-playlist-maker/id1631703551
I don't think its affecting every family plan account from what I can tell, but for those who are affected it happens 100% of the time when creating a playlist

Note that I tested your app with my family plan member account and my Dad's family plan manager account and I could not reproduce the bug.

Appreciate you testing it. I have some Spotify engineers on Twitter I'll try reaching out to them and see if they are aware of anything. Might have to create a family plan myself as well. I guess I can close this unless you have other ideas

I suggest that you post about it at the Spotify Developer Forums.

See #41. This could be the cause of the issue. Test your app with a user with underscores in the user ID (not the display name).

@Peter-Schorn thank you

Can confirm that this fixed the issue


ApiGatewayV1Api: cannot set restApiName

Closes #1382

Hello dear,
i looked for this github project, its very useful for development purposes,
i want try work on familiar projects ,i like android development with
native languages .
thanks for your efforts.
cordially
On Fri, Feb 18, 2022 at 11:21 PM Frank @.***> wrote:

Closes #1382
https://github.com/serverless-stack/serverless-stack/issues/1382
You can view, comment on, or merge this pull request online at:
https://github.com/serverless-stack/serverless-stack/pull/1388
Commit Summary

b0c7620
https://github.com/serverless-stack/serverless-stack/pull/1388/commits/b0c762068b555a4f31721d0ef996d1aa90772906
ApiGatewayV1Api: cannot set restApiName

File Changes
(2 files
https://github.com/serverless-stack/serverless-stack/pull/1388/files)

M packages/resources/src/ApiGatewayV1Api.ts
https://github.com/serverless-stack/serverless-stack/pull/1388/files#diff-cbc89ccf12da6faf551546dea773a13276e3b2966050bbd992e2be04fd6a225d
(2)
M packages/resources/test/ApiGatewayV1Api.test.ts
https://github.com/serverless-stack/serverless-stack/pull/1388/files#diff-e0245e661f2176cd4a89832799b079782aac8a3ffdbb1d82d7639efe1d361c36
(7)

Patch Links:

https://github.com/serverless-stack/serverless-stack/pull/1388.patch
https://github.com/serverless-stack/serverless-stack/pull/1388.diff

—
Reply to this email directly, view it on GitHub
https://github.com/serverless-stack/serverless-stack/pull/1388, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AJTAY3OKE536TOFVGBOLU3LU33A7DANCNFSM5OZHNVOQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


How to use stereopy for registration of histological images and gene expression images

Hello
Thanks for developing the stereopy. Recently, when I used stereopy for tissue segmentation (function: tissue_extraction_to_bgef), I found that the axolotl data on the official website of Huada spatiotemporal omics (https://db.cngb.org/stomics/artista/download.html) was not aligned with the genetic data. I checked the paper, mentioned that the author performed the alignment manually.  so I want to know if there is a way to align and complete the segmentation of the tissue through the stereopy package?
gene scatter plot:

image:

tissue cut scatter plot:

hi, could you tell me the download link of the image (sn.tif) mentioned here ? ( and also the gene expression matrix (gem.gz/gef)

link of image: https://ftp.cngb.org/pub/SciRAID/stomics/STDS0000056/Image/2DPI_1.tif
gem file: https://ftp.cngb.org/pub/SciRAID/stomics/STDS0000056/Bin1_matrix/2DPI_1.gem.gz
the spatiotemporal database link: https://db.cngb.org/stomics/artista/download.html

link of image: https://ftp.cngb.org/pub/SciRAID/stomics/STDS0000056/Image/2DPI_1.tif gem file: https://ftp.cngb.org/pub/SciRAID/stomics/STDS0000056/Bin1_matrix/2DPI_1.gem.gz
the spatiotemporal database link: https://db.cngb.org/stomics/artista/download.html
Hi,you can follow the steps below to run.
First,You need to use imageQC1.0.8(https://github.com/BGIResearch/imageQC) to control your tif.
Second,Use SAW4.1.0(https://github.com/BGIResearch/SAW) to register images.
Finally, get the image from the registration result.

link of image:https://pan.genomics.cn/ucdisk/s/2ey6F3

link of image: https://ftp.cngb.org/pub/SciRAID/stomics/STDS0000056/Image/2DPI_1.tif gem file: https://ftp.cngb.org/pub/SciRAID/stomics/STDS0000056/Bin1_matrix/2DPI_1.gem.gz
the spatiotemporal database link: https://db.cngb.org/stomics/artista/download.html
Hi,you can follow the steps below to run.
First,You need to use imageQC1.0.8(https://github.com/BGIResearch/imageQC) to control your tif.
Second,Use SAW4.1.0(https://github.com/BGIResearch/SAW) to register images.
Finally, get the image from the registration result.

Hi, I have install imageQC1.0.8, when I drag and drop the image files (2DPI_1.tif) into the program window, requested to fill in image parameters. I don't know where to get these parameters and the ChipNo. Should I use the default ExtraInfo parameters?

You can use this tif(https://pan.genomics.cn/ucdisk/s/2ey6F3) file directly in  stereopy.Please refer to the user manual for the use method of imageQC1.0.8.

You can use this tif(https://pan.genomics.cn/ucdisk/s/2ey6F3) file directly in stereopy.Please refer to the user manual for the use method of imageQC1.0.8.

hi, I am very grateful for your help. I want to research cell segmentation algorithm based image, so I also need to register other data of axolotl  dataset.
Following your method, I have successfully registered the data of 2DPI slices by use default parameters of ImageQc. However, when registering other data, I encountered a new problem, that is, the image can pass imageQC, but an error occurs when using the SAW registration tool.
example:
gem file link: https://ftp.cngb.org/pub/SciRAID/stomics/STDS0000056/Bin1_matrix/10DPI_1.gem.gz
image file link: https://ftp.cngb.org/pub/SciRAID/stomics/STDS0000056/Image/10DPI_1.tif
the error:

log:
20221014-19-07-25.log

Please move to https://github.com/BGIResearch/SAW/issues/17


Conditional breakpoints need to better report stale references

Type: Bug
When I started my program I was getting an error about an invalid variable. The error was obscure until I figure out the problem was that I had changed the name of a variable but a conditional breakpoint was using the old name.
The simple fix is to provide an explicit messaging saying that was the problem rather than a long stake trace with a strange message about a variable not being found.

I had a condition breakpoint and then deleted the variable it was referencing, but I didn't notice the conditional breakpoint was there. The error message was very confusing since it treated it as happening in a line of code rather than mentioning the breakpoint.
Ideally, the fix is to just mention that the problem is in the conditional breakpoint.

If I enable breaking on "caught exceptions" and have an error in a conditional breakpoint expression, then I break on something like this

and clicking the frame doesn't open any editor, so it's a little confusing @connor4312 . I don't see the error message you describe but you could have something else that ends up logging it.

vscode-debugadapter-e145a833.json.gz


Better Czech localization

After using Czech translation for a while, my colegues give me some feedback. The main difference is how to name a "repository".
There are also some more fixes in spelling of some words.

Fantastic!  Thank you.


added termination_on_expiration_policy variable

Spot fleet instances didn't terminate after request expiration.
added termination on expiration policy variable.

Please update README.md accordingly to changes

Done.


Replace deprecated pre-build hook with pre-build-buildstep

Open to suggestions as to allowing for backwards compatibility. Anyone requiring that functionality could also just create a symlink after cloning. A stub pre-build could also be implemented that will just call pre-build-buildstep with its arguments.

this is to enable dokku 0.3.15 compatibility


created ABOUT.md with language description

This should take care of #53. I think the grammar and spelling is good, but let me know if you would like anything changed.

Thanks!


Cannot disable type checking when diagnosticSeverityOverrides is set

Environment data

Language Server version: v2023.5.40
OS and version: Windows 10 Enterprise build 19044.1826
Python version (& distribution if applicable, e.g. Anaconda): 3.10

Code Snippet

a: int = "hi"

with the following settings.json:
"python.analysis.diagnosticSeverityOverrides": {
	"reportGeneralTypeIssues": "warning"
}

Repro Steps

Set the severity override for reportGeneralTypeIssues in settings.json (to anything)
Disable pylance type checking in settings (should already be disabled by default)

Expected behavior
Type errors should not be highlighted in the editor when type checking is disabled.
Actual behavior
Type errors are highlighted in the editor, even though type checking is disabled. The severity matches whatever it was set to in settings.json
Removing the python.analysis.diagnosticSeverityOverrides setting causes type checking to be disabled as expected.
Logs

2023-05-25 03:37:59.042 [info] 2023-05-25 03:37:59.040709 [I:onnxruntime:, inference_session.cc:1488 Initialize] Session successfully initialized.

2023-05-25 03:37:59.043 [info] (67040) Created IntelliCode session.
2023-05-25 03:37:59.043 [info] (67040) Initialize deeplearning succeeded
2023-05-25 03:38:06.791 [info] (67040) [FG] parsing: $userdir\.vscode\extensions\ms-python.vscode-pylance-2023.5.40\dist\typeshed-fallback\stdlib\typing.pyi [fs read 0ms] (12ms)
2023-05-25 03:38:06.799 [info] (67040) [FG] binding: $userdir\.vscode\extensions\ms-python.vscode-pylance-2023.5.40\dist\typeshed-fallback\stdlib\typing.pyi (7ms)
2023-05-25 03:38:06.803 [info] (67040) [FG] parsing: $userdir\.vscode\extensions\ms-python.vscode-pylance-2023.5.40\dist\typeshed-fallback\stdlib\typing_extensions.pyi [fs read 0ms] (2ms)
2023-05-25 03:38:06.805 [info] (67040) [FG] binding: $userdir\.vscode\extensions\ms-python.vscode-pylance-2023.5.40\dist\typeshed-fallback\stdlib\typing_extensions.pyi (2ms)
2023-05-25 03:38:06.830 [info] (67040) [FG] parsing: $userdir\.vscode\extensions\ms-python.vscode-pylance-2023.5.40\dist\typeshed-fallback\stdlib\_typeshed\__init__.pyi [fs read 1ms] (8ms)
2023-05-25 03:38:06.832 [info] (67040) [FG] binding: $userdir\.vscode\extensions\ms-python.vscode-pylance-2023.5.40\dist\typeshed-fallback\stdlib\_typeshed\__init__.pyi (1ms)
2023-05-25 03:38:06.838 [info] (67040) [FG] parsing: $userdir\.vscode\extensions\ms-python.vscode-pylance-2023.5.40\dist\typeshed-fallback\stdlib\types.pyi [fs read 1ms] (6ms)
2023-05-25 03:38:06.843 [info] (67040) [FG] binding: $userdir\.vscode\extensions\ms-python.vscode-pylance-2023.5.40\dist\typeshed-fallback\stdlib\types.pyi (5ms)
2023-05-25 03:38:06.853 [info] (67040) [FG] parsing: $userdir\.vscode\extensions\ms-python.vscode-pylance-2023.5.40\dist\typeshed-fallback\stdlib\abc.pyi [fs read 0ms] (1ms)
2023-05-25 03:38:06.853 [info] (67040) [FG] binding: $userdir\.vscode\extensions\ms-python.vscode-pylance-2023.5.40\dist\typeshed-fallback\stdlib\abc.pyi (0ms)
2023-05-25 03:38:06.858 [info] (67040) [FG] parsing: $userdir\.vscode\extensions\ms-python.vscode-pylance-2023.5.40\dist\typeshed-fallback\stdlib\decimal.pyi [fs read 0ms] (1ms)
2023-05-25 03:38:06.865 [info] (67040) [FG] binding: $userdir\.vscode\extensions\ms-python.vscode-pylance-2023.5.40\dist\typeshed-fallback\stdlib\decimal.pyi ...
2023-05-25 03:38:06.865 [info] (67040) [FG]   parsing: $userdir\.vscode\extensions\ms-python.vscode-pylance-2023.5.40\dist\typeshed-fallback\stdlib\_decimal.pyi [fs read 0ms] (6ms)
2023-05-25 03:38:06.866 [info] (67040) [FG]   binding: $userdir\.vscode\extensions\ms-python.vscode-pylance-2023.5.40\dist\typeshed-fallback\stdlib\_decimal.pyi (2ms)
2023-05-25 03:38:06.867 [info] (67040) [FG] binding: $userdir\.vscode\extensions\ms-python.vscode-pylance-2023.5.40\dist\typeshed-fallback\stdlib\decimal.pyi (9ms)
2023-05-25 03:38:06.872 [info] (67040) [FG] parsing: $python_file [fs read 0ms] (1ms)
2023-05-25 03:38:06.872 [info] (67040) [FG] binding: $python_file (0ms)
2023-05-25 03:38:06.884 [info] (67040) [FG] parsing: $userdir\.vscode\extensions\ms-python.vscode-pylance-2023.5.40\dist\typeshed-fallback\stdlib\re.pyi [fs read 0ms] (6ms)
2023-05-25 03:38:06.886 [info] (67040) [FG] binding: $userdir\.vscode\extensions\ms-python.vscode-pylance-2023.5.40\dist\typeshed-fallback\stdlib\re.pyi (2ms)
2023-05-25 03:38:06.893 [info] (67040) [FG] parsing: $userdir\.vscode\extensions\ms-python.vscode-pylance-2023.5.40\dist\typeshed-fallback\stdlib\enum.pyi [fs read 0ms] (3ms)
2023-05-25 03:38:06.894 [info] (67040) [FG] binding: $userdir\.vscode\extensions\ms-python.vscode-pylance-2023.5.40\dist\typeshed-fallback\stdlib\enum.pyi (1ms)
2023-05-25 03:38:06.903 [info] (67040) [FG] parsing: $userdir\.vscode\extensions\ms-python.vscode-pylance-2023.5.40\dist\typeshed-fallback\stdlib\collections\abc.pyi [fs read 0ms] (1ms)
2023-05-25 03:38:06.903 [info] (67040) [FG] binding: $userdir\.vscode\extensions\ms-python.vscode-pylance-2023.5.40\dist\typeshed-fallback\stdlib\collections\abc.pyi ...
2023-05-25 03:38:06.903 [info] (67040) [FG]   parsing: $userdir\.vscode\extensions\ms-python.vscode-pylance-2023.5.40\dist\typeshed-fallback\stdlib\_collections_abc.pyi [fs read 0ms] (0ms)
2023-05-25 03:38:06.904 [info] (67040) [FG]   binding: $userdir\.vscode\extensions\ms-python.vscode-pylance-2023.5.40\dist\typeshed-fallback\stdlib\_collections_abc.pyi (1ms)
2023-05-25 03:38:06.904 [info] (67040) [FG] binding: $userdir\.vscode\extensions\ms-python.vscode-pylance-2023.5.40\dist\typeshed-fallback\stdlib\collections\abc.pyi (1ms)
2023-05-25 03:38:06.919 [info] (67040) Background analysis message: getDiagnosticsForRange
2023-05-25 03:38:06.919 [info] (67040) Background analysis message: getDiagnosticsForRange
2023-05-25 03:38:06.922 [info] (67040) Background analysis message: getDiagnosticsForRange
2023-05-25 03:38:06.922 [info] (67040) Background analysis message: getDiagnosticsForRange
2023-05-25 03:38:06.937 [info] (67040) Background analysis message: getDiagnosticsForRange
2023-05-25 03:38:06.937 [info] (67040) Background analysis message: getDiagnosticsForRange

This is by design. Setting python. analysis.typeCheckingMode changes the defaults of all the diagnostics to none. But then you're changing reportGeneralTypeIssues to warning, which takes precedence over the defaults.

I think there's a strong use-case for allowing both to be done independently though, where I would want to change the severity of some errors but still be able to disable type checking sometimes.
This is especially so considering there is now a toggle button in the status bar that allows one to toggle type checking frequently and on the fly.

I think there's a good argument for allowing both to be done independently though

That wouldn't make sense because these are not independent controls. They control the same thing — the severity levels for diagnostic rules. The typeCheckingMode is simply a shortcut for setting the severity levels for many diagnostic rules at once. You can always set the severity for these rules manually as well (which overrides the defaults set by the typeCheckingMode).
You may find this table useful. It explains what severity levels are applied by default for each rule when you set the typeCheckingMode to "off", "basic" or "strict".

I see, so that setting is better understood a specific mode called "off", rather than an actual off switch for the type checker. Could we have an actual off switch for the type checker then, that disables everything?
Especially in combination with the toggle button in the status bar, I would find such a button that fully turns it off to be more intuitive than the current behaviour.

 Can you explain why you'd want to do this? If you don't want to enable certain diagnostic rules, you can simply not enable them in your settings.

You could call it that, but I prefer like to think of it as a hard off switch for the entire type checking system.
One possible scenario would be if I were editing a project that already has multiple type errors (e.g. if the previous author used type hints more as loose annotations). I would still like to use the type-checker selectively when I adding new features, but would prefer to turn them off when browsing the existing code to minimise distractions.
If I wanted to customise the severity levels of some rules, this would no longer work. There would be no simple way to toggle the type checker on and off while having my custom options, other than to edit settings.json and add or remove the entire python.analysis.diagnosticSeverityOverrides option every time, which is obviously not very practical.

If it helps, the way I imagine it working would be essentially analogous to the grammar checker in Microsoft Word, where there are options for toggling each individual rule on and off, and a big switch to turn the whole grammar-checking feature on and off.


decreased duration of mitotic M-phase

https://github.com/pombase/fypo/issues/3275
I forgot (or cant find!)
decreased duration of mitotic M-phase

nope, we actually didn't have it before
decreased duration of mitotic M phase FYPO:0006490
edit file: c97b3b6deffa646901716342b8e38d25e85ae32e
release: 26292d5dada779e4c2e25ace9ed0d27185395cda

hmmph, I have no idea what this was for now.....


use go1.17 in benchstat

In previous changes from #6, .github/workflows/build.yml was updated to require at least go1.17 but benchstat.yml was neglected, resulting in issue running bench:
29  Error: ../../../go/pkg/mod/github.com/rqlite/go-sqlite3@v1.27.1/sqlite3_opt_serialize.go:41:26: undefined: math.MaxInt
30  note: module requires Go 1.16

The problem is that module github.com/rqlite/go-sqlite3

requires 1.16 in its go.mod
and also uses math.MaxInt that was added in 1.17

See #9


docs: Make badges consistent, run markdownlint on all .md files

Consistently generate badges for GH workflows in README RHELPLAN-146921
Run markdownlint on all .md files
Add custom-woke-action if not used already
Rename woke action to Woke for a pretty badge

LGTM.


Add Unleash Psyche effect

Test it without afterDerived. I think at a certain point we made @spell no longer require that.

Looks like that works. Alright, removed.


[Add] huetiful-js to LinksHub

Site URL
https://prjctimg.github.io/huetiful
Description of the above provided site
JavaScript library for simple color manipulations based on the Culori API.
Category
Frontend
Sub Category
"colors"
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue (optional)

@prjctimg thanks, assigned to you, you can start working on this issue now


Unable to set cookie header

I'm dealing with a funky authentication process that requires me to:

Hit an endpoint with redirects set to 0 or handle all responses in the chain
Take the Set-Cookie header value from the first response and supply it with all future requests in the Cookie header

I'm attempting to accomplish this through a custom implementation of ServiceClientCredentials utilizing Axios(for now at least). For the life of me, whatever I try, the cookie header ends up blank on the wire. It appears some effort was made to get cookies working with fetch, however I can't get them working for myself :)
Any guidance on this would be much appreciated(or if I could convince you to utilize Axios :D ).

Which environment is the redirect failing in (nodejs or browser)?
Axios is something we've considered moving to under the hood due to some features we need not being available in the fetch polyfills yet.

I'm currently testing with nodejs, however the redirect is not an issue as I'm handling that with Axios.
The issue I'm facing now is that after I set the cookie header in the ServiceClientCredentials implementation, the subsequent requests have no value for the cookie header; it has been cleared out. I have not attempted to get the cookie set naturally as it doesn't appear that fetchHttpClient is exported from the package.

My guess is that the runtime has been simply overwriting the Cookie header on the outgoing request based on what it thinks the cookie should be. I'll try and do something about this.


:stuck_out_tongue_closed_eyes: link text updated

:rocket: deployment of profiles-pr-98 in nhsuk-dev succeeded (http://profiles-pr-98.dev.c2s.nhschoices.net)


changes tone analyzer service to nlu

This updates the dependencies from tone analyzer to natural language understanding and also adds a couple of tests

@jweisz I made the fixes and added a couple of other things if you can take a look


Issue using certain versions of fluid framework

npm i<EMAIL_ADDRESS>does not work. but npm i<EMAIL_ADDRESS>npm and pnpm are unable to resolve versions with -. How do you suggest we handle these?

@tylerbutler or @Abe27342, do either of you have suggestions here?

@tummalasujith  What version of npm are you on and how is your .npmrc configured? It works for me pointing at npmjs:

I'm guessing this is a feed configuration issue. I see you're in the Microsoft org--if you're pointing at an ADO feed that upstreams npmjs, IIRC there's a possibility that you don't have permissions to cause the internal feed to grab packages from its upstreams.

My npm is at 6.14.16. Right, we are pointing to OneDrive internal feed. We tried piercing cache and multiple people try it. exact version works, but not with caret. Sound like we need to look into our feed configuration to see if there is a setting to avoid prerelease versions when using caret.

@tummalasujith,  Just to close on this issue: npm semver ^ ranges doesn't consider any prerelease versions unless the version also has a prerelease tag:
e.g. ^0.59.4002-0 will match 0.59. where patch >=4002 and prerelease version with prefix 0.59.4002 only
But ^0.59.4002 will only match 0.59.  where patch >=4002  and no prerelease versions.


Java Lambda triggered through API Gateway is not continuing a trace rather it starts a new trace

Describe the bug
A Java 11 Corretto Lambda receiving a request through API Gateway does not continue the parent trace. The lambda starts a new trace ignoring the fact it has been called by another service that has already initiated a trace.
For example I have Spring Boot Service A -> API Gateway -> Lambda B
I have also tried Lambda A -> API Gateway -> Lambda B both behave the same
The trace collected in Elastic APM (or Zipkin, basically I think anything that isn't XRAY) captures each service as its own distinct trace. If I just have two spring boot applications then the trace is correctly captured.
Steps to reproduce
Create a local spring boot service that makes a http call to a Lambda through API Gateway.
What did you expect to see?
A single trace starting in Service A, showing the http request and linking this to Service B
What did you see instead?
No link and two separate traces
Environment
The Lambda is using the following layer: arn:aws:lambda:eu-west-1:901920570463:layer:aws-otel-java-agent-amd64-ver-1-19-2:1
The Spring Boot service is using the latest AWS OTEL version (as of today) downloaded from here: https://aws-otel.github.io/docs/getting-started/java-sdk/trace-auto-instr

Can you provide some additional details?  It would be helpful to have debug logs from your collector instance, including with the logging exporter at the detailed/debug level.

Sure. I've zipped up the logs from both lambdas which includes the applications logs and also the openTel logging. I've also included the logs of stand alone collector there both pushing to. I have also included the config for the 2 lambdas and the standalone collector
Archive.zip

@Sutty100 can you make sure that the call that is performed from Lambda A to Lambda B is performed using a library that supports auto instrumentation? We recommend using the Okhttp library https://square.github.io/okhttp/

Marking the issue as stale was we are strongly suspecting that the problem is the usage of an library that does not support auto-instrumentation, which would lead to having two trace ids.

The issue might not be the library to perform the http calls. I will update this ticket soon with findings. I'm suspecting about the auto-instrumentation library not being able to identify all the type of lambda handler types for java (Stream vs event based handlers).


Set compiler flags per file

Hi,
I'm using gradle 6 with C++ Library plugin.
I have a cpp library with multiple src files and I want to control a specific file compiler args differently than other files.
For my understanding the CppCompile.compilerArgs is uniform for all src files.
Is there a way I can set / add / override compilerArgs for a specific cpp file?
Thanks

Unfortunately, there isn't a way to achieve this at this point. Once version 0.5 of Nokee (https://nokee.dev) is out the door, single file flag configuration is on the candidate list to work on. Note that Nokee is a separate native support for Gradle than the one offered by the Gradle team.
Could you expand on the type of single file configuration you wish to do? Is it turning on debugging on specific files only or something else?

Currently I want to manipulate /arch and floating point precision but i guess usecases for other flags might be required too.
regarding Nokee - is it the preferred future way for developing C++ using gradle or does the current gradle plugins will remain the official way for this?
Thanks.

I can't talk for the Gradle team but Nokee is a continuous effort on developing first class support for native in Gradle.


Support idle watchers?

See https://github.com/amphp/amp/issues/214.

Idle watcher is a bad name, but I got that from http://docs.libuv.org/en/v1.x/design.html#the-i-o-loop. Basically watchers which are invoked on every loop iteration, but they shouldn't trigger an immediate new iteration IMO (like nested defers).

https://github.com/kelunik/loop-block is one possible use case. Another would be moving signal handlers out of the event loop and into these idle handlers.

This could also hook into #5 and expose a metric for time between loop ticks

Rather than being able to measure ticks, it's probably more important to measure callbacks blocking.

While libuv does have idle watchers, they're implemented in a way we can already emulate. Any active idle watchers result in a timeout of 0 while waiting for I/O, so this is very similar to scheduling a defer watcher inside itself every time until some condition is satisfied. Until then, the event loop will busy wait.


map: refactored HDMapImpl

map:

refactored HDMapImpl class by creating a hierarchy of classes for lanes, junctions, signals, crosswalks, stop-signs, yield-signs, speed-bumps, overlaps, roads, PNC junctions and RSUs, removing duplicate code;
faster map initialization by preallocating memory for vectors, unordered_maps and unordered sets;
faster lookup using unordered sets instead of ordered sets;
uniform access to proto-buffer object in newly created classes by using inner_object() method following DRY principle;
optimized serial if-else based look-up by using switch instruction;
smaller refactorings removing duplicate code;

dreamview, perception, planning, prediction, v2x:

updated code according to map related changes;

ran code format, code linting, map unit tests and dreamview simulation.

The previous pull-request/merge seems it failed because my own fork was overwritten by an automatic update app installed by me.

@zhouyao4321 Could you please review again this pull request?

Could someone restart TeamCity build? My 2nd commit was not included in the build.

I'm wondering, why this PR was not reviewed until now?

Modifying the map model needs to consider the impact on other modules, so we will be more cautious. I will review and give feedback


freeinput & itemValue

I have made a small feature addition in order to use object value/text with free input.
This makes each freeinput tag addition attempt with a new event beforeFreeInputItemAdd

hi dkavraal,
I had the same problem that free-text input is not cleared after enter key pressed and I found the solution. The cause of this issue is because typeahead holds the text you input. You need to clear the text hold by typeahead when 'itemAdded' event is fired.
$('input').on('itemAdded', function(){
$('input').typeahead('val','');
}).tagsinput({...});

Any updates on this ?

Thanks for your work,
I really really need this !!


Remove untrue statement from README.md (log-upload)

This PR removes the information that this repository contains the log-upload part of the Corona-Warn-App, since this is repository doesn't contain the log-upload part, from the README.md.

Thanks for your PR!
We will check the rest of the repository If there are still some left overs from log-upload.


dependentKeys as a function?

My dependentKeys are dynamic and can't be known at run time. Is it possible to use a function that returns an array of dependentKeys?

Alternatively, is there a way to force revalidation on validators?

I was able to force this validation to retrigger with some hacking, but surely there is a better way:
// Force a revalidation by changing customFields to a new ember object with the same content
        items.forEach(i => {
          i.set('customFields', EmberObject.create(i.get('customFields')));
        });```

Last thing I'll add in case it helps, the actual inline validator:
customFields: validator('inline', {
    dependentKeys: ['model.type', 'model.customFields'],
    validate(value, options, model) {
      const fields = getTypeFields(model);
      const errors = [];
      fields.forEach(field => {
        if (field.get('isRequired') && isBlank(model.get(`customFields.${field.get('name')}`))) {
          errors.push(field.get('name'));
        }
      });
      model.set('errorMessages', errors);
      if (Object.values(errors).length) {
        return Object.values(errors).join(', ');
      }
      return true;
    },
  }),


Tests that are not run due to build configuration

I've noticed that some tests are #ifdef out on certain platforms due to missing features (one I can think of is between 32 and 64 bit builds, related to 64-bit longs).  This means that there is a difference in the number of tests that are executed on these platforms.  I'm wondering if this is the best way to handle these tests, or if they should show as ignored on platforms that don't support them.  I think this also ties a little bit into the discussion on issue #343, since the two would affect each other some even though they are otherwise different ideas/features/etc...
Other people's thoughts on this?

Some of the tests can't compile on other platforms so then it can't be ignored (e.g. 32/64 bit)

I guess I didn't quite think that through. I suppose it would be possible to write the macros/ifdefs such that it still got flagged as ignored but was otherwise not built, but that might be more wonky than just not compiling the test in the first place.

Wouldn't it be simple to put an #else with the #ifdef that contains an ignored test by the same name, which is a dummy (possibly with a printed message explaining why a test was ignored)?

Talking about ignored tests -- I noticed CppUTest has a couple of those and it would be helpful to have some sort of message (or at least comment) clarifying why it was ignored. Maybe the reason for ignoring them doesn't even exist any more (who knows) or maybe something should get fixed (again, who knows)?

So, I tend to agree with @Andne and I think it would be quite easy to do.

@Andne -- doing
#ifndef WIN
TEST(foo, something_testing_64bit_longs)
{
     // actual test here
}
#else
IGNORE_TEST(foo, something_testing_64bit_longs)
 {
      UT_PRINT("64 bit Windows has 32 bit longs");
}
#endif

would not be wonky, right? Also, pretty much self-documenting, I think.

That would be too bad, except I'm not sure like repeating the test name.  I think we need to go through the list of tests that are ignored and see why, some maybe be able to be tricked out through various things to always build, but thinking more others will need something like this - such as ones that test exception behavior, they don't apply at all without exceptions. Kind of makes me think there should be a distinction between tests that are ignored because they haven't been fixed (ok short term, bad long term) and tests that don't apply to the current platform.  I'd still like to see the later be flagged in results so that they can be accounted for, but they are not passed or failed, and using ignored seems to apply two very different definitions to the term.

It would be too bad ???
You are right about two distinctly different uses for ignored tests. This is only about the one kind -- a test cannot be run at all under certain conditions, but can be run under other / normal conditions.
In this case, I think the exact same test name is not only possible, but beneficial. Because of the #ifdef ... #else ...  #endif involved there will be no error. Only if (by mistake) something is messed up, a compile error would result, which is a GOOD thing, in  my opinion.
At least in case of the 64bit long test case, I would like to see this.
I don't know about other possible applications.
And I certainly agree that some spring-cleaning of ignored tests would be quite desirable...

Oops, that's what happens when I'm typing on a tablet and don't proofread...
I agree, the same test name is good, and it won't cause errors due to the #ifdef/#else.  My concern was that someone would change a test name and not change it in both lines.  I know it's a pretty minor edge case, but still seems to be something we wouldn't want.
I agree that your suggestion only makes sense for tests that can't be run due to platform differences.  I was just saying that I wish there was a way to distinguish 'can't run because of the platform' and 'not run because someone said not to'.  Unfortunately, I don't think that most test frameworks or even the other tools that log tests have any support for another category like that.

Maybe a macro such as TEST_IF(bool expression) could actually work. The only pain is that such a macro will need an #endif. But then you can do the ignore there also and get rid of most #ifs.

That's an interesting thought.  I'm not sure how well the macros would accept generating a conditional preprocessor tree, but it's something to try and see how it works out.

TEST_IF() would print a dot when executing. If the condition is false, what would it print? A dash, perhaps? Or would IGNORE_TEST() be used if the condition is false and an ! be printed?

I'd say probably treat it as an ignored test, since that's all other tools have support for anyways.
Unfortunately, I'm not having much luck figuring out how to match up a conditional compilation block with a macro.  Trying to write it myself causes build errors, # becomes the stringify operator when working inside of macros, so you can't put extra directives inside a macro block.  Searching online turns up few results, but what there is pretty much says no, you can't do that.  While there still probably is a way to make a TEST_IF macro, I don't see how it could disable the compilation of the test code itself.

Yeah, I guess you are right. The only way then would be to have specific TEST_IF_64_BIT-style macros.
That would be too bad though :(

Close this?
Current approach is to use IGNORE_TEST() with conditional compilation where possible ("possible" meaning a failed test would result, rather than a compiler error).
I thought I had seen a "TEST_IF"-style macro in the existing tests, but couldn't find it anymore.

Sure, I suppose we can close it.  I tried to write a TEST_IF macro, and ran into some issues were you can't have #if inside the macro, so it didn't work.


esdoc(SourceMapDevToolPlugin): add some esdoc definitions

https://github.com/webpack/webpack/issues/3277

What kind of change does this PR introduce?
documentation

Did you add tests for your changes?
No

Does this PR introduce a breaking change?
No

What needs to be documented once your changes are merged?
Nothing

Thanks


Which env version for Mujoco tasks?

Hi, which Mujoco env version is used to generate the offline data, e.g., Hopper-v1 or Hopper-v3? And how to evaluate the agent's performance after training? By testing the agent in an online way, i.e., running it in the env? Thank you.

They correspond to the v2 envs (i.e. Hopper-v2). And yes, we recommend evaluating the agent in the online way by running them in the env. We are currently working on some off-policy evaluation benchmarks but that is still in progress.


Nix CI: push artifacts to (alternative) container registries

Nix CI already produces docker images of Mina. Push them somewhere to make accessing (and testing) them easier.

We can upload tarballs that Nix produces straight to the registry with skopeo copy --insecure-policy --dest-registry-token $(gcloud auth application-default print-access-token) docker-archive:$(readlink result) docker://us-west2-docker.pkg.dev/o1labs-192920/nix-containers/mina:latest . The only thing left to do is to make a service account with access to the registry.

Probably also add an upload job for mina-daemon-docker


Fixed more copyright python file mishaps.

I double-checked that there was no other python file in the AI85/AI87 that has this copyright mishap.


Add a few more my interfaces

Addresses https://github.com/karlicoss/HPI/issues/64

Withings
Level Health
Eight Sleep
ActivityWatch (local-only RescueTime, also write my own synchronization code so you can sync multiple computer's AW data)
Typeform
Todoist

Unfortunately the extractors are written in node then I wrote DALs for each in Python. Maybe I am being presumptive but I think exposing the idea that HPI doesn't have to be solely in python is fun. Love being able to leverage Python's data library ecosystem though, much better than node for this :joy:.
The my interfaces are pretty barebones and may not be up to standards, so happy to modify if there is a certain standard or direction you like better.

Ah, I didn't realize you weren't watching this repo, so you may have not seen the issue I raised here
To expand on it a bit;
I used to maintain a direct fork/merge commits from this repo back into my own copy of HPI here, but that became more and more difficult to as time went on...
The solution for me was in the SETUP.org file all along (described as the 'proper way')
I've since stopped merging commits that, and now install both packages as editable, which means I keep up to date with this repository and can still add/overlay modules on top of that
i.e. you can have multiple my/ directories installed at the same time, if they're installed as namespace packages (relevant lines in setup.py here and here):
 $ python3
In [1]: from my.coding.commits import commits  # something from karlicoss/HPI

In [2]: from my.zsh import history  # something from my own repo

Another example of an overlay here
Not saying these items can't be merged into here, thats up to karicoss, I've just found HPI to be more useful as a configuration/caching layer, giving me all the tools to load my data, but everyone uses different services/data sources, so I'd rather keep modules on my own branch. Otherwise, we'd eventually have hundreds of data sources in this one repository, and I don't know if anyone wants to maintain that :sweat_smile:

Unfortunately the extractors are written in node then I wrote DALs for each in Python

No that's definitely cool, and I'm definitely pro using whatever tools people can use!

Not saying these items can't be merged into here, thats up to karlicoss, I've just found HPI to be more useful as a configuration/caching layer, giving me all the tools to load my data, but everyone uses different services/data sources, so I'd
rather keep modules on my own branch.

Yep, definitely suggest trying this approach out too!

Otherwise, we'd eventually have hundreds of data sources in this one repository, and I don't know if anyone wants to maintain that

Having too many modules is definitely an issue -- I dread the fate of the Spacemacs maintainer! I think for now I'm basically happy to merge things, stuff doesn't seem to break much; but need to start thinking of ways to making it more manageable in advance perhaps.
Although I would prefer if they had tests/ example data pointers somewhere, since otherwise I can't guarantee I won't break it :)


Initial version of method receiver to class bindings

This feature implements defining PHP classes based on Go method receivers (of any type). It allows for getting and setting internal properties (in the case of struct method receivers with publicly-accessible fields) and calling methods attached to the receiver passed.

Documentation is missing, code and tests are very rough, but this works for getting, setting and calling properties and methods on method receivers as PHP classes.


python3Packages.flask-migrate: fix build

Motivation for this change
Test data was not part of the PyPi release.
Closes: #138330
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

Also please add a pythonImportsCheck


Warnings and the server doesn't start

I'm not sure what's going on. I tried reinstalling http-server, but it still isn't working. When I run http-server, it says
Starting up http-server, serving ./ on port: 8080
Hit CTRL-C to stop the server
But if I go to http://localhost:8080/, I get a "No Data Received" error in Chrome.
When I installed and reinstalled http-server, I got this error.

Can you confirm and reopen if it isn't working? Seems to be working as of 0.7.5.


Update RP Conf Role

The current role updates a current entry.  However there are times when would like to "add" an entry to an existing stanza. (ie. logcfg=blah,blah,blah)
I tried creating a new role that uses the action ibmsecurity.isam.web.reverse_proxy.configuration.entry.add.   This is the message i get.
"msg": "Error> action does not have the right set of arguments or there is a code bug!

There are three roles:
update_reverseproxy_conf
set_reverseproxy_conf
add_reverseproxy_conf

The last one is the one you want for your current use case. Use update to update an existing single value, add to append to existing or non-existing entry and finally set to ensure that you get the exact set of multi-values you want assigned to an entry. It will ensure there is no more or less when you use set.

Thanks Ram.   Sorry for missing those other roles


fix "some duration before noon/dinner/etc"

See PR #160 for details -- backfilling for release notes

fixed by PR #160


Add option g:cm_complete_start_delay, for fast-typing performance.

This PR address #118
@felixjung
Please test this PR
I've only tested it by holding the <backspace> key as in the gif you posted.
Tweak the value of g:cm_complete_start_delay if necessary.  Current default is 50ms. I could delete about 30 characters per second on my nvim, when holding the <backspace> key.

The default is now chaged to 0, since it's typically useful for ultisnips users.
Use this to tweak your option.
let g:cm_complete_start_delay = 80


chore: remove unused dependency features from fedimint-core

Merge conflict, oof.

Oooh, so close!
Rebased

wasm broken

@tvolk131 Ping me directly on Discord if you ever need it quickly approved, so we can get it through somehow. :D

Same. (currently WASM is still broken though)

Cross-compiling is failing.

serde feature on url somewhere missing

I pushed a suspected fix for the CI failure, feel free to squash/delete if it works/doesn't.

Wasm xcompile failing.

Too much work, not worth the effort 🤦


Incorrect Solution

https://github.com/kamyu104/LeetCode/blob/master/Python/maximum-subarray.py#L15
for test case [-1] should return -1

This can be added to handle such cases:
    if max(nums) < 0:
        return max(nums)

Thanks. I'll fix it soon!


./mvnw package -Dquarkus.package.type=uber-jar does not work!

....
[INFO] -------------------------------------------------------
[INFO]  T E S T S
[INFO] -------------------------------------------------------
[INFO] Running org.acme.endpoint.ExampleEndpointTest
2022-03-23 17:00:04,895 INFO  [org.jbo.threads] (main) JBoss Threads version 3.4.2.Final
17:00:05 WARN  traceId=, spanId=, sampled= [io.qu.mi.de.bi.mp.MicroprofileMetricsProcessor] (build-14) This application uses the MP Metrics API. The micrometer extension currently provides a compatibility layer that supports the MP Metrics API, but metric names and recorded values will be different. Note that the MP Metrics compatibility layer will move to a different extension in the future.
17:00:07 INFO  traceId=, spanId=, sampled= [io.ag.pool] (main) Datasource '<default>': Initial size smaller than min. Connections will be created when necessary
Hibernate:

    drop table if exists Message CASCADE
Hibernate:

    drop sequence if exists message_id_seq
Hibernate: create sequence message_id_seq start with 1 increment by 1
Hibernate:

    create table Message (
       id bigint not null,
        content varchar(255),
        locale varchar(255) not null,
        primary key (id)
    )
Hibernate:
    INSERT INTO message (id, content, locale) VALUES (1000, 'bonjour','fr')
[ERROR] Tests run: 5, Failures: 0, Errors: 1, Skipped: 4, Time elapsed: 6.924 s <<< FAILURE! - in org.acme.endpoint.ExampleEndpointTest
[ERROR] org.acme.endpoint.ExampleEndpointTest.findNonExistingLanguage  Time elapsed: 0.006 s  <<< ERROR!
java.lang.RuntimeException: java.lang.RuntimeException: Failed to start quarkus
Caused by: java.lang.RuntimeException: Failed to start quarkus
Caused by: java.lang.RuntimeException: Unable to start HTTP server
Caused by: java.util.concurrent.ExecutionException: io.quarkus.runtime.QuarkusBindException
Caused by: io.quarkus.runtime.QuarkusBindException

[INFO] Running org.acme.metric.counter.DynamicMultiTaggedCounterTest
[INFO] Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.145 s - in org.acme.metric.counter.DynamicMultiTaggedCounterTest
[INFO] Running org.acme.metric.counter.DynamicTaggedCounterTest
[INFO] Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.001 s - in org.acme.metric.counter.DynamicTaggedCounterTest
[INFO] Running org.acme.metric.timer.DynamicMultiTaggedTimerTest
[INFO] Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 1.037 s - in org.acme.metric.timer.DynamicMultiTaggedTimerTest
[INFO] Running org.acme.metric.timer.DynamicTaggedTimerTest
[INFO] Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 1.005 s - in org.acme.metric.timer.DynamicTaggedTimerTest
[INFO]
[INFO] Results:
[INFO]
[ERROR] Errors:
[ERROR]   ExampleEndpointTest.findNonExistingLanguage » Runtime java.lang.RuntimeExcepti...
[INFO]
[ERROR] Tests run: 11, Failures: 0, Errors: 1, Skipped: 4

I am on Linux (Ubuntu 20.04) with Java openjdk version "11.0.14" 2022-01-18.

Hi,
I just run this and did not experience the same issue. Please pay attention that if you previously packaged in a different mode, you will use some of the previous package when executing ./mvnw package -Dquarkus.package.type=uber-jar command. Try cleaning your previous results and re-run it.

Event with mvn clean package -Dquarkus.package.type=uber-jar I still have
[INFO] Running org.acme.endpoint.ExampleEndpointTest
2022-03-23 17:28:30,536 INFO  [org.jbo.threads] (main) JBoss Threads version 3.4.2.Final
17:28:31 WARN  traceId=, spanId=, sampled= [io.qu.mi.de.bi.mp.MicroprofileMetricsProcessor] (build-8) This application uses the MP Metrics API. The micrometer extension currently provides a compatibility layer that supports the MP Metrics API, but metric names and recorded values will be different. Note that the MP Metrics compatibility layer will move to a different extension in the future.
17:28:32 INFO  traceId=, spanId=, sampled= [io.ag.pool] (main) Datasource '<default>': Initial size smaller than min. Connections will be created when necessary
Hibernate:

    drop table if exists Message CASCADE
Hibernate:

    drop sequence if exists message_id_seq
Hibernate: create sequence message_id_seq start with 1 increment by 1
Hibernate:

    create table Message (
       id bigint not null,
        content varchar(255),
        locale varchar(255) not null,
        primary key (id)
    )
Hibernate:
    INSERT INTO message (id, content, locale) VALUES (1000, 'bonjour','fr')
[ERROR] Tests run: 5, Failures: 0, Errors: 1, Skipped: 4, Time elapsed: 6.355 s <<< FAILURE! - in org.acme.endpoint.ExampleEndpointTest
...
INFO] Results:
[INFO]
[ERROR] Errors:
[ERROR]   ExampleEndpointTest.findNonExistingLanguage » Runtime java.lang.RuntimeExcepti...
[INFO]
[ERROR] Tests run: 11, Failures: 0, Errors: 1, Skipped: 4

Do you happen to have more details of the RuntimeException? I am not getting the same error. Ran it multiple times and have the same openjdk version as you.


CoC: add committee members to CoC text

Solving https://github.com/pydata/pandas-governance/commit/1376493cdbfaa22adebd72cf1891db415f9f6923#commitcomment-18587703
But, we also have to decide on how people can contact those members. So either adding the email addresses of the different people listed there, or making a generic email address that is forwarded to those people.

Guess we should create a private CoC list that can receive incoming e-mail. What's the best service for this, Google groups?

created<EMAIL_ADDRESS>I'm inviting the initial members. Can you add this e-mail address to the patch and say that anyone can e-mail the CoC and these messages are only visible to the CoC / not publicly?

Added the email address

Thank you. Merging!


some questions about rgb feature and flow feature

I had extracted flow using denseflow with the command:denseflow videolist_all.txt -b=20 -a=tvl1 -s=2 -v. It work well.
The flow had been saved by flow_x_{:%d5}.jpg and flow_y_{:%d5}.jpg file format. e.g:there has 6 rgb frames,index from 0 to 5.so the flow data have 10 flow frames,index from flow_x_00000.jpg to from_x_00004.jpg and flow_y_00000.jpg to flow_y_00004.jpg respectively，so the number of flow frames is 10.
And then i use tsn_feature_extraction.py to extract rgb feature and flow feature,i get the rgb feature shape is [6,2048] and flow feature is [10,2048],but, i think the flow feature shape should be [5,2048].So,is there any problem here？
My data pipeline is data_pipeline = [
dict(
type='CollectFrames',
clip_len=args.clip_len,
frame_interval=2,
# num_clips=25,
test_mode=True),
dict(type='RawFrameDecode'),
dict(type='Resize', scale=(-1, 256)),
dict(type='TenCrop', crop_size=224),
dict(type='Normalize', **args.img_norm_cfg),
dict(type='FormatShape', input_format='NCHW_Flow'),
dict(type='Collect', keys=['imgs'], meta_keys=[]),
dict(type='ToTensor', keys=['imgs'])

@SugarMuzi Hi, Has the problem been solved? I am also extracting. Can you share the detailed extraction process?

@SugarMuzi @prozzzzzz Have you solved the problem?

@SugarMuzi @SugarMuzi Hi, Has the problem been solved?

@SugarMuzi @SugarMuzi Hi, Has the problem been solved?

hi~
I think I have found the problem. In other issues of this GitHub repository, someone asked about the handling after extracting optical flow features using bn-inception. The author replied that they used average pooling. Therefore, after extracting optical flow features using tsn_feature_extraction, you just need to use average pooling for dimensionality reduction.

@SugarMuzi Hi, Has the problem been solved? I am also extracting. Can you share the detailed extraction process?

hi~ The above answer is my solution.

@SugarMuzi @SugarMuzi Hi, Has the problem been solved?

hi~ I think I have found the problem. In other issues of this GitHub repository, someone asked about the handling after extracting optical flow features using bn-inception. The author replied that they used average pooling. Therefore, after extracting optical flow features using tsn_feature_extraction, you just need to use average pooling for dimensionality reduction.

Thank you for your detailed response!


LPAL-866 infrastructure to GitHub actions [DO NOT MERGE]

Purpose
Testing initial migration of Terraform + Build process to Github Actions
Fixes LPAL-866
Approach
Remove .circleci and add files to .github/workflows
Learning
Any tips and tricks, blog posts or tools which helped you. Plus anything notable you've discovered about the LPA service
Checklist

[ ] I have performed a self-review of my own code
[ ] I have updated documentation (Confluence/GitHub wiki/tech debt doc) where relevant
[ ] I have added tests to prove my work
[ ] I have added mandatory tags to terraformed resources, where possible
[ ] If I have a new OPG component dependency, I have updated the metadata.json with the repo location.
[ ] If I added a package.json or composer.json, I also made sure this is included in the script in .github/workflows/dependabot-update.yml
[ ] The product team have tested these changes

I came to take your post merge cleanup workflow and spotted that it didn't specify permissions. This is useful for ensuring the workflows only have the access they need to complete. Here's info on permissions for github actions workflows https://docs.github.com/en/actions/using-jobs/assigning-permissions-to-jobs


moved the changes in hugo from ci to development Dockerfile

please make sure you think my changes are ok for dagger

lgtm


Update issue templates

Include the "New User Story"  template.

@eazeredo it fails on one of the checks commit lint which validates the commit message.
Our team follows the convention called Conventional Commits which is used mostly in the community.
https://www.conventionalcommits.org/en/v1.0.0/#summary
so for this commit, you can change it something like:
chore: update issue templates


Hi 👋
I tried to reproduce this from within a container, however, docker pull was successful for me.
I used this dev config for creating the dev container👇
{
	"image": "mcr.microsoft.com/devcontainers/base:ubuntu",
	"features": {
		"ghcr.io/devcontainers/features/docker-in-docker:2": {}
	}
}

@rvanlaar I wonder if the issue is with the underlying OS for your container?

After your comment I did check again.
Created two debian 11.6 containers. One 'unprivileged'.
This error is reproducible on the 'unprivileged' container.

This error is reproducible on the 'unprivileged' container.

I see, @rvanlaar could you help me out with repro steps? thanks!

I see, @rvanlaar could you help me out with repro steps? thanks!

I use proxmox https://www.proxmox.com/en/ on the host and use their functionality to create containers:
https://pve.proxmox.com/wiki/Linux_Container
Used debian 11.6 and Inside the container I installed Docker.

Thanks, @rvanlaar  can you open an issue for proxmox ? Looks like the issue is on their end as these images can be pulled with dev containers, locally with docker desktop and also with Codespaces.

Thanks for your time looking into the issue.


[3287] Add 0 allocation request validations

Context
We shouldn't accept 0 allocation requests for initial requests.  "Repeat"
request should still allow "0" places because this is where we don't have date
for the training provider.
Changes proposed in this pull request
Validate number_of_places is not zero if the request type is "initial"
Guidance to review

try to create new allocations with the following params
provider_code: "B20", provider_id: 15052, request_type: "initial", number_of_places: 0

Note: when I tried adding presence: true Publish blows up
Checklist

[ ] Make sure all information from the Trello card is in here
[ ] Attach to Trello card
[ ] Rebased master
[ ] Cleaned commit history
[ ] Tested by running locally

 [S] Add 0 allocation request validations