Fixing issue where light curve computation fails because of incorporating JPU units

This PR addresses the issue raised in #84 where the LimbDarkLightCurve class's light_curve function raises an error because of a TypeError. This error is caused by passing pint.Quantity objects into jnp functions even though a JAX array is expected. Since pint.Quantity objects can be correctly handled by the jpu.numpy functions, substituting the current jax.numpy function calls with the corresponding jpu.numpy function calls when these objects are being passed fixes this issue.
Other small changes were made so that the exposure time integration still works with the units. There is one hacky part, dt = jnp.outer(texp.magnitude, dt) * texp.units, where since jnpu does not support the outer function yet I'm manually separating the magnitude and the unit of texp, doing the computation using the usual jnp.outer function, and then reattaching the unit. @dfm: Maybe we could implement the outer function in jpu.numpy?
The returned light curves are now 1-D vectors so indexing is not necessary (unless you actually want the first flux calculation).

Thanks @soichiro-hattori! Can you add a test to tests/light_curves_test.py (it would be fine to just copy the example from #84 minus the plotting)?

Maybe we could implement the outer function in jpu.numpy?

Good idea - I'm actually surprised that it's not already there! Can you open an issue on that repo?

The returned light curves are now 1-D vectors so indexing is not necessary (unless you actually want the first flux calculation).

This is actually a little subtle - it depends on the type of orbit! But I think that's a conversation for another thread.

Thanks @soichiro-hattori! Can you add a test to tests/light_curves_test.py (it would be fine to just copy the example from #84 minus the plotting)?

The updated commit should have the test in there!

Maybe we could implement the outer function in jpu.numpy?

Good idea - I'm actually surprised that it's not already there! Can you open an issue on that repo?

Will do!

Since I've added a PR on the jpu repo to add the outer function, I've updated this commit to use the jnpu.outer function instead of the hacky approach mentioned above.

Looks great to me - thanks!!


chore: update action usage (use stable toolchain)

https://github.com/Homebrew/homebrew-core/pull/152159

Homebrew/homebrew-core#152159

thanks!


V2.0.0 pull request

Sam - there are two conflicts with this merge request, you can accept my changes for both these conflicts. I could not resolve the conflicts for some reason.


vertical alignment has no bottom

i have some code to grab the horizontal and vertical alignments :
   if let Some(varA) = cell.get_style().get_alignment() {
      tmp_cell_data["alignment"] = json!({
          "horizontal": varA.get_horizontal().get_value_string(),
          "vertical": varA.get_vertical().get_value_string(),
          "rot": varA.get_text_rotation(),
          "wrap": varA.get_wrap_text(),
      })
   }

given the following doc:

I get the following for vertical alignment:
top *A*
top *B*
top G
top H
top D
center M
none
none
top I
top J
none
center F
top C
top N
top K
top E

this is problematic given that most cells are bottom aligned.

@taylorh140, Looking at repo code it looks like default value of the VerticalAlignmentValues is set to as Top Variant. It is my guess that there is a problem reading and storing it in Alignment struct, the get_vertical() would yield its default value which happens to be Top.

@agentjill Any suggestions on how to work around this? or maybe I need to rebuild or PR with Bottom as default?

or maybe I need to rebuild or PR with Bottom as default?

I dont think that is a good idea. The default behaviour is correct to be Top. The problem may be due to reading the alignment. I think it may be addressed in next version (may be).

Thank you for your report.
Hmmm, agentjill is right, there may be something wrong with reading the alignment.
I will investigate.

The latest version has been released.
Please check to see if any problems have been resolved.

This seems to be resolved!


OPT_OUT in environment should override .opt-in file

Having opted in with an .opt-in file, I should then still be able to opt out for individual commands with an OPT_OUT=pre-commit environment variable. Right now the OPT_OUT environment variable does not override the .opt-in file.

Hæ @gthb
Do you have a particular use case for this or is this a theoretical case?
Originally, the functionality was intentionally limited to having commands from which a user can opt out from (or opt into); and to do so in a "flat" manner (that is, only creating a single local source of truth to opt into or out of certain commands).
What I thought about, from time to time, for quite some time now and which is now refreshed by this issue you raised, would be to make a v2 release that actually evaluates all the opt in and opt out variables available, in a hierarchical manner (that is: env variable overrides dot-file, which itself overrides package.json config object; opting out of an option on a level overrides opting into it on the same level and below).
Would this align with your idea(s) and/or requirement(s)?

Kveðja til uppáhalds eyjunnar minnar!

Hæ @gthb
Do you have a particular use case for this or is this a theoretical case?
Originally, the functionality was intentionally limited to having commands from which a user can opt out from (or opt into); and to do so in a "flat" manner (that is, only creating a single local source of truth to opt into or out of certain commands).
What I thought about, from time to time, for quite some time now and which is now refreshed by this issue you raised, would be to make a v2 release that actually evaluates all the opt in and opt out variables available, in a hierarchical manner (that is: env variable overrides dot-file, which itself overrides package.json config object; opting out of an option on a level overrides opting into it on the same level and below).
Would this align with your idea(s) and/or requirement(s)?

Kveðja til uppáhalds eyjunnar minnar!

Hæ! Yep, that would align with our requirements, and was the behavior I expected, until I tried it. :)
My use case was that I was introducing a lint-the-modified-files pre-commit hook to my team and proposing that people opt in with an .opt-in file, and then just opt out as needed, case-by-case, with OPT_OUT=pre-commit git commit.  The idea is to ease the concern of “obtrusiveness”, so that the opt-in is an easier sell: “it's easy to skip when needed, so it won't block you, just nudge you.”)

Hæ! Yep, that would align with our requirements, and was the behavior I expected, until I tried it. :)
My use case was that I was introducing a lint-the-modified-files pre-commit hook to my team and proposing that people opt in with an .opt-in file, and then just opt out as needed, case-by-case, with OPT_OUT=pre-commit git commit.  The idea is to ease the concern of “obtrusiveness”, so that the opt-in is an easier sell: “it's easy to skip when needed, so it won't block you, just nudge you.”)

Kveðja til uppáhalds eyjunnar minnar!

Ég skila því!

Kveðja til uppáhalds eyjunnar minnar!

Ég skila því!


chore(Dependencies): Fix ssh2 version

Actually some ssh hosts might not work with sha1 or md5 as hosthash, so ssh2 only handled it as none in =< 0.4.14 versions.

Related issues

#321
#324
#326

Workaround
To get it done before the PR get accepted, open the Remote-FTP folder in atom packages (probably $HOME/.atom/packages/Remote-FTP in unix) and change in package.json:
...
"ssh2": "^0.4",
...

then remove the node_modules/ folder and run npm install.

Beautiful, this works great. Thanks!

This works. Ty! Funcionou para mim também. Valeu!

Thanks, it works.


Add test for derived cache initialization when parent is already started

This scenario is currently untested, see https://coveralls.io/builds/13267639/source?filename=src%2Fmain%2Fscala%2Fcom%2Fgilt%2Fgfc%2Fcache%2FDerivedSyncCache.scala#L107

Wrong repo 🤦♂


Isolate node code cache on apple silicon

Steps to repro:

Install and run x64 version on M1
Install and override x64 with arm64
Crash on startup in v8 when trying to read the CachedData

Workaround:
Start the app with --no-cached-data
Proposed solution:
Split out the cache data with arch prefix CachedData-x64 , CachedData-arm64

code pointer: https://github.com/microsoft/vscode/blob/c87c95a1a04b01107328fe1987b1865b2eb367a8/src/main.js#L510

code pointer: https://github.com/microsoft/vscode/blob/c87c95a1a04b01107328fe1987b1865b2eb367a8/src/main.js#L510

This is now fixed. The root directory is still CachedData but the individual hash each cache file uses considers now the arch as well. Keeping the root directly the same helps with the cached data cleaning

This is now fixed. The root directory is still CachedData but the individual hash each cache file uses considers now the arch as well. Keeping the root directly the same helps with the cached data cleaning


Don't wait forever in TokNext TokMustReply

In the (TokNext TokMustReply) state the responder has indicated that it
currently doesn't have a new tip for us and we should wait.
This patch limits the time we wait to 70s which means that we don't end
up waiting forever incase of network problems.

bors merge


Update index.md

Why:
Closes ISSUE

What's being changed (if available, include any code snippets, screenshots, or gifs):

Check off the following:

[ ] I have reviewed my changes in staging (look for the "Automatically generated comment" and click the links in the "Preview" column to view your latest changes).
[ ] For content changes, I have completed the self-review checklist.

This seems to have been opened accidentally, I am going to close it now :yellow_heart:
Take a look at the contributing.md to learn how to contribute in this repo. You can open a discussion if you have any questions or check out issues with the help wanted label. :sparkles:

Quiero borrar mi suscripción
2022-12-28 9:09 GMT-06:00, welcome[bot] @.***>:

Thanks for opening this pull request! A GitHub docs team member should be by
to give feedback soon. In the meantime, please check out the contributing
guidelines.
--
Reply to this email directly or view it on GitHub:
https://github.com/github/docs/pull/22885#issuecomment-1366717972
You are receiving this because you are subscribed to this thread.
Message ID: @.***>


Add new Website Clones .

Can include clones of various  famous websites

Hey @Rayman-Sodhi ,
In coming days I want to add a  "  NETFLIX  "  website clone .
What do you think about this , Let me know ..................

Hey @Rayman-Sodhi
I would like to add a " Disney+ Hotstart " clone both backend and frontend.

@anugoyal998  you can start working on it ,but I will add the labels and then assign you . Most probably by 11 the scoring system will work

@anugoyal998  you can start working on it ,but I will add the labels and then assign you . Most probably by 11 the scoring system will work

Ok

@anugoyal998   you can add your role in the excel sheet and start working on this issue

@anugoyal998 you can add your role in the excel sheet and start working on this issue

which excel sheet?

check in discord

@Rayman-Sodhi i am interested in creating Zomato website clone

@VikasYadav-1  Make sure to add project id with your name  in excel sheet

Hey I am interested in making a clone of Linktree

@harshita214  Make sure to add project id with your name in excel sheet and start working

@anugoyal998 i want to make hulu clone project... pls assign me

@anugoyal998 i want to make hulu clone project... pls assign me

Sorry, I'm not the project admin

@Rayman-Sodhi i am interested in creating Zomato website clone

u haven't added it yet.

I am GSSOC-22 contributor I am interested to a contribute google keep clone, please assign me

hey u can start once contribution peroid starts.  Also which clone you want to add

Hey @Rayman-Sodhi , I'm a participant of GSsOC '22. I am interested in making a clone of inshorts(news) app.

Hello @Rayman-Sodhi
Can I make google classroom's website clone?
can you please assign me for this task?

Hi  @Ajoe12  and @sakshi0210  you can work on this issue ur project should have a working link and you should add that in the main website too once done with creating it .
Deadline -  5-6 days

@sakshi0210 are you still working on issues?

Yes
On Sat, Mar 19, 2022, 10:57 PM Gurjeet Singh Virdee <
@.***> wrote:

@sakshi0210 https://github.com/sakshi0210 are you still working on
issues?
—
Reply to this email directly, view it on GitHub
https://github.com/Rayman-Sodhi/Clone-IT/issues/5#issuecomment-1073049137,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARNOOYTXPOWHG5XRX6DHIYLVAYFAPANCNFSM5JKKG7KA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

@sakshi0210 then plz make a new issue, I wamt to know what are you working , I can't  add label here because this place foe discussion
Make a new issue

Okay 👍🏻
On Thu, Mar 24, 2022, 10:22 AM Gurjeet Singh Virdee <
@.***> wrote:

@sakshi0210 https://github.com/sakshi0210 then plz make a new issue, I
wamt to know what are you working , I can't add label here because this
place foe discussion
Make a new issue
—
Reply to this email directly, view it on GitHub
https://github.com/Rayman-Sodhi/Clone-IT/issues/5#issuecomment-1077104902,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARNOOYRN4VDT43AI3PCSZKLVBPYITANCNFSM5JKKG7KA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hey, I am interested in making a clone of a website. I was thinking about pizza fan (https://www.pizzafan.gr/en). Could u please assign me?

@GNtrazios make a new issue plz then I will assign you


Create DeleteWorkspaceAvatar

Details

Fixed Issues

$ https://github.com/Expensify/Expensify/issues/219963
Tests/ QA Steps

In Newdot, create a workspace if you don't have one and open the General settings:

If you don't have an avatar, add one:

Once the avatar is added, remove it:

Confirm it immediately goes back to the default avatar.

Add an avatar again.

Go offline

Remove the avatar. It should immediately go back to the default avatar, but it should display semitransparent:

Go back online. The default avatar should go back to fully opaque.

Add an avatar again. Do not leave the General settings.

In a different tab, open your account in OldDot and go to Policies->group and delete the workspace:

Go back to newdot and remove the avatar. After a bit, an error should show up:

Click the X. The error should go away.

[ ] Verify that no errors appear in the JS console

OFF WIP!

Comments addressed!!

OFF HOLD!

It should still be on HOLD until the Web PR is deployed, otherwise it might fail on staging and create deploy blockers.

It should still be on HOLD until the Web PR is deployed, otherwise it might fail on staging and create deploy blockers.

Really? I thought they could move in parallel! My bad!

Web PR is on Staging. This is OFF HOLD!
@MonilBhavsar, go for it!

Web PR is on Staging. This is OFF HOLD!

I appreciate the urgency, but App staging calls the production Web api, so we need to wait until it's on prod 😞

Web PR is on Staging. This is OFF HOLD!

I appreciate the urgency, but App staging calls the production Web api, so we need to wait until it's on prod 😞

Oh, damn :(

Web was just deployed to production so we can merge!

🚀 Deployed to staging by @MonilBhavsar in version: 1.1.90-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

Testing well in staging


Name parsing of generic datatypes

The function parseName of Name is currently not well suited well to parse FQNs that contain type arguments.
For example, the String "java.util.Optional<java.lang.String>" comes out as
parts = {"java", "util", "Optional<java", "lang", "String>"}
I suggest this: instead of splitting the FQN by a delimiter, it could be matched against a pattern describing the parts that should be kept together:
(\\w+(?:<[\\w\\., <>]+>)?) describes a pattern that should work well in Java, C++, and Rust, if I am not mistaken.
It would part "java.util.Optional<java.lang.String>" into the matches {"java", "util", "Optional<java.lang.String>"} and even
"java.util.Optional<java.util.Map<java.lang.String, java.lang.Integer>>" into {"java", "util", "Optional<java.util.Map<java.lang.String, java.lang.Integer>>"}.
In Go and Python, it seems that generic arguments are given in between square brackets. In Go, there seem to be other extensions to the type system that I am not really familiar with.
The language-specific characters for type arguments could be stored in the respective Language object, and the regular expression could be built dynamically from that, I suppose.
I am specifically talking about in ObjectTypes here, FunctionTypes may be a whole story altogether.

Thats a good Idea. We should indeed use a generics delimiter for type arguments that we store in the language frontend. However, we probably should separate the type name java.util.Optional and the list of generic arguments <java.lang.String>.

I would actually try to avoid going down this route... parseName is designed to parse names, not types! In case you end up with a "name" like java.util.Optional<java.lang.String> my suspicion is that at some point you were using a toString representation of a type instead of using the type directly. Native java parser types are converted using the typeOf function of the language frontend.


failed to reproduce DSNAS paper performance

Dear author,
thank you for this great work and code.
I tried to reproduce the performance of DSNAS by following the instructions on README (firstly search supernet for 80 epochs, and then continue searching with train_imagenet_child.py)
but failed to reproduce the results. As shown in the attached curve below, I got only 73.57 top-1 accuracy, which is much lower than 74.4 as the paper claims.

Very thanks for running our codes. It seems that the evaluation curve accuracy is a little bit strange. Could you send me your generated log files and script folder to my email (skhu [at] se [dot] cuhk [dot] edu [dot] hk)? I will have a check and give you the response.

Since "loss_alpha" is a loss value, it should be in [0, inf]. However, according to the current implementation it is in [-inf, 0], so the sign should be changed. Is this a simple bug, or am I misunderstanding something?
https://github.com/SNAS-Series/SNAS-Series/blob/568102e527e934d21a8b4eec7ee8b1e0f1204f52/DSNAS/network.py#L119
https://github.com/SNAS-Series/SNAS-Series/blob/568102e527e934d21a8b4eec7ee8b1e0f1204f52/DSNAS/network.py#L141

No, it's not a bug. Here we use loss_alpha to calculate the gradient of the first part in the following equation.

Thanks for your interest in our paper. We run the experiments twice in the second searching process by using the checkpoint (model.pth-80) you sent us  and get the results (74.4 and 74.2). The corresponding tensorboard log file is in https://drive.google.com/drive/folders/1DNh55QVCuOFLJNBkvXcLpxBRukCI7ujU?usp=sharing. It's possible that dropout layer makes the variance larger, as we have also found a very small variance with the absense of dropout (74.2+-0.1). Maybe you can also run the experiments several times to check the result with/without dropout layer. If you have any further questions, welcome to discuss with us.

Hello @skhu101
Hope you are doing well.
I am doing some research and have used your library (which is excellent :) ) as a baseline.
I have managed to replicate your accuracy (74.2, not 74.4). However, my training accuracy is always much higher when compared to the training accuracy curve your tensorboard has. Should I be worried? Maybe I am leaving some performance on the table. Note that in the image I have uploaded, 'ours' refers to my replication of the 'ref_sim' which is the one you have uploaded. 'ours' uses a different model with 309 MFLOPs.

I also get a lot of these errors (maybe 14 per epoch) during the training. Maybe this is causing skipping of entire mini batches?
/opt/conda/lib/python3.6/site-packages/PIL/TiffImagePlugin.py:754: UserWarning: Possibly corrupt EXIF data.  Expecting to read 2555904 bytes but only got 0. Skipping tag 0
  " Skipping tag %s" % (size, len(data), tag))

Do let me know your thoughts! Thank you!

Hope you are doing well! Such phenomenon is possible due to the effects of random seeds. I think this should not matter so much!


Terminal turns black and crashes VSCode after long tern using(reposted)

This happens to me as well. In my case, @Tyriar 's solution does not help.
The only way that I can make the windows to show up again is open a new terminal.
Version : VS Code 1.54.3
OS : Linux x64 5.4.0-65-generic

The black issue is fixed in Insiders, as for the weird viewport problem that's https://github.com/Microsoft/vscode/issues/57803
Originally posted by @Tyriar in https://github.com/microsoft/vscode/issues/65958#issuecomment-451981428

I am seeing the same issue.

Version: 1.56.2
Commit: 054a9295330880ed74ceaedda236253b4f39a335
Date: 2021-05-12T16:45:26.313Z
Electron: 12.0.4
Chrome: 89.0.4389.114
Node.js: 14.16.0
V8: <IP_ADDRESS>-electron.0
OS: Linux x64 5.4.0-66-generic

Does this only happen when a terminal is reconnected or only on the first terminal created? Does setting terminal.integrated.gpuAcceleration to false fix it?

It happens when vs code has been running for some time, not when the terminal is first created. It happens to all vscode windows at the same time. I can still type into the terminals, and the white box moves, but no text appears. I have tried to disable GPU acceleration, i will report back if it still fails.
My graphics are
00:02.0 VGA compatible controller: Intel Corporation HD Graphics 530 (rev 06)
the driver is i915 from 5.4.0-66.
The mesa version is OpenGL version string: 4.6 (Compatibility Profile) Mesa 20.2.6
X.Org version: 1.20.9
Everything is running on an up to date Ubuntu 20.04.2 LTS

@meganrogge maybe a problem with webgl context loss?

@Tyriar did you see Webgl could not be loaded. Falling back to the canvas renderer type. as a warning?

Does this only happen when a terminal is reconnected or only on the first terminal created? Does setting terminal.integrated.gpuAcceleration to false fix it?

After setting 'terminal.integrated.gpuAcceleration' as 'Off', all terminals show up again. It's like miracle. Thank you.


add check secrets and update GHA workflow files

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@gruebel It seems something in CodeQL didn't like your changes @gruebel

I think it has some trouble to figure out I moved it, but it should be fine after merging. The CodeQL scan ran as it should, just grabbing the results didn't work.


Removed 'Improved Logs' from Readme

Related to Issue #47

Thanks. Check out #45 and its associated PR as well. Performance optimizations right now are the most essential patches but also looking to reimplement logging in a more user-friendly way. So if you have any ideas related to that, let me know, thanks.


Allow IP address as node name

Fixes #309

Can one of the admins verify this patch?

We have a CCLA as whitepages.com .

Thanks for the PR! LGTM except for the one nit.

Stand by -- integration testing shows this didn't solve the issue.

Looks like there's a deeper issue for us here -- node.Status.Addresses is not getting populated. I'm tracing through why this may be.

This issue turned out to be very close to #202, but I think we've addressed the underlying problem that forced reverting that change with #210.
LegacyHostIP is documented as equivalent to PublicIP in the cloud providers. With this change, the algorithm for choosing a node's IP is as follows:
If the node has an InternalIP, use that address;
otherwise, if the node has a hostname that differs from its PublicIP, resolve the hostname and use that address;
otherwise, use the node's PublicIP (which may have been passed as LegacyHostIP.)
This version is working as expected in our cluster.

Looks like I need to add more unit tests to pass the coverage check.

make test-unit-cov is passing on my box; what am I missing here?

Unit tests are failing. Can you run make test-unit?

Coverage is not enforced as of now.

