Glove randomly produces NaN weights or nonsensical similarities

Issue Description
In version 1.0.0-beta, there is (still) a bug in org.deeplearning4j.models.glove.Glove that causes it to produce a org.deeplearning4j.models.embeddings.WeightLookupTable with NaN as weights. This can be observed when running org.deeplearning4j.examples.nlp.glove.GloVeExample or any other code that uses dl4j's Glove implementation. Here's some of the logging from the GloVeExample that demonstrates this behaviour:
09:44:12.864 [VectorCalculationsThread 3] INFO org.deeplearning4j.models.embeddings.learning.impl.elements.GloVe - Processed [47194] pairs, Error was [3.370708701478753E7]
09:44:14.409 [VectorCalculationsThread 3] INFO org.deeplearning4j.models.embeddings.learning.impl.elements.GloVe - Processed [94388] pairs, Error was [NaN]
09:44:16.074 [VectorCalculationsThread 3] INFO org.deeplearning4j.models.embeddings.learning.impl.elements.GloVe - Processed [141582] pairs, Error was[NaN]
09:44:17.680 [VectorCalculationsThread 3] INFO org.deeplearning4j.models.embeddings.learning.impl.elements.GloVe - Processed [188776] pairs, Error was[NaN]
09:44:19.287 [VectorCalculationsThread 3] INFO org.deeplearning4j.models.embeddings.learning.impl.elements.GloVe - Processed [235970] pairs, Error was[NaN]
09:44:21.388 [VectorCalculationsThread 3] INFO org.deeplearning4j.models.embeddings.learning.impl.elements.GloVe - Processed [283164] pairs, Error was[NaN]
09:44:22.996 [VectorCalculationsThread 3] INFO org.deeplearning4j.models.embeddings.learning.impl.elements.GloVe - Processed [330358] pairs, Error was[NaN]
09:44:24.757 [VectorCalculationsThread 3] INFO org.deeplearning4j.models.embeddings.learning.impl.elements.GloVe - Processed [377552] pairs, Error was[NaN]
09:44:26.548 [VectorCalculationsThread 3] INFO org.deeplearning4j.models.embeddings.learning.impl.elements.GloVe - Processed [424746] pairs, Error was[NaN]
09:44:28.314 [VectorCalculationsThread 3] INFO org.deeplearning4j.models.embeddings.learning.impl.elements.GloVe - Processed [471940] pairs, Error was[NaN]
09:44:30.073 [VectorCalculationsThread 3] INFO org.deeplearning4j.models.embeddings.learning.impl.elements.GloVe - Processed [519134] pairs, Error was[NaN]

What is important is that NaN values are not always produced. Sometimes real numbers are produced, sometimes it's NaN.
Another problem is that, even if real numbers are produced, the word similarities are off the mark. E.g.
o.d.e.n.g.GloVeExample - Day/night similarity: 0.06969143450260162
o.d.e.n.g.GloVeExample - Nearest words to 'day': [next, yesterday, a, season, place, one, who, program, best, house]
There are already several tickets relating to this issue, but they have either been closed or pertain to older versions of dl4j cf: https://github.com/deeplearning4j/deeplearning4j/issues/1622, https://github.com/deeplearning4j/deeplearning4j/issues/3462
Therefore as requested by @RobAltena I'm creating a new ticket. I've also created a mini-project with a unit-test that runs the GloveExample multiple times and fails if any of the weights are NaN. As can be seen, the test fails after a several iterations (or sometimes even the first). If the test doesn't fail upon the first time you run it, please run it several times over. Eventually it will fail.
Please note that changing the org.nd4j.graph.DataType does in no way remedy this problem.
The code can be found in these gists: GloveRunner, GloveRunnerTest, pom.xml
I've also added it as a github project here: https://github.com/daanvdn/glove-nan-bug
Version Information
deeplearning4j version 1.0.0-beta

Thank you! I have send you a PR https://github.com/daanvdn/glove-nan-bug/pull/1 to clean up the code a little. Should make it easier to move your junit test into deeplearning4j.

your're welcome @RobAltena. I just approved the PR and merged. Do you need any other input from me?

Lets wait from some hints/tips from the core devs.

hi @RobAltena is there any news about this issue? Is there any one I can reach out to?

It's enqueued, but it's considered low priority.

Closing due to GloVe feature removal. See https://github.com/eclipse/deeplearning4j/issues/7855#issuecomment-624616313


Expose use_emulator option for GCS

Which issue does this PR close?
Closes #2627
Rationale for this change
Currently a test-only option is exposed, to configure an unsafe client. To avoid exposing third party types in the public API, I moved the same logic as used in test behind a use_emulator option.
Also after some trials around copy-if-not exists, it seemed to me fake-gcs so far does not support this. fsouza/fake-gcs-server#909.
What changes are included in this PR?
New builder option for GCS object sore.
Are there any user-facing changes?
yes

I wonder if we should instead be configuring fake-gcs to use HTTP, we could then provide an with_allow_http like the we do for S3 🤔
I'll have a quick play

This is likely much better - consistent and infallible 😀.

I switched away from using insecure TLS in https://github.com/apache/arrow-rs/pull/2632 PTAL

From my perspective, your approach using http is superior, so going that way makes more sense. Given the need is only to do integration test "somehow"...

Ok let's move forward with that then


FIO JSON output format has changed.

Parsing scripts break on the latest version of FIO Json output.

@tigerquoll can you provide the output of the following?
fio --version

Good call - it was as the earlier version of FIO in the mix - RHEl 7.4 repos that our boxes were talking to used FIO 2.x, while the RHEL 7.5 box that were using as the master automatically pulled down 3.x.  FIO does not seem to detect when it is trying to remotely drive an older version - it will silently fail.  Updating all FIO versions to the same 3.x version fixed the issue.


Flutter doctor not generated in Ubuntu

Describe the bug
Ubuntu didn't generate flutter doctor -v
Steps to reproduce

run issue creation on Ubuntu.

Expected behavior

See flutter information on the issue.

Actual results

No flutter information.

Additional context
Screencast from 18-10-22 16:11:18.webm
This issue was created using issue.

Flutter doctor -v
[✓] Flutter (Channel stable, 3.3.4, on Ubuntu 22.04.1
    LTS 5.15.0-50-generic, locale en_US.UTF-8)
    • Flutter version 3.3.4 on channel stable at
      /home/bri/sdk/flutter-stable
    • Upstream repository
      https://github.com/flutter/flutter.git
    • Framework revision eb6d86ee27 (2 weeks ago),
      2022-10-04 22:31:45 -0700
    • Engine revision c08d7d5efc
    • Dart version 2.18.2
    • DevTools version 2.15.0

[✓] Android toolchain - develop for Android devices (Android SDK version 32.1.0-rc1)
    • Android SDK at /home/bri/Android/Sdk
    • Platform android-32, build-tools 32.1.0-rc1
    • Java binary at: /home/bri/app/android-studio/jre/bin/java
    • Java version Java(TM) SE Runtime Environment (build 17.0.1+12-LTS-39)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at google-chrome

[✓] Linux toolchain - develop for Linux desktop
    • Ubuntu clang version 14.0.0-1ubuntu1
    • cmake version 3.22.1
    • ninja version 1.10.1
    • pkg-config version 0.29.2

[✓] Android Studio (version 2021.2)
    • Android Studio at /home/bri/app/android-studio
    • Flutter plugin version 70.2.2
    • Dart plugin version 212.5744
    • Java version Java(TM) SE Runtime Environment (build 17.0.1+12-LTS-39)

[✓] VS Code (version 1.71.2)
    • VS Code at /usr/share/code
    • Flutter extension version 3.50.0

[✓] Connected device (2 available)
    • Linux (desktop) • linux  • linux-x64      • Ubuntu 22.04.1 LTS 5.15.0-50-generic
    • Chrome (web)    • chrome • web-javascript • Google Chrome 105.0.5195.125

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

• No issues found!

fixed in version 0.0.4. closing...
feel free to reopen if you feel otherwise.


Traffic Pattern Dependencies

Dear Arzani et al.,
Your paper in NSDI was quite enjoyable to read, and the system proposal I find interesting. I have a question regarding line 11 of Algorithm 1: what traffic pattern assumptions does it make? I cannot seem to find it (or the Algorithm 1 itself) in the code. In the text is stated:

Namely, we iteratively pick the most voted link l_max and estimate the portion of votes obtained by all other links due to failures on l_max. This estimate is obtained for each link k by (i) assuming all flows having retransmissions and going through l_max had drops due to l_max and (ii) finding what fraction of these flows go through k by assuming ECMP distributes flows uniformly at random.

In particular, P is set as the set of all possible paths, and if a link shares at least one path with l_max, its score is "adjusted". Is this adjustment done by making use of the knowledge what flows were active on which path, or done obliviously by calculating the ECMP spreading under the assumption of an all-to-all traffic matrix?

Hi Simon,
Basically, the number of votes a link gets (l_max for example) shows the
number of connections that went through that link and that had packet drops
(remember votes are just the number of flows that had dropped packets
divided by the path length). What we did for the work was to assume a
uniform distribution of flows across ToRs (or hosts) and used this
assumption when backtracking . What this means is that, by using this
assumption, if you are on a T_1 link you can calculate what fraction of the
flows would go through a T_2 link and which fraction would go down to
another ToR. (we ignored T_3 for this work). Note that you get less and
less contribution to the votes on a link as you move away from it.
Another approach which is equally valid (however requires you to keep track
of all tracerouted paths) would be to take advantage of the fact that you
know the path of all flows that contributed to the votes gotten by l_max,
you can go back and set the votes of the flows that went through that link
to 0 for all other links that they went through and re-compute the votes.
This is more computationally expensive, but at the same time is also more
accurate as it removes the assumption that flows are distributed uniformly
at random across ToRs.
Hope this helps,
On Wed, Oct 10, 2018 at 4:28 AM Simon<EMAIL_ADDRESS>wrote:

Dear Arzani et al.,
Your paper in NSDI was quite enjoyable to read, and the system proposal I
find interesting. I have a question regarding line 11 of Algorithm 1: what
traffic pattern assumptions does it make? I cannot seem to find it (or the
Algorithm 1 itself) in the code. In the text is stated:
Namely, we iteratively pick the most voted link l_max and estimate the
portion of votes obtained by all other links due to failures on l_max. This
estimate is obtained for each link k by (i) assuming all flows having
retransmissions and going through l_max had drops due to l_max and (ii)
finding what fraction of these flows go through k by assuming ECMP
distributes flows uniformly at random.
In particular, P is set as the set of all possible paths, and if a link
shares at least one path with l_max, its score is "adjusted". Is this
adjustment done by making use of the knowledge what flows were active on
which path, or done obliviously by calculating the ECMP spreading under the
assumption of an all-to-all traffic matrix?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/behnazak/Vigil-007SourceCode/issues/1, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AEzGNDIxbWwr0FNGVAQk34FIsEKAx7Gkks5ujdn4gaJpZM4XVIgC
.


Add support for universe/ids POST endpoint

Added support for universe/ids POST endpoint.
Improvements could be made to pull the categories from the SDE directly, versus manually coding them here.
Also very open to a better solution, this is just the one I came up with.

I pushed a commit to resolve the merge conflicts.

I fixed the formatting; I think I messed up the replacement suggestion indent.

Do me a favor and confirm that new signature works for you, and then I'll get it merged.

Yes, this works for me. Thanks!

Super, thank you for the contribution!

Released in 0.32.0.


Fix homepage_url in manifest

The incorrect URL causes an issue where clicking on the extension takes you to a 404.

Has been fixed upstream


Inline Guava's Iterators#limit

A subset of #213 that is hopefully non-controversial and safe to land in the short term because it does not involve Caffeine or shading.

I really don't think this is an issue, but if someone wants to reimplement this then feel free.

FYI @jtnord

As long as the license header is kept intact it would be ok I expect?


Add tracing disable toggle

As suggested by @stayallive in https://github.com/getsentry/sentry-symfony/pull/466#issuecomment-800084751

I'm not sure if this needs more tests, otherwise it's ready for review.

@ste93cry I would prefer not to mix the two PRs to avoid making the review harder. I've switched them on by default, you can go ahead afterward with your PR with just fixing the small conflict.

Ok, a full-red CI has shown me why that wasn't a good idea... Let's merge #466 first then, and then we can come back here.

Ok, scratch that: I've merged #466 + rebased, I'll address the comments coming from that PR and we should be good to go!


[type:refactor] refactor admin-listener.

Make sure that:

[ ] You have read the contribution guidelines.
[ ] You submit test cases (unit or integration tests) that back your changes.
[ ] Your local test passed ./mvnw clean install -Dmaven.javadoc.skip=true.

Codecov Report

Merging #5294 (b5bc854) into master (077f889) will decrease coverage by 0.08%.
The diff coverage is 85.71%.

@@             Coverage Diff              @@
##             master    #5294      +/-   ##
============================================
- Coverage     61.65%   61.58%   -0.08%
+ Complexity     8788     8645     -143
============================================
  Files          1262     1238      -24
  Lines         38780    38113     -667
  Branches       3701     3665      -36
============================================
- Hits          23908    23470     -438
+ Misses        12765    12554     -211
+ Partials       2107     2089      -18

Files
Coverage Δ

...henyu/admin/config/WebSocketSyncConfiguration.java
100.00% <100.00%> (ø)

...tener/http/HttpLongPollingDataChangedListener.java
5.60% <ø> (ø)

...admin/config/HttpLongPollingSyncConfiguration.java
66.66% <66.66%> (ø)

... and 28 files with indirect coverage changes
:mega: Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!


Corrects partners collection URL configuration

Description
Corrects partner page collection URL configuration by including the same configuration in _config.yml into _ config-dev.yml.
Check List

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

By submitting this pull request, I confirm that my contribution is made under the terms of the BSD-3-Clause License.

This was not the correct resolution to the problem of broken partner page URLs. Closing PR.


Jenkins: update inline jenkins build data as they change states or are added to the list.

Builds can show up when the PR is first made, they can finish, or they can restart on "retest this please" -- it'd be cool if those state changes didn't require refreshing.

Or, at least, add a refresh button


the vendor_cmds requester unit_test has some error

When any requester unit_test is called after the vendor_cmds requester unit_test, the GCC release build failed.
There is a example PR:
https://github.com/Wenxing-hou/libspdm/pull/17

Removing -flto from compilation will let this pass. Seems to be a compiler bug. Tested on

gcc (Ubuntu 13.2.0-23ubuntu4) 13.2.0

Also reproduces in

gcc-14 (Ubuntu 14-20240412-0ubuntu1) 14.0.1 20240412 (experimental) [master r14-9935-g67e1433a94f]

This is "fixed" by moving https://github.com/DMTF/libspdm/blob/a3e5b9966da2021a5908420a5bfa51634eebc1e5/unit_test/test_spdm_requester/vendor_request.c#L222-L227 and https://github.com/DMTF/libspdm/blob/a3e5b9966da2021a5908420a5bfa51634eebc1e5/unit_test/test_spdm_requester/error_test/vendor_request_err.c#L219-L224 inside their respective _main functions. This should be done anyways since those values are not needed at file scope and definitely do not need external linkage.

https://github.com/DMTF/libspdm/blob/704d5652b2eeb451fffbd1bb1129046e377c9cf3/unit_test/test_spdm_requester/vendor_request.c#L130-L131 https://github.com/DMTF/libspdm/blob/704d5652b2eeb451fffbd1bb1129046e377c9cf3/unit_test/test_spdm_requester/error_test/vendor_request_err.c#L118-L119
Also use of spdm_test_context is incorrect. It should be spdm_context. I'm surprised the call to the function is successful, but that's C for you. This issue was found by Rani at NVIDIA using his formal verification tool.


Debugger fails to start under Python 3.13 when user has no name

https://github.com/python/cpython/pull/29739
getpass.getuser() now raises OSError if the current user does not have a username:
Traceback (most recent call last):
  File "/usr/local/lib/python3.13/getpass.py", line 173, in getuser
    return pwd.getpwuid(os.getuid())[0]
           ~~~~~~~~~~~~^^^^^^^^^^^^^
KeyError: 'getpwuid(): uid not found: 1000'

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  [ own code removed ]
  File "/mp/raphson_mp/__main__.py", line 31, in handle_start
    app.run(host=args.host, port=args.port, debug=True)
    ~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.13/site-packages/flask/app.py", line 625, in run
    run_simple(t.cast(str, host), port, self, **options)
    ~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.13/site-packages/werkzeug/serving.py", line 1081, in run_simple
    application = DebuggedApplication(application, evalex=use_evalex)
  File "/usr/local/lib/python3.13/site-packages/werkzeug/debug/__init__.py", line 297, in __init__
    if self.pin is None:
       ^^^^^^^^
  File "/usr/local/lib/python3.13/site-packages/werkzeug/debug/__init__.py", line 315, in pin
    pin_cookie = get_pin_and_cookie_name(self.app)
  File "/usr/local/lib/python3.13/site-packages/werkzeug/debug/__init__.py", line 175, in get_pin_and_cookie_name
    username = getpass.getuser()
  File "/usr/local/lib/python3.13/getpass.py", line 175, in getuser
    raise OSError('No username set in the environment') from e
OSError: No username set in the environment

Under Python 3.12, the exact same code runs without an error and is able to start the development environment.
Environment:

Python version: 3.13
Werkzeug version: 3.0.4

Thanks!


obs-studio-plugins.obs-nvfbc: 0.0.4 -> 0.0.5

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/obs-nvfbc/versions.
meta.description for obs-studio-plugins.obs-nvfbc is: "OBS Studio source plugin for NVIDIA FBC API"
meta.homepage for obs-studio-plugins.obs-nvfbc is: ""
meta.changelog for obs-studio-plugins.obs-nvfbc is: ""
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 0.0.5 in filename of file in /nix/store/grmlzgdpy5yypf9b2hi93wkj6fygk72a-obs-nvfbc-0.0.5

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

2 package rebuild(s)

2 x86_64-linux rebuild(s)
1 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
docbookrx
obs-studio-plugins.obs-nvfbc

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/grmlzgdpy5yypf9b2hi93wkj6fygk72a-obs-nvfbc-0.0.5 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A obs-studio-plugins.obs-nvfbc https://github.com/r-ryantm/nixpkgs/archive/3e2c4ccff17c05bb3bccfe8130fa4713eb11bc4a.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/grmlzgdpy5yypf9b2hi93wkj6fygk72a-obs-nvfbc-0.0.5
ls -la /nix/store/grmlzgdpy5yypf9b2hi93wkj6fygk72a-obs-nvfbc-0.0.5/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

obs-studio-plugins.obs-nvfbc

Maintainer pings
cc @babbaj for testing.

works on my machine
also tested by https://github.com/NixOS/nixpkgs/pull/152697#issuecomment-1003370599


Yellow text (Debugger with Warnings) is unreadable when using light themes

Godot version
4.0rc1
System information
Linux/Manjaro
Issue description
When using light themes, the warnings in the debugger are almost indistinguishable from the light background.

Steps to reproduce
Use a light theme and run a project with warnings
Minimal reproduction project
N/A

Hey @Yagich !
Hope you're doing well
I would like to work on the issue.
Can you please assign this to me?

