feat: Add support for Dynamic Sampling

This adds support for Dynamic Sampling, introduced in https://github.com/getsentry/sentry-php/pull/1360.
Once we released the PHP SDK, we can progress here.
Remaining ToDos

[x] In the Middleware, I'm setting TransactionSource::url(), but maybe TransactionSource::route() is more appropriate.
[x] The LighthouseIntegration.php is currently not setting a TransactionSource, as we might not have access to the context here.

@stayallive Thanks, LGTM! :shipit:


Round properly to nearest page in TabBarView.

This logic is described in the test as looking for a scroll ending
very close to a new page, but in fact its behavior is more like
"very close to a page to the right": if we're not very, very close
to any page, it will pick the page to the left, not an old page.
There's no reason this should be left-right asymmetrical.
Instead, pick the nearest page.
In practice, the case where this makes a difference never arises when
the scroll runs undisturbed to completion; but when the user taps on
the page to hold or drag, the scroll will be interrupted before it
gets within tolerance of a particular page, and this case does arise.
This fixes a glitch that is hard to trigger without time dilation,
but is quite conspicuous with it:

Open a tab view with at least 4 tabs, e.g. the Buttons screen
of the gallery (with "Animate Slowly" on.)
Starting at tab 0, tap tab 2.
When the animation is nearly complete, tap the page a couple
of times, as if to drag it around to scroll.  Then let the
page view settle ballistically toward page 2.
Before it finishes, tap tab 3.
Suddenly page 1 fills the view, replacing page 2, before we
scroll from there to page 3.

With this fix, the animation in the last step moves smoothly from
where we are when it starts onward to page 3.

I'll try and give this a review today ... sorry it's taken so long!

This looks correct to me, but maybe @HansMuller can remember why we didn't go this route before?
Anyway, unless @HansMuller has an objection, LGTM and one of us will land this when our tree is green.
Thanks for the contribution!

It took a while, but I have been able to reproduce your slo-motion scenario.
I agree that _controller.index = _pageController.page.round(); instead of effectively setting it to ..floor() sounds sensible. However I don't think it really matters so long as the mapping from the page controller's page to the tab controller's index (selection) is consistent.
If you drag-scroll the TabBarView while TabBar is in the middle of "warping" towards the new selected tab then things do get confused if you scroll far enough to change the mapping from the page controller's page to the tab controller's index. This can happen if the index is defined to be page.floor() or page.round().
At least that's the way it looks late Friday afternoon.
I think the TabBar could do a better job and dealing with cases where the TabBarView is scrolled while the TabBar is trying responding to a tap by warping to a new tab. It hasn't been a priority so far, because it's rather difficult to do at normal speed.
Did you look into this because you were seeing animation glitches that occurred with timeDilation=1.0?

I spoke to Hans offline. It sounds like what we should do is first document the exact problems that the current system has, and then see if there is a more holistic solution to those problems.

@HansMuller So looking at this a bit more, it seems to me like it does improve the logic a bit -- moving from dealing with tolerances and potentially being biased to one side to being more straight-forward. Even if it doesn't actually fix any underlying bugs it still seems like a net improvement to this logic, which would help with further refactorings and bug fixing. I think we should land this and then consider whether there are any remaining issues separately.
The only thing I think we should do before we land it is adjust the tests so that it still tests the old "VERY close" behaviour as well.

Fixes #15126
We should add a test to check that the specific issue in #15126 get resolved too.


Update cvarlist.md

Would it be possible to regenerate this list towards the current client?

Can you paste the output of cvarlist as a comment here? Then I can format it in Markdown.


Incorrect results?

It appears that this library sometimes returns incorrect results. The example below shows two failure cases. For both problem instances, this library incorrectly assigns both rows to the same column. For reference, I have also included the (presumably correct) results returned by the scipy implementation.
import scipy.optimize
import lapjv
import numpy as np

C1 = np.array([[8., 9.],
               [1., 4.]])
C2 = np.array([[5., 1.],
               [8., 9.]])

for C in [C1, C2]:
    print('\n%s' % C)
    print('LAPJV result: %s, %s' % lapjv.lapjv(C)[1:])
    print('Scipy result: %s, %s' % scipy.optimize.linear_sum_assignment(C))

Result
[[8. 9.]
 [1. 4.]]
LAPJV result: [1 0], [1 0]
Scipy result: [0 1], [1 0]

[[5. 1.]
 [8. 9.]]
LAPJV result: [1 0], [1 0]
Scipy result: [0 1], [1 0]

Please let me know if I'm using the library incorrectly, or if there are certain types of inputs to avoid.

Thanks @vmarkovtsev! I was interpreting the output format incorrectly.
For reference: gatagat/lap#10

Hi guys! Just one question. From this implementation it’s possible to return the dual variables?

@HildebrandoBarros this is not related to the issue, please open a new one.


Error after MemGator's go get ... command

I have setup my go environment then cd $GOPATH, which puts me at /Users/MYUSERNAME/go. I then run go get github.com/oduwsdl/memgator and receive the error:
# github.com/oduwsdl/memgator
src/github.com/oduwsdl/memgator/main.go:128: unknown http.Client field 'Timeout' in struct literal

MacOS X 10.10.5. go version go1.2.2 darwin/amd64. Occurs with no subdirectories within

Please try upgrading your Go to the latest version. The Client.Timeout requires at least Go 1.3. According to the Go 1.3 Release Notes:

The net/http package's HTTP client now has an optional Client.Timeout field to specify an end-to-end timeout on requests made using the client.

I upgraded go:
$ go version
go version go1.5.1 darwin/amd64

then:
$ cd $GOPATH
$ go get github.com/oduwsdl/memgator

returns
# github.com/oduwsdl/memgator
src/github.com/oduwsdl/memgator/main.go:7: import /Users/MYUSERNAME/go/pkg/darwin_amd64/github.com/oduwsdl/memgator/pkg/mflag.a: object is [darwin amd64 go1.2.2 X:none] expected [darwin amd64 go1.5.1 X:none]

Scratch that. I cleared everything in the $GOPATH, reran, and it works as expected. We should recommend a certain version of Go (> 1.3) in the MemGator requirements, @ibnesayeed .


[firebase_analytics] Fix deprecated api

Standart eccomerce_purchase event (and many others, I guess) has been deprecated. Need a support for new events API
https://firebase.google.com/docs/reference/android/com/google/firebase/analytics/FirebaseAnalytics.Event#public-static-final-string-ecommerce_purchase

I am also facing this issue, any progress or ETA?
Would also need the functionality for multiple items being purchased in the one transaction.

Closing this in favor of https://github.com/FirebaseExtended/flutterfire/issues/3876


Run inference using UK metoffice data

Detailed Description
UK Metoffice is now available on open meteo, so we should add a option to use this in the model.
Context

currrently use ICON data
model was trained on met office data
#49
https://open-meteo.com/en/docs/ukmo-api#hourly=

Possible Implementation
Add option here

Hello @peterdudfield
Can I work on this issue?


Reduce 1 nested loop, unecessary CompletionIterator

What changes were proposed in this pull request?
inUseFiberCache array is released whenever necessary:

In update
In close of OapIterator

Accordingly, no need an extra nested CompletionIterator
How was this patch tested?
N/A

The CompletionIterator is useful to release the fibers earlier and there should be no overhead. Considering the last iter, there is no update method to release the fibers. If you remove the CompletionIterator, you will wait until the task completion for the release. Otherwise it can be released earlier just after the iter is consumed.

@carsonwang I agree with you except

should be no overhead
One more nested loop is added.
I just merge the CompletionIterator's functionality into OapIterator, it will solve the later relase issue.

This should work now.  One little drawback is that user need specify both completionFunction and close which are actually doing the similar thing.

@carsonwang Yes, they are just slightly different. I have addressed the difference in comments : )

LGTM. Thanks @gczsjdy .


150GB in RedisQFork_####.dat files

I found several files in C:\Windows\ServicesProfiles\NetworkService\AppData\Local\Redis that add up to nearly 150GB.
For some reason, them being in this folder means that Explorer and other programs cannot find them without being ran as administrator.
Is there any reason for all of these files? I didn't even have any data in my redis.

Hi @PaulBGD,
do you still have the list of the files?

Sorry I don't, I had less than a GB left and I needed the space.

@PaulBGD can you at least tell me what name/file extension they had?

I've also have a huge file named RedisQFork_2568.dat over there. can we
delete it safely?
Thanks,
Mehdy Haghy
Chief Technology Officer
Intellirose LLC
Cell:<PHONE_NUMBER> <%28562%29552-5460>
Phone:<PHONE_NUMBER>
On Fri, Jul 31, 2015 at 2:52 PM, Enrico Giordani<EMAIL_ADDRESS>wrote:

@PaulBGD https://github.com/PaulBGD can you at least tell me what
name/file extension they had?
—
Reply to this email directly or view it on GitHub
https://github.com/MSOpenTech/redis/issues/283#issuecomment-126796804.

@mehdymohajery
that is the the memory mapped filed used by Redis on Windows, if Redis is running it can't be deleted.

@enricogior As I said in the title, they all looked like RedisQFork_####.dat where each # was a digit.

@PaulBGD sorry... it's 10PM here and I guess I'm getting tired ;)
The .dat files are used for the memory map mechanism, the expected behavior is that one single .dat file is created when redis-server.exe starts and the file is supposed to be deleted when redis-server.exe exits.
If multiple instances of redis-server.exe are running at the same time, multiple .dat files are created.
You can change the default location of the .dat file editing the redis configuration file and setting the heapdir flag for example heapdir c:\redis-heap

Found a 32GB file. Why not save in the temp directory by default?

Hi @starrychloe,
I honestly don't know the reason behind that initial choice, but your suggestion seems reasonable.
Thank you.

Should I close this? Since I deleted the files and can't reproduce it on the latest version, the issue seems meaningless.

@PaulBGD,
sure, feel free to close it.
Thank you.

Just a heads up, this happened to me yesterday (2015-10-01). Very random. I am not able to reproduce it either. Perhaps something to be aware of for the team?


Minimum network configs for Establishing Peer-to-Peer Connection with Netbird

I am seeking guidance on configuring networks for peer-to-peer (P2P) connections between specific peers in our setup. Specifically, I need the peers in the upper part of the attached illustration to establish P2P connections with peer X, located in the lower part of the illustration. Connections among the peers in the upper part can remain relayed.

Constraints:

We have limited control over the firewalls and network configurations in the upper part, but we can request the network owners to implement certain minimum requirements.
Firewall X and its associated network configuration are fully controlled by us.

Questions:

What are the minimum network requirements (A) that the network owners in the upper part must implement to facilitate a P2P connection with peer X?
Given that peer 1's firewall and network are configured accordingly, what are the minimum requirements (B) that we should apply to the firewall and network of peer X to achieve a P2P connection between peer 1 and peer X?