Unit tests are green on my system.
GOOS=darwin GOARCH=amd64 CGO_ENABLED=0 godep go build -a github.com/GoogleCloudPlatform/heapster
GOOS=darwin GOARCH=amd64 CGO_ENABLED=0 godep go test --test.short github.com/GoogleCloudPlatform/heapster/...
?   	github.com/GoogleCloudPlatform/heapster	[no test files]
?   	github.com/GoogleCloudPlatform/heapster/clusters/coreos	[no test files]
?   	github.com/GoogleCloudPlatform/heapster/extpoints	[no test files]
ok  	github.com/GoogleCloudPlatform/heapster/integration	0.015s
?   	github.com/GoogleCloudPlatform/heapster/manager	[no test files]
?   	github.com/GoogleCloudPlatform/heapster/sinks	[no test files]
ok  	github.com/GoogleCloudPlatform/heapster/sinks/api	0.063s
ok  	github.com/GoogleCloudPlatform/heapster/sinks/gcl	0.012s
?   	github.com/GoogleCloudPlatform/heapster/sinks/gcm	[no test files]
ok  	github.com/GoogleCloudPlatform/heapster/sinks/influxdb	0.011s
ok  	github.com/GoogleCloudPlatform/heapster/sources	0.014s
?   	github.com/GoogleCloudPlatform/heapster/sources/api	[no test files]
ok  	github.com/GoogleCloudPlatform/heapster/sources/datasource	1.321s
ok  	github.com/GoogleCloudPlatform/heapster/sources/nodes	0.014s
?   	github.com/GoogleCloudPlatform/heapster/util/gce	[no test files]
ok  	github.com/GoogleCloudPlatform/heapster/util/gcstore	0.015s
?   	github.com/GoogleCloudPlatform/heapster/validate	[no test files]
?   	github.com/GoogleCloudPlatform/heapster/version	[no test files]
201345m0:heapster jfoy$ echo $?
0

@cadvisorJenkinsBot: ok to test

The failing test appears to get non-deterministic input:
func TestFuzzInput(t *testing.T) {
    var input source_api.AggregateData
    fuzz.New().Fuzz(&input)
    timeseries, err := NewDecoder().Timeseries(input)
    assert.NoError(t, err)
    assert.NotEmpty(t, timeseries)
}

Is there a way to get the tests to show a seed, to reproduce a particular fuzzed input?

The failed integration builds appear to be an infrastructure error:
 failed to load docker image "heapster:e2e_test" using temp file "/tmp/kubernetes-minion-71yx171296728" on host "kubernetes-minion-71yx" ("exit status 1") -
"Warning: Permanently added '<IP_ADDRESS>' (ECDSA) to the list of known hosts.
time=\"2015-06-04T19:01:01Z\"
level=\"fatal\"
msg=\"Error: Untar fork/exec /usr/bin/docker: argument list too long\"
ERROR: (gcloud.compute.ssh) [/usr/bin/ssh] exited with return code [1]."

I am working on fixing the integration framework. The fuzzed test is an
issue. I will fix that test via a separate PR.
On Thu, Jun 4, 2015 at 12:09 PM, Jack Foy<EMAIL_ADDRESS>wrote:

The failed integration builds appear to be an infrastructure error:
failed to load docker image "heapster:e2e_test" using temp file "/tmp/kubernetes-minion-71yx171296728" on host "kubernetes-minion-71yx" ("exit status 1") -
"Warning: Permanently added '<IP_ADDRESS>' (ECDSA) to the list of known hosts.
time="2015-06-04T19:01:01Z"
level="fatal"
msg="Error: Untar fork/exec /usr/bin/docker: argument list too long"
ERROR: (gcloud.compute.ssh) [/usr/bin/ssh] exited with return code [1]."
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/heapster/pull/310#issuecomment-109014540
.
--
You received this message because you are subscribed to the Google Groups
"cadvisor jenkins bot" group.
To unsubscribe from this group and stop receiving emails from it, send an
email to<EMAIL_ADDRESS>To post to this group, send email to<EMAIL_ADDRESS>To view this discussion on the web visit
https://groups.google.com/a/google.com/d/msgid/cadvisor-jenkins-bot/GoogleCloudPlatform/heapster/pull/310/c109014540%40github.com
https://groups.google.com/a/google.com/d/msgid/cadvisor-jenkins-bot/GoogleCloudPlatform/heapster/pull/310/c109014540%40github.com?utm_medium=email&utm_source=footer
.

LGTM.

Thanks!


[Feature Request] Add support for openapi explorer documentation viewer

Description of the feature
Similar to redoc, add support for https://github.com/Rhosys/openapi-explorer.
Code example (if possible)
Couldn't find a standalone JS file. Might have to build it.

Found a google vanilla JS code example: https://github.com/Rhosys/openapi-explorer/blob/release/0.10/docs/examples.md#vanilla-javascripthtml


fix: make run-kube-local failed

Fixes: #387
Signed-off-by: Xunzhuo<EMAIL_ADDRESS>

/lgtm. I'll let @arkodg tag since he did the first review.


Apollo Store lock contention

Use case
Hello,
Did some systracing. Launched 50 duplicate API calls, result shows there are around 60 RxCachedThreadS and 60 DefaultDispatch threads all contending for monitor lock.
This is only theoretical tests. However, our app at peak (home feed) can have 200+ threads running. Even if there will be only dozen of remote API calls (not counting other Apollo store cache reads), if we migrate all APIs to GraphQL, this could present a performance penalty.
Also in RXJava world, it is not uncommon for dev to call an API to retrieve a list of items, and use flatmap to make a seperate API call for each item in the list, so 50 API calls is not so uncommon and it gets worse as user scroll quickly on a list.
Describe the solution you'd like
Question
Is it possible to optimize the locking a bit more? Possibly even sacrificing consistency for performance (or having it as a configurable option)?
MemoryCache has a monitor lock: CacheLock. (This theoretically should be very fast since synchronized block guards just a few LinkedHashMap calls)
DefaultApolloStore has a ReentrantReadWriteLock. (A single writer will block all readers, a single reader will block all writers)

Java/Kotlin method tracing
DefaultDispatch: It seems the ReentrantReadWriteLock does a couple of things such as Invoking JsonWriter, not just directly save data to a LinkedHashMap.

RxCachedThreadS: It's harder to understand the Flame Chart and why the thread exist.

systrace

Thank you for providing this, this is interesting data, and will dig into it.
Just a quick reaction to

it is not uncommon for dev to call an API to retrieve a list of items, and use flatmap to make a seperate API call for each item in the list

Not sure if you are using GraphQL already (if yes you can disregard my comment), but one of its advantage when compared to REST is the ability to avoid this, by having the fields you need at your disposal to be used in one query. Of course the ability to do it will depend on the server's schema, but in many cases you should be able to avoid some round trips.

Not sure if you are using GraphQL already
Yes, to provide more context, REST and GraphQL will co-exist for quite a while for us. Migration will take a lot of time.

I wasn't sure if it is possible to build a LRU cache that's more performant than a synchronized LinkedHashMap. I Looks like Guava cache is potentially more performant (hard to tell since Guava cache maybe better used for server than low power mobile devices.

The Least Recently Used page replacement algorithm was chosen due to its simplicity.
The basic strategy is to subdivide the table among Segments, each of which itself is a

concurrently readable hash table. The map supports non-blocking reads and concurrent writes
across different segments.

A few additional notes:

About your last comment about LRU, this is interesting. My hunch is that optimizing the LRU cache itself won't have a big impact (I could be wrong!) but that the locking situation showed with your traces could be improved
The need for the synchronized block in MemoryCache.loadRecord() is that, although it is already called under a read lock (from DefaultApolloStore.readOperation), the code actually alters the inner state, and we need to make sure that only one thread is doing that at a time. As you noted above, updating the LRU cache state itself should be a very fast operation however.
But inside the same synchronized block the code also reads the data of the next cache. This is usually the disk cache and does IO, so not a fast operation. Locking this part is also unnecessary since we're already under the read lock. We could improve this by reading the disk cache outside of the synchronized block, and only do the memory changes under  synchronized block. This would allow multiple threads to read from the disk cache at the same time, as intended, probably a good improvement to make.

But inside the same synchronized block the code also reads the data of the next cache. This is usually the disk cache and does IO, so not a fast operation.
Great idea! What's the class name of the disk cache? If you are referring to http cache, we don't use it.

No sorry I was referring to the SQLite cache (SqlNormalizedCache). Maybe you don't need it either actually - it's only useful if you need a persistent cache in your app of course.

We don’t use sql cache at least for now (may do in future to persist user data such as making a post and save). So that lock contention is caused by something else.
Sent from my iPhone
On Jan 5, 2023, at 1:55 PM, Benoit Lubek @.***> wrote:

No sorry I was referring to the SQLite cachehttps://www.apollographql.com/docs/kotlin/caching/normalized-cache/#sqlite-cache (SqlNormalizedCachehttps://github.com/apollographql/apollo-kotlin/blob/05f2a3295517fb7a64c2259c38b453a37f3c4d19/libraries/apollo-normalized-cache-sqlite/src/commonMain/kotlin/com/apollographql/apollo3/cache/normalized/sql/SqlNormalizedCache.kt#L13). Maybe you don't need it either actually - it's only useful if you need a persistent cache in your app of course.
—
Reply to this email directly, view it on GitHubhttps://github.com/apollographql/apollo-kotlin/issues/4605#issuecomment-1372608556, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AANZ66UYE7AV6FABZYWTS2DWQ4KK7ANCNFSM6AAAAAATRLN4LU.
You are receiving this because you authored the thread.Message ID: @.***>

Thanks for clarifying. It's difficult to pinpoint the source of the contention. FWIW, what I observe in a similar test - albeit without Rx - looks a bit different.
I'm wondering if it would make sense to get rid of a high level lock on ApolloStore and let the NormalizedCaches handle their own locking instead.
Overall, we think the normalized cache could benefit from a revamp, including performance improvements (in this area but also in terms of storage size, access time, etc.), as well as new features (see #2331) - but this will be a major task.

I'm wondering if it would make sense to get rid of a high level lock on ApolloStore and let the NormalizedCaches handle their own locking instead.
I am not familiar with internals of everything the lock currently guards.

Regarding the our own repository layer implementation for comparison with Apollo Store, it does not hold a lock under

It basically concats 3 streams of data and optionally take the first that comes back. The result observable can be subscribed on IO thread. We basically do not hold any lock anywhere. Android LRU cache implementation is thread safe.
A: in memory data source with Android LRU cache implementation.
B: DB data source (very rare)
C: remote REST data source.

Hey, just sharing some ideas I found for improving performance of readers/writers
JakeW's DiskLruCache used by Okhttp and Bumptech/Glide image loading.

Multiple readers can read the cache even if there is a writer that is currently writing. Writer write to a temporary file, and swap with the main file when it's done. It sacrifice consistency for performance.
Does not allow multiple writers to the same file.

Running into performance issues with this with large queries.
Found that there was some extra logic inside of the locks that could be moved out: https://github.com/apollographql/apollo-kotlin/pull/5101
This was minor performance boost when testing these modifications on our data.
It appears on a query cache read I am trying to boost performance, a lot of the time is spent in JSON serialization logic trying to build out the records.

Thank you so much @ashare80 💙 . The investigation and PR are awesome! Is there any chance you could contribute your data? We could include that in benchmarks and that'd be a good data point for future optimisation. No worries at all if not, we can always forge tests but real life data always has more flavour :)

@martinbonnin We might be able to work something out for that. What would be the best way to share this? Would a sample response be sufficient?

Nice! If you can share the schema + query too, it'd be ideal. If not we can craft something out but it's a bit painful + might not be able to infer the polymorphism if any.


Zip file enhancements

Resolves issue #350
Added ZIP file support to Add Data > File > Browse... > Select a *.zip file.
Utility will unzip the file and allow User to import one or more of the files inside.
Testing steps:

Put a few CSV and KML files into a ZIP file.
Click Add Data > Open File/URL > Browse > Select the ZIP file
Click Next when contents are unzipped
Deselect/Select one or more files and click Done
Complete the regular Import Data process for each of the files selected

Surge -- http://puzzled-record.surge.sh/opensphere/

At this moment I'm not sure how it was working before -- but somehow the Wizard blocked the deferred chain from completing until each chained Importer finished.
Now that I'm just extending from FileImportUI, all of the sub-imports are popping open at once.  I'll figure out what's going on when I get back on Monday.  Sorry for the delay.

I added an additional Event listener to the FileImport callback and updated all of the imports to configure it on the config object.  Surge is up to date.


Ncs v2.2 rebase

Regular rebase as release prep

The following west manifest projects have been modified in this Pull Request:

Name
Old Revision
New Revision
Diff

hal_nordic
https://github.com/zephyrproject-rtos/hal_nordic/commit/4ee20fd6ca25cd4bf78918f4fffee016a4ef47b9
https://github.com/zephyrproject-rtos/hal_nordic/commit/fe0be3c3546e136a67264084c0e3edfd7354645a (master)
<EMAIL_ADDRESS>

Note: This message is automatically posted and updated by the Manifest GitHub Action.


Add min_by enumerable

This pull request resolves the issue that I raised in #84
@josh-works suggested that I make this pull request.

Closing this PR. See the thread in issue #84 for more details.


Use @DIR instead of Pkg.dir

this allows installing and loading the package from elsewhere

Coverage remained the same at 90.626% when pulling 59e26311e09897e0831d8f7cd091bb28c486a944 on tkelman:nopkgdir into 3cc43fafd2958668a2e80b372167a0c85de2f393 on JuliaFEM:master.

Thank you


Warning server path handling problem using Nextcloud

Calling client.principal().calendars() using Nextcloud as the caldav server results in the following warning:
Possibly the server has a path handling problem, possibly the URL configured is wrong.
Path expected: //remote.php/dav, path found: ['/remote.php/dav/'] Please raise an issue at https://github.com/python-caldav/caldav/issues or reach out to<EMAIL_ADDRESS>include this error and the traceback and tell what server you are using.
Continuing, probably everything will be fine

However, a correct list of the user's calendars is still returned.
So far I have only noticed this with client.principal().calendars(). When deleting or creating events, via client.calendar(...).add_event(...), this warning does not appear.
I am currently using the latest Nextcloud Docker image (Nextcloud Hub 5 (27.0.2)) with a SQLite database.

It seems harmless, but I ought to look into it at some point.


API: update individual

when someone edits their profile, needs to keep same donorfy user when someone updates their email address.
Also when we upsert users we should check if their name is null and set it if it is.

this is all good


select_source does not handle playlists that have not yet been synced to the device

Currently the list of playlists (galleries) that can be selected as an input source are created by combining all available device galleries with all available user galleries. It is possible for a user gallery to exist, but not be synced to the device yet. select_source will attempt to switch to this unsynced gallery using a local call and fail. We should instead detect if a gallery has not been synced to the device yet, and then use a Meural API call to upload it to the device.

Changed approach. Now we only list all local device galleries available. To upload a non-synced user gallery to your device, use the Meural app instead.
See: https://github.com/GuySie/ha-meural/commit/766da1137717891935e54b65a5b7e3ecfd5f780e


Trying to update an already up-to-date cluster breaks it

I tried running sudo flynn-host update on a freshly updated cluster, and I got the following error:
INFO[05-19|21:21:19] pulling binaries and config              host=ip172311695
INFO[05-19|21:21:19] pulling binaries and config              host=ip172318138
INFO[05-19|21:21:22] binaries and config pulled successfully  host=ip1723131163
INFO[05-19|21:21:23] binaries and config pulled successfully  host=ip172312166
INFO[05-19|21:21:23] binaries and config pulled successfully  host=ip172311695
INFO[05-19|21:21:23] binaries and config pulled successfully  host=ip17231228
INFO[05-19|21:21:24] binaries and config pulled successfully  host=ip172318138
INFO[05-19|21:21:24] validating binaries
INFO[05-19|21:21:24] updating flynn-host daemon on all hosts
INFO[05-19|21:21:24] flynn-host updated successfully          host=ip172318138
INFO[05-19|21:21:24] flynn-host updated successfully          host=ip1723131163
INFO[05-19|21:21:24] flynn-host updated successfully          host=ip17231228
INFO[05-19|21:21:24] flynn-host updated successfully          host=ip172312166
INFO[05-19|21:21:24] flynn-host updated successfully          host=ip172311695
EROR[05-19|21:21:25] error getting cluster status             err="Get http://status-web.discoverd: dial tcp: lookup status-web.discoverd: no such host"
21:21:26.423601 host.go:138: exec: job exited with status 1

The cluster is now completely down and unresponsive. I collected the logs here.
(This is the 3rd time it happens to me this week, with different versions of Flynn, so I don't think it's an isolated problem. On the bright side, the in-place update has started to work for me this week as well).

I had same issue. how can I fix it? doing a backup/restore failed on 05:04:17.687000 status-check status error: bootstrap: timed out waiting for http://status-web.discoverd, last response Get http://status-web.discoverd: dial tcp: lookup status-web.discoverd on <IP_ADDRESS>:53: no such host
I have same issue as #2802 after trying to restore the backup.

FYI I believe this was fixed during the migration to libcontainer (#3030, specifically this change). Previously every update would "resurrect" another discoverd job even though one was already running, causing issues.
If you continue to have the discoverd issue with in-place updates on releases since v20160711.0 then let us know.


Fail to decode Caesar Cipher

Problem:
I try one of the easiest encryption methods which is the Caesar Cipher technique. However, after 15 guesses the program could not decrypt the message.
For example:
Expected: My name is Henry
$ ciphey -t 'Kw}l_kc}gq}Fclpw]='
Possible plaintext: 'Co}d cu}yi}Xudho]=' (y/N): n
Possible plaintext: '=]ohduX}iy}uc d}oC' (y/N): n
Possible plaintext: 'Es}d em}iy}Jmdzs]=' (y/N): n
Possible plaintext: '=]hosbY}nx}bt s}hT' (y/N): n
Possible plaintext: 'Se}t sk}oy}Nktxe]=' (y/N): n
Possible plaintext: 'Am}b as}wg}Vsbfm]=' (y/N): n
Possible plaintext: '=]ovziF}ue}ia z}oA' (y/N): n
Possible plaintext: '=]ubfoL}ak}og f}uG' (y/N): n
Possible plaintext: '=]cjnwT}is}wo n}cO' (y/N): n
Possible plaintext: 'Ui}t uc}yo}Zctpi]=' (y/N): n
Possible plaintext: '=]rkgxA}lb}xf g}rF' (y/N): n
Possible plaintext: '=]iptcZ}oy}cu t}iU' (y/N): n
Possible plaintext: 'Ma}l mu}qg}Rulha]=' (y/N): n
Possible plaintext: '=]ahluR}gq}um l}aM' (y/N): n
🌓  Thinking...

Hello. This isn't a bug. Your plaintext is My}n_me}is}Henry]= which has symbols between the letters and Ciphey's checkers can't detect it.
You can run ciphey -t "Ftue ue mz qjmybxq ar ftq Omqemd oubtqd" as an example instead.


Fixed #8809 -- ExcelDateFormatter generates invalid format strings

The format strings generated previously would not work correctly in macOs Preview and Numbers (see also #8809).
I've aligned the format string conversion using the documentation found here:
https://docs.djangoproject.com/en/stable/ref/templates/builtins/#date
https://support.microsoft.com/en-us/office/format-numbers-as-dates-or-times-418bd3fe-0577-47c8-8caa-b4d30c528309#bm2
Dates are now formatted correctly in Preview and Numbers (and remain correct in other spreadsheet software):
Preview:

Numbers:

Google Sheets:

LibreOffice Calc:

The test failure (test-postgres (3.10 Django main)) is unrelated to this PR.

Merged in 12ba7d8161a0055211541402f759f92acfbd0ba3

Thanks for fixing this @jaap3

No problem @lb-, would you mind having a look at my other PRs? :grin:

@jaap3  I know you have a few up - I will have a go when I get some time.


Simplify Component Usage with Tailwind/Bootstrap Integration

Description: To speed up development and ensure consistent styling, we can integrate Tailwind CSS or Bootstrap. This will allow the use of pre-built, responsive components and utility classes, reducing the need for manual CSS.
Benefits:
Faster development with pre-built components.
Consistent design across the project.
Easy customization with utility classes.

Assign it to me, I will change the manual CSS to tailwind CSS @divyanshudhruv

@Riddhi050803 we will change it later. Currently we are focusing on the basic UI or the portfolio. But thank you for the idea :)


Error: Only named exports may use 'export type'.

Hi,
I just upgraded to the newest version 0.3.21 and got the following error when trying to build my project:
> tsc -p tsconfig.package.json

../../node_modules/@jridgewell/trace-mapping/dist/types/trace-mapping.d.ts:4:1 - error TS1383: Only named exports may use 'export type'.

4 export type * from './types';
  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Found 1 error in ../../node_modules/@jridgewell/trace-mapping/dist/types/trace-mapping.d.ts:4

I think it´s related to this commit https://github.com/jridgewell/trace-mapping/commit/d04e7fe0bda0952e1542dd89a6ae3ff85967346a
After downgrading back to 0.3.20 the error is gone. Any ideas?

You're probably using an older TS version? I think TS v5 added export type * syntax.

As @lucb31 comments, the error is gone when downgrading to 0.3.20.

@jridgewell thanks for the fix. Works again with the most recent version. Makes sense to not rely on TS 5 syntax here IMO since quite a few other packages depend on this and none explicitly state that they require TS v5.
Appreciate your effort!


EnergyBarUI NonAnimatedHealthBar

Add healthbar images to HUD
Remove progress bar indicating remaining health
Add scalable health bar to HUD to replace progress bar
Bind the visibility of each segment to the player's energy level
Bind the health bar image to the player's energy level

Closing since I re-implemented on top of main on a different branch


Feature Request (Marathon): Support refreshing DC/OS authentication token

Do you want to request a feature or report a bug?
Feature
Problem
Currently, traefik supports supplying a static DC/OS auth token via dcosToken in the configuration, which is supported by go-marathon. The problem is that the tokens generated by DC/OS eventually expire (I believe 5 days).
It's possible to work around this by creating tokens with long expiry times, but that's more of a band aid (and they still eventually expire). It would be nice if traefik could support automatically refreshing the authentication token.
Details
Issue 292 was logged in go-marathon for this functionality. There is a proposal about making use of the http transport from dcos-go to create the HTTPClient that is passed to go-marathon, documented in my comment here, but there are also some caveats that are discussed in that comment thread.
The discussion made it sound like the best place to use dcos-go for this functionality might not be in the go-marathon library itself, but in the consuming projects. Mainly, this is because dcos-go depends on the jwt-go library (currently 2.6.0). The consumer of go-marathon (in this case, traefik) would then be responsible for making the choice of using the dcos-go http transport and vendoring the correct dependencies to make it work.
I wanted to open the discussion here to see if this would be a desired feature in traefik, and if it makes sense to implement that functionality in traefik instead of go-marathon.
I do have a working example of what utilizing the dcos-go http transport in traefik would look like.

For obvious reasons, I endorse the client-based approach. :-)

Hello,
In the upcoming major version of Traefik (v3), support for the Marathon provider is being dropped.
Due to this change, we won't be able to accept or work on new features related to the Marathon provider.
For this reason, we are closing this issue.


Zulip Components

Can you describe what each component that compromises the Zulip stack is used for? I would like to better understand how each piece works together. From what I can see, Zulip uses these technologies:

postgres
redis
rabbitmq
nginx - serves python (the web frontend)
erlang

I've been meaning to write a more detailed doc on the architecture but here's the short version:

nginx serves static assets and proxies to django + tornado
django is the main web application server
tornado runs the server -> client real-time push system
postgres is the database and stores basically all the data
memcached is used for caching of database model objects
redis is used for a few very short-term data stores; e.g. for rate limiting.
rabbitmq (which is implemented in erlang) is used for queuing expensive work (e.g. sending emails triggered by a message, push notifications, some analytics, etc.) that require reliable delivery but you don't want to do on the main thread.  It's also used for communication between the application server and the tornado push system.

Feedback on what you'd like to learn more about is very welcome since I'm likely to expand these notes into the architecture docs.

supervisor to manage all programs(listing down all management commands which we run via supervisor and their roles).
webpack(node)/django-pipeline setup for frontend.

webpack actually currently doesn't do anything useful, but yeah, it's there :)

Thanks, this helps to clarify things! Two things that interest me are:

what I need to do to make complete backups that I can restore later (e.g. if the server crashes). If I were to just use pg_dump to dump the zulip database, is that all I would need to restore to fully recover my zulip install on a new host?
if I want to make changes/enhancements to the web interface or backend, which services do I need to restart to apply the changes (e.g. scripts/restart-server)? I read through the New Feature Tutorial, but it was unclear exactly what needs to be restarted/reloaded to read in the modified files

@asmartin there are docs on backups in README.prod.md -- check them out.
scripts/restart-server is for restarting everything in the production environment; with the development server, it will auto-restart automatically whenever you change python code (and reloading the browser will get the latest static assets).

Thanks for the clarification, that is cool that it auto-restarts when changes are detected!

I'm working on this - @timabbott please assign it to me, as I can't seem to assign it to myself. Thanks.

GitHub only allows project "members" who have write access to the repo to assign projects (or label issues, etc.).  While I could add you, really I wish GitHub let me allow anyway to assign projects to themselves if they like.

@timabbott per #16 - is it fair to say that memcached is used for legacy reasons and we're interested in replacing it with Redis throughout?

There are some technical challenges involved with finishing #16, so I think I'd just say we use memcached for caching.  We can update the docs later if we're able to make redis provide the same functionality in a way we're happy with.


added more competitive codes

please accept the pull request, i have added more codes about 20

Hii, @warriorwizard  please check for the conflict.


[BUG] Mujoco: WARNING: Unknown warning type Time = 0.4000.

Describe the bug
When running mujoco env, with some configuration it raises this warning.
To Reproduce
On my laptop
$ cd benchmark
$ python3 test_envpool.py --env mujoco --num-envs 17 --batch-size 17
Namespace(batch_size=17, env='mujoco', num_envs=17, num_threads=0, seed=0, thread_affinity_offset=0,
total_step=50000)
  0%|                                                                         | 0/50000 [00:00<?, ?it/s]
WARNING: Unknown warning type Time = 5.0500.

  0%|▍                                                            | 102/50000 [00:00<00:49, 1011.42it/s]
WARNING: Unknown warning type Time = 5.0500.

WARNING: Unknown warning type Time = 5.0500.

WARNING: Unknown warning type Time = 5.0500.

WARNING: Unknown warning type Time = 5.0500.

Expected behavior
No warning
System info
My laptop. But this phenomenon also exists in other hardware, e.g., workstation: 50 wait 20.
Additional context
Related issue: https://github.com/openai/mujoco-py/issues/340
This error only occurs when batch_size > 16
Manually checked with NaN and inf but everything looks good.
The sleep solution to some extent helps, but it's not a good solution, i.e., adding print will reduce the possibility of this warning's occurance.
Reason and Possible fixes
Wait until deepmind release the mujoco source code (hopefully in mid-May). Add traceback and find the root cause.
This error generates from libmujoco.so.2.1.5
A pure C++ working example to reproduce this bug would be very helpful.
Checklist

[x] I have checked that there is no similar issue in the repo (required)
[x] I have read the documentation (required)
[x] I have provided a minimal working example to reproduce the bug (required)

Out of curiosity, have you tried reducing the physics timestep?

I set the number of substep to 1 but it can still produce this wraning.


[BUG] Scout soft deletes requires all models to support softdeletes trait.

Describe the bug
If Scout is configured to do soft deletes then currently all models are required to support soft deletes as DefaultImportSource::newQuery() doesn't check for existence of withTrashed() when soft deletes are turned on. Is it possible to have the code updated so that soft deletes on a model are optional when soft deletes are turned on in Scout.
To Reproduce
Steps to reproduce the behavior:

Create a model with softdelete trait and searchable trait.
Create a model with a searchable trait but no softdelete trait.
Set config.scout.soft_delete to true
Run php artisan scout:import
See error

Expected behavior
I would expect the import to work with no errors.
I have for the time being created a custom ImportSourceFactory that uses a custom copy of DefaultImportSource, but this is likely to be a temporary solution and there may be other instances in the code where withTrashed() is assumed to exist.

What version do you use?
There is a fix in 3.0.1 https://github.com/matchish/laravel-scout-elasticsearch/pull/89
Don't it fix the issue?

Apologies I didn't see that PR, should have checked those. The managed cloud service I am using only supports ES 6, so I'm having to use 2.1 for the moment :( But I will see if I can incorporate that change from the PR). Thanks for getting back to me.

No worries) Feel free to send a PR to a 2.x branch if you fix the issue using changes from the PR.