Godot engine doesn't practice assigning issues. Just make a PR

If this is still an issue, it could be a desktop environment issue. I am on a pretty vanilla Ubuntu 20 system and I am not able to reproduce this.

I'm on Windows and it's pretty hard to read

You might need to restart the editor after changing the theme for the change to fully take effect.
Can you confirm that it works fine after a restart @Yagich?

Ah yes, after restart it looks fine!


Fix visualization and default model for text detection PP-OCR

Minor PR to correct the default model path and address issues with visualization. The issue was partially discussed in https://github.com/opencv/opencv_zoo/issues/135#issuecomment-1962323969.

@WanliZhong Please take a look at this PR.


Add support for Mach-O file format used by Mac OS X

Please consider adding support for the Mach-O file format so Mac OS X users can enjoy this project natively too.

Thanks a lot for your suggestion. However, right now I have no experience for Mac OS, and no machine either... So sorry.

@filipochnik I added a test to build a "real" executable file (simple command to output "hello" on stdout). And also updated the test to append the zip file contents on it. Now, everything seems to be OK on travisCI with OSX. So I think I can say that this library is OK with OSX too.

I will now close this issue.


Remove versioning from Flask in requiremnts.txt to get rid of Werkzeu…

…g version bug. This is a simple app and doesn't need to be pinned to a version.

#sign-off


Remove webroot_paths cycling to match domains list

When fewer webroot_paths than domains are supplied, certbot/LE will use
the last webroot path given rather than the module cycling through
webroot_paths. When webroot_paths is one element, there will be no
behaviour difference.
Fixes GH-28

:+1: thanks!


Fix the error when save adv model and use modelcheckpoint callback

@wangbingnan136，你能看一下并在上面的评论中签署贡献者许可协议吗？

It looks like I have already signed

I think the checks were done,sorry this is my first time make a contribution to tensorflow~

ok,I got it!

It is done

Thanks for the fix! Let me polish the example notebook a bit before merging the PR.

Okk

I found that when we load the model，the "fit" and "predict" methods works as expected.But the "evalaute" has some bugs.I will fix this problem when I am not busy.

Sorry for the delay. I made some updates on the notebook and will proceed to merge this PR.
Regarding the bug on evaluate, what I encountered is that the accuracy number was off for the loaded model (0.09 vs. 0.98). The root cause turns out to be that metrics=['acc'] is interpreted differently. (Keras has some internal heuristic to distinguish what "accuracy" means to a model.) What we expect is tf.keras.metrics.SparseCategoricalAccuracy, which is the case for the adv_model. But the loaded model somehow got tf.keras.metrics.CategoricalAccuracy, so the evaluation result became different.  To workaround this issue, I changed metrics=['acc'] to metrics=[tf.keras.metrics.SparseCategoricalAccuracy()], which then produced consistent result before and after saving/loading.


Q: Does this support assigning PRs/Issues to people?

I might be missing something — this implementation looks like it only handles labeling and comments. Does it also respect a codeowners doc and handle assignments?
I found this https://github.com/technote-space/github-action-pr-helper which looks similar, and includes an assign-author action too

It respects the CODEOWNERS automatic functionality. So if you have a codeowners it will detect that there are reviewers assigned and add a label. While there are reviews outstanding it will also not automatically merge. I'll check the doco to make sure this is clear
Pretty sure that assign-author action is just assigning the author of the Issue or PR to the assignees (which is best practice). That is different to a reviewer based on codeowners or some other type of file ownership.
I do have on the roadmap to add functionality to include the ability to automatically assign reviewers based on frontmatter. So that if your code base is mostly markdown for example this could act as an alternatively or in addition to using the CODEOWNERS.

@Matticusau can the bot parse GH issues' content? A key:value at the top of an issue for url: ftakb.azurewebsites.net/docs/etc could provide the doc a place to look for front matter
AWESOME stuff on the front matter scraping/parsing

@wbsmolen I'm actively looking into the best way to implement something like that.
Some of the similar systems seem to add front-matter to the body of the issue through the "feedback" mechanism that directs the user to the new issue page from the Jekyll blog site. That front-matter includes the owner/assignee and other context for automation.
An alternative approach is certainly as you suggest to simply place the related URL/File path in the issue front-matter which then the bot/action could read, find the referenced file, read that front-matter and then assign the document owner.
From a Jekyll perspective which do you think is better to pass in a query param, the URL or Git File Path?

For the Issue assignee automation am branching this out into a different Action to ensure this one stays focused on the PR side of things.
https://github.com/Matticusau/issue-helper-jekyll


Spyder kernel doesn't launch

Issue Report Checklist

[X] Searched the issues page for similar reports
[ ] Read the relevant sections of the Spyder Troubleshooting Guide and followed its advice
[X] Reproduced the issue after updating with conda update spyder (or pip, if not using Anaconda)
[ ] Could not reproduce inside jupyter qtconsole (if console-related)
[ X] Tried basic troubleshooting (if a bug/error)

[X] Restarted Spyder
[ X] Reset preferences with spyder --reset
[ X] Reinstalled the latest version of Anaconda
[ X] Tried the other applicable steps from the Troubleshooting Guide

[ X] Completed the Problem Description, Steps to Reproduce and Version sections below

Problem Description
Kernel refuses to launch
What steps reproduce the problem?

Launch Kernel

What is the expected output? What do you see instead?
If I restart the kernel it send this message
"Cannot restart a kernel not started by Spyder"

An error ocurred while starting the kernel
The error is:

Traceback (most recent call last):
File "C:\Users\Eduardo Nagan\anaconda3\envs\gym\lib\site‑packages\spyder\plugins\ipythonconsole\plugin.py", line 1209, in create_kernel_manager_and_kernel_client
kernel_manager.start_kernel(stderr=stderr_handle, **kwargs)
File "C:\Users\Eduardo Nagan\anaconda3\envs\gym\lib\site‑packages\jupyter_client\manager.py", line 267, in start_kernel
self.kernel = self._launch_kernel(kernel_cmd, env=env, **kw)
File "C:\Users\Eduardo Nagan\anaconda3\envs\gym\lib\site‑packages\jupyter_client\manager.py", line 211, in _launch_kernel
return launch_kernel(kernel_cmd, **kw)
File "C:\Users\Eduardo Nagan\anaconda3\envs\gym\lib\site‑packages\jupyter_client\launcher.py", line 135, in launch_kernel
proc = Popen(cmd, **kwargs)
File "C:\Users\Eduardo Nagan\anaconda3\envs\gym\lib\subprocess.py", line 854, in __init__
self._execute_child(args, executable, preexec_fn, close_fds,
File "C:\Users\Eduardo Nagan\anaconda3\envs\gym\lib\subprocess.py", line 1307, in _execute_child
hp, ht, pid, tid = _winapi.CreateProcess(executable, args,
FileNotFoundError: [WinError 2] O sistema não pode encontrar o arquivo especificado

See #11819


EIP is not associated to EC2::Instance: no internet connection from Manager

Hello there,
first I want to thanks all the people that has contributed to this project. I think it a very good starting point to learn how to deploy gitlab runner cluster within AWS.
However I'm stacked with a problem related to internet connection from within the Manager EC2 Instance. When executing the UserData, there is not Internet Connection and the commands fail with connection refused. I've noticed that the Elastic IP defined in the stack is not automatically associated to the EC2::Instance resource defined, even if the correct InstanceId property is specified. Indeed, if I attach an elastic IP and execute the UserData commands by hand the stack is correctly created and a new Elastic IP is associated to the EC2::Instance.
I've tried to use EIPAssociation (without success) and also to define a NetworkInterface to be attached to the EC2 instance: same error result. My question is: there are specific requirements for the definition of the VPC that can block this association? I've defined a VPC with active InternetGateway: is there any other property/feature that needs to be set in the VPC?
Thanks for the attention to this matter.

Solved using AWS VPC quickstart. Link for guide here.


Biber doesn't sort by entrytype

Consider
\documentclass{article}

\usepackage[backend=biber]{biblatex}
\addbibresource{biblatex-examples.bib}
\DeclareSortingTemplate{entrytype}{
  \sort{\field{entrytype}}
}

\begin{document}
\nocite{cicero,sigfridsson}

\begin{refcontext}[sorting=entrytype]
\printbibliography
\end{refcontext}
\end{document}

this sorts cicero > sigfridsson even though sorting by entrytype should give sigfridsson (@article) > cicero (@book)
The .bbl contains
[651] Internals.pm:1102> DEBUG - Sorting object for key 'cicero' -> ["entrytype"]
[651] Internals.pm:1102> DEBUG - Sorting object for key 'sigfridsson' -> ["entrytype"]

but of course we expected the values book and article.
This came up in https://tex.stackexchange.com/a/346895/35864

I suppose 2.10 and biblatex 3.10 have already been sent off to CTAN? So this came in just a few hours to late....

No not yet as I’m waiting for the biber builders to commit so I should be able to fix this

Fixed in v2.10 - this will make to the CTAN release.

Works brilliantly with the version from sourceforge. Thank you very much.


Arcgis connector check is flipped at JS import modal

Both the backend and user_data have correct value for arcgis_datasource, but the import modal shows the opposite behaviour: If datasource active appears as without arcgis, if datasource disabled appears as with arcgis.

Just subscribing me to the issue :smile:
/cc @jaimecartodb


Fixes the master branch failing tests

Title!
the only changes are that tests now pass proper
contract debug spawning now only kicks in if you enable the debug.citadel.contractspawning cvar (otherwise the unit test that deletes everything ends up with contracts spawning themselves in right after)
bank commands are now locked behind admin debug perms
toolshed commands now have descriptions that may or may not be accurate
contracts and their subcomponents are no longer able to be saved in maps

self-mergin on request of other maints


[Feature] Add Merge layers (Add, Subtract, Multiply, Concatenate, Maximum, Minimum)

All these layers were implemented and merged. I close this issue


Flip Tile To-Do

[ ] Add a reset button
[ ] Add a skip to next puzzle button
[ ] Add support for even-numbered square puzzles (need to change how position of tiles are calced)
[ ] Add a difficulty value that can increase or decrease at the press of buttons 2x2 matrix, 3x3, 4x4, 5x5
[ ] Add a reveal solution button which resets the puzzle and then shows which ones you need to press to solve it
[ ] Add pointer on hover of tiles and make them grow in size when hovered as an anim
[ ] Add onClick anim for tiles too (maybe they spin or have a cool color thing happen)

[ ] Add a score system
[ ] Add leaderboards? This may need to be turned into a longer term ticket since that may make the website no longer static
[ ] Add a cool background that is animated and moves?

[ ] Fix up the while loop such that it has error handling and tries a new set of tiles if the current one is not solvable
[ ] Link the youtube video I found that talks about how to solve these in the code where I calculate how to solve these puzzles w/ lin alg

[ ] Add hover pointer / maybe enlarge for the toggle boxes

[ ] Make it show to a player the current difficulty above the puzzle so that its clear the current difficulty is not always the same as the selected difficulty if you havent clicked new puzzle yet. Look at ipad notes for things like score etc. that should be on top as well
[ ] Spread out tiles more for phone

[ ] Add a daily puzzle option that gives a random configuration that all players can see (random number whose seed is related to the date so that all players have the same one).
[ ] Have daily puzzle be like Wordle so you can send it to friends when you solve it.
[ ] Have a countdown timer in the top right saying “next daily puzzle in HH:MM” and a button you can click to show the current daily puzzle
[ ] Need to find a way to alter the share button / text like in Wordle where it shows the pattern when you send it. Maybe it Shows the pattern of the puzzle and how many taps it took you to get it?
[ ] Stretch goal: Need some way of it knowing if you’ve already played the daily puzzle so you can't re-do it on accident

[ ] Buttons stay stuck selected after tapping them on phone (like the animation stays enlarged). Fix this


Issues with ticket registration form

Bugs-

Ticket buyer information is not asked for one ticket
Ticket holder information should not editable

Enhancement -

Provide option to input same details in holder info as the buyer info

Would you like to work on the issue?

[x] Yes

Need more work as discussed in the meeting.


[Dribbblish] sidebar icons not working

sidebar icons are not working. I have done all the things currently but still does not work

yeah same thing has happened to me

fixed with the new spicetify update


LTS 3006.9

This PR won't be merge to master.
It will be used to validate tests and then to create a new release from this branch.
This image contains LTS 3006.9 salt-master. More info at: https://docs.saltproject.io/en/3006/topics/releases/3006.9.html

Tests have passed and a new release has been created.


OpenBLAS-0.3.7, Compile problems with PGI 19.4

CPU: AMD EPYC 7601
OS: CentOS 7.6
Compiler：PGI 19.4
Problem 1
[root@n212 OpenBLAS-0.3.7]# make CC=pgcc FC=pgf90
pgcc-Error-Unknown switch: -march=native
make: *** [getarch] Error 1
Makefile.system:216: Makefile.conf: No such file or directory
make: *** No rule to make target `Makefile.conf'.  Stop.
[root@n212 OpenBLAS-0.3.7]#
Problem 2
First, modify Makefile.system 148 line:
146 ifeq ($(ARCH), x86_64)
147 ifneq ($(C_COMPILER), PGI)
148 #GETARCH_FLAGS += -march=native
149 endif
150 endif
And,
[root@n212 OpenBLAS-0.3.7]# make CC=pgcc FC=pgf90 1>make.log 2>make.err
[root@n212 OpenBLAS-0.3.7]# tail -n 3 make.log
pgcc -c -O2 -DMAX_STACK_ALLOC=2048 -tp p7-64 -DF_INTERFACE_PGI -fPIC -DNO_AVX512 -DSMP_SERVER -DNO_WARMUP -DMAX_CPU_NUMBER=32 -DMAX_PARALLEL_NUMBER=1 -DVERSION="0.3.7" -DASMNAME=dtrmm_outucopy -DASMFNAME=dtrmm_outucopy_ -DNAME=dtrmm_outucopy_ -DCNAME=dtrmm_outucopy -DCHAR_NAME="dtrmm_outucopy_" -DCHAR_CNAME="dtrmm_outucopy" -DNO_AFFINITY -I..  -DDOUBLE  -UCOMPLEX  -DDOUBLE -UCOMPLEX -DOUTER -ULOWER -UUNIT generic/trmm_utcopy_8.c -o dtrmm_outncopy.o
make[1]: Leaving directory `/root/src/OpenBLAS-0.3.7/kernel'
[root@n212 OpenBLAS-0.3.7]#
[root@n212 OpenBLAS-0.3.7]# cat -n make.err
...
PREPRO-I-0222-Redundant definition for symbol extension (../kernel/x86_64/dgemm_kernel_4x8_haswell.S: -1)
PREPRO-I-0222-Redundant definition for symbol extension (../kernel/x86_64/gemm_beta.S: -1)
:2:13: error: invalid operand for instruction
addq    $128, %dl
^~~
:5:15: error: invalid base+index expression
vmulpd  -128(%dl), %xmm0, %xmm4
^
:6:15: error: invalid base+index expression
vmulpd  -112(%dl), %xmm0, %xmm5
^
:7:15: error: invalid base+index expression
vmulpd   -96(%dl), %xmm0, %xmm6
^
:8:15: error: invalid base+index expression
vmulpd   -80(%dl), %xmm0, %xmm7
^
:9:15: error: invalid base+index expression
vmulpd   -64(%dl), %xmm0, %xmm8
^
...
[root@n212 OpenBLAS-0.3.7]#

Ah, sorry - I see now that the line 147 ifneq ($(C_COMPILER), PGI) can not work (C_COMPILER is only set by c_check, which is run at the same - later - time as getarch). Will probably need to check for HOSTCC matching "pgcc" instead.
Could you please check if the assembler errors go away when you change the "-tp p7-64" in Makefile.system around line 700 to "-tp haswell" ?

Actually it is probably best to simply change these -tp options to the same simple -m64 (or -m32 as appropriate) as used on most other compilers, leaving the PGI compiler to use its own defaults for the cpu-specific options. At least the "nollvm" version of PGI 19.4 certainly understands the -m64, the previous options must have been necessary for some much earlier version, ten or more years ago.
Unfortunately I cannot get rid of the assembler errors when I use the default llvm-based compiler,
looks like it has serious problems with inline assembly. When I add -Mnollvm to the compile options, it builds correctly but cannot run the openblas_utest binary due to a segmentation fault
in the startup code of libpgpmp.so. When I point CC and FC at the binaries in the linux-x86_64-nollvm/bin/ branch directly, everything works as normal.

Ah, sorry - I see now that the line 147 ifneq ($(C_COMPILER), PGI) can not work (C_COMPILER is only set by c_check, which is run at the same - later - time as getarch). Will probably need to check for HOSTCC matching "pgcc" instead.
Could you please check if the assembler errors go away when you change the "-tp p7-64" in Makefile.system around line 700 to "-tp haswell" ?

I set the env for pgi compiler and used the directory /opt/pgi/linux86-64-nollvm (not /opt/pgi/linux86-64-llvm). And, modify  the content of Makefile.system to "-tp p7-64" or "-tp haswell" or "-tp zen", all compiled successfully.

Thanks. I will leave the "-tp p7-64" unchanged then, and just correct the wrong "-march=native".


update renovate config

lets try again…
Testing

Trade-offs
dependencies that are behind in major are pinned to that major by only enabling patch and minor updates
same for all devDependencies

Dependencies

Screenshots

:tada: This PR is included in version 2.66.3 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


test: Conduct scale tests

Regular scale testing is important, but is currently performed ad hoc. We should automate the following scenarios:

Primary mount only

Deploy 1000 pods; measure attach latency and overall time to completion
Compare with v1 driver

Replica mounts

Deploy 1000 pods; measure attach latency and overall time to completion

Failover

Deploy 1000 pods
Failover concurrently and measure latency
Compare with v1 driver

It may be helpful to leverage the existing pod failover tester.


C# backend - generate ref params even for *mut _

Currently *mut pointers generate out parameters on the C# backend.
This works for patterns where the C# side is expected to only write to the pointer but breaks down when you want to read from the pointer first.
This PR changes *mut to generate ref arguments like *const.
This is a breaking change for existing code so perhaps there is a better solution?

Thanks for the PR.
This basically boils down to how to translate pointer attributes that don't exist in one language to another. Variants of this came up before (e.g., #68, #71) and I don't think there is a simple solution.
I think the best approach so far is described in #71, and that's also my preferred solution for C# ref and out.

Understood, thanks @ralfbiedert


PDFs intermittently working in Chrome when using addHTML()

Hi there,
Thanks first of all for providing such a fantastic library!  Very impressive stuff :)
One issue that we've been facing with generating PDFs and then displaying them either in an iframe, a new tab or window via datauri, or opening directly via download link with Chrome's PDF viewer is that some PDFs that we generate using addHTML() don't work.  It's like Chrome sees them as invalid PDFs.  The same PDF opens up with no problem in a system viewer, or in Firefox's PDF viewer for instance.
About 50% of the PDFs behave this way in Chrome and if one fails to open it always fails to open so it is at least consistent.
Just wondering if anyone else has noticed this behaviour in Chrome?
My exact version is 43.0.2357.65 on Ubuntu 14.04.  Unfortunately due to the sensitive nature of the PDF content I'm unable to attach a datauri to this ticket.
Many thanks,
Duncan

Hmm that is strange. Can you post example code for a working and
non-working PDF?
On Thu, May 28, 2015 at 16:50 duncan-at-hiveit<EMAIL_ADDRESS>wrote:

Hi there,
Thanks first of all for providing such a fantastic library! Very
impressive stuff :)
One issue that we've been facing with generating PDFs and then displaying
them either in an iframe, a new tab or window via datauri, or opening
directly via download link with Chrome's PDF viewer is that some PDFs that
we generate using addHTML() don't work. It's like Chrome sees them as
invalid PDFs. The same PDF opens up with no problem in a system viewer, or
in Firefox's PDF viewer for instance.
About 50% of the PDFs behave this way in Chrome and if one fails to open
it always fails to open so it is at least consistent.
Just wondering if anyone else has noticed this behaviour in Chrome?
My exact version is 43.0.2357.65 on Ubuntu 14.04. Unfortunately due to the
sensitive nature of the PDF content I'm unable to attach a datauri to this
ticket.
Many thanks,
Duncan
—
Reply to this email directly or view it on GitHub
https://github.com/MrRio/jsPDF/issues/513.

