Window is not resized because of limit, but window content do

I think this is a bug.
https://user-images.githubusercontent.com/18114381/103560352-38f7a680-4eea-11eb-98f8-6328c1d95a03.mp4

This is actually intended (if admittedly confusing) behavior!
You are attempting to make the window thinner than the widest widget, and that's not possible (because there is no horizontal scrolling which could allow that). So why is Egui so early wrapping the text only to then not make the window thin?
It turns out that because Egui is an immediate mode GUI, Egui can't know whether or not there will be a wide widget in the window until that widget is added (which is outside the frame of this video). So Egui will optimistically make all widgets fit in to the thin window (which is why you see the text wrap etc) but will then "abort" the shrinking when the widget widget is added. But then it is too late to go back!
There is, as far as I can tell, only one way to solve this (in an immediate mode GUI), which is to add horizontal scroll bars to windows (and thereby forcing you to scroll sideways when you shrink a window by a lot). I haven't gotten around to implementing that yet, but even if I had I think I would disable it by default for most windows (because I'd prefer not to be able to shrink windows beyond the widest widgets).
I do agree that it looks quite glitchy, and I'm open to suggestions on how to improve that. Maybe there could be some helpful visual feedback to show that the window couldn't become as thin as requested.

This is actually intended (if admittedly confusing) behavior!
You are attempting to make the window thinner than the widest widget, and that's not possible (because there is no horizontal scrolling which could allow that). So why is Egui so early wrapping the text only to then not make the window thin?
It turns out that because Egui is an immediate mode GUI, Egui can't know whether or not there will be a wide widget in the window until that widget is added (which is outside the frame of this video). So Egui will optimistically make all widgets fit in to the thin window (which is why you see the text wrap etc) but will then "abort" the shrinking when the widget widget is added. But then it is too late to go back!
There is, as far as I can tell, only one way to solve this (in an immediate mode GUI), which is to add horizontal scroll bars to windows (and thereby forcing you to scroll sideways when you shrink a window by a lot). I haven't gotten around to implementing that yet, but even if I had I think I would disable it by default for most windows (because I'd prefer not to be able to shrink windows beyond the widest widgets).
I do agree that it looks quite glitchy, and I'm open to suggestions on how to improve that. Maybe there could be some helpful visual feedback to show that the window couldn't become as thin as requested.


Components section not rendered when serializing OpenAPI doc as V2 with inline reference settings

The components section does not get rendered when serializing an OpenAPI doc as V2 with inline reference settings - in the case of cyclic references. This section gets added as a reference yet a definitions section gets rendered.
demo:

OpenAPI V2 docs do not have components sections.  There were however a bunch of issues with inlining in v2.  The 1.2.0-preview.2 release has a bunch of fixes for inlining a v2 document.


Need to expose blend mode on NME preview window when in Particle Mode.

We need to have an option to change the blend mode of the particles in the preview window of NME, when NME is set to particle mode. This will allow the user to see proper representation of their particle effect.
@PatrickRyanMS - for the design
@msDestiny14 - for the implementation

I can do the implementation as soon as we have a design

I can do the implementation as soon as we have a design

Here we are

Awesome!!!!


[SPARK-47721][DOC] Guidelines for the Structured Logging Framework

What changes were proposed in this pull request?

As suggested in https://github.com/apache/spark/pull/45834/files#r1549565157, I am creating initial guidelines for the structured logging framework.
Why are the changes needed?

We need guidelines to align the logging migration works in the community.
Does this PR introduce any user-facing change?

No
How was this patch tested?

It's just doc change.
Was this patch authored or co-authored using generative AI tooling?

Yes. Generated-by: GitHub Copilot <IP_ADDRESS>87

cc @panbingkun @pan3793 @dtenedor @dongjoon-hyun @cloud-fan @itholic

@dongjoon-hyun Thanks for pointing it out. I just updated the PR description.

Merged to master.

Thanks for writing this. It's helpful


Install ice via composer instead of pecl

Hi,
Following https://github.com/dependabot/dependabot-core/issues/1990, you meqan @mruz that composer, with
{
  "require": {
    "iceframework/framework": "~1.8.0"
  }
}

could handle ice compilation
That's not what I have using composer install --no-dev --prefer-dist --classmap-authoritative (I'm using https://github.com/the-benchmarker/web-frameworks to try this).
Regards,

Note: Only scripts defined in the root package's composer.json are executed. If a dependency of the root package specifies its own scripts, Composer does not execute those additional scripts. -https://getcomposer.org/doc/articles/scripts.md#what-is-a-script-

You could need to define a script which calls ice's script:
{
  "require": {
    "iceframework/framework": "~1.8.0"
  },
  "scripts": {
    "post-install-cmd": [
      "cd vendor/iceframework/framework && ./install"
    ]
  }
}

It is defined, but composer does not execute those additional scripts from packages, only from root.


Add touch events to mobile Carousel

Sadly this is not working properly on Android 6 (Chrome), at the point that is not usable in the current state. :cry:
In iOS is working great.
Any ideas?

@battaglr Let´s invite the author @1987santiago to the conversation 😉

@atma, my bad! :expressionless:


Updated validation.py python script Closes: #1

Updated validations.py and is working fine now.
Hope you now happy with it :)
Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[x] Tests pass
[x] Appropriate changes to README are included in PR

@googlebot I signed it!


make javadoc work with UTF-8

Tentative fix for issue #1.
This patch tells javadoc to process UTF-8 encoded files.


url-0.5.9: failed to parse manifest

Trying to vendor cargo-vendor itself, I'm getting this error:
$ cargo vendor --verbose
 Downloading pnacl-build-helper v1.4.10
 Downloading gdi32-sys v0.2.0
error: unable to get packages from source

Caused by:
  failed to parse manifest at `/home/krig/.cargo/registry/src/github.com-88ac128001ac3a9a/url-0.5.9/Cargo.toml`

Caused by:
  could not parse input as TOML
/home/krig/.cargo/registry/src/github.com-88ac128001ac3a9a/url-0.5.9/Cargo.toml:14:3 expected a newline after table definition

error: third party subcommand `cargo-vendor` exited unsuccessfully

To learn more, run the command again with --verbose.

cargo build succeeds without complaint, so there's somehow a difference in how it parses TOML, because it does download and compile url-0.5.9 as well.

Ah yeah this is fixed in more recent versions of Cargo (as a library), and this can probably just be fixed by updating dependencies

The strange thing is that I build cargo version 0.10.0 and it works fine there, and when I build cargo-vendor I also build against cargo 0.10.0. The additional dependencies are exactly the same as for cargo. So it's not quite that simple..

Something like rust-lang/cargo#2680 is probably the fix that is needed here.

The odd thing is that I can't find a version of cargo which implements this behaviour. Before using toml 0.1.29, cargo would allow the malformed TOML. After 0.1.29, cargo sets require_newline_after_bool to false and warns about it. I'm clearly missing something here (still a rust newbie) so I think I'm going to have to wait for you to fix it ;)

Apologies for the spam here, but for the record I figured it out: Cargo.lock in cargo-vendor has toml 0.1.30 whereas Cargo.lock in the 0.10.0 branch of cargo has toml 0.1.28. Mystery solved, and the fix is simply to roll back to 0.1.28 in cargo-vendor until the next version of cargo is released.


[Java] Provide shaded arrow-memory-netty

Describe the enhancement requested
Offer downstream projects a lifeline
Component(s)
Java

This may be hard to do/pointless due to how DefaultAllocationManagerFactory works


FACTURAS

DOCUMENTOS XML

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

ID: 4fb42032-a67b-b601-7ef0-bdf1c8838f73
Version Independent ID: 4d052b88-6fe1-358e-47a7-4d2687f02c10
Content: App Center Push for Universal Windows Platform - Visual Studio App Center | Microsoft Docs
Content Source: docs/sdk/push/uwp.md
Service: vs-appcenter
GitHub Login: @elamalani
Microsoft Alias: emalani

@wars23 Do you have a question or comment about the docs?


Help!! Unable to predict CNN model: How to resolve incompatible layers?

Please make sure that this is an issue related to performance of TensorFlow.
As per our
GitHub Policy,
we only address code/doc bugs, performance issues, feature requests and
build/installation issues on GitHub. tag:performance_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Windows 10
TensorFlow installed from (source or binary): pip
TensorFlow version (use command below): 2.2.0
Python version: 3.8.4rc1

Describe the current behavior
Can't predict model due to unexpected layer (expects 100,352 instead gets 131,072)
Standalone code to reproduce the issue
Will provide a jupyternotebook link soon
Other info / logs Include any logs or source code that would be helpful to
diagnose the problem. If including tracebacks, please include the full
traceback. Large logs and files should be attached.
ISSUE
Help!! I can't predict my model because it's giving me the error:
ValueError: Input 0 of layer dense_3 is incompatible with the layer: expected axis -1 of input shape to have value 100352 but received input with shape [None, 131072]
I just finished training a CNN using a ResNet50 architecture and a few top layers.
This is the code I used for creating the model...
base_model = ResNet50(weights = None, include_top=False, input_shape=(200, 200, 3))

x = base_model.output
x = Flatten()(x)
x = Dropout(0.2)(x)
x = Dense(32, activation='relu')(x)
x = Dense(16, activation='relu')(x)
predictions = Dense(num_class, activation='softmax')(x)

# The model to be trained
model = Model(inputs=base_model.input, outputs=predictions)

model.compile(loss='binary_crossentropy',
              optimizer='rmsprop',
              metrics=['accuracy'])

callbacks_list = [keras.callbacks.EarlyStopping(monitor='val_acc', verbose=1)]
model.summary()

As you can see, I only used a few layers on top of the ResNet50 architecture. I also used an image size input of 200, 200, 3
Went it calls back, this is the summary for the FLATTEN to the 2ND LAST DENSE LAYER.
__________________________________________________________________________________________________
flatten_1 (Flatten)             (None, 100352)       0           conv5_block3_out[0][0]
__________________________________________________________________________________________________
dropout_1 (Dropout)             (None, 100352)       0           flatten_1[0][0]
__________________________________________________________________________________________________
dense_3 (Dense)                 (None, 32)           3211296     dropout_1[0][0]
__________________________________________________________________________________________________

