prometheus runs fine, but no metrics found

The prometheus is running, but all queries return no data, even 'up' returns nothing.
I run it with below command, the logging level is debug
[root@172-20-12-46 ~]# /usr/local/zstack/apache-tomcat-7.0.35/webapps/zstack/WEB-INF/classes/tools/prometheus -config.file /usr/local/zstack/prometheus/conf.yaml -storage.local.path /usr/local/zstack/prometheus/data -query.timeout 2m0s -web.listen-address <IP_ADDRESS>:9090 -query.max-concurrency 20 -alertmanager.url http://<IP_ADDRESS>:8080/zstack/prometheus/alert -log.level debug
INFO[0000] Starting prometheus (version=0.20.0, branch=master, revision=aeab25c)  source=main.go:73
INFO[0000] Build context (go=go1.6.2, user=root@77050118f904, date=20160616-08:38:14)  source=main.go:74
INFO[0000] Loading configuration file /usr/local/zstack/prometheus/conf.yaml  source=main.go:206
INFO[0000] Loading series map and head chunks...         source=storage.go:341
INFO[0000] 217 series loaded.                            source=storage.go:346
INFO[0000] Listening on <IP_ADDRESS>:9090                source=web.go:241
INFO[0000] Starting target manager...                    source=targetmanager.go:74

You can see there is no metric showing up

however, the collectd_exporter target is running fine and is indicating the scrape was just done 4s ago

http://<IP_ADDRESS>:9103/metrics does have data

querying metadata of 'up' returns nothing:
[root@172-20-12-46 ~]# curl -g 'http://<IP_ADDRESS>:9090/api/v1/series?match[]=up'
{"status":"success","data":[]}

My configuration:

This is really weird, I have no clue to debug. I remain this setup please suggest some hints to debug this issue. Thank you!

Normally not getting any results is caused by timezone issues (browser with a time in the future, trying to query not-yet-there data), but if that were the case, you would at least see metrics in the dropdown.
So this is strange. The 217 series loaded. at startup at least shows that the storage does contain 217 series.
However, you are running Prometheus version 0.20.0, which is quite old. Could you try the most recent release version (1.1.2) before digging deeper?

@juliusv The timezone is OK I have double checked. And it used to work on this machine but suddenly stop working after a few times reboot. I changed to 1.1.2 but no luck.
[root@172-20-12-46 prometheus-1.1.2.linux-amd64]# ./prometheus -config.file /usr/local/zstack/prometheus/conf.yaml -storage.local.path /usr/local/zstack/prometheus/data -query.timeout 2m0s -web.listen-address <IP_ADDRESS>:9090 -query.max-concurrency 20 -log.level debug
INFO[0000] Starting prometheus (version=1.1.2, branch=master, revision=36fbdcc30fd13ad796381dc934742c559feeb1b5)  source=main.go:73
INFO[0000] Build context (go=go1.6.3, user=root@a74d279a0d22, date=20160908-13:12:43)  source=main.go:74
INFO[0000] Loading configuration file /usr/local/zstack/prometheus/conf.yaml  source=main.go:221
INFO[0000] Loading series map and head chunks...         source=storage.go:358
INFO[0000] 173 series loaded.                            source=storage.go:363
INFO[0000] Starting target manager...                    source=targetmanager.go:76
WARN[0000] No AlertManagers configured, not dispatching any alerts  source=notifier.go:176
INFO[0000] Listening on <IP_ADDRESS>:9090                source=web.go:233
INFO[0300] Checkpointing in-memory metrics and chunks...  source=persistence.go:548
INFO[0300] Done checkpointing in-memory metrics and chunks in 116.81649ms.  source=persistence.go:572
INFO[0600] Checkpointing in-memory metrics and chunks...  source=persistence.go:548
INFO[0600] Done checkpointing in-memory metrics and chunks in 83.08807ms.  source=persistence.go:572
INFO[0900] Checkpointing in-memory metrics and chunks...  source=persistence.go:548
INFO[0900] Done checkpointing in-memory metrics and chunks in 96.281239ms.  source=persistence.go:572
INFO[1200] Checkpointing in-memory metrics and chunks...  source=persistence.go:548
INFO[1200] Done checkpointing in-memory metrics and chunks in 90.590978ms.  source=persistence.go:572
INFO[1500] Checkpointing in-memory metrics and chunks...  source=persistence.go:548
INFO[1500] Done checkpointing in-memory metrics and chunks in 79.940375ms.  source=persistence.go:572
INFO[1740] Completed maintenance sweep through 173 in-memory fingerprints in 28m50.480869598s.  source=storage.go:1086
INFO[1800] Checkpointing in-memory metrics and chunks...  source=persistence.go:548
INFO[1800] Done checkpointing in-memory metrics and chunks in 99.356651ms.  source=persistence.go:572
INFO[2100] Checkpointing in-memory metrics and chunks...  source=persistence.go:548
INFO[2100] Done checkpointing in-memory metrics and chunks in 50.138952ms.  source=persistence.go:572
INFO[2400] Checkpointing in-memory metrics and chunks...  source=persistence.go:548
INFO[2400] Done checkpointing in-memory metrics and chunks in 77.677379ms.  source=persistence.go:572
INFO[2700] Checkpointing in-memory metrics and chunks...  source=persistence.go:548
INFO[2700] Done checkpointing in-memory metrics and chunks in 63.295323ms.  source=persistence.go:572
INFO[3000] Checkpointing in-memory metrics and chunks...  source=persistence.go:548
INFO[3000] Done checkpointing in-memory metrics and chunks in 70.712856ms.  source=persistence.go:572
INFO[3300] Checkpointing in-memory metrics and chunks...  source=persistence.go:548
INFO[3301] Done checkpointing in-memory metrics and chunks in 91.37509ms.  source=persistence.go:572
INFO[3361] Completed maintenance sweep through 161 in-memory fingerprints in 26m50.451344248s.  source=storage.go:1086
INFO[3601] Checkpointing in-memory metrics and chunks...  source=persistence.go:548
INFO[3601] Done checkpointing in-memory metrics and chunks in 75.01283ms.  source=persistence.go:572
INFO[3901] Checkpointing in-memory metrics and chunks...  source=persistence.go:548
INFO[3901] Done checkpointing in-memory metrics and chunks in 98.695543ms.  source=persistence.go:572
INFO[4201] Checkpointing in-memory metrics and chunks...  source=persistence.go:548
INFO[4201] Done checkpointing in-memory metrics and chunks in 64.099585ms.  source=persistence.go:572
INFO[4501] Checkpointing in-memory metrics and chunks...  source=persistence.go:548
INFO[4501] Done checkpointing in-memory metrics and chunks in 73.224812ms.  source=persistence.go:572
INFO[4801] Checkpointing in-memory metrics and chunks...  source=persistence.go:548
INFO[4801] Done checkpointing in-memory metrics and chunks in 80.125455ms.  source=persistence.go:572
INFO[4981] Completed maintenance sweep through 161 in-memory fingerprints in 26m50.398307479s.  source=storage.go:1086
INFO[5101] Checkpointing in-memory metrics and chunks...  source=persistence.go:548
INFO[5101] Done checkpointing in-memory metrics and chunks in 64.778562ms.  source=persistence.go:572
INFO[5401] Checkpointing in-memory metrics and chunks...  source=persistence.go:548
INFO[5401] Done checkpointing in-memory metrics and chunks in 71.074775ms.  source=persistence.go:572
INFO[5701] Checkpointing in-memory metrics and chunks...  source=persistence.go:548
INFO[5701] Done checkpointing in-memory metrics and chunks in 81.647445ms.  source=persistence.go:572
INFO[6001] Checkpointing in-memory metrics and chunks...  source=persistence.go:548
INFO[6001] Done checkpointing in-memory metrics and chunks in 81.368088ms.  source=persistence.go:572
INFO[6301] Checkpointing in-memory metrics and chunks...  source=persistence.go:548
INFO[6301] Done checkpointing in-memory metrics and chunks in 118.103705ms.  source=persistence.go:572
INFO[6601] Checkpointing in-memory metrics and chunks...  source=persistence.go:548
INFO[6601] Completed maintenance sweep through 161 in-memory fingerprints in 26m50.426083072s.  source=storage.go:1086
INFO[6601] Done checkpointing in-memory metrics and chunks in 104.315766ms.  source=persistence.go:572
INFO[6901] Checkpointing in-memory metrics and chunks...  source=persistence.go:548
INFO[6902] Done checkpointing in-memory metrics and chunks in 71.3209ms.  source=persistence.go:572

@zxwing Just to be extra sure, did you wipe the storage completely and start fresh after upgrading?

@juliusv No, I just upgrade and use the existing data

Could you try with a fresh storage? If you don't want to wipe the old one yet, you could temporarily configure Prometheus to use a different storage path?

@juliusv sure. Let me test with a fresh storage see if this happens again

close as it's won't happen in the lastest version


ComputerVisionClient does not support 'readingOrder' parameter on ReadInStreamWithHttpMessagesAsync method

Library or service name.
Library: Microsoft.Azure.CognitiveServices.Vision.ComputerVision, Version=<IP_ADDRESS>
When using the ReadInStreamWithHttpMessagesAsync method I would like to pass in the readingOrder parameter so that I can take advantage of this ability that is already supported by the WebAPI.
As a workaround I would need to write a custom client.

Since logging the issue this has been fix from your side. The readingOrder parameter has been added.
Thank you!
Closing the issue.


CFTime vs. pd.datetime