Hi,
Sure, no problem!  It'll take me a little while as our code wraps jspdf quite a lot so it'll need a bit of time to unwrap it, but I'll provide it asap.
Thanks for the reply,
Duncan


seemingly no way to take events from ZwpInputMethodContextV1?

I'm new to using smithay, and I've been trying to implement an IME. From what I can tell, you have to register a global ZwpInputMethodV1, listen for the Activate { id: ZwpInputMethodContextV1 } event, and then additionally listen for events on that id, However, ZwpInputMethodContextV1 doesn't seem to provide any methods that take a QueueHandle. Am I missing something?

Hello, sadly I haven't ever worked with "Input method v1" and due to its limited support I don't have a way to test it out. However from a quick glance, you are correct that you have to register ZwpInputMethodV1 global and listen to the Activate event, however I'm not exactly sure what you mean by "listen for events on that id".
In any case, my assumption is that when you register ZwpInputMethodV1 and it emits activate event with ZwpInputMethodContextV1. ZwpInputMethodContextV1 will also automatically start emitting events until it is deactivated. However again I don't have a way to test it out so I might be wrong.
You can test it out by just listening if ZwpInputMethodContextV1 is emitting any events with code similar to this.
impl Dispatch<ZwpInputMethodContextV1, ()> for Delegate {
    fn event(
        _: &mut Self,
        _: &WlRegistry,
        event: wl_registry::Event,
        _: &GlobalListContents,
        _: &Connection,
        _: &QueueHandle<Self>,
    ) {
    println!("{:?}", event);
    }
}

It is also worth mentioning that there is Input method v2 which appears to be more widely supported.

Objects created by an event (like ZwpInputMethodContextV1) are automatically registered to the same event queue as their parent object, so indeed you just need to have the appropriate Dispatch<_> implementation.
One part that is a little clunky is the initialization of the user-data associated with these objects, which is currently done through the event_created_child! macro.

how does a trait implementation even affect behaviour? the current code doesn't complain about Dispatch<ZwpInputMethodContextV1> missing, so I don't think implementing it would change much?


Adding a difficulty tag.

Expected Behavior
The rebuses should get progressively tougher as one keeps solving them (that is what a gamer would expect).
Actual Behavior
The difficulty of successive rebuses is quite random.
Steps to Reproduce the Problem

Add a new parameter "difficulty" as a number (maybe from 1-5) in each object of the rebuses.js array.
Sort the objects according to their difficulty and display them to the user from low to high difficulty.
Add a difficulty logo to each rebus card (maybe on top-right).

Specifications

Browser: Should work for all browsers.

Hey @anuva04, i would like to work on that issue. Please assign it to me. Thanks.


Implement channel upgrade relaying

Summary
A new channel upgrade specs has been added with two new channel states: UPGRADE_INIT and UPGRADE_TRY.
This needs to implemented in order to support upgrading channels to enable fees once ICS29 is released.

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate milestone (priority) applied
[ ] Appropriate contributors tagged
[ ] Contributor assigned/self-assigned

Done by https://github.com/informalsystems/hermes/pull/3228


[Bloc] Tasks should wake up their Space using Waker instead of polling

Now spaces have to loop indefinitely while the task queue is not empty. If a task executes code in the background thread we should wake up the space using a Waker.

Tasks are no longer polling-only.


Add support for Google searches using Google.Apis.Customsearch.v1

Motivation and Context
Allow the usage of Google search equivalent to current Bing search implementation.
Description
Uses the Google.Apis.Customsearch.v1 packages to interface with the Google search API, this requires an API key from GCP and a custom search engine ID
Contribution Checklist