The dense layer expects an input of shape 100,352 but instead it receives an input of 131,072!! Hence, the value error when running the predict code via...
img_path = 'train/10_right.jpeg'
img = image.load_img(img_path, target_size =(256,256))
x = image.img_to_array(img)
x = np.expand_dims(x, axis=0)

preds = model.predict(x)

@JacobOfCorns
Please share complete stand alone code for us to replicate the issue or if possible share a colab gist with the error reported.

Note that your input image has size 256 x 256. Assuming it has two channels, that gives you a vector of size 131072. In your network definition, you wrote input shape=(200, 200, 3). Please check that.


/api/v1/repos/search -> sending too much informations back

Gitea version (or commit ref): 1.14.0
Git version:
Operating system:

Database (use [x]):

[x] PostgreSQL
[ ] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ ] No

Log gist:

Description

I analyzed which api calls are you doing for this view.

Cause when I search for some repos and when I click << < x > >> than I see this steps:

176 is changing to 0
sending /api/v1/repos/search with parameters
show next founded repos
0 is changig to 176

I was wondering why the switch 176 <-> 0?
This is the response from /api/v1/repos/search api call:

And if you take deep look on the data you see

I would say, that you don't need 95% of the informations for this view.

There's always someones usecase that potentially uses these fields, so if someone is looking into making a PR for this perhaps a field filter?
(relevant xkcd: https://xkcd.com/1172/)

We could introduce a new route /api/ui/repos/search to only provide necessary fields.

I seriously doubt this is a major performance issue. Yes it's wasteful but we have much bigger fish to fry.

I seriously doubt this is a major performance issue. Yes it's wasteful but we have much bigger fish to fry.

I like fish! ;)
With the new /api/ui/repos/search route it will be better. But you are showing informations for others which shouldn't be shown for people, who has no admin access to the projects. Just show informations which are needed for this view.


Enhancement | Dark / Light mode

Could this be implemented with Shadcn UI as it's a new project.

Yes, Shadcn is already added to the project.

I'd like to implement it. Any theme specifications ?

@vverma022 - Sorry. This issue has been resolved. the feature is live.


High cpu & threads

For a large filesystem the first update takes a lot of time and puts the whole server to 100% cpu for hours. Is there a way to optimize this, for e.g:

Slow scan, takes longer but doesn't choke the CPU
Limit threads
I am not sure whether if I kill the process after , say 15 mins, it will resume the scan from the same point. If the update is resumable that also can help a lot!

Not sure how much data it is so that the process takes hours? But yeah it's possible to use fewer threads but you'd need to patch the source for that. I'd suggest trying something else first, in constants.hpp:
// File data compression level, 0-9
const int kFileDataCompressionLevel = 5;

Try reducing this to 0 and see if that helps. If it does I could add this as an official option.
It's possible to use fewer threads but that would mean the process takes much longer to complete so that's probably not desirable. The update process is not resumable/restartable.

P.S. If the above doesn't help, it would be helpful to get the profile of qgrep as it runs (eg using perf), so that I have a more precise idea of the source of performance problem.

I don't have the tools to re-compile after the changes. I am going to use nice and cpulimit to limit the cpu.
Also, I am getting this
qgrep: src/build.cpp:583: void buildAppendFilePart(BuildContext*, const char*, unsigned int, const char*, size_t, uint64_t, uint64_t): Assertion `file.startLine < startLine' failed.
Aborted (core dumped)5243 Mb in, 1742 Mb out

The assertion has been fixed a few months ago, the next version (which should release this week) will fix that (see https://github.com/zeux/qgrep/issues/17). I'm going to double check if there's something meaningful that can be done for update performance, a few optimizations have been committed already but they don't address the bulk of the cost. Regardless using nice is a reasonable workaround.

Thank you @zeux . Appreciate the response, will wait for your update.

Please do a release for buildAppendFilePart we are facing a lot of issues.

You can download a pre-release build from artifacts on this page https://github.com/zeux/qgrep/actions/runs/3456466725

A new release has been published. It accumulates several performance improvements to project update as well as the assertion fix. I don't think there's much else that can be done here, as there's not a lot of performance left on the table after the recent fixes, so I'm going to close this.

Thanks. How can I sponsor this project?


Error specifying fieldmaps using aas_addsubject

Hello,
I’m trying to specify fieldmaps using aas_addsubject with the following:
aap = aas_addsubject(aap, 'sub-01', ‘sub-identifier, ’fieldmaps',[11 12]);
Tasklist:
 <module><name>aamod_get_dicom_fieldmap</name></module>               <module><name>aamod_convert_fieldmaps</name></module>
However, the fieldmap modules in the task list do not get executed.

Could you please provide an example of the correct way to do this?
Is it necessary that other functional series from the scan also be specified at the same time?

Thank you.
Aahana

Hi Aahana1,
I can see several issues with your script, but it is hard to judge without seeing the whole.
Anyway, you can find some fitting examples here: UMS and tasklist. I hope they help.
Let me know how you proceed.
Tibor

Hi @Aahana1, did you have any luck with the examples? I have also realised that the main issue might be related to your second question. Yes, you do need functional (or diffusion) series, because fieldmaps are interpreted only in relation with them.

Hi @tiborauer, I've tried implementing the script with fieldmaps + functionals and also fieldmaps + diffusion series but didn't have much luck with either. The problem might also be due to differences in the internal file structures that I'm working with. I haven't had a chance to work further on these lately, I will update you if something works on my end once I get back to it.
Thanks for your help.
Aahana

Hi @Aahana,
Let me know if you need further help. Regarding file structure, you can use the example demo data https://files.osf.io/v1/resources/umhtq/providers/osfstorage/5b7465680b87a00018c6a76f  and the corresponding script https://github.com/automaticanalysis/automaticanalysis/blob/master/examples/aa_user_demo.m . Unlike the example in my previous message, this one is not with fieldmap, though.
The main pint is that you do not have to accommodate your file structure to aa, because you can tell aa about your file structure using the aap_parameterset. The most corresponding field are:

rawdatadir desc: Directories to find raw MRI data
subjectoutputformat: sprintf formatting string to get subject directory from identifyer
seriesoutputformat: sprintf formatting string to get series directory from number (e.g. Series_%03d*). It is only relevant if you organise your DICOM images into subfolders according to series.
protocol_fieldmap, protocol_structural, protocol_t2: string patterns to autoidentify protocols for fieldmap, structural, and T2 (the protocol DICOM tag varies among vendors: SeriesDescription for GE and Philips, and ProtocolName for Siemens)
dicomfilter: string pattern to filter DICOM images (e.g. *.dcm)

Vale,
Tibor
Auer, Tibor (Ph '99)
From: Aahana1<EMAIL_ADDRESS>Sent: 19 October 2019 01:33
To: automaticanalysis/automaticanalysis<EMAIL_ADDRESS>Cc: Tibor Auer<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [automaticanalysis/automaticanalysis] Error specifying fieldmaps using aas_addsubject (#201)
Hi @tiborauer https://github.com/tiborauer , I've tried implementing the script with fieldmaps + functionals and also fieldmaps + diffusion series but didn't have much luck with either. The problem might also be due to differences in the internal file structures that I'm working with. I haven't had a chance to work further on these lately, I will update you if something works on my end once I get back to it.
Thanks for your help.
Aahana
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/automaticanalysis/automaticanalysis/issues/201?email_source=notifications&email_token=ABEXZKPNQDIYWDZCYVI44EDQPJISBA5CNFSM4IPEKIKKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBWSLUQ#issuecomment-544024018 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ABEXZKIR4VYSIBYDMCDXP6LQPJISBANCNFSM4IPEKIKA .  https://github.com/notifications/beacon/ABEXZKN73SJA4CPPLWAKSKDQPJISBA5CNFSM4IPEKIKKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBWSLUQ.gif

Hi @tiborauer ,
I think part of my fieldmap issue might be because of the following.
This is the output from aamod_autoidentifyseries_timtrio.m which suggests that aa is able to find the field map and diffusion files.
Series 11 (SpinEchoFieldMap_AP_2p0mm ) with 3 dicom files
Series 12 (SpinEchoFieldMap_PA_2p0mm ) with 3 dicom files
Series 21 (ep2d_diff_dbsi_99_iso2_wholebrain ) with 6930 dicom files
Series 22 (ep2d_diff_dbsi_99_iso2_wholebrain_AP) with 70 dicom files
Series 23 (ep2d_diff_dbsi_99_iso2_wholebrain_PA) with 70 dicom files
I have two different tags ’SpinEchoFieldMap_AP_2p0mm’ and ‘SpinEchoFieldMap_PA_2p0mm’ for fieldmaps and I'm assuming aa is looking for 1 generic tag for the protocol.
I’m not sure how to accommodate for this in setting the aap.directory_conventions.protocol_fieldmap
On a different note, I tried running my code with your new PR #202 since it also had some changes to aamod_convert_filedmaps.m.
I noticed that the end statement in line 45 might have been misplaced. When the if statement in line 24 is false, Vfm never specified. This might not have much to do with my issue but just thought i'd let you know.
Thanks,
Aahana

You do not have to specify the full name of the protocol, because aa uses a pattern search. However, you have to make sure that the pattern you specify is distinguishing enough. So I would suggest "SpinEchoFieldMap" as protocol_fieldmap.
Another issue is that topup/opposing PE is not implemented for fMRI but for diffusion only in aa. It is something I am working on, but I have nothing to share at the moment.
Thank you for your test and notice for PR #202. I will update the code ASAP.

Hi, @Aahana1, I have updated the PR #202

@Aahana1, Do you still have the issue or shall we close it?

Hi @tiborauer , I've closed the issue for now. Thank you.


Closes: #1

Updated validations.py python script.
Fixed the behavior of validate_user function in validations.py.
Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[ ] Tests pass
[ ] Appropriate changes to README are included in PR

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
View this failed invocation of the CLA check for more information.
For the most up to date status, view the checks section at the bottom of the pull request.
Check again


SPIKE: Auto open  (to onboard / connect) on load of the dapp

When you open console, open the wallet
Could be that if you connected once before, auto trigger it in the future
Spike
To establish what the unknowns are to achieve the story below.
Story
As a Vega user
I want connecting to my recently installed wallet to happen as quickly as possible
So that I can get started on Console without having to think about or find the connect wallet button
Acceptance Criteria

[ ] Given I have just installed the vega wallet and returned to Console / a Vega dapp - I want to immediately be prompted to complete onboarding or connect to dapp
[ ] There is a way to prompt the wallet to open from a dapp e.g. the tab auto refreshes and looks for a wallet installed on the right port, or something else

Tasks

[ ] Detailed sketch
[ ] Visual design
[ ] Frontend tasks:

[ ] Implement UI
[ ] Integrate with backend
[ ] Write unit tests

[ ] Backend tasks:

[ ] TBC
[ ] Write unit tests

[ ] Write end to end tests
[ ] Code review
[ ] Team verify acceptance criteria met
[ ] Exploratory QA testing
[ ] Capture bugs and new edge cases in tickets (this one or new TBD)
[ ] Write automated regression tests for the A/Cs (expanding on A/Cs where possible)
[ ] Add story and A/Cs to spec doc
[ ] Review with stakeholders
[ ] Release

Notes
This is about finding a way to join up the journey of installing the wallet and coming back to your console tab e.g. triggering it to open to onboard / connect
For Firefox if we can't force the pop up to open we can open in a new window the same size and position as the pop up instead.
To make that work we'd need Console to refresh or something, might be some weird trickery required.. unless something can be built into the wallet itself
Wallet could look at your tabs, tell you to refresh your console tab if you have one open? And that could trigger connecting... could we include something in the URL to auto find tabs in that state - could apply to any dapp so any dapp can hook in and join up to the onboarding flow
Sketch

@mattrussell36 I'm not sure if we are / shoudl be / want to do this ... lets discuss next week  Doesnt seem to have come up in any of our plannig.

This should presumably be @macqbat and @dexturr on this.  Barney confirmed its pretty high priority for onboarding.


Run against SWE-Bench

Feature description
Devin, the "First AI software engineer" is using their SWE-Bench performance as primary evidence of their capability.
Motivation/Application
Provide a benchmark to evaluate Devin's claims, and solidify gpt-engineer's reputation as a legitimate autonomous coding agent.

I also had the urge to implement this after the Devin announcement 😄
But there is already an issue for it: https://github.com/gpt-engineer-org/gpt-engineer/issues/913
Closing as duplicate, continuing the discussion in the above issue.


Question: do you plan to further develop this?

Hi, this is an excellent project with great utility. Seems work on this was stopped or did it move elsewhere? Is there any plan to continue working on this?

Hi @shafqatevo
currently, the features supported already achieve what I need in my projects.
If you have any idea that could make this project better, we're open to discussion!

Thanks @wwwy3y3!
We are very interested to port it to Dart to be used with graphql-flutter package and for the new FireStore database besides or instead of RTDB. graphql-flutter project has link support in their latest codebase - https://github.com/zino-app/graphql-flutter/pull/76/commits/2205b940d8d7279e0a7d3e55d35e7d83b0f6c05a)
This is the missing piece now to use client-side-only GraphQL in Flutter or AngularDart with Firebase in a proper way.
I am naively assuming this won't be much effort since you have a good structure already. One of our team members will work on this and we surely need some guidance from you and if you're also on Dart/Flutter-Firebase, we could work together.

@shafqatevo Firestore support sounds nice!
I did think about firestore before, but the API of firestore is more complicated than real-time database, and I didn't have time to think through the directives design of firestore.
We should have sth like this
const query = gql`
  query getUsers {
    users @firestore(collection: "users", where: [{name: "country", operator: "==", value: "US"}]) {
      id @key
      name
    }
  }