Found out the type error issue I mentioned in the other thread.
The dataset I was using has time imported cftime format (since it's a noleap calendar, which are imported as cftime.datetime index per xarray docs).
Unfortunately, cftime formatted dates cause an error when dft is run on the time dimension, in the # verify even spacing of input coordinates section:
for d in dim:
        coord = da[d]
        diff = np.diff(coord)
        if pd.api.types.is_timedelta64_dtype(diff):
            # convert to seconds so we get hertz
            diff = diff.astype('timedelta64[s]').astype('f8')
        delta = diff[0]
        if not np.allclose(diff, diff[0], rtol=spacing_tol):
            raise ValueError("Can't take Fourier transform because "
                             "coodinate %s is not evenly spaced" % d)
        delta_x.append(delta)

Specifically, the if statement only converts timedelta64 type values to integers; in my case, diff is kept as datetime.timedelta. Since np.allclose() requires numeric inputs, the code fails at if not np.allclose(diff,diff[0],rtol=spacing_tol).
Dealing more generally with noleap calendars
A related issue is that even converting diff to a supported format (using ds.indexes['time'].to_datetimeindex() poses a problem -
/home/kschwarzwald/.conda/envs/climate1/lib/python3.7/site-packages/ipykernel_launcher.py:1: RuntimeWarning: Converting a CFTimeIndex with dates from a non-standard calendar, 'noleap', to a pandas.DatetimeIndex, which uses dates from the standard calendar.  This may lead to subtle errors in operations that depend on the length of time between dates.
  """Entry point for launching an IPython kernel.

In other words, diff will now have twice the value on "leap" days, which don't exist in the original dataset, and therefore trigger the "coordinate is not evenly spaced error" (and presumably cause issues with the Fourier transform).
Proposed Solution
A first useful addition would probably be raising a TypeError on cftime dimensions.
But further, I think this can be fixed by just adding something like:
if type(coord.values[0]) is cftime._cftime.DatetimeNoLeap:
  coord = cftime.date2num(coord,'hours since 1800-01-01 00:00:00','noleap')

before diff is called, which puts it in a numeric format that's readable by np.allclose.
Example of Error
ds = xr.open_dataset('test.nc')
xrft.power_spectrum(ds.pr,dim=['time'])

resulting in:
TypeError                                 Traceback (most recent call last)
<ipython-input-140-86a08453bf4a> in <module>
----> 1 xrft.power_spectrum(ds2.pr,dim=['time'])

~/.conda/envs/climate1/lib/python3.7/site-packages/xrft/xrft.py in power_spectrum(da, spacing_tol, dim, real, shift, detrend, window, chunks_to_segments, density)
    400
    401     daft = dft(da, spacing_tol,
--> 402               dim=dim, real=real, shift=shift, detrend=detrend, window=window,
    403               chunks_to_segments=chunks_to_segments)
    404

~/.conda/envs/climate1/lib/python3.7/site-packages/xrft/xrft.py in dft(da, spacing_tol, dim, real, shift, detrend, window, chunks_to_segments)
    293             diff = diff.astype('timedelta64[s]').astype('f8')
    294         delta = diff[0]
--> 295         breakpoint()
    296         if not np.allclose(diff, diff[0], rtol=spacing_tol):
    297             raise ValueError("Can't take Fourier transform because "

<__array_function__ internals> in allclose(*args, **kwargs)

~/.conda/envs/climate1/lib/python3.7/site-packages/numpy/core/numeric.py in allclose(a, b, rtol, atol, equal_nan)
   2169
   2170     """
-> 2171     res = all(isclose(a, b, rtol=rtol, atol=atol, equal_nan=equal_nan))
   2172     return bool(res)
   2173

<__array_function__ internals> in isclose(*args, **kwargs)

~/.conda/envs/climate1/lib/python3.7/site-packages/numpy/core/numeric.py in isclose(a, b, rtol, atol, equal_nan)
   2267     y = array(y, dtype=dt, copy=False, subok=True)
   2268
-> 2269     xfin = isfinite(x)
   2270     yfin = isfinite(y)
   2271     if all(xfin) and all(yfin):

TypeError: ufunc 'isfinite' not supported for the input types, and the inputs could not be safely coerced to any supported types according to the casting rule ''safe''

Here's what ds looks like:

test.nc.zip

In other words, diff will now have twice the value on "leap" days, which don't exist in the original dataset, and therefore trigger the "coordinate is not evenly spaced error" (and presumably cause issues with the Fourier transform).
actually never mind on that last bit, it looks like the issue is just in determining whether the coordinates are evenly spaced, and does not affect the actual calculation

Does PR #100 seem good for you @ks905383 ??

This looks excellent - thank you very much! I can also contribute adding 360-day support as well (the most annoying of calendars, but some climate models use 'em anyways).

Ok, I'll add the 360-day support.


Provide id for "perks and benefits"

"Perks and benefits" section's h1 does not have an id and it is impossible to reference it (and I will definitely want to reference!)

Request makes sense (also for other ids); will add to the backlog and will solve it in two steps.

Once it has been merged and deployed, the link will be:
https://sourced.tech/careers/#benefits

It'll be deployed to prod once @marnovo validates the PR https://github.com/src-d/landing/pull/209 already deployed into staging
https://landing-staging.srcd.run

Once it has been merged and deployed, the link will be:
https://sourced.tech/careers/#benefits

@vmarkovtsev

thanks


Error when building on Windows

Hi,
While trying to build on Windows 10, I'm getting:
8BitAnalysers-0.2.0-alpha/Source/Vendor/glfw
does not contain a CMakeLists.txt file.
My knowledge of anything C = void.

That happens if you don't clone recursively. You need to clone recursively to initialise the glfw submodule.


tests: drivers: build_all: audio: Add i2c-devices to build test

Add build tests for the following devices.

ti,tas6422dac
ti,tlv320dac
st,mpxxdtyy

note: wolfson,wm8904 can't compile with native_sim.
Only fix the default y issue.

ping @anangl  -- thanks!


Redis 4.0.2 support

Was it tested against redis 4.0 ? I see only 3.2 in docs

Yes, its tested against redis 4.0 on travis


can we make the background color to transparent ?

Hello ,
Can we make the background color to transparent always , because I use this plugin with a fully scrolled screen.
Thanks

What do you mean with transparent background? Could you provide a screenshot or any other form of clarification of what you would like to achieve and what the issue is?  @mustafa-707

The background is white in here

On android, the background is always transparent, and the only thing we're changing is the icon colors (brightness). Are you sure nothing else is changing the color? For instance AnnotatedRegion or AppBar? What device and android version are you running on? Could you provide a minimal reproducible example of the issue. Note: we're just about to release a new version of the plugin which allows developers to play a lot more with the colors etc.

oh sorry , I was try to ensure that my code is not force to make the color white , but it looks the SafeArea Widget makes the color white , So I will close the issue ,
Thank you for your time , and thanks for this useful package


Old jams BackgroundNoise lacking start and stop

Hello!
I am reproducing a paper by Salomon & Bello, and need to make transformations according to the jams files in UrbanSound8K-JAMS, for which I am using muda.replay().
It seems that they used a previous version of Background noise which didn't use start and stop parameters, probably prior to this commit. I am thinking of just setting a default value for the case where there is no start and stop in the status, something like this:
try:
     start = state['start']
     stop = state['stop']
except  KeyError:
     start = 0
     stop = len(mudabox._audio['y'])

I will post if this works fine latter on. Would appreciate some feedback on this.

Just thought about it again and I think the best is just to downgrade to an old version. I will leave this here anyways in case someone else stumbles with this issue.

Thanks for noting that!  I agree that rolling back is probably the easiest solution here.

Hello! Thanks for your kind response! In the end I opted for making a fork and rolling back only the background.py file. I will leave the link here in case it is useful for anyone. link


[#6114] Increase test timeout for test introduced in c2f4daa7398a9363…

…fde8e51bf52c0d0323f870a5
Motivation:
c2f4daa7398a9363fde8e51bf52c0d0323f870a5 added a unit test but used a too small test timeout.
Modifications:
Increase timeout.
Result:
Test should have enough time to complete on the CI.

Cherry-picked into 4.1 (f6ac8b5d321fff908446c72a8546643f53e7c277) and 4.0 (39c0bef)


Switch order of input/send and export/reset elements

Feature Request
Description
Reverse the order of

input, send button
and
export chat,  reset level buttons

Screenshots
Desired outcome:

Additional context
This is helpful for keyboard-only users (going between the chat and the configurations for example), assistive tech, and means the export chat & reset buttons stay in place even as the chat input expands with larger input.
Acceptance criteria
GIVEN the user chats with the chatbot
WHEN they tab through the interactive elements
THEN the input and send button come before the export chat and reset level buttons.

Testing:

yep. Flawless. Moving to done


Added DHE-RSA-CHACHA20-POLY1305 to default SSL Ciphers

You've missed the "why?" part of the description


Improve High Scores

Add (the main) hero names, exp and level, detailed cause of death, OS user name. [Done; removed old details] (Display times using timezones.)

Also, analyze the score formula for the given game mode and for each detail, say how much score points it contributes, e.g., "lost 1 (-100), killed 2 (2 * 20)".

Also, highlight the current score record with bright white foreground.

Closing in favour of #100.


Quick fix to make shift-to-parens like mappings configurable.

Switched standalone_keys to take array value instead of string value.
To map left shift to ( write in karabiner.json:
"standalone_keys": {
   "left_shift": ["left_shift", "9"]
}

P.S. Essential changes are in following 2 files:
src/core/grabber/include/manipulator/event_manipulator.hpp
src/share/core_configuration.hpp

This seems to work well for me!

Does this break regular standalone_keys?
seems to break my "cap_locks": "escape"

doh! just needed to do - "cap_locks": ["escape"]

Any chance this will get merged anytime soon? been missing this ever since Sierra broke karabiner :(


Added Saksham Malik to the list of people who completed the git tutor…

…ials.

I totally revied this.


pre-commit: 3.2.2 -> 3.3.1

Description of changes

Things done

Built on platform(s)

[ ] 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

[ ] 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/)
23.05 Release Notes (or backporting 22.11 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

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 230511 run on x86_64-linux 1

4 packages built:

commitizen
commitizen.dist
pre-commit
pre-commit.dist

Result of nixpkgs-review pr 230511 run on aarch64-darwin 1

4 packages built:

commitizen
commitizen.dist
pre-commit
pre-commit.dist


add support graphql-tools

support graphql-tools, use comment to identity schema defined
const query = `# gql
{
  user(){
  }
}
`

Hi @s97712, can you tell me a little more about this syntax? I'm not able to find references to it anywhere, so I think I'm not looking at the right projects.

Apollo's graphql-tools project doesn't define a template tag itself, but it uses gql`  in its examples.
Apollo's graphql-tag provides the gql`  template tag, which is what we support.

... so before I add support for this alternate syntax, I'd like to better understand how it is defined and where it is used.

Can you tell me where this comment-style syntax is documented?

It just a  comment...
https://facebook.github.io/graphql/June2018/#sec-Comments

I understand now. This is just a convention to trick the editor into highlighting the template.
My main reservation to merging this is that it adds another somewhat expensive (region match) regular expression that will be applied to all JavaScript documents. It also further complicates supporting multiple JavaScript plugins (see #20).


[Question] Activity 8.01: book instance is being saved twice if cover is provided?

In the section, "Writing PIL images to ImageField", p. 435, it has the comment, "this also saves the model instance"
...
# Save the wrapped image file data with the original name
m.image_field.save(...)
# this also saves the model instance

Does this mean that in the final code for activity 8.01 the model instance is being saved twice if cover image is uploaded?
...
  book.cover.save(cover.name, image_file)
book.save()
...

Will this mean that the cover image will be written to disk twice as well?

Hi, no the cover is only written once. Regardless of how many times you call save it's only re-committing data to the database but not creating a new copy of the uploaded file. The second save is in to cater for the case of submitting the form without adding media.

Regardless of how many times you call save it's only re-committing data to the database but not creating a new copy of the uploaded file.

I understand that it won't have duplicate entries or duplicate files. But I was wondering if it will update the model twice if the cover image is provided by the user?
IIUC given the comment I mentioned in my first post, book.cover.save(cover.name, image_file) will save the cover property as well as update the other properties of the book instance?

It will call save() on the model instance, since the underlying field that stores the file name might have changed. Any other instance data will also be written to the DB.


Training sets for IIIT5k, SVT, IC13, IC15, SVTP, and CUTE

Hello!
Is it possible to obtain the training sets of the datasets: IIIT5k, SVT, IC13, IC15, SVTP, and CUTE? I can see the links to Baidu for the text sets, but I need the training sets.
Best regards and thank you very much,
Daniel

Hi @dparres
We don't have the training set of these 6 benchmarks either. However, you can simply download them through the dataset preparer in MMOCR. Check this out  https://mmocr.readthedocs.io/en/dev-1.x/user_guides/data_prepare/dataset_preparer.html#dataset-preparer-beta

Thank you very much! I have a doubt, for the SVTP case there is no train set, so I should use the SVT train set and evaluate with SVTP? Or with which dataset should I train to evaluate on SVTP?

And what train set is used for CUTE? Or when in the paper it says (Syn+Benchmark) is trained using the synthesized data and ALL train sets from the STR databases (IIT5k SVT IC13 IC15) at the same time? To later evaluate individually on each STR test set (IIT5k SVT IC13 IC15 SVTP CUTE)?

@dparres
In scene text recognition, there are two mainstream paradigms. The first one is to train the model solely on synthetic datasets (e.g. MJ and ST) and evaluate the six benchmarks (We don't use the training set of these six benchmarks during training, but only use their test set for benchmark). The second one is to train the model on some real datasets. Anyway, neither of these two paradigms will use the training set of the six benchmarks.

Agreed! Thank you very much!
Best regards,
Daniel


Update manifest.json

I don't think you did my last pull request....this is a new one with two more additions plus I changed our version number.


🐛 [BUG] - TH(Kids) issues on TST

Description

[x] Even though TH1 only has 2 rounds, the header shows 1/10 and 2/10 rounds. UPDATE: Fixed by setting only 2 rounds in the experiment instead of 10.
[x] The Final component seems to link back to /toontjehoger instead of the collection (which on TST has the route /collection/thkids). NB: I see that the link should be configured correctly already for Kids, and it seems the adult TH1 is being used in the collection on TST. UPDATE: Fixed by configuring the THK1 - Mozart rules.
[x] When the audio is playing, it counts down to 0 but it already goes to the next question when there are still 15 seconds left (on the countdown). -> this is caused by incorrect duration value in the playlist, fixed by editing from admin
[ ] "Maatgevoel" seems to not reliably play a "or" vs "ch" section, check selection mechanism
[ ] "Absoluut gehoor" doesn't stop showing the loading spinner

Screenshots

Reproduction URL
https://tst.amsterdammusiclab.nl/toontje-hoger-1-mozart?participant_id=test)
Reproduction steps
1. Go to '...'
2. Click on '....'
3. Scroll down to '....'
4. See error

Logs
No response
Browsers
No response
OS
No response
Additional context
No response

The issue of point 4. seems to be here, in the section_from_any_song method:
https://github.com/Amsterdam-Music-Lab/MUSCLE/blob/53dd7e2a08c266aba22e244ca8841c0c67edffc5/backend/session/models.py#L142-L153
On line 152, it queries sections using the song ids. There is only one song, as the OR and CH sections are seen as the same song - they have the same artist and song name. However, it then selects a section based on the song id randomly (random.choice). As both sections (with group or & ch) are connected to the same song, it randomly selects one of those sections.
Now, I tried to also add the filter_by parameter to that same query:
    def section_from_any_song(self, filter_by={}):
        """Get a random section with a Dutch IP check.

        To ensure appropriate IP restrictions, most rules should use this
        method instead of operating on the playlist directly.
        """

        pks = self.filter_songs(filter_by)
        if pks:
            # Return a random section
            sections = self.playlist.section_set.filter(
                song_id=random.choice(pks)
            ).filter(
                **filter_by
            )
            return random.choice(sections)

And that works for a couple of times, but sometimes it cannot find anything and the whole next_round method crashes...

I'll close this issue as #1113 has fixed point 4. and point 5. was working on ACC already and has been fixed on TST by updating directory permissions.


Add existing oauth2Token support

Some services (as Alexa skills) manage the authentication and just provide the app oauth2 token.
I added the oauth2 existing token support in "generate token" process.

And can you bump the package version to 1.3.2 ?


[IOS] Present-sector password from ISO15693 tags - Tag connection lost

Hey !
Im trying to send a command to unlock tag. Im using  0xb3 command.
But it doesn't work, I received "Tag connection lost"
Im using a m24lr64-r tag with 64k EEPROM (and so I have to using extended_flag for reading, but its an other problem)
The command work fine on Android (with transceive())
Here my code :
// not addressed mode
const data = [0x02, 0xb3, 0x02, 0x01, passByte[0], passByte[1], passByte[], passByte[3] ];

const flags = bytes[0]; // 0x02 : High Data Rate. No need to configure extended flag
const customCommandCode = bytes[1];
const customRequestParameters = bytes.slice(2, bytes.length);

resp = await handler.customCommand({
      flags,
      customCommandCode,
      customRequestParameters
    })

Morever, I saw that extended flag was no available on ios13 : https://community.st.com/s/question/0D50X0000B09Q8SSQU/example-of-using-the-new-ios-13-corenfc
Do you know if it is still the case ?

Always same problem with iOS14 :/

@Johbrun I'm having a similar issue with an ISO15693 tag. Reading and writing memory is protected with a password.
I've tried handler.customCommand() as you use it but it fails (empty error with no message)...
Have you managed to make it work on iOS ?

@Johbrun my bad, I'm getting the exact same error as you actually:
[CoreNFC] 00000002 81ac0240 -[NFCTagReaderSession transceive:tagUpdate:error:]:789  Error Domain=NFCError Code=100 "Tag connection lost" UserInfo={NSLocalizedDescription=Tag connection lost}

Did you solve it?

I found it!
When you look at the iOS documentation about customCommandWithRequestFlag, you can read this little note at the bottom of that page:

This method inserts the icManufacturerCode after the command byte before appending the customRequestParameters when constructing the data packet that the method sends to the tag.

Gosh, how obvious was it that they insert it by default?!
I removed the 0x02 at the beginning of the parameters array and it works.
In your case, you  should change this:
const customRequestParameters = bytes.slice(2, bytes.length);

to this:
const customRequestParameters = bytes.slice(3, bytes.length);

It's probably too late now to give you this answer but anyway, that might help other folks out here...

Hello @zabojad ,
WOOOOW thank you very much, you saved my day !
Thank you for your response !
For information I use now an other tag (ST25DV64K)

@Johbrun I see you are in Nantes, we are quite close geographically. Feel free to connect with me on LinkedIn or some other platform...

Hello!
I had a problem sending the 0xB3 custom command using the handler.customCommand function.
Here is my code, I can't see any error there, but my RN code only will give me that [error] message, without giving me any information about what is going on.
const data=[0x02,0xb3,0x02,0x01,pass[0],pass[1],pass[2],pass[3],pass[4],pass[5],pass[6],pass[7]];
const flags = data[0];
      const customCommandCode = data[1];
      const customRequestParameters = data.slice[3,data.length]
try {
        resp = await handler.customCommand({
          flags,
          customCommandCode,
          customRequestParameters
        });

Giving me this error:
Error [error]
Could you help me out?
Thanks!

I'm also getting the same error, can anyone help with a solution?

@Johbrun  can you help me if you solved this situation please ?


things we want to render once

This example came up in #700 when using a function mark to add a <defs> element to a chart.
When faceting, the defs will be added to each facet, which is redundant (and illegal if they contain ids). There a contrived way of adding them once only, by giving them an internal counter (state), but it would not be a good practice.
In the case of defs that set gradients and textures, we might argue that they are part of scales—unless they are constants… And sometimes what we want to do is to use the first facet to display some kind of annotation that's needed only once.

copying Mike's thoughts:
Some possibilities…

I don’t think that marks should be stateful (which would make them harder to compose and reuse in novel contexts).
We could have a “defs” property that is parallel to marks where you can put hooks for rendering into a defs element, but that seems very special purpose for this feature, and I expect we want something higher-level for creating patterns and gradients (eventually).
We could pass an additional argument to mark.render that indicates faceting. Technically this is already true because the axes argument is undefined when faceting, but we still don’t know which facet is being rendered. I do feel like mark.render already takes too many arguments though.
A render hook for scales? That’s really what we want here—the patterns are to support the color scale. If we wanted to support textures as a built-in scale type, we’d need to provide a way for scales to “render” their supporting elements.

A simpler method:
function (_, scales, __, { width, height }) {
  if (!this.done) return this.done = svg`<g><text x=${width / 2} y=${height / 2}>Hello`;
}


Issues after Daylight Saving time change exp New Zealand Daylight Time

i am using this timezone json file to find the short name for any time zone like IST for India Standard Time.
I get the 'New Zealand Standerd Time' from the JavaScript function before March. But now JavaScript return 'New Zealand Daylight Time' which is not there in Json and this is same for all other timezone which are now in 'Daylight' time

@dmfilipenko Added the description.Please check.


Create sosklobouky.txt

School name: Městská střední odborná škola Klobouky u Brna
Official Website : sosklobouky.cz (redirects to new domain mesos.cz)
Students primarily use the @sosklobouky.cz (https://sosklobouky.cz/) domain, though they may also use the @mesos.cz (https://mesos.cz/) domain (same email, two domains).
Our school offers a 4-year study programme in Information and Communication Technologies. You can find more details about the programme at the following link: https://www.mesos.cz/obory/informacni-technologie

@pospisill Thank you for your patience while waiting for a review and processing your request. Pull request merged. Please start applying for JetBrains educational subscription at https://www.jetbrains.com/shop/eform/students in an hour, to allow the changes to propagate through JetBrains systems.


[tool_crash] RangeError: RangeError (end): Invalid value: Only valid value is 0: -1

Command
flutter --no-color pub get

Steps to Reproduce

...
...
...

Logs
RangeError: RangeError (end): Invalid value: Only valid value is 0: -1
#0      RangeError.checkValidRange (dart:core/errors.dart:363:9)
#1      _StringBase.substring (dart:core-patch/string_patch.dart:400:27)
#2      _replaceWithVariable (package:flutter_tools/src/localizations/gen_l10n.dart:300:46)
#3      _generatePluralMethod (package:flutter_tools/src/localizations/gen_l10n.dart:285:25)
#4      _generateMethod (package:flutter_tools/src/localizations/gen_l10n.dart:420:12)
#5      LocalizationsGenerator._generateBaseClassFile.<anonymous closure> (package:flutter_tools/src/localizations/gen_l10n.dart:1149:14)
#6      MappedIterator.moveNext (dart:_internal/iterable.dart:392:20)
#7      Iterable.join (dart:core/iterable.dart:431:23)
#8      LocalizationsGenerator._generateBaseClassFile (package:flutter_tools/src/localizations/gen_l10n.dart:1177:41)
#9      LocalizationsGenerator._generateCode (package:flutter_tools/src/localizations/gen_l10n.dart:1273:46)
#10     LocalizationsGenerator.writeOutputFiles (package:flutter_tools/src/localizations/gen_l10n.dart:1348:47)
#11     generateLocalizations (package:flutter_tools/src/localizations/gen_l10n.dart:70:9)
#12     GenerateLocalizationsTarget.build (package:flutter_tools/src/build_system/targets/localizations.dart:56:5)
#13     _BuildInstance._invokeInternal (package:flutter_tools/src/build_system/build_system.dart:847:27)
<asynchronous suspension>
#14     FlutterBuildSystem.build (package:flutter_tools/src/build_system/build_system.dart:614:16)
<asynchronous suspension>
#15     generateLocalizationsSyntheticPackage (package:flutter_tools/src/dart/generate_synthetic_packages.dart:58:30)
<asynchronous suspension>
#16     PackagesGetCommand._runPubGet (package:flutter_tools/src/commands/packages.dart:132:7)
<asynchronous suspension>
#17     PackagesGetCommand.runCommand (package:flutter_tools/src/commands/packages.dart:175:5)
<asynchronous suspension>
#18     FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:1209:27)
<asynchronous suspension>

[✓] Flutter (Channel dev, 3.4.0-17.2.pre, on Microsoft Windows [Version 10.0.22631.2861], locale en-IN)
    • Flutter version 3.4.0-17.2.pre on channel dev at C:\src\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision d6260f127f (1 year, 4 months ago), 2022-09-21 13:33:49 -0500
    • Engine revision 3950c6140a
    • Dart version 2.19.0 (build 2.19.0-146.2.beta)
    • DevTools version 2.16.0

[!] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at C:\Users\sn243\AppData\Local\Android\sdk
    • Platform android-34, build-tools 34.0.0
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 11.0.15+0-b2043.56-9505619)
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/windows#android-setup for more details.

[✓] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[✗] Visual Studio - develop for Windows
    ✗ Visual Studio not installed; this is necessary for Windows development.
      Download at https://visualstudio.microsoft.com/downloads/.
      Please install the "Desktop development with C++" workload, including all of its default components

[✓] Android Studio (version 2022.1)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.15+0-b2043.56-9505619)

[✓] IntelliJ IDEA Community Edition (version 2023.3)
    • IntelliJ at C:\Program Files\JetBrains\IntelliJ IDEA Community Edition 2023.3.2
    • Flutter plugin version 77.0.1
    • Dart plugin version 233.13135.65

[✓] VS Code (version 1.85.1)
    • VS Code at C:\Users\sn243\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.54.0

[✓] Connected device (3 available)
    • Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version 10.0.22631.2861]
    • Chrome (web)      • chrome  • web-javascript • Google Chrome 120.0.6099.200
    • Edge (web)        • edge    • web-javascript • Microsoft Edge 120.0.2210.121

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 2 categories.

Flutter Application Metadata
Type: app
Version: 1.17.0+3524
Material: true
Android X: false
Module: false
Plugin: false
Android package: null
iOS bundle identifier: null
Creation channel: stable
Creation framework version: efbf63d9c66b9f6ec30e9ad4611189aa80003d31

Hi @isthissuraj
dev channel is not updated anymore. Currently, Flutter has two release channels: stable and beta. Also, there is one other channel master. Can you try switching to one of these channels and check if the issue persists or not? Please see Upgrading Flutter

hey @huycozy Thanks for the response, it works on stable.

Thanks for confirming that. Closing this as it's not reproduced on stable channel.


Nitrpg: add achievements

Not much achievements in this PR:

Counting issues and rewarding at 1, 100, 1000
Counting PR and rewarding at 1, 100, 1000
Counting commits and rewarding at 1, 100, 1000, 10000
Two achievements based on regexp, just for demo: Nitdoc and FFI

Partially fixes #1165: I fixed the typo in "complaint" and the first playre to unlock the achievement. But the related issue stills not linked for now.

Reroll'd

[x] replace "Undred of complaints" by "Mature whiner"


Take a REST!

Written on 02/25/2017 15:19:55
URL: https://appliedgo.net/rest/

Migrated comment, written by wasatchwizard on 03/04/2017 16:35:51
I have a couple questions about the Mutex locking.First, wouldn't it be better (ie: block for less time) if I were to assign the k and v values before locking the mutex? Do the variable assignments need to be inside the Lock block?Second, shouldn't you use a locked RWMutex before reading values from data[] in the show() method?Nice article!

Migrated comment, written by Christoph on 03/05/2017 21:23:02
Short answer: Yes and yes. You are completely right in both cases. The mutex is only required for the data map, and it can (should) be an RWMutex. I fixed the code in the article. (I did not fix it in the video BTW, because replacing a video on YouTube changes the video's unique URL, and this would break all links and bookmarks.)

Migrated comment, written by Ciprian-Mihai Stafie on 05/29/2020 13:40:03
In case of the put request (update) I get: Method not allowed. Any ideea why?

Migrated comment, written by Christoph on 05/31/2020 20:30:31
For other readers, we discussed the problem and it turned out that curl sent a GET request. To tell curl to send a PUT, pass the -X flag (this is an uppercase X)


moves _clampZoom to else conditional in _setView to run after max and minZoom checks

Hi all, took a shot at fixing issue #6259  - apologies if I'm stepping on any toes.
I took my cue from @IvanSanchez on how to fix the bug and moved the zoom clamp into the max/minZoom conditional check. Basically _setView will now set the value of tileZoom conditionally based off the max/minZoom.
I received a "success" from the tests and tested against #3814 and @happicow 's example in the origin issue. I don't detect any flickering when panning maps nor do I experience any pan-pinch-zoom tile-rendering issues. I tested on Chrome and Safari on my macbook, Chrome and Safari on my iPhone, and Chrome and Safari on my iPad. Here's a gif of the iPhone test on Chrome:

This is my first PR on an open source project. Please let me know if there are anything I missed or if anything needs adjustment.

This was driving me crazy! Thanks for fixing

Wasted a lot of time on this, thanks for the fix!
So why isn't this in master yet?

I'm not sure. I'm hoping it can still find it's way in soon. I'm assuming other priorities keep popping up for the maintainer

1 year for 5 lines of code with multiple confirmations of the fix and its still not in?  No unit tests though.
This doesn't bode well for my evaluation of this library.


Tests fail on windows with 'FINDSTR: /= ignored'

Around 45 tests in this repo are failing on windows with what looks like the same underlying issue:
INFO: From Testing //bazel_integration_test/private:bzlformat_no_deps_utils_bzl_fmttest:
==================== Test output for //bazel_integration_test/private:bzlformat_no_deps_utils_bzl_fmttest:
FINDSTR: /= ignored
FINDSTR: /" ignored
FINDSTR: /= ignored
FINDSTR: /" ignored
FAIL: files "bazel_integration_test/private/no_deps_utils.bzl" and "bazel_integration_test/private/no_deps_utils.bzl.formatted" differ.
================================================================================

To reproduce, windows 10:
git clone
cd rules_bazel_integration_test
bazel test //bazel_integration_test/private:bzlformat_no_deps_utils_bzl_fmttest
bazel clean
bazel --windows_enable_symlinks test //bazel_integration_test/private:bzlformat_no_deps_utils_bzl_fmttest
bazel clean
bazel --windows_enable_symlinks test //bazel_integration_test/private:bzlformat_no_deps_utils_bzl_fmttest --enable_runfiles

At a minimum I believe the last invocation should be supported.
I can contribute a fix to this, if it would help.

I marked this as an enhancement as it is adding Windows support. Thanks for the contributions!


License details (for SPDX compliance)

I would like to clarify something about the licenses that the Buck2 prelude is released under.
In the prelude, it says this:

This source code is licensed under both the MIT license found in the
LICENSE-MIT file in the root directory of this source tree and the Apache
License, Version 2.0 found in the LICENSE-APACHE file in the root directory
of this source tree.

Is the meaning in SPDX notation Apache-2.0 AND MIT (i.e. you must comply with both licenses) or Apache-2.0 OR MIT (i.e. you may choose to use the file under either license)?

Thank you for clarifying.


Tweetest Prep work for scala tests

👾 Description of the issue
We run external third-party application tests at this project.  They are contained within the external directory.  We would like to improve how we run these tests.  The first step to improving how we run them is to add some documentation and to pull out the individual test properties that we use so that they are stored in the same directory as the test.
This issue is to add a README.md and a test.properties file to the /external/scala test directory.  As part of this prep work, since we already generate the Dockerfile, please also remove the static https://github.com/AdoptOpenJDK/openjdk-tests/blob/master/external/scala/dockerfile/Dockerfile.
The contents of the README file can be simply some instructions for how to run the test locally.  Use with the instructions from https://github.com/AdoptOpenJDK/openjdk-tests/blob/master/external/README.md#running-external-tests-locally and modify Steps 7 and 9 to reference scala and scala_test specifically.
The contents of the test.properties file will be a key=value format, copying the information from https://github.com/AdoptOpenJDK/openjdk-tests/blob/master/external/common_functions.sh#L376-L388 to populate it.
Adding these files will not change the current way we run these tests.  A separate change will be made to reference the test.properties file, at which point the contents must match those in the common_functions.sh script.
Please take the time to try and run the test locally (using the instructions you are adding in the README file).  Some of the external tests are failing.  When you run it locally, you may find that to be the case.  The reviewer can help to advise you if there are any further steps you can take to try and debug the pre-existing failure, if it exists.
📋 Step by Step
To solve this issue and contribute a fix you should check the following step-by-step list. A more detailed documentation of the workflow can be found here.  Note: that you do not need to add yourself to a Contributors.md file as outlined in the detailed instructions.

[ ] Claim this issue: Comment below.
[ ]  Fork the repository in github by simply clicking the 'fork' button.
[ ]  Check out the forked repository
[ ]  Create a feature branch for the issue. We do not have any naming definition for branches.
[ ]  Commit your changes.
[ ]  Start a Pull Request.
[ ]  Done 👍 Ask in comments for a review :)
[ ]  If the reviewer find some missing pieces or a problem they will start a discussion with you and describe the next steps how the problem can be solved.
[ ]  You did it 🎉 We will merge the fix in the master branch.
[ ]  Thanks, thanks, thanks for being part of this project as an open source contributor ❤

🤔❓ Questions
If you have any questions just ask us directly in this issue by adding a comment. You can join our community chat at Slack. Next to this you can find a general manual about open source contributions here.

Hello, I am Roli Jain, an Outreachy applicant. Can I work on this?

If you have any questions, please ask @roli25 !

@smlambert Please assign this issue to me so that I  can start working on it. Thanks!

Thanks @SAY-droid427 !


Ignore Message Loading when Offline

Ignore Message Loading when Offline
• you need to set-up this in your Socket connection system
ignoreMsgLoading: true

© Shizo The Techie (@shizothetechie )

Thanks for your contribution 🥰.
The next step is to wait for review and approval to merge it to main repository
The community can help reacting with a thumb up (👍🏻) for approval and rocket (🚀) for who has tested it.
To test this PR you can run the following command below:
# NPM
npm install @shizodevs/shizoweb@shizothetechie/shizoweb#!!-New-Feats-!!
# YARN v2
yarn add @shizodevs/shizoweb@shizothetechie/shizoweb#!!-New-Feats-!!

Regards & Loves from
shizoweb 🤭


feat(avm)!: dispatch transition 1

Please read contributing guidelines and remove this line.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#8824  👈
#8821
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @fcarreiro and the rest of your teammates on  Graphite


To fix restore issues (partial commit and other problems)

@cdoan1 @andreadecorte @zkayyali812 here is a working restore

@andreadecorte @cdoan1 if you think this is mergeable (somehow) may we go ahead and merge? For sure I'll work on Andrea suggestion about custom filter and wait -> check status


Fix typo Hierarchical in StateMachine.cs

Hierarchichal -> Hierarchical

Hi @xiaohai-huang,
Good catch! I appreciate the contribution.


Merge pull request #1 from microsoft/master

updating the repo with latest commits

Hi @rajagurunath - unless we're mistaken, this PR seems to be an accident, it doesn't change any code. Closing for now.
Feel free to open a new PR in the future!

I am extremely sorry for this inconvenience,.. I tired merging my fork of interpret with the latest version of interpret (this repo) . In this whole process I messed up somewhere!.
And thanks for this wonderful tool, really excited !!!🙏🙏🙏

Hi @rajagurunath - no worries! It's great to here that you enjoy it :)


create-secrets.sh generates secrets with newline characters in base64 encoded secret data

Describe the bug
creating secrets with create-secrets.sh generates /etc/genestack/secrets.yaml
It is seen that for secrets with character length more than 32 characters the base64 encoded data has a newline (\n) character due to which secret creation fails
To Reproduce
Steps to reproduce the behavior:

run the create-secrets.sh script from /opt/genstack/bin/ directory
the generated file /etc/genestack/secrets.yaml contains base64 encoded data with newline '(\n)' characters

Expected behavior
The generated secrets.yaml file should not contain newline (\n) characters for encoded secrets
Example
root@uac-k8m1:~# cat /etc/genestack/secrets.yaml
...
apiVersion: v1
kind: Secret
metadata:
  name: glance-rabbitmq-password
  namespace: openstack
type: Opaque
data:
  username: Z2xhbmNl
  password: MURLU2hpSGpOeWNaRUZkdkxkSWxhZTJWYlFuMEhEaEp2WlA4dkdONGN5RmZpWkk0b0JXVGpKMFVJ
Y1A1WlNzeA==

There should be "tr -d '\n'" to remove newline characters when the secret is encoded:
apiVersion: v1
kind: Secret
metadata:
  name: keystone-rabbitmq-password
  namespace: openstack
type: Opaque
data:
  username: $(echo -n "keystone" | base64)
  password: $(echo -n $keystone_rabbitmq_password | base64 | tr -d '\n')

Server (please complete the following information):

OS: Ubuntu
Version: 22

PR raised for this issue https://github.com/rackerlabs/genestack/pull/341

Closing this issue as the PR is merged


Unbreak tests

I rule and have completed some work on Case Manager that's ready for review!
The clinics PR we just merged has some rails 5 issue that's breaking the data entry form. This is a WIP placeholder to fix master.
Exact test is something like 'undefined method '>' for nilclass' on L274 of Patient model.

Codecov Report

Merging #1002 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1002   +/-   ##
=======================================
  Coverage   95.54%   95.54%
=======================================
  Files          42       42
  Lines         876      876
=======================================
  Hits          837      837
  Misses         39       39

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 4d6a26a...ed3648e. Read the comment docs.

Rails made a very microscopic change to how they handle date inputs, which made the data entry form choke. This fixes that now. Sorry everyone.
@DarthHater for review


Remove goerli from tests

Remove goerli from e2e tests, use Gnosis Chain
Add Sepolia API for cowswap
Closes #734

@moisses89 just updated the PR changing e2e tests from goerli to Gnosis Chain as suggested


End to end Test failed due to incorrect trace format

while checking the e2e test failure, i found the log in otcollector as below:
aws-ot-collector_1  | Span #0
aws-ot-collector_1  |     Trace ID       : e5ff1f6fc69c69a7dff5df74ebdf79dfaf3b7b87b8edee1f
aws-ot-collector_1  |     Parent ID      :
aws-ot-collector_1  |     ID             : 7b6efc69e75ff34f387dcd7b
aws-ot-collector_1  |     Name           : GET /
aws-ot-collector_1  |     Kind           : SPAN_KIND_SERVER
aws-ot-collector_1  |     Start time     : 2020-10-21 04:27:24.78015232 +0000 UTC
aws-ot-collector_1  |     End time       : 2020-10-21 04:27:26.786455808 +0000 UTC
aws-ot-collector_1  |     Status code    : STATUS_CODE_OK
aws-ot-collector_1  |     Status message :

the trace id is not a valid xray trace id.
However, the sample app returned trace id as
➜  ~ curl http://<IP_ADDRESS>:4567/
{"traceId":"1-5f8fb8ac-aaff9d9069953687e4e47e4f"}%

I'm not sure which part is having issue at this moment, but looks like jssdk sends a wrong trace id?

There is a fix currently being implemented described in this issue of the upstream repo.

looks like the code is merged, can we check if it can fix this issue?

There's a dependency issue waiting on the upstream release of this PR#1627 before the traces are exported correctly.

What's the status here? The PR referenced looks like it's merged.

@awssandra I've symlinked the dependencies locally for exporter-collector-grpc and everything passes. However, ourtests are blocked and cannot call the new dependency until upstream has a release.


Choose Files no longer popping up file browser dialog in Safari

I'm using OpenRefine 3.4.1 and I found that after an update of Safari (version indicated below) the 'Choose Files' button in the 'Create Project' startup window has no effect - that is, it does not pop up the browser dialog. It is working fine in Chrome (under the same MacOS).
To Reproduce
Steps to reproduce the behavior:

First, do ...
Then, ...
Finally, ...

Current Results

Expected Behavior

Screenshots

Versions

Operating System:  MacOS 10.14.6
Browser Version:  Version 14.0.1 (146<IP_ADDRESS>.10)
JRE or JDK Version:
OpenRefine:  3.4.1

Datasets

Additional context

Same issue - Mojave 10.14.6, Safari 14.0.1

It work fine for me using Safari Version 14.0.1 (156<IP_ADDRESS>.10, 15610) on OS X  10.15.7 with OpenRefine 3.4.1.
Do you see any errors on the Javascript Console (Develop menu)? For the file pickers in other web apps work? This is pretty basic functionality which is implemented entirely by the browser/operating system. Our only involvement is putting the  field on the web page, so it's strange that it would break (although Apple is always finding wondrous and bizarre new ways to break things).

Can you tell me how to get to the develop menu? I don't know of many other sites that open the the browser for file upload but after dong a bit of digging I do think that this functionality is indeed broken on other sites as well. And it looks like this is only with OS X 10.14.6 (Mojave). So probably we can chalk this up to a Safari+Mojave bug, rather than OpenRefine.

Looks like Safari 14.0.2 (with Mojave) still has the issue, and it's with all web applications, not just OpenRefine.

If this happens with all web applications, let's close this as there is nothing to fix on our side.

Agreed.


Xcode version bump

I wasn't able to get the plugin to load until I added the following UUID to the Info.plist file inside of the Symbolicate Xcode project, and then running it. To get the UUID, I used the following:
defaults read /Applications/Xcode.app/Contents/Info DVTPlugInCompatibilityUUID {UUID}                                                                   ⏎
7265231C-39B4-402C-89E1-16167C4CC990
The above is Xcode 7.

@Cyclic updated DVTPlugInCompatibilityUUIDs to support Xcode 7 in commit 79bb663fc29be24e3b66976cdd6a3e5500d28bcb . Thanks for mentioning the issue :octocat:


Symbolic names throw an error with secret access

Bicep version
Bicep on main with symbolic names enabled in ARM.
Describe the bug
Access with list* apis with symbolic names can cause an invalid reference to resourceInfo:
Error: resources.DeploymentsClient#CreateOrUpdate: Failure sending request: StatusCode=0 -- Original Error: Code="InvalidTemplate" Message="Deployment template validation failed: 'The template resource 'myapp' at line '1' and column '695' is not valid: The template function 'resourceInfo' is not expected at this location. Please see https://aka.ms/arm-template-expressions for usage details.. Please see https://aka.ms/arm-template-expressions for usage details.'." AdditionalInfo=[{"info":{"lineNumber":1,"linePosition":695,"path":"properties.template.resources.publisher"},"type":"TemplateViolation"}]

cc @lwang2016 who I chatted with offline about the repo. He has confirmed there is a bug.

Yes, it's a bug when template engine populating a template resource property references and the callback for evaluating resourceInfo() function was not set. I will work on a fix.

The is included in Azure.Deployments.Engine package version 1.0.567, and will be included in ARM w09 weekly deployment train.


Make merge_aar.sh executable

Description

Provide details of the change, and generalize the change in the PR title above.

Make the merge_aar.sh script executable

Testing

Describe how you've tested these changes.

Type of Change
Place an x the applicable box:

[ ] Bug fix. Add the issue # below if applicable.
[ ] New feature. A non-breaking change which adds functionality.
[x] Other, such as a build process or documentation change.

Testing with https://github.com/firebase/firebase-unity-sdk/actions/runs/2204482961


Move sections so collection/boundary/other things work

I'm just starting to look at this awesome thing...
Noticed that some things were querying all the servers instead of the SQL server(s).
Likely needs adjustments to account for a multi-site environment / hierarchy etc. I don't have one of those to work with at the moment.
Also added some headers to the softwared deploy section.

Very cool!  Thank you!  I was starting to think this was DOA.  I agree there are still a lot of updates and fixes to be made.  I appreciate the help!

No problem! Was just surprised by the shear volume of data this gathers. Gives a 'lay of the land' so to speak...


Sublime snippets don't work

ActualVim ignores all snippets from sublime when in insert mode. Emmet also doesn't seem to be working unless I trigger it from the command palette.

Unfortunately Sublime won't trigger keystroke handlers on intercepted keys. This affects automatic autocomplete and snippet popup. I don't know what to do about this besides asking Sublime for a fix or recommending a solution on the Vim side.

What does Emmet working look like?
I might actually be able to work around snippets and autocomplete by reimplementing the Sublime version using Phantom. For snippets, do you rely on the popup menu much, or just type and press tab? You could bind the snippet completion button to another key that works while inside Vim, or make a Vim command that passes the tab back to Sublime.

Emmet expands everything dynamically, so typing something like ul>li*2>atab would generate:

This kind of HTML completion works just fine with the Vim version of Emmet, but it doesn't support CSS expansions like mt15tab to get margin-top: 15px.
Is there a way to pass the tab back to sublime from vim? It probably would be the best way to solve this.

I think #97 will solve this, but if it doesn't, there are a few options for giving Sublime Text the tab key.

Add a keybinding with sufficient context to steal it from Vim (so Vim never sees it). This would probably involve the mode-based settings (in ActualVim settings, you can add arbitrary settings to change on the view based on current vim mode). If you set a "av_insert" setting on the view while in insert mode, then added a keybinding with that setting as the context, it might be able to override the ActualVim input handler. Looks like they rebind tab, so you could too: https://github.com/sergeche/emmet-sublime/blob/master/Default (OSX).sublime-keymap#L372

Add a generic run Sublime command from Vim callback, then bind the tab key from within Vim to call the Sublime Emmet command instead of the Vim one. Vim scripts can talk to ActualVim via rpc_notify and rpc_request (which is how the autocompletion integration works). Then you can bind tab to something that calls: rpc_notify('actualvim_view_command', 'emmet_command_name', 'arguments')

This is probably not as good of an idea, but add a setting to hook specific keys per mode and call a Sublime command instead. This could maybe use Sublime contexts too I think. It might look like this in the ActualVim settings:

mode_keybind: {
    "insert": [
        {"tab": "emmet_command", "context": "sublime context query"},
    ],
}

This would be kinda cool, and the context stuff would be better than you could do from Vim, but it might require reimplementing more Sublime functionality (I don't remember how much of the context queries you can do from the Python API), and it would fail (maybe silently unless I keep track) when you try to use a key not mapped to ActualVim's handler.

But honestly #97 should be the best general way :)

(2) is implemented in latest master:
call ActualVimViewCmd('command', {args})
This is the mapping:
ActualVimWinCmd: window.run_command
ActualVimTextCmd: view.run_command
ActualVimAppCmd: sublime.run_command


Identify layer features Sample

Description
PR to add a new Kotlin sample "Identify layer features" in Search and Query category.
Links and Data
Sample Epic: runtime/kotlin/issues/2934
What To Review

Review the code to make sure it is easy to follow like other samples on Android
README.md and README.metadata.json files

How to Test
Run the sample on the sample viewer or the repo.

@shubham7109 Thanks for the review suggestions! Applied. please re-review?

@rchintapalli sample design doesn't do feature selecting/highlighting. So leaving this as is. Thanks for your review.


DataManagement update

Decoupling of data structures and interoperability module

[x] Low-level oneDAL data management wrappers
[x] Isolation of data conversion functionality to Interoperability modules
[x] Removing compile-time dependency on DPCTL

Interoperability modules:

[x] PyBuffer protocol (via pybind11 intermediate) - input & output
[x] dlpack protocol - input only
[x] Sycl Usm Array Interface aka sua - input & output

Tests:

[x] Test conversion to and from array
[x] Test conversion to and from CSR table
[x] Test conversion to and from homogenous table

@KulikovNikita please rename inteoperability.dpctl to inteoperability.sua_iface, because actually it is interop for sua iface, that support by dpctl.tensor and dpnp.ndarrays

/intelci: run

@KulikovNikita we have to make versioning for 2024.0.0 features, since this not available yet.


[Bug]: During training, the console displays incorrect values for the epoch and step count in brackets.

Project Version
3.2.6
Platform and OS Version
Windows 11
Affected Devices
All
Existing Issues
No response
What happened?
During training, the console displays incorrect values for the epoch and step count in brackets.
Steps to reproduce
Train a voice model.
Expected behavior
Console should either show correct values for the epoch and step count in brackets or should not show the brackets part at all.
Attachments
No response
Screenshots or Videos

Additional Information
No response

you've highlighted the epoch and step with the lowest loss value, not the current epoch

OK, I didn't realize that the values in brackets are not the current epoch and step count, but rather the epoch and step with the lowest loss value. Thank you for the clarification.


APB-7202 [CS] update name group page content

Changes also applied to /agent-permissions/rename-custom-group/:id

I couldn't find anything in the slide deck or Figma about changes that clash, I assume either the labels should match create (current) or apply the diff "What do you want to call this access group?" -> "What do you want to call this group?"

Label for rename group has been updated to "What do you want to call this group?" as agreed


New version: Mozilla.Thunderbird.Beta version 118.0b2

Pull request has been created with WinGet Automation :rocket:
Microsoft Reviewers: Open in CodeFlow

Manifest Error: DisplayVersion declared in the manifest has overlap with existing DisplayVersion range in the index. Existing DisplayVersion range in index:  [ [<IP_ADDRESS>, <IP_ADDRESS>]]


Show total number of people on the payment summary for buyer

At the moment we only list present Bioklubbkorsnummer, risks missing people without.

Closed by #107


Shell ignores extra parameters

Feature Request
Proposal: trigger an error when launching the shell with extra parameters (arguments, not options).
Current behavior: when you launch the shell, arguments are simply ignored. This can be misleading.
Each database has its own CLI, and you could forget whether you need to pass the hostname to connect to as an option or as an argument. If you pass the hostname as an argument and have a local InfluxDB database running, it will connect to your local DB, but you might think it successfully connected to the remote host.
$ influx host_I_want_to_connect_to
Connected to http://localhost:8086 version v1.3.0
InfluxDB shell version: v1.3.0
>

Desired behavior: an error would be ideal. It should print the usage and indicate that the extra argument is not valid.
Use case: this just saves time in case people make a mistake in their command line.

Hi! I would like to take this if nobody is working on it.

@stop-start I believe everyone on the Influx team is focused on other efforts right now - feel free to pick this up if you'd like. Thanks.


Example of stacking LTC with convolutional layers on pytorch version

Hello!Thank you for having such a great work for many beginners like me to learn, recently I am using pytorch framework to build LTC model, but I can never use convolutional layer and LTC well together, always report the error of data dimension mismatch makes me very distressed, hope to get a pytorch framework of convolutional layer and LTC stacking example, looking forward to your reply!

Hello,
The Atari behavior cloning tutorial contains an example of convolutional layers stacked with an LTC/CfC:
https://ncps.readthedocs.io/en/latest/examples/atari_bc.html

Thank you very much for your help, I have benefited a lot!


WIP: Support webpack publicPath option

Right now it's impossible to make a build with relative paths that you could later put in any folder.
This is a breaking change, let me know what you think.

Looks good. Add some docs and I'll merge and publish. I don't mind adding release rights for you.

Add some docs and I'll merge and publish.

Will do!

I don't mind adding release rights for you.

Sounds good! I'd add some basic tests, linting, etc.

Done with docs ;-)


Docker app standards

User Need
As a developer
I want a template or clearly defined set of minimum standards for this tool
so that I can deliver quickly, consistently and in line with best practice.
Value
Ensure our products are consistent and follow best practice.
Functional Requirements:

Easy to use template

Non-Functional Requirements:

Acceptance Criteria:

[ ] Define and record standard (in ADR?)
[ ] Create template to exemplify standard (in operations-engineering-example app)
[ ] Easy to use template

Notes:

Top be revisited as part of App Standards work.


Detect changes to JS objects in global scope

https://privacy-test-pages.glitch.me/security/js-leaks.html
Is there anything here we're not already picking up on? PS, it's massively worse in nightly
Here is my main pimped arkenfoxed FF92 vs Firefox 92

serviceworkers are disabled <-- is in TZP, and not considered stable for a FP
beacon is disabled <-- is in TZP
wtf are the other three

Properties Added

    new property window.applicationCache
    new property window.sidebar

Properties Removed

    mising property window.clientInformation.sendBeacon
    mising property window.clientInformation.serviceWorker
    mising property window.navigator.sendBeacon
    mising property window.navigator.serviceWorker

Properties Changed

    window.location.toString changed configurable, enumerable, writable, type, value

here's my nightly vs Firefox 92
Properties Added

    new property window.onsecuritypolicyviolation
    new property window.onslotchange
    new property window.reportError

Properties Removed

    mising property window.clientInformation.sendBeacon
    mising property window.navigator.sendBeacon
    mising property window.ondragexit

Properties Changed

    window.document changed configurable
    window.location changed configurable
    window.location.toString changed configurable, enumerable, writable, type, value
    window.top changed configurable
    window.window changed configurable

So I guess we need to look at the full list, not the differences

This is similar to comparing keys or hashes of the window object, but this script goes further and stores a snapshot json file to compare each object (and sub objects) descriptor, properties, and methods.
Here's the snapshot function. After gather this, we would fetch the json file and then perform a deep diff check.
function collectProps () {
    const objCache = new Set();
    // skip these
    objCache.add(window.self);
    // prevent us decending into the DOM
    objCache.add(document);
    objCache.add(document.activeElement);
    objCache.add(event?.currentTarget);
    const recursionAllowed = new Set();
    recursionAllowed.add('navigator');

    function extractObjectProps (obj, breadcrumbs = [], depth = 0) {
        if (depth >= 5) {
            return {};
        }
        const props = {};
        const proplist = [];
        for (const prop in obj) {
            proplist.push(prop);
        }
        proplist.sort().forEach((prop) => {
            try {
                const desc = Object.getOwnPropertyDescriptor(obj, prop) || {};
                desc.type = typeof obj[prop];
                if (desc.type === 'function') {
                    desc.value = obj[prop].toString();
                } else if (desc.type === 'object') {
                    if (!objCache.has(obj[prop]) || recursionAllowed.has(prop)) {
                        objCache.add(obj[prop]);
                        desc.properties = extractObjectProps(obj[prop], [...breadcrumbs, prop], depth + 1);
                    }
                    delete desc.value;
                }
                props[prop] = desc;
            } catch (e) {
                console.warn(`Error for ${breadcrumbs.join('.')}.${prop}`, e);
            }
        });
        return props;
    }

    return {
        window: extractObjectProps(window)
    };
}

result = collectProps().window
jsonPrettified = JSON.stringify(result, null, 2)

After gathering this, we would fetch the json file and then perform a deep diff check

IDK about you, but I wouldn't want to generate a diff, just a hash. SO the question becomes, what does this add in terms of FPing: is it stable, or affected by extensions - is it mostly/all just equivalency of other measurements already collected and/or of platform/engine?


replace doLock/doUnlock with runLocked

pass function via alias parameter to allow inlining

I tried the benchmarks for win32 and didn't notice any performance regressions or improvements, so I'm fine with the refactoring.
Unfortunately compiling with -debug=PROFILE_API results in
..\src\gc\gc.d(349): Error: function gc.gc.GC.runLocked!(mallocNoSync, mallocTime, numMallocs, uint, uint, uint, const(TypeInfo)).runLocked cannot inline function
..\src\gc\gc.d(349): Error: function gc.gc.GC.runLocked!(sizeOfNoSync, otherTime, numOthers, void*).runLocked cannot inline function
..\src\gc\gc.d(349): Error: function gc.gc.GC.runLocked!(queryNoSync, otherTime, numOthers, void*).runLocked cannot inline function
..\src\gc\gc.d(349): Error: function gc.gc.GC.runLocked!(go, otherTime, numOthers, Gcx*, void*).runLocked cannot inline function
..\src\gc\gc.d(349): Error: function gc.gc.GC.runLocked!(go, otherTime, numOthers, Gcx*, void*).runLocked cannot inline function
..\src\gc\gc.d(349): Error: function gc.gc.GC.runLocked!(go, otherTime, numOthers, Gcx*, void*, uint, const(TypeInfo)).runLocked cannot inline function
..\src\gc\gc.d(349): Error: function gc.gc.GC.runLocked!(go, otherTime, numOthers, Gcx*, void*).runLocked cannot inline function

BTW: With the current version I could not measure any difference anymore when compiling with PROFILE_API (but not using it). Maybe we can always enable it into the build.

Stupid inliner, will try to resolve this.
We can reduce it to a single GC.config.profile > 1 branch, so to use only a single branch predictor slot, then it should be fine to always enable the profile API.

People should stop marking everything with pragma inline.

People should stop marking everything with pragma inline.

Not at all, we have measurable problem (a few %) if this isn't inlined.

Not at all, we have measurable problem (a few %) if this isn't inlined

As I understood it, marking things as inline changes nothing.  Your only turning non-inlineable code into an error.

As I understood it, marking things as inline changes nothing.

At some point it's supposed to become an inline hint.

Your only turning non-inlineable code into an error.

I explicitly added it b/c it is a performance regression to change the function in a way that it becomes non-inlineable.

Fixed for debug(PROFILE_API), dmd isn't capable of inlining the code, so I separated the function.
Until the inlining issues isn't resolved we can't enable PROFILE_API by default.

Why don't you just disable the pragma(inline) when PROFILE_API is on?

Why don't you just disable the pragma(inline) when PROFILE_API is on?

pragma(inline) only accepts tokens true and false as argument, not a compile time expression, making it very clumsy to use conditionally. But IIRC it has the rather unexpected version where you can put the pragma into the function body.

Yes, using it as a statement should work.

updated

Thanks.

Auto-merge toggled on

This is now blocking my inlining improvement https://github.com/D-Programming-Language/dmd/pull/5292. With dmd git master, the instantiation runLocked!(reallocNoSync, ...) at src/gc/gc.d L552 is not inlined because reallocNoSync returns void and is called directly on ExpStatement at L356. By the dmd PR 5292, the call will also be tried to inline forcibly, but reallocNoSync is too big to expand its body, then it fails.
In my humble opinion: druntime is built with -inline by default. So the new runLocked API is needlessly limited by the compiler's inliner capability due to the use of pragma(inline, true). Removing it would have no problem.

I don't care too much about reallocNoSync, but runLocked itself.

With dmd git master, the instantiation runLocked!(reallocNoSync, ...) at src/gc/gc.d L552 is not inlined because reallocNoSync returns void and is called directly on ExpStatement at L356. By the dmd PR 5292, the call will also be tried to inline forcibly, but reallocNoSync is too big to expand its body, then it fails.

Why would it forcibly inline reallocNoSync? This is not the sematics of the pragma(inline) inside the function body (albeit a bad aberration from other attributes).

In my humble opinion: druntime is built with -inline by default. So the new runLocked API is needlessly limited by the compiler's inliner capability due to the use of pragma(inline, true). Removing it would have no problem.

If runLocked cannot be inlined we have seen a performance regressions in the past, with the druntime benchmarks running a few percents slower. The most problematic case is the malloc, though, so we wouldn't care too much for all the other calls.

Why would it forcibly inline reallocNoSync? This is not the sematics of the pragma(inline) inside the function body (albeit a bad aberration from other attributes).

In short, it's current behavior. (Note that, today pragma(inline, true) auto func() { ... } and auto func() { pragma(inline, true); ... } are exactly same behavior.)

If runLocked cannot be inlined we have seen a performance regressions in the past,

The code performance with -inline is only limited by the compiler's inlining capability. pragma(inline, true) would introduce no additinal performance. See also: https://github.com/D-Programming-Language/druntime/pull/1496#issuecomment-181650286

But it fails to compile when a function (runLocked) cannot be inlined, which is what we want here to prevent someone from introducing a performance regression.

Maybe we should leave that few percent to more serious backends?

But it fails to compile when a function (runLocked) cannot be inlined, which is what we want here to prevent someone from introducing a performance regression.

But current dmd inliner has known problems. By using pragma(inline, true), druntime is now relying on its bug-prone behavior. And sticking on current performance would prevent further inliner improvements. Actually my https://github.com/D-Programming-Language/dmd/pull/5292 is now dying.


Make it easier to run schema version check locally

User story
As a user, I want to more easily test that the schema versions have been incremented, so I can no longer have to wait for github actions
Acceptance criteria

[ ] Given an s3 endpoint, when a user runs a command in the command line, then the latest version of aind-data-schema will be downloaded from s3 and compared against the versions changed on the branch.
[ ] Alternatively, if there is an easy way to compare it against the local main branch, that might be more ideal

Sprint Ready Checklist

[ ] 1. Acceptance criteria defined
[ ] 2. Team understands acceptance criteria
[ ] 3. Team has defined solution / steps to satisfy acceptance criteria
[ ] 4. Acceptance criteria is verifiable / testable
[ ] 5. External / 3rd Party dependencies identified
[ ] 6. Ticket is prioritized and sized

Notes
Add any helpful notes here.

Will potentially be implemented when this gets fixed: https://github.com/AllenNeuralDynamics/aind-data-schema/issues/688
@jtyoung84 Convert this to user story


[DOCS]

Description
The "HOW TO" readme in public/server has a TODO that says you need to add instructions r contributing to the main readme. This has already been done actually. Should I update the README (I mean the one in public/server)?
Screenshots

Additional information
No response

There have been some major updates. With the new merge, this is no longer an issue. I think it's okay to close this issue. What do you think?

That's fine. Since it's been solved 👌🏾


feat: custom native library location

Added brotli4j.library.path property for specifying native library location, This gives more flexibility to load brotli native library.

Thanks a lot!


[blend2d] Remove vcpkg_fail_port_install.

Only wasm32 was missing.
Also ci.baseline.txt was out of date.
In support of https://github.com/microsoft/vcpkg/pull/21502

Thanks!


Add screenshots on the README

Checklist

[X] There are no similar issues or pull requests for this yet.

Is your feature related to a problem? Please describe.
No problem...
Describe the solution you would like.
I shared this package with a friend, and he said: "some screenshots on the README would be great...".
Just sharing his feedback here... :eyes:
Describe alternatives you considered
No response
Additional context
No response

Hey, @Kludex
Thanks for the feedback. Yeah you're right, I'll add them.
By the way you can check the online demo too.

I added them, thank you 👍

Thank you :pray:


ADD font to child item when isMultiSelect is enabled

FIX bug when you have isMultiSelect enabled the font was not setted to the cell and make it a little more cleaner with a configureCell function .
Remove test target that was empty and adds compilation errors when XCode version updates.

deprecated


Reconsider currying

The spec currently require that all methods of each type must be curried. Which is cool, but might be problematic sometimes.
Some problems are:

It might affect performance
It makes call stack ugly which is a problem when debugging etc.
One have to depend on a module that provides a curry function
We can't have optional arguments in a curried function

We could change the spec in a way that it won't require methods to be curried, but also won't require otherwise, so people could choose to use currying or not by themselves.
Related: #2

Another question we should ask ourselves "Is point free actually a good thing?". Take a look at the example:
foo(T.bar(x))

foo(y => T.bar(x, y))

In the second line one can tell much more about what is going on by just looking at the line itself, while to understand the first line they will need to check docs, other files, etc.

Plan

[ ] Remove note from spec.md
[ ] Rewrite build in types and derive utils without currying
[ ] Remove utils for currying

Currying and function composition are fundamental concepts of FP. Being point-free is just one property of them. static land makes this unnecessary and thus should embrace function composition, which is greatly facilitated by currying. Functions in Haskell are curried by default and isn't fantasy/static land the attempt to transfer Haskell idioms to Javascript?

It might affect performance

I guess this is merely micro-optimization. If you encounter performance penalties in a particular portion of your code, optimize that portion.

it makes call stack ugly which is a problem when debugging etc.

Good point.

One have to depend on a module that provides a curry function

Arrow functions provide a succinct syntax to manually define curried functions. No need for a programmatic solution.

We can't have optional arguments in a curried function

Optional arguments are evil, just like variadic functions. With curried functions you simply don't need them:
const f = opt => x => y => ...;
const g = f("whatsoever");

Would you re-reconsider currying?

Before this issue, when currying was in the spec, it was required for all methods to support both T.foo(a)(b) and T.foo(a, b) modes (Ramda-like currying). So arguments I was making were about no currying on one hand and Ramda-like currying on the other.
You suggest a bit different approach to support only T.foo(a)(b). In this case the arguments against currying are indeed weaker. But we will (need)(to)(do)(this) all the time. So not sure...

And this can really hurt performance actually. Consider some function like fold that accepts a static-land Monoid. If it called on a collection with a lot of items and instead of T.concat(a, b) we're doing T.concat(a)(b), creating a temporary function on each step, the performance hit can be huge.

Another idea. Currying can be implemented as a library that accepts a normal (current spec) static-land dictionary and returns a new one with curried methods. This way people can enjoy currying for any SL type without spec requiring it.

First of all I think that the approach static land specifies will hit the big-time. I've played around a bit with an implementation and it actually solved problems and simplified code. Apart from the verbosity I haven't been faced with any drawbacks yet.
Back to the topic: I don't want to be dogmatic here. My experience is that whenever I have an uncurried function in my toolset, I sooner or later re-implement it as a curried version. Also I've never experienced performance hits caused by currying.
On the other hand I cannot assess the impact of currying on the spec or on corresponding implementations. Does it really matter? I would say yes, but that is rather intuitively. I need more experience.
Your last suggestion sounds promising. Let's wait and see if currying is also an issue for others.

Sounds good, @ivenmarquardt , let's wait for more feedback.
Just one more option I want to mention, we always can use a helper like partial e.g., lifted = partial(T.map, f). I probably like this the most because it's explicit.

Like @ivenmarquardt, I also find myself currying functions as I am a proponent of the compositional / point-free style.
Like anything, I think there is a balance to be struck when using point-free in regards to readability.

Interesting article on stack-trace / debugging:
<EMAIL_ADDRESS>

@rpominov Re; (need)(to)(do)(this), Sanctuary recently changed all their APIs to simple currying, and they find that appropriate (use) (of) (whitespace) makes things look OK despite the noisier function application syntax in JS. Additionally, JS now has a proposal for the reverse application |> operator, which plays really nicely with curried, static functions:
  inputObservable
    |> Obs.map(x => x * 2)
    |> Obs.flatMap(x => Obs.delay(1000, x))
    |> Obs.filter(x => x < 50)
    |> Obs.scan(x => y => x + y)

@masaeedu Btw., you can have function composition with arbitrary number of curried functions along with simple recursion:
// right to left
const comp = f => Object.assign(g =>
  comp(x =>
    f(g(x))),
    {runComp: f});

// left to right
const pipe = f => Object.assign(g =>
  pipe(x =>
    g(f(x))),
    {runComp: f});

const inc = n => n + 1;
const sqr = n => n * n;

comp(sqr)
  (inc)
    (inc)
      (inc)
        (inc)
          .runComp(0);

I prefer this recursive encoding over the spread operator (e.g. pipe(sqr, inc, inc, ...), because the bracket notation indicates function application more naturally. The Lisp style code indentation is merely my personal style.

I still feel like forcing everyone to call functions in weird way is not a good idea. I know many people are okay with it, but probably many will not be. Also performance issue still looks like a real issue to me. It's true that in most cases it won't matter, but if a curried function call appears in a hot path it will be a problem. I hope https://github.com/tc39/proposal-partial-application lands soon, seems like it solves all problems.

I still feel like forcing everyone to call functions in weird way is not a good idea

Since the introduction of arrow functions, manually defining curried functions has become easy, and the only way to call
const K = x => _ => x;

is
K(42)(null);
// or
K (42) (null);

I'll admit it took me a little getting used to (a couple of days), but we're talking about a different programming paradigm here, and the difference in calling patterns emphasizes this.

Spaces are crucial. This expression would be difficult to read without spaces:
B (B (B (B (eitherToMaybe)))) (encaseEither3 (I))

I don't want to start this discussion all over. I was just wondering if curried functions still conform to the spec as it was suggested by the original poster. The only hint in the specification is implicit in the section about module-signatures, where  a => b => c seems to be different from (a,b) => c, so my guess is, curried functions do not conform anymore, am I correct?

@RaoulSchaffranek You can define a function in a way that both fn(a, b) and fn(a)(b) would work. In this case it will be compatible, since the specification only requires fn(a, b) to work.


implementing slurm.schedmd.com/pam_slurm_adopt.html to add cgroups, pam_slurm_adopt and ssh admin access

This script is doing 3 actions following the documentation from https://slurm.schedmd.com/pam_slurm_adopt.html

add cgroup enforcement to fence jobs, memory for exemaple with MaxRAMPercent
add pam_slurm_adopt support to prevent user to ssh without jobs running on that node
add wheel group support to allow ssh

pam_slurm_adopt will always allow the root user access.
To allow other admins to the system, there are 2 PAM implemented options to allow users to ssh:

pam_access.so using ${access_conf}
pam_listfile.so using ${wheel_list}

pam_slurm_adopt
The purpose of this module is to prevent users from sshing into nodes that they do not have a running job on, and to track the ssh connection and
any other spawned processes for accounting and to ensure complete job cleanup when the job is completed. This module does this by determining the job
which originated the ssh connection. The user's connection is "adopted" into the "external" step of the job.
When access is denied, the user will receive a relevant error message.

Is the latest script ready for testing?


Add EMD for Rare Phenomena at Colliders Paper

Resolves #74
Add "Use of a Generalized Energy Mover’s Distance in the Search for Rare Phenomena at Colliders by M. C. Romao, @nfcastro, J. Milhano, R. Pedro, and T. Vale

Thanks!


Update SPIRV-Tools and SPIRV-Headers

Pick up first SPIRV-Tools v2017.1-dev commits

Tested on Release and Debug builds.
LGTM


After latest windows update, all visual studio code shortcuts, executables, etc are gone. Like it vanished.

Issue Type: Bug
The title says it all.  I updated Windows with the required Windows Update Friday night (5/27).  Saturday when I go to look at the code I'd worked on in MS VSC, I noticed the taskbar icon is missing.  Then I hit the Windows key and start typing visual code, nothing returns  - only MS Visual Studio 2017. I was worried I'd lost a ton of code from the night before but luckily, the source code in my local repository was updated. But it was a huge scare as I was gone this weekend for a wedding I was to be in.  I just got home and was able to see the code was still there... seriously... thankfully.  The way I got VSC to run was at this path: C:\Program Files (x86)\Microsoft VS Code_\bin\code (windows command script).
 You should probably file an issue to the Windows team: https://support.microsoft.com/en-us/help/27932/windows-10-send-us-feedback
As for Code, just reinstall it, you should be fine.


Limit extract one time

Extraction button will now show a successful/failed extraction based on first attempt or not.
@laudanum123 tried to declutter adventure_manager.vue but failed, if you have any ideas to move the notification to another component, please let me know

Hi @YazidKurdi ,
Could you have a look at this PR, it seems due to my npc creator stuff now we have some merge conflicts here. Could you check which changes are required for your code to work?
Regards


[SPARK-39572][PYTHON][TESTS] Fix test_daemon.py to support IPv6

What changes were proposed in this pull request?
This PR aims to fix test_deamon.py to support both IPv6 additionally.
Why are the changes needed?
To add IPv6-only test coverage.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Pass the CIs and do the following manually.
$ build/sbt package
$ SPARK_PREFER_IPV6=true python/run-tests.py --modules pyspark-core

Maybe we can even set a scheduled job for this once the tests pass ..

Thank you, @HyukjinKwon ! Yes, currently, I'm running Jenkins locally. We should build IPv6 CI.
Merged to master.


how can I get detection results?

I want visualize the predicted image, can you share the demo for visualization? thank you very much.

i have solve this problem. thanks

Hi @binyisu
could you share the code for visualization? It would be great help.


Create WinGet package for March 2024 GDK Update 2

Create WinGet package for March 2024 GDK Update 2
Tested locally with winget validate and winget install
Microsoft Reviewers: Open in CodeFlow

@microsoft/xbox-developer-experience-toolbox for review

LGTM

Hi @aaron-stebner,
This manifest removes Apps and Features entries that are present in previous PR versions. These entries should be added to this version, to maintain version matching, and prevent the 'upgrade always available' situation with this package.
(Automated response - build 901.)

Hi @aaron-stebner,
This manifest removes Apps and Features entries that are present in previous PR versions. These entries should be added to this version, to maintain version matching, and prevent the 'upgrade always available' situation with this package.
(Automated response - build 901.)

I'm not seeing any Apps and Features entries get removed.  Can you elaborate on this more please?


[FEATURE] Enable user play fun games whiles waiting to be paired with a new buddy

Description
This could keep users on the app for some time, and make the waiting time less boring and more fun.
The game doesn't have to be complex.
Users may even choose the type of game they'd like to play from the list of available options.
And maybe somewhere in the future, they can play it with their buddy?
Screenshots
No response
Additional information
No response
👀 Have you checked if this issue has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Contributing Guidelines?

[X] I have read and understood the rules in the Contributing Guidelines

Are you willing to work on this issue ?
None

good idea, but no


Fixed filter in UI

First of all, many thanks for making this theme. This is so far the best theme I have seen for Django admin.
Issue:
Is there a way to make filter on sidebar fixed in UI instead "click and reveal" behaviour?
I use it frequently, and clicking and selecting multiple filters can be annoying. It becomes useful when you have more fields to display on the admin list page. You can move boolean fields to the sidebar for quick filtering instead of displaying them in the list.

I know what you mean, I was already thinking about it. This task would require big template (UI & code) overhaul and introduction of new flag in ModelAdmin to switch between different layouts.


Quantum bridge card compatibility with compact machines

Although the card works across dimensions (including custom dimensions) it doesn't inside a compact machine.

Is the network loaded?
there shouldn't be anything special about the compact machines dimension

Chunk? yes.....
The inventory shows briefly then disappears

https://user-images.githubusercontent.com/8942699/214384954-65aa26db-7c05-4655-ac7d-bda528d677a2.mp4

I have seen this issue before, but I don't quite know what causes it.
there are two options I can think of:

Your network doesn't have enough power / it uses more power in a tick than it can store (one dense cell would be enough)
the chunkloading doesn't work

Spot on, one dense cell and issue resolved, thanks.
(Weird that it's ok in other dimensions though /shrug)


pythonPackages.tinyprog: init at 1.0.21

Motivation for this change
Add python lib. This required to add jsonmerge library and bump version for intelhex (done in separate commits).
Things done

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

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

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

@andrey-komarov are you able to address the above comments and continue with this PR?

Closing since #66521 was merged


[ZEPPELIN-5900] Fix RemoteSchedulerTest

What is this PR for?
This PR changes the RemoteScheduler
The main change is the executor of the RemoteScheduler. It's now a separate FIFO Executor (one Thread). As a result, jobs are now queued in the Zeppelin server and not in the RemoteInterpreter.
Since previously the jobs were queued in the remote interpreter due to parallelism in the Zeppelin server, the cancellation of jobs did not work, which is why the test failed.
The interrupt behaviour has also been improved.
What type of PR is it?
Bug Fix
What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-5900
How should this be tested?

CI

Questions:

Does the license files need to update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

As this PR changes a very important component of Zeppelin, I need an accurate review. If you want me to describe the changes in more detail, please let me know.

Thanks @Reamer  so much for fixing the failed test case ! It's a great job.
It's weird that the RemoteSchedulerTest runs well under Junit 4, Do you know why?

BTW it's weird that the RemoteSchedulerTest used to runs fine on Junit 4, Do you know why?

Unfortunately no.
However, since only the timing matters due to the six threads involved, JUnit-5 may have improved things here.
But perhaps the CI-VM is so slow that the error occurs less frequently there. However, I have seen the error several times under JUnit 4.

Does this FIFO Executor make global share interpreter cannot run in concurrently  and invalidate parameter like zeppelin.jdbc.concurrent.use ？ @Reamer

I am currently on vacation without a PC. Unfortunately, I don't know the implemention of the configuration. Can you send me code links where the configuration is used.

The configuration zeppelin.jdbc.concurrent.use is used in function isConcurrentExecution(). You can find it in codes above