If my questions reveal the gaps in my networking knowledge, please feel free to suggest alternative approaches.

I have this same kind of situation and interested to get solutions as well.


ValueError while training triplet loss

@davidsandberg I am facing the ValueError when I am training the triplet loss. Please can you suggest the solution?
python src/align/align_dataset_mtcnn.py /home/poonam/Downloads/Bright_IMG /home/poonam/Downloads/Bright_IMG1 --image_size 182 --margin 44
Creating networks and loading parameters
Traceback (most recent call last):
File "src/align/align_dataset_mtcnn.py", line 161, in
main(parse_arguments(sys.argv[1:]))
File "src/align/align_dataset_mtcnn.py", line 57, in main
pnet, rnet, onet = align.detect_face.create_mtcnn(sess, None)
File "/home/poonam/projects/facenet/src/align/detect_face.py", line 283, in create_mtcnn
pnet.load(os.path.join(model_path, 'det1.npy'), sess)
File "/home/poonam/projects/facenet/src/align/detect_face.py", line 85, in load
data_dict = np.load(data_path, encoding='latin1').item() #pylint: disable=no-member
File "/home/poonam/.virtualenvs/facenet/lib/python3.6/site-packages/numpy/lib/npyio.py", line 447, in load
pickle_kwargs=pickle_kwargs)
File "/home/poonam/.virtualenvs/facenet/lib/python3.6/site-packages/numpy/lib/format.py", line 692, in read_array
raise ValueError("Object arrays cannot be loaded when "
ValueError: Object arrays cannot be loaded when allow_pickle=False

check if your model_path is correct and det1.npy exists in directory
It seems your mtcnn face detection model is not loaded correctly in line pnet.load(os.path.join(model_path, 'det1.npy'), sess)

@thelastfunction Thanks.

@PoonamZ How You Solve that??

I have the same issue on line 283 of the file detect_face.py,
I also add a print before to confirm is the model_path is correct,  it is OK.
the det1.npy, det2.npy, det2.npy just right in the folder of /src/align/
There are also many warnings, indicating some useage of code is deprecated!
`W0622 18:15:39.963512<PHONE_NUMBER>83392 deprecation_wrapper.py:119] From src/align/align_dataset_mtcnn.py:52: The name tf.GPUOptions is deprecated. Please use tf.compat.v1.GPUOptions instead.
W0622 18:15:39.964641<PHONE_NUMBER>83392 deprecation_wrapper.py:119] From src/align/align_dataset_mtcnn.py:53: The name tf.Session is deprecated. Please use tf.compat.v1.Session instead.
W0622 18:15:39.964763<PHONE_NUMBER>83392 deprecation_wrapper.py:119] From src/align/align_dataset_mtcnn.py:53: The name tf.ConfigProto is deprecated. Please use tf.compat.v1.ConfigProto instead.
2019-06-22 18:15:39.965052: I tensorflow/core/platform/cpu_feature_guard.cc:142] Your CPU supports instructions that this TensorFlow binary was not compiled to use: AVX2 FMA
WARNING: Logging before flag parsing goes to stderr.
W0622 18:15:39.969522<PHONE_NUMBER>94368 deprecation_wrapper.py:119] From src/align/align_dataset_mtcnn.py:52: The name tf.GPUOptions is deprecated. Please use tf.compat.v1.GPUOptions instead.
W0622 18:15:39.971852<PHONE_NUMBER>94368 deprecation_wrapper.py:119] From src/align/align_dataset_mtcnn.py:53: The name tf.Session is deprecated. Please use tf.compat.v1.Session instead.
W0622 18:15:39.973070<PHONE_NUMBER>94368 deprecation_wrapper.py:119] From src/align/align_dataset_mtcnn.py:53: The name tf.ConfigProto is deprecated. Please use tf.compat.v1.ConfigProto instead.
2019-06-22 18:15:39.973934: I tensorflow/core/platform/cpu_feature_guard.cc:142] Your CPU supports instructions that this TensorFlow binary was not compiled to use: AVX2 FMA
WARNING: Logging before flag parsing goes to stderr.
W0622 18:15:39.978578<PHONE_NUMBER>36448 deprecation_wrapper.py:119] From src/align/align_dataset_mtcnn.py:52: The name tf.GPUOptions is deprecated. Please use tf.compat.v1.GPUOptions instead.
WARNING: Logging before flag parsing goes to stderr.
W0622 18:15:39.979246<PHONE_NUMBER>36448 deprecation_wrapper.py:119] From src/align/align_dataset_mtcnn.py:53: The name tf.Session is deprecated. Please use tf.compat.v1.Session instead.
W0622 18:15:39.979369<PHONE_NUMBER>36448 deprecation_wrapper.py:119] From src/align/align_dataset_mtcnn.py:53: The name tf.ConfigProto is deprecated. Please use tf.compat.v1.ConfigProto instead.
2019-06-22 18:15:39.979740: I tensorflow/core/platform/cpu_feature_guard.cc:142] Your CPU supports instructions that this TensorFlow binary was not compiled to use: AVX2 FMA
2019-06-22 18:15:39.980931: I tensorflow/core/platform/profile_utils/cpu_utils.cc:94] CPU Frequency:<PHONE_NUMBER> Hz
W0622 18:15:39.977033<PHONE_NUMBER>46784 deprecation_wrapper.py:119] From src/align/align_dataset_mtcnn.py:52: The name tf.GPUOptions is deprecated. Please use tf.compat.v1.GPUOptions instead.
W0622 18:15:39.982066<PHONE_NUMBER>46784 deprecation_wrapper.py:119] From src/align/align_dataset_mtcnn.py:53: The name tf.Session is deprecated. Please use tf.compat.v1.Session instead.
W0622 18:15:39.982222<PHONE_NUMBER>46784 deprecation_wrapper.py:119] From src/align/align_dataset_mtcnn.py:53: The name tf.ConfigProto is deprecated. Please use tf.compat.v1.ConfigProto instead.
.....
`

@doyanger you may be using tensorflow 2.0 version, thats why you are getting these warnings ......you can ignore them ... or you can downgrade it to tensorflow 1.8

Hi Guys can someone help me with this I get the same error
the det1.npy file is existed in the directory, still I get this error
detect_face.py
 print("this is data path="+os.path.join(model_path, 'det1.npy'))         pnet.load(os.path.join(model_path, 'det1.npy'), sess)
this is data path=/Users/mohammadkhaleghi/AI/facenet/src/align/det1.npy Traceback (most recent call last):   File "align_dataset_mtcnn.py", line 160, in <module>     main(parse_arguments(sys.argv[1:]))   File "align_dataset_mtcnn.py", line 56, in main     pnet, rnet, onet = detect_face.create_mtcnn(sess, None)   File "/Users/mohammadkhaleghi/AI/facenet/src/align/detect_face.py", line 284, in create_mtcnn     pnet.load(os.path.join(model_path, 'det1.npy'), sess)   File "/Users/mohammadkhaleghi/AI/facenet/src/align/detect_face.py", line 85, in load     data_dict = np.load(data_path, encoding='latin1').item() #pylint: disable=no-member   File "/opt/anaconda3/lib/python3.7/site-packages/numpy/lib/npyio.py", line 453, in load     pickle_kwargs=pickle_kwargs)   File "/opt/anaconda3/lib/python3.7/site-packages/numpy/lib/format.py", line 722, in read_array     raise ValueError("Object arrays cannot be loaded when " ValueError: Object arrays cannot be loaded when allow_pickle=False

File "align\align_dataset_mtcnn.py", line 170, in
main(args)
File "align\align_dataset_mtcnn.py", line 53, in main
pnet, rnet, onet = align.detect_face.create_mtcnn(sess, None)
File "C:\Users\Ritesh's Laptop\Desktop\facial-recognition-video-facenet\src\align\detect_face.py", line 283, in create_mtcnn
pnet.load(os.path.join(model_path, 'det1.npy'), sess)
File "C:\Users\Ritesh's Laptop\Desktop\facial-recognition-video-facenet\src\align\detect_face.py", line 85, in load
data_dict = np.load(data_path, encoding='latin1').item() #pylint: disable=no-member
File "C:\Users\Ritesh's Laptop\AppData\Local\Programs\Python\Python36\lib\site-packages\numpy\lib\npyio.py", line 447, in load
pickle_kwargs=pickle_kwargs)
File "C:\Users\Ritesh's Laptop\AppData\Local\Programs\Python\Python36\lib\site-packages\numpy\lib\format.py", line 692, in read_array
raise ValueError("Object arrays cannot be loaded when "
ValueError: Object arrays cannot be loaded when allow_pickle=False

@soniritesh124 This may due do the version complexity in numpy, what numpy version are you using?

They didn't give a specific version of package so these are broke.
Please try install this versions. Problem is solved.
pip install numpy==1.16
pip install scipy==1.1.0

@us
Thank you, it works now


Excel Lab opens a blank panel.

sorry, new here and learning.  Hopefully the picture loads.  It isn’t a load failure it just does nothing.  Just started today.  Have been using AFE and now Excel labs heavily for over a year.

In the blank panel could you:

right click > inspect (opens dev tools)
find the "Console" tab along the top and switch to that
see if there is any text or errors there

Hi,
I am encountering the same issue.  I found the Console and noted a few errors - the last one relates to rendering and may be the issue but I am not sure.  Please see the attached screenshot of the console.

fyi @NWilson


Normalize Data

Normalize the extracted data to ensure consistency and compatibility with the target data schema. Normalization involves transforming the data into a standard format, addressing any discrepancies or variations in the raw data. This process is essential for maintaining data integrity and facilitating accurate analysis.

First version in place

Need to re-evaluate the current strategy. Potential adjustments:

Concatenate long-form texts prior to partitioning
Eval OpenAPI spec to classify long-form texts and quantitative/qualitative metadata fields

To consider in #22

Metadata as JSON or converted into a markdown table


Gh 2723 refactor to open cypher csv

Refactored the csv export functionality. A user can now pass a csv format which is then used to build its respective generator. This makes use of the existing ToCsv operation and handler rather than building a new op and handler for each format

Codecov Report

:exclamation: No coverage uploaded for pull request base (v2-alpha@9a66133). Click here to learn what that means.
The diff coverage is n/a.

:exclamation: Current head 2d7ce1e differs from pull request most recent head 63e50c6. Consider uploading reports for the commit 63e50c6 to get more accurate results

@@             Coverage Diff             @@
##             v2-alpha    #2751   +/-   ##
===========================================
  Coverage            ?   69.17%
  Complexity          ?      487
===========================================
  Files               ?       78
  Lines               ?     1502
  Branches            ?      119
===========================================
  Hits                ?     1039
  Misses              ?      380
  Partials            ?       83

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

The file rest-api/core-rest/src/test/java/uk/gov/gchq/gaffer/rest/service/v2/OperationServiceV2IT.java is showing as only copyright changed. I think this can be undone as well, such that the file is not mentioned in the changes at all, it should never have been there.


<IP_ADDRESS>-rc1 release notes

Signed-off-by: aksingh-es<EMAIL_ADDRESS>Description
OS 2.0.0-rc1 supported release notes
Fixed review comments from https://github.com/opensearch-project/anomaly-detection/pull/510
Modified Infrastructure entries to avoid unncessary entried
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #512 (9fa8e95) into main (a44ffec) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main     #512      +/-   ##
============================================
- Coverage     78.93%   78.92%   -0.02%
+ Complexity     4193     4190       -3
============================================
  Files           296      296
  Lines         17663    17662       -1
  Branches       1878     1878
============================================
- Hits          13943    13939       -4
- Misses         2819     2820       +1
- Partials        901      903       +2

Flag
Coverage Δ

plugin
78.92% <ø> (-0.02%)
:arrow_down:

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

Impacted Files
Coverage Δ

...ava/org/opensearch/ad/task/ADHCBatchTaskCache.java
88.88% <0.00%> (-1.24%)
:arrow_down:

...rch/ad/transport/AnomalyResultTransportAction.java
80.13% <0.00%> (-0.69%)
:arrow_down:

...opensearch/ad/indices/AnomalyDetectionIndices.java
71.93% <0.00%> (-0.19%)
:arrow_down:

...ava/org/opensearch/ad/model/TimeConfiguration.java
100.00% <0.00%> (ø)

.../main/java/org/opensearch/ad/ml/CheckpointDao.java
70.19% <0.00%> (+0.64%)
:arrow_up:

Check if all PRs after 1.3 release have correct tag, for example, https://github.com/opensearch-project/anomaly-detection/pull/498 and https://github.com/opensearch-project/anomaly-detection/pull/509 seems don't have correct tag

Check if all PRs after 1.3 release have correct tag, for example, #498 and #509 seems don't have correct tag

@ylwu-amzn  I have removed certain CRs which do not have consequence for the users.(related to Uts,intermediate changes etc)


[BUG]: Trouble Adding Description to Developer Compliance Center Canvas App

Describe the bug
Adding Description to Developer Compliance Center Canvas App.  Instruction say to click on refresh button below -- however the only button visible is a submit button.
Component (please tell us which flow or app you are experiencing issues with):

Solution [e.g. Core, Nurture] -->  Nature
Version [e.g. 1.29]  --> Core 1.60 Nature 1.15
App or Flow [e.g. Sync Template (v2)]  -->   Developer Compliance Center Canvas App

To Reproduce
Steps to reproduce the behavior:

Go to Developer Compliance Center
Click on '.App Catalog' Details button
Description says to go to app settings > edit the app > settings > add description > save and publish > return here and hit refresh
See screen attached screen shot

Expected behavior
Expect description to be populated as stated above

Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

The save button is to the left of the screenshot you have captured. After you hit save you will be surfaced UX to publish

Closing out issues with no follow up on our side. Thanks for using CoE!


Country info card link to data details pages

The "See details" call to action looks too small and on my screen and there is too much empty space.
I also automatically tried to click on the title "Germany Digiwhist" in the example, which is common UX design practice, so think we should hyperlink this too.

I also think the text OCDS in the label looks off centre:

Make the whole design narrower so that we get rid of the empty spaces (the is a lot of other ugly empty spaces)

The whole box is now clickable and the click redirects to the detail page.

The narrower layout was set. The content width doesn't exceed 1400px.

The narrower layout is much better. Thank you
I actually didn't notice that the box changes colour slightly upon hover. I also missed that the "See details" button gets underlined. It's much better, but if we could also underline the title when the user hovers over the box, the feedback will be much stronger.
The OCDS label I mentioned before is still an issue

I also think the text OCDS in the label looks off centre:

I just checked again and the label is also coming up as a contrast error. Possibly because the spacing between the characters is not great? Can you take a look?

@sabahfromlondon actually it now combines two styles. There is the See details link which can be combined with the title link.
The second style is that the whole card is a link (that's why the box changes colour slightly upon hover.). In this case it makes no sense to also have the See details link.
I like the style when I can click the whole box as I don't have to search for links a move to them.
Which one should we apply?

@hrubyjan You can't not have a call to action button. We need to keep the see details button.
I've said before, the different shades of grey for the are not obvious. Someone with a visual impairment will not see the colour change. Nor is it a standard design pattern, so has the potential to cause confusion on it's own.
I'm perfectly fine with the whole box being clickable. My suggestion is about feedback to the user the something is clickable. It's why I asked for the title to look like a hyperlink upon hover on the box.

It's why I asked for the title to look like a hyperlink upon hover on the box.

Done - a non-color designator (text underline) is added to the heading on hover.
For the other issue mentioned (alignment of "OCDS"), this version of the registry ONLY has OCDS datasets, so I simply removed those badges. As I understand, they had a function when the plan was to include non-OCDS datasets.

@jpmckinney great thank you! :)
And yes fine to remove the OCDS label for now.


Download regions created by embedding algorithms

I wonder if it could be possible to download the region files generated in the 2D embedding feature for further inspection outside the software?

Yeap that works!


config service(version 2.0.0-RC1 or above) return 404 when use double slash "//" in http path

[x] I have checked the discussions
[x] I have searched the issues of this repository and believe that this is not a duplicate.
[x] I have checked the FAQ of this repository and believe that this is not a duplicate.

Describe the bug
apollo client read config fail when there is a slash in the suffix of apollo.meta, for example
apollo.meta=http://ip:port/

but it work well when / doesn't occur in the suffix. i.e
apollo.meta=http://ip:port

Reproduce
run
curl http://localhost:8080//services/config

to access config service, version 2.0.0-RC1
get
{"timestamp":"2023-08-01T21:49:31.123+0800","status":404,"error":"Not Found","path":"//services/config"}

watch out that // after port 8080

Expected behavior
get json like
[{"appName":"APOLLO-CONFIGSERVICE","instanceId":"localhost:apollo-configservice:8080","homepageUrl":"http://<IP_ADDRESS>:8080/"}]

Additional Details & Logs

Platform and Operating System

$ java -version
openjdk version "1.8.0_362"
OpenJDK Runtime Environment (build 1.8.0_362-8u372-ga~us1-0ubuntu1~20.04-b09)
OpenJDK 64-Bit Server VM (build 25.362-b09, mixed mode)

Version 2.0.0-RC1 or above

config service's version
double slash "//" work or not

1.8.2
work well

1.9.2
work well

2.0.0-RC1
404

2.0.1
404

more detail from version 1.8.2 to 2.0.1
1.8.2
curl http://localhost:8080/services/config
[{"appName":"APOLLO-CONFIGSERVICE","instanceId":"localhost:apollo-configservice:8080","homepageUrl":"http://<IP_ADDRESS>:8080/"}]

curl http://localhost:8080//services/config
[{"appName":"APOLLO-CONFIGSERVICE","instanceId":"localhost:apollo-configservice:8080","homepageUrl":"http://<IP_ADDRESS>:8080/"}]

1.9.2
curl http://localhost:8080/services/config
[{"appName":"APOLLO-CONFIGSERVICE","instanceId":"localhost:apollo-configservice:8080","homepageUrl":"http://<IP_ADDRESS>:8080/"}]

curl http://localhost:8080//services/config
[{"appName":"APOLLO-CONFIGSERVICE","instanceId":"localhost:apollo-configservice:8080","homepageUrl":"http://<IP_ADDRESS>:8080/"}]

2.0.0-RC1
curl http://localhost:8080/services/config
[{"appName":"APOLLO-CONFIGSERVICE","instanceId":"localhost:apollo-configservice:8080","homepageUrl":"http://<IP_ADDRESS>:8080/"}]

curl http://localhost:8080//services/config
{"timestamp":"2023-08-01T21:54:25.152+0800","status":404,"error":"Not Found","path":"//services/config"}

2.0.1
curl http://localhost:8080/services/config
[{"appName":"APOLLO-CONFIGSERVICE","instanceId":"localhost:apollo-configservice:8080","homepageUrl":"http://<IP_ADDRESS>:8080/"}]

curl http://localhost:8080//services/config
{"timestamp":"2023-08-01T21:49:31.123+0800","status":404,"error":"Not Found","path":"//services/config"}

trouble shoot
apollo-client
https://github.com/apolloconfig/apollo-java/blob/265fe49ddf0a4becad40ecf5bcd6bd1c86d9826b/apollo-client/src/main/java/com/ctrip/framework/apollo/internals/ConfigServiceLocator.java#L248C26-L248C35
    return domainName + "/services/config?" + MAP_JOINER.join(queryParams);

when domainName(apollo.meta) is http://ip:port/ will get double slash // in http://ip:port//services/config?...
config-service
The path mapping in
https://github.com/apolloconfig/apollo/blob/99b76dccb0d5700202807d8d75a058006896eee9/apollo-configservice/src/main/java/com/ctrip/framework/apollo/metaservice/controller/ServiceController.java#L48
Guess that when Spring MVC or Spring WebFlux update in https://github.com/apolloconfig/apollo/pull/4295 cause it?
I.e '//' cannot access again.

solution:

apollo-client: delete the suffix '/' of apollo.meta on then runtime?
config service: keep the compability? For example add a servlet filter when meet '//' then convert it to '/'

According to https://stackoverflow.com/questions/70606716/getting-404-error-when-url-contains-double-slash-after-spring-boot-2-5-to-2-6-mi
Just add a property in config service's .properties file
spring.mvc.pathmatch.matching-strategy=ant-path-matcher

Everything is great again!

Reference:

https://docs.spring.io/spring-boot/docs/2.7.14/reference/html/web.html#web.servlet.spring-mvc.content-negotiation
https://github.com/spring-projects/spring-boot/blob/191ac10009cdf1f0663221dad16993493ca50205/spring-boot-project/spring-boot-autoconfigure/src/main/java/org/springframework/boot/autoconfigure/web/servlet/WebMvcProperties.java#L103

The release notes for Spring Boot 2.6 include a section titled PathPattern Based Path Matching Strategy for Spring MVC. This particular change seems to have escaped my attention when issue #4295 was raised. To maintain the original logic, should we set the following configuration as the default?
spring.mvc.pathmatch.matching-strategy=ant-path-matcher


Remove builds of JRuby that bundle Graal

Thanks very much for accepting our definition files for JRuby that bundled Graal (our new JIT compiler for Java). This was really useful when there was no other way to get Graal, but now there are official experimental releases available from Oracle https://github.com/jruby/jruby/wiki/Downloading-GraalVM and soon it will be possible to just use an early access of Java 9 https://github.com/jruby/jruby/wiki/Using-Graal-in-JDK-9-EA-Builds, so these bundled versions aren't necessary any more and we can remove them.

Thanks for the update. What would be the steps for ruby-build/rbenv users to continue trying out the GraalVM after this is removed? Could we instead update the definitions to download the new build from Oracle?

Thanks for the update. What would be the steps for ruby-build/rbenv users to continue trying out the GraalVM after this is removed? Could we instead update the definitions to download the new build from Oracle?

I'm afraid the new builds are click-through licensed as we're now basing our public builds on Oracle's JDK rather than Open JDK.
I've put up an FAQ with the answer for anyone saying 'how do I do it now?'
https://github.com/jruby/jruby/wiki/Truffle-FAQ#how-do-i-get-jrubytruffle

I'm afraid the new builds are click-through licensed as we're now basing our public builds on Oracle's JDK rather than Open JDK.
I've put up an FAQ with the answer for anyone saying 'how do I do it now?'
https://github.com/jruby/jruby/wiki/Truffle-FAQ#how-do-i-get-jrubytruffle


feat: initial revision for e2e tests

This PR adds End-to-end tests to the SDK.
Closes: https://topsort.atlassian.net/browse/API-922

Ok to land but lets do the other PR first.


Fix bug where destination isn't loaded in app.config

Currently, connection strings for destinations are saved in "add" elements with names prefixed with "NBug.Properties.Settings.Connection". This does not match the settings configured in NBug.Properties.Settings, so if NBug attempts to load an "app.config", it will not load the destinations properly. This commit changes the Settings class so it saves destination connnection strings with the prefix "NBug.Properties.Settings.Destination", which does match.

Oh man good catch. I wonder why no one noticed this before. Everything was fine in my tests way back when I was using the configurator tool. I need to verify this first.

I think this was introduced in another PR which enabled custom destinations and renamed connection to destination. Can you verify this works as intended now (i.e. create basic mail destination and test it with the test button). If so I'll merge this.

I have tested this on my application and it now works properly. This probably went unnoticed because the bug only occurs when using an app.config (i.e. MyProduct.exe.config inside the same folder with the main executable 'MyProduct.exe'). In this case, the "LoadAppconfigSettings()" method is called, which tries to use "NBug.Properties.Settings" class to get the configuration settings. This fails, because the name of the destination properties don't match what has been configured in the "Settings.settings" file. When you use the test app in the configurator, it calls the "LoadCustomSettings()" method, which doesn't have this problem, since it reads the config file directly and doesn't care what the names of the destination properties are.

Alright, I must admit I've only tested with the test app and never with the *.exe.config. Thanks for the detailed explanation. Will run more detailed tests after big PRs changing destinations etc. next time.
Merging.


Ignore .idea folder

Add it to your global gitignore, not here. Because we use different IDEs, either everyone here, or neither
https://stackoverflow.com/a/7335487

It needs to apply by every contributor machines (even if the one has many) or just clone the repo and everything will work great.
The world has a few IDE's so that's why I don't see any problems to have 2 or 3 or even 10 line in .gitignore file that changes very rarely.
Anyway as you wish.

You might be right. no bad for a few lines, could you please add .vscode also then I merge?

Done


Before and after options for real-time workflows

This is related to this existing issue: https://github.com/MicrosoftDocs/flow-docs/issues/37
The documentation above about Initiating real-time workflows before or after status changes suggests that the before and after options only apply to status changes. The UI certainly seems to suggest that, but I believe it applies to all of the following options, not just the first one. Each of these represent an update operation.

Record status changes
Record is assigned
Record fields change options

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

ID: a013bf3e-675d-6188-6115-bef65547c566
Version Independent ID: a6c6e501-806e-0376-94aa-56b004c5c3c9
Content: Configure workflow stages and steps in PowerApps - Microsoft Flow
Content Source: flow-docs/configure-workflow-steps.md
Service: flow
GitHub Login: @Mattp123
Microsoft Alias: matp

@JimDaly Thanks for your feedback. I see that this one has been sitting a while. I am evaluating whether there is a documentation issue that needs to be addressed. Can you clarify what you think needs to be updated or added? The same applies to your releated issue. Thanks!

The issue had to do with real-time workflows.
Now the content has been split and all discussion of realtime (synchronous) workflows is in the CDS documenttation:
https://docs.microsoft.com/en-us/powerapps/maker/common-data-service/configure-workflow-steps rather than in the Power Automate content that focuses on background workflows.
The specific link above is no longer in this topic, it is now here:
https://docs.microsoft.com/en-us/powerapps/maker/common-data-service/configure-workflow-steps#initiating-real-time-workflows-before-or-after-status-changes


Setup for a lerna repo

Question

I'm trying to add stryker to a lerna repo. I'm getting unresolved imports for my sandboxed spec files. it's not importing my __mocks__/*.ts files.
i.e
Cannot find module './mocks/some-models' from 'myTest.spec.ts'

Stryker environment

<EMAIL_ADDRESS>+-- "@stryker-mutator/core": "2.0.2",
+-- "@stryker-mutator/html-reporter": "2.0.2",
+-- "@stryker-mutator/jest-runner": "2.0.2",
+-- "@stryker-mutator/typescript": "2.0.2",

+-- "jest": "^24.7.1",

Additional context
stryker.conf.js file
module.exports = function(config) {
  config.set({
    mutator: 'typescript',
    packageManager: 'npm',
    reporters: ['html', 'clear-text', 'progress'],
    testRunner: 'jest',
    transpilers: ['typescript'],
    coverageAnalysis: 'off',
    tsconfigFile: 'tsconfig.json',
    mutate: [
      'packages/**/src/**/*.ts',
      '!packages/**/src/__mocks__/**/*.ts',
      '!packages/**/index.ts',
      '!packages/**/*.spec.ts',
    ],
    testMatch: ['<rootDir>/packages/**/*.spec.ts'],
    jest: {
      projectType: 'custom',
      config: require('./jest.config'),
      enableFindRelatedTests: true,
    },
    htmlReporter: {
      baseDir: 'coverage/stryker-reports/',
    },
  })
}

jest.config.js
module.exports = {
  moduleFileExtensions: ['ts', 'tsx', 'js', 'jsx', 'json', 'node'],
  transform: {
    '.(ts|tsx|js|jsx)': '<rootDir>/node_modules/babel-jest',
  },
  testEnvironment: 'node',
  testRegex: '(\\.(test|spec))\\.(ts|tsx|js)$',
  transformIgnorePatterns: [
    '<rootDir>/node_modules/',
  ],
  modulePathIgnorePatterns: ['dist'],
  coveragePathIgnorePatterns: ['/node_modules/'],
  coverageThreshold: {
    global: {
      branches: 90,
      functions: 95,
      lines: 95,
      statements: 95,
    },
  },
  collectCoverageFrom: [
    '**/src/**/*.{js,ts}',
  ],
  coverageReporters: ['text', 'cobertura'],
  setupFilesAfterEnv: ['<rootDir>/fetch.js', '<rootDir>/jest-extended.js'],
  moduleNameMapper: {
    '^@getgo/(.*)': '<rootDir>/node_modules/@getgo/$1',
    '^@types/(.*)': '<rootDir>/node_modules/@types/$1',
  },
}

Hi @lbenie thanks for opening this issue.
Stryker should work with lerna, no problem (we're using it ourselves as well). Could you specify where the mocks/some-models is located on disk? Is it checked into source control?
I think you should also remove the transpiler here. Since you're using jest, it will be responsible for the transpiling (not stryker).

Hi @lbenie thanks for opening this issue.
Stryker should work with lerna, no problem (we're using it ourselves as well). Could you specify where the mocks/some-models is located on disk? Is it checked into source control?

some are located in
/packages/package1/__mocks__
and yes they are checked in source control

I think you should also remove the transpiler here. Since you're using jest, it will be responsible for the transpiling (not stryker).
Thanks I will try that!

@lbenie Were you able to solve the problem?
I have a lerna project and I have been able to run stryke on a single package and a single test. The test does not have dependencies on other packages so it works fine.
I tried with my suite of tests and as stryker is not copying the modules onto the temporal folder it does not find them.
I have also looked at the code of stryker itself to see how the project is managed but I don't see an obvious thing that I'm missing
Did you find a solution?

@lbenie Were you able to solve the problem?
I have a lerna project and I have been able to run stryke on a single package and a single test. The test does not have dependencies on other packages so it works fine.
I tried with my suite of tests and as stryker is not copying the modules onto the temporal folder, it does not find them.
I have also looked at the code of stryker itself to see how the project is managed but I don't see an obvious thing that I'm missing
Did you find a solution?

Hey man @amrondop, I haven't tried in a while. Had to get on some other features. If you find anything keep me posted!
I'll try next month when I get some more free time

Hi! I'm closing this issue for now. If it still persists with Stryker version 4, please open a new issue. Thanks!


Fix missing symlink explorer decoration if file is opened from quickpick

This PR fixes #147628

Bump @jrieken :)


refactor(v2): move scripts/stylesheets injection to server side

Motivation
Move scripts/stylesheets siteConfig client side injection to server side using injectHtmlTag API
The benefit of this approach is that there won't be weird data-helmet attribute on the injected head tags and theoretically server side should be faster than react helmet.
Have you read the Contributing Guidelines on pull requests?
yes
Test Plan
Nothing broke

Add random scripts and stylesheets into config

To consider
Deprecate scripts and stylesheets option ? But that would require v1 user migrating having to create a local plugin which can be slightly annoying

sounded a bit peculiar to me at first but actually quite legit on a second thought, smart idea （╹◡╹）

because its a global tag and not page specific.


Minor functionality broken with JQuery UI 1.12.1

Noticed the following behavior when JQuery UI 1.12.1 was used alongside this plugin:

Close button/icon inside a filtering box (which is intended to remove the filter object from the list) does not remove the item, and instead opens it for editing

Text description of filtering behaves oddly when being edited after initial creation. Would end up with odd strings like "Room does not contain 'a' does not contain 'b' does not contain 'c'"

Excellent plugin btw.

Done.


Setup wizard

Setup wizard tutorial updated with the Franka Panda arm.

don't forget to delete your branches after merging


Seeding fails to produce deterministic results

Hi, thanks for this great code. I've been using it for some experiments and have been having some issues with replicability. One thing I notice is that learning curves are different even when I try and set the same random seed. I still get different results even if I do:
    torch.manual_seed(seed)
    torch.cuda.manual_seed(seed)
    torch.cuda.manual_seed_all(seed)  # if you are using multi-GPU.
    np.random.seed(seed)  # Numpy module.
    random.seed(seed)  # Python random module.
    torch.manual_seed(seed)
    torch.backends.cudnn.benchmark = False
    torch.backends.cudnn.deterministic = True

https://github.com/pytorch/pytorch/issues/7068
Have you noticed this issue and is there a way to resolve it?
Also, a quick unrelated question: for Treasure Collection I notice that by default you use substantial reward shaping. Was this shaping used for the Actor-Attention-Critic paper? Were you able to successfully train without it?

Hi,
Thanks for the interest! Not sure exactly what the issue is, but one thing you might try is to instantiate a numpy RandomState object (with the given seed) in the environment when calling env.seed and use that in place of np.random in the env code. There might be something weird going on with setting the global random seed in each environment since they're running in parallel in subprocesses.
The reward shaping was used for the paper. From what I can remember, it may have worked a little without reward shaping but not nearly as well.

@shariqiqbal2810 Thanks a lot for your quick response.
Nice idea to fix the randomness of the environment - I just tried a quick hack by simply eliminating any randomness in the 'simple.py' environment but unfortunately am still getting slight differences in the reward curves. I'm a bit time constrained at the moment, so may have to investigate this later. Are you able to confirm that you don't run into this issue on your machine or have you not had a chance to try yet?
And thanks for the info on reward shaping. I suppose solving the environment without it would be a nice target to aim for. :)

Hmm in this case I'm not sure what the problem is, especially since you have enabled deterministic mode for cudnn. My compute resources are being 100% utilized right now, so I'm unable to verify whether I also have this issue on my machine, unfortunately. Please let me know if you figure anything out!

I understand, will do!

Just in case anyone is curious regarding my other question, MAAC does indeed solve collect treasure when shaping is set to False.  I tried it on a version of the problem with 5 agents (stopped early).

Just in case anyone is curious regarding my other question, MAAC does indeed solve collect treasure when shaping is set to False. I tried it on a version of the problem with 5 agents (stopped early).

Ah, good to know!

I am a bit mystified as to what changed, but fixed random seeds are now giving the desired fixed results, even in random environments, so will take the win!

If someone else runs into this issue perhaps try:
    torch.manual_seed(run_num)
    torch.cuda.manual_seed(run_num)
    torch.cuda.manual_seed_all(run_num)  # if you are using multi-GPU.
    np.random.seed(run_num)  # Numpy module.
    random.seed(run_num)  # Python random module.
    torch.manual_seed(run_num)
    torch.backends.cudnn.benchmark = False
    torch.backends.cudnn.deterministic = True

Great! Perhaps it's a package version thing?

It's possible but I have no idea to be honest!


function naming is ambiguous

Just some suggestions in function naming
I find the usage ambiguous (the compiler doesn't like it as well)
esp_err_t outputVoltage(uint8_t value);
esp_err_t outputVoltage(float voltage);

The compiler still prefer double over float
maybe change it to:
esp_err_t outputValue(uint8_t value);
esp_err_t outputVoltage(float voltage);

Hi there, calling an overloaded function without typecasting the argument(s) passed is generally not a good idea. The compiler might speculate right, often not. In latter case the problem can stay unnoticed esp. without compile error.
Below line will compile without hassle and delivers the result you want:
...
dac.outputVoltage((float)(sin(f) * 1.5 + 1.6));
...
When coding don't let the compiler guessing. Try to make sure the argument is interpreted properly and all participants are happy. Regards Th

Just as a further remark, the floating argument passed to outputVoltage() is supposed to be in the range 0...CHANNEL_VOLTAGE_MAX (3.3 by default). An argument with higher value than CHANNEL_VOLTAGE_MAX is accepted though but always results in max possible output voltage.

The reason I found it ambiguous is that the overload both accept one numerical argument, however they accept arguments with different meaning. (One is byte range, the other is actual voltage) Directly naming the byte range one outputVoltage might not be a good idea since it's not a voltage, so I want to change it into a more declarative name.
(Sorry for not being clear enough on the issue description)
On the other hand, I accept the point that calling overloaded function without type casting is a bad idea, and changing function name is a breaking change, which might be a worse idea.

Well, I get your point. I named it outputVoltage() because arguments of both types will finally result in a certain "output voltage" level, represented by the functions argument. I guess using float makes it easier for humans to recognize/read without having to calculate uint8Value / 255 * 3.3V.
Applying a typecast in code however is a very precise and declarative way to say what you want and what the argument represents. Maybe that helps...

Yeah. Maybe I'll close the issue?

Just done. Thanks for the discussion. Regards Th


coder: init at 0.12.4

Description of changes
Fixes #198207

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

I think this package does not work as expected as it does not include the web UI (Try running coder server and visiting the url provided).
There is a much more detailed PR open currently, which also includes a nixos module - https://github.com/NixOS/nixpkgs/pull/197613

I did describe this change as "I have not tested the output binary", but it does compile. I was hoping that we might get some feedback from real users, as I was packaging it.
Should we merge another change to mark this as broken?


Large eddy simulations of atmospheric boundary layer

Dataset Name
ABL-LES
Dataset URL
No response
Description
The ABL-LES dataset simulates the atmospheric boundary layer across a wide range of turbulent regimes, from channel flow to strong convective cases, with various boundary conditions (e.g., surface heating, geostrophic wind, free tropospheric lapse rate). The data also includes several implemented scalars (e.g., chemicals with various sources). This dataset is ideal for studying turbulence and transport under a wide range of conditions, making it great for data-driven approaches, specifically for developing parameterizations of the atmospheric boundary layer that can be implemented in climate or weather models.
Size
10-15 TB
License
Unknown
Data Format
NetCDF
Data Format (other)
No response
Access protocol
Globus
Source File Organization
Data are organized by the simulations (set of boundary conditions), and further diveded to snapshots, and statistics.
Example URLs
No response
Authorization
Globus login
Transformation / Processing
Data does not require cleaning but it requires to be better organized
Target Format
Zarr
Comments
No response

Hey @Sshamekh, thanks for submitting this. Where is the data accessible?

Hi Julius,
They are on NCAR HPC

Ah ok that makes things (possibly) harder. Can you ask if there is a way for them to make them accessible via http or ftp in some way?

Thanks Julius.
I will reach out to the NCAR helpdesk to inquire about HTTP or FTP access. However, I wanted to check if using Globus would also be an option beforehand.

Possibly. Could you check if they subscribe to the Globus S3 Connector? If that is the case we might be able to move the data to an OSN bucket.

Actually did you ever get an answer to this? If they have their own setup, that might make things easier.

Here is their Globus : https://ncar-hpc-docs.readthedocs.io/en/latest/storage-systems/data-transfer/globus/
But they didn't mention S3 connector. I will check with them

Hi @Sshamekh, just wanted to check in on the status of the transfer? Do you have some time to let it run some time soon?


Incompatiblity with More Axolotl by KxmischesDomi

Mod Name
More Axolotl
Mod Homepage
https://www.curseforge.com/minecraft/mc-mods/more-axolotls
Minecraft version
1.20.1
Describe the bug
Failing during launch
Steps to reproduce
1.Have the connector and reforiged fabric mod (i tried several versions connector and reforgied fabric and presented the last crash along with the newest crash) and More Axolotl 2.0.1-1.20
-also had infinite banners installed which works great with your mod
2. Launch the game
3. Watch it fail to launch because it said its unable to read 'main' due to errors
...
Logs

Additional context
crash-2024-03-19_04.18.26-client.txt

Duplicate of #674


Initial commit.

Extracted from https://github.com/digitalbazaar/http-signature-header/pull/19
Extracted from https://github.com/digitalbazaar/http-signature-zcap-invoke/blob/master/main.js#L35
WIP, todo:

[ ] Find an isomorphic way to base64-encode strings (our base64url-universal does not expose non-url encoding)
[ ] Find some test strings for multihash digest?

Switched to using universal base64 and base64url libs. Except that something is off with the karma tests, giving me
AssertionError: expected 'SHA-256=OTUsMTQzLDQsMjQ2LDE2MywxNjgsMTQ2LDE3MCwxNzEsMTg5LDIxOSwxMDgsMjQyLDExNSwxMzcsNjgsMTQ3LDExOSw1Nyw5NiwyMTIsMTYzLDM3LDE3Nyw1LDI1NCwyMjgsMTEwLDIzOSw2Nyw0LDI0MQ==' to equal 'SHA-256=X48E9qOokqqrvdts8nOJRJN3OWDUoyWxBf7kbu9DBPE='

Investigating.

please merge.

Thanks @dlongley, will clean those up now..

@OR13 @dlongley also renaming this repo to http-digest-header (since now Digest header has its own spec).


pkg/k8s: fix k8s_event_lag_seconds for negative time

In some occasions the metric k8s_event_lag_seconds could be presented
as an overflown value such as<PHONE_NUMBER>854775807. This commit fixes
this by checking if the calculated value is less than zero by only
setting this metric for positive times.
Fixes: 4e2913004340 ("pkg/endpoint: calculate Kube API-Server lag from pod events")
Signed-off-by: André Martins<EMAIL_ADDRESS>Fixes: https://github.com/cilium/cilium/issues/14309
Fixe possible overflow in values presented in the `k8s_event_lag_seconds` metric.

test-me-please

hit https://github.com/cilium/cilium/issues/14125

retest-runtime

hit https://github.com/cilium/cilium/issues/14330


Support backing up user-provided list of files

Output of restic version
restic 0.9.6 (v0.9.6-204-gf0b2dee7) compiled with go1.12.9 on linux/amd64

What should restic do differently? Which functionality do you think we should add?
Support a well-specified and simple way for users to tell restic to backup a specific set of files.
For example, consider this JSON list of files and assume that there are files with these exact names on the filesystem
[
  "/src/**",
  "/src/\n",
  "/src/\\"
]

restic could accept them as a JSON array, like
echo '["/src/**","/src/\n","/src/\\"]' | restic backup --files-from-json -

Alternatively, restic could accept them with NUL-delimiters, like
printf '%s\0' '/src/**' '/src/\n' "/src/\\" | restic backup --files-from-z -

which aligns with similar features in several other tools for producing output and receiving input in a consistent way: find (-print0), xargs (-0), git ls-files (-z).
Personally, I prefer the JSON-based approach since we can provide more meaningful errors, and there are fewer edge cases or decisions to make around the parsing of it (like how to treat an empty field or whether to consider a trailing newline as extraneous or as a path), but it may be a little less conventional.
To be forward-compatible, it may be worth putting the list of files under a key, like {"files": []}. I really do not want restic to do any processing at all here, but later someone may want to be able to provide globs in a similar way, without the caveats of --files-from. {"patterns": []} could be simpler than another argument in that case.
What are you trying to do? What problem would this solve?
I have a specific list of files to back up, and want to use restic to do it. I want to back up only these specific files, and the paths may contain any character the OS/filesystem considers valid.
Currently, to be understood by restic in a compatible way, I have to carefully consider each path. The logic looks like:

Leave $ as-is, since $ does not need to be escaped in the paths provided via --files-from (as opposed to exclude files)
Make every path absolute

avoids needing to escape leading whitespace, which would be trimmed if in a --files-from file
avoids needing to escape a leading #, which would cause the line to be skipped if in a --files-from file

Pass paths that contain newlines via positional command-line arguments

files passed to --files-from are split by newline, so newline-having paths would get split incorrectly
currently, no other processing is done on paths provided this way
only passing newline-having paths as arguments, as opposed to all paths, reduces the chance of hitting an OS-specific limit (for example, ARG_MAX on Linux)

For paths without newlines, escape as described below and save them one-per-line to a file and pass the file path to --files-from

Surround each [, *, and ? with []

necessary since all paths derived from --files-from are passed through filepath.Glob
putting special characters in a character class causes filepath.Glob to treat them as normal characters, without relying on \-escaping, which is not supported in filepath.Glob on Windows

If there is any trailing whitespace, surround the last character with []

currently, restic strips any leading and trailing whitespace from each line derived from a --files-from argument
preserves the trailing whitespace by eliminating it in a way that will be interpreted correctly by filepath.Glob

On non-Windows OSes, replace \ with \\

\\ will be interpreted correctly as \ by filepath.Glob

Which has these downsides:

Not obvious at all
Requires additional code to integrate with restic in a generic way
Minor performance drawback since we may be globbing for what are essentially plain paths to individual files, and taking the slower path in the Go std library
Very fragile. Any of the above could easily change if untested, and I wouldn't hold it against anyone that broke it either

If restic supports this feature, then none of the above is necessary and it gets a lot easier to use restic as a building block for different backup workflows.
Did restic help you today? Did it make you happy in any way?
Yes! I like the content-hash-based de-duplication a lot, especially where the modified time of a file may change, but not its contents. Other approaches like rclone with a crypt backend would unfortunately reupload the file.

Do you know about restic backup --files-from?

@chrahunt Thanks for being so elaborate in your description, that's appreciated!
That said, do you have an actual concrete/practical use case where you need this? I have a hard time relating to anyone I know ever having needed this. Just as an example, when would you have newlines in your filenames?
The use case strikes me as very uncommon and for such a specialized case I spontaneously think it might be fair to have to do that externally.

do you have an actual concrete/practical use case where you need this?

I don't want to rely on any specific app's include/exclude rule syntax or restrictions so I generate my own list of files to backup. I have a script that generates those files so I don't have to manually consider every single file. If any of those files happen to have any of *, \, \n, [, or ? then just writing them into --files-from would lead to undesired behavior. That's not acceptable to me if I can help it. I need to run on both Linux and Windows, so I can't use \ to escape characters. I can't guarantee that my files do not have newlines, so I can't blindly write into --files-from.

Just as an example, when would you have newlines in your filenames?

There are two situations I've seen this specifically:

apps that use user-provided text as a filename without any encoding
test files, like those used to test that a script correctly reads and escapes filenames that contain \n :)

To me, omitting these files from the backup is not ideal.

The use case strikes me as very uncommon and for such a specialized case I spontaneously think it might be fair to have to do that externally.

I don't think this is that uncommon. Consider #2276, #2448, and other forum posts. They essentially boil down to "provide restic a list of paths to back up", but users are getting caught off-guard by the interpretation of the paths. In some cases users are stuck with sub-optimal workarounds because of platform limitations.
We could also consider that similar features exist in other backup tools:

rsync has --files-from, which takes newline-delimited paths verbatim and --from0, which causes --files-from to expect null separators instead
rclone has --files-from-raw, which takes newline-delimited paths verbatim
duplicity has --include-filelist, which takes newline-delimited globs and --null-separator, which causes --include-filelist to expect null separators instead
borg: has --patterns-from, which takes a newline-delimited of patterns, prefixing a line with pf: causes that line to be interpreted verbatim


Make OnlineStat usable as a reducing function

Codecov Report

Merging #264 into master will decrease coverage by 6.57%.
The diff coverage is 70.00%.

@@            Coverage Diff             @@
##           master     #264      +/-   ##
==========================================
- Coverage   90.71%   84.13%   -6.58%
==========================================
  Files          20       20
  Lines        1346     1349       +3
==========================================
- Hits         1221     1135      -86
- Misses        125      214      +89

Impacted Files
Coverage Δ

src/interop/onlinestats.jl
66.66% <70.00%> (-33.34%)
:arrow_down:

src/show.jl
68.50% <0.00%> (-21.34%)
:arrow_down:

src/basics.jl
66.66% <0.00%> (-19.05%)
:arrow_down:

src/progress.jl
75.78% <0.00%> (-12.64%)
:arrow_down:

src/processes.jl
77.01% <0.00%> (-10.35%)
:arrow_down:

src/reduce.jl
70.00% <0.00%> (-10.20%)
:arrow_down:

src/library.jl
91.78% <0.00%> (-2.24%)
:arrow_down:

src/core.jl
88.88% <0.00%> (-1.59%)
:arrow_down:

src/unordered.jl
95.74% <0.00%> (-0.41%)
:arrow_down:

Continue to review full report at Codecov.

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


Filters are lost in both list & table views when navigating to next page

Describe the bug
When filtering the list/table views that have enough results to be paginated, selecting Next/Previous/Last/First at the bottom of the view will result in navigating to the selected page of All Submissions rather than the filtered criteria
To Reproduce
Steps to reproduce the behavior:

Navigate either the list or table view
Use a broad filter to cause pagination (ie. No Screening)
Scroll down to the page nav buttons at the bottom (ie. Next/Previous/Last/First`)
Select any of the buttons & notice page has been changed but filters are gone

Expected behavior
Filters are preserved while the page is changed
Priority

Medium (slowing you down)
Screenshots

Additional context
Manually entering the page query param in the URL causes the filter to be preserved while navigating to the intended page.

Definitely a bug. The search query is missing from the pagination links, /apply/submissions/all/?page=2 instead of e.g. /apply/submissions/all/?page=2&status=dismissed.
If I manually enter /apply/submissions/all/?page=2&status=dismissed it works as it should so should be an easy fix.

sweet! I think I fixed it w a pagination tag to add the params


Question: is onEntry supposed to be executed for the initial state?

Bug or feature request?
Description: In the example lightMachine, I added onEntry for the initial state "green" as follows:
const lightMachine = xstate.Machine({
key: 'light',
initial: 'green',
states: {
green: {
on: {
TIMER: 'yellow',
POWER_OUTAGE: 'red'
},
onEntry: 'setLightGreen',
},
....
I expect setLightGreen to be executed as soon as lightMachine is created because it would transition to "green" as the initial state. But it wasn't. Did I misunderstand the default behavior?
(Bug) Expected result:
(Bug) Actual result:
(Bug) Potential fix:
Required! If you found a bug and can consistently reproduce it, please take a minute to think about what the potential fix might be.
Even if the fix is incorrect, this can help diagnose and find the solution to the issue as quickly as possible.
I also strongly encourage you to create a pull request if you are confident in the potential fix.
(Feature) Potential implementation:
Required! If you would like to see a feature or enhancement make its way to xstate, please describe:

What the API would look like
If this is a breaking change
If this is part of the existing SCXML specification

Link to reproduction or proof-of-concept:
You can use CodePen or Code Sandbox to create a reproduction/POC.

It should be and it is in the master branch. Minor release coming soon!

Yes, always.


CAPI: Experiment Decision Time

🚀 Deployed on https://657c9ccac7a0a51d1493b061--cozy-fox-0defec.netlify.app


Prints warnings abut missing .eslintrc.js and "undefined" on console

Each time it completes processing a module ts-for-gjs prints "undefined" on the console. I haven't managed to find out what's causing this.

I was going to report a separate issue, but they might both be caused by the same thing. Just before each "undefined" message I get a pair of warnings about ESLint like this:
Error: No ESLint configuration found in /home/tony/Code/tvfrontend/gjs/tsforgjs/@types/Gjs.
[gjs][GLib-2.0] WARN: Can't prettify file<EMAIL_ADDRESS>please check your .eslintrc.js in your working directory

Could the program check a .eslintrc.js exists before trying to run ESLint, or perhaps provide a suitable template?

I think I can fix this when I am back from vacation

Should be fixed in #45


Use SuspendingObservableCollection

Suspend usage of ObservableCollection when doing mass updating.  Only have one and modify it.
CNX-799
https://linear.app/speckle/issue/CNX-799/observablecollection-errors

Can we put things on an interface and/or comments about what is needed for the javascript UI?

Can we put things on an interface and/or comments about what is needed for the javascript UI?

@adamhathcock
Initially DiscriminatedObject was the idea of this to mark they are data classes for JS. But I guess it is semi-broken now. You are right. We need those interfaces to imply that data/prop is needed for UI.


⚠ DSO-API has degraded performance

In 1fb2ee8, DSO-API ($SITE_URL_DSOAPI) experienced degraded performance:

HTTP code: 200
Response time: 7850 ms

Network hickup.


Add changelog or release notes

This is part of doing releases better. Ideally I would like to see bullet points about what changed in each version on the releases page. Why would I upgrade to 0.4.0? What are the risks?

Yes, this is a big part of what I want to do for 1.0. I'm not doing it a whole ton right now because the changes are pretty much internal; I've been trying to get rid of the hand-rolled parser for a long time.
I also need to thoroughly document semver-parser as well.

It looks like 1.0 was a breaking release because of (at least) the new Error type.  Would it be possible to add notes to the 1.0 release in GitHub (or wherever is appropriate) about the changes and what users should do to feel confident in upgrading?


FYI: ROVA is no longer on Afvalwijzer

Feature request
Hi @xirixiz,
As an FYI: ROVA is no longer on Afvalwijzer. It uses a custom API from now on.
You can look at my integration if you need help with the new API:
https://github.com/pippyn/Home-Assistant-Sensor-Afvalbeheer/blob/b9898ade57533136f63a0a7c37e1f36c77625968/custom_components/afvalbeheer/API.py#L1062
Kind regards,
Pippijn
What version of the integration are you using?
N.A.
Anything in the logs that might be useful for us?
No response
Additional information
No response

Hi, this is odd. Somehow I missed all open issues. I just got a new laptop and suddenly a lot of open issues are visible. I`ll have a look and come back to it asap. Thanks!

Added to a new release. Thanks pippyn, very much apreciated!

ROVA is still not updating for me.:

Do I need to change anything in my config?


option to not use orm ?

Hello,
I have a project without use database. When I can ignore connexion ?
Thanks

I seem to found solution : desactivate in app.ts lines of typeorm ....


Clarification requested on invalidation

From the readme:

Invalidation
This cache is updated when an entity is changed (insert, update, or delete) via a DbContext that uses this library. If the database is updated through some other means, such as a stored procedure or trigger, the cache becomes stale.

I expect that the cache is updated when an entity is changed in the relevant DbSet.  Is that correct?  Or are all caches invalidated when any entity has been touched?

Is that correct?

Yes. From the final SQL query -> Extracts the real in use table_name's from it -> To create the cache dependencies. Then after each CRUD operations these related dependencies will be invalidated.

Is that correct?

Yes. From the final SQL query -> Extracts the real in use table_name's from it -> To create the cache dependencies. Then after each CRUD operations these related dependencies will be invalidated.


[Icerberg REST] Namespace separator mismatch

Describe the bug
The current Iceberg dependency version is 1.5.2, which uses %1F as namespace separator character in its client implementation.
Unity however uses . character which causes client calls such as listTables() or loadTable() to fail if a Namespace is defined with more levels.
To Reproduce
Steps to reproduce the behavior:

Create a uniform table such as unity_test.default.mytable
Use the Iceberg client and call listTables(... , Namespace.of("unity_test", "default"));
This yields to 400 Bad request

Expected behavior
Retrieve a TableIdentifer for mytable.

Thanks @szlta - we will review this shortly!

Hi I will be looking at making some changes to address this


fix-#445: Absence of a Proper Loading skeleton for the AddBlog Section #pr446

Summary
This PR provides a Loading Skeleton for the AddBlog section so that the user has a seamless experience
Description
AddBlogPostSkeleton Component has been added and has been used to replace the previous Loader component that was used.
Images

Issue(s) Addressed
Closes #445
Enter the issue number of the bug(s) that this PR fixes
Template should be strictly Closes <issue_number>
Example: Closes Centralised Error Handling #1
Prerequisites

[ X] Have you followed all the CONTRIBUTING GUIDELINES?

i have made the changes

Please add snapshots of both the loaders

i have added the screenshots, in the below screenshot of the loader working the footer is also seen, my screenshot is unable to capture it. But in both the loaders the footer is seen. @krishnaacharyaa  i have added the screenshots.

Hey sorry yaar @MetiUmashri , i thought i had commented, figured out i hadn't published it was in pending state, sorry for that, a bit busy with workplace, excuse me for the delay

Hey @krishnaacharyaa
@MetiUmashri  and I thought of improving the next loader in a different PR 😁
Both of us would like to work on this as well....


BUG: WEBSITE IS NOT WORKING

WE SHOULD DEPLOY IT PERFECTLY..

/assign

@sanjay-kv is there any environment variable ? if yes then please tell me

@sanjay-kv Please Review PR..

@sanjay-kv I deploy the website you can use it..


Small fixes for Patch 2.13

Updated RED4ext.SDK

Updated AddressHashes::CPhotoMode_SetRecordID

Note: CScript_RegisterMemberFunction seems to not be used by anything

Seems to work by my untrained eye, I can run around using CET mods and spawn NPCs

Done


Don't rearrange views behind swaylock.

This fixes #481 by making the swaylock views floating so they don't affect the arranging of other tiled views on the same workspace.

Nice. Will merge when build passes.


Fix #55: allow null group and subtitle

@akbarpulatov are you planning to merge this change and create a new version on pub.dev?


Proposal: alternative approach to root dependencies

Was talking about root dependencies. Manypkg breaks from its predecessor bolt in using versions within leaf* package.json files, and allowing them to be excluded from root AND that all dependencies in root must be listed under dependencies, not devDependencies
The benefits of the new approach:

Calculable source of truth around package versions between packages
packages become easier to remove - packages not used anywhere simply vanish
shorter root package.jsons

The benefits of the old approach:

single source of truth that can be referenced for what packages at what versions are installed

So question 1 then becomes "How important for repository maintainers is having a single source of truth of what packages at what versions exist entirely within the multi-package repository"
I've been talked around that having this single source of truth that is easy to reference is genuinely valuable.
There's an... alternate new approach if we want:

single source of truth that can be referenced for what packages at what versions are installed
packages become easier to remove - packages not used anywhere simply vanish

The new rules are:

All installed packages used in leaf packages must exist in dependencies in the root package.json
All installed packages not used in leaf packages must exist in devDependencies of the root

Complaints with this approach

This makes the tooling harder to understand, as it has more specific rule about where to place things

True, but we already have a lot of opinions, and things remain as easy to alert and fix as they currently are (thing mentioned in a package but not at root is a new fix, aligning versions changes slightly)

people can throw everything in dev dependencies and then we are no better off

A risk, but also means people unconcerned with removing packages just don't care.
Three possible decisions

Stick with what is in manypkg now, decide single source of truth for package versions is an okay trade-off
1a. Add a manypkg command to get this list, to counteract the trade-off
Revert to bolt's rules, add everything into dependencies still, decide removing unused packages being harder is an okay trade-off
Implement the new option proposed here, decide the added rules complexity is an okay trade-off.

*New term - picked 'leaf' to match 'root' - may be bad. Technically we only have root and non-root, but that's not ideal either.

I've been talked around that having this single source of truth that is easy to reference is genuinely valuable.

I'd love to hear more about this, what it's used for, etc. I haven't personally found that as a thing I need but I assume this a Atlaskit and other things scale that makes it more useful?

1a. Add a manypkg command to get this list, to counteract the trade-off

Isn't this necessary for all three of the options? I would kinda argue it's more necessary because if I want to add a new dep on a package, I have to add it in two places with options 2 and 3 but only one with option 1.

I assume this a Atlaskit and other things scale that makes it more useful?

This. The use-case is effectively to help people who are maintaining a repository but aren't directly responsible for individual code. Atlaskit hits this a lot, but as noted, we don't as much because often the people maintaining the repository are very close with a lot of the code in the repository.
It's also in part a cost/benefit thing. Having a long list of root deps is low cost is an assumption.

Isn't this necessary for all three of the options?

Assuming we're still adding manypkg add, my assumption is that it would take care of the double add, and manypkg --fix ends up with a tweaked behavior, but is equally solvable (it's what bolt-check is already doing I think)
I think of it as 'run one command to add in any of the three scenarios, fix does something different'.
On 1a specifically, if the list is just embedded in the root package.json, there's no reason to add a command to generate the list of all dependencies and their versions.

root deps thing

Would you have any examples of where they use it?

Having a long list of root deps is low cost is an assumption.

I kind of think it might not be a good thing because I've been looking at https://pnpm.js.org as a thing that we might want to use(very much a probably not but maybe thing), largely because it's more strict about not letting workspaces access things that it doesn't actually depend on but not as "break the whole ecosystemy" in the way Yarn PnP is and having things in the root deps means anything can import it which is annoying because the ESLint rule isn't perfect for things like @babel/runtime because the import is written by a babel plugin.

command thing

Whoops, sorry. I read it wrong and thought you meant we would only need an add command for option one. Makes sense.

it's more strict about not letting workspaces access things that it doesn't actually depend on

tbh, this is a huge win to me and worth keeping on track with our current approach. Will discuss more with Luke sometime but I think we don't want to pivot. The conclusion here is that things not at root is not low cost, even though it may appear low cost. It leads to a subtle annoying error class.

Would you have any examples of where they use it?

Occasionally audits are done around pulling in multiple similar packages, or what the most bloat-iest packages in a repo are.
If a design-system is pulling in three different external select components, or has 4 ways to pull in lodash that means if some-component is rendered it pulls in all of lodash, AND something it depends in pulls in lodash.snakecase and some more nonsense like that, seeing these listed in the root allows you to do an 'aha, a thing to fix'.
Additional things is packages that people keep adding which you have banned from the repository (roll your own is lowish effort and package is way too heavy), this makes it clearly surfaced.
There's other things which you keep track of but it's not really where you're tracking them. A single react version for example is required, and the repo maintainer probably has strong opinions about what that is (and when upgrades happen). Since these lists are so long, it would be a nonsense to say that this is where the information for this 'lives'.

So I think we’re happy with a command that lists all the deps and probably also a package to get it programmatically?

Yep. Closing this and making that issue


Add image handling package(s) to Tcl

From @GoogleCodeExporter on May 31, 2015 11:25
How would I go about adding image handling for Tcl?

I am personally not familiar with the structure of the interpreters for
this environment, and don't have the know-how to patch them myself... Is
this an exceptionally difficult request?

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 1 Apr 2010 at 7:25
Copied from original issue: damonkohler/android-scripting#275

From @GoogleCodeExporter on May 31, 2015 11:25
Tcl was provided by an external contributor. His web page is here:
http://www.patthoyts.tk/tclkit/

I'd suggest getting in touch with him about it.

Original comment by<EMAIL_ADDRESS>on 2 Apr 2010 at 12:33

Changed title: Add image handling package(s) to Tcl


Refactor into plain JS (remove typescript)

I want to merge this into the main repo, and introducing typescript into the main repo is a bad idea at this point.

Typescript has had some major releases in the meanwhile, I've tried updating, but i know very little of typescript and the changes they went through, and got stuck.

I've just had a try at this, will create PR soon

Awesome @psimyn !


Support the simple_pattern and simple_pattern_split tokenizers

Hello!
Recently I wanted to use simple_pattern_split tokenizer and didn't found a way to set it up with TokenizerDescriptor. I used pattern tokenizer instead but faster simple_pattern_split is more optimal for my case because the pattern is very simple.
So I thought maybe it will be a good idea to support simple_pattern_split and simple_pattern in Nest. Something like:
await elasticClient.Indices.CreateAsync(
    config.IndexName,
    c => c
        .Settings(s => s
            .Analysis(a => a
                .Tokenizers(t => t
                    .SimplePatternSplit("email", f => f.Pattern(@"\@|\."))))))

Thanks for raising this @tkirill. This is certainly something we can and should look at supporting via NEST. I'll take a look at this.

@tkirill I've merged in the inclusion of these tokenizers in NEST. Those will be available in the 7.14.0 release.


check compatibility with hop 0.10

See if hop 0.10 can be supported. It looks like we've been relying on hop 0.9 for at least 18 months.

Fresh installs pull in 0.10 on their own.  So, updating a snews_cs then running through the canned battery of tests might be an easy test.  Will report results here, could do this as part of the next mini-firedrill.

Melih ran the server with 0.10, and me with three 0.10 clients, and the comms all worked ok for a coincidence test.
Running the checks with 0.10 and the server was at 0.09 also worked.


coding guidelines

As part of #60 I want to enable all clippy lints that seem useful.
This also includes adding documentation to all methods and modules in the backend.
After all clippy lints are active there might not even be a need to write additional coding guidelines (as clippy can be quite strict).

@markus2330 Please have a lot at the metadata I added to the toml file. I put in there what clippy wanted and tried to piece together what I think would make sense.

I'm kind of finished with what I can do in this PR for now (not yet to be merged).
I documented where I could in code and left TODOs where I didn't really know what to write (e.g. struct fields that represent the database). For that I would need some good documentation or some help to tell me what the individual fields are actually for (maybe discuss that on Tuesday or somebody helps me after the meeting).
Then there is still #![allow(clippy::module_name_repetitions)]. I don't actually know whether we want that, but I can't really do anything about that currently as I'm kind of blocked from renaming file by #48 (we did some renaming there and that would result in a huge merge conflict mess).

@markus2330 To view the docs type cargo doc --open and the generated html files should open in the browser. It's easier to look through code that way.

@lukashartl Once this PR is merge you probably need to change CI. There should be an additional check whether documentation links are broken. The command for that is cargo doc.
@markus2330 It is possible to set a flag in rust to deny warnings (https://github.com/rust-lang/cargo/issues/8424#issuecomment-774662296). We might want to enable that.

@lukashartl Once this PR is merge you probably need to change CI. There should be an additional check whether documentation links are broken. The command for that is cargo doc.

@markus2330 It is possible to set a flag in rust to deny warnings (https://github.com/rust-lang/cargo/issues/8424#issuecomment-774662296). We might want to enable that.

I created #98 to track this, the deployment is higher priority though.

Great work! Now please concentrate on documentation, architecture, big picture of backend etc., so that we can finish the first milestone without bigger merge conflicts.

Great work! Now please concentrate on documentation, architecture, big picture of backend etc., so that we can finish the first milestone without bigger merge conflicts.

There are still a few smaller issues open. I will quickly fix them and afterwards start with the doc/decisions.


footer

the footer UX can be made better as it looks like two footers overlapping which can be done in a single one clearly
please assign this to me under gssoc

#95 , add your comment here, @Lakshyyaa . Thanks.


Input component style issues in dark theme

It is a slightly different color from the form background, but only barely:

In the same vein, border on TextArea:

Thanks for posting - just noticed that and came here to say the same!

Like the first one - both an improvement of course but first one seems like it would be a clearer contrast with the on position.

Ok, agree - the only concern I had about using a box shadow was that it overrides an !important rule in blueprint.css (see lines 2010,2011). Kind of breaking a pattern but I think it makes sense for this case.


convolution encoder better result then vit

Thank you for this work.
The generalizing ability of neural networks based on convolution layers is much greater.
class Model(torch.nn.Module):
    def __init__(self):
        super(Model, self).__init__()
        from efficientnet_pytorch import EfficientNet
        self.model = EfficientNet.from_pretrained('efficientnet-b4')
        self.model = EfficientNet.from_pretrained('efficientnet-b4')
        self.model._fc = torch.nn.Linear(1792, 1024)
        self.conv1D = torch.nn.Conv1d(1, 128, 3, padding='same')

    def forward(self, x):

        x = self.model(x)
        x = torch.unsqueeze(x, 1)
        x = self.conv1D(x)
        # return (batch, seq, dim)
        return x

thanks！could you give me a pretrained mdoel ?

Yes!Check on validation dataset.(Train, Test, Validation)On this week.Is that okay with You?Good luck!05:40, 4 марта 2023 г., "JackWhite-rwx" @.***>:
thanks！could you give me a pretrained mdoel ?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>
-- Отправлено из мобильного приложения Яндекс Почты

thank you very much!

Jack Please read the readme file.The model is retrained, but shows good results. VIT did not recognize most of the validation dataset. This network recognizes almost everything, but writes a lot of associations that are not in the photo. This happened after retraining the model.I hope this helps You. 04.03.2023, 06:25, "JackWhite-rwx" @.>:  thank you very much!—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.>
К письму приложены файлы на Яндекс Диске: CoCaPretrain.zip (315397104)

Thank you again,but I can't find the 'CoCaPretrain.zip' file in the attachment.Could you tell me where to download it?
------------------ 原始邮件 ------------------
发件人:                                                                                "lucidrains/CoCa-pytorch"                                                                                @.>;
发送时间: 2023年3月9日(星期四) 凌晨5:02
@.>;
<EMAIL_ADDRESS>主题: Re: [lucidrains/CoCa-pytorch] convolution encoder better result then vit (Issue #14)
Jack Please read the readme file.The model is retrained, but shows good results. VIT did not recognize most of the validation dataset. This network recognizes almost everything, but writes a lot of associations that are not in the photo. This happened after retraining the model.I hope this helps You. 04.03.2023, 06:25, "JackWhite-rwx" @.>:  thank you very much!—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.>
К письму приложены файлы на Яндекс Диске: CoCaPretrain.zip (315397104)
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>

Does your mail server not allow you to download attached files?You could not open the email via gmail or yandex.Or report the gmail address.If you do not have such an opportunity, I will put it on google disk and give you a link.Which option will suit you?Message ID: @.***>

Yes，I can not receive the file at qq email. Could you put it on google disk and give me a link, or please send the file to    @.***     . Thank you very much.
---Original---
From: @.>
Date: Thu, Mar 9, 2023 15:52 PM
To: @.>;
Cc<EMAIL_ADDRESS>Subject: Re: [lucidrains/CoCa-pytorch] convolution encoder better result thenvit (Issue #14)
  Does your mail server not allow you to download attached files?You could not open the email via gmail or yandex.Or report the gmail address.If you do not have such an opportunity, I will put it on google disk and give you a link.Which option will suit you?Message ID: @.>
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.>

Please let us know, if possible, how satisfied you are with the result of CoCa + EfficientNet_b4" ThanksMessage ID: @.***>

Jack sorry, could you give an estimate CoCa + EfficientNetMessage ID: @.***>

Of course, I will give you feedback when I have finished testing on the few shot classification and other low level task. Sorry for the delay in replying to your email
---Original---
From: @.>
Date: Tue, Mar 14, 2023 06:03 AM
To: @.>;
Cc<EMAIL_ADDRESS>Subject: Re: [lucidrains/CoCa-pytorch] convolution encoder better result thenvit (Issue #14)
  Jack sorry, could you give an estimate CoCa + EfficientNetMessage ID: @.>
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.>

Now I am training another network based on the transformer architecture. They have the same mistakes. I believe the inaccuracy of coca net is not in the architecture, but in the dataset. There are topics in which the network works perfectly. Others have a lot of excess, a lot of associations.Take a look at this image.sunrise or sunset on the river bank.The network writes that a man is surfing.During the training in the dataset there was no description of nature, sunrises, sunsets. She saw this background when a man was surfing. This is what I call an association.And such examples can be seen throughout the dataset. To get the desired result, you need to correctly create a dataset by topic, this dataset is created from random images.a person holding a surfboard standing on a beach . a person holding a surfboard near the waves in the ocean . a shirtless man standing on the beach  holding a red surfboardMessage ID: @.***>


DataFrames with DatetimeIndex have dates rendered off by a day.

Hi, I was also looking into rendering dataframe with timestamp (mine has microseconds as precision). Is there a way to increase the time resolution for the qgrid dataframe filter tool?

@epifanio there's no way to do that, sorry.  I don't expect we'll add support for microseconds anytime soon because it's a pain to handle in Javascript.

Fixed a while ago.


Ensure progress widgets respect UseVT

Otherwise the Spinner and Progress widgets merrily set the foreground colour but TextFormat::Default is never sent because the Reporter actually checks the VT setting!
Fixes #665
Microsoft Reviewers: Open in CodeFlow

/azp run


Panic in wgpu_render_pass

Bevy version
This bug is confirmed to not occur on 6b004f7d16795c413ca64ff8e3973dbb453da66c, but it does occur on f54eb12efc3dc7fd2cedfa13726808152443b087. The versions in between don't compile for me, but I'm assuming this issue is related to the recent rendering optimizations.
Operating system & version
macOS 10.14.6
What you did
Minimal repro:
use bevy::prelude::*;

fn main() {
    App::build()
        .add_plugins(DefaultPlugins)
        .add_resource(Option::<Entity>::None)
        .add_startup_system(startup)
        .add_system(sys)
        .run();
}

fn startup(commands: &mut Commands, mut entity: ResMut<Option<Entity>>) {
    commands.spawn(SpriteBundle::default());
    *entity = commands.current_entity();
    commands.spawn(Camera2dBundle::default());
}

fn sys(
    mut meshes: ResMut<Assets<Mesh>>,
    mesh_query: Query<&Handle<Mesh>>,
    entity: Res<Option<Entity>>,
) {
    if let Some(entity) = *entity {
        let mesh = mesh_query.get(entity).unwrap();
        meshes.get_mut(mesh);
    }
}

What actually happened
The window opens and renders a gray screen briefly, before panicking with this error message:
thread 'main' panicked at 'called `Option::unwrap()` on a `None` value', /Users/martin.svanberg/.cargo/git/checkouts/bevy-f7ffde730c324c74/f54eb12/crates/bevy_wgpu/src/wgpu_render_pass.rs:44:66
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

Resolved by #984


feat!: square size independent message commitments

Description
Implementation for square size independent message commitments. This PR is ready for review but we should hold off on merging until we make a final decision on the ADR.
Screenshot

Sorry I can't comment on https://github.com/celestiaorg/celestia-app/pull/937#discussion_r1010064147 but thanks for the tip! I reverted this test timeout decrease and changed my local VSCode go test timeout config.

Codecov Report

:exclamation: No coverage uploaded for pull request base (feat/square-size-independent-commitments@9769b75). Click here to learn what that means.
The diff coverage is n/a.

@@                             Coverage Diff                             @@
##             feat/square-size-independent-commitments     #937   +/-   ##
===========================================================================
  Coverage                                            ?   25.79%
===========================================================================
  Files                                               ?       77
  Lines                                               ?     9272
  Branches                                            ?        0
===========================================================================
  Hits                                                ?     2392
  Misses                                              ?     6653
  Partials                                            ?      227

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Does this PR preserve the current non-interactive default rules? As mentioned in https://github.com/celestiaorg/celestia-app/issues/941#issuecomment-1305410774, the current non-interactive default rules would be required to ensure that message inclusion proofs remain O(log(n)).

@musalbas good question and thanks for checking on this. This PR does not change the existing non-interactive defaults.
This function and its usage remains unmodified https://github.com/rootulp/celestia-app/blob/671bc800acecfdeef426db580811fc0baeeda037/pkg/shares/non_interactive_defaults.go#L39-L80
only code relating to which subtree roots are used to calculate the commitment has been changed.

Will merge this to the feature branch feat/square-size-independent-commitments so that I can update docs in a separate PR.


Weighted average calculation

I've been sketching up an API for grade calculation iteratively. This is all in very early stages, and feedback is welcome.

Related: #134

Ready for review. The testing could be more thorough, so feel free to ask for more tests if you feel it is lacking.

I'm going to write some unit tests for the formulas right now.