[x] The code builds clean without any errors or warnings
[x] The PR follows SK Contribution Guidelines (https://github.com/microsoft/semantic-kernel/blob/main/CONTRIBUTING.md)
[x] The code follows the .NET coding conventions (https://learn.microsoft.com/dotnet/csharp/fundamentals/coding-style/coding-conventions) verified with dotnet format
[x] All unit tests pass, and I have added new tests where possible
[x] I didn't break anyone :smile:

@microsoft-github-policy-service agree

@ScottKane please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
@microsoft-github-policy-service agree [company="{your company}"]

Options:

(default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.

@microsoft-github-policy-service agree

(when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.

@microsoft-github-policy-service agree company="Microsoft"

Contributor License Agreement

@microsoft-github-policy-service agree

@microsoft-github-policy-service agree

A few thoughts:

the this. keywords can be safely removed everywhere, since they do not contribute anything.
the exact version of the Google.Apis.Customsearch.v1 package should not be hardcoded, such that bugfixes of the library will not be missed in future builds. Only fix the MAJOR.MINOR part of the version.
consider setting <ImplicitUsings>true</ImplicitUsings> in order to clean up the using section

The original code written was without this. however StyleCop was enforcing the use of them so I assumed this was required. Happy to make any changes required as soon as we can iron out the listed questions surrounding the interface and naming conventions. The BingConnector also takes a string apiKey so should that change to an Option type too?

The original code written was without this. however StyleCop was enforcing the use of them so I assumed this was required. Happy to make any changes required as soon as we can iron out the listed questions surrounding the interface and naming conventions. The BingConnector also takes a string apiKey so should that change to an Option type too?

Yes, current repo style guides are to use "this.". You should also run "dotnet format" in the "/dotnet" directory to gets most of the style adjustments made for free. This will help the "dotnet-format" gate pass.

Ok I ran dotnet format as requested, I think ommitting Search from the Bing/Google connectors is a better descision, they both implement IWebSearchEngineConnector which can tell you're working with a search engine while leaving the posibility for the same connectors to implement different interfaces in the future, e.g both could implement an IWebMapsConnector or something, at which point having Search in the name might not be so sensible.

@ScottKane thanks for bringing this in! Unfortunately you unchecked the "allow maintainers to edit the PR" and we could not address some important changes, e.g. the dependency on an old nuget. I tried working around the block but between this and several conflicts it was taking too long, so I recreated your PR here https://github.com/microsoft/semantic-kernel/pull/737 and added you as one of the contributors in the commit log.

@dluc no problem, sorry I will look out for that next time I raise a PR


Add app helper nodes actions screen to datalayer wear sample

WHAT
Add app helper nodes actions screen to datalayer wear sample.

WHY
Show case app helper features.
Checklist :clipboard:

[N/A] Add explicit visibility modifier and explicit return types for public declarations
[x] Run spotless check
[x] Run tests
[N/A] Update metalava's signature text files

In Node actions, why the big gap to the shrunken chip?

why the big gap to the shrunken chip?

I added the gap to easily see the difference between the items of the list of devices and a button that is not an item of that list. Maybe a header would do, but it seemed too much just for a single chip.
I'm not UI expert so happy to hear suggestions.

why the big gap to the shrunken chip?

I see there is no padding defined there, so not sure why it has a bigger gap, to be checked.

Same with ChipCompact from wear-compose directly.
It might be related to the minimum tappable size, but I'm speculating now.


Support string values

Hi, I've got a few battery powered devices I monitor, where the unit is: normal or low
When using your battery-entity-row, what unit should I be using for these devices?
Defaulting to % is great, in most cases, for these devices, it just shows 0%, when in-fact the state is actually normal. I've tried "unit:  " & "unit: string" but not getting the result I'd expect. Are their more options for unit than "%"?
cheers

Try unit: false. However, this card currently only supports numeric battery values, not strings like normal and low.

Understood. I tried it, after upgrading, no luck. Any chance of making this a feature request?

Make sure you clear cache/hard refresh your HA tab.. Hiding the unit works fine here.
I'll try to look into supporting string values. Let me know what the different values are. high, medium, normal, low, empty, etc..?

I've tried clearing the cache/hard refresh, and the binary sensor shows "Unknown" when the state of the sensor is actually "Low". I've got several binary_sensors, the 2 states I've seen are "Normal" or "Low".

Version 1.2.0 should now support string values like normal and low (and some others).


Phase2 QA and closed picture issues

@NewGraphEnvironment before you merge this just one question. I hand picked extra photos from the repo and labeled them with a _k_ and copied them over to the submission portal, to fill out the 5 required photos that I didn't already submit to pscis. Do you want me committing these extra photos to github? I didn't do this in the PR below. I believe some of them got into the photo_metadata object when I was fixing issue #116. So some are showing up on the interactive map. Is it a big deal if we have a few extra photos on the map?

ya - tons of photos really clutters the maps and makes them crappy to use. I stopped adding tonnes back in the day for that reason.  I think the thing to do is for us to file and issue and remedy it by adding an additional _nm_ (no map) tag.  then we can filter them out from the photo_metadata object but they will still be committed.
It is great to use the scripts to move them to the pscis submission so we can just leave the _k_ there and add nm_.  Way to do it would be:

type git status >> data/inputs_extracted/photos_to_rename.csv into the terminal to print the names of the untracked files to a csv.
import thephotos_to_rename.csv, filter out everthing that is not a photo and mutate a new column to sub out _k_ for _k_nm_.  Then rename the photos with a wee script like the pscis transfer.  If after an hour or so you get stuck you can always rename by hand but at least you will have a list of which ones need to be dealt with
filter out photos with _nm_ from the photo_metadata object


Confirm email settings

For Feature Requests
Desired Feature: Page to confirm system settings, i.e. if mail is configured correctly
When going to the "Forgot Password" for example, I see "Whoops, looks like something went wrong.". I imagine my mail config is incorrect.

This is covered in a previous feature request:
https://github.com/BookStackApp/BookStack/issues/408
Might be worth commenting there to show support!


THRIFT-4591:cannot read call result via lua lib

THRIFT-4591: [cannot read call result via lua lib]
Client: [lua,  lib/lua]

What is the underlying transport ?
It looks like an issue of underlying transport implementation where data written by multiple write call cannot properly be read by receiver side.

@jeking3 the server is writen with c++ library, i donot know if the C++ library should be supported separated reading function, write "ab" is faster than write "a" and then write "b"; otherwise this correction can fix the problem, after all it may not match the root cause.

And the attachment upload to jira show the correction work fine!!!

I believe the error is in the implementation of TFramedTransport in the C++ library.  Specifically, TFramedTransport readSlow() attempts to read a size off the underlying transport and then attempts to read as much as possible from the underlying transport.  If it doesn't read the entire message indicated by the size, it will return a partial read.  The upper layer will then call readSlow again, which will try to read another size off the wire, destroying message integrity.  It's amazing to see such a bug in code that's been around for a long time.  Funny thing is I have been adding support for message bus transports based on TFramedTransport being correct, and obviously it is not.
So the lua library is just fine writing size and then writing payload.  I think that's still better than copying the size and payload into another buffer (increasing memory requirements to write by 2x) and then writing all at once.
So you correctly identified there was a problem but the issue is on the C++ side, not on the lua side.  If another client did the same thing, the C++ server would fail the same way.
That said, I'd like to take over this issue in Jira and I will fix TFramedTransport, hopefully this week.  My current employer is providing me with the time to work on message bus support for thrift, and I need TFramedTransport to work properly.  In fact, I may add a test that specifically targets this issue to see if it is present on any other library implementation.
I'm closing this PR as this change is not necessary.  Thanks.

python library does the same thing as this change so I am going to merge it.  It previously passed all CI tests.

"THRIFT_EAGAIN (unavailable resources)" means your process ran out of resources; you may want to look at your process limits, or see if you are sending a message too large (but not larger than the maximum frame size).


Better exception handling/logging

There's a few things going on in this commit.
The first was to refactor the fetchers to use a connection() method to make everything a bit easier to test.
The other is to specify the exception caught, and to log the error and backtrace to either the provided logger, Rails.logger (if it's there), or a stdlib Logger object.

I'd like to 2nd this request.  I'm currently stuck in a loop somewhere, and having a heck of a time finding out where.  There is no email to be processed to my knowledge, but fetcher seems to be pulling emails previously processed....  I can't find them either....


Update README.md

Update readme to align with the new website (https://neuroinformatics.dev/projects/swc-bids.html) and mention the ongoing proposal to extend BIDS to non-human research.
I started basically copying verbatim from website with a few extra additions, but maybe this would read strangely if you navigate straight from the website, so changed it up a little.

I started basically copying verbatim from website with a few extra additions, but maybe this would read strangely if you navigate straight from the website, so changed it up a little.

In general I don't see any reason why we can't have the same info in >1 place.
Unless I'm missing something, this PR changes the readme to better match the NIU website, but then differs from the SWC-BIDS site. I would favour the information on the NIU site being very basic (and hardly ever needing to be updated) and then the repo readme and project website (ideally the same content) be more in-depth.

I started basically copying verbatim from website with a few extra additions, but maybe this would read strangely if you navigate straight from the website, so changed it up a little.

In general I don't see any reason why we can't have the same info in >1 place.
Unless I'm missing something, this PR changes the readme to better match the NIU website, but then differs from the SWC-BIDS site. I would favour the information on the NIU site being very basic (and hardly ever needing to be updated) and then the repo readme and project website (ideally the same content) be more in-depth.

If I understand correctly, I believe the landing page of the swc-bids website now links to the GitHub readme (so it should change accordingly)

If I understand correctly, I believe the landing page of the swc-bids website links to the GitHub readme (so it should change accordingly)

Oh perfect. In that case LGTM.


Resources for Tasks with many images in the same registry are repetitive

Expected Behavior
If I'm creating a Task that needs to build a lot of images, that means I need to provide many image resources. If these all need to get pushed to the same image registry, it would be great if I only needed to provide this information once.
Actual Behavior
While working on #528 I started to create a Task that can publish all the images required for this project. There are a lot of them tho and so what I've currently got is something like this:
---
apiVersion: tekton.dev/v1alpha1
kind: PipelineResource
metadata:
  name: entrypoint-image
spec:
  type: image
  params:
   - name: url
     value: gcr.io/christiewilson-catfactory/entrypoint # Replace this with your own image registry
---
apiVersion: tekton.dev/v1alpha1
kind: PipelineResource
metadata:
  name: kubeconfigwriter-image
spec:
  type: image
  params:
   - name: url
     value: gcr.io/christiewilson-catfactory/kubeconfigwriter # Replace this with your own image registry
---
apiVersion: tekton.dev/v1alpha1
kind: PipelineResource
metadata:
  name: creds-init-image
spec:
  type: image
  params:
   - name: url
     value: gcr.io/christiewilson-catfactory/creds-init # Replace this with your own image registry
---
apiVersion: tekton.dev/v1alpha1
kind: PipelineResource
metadata:
  name: git-init-image
spec:
  type: image
  params:
   - name: url
     value: gcr.io/christiewilson-catfactory/git-init # Replace this with your own image registry
---
apiVersion: tekton.dev/v1alpha1
kind: PipelineResource
metadata:
  name: nop-image
spec:
  type: image
  params:
   - name: url
     value: gcr.io/christiewilson-catfactory/nop # Replace this with your own image registry
---
apiVersion: tekton.dev/v1alpha1
kind: PipelineResource
metadata:
  name: bash-image
spec:
  type: image
  params:
   - name: url
     value: gcr.io/christiewilson-catfactory/bash # Replace this with your own image registry
---
apiVersion: tekton.dev/v1alpha1
kind: PipelineResource
metadata:
  name: gsutil-image
spec:
  type: image
  params:
   - name: url
     value: gcr.io/christiewilson-catfactory/gsutil # Replace this with your own image registry
---
apiVersion: tekton.dev/v1alpha1
kind: PipelineResource
metadata:
  name: controller-image
spec:
  type: image
  params:
   - name: url
     value: gcr.io/christiewilson-catfactory/controller # Replace this with your own image registry
---
apiVersion: tekton.dev/v1alpha1
kind: PipelineResource
metadata:
  name: webhook-image
spec:
  type: image
  params:
   - name: url
     value: gcr.io/christiewilson-catfactory/webhook # Replace this with your own image registry

If someone wants to use the Task that consumes these with their own registry, they're going to have to change each of those url values.
Additional Info
Note that these images are built using ko, which doesn't take as input information about individual images, it just takes the KO_DOCKER_REPO

Would need to be overridable per-image, I'd assume?

I think the future of the image resource is going to look very different from what we have today (#1673) - and with Tasks able to emit results (e.g. in this example https://github.com/tektoncd/pipeline/blob/master/docs/migrating-v1alpha1-to-v1beta1.md#replacing-pipelineresources-with-tasks) it seems like image resources really arent that useful. So I'm going to close this for now!


Add extra step for token verification in email authentication

This is more of a feature request than an issue if we don't mind the possibility of automatic signins, i would like to use a recaptcha to prevent this.
One way could be instead of checking the token right away after user clicks the link in the email, it could be forwarded to a page where a recaptcha could verify its not a bot and then perform the token checking via ajax, i assume the recaptcha validation is only needed for the email authentication not for the identity providers, this way only the email authentication flow gets validated by recaptcha.
This page could also set the token in localstorage triggering the 'storage' event letting the other open tab know the user is now logged and sync.
(Sorry if last part is unrelated, i think this package is amazing, thanks btw)

Hi there,
These are good request and there is certainly need for better email token handling (specifically, I've been meaning to add an expiry timestamp).
Perhaps the best place for captcha with next-auth is possibly on signing up via email, to prevent account creation by bots (before an account and token is even generated).
A good place to put this might be in the nextjs-starter project - I will look at adding reCAPTCHA to it (and also write it up on the wiki here). If I can keep it simple and small I'l also include it directly next-auth example too. I'd be very happy to include helper functionality in next-auth and next-auth-client if it makes it easier and makes sense to have there though, as I agree it's something to encourage people to implement.
Having the next-auth-client trigger updating of other tabs is definitely a thing I'd like to have but requires some thought and I'm not sure if there is a really good way to do it that would work for everyone yet.
Currently session state is reflected next time you load a page in an old tab (because as soon as you load a page, it sees the new cached session state in localStorage and sees that the user is now logged in), but it would be nice to be able to update pages immediately.
The problem with doing that is - even if we used a store instead of just props to access session state, so could update the state of them dynamically  - pages render differently when you are logged in and not logged in. Often it's small things in the UI (like navbar contents) but sometimes it's more complex behaviour, like rendering different form options when a user is logged in. I wouldn't want to force re-rendering of the current page as data might be lost (e.g. the contents of form fields, or unsaved app state).
Perhaps a good approach is to fire an event people can bind to, or even to reload pages by default when signed in/signed out in another tab, but let people pass an option to specific pages to disable the automatic reloading behaviour and/or handle it manually.

I agree, the captcha should be placed before account and token creation, i was thinking how zeit site might work.
I think i could place the captcha in singin page or load it after email submit click, an example of how to implement it would be cool, thanks.
I was looking for options of how to make tabs update but as you say is hard to find one with full browser support, i think the best way would be done all in the browser (no websockets) and even though im not very familiar with web workers it seems like the way to go. In the other hand, the most supported option i found would be using localStorage and the "storage" event, using local-storage package that brings better support as stated in this blog post.

Thanks for the feedback and suggestions. Am going to leave this one open and work on it.
I think using the storage API is a really great way to listen for / fire an an event across tabs and seems like the best way to make it easier to do live updating when session states change.
Maybe a good way to start would be if I made it easy to listen to session changes in pages by adding an easy hook in the next-auth/client to handle either the opt-in or opt-out automatic page reload behaviour.
A future release, maybe something to save for 2.0, could refactor to use a store for session changes, to make it easier to handle gracefully in more situations.

While it doesn't exactly address this issue, for anyone reading who is interested in this as of 1.8.0 you can now define a custom signIn() function - and optionally disable the built in email token system - and write your own function to handle sign in form submission as a way of doing this.
Like the other sign in methods it works with and without JavaScript in the client, and you can handle processing any request you like. An example signIn() function and example page showing how to use it is provided.

The new features in 2.0, which include custom signin() callbacks, Credentials Provider and fully customisable emails, custom sign in pages address the issue raised in this ticket.
Additionally, Users accounts are no longer created until after an email sign in email has been activated, and tokens are stored more securely in the database, to prevent abuse.


Fixup free_memory to deal with garbage collection

What does this PR do?
Tweaks Accelerator.free_memory to take in *objects and return *objects. This is because we need to fully replace the values, doing just del doesn't actually help as much as it should.
Solves https://github.com/huggingface/accelerate/issues/2715, graph:
Solves https://github.com/huggingface/transformers/issues/28178

(Weird read, I know, I think it's because they delay the reading of the computer for a few seconds). But I think because the graphs are identical that's good
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you read the contributor guideline,
Pull Request section?
[ ] Was this discussed/approved via a Github issue or the forum? Please add a link
to it if that's the case.
[ ] Did you make sure to update the documentation with your changes? Here are the
documentation guidelines, and
here are tips on formatting docstrings.
[ ] Did you write any new necessary tests?

Who can review?
Anyone in the community is free to review the PR once the tests have passed. Feel free to tag
members/contributors who may be interested in your PR.
@SunMarc

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

I am running into a memory leak and noticed this PR
There is also a issue with deepspeed engine's destroy() not being called which is supposed to remove torch hooks that known to leak memory:
https://github.com/huggingface/transformers/issues/28178
What would be the best way to accomodate this ^ too

@SunMarc that is indeed expected (but no, we don't care about this use-case really).
I'll be sure to include .destroy() in deepspeed as part of this, noticed this bug first when I was starting to look into it

I am running into a memory leak and noticed this PR

There is also a issue with deepspeed engine's destroy() not being called which is supposed to remove torch hooks that known to leak memory:
https://github.com/huggingface/transformers/issues/28178

What would be the best way to accomodate this ^ too

Thank you @chiragjn for the pointers, helpful!


[BUG] Website Content Security Policy blocking font injection via data URL

What happened?
I am working on a project that injects a custom overlay UI that has it's own injected font. Font injection works fine on most of it's target websites, but there's one that has a content security policy that prevents loading data: urls, and this is blocking the font injection on that website. I know fonts can be imported via url in regular browser extensions so i know a workaround/solution is possible. I'd be grateful if someone who has encountered a similar issue can describe how it was worked around.
Thanks.
Version
Latest
What OS are you seeing the problem on?
Windows, MacOSX, Linux
What browsers are you seeing the problem on?
Chrome, Microsoft Edge, Firefox
Relevant log output
No response
(OPTIONAL) Contribution

[ ] I would like to fix this BUG via a PR

Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I checked the current issues for duplicate problems.

Found a workaround. Solved my problem, but I don't believe it's the best case solution.
What I did:
Install Styled Components via npm install styled-components
Create a global style component like so:
const GlobalStyle = createGlobalStyle`   @import url('https://fonts.googleapis.com/css2?family=Nunito:ital,wght@0,200;0,300;0,400;0,500;0,600;0,700;0,800;0,900;0,1000;1,200;1,300;1,400;1,500;1,600;1,700;1,800;1,900;1,1000&display=swap');  /* Add any additional global styles here */ `;
Add it to your base component:
<GlobalStyle />

Found a workaround. Solved my problem, but I don't believe it's the best case solution.
What I did:
Install Styled Components via npm install styled-components
Create a global style component like so:
const GlobalStyle = createGlobalStyle`   @import url('https://fonts.googleapis.com/css2?family=Nunito:ital,wght@0,200;0,300;0,400;0,500;0,600;0,700;0,800;0,900;0,1000;1,200;1,300;1,400;1,500;1,600;1,700;1,800;1,900;1,1000&display=swap');  /* Add any additional global styles here */ `;
Add it to your base component:
<GlobalStyle />


Drop iOS 7 support

How about dropping iOS 7 support due to use of the default cell sizing mechanism provided by UITableView since iOS8?

Thanks kefejo. I will drop os7 support in the next version.


Free-text inputs do not preserve line break formatting on review page

Sample input on register page:

Review page:

@pcraig3 looks like white-space: pre-line would be a quick fix.
https://developer.mozilla.org/en-US/docs/Web/CSS/white-space

This was closed in PR 457


Add Code of Conduct.md file

The repository does not contain the code of conduct.md file.

@preronagit thank you, i appreciate it.


Add config on android to controlled setInvalidatedByBiometricEnrollment

Changes proposed to fix the issue described in the issue #114.
Adds a new optional config argument on Android used to set the property setInvalidatedByBiometricEnrollment as described on https://developer.android.com/reference/android/security/keystore/KeyGenParameterSpec.Builder.html#setInvalidatedByBiometricEnrollment(boolean)

Thank you, mate!

Hi @mCodex, I just received some feedback from one user and it looks like I missed one thing on the PR when I was migrating my local changes into the project.
Sorry about the inconvenience, I'll be pushing a new PR with the fix. I might have to change few things on the code.

@AJM88 don't worry! Thanks!!


One-line syntax specs don't show proper documentation when dry-running

Using rspec -fd --dry-run on  a project with one-liner specs like
it { is_expected.to eq true }

only show up as
   example at ./spec/path/to/spec.rb:24

When actually running the specs I get the correct output.
I know that dry-running does not execute the code, but only collects the specs and their description. I would love to have the specs also evaluate the expectation strings when dry-running one-liner. Maybe another run-mode might be appropriate to not slow down dry-running, but to get the expactation evaluation. After all, I would like to use this to get a readable and presentable documentation of all my specs including the one-liner without actually running the tests.

Thanks for the feedback, but as you point out  --dry-run doesn't execute the spec, thus it isn't possible to gather a description for the one line syntax.
This is one of the many reasons we advocate writing your spec doc strings yourself instead of relying on the auto-generated ones, we simply are often unable to guess your meaning, writing them yourself is the best way to get readable and presentable documentation :)

@JonRowe Just arrived to this issue after having been bitten by this issue.
In my opinion it's extremely interesting to take into account the comment made 5 years ago by @dchelimsky regarding this topic @ https://github.com/rspec/rspec-core/issues/354:

Self-describing matchers/one-liners, which must be executed in order to generate their documentation, render --dry-run unusable for most users, so it's not included in rspec-2.

I'd like to ask you folks (@samphippen @myronmarston also since they reviewed https://github.com/rspec/rspec-core/pull/1028): what do you think about adding a note regarding this to the --dry-run docs @ rspec-core?

I'd like to ask you folks (@samphippen @myronmarston also since they reviewed https://github.com/rspec/rspec-core/pull/1028): what do you think about adding a note regarding this to the --dry-run docs @ rspec-core?

I'm in favor.  Want to open a PR to improve the docs?


[SPARK-43286][SQL] Updates aes_encrypt CBC mode to generate random IVs

What changes were proposed in this pull request?
The current implementation of AES-CBC mode called via aes_encrypt and aes_decrypt uses a key derivation function (KDF) based on OpenSSL's EVP_BytesToKey. This is intended for generating keys based on passwords and OpenSSL's documents discourage its use: "Newer applications should use a more modern algorithm".
aes_encrypt and aes_decrypt should use the key directly in CBC mode, as it does for both GCM and ECB mode. The output should then be the initialization vector (IV) prepended to the ciphertext – as is done with GCM mode:
[16-byte randomly generated IV | AES-CBC encrypted ciphertext]
Why are the changes needed?
We want to have the ciphertext output similar across different modes. OpenSSL's EVP_BytesToKey is effectively deprecated and their own documentation says not to use it. Instead, CBC mode will generate a random vector.
Does this PR introduce any user-facing change?
AES-CBC output generated by the previous format will be incompatible with this change. That change was recently landed and we want to land this before CBC mode is used in practice.
How was this patch tested?
A new unit test in DataFrameFunctionsSuite was added to test both GCM and CBC modes. Also, a new standalone unit test suite was added in ExpressionImplUtilsSuite to test all the modes and various key lengths.
build/sbt "sql/test:testOnly org.apache.spark.sql.DataFrameFunctionsSuite"
build/sbt "sql/test:testOnly org.apache.spark.sql.catalyst.expressions.ExpressionImplUtilsSuite"

CBC values can be verified with openssl enc using the following command:
echo -n "[INPUT]" | openssl enc -a -e -aes-256-cbc -iv [HEX IV] -K [HEX KEY]
echo -n "Spark" | openssl enc -a -e -aes-256-cbc -iv f8c832cc9c61bac6151960a58e4edf86 -K<PHONE_NUMBER>666768696a6b6c6d6e6f7031323334353637384142434445464748

@sweisdb Could you remove the error class AES_SALTED_MAGIC from error-classes.json, and related function aesInvalidSalt() + the test "INVALID_PARAMETER_VALUE.AES_SALTED_MAGIC: AES decrypt failure - invalid salt"

Updated to address this.

@sweisdb Could you re-trigger GitHub actions, please. Highly likely the failure of the test org.apache.spark.storage.BlockManagerBasicStrategyReplicationSuite is not related to your changes but just in case let's re-run all tests.

@MaxGekk Sure, that worked and it's all green now.

+1, LGTM. Merging to master.
Thank you, @sweisdb.

@sweisdb Congratulations with your first contribution to Apache Spark!


[datashare] sort api's and definitions

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[ ] I commit to follow the Breaking Change Policy of “no breaking changes
[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[ ] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
Adding a new service

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from API Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable version with Breaking Change Validation errors
[ ] Updating API(s) in preview over 1 year

Please follow the link to find more details on PR review process.

/azp run automation - sdk

/azp run automation - sdk

Just sorting the API's alphabetically. This will ensure reviews going forward are smooth.

Can one of the admins verify this patch?

No breaking change. Only re-ordering of API's.
Python API's also shouldn't be breaking change. We just added query parameters. Refer: https://github.com/Azure/azure-rest-api-specs/pull/10209


Created a callback exactly when the user touches the button and a cre…

Created a new method to notify the listener exactly when the button was tapped, and a touchable/untouchable switch ;D cheers.

Thanks for this! i was looking for a way to be able to do this and get an instant callback. much appreciated


PCI-E Splitter Support?

Hi,
I am in the process of switching from windows to Ubuntu linux. The problem I am having is that the ROCm drivers do not work with my 4 in 1 PCI-E splitter. This worked fine with Windows. Is this a known issue? Is there any way to resolve this?

ROCm support PCIe Switches.   It dependent on you  which GPU you are running.  If you running Polaris cards you need PCIe Gen3 base PCIe switch since they need PCIe Atomic support.   Not windows driver does not use PCIe Atomics

@gstoner hmm it was for Vega 56 in a PCIe 3 slot. I have changed my motherboard now without splitter


yyyy/mm/dd date format

A user in Japan has noticed that his locale's customary yyyy/mm/dd format is not supported. This issue was briefly discussed in #70 with a potential solution at least when the year has 4 digits that would detect the yyyy/mm/dd format based on the position of the 4-digit sequence, regardless of the current locale's preferred date format. I think that's reasonable. Installing ICU didn't seem to help because this was skipped by the RE_DATE regex.

According to Wikipedia, only Kazakh uses yyyy.dd.mm, all other national date formats beginning with yyyy are in the order yyyy.mm.dd. I realized that ISO 8601 standard format yyyy-mm-dd is not matched either, so I will handle that case too.
It seems like the yyyy/mm/dd format should have worked for this user when ICU was installed, I'm not quite sure why it did not.


Minor grammatical and accuracy changes.

Minor grammatical and accuracy changes.

@BaileyLawson : Thanks for your contribution! The author, @rayne-wiselman, has been notified to review your proposed change.


feat: mount WebR file system

See docs here: https://docs.r-wasm.org/webr/latest/mounting.html

#334


Removed old/deprecated UI libraries

Description

Removed old/deprecated libraries from the UI category

          1 Warning

:warning:
Found 15 link issues, a project collaborator will take care of these, thanks :)

Link issues by awesome_bot

Line
Status
Link

4
302
https://triplebyte.com/a/azelHFa/d redirects tohttps://triplebyte.com/

276
301
https://github.com/analytics-pros/Swift-GA-Tracker-for-Apple-tvOS redirects tohttps://github.com/adswerve/Swift-GA-Tracker-for-Apple-tvOS

474
404
https://github.com/czater/Colorify

767
301
https://github.com/igormatyushkin014/Sensitive redirects tohttps://github.com/hellowizman/Sensitive

935
301
https://github.com/onmyway133/Anchors redirects tohttps://github.com/onmyway133/EasyAnchor

1333
404
https://github.com/jorik041/PerfectAPIClient

1382
301
http://www.catapush.com/ redirects tohttps://www.catapush.com/

1452
301
https://github.com/adrianmateoaea24/magic-mapper-swift redirects tohttps://github.com/adrianmteo/magic-mapper-swift

1532
301
https://github.com/IvanVorobei/SPPermission redirects tohttps://github.com/ivanvorobei/SPPermissions

1615
301
https://github.com/facebook/facebook-swift-sdk redirects tohttps://github.com/facebookarchive/facebook-swift-sdk

1987
301
https://github.com/Babylonpartners/DrawerKit redirects tohttps://github.com/babylonhealth/DrawerKit

2074
301
https://github.com/soheil/RadialLayer redirects tohttps://github.com/soheil-zz/RadialLayer

2115
301
https://github.com/ML-Works/Overlap redirects tohttps://github.com/MLWOS/Overlap

2525
301
https://github.com/OpenFeyn/KafkaRefresh redirects tohttps://github.com/HsiaohuiHsiang/KafkaRefresh

3228
301
http://jamesonquave.com/blog/tutorials/ redirects tohttps://jamesonquave.com/blog/tutorials/

  Generated by :no_entry_sign: Danger


@ApiModelProperty(hidden = true) removes response fields from the API payloads, not just Swagger

Hi all,
I'm trying to hide a response body field from Swagger UI by annotating the field with @ApiModelProperty(hidden = true) on the field. However, by doing so, the field is also removed from the actual API response body, not just the Swagger UI.
I'm using Lombok for the getters/setters, but I have tried annotating an explicit getter but seeing the same results (referencing https://github.com/swagger-api/swagger-core/issues/2061).
For example:
@EqualsAndHashCode(callSuper = true)
@NoArgsConstructor
public class SomeClass extends SuperClass {

    private static final long serialVersionUID = 1L;

    private Integer fieldA;

    @ApiModelProperty(hidden = true)
    private Boolean fieldB;
}

This results in fieldB failing to return in the API response.
Similarly, the following also fails to return in the API response:
@EqualsAndHashCode(callSuper = true)
@NoArgsConstructor
public class SomeClass extends SuperClass {

    private static final long serialVersionUID = 1L;

    @Getter
    @Setter
    private Integer fieldA;

    private Boolean fieldB;

    @ApiModelProperty(hidden = true)
    public Boolean getFieldB() {
        return fieldB;
    }
}

Any ideas as to what could be causing this behaviour? Is there a known workaround?
Thanks,
Taylor

@zebugman try @Schema(hidden=true) ?


Typing time from keyboard displays random minutes after selection, when I type 0 in minutes

Current behaviour
Typing time from keyboard displays random minutes after selection, when I type 0 in minutes
Like typing, 9 in hour is fine,
00 in minutes is fine
0 in minutes appends random trailing number.
Expected behaviour
0 minutes should take 00 minutes in selected time
How to reproduce?
running it in android platform, or android web ..
    <TimePickerModal
        use24HourClock={false}
        visible={modalVisible}
        onDismiss={toggleModal}
        onConfirm={(selectedDate) => {
          console.log(selectedDate);
          setSelectedTime(moment(selectedDate).format('HH:mm:ss.SSS[Z]'));
          toggleModal();
        }}
        hours={selectedTime ? moment(selectedTime).hours() : moment().hours()}
        minutes={
          selectedTime ? moment(selectedTime).minutes() : moment().minutes()
        }
      />

Is the module usage with my random data/entries
Preview
())
What have you tried so far?
Tried with different times
Your Environment

software
version

ios
x

android
x

react-native
0.73.6

react-native-paper
0.20.5

node
20

npm or yarn
x.x.x

expo sdk
50.0.15

PRs are welcome!


Rename Alpine shared fx to "linux-musl"

React to changes made in core-setup to use "linux-musl" instead of "alpine3.6".
Need to ensure:

Packages such as SQLitePCLRaw still have their native binaries copied into the sharedfx we build
Rename symbols packages
Rename installer files
Update dotnet/dotnet-docker

Part 1: https://github.com/aspnet/Universe/pull/1104 adds linux-musl-x64.
Part 2: TBD - drop alpine.3.6-x64. We can do this later.

Done in https://github.com/aspnet/Universe/pull/1121


Support with SF 3.0

Fix https://github.com/florianbelhomme/FlobFoundationBundle/issues/35

As you change the configuration of the dependencies, can you update the composer.json file to match the version of the bundles with this configuration? Thanks.

The tests are failing, fix them please.


MECH 8 - Create blueprint for base weapon

Definition of Done:
Base Weapon blueprint exists with basic firing functionality
Implementation:
The blueprint will include base functionalities that will be shared amongst different weapons
BaseWeapon base functionality includes:

[ ] Fire
[ ] Reload
[ ] Aim

Additional sub classed blueprints should include:
*This is necessary as these are parent classes to different weapons

[ ] Hitscan weapon
[ ] Projectile weapon

Scope Transfer:
Scope from this ticket has been transferred into ANIM-16. This is due to requirements of additional Unreal Engine work to test the correctness of the animations.