Closing Popup by pressing Escape doesn't work in all cases

Description
The keyboard handler adds the keydown-Listener to the whole document and not to the map-container element.
In order not to interfere with other keyboard-events, the keydown-listener gets registered when the map-container receives focus and gets removed when a blur event on the map-container occurs.
https://github.com/Leaflet/Leaflet/blob/37d2fd15ad6518c254fae3e033177e96c48b5012/src/map/handler/Map.Keyboard.js#L56-L59
https://github.com/Leaflet/Leaflet/blob/37d2fd15ad6518c254fae3e033177e96c48b5012/src/map/handler/Map.Keyboard.js#L71-L74
https://github.com/Leaflet/Leaflet/blob/37d2fd15ad6518c254fae3e033177e96c48b5012/src/map/handler/Map.Keyboard.js#L132-L138
When the marker gets clicked, it receives focus, therefore the map container looses focus and triggers a blur event. The blur event disables the Keyboard-Handler and the opened popup can't be closed by pressing Esc because the Keyboard-Handler is disabled.

[x] I've looked at the documentation to make sure the behavior is documented and expected
[x] I'm sure this is a Leaflet code issue, not an issue with my own code nor with the framework I'm using (Cordova, Ionic, Angular, React…)
[x] I've searched through the issues to make sure it's not yet reported

Steps to reproduce
Steps to reproduce the behavior:

Add a marker to a map
Bind a popup to that marker
Click the marker to open the popup
Try to close the opened popup by pressing Esc

Expected behavior
The opened popup should be closed then the Esc-Key is pressed
Current behavior
The popup get's only closed when the layer the popup is bound to is not focusable
Environment

Leaflet version: 1.4
Browser (with version): Chrome 73
OS/Platform (with version): Windows 10

Minimal example reproducing the issue
https://next.plnkr.co/edit/Soh1otCSsfdBBCi0

[x] this example is as simple as possible
[x] this example does not rely on any third party code

I'm wondering why the keydown listeners are added to the document and not the map-container.
Is it necessary for some legacy browser?
I've made a local branch where the KeyBoard-Handler add the listeners to container and note the whole document. By doing that it is not necessary to add and remove the listeners on focus and blur and it seems somewhat simpler.

Because keyboard events only fire/bubble on HTMLElements that the browser deems focusable. Let me quote from https://w3c.github.io/uievents/#events-keyboard-event-order , emphasis mine:

The event target of a key event is the currently focused element which is processing the keyboard activity. This is often an HTML input element or a textual element which is editable, but MAY be an element defined by the host language to accept keyboard input for non-text purposes, such as the activation of an accelerator key or trigger of some other behavior. If no suitable element is in focus, the event target will be the HTML body element if available, otherwise the root element.

Because a Leaflet map container is often a non-focusable-by-default <div>, markers can have non-focusable DivIcons and pop-ups have non-focusable containers, and on top of that there's multiple browser implementations, the only reliable way of handling keyboard input is to attach the event handlers to the document.
The current code attaches focus/blur and keyboard event handlers to the map container at ...
https://github.com/Leaflet/Leaflet/blob/master/src/map/Map.js#L1326-L1327
...since https://github.com/Leaflet/Leaflet/pull/6421, but anyway this is one of the areas where I must ask for testing in several browsers and environments.

Let me quote from https://w3c.github.io/uievents/#events-keyboard-event-order , emphasis mine:

If no suitable element is in focus, the event target will be the HTML body element if available, otherwise the root element.

https://github.com/Leaflet/Leaflet/issues/2199#issuecomment-779416114 suggests that focus should be sent to the popup's container after it's been opened (a requirement for screen readers to announce the content of the popup) which would then mean there's a suitable element in focus.

https://github.com/Leaflet/Leaflet/issues/6551#issue-419579336:

When the marker gets clicked, it receives focus, therefore the map container looses focus and triggers a blur event. The blur event disables the Keyboard-Handler and the opened popup can't be closed by pressing Esc because the Keyboard-Handler is disabled.

https://github.com/Leaflet/Leaflet/issues/6551#issuecomment-471714430:

Because keyboard events only fire/bubble on HTMLElements that the browser deems focusable

Because [...] pop-ups have non-focusable containers

The solution to https://github.com/Leaflet/Leaflet/issues/2199 is to make the popup's container programmatically focusable via tabindex="-1", perhaps that'd be sufficient to have the Esc key close popups more reliably? Just an observation.