`;

Maybe we can open another issue talking about firestore support, and I'd love to give you guys guidance and some technical support to support firestore in this module.

Thanks @wwwy3y3 ! I will open another issue for this.


preserve data after delete block device claim

Description

When I tried the OpenEBS local device featuers, I found that if I add an already formatted block device to OpenEBS, it didn't format it and mounted it into the pod, which worked like a charm. However, after I deleted the pod and the PVC, OpenEBS would delete the underlying PV, BlockDeviceClaim and clear the block device. What can I do to perserve data after this process?

OK, after went through related codes, I found that this part of job is done by the node disk manager, and add openebs.io/reconcile=false this annotation to the block device object will disable the auto disk format procedure. However, you need to manually edit the block device's status and finalizer after delete related PVC and BDC. But this meet my requirement, which ensures my data's safety.


Mii sample jrf fix

Maggie - Please test this fix.

With this PR's change now initial JRF domain is up with both admin server and 2 managed servers
kubectl get pods -n sample-domain1-ns --watch
NAME                                READY   STATUS    RESTARTS   AGE
sample-domain1-admin-server         1/1     Running   0          2m40s
sample-domain1-managed-server1-c1   1/1     Running   0          64s
sample-domain1-managed-server2-c1   1/1     Running   0          64s


feat(services): add SFN client

Current dependencies on/for this PR:

main

PR #57

PR #58   👈

This comment was auto-generated by Graphite.


Add ability to define different config files for dev and production

I agree with the advice in "Build Awesome Command-Line Applications in Ruby" about placing config files in a user's home directory. Doing so is great but adds one complication to development. Without additional logic, using:
config_file File.join(ENV['HOME'], ".app_config.yml")

means that both the development and production versions of the script will be using the same config file. When using Cucumber or another testing tool, it's easy to overwrite ENV['HOME'], but sometimes it's helpful to run the development version of the app directly from the command line.
Two solutions I can think of are:

Add a new config_file_for_dev method that would be used in conjunction with config_file. If the file defined in config_file_for_dev exists, use it. Otherwise, use the file defined in config_file as normal.
Update config_file to have the optional ability to accept a hash with keys for dev and prod. Again, if the dev file exists, it's used. Otherwise, the prod file is.

The more I think about it, the more I like the first option. Especially since it doesn't add a new interface to "config_file" so there's less potential problems with backwards compatibility.

Incidentally, the way I'm solving this now is to do the following in my bin/app_name file:
executable_directory = File.expand_path(File.dirname(__FILE__))
config_path_for_dev = File.join(File.dirname(executable_directory), 'config_for_dev.yml')
config_path_for_prod = File.join(ENV['HOME'], "#{GLI::App::exe_name}.yml")

if File.exists? config_path_for_dev
  config_file config_path_for_dev
else
  config_file config_path_for_prod
end

In the apps development location, I put a "config_for_dev.yml" file at the root. When running bundle exec bin/app_name, its found and used. When I use gem to deploy the tool, it uses the config file from the home directory. I've only done a little testing with this, but so far this seems to work fine. (Note that when using Cucumber, etc... they will see the and use the "config_for_dev.yml" file too. I don't think this is ideal, but it allows separation between dev and prod.)

An option I would prefer instead would be allow you to pass in a global option to specify the config file to use.  This would be trivial to do if the config_file command would support a lambda, or something else that would allow it to be expanded at runtime vs. a hardcoded string.

FYI, this was trivial to add in.  I patched this in to my application, but am providing it here in case you want to add it in to gli itself.
lib/gli/commands/initconfig.rb:
action do |global_options,options,arguments|
+         unless global_options[:config].nil?
+           @filename = global_options[:config]
+         end
          if options[:force] || !File.exist?(@filename)

This allows everything to work as you would expect.  If you don't specify -c|--config, then the value passed in is used.  If -c|--config are used, then it uses that.

Consolidated into #305


PyTorch data loader convergence benchmark on rossmann

Created two notebooks to visualize the convergence of metric (e.g. valid_rmspe) on the rossmann sales dataset.
It compares native PyTorch data loader and NVTabular data loader for PyTorch.

run tests


Compilling on SheevaPlug : ARM processor

I can't compile and build a deb package on my SheevaPlug.

It's a ARM processor.
I use Debian Lenny 5 with Apache2, PHP5 and MySQL.

Can you build it ?
Can you tell me for compiling and building a deb package for this architecture.

Thanks.

Original issue reported on code.google.com by colundrum on 17 Jan 2011 at 10:21

This is a very old thread but attempting to resurface due to the increasing need to compile for arm64. There is a similar discussion for Nginx as well. https://github.com/apache/incubator-pagespeed-ngx/issues/1213


error running modbamtools

Hi!
I´m trying to run modbamtools with my megalodon output to vizualize the data, but Iḿ getting some errors and I don´t spot the issue.
modbamtools plot -r CM039687.1:2914823-19333007 -bd ./modbamtools/ -o ./modbamtools/ ./modbamtools/mod_mappings_sorted.bam
This is my code, and in the modbamtools/ folder I have the bam file (with its .csi index file), the modified_bases.5mC.bed and also a modified_bases.5mC.bedgraph which I parsed from the bed file.
I get this error.
Traceback (most recent call last):
  File "/usr/local/bin/modbamtools", line 8, in <module>
    sys.exit(cli())
  File "/usr/local/lib/python3.8/dist-packages/click/core.py", line 1128, in __call__
    return self.main(*args, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/click/core.py", line 1053, in main
    rv = self.invoke(ctx)
  File "/usr/local/lib/python3.8/dist-packages/click/core.py", line 1659, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/usr/local/lib/python3.8/dist-packages/click/core.py", line 1395, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/usr/local/lib/python3.8/dist-packages/click/core.py", line 754, in invoke
    return __callback(*args, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/modbamtools/cli.py", line 300, in plot
    fig = Plotter(
  File "/usr/local/lib/python3.8/dist-packages/modbamtools/modbamviz.py", line 36, in __init__
    self.tracks, self.num_tracks = get_tracks(
  File "/usr/local/lib/python3.8/dist-packages/modbamtools/tracks.py", line 235, in get_tracks
    b = parse_bedgraph(bedgraph, chrom, start, end)
  File "/usr/local/lib/python3.8/dist-packages/modbamtools/tracks.py", line 27, in parse_bedgraph
    with open(bedgraph_path) as b:
IsADirectoryError: [Errno 21] Is a directory: './modbamtools/'

I quite lost with the PATH options, maybe Iḿ run it wrong.
Could your give any hint?
Thanks!

Hi!
option -bd should be used with a path to a bedgraph format file but you are providing a path to a directory. However if you are trying to point to megalodon bed files, that is unnecessary as modbamtools automatically plots methylation frequency on top of the single read plots. Let me know if that works!

Ohhh, so I just could run it with the .bam file like this:
modbamtools plot -r CM039687.1:2914823-19333007 -o ./modbamtools/ ./modbamtools/mod_mappings_sorted.bam
Because I did, but my RAM can´t handle it. After a wile the process kiled.
I have 16Gb, could it be because the range I selected is to wide? like 18 Mbs?

Hi! As expected I selected a shorter window and run just fine! Thanks!
Let me ask you another question. Is it possible to plot two samples (i.e. two genotypes) in the same plot?
What do you recommend for the downstream analysis like find DMRs between experiments?
Thanks again

yes you can! just do modbamtools plot options bam1.bam bam2.bam ...
you can change the sample names to whatever you want with --samples option.
as far as DMR detection, this is something I am actively working on but it's not available yet. you could potentially use megalodon bed files with other tools to find DMRs in the meantime

Hi! Great! Thanks for the replay!
Late night yesterday I run the modbamtools calcMeth and it run till morning (8hrs aprox) and then the RAM colapse and kill the processes.
Could it take that long? is because have to calculate de methylation on all the bed file (all the genome?)
modbamtools calcMeth -t 10 -o ./modbamtools/ -b ./modbamtools/modified_bases.5mC.bed ./modbamtools/mod_mappings_sorted.bam

Ohh, you are right!!! My bad. I don´t have an annotation file for this organism (a grass), but I think I could "make" a file with region of interest.
Is it possible that the plot also display others methylation contexts? like CHG and CHH (very important in plants?) Or the bam file don´t have that information? I think I have read on one of the megalodon issues that is something the want to adress.
An another quick question, what are representing the colors of the reads on the plot?
thanks a lot!
Diego

there is an all context 5mC model available for megalodon or guppy/bonito that you can use.
Red color represents methylated bases and blue represents unmethylated bases.


Allow BlockAreaName translations

This would allow translations to have
fi:
Block:
BlockAreaName:
MainContent: 'Pääsisältö'

Thanks!


Tweak BuildPulse/rspec-retry logic

BuildPulse is trying to find flaky tests for us but, given the previous model of using rspec-retry, it would rarely find them. Instead, let's try to always rerun brew tests multiple times, report to BuildPulse each time (by moving the reporting logic into brew tests) and disable rspec-retry when using BuildPulse.
While we're here, let's enable rspec-retry locally so we don't have flaky tests biting maintainers/contributors there.
CC @jasonrudolph for thoughts

Review period skipped due to critical label.


    public virtual SelectExpression PushDownSubquery() forgot predicate

see Join_customers_orders_with_subquery_predicate_with_take
That create this SQL statement
                @"SELECT [c].[ContactName], [t0].[OrderID]
FROM [Customers] AS [c]
INNER JOIN (
    SELECT TOP(5) [o2].*
    FROM [Orders] AS [o2]
    ORDER BY [o2].[OrderID]
) AS [t0] ON [c].[CustomerID] = [t0].[CustomerID]
WHERE [t0].[CustomerID] = 'ALFKI'"

Instead of
                @"SELECT [c].[ContactName], [t0].[OrderID]
FROM [Customers] AS [c]
INNER JOIN (
    SELECT TOP(5) [o2].*
    FROM [Orders] AS [o2]
    WHERE [o2].[OrderID] > 0 //************where part was missing
    ORDER BY [o2].[OrderID]
) AS [t0] ON [c].[CustomerID] = [t0].[CustomerID]
WHERE [t0].[CustomerID] = 'ALFKI'"

When runnig test, I get fails because the text that's generated contains \r\n and the expexted only \n.
I have replaced all Sql with Sql.Replace("\r",''") to pass all test but there must be a better way to do this.
Also I can't buld the last DEV version. Is there a way I can see the last succesfull build like the MVC project history mvc

I can build the latest dev, what issues do you see?
The linefeed issue is due to your git configuration, change and get files again.

@ErikEJ Thx. I had the line ending problem in the delivery zip file of the Beta 6 pacakge.
The error is

Total build time elapsed: 00:00:03.4769526
Total projects built: 1
warn: Exit code -1 from cmd
verbose: Stack trace:
   at View2ecd540af9644e8da975776bf95845ef.Exec(String program, String commandline) in c:\Users\Thom\AppData\Local\Temp\561d48d715ce42d3a845e5aff9d6578c-1.cs:line 271
   at View2ecd540af9644e8da975776bf95845ef.DnuPack(String projectFile, String kpmPackOutputDir, String configuration) in c:\Users\Thom\AppData\Local\Temp\561d48d715ce42d3a845e5aff9d6578c-1.cs:line 645

   at View2ecd540af9644e8da975776bf95845ef.<>c__DisplayClass9f.<RenderViewLevel0>b__61(String projectFile) in c:\Users\Thom\AppData\Local\Temp\561d48d715ce42d3a845e5aff9d6578c-1.cs:line 7076
   at System.Collections.Generic.List`1.ForEach(Action`1 action)
   at View2ecd540af9644e8da975776bf95845ef.<>c__DisplayClass9f.<RenderViewLevel0>b__5f() in c:\Users\Thom\AppData\Local\Temp\561d48d715ce42d3a845e5aff9d6578c-1.cs:line 7076
   at Sake.Engine.Builder.BuilderBase`1.CallTarget(String name)
   at Sake.Engine.Builder.BuilderBase`1.StartingTarget(String name)
   at View2ecd540af9644e8da975776bf95845ef.<RenderViewLevel0>b__47() in c:\Users\Thom\AppData\Local\Temp\561d48d715ce42d3a845e5aff9d6578c-1.cs:line 1830
   at Sake.Engine.Builder.BuilderBase`1.CallTarget(String name)
   at Sake.Engine.Builder.BuilderBase`1.StartingTarget(String name)
   at View2ecd540af9644e8da975776bf95845ef.<RenderViewLevel0>b__48() in c:\Users\Thom\AppData\Local\Temp\561d48d715ce42d3a845e5aff9d6578c-1.cs:line 1847
   at Sake.Engine.Builder.BuilderBase`1.CallTarget(String name)
   at Sake.Engine.Builder.BuilderBase`1.StartingTarget(String name)
   at View2ecd540af9644e8da975776bf95845ef.<RenderViewLevel0>b__49() in c:\Users\Thom\AppData\Local\Temp\561d48d715ce42d3a845e5aff9d6578c-1.cs:line 1864
   at Sake.Engine.Builder.BuilderBase`1.CallTarget(String name)
   at View2ecd540af9644e8da975776bf95845ef.<RenderViewLevel0>b__41() in c:\Users\Thom\AppData\Local\Temp\561d48d715ce42d3a845e5aff9d6578c-1.cs:line 1720
   at Sake.Engine.Builder.BuilderBase`1.CallTarget(String name)
   at Sake.Engine.SakeEngine.Execute(Options options)
   at Sake.Engine.SakeEngine.Execute(String[] args)
   at Sake.Program.Main(String[] args)

I think there are problems with nuget. Yesterday and today i got connection errors/ file not found

I have clear the nuget cache and get these errors again

Did you run
build initialize

Yep that works.
How do I run all test.
The error I got is in building
Building EntityFramework.Commands for DNX,Version=v4.5.1 and
Building EntityFramework.Commands for DNXCore,Version=v5.0
But 4.5 has no error
Building EntityFramework.Commands for .NETFramework,Version=v4.5

I am not sure that EF.Commands build properly under those platforms - @lajones ?

Merged


c-dark-theme toggle between pages

I currently have coreui dark theme toggle working, it correctly toggles between light and dark mode. The problem is it doesn't remember the setting between pages or reloads.
Things i have done to debug:
double checked all tags on the button (i am using fontawesome icons instead of coreui)-

                   <button class="c-class-toggler c-header-nav-btn" type="button" id="header-tooltip" data-target="body" data-class="c-dark-theme" data-toggle="c-tooltip" data-placement="bottom" title="Toggle Light/Dark Mode">                                                             </button>

all updated scripts are called correctly
all css scripts are called correctly and updated
no JS errors on page load

Your question was from March so not sure if still relevant / needed but I've done something like this for now during development (as this isn't builtin functionality):
HTML:

        <button class="c-header-nav-btn" type="button" title="Toggle Light/Dark Mode" onclick="toggleColorScheme()">

        </button>

Script at the end of body:


Add complete session start time to FicTrac

When working on the Clandining conversion I realized that the first line of the data header has non-valid values save the timestamp. The timestamp is the unix time epoch in milliseconds which can be used to extract a more complete session_start_time as previously we were extracting only the date.
I did some other improvements to the code like adding starting_time when they are not zero which were not done before.

Oh, I thought that ZoneInfo was introduced in python 3.8 but turns out is not available until python 3.9 : (
Related:
https://github.com/catalystneuro/neuroconv/issues/599


Check if minecraft world exists

Hello,
the script should check whether the given minecraft world save exists before starting its calculations.
Currently, the script runs just to abort when the file could not be saved in the end.
Kind regards

Thanks for the idea, I'll include this check!


Generated URL has an extra forward slash?

I'm trying to use the Cloud Storage API in the PHP library and I'm getting the following error:
Type: Google_Service_Exception
Message: Error calling POST https://www.googleapis.com//upload/storage/v1/b/gauntface-site-uploads/o?name=download.png&uploadType=multipart: (400) Uploads must be sent to the upload URL. Re-send this request to https://www.googleapis.com/upload/storage/v1/b/gauntface-site-uploads/o?name=download.png&uploadType=multipart
I've looked through the code and there are two places I can think of that could be tweaked to fix this issue.
https://github.com/google/google-api-php-client/blob/master/src/Google/Http/MediaFileUpload.php#L241
or
https://github.com/google/google-api-php-client/blob/master/src/Google/Service/Storage.php#L62
I don't know the library well enough to say which would be the best, or whether there is something else that should be happening to make everything consistent.
Happy to make a PR if anyone can direct me on the right solution.

Getting the same issue when I have updated this lib. How did you solve it?

Same here, hopefully this gets solved soon

This is no longer a bug in v2. This can potentially be closed. I am not sure if v1 will still be maintained.


Upgrade ZeroBraneStudio.app to v1.20

Thank you for the contribution. It was merged directly as dd89445e0e3c2aee01cea590e09c326caad8bbbd to keep commit history cleaner. Your contribution is still credited to you.

Thank you for the contribution. It was merged directly as a072c6be9bcbdc8b25ed4e08956ae19abf885932 to keep commit history cleaner. Your contribution is still credited to you.


cc-wrapper, bintools-wrapper: Remove now unneeded set {+,-}u

Motivation for this change
I noticed late in https://github.com/NixOS/nixpkgs/pull/72347 that I missed a few in the wrappers. This could cause more breakage since the set +u now "leaks" with out the stdenv cleanup, so I figured rather than last-minute add to the last PR, I'd make another.
https://hydra.nixos.org/jobset/nixpkgs/bash-no-undef-vars
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

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

Notify maintainers
cc @

OK, basically all the Darwin errors are due to unreated python and swift problems, and I just fixes the last linux errors I see, so I think this is ready for staging. I'll just watch staging if any darwin problems get uncovered.

Yeah that sounds good


Allows setting of Optional fields with Scala Options

Adds optional setters for people working with Options (Scalaland)

Sample of Code Generated by this Function:
  public TEmail setEmailAddress(scala.Option<java.lang.String> emailAddressOpt) {
    if(emailAddressOpt.isDefined()) {
      this.emailAddress = emailAddressOpt.get();
    }
    return this;
  }

I'll merge this once I confirm more thoroughly that it functions as intended.


Defining Arrow extension types

The Arrow format has the concept of "extension types" (https://arrow.apache.org/docs/format/Columnar.html#extension-types), which is a way to extend the built-in types by annotating one of those built-in types with an extension name and metadata.
The underlying built-in type that is being annotated is called the "storage" type, and the name and metadata can be used to specify how the storage type should be interpreted. In serialized form (eg when passing data through IPC), the extension type is defined by field metadata, with keys "ARROW:extension:name" (identifying string name) and "ARROW:extension:metadata" (optional additional information to reconstruct the type).
Because it annotates an existing arrow logical type, even if a certain Arrow implementation does not support the extension type (i.e. does not recognize the extension name), it can still handle the underlying data.
So for being able to faithfully pass geospatial data between Arrow implementations, we can define a set of agreed extension types that annotate the actual data (the nested lists format as being proposed in https://github.com/geopandas/geo-arrow-spec/pull/12).

Extension name
We can use a "geoarrow" namespace, followed by the specific geometry type that is being stored (for now, https://github.com/geopandas/geo-arrow-spec/pull/12 only defines arrays with a single, fixed geometry type).
In practice this would give names such as "geoarrow.point", "geoarrow.multipolygon", etc
Extension metadata
The name and actual storage type can already fully define the actual geometries. But there is some useful additional metadata that we can store:

Coordinate reference system (CRS) of the geometries (see also https://github.com/geopandas/geo-arrow-spec/issues/8)
Whether the geometries should be interpreted as geodesic (spherical coordinates) or planar
The dimension of the geometries? (xy vs xyz vs xym) Athough this can in principle be inferred from the storage type

This metadata can semantically be interpreted as a key-value mapping like {"crs": "...wkt string..", "dim": "xy", "geodesic": False}. How to exactly store this in practice in serialized form (as a JSON string, or some binary format, ..) is still something to decide, but there is a thread about this on the Arrow mailing list, so we can for now leave that discussion for on that list.

What is being written down here is largely based on what @paleolimbot has been prototyping in https://github.com/paleolimbot/geoarrow, and myself in https://github.com/jorisvandenbossche/python-geoarrow
Some open questions I am aware of:

Do we also want to formalize a "geoarrow.wkb" extension type? (probably yes, since we already use WKB for the current Parquet files, and we probably also want to fall back to WKB for mixed geometry types) And others? (WKT, GeoJSON, ..)

For the arrow-native types defined in https://github.com/geopandas/geo-arrow-spec/pull/12, do we only annotate the top-level type, or also the different nesting levels?
For example, for a "multipolygon" type (which is stored as List<List<List<FixedSizeList<double>[2]>>>), the type itself is annotated as "multipolygon", but the value in the first list level could also be annotated as "polygon", and the inner-most level as "point".
Some questions related to this:

What is the advantage of this? (-> you can extract the values of the MultiPolygon column, and get an array of polygons (the "exploded" MultiPolygons) that is still recognized as a geoarrow type)
What with metadata? Are those repeated at each level, or only on the outer or inner annotation?
Given that gives some complexity, do we see this as "required", or rather something optional?

For the multi-part geometries, do we define separate extension types (i.e. "geoarrow.multipoint", "geoarrow.multilinestring", "geoarrow.multipolygon"), or a single "geoarrow.multi" that can hold all three of them (and which it is can be inferred from the value type in the first list level, which will be points, linestrings or polygons, respectively) ?
(I think defining them as separate types is probably the simplest / clearest for users)

Do we also want to formalize a "geoarrow.wkb" extension type?
Seems resonable, given the existing precedent

And others? (WKT, GeoJSON, ..)
These seem like they undo the benefits of using a compact binary encoding, and in particular GeoJSON leads to ambiguity in what is stored: geometries vs features.  It seems very reasonable to leave these out until there is a very clear reason to add them.

do we define separate extension types
This also seems reasonable; it seems like it allows greater clarity without undue complexity / computational overhead

Whether the geometries should be interpreted as geodesic (spherical coordinates) or planar
Is the idea that this provides hints as to the geometry engine (GEOS / S2 / etc) that should be used on derserialization of those coordinates?  I can see the advantage of splitting this out from CRS, since you don't necessarily always intend to represent geographic coordinates in a geodesic engine.

If I'm following along here, it doesn't seem like there is a downside in case there is no geodesic engine available upon deserialization; it could fall back to using a planar engine since you are only storing coordinates and not a representation that requires a geodesic engine to deserialize them properly.

Whether the geometries should be interpreted as geodesic (spherical coordinates) or planar

Is the idea that this provides hints as to the geometry engine

Yes, in the context of this metadata, it is just a "hint" for users that can be used (and eg currently geopandas doesn't support spherical coordinates, so would likely just ignore it or raise a warning about it). AFAIU for libraries with such support (eg based on S2), this hint can be useful though (geometries that are valid when interpreted as planar coordinates are not necessarily valid when interpreted as spherical coordinates).

GeoJSON does have a pretty standard interpretation when used in this context (e.g., both GEOS and BigQuery will happily read it for you)

Just to be sure: is this "GeoJSON" about a string representation of it? (a json dump) Or an actual nested data type? I suppose the string version since otherwise you can't use that to load data into a database using COPY?


vis: 2016-07-15 -> 2016-08-24

Motivation for this change

update.
removed superfluous dependency on unzip.
moved pkgconfig to nativeBuildInputs.
renamed vis-nightly -> vis-unstable following new naming guidelines.
fixed VIS_PATH with $HOME/.config to allow personal configurations and highlighters following upstream's usage of the variable.
added a desktop shortcut.
added myself to maintainers.

Things done

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

[x] NixOS
[ ] OS X
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

@RamKromberg I am unable to test it out it currently (and merge), as I am in a place with sporadic low bandwidth net access for a month. Your changes look good otherwise. I hope someone else can test it and merge it.

@vrthra don't worry about it I have like a dozen PRs waiting for a couple of weeks so this one can queue up right along with the rest of them just fine :)
This specific fix \ update been stuck in my /etc/nixos for a while since I got busy with other things but now that's it's the weekend... :D Why, I even have an updated standalone version of sam ripe for the picking if you're curious: https://github.com/NixOS/nixpkgs/pull/18305
So really, don't rush things on my account :)

Thanks!


format_dict: only unescape quotes when removing outer quotes

If the outer quotes are removed, then it's safe and necessary to remove the escapes on the inner quotes. Otherwise they should be left alone or they will break the double-quote terminated string.
Only remove double quote escapes if we are for sure removing outer quotes with a curly brace; a string containing the pattern }" will never match this regex because the quote following the curly will always be escaped by the JSON encoder as }\". So our regex here will only ever match a curly followed by JSON encoder's closing double quote (which is always unescaped).
This allows rx.html strings to contain double quotes and generally fixes weird quoting issue described in REF-226.

This blows up reflex-web with the following error
Server Error

Error: Invalid tag: {({node, ...props}) => <Text {...props}  />}

and that's all it gives... 🤔

This blows up reflex-web with the following error

The problem was exposed in the <ReactMarkdown> component, which was taking a big dictionary of props with Vars interspersed. This was being formatted as a long string, which itself had curly braces within it, so the moment the regex hit the closing curly brace }, it stopped matching and broke the formatter.
This was fixed by instead doing a non-greedy match on anything between "{ and }", as long as the opening quote-brace was not preceeded by an escape character.
The latest change also switches to re.VERBOSE mode and adds comments for each element.


Update submission instructions on Meetup page

In "About" roll over on Meetup.com it currently says to submit to the Wufoo form, and this link was just in a tweet. Also, can a non-collaborators add labels to an issue? Maybe I'm missing that.

@laras126 Good point....we have to get that updated ASAP...thanks for pointing that out!


try diff branch

try a new PR to fix tests

@tazarov figured out the issue here - closing this in favor of https://github.com/chroma-core/chroma/pull/1032


Registration not working

Email is not sent.

Email is sent..after 15 minutes.

We're now using Google for login


mavlink: 2019.12.30-1 in 'melodic/distribution.yaml' [bloom]

Increasing version of package(s) in repository mavlink to 2019.12.30-1:

upstream repository: https://github.com/mavlink/mavlink.git
release repository: https://github.com/mavlink/mavlink-gbp-release.git
distro file: melodic/distribution.yaml
bloom version: 0.9.0
previous version for package: 2019.12.12-1

Same as for #23382.


Add support to query errors in run_all_tests.py

run_all_tests.py can not handle errors that was issue during query
please notice:
when implementing this issue, please remove the following lines from run_all_tests.py :
if file == 'scan-dir-test-scheme.yaml':
continue

not relevant, we are using query_not_executed mechanism


Proposal: Please add CarouselPanel here

Proposal: Can we get CarouselPanel in here
Summary
Related to https://github.com/Microsoft/microsoft-ui-xaml/issues/36, in that CarouselPanel is used by UWP ComboBox.  I did look for a feature request just for ComboBox, but I didn't see one so I'm creating this so as to encourage a high performance ComboBox as the CarouselPanel shoud virtualize.
Rationale
High performance ComboBox's which can have lots (000s ) of items in the ItemsSource are often used in LOBs.  Often just for the purpose of converting a key to a name with only the option to open which is often not exercised.

Functional Requirements

Important Notes

Open Questions

The carousel functionality is useful but CarouselPanel itself isn't something we can easily expose (especially since it uses a bunch of internal hooks). But it would be beneficial to build something like that on top of ItemsRepeater. @micahl @ranjeshj do you have any samples like this?

The carousel functionality is useful but CarouselPanel itself isn't something we can easily expose (especially since it uses a bunch of internal hooks). But it would be beneficial to build something like that on top of ItemsRepeater. @micahl @ranjeshj do you have any samples like this?

I don't have a sample, but it should be relatively easy to build it on top of ItemsRepeater i think.

Thanks for the replies.  To be honest I wasn't aware of the ItemsRepeater.   If that could be used in the ComboBox ItemsPanel to get virtualization then that would be good.  I'll close this.

I'd like to keep this open to track at least getting a CarouselPanel sample built using ItemsRepeater.

@yowl, the ComboBox's CarouselPanel does UI virtualization today.  Are you hitting a situation where it's not?

@micahl, you mean the UWP one? That is fine.  I was refering to this code repo, if I search for CarouselPanel here, https://github.com/Microsoft/microsoft-ui-xaml/find/master, nothing comes back.  I suppose a bit of background to my question is in order as I can see why there is confusion.  Some other projects would benefit from the source code to a virtualizing combobox, and I was thinking of https://github.com/nventive/Uno, and I thought that if the UWP combox source code along with its CarouselPanel was available it would benefit that project.  Maybe you see my request as a bit underhand and if so I apologize for not being upfront about my reason for this issue.

@yowl, thanks for clarifying.  It sounds like the ask here is similar to #217.  You may find it useful to see these two docs (one which I just posted yesterday)
Custom collection views / ItemsRepeater guidance
Attached Layouts - talks about creating a virtualizing layout
They don't necessarily cover everything about creating a virtualizing control, but I think you and others will still find them useful.  There's a lot that can go into building a custom virtualizing control so hearing from you and others on what you need will help prioritize what additional content gets created.

And something to mitigate myself.  We have a large investment in Silverlight and hence looking for ways to not throw that away.

I'm closing this issue in favor of the general WinUI 3.0 issue (#717).  Adding CarouselPanel here would be included in that work since we plan to make WinUI 3.0 open source.
Fwiw, to see a virtualizing layout that carousels based on a newer architecture you can take a look at this layout that we used in this sample.  We're considering a more complete CarouselView control (#771) that would use it.


Add a sitemap

A brief exchange with the "Search On Math" team reminded me that ar5iv should ideally have a sitemap in order to allow for quick direct access to all article pages.

But now that the site is getting retired soon, maybe this is the wrong way to spend effort...


Generate amino types mainnet

Should we add it before adding some test?

I'll add it, with tests for proto messages and also tests for amino, so we can easy deploy it on the main branch to support all the basic features for the mainnet


@material-ui/core package is disappear from npm

$ yarn install

error An unexpected error occurred: "https://registry.yarnpkg.com/@material-ui%2Fcore: Not found".

https://www.npmjs.com/package/@material-ui/core

ESOCKETTIMEDOUT

Does anyone know what is going on? Looks like the @material-ui/core package is removed from npm, does anyone know how to fix this?

So was react-dom https://www.npmjs.com/package/react-dom.

😢

@oliviertassinari
the same https://www.npmjs.com/package/@emotion/core
wondering what is going on??

Something is going on:
https://status.npmjs.org/

Not found
<EMAIL_ADDRESS>

if it can help i have this error :
Not Found - GET https://registry.npmjs.org/@material-ui%2fcore - Not found npm ERR! 404  npm ERR! 404  '<EMAIL_ADDRESS>is not in the npm registry.
i hope

Hi,
Same issue with yarn :-(
$yarn add @material-ui/core
yarn add v1.22.4
warning package.json: No license field
warning No license field
[1/4] Resolving packages...
error An unexpected error occurred: "https://registry.yarnpkg.com/@material-ui%2Fcore: Not found".
info If you think this is a bug, please open a bug report with the information provided in "/tmp/yarn-error.log".
info Visit https://yarnpkg.com/en/docs/cli/add for documentation about this command.

This is also there for us, preventing our CI.

Same issue happenig for us as well.
$ npm install @material-ui/core npm ERR! code E404 npm ERR! 404 Not Found - GET https://registry.npmjs.org/@material-ui%2fcore - Not found npm ERR! 404 npm ERR! 404  '@material-ui/core@latest' is not in the npm registry. npm ERR! 404 You should bug the author to publish it (or use the name yourself!) npm ERR! 404 npm ERR! 404 Note that you can also install from a npm ERR! 404 tarball, folder, http url, or git url.

Having the same issue.

The ongoing issue may also affect installation of scoped packages. We continue investigating these issues.

Please subscribe to https://status.npmjs.org/incidents/cksjqc1w11v5?u=33mgrqjjqdb1 to be notified of updates. There's nothing we can do about it


Sometimes cannot go down with down arrow

Hey all, thanks for making this. I was playing around with the demo on the website and found two bugs that I'll lump in one here.
First if I go to the Framer example and type "baa", then delete the last "a", the only result showing is not highlighted which seems like a bug to me. However I can highlight it by going down with the down arrow.
Then if I delete another "a", I see two results, but I cannot go up/down with the arrows anymore.
Here's a video where I'm pressing up/down arrow at the end.
https://user-images.githubusercontent.com/4534692/183738721-7e2a009e-c38e-484d-808c-e1f3e369868d.mov

Thanks for the report, was able to reproduce and trace it back to a change made recently.
Should be fixed in #27 and on this preview:
https://cmdk-website-git-paco-fix-unstable-ids-paco.vercel.app


fix: add unusedFlag to avoid job run used GroupExprssion

Proposed changes
Issue Number: close #xxx
Problem summary
Describe your changes.
Checklist(Required)

Does it affect the original behavior:

[ ] Yes
[ ] No
[ ] I don't know

Has unit tests been added:

[ ] Yes
[ ] No
[ ] No Need

Has document been added or modified:

[ ] Yes
[ ] No
[ ] No Need

Does it need to update dependencies:

[ ] Yes
[ ] No

Are there any changes that cannot be rolled back:

[ ] Yes (If Yes, please explain WHY)
[ ] No

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 35.24 seconds
load time: 491 seconds
storage size:<PHONE_NUMBER>8 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230111140216_clickbench_pr_77845.html


AWS Lambdas not updating from cloudformation properly

Reporting Bugs
Issue
I'm deploying lambdas with serverless, and localstack, and the issue we are running into is that the old lambdas are not being deleted which I believe is leading to the newly created lambdas not being able to be created.
Example: I have a serverless application called my-app, I deploy to localstack and the first time it works perfectly. I make changes, and deploy again, and the function is still only the function from the first deploy.
There is some traceback, but from what I've read it may not be an issue.
Traceback
2019-07-16T11:51:46:ERROR:werkzeug: Error on request:
Traceback (most recent call last):
File "/usr/local/lib/python2.7/site-packages/werkzeug/serving.py", line 303, in run_wsgi
execute(self.server.app)
File "/usr/local/lib/python2.7/site-packages/werkzeug/serving.py", line 291, in execute
application_iter = app(environ, start_response)
File "/usr/local/lib/python2.7/site-packages/moto/server.py", line 104, in __call__
return backend_app(environ, start_response)
File "/usr/local/lib/python2.7/site-packages/flask/app.py", line 2309, in __call__
return self.wsgi_app(environ, start_response)
File "/usr/local/lib/python2.7/site-packages/flask/app.py", line 2295, in wsgi_app
response = self.handle_exception(e)
File "/usr/local/lib/python2.7/site-packages/flask/app.py", line 1741, in handle_exception
reraise(exc_type, exc_value, tb)
File "/usr/local/lib/python2.7/site-packages/flask/app.py", line 2292, in wsgi_app
response = self.full_dispatch_request()
File "/usr/local/lib/python2.7/site-packages/flask/app.py", line 1815, in full_dispatch_request
rv = self.handle_user_exception(e)
File "/usr/local/lib/python2.7/site-packages/flask/app.py", line 1718, in handle_user_exception
reraise(exc_type, exc_value, tb)
File "/usr/local/lib/python2.7/site-packages/flask/app.py", line 1813, in full_dispatch_request
rv = self.dispatch_request()
File "/usr/local/lib/python2.7/site-packages/flask/app.py", line 1799, in dispatch_request
return self.view_functions[rule.endpoint](**req.view_args)
File "/usr/local/lib/python2.7/site-packages/moto/core/utils.py", line 140, in __call__
result = self.callback(request, request.url, {})
File "/usr/local/lib/python2.7/site-packages/moto/core/responses.py", line 117, in dispatch
return cls()._dispatch(*args, **kwargs)
File "/usr/local/lib/python2.7/site-packages/moto/core/responses.py", line 200, in _dispatch
return self.call_action()
File "/usr/local/lib/python2.7/site-packages/moto/core/responses.py", line 274, in call_action
response = method()
File "/usr/local/lib/python2.7/site-packages/moto/cloudformation/responses.py", line 294, in update_stack
tags=tags,
File "/usr/local/lib/python2.7/site-packages/moto/cloudformation/models.py", line 330, in update_stack
stack.update(template, role_arn, parameters=parameters, tags=tags)
File "/usr/local/lib/python2.7/site-packages/moto/cloudformation/models.py", line 112, in update
self.resource_map.update(self.template_dict, parameters)
File "/usr/local/lib/python2.7/site-packages/moto/cloudformation/parsing.py", line 526, in update
resource_name, resource_json, self, self._region_name)
File "/usr/local/lib/python2.7/site-packages/moto/cloudformation/parsing.py", line 323, in parse_and_delete_resource
resource_class.delete_from_cloudformation_json(
AttributeError: type object 'LambdaVersion' has no attribute 'delete_from_cloudformation_json'

I added in a temporary dummy method, to fix that, and then I got almost the same error with a similar traceback, with the error
...
File "/usr/local/lib/python2.7/site-packages/moto/cloudformation/models.py", line 330, in update_stack
stack.update(template, role_arn, parameters=parameters, tags=tags)
File "/usr/local/lib/python2.7/site-packages/moto/cloudformation/models.py", line 112, in update
self.resource_map.update(self.template_dict, parameters)
File "/usr/local/lib/python2.7/site-packages/moto/cloudformation/parsing.py", line 545, in update
raise last_exception
AttributeError: type object 'LambdaFunction' has no attribute 'update_from_cloudformation_json'

Thanks for opening; setting this as a feature request.

https://github.com/spulec/moto/issues/793


Uninstall Shortcut

Hi
Why don't create a shortcut like bower i PolymerElements/paper-button for also the UNINSTALL command.
Like this : bower u PolymerElements/paper-button
Good or Bad Idea?
Thanks
Riccardo Amadio

@rokity bower is deprecated, new features arn't going to be accepted.


Update testnet to 2.223.0

Description of proposed changes
Update Testnet to 2.223.0

Practical tips for PR review & merge:

[ ] All GitHub Actions report PASS
[ ] Newly added code/functions have unit tests

[ ] Coverage tools report all newly added lines as covered
[ ] The positive scenario is exercised
[ ] Negative scenarios are exercised, e.g. assert on all possible errors
[ ] Assert on events triggered if applicable
[ ] Assert on changes made to storage if applicable

[ ] Modified behavior/functions - try to make sure above test items are covered
[ ] Integration tests are added if applicable/needed

Codecov Report

Merging #1203 (8354bbb) into testnet (7a6f5ca) will decrease coverage by 67.77%.
The diff coverage is 28.02%.

@@             Coverage Diff             @@
##           testnet   #1203       +/-   ##
===========================================
- Coverage    74.62%   6.85%   -67.77%
===========================================
  Files           74      25       -49
  Lines        10686     744     -9942
  Branches         0      87       +87
===========================================
- Hits          7974      51     -7923
+ Misses        2712     693     -2019

Impacted Files
Coverage Δ

scripts/cc-cli/src/commands/balance.ts
0.00% <0.00%> (ø)

scripts/cc-cli/src/commands/bond.ts
0.00% <0.00%> (ø)

scripts/cc-cli/src/commands/distributeRewards.ts
0.00% <0.00%> (ø)

scripts/cc-cli/src/commands/newSeed.ts
0.00% <0.00%> (ø)

scripts/cc-cli/src/commands/send.ts
0.00% <0.00%> (ø)

scripts/cc-cli/src/commands/setKeys.ts
0.00% <0.00%> (ø)

scripts/cc-cli/src/commands/status.ts
0.00% <0.00%> (ø)

scripts/cc-cli/src/commands/unbond.ts
0.00% <0.00%> (ø)

scripts/cc-cli/src/commands/validate.ts
0.00% <0.00%> (ø)

scripts/cc-cli/src/commands/wizard.ts
0.00% <0.00%> (ø)

... and 4 more

... and 85 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


I2C doesn't work with repeating starts

This is part of the spec, and is required by some devices. Fixed in this PR. Copy+pasted write_read method from the H7 HAL.
Pre-fix: https://i.imgur.com/wONAYTl.png
Post-fix: https://i.imgur.com/8NhBxVk.png
Note the Sr (Repeating start) flag towards the left on the fixed one, replacing the P S (Stop, start) on the current impl. And note the correct data on the fix towards the right, compared to 0xFF.
Per the Embedded Hal docs, this is how WriteRead is expected to behave.

Just weighing in here to point to the I2C specification corroborating this. See: Figure 13 in section 3.1.10 of UM10204.

Just weighing in here to point to the I2C specification corroborating this. See: Figure 13 in section 3.1.10 of UM10204.

I tested on stm32f303x8 and can't reproduce the issue...

I tested on stm32f303x8 and can't reproduce the issue...

What device did you test with? I2C works on many devices as-is, but the current non-compliant implementation fails on others. This brings it closer to I2C standards. (There's still a problem of Nacking at the end I haven't solved)

What device did you test with? I2C works on many devices as-is, but the current non-compliant implementation fails on others. This brings it closer to I2C standards. (There's still a problem of Nacking at the end I haven't solved)

According to section 3.1.10, Master-receiver Nacking at the end is a spec compliant behavior.

According to section 3.1.10, Master-receiver Nacking at the end is a spec compliant behavior.


no example, no demo, no doc...

no example, no demo, no doc...

@leonardgithub check out the /example/ReduxRouter folder for an example and the README for installation and usage instructions.


LZ4 decompression for unknown original size

Is your feature request related to a problem? Please describe.
If we use LZ4 compression for replication then we should adjust a client code to decompress values.
To decompress a value of unknown length, we have to reserve a large buffer since we don't know the original size of a value or we have to have tricky logic of increasing this buffer incrementally if a decompressed value doesn't fit in.
Describe the solution you'd like
Write the original uncompressed size (maxCompressedLength) as a leading integer in the compressed value byte array. In this case, we will know the exact size of a buffer we should use for decompression on a client size. Or just use LZ4CompressorWithLength. Similar to this.
Describe alternatives you've considered
I'd also appreciate any solution of how to decompress a value of unknown length with lz4-java in an efficient way.

Thanks @nwheeler81 for fixing it so quickly!
I think we may want to change
val sizePrefix = ByteBuffer.allocate(4).putInt(input.length).array()

to
val sizePrefix = ByteBuffer.allocate(4).putInt(inputBytes.length).array()

otherwise, the size will be wrong for strings with Unicode chars that take >1 byte.
Also, any reason to not use LZ4CompressorWithLength?

@nwheeler81 yes, it's readable from python-lz4
This is the code I've used to test it
In Java, to generate lz4 file:
public class Main {
  private static final String STR = Strings.repeat("test", 32);

  public static void main(String[] args) throws IOException {
    LZ4Factory factory = LZ4Factory.fastestInstance();
    LZ4Compressor compressor = factory.fastCompressor();
    LZ4CompressorWithLength compressorWithLength = new LZ4CompressorWithLength(compressor);
    byte[] compressed = compressorWithLength.compress(STR.getBytes(StandardCharsets.UTF_8));
    Files.write(Paths.get("/tmp/test.gz"), compressed);
  }
}

and in python3, to read and decompress it:
import lz4.block

f = open('/tmp/test.gz', 'rb')
b = f.read()
f.close()

str = lz4.block.decompress(b)

print(str.decode('utf-8'))
print(int.from_bytes(b[:4], 'little'))

and it gives me
python3 test.py

testtesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttest
128

Note that we don't even need to skip the first 4 bytes using python-lz4.
It uses the same schema to store uncompressed size as java-lz4
On the other hand, Golang and its lz4 module doesn't have this feature. So you'll have to deal with it manually:
import (
	"encoding/binary"
	"github.com/pierrec/lz4"
	"os"
)

func main() {
	s, err := os.ReadFile("/tmp/test.gz")
	if err != nil {
		panic(err)
	}

	size := binary.LittleEndian.Uint32(s[:4])

	var d = make([]byte,size)

	_, err = lz4.UncompressBlock(s[4:], d)
	if err != nil {
		panic(err)
	}

	println(string(d))
}

So, IMHO, it's safe to switch your code to LZ4CompressorWithLength.
But I'd also provide more details in the README about how the length is stored.
In particular, I'd mention that it's int32 with little-endian ordering - just to help people to decode it correctly if their lz4 lib doesn't support it.

LGTM


Support async sessions in strawberry_sqlalchemy_mapper and deprecate sync.

I left a note in the loader's init, but to duplicate that here: Making blocking database calls from within an async function (the resolver) has catastrophic performance implications. Not only will all resolvers be effectively serialized, any other coroutines waiting on the event loop (e.g. concurrent requests in a web server), will be blocked as well, grinding your entire service to a halt.
There's no reason for us to support a foot bazooka, hence the deprecation.

Thanks for adding the RELEASE.md file!

Here's a preview of the changelog:

Adds support for async sessions and deprecates sync sessions due to performance reasons.

Codecov Report

Merging #53 (e2ace34) into main (070ec9a) will increase coverage by 1.13%.
Report is 3 commits behind head on main.
The diff coverage is 94.02%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #53      +/-   ##
==========================================
+ Coverage   77.37%   78.51%   +1.13%
==========================================
  Files          10       10
  Lines         725      782      +57
  Branches      108      116       +8
==========================================
+ Hits          561      614      +53
- Misses        135      137       +2
- Partials       29       31       +2

I'd love to get Erik's opinion on this 😊

I'd love to get Erik's opinion on this 😊

However, we must ensure that we don't compromise synchronous support in the process. As previously discussed in another PR, the halting of the event loop executing the GQL Request in a WSGI Webserver isn't particularly concerning due to the presence of other worker threads.
Given that Strawberry caters to both ASGI and WSGI, it's essential for us to provide comprehensive support for both async and sync sessions throughout this library. While striving for this, it's critical to:

Assess the clarity and elegance of the dual support implementation.
Ensure that the resulting codebase remains maintainable.

It's important to acknowledge that there are genuine reasons why some might opt for a synchronous driver over an asynchronous one. These reasons remain relevant, even if there's a need to incorporate some asynchronous code, such as for the Dataloader pattern.
I completely agree on emphasizing the distinctions and considerations between sync and async. It's a crucial topic that users should be well-informed about. The best place for that is the documentation of this library. We should get that started soon 😊

Thanks for your thoughts Erik!
I'm still extremely skeptical of this sync bind, but there's nothing forcing us to add support for async and deprecate sync at the same time, so for now I got rid of the deprecation in this PR and we can take more time to discuss further and come to recommendations that we document (good call out on the documentation!)
Let's chat more when you and Patrick have time about the possibility of deprecating sync. For now, I updated the PR description and title.


icon-pack2 : stick-figures

closes #671
added dropdown + icons
icons fetched from https://svgsilh.com/tag/stickman-1.html

- I have participated in GSSOC and CrossWoC

In stick figure light the last icons is not a part of that illustration set.
Dark ones the first 2 are a bit off.

remove the 4,5,7 from them too

which pack? thin or bold?

Light


Make validate_call return ValidateCallWrapper

Change Summary
This is sort of the revert of #8268. The reason to switch back to ValidateCallWrapper is because I found that supporting Self in validate_call (WIP) can be much simplified with descriptors, since we now have access to objtype from __get__.

Related issue number

Checklist

[ ] The pull request title is a good summary of the changes - it will be used in the changelog
[x] Unit tests for the changes exist
[ ] Tests pass on CI
[ ] Documentation reflects the changes where applicable
[ ] My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

@Viicos @sydney-runkle Could you give some thoughts on this?
(Currently tests failed below Python 3.11 because inspect.markcoroutinefunction is added in Python 3.12)

Hey @kc0506,
We've really been appreciating your work here. Before you start work on a PR, could you please open an issue so we could discuss the plan? We've seen a lot of churn in the validate_call space that leads to a hard to follow blame + higher likelihood of introducing bugs in a minor version.
Thanks again for your work - we're looking forward to future collaboration!

No problem. I will note that in the future. Should I open a new issue to discuss this PR then?


Is ApiClient threadsafe?

I can't find any documentation specifying whether we can use a single ApiClient instance, or if we need a separate ApiClient for each thread.

Thanks! Thread unsafe works just fine, easy to solve with ThreadLocal. Just wanted a doc on what the intent was.

Hey @nedtwigg
actually, my assumption seem to have been wrong. The http client used by the library is thread safe. However I did a small release today, version 1.10.21 with small fine tuning to it.
Feel free to use the latest version, and you should be good to go.


Combobox in datagrid filter not working properly

Describe the bug
When a datagrid filter contains a multiselect combobox, there are some strange behaviors:

When a new (non selected) option is clicked from the list, it is added to the selected ones, but the whole filter disappears (the filter probably does not recognize the combobox's list as its own child component)

When an already selected option is clicked again, the filter disappears, BUT the list of options is still shown.

When the last selected option is removed, the ngModel (which is a list) becomes null.

How to reproduce
See on stackblitz: https://stackblitz.com/edit/clarity-v4-combobox-issue-datagrid?devtoolsheight=33&file=src/app/app.module.ts
Expected behavior
When clicking on a non-selected option, the filter should still be shown, and selected option should be added to the list.
When clicking on an already selected option, the filter should still be shown, the list should disappear and the selected option should remain the same.
When removing the last item which is selected, the ngModel attached to the combobox should become an empty list.
Versions
App

Angular: 10
Node:
Clarity: 4.0.0

Device:

Type: Lenovo P50
OS: Windows
Browser Chrome
Version 86

Additional notes
ngModelChange is called for every change twice, but I think that's not related to combobox.

An additional note from my side: we also think that both components (filter and combobox) using popovers might be related to the issue.

I agree with your findings about both components using popover... though it's a bit vague if the issue is in the combobox, or in the filter.


Request for Method to Calculate FLOPs and Params for vision-rwkv Model

Hello,
Could you provide a method to calculate the FLOPs and parameters for the vision-rwkv model? I have attempted to use the get_flops tool from mmpretrain, but encountered the following error:
File "mmpretrain/mmcls_custom/models/backbones/vrwkv.py", line 217, in forward
    x = _inner_forward(x)
File "mmpretrain/mmcls_custom/models/backbones/vrwkv.py", line 208, in _inner_forward
    x = RUN_CUDA(B, T, C, self.spatial_decay / T, self.spatial_first / T, k, v)
File "mmpretrain/mmcls_custom/models/backbones/vrwkv.py", line 90, in RUN_CUDA
    return WKV.apply(B, T, C, w.cuda(), u.cuda(), k.cuda(), v.cuda())
File "/home/miniconda3/envs/openmmlab/lib/python3.8/site-packages/torch/autograd/function.py", line 506, in apply
    return super().apply(*args, **kwargs)  # type: ignore[misc]
RuntimeError: _Map_base::at

Any help or guidance on how to resolve this would be greatly appreciated.
Thank you!

@xiaojieli0903
Hi, the script for calculating the FLOPs and parameters is uploaded(get_flops_backbone.py).
You can use it as follows:
cd Vision-RWKV/classification
python get_flops_backbone.py configs/vrwkv/vrwkv_tiny_8xb128_in1k.py

@duanduanduanyuchen
Thank you for providing the tool. I have successfully tested it.


fix: correctly filter my items within marketplace [wip]

Resolves #178

[x] Items where I am seeker and are not done, canceled or resolved
[ ] Items where I am provider and are not done, canceled or resolved
[ ] Items where I am markedplaceOwner and are disputed

@filoozom any idea how to do this efficiently? IMHO we should provide such functionality in SDK

@vojtechsimetka I pushed the code to filter out the provider's items as well, and some "pseudo code" for the marketplace owner. But as the whole marketplace owner part is not done yet, I'd suggest we already merge this and keep that part for later?


Update ubuntu to 22.04 from 20.04

Description
Update ubuntu to 22.04 from 20.04
Context
We are updating all ubuntu base images in SDF projects to 22.04 LTS.
Testing
I built the docker image and it built successfully.
Documentation
NA
Known limitations
NA

@satyamz Thanks!


Error in running the test.sh file

Hello, can you please help for this error, Thanks
Traceback (most recent call last):
File "", line 1, in
File "/usr/local/lib/python2.7/dist-packages/SimpleITK-0.9.1.post2-py2.7-linux-x86_64.egg/SimpleITK/SimpleITK.py", line 7724, in ReadImage
return _SimpleITK.ReadImage(*args)
RuntimeError: Exception thrown in SimpleITK ReadImage: /usr/src/SimpleITK/Code/IO/src/sitkImageReaderBase.cxx:71:
sitk::ERROR: The file "/output/images/cspca-detection-map/cspca_detection_map.mha" does not exist.
Expected detection map was not found...
Traceback (most recent call last):
File "", line 1, in
IOError: [Errno 2] No such file or directory: '/output/cspca-case-level-likelihood.json'
Expected case-level prediction was not found...
picai_baseline_nnunet_processor-output-8302c9287b

Hi,
This error is generated by the final part of the test.sh script and indicates that the detection algorithm did not produce an output. You should have more output logs that indicate what went wrong with the algorithm itself. What more do the logs indicate?
Best, Joeran

Thanks for the update; the issue is solved, as there was a mounting issue with the image's path at input volume.


Sort resources by name

Hello, I added a python script that will automatically sort resource files links. After inserting new material links, you can just run the script and the file will be sorted.
I'm open to any remarks and changes if needed.

Hi @milkiyd, did you use python code to sort automatically all links in the files? (I mean I'm seeing it, but just to confirm)

Yes it is python

Here we have the learning resource starting by S but, still stay on the bottom.

It may due to the fact that the resource name is starting with a lower case 's' which have an ascii code (115) greater than the uppwercase 'W' (87). Sorry i didn't checked that before commiting.
One fix could be to make sure that all resource names start with an uppercase letter, or I could try to handle these cases directly in the code.

Hi @milkiyd, did you use python code to sort automatically all links in the files? (I mean I'm seeing it, but just to confirm)

Yes it is python

Alright, nice

Here we have the learning resource starting by S but, still stay on the bottom.

It may due to the fact that the resource name is starting with a lower case 's' which have an ascii code (115) greater than the uppwercase 'W' (87). Sorry i didn't checked that before commiting.
One fix could be to make sure that all resource names start with an uppercase letter, or I could try to handle these cases directly in the code.

No worries @milkiyd, always start with capital letter any title or sentence, but for enhancement of the code, I agree that if possible, because it'll serve you in future project I guess and sure, so you can handle if it's a lower letter or not.

Hello @geoffreylgv , I updated the script according to your reviews, and added the feature to handle resources containing subsections.
The CONTRIBUTING.md file has also been updated.

Hello @geoffreylgv , I updated the script according to your reviews, and added the feature to handle resources containing subsections. The CONTRIBUTING.md file has also been updated.

Hi @milkiyd, I will look into it soon, well done


suggesttion: add connection pool support

Redis is single threaded usually there is no need for the client to implement a connection pool, but for some databases that using redis protocol, who handles each client with  one thread, a clinet with  multiple
connection may enhance performance.

That's already in place.


启动失败

bogon:starup-game-python username$ pip3 install -r requirments.txt
Collecting docopt==0.6.2 (from -r requirments.txt (line 1))
  Downloading docopt-0.6.2.tar.gz
Requirement already satisfied: termcolor==1.1.0 in /usr/local/lib/python3.6/site-packages (from -r requirments.txt (line 2))
Building wheels for collected packages: docopt
  Running setup.py bdist_wheel for docopt ... done
  Stored in directory: /Users/username/Library/Caches/pip/wheels/b2/16/5f/c33a2bb5f2dce71205f8e65cbfd05647d79d441282be31fd82
Successfully built docopt
Installing collected packages: docopt
Successfully installed docopt-0.6.2
bogon:starup-game-python username$ python game.py start
Traceback (most recent call last):
  File "game.py", line 17, in <module>
    from docopt import docopt
ImportError: No module named docopt

bogon:starup-game-python username$ pip3 install docopt
Requirement already satisfied: docopt in /usr/local/lib/python3.6/site-packages
bogon:starup-game-python tuchang$ python3 game.py start
Traceback (most recent call last):
  File "game.py", line 17, in <module>
    from docopt import docopt
ImportError: No module named 'docopt'

不是写了只支持python2么？

稍微改改括号啊什么的 好像也能进去

怎么乱码了兄弟？


VCV Module: Select random input

Should be pretty simple ... poly input, single out, random generator, trigger. Hrm. But it might be nice to order the inputs, and select them by round-robin, randomly, random gaussian, etc. Maybe two modules?

Sources of inspiration:

Overall form-factor, Merge:

code: https://github.com/VCVRack/Fundamental/blob/v2/src/Merge.cpp
svg: https://github.com/VCVRack/Fundamental/blob/v2/res/Merge.svg

Selecting one of several inputs as the output, Switch 4:

code: https://github.com/countmodula/VCVRackPlugins/blob/master/src/modules/Switch4.cpp
svg: https://github.com/countmodula/VCVRackPlugins/blob/master/res/Switch4.svg

Selection with a trigger, Rotator:

code: https://github.com/mhetrick/hetrickcv/blob/master/src/Rotator.cpp
svg: https://github.com/mhetrick/hetrickcv/blob/master/res/Rotator.svg

Normal/Guassian distribution in C++:

https://cplusplus.com/reference/random/normal_distribution/
https://stackoverflow.com/questions/19944111/creating-a-gaussian-random-generator-with-a-mean-and-standard-deviation


Add Makefile README

Suggest to add a link in README.md,  to link to README.Makefile.md

agree, we can put the link in the "install from source code" section in GetStarted.md