Thank you I will do that.


Consider React-Redux-Form?

It seems like many of the concerns addressed in Shasta forms are accomplished in react-redux-form, and gives much more control to the developer as far as how they structure their forks, however complex. Would you be willing to incorporate that in your project? I'm willing to collaborate to make sure that ideologies with Shasta and RRF are in alignment.

definitely open to it, will review and see where we can collaborate and where we're duplicating efforts


[FEATURE] Combine it with slash commands action for further fine-tuning

Provide some commands that can be executed through slash-commands to provide more functionality like /assign, /review, /close, etc...

Added slash commands. Now we only need to extend them.


Run.set_folder_details appears to add additional path depth

When executing:
with simvue.Run() as run:
    run.init()
    run.set_folder_details(
        path="/simvue_unit_test_folder/simvue_unit_test_folder",
        tags=["test_tag"]
    )

A new folder appears to have been created which has the address:
/simvue_unit_test_folder/simvue_unit_test_folder/simvue_unit_test_folder
Suggesting that this is creating a new folder within the target folder.

Cannot seem to reproduce this, also related to #480


New type for RM4 Pro

I tried to use the discover node and the debug messaged showed up with:
typeid: "61a2"
type: "Broadlink: Unconfigured Device Type: Log an issue with the device details and typeid returned as this may be a new device type"
Learn/Send command doesn't do anything when i tried to use it.

Updated in v2.1.1 to include the 61a2 model.


Numary UI

HI there,
I tried opening the Dashboard via the SSH  link i have, but it just came up with the following error:
2021/07/22 12:57:08 exec: "xdg-open": executable file not found in $PATH
I have already changed the HTML bind address to the external IP.
Is there some URL I can visit to open the Dashboard or does it have to be started manually?
thanks,
Philip

Hey @thephilluk,
This was an issue with xdg-open sometimes not existing on ubuntu-server, this gets fixed with Alpha 24 - will just cost you another curl :)
Thanks for the heads up!
Cheers,

Hi,
thats great, but it seems to just bind to localhost, not the address in the config.
numary ui
Numary control is live on http://localhost:3078

tried connecting with the external ip, but i get an  empty response.
thanks,
Philip

Ha that' right, the address in config is actually used for the ledger http api and not the UI. Adding a new parameter to achieve this (see #24) - should be ready in the afternoon.
Cheers!

Alright this is now available in Alpha 25 @thephilluk
With env
NUMARY_UI_HTTP_BIND_ADDRESS=localhost:3070 \
numary ui

With config file
ui:
  http:
    bind_address: localhost:3070

Thanks for bringing that up!

Thanks for the fix.
I was following the README and ran into this (if the server is running the port is already in use, if it’s not the UI start but has nothing to connect to).
Maybe it would be helpful to have a different default port in the UI? Or document it in the README? IMO It would make for a better first-time experience if you could follow the Getting Started guide and have both things running immediately with no friction.


Value error fixed in keras-io/guides/keras_nlp /transformer_pretraining.py documentation

This code fixes the small bug in the transformer_pretraining.py documentation which leads to Value error,It has been tested
pooled_tokens = keras.layers.GlobalAveragePooling1D()(encoded_tokens[0]) encoded_tokens[0] is giving the following error
line 437
ValueError`                                Traceback (most recent call last)
[<ipython-input-23-599e891a0cf7>](https://localhost:8080/#) in <cell line: 8>()
      6 # Encode and pool the tokens.
      7 encoded_tokens = encoder_model(inputs)
----> 8 pooled_tokens = keras.layers.GlobalAveragePooling1D()(encoded_tokens[0])
      9
     10 # Predict an output label.

1 frames
[/usr/local/lib/python3.10/dist-packages/keras/src/engine/input_spec.py](https://localhost:8080/#) in assert_input_compatibility(input_spec, inputs, layer_name)
    233             ndim = shape.rank
    234             if ndim != spec.ndim:
--> 235                 raise ValueError(
    236                     f'Input {input_index} of layer "{layer_name}" '
    237                     "is incompatible with the layer: "

ValueError: Input 0 of layer "global_average_pooling1d" is incompatible with the layer: expected ndim=3, found ndim=2. Full shape received: (128, 256)

pooled_tokens = keras.layers.GlobalAveragePooling1D()(encoded_tokens) fixes the code

Could you please attach the working Gist with all the outputs, Thanks!

Could you please attach the working Gist with all the outputs, Thanks!

I executed the code in Keras NLP Guide, here is the collab (contain cell with error as well corrected ones).
# Reload the encoder model from disk so we can restart fine-tuning from scratch.
encoder_model = keras.models.load_model("encoder_model.keras", compile=False)

# Take as input the tokenized input.
inputs = keras.Input(shape=(SEQ_LENGTH,), dtype=tf.int32)

# Encode and pool the tokens.
encoded_tokens = encoder_model(inputs)
pooled_tokens = keras.layers.GlobalAveragePooling1D()(encoded_tokens[0])

# Predict an output label.
outputs = keras.layers.Dense(1, activation="sigmoid")(pooled_tokens)

# Define and compile our fine-tuning model.
finetuning_model = keras.Model(inputs, outputs)
finetuning_model.compile(
    loss="binary_crossentropy",
    optimizer=keras.optimizers.AdamW(FINETUNING_LEARNING_RATE),
    metrics=["accuracy"],
)

# Finetune the model for the SST-2 task.
finetuning_model.fit(
    finetune_ds,
    validation_data=finetune_val_ds,
    epochs=FINETUNING_EPOCHS,
)

OUTPUT
ValueError                                Traceback (most recent call last)
[<ipython-input-10-b8248bf66c90>](https://localhost:8080/#) in <cell line: 9>()
      7 # Encode and pool the tokens.
      8 encoded_tokens = encoder_model(inputs)
----> 9 pooled_tokens = keras.layers.GlobalAveragePooling1D()(encoded_tokens[0])
     10
     11 # Predict an output label.

1 frames
[/usr/local/lib/python3.10/dist-packages/keras/src/engine/input_spec.py](https://localhost:8080/#) in assert_input_compatibility(input_spec, inputs, layer_name)
    233             ndim = shape.rank
    234             if ndim != spec.ndim:
--> 235                 raise ValueError(
    236                     f'Input {input_index} of layer "{layer_name}" '
    237                     "is incompatible with the layer: "

ValueError: Input 0 of layer "global_average_pooling1d" is incompatible with the layer: expected ndim=3, found ndim=2. Full shape received: (128, 256)

CORRECTED
# Reload the encoder model from disk so we can restart fine-tuning from scratch.
encoder_model = keras.models.load_model("encoder_model.keras", compile=False)

# Take as input the tokenized input.
inputs = keras.Input(shape=(SEQ_LENGTH,), dtype=tf.int32)

# Encode and pool the tokens.
encoded_tokens = encoder_model(inputs)
pooled_tokens = keras.layers.GlobalAveragePooling1D()(encoded_tokens)

# Predict an output label.
outputs = keras.layers.Dense(1, activation="sigmoid")(pooled_tokens)

# Define and compile our fine-tuning model.
finetuning_model = keras.Model(inputs, outputs)
finetuning_model.compile(
    loss="binary_crossentropy",
    optimizer=keras.optimizers.AdamW(FINETUNING_LEARNING_RATE),
    metrics=["accuracy"],
)

# Finetune the model for the SST-2 task.
finetuning_model.fit(
    finetune_ds,
    validation_data=finetune_val_ds,
    epochs=FINETUNING_EPOCHS,
)

OUTPUT
Epoch 1/3
2105/2105 [==============================] - 110s 46ms/step - loss: 0.3999 - accuracy: 0.8145 - val_loss: 0.4125 - val_accuracy: 0.8326
Epoch 2/3
2105/2105 [==============================] - 81s 38ms/step - loss: 0.2566 - accuracy: 0.8950 - val_loss: 0.4090 - val_accuracy: 0.8429
Epoch 3/3
2105/2105 [==============================] - 80s 38ms/step - loss: 0.1907 - accuracy: 0.9253 - val_loss: 0.4345 - val_accuracy: 0.8291
<keras.src.callbacks.History at 0x799fd0b56650>

I cannot reproduce the issue with the latest version of keras-nlp (0.7.0). What version are you using?

versions
keras_nlp:0.7.0
tensorflow:2.15.0
@fchollet  I have uploaded the whole [collab] (https://colab.research.google.com/drive/1NGvp8IWv9_S1QrGUdJWivFhsknN5zTz4?usp=sharing) with error and correction

I think you seem to be using Keras 2. Try to upgrade to Keras 3. Run pip install keras --upgrade after you've installed KerasNLP and TF.


How to insert a page break ?

I know QXlsx doesn't support printing.But how about inserting a page break?
I couldn't find any document about it.Please tell me if there is the feature,thank you.

Dear @andthen255
Thanks for reporting.

I will register this issue to an enhancement.
and an example of printing on a paper


Allow for flags

Providing a means to pass flags would be useful. I'd like

-v/--verbose: display successful renames, failed renames, and how long the process took
-h/--help: a small set of instructions

Current implementation plan

Options for parsing flags include getopt or argp. Argp seems to be more powerful but getopt is standard and portable.

Initialized implementation in fa3382b7694284df453a4806a183c4477674f6b1

getopt_long() is not posix-compliant
the end of the long option struct requires {NULL, 0, NULL, 0} or {0, 0, 0, 0} to be initialized correctly. Without it, segfaults can occur when an unexpected long option is given

I prefer the idea of remaining posix-compliant, even though the rest of the program isn't right now


small fix for binned distribution

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Job PR-352/1 is complete.
Docs are uploaded to http://gluon-ts-staging.s3-accelerate.dualstack.amazonaws.com/PR-352/1/index.html

Codecov Report

Merging #352 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #352      +/-   ##
==========================================
+ Coverage   80.16%   80.16%   +<.01%
==========================================
  Files         146      146
  Lines        8324     8325       +1
==========================================
+ Hits         6673     6674       +1
  Misses       1651     1651

Impacted Files
Coverage Δ

src/gluonts/distribution/binned.py
96.22% <100%> (+0.03%)
:arrow_up:

Job PR-352/2 is complete.
Docs are uploaded to http://gluon-ts-staging.s3-accelerate.dualstack.amazonaws.com/PR-352/2/index.html


Test bot on baobab

related issues: https://github.com/Bisonai/orakl/issues/220 https://github.com/Bisonai/orakl/issues/200

let's keep just 1 issue for test bots and operator


[revolution-theme] latest videos page

Issues related to the theme will go his repository
https://github.com/you-space/revolution/issues


Send Buttons is not working in whatsapp beta

send buttons not working in whatsapp beta version and working  in stable version. please check and fix it.

@Zinguchi , first: fill all the form when you are opening an issue
second: there are no support by WhatsApp to use buttons from Beta Version


basic babel plugin

Started on a basic babel plugin. It looks like it works but I still need to add automated testing and some more complex test cases.

Fixes #536 when complete.

This works fine with simple setups and inline maps. It transpiles to JS then hooks into the default babel parse. In theory it could better hook into the TypeScript babel plugins but I have no idea how they work despite reading through the code.
https://github.com/babel/babel/blob/5f74b510ff251af5bf7fbbf25c1c934ffcb6df52/packages/babel-plugin-syntax-typescript/src/index.ts
https://github.com/babel/babel/blob/5f74b510ff251af5bf7fbbf25c1c934ffcb6df52/packages/babel-plugin-transform-typescript/src/index.ts
https://github.com/babel/babel/blob/5f74b510ff251af5bf7fbbf25c1c934ffcb6df52/packages/babel-preset-typescript/src/index.ts
https://github.com/babel/babel/blob/5f74b510ff251af5bf7fbbf25c1c934ffcb6df52/packages/babel-plugin-syntax-jsx/src/index.ts
Still need to add JSX support.

I wonder if you could do this kind of code to enable TypeScript when we don't have javascript turned on?
https://github.com/babel/babel/blob/5f74b510ff251af5bf7fbbf25c1c934ffcb6df52/packages/babel-plugin-syntax-typescript/src/index.ts#L60-L61
Alternatively, does TypeScript (and JSX) just work if those plugins are enabled along with this one? Seems fine to require the user to enable the relevant plugins.  And given that you're passing opts through as is, I would have expected code like the above to pass the correct options to the parser. (What error are you getting?)

I'm not going to work on this any more right now but it may be useful for some projects using Babel. If anyone tries it and hits specific issues I can try to look into them when I have time. I figure that getting this out is better that it just sitting in limbo forever.


Add container disabled style

👍

The solution we arrived at in a Slack discussion is to pass in a different style when you disable the button:
<Button
  disabled={isDisabled}
  containerStyle={!isDisabled ? normalButtonStyle : disabledButtonStyle}
/>


No output with vmwgfx (regression)

PR #2169 broke output for vmwgfx (Mesa's VMware SVGA3D driver).
Sway log: http://ix.io/2lRS

Can you run drm_info?

http://ix.io/2lSy

Segmentation fault

Eh, this isn't supposed to happen. Can you run it in GDB and then type bt full?

The wlroots issue might be a duplicate of #2187
(The drm_info crash is something else)

GDB backtrace: http://ix.io/2lSS

Can you try https://github.com/swaywm/wlroots/pull/2194?

That fixed it :)


TD-199 All courses page: поменять логику запроса к фильтрам

Task https://tessera-mosaic.atlassian.net/browse/TD-199
Changes:

removed old logic for response with pagination

@zigfrei
we don't have sorting "by default", we have "Recommended" (рекуомендуемые) that should be a default option. Could you change "by default" for "recommended"?

@ReneNorth In this case, how would you remove the sorting if a sorting filter was selected? For example, you chose the recommended sorting and then you want to remove the sorting and leave the filters?

@zigfrei  That's a question for our designers. I think clicking a chosen sorting option should change it to a default state without any sorting. But I believe that "no sorting" should indicate the absence of applied sorting rather than being a separate option.

@ReneNorth done


[GIT PULL] fix build error related to statx when using glibc >= 2.28

fix build error related to statx when using glibc >= 2.28
glibc >= 2.28 supports statx out of the box. This means we get an error when attempting to build liburing when glibc is >= 2.28, for e.g. in my glibc version 2.33 I get the following errors
:26:8: error: redefinition of ‘struct statx_timestamp’
   26 | struct statx_timestamp
      |        ^~~~~~~~~~~~~~~
In file included from statx.c:14:

or
error: redefinition of ‘struct statx’
   30 | struct statx
      |        ^~~~~
In file included from statx.c:14:

This PR fixes the make error by choosing to use the glibc version of statx if available. Additionally, I have replaced /bin/bash with a more portable version /usr/bin/env bash so that we can also run tests in systems that doesn't include '/bin/bash', e.g. NixOS.
glibc 2.28 release notes:
https://sourceware.org/git/?p=glibc.git;a=blob;f=NEWS;h=154ab22d7ca065af5233406927302bb7f6a66890;hb=3c03baca37fdcb52c3881e653ca392bba7a99c2b#l84
git request-pull output:
The following changes since commit 7b5681f82bf53b42afc7cb35482036559d5bad42:

  test/io-cancel: -ECANCELED is a valid return value (2021-11-06 15:16:23 -0600)

are available in the Git repository at:

  origin/glibc-statx

for you to fetch changes up to 59b3547104367a9e0e5ac149482fa72119f61ac1:

  configure: check for glibc statx support. Use glibc statx instead of <sys/stat.h> if available (2021-11-12 15:02:07 +0000)

----------------------------------------------------------------
Bikal Lem (1):
      configure: check for glibc statx support. Use glibc statx instead of <sys/stat.h> if available

 configure              | 31 ++++++++++++++++++++++++++++++-
 src/include/liburing.h |  1 -
 test/Makefile          |  6 +++++-
 test/file-verify.c     |  1 +
 test/rename.c          |  1 +
 test/runtests-loop.sh  |  2 +-
 test/runtests.sh       |  2 +-
 test/unlink.c          |  1 +
 8 files changed, 40 insertions(+), 5 deletions(-)

The Signed-off-by tag should be in the commit message. Not in the pull request description.
Please also keep your commit message title (the first line) at most 72 chars.
You can write extra explanation in the description. Please read the pull request guidelines carefully.

Done.

Can you split this into multiple patches? It's a good hint that when the commit message isn't just describing a single issue, then that should be separate patches.
Also, I really like your description in here, but the patch commit message is very lacking. All the detail should go into the patch itself, github is just a side channel, the git repo is the canonical truth for figuring out why a particular change was done and needed.

The Signed-off-by tag should be in the commit message. Not in the pull request description.
Please also keep your commit message title (the first line) at most 72 chars.
You can write extra explanation in the description. Please read the pull request guidelines carefully.

Done.

Not strictly necessary, but you might as well update the diffstat (git request-pull output).
A bit off-topic. BTW, I am not sure why people easily misunderstand about Signed-off-by tag, they think it's for pull request description, but that should be in the commit message.
Did I write the PR guidelines badly? FYI, I am not a native English, so my words may not be so good.

Can you split this into multiple patches? It's a good hint that when the commit message isn't just describing a single issue, then that should be separate patches.

I  have split this PR into 2 (one to address make errors and another to address make runtests errors). This PR only addresses make errors related to statx.
I will submit another one for make runtests after this one is merged.

@ammarfaizi2 I will address your suggestions in another PR.

@ammarfaizi2 @axboe The PR is ready now after wrangling with some git foo for a bit.

This breaks compile for me:
file-verify.c: In function ‘test_truncate’:
file-verify.c:79:13: warning: implicit declaration of function ‘fstat’ [-Wimplicit-function-declaration]
   79 |         if (fstat(fd, &sb) < 0) {
      |             ^~~~~
file-verify.c:85:13: warning: implicit declaration of function ‘S_ISREG’; did you mean ‘S_IFREG’? [-Wimplicit-function-declaration]
   85 |         if (S_ISREG(sb.st_mode)) {
      |             ^~~~~~~
      |             S_IFREG
file-verify.c:87:20: warning: implicit declaration of function ‘S_ISBLK’; did you mean ‘S_IFBLK’? [-Wimplicit-function-declaration]
   87 |         } else if (S_ISBLK(sb.st_mode)) {
      |                    ^~~~~~~
      |                    S_IFBLK

Needs at least this:
diff --git a/test/file-verify.c b/test/file-verify.c
index 50cad450e8e5..327cb1d19bc3 100644
--- a/test/file-verify.c
+++ b/test/file-verify.c
@@ -12,6 +12,7 @@
 #include <assert.h>
 #include <string.h>
 #include <sys/ioctl.h>
+#include <sys/stat.h>
 #include <linux/fs.h>

 #include "helpers.h"
diff --git a/test/rename.c b/test/rename.c
index af09d65a85dc..7798d4334d78 100644
--- a/test/rename.c
+++ b/test/rename.c
@@ -9,6 +9,7 @@
 #include <stdlib.h>
 #include <string.h>
 #include <fcntl.h>
+#include <sys/stat.h>

 #include "liburing.h"

diff --git a/test/unlink.c b/test/unlink.c
index f8c76397f74f..e4c210bf443f 100644
--- a/test/unlink.c
+++ b/test/unlink.c
@@ -9,6 +9,7 @@
 #include <stdlib.h>
 #include <string.h>
 #include <fcntl.h>
+#include <sys/stat.h>

 #include "liburing.h"

I have force pushed it. Can do make clean && make okay now.


Occasional races and file access issues in Provisioner (azure cli)

example stacktrace in a project that wants to deploy a storage account and eventhubs namespace:
Provisioning eventhubns...
WARNING: The configuration value of bicep.use_binary_from_path has been set to 'false'.
ERROR: The command failed with an unexpected error. Here is the traceback:
ERROR: [WinError 32] The process cannot access the file because it is being used by another process
Traceback (most recent call last):
  File "D:\a\_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\knack/cli.py", line 233, in invoke
  File "D:\a\_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 664, in execute
  File "D:\a\_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 731, in _run_jobs_serially
  File "D:\a\_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 701, in _run_job
  File "D:\a\_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 334, in __call__
  File "D:\a\_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/command_operation.py", line 121, in handler
  File "D:\a\_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/command_modules/resource/custom.py", line 4476, in build_bicep_file
  File "D:\a\_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/command_modules/resource/_bicep.py", line 107, in run_bicep_command
  File "D:\a\_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/command_modules/resource/_bicep.py", line 305, in _run_command
  File "subprocess.py", line 548, in run
  File "subprocess.py", line 1026, in __init__
  File "subprocess.py", line 1538, in _execute_child
PermissionError: [WinError 32] The process cannot access the file because it is being used by another process
To check existing issues, please visit: https://github.com/Azure/azure-cli/issues
Error provisioning eventhubns.
System.InvalidOperationException: Command C:\Program Files\Microsoft SDKs\Azure\CLI2\wbin\az.cmd bicep build --file "C:\Users\OisinGrehan\AppData\Local\Temp\aspirew3d0p52w.toj\eventhubns.module.bicep" --stdout returned non-zero exit code 1
   at Aspire.Hosting.Azure.Provisioning.BicepProvisioner.ExecuteCommand(ProcessSpec processSpec) in C:\p\github\aspire\src\Aspire.Hosting.Azure\Provisioning\Provisioners\BicepProvisioner.cs:line 360
   at Aspire.Hosting.Azure.Provisioning.BicepProvisioner.ExecuteCommand(ProcessSpec processSpec) in C:\p\github\aspire\src\Aspire.Hosting.Azure\Provisioning\Provisioners\BicepProvisioner.cs:line 368
   at Aspire.Hosting.Azure.Provisioning.BicepProvisioner.GetOrCreateResourceAsync(AzureBicepResource resource, ProvisioningContext context, CancellationToken cancellationToken) in C:\p\github\aspire\src\Aspire.Hosting.Azure\Provisioning\Provisioners\BicepProvisioner.cs:line 193
   at Aspire.Hosting.Azure.AzureProvisioner.ProcessResourceAsync(IConfiguration configuration, Lazy`1 provisioningContextLazy, IAzureResource resource, CancellationToken cancellationToken) in C:\p\github\aspire\src\Aspire.Hosting.Azure\Provisioning\Provisioners\AzureProvisioner.cs:line 246

@oising were you hitting this with the eventhubs playground sample? I've not seen this before and I've done a lot of deployments. So I'm trying to figure out if this issue is still occuring.

Haven't seen it since, @mitchdenny - it was that day.

OK I'm going to close this. If it pops up again feel free to re-open.


General: enhance create/import exercise button with exercise type icons on desktop

Checklist
General

[x] This is a small issue that I tested locally and was confirmed by another developer on a test server.
[x] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[x] I chose a title conforming to the naming conventions for pull requests.

Client

[x] Important: I implemented the changes with a very good performance, prevented too many (unnecessary) REST calls and made sure the UI is responsive, even with large data.
[x] I followed the coding and design guidelines.
[x] Following the theming guidelines, I specified colors only in the theming variable files and checked that the changes look consistent in both the light and the dark theme.
[x] I added multiple integration tests (Jest) related to the features (with a high test coverage), while following the test guidelines.
[x] I added authorities to all new routes and checked the course groups for displaying navigation elements (links, buttons).
[x] I documented the TypeScript code using JSDoc style.
[x] I added multiple screenshots/screencasts of my UI changes.
[x] I translated all newly inserted strings into English and German.

Motivation and Context
Stephan suggested adding exercise-type icons for the Create/Import Exercise buttons in the exercise group management view to improve UX.
Description
These icons are already displayed on mobile, when the button label is not displayed anymore. This PR displays them also on desktop after the label.
Edit: Same goes for the course exercise create/import buttons and the order of create and import buttons is unified to first the "Create" Button and then the "Import" Button.
Steps for Testing

Prerequisites:

1 Instructor
1 Exam with 1 Exercise Group
View the control section there on mobile and desktop
Verify that the exercise-type icons are displayed on both

Review Progress

Code Review

[x] Code Review 1
[x] Code Review 2

Manual Tests

[ ] Test 1
[ ] Test 2

Test Coverage

Screenshots

Desktop:

Mobile:

course exercises:

Thank you for your feedback, I sticked to the exercise icon at the end, added the icon also in the course exercise views and unified the order of create/import buttons. Please have another look at the PR.


code to compute fs3

Thanks for your awesome works!.
I have a new GAN model. Where code to compute fs3.

I found it!


Integration with Spring

Spring  supports the notion of MessageListener with AMQP/Kafka/JMS where you can define the concurrent instances that will be created by container as a recipient of messages from Queue.
I am trying to understand how similar thing is possible with Nats Streaming Java API.
What I could gather from documentation is as below (Please correct if I am mistaken here):

Inject ConnectionFactory (seems a candidate for being a Singleton)
Create an Instance of Connection as subscriptions are tied to Connection as evident from following code block:

ConnectionFactory cf = new ConnectionFactory(clusterID, clientID);
Connection sc = cf.createConnection();
Subscription sub = sc.subscribe("foo", new MessageHandler() {
  public void onMessage(Message m) {
    latch.countDown();
    System.out.printf("Received a message: %s\n", new String(m.getData()));
  }
}, new SubscriptionOptions.Builder().deliverAllAvailable().build());

It seems possible that we define MessageHandler as a Bean with prototype scope so that instances are created on-demand

Questions:

How do we create certain number of listener instances (message handler) during container bootstrap?
Is subscription always tied with one single connection?
Is there a notion of connection pooling of some sort?

thanks.

@sachinwalia2k8 , Thank you for using NATS!  To answer your questions:

How do we create certain number of listener instances (message handler) during container bootstrap?

One approach might be to have more than one subscription.  You can create multiple subscriptions on the same subject, and your application can restrict the number of subscriptions.  Since message listeners and subscriptions have a 1:1 relationship, this may be a cleaner way for you to control message listener instances.

Is a subscription always tied with one single connection?

Yes, a subscription is tied to the connection it was created on.

Is there a notion of connection pooling of some sort?

No, there isn't at this time.  Each connection requires a unique client ID (to maintain client state at the server), so pooling could result in unpredictable behavior.
Did this answer your questions?
Regards,
Colin