hello, I've attempted a solution by modifying the Popup class's Layer to include a keydown eventListener to close the   popup. ( #8239 )
Edited Plunkrs for review
https://plnkr.co/edit/pkOzSCsI5QvtAcAe
https://plnkr.co/edit/4hXgn7EMinn9Wwn6


Zed deletes symlinked settings.json file when changing theme via theme selector: toggle

Check for existing issues

[X] Completed

Describe the bug / provide steps to reproduce it
When ~/.config/zed/settings.json is a symlink, changing the theme via the command palette seemingly performs an atomic write, deleting the symlink then writing a new JSON file in its place. This affects anyone that wants to store their Zed config in a central "dotfiles" repository with a symlink to the correct location
I can't think of any other commands that would change the config file, off the top of my head, but I'd guess they're affected by this issue too.
Environment
Zed: v0.104.3 (preview)
OS: macOS 13.5.2
Memory: 16 GiB
Architecture: aarch64
If applicable, add mockups / screenshots to help explain present your vision of the feature
No response
If applicable, attach your ~/Library/Logs/Zed/Zed.log file to this issue.
If you only need the most recent lines, you can run the zed: open log command palette action to see the last 1000.
No response

Just wanted to add something on to this from discord - I don't want to make any presumptions about the way this is fixed, but presuming it's fixed without implementing live config reload, I could easily live without that presuming there was an action to reload the config that I could configure myself, such as "cmd-,": "zed::ReloadSettings"

Alternatively, have some way for zed to use a configurable settings path, so the file in .config/zed can just point to the real one stored elsewhere?


VISIBLE keyword causes parser error

Describe the bug
Reading an SQL DDL that uses the VISIBLE keyword causes an error and the entire table definition is not read
To Reproduce
Steps to reproduce the behavior:

Write an SQL create table statement with an index that uses the VISIBLE keyword
Read the SQL with the ddl parser
See the error generated

Expected behavior
DDL parser should read SQL DDL statements with the VISIBLE keyword

released in version 1.1.0,  added in PR: https://github.com/xnuinside/simple-ddl-parser/pull/246/files
if will be needed anything else - feel fre to re-open this issue or create new one


deleted error-causing old annotation

related to #760

Looks good.  Might be best to switch fbbt_ext.owl to functional syntax after you've merged this.  Safer for hand editing.


oauth2_proxy with DEX on Kubernetes

Hello,
Is there someone who succesfully use ouath2 on K8S with DEX and can share a deployment conf , how to start it with correct issuer-url and provider ?

Looking for the same thing
@Drupi Did you find any documentation? Mind to share?

@Drupi @exodussystem check out the https://github.com/jenkins-x/sso-operator. It has a helm chart for dex with cert-manager integration, and also you can provision the oauth2_proxy as a Kuberntes CRD in front of a service.


How to conduct model fine-tuning training and what is the required data magnitude?

Hi, you can follow our instructions on model training to finetune the model on your datasets. If you fine-tune the pretrained InstanceDiffusion model with LORA, the data magnitude can be significant reduced. The exact data magnitude depends on your task.

Do you have any plans to open source for lora training?

Hi, this repo is mainly for reproducing the results reported in our paper. Currently, we don't have plans to support additional new tasks or lora training. But you can check https://github.com/cloneofsimo/lora to learn how to add LORA to stable diffusion (the base model we used in this repo). Thanks!

Thanks you for your great job,Can I use your model to do simpler training on a RTX3090 24g?

You may want to use flash attention (and or deepspeed) during the training time if you want to train the model using RTX 3090. Flash attention was implemented already, you can set it as True in the .yaml config file.
Hope it helps.

Closing it for now, please reopen-it if you have more questions.


Consolidate encrypt and decrypt functions

The current architecture, with function pointers, etc., is pretty heavyweight. The draft now supports a much-streamlined approach where there's a single connection ID encoding and there are optional encryption techniques applied to it depending on the parameters.

fixed by #16


Fix managed parameters used for the Deploy-Private-DNS-Zones Policy Assignment

Overview/Summary
This PR updates the managed parameters set for the Deploy-Private-DNS-Zones Policy Assignment to ensure complete mapping of private DNS zones for private endpoints.
Additionally, I have updated the list of private DNS zones deployed by the test framework to match those of the policy assignment.
Includes updates to RELEASE.md for other missed changes added prior to this PR.
This PR fixes/adds/changes/removes

Fixes #544
Fixes #545 (duplicate)

Breaking Changes
None
Testing Evidence
Please see the following image showing the correctly populated policy assignment:

In place updates were tested manually during implementation of these code updates. The following shows successful update of the policy assignment:

As part of this Pull Request I have

[x] Checked for duplicate Pull Requests
[x] Associated it with relevant issues, for tracking and closure.
[x] Ensured my code/branch is up-to-date with the latest changes in the main branch
[x] Performed testing and provided evidence.
[x] Updated relevant and associated documentation.
[x] Updated the "RELEASE" page with details needed for when this code change is released.

/azp run update

/azp run unit


Remove ResourceCheck based test filtering

backport of https://github.com/querydsl/querydsl/pull/1256

It wasn't really used that much, and MSSQL based tests are filtered out also via other means.

For consistency I'll merge also this already.


slidev build --watch no work

Describe the bug
The slidev build --watch command is not working as expected. It only performs a single build operation and does not continue to watch for changes and rebuild.
Desktop (please complete the following information):

Slidev version: 0.49.2

Reproduced.
Could you please tell us why you need to build the slides in watch mode? Run the slidev command itself will start a dev server.

轉載。
您能否告訴我們為什麼需要在觀看模式下建立投影片？運行slidev命令本身會啟動一個開發伺服器，它可以反映原始程式碼的變化，甚至不需要刷新瀏覽器頁面。

I saw this usage in the documentation.

Yes, and this is not working😓. And since we have the dev server, this option is almost useless, so we are planning to remove it. Want to know why you want to use this option?

Yes, and this is not working😓. But since we have the dev server, this option is almost useless, so we are planning to remove it. Want to know why you want to use this option?
中文版：这个功能坏了，但是因为可以直接运行slidev命令启动开发服务器，所以我觉得这个功能其实没有实际用处。想问一下您为什么需要这个功能

Thank you for your response. I have found another way to achieve this, so I am closing this issue.


MVP: Implement a batch/apply style function

Feature request
Blocks #9. We want to apply a function, such as a periodogram function, to all lightcurves stored within a nested structure. We should implement a simple function for applying this function to the lightcurves. With TAPE, this motivated the batch function for grouping the sources and applying a function to them. Our data structures motivate a custom function a bit less, as the data is pre-grouped. So using the base apply may potentially be fine, however we may be able to provide a more intuitive API function for doing this operation and can potentially optimize for how it accesses the lightcurve data under the hood. Usage should look something like this:
mean_flux = nf.batch(my_mean_flux, "my_nested.flux")

Before submitting
Please check the following:

[x] I have described the purpose of the suggested change, specifying what I need the enhancement to accomplish, i.e. what problem it solves.
[x] I have included any relevant links, screenshots, environment information, and data relevant to implementing the requested feature, as well as pseudocode for how I want to access the new functionality.
[x] If I have ideas for how the new feature could be implemented, I have provided explanations and/or pseudocode and/or task lists for the steps.

A small suggestion here is that I would avoid the name "batch" here since it has the connotation that we're doing batch processing of splitting the job into multiple computable batches. That connotation makes less sense here where there is no applied parallelism.


[1] Check if the S3 API can be used for Harbor on Cleura

What should be investigated.
We currently use Swift as the Harbor image/chart storage because there were some issues when uploading/downloading large images using the S3 API. This was introduced over a year ago and things may have been fixed in the meantime.
Let's investigate if S3 can be used as this would remove some extra configuration and management burden.
What artifacts should this produce.
Try uploading charts and images (especially really large ones) to Harbor while using S3 on CityCloud. If this works, create a follow up feature issue.

Have tested on Cleura / Fra1 and it worked as expected so we should be able to start using S3 as default on Cleura.

Update: do remember to check if the naming convention discrepancy between swift and s3 is a problem.

If we can switch to s3, then we also need some migration path for environments that currently use swift.


how i can get only TimeStamp Value

hi today i used goapcket and i have issue:
how i can get only Timestamp value from tcp.Options.. i make for loop on it and have 3 key and value 0 1 2, but when i chosse [2] i got error slice out of range
thanks

Note that many TCP packets don't have the timestamp option set in the
packet.  If you'd like to iterate over all options, use:
for _, opt := range tcp.Options {
...
}
If you'd like to get the capture timestamp for a packet, use:
packet.Metadata().Timestamp
On Mon, Dec 4, 2017 at 3:07 PM, SaifSalah<EMAIL_ADDRESS>wrote:

hi today i used goapcket and i have issue:
how i can get only Timestamp value from tcp.Options.. i make for loop on
it and have 3 key and value 0 1 2, but when i chosse [2] i got error slice
out of range
thanks
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/google/gopacket/issues/393, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABJ9tupeTtz4Y_xDRshuli1rjgTZc0XPks5s9G01gaJpZM4Q1ZhB
.

srry i think you don't understand me what  i mean
this my code
`	tcpLayer := packet.Layer(layers.LayerTypeTCP)
if tcpLayer != nil {
tcp, _ := tcpLayer.(*layers.TCP)
fmt.Println("SrcPort: ", tcp.SrcPort)
fmt.Println()
for i,val  := range tcp.Options {
			fmt.Println("Key: ",i,"Value :", val)
		}
	} else {
		fmt.Println("NULL.")
	}`

this my result from capture
Key:  0 Value : TCPOption(NOP:)
Key:  1 Value : TCPOption(NOP:)
Key:  2 Value : TCPOption(Timestamps:743883130/749163709 0x2c56c17a2ca754bd)
i want get only this value ( TCPOption(Timestamps:743883130/749163709 0x2c56c17a2ca754bd )

thank you, i solved issue xD
for i,_  := range tcp.Options {
			if i == 2{

				fmt.Println(tcp.Options[i])
			}

		}

<3


Proxy should ignore traps whose value is null

const obj = new Proxy({}, {
    getPrototypeOf: null
});

print(Object.getPrototypeOf(obj));

This code should print [object Object], but it throws TypeError: 'getPrototypeOf' is not a function
The trap should be ignored if it is either undefined or null, but ChackraCore only ignores it in the first case.
Specification:
9.5.1 [[GetPrototypeOf]] ( )
  When the [[GetPrototypeOf]] internal method of a Proxy exotic object O is called, the following steps are taken:
    ...
    5. Let trap be ? GetMethod(handler, "getPrototypeOf").
    6. If trap is undefined, then
        a. Return ? target.[[GetPrototypeOf]]().

7.9.3 GetMethod (V, P)
    1. Assert: IsPropertyKey(P) is true.
    2. Let func be ? GetV(V, P).
    3. If func is either undefined or null, return undefined. <---

Can I tackle this issue?

sure go ahead.


Future is not subtype of Future at requestWillBeSent

I've just implemented Stetho in this one project. Requests are failing due to issue:
type 'Future<dynamic>' is not a subtype of type 'Future<String>' where
  Future is from dart:async
  Future is from dart:async

#0      MethodChannelController.responseHeadersReceived (package:flutter_stetho/src/method_channel_controller.dart:17:16)
#1      StethoHttpClientRequest.close (package:flutter_stetho/src/http_client_request.dart:62:29)
<asynchronous suspension>
#2      Stream.pipe.<anonymous closure> (dart:async/stream.dart:609:70)
#3      _RootZone.runUnary (dart:async/zone.dart:1381:54)
#4      _FutureListener.handleValue (dart:async/future_impl.dart:129:18)
#5      Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:633:45)
#6      Future._propagateToListeners (dart:async/future_impl.dart:662:32)
#7      Future._completeWithValue (dart:async/future_impl.dart:477:5)
#8      Future._asyncComplete.<anonymous closure> (dart:async/future_impl.dart:507:7)
#9      _microtaskLoop (dart:async/schedule_microtask.dart:41:21)
#10     _startMicrotaskLoop (dart:async/schedule_microtask.dart:50:5)

Can't really help much. The requestWillBeSent method expect Future<String>, but it does't get that. First time using this package. Any idea why is this happening?

My gosh, somehow wasn't watching the repo and missed your message. My bad...
Luckily, @thejakeofink has a fix up. I'll verify that and publish a new version.

Fix published as 0.1.1.


Cannot provide pre-parsed AsyncAPI documents as input

Describe the bug
Based on this: https://github.com/asyncapi/cupid/issues/67#issuecomment-968961039
I tried to pre-parse my AsyncAPI documents and then give Cupid those pre-parsed documents as inputs.
How to Reproduce
Follow https://github.com/asyncapi/cupid/issues/67#issuecomment-968961039 example.
Expected behavior
By changing the validation function: https://github.com/asyncapi/cupid/blob/c9cdb3df6675d992a42ccbd9e92bd118a21a11d4/lib/utils.js#L9
To something like the following everything works:
function validate(asyncApiDocs) {
  return Promise.all(asyncApiDocs.map(async doc => {
    if (typeof doc === 'object' && doc.ext && doc.ext('x-parser-spec-parsed')) {
      return doc;
    }
    return parser.parse(doc);
  }));
}

This is because the provided document does not have a property called x-parser-spec-parsed in the root object, it is hidden under _json, so it would be an idea to use the functions to access the extension 🙂

Working on it.

it was solved, but just not properly mentioned in PR thus not closed automatically


Module could not be found

If I try and run the launcher file it says that the module could not be found and closes the tab. Not sure what's going on exactly

Northstar version: 1.4.0
Platform: Origin

Could you not open a new issue and check #faq in discord?
It seems likely that you installed Northstar in an incorrect directory.

Additionally, please provide screenshots to further help us investigate the issue.

Does this section in the wiki describe your issue?

@GeckoEidechse could a label for "possibly resolved" be added for issues waiting on author input

That's what 'info-needed' is for, I'm closing this since the issue template wasn't followed


TypeError: int returned non-int (type NoneType)

After installing all requirements (python-midi and requirements.txt) I get this error message when executing generate.py:
Using TensorFlow backend.
Traceback (most recent call last):
  File "generate.py", line 153, in <module>
    main()
  File "generate.py", line 142, in main
    models = build_or_load()
  File "/Users/frederikriedel/Developer/DeepJ/util.py", line 15, in build_or_load
    models = build_models()
  File "/Users/frederikriedel/Developer/DeepJ/model.py", line 149, in build_models
    notes_out = naxis(time_out, chosen, style)
  File "/Users/frederikriedel/Developer/DeepJ/model.py", line 111, in f
    dense_layer_cache[l] = Dense(int(x.get_shape()[3]))
TypeError: __int__ returned non-int (type NoneType)

Do you maybe know what I'm missing here?

Hmm, it could be a Tensorflow version mismatch? Maybe you're using a newer version than the one DeepJ was developed in?

That was a great hint. It wasn't the tensorflow version itself that caused the issue, but keras.
I installed an older version of keras now and it seems to work!

@frogg Great to know. If you want , feel free to submit a PR to update our code to the latest Keras so others won't run into the same issue. Seems like it would be a one-line change.

@frogg thanks a lot, your re help me

@frogg @calclavia I am using keras 2.1.5 and I got the same error. Which version of Keras should I use? Please let me know.

@frogg  Could you please let me know the tensorflow and keras version.

@swarna-a-26 I think I was using keras 1.2.1.
Let me know if that works. Maybe @dannywu19910524 remembers it better than me.

@frogg @dannywu19910524 I used tensorflow1.6.0 and keras 2.0.0.That seems to remove the above error.Thanks so much.
I am getting the below error when I try to train the data.
volume_dense (Dense)             multiple              129

concatenate_3 (Concatenate)      (None, 128, 48, 3)    0
Total params: 1,269,476.0
Trainable params: 1,269,476.0
Non-trainable params: 0.0

Unable to load model from file.
Loading data
Training
Traceback (most recent call last):
File "train.py", line 32, in
main()
File "train.py", line 16, in main
train(models)
File "train.py", line 29, in train
models[0].fit(train_data, train_labels, epochs=1000, callbacks=cbs, batch_size=BATCH_SIZE)
File "/usr/local/lib/python3.6/site-packages/keras/engine/training.py", line 1405, in fit
batch_size=batch_size)
File "/usr/local/lib/python3.6/site-packages/keras/engine/training.py", line 1295, in _standardize_user_data
exception_prefix='model input')
File "/usr/local/lib/python3.6/site-packages/keras/engine/training.py", line 121, in _standardize_input_data
str(array.shape))
ValueError: Error when checking model input: expected input_1 to have 4 dimensions, but got array with shape (0, 1)
Any idea about this error? I created a data folder and have three mid files there. Any thoughts to overcome this issue? Is this one also related to keras and tensorflow version? any thoughts will be helpful.I am using MAC 10.13.1

@swarna-a-26 i didn't get this error i'll check my keras and tensorflow version later for u

@swarna-a-26 tensorflow-gpu 1.2 keras 2.0

can you please recommend a way out of the error @swarna-a-26 mentioned?
I've used python3, tensorflow 1.2.0 (no GPU), keras 2.0 and also got
ValueError: Error when checking model input: expected input_1 to have 4 dimensions, but got array with shape (0, 1)
please let us know if you can

@slave2sync @swarna-a-26 Have you solve the problem? And can you share the way out of the error:
ValueError: Error when checking model input: expected input_1 to have 4 dimensions, but got array with shape (0, 1)

any news?

@calclavia @slave2sync @swarna-a-26
I am on Ubtuntu Linux with  TF 1.12 with GPU (without GPU) and Keras 2.0.
I am getting the same ValueError:  Error when checking model input: expected input_1 to have 4 dimensions, but got array with shape (0, 1)
Any help would be much appreciated?

I am getting the same ValueError: Error when checking model input: expected input_1 to have 4 dimensions, but got array with shape (0, 1)

Hey, I am also getting the same error. If you fixed it please, let me know.
Thank you.

I am getting the same ValueError: Error when checking model input: expected input_1 to have 4 dimensions, but got array with shape (0, 1)

Hey, I am also getting the same error. If you fixed it please, let me know.
Thank you.

Hey,I had the same error before,than i solved it thanks to this issue #72
You need to construct the training data fold like the styles and genres in 'constant.py', than the error would disappear(at least for me).
Hope this would help!


Polishing Asciidoctor

StringUtils.hasText already includes non null check so the additional check is not needed.
Parsing the module in renderConfigurationProperties isn't needed anymore because the ModuleProperty list is already module specific.

That's merged, thanks!


[SPARK-35486][CORE] TaskMemoryManager: retry if other task takes memory freed by partial self-spill

What changes were proposed in this pull request?
When a memory reservation triggers a self-spill, ExecutionMemoryPool#releaseMemory() will immediately notify waiting tasks that memory has been freed. If there are any waiting tasks with less than 1/2N of the memory pool, they may acquire the newly-freed memory before the current task has a chance to do so. This will cause the original memory reservation to fail. If the initial spill did not release all available memory, the reservation could have been satisfied by asking it to spill again.
This PR adds logic to TaskMemoryManager to detect this case and retry.
Why are the changes needed?
This bug affects queries with a MemoryConsumer that can spill part of its memory, such as BytesToBytesMap. If the MemoryConsumer is using all available memory and there is a waiting task, then attempting to acquire more memory on the MemoryConsumer will trigger a partial self-spill. However, because the waiting task gets priority, the attempt to acquire memory will fail even if it could have been satisfied by another spill.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Added a test to MemoryManagerSuite that previously failed and now passes.

cc @Ngone51 @dongjoon-hyun @cloud-fan

Test build #138815 has started for PR 32625 at commit f7f4847.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43337/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43337/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/43337/

Test build #138815 has finished for PR 32625 at commit f7f4847.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/138815/

Test build #138836 has started for PR 32625 at commit b293e90.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43358/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43358/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/43358/

Test build #138836 has finished for PR 32625 at commit b293e90.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/138836/

Thank you for pinging me, @ankurdave .

Fine with me to consider this for master only.

cc @tgravescs @mridulm

Test build #138910 has started for PR 32625 at commit 3f85e39.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43432/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43432/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/43432/

Test build #138910 has finished for PR 32625 at commit 3f85e39.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/138910/

Thanks, merged to master!


Port terra's assemble() to be a function for the ibmq provider

What is the expected enhancement?
The assemble() function in qiskit-terra is ibm quantum api specific. It's used solely to generate a payload for ibm's job payload format and is only used by the ibmq/ibm provider currently. In the past it was used as the job format submitted to all backend's run() methods but this is deprecated and slated for removal. Moving forward it would be good if we could drop the assemble method from terra so we can avoid needing IBM specific logic (like: https://github.com/Qiskit/qiskit-terra/pull/6167 ). If qiskit-ibm took ownership of assemble() we could start the process of deprecating it in terra.

Probably this should include the entire qiskit.qobj module too

Closing since provider doesn't use assemble() anymore either.
Having said that, @mtreinish please let me or @taalexander know if you do plan on deprecating it, as it's used for other paths.

There aren't any plans this year, but probably we'll deprecate it for the April release next year. This will also apply to all the models objects: BackendConfiguration, etc


Editor Debugger: auto scrolling is activated at every log start.

@dankan1890 Is this pull request no longer valid somehow? It would be better to leave it open if it still can be merged.
(Sorry for the delays in reviewing pull requests. The master branch is undergoing refactoring as we speak.)

Sorry, my mistake :(

@Faless I think you slept on this one :)

I think you slept on this one :)

I did :) . Is this still relevant? The Editor log now gets cleared every time you press play, (and it will restart following if you were not following).
So, I think this behaviour is already the default.
Additionally, I think that now that we have multiple debugger instances it's time to split the output log from sessions in their own tab (I know reduz didn't like this originally, so I guess we'll have to discuss it in a meeting :) ).

Closing, as this behaviour should already be the default, but feel free to comment/re-open if you think it's still useful.


Track and show individual subscriptions to input streams

Hi.
This branch adds logic to keep track of the individual subscriptions to the input sequences and render them as marble diagrams beneath the output marble diagram.
Not very exciting for "simple" operations like filter.  But can be very useful to understanding more complex operators like amb or takeUntil or pausable, etc.
The display of these subscriptions can be suppressed by setting showSubscriptions property to false on the sandbox component.
I think the model and view changes in this PR will also be useful when adding support for the nested observable operators (groupBy, flatMap, etc).
Here are some examples:
merge (not very exciting)

concat (a little more interesting)

some (helps to show its short-circuiting nature)

pausable (ever wonder what it does on a cold observable?)

amb

repeat (added this one)

Thank you for all the improvements, there are several of them! I need to take some time to go through all the changes (there are things such as ghost part of the arrow, which I'm not sure I'll adopt since I'm trying to stay close to the original RxJava diagram look-and-feel), rearranging it also from a UX perspective (probably I'll make the subscription details part toggleable behind some switch or button, to keep the default view simple and uncluttered). But in any case, thank you very much for this PR.

thanks for the project.  It is pretty cool.
Let me know if you want me to make some of those changes.  I'd rather leave the actual UX and UI event handling to you, but I can wire up the rendering to make it easier.
I already added a property to the sandbox to allow you to turn on/off the subscription details so it should be easy to wire up to a toggle.
If you like, I can add a property to the diagram component to control ghosting on a per-diagram basis.  Then it could be turned off for all the regular diagrams and only on for the subscription diagrams.
Otherwise, I'm going to work on support for nested observable operators (window, groupBy, flat*, ...).  It occurred to me this morning that I can inject a method into the example apply methods which would allow them to capture and render child observable subscriptions the same as the input subscriptions are captured.  So you'd be able to write something like:
"flatMap": {
  inputs: [ ... ],
  apply: function (Rx, scheduler, capture) {
    return inputs[0].flatMap(x => capture(Rx.Observable.of(x).delay(10, scheduler)));
  }
}

You'd end up with the output diagram as well as the diagrams for each child observable.  Would make it really easy to understand the difference between the different flattening operators.

Let me know if you want me to make some of those changes. I'd rather leave the actual UX and UI event handling to you, but I can wire up the rendering to make it easier.

Sounds like a good plan.

If you like, I can add a property to the diagram component to control ghosting on a per-diagram basis. Then it could be turned off for all the regular diagrams and only on for the subscription diagrams.

That would be good.

Otherwise, I'm going to work on support for nested observable operators (window, groupBy, flat*, ...). It occurred to me this morning that I can inject a method into the example apply methods which would allow them to capture and render child observable subscriptions the same as the input subscriptions are captured.

For flatMap and similar, I want to render a diagram component inside the operator box, like this: http://reactivex.io/documentation/operators/flatmap.html
Rendering the child observable subscription isn't at important as the above.

Do you want that to be somehow data-driven?  Or more of a "const pre-defined" diagram specified in the example definition?

It should be visual, and above all, draggable just like other diagrams.

:cool:

@staltz @bman654 Hey guys, what happened here? This was a decent update :)

Disappointing that this project is abandoned. If the creator doesn't want to maintain it, he should hand off control to someone who will.


NoneType object is not callable when shutting down replay server

Anyone encountered this exception when stopping replay server?
[reverb/cc/platform/default/server.cc:63] Shutting down replay server
Exception ignored in: <function Server.del at 0x7f844a1f8c10>
Traceback (most recent call last):
File "/pythonProject/venv/lib/python3.8/site-packages/reverb/server.py", line 295, in del
TypeError: 'NoneType' object is not callable

Hey,
Would you mind running the following command in a terminal and posting the result to the issue?
awk 'NR>280&&NR310<' /pythonProject/venv/lib/python3.8/site-packages/reverb/server.py

Hey, this is what i got
awk: line 1: syntax error at or near end of line

Ops, sorry that was supposed to be:
awk 'NR>280&&NR<310' /pythonProject/venv/lib/python3.8/site-packages/reverb/server.py

if checkpointer is None:
checkpointer = checkpointers.default_checkpointer()
self._server = pybind.Server([table.internal_table for table in tables],
                             port, checkpointer.internal_checkpointer())
self._port = port

def del(self):
"""Stop server and free up the port if was reserved through portpicker."""
if hasattr(self, '_server'):
self.stop()
if hasattr(self, '_port'):
  portpicker.return_port(self._port)

def repr(self):
return repr(self._server)
@property
def port(self):
"""Port the gRPC service is running at."""
return self._port
def stop(self):
"""Request that the ReverbService is terminated and wait for shutdown."""
return self._server.Stop()
def wait(self):

Ok so the error is coming from the line portpicker.return_port(self._port). Not really sure why this is happening but it shouldn't really be causing any issues for you anyway since the message is coming from the garbage collection.
Just out of curiosity. What happens if you open up a python interpreter and do:
>>> import portpicker
>>> port = portpicker.pick_unused_port()
>>> port
23075
>>> portpicker.return_port(port)

?

Yeah it's not causing any issues just wanted to know what might be the cause.
Return_port isn't outputting anything so i guess that's why i get NoneType error, no clue why it's happening though. In any case thanks for your answer!

import portpicker
port = portpicker.pick_unused_port()
port
24178
portpicker.return_port(port)

Yeah I also have no idea why it is happening but I can't imagine this causing any issues (apart from noisy logs) so I'm going to close this issue down.
Thanks for raising the issue though!


Angular peer dependencies in non-Angular projects

Since we now have a significant number of dependencies that won't be needed for non-Angular projects, what do you think of updating the "Installation" section of README.md to provide more explicit guidance about using npm view vs npx install-peerdeps?
I'm thinking something like a note in the install-peerdeps section saying "For non-Angular projects, this will install a significant number of unneeded dependencies"
Moved from a comment from @jattasNI  in PR#40.

It has been identified that a future version of npm may install sub-peer dependencies which would support changing the organization of the rules package. @rajsite will verify the behavior, and then we'll make a proposal for packaging and documentation.

The process and validity test for npm v7's automatic peer dependency installation can be found in RFC 25.


Cannot navigate to /checkout/shipping from /checkout/customer with guest user

What is your question / Please describe your issue
I've installed vendure and added vue-storefont as described in docs.
From the first glance everything works as expected, but i can't navigate to shipping step with unauthorized user.
When i filled in name/email in /checkout/customer route, and clicked "Continue Shipping" button, I'am redirected to Home page.
Steps to reproduce the behavior:

Add item to basket

Go to '/checkout/customer'

Fill in user name and email

Click 'Continue Shipping'

In api request setCustomerForOrder there is error:

{ "errorCode": "NO_ACTIVE_ORDER_ERROR", "message": "There is no active Order associated with the current session", "__typename": "NoActiveOrderError" }
Environment :
@vue-storefront/template-vendure: 1.0.0
@vendure/core version: 1.4.5
Nodejs version: v14.16.1
Database : postgres
What version of vendure are you using?
0.1.0
Code of Conduct

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

Restarted server and now it works.


[freetype] provide copy of 3rd party library freetype binary's

Hello.
As examples required to link with 3d party library freetype it is be good to copy binary of this library into bin directory of current example.
Thanks.

Well, it's not specified in readme how to compile Freetype library in order to use it in lantern, but it's assumed that it's compiled as a static library, otherwise you'll have to change cmake file a little (like you did)
For now I don't see any point in changing this behaviour since it's more like "one-person" project
I'll consider this if this will change in future
Anyway, thanx for pointing this out, I'll add a note about it in readme file

At first time I used binary package freetype-2.3.5-1-bin.zip but soon also check to do same with source. So by the way, it also possibility to check to what type of package - source or binary, set path at environment variable. For MinGW and Visual Studio I do such checking
But making from source required to put also config file with applied advice.
Ok. Let it be.


ocamlPackages.csexp: fix minimum OCaml version

Motivation for this change
Source: https://opam.ocaml.org/packages/csexp/csexp.1.3.2/
Things done

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

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

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

@GrahamcOfBorg build ocaml-ng.ocamlPackages_4_02.csexp

Apologies for not having done a more thorough test on this. I thought the OCaml 4.08 requirement was just about Dune 1 vs Dune 2. I now realize there is also the issue of the result package. I'll update result to 1.5 (which is compatible for OCaml >= 4.08) and remove the postPatch step here.

@GrahamcOfBorg build ocamlPackages.dune-configurator ocaml-ng.ocamlPackages_4_02.csexp ocaml-ng.ocamlPackages_4_08.csexp

@GrahamcOfBorg build dot-merlin-reader

@GrahamcOfBorg build ocaml-ng.ocamlPackages_4_02.dune-configurator

There is something wrong about dune-configurator: probably a missing minumumOCamlVersion.

Yes, will fix in https://github.com/NixOS/nixpkgs/pull/99265

Thanks!


[Feature]: Add MINTraketen award to projects

Create MINTraketen award
Upload image for award on stage and get the hashed folder path
Transfer this image to prod
Add award to MINTraketen projects through database operation

[ ] Create MINTraketen award
[ ] Add it to the MINTraketen projects


also remove function

@cheungpat, thanks for your PR! By analyzing the annotation information on this pull request, we identified @rickmak to be a potential reviewer


use scalafixAll

Uses https://github.com/scalacenter/sbt-scalafix/pull/126
This slightly improves runtime as migrations can be run in parallel for Compile and Test.
Note that this will NOT run migrations on IntegrationTest or any other custom configuration, as Scalafix needs to be enabled explicitly on them.

Note that this will NOT run migrations on IntegrationTest or any other custom configuration, as Scalafix needs to be enabled explicitly on them.

Maybe slightly off topic, but I wonder if there is a demand for that in the context of scala-steward? If so, it could be interesting to provide a scalafixEnableAll.

Maybe slightly off topic, but I wonder if there is a demand for that in the context of scala-steward? If so, it could be interesting to provide a scalafixEnableAll command.

Nobody has asked for this yet or complained about it but I guess it makes sense to have something like scalafixEnableAll so that Scala Steward migrates all sources in a project and not only the sources of the Compile and Test configuration.

FTR https://github.com/scalacenter/scalafix/issues/1197


feat: optionally use token from env, allow selecting repos

When prompting the user for a token give the option of using a token from the env ($GITHUB_TOKEN)
Allow selecting all, or some repositories

Then if this gets merged - this might be another little fix - https://github.com/domdomegg/github-repo-settings-editor/pull/3

Totally agree with all your comments @domdomegg , thanks so much for taking the time to write them, I'll cut up a cleaned up version soon 😄

closing for now as have become insanely busy all of a sudden, might completely redo this in the future.


containerapp allow yaml without environment

This checklist is used to make sure that common guidelines for a pull request are followed.
Related command

When --environment is provided and the environmentId value does not exist in --yaml, the value in --environment is used.
When the environmentId value existed in --yaml, use the environmentId in --yaml.
az containerapp create --yaml --environment
az containerapp job create --yaml --environment

General Guidelines

[ ] Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
[ ] Have you run python scripts/ci/test_index.py -q locally?

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

Thank you for your contribution! We will review the pull request and get back to you soon.

Hi @zhoxing-ms
This PR has been reviewed by zunli, could you please help to review and merge? Thanks a lot.


Implement Methods from interface in php language

It would be much better to have this feature as it works well on PHPStorm IDE

Our builtin PHP support is very basic, but this would be a good feature for an extension to implement


Rails 5 issue with boolean type

I've got an error when using the boolean typecast within Rails 5.0.0.beta3. All other types (string, datetime) work very well.
NoMethodError: undefined method `type_cast_from_user' for #<ActiveModel::Type::Boolean:0x007fc070def5c0>
	from /usr/local/var/rbenv/versions/2.3.0/lib/ruby/gems/2.3.0/gems/hstore_accessor-1.0.3/lib/hstore_accessor/active_record_4.2/type_helpers.rb:27:in `cast'
	from /usr/local/var/rbenv/versions/2.3.0/lib/ruby/gems/2.3.0/gems/hstore_accessor-1.0.3/lib/hstore_accessor/serialization.rb:26:in `block in <module:Serialization>'
	from /usr/local/var/rbenv/versions/2.3.0/lib/ruby/gems/2.3.0/gems/hstore_accessor-1.0.3/lib/hstore_accessor/serialization.rb:45:in `deserialize'
	from /usr/local/var/rbenv/versions/2.3.0/lib/ruby/gems/2.3.0/gems/hstore_accessor-1.0.3/lib/hstore_accessor/macro.rb:76:in `block (3 levels) in hstore_accessor'
	from /usr/local/var/rbenv/versions/2.3.0/lib/ruby/gems/2.3.0/gems/hstore_accessor-1.0.3/lib/hstore_accessor/macro.rb:80:in `block (3 levels) in hstore_accessor'
	from (irb):7
	from /usr/local/var/rbenv/versions/2.3.0/lib/ruby/gems/2.3.0/gems/railties-5.0.0.beta3/lib/rails/commands/console.rb:65:in `start'
	from /usr/local/var/rbenv/versions/2.3.0/lib/ruby/gems/2.3.0/gems/railties-5.0.0.beta3/lib/rails/commands/console_helper.rb:9:in `start'
	from /usr/local/var/rbenv/versions/2.3.0/lib/ruby/gems/2.3.0/gems/railties-5.0.0.beta3/lib/rails/commands/commands_tasks.rb:78:in `console'
	from /usr/local/var/rbenv/versions/2.3.0/lib/ruby/gems/2.3.0/gems/railties-5.0.0.beta3/lib/rails/commands/commands_tasks.rb:49:in `run_command!'
	from /usr/local/var/rbenv/versions/2.3.0/lib/ruby/gems/2.3.0/gems/railties-5.0.0.beta3/lib/rails/command.rb:20:in `run'
	from /usr/local/var/rbenv/versions/2.3.0/lib/ruby/gems/2.3.0/gems/railties-5.0.0.beta3/lib/rails/commands.rb:18:in `<top (required)>'
	from bin/rails:9:in `require'
	from bin/rails:9:in `<main>'

Thanks!

Hey @marcusg There are a number of changes to ActiveRecord for Rails 5.0. We're working on getting something cooked up. In the meantime, use at your own risk.

Would love to see this gem become rails 5 compatible. It's the last gem on my list :)

I have the same problem! Any update?

We released a beta 1.0 branch which Rails 5 compatibility a couple weeks ago. Please check it it out. We'd love you feedback.

On Nov 22, 2016, at 7:24 AM, Gerardo Ortega<EMAIL_ADDRESS>wrote:
I have the same problem! Any update?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Hi @jhirn, I already tried it, but I had the same issue, then I tried using the fork of @hanselmw, and It's working well

Ah I'm sorry I answered this early in the morning and realize this is for
hstore_accessor. We've paid much attention to this gem since we're mostly
using jsonb_accessor these days.
I'll take a look at the pull request from @hanselmw soon and get it merged
in if all looks well.
On Tue, Nov 22, 2016 at 11:22 AM, Gerardo Ortega<EMAIL_ADDRESS>wrote:

Hi @jhirn https://github.com/jhirn, I already tried it, but I had the
same issue, then I tried using the fork of @hanselmw
https://github.com/hanselmw, and It's working well
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/devmynd/hstore_accessor/issues/69#issuecomment-262305756,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAKMsRW24blwNc2zPHTYJFev4NimnoVIks5rAyTvgaJpZM4IJqSA
.

Hi, it would be wonderful to see an update to support Rails 5. Love this gem.

Bump. Please :)

That PR seems to work. Please merge and release a new version. Thanks

@jhirn any timeframe on landing that PR?

v1.1 was just released. All of the tests pass against ActiveRecord 5.0 and 5.1.


fix: evm fee drawer correcly updates tx after changing fee strat (aligned with BTC)

✅ Checklist

[x] npx changeset was attached.
[ ] Covered by automatic tests.
[ ] Impact of the changes:

...

📝 Description
BTC does this https://github.com/LedgerHQ/ledger-live/blob/1f7406327583cb61944cab18023e966b5ac467aa/apps/ledger-live-desktop/src/renderer/families/bitcoin/SendAmountFields.tsx#L92-L97
Imo it is better than prepareTransaction again after and refetch gasValues
❓ Context
- JIRA or GitHub link:

🧐 Checklist for the PR Reviewers

The code aligns with the requirements described in the linked JIRA or GitHub issue.
The PR description clearly documents the changes made and explains any technical trade-offs or design decisions.
There are no undocumented trade-offs, technical debt, or maintainability issues.
The PR has been tested thoroughly, and any potential edge cases have been considered and handled.
Any new dependencies have been justified and documented.
Performance considerations have been taken into account. (changes have been profiled or benchmarked if necessary)

closing. transactions should be prepared after each update.


make fails on git source

Builing from git source fails:
$ git clone https://github.com/ncw/rclone.git
$ cd rclone
$ make
touch fs/version.go
go install -v --ldflags "-s -X github.com/ncw/rclone/fs.Version=v1.36-47-gd86ea86"
rclone.go:9:2: cannot find package "github.com/ncw/rclone/cmd" in any of:
    /usr/lib/go-1.6/src/github.com/ncw/rclone/cmd (from $GOROOT)
    ($GOPATH not set)
rclone.go:10:2: cannot find package "github.com/ncw/rclone/cmd/all" in any of:
    /usr/lib/go-1.6/src/github.com/ncw/rclone/cmd/all (from $GOROOT)
    ($GOPATH not set)
rclone.go:11:2: cannot find package "github.com/ncw/rclone/fs/all" in any of:
    /usr/lib/go-1.6/src/github.com/ncw/rclone/fs/all (from $GOROOT)
    ($GOPATH not set)
Makefile:15: recipe for target 'rclone' failed
make: *** [rclone] Error

I had expected either the system would build or it would tell me how to install the missing packages. This information could also be in the file INSTALL.
May I suggest you try building on a newly installed distribution? This will make it clear which packages, software and environment settings you depend on for building. Then document this in INSTALL or make the makefile more intelligent.
This is similar to #1395.

Yo need to setup your $GOPATH in order to build from source.
See https://golang.org/doc/code.html#Workspaces for more details.

In the binary package I found README.txt (It should also be in the source package and in git):
$ go get -u -v github.com/ncw/rclone
github.com/ncw/rclone (download)
package fmt: unrecognized import path "fmt" (import path does not begin with hostname)
package bufio: unrecognized import path "bufio" (import path does not begin with hostname)
[... many more ...]

@sjurtf If setting $GOPATH is needed it should say so in the installation instructions. Non-go-developers should be able to compile from source, just like non-C-programmers can compile most C programs if they follow the instructions.
I have now tried:
$ GOPATH=$HOME/gopath
$ export GOPATH
$ go get -u -v github.com/ncw/rclone
github.com/ncw/rclone (download)
package fmt: unrecognized import path "fmt" (import path does not begin with hostname)
package bufio: unrecognized import path "bufio" (import path does not begin with hostname)
package bytes: unrecognized import path "bytes" (import path does not begin with hostname)

You need to set your GOPATH - all go packages need that.
I'd suggest
mkdir ~/go
export GOPATH=~/go
go get -u -v github.com/ncw/rclone

Like it says in the installing from source section in the docs.

The error

package fmt: unrecognized import path "fmt" (import path does not begin with hostname)

Indicated you've set GOROOT I think.  Do unset GOROOT and all should be fine hopefully.

That solved it.
May I suggest the above either goes into github.com/ncw/rclone/INSTALL.txt or that the Makefile does that?

Well, it does say in the documentation how to do it.
https://rclone.org/install/#install-from-source

At the very least split MANUAL.txt into:
INSTALL (with all instructions for installation)
Changelog (with the changelog)
LICENSE (with the license)
FileFormat (with the file format description)
Also consider formatting rsync.1 to look like a normal man page.

I have the same problem with compiling from sources, when I try to run go build - I got the errors:
$ go build
rclone.go:7:2: cannot find package "github.com/ncw/rclone/backend/all" in any of:
        /usr/lib/go-1.10/src/github.com/ncw/rclone/backend/all (from $GOROOT)
        /home/murz/go/src/github.com/ncw/rclone/backend/all (from $GOPATH)
rclone.go:8:2: cannot find package "github.com/ncw/rclone/cmd" in any of:
        /usr/lib/go-1.10/src/github.com/ncw/rclone/cmd (from $GOROOT)
        /home/murz/go/src/github.com/ncw/rclone/cmd (from $GOPATH)
rclone.go:9:2: cannot find package "github.com/ncw/rclone/cmd/all" in any of:
        /usr/lib/go-1.10/src/github.com/ncw/rclone/cmd/all (from $GOROOT)
        /home/murz/go/src/github.com/ncw/rclone/cmd/all (from $GOPATH)

If I change $GOPATH to my custom folder location (/home/murz/rclone-from-git) - go continue lookup packages into /home/murz/rclone-from-git/src/github.com/ncw/rclone/backend/all subfolder instead of /home/murz/rclone-from-git/backend/all.
How can I fix this problem?

This issue seems very obsolete already. rclone has switched to go mod and you'll need go 1.11 or above to build it. Also, you don't need to touch $GOPATH any more.

Small note that the Install documentation states that "Make sure you have at least Go 1.7 installed". I installed go using apt-get install golang-go which installed version 1.10.4.
I saw the same errors as MurzNN ("cannot find package" 3 times). Running go get -u -v github.com/ncw/rclone worked for me.
Should the documentation state that you need go 1.11?
(Sorry, I have no knowledge about go -- just my personal experience above trying to compile from source) :-) Cheers

Small note that the Install documentation states that "Make sure you have at least Go 1.7 installed". I installed go using apt-get install golang-go which installed version 1.10.4 (so I thought I was ok).
I saw the same errors as MurzNN ("cannot find package" 3 times). Running go get -u -v github.com/ncw/rclone worked for me.
Should the documentation state that you need Go 1.11?
(Sorry, I have no knowledge about Go -- just my personal experience above trying to compile from source) :-) Cheers