Thanks @ColinSullivan1 for your response. So if we follow this strategy then one way to achieve this would be to spawn threads from within listener itself. Since creating on-demand thread in java is expensive so I guess bootstrapping a thread pool during startup could be one strategy to reduce the latency. So at this time it sounds like listener could be envisaged as a control plane for launching further consumers. Let me know if this sounds about right or if there is a scalability/consistency issue?
thanks.

@sachinwalia2k8 , absolutely, happy to help.  What you described sounds good; I don't see any concurrency issues.
One thing to consider is manually acknowledging messages since you are handing them off to another thread.  An example of using setManualAcks(true) to do this can be found here.
Please let us know how this turns out for you!
Thanks,
Colin

Thanks @ColinSullivan1 I am going to try this approach today. My project has mix of both Go and Java based services that needs to intercommunicate. From Go it is fairly straightforward but I wasn't sure the right approach on the Java side.

Closing as part of housekeeping.  If this is still an issue, please reopen this.


TypeError

Hi,
When i start my server gives many of these messages
And my bots not coming to me

Same, more my bots work perfect

Same :(

http://www.mediafire.com/download/8lp1gft898y2h2u/SquareBots_V1.3.zip

https://join.skype.com/vHEYCCOZ3IbK


Shared SPI bus between devices with different SPI modes

I have a small board that has 2 devices attached to the same SPI bus

AD9833 DDS that uses SPI MODE 2
MCP41010 Digital Potentiometer that can use SPI MODE 0 or MODE 3

Each device has its own CS pin.
I am using the following two drivers

https://github.com/eldruin/ad983x-rs.git
https://github.com/dan-corneanu/mcp4x-rs.git

The problem that I am facing is that my program can only communicate with one of the devices because
the SpiDevice can not change the mode of the SpiBus.
I think this design decision was discussed back in 2022 in this comment https://github.com/rust-embedded/embedded-hal/pull/351#issuecomment-1043686888.
Could you please advise on how I could address this problem in my code?
If interested in the code of my application you can have a look at this branch https://github.com/dan-corneanu/ad983x_signal_generator/tree/features/configure_ushell.

At the moment this must be implemented in a HAL-specific way. The difficulty will depend on the HAL you are using (I cannot access your application repo) but it will probably involve rewriting a bit of embedded-hal-bus so that when a device is activated, the bus is reconfigured.

Have made it public.


Fix codegen issues and update travis CI for CoreNEURON tests

#196 introduced regression in the sense that the perfvisitor
was not running just before codegen. Because of this, read/write
count was not updated resulting in const/non-const issues.
update travis configuration so that coreneuron is built and run
on every PR to avoid issues like above

fixes #226 #227

The code fix is fine. I figured out what is going on with the symtab updates and PerfVisitor passes after developing the related PR. I am sorry..

that's fine @iomaganaris :) I should have enabled test when I setup travis first time :)

Edit: It doesn't run the tests in the Linux build but runs them in the mac os, so I guess it is fine?

Ahh I didn't notice that. I will fix that. Most likely boost-test package.


Restrict usage questions to SO

I suggest we forward all usage questions (like you practiced some time ago) to Stackoverflow. This will free up some of your time and will help build some hype on SO, as the more questions, the more attention SUI gets. Am I right?
And maybe you could once a week rummage through long unanswered questions on SO and help some folks in case they get really stuck..
If you agree, I would amend the guidelines.

I understand when people aren't sure whether their issue is a bug or just improper usage. It seems to me for people who are uncertain to post here. Although I definitely need some help moderating.
I am going to start being a bit harsher on people who don't bother following the bug report guidelines, omitting test cases etc. Not worth the comment anymore.

btw, I've already amended them once!
strange, that you've removed https://github.com/Semantic-Org/Semantic-UI/commit/9b661cca3dedb124222cb7a1098d91769942149f#diff-6a3371457528722a734f3c51d9238c13

I certainly support shunting away entry-level usage questions to SO.  GitHub issues can be used for whatever, obviously, but when issues bloat up, it creates mental baggage (for you, as a the project head,  that they need to be cleaned up), and it dilutes the experience for people like me, which check up on new issues once a week or so.  SO is a better place for Q&A, and provides better discoverability and later evolution of the answers.

Actually, I see your point and I think I agree.

This can be closed, as it's already being done  (gitter).


require_config: remove modernizr from deps list

Modernizr isn't necessary anymore and as such not imported by bower. By accident it is still listed as dependency for RequireJS, which leads to failing tests.

Fixed on master (v1.0.0).


Refactor filenames and directory structure

Our client suggested a few changes

Grouping files into folders by class type, Changes, Generators, and Data Types
Making the file names more consistent. For example the placement of the word Spanner in the java class is sometimes in the beginning, middle and end.

I've updated the filenames, the class names, and the affected test files.
They've also suggested to rename the main CloudSpanner.java to be DatabaseSpanner.java ? Just to show it inherits from the Abstract Database class. I've not implemented this in case it breaks backwards compatibility.

Note -- more refactoring will be needed to be done post this change.


Using another model despite of provided in android sample

@ry @jmhodges
I have replaced the tensorflow lite (Detect.tflite) model in the android sample.
I ended up with several errors on building the project.
One of them is
 java.lang.IllegalArgumentException: Cannot convert between a TensorFlowLite buffer with 1080000 bytes and a ByteBuffer with 270000 bytes.
After some try and error I ended up with another error mentioned below
java.lang.IllegalArgumentException: Cannot copy between a TensorFlowLite tensor with shape [1, 1917, 4] and a Java object with shape [1, 3].
Please look it into asap so can move forward with the sample.

Please provide details about what platform you are using (operating system, architecture). Also include your TensorFlow version. Also, did you compile from source or install a binary?
Make sure you also include the code snippet to reproduce the issue. If you are unclear what to include see the issue template displayed in the Github new issue template.
We ask for this in the issue submission template, because it is really difficult to help without that information. Thanks!

@ravikyram
I have tried running the Demo on my samsung A50.
Simply I have replaced the detect.tfite with my .tflite file and label file with my file, and ended up with above errors.
Please assist.

Simply I have replaced the detect.tfite with my .tflite file and label file with my file

You can't simply drop in an arbitrary model and expect the sample to function without any modifications. In particular, depending on your model, the input/output tensors will differ, and you'll need to adjust your usage of the TFLite Interpreter accordingly. Without knowing any additional details about your model, it's impossible to say what is going wrong here, but it's pretty clear that your new model has fundamentally different input shapes and output types than what are used in the sample.
Have a look at this guide for how to inspect your graph to determine the format of inputs/outputs you should provide.


Debuggers: Add a tooltip for value on key hoverover

VSCode Version: 1.8.1
OS Version: Win10x64
Observed on: Node2-, Chrome-, Firefox debuggers

There is a tooltip showing the value, when hover over the value, but no tooltip, when hovering on the corresponding key. So if the key is longer than the inspector width - there is no way to see the value other than to resize the inspector.

Thank you.

Just noticed that the tooltip does show for non primitive values (i.e. the line is expandable)

On name hover we show the type of the variable (if it is provided by the debug extension).
If type is not provided we now show the name to accomodate for the use case you have described

Why?

Can you elaborate please

Why does it have to be type? How is this useful? One rarely needs it and if so, it can be determined through value color (if primitive) or it is specifically named if it is complex typed one. Value is more important and as described in my OP it sometimes inaccessible unless the inspector is resized. Why is this bugfix is useless?.

More details about this can be found here https://github.com/Microsoft/vscode/issues/8040
You can also find a lot of related issues linked
One of the main drivers for this is C# debugging


"special_workflow" config_automation.yml option

Purpose/implementation Section
What changes are being implemented in this Pull Request?
This is in response to a BaseCamp conversation.
How do we have other workflows that need to be run added into the pull_request.yml without accidentally overwriting them in sync PRs?
We can try to specify one in config_automation.yml (at least that's what I'm trying to do here. Let's see 🤞)

I'm really close on this.
Error: Can't find 'action.yml', 'action.yaml' or 'Dockerfile' under '/home/runner/work/AnVIL_Template/AnVIL_Template/.github/workflows/custom-workflow.yml'. Did you forget to run actions/checkout before running your local action?

This error is saying that it can't find the file, but I've done multiple tests to show that the file is there and where it says.
But as far as I can tell, this is following what the documentation says: https://docs.github.com/en/actions/using-workflows/reusing-workflows#creating-a-reusable-workflow
So I'm a bit stumped. I can look at it again later, unless @avahoffman wants to take a whack at it.

Ah, I see. It needs a metadata file: https://docs.github.com/en/actions/creating-actions/metadata-syntax-for-github-actions
I'll whip one up.

@cansavvy are you still working on this?

@cansavvy are you still working on this? I like the direction this is headed. If you get to a stopping point or are stuck I'm happy to jump in (but don't want to duplicate too much effort) 🙂

No worries! Yea I'll be returning to it later this week but if you want to take a whack at it in the meantime, that is great!

@cansavvy I think I have things working.. but for some reason now the access token isnt working.
One thing that might need rethinking is the custom workflow toggle. As is, it's necessary for render-preview. So if the custom-workflow toggle is off, the preview doesn't get rendered regardless of what its toggle is set to.
Maybe what I'm trying to say is that the custom workflow should just be a dummy workflow for OTTR, but customized for AnVIL. Thoughts?

Maybe what I'm trying to say is that the custom workflow should just be a dummy workflow for OTTR, but customized for AnVIL. Thoughts?

I was writing this with mainly OTTR general purposes in mind, but you should definitely customize it for this anvil template for what you need. I'll bring over the useful things to the main OTTR template after we've developed it here.

Maybe what I'm trying to say is that the custom workflow should just be a dummy workflow for OTTR, but customized for AnVIL. Thoughts?

I was writing this with mainly OTTR general purposes in mind, but you should definitely customize it for this anvil template for what you need. I'll bring over the useful things to the main OTTR template after we've developed it here.

@cansavvy Totally. I'm not doing a good job explaining 😛 . I'm thinking OTTR would:

have something extremely basic in the custom workflow file.
ignore the custom workflow file downstream. That way no OTTR updates overwrite it.
not have a custom workflow toggle. It will always be run.

Happy to discuss if helpful!!

have something extremely basic in the custom workflow file.

ignore the custom workflow file downstream. That way no OTTR updates overwrite it.

not have a custom workflow toggle. It will always be run.

This sounds like a good plan.


Limited CHGM alert resolve investigations to clusters that have the CHGM LS reason

What?

Limited CAD CHGM resolve investigation to clusters that CAD put into LS itself, as we everything else is a NOOP for CAD.
Renamed LimitedSupportReasonExists to IsInLimitedSupport to make the function names less confusing

Why?
This change allows us to use less resources for CAD pipelines and removes unnecessary long pipeline runs.
https://redhat-internal.slack.com/archives/C04PN0YCWGL/p1676498065888859?thread_ts=1676463360.065339&cid=C04PN0YCWGL

/retest