Go 1.10 is a supported target for rclone (see the travis builds: https://travis-ci.org/ncw/rclone) so it should work, however you will need to unpack rclone in the GOPATH.  The easiest way is to unpack rclone at ~/go/src/github.com/ncw/rclone as the default GOPATH is ~/go.
One of the aims of go mod was to remove this friction setting GOPATH.


nf22: Add README.md and config.json for roadway impact detector.

Looks good. You might want to add links to the relevant API docs too.

Looks good. You might want to add links to the relevant API docs too.

Sure. Do you just mean links for things like card.motion.sync etc.?

Looks good. You might want to add links to the relevant API docs too.

Sure. Do you just mean links for things like card.motion.sync etc.?

Yep. Take a look at NF20 for an example.


Error while making video using brainrender <IP_ADDRESS>: AttributeError: 'NoneType' object has no attribute 'Elevation'

Hello,
I am using brainrender Version: <IP_ADDRESS> within jupyter notebook inside Chrome.
I am unable to recreate the video example mentioned in old Github "examples" repository [here].(https://github.com/brainglobe/brainrender/blob/4c087bc4cb8f94ba394dc0cf75baa646065ea44b/Examples/advanced/videomaker.py)
Following is my code:
import brainrender
brainrender.SHADER_STYLE = 'cartoon'
from brainrender.scene import Scene
from brainrender.animation.video import BasicVideoMaker as VideoMaker

scene = Scene()
scene.add_brain_regions(['RN'], color="violet")

scene.add_cells_from_file("C:\\Users\\7057mehrav\\Documents\\164-test\\cells_in_standard_space.h5", radius=15,alpha=0.5)

# Create an instance of VideoMaker with our scene
vm = VideoMaker(scene, niters=30)  # niters = number of frames

# Make a video!
vm.make_video(
    elevation=1, roll=5
)  # specify how the scene rotates at each frame

After promising messages, I get an error:
Bg_atlasapi: allen_mouse_25um version 1.1 is not the latest available (1.2). To update the
atlas run in the terminal:
    brainglobe update -a allen_mouse_25um
allen mouse atlas (res. 25um)
From: http://www.brain-map.com (Wang et al 2020, https://doi.org/10.1016/j.cell.2020.04.007 )
Setting 'display_inset' to False as this feature is not                             available in juputer notebooks
Setting window size to 'auto' as whole screen is not available in jupyter
Added 15473 cells to the scene
Saving video in C:\Users\7057mehrav\.brainrender\Output/Videos
('\x0bideo Video', 'brainrender_video__20201203_123641', 'is open...')
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
<ipython-input-4-3ee3c505a465> in <module>
     15 # Make a video!
     16 vm.make_video(
---> 17     elevation=1, roll=5
     18 )  # specify how the scene rotates at each frame

c:\users\7057mehrav\appdata\local\continuum\anaconda3\envs\brainrender\lib\site-packages\brainrender\animation\video.py in make_video(self, azimuth, elevation, roll, **kwargs)
    144         for i in range(self.niters):
    145             self.scene.plotter.show()  # render(interactive=False, video=True)  # render the scene first
--> 146             self.scene.plotter.camera.Elevation(elevation)
    147             self.scene.plotter.camera.Azimuth(azimuth)
    148             self.scene.plotter.camera.Roll(roll)

AttributeError: 'NoneType' object has no attribute 'Elevation'

What could be causing this? I have already looked in the location where it tries to save the output but that folder is empty. Is it because I am running this functionality from within jupyter?

Hi, thanks for the report.
The short answer to this

Is it because I am running this functionality from within jupyter?

is yes.
Running brainrender in notebooks can incur into limitation, as discussed in several issues and in the documentation
You can see if setting up brainrender to pop up the rendering window (see instructions here) helps, but the example linked is for the latest brainrender 2.0 so you might have to adjust it to make it work with the older version you're using.
A simpler solution is just to not use a notebook, this error should disappear if you run brainrender from a script

Thanks for the response @FedeClaudi
Unfortunately, running with the script also doesn't work. I do get a Win32 Open GL window but after few seconds, it crashes. Attached is the image of that window. Can we simply save the video and not open it?

Can we simply save the video and not open it?

With brainrender 2.0 you can, in fact it does that by default.
Do you get any error, anything happens before it crashes?
Can you render the scene normally?
Is there any reason against moving to brainrender 2.0? The video an animation functionality is much improved now and the transition is almost effortless

Can you render the scene normally?

Same error with rendering also when running the script (which works in jupyter using Plotter). No specific error. After this piece of info:
Bg_atlasapi: allen_mouse_25um version 1.1 is not the latest available (1.2). To update the atlas run in the terminal:
    brainglobe update -a allen_mouse_25um
allen mouse atlas (res. 25um)
From: http://www.brain-map.com (Wang et al 2020, https://doi.org/10.1016/j.cell.2020.04.007 )
Added 15473 cells to the scene
Saving video in C:\Users\7057mehrav\.brainrender\Output/Videos
('\x0bideo Video', 'brainrender_video__20201203_150342', 'is open...')

just a  "python has stopped working" dialog box

Is there any reason against moving to brainrender 2.0?

Not a huge reason. Just wanted to keep stuff consistent with stuff I have already created for a website I am working on. I will give it a shot now. Congratulations for the new version!

Thanks, I hope you'll find is much improved and that hopefully the transition is relatively painless.
Given your error message, are you rendering after creating the video? Can you try rendering intead of creating the video?
If that's still a problem perhaps you can try without adding the cells to the scene. It's not a very large number of cells you're displaying, but if you are running window 32bit perhaps that's a bit too much?

Not a huge reason. Just wanted to keep stuff consistent with stuff I have already created for a website I am working on. I will give it a shot now. Congratulations for the new version!

This is so cool! 🎉
Let me know how that goes, if you can. I've been curious about the possibility of having brainrender being used for online content for a while. And happy to chat/add new features that could make your life a bit easier with that.

Can you try rendering intead of creating the video?

Yes, unfortunately did not work.

Let me know how that goes, if you can.

Sure, will let you know when website goes live.

mmm, weird.
Can you give me some more info, what OS are your using, what version of python, brainrender and vedo and any output that you get while trying to render the scene. Thanks.
Also, I suppose that you have been able to get brainrender to work before, did anything change?
Can you get any of the example scripts to run?

I just tried this video example on latest brainrender version. My anaconda prompt is just stuck with this.

Windows 10
Python - 3.7.9
Name: vedo
Version: 2020.4.2

Name: brainrender
Version: <IP_ADDRESS>

I am running brainrender from within a conda environment.

I was able to make the latest brainrender's video example work on my personal Macbook. Although support for loading cells from h5 file has been removed?  :(

Hey,
I see. I have no idea why it would work on mac but not on windows. I suppose that you have ffmpeg correctly installed in both machines?
On windows, can you get any of the other examples to work? Can you please check which version of vtk you have?

p.s.: There is a typo in the error (in bold , it should be "not"):

Thanks, fixed.

Although support for loading cells from h5 file has been removed? :(

That's right, sorry. The previous approach required users to have dataframes with specific column names, using numpy arrays instead of dataframes should give more flexibility to the users. It does mean that you have to load the file directly though.
A simple function would take care of this
def load_cells(path):
    return pd.read_hdf(path).values

assuming that your dataframe only has 3 columns and these correspond to the coordinates.

Hi,
If you use embedWindow(None) video generation in Jupyter notebooks works:

hey @rdbcasillas did you get a chance to try this out?
Everything seems to work okay at my end, but I wanted to make sure that your problem is fixed!

Hey @FedeClaudi,
I haven't tested it yet because I am still working with h5 files which new version doesn't support. We created a lot of h5 files using cellfinder's function "points_to_brainrender" which converts xml output from cellfinder to h5. So we will have to continue using older version of brainrender for a while.
Thanks!

I see, okay.  To use the .h5 files you just need a function that loads the file and gets the coordinates :)
def load_cells(path):
    return pd.read_hdf(path).values

Anyway, I'll close this for now then but let me know if you bump into other problems


CVAT not starting due to timeout error wait-for-it.sh: waiting for cvat_db:5432 without a timeout

Actions before raising this issue

[x] I searched the existing issues and did not find anything similar.
[X] I read/searched the docs

Steps to Reproduce
1- Set the CVAT_HOST environment variable:
export CVAT_HOST=MY-IP-ADDRESS
2- Run the following command to bring up the containers:
docker compose up -d
Expected Behavior
No response
Possible Solution
No response
Context
Hello,
I'm encountering an issue where CVAT is not starting on my PC. It seems to be stuck with the following error message:
wait-for-it.sh: waiting for cvat_db:5432 without a timeout
I have already tried restarting the application and checking my Docker setup, but the problem persists. It looks like the database connection is timing out, and CVAT is unable to initialize properly.
Has anyone else experienced this issue or knows how to resolve it? Any guidance or advice would be greatly appreciated.
Environment
OS: Ubuntu 20.04
Docker version: 27.3.1, build ce12230
CVAT version: v2.19.0

Any logs from cvat_db?

Thank you for your response! After further investigation, I realized the issue was likely caused by a problem with my local setup. I decided to reinstall Ubuntu on my machine, and after doing so, the issue no longer occurs. Everything seems to be working fine now. Apologies for the confusion, and thank you again for your time and support!


Command line arguments specification with '=' doesn't work

Example:

Doesn't work:

ttyper --language-file=~/.config/ttyper/words

Executes well:

ttyper --language-file ~/.config/ttyper/words

Could you please elaborate on why you want that = feature?

Because this is common behaviour for a lot of console applications. For example, podman, or gcc (first things that came to my mind).  I think that many people are already used to it.

ttyper does support this option syntax. This example doesn't work because your shell isn't expanding the ~ character to your home directory. If you try with an absolute path it will work.
It is possible to make this corner case work by expanding tildes within ttyper, but I'm not a huge fan of doing this because

relative file paths beginning with ~ are valid, so there would have to be some bespoke, nonstandard way to escape these file paths.
This is generally understood to be the shell's responsibility.

Ok, now I get it. Didn't think of it this way. Thanks!


Extract absolutize_relative_to to scolapasta-path

This path normalization utility lives in artichoke-backend.
https://github.com/artichoke/artichoke/blob/2fe83803834963deef45e5fa8a295188902b72b7/artichoke-backend/src/load_path.rs#L66-L246
Move it and its tests to scolapasta-path. Does this need more tests?
I'm not convinced that this function is the best way to do this and maybe we can delegate to a third party crate. Some requirements:

Does not rely on std::fs::canonicalize because this routine may be used on paths in virtual filesystems.
Correctly handles various Windows path prefixes including verbatim paths: https://doc.rust-lang.org/stable/std/path/enum.Prefix.html
Produces paths with / as the path separator (for non-verbatim paths on Windows). This can be done by collecting the Components into an OsString with Component::as_os_str.

It looks like the current implementation is missing tests that the string representation of the path has / path separators on Windows.

As we do this extraction, let's do the trick that minimizes the code bloat from monomorphization while still getting the ergonomics with AsRef<Path> and restructure the code like:
pub fn absolutize_relative_to<T, U>(path: T, cwd: U) -> PathBuf
where
    T: AsRef<Path>,
    U: AsRef<Path>,
{
    absolutize_relative_to_inner(path.as_ref(), cwd.as_ref())
}

fn absolutize_relative_to_inner(path: &Path, cwd: &Path) -> PathBuf {
    todo!();
}

Hello @lopopolo .
I am aware of a Windows-specific requirement for this issue - I am sadly not in the position to check it because I lack access to any windows machines (all of my machines are aarch64).
Let me know if there's something else I can do to either fulfil the requirement or otherwise compensate for the lack of it.

Thanks @NlightNFotis. I can take a look at the windows bits but it probably won't be until the new year. Happy holidays and thanks for the PR.


[FEAT] /docs route to main site

This is a terrible code and I don't even know if it should exist in production but this is the best I could do

OH MY GOD AIN'T NO WAY


Update compute-api.json and compute-gen.go

There are new features on API v1, such as Instance setLabels (https://cloud.google.com/compute/docs/reference/v1/instances/setLabels), but to use it compute-api.json and compute-gen.go need to be updated.

I believe that's been in the actual repo for a couple of days at least (so go get -u should have fetched it).
I just mirrored to this repo.


[occm]: install python3-pip on focal

Task: https://logs.openlabtesting.org/logs/84/1384/a2a6c42d771b29e4a364b78145904ec971e28072/cloud-provider-openstack-acceptance-test-keystone-authentication-authorization/cloud-provider-openstack-acceptance-test-keystone-authentication-authorization/bba5fc1/job-output.txt.gz
Error:
2021-02-09 08:58:40.811199 | ubuntu-focal | + apt-get install python-pip -y
2021-02-09 08:58:40.874150 | ubuntu-focal | Reading package lists...
2021-02-09 08:58:40.989569 | ubuntu-focal | Building dependency tree...
2021-02-09 08:58:40.990325 | ubuntu-focal | Reading state information...
2021-02-09 08:58:41.104459 | ubuntu-focal | E: Unable to locate package python-pip
2021-02-09 08:58:41.507686 | ubuntu-focal | ERROR

@wangxiyuan another fix for a package name

@wangxiyuan thanks. we're close to success :) manila csi jobs are already green :tada: https://github.com/kubernetes/cloud-provider-openstack/pull/1384#issuecomment-775785797


Need to allow multiple PDB models for comparison

Need to allow up to 2 PDB files as input, which could then be switched to from a button within iris

Fixed in #11


[bn] Localize for client-server-architecture.md

Signed-off-by: Sajib Adhikary<EMAIL_ADDRESS>Localizes content/bn/client-server-architecture.md  to Bengali
This is an initial PR as it may need some improvements!
Tracking issue: #410

We are going to merge some changes to dev-bn. As soon as the changes go live, we will merge this file.  #1688

Why lines are not together? Where is there so many space?

Because the lines in the original file content/en/client-server-architecture.md were not connected. So I tried to maintain the structure of the original file.


Clean Hansard Debate Titles

First install tidyverse on M2:
install.packages("tidyverse")

Use this code to access Hansard debate titles on M2:
library(tidyverse)

hansard <- read_csv("/scratch/group/pract-txt-mine/hansard_justnine_12192019.csv")

hansard <- hansard %>%
  select(sentence_id, debate, text)

debate_titles <- hansard %>%
  distinct(debate)

@HaileyHazen -- let's catch up on where you are with this. Schedule a brief meeting with me for next week?

Yes, I can definitely schedule a meeting regarding this project next week. When are you available?


Dealer

I would like to submit this for inclusion in the Roll20 repository, and One-Click install if possible.
This script deals and takes cards to selected player from specified decks.
Syntax is !deal --[give,take] [number of cards as integer] --[deck name]

I have updated the script. How does this worK? It now begins with:
// Dealer
// Last Updated: 2019-08-16
// A script to deal and take cards to selected users from specified decks.
// Syntax is !deal --[give,take] [number of cards as integer] --[deck name]
log('-=> Dealer v' + version + ' <=-');
on('chat:message', (msg) => {
if ('api' === msg.type && /!deal\b/i.test(msg.content) && msg.selected) {
const version = '1.0.0';

On Aug 18, 2019, at 6:17 AM, Jakob Oesinghaus<EMAIL_ADDRESS>wrote:
@joesinghaus commented on this pull request.
In Dealer/Dealer.js https://github.com/Roll20/roll20-api-scripts/pull/850#discussion_r314985164:

@@ -0,0 +1,130 @@
+const version = '1.0.0';
Hi Keith,
you should probably move this part into some inner scope (like an on('ready') callback). Variables in the outermost scope are shared between all API scripts, so no other script could define a "version" variable (they also shouldn't do that, but better to be defensive here) and the API would fail with a syntax error.

Another thing, it's a good idea to begin your script with a comment due to the way all scripts are concatenated.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/Roll20/roll20-api-scripts/pull/850?email_source=notifications&email_token=AF7RO4HJPBXTSHZ75YPZNWDQFFDU7A5CNFSM4IMOLRNKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCB32XDQ#pullrequestreview-276278158, or mute the thread https://github.com/notifications/unsubscribe-auth/AF7RO4GH2E6AUM5I7EL3FHLQFFDU7ANCNFSM4IMOLRNA.

I have gone back and wrapped them into an on(ready handler and tested the
code. Hopefully it will make the cut now.
Thanks!
Keith
On Sun, Aug 18, 2019 at 6:38 PM kac<EMAIL_ADDRESS>wrote:

I have updated the script. How does this worK? It now begins with:
// Dealer
// Last Updated: 2019-08-16
// A script to deal and take cards to selected users from specified decks.
// Syntax is !deal --[give,take] [number of cards as integer] --[deck
name]
log('-=> Dealer v' + version + ' <=-');
on('chat:message', (msg) => {
if ('api' === msg.type && /!deal\b/i.test(msg.content) && msg.selected) {
const version = '1.0.0';
On Aug 18, 2019, at 6:17 AM, Jakob Oesinghaus<EMAIL_ADDRESS>wrote:
@joesinghaus commented on this pull request.
In Dealer/Dealer.js
https://github.com/Roll20/roll20-api-scripts/pull/850#discussion_r314985164
:

@@ -0,0 +1,130 @@
+const version = '1.0.0';

Hi Keith,
you should probably move this part into some inner scope (like an
on('ready') callback). Variables in the outermost scope are shared between
all API scripts, so no other script could define a "version" variable (they
also shouldn't do that, but better to be defensive here) and the API would
fail with a syntax error.
Another thing, it's a good idea to begin your script with a comment due to
the way all scripts are concatenated.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Roll20/roll20-api-scripts/pull/850?email_source=notifications&email_token=AF7RO4HJPBXTSHZ75YPZNWDQFFDU7A5CNFSM4IMOLRNKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCB32XDQ#pullrequestreview-276278158,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF7RO4GH2E6AUM5I7EL3FHLQFFDU7ANCNFSM4IMOLRNA
.

This will work! I would do it like this to start everything with a comment:
// Dealer
// Last Updated: 2019-08-16
// A script to deal and take cards to selected users from specified decks.
// Syntax is !deal --[give,take] [number of cards as integer] --[deck name]
on('ready',()=>{

Easily remedied, done. Thanks, I'll keep this formatting in mind for any
future submissions.
Keith
On Mon, Aug 19, 2019 at 12:54 PM Jakob Oesinghaus<EMAIL_ADDRESS>wrote:

This will work! I would do it like this to start everything with a comment:
// Dealer
// Last Updated: 2019-08-16
// A script to deal and take cards to selected users from specified decks.
// Syntax is !deal --[give,take] [number of cards as integer] --[deck name]
on('ready',()=>{
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Roll20/roll20-api-scripts/pull/850?email_source=notifications&email_token=AF7RO4GVP6YYJO5XQU2MZDTQFL26VA5CNFSM4IMOLRNKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4UDMBY#issuecomment-522728967,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF7RO4FA42O2XWTRCM74J7DQFL26VANCNFSM4IMOLRNA
.


Change the Title Color

How can I change the titles Color ?
Thanks
Ben

Hi Ben,
You can change the colour by applying a class to the axis and then using CSS to target the element whose colour you want to change.

Hi Alex, i've tried to use this code in my CSS :
text.ct-axis-title {     color: white; }

@ilhamsr
You should use fill instead of color as you are changing an svg element.
Like:
text.ct-axis-title { fill: white; }


Snyk not being triggered in Artifactory 7?

Hi!
I'm aware that official support is only for AF6, but I was told by our contact at Jfrog, that AF7 should be compatible as well. Hence, we are trying to run the plugin in AF7, and so far it seems to install correctly. However trying to trigger a scan does not seem to work. I'm not sure whate the conditions are to run a scan, but I'm trying to download a vulnerable npm package version from one of our cached remote repos via CLI. This is the the log output Artifactory for that download:
2021-02-08T11:14:01.278Z [jfrt ] [INFO ] [6324ae19bf9a54a1] [o.a.r.HttpRepo:443            ] [ttp-nio-8081-exec-15] - registry-npmjs-org downloading https://registry.npmjs.org/jquery Unknown content length
2021-02-08T11:14:01.336Z [jfrt ] [INFO ] [6324ae19bf9a54a1] [o.a.r.HttpRepo:456            ] [ttp-nio-8081-exec-15] - registry-npmjs-org downloaded  https://registry.npmjs.org/jquery 111.99 KB at 1,957.44 KB/sec
2021-02-08T11:14:01.813Z [jfrt ] [INFO ] [5dc7ce63cd6b2bb6] [o.a.r.HttpRepo:443            ] [ttp-nio-8081-exec-45] - registry-npmjs-org downloading https://registry.npmjs.org/jquery/-/jquery-3.3.1.tgz 405.64 KB
2021-02-08T11:14:01.898Z [jfrt ] [INFO ] [5dc7ce63cd6b2bb6] [o.a.r.HttpRepo:456            ] [ttp-nio-8081-exec-45] - registry-npmjs-org downloaded  https://registry.npmjs.org/jquery/-/jquery-3.3.1.tgz 405.64 KB at 4,771.24 KB/sec
2021-02-08T11:14:01.964Z [jfrt ] [WARN ] [5dc7ce63cd6b2bb6] [i.s.p.a.s.ScannerModule:53    ] [ttp-nio-8081-exec-45] - Artifact 'registry-npmjs-org:jquery/-/jquery-3.3.1.tgz' will not be scanned, because the extension is not supported

I'm not sure if the last line mentioning the file not being scanned is coming from the Snyk plugin and if it is, I'm a bit confused as to why it would not scan here.
The plugin seemingly is loaded correctly:
2021-02-05T16:23:16.624Z [jfrt ] [INFO ] [156aacebc696052f] [o.a.a.p.PluginsAddonImpl:119  ] [ttp-nio-8081-exec-15] - Reloading plugins
2021-02-05T16:23:16.716Z [jfrt ] [INFO ] [156aacebc696052f] [o.a.a.p.GroovyRunnerImpl:313  ] [ttp-nio-8081-exec-15] - Loading script from 'snykSecurityPlugin.groovy'.
2021-02-05T16:23:21.158Z [jfrt ] [INFO ] [156aacebc696052f] [.a.a.p.e.ExecutePluginImpl:189] [ttp-nio-8081-exec-15] - Groovy execution 'snykSecurityReload' has been successfully registered.
2021-02-05T16:23:21.213Z [jfrt ] [INFO ] [156aacebc696052f] [o.a.a.p.GroovyRunnerImpl:357  ] [ttp-nio-8081-exec-15] - Script 'io.snyk.plugins.artifactory.snykSecurityPlugin' loaded.

Any pointers as to what goes wrong here?
Is there a test that I can run to see if the plugin works correctly?

Hey @bt-nia,
at the moment our plugin doesn't scan artifacts in AF7, we're working on it. I'll ping you back as soon we have a new release with AF7 support.
Thanks, Pavel.

Is scanning explicitly turned off for AF7, hence the message or is this an incompatibility bug triggered because of the version differences?

There are internal API changes between A6 and A7.
Could you share how your registry-npmjs-org repository is configured? You can find it under Administration -> Repositories. So I can add it to our integration tests. Thanks.

Hey pavel,
It is a very basic configuration. Hope this helps:

I could not find notable differences in the documentation between A6 and A7, but something must me going wrong during the processing of the repo meta data. Maybe the getExtension call returns a different extension compared to before? (e.g. including the .)

Thanks for screenshots. I might know what kind of issue is here.
Please check your npm-default layout. The correct artifact path pattern for npmjs registry should look so: [orgPath]/-/[module]-[baseRev](-[fileItegRev]).[ext]. Notice the dash between orgPath and module.
You can test whether artifacts metadata can be parsed using 'Test Artifact Path Resolution` feature in layouts. I attach screenshots from my internal system so you can see differences.

Awesome!
The default policy was set to [orgPath]/-/[module]-[baseRev](-[fileItegRev]).tgz, so the getExt call could not retrieve the extension at all.
After changing it to: [orgPath]/-/[module]-[baseRev](-[fileItegRev]).[ext] scanning works and I'm getting a 403 when trying to download a vulnerable library.

Glad to here your issue is solved 💪 .
We'll improve our documentation for the plugin, so on-boarding and configuration experience with the plugin will be smoother.


Cartesian Product pattern using hadoop

Using this PR -

of.txt and hog.txt are converted to a single sequence file
cartesian product pattern is used by overriding hadoop FileInputFormat and RecordReader
(UPDATE) Now map jobs receive serialised FeatureVector which contains computed feature for HOF and HOG series. Previously they received file paths for of.txt and hog.txt
Issue was that we are converting String to Double 700K times in each map. Now with serialised object in sequence file we don't need to parse strings

Now the flow looks like below -

This also includes commit in https://github.com/USCDataScience/hadoop-pot/pull/9

Below are observation after running jobs on 1000 video set -
MeanChiSquareDistanceCalculation -
"# M/R jobs"	"cores X machine"	"Time"
0.5 million		32 X 1			1.5 hours

SimilarityCalculation -
"# M/R jobs"	"cores X machine"	"Time"
0.5 million		32 X 1			4.5 hours


Replacing VocabularyCollection with sub-Vocabularies

Coverage remained the same at 100.0% when pulling 4b1beb4531a1851565597d554a045c26efc68272 on sub-vocabularies into d927747e2b8aedf784a5b052dfe554cc76abb0bc on master.

Coverage remained the same at 100.0% when pulling 4b1beb4531a1851565597d554a045c26efc68272 on sub-vocabularies into d927747e2b8aedf784a5b052dfe554cc76abb0bc on master.


v3.1.3 Cherry-pick

[ ] jibuji: fix: support archive.none for OSS directory artifacts (#6312) 40b08240d
[x] Saravanan Balasubramanian: fix(controller): Same workflow nodes are not executing parallel even semaphore locks available (#6418) 7ec5b3ea9
[x] Saravanan Balasubramanian: fix(controller): Randomly expr expression fail to resolve (#6410) c29b275d5
[ ] Yuan (Bob) Gong: docs(executor): emissary executor also runs on GKE autopilot (#6430) f1792f68c
[ ] Huan-Cheng Chang: fix(controller/cli): Resolve global artifacts created in nested workflows (#6422) dd3c11252
[ ] Antoine Dao: fix(emissary): throw argo error on file not exist (#6392) b17d1bf7b
[x] Alex Collins: fix(executor): Remove 15s guard for Docker executor. Fixes #6415 (#6427) 946e4a4a6
[ ] Luces Huayhuaca: chore: upgrade to k8s v0.20.4 (#6143) 96b165e18
[x] Alex Collins: fix(executor): remove unused import preventing compilation 29ebc2a6a
[ ] Joe McGovern: feat(controller): opt-in to sending pod node events as pod (#6377) cc701a1af
[ ] Tianchu Zhao: feat(artifact): enable gcs ListObjects (#6409) 959ce6b7f
[ ] Alex Collins: chore: correct CODEOWNERS syntax 896600648
[ ] Alex Collins: chore: add @alexec as proto codeowner f47ada3cd
[x] Yuan (Bob) Gong: fix(executor/emissary): fix nonroot sidecars + input/output params & artifacts (#6403) 30e2518c2
[x] Saravanan Balasubramanian: fix(controller): Global parameter is not getting updated (#6401) 4da8fd940
[ ] Alex Collins: docs: updated sidecar-injection.md c4bdc217e
[x] Alex Collins: fix(controller): Force main container name to be "main" as per v3.0. Fixes #6405 (#6408) f2d24b1d9
[ ] Tianchu Zhao: fix(executor): fix GCS artifact retry (#6302) 2df5f66a3
[ ] Dominik Deren: docs: updating documentation on how to setup IRSA on AWS (#6406) c80f4bc73
[ ] Alex Collins: docs: Updating upgrading.md. Closes #6314 10c0fa5f9
[x] Alex Collins: fix(controller): Mark workflows wait for semaphore as pending. Fixes #6351 (#6356) 092b4271b
[x] Tianchu Zhao: fix(controller): allow initial duration to be 0 instead of current_time-0 (#6389) 453539690
[ ] Yuan Tang: docs: Add note on additional required permission for createBucketIfNotPresent for OSS driver (#6378) 2bf711b32
[ ] Thiago Bittencourt Gil:  docs: Add 4intelligence (#6400) b15a79cc3
[x] Alexander Matyushentsev: fix: Server crash when opening timeline tab for big workflows (#6369) f4b89dc8e
[ ] Alex Collins: Revert "fix: examples/ci.yaml indent (#6328)" 99359a095
[ ] Antoine Dao: fix(gcs): throw argo not found error if key not exist (#6393) 66c441006
[ ] Zadkiel: chore: import grafana dashboard (#6365) 2c2525981
[ ] KUNG HO BACK: fix: examples/ci.yaml indent (#6328) 3f72fe506
[ ] Tianchu Zhao: fix(controller): fix retry on transient errors when validating workflow spec (#6370) 9233a8de7
[x] Tianchu Zhao: fix(controller): allow workflow.duration to pass validator (#6376) 488aec3ca
[ ] Yuan Tang: feat(controller): Allow configurable host name label key when retrying different hosts (#6341) d6ec03238
[x] Antoine Dao: fix(fields): handle nexted fields when excluding (#6359) bd5a8a99b
[x] BOOK: feat(controller): sortDAGTasks supports sort by field Depends (#6307) cfab7db53
[ ] Yuan Tang: fix(cli): Overridding name/generateName when creating CronWorkflows if specified (#6308) 6e58b35c3
[ ] Tianchu Zhao: docs(users): Add WooliesX (#6358) 5f0d6ab87
[ ] github-actions[bot]: docs: updated CHANGELOG.md (#6348) 16308fa61
[x] Tianchu Zhao: fix(crd): temp fix 34s timeout bug for k8s 1.20+ (#6350) b388c63d0
[ ] Yuan Tang: docs: Add Alibaba Group to USERS.md (#6353) eced49411

@terrytangyuan do you want to mark any for OSS?
@tczhao there are a few fixes for GCS is this list. Do you think they should be included?

I'd like to include #6434, once it is merged.

[ ] Alex Collins: fix(controller): Reinstate support for outputs.results for containers. Fixes #6428 (#6434) 51c15764d

Nothing to include for OSS for this release but could you include the following commit?
Yuan Tang: feat(controller): Allow configurable host name label key when retrying different hosts #6341 d6ec032

I'm keen to include the GCS fixes

Tianchu Zhao: feat(artifact): enable gcs ListObjects (feat(executor): enable gcs artifact ListObjects, supports data-transformations #6409) 959ce6b
Tianchu Zhao: fix(executor): fix GCS artifact retry (fix(executor): fix GCS artifact retry #6302) 2df5f66

Released!

Hi @alexec @sarabala1979, I think we missed cherry picking my first PR fix for emissary with nonroot: https://github.com/argoproj/argo-workflows/pull/6247/files. It's also required for running emissary with nonroot containers.
https://github.com/argoproj/argo-workflows/blob/release-3.1/workflow/executor/emissary/binary.go#L19

@terrytangyuan  We already released v3.1.3. Which includes only the checked once.  Sorry, we missed your comments.  you might be commented after we start releasing process. I can do 3.1.4 this week. I will create cherry-pick issue tomorrow.

No worries, thank you for the quick response! Looking forward to the next release.


Responsive Sticky Navbar

Is your feature request related to a problem? Please describe.
Yes, the current navigation system lacks a responsive sticky navbar. This can be frustrating for users who need to scroll back to the top of the page to access the main navigation links, especially on mobile devices or long pages.
Describe the solution you'd like
I would like to see the addition of a responsive sticky navbar that:
Sticks to the Top: Remains visible at the top of the page as users scroll down.
Responsive Design: Adjusts seamlessly to different screen sizes, ensuring optimal usability on both desktop and mobile devices.
Customizable: Allows for customization of its appearance and functionality to match the site's design and user needs.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Static Navbar: Keeping the navbar static at the top, which requires users to scroll back up to access it.
Sidebar Navigation: Using a sidebar navigation that remains fixed, but this can be less intuitive and takes up valuable screen space, especially on mobile devices.
Floating Action Button: Implementing a floating action button for navigation, but this doesn't offer the same comprehensive access to multiple links as a navbar does.

Currently not accepting sample projects. KIndly raise new issues

@Sulagna-Dutta-Roy please assign this to me i am GSSOC'24 contributor with 2+ years of experience in development


feat: enhanced provider API / @tanstack/react-query peer dependency

Changes

Added @tanstack/react-query as a peer dependency and introduced enhanced provider API.
Implemented fallback to fetch ENS name from enhanced provider API when mainnet is unavailable.

What to test

Go to this preview link: https://rainbowkit-example-h3v2divmt-rainbowdotme.vercel.app/
Open up network tab by right clicking -> inspect element -> network tab
If you connect to a wallet that has ENS name you should be able to see it making request to our enhanced provider API

@magiziz Noticed the address -> resolved name is a bit slow, even after the first fetch. Seeing a few network requests after leaving the page idle too. Are we taking advantage of react-query caching? https://github.com/rainbow-me/rainbowkit/assets/4412473/307f133e-0b26-49dd-b092-91b215adaf9d

Also pushed up some tweaks, mainly one that injects the API key at build-time. This can be a bit messy because it also needs to be available within the CI tests (where we can't statically replace the API key). Lmk if you have other thoughts! It will be skipped in CI here because we need to add a new ENV.

Force pushed to cleanup the git history here


Update version: Docker.DockerDesktop version 4.21.0

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] This PR only modifies one (1) manifest
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.4 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: codeflow:open?pullrequest=https://github.com/microsoft/winget-pkgs/pull/111200&drop=dogfoodAlpha

[Policy] Needs-CLA


Different videos with same title in search results both get highlighted when one chosen.

When I choose a video from search results in Kaku, where there are multiple identically named videos as results, all with that name get highlighted.
For example: Search for 'Billy Joel Piano Man'. The two first results are both called 'Billy Joel - Piano Man'. When you click on one, both get highlighted.

This is a known bug, thanks for reporting.

Use uploader's name as a quickfix for the time being.


Travis build cancelation

There is beta feature in Travis that cancels existing job if new commit in branch is created or the branch is force-pushed to. It would greatly reduce load on Travis thus probably increasing the throughput.
Would everyone be ok with enabling that feature? @whyrusleeping @diasdavid @dignifiedquire @VictorBjelkholm @lgierth
I would have to email the support and ask them to do it as it is still in "closed" beta, they might want authorization from owner of the org but we will see.

I like it

Yes please :D

yes, yes and yes

Makes a lot of sense, go for it!

Sent the email.

The feature was enabled for our repos in libp2p and ipfs.
If you have some repos where you frequently go through push,dev,push,dev,push,dev process enable this feature in settings tab of this repo.
I have already enabled it for go-ipfs.

Thank you @Kubuxu


Updating Haxe

What is your question?
If I update Haxe to 4.2.4 for Psych Source 0.5 and further, will I still be able to compile past versions of Psych (0.4.2 and below)?

yes

yeah i use haxe 4.2.4 and i can still compile the other versions of psych just fine

Ok, thank you.


Migrate to unity catalog and terraform

What type of PR is this?

Refactor
Bug Fix
Optimization
Deployment (Azure, CI/CD, PowerShell, ...)

Description

Introduce a test deployment with full Unity Catalog
Change the spetlr version to be the same as the Databricks Runtime Version used in testing - 14.3.* for now
Test only on 1 DBR version
Make Speltr UC compatible (see below)
Change the test deployment to be completely in terraform
SQL server test were not ported to terraform yet, we invite future PRs to re-introduce this part of the tests.

Functional changes to Speltr

The DeltaHandle by default no longer reads or writes by path. All read and writes are done using the name. Unity Catalog tables are either managed, residing in UC managed storage or external, residing in storage that is known to UC, but not exposed to the user. In bothe cases UC wants the user to go trough UC when accessing the table so that permissions can be applied. Therefore all direct access to the underlying data (regardless if that may save a few milliseconds) is not allowed.

Deployment
Deployment is split into several stages:

Permanent resources (metastore_and_pemanent) deploys the metastore and the permanent resources (Cosmos account)
"integration_resources" deploys all azure resources, the captain service principal and the workspace.
"integration_databricks" then uses that workspace API and deploys the catalog, workspace association, external locations and permissions so that the SPN can deploy the test job
Test job submission is a simple step. The spetlr-tools repos could in fact use the pipeline to here and deploy its own set of commands at this point.

Next steps after this

Add sql server tests back in.
Create a cleanup pipeline that deletes the captains and catalogs from databricks.

Work completed:

integration resource as well as databricks deployment is now based on local terraform state
Deployments will be forgotten and need to be cleaned up outside terraform. This is for speedup of the pipeline.
Azure cleanup script is done.
Databricks cleanup (Catalogs, credentials, SPNs) is TODO.

Work completed:

integration resource as well as databricks deployment is now based on local terraform state
Deployments will be forgotten and need to be cleaned up outside terraform. This is for speedup of the pipeline.
Azure cleanup script is done.
Databricks cleanup (Catalogs, credentials, SPNs) is TODO.

Next step:

pipeline fails on missing permission to create databricks_storage_credential.ex_storage_cred.

The Databricks cleanup can maybe be done by using terraform destroy?

Re: @LauJohansson I intentionally went away from terraform destroy. Terraform destroy carefully rolls back all changes and waits until this has been completed. We use new names every run, so there is no collision. By simply deleting the resources, we save about 10 minutes of pipeline runtime. I would prefer not to go back to using terraform destroy.

Development Notes:

I simplified the deployment to only 1 catalog. This is my preferred way 1 catalog to 1 workspace sharing the same name.
During deployment, the pipeline identity will be used. This is already an account admin.
TODO: for running, the captain needs read and write access to the catalog and nothing else. This needs a new group.
The captain should not need account access.

Current state: Test job is launched but gives error: Cannot read the python file /Volumes/spetlr114571358001/volumes/init/main.py.
The library is correctly installed from the same volume, and logging into the workspace I can download main.py from the given location.
I do not currently understand the problem and am going to let it be for now and come back to it in a a few days. meanwhile, here are the driver logs:
stderr (2).txt
log4j-active (3).txt
stdout (2).txt

Status: I just need to tune the test main job a little bit. The test job launches and output can be seen. Unity catalog setup seems good on inspection.

Status:
The deployment is working. The test job is runnning. All the test now need to be changed to actually work with unity catalog.

This PR is ready to merge. Please review.


recommended that you use use_reentrant=False

I see below warning in logs when running a LoRA training , can this be ignored ?
/text-generation-webui-main/installer_files/env/lib/python3.11/site-packages/torch/utils/checkpoint.py:429: UserWarning: torch.utils.checkpoint: please pass in use_reentrant=True or use_reentrant=False explicitly. The default value of use_reentrant will be updated to be False in the future. To maintain current behavior, pass use_reentrant=True. It is recommended that you use use_reentrant=False. Refer to docs for more details on the differences between the two variants.   warnings.warn(

settings used
Text: (Service_API.txt) has 1604 blocks (Block Size 256 tokens)
[Batch Size: 7, Epochs: 3.0, Gradient Accumulation: 1]
Total number of steps: 690
Steps per each Epoch: 230
Suggestions:
Checkpoints: Save every 69 - 138 steps (Current: 192)
Warmup steps: 69 (Current: 96)

It could be ignored. I thought I disabled the warning... have to check


Support New RM Commands

Azure no longer supports the Switch-AzureMode command in favor of modified commandlets using (verb)-AzureRM(noun)

+1

+1


Failing Cosmos VALUE Query

SELECT VALUE IS_PRIMITIVE(r["Enrich"]["BotInfo"]["ModelCandidate"]) FROM r
Results in Cannot set property 'headers' of undefined
Repros in the portal and locally

This now works on master with the new groupBy work


Subtler icon, in address bar

The icon color is not in line with the subtle other default icons anymore.

Plus, can it show up in the address bar?

SVG icon 🎉
But we cannot put it into the nav box, just the nav bar next to all the other icons.