Add BaseDataset and FileSystemDataset classes

Incorporate the changes from the reviews for https://github.com/woven-planet/wicker/pull/57 in this PR:

We leave the AbstractDataset class alone so that it remains a pure interface.
As suggested in the previous review, we create the BaseDataset class as a concrete implementation of AbstractDataset that is orthogonal to any filesystem or bucket-specific assumptions (it will rely on the storage and column file reader instances for this logic).

The S3Dataset class is updated so that it is a subclass of BaseDataset (there is still a little bit of S3-specific logic that is left in this class for backwards compatibility).
We create the new FileSystemDataset class, but it only exists so that its constructor arguments can be constrained to be compatible with local storage (e.g. the storage argument must have the type FileSystemDataStorage and the Arrow filesystem has to be a LocalFileSystem). However, I would be ok to remove this class if the reviewers feel it is unnecessary.
Testing:

Unit tests added to cover the new FileSystemDataset class and its new behavior to support local storage (together with BaseDataset).
Running test training jobs is in progress; they will be linked here.

I have been able to make this PR is totally orthogonal from https://github.com/woven-planet/wicker/pull/73 . Although I think #73 has good changes for Wicker, actually this PR is the one we really need to enable reading Wicker datasets from local filesystems.
Note: One thing Zhenyu proposed in his review of Isaak's PR is to change the user experience so that they use a factory to return instances of AbstractDataset. We want to leave this new user experience up to the platform team. However, the new factory methods should work well with the changes in this PR!

Thank you @zhenyu for the review so far! Since we have made some changes together, let me get another review when you can (especially for the new factory class and functions)!

@zhenyu I fixed our new builder function to accept a dataset_config parameter to enable the user to easily specify the dataset type (and I updated the wickerconfig.json file to support FileSystemDataset configuration). Everything looks good. But now as you can see from my last comment I am wondering if the wickerconfig.json file should support multiple WickerAwsS3Config or WickerFileSystemConfig named entries, do you think this is a usecase or I am over-thinking it.

@zhenyu I think I have got it. Now you can have wickerconfig.json like this:
{
  "aws_s3_config": {
    "s3_datasets_path": "s3://fake_data/",
    "region": "us-west-2",
    "boto_config": {
      "max_pool_connections":10,
      "read_timeout_s": 140,
      "connect_timeout_s": 140
    }
  },
  "filesystem_configs": [
    {
      "config_name": "filesystem_1",
      "root_datasets_path": "/mnt/bucket_1/"
    },
    {
      "config_name": "filesystem_2",
      "root_datasets_path": "/mnt/bucket_2/"
    }
  ],
  ... (other stuff)
}

There is still just the one aws_s3_config, but now you can have multiple filesystem configs for mounting multiple FileSystemDataset volumes into the training job. In the config, you have to give them a configuration_name property so that in the builder function you can know for which one you want to return a FileSystemDataset instance.

@convexquad Thanks so much for this PR. The class design is much better now although we could improve even further, let us do it little by little. For this PR, I am fine with the class design now. Please make sure enough test, the CI pipeline is not so convincing now.
Ping me for an approval, once you think the test enough. Thanks again

@zhenyu thanks for all the reviews! I am running test cloud jobs to make sure everything is good with this. I might leave this PR open for a short while.

@zhenyu could you give me an approving review for this PR? I have completed test training jobs with both FUSE-mounted local FileSystemDataset instances as well as S3Dataset instances from this branch.
I cannot add a link to the training job since this is a public GitHub repository, but I have tested this PR in our ML repository on my branch dev/abain/test_wicker_s3 with run ID 9eb6efad-6c64-4260-abec-a5d3c0c59779. The test training job loads many S3Dataset instances.


Make Orienteer-Taucharts to use REST for data retrieval instead of inplace JSON

Fixed here: https://github.com/OrienteerBAP/Orienteer/commit/b49d097e33a75b1c27e77ec8750f85d19e445bdb


Handling missing ~/.nuget/packages directory

An ugly exception (see below) is thrown if the ~/.nuget/packages directory is missing.
FWIW, I suspect I may have deleted that directory on this system during a manual disk cleanup / space reclamation binge before I found out about nuget-gc ;)
Repro steps:

dotnet tool install dotnet-nuget-gc -g
rm -fR ~/.nuget/packages
dotnet nuget-gs

Failure mode:
$ dotnet nuget-gc

Unhandled Exception: System.IO.DirectoryNotFoundException: Could not find a part of the path '/home/MYUSER/.nuget/packages'.
   at System.IO.Enumeration.FileSystemEnumerator`1.CreateDirectoryHandle(String path, Boolean ignoreNotFound)
   at System.IO.Enumeration.FileSystemEnumerator`1..ctor(String directory, EnumerationOptions options)
   at System.IO.Enumeration.FileSystemEnumerable`1..ctor(String directory, FindTransform transform, EnumerationOptions options)
   at System.IO.Enumeration.FileSystemEnumerableFactory.DirectoryInfos(String directory, String expression, EnumerationOptions options)
   at System.IO.DirectoryInfo.InternalEnumerateInfos(String path, String searchPattern, SearchTarget searchTarget, EnumerationOptions options)
   at System.IO.DirectoryInfo.GetDirectories(String searchPattern, EnumerationOptions enumerationOptions)
   at System.IO.DirectoryInfo.GetDirectories()
   at NugetCacheCleaner.Program.CleanCache(Boolean force, TimeSpan minDays) in /home/vsts/work/1/s/src/Program.cs:line 75
   at NugetCacheCleaner.Program.Main(String[] args) in /home/vsts/work/1/s/src/Program.cs:line 40

Possibly related issues: #7

I submitted PR #13 to address this issue.

This repo seems to be abandoned. Check out the commit history.

Fixed by #13


Add internal API Reference Section and Missing APIs

Document undocumented APIs including internally used ones. #4845 #447

@ChrisHuie can you add this at the bottom of the "Adding an Adapter" section by adding a reference in the sidebar.yml

https://github.com/prebid/prebid.github.io/blob/2ad92c2c6918ac6d158918c2215ddd78b5570e6a/_data/sidebar.yml#L400


Interaction with window.scrollTo/scrollBy

I think the interaction of the visual viewport with the window.scrollTo/scrollBy is also worth specifying as part of this proposal. The intuitive (to me, anyway) behaviour from the author's point of view would be that calling window.scrollTo also resets the visual viewport to the top-left of the layout viewport, and that calling window.scrollBy leaves the visual viewport as-is relative to the layout viewport.

Agreed that this needs some thought but I'm not sure yet how these should work. My intuition is that if we add visualViewport.scrollTo/By then the old window versions should behave as if the visual viewport didn't exist. That's not the case for us today.


Type the result property on IDBRequest

Fixes #25547

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: LinusU sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

You should edit this into TSJS-lib-generator repository.
Please read: https://github.com/Microsoft/TypeScript/blob/master/CONTRIBUTING.md#contributing-libdts-fixes.

Hmm, I thought that was for src/lib/dom.generated.d.ts and src/lib/webworker.generated.d.ts? But this is touching lib/dom.d.ts and lib/lib.webworker.d.ts 🤔
I'm probably missing something though 😄

lib/dom.d.ts and lib/lib.webworker.d.ts are generated from src/lib/dom.generated.d.ts and src/lib/webworker.generated.d.ts (and other files). And these are generated from TSJS-lib-generator.

Got it 👍
Closed in favour of https://github.com/Microsoft/TSJS-lib-generator/pull/530


eliminate jet

everyone agrees it’s objectively bad 😉
therefore the characters “jet” should only appear in the commit message “jettisoned bad colormap defaults”

https://jakevdp.github.io/blog/2014/10/16/how-bad-is-your-colormap/
http://cresspahl.blogspot.de/2012/03/expanded-control-of-octaves-colormap.html
http://stats.stackexchange.com/questions/223315/why-use-colormap-viridis-over-jet
https://eagereyes.org/basics/rainbow-color-map
https://courses.washington.edu/engageuw/why-you-should-dump-the-rainbow/
http://researchweb.watson.ibm.com/people/l/lloydt/color/color.HTM

i’ll do it if you want, and i’ll make sure contrast and distinction is preserved in all figures

that would be nice! :) but give it another day.
for the categorical case, as is very important, for example, here
https://github.com/falexwolf/collab_alex/blob/master/scanpy/examples/nestorowa16.md
i'm not sure whether one can actually do better than jet. but ok, you will know better. :)

haha with ease. you can observe the inhomogeneous contrast distribution with C2 and C10 there: the colors are indistinguishable dark blue while C7 and C8 go from snot green all the way to orange.
that would be horrible for continuous data, but merely makes C2 and C10 indistinguishable for categorical colors and unnecessarily reduces contrast there (as it’s a color map and no palette).

check out e.g. color brewer’s paired, which has 12 easily distinguishable colors (although that yellow is too light for a white background):

looks good, i'll take one of these.

so, https://github.com/theislab/scanpy/commit/d5d2a099a5b2d6f34144f331c5074df386048100 uses now a smart choice of color maps and plotting layout, though i still didn't get good results for the example above with the categorical palettes

now also dpt calls plotting.plot_tool, so if you want to improve the choice of color maps, go ahead! you'll only have to do it in there. :)

OK 😋


Only allow plans to accept Sequences of detectors

Allow the plans to only accept lists or tuples of hardware devices eg. detector to avoid unexpected behaviour.
Description

Adds a check on the type of the detector argument passed to the main plans used and raises an error if not a Sequence.
Adds type checking to plans and plan stubs.

Pulled in PR#1610 which had started the work on type checking.
Motivation and Contex
Currently plans will still run to completion if passed an iterable different from a generator as input, but will do the wrong thing in the background.
See #1613 .
How Has This Been Tested?

Type annotations should not change the functionality of the code.
Small additions to unit tests to verify that plans raise a TypeError if a generator is passed instead of a list.

Docs build is failing because numpydocs doesn't accept having both Yields and Returns sections in the docstring. It seems it never has even in older versions and adding type hints may have triggered this check

Numpydoc PR to address this: https://github.com/numpy/numpydoc/pull/542

There is a numpydoc release now (which is why I tried to rebase #1610 because I thought that one was this one).

I see that @noemifrisina has CI passing. Were your concerns above adequately addressed, @callumforrester?


Remove typecast

Codecov Report

Merging #58 (9148063) into main (39e845f) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##             main      #58   +/-   ##
=======================================
  Coverage   18.25%   18.25%
=======================================
  Files           7        7
  Lines         400      400
=======================================
  Hits           73       73
  Misses        327      327

Impacted Files
Coverage Δ

torchts/utils/graph.py
0.00% <0.00%> (ø)

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 39e845f...9148063. Read the comment docs.


Replace ctr i pull/push with nerdctl -q

Issue #, if available:
Closes #403
Description of changes:
Replace ctr i pull/push with nerdctl -q to quieten log output during image operations.
Testing performed:
Ran integration test suite successfully.
Also ran one test, LazyPullWithSparseIndex, with and without this change, to compare output verbosity:
% wc out.before
  1372   8285 248098 out.before

% wc out.after
  1166   7415 215406 out.after

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

How does pushing to the registry work without credentials?

@Kern-- credentials are cached when newShellWithRegistry does nerdctl login ...


Add finalisation status to channel

This adds the finalization status to the ChannelState that gets passed into protocols.
This sets us up to easily check the finalization status in the close channel protocol for this

I'm going to close this and just include these changes with the changes for #3078

I'm going to close this and just include these changes with the changes for #3078

I'm going to close this and just include these changes with the changes for #3078

Actually, on second thought, this should probably be its own chunk of work.

I'm going to close this and just include these changes with the changes for #3078

Actually, on second thought, this should probably be its own chunk of work.


Large regions are generated during running sysbench insert

Bug Report

What version of TiKV are you using?
v4.0.15
What operating system and CPU are you using?
2TiDB、2PD、4TiKV
8 core、16G
Steps to reproduce
1、run sysbench prepare
tabNum=1
threads=10
What did you expect?
do not generate Large regions
What did happened?
generate Large regions

/assign gengliqi

/type question


2.x Fix deprecated() function

Ticket: #2146, #2154
Issue
There are many test errors in 2.x, because I got a bailout wrong in #2146. Sorry for that.
Solution
Fix bailout.
Impact
Tests should work again.
Usage Changes
None.
Considerations
None.
Testing
None.

just on ❕

all it took was just one !, huh?

Yeah, apparently 🤪. Even if it’s only a bugfix for a mishap of mine, this has to be my smallest pull request ever.

@jarednova  looks like I was late to the party and #2154 was the wrong approach.
Thanks for this fix @gchtr  - I merged this back into 2.x-factories as well.


[nrf toup] zephyr: update name of nrf5340 dk

This to correctly set the kconfig default value.
Ref: NCSDK-5599
Signed-off-by: Håkon Øye Amundsen<EMAIL_ADDRESS>

Some checks failed. Please fix and resubmit.
Nits issues
Missing newline at end of '.github/workflows/backport.yml'. Check your text editor settings.

Gitlint issues
Commit e88113bbeb:
1: UC6 Body has no content, should at least have 1 line.
Commit 014f42df09:
1: UC5 Title exceeds max length (78>72): "zephyr: Replace DT_FLASH_DEV_NAME with DT_CHOSEN_ZEPHYR_FLASH_CONTROLLER_LABEL"
Commit c95d684072:
1: UC3 Title does not follow [subsystem]: [subject] (and should not start with literal subsys:): "[nrf toup] clean up multi-image documentation"
Commit 23a7836ec5:
11: UC4 Line exceeds max length (94>72): "6bd9099 [nrf temphack] ci: skip gitLint if there is a [nrf mergeup] sauce tag on the PR title."
15: UC4 Line exceeds max length (79>72): "55d9cca [nrf noup] ci: CI failure on master or PR will send email to code owner"
pylint issues
************* Module assemble
scripts/assemble.py:23:0: W0611: Unused import io (unused-import)

Identity/Emails issues
2cdccc5f7d5d71dbb806c41540de2fd17212ac17: author email (Andrzej Puzdrowski<EMAIL_ADDRESS>needs to match one of the signed-off-by entries.
ee64382dd18192e5e793b99b78847ea28c8d41ea: author email (Andrzej Puzdrowski<EMAIL_ADDRESS>needs to match one of the signed-off-by entries.
66e9db267cec3d126178036dfad52ed3fd7ef6d5: author email (Andrzej Puzdrowski<EMAIL_ADDRESS>needs to match one of the signed-off-by entries.
278e2e377679f7638bc17a63394d7249a45f1aa1: author email (Andrzej Puzdrowski<EMAIL_ADDRESS>needs to match one of the signed-off-by entries.
d15b6379460c9f7fc2b3f6f886b24561b455ca04: author email (Andrzej Puzdrowski<EMAIL_ADDRESS>needs to match one of the signed-off-by entries.
72e596d573e2db5c3abd8850c57a4188b03bb32b: author email (Andrzej Puzdrowski<EMAIL_ADDRESS>needs to match one of the signed-off-by entries.
2b3aee45bffecb19b1a008f4a3b52d5ec1a6ab15: author email (Andrzej Puzdrowski<EMAIL_ADDRESS>needs to match one of the signed-off-by entries.
7322545d82f77e1c7eecf4186e64000d344b0ba9: author email (Andrzej Puzdrowski<EMAIL_ADDRESS>needs to match one of the signed-off-by entries.
238eece1b4d19fef73de0e1254a0759d53e52721: author email (Andrzej Puzdrowski<EMAIL_ADDRESS>needs to match one of the signed-off-by entries.
498190696eb9fb044e75ca59cdd9e9b7dfae122a: author email (Andrzej Puzdrowski<EMAIL_ADDRESS>needs to match one of the signed-off-by entries.
21b2467282030ef41986ae0347f179ba0db70b22: author email (Andrzej Puzdrowski<EMAIL_ADDRESS>needs to match one of the signed-off-by entries.
8eac2f27021c41b1fb5d42a533edb9e5dfbb5fb0: author email (Andrzej Puzdrowski<EMAIL_ADDRESS>needs to match one of the signed-off-by entries.
43fd424caf655be112864b8904b91daf592d6c64: author email (Andrzej Puzdrowski<EMAIL_ADDRESS>needs to match one of the signed-off-by entries.
ab0bad8af0d0c36b1c2cc44ba94447ff26df8833: author email (Emil Obalski<EMAIL_ADDRESS>needs to match one of the signed-off-by entries.
c60d3d84c96e1c3d9ac94162c242ecfbd17f13bc: author email (Emil Obalski<EMAIL_ADDRESS>needs to match one of the signed-off-by entries.
4ccee67fe77dbd40e22c8d37330d34a7692aac39: author email (Emil Obalski<EMAIL_ADDRESS>needs to match one of the signed-off-by entries.
409fd90574b6ebefeb9204d5bcc17bb3b3eaca28: author email (Emil Obalski<EMAIL_ADDRESS>needs to match one of the signed-off-by entries.
6b5157bc62fe0298085c286f61e5d47813062329: author email (Emil Obalski<EMAIL_ADDRESS>needs to match one of the signed-off-by entries.
4a4987a52e7f50853d25d2bb516e931c132a44c5: author email (Emil Obalski<EMAIL_ADDRESS>needs to match one of the signed-off-by entries.
b562c7f6a27f2ef59ae8540081c992b6dc5bb8ef: author email (Emil Obalski<EMAIL_ADDRESS>needs to match one of the signed-off-by entries.
101bdad99add39223947c7e4108f45d1b8f31ec9: author email (Emil Obalski<EMAIL_ADDRESS>needs to match one of the signed-off-by entries.
2444a580004d1253ef57f0fe9dda3e2fb910be48: author email (Emil Obalski<EMAIL_ADDRESS>needs to match one of the signed-off-by entries.
cf314a529542d7cf11d640cda1a02ad7219e3f5a: author email (Emil Obalski<EMAIL_ADDRESS>needs to match one of the signed-off-by entries.
5c201512819a6a038f97a42f159784e74b4b7656: author email (Emil Obalski<EMAIL_ADDRESS>needs to match one of the signed-off-by entries.
b3be3d03118b3ed046fe313613502439e00e7045: author email (Emil Obalski<EMAIL_ADDRESS>needs to match one of the signed-off-by entries.
5fbcb460c4c636a2eee6dea7ac51b19fadc76ff1: author email (Emil Obalski<EMAIL_ADDRESS>needs to match one of the signed-off-by entries.
c95d684072913d1c97b2d67a7c5d426029550b73: author email (Marti Bolivar<EMAIL_ADDRESS>needs to match one of the signed-off-by entries.
Tip: The bot edits this comment instead of posting a new one, so you can check the comment's history to see earlier messages.


Add a guide for first-time contributors

Some considerations:

If they're at all unfamiliar, suggest practicing here: https://github.com/firstcontributions/first-contributions
There should be a way to determine at a glance which issues are suitably easy
Links to all important concepts and topics one might need to read up on first should be provided
Contribution guidelines? Might be a losing battle...

Okay I will do this


UI changes in checker configuration dialog

Feedback from our QA team:

The "Files URLs" header of the table can be hidden, there is a label above it explaining the same thing.
Move "From all available condition sets" radio button before the "From condition sets"
Check images => Check for references to missing images
Check external links => Check for broken links to external web sites
Check internal links => Check for broken links to internal targets
Use profiling conditions(filters) => add extra space before (

The (2) issue seems not to be fixed. Attaching a suggestion from our QA team.

Fixed in: 92222a8a194671fd9ceb037df93aeaf5185df1be


feat: sort yearn vaults on the earn page

Description
PR for https://github.com/shapeshift/web/issues/877
Notice
Before submitting a pull request, please make sure you have answered the following:

[X] Have you followed the guidelines in our Contributing guide?
[X] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Pull Request Type

[ ] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[ ] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[X] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)
If applicable, please link to the github issue and put closes #XXXX in your comment to auto-close the issue that your PR fixes.
Testing
Please outline all testing steps

Pull branch locally and run yarn to install new deps
etc...

Screenshots (if applicable)

Please fix type cast, otherwise looks great

Done ! thanks 👍


SPIKE: Finalize Database Schema

Context
Before we can begin actually defining models in the code and running migrations, we need to know exactly what tables and fields we need
Definition of Done

An initial database schema is thoroughly documented and laid out for us to use
Schema is approved and placed in team documentation

Engineering Details
this does not need to be perfect as data will always evolve, but should be a good foundation of the minimum necessary tables and fields to get the app up and running

should this be consolidated with the data model ticket I put?

yeah, makes sense, I'll mark this as a duplicate of #11


[Candidate 4v4 Lost Cause ZH v4] Units can be air dropped at invalid places

Observed behavior
Units can be air dropped at invalid places.
Expected behavior
Units can not be air dropped at invalid places.
Screenshots

fixed. Just the ranger problem as always will probably still reside

Vehicle drop fixed.


Can't add repository key of rocm in ubuntu 20.04

hello rocm team
plz help, the terminal is showning
wget -q -O - http://repo.radeon.com/rocm/rocm.gpg.key | sudo apt-key add - gpg: invalid key resource URL '/tmp/apt-key-gpghome.KMH3OusvRn/home:manuelschneid3r.asc.gpg' gpg: keyblock resource '(null)': General error gpg: key 1488EB46E192A257: 1 signature not checked due to a missing key gpg: key 1488EB46E192A257: 1 signature not checked due to a missing key gpg: key 3B4FE6ACC0B21F32: 3 signatures not checked due to missing keys gpg: key D94AA3F0EFE21092: 3 signatures not checked due to missing keys gpg: key 871920D1991BC93C: 1 signature not checked due to a missing key gpg: Total number processed: 7 gpg:       skipped new keys: 7

try wget -q -O - http://repo.radeon.com/rocm/apt/debian/rocm.gpg.key | sudo apt-key add -

try wget -q -O - http://repo.radeon.com/rocm/apt/debian/rocm.gpg.key | sudo apt-key add -

thanks,
but its still same

That first wget should work, it uses the new location of the ROCm key. As a matter of fact, it does work on my computer. The error doesn't seem to point to an issue with the ROCm key, but to other keys. People have mentioned that the manuelschneid3r may have expired. That one may have to be updated. I wonder if the ROCm key hasn't been added after all. What does 'apt-key list' show? If it's showing the ROCm key, it most likely has added the key successfully, and it may be complaining about other keys.

Hi @AtulPremNarayan
Request you to close this issue, if not reproduced.
Thank you.

Closing this issue as there is no response from reporter for a long time.
Request @AtulPremNarayan to open a new issue if there is still a problem.


allow unknown fields

Changes

Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:

[ ] Run make test lint before submitting a PR
[ ] Includes tests (if functionality changed/added)
[ ] Includes docs (if user facing)
[ ] Commit messages follow commit message best practices

See the contribution guide for more details.
Release Notes

解决 DEVOPS-23540


Expose more APIs for onnx cpp backend

User wants to peek init_net and pred_net in some cases.

@onnxbot retest this please


Author's avatar name triggering cl.Text hyperlink.

Hi,
I think I spotted a Bug when creating a a message such as :
await cl.Message(         content="A random message containing an author name ... ",         author="random",     ).send()
And then setting an avatar to the author "random" :
 await cl.Avatar(         name="random",         url="blank",     ).send()
It will then trigger the hyperlink to an empty cl.Text for the first message on the word "random".
Is this a desirable behavior ?

Nice catch, will be fixed soon!

Should be fixed with the latest release.


Features/fix fake netlify

We're not sure about the yml date type, but from what we checked on the internet it seems to be valid. We will need to test it when this is merged. It should fix the Travis error 'date is not a string'

We've also modified the jsyaml.dump config so it wouldn't use multiline strings as much
Finally, we took a look at the tags validation and found it very tricky to apply individually to all fields, especially since the inputs other than the first are added dynamically. So what we've done is make sure validation is applied to the first fields (for both en and fr) following the idead that at least one tag should be added for each. But then all subsequent fields aren't validated, but we also only add them to the yaml file if their value isn't empty.

Following the build failure, date doesn't seem to work, we'll investigate further


initialFocusRef in Modal should open keyboard

Description
When using initialFocusRef property in Modal, the keyboard should open, instead the cursor blinks the selected input, but you have to click on it in order to open the keyboard.
CodeSandbox/Snack link
https://snack.expo.dev/@faf0/64fa14
Steps to reproduce
You can see it on the documentation example. Just click on open on a mobile device. The modal opens, the focus is on the correct input, but the keyboard closed
NativeBase Version
Any above 3.2
Platform

[X] Android
[ ] CRA
[ ] Expo
[ ] iOS
[ ] Next

Other Platform
No response
Additional Information
No response

Sorry I put  temporary link to the snack. This one should be ok:
https://snack.expo.dev/@faf0/initial-focus-on-modal

Hi @fafo, Thank you for reporting this issue. We will try to fix this at the earliest.

Hi @fafo, initialFocusRef can't open the keyboard. In your case, it's inputText but Sometimes the user wants to focus on radio or button when Modal is open. So this doesn't align with the component library perspective. I am moving this to the discussion. We can have a conversation about it there.


Carrier event property is not the active carrier

Integration Method: CocoaPods
Xcode Version: 10.1
Library Version: 3.3.9
Platform: iOS
Language: Swift / Objective-C
Description:
iPhone Xr on iOS 12.1.2 with physical SIM from vodafone IE and eSIM from AT&T installed. The physical SIM is turned off and the eSIM is turned on in the OS settings. The carrier value reported for all events is vodafone IE, when I would expect it to be the carrier for the active SIM, AT&T.

HI @aldukeman ,  we released a new version v3.6.1 which contains a fix for this issue. I'm closing this issue for now, please feel free to to reopen it if the problem still exists.


[OCDM] Enhancements

 1. Support passing Encryption Scheme and Pattern to Decrypt
 2. New Gstreamer Adapter method that parses Sample and Subsample info
    from GstStructure in ProitectionMetdata of GstBuffer
 3. New gstreamer adapter API supports passing Subsample Info to the DRM
    implementation without processing it.

Closing older PRs. Pls bring changes to latest sprint.


(WIP) stdio: ensure stdio devices are consistent with sys as char devices

I originally noticed this because I was trying to run this Python interpreter but the REPL hung.
The main culprit is that we are stubbing stdin/out/err and fs.ModeCharDevice must imply fs.ModeDevice; this in turn causes
https://github.com/tetratelabs/wazero/blob/a4226906cf30ba5261d2fab2ff4df62a8109d6ea/imports/wasi_snapshot_preview1/fs.go#L360-L365
to return UNKNOWN instead (value 0)
I have dug further, and it seems that we are defaulting stdin/stdout/stderr to be block devices instead of CharDevices
https://github.com/tetratelabs/wazero/blob/372e0222ddb7dbc64bc7829fc112dd0b88a15e9e/internal/sys/fs.go#L83-L87
this in turn causes my new TestStdioStat to fail, but if I flip those defaults I cause other tests to fail.
Hence, it follows that we are not really being consistent with the operating system, but we are testing our own mocks (at least in some cases) 🤔
Draft for now.

In practice, my doubts amount to: "shouldn't we stat stdin/stderr/stdout at boot time instead of mocking?"


[stable/inginx-ingress] added support for a global image registry option

Signed-off-by: Mo Kassem<EMAIL_ADDRESS>
Added a global.imageRegistry at the top of the values.yaml (commented out). This value if set will override the image registry for all the images, including dependencies, configured to use the global value.
Added two helper templates nginx-ingress.image and defaultBackend.image to return the proper image's name for the controller and the default backend
Updated the README with the description of the new added values

/assign @jackzampolin

Is there anybody available to give approval for this PR?

/ok-to-test

/approve

/ok-to-test


Express section

Upper row with icon, line, p, h2
Lower row boxes with images, h3, p
Boxes hover animation + shadow


OBC updates ignored once bucket is provisioned

This is for public FYI:
Today, once a bucket is provisioned all subsequent OBC updates are ignored. The implications of this are changes in (future) bucket properties such as ACL, size limits, etc won't be processed by  the lib.

we should add this to the design doc

#97 addresses this.


With -Dsetup uma tables are not get created

When we delete the existing database and start the server with -Dsetup uma scripts are not get executed.

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


⚠ Bofrost.de has degraded performance

In 9593b0b, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1029 ms

Resolved: Bofrost.de performance has improved in f062546 after 6 minutes.


Update auto complete prop names

Describe the bug / expected behavior
Capitalization of the allowCustomtag prop — should be allowCustomTag.
Rename the limitCharacterCountTag prop to tagCharacterLimit.
What are the steps to reproduce?

Go to https://github.com/etn-ccis/blui-react-native-component-library/blob/dev/components/src/core/AutoComplete/AutoComplete.tsx#L62
and https://github.com/etn-ccis/blui-react-native-component-library/blob/dev/components/src/core/AutoComplete/AutoComplete.tsx#L44

Screenshots / Screen recording
Code snippet / Link to minimum reproduction example

CODE HERE

Your environment information

Suggested fix

Anything else to add?

Prop name change is a breaking change, hence we need to add it into a major change.


test environment - "SupportsHiddenColumns": false by default - why?

Latest versions of localdb (build 13.0.4001 or later) supports system versioned tables, so i should be "true" since this is available in the localdDB version that comes with VS 2017

We could. So far our testing SqlServer version has been 2014.

Maybe it is time to update, that is 2 released versions behind 😄


Popover position incorrect on page load

When using the new feature options.showOnLoad the popover is positioned incorrectly on page load.

This is due to bootstrap. bootstrap will position the popover to be inside the viewport when triggered.  The workaround is to position the popover on the top or bottom of the element it is attached to.
This workaround was done for 1.x in #223 popover docs cleanup


Fix redirected progress output to be less verbose.

Currently, when the progress output is redirected, it prints out a lot of
percentages and is very verbose.
This commit changes the redirected output to print a period for each 10 percent
that completes, resulting in much less output but still showing progress.
Fixes dotnet/cli#8288.

@livarcocc how do we roll a new build of this to include the fix in CLI?

Look at VSO for the build for this merge and update DependencyVersions in the CLI with said build.

So that I'm on the same page:

Bump version number in https://github.com/dotnet/dotnet-cli-archiver/blob/master/build/Versions.props
Kick off VSTS build which publishes new package.
Update CLI dependency to match new version number.

Is that correct?

You shouldn't have to do 1 or 2. Just merging the PR should have kicked a build.
Pick it up here: https://devdiv.visualstudio.com/DevDiv/Default/_build/index?definitionId=5619&_a=completed and update the CLI.

Isn't that for the parser and not the archiver?

sorry, handling a fire now and doing too many things at once. I guess you just need to kick off a build on the archiver. Not sure why there is no trigger. ping @tannergooding to confirm and see why there is no trigger.

Will do, thanks!


pcss vs css and tailwind integration

What is the difference between pcss and CSS, in the way it is used in the code?
And you have tailwindcss, and I don't see it in the code. Can you give an example of using tailwind with content script, and shadow house + CSS modules, for components?

This is out of scope of this issue, you can search that pattern up!


Get username from backend

Get the username from ardana-service to display in the servers selection page.

My bad.  I should've removed it in the first place.  Done


Splitting widget and app code

Copied from https://github.com/house-of-abbey/GarminHomeAssistant/issues/113.
Goals

Having a code base for the app version without "legacy" widget code for achieving space to add new features.
Set the widget version to maintenance only mode (feature freeze, only bug fixing etc.)

Proposal
New repository under "house-of-abbey" for the widget version. For example: "GarminHomeAssistantWidget".
Cloning the current code base into it. Removing all the "legacy" widget related code from the app repo and removing all the glance code in the widget version.
Set the widget repo to "maintenance only mode" by making this clear in the readme.

At the moment, I replacing all URLs etc. to "GarminHomeAssistantWidget" repo:
I propose, that we provide a dedicated version of the web-based editor for the widget version (so future changes in the "app" version won`t break it).
Suggested URL: https://house-of-abbey.github.io/GarminHomeAssistantWidget/web/
What do you think? @philipabbey @JosephAbbey

It is now available at that URL.

Just found a bug in compile_sim.cmd where I created but never used a variable.
--jungles %SRC%\%JUNGLE% ^

I realise you're on Linux, but could you amend the (simplified) file for the widget too please? I'll test on review.
Thanks.

I now have a version of the application working with widget code removed. See https://github.com/house-of-abbey/GarminHomeAssistant/tree/113-splitting-widget-and-app-code. I've started to look at the docs too, but some of that needs to be coordinated with your MR here.

Just found a bug in compile_sim.cmd where I created but never used a variable.
--jungles %SRC%\%JUNGLE% ^

I realise you're on Linux, but could you amend the (simplified) file for the widget too please? I'll test on review.
Thanks.

I added your version (and change the "default" device to "Venu"):
https://github.com/house-of-abbey/GarminHomeAssistantWidget/blob/1-splitting-widget-and-app-code/compile_sim.cmd

I now have a version of the application working with widget code removed. See https://github.com/house-of-abbey/GarminHomeAssistant/tree/113-splitting-widget-and-app-code. I've started to look at the docs too, but some of that needs to be coordinated with your MR here.

Yes, I wrote a proposal:
https://github.com/house-of-abbey/GarminHomeAssistantWidget/compare/main...house-of-abbey:GarminHomeAssistantWidget:1-splitting-widget-and-app-code#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5
I think, this can be use as a inspiration for the main repo.

I will finally check my changes tomorrow, before I will do the PR.

Just a reminder: The app store descriptions for the app and widget versions has to be updated too.

Just found a bug in compile_sim.cmd where I created but never used a variable.\n\n--jungles %SRC%\%JUNGLE% ^\nI realise you're on Linux, but could you amend the (simplified) file for the widget too please? I'll test on review.\n\nThanks.\n\nI added your version (and change the "default" device to "Venu"):

This will be fine as long as the Jungle file is renamed too.
Thanks

Yes, I wrote a proposal:\n\nmain...house-of-abbey:GarminHomeAssistantWidget:1-splitting-widget-and-app-code#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5\n\nI think, this can be use as a inspiration for the main repo.

Will take a look, I've already started to make changes to the README for the application.

Just found a bug in compile_sim.cmd where I created but never used a variable.\n\n--jungles %SRC%\%JUNGLE% ^\nI realise you're on Linux, but could you amend the (simplified) file for the widget too please? I'll test on review.\n\nThanks.\n\nI added your version (and change the "default" device to "Venu"):

This will be fine as long as the Jungle file is renamed too.
Thanks

Yes, I renamed all files back to "default" names, as part of transforming the repo to widget only.

On Tuesday, I did a PR for this branch.. but for the wrong repository 🙈 (because this repo is a fork from GarminHomeAssistant.. the default behavior is a PR for GarminHomeAssistant). So now for the right repo...

@JosephAbbey can you check the default behaviour here please?


Question : Handling a specific customization where file needs to be saved in directory?

Hi Team,
Good Morning.
We are working on a functionality where we need to export some files i.e. PDF, Excel from NAV and save it on some directory (Client or Server). Is there any way to do this on business central because i'm not sure how or what path we will define to export files.
Same is the query in case if we need to import file into NAV.
Any suggestions..
Regards,
Anuj Nagpal

Not supported (unless you set the compiler target to 'internal' for on-prem extensions)
https://github.com/Microsoft/AL/issues/2981
What you can try is to engage a cloud storage provider and then exchange the files using Streams, TempBlob table and the BLOB fields in table 1220 Data Exch. with it.


Jib-core: creation date ignored when pushing to repo

I understand that for reproducibility reasons  we need to set the creation timestamp to 0 (epoch). But in some cases, we are not rebuilding an image, but simply push it, maybe we could keep the creation date of the image.
For example, in my project, we use jib-core's JibContainerBuilder and I need to build the image and save it, then push the image later (we do that in order to take advantage of the caching system of our CI). In jib-core, the only way to push an image is actually to rebuild it from the tar:
Jib.from(<tar>)
  .containerize(
      Containizer.to(
          RegistryImage(... <image ref> ... )
     )
  )

In this example it's not a real "rebuild", it's a "push", so the creation date could be read from the tar. But it's not and it's set to zero.

Proposals:

Can we imagine specific values to setCreationDate() like:

USE_FROM_TIMESTAMP where it keeps the creation date from the from() image?
Keep the creation date from the from() if we are not adding any new layer
USE_CURRENT_TIMESTAMP like in maven's jib setting the current timestamp

or even, having an explicit push(..) method which will simply push a tar without rebuilding it, keeping the creation date?

I currently work it around by storing the creation date in a file alongside with the image tar, call the setCreationDate() with the content of this file.

Thanks for the input.

USE_FROM_TIMESTAMP where it keeps the creation date from the from() image?
Keep the creation date from the from() if we are not adding any new layer

In this example it's not a real "rebuild", it's a "push"

Allow me to share our views on this. Jib is a tool to build an image in a declarative manner. When you run Jib, you are always creating a new image, as it is what Jib does. Even if you have nothing after from(), you will create a new container configuration and a manifest, which means you will create a different image than the base image. The newly created image will always be different, as these metadata are really different. Jib is not a tool to just move or copy an existing image (whether the image is stored locally as a tar, in a Docker daemon cache, or in a remote registry). For such operations, there are other great tools specifically designed for that.
And the way we want Jib to build images is declaratively and  reproducibly. Image building is then conceptual. If you have the same source code that calls the Jib API interface in the exact same way, then whenever and wherever you check out the code and build an image, it should build the exact same image every time. That is, as soon as you have written a piece of code, it becomes irrelevant whether you have physically created and stored an image somewhere; all that matters is that you have the code that can build an image. But if you make Jib read a timestamp of a local file, then the outcome of running Jib will be different if, for example, you run it on a different machine or OR, or a month layer.

USE_CURRENT_TIMESTAMP like in maven's jib setting the current timestamp

This is very easy to do in Java, so I don't think we really need to change the Jib API. Just get the current time and set it.

I am closing the issue, but feel free to update for any feedback. Thanks!


Add frost free season indicators [rebase]

Pull Request Checklist:

[x] This PR addresses an already opened issue (for bug fixes / features)

This PR fixes #887

[x] Tests for the changes have been added (for bug fixes / features)
[x] Documentation has been added / updated (for bug fixes / features)
[x] HISTORY.rst has been updated (with summary of main changes)

[x] Link to issue (:issue:number) and pull request (:pull:number) has been added.

[x] bumpversion patch has been called on this branch
[ ] The relevant author information has been added to .zenodo.json

What kind of change does this PR introduce?

adds a few indicators with metadata specific to frost free season
add growing_season_start Indicator based on freshet_start indice
add thawing_degree_days
add freezing_degree_days
add French translations for new indicators
Correct usage of "." in description, long_name, and title strings

Does this PR introduce a breaking change?
I've added a default suggested value to some growing_degree_day calculations. This might affect some users. If this should be undone, let me know.
Other information:
There's an identical indice proposed here, solely with differences in metadata (freshet_start = frost_free_season_start). I wanted to propose this since "freshet" is not as commonly used in literature as "frost free season". If this ruffles feathers, I can remove it.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

6 of 6 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.004%) to 90.247%

Totals

Change from base Build<PHONE_NUMBER>:
0.004%

Covered Lines:
12482

Relevant Lines:
13831

💛  - Coveralls

@juliettelavoie I've added a growing_season_start indice to fix the issues you mentioned. Thanks for the review comment.
This PR is good for final review IMO.

Added freezing_degree_days since well, why not. Thanks for the reviews both of you!

I just realized that growing_season_lenght used tas, not tasmin as described and as would make sense for frost_free_season_length.

I just realized that growing_season_length used tas, not tasmin as described in the metadata of frost_free_season_length. frost_season_length uses tasmin.
(same for _start and _end)

Right. I really shouldn't be rushing this PR through like this. Will make the needed modifications.

I just realized that growing_season_length used tas, not tasmin as described in the metadata of frost_free_season_length. frost_season_length uses tasmin.
(same for _start and _end)

Right. I really shouldn't be rushing this PR through like this. Will make the needed modifications.

and I think it makes sense that frost_free_season_length should use tasmin, so I don't know that we can even use growing_season_length ( or maybe modify it to be more flexible to be able to choose btw tas and tasmin)

and I think it makes sense that frost_free_season_length should use tasmin, so I don't know that we can even use growing_season_length ( or maybe modify it to be more flexible to be able to choose btw tas and tasmin)

We can't leverage growing_season_* indides, no, The good news is this will correct some underlying data. It's just a bit more code.

Since #873, you can use input={'tasmin': 'tas'} to switch a variable in the compute's signature. Could that be useful here?
As for the flexibility question, we have no current way of making this. But maybe we should.


Publish nbsite to two new channels: tooling_dev and tooling

I would like to be able to build hvPlot's docs without any HoloViz dev release but with a dev release of nbsite. This is currently not possible (or at least not easily). With this PR nbsite will get additionally published to tooling_dev (dev releases) and tooling (main releases) channels.

I don't know if we need two channels for this, as it is an internal tool on our channel.

Do you mean 1 channel instead? Just tooling_dev? Might be fine yes.

Do you mean 1 channel instead? Just tooling_dev? Might be fine yes.

Yes, maybe just tooling to keep it short.

Hmm sometimes I may want pyviz/label/dev but not a dev version of nbsite.

Your suggested changes won't solve that either, right? As nbsite is still upload to dev.

Ah yes true 🤦 In which case I'd suggest no longer publishing nbsite to dev and main.

I'm not a fan of that. Wouldn't it be easy enough to pin nbsite in that case?

I'm not a fan of that. Wouldn't it be easy enough to pin nbsite in that case?

It depends on what you want to achieve. Pinning is imo an okay solution short term but is annoying long term. I've had a quick look at these repos and saw that most of them build the docs with the pyviz/label/dev channel, I'm not sure that's the right thing to do? Perhaps this channel was set up at some point to get a dev release of nbsite and then forgotten, I'm not sure. I also noticed nbsite is pinned to 0.8.4 on Param's doc build.

HoloViews: pyviz/label/dev
GeoViews: pyviz/label/dev
Panel: pyviz/label/dev
Datashader: pyviz/label/dev
hvPlot: conda-forge with dev install of nbsite from pip
Lumen: pyviz/label/dev
Param: nbsite ==0.8.4" from pip

Pinning is imo an okay solution short term but is annoying long term

But we control the whole chain, so we can always push a fix if we need it.

build the docs with the pyviz/label/dev channel, I'm not sure that's the right thing to do?

I feel like we have this talk every six months or so :upside_down_face:
In my opinion, it is not a black-and-white case. Sometimes, you need to have the latest dev release for one of our package to be able to publish the documentation. Sometimes, it doesn't matter if it is dev version or full release, and sometimes, we should use the full release.
I don't think we can make a definite rule, as it will almost always depend on the case.

Perhaps this channel was set up at some point to get a dev release of nbsite and then forgotten

This only argues for a tooling-only channel, not the removal of nbsite for the existing channels, IMO.

@hoxbro are you okay with a tooling_dev channel only?


Clue Update 2019

Here are all the clues that the current system supports that we need to support

[x] Anagrams:

[x] Beginner:

[x] The anagram reveals who to speak to next: AN EARL
[x] The anagram reveals who to speak to next: CARPET AHOY
[x] The anagram reveals who to speak to next: DISORDER
[x] The anagram reveals who to speak to next: I CORD
[x] The anagram reveals who to speak to next: IN BAR
[x] The anagram reveals who to speak to next: RAIN COVE
[x] The anagram reveals who to speak to next: RUG DETER
[x] The anagram reveals who to speak to next: SIR SHARE RED
[x] The anagram reveals who to speak to next: TAUNT ROOF

[x] Medium:

[x] This anagram reveals who to speak to next: CALAMARI MADE MUD
[x] This anagram reveals who to speak to next: CLASH ION

[x] Master:

[x] This anagram reveals who to speak to next: A ELF KNOWS
[x] This anagram reveals who to speak to next: RIP MAUL

[x] Cipher

[x] Hard:

[x] The cipher reveals who to speak to next: BSOPME MZETQPS
[x] The cipher reveals who to speak to next: IWPPLQTP

[x] Coordinates:

[x] Medium:

[x] 02.16N 12.07E
[x] 17.39N 37.16W
[x] 23.01N 41.33E

[x] Hard:

[x] 16.45N 03.05W
[x] 19.09N 21.58E
[x] 24.22N 17.15E

[x] Elite:

[x] 17.58N 19.05E
[x] 20.24N 03.52W

[x] Master:

[x] 18.26N 37.15W

[x] Cryptic:

[x] Beginner:

[x] Always walking around the castle grounds and somehow knows everyone's age.
[x] In a village of barbarians, I am the one who guards the village from up high.
[x] In the place Duke Horacio calls home, talk to a man with a hat dropped by goblins.
[x] Near the open desert I reside, to get past me you must abide.

[x] Easy:

[x] Search the chest in Fred the Farmer's bedroom.
[x] Search the eastern bookcase in Father Urhney's house.
[x] Search the boxes next to a chest that needs a crystal key.
[x] Search the crates in Falador General store.
[x] Search the crate in Rommiks crafting shop in Rimmington.
[x] Search the single crate in Horvik's smithy in Varrock.
[x] Talk to Ali the Leaflet Dropper north of the Al Kharid mine.
[x] Talk to Charles at Port Piscarilius.
[x] Talk to the cook in the Blue Moon Inn in Varrock.
[x] Talk to Morgan in his house at Draynor Village.
[x] Talk to Turael in Burthorpe.
[x] Talk to Wayne at Wayne's Chains in Falador.

[x] Elite:

[x] More resources than I can handle, but in a very dangerous area. Can't wait to strike gold!
[x] Observing someone in a swamp, under the telescope lies treasure.

[x] Emotes:

[x] Beginner:

[x] Blow a raspberry at Gypsy Aris in her tent. Equip a gold ring and a gold necklace.
[x] Bow to Brugsen Bursen at the Grand Exchange.
[x] Spin at Flynn's Mace Shop.
[x] Panic at Al Kharid mine.
[x] Cheer at Iffie Nitter. Equip a chef hat and a red cape.
[x] Clap at Bob's Brilliant Axes. Equip a bronze axe and leather boots.

[x] Hot/Cold:

[x] Inside the Ruins north of the Graveyard of Shadows, level 27 Wilderness.
[x] North of Venenatis, level 32 Wilderness

In addition, the following new types of clues were added, and could do with support:

[ ] Beginner Hot/Cold
Base Clue: Buried beneath the ground, who knows where it's found. Lucky for you, a man called Reldo may have a clue.
Has 5 locations it can be in

[x] Music Clues

[x] Beginner Map Clues
These work like no other maps clues in the game, using 1 ItemID for all 5 map types. As a result they can't be added easily to the current maps

[x] Charlie Tasks
Similar to Sherlock Challenges
Consists of 3 parts:
A Clue telling the player to go to Charlie
A Challenge telling the player what to do
A Challenge telling the player to return to Charlie with the item
Current known challenges:

		I need to cook Charlie a pike.
		I need to take the cooked pike to Charlie.

		I need to fish Charlie a herring.
		I need to take a raw herring to Charlie.

		I need to fish Charlie a trout.
		I need to take a raw trout to Charlie.

		I need to cook Charlie a trout.
		I need to take the cooked trout to Charlie.

		I need to smith Charlie one iron dagger.
		I need to take the iron dagger I smithed to Charlie.

		I need to take the iron ore to Charlie.
		I need to mine Charlie a piece of iron ore from an iron vein.

		I need to craft Charlie a leather body.
		I need to take the leather body I crafted to Charlie.

		I need to craft Charlie some leather chaps.
		I need to take the leather chaps I crafted to Charlie.

I'll look at taking a crack at abstracting the master clue hot/cold solver to apply to these new hot/cold beginner clues, and implementing the beginner map clue solver for next week.

With #8607 merged, the 2 new hot/cold clue locations (that aren't beginner) are done.

Charlie challenge red chat notification text (note the run-on sentence is accurate currently in game at least for this specific clue):

You have completed Charlie's task, go back and give him the Pike.

Charlie challenge changed clue text after the challenge is complete:

I need to take the cooked pike to Charlie.

Currently, reading a clue after accepting or completing Charlie's challenge does not create a clue overlay.

That's expected, since we don't have code which handles Charlie (or Sherlock, for that matter) clues. We're hoping to have that implemented in the near future.

Are you referring to public class ThreeStepCrypticClue that lacks a Set<> of actual clue info? I didn't see a trivial way to pull request the info, then found this issue with search. My Java skills are mediocre (i.e. borrowed from C#), so it's probably best I don't try to finish that class if anyone else has plans to do it.

ThreeStepCrypticClue has nothing to do with Sherlock or Charlie. I've got a PR for it 95% complete however


Remove some standard columns and re-order others in export

Current order:

Sheet ID
Name (of Design)
Description (of Design)
Sheet Creation Date
Project
Site
Subject
Acrostic
Status
Creator
Event Name

Proposed new order (off the top of my head):
Note: Status has been deleted in the interim.
Delete: Description, Project

Subject
Acrostic
Site
Name (of Design)
Event Name
Sheet ID
Creator
Sheet Creation Date

I think that's a fairly good order. The only change I'd suggest is removing creator from some exports. The only time I could envision that field being used is to double check all the data entry by someone that we knew messed everything up (and even then it wouldn't track fixes by last_updated_by). I think we can safely leave it out of the SAS/R exports (and potentially others).
Side note: It'd be nice one day be able to get exports with 1 row/subject-event, rather than 1 row/subject-design, a la REDCap exporting

Nice, thanks for the feedback, we'll try this for a list:

Subject
Acrostic
Site
Name (of Design)
Event Name
Sheet ID
Creator
Sheet Creation Date


bcfishpass.falls_upstr_anadromous_vw - include current CABD barrier status

Currently 406 records with >= 5 salmon observations upstream, but only 37 of these are tagged as barriers - it would be better if this was available directly in the view.
Less important but also potentially useful would be including all species (particularly BT) in this summary, just for easier identification of CABD location / FWA stream matching issues.

Also include the user barrier status, otherwise things get really confusing - Elk Falls (in Campbell River) has 10 upstream salmon observations since 1990 but is pinned as a barrier by user_barriers_definite_control.

Identifying records where the barrier status in the fix table overrides salmon observations upstream is straightforward with a query to the db, it could be added to the view... but usage is rather opaque - it is not possible to tell which records these are directly in the fix table.
Before completing this view, reworking of the falls/CABD fix tables would be helpful (and a cleanup is required anyway).
This requires a larger discussion - there is a lot of QA data in user_barriers_definite_control that seems to now be redundant - but it is hard to tell. Do these records need to be retained to to track some of what has been reviewed?


Configure remote naming

Some straight recipes require a particular naming convention for the remotes. So it would be good if that was configurable. See original problem here:
https://github.com/raxod502/straight.el/issues/128#issuecomment-324214018

Probably this will look like :remote keys at the top level and in the :upstream sub-plist. It should be pretty easy to do, just find all the places where straight-vc-git-primary-remote and straight-vc-git-upstream-remote are used, and have them try those values from the recipe before defaulting back to the variables.

It looks like 6abe699e1ecf9431e65f044f1ce07f0ab0ca2247 took care of this.
Please reopen if I'm mistaken.


Deleted Results Are Returned Next Search

I've noticed that when I delete results (that are not of interest) the next time the search executes all of the results that I just deleted are returned as new results again.
Maybe this is a misunderstanding of how Scumblr works.  Should repeated results get returned after a delete?  Is there a way where I can remove unwanted results and not have them reappear with the next search?

Hi @jwilczek,
This would be expected behavior. Search providers look for results and add them to the database if they don't already exist. So if you delete them they will just be added against since they don't exist anymore.
Usually the best way to deal with unwanted results is to set them to a closed status (you can create a specific status like "False positive" or "Not interesting" if you want so you can track how many irrelevant results are being returned by search search provider).
This way the results won't show up in the default search (unless you have checked "Include closed results") and they won't reappear when the search provider is rerun.
If this doesn't work for you reopen the issue, let me know what's missing, and I'll see if I can assist further.

Thanks!  I thought that may be the case.


Node v6 image is being shipped with unsupported pm2 version

The latest node v6 image ships with a version of pm2 that no longer supports node v6. Apps fail to start if I try using this image. The problem is that the latest version of pm2 is always being installed when the image is built instead of a node v6 compatible version.
https://hub.docker.com/layers/keymetrics/pm2/6-alpine/images/sha256-fe5adedd68a040154db0689357b5abda3b28cd5851a9b8b620045af487f207a7

https://github.com/keymetrics/docker-pm2/commit/f4af754d946b6f811a468d3d2741c3848f1964ef


xy stage limits- hitting sides

-on start the xy stage hits side if last closed in a different location
-if input a number larger than max x or y the stage hits side

I owe Anna the mechanical limits.   Maybe we can both measure ours and report them here?
What is the maximum value for XYZ at the far limit on Proto 1 and proto 4.

Kevin found the max range of X = 122 mm, the Y is 81 mm.
Let's simplify those ranges to:
Ranges: | mm | Microsteps
X axis (mm) | 120 | 2418898
Y axis (mm) | 80 | 1612598
Z axis (mm) | 15 | 2560000
I would also like to add 0.5mm to the XY and 0.25 mm to the Z as the initial value.
0.5mm on XY is 10079 microsteps.
0.25 mm on Z is 42667 microsteps
So the usable range of X would be 10079 to 2428976
Usable range of Y would be 10079 to 1622677
Usable range of Z would be 42667 to 2602667
(This would all be a lot simpler if we could keep these in mm/um...).
We may want to refine these numbers in the future so would be best if these could be in an ini file.

On our prototype, after homing. I identified the distance needed to un-set the X and Y motor left limit switches.
The GUI indicates 0.06mm for both X and Y
This was determined by observing the limit switch LED's on the board achieve full brightness after successive single clicks of the button as shown. They are not always fully dim after homing. (mechanical flag position ? need pull-up/down resistor?)

The limit switches will have Schmidt trigger on the final board, so they will be digital on/off on the real board.  Right now there is some fuzziness on when it triggers.  The XY is switching from optical to Hall effect switches which are also digital. But the Turret + Z will have optical, so the Schmidt triggers will help with this to sharpen the response.

Limits set.


CGEMM & ZGEMM using C code, Power only, P10 only.

We originally uploaded this nearly a year ago, but discovered a bug in the XL compiler.
We waited for the compiler to be fixed for AIX, and it is now fixed in version XL <IP_ADDRESS>.
We are now ready to upload this code again!

Great, thanks.

That one failure looks pretty far away from what I'm changing...

sure, I just haven't gotten round to helping osx/homebrew find its android-ndk package again


Support format argument capture

Basically, defmt::info!("{x}"); should be equivalent to defmt::info!("{x}", x = x);.
The upstream feature is: https://github.com/rust-lang/rust/issues/67984

I like that proposal!


[Bug] Renaming or removing a contracted model should raise a BreakingChange warning/error

Is this a new bug in dbt-core?

[X] I believe this is a new bug in dbt-core
[X] I have searched the existing issues, and I could not find an existing issue for this bug

Current Behavior
When a contracted model changes or disables its contract, dbt raises a warning (UnversionedBreakingChange) or error (ContractBreakingChangeError) within the same_contract check triggered by state:modified selection/comparison.
However, the same warning/error does not occur if the user simply deletes or renames a contracted model. I believe it should — until/unless unless that contracted model has passed a defined deprecation_date, at which point deletion is allowed.
Expected Behavior
If a contracted model is present in the comparison manifest, and missing in the current manifest, raise a warning (if unversioned) or error (if versioned) accordingly. Add a new type of "breaking change" for "renamed or removed."
Naïve implementation for illustrative purposes only:
class StateSelectorMethod(SelectorMethod):
    ...
    def check_for_deleted_contracted_models(self) -> None:
        old_contracted_nodes = set(
            k for k, v in self.previous_state.manifest.nodes.items() if v.config.contract.enforced
        )
        new_contracted_nodes = set(
            k for k, v in self.manifest.nodes.items() if v.config.contract.enforced
        )
        for unique_id in old_contracted_nodes - new_contracted_nodes:
            node = self.previous_state.manifest.nodes[unique_id]
            if (
                node.deprecation_date
                and node.deprecation_date < datetime.datetime.now().astimezone()
            ):
                # Passed its deprecation_date, so deletion is allowed
                continue
            if node.version is None:
                print("WARNING! Unversioned contracted model renamed or removed")
                # This should be warn_or_error(UnversionedBreakingChange), and we probably want a new "breaking_change" reason
            else:
                print("ERROR! Versioned contracted model renamed or removed")
                # This should raise ContractBreakingChangeError
    ...
    def search(self, included_nodes: Set[UniqueId], selector: str) -> Iterator[UniqueId]:
        if self.previous_state is None or self.previous_state.manifest is None:
            raise DbtRuntimeError("Got a state selector method, but no comparison manifest")
        self.check_for_deleted_contracted_models()
        ...

Steps To Reproduce

Create a model configured with contract: {enforced: true}
Parse the project / run the model
Move the manifest to a folder named state/
Delete or rename the model
dbt list -s state:modified --state
No warning or error

Relevant log output
No response
Environment
- OS: macOS
- Python: 3.10.11
- dbt: v1.8 / main

Which database adapter are you using with dbt?
No response
Additional Context
No response

@MichelleArk link an example of the current implementation for similar errors in the contracted model.

The existing call sites for UnversionedBreakingChange and ContractBreakingChangeError are in same_contents:
https://github.com/dbt-labs/dbt-core/blob/03b621f390a24a0de3fcd3e3ef98acabbb96ea06/core/dbt/contracts/graph/nodes.py#L573-L755
However, in the case of a model that's been renamed/removed, we can't rely on same_contents, because that is called once for each node within the current project.
Acceptance criteria

Add new attribute (renamed_or_removed: bool) to UnversionedBreakingChange
Add check for old nodes missing in current manifest to state:modified (naïve example above). Depending on whether the node is versioned, and whether it's passed its deprecation_date, fire UnversionedBreakingChange event or raise ContractBreakingChangeError error. These should include "The model has been renamed or removed" within their list of breaking_changes.
Add tests

Opened a new issue in dbt-labs/docs.getdbt.com: https://github.com/dbt-labs/docs.getdbt.com/issues/5574


Making sure generated performance counter names are correct

this fixes #2905
flyby: fixing lock held during suspension problem in resource_partitioner

@zbyerly please verify whether this fixes your issue.

This fixes the issue.


Cant load custom fonts with nextJS

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯
I have been trying to load custom fonts in MUI for my nextJS project. But it errors out. According to the docs we need a loader for it parse custom fonts but I haven't been able to find any information related to that. How should I configure Next and MUI to load the fonts?
The only workaround I have been able to find is to define the font family through a global css file.  But then the fonts don't work inside components.
Here's my code for theme.ts
import { createTheme } from '@mui/material';
import proximaRegular from './public/fonts/Proxima-Nova.ttf';
import proximaBlack from './public/fonts/Proxima-Nova-Black.otf';
import proximaSBold from './public/fonts/Proxima-Nova-SBold.ttf';

const theme = createTheme({
  typography: {
    fontFamily: 'ProximaRegular, ProximaBlack, ProximaSBold, Arial, Roboto',
  },
  palette: {
    background: {
      default: '#FFD600',
    },
  },
  components: {
    MuiCssBaseline: {
      styleOverrides: `
              @font-face {
          font-family: 'ProximaRegular';
          font-style: normal;
          font-display: swap;
          font-weight: 400;
          src: url(${proximaRegular}) format('ttf');
          unicodeRange: U+0000-00FF, U+0131, U+0152-0153, U+02BB-02BC, U+02C6, U+02DA, U+02DC, U+2000-206F, U+2074, U+20AC, U+2122, U+2191, U+2193, U+2212, U+2215, U+FEFF;
        }

        @font-face {
          font-family: 'ProximaBlack';
          font-style: normal;
          font-display: swap;
          font-weight: 400;
          src: url(${proximaBlack}) format('otf');
          unicodeRange: U+0000-00FF, U+0131, U+0152-0153, U+02BB-02BC, U+02C6, U+02DA, U+02DC, U+2000-206F, U+2074, U+20AC, U+2122, U+2191, U+2193, U+2212, U+2215, U+FEFF;
        }

        @font-face {
          font-family: 'ProximaSBold';
          font-style: normal;
          font-display: swap;
          font-weight: 400;
          src: url(${proximaSBold}) format('ttf');
          unicodeRange: U+0000-00FF, U+0131, U+0152-0153, U+02BB-02BC, U+02C6, U+02DA, U+02DC, U+2000-206F, U+2074, U+20AC, U+2122, U+2191, U+2193, U+2212, U+2215, U+FEFF;
        }
      `,
    },
  },
});

export default theme;

and here's the error I'm getting during the process
error - ./public/fonts/Proxima-Nova-Black.otf
Module parse failed: Unexpected character '' (1:4)
You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders
(Source code omitted for this binary file)

Expected behavior 🤔
The custom fonts should be loaded
Steps to reproduce 🕹
Steps:

import the custom fonts inside theme.ts
Create the appropriate style Overrides for them
Build the app.

Context 🔦
No response
Your environment 🌎

`npx @mui/envinfo`
System:
    OS: Windows 10 10.0.19042
  Binaries:
    Node: 14.17.0 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.11 - ~\AppData\Roaming\npm\yarn.CMD
    npm: 6.14.13 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Chrome: Not Found
    Edge: Spartan (44.19041.1023.0), Chromium (91.0.864.67)
  npmPackages:
    @emotion/react: ^11.7.0 => 11.7.0
    @emotion/styled: ^11.6.0 => 11.6.0
    @mui/base:  5.0.0-alpha.59
    @mui/material: ^5.2.3 => 5.2.3
    @mui/private-theming:  5.2.3
    @mui/styled-engine:  5.2.0
    @mui/system:  5.2.3
    @mui/types:  7.1.0
    @mui/utils:  5.2.3
    @types/react: ^17.0.37 => 17.0.37
    react: 17.0.2 => 17.0.2
    react-dom: 17.0.2 => 17.0.2
    typescript: ^4.5.2 => 4.5.2

I'm using Chrome v96

This should help - https://stackoverflow.com/questions/60134693/why-is-my-local-font-not-being-applied-with-emotion-global-styling You can define the font face using link and reference it then in the global styles. If I need to guess the URLs are not correct.

I tried defining them into the global css file and then had MUI try to access them inside typography.fontFamily but it doesn't seem to work. As when I check the fonts they aren't applied

I figured out the problem, I was defining the formats of the font incorrectly, which was why they weren't loading. For others here's how this'll work
in your global css define the fonts
@font-face {
  font-family: 'ProximaRegular';
  font-style: normal;
  font-display: swap;
  font-weight: 400;
  src: url('/fonts/Proxima-Nova.ttf') format('truetype');
  unicoderange: U+0000-00FF, U+0131, U+0152-0153, U+02BB-02BC, U+02C6, U+02DA,
    U+02DC, U+2000-206F, U+2074, U+20AC, U+2122, U+2191, U+2193, U+2212, U+2215,
    U+FEFF;
}

@font-face {
  font-family: 'ProximaBlack';
  font-style: normal;
  font-display: swap;
  font-weight: 400;
  src: url('/fonts/Proxima-Npva-Black.otf') format('opentype');
  unicoderange: U+0000-00FF, U+0131, U+0152-0153, U+02BB-02BC, U+02C6, U+02DA,
    U+02DC, U+2000-206F, U+2074, U+20AC, U+2122, U+2191, U+2193, U+2212, U+2215,
    U+FEFF;
}

@font-face {
  font-family: 'ProximaSBold';
  font-style: normal;
  font-display: swap;
  font-weight: 400;
  src: url('/fonts/Proxima-Nova-SBold.ttf') format('truetype');
  unicoderange: U+0000-00FF, U+0131, U+0152-0153, U+02BB-02BC, U+02C6, U+02DA,
    U+02DC, U+2000-206F, U+2074, U+20AC, U+2122, U+2191, U+2193, U+2212, U+2215,
    U+FEFF;
}

and then refer them inside your theme object as how you do for regular fonts
Thanks

@Altair2169 one note though, there is no such thing as unicoderange, try unicode-range, also I don't think you should every font as font-weight: 400, it could break some third party library if you'd use it


Popup window not recognized as popup on X11

Godot version
4.0 beta10
System information
Linux
Issue description
This is a problem of the Godot editor.
_NET_WM_WINDOW_TYPE of all windows is _NET_WM_WINDOW_TYPE_NORMAL. When using a tiling window manager, the WM tries to tile every popup as well.
It should be _NET_WM_WINDOW_TYPE_DIALOG.
Steps to reproduce

Open any Godot popup, like "Save or Discard"
use xprop and click on the popup
check _NET_WM_WINDOW_TYPE(ATOM) in xprop's output

Minimal reproduction project
N/A

Duplicate of https://github.com/godotengine/godot/issues/55415?

duplicate


Is this Q*?

Hi,
Is this basically Q*?

Yes. But please don't tell anyone.

Haha 🤣 - but are the principles similar?

We don't know what Q* really is, but it seems to be related to sequential decision making, which is also what Pearl and RL in general do. But this is just a vague, abstract similarity.

@fakerybakery thanks for reaching out! Like what @rodrigodesalvobraz said, we don’t exactly know what Q* is but I would suspect it is a combination of multiple RL features, likely some model free + model based learning approach. We have most of the popular model-free support on a lot of different dimensions of an RL agent and we are working towards supporting the same for model-based approaches.
Pearl’s capability of combining multiple RL features/capabilities in a single agent will help build intelligent agents that are more capable than  what they are able to do today. We hope that Pearl will incubate more powerful AI agents than these built by the mysterious Q* algorithm.

Thank you for the detailed explanation!


Add emphasis on not using non-stock emulator images

Attempting to make the note about incompatibility with non-stock emulator images (such as Oreo at time of writing), a bit more prominent as there seems to be a few reported issues around that.
I know it's stated in the How it works part but not sure if it's enough.

Addresses #71

Looks good, thanks for helping with this!


BioClip answer without relevant score

There is not tag in the reply
https://bsky.app/profile/botensky.bsky.social/post/3ladnqmobdl2c
Maybe could we add some plugins related tags

no keep negative case without tags is ok


Can't put text at bottom of the card

When creating a card for teams channel using the legacy documentation I am unable to get the text to show at the bottom of the card.
If I put the text at the end of section. It shows up before the facts.
If I add the text to the card it shows up before the section.
How do you get it to show bellow the section?

Can you share your card JSON?

{
"@type": "MessageCard",
"summary" : "error",
"title": "HUB Error",
"sections": [
{
"facts": [
{
"name": "URL",
"value": "https://test.com"
}
],
"text": "sample text"
}
]
}
and I've tried
{
"@type": "MessageCard",
"summary" : "error",
"title": "HUB Error",
"sections": [
{
"facts": [
{
"name": "URL",
"value": "https://test.com"
}
]
],
"text" : "sample text"
}

You need your text in a separate section, like so:
{
  "@type": "MessageCard",
  "summary": "error",
  "title": "HUB Error",
  "sections": [
    {
      "facts": [
        {
          "name": "URL",
          "value": "https://test.com"
        }
      ]
    },
    {
      "text": "sample text"
    }
  ]
}


npm ERR! ERESOLVE unable to resolve dependency tree (Svelte 3.44.0)

First: thank you so much for making this library! It seems like it will fill my exact current need. However, I am having trouble installing it...
This issue popped up before (https://github.com/hmsk/vite-plugin-markdown/issues/324) but seems to be a problem again.
When I try to install, I receive the following error:
npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/vite
npm ERR!   dev<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>|| ^3.0.0" from<EMAIL_ADDRESS>npm ERR! node_modules/vite-plugin-markdown
npm ERR!   dev vite-plugin-markdown@"*" from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.
npm ERR!
npm ERR! See /Users/stephennixon/.npm/eresolve-report.txt for a full report.

npm ERR!      /Users/stephennixon/.npm/_logs/2022-09-03T20_29_59_223Z-debug-0.log

Is there a suggested workaround for this?

It seems that adding the --force flag to npm install will download things. Getting it to work, however, is a little challenging... so I can’t say for sure whether forcing it is a good or bad idea yet.

SvelteKit is now using<EMAIL_ADDRESS>so should work


Pylint only checks first level of sub-directories

In .travis.yml the pylint call - pylint */*.py only checks a few directories. A call of pylint experiment_1 tests oemoflex exposes way more issues, most of all a lot of R0801: Similar lines in x files.

Duplicate of #51


Please consider fixing issues detected by repository checker

Notification from ioBroker Check and Service Bot
Dear adapter developer,
I'm the ioBroker Check and Service Bot. I'm an automated tool processing routine tasks for the ioBroker infrastructure. I have recently checked the repository for your adapter kostal-piko-ba for common errors and appropiate suggestions to keep this adapter up to date.
This check is based the current head revisions (master / main  branch) of the adapter repository
Please see the result of the check below.
ioBroker.kostal-piko-ba
   -
ERRORS:

[x] :heavy_exclamation_mark: [E507] missing size attributes [xs,sm,md,lg,xl] for checkbox at admin/jsonConfig.json/items/_AnalogTab/readanalogs

WARNINGS:

[ ] :eyes: [W522] Why did you decide not to use i18n support?

:thumbsup: No suggestions found
Please review issues reported and consider fixing them as soon as appropiate.
Errors reported by repository checker should be fixed as soon as possible. Some of them require a new release to be considered as fixed. Please note that errors reported by checker might be considered as blocking point for future updates at stable repository.
Warnings reported by repository checker should be reviewed. While some warnings can be ignored due to good reasons or a dedicated decision of the developer, most warnings should be fixed as soon as appropiate.
You may start a new check or force the creation of a new issue at any time by adding the following comment to this issue:
@iobroker-bot recheck
or
@iobroker-bot recreate
Please note that I (and the server at GitHub) have always plenty of work to do. So it may last up to 30 minutes until you see a reaction. I will drop a comment here as soon as I start processing.
Feel free to contact me (@iobroker-bot) if you have any questions or feel that an issue is incorrectly flagged.
And THANKS A LOT for maintaining this adapter from me and all users.
Let's work together for the best user experience.
your
ioBroker Check and Service Bot
@mcm1957 for evidence
Last update at Fri, 22 Nov 2024 04:01:29 GMT based on commit 46c313041fdcc62e4ec8963a80cfe1e6c38b5364
ioBroker.repochecker 3.2.2

@iobroker-bot recreate


First example throws error

Hi, I was just trying the first example you have and I get an error:
HTTPError: 400 Client Error: Bad Request for url: https://api.eia.gov/v2/natural-gas/pri/fut/data/?api_key=c4wW96H5wHjJf0C9RKda6nseO5gjZV8VqHz8zdzB&data[]=value&frequency=daily&start=1963-09-23&end=2073-08-26&facets[None][]=RNGC1&sort[0][column]=period&sort[0][direction]=desc
{"error":"Invalid facet 'None' provided. The only valid facets are 'duoarea', 'product', 'process', and 'series'.","code":400}.
The second works fine. Just FYI. Cheers.

Your documentation is also out of date, start parameter is now start_date etc.

seriesId -> series

Thank you for noticing! Fix it now in 0.3.5.
pip install --upgrade myeia

No problem, cheers!


The lowering of the thresholds creates a ton of false flags.

Spesifically it is the teleportation that misdetects the most of the time.
I think the current way of setting back the player is via teleportation. However this isn't ideal as the problem isn't how much they can change their position in one instant but how much total distance they can gain. The current method of distance gain detection relies on having the teleportation checks really low so it can limit the distance gained. However this is undesired and does cause a lot of false flags. For example 2 studs isn't really a lot and I wouldn't really consider it teleporting. How it should in my opinion work is that the teleportation distance would be risen to much higher and that there would be a check where it would check for example how much total distance is a player gaining and if it is too much teleport them back to a desired position.

The problem primarily with this is how to measure "gained distance." Without a reference point I have no way to measure the distance gained.
Say, for example, the threshold is raised to 5 studs per second. The player might teleport 5 studs, wait 1 second (~60 frames), teleport 5 studs again, etc, etc. How can I know when to assume the player is no longer "gaining" distance? I would have to mark some start position, currently this is per frame, and then I would have to decide when to unmark that position. There doesn't seem to me to be a good way to do this.
The best solution I can come up with is to give a time frame parameter, e.g. 5 seconds. I can then say that, if the player has moved a total of X studs more than expected in that time frame, I lag them back. This is probably what I will end up doing, but, it most likely won't solve the issue.
Additionally, can you describe to me exactly how these false positives are occurring for you, and what impact they are having? Is this occurring due to network lag, for example? Is it noticeable to you as a player, e.g. is the prediction off enough to be annoying? My concern is not around whether false positives exist, false positives will always exist and I expect them to be frequent. My concern is mostly around the impact this will have on players vs the freedom it will give exploiters. False positives are only a concern to me if they annoy or detriment the player in some way, this is a benefit of my methodology on how I create my anticheat.

I've raised the thresholds again based on testing, hopefully this should be better.
I will be implementing a better prediction system and modularizing the anticheat in the next major update.

The jumping was the thing that false flagged the most wa jumping.  I am not sure if they are fixed as I cannot test them right now.

Seems like this is fixed


Adds  option fetch_merge. Closes #28

Thanks for the PR. I'm going to add a test for the git failure.

@ckaznocha just a heads up, I made this a param on a get. This way a user has the option to pull in either type within the same pipeline.
Just a heads up, so you can update your pipelines.


[release-4.15] OCPBUGS-30465:  fix: bump google.golang.org/protobuf

What this PR does / why we need it:
bump google.golang.org/protobuf as versions less than 1.33.0 are vulnerable to CVE-2024-24786
See-also: CVE-2024-24786
Checklist

[x] Subject and description added to both, commit and PR.
[x] Relevant issues have been referenced.
[ ] This change includes docs.
[ ] This change includes unit tests.

/area ci-tooling

/jira refresh

/retest-required

/retest-required

/retest-required

/hold
Going to wait for #4625 to be merged first

/jira refresh

/jira refresh

/approve

/hold cancel

/lgtm

/retest-required


test: test directory added

test directory added

I  think you can make this PR redy for review now. :slightly_smiling_face:


Adobe Illustrator generated svg files are not getting sprited

Because of these attributes on the SVG element
xmlns:x="&ns_extend;" xmlns:i="&ns_ai;" xmlns:graph="&ns_graphs;"

when I try to run svg-sprite on files like this it fails with the following error
events.js:72
throw er; // Unhandled 'error' event
Error: Skipping "%s" (%s)
I guess the SAX parser in SVGO is not recognizing this as valid syntax. Any way to turn this strict mode off from gulp-svg-sprite?
My SVG file looks like this
<?xml version="1.0" encoding="utf-8"?>
<!-- Generator: Adobe Illustrator 16.0.3, SVG Export Plug-In . SVG Version: 6.00 Build 0)  -->
<!DOCTYPE svg PUBLIC "-//W3C//DTD SVG 1.1//EN" "http://www.w3.org/Graphics/SVG/1.1/DTD/svg11.dtd" [
	<!ENTITY ns_extend "http://ns.adobe.com/Extensibility/1.0/">
	<!ENTITY ns_ai "http://ns.adobe.com/AdobeIllustrator/10.0/">
	<!ENTITY ns_graphs "http://ns.adobe.com/Graphs/1.0/">
	<!ENTITY ns_vars "http://ns.adobe.com/Variables/1.0/">
	<!ENTITY ns_imrep "http://ns.adobe.com/ImageReplacement/1.0/">
	<!ENTITY ns_sfw "http://ns.adobe.com/SaveForWeb/1.0/">
	<!ENTITY ns_custom "http://ns.adobe.com/GenericCustomNamespace/1.0/">
	<!ENTITY ns_adobe_xpath "http://ns.adobe.com/XPath/1.0/">
]>
<svg version="1.1" id="Layer_1" xmlns:x="&ns_extend;" xmlns:i="&ns_ai;" xmlns:graph="&ns_graphs;" xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink" x="0px" y="0px" width="51px" height="51px"
	 viewBox="0 0 51 51" enable-background="new 0 0 51 51" xml:space="preserve">
<switch>
	<foreignObject requiredExtensions="&ns_ai;" x="0" y="0" width="1" height="1">
		<i:pgfRef  xlink:href="#adobe_illustrator_pgf">
		</i:pgfRef>
	</foreignObject>
	<g i:extraneous="self">
		<g>
			<g>
				<circle fill="#FFFFFF" cx="25.5" cy="25.5" r="25"/>
				<path fill="#D0D2D3" d="M25.5,51C11.4,51,0,39.6,0,25.5S11.4,0,25.5,0C39.6,0,51,11.4,51,25.5S39.6,51,25.5,51z M25.5,1
					C12,1,1,12,1,25.5C1,39,12,50,25.5,50S50,39,50,25.5C50,12,39,1,25.5,1z"/>
			</g>
		</g>
		<g>
			<rect x="11.4" y="10" fill="#00A6A0" width="6.7" height="28"/>
			<rect x="28.1" y="17.8" fill="#00A6A0" width="6.7" height="20.2"/>
			<rect x="18.1" y="22.3" fill="#6E6F71" width="6.7" height="15.7"/>
			<rect x="34.8" y="27.9" fill="#6E6F71" width="6.7" height="10.1"/>
		</g>
	</g>
</switch>
<i:pgf  id="adobe_illustrator_pgf">
	<![CDATA[
	... lot of data here ...
	]]>
</i:pgf>
</svg>

Hey @rathishc24,
that's a good question. In fact, I'm not too much into SVGO, but you should be able to pass any valid configuration to the SVGO process. Please see here for an example enabling / disabling particular SVGO plugins. svg-sprite will use the value of shape.transform.svgo (when given as an object) as argument for the SVGO() constructor, so if SVGO provides the option you need, you should be able to use it.
I'll close this issue for now, as it seems rather SVGO related, but please let me know if there's anything I can do from the svg-sprite side of things.
Cheers,
Joschi


Error of losing horizontal lines Led P2.5

Hi!
I performed test 2_PatternPlasma on led panel p.25 with horizontal line loss.
Chip: ICN2038S

Please help me to solve!

It looks like a hardware issue to me. Check the ABCD(E) inputs to the display PCB.

Yeah, looks like a B pin issue.


Feature/zephyr

Added Zephyr topology support to DWaveMockSampler which can be the basis for further development of code supporting forthcoming Zephyr features.

AttributeError: module 'dwave_networkx' has no attribute 'zephyr_graph'
is the cause of test failures. I think this is an issue with the required version for the repository.

@jackraymond, be sure to also update requirements in setup.py to dwave-networkx>=0.8.10, and in requirements.txt to dwave-networkx==0.8.10.

Codecov Report

Merging #432 (c506d05) into master (ddb219f) will decrease coverage by 3.03%.
The diff coverage is 73.68%.

@@            Coverage Diff             @@
##           master     #432      +/-   ##
==========================================
- Coverage   90.54%   87.51%   -3.04%
==========================================
  Files          22       22
  Lines        1492     1498       +6
==========================================
- Hits         1351     1311      -40
- Misses        141      187      +46

Impacted Files
Coverage Δ

dwave/system/testing.py
94.82% <73.68%> (-1.54%)
:arrow_down:

dwave/system/samplers/leap_hybrid_sampler.py
61.72% <0.00%> (-13.88%)
:arrow_down:

dwave/system/samplers/clique.py
77.35% <0.00%> (-5.04%)
:arrow_down:

dwave/system/samplers/dwave_sampler.py
84.47% <0.00%> (-3.11%)
:arrow_down:

dwave/system/composites/embedding.py
95.85% <0.00%> (-1.19%)
:arrow_down:

Continue to review full report at Codecov.

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


Bitalino Devices support

Hello,
Is there any future support for Bitalino planned?
Thanks
Luca

Dear Luca/@ilpolemico,
Thank you very much for your pertinent question, the PLUX team strongly hopes that everything is well with you.
Regarding the support for BITalino devices, fortunately, despite not explicitly declared in the repository, the Unity API should already support the interaction with the major functionalities provided by BITalino, namely, starting/stopping a real-time acquisition and receiving the collected data samples.
We sincerely wish that the previous information can be helpful, however, if you don't consider it totally adequate or if any further questions arise, please, do not hesitate to alert us.
In the meantime, I am pleased to wish you a continuation of an excellent day!
My best and sincere regards,
@GRamosPlux


Error running one diagnostic from schlund20jgr/recipe_schlund20jgr_gpp_change_rcp85.yml

@schlunma @sloosvel I gets this in stdout:
esmvalcore._task.DiagnosticError: Diagnostic script mlr/postprocess.py failed with return code -9. See the log in /scratch/b/b382109/esmvaltool_output/recipe_schlund20jgr_gpp_change_rcp85_20221011_110207/run/diag_mmm/postprocess/log.txt
and in log:
run_dir: /scratch/b/b382109/esmvaltool_output/recipe_schlund20jgr_gpp_change_rcp85_20221011_110207/run/diag_mmm/postprocess
script: postprocess
themes:
- ML
- EC
- carbon
version: 2.7.0rc2
work_dir: /scratch/b/b382109/esmvaltool_output/recipe_schlund20jgr_gpp_change_rcp85_20221011_110207/work/diag_mmm/postprocess

2022-10-11 11:04:56,258 [344025] INFO     esmvaltool.diag_scripts.shared._base,550      Creating /scratch/b/b382109/esmvaltool_output/recipe_schlund20jgr_gpp_change_rcp85_20221011_110207/work/diag_mmm/postprocess
2022-10-11 11:04:56,258 [344025] INFO     esmvaltool.diag_scripts.shared._base,550      Creating /scratch/b/b382109/esmvaltool_output/recipe_schlund20jgr_gpp_change_rcp85_20221011_110207/plots/diag_mmm/postprocess
2022-10-11 11:04:56,319 [344025] INFO     __init__.py,532       Ignoring files with [{'prediction_name': None}, {'var_type': None}]
2022-10-11 11:04:56,319 [344025] INFO     postprocess.py,747    Processing tag 'GPP_change'
2022-10-11 11:04:56,319 [344025] INFO     postprocess.py,750    Processing prediction 'OBS'
2022-10-11 11:04:56,319 [344025] INFO     postprocess.py,686    Found mean prediction dataset ('prediction_output') for tag 'GPP_change' of prediction 'OBS': /scratch/b/b382109/esmvaltool_output/recipe_schlund20jgr_gpp_change_rcp85_20221011_110207/work/diag_mmm/mlr/mmm_GPP_change_prediction.nc (used as reference)
2022-10-11 11:04:56,319 [344025] INFO     postprocess.py,701    Found error datasets ('prediction_output_error') for tag 'GPP_change' of prediction 'OBS':
2022-10-11 11:04:56,319 [344025] INFO     postprocess.py,704    ['/scratch/b/b382109/esmvaltool_output/recipe_schlund20jgr_gpp_change_rcp85_20221011_110207/work/diag_mmm/mlr/mmm_GPP_change_squared_prediction_error.nc']
2022-10-11 11:04:56,319 [344025] INFO     postprocess.py,720    Found 'prediction_input' dataset for covariance structure estimation for tag 'GPP_change' of prediction 'OBS': /scratch/b/b382109/esmvaltool_output/recipe_schlund20jgr_gpp_change_rcp85_20221011_110207/work/diag_mte_for_cov_estimation/preprocess/OBS_MTE_reanaly_May12_Lmon_gpp_1982-2011_prediction_input.nc
2022-10-11 11:04:56,337 [344025] INFO     postprocess.py,623    Postprocessing mean prediction cube Carbon Mass Flux out of Atmosphere due to Gross Primary Production on Land (divided by ref) / (%) (latitude: 68; longitude: 180)
2022-10-11 11:04:56,355 [344025] INFO     esmvaltool.diag_scripts.shared.io,211 Wrote /scratch/b/b382109/esmvaltool_output/recipe_schlund20jgr_gpp_change_rcp85_20221011_110207/work/diag_mmm/postprocess/mmm_GPP_change_prediction.nc
2022-10-11 11:04:56,355 [344025] INFO     postprocess.py,629    Mean prediction: 0.5882820453391052 1
2022-10-11 11:04:56,358 [344025] INFO     postprocess.py,578    Postprocessing errors using mean prediction cube Carbon Mass Flux out of Atmosphere due to Gross Primary Production on Land (divided by ref) / (%) (latitude: 68; longitude: 180) as reference
2022-10-11 11:04:56,358 [344025] WARNING  postprocess.py,458    No covariance dataset (i.e. dataset with short_name == '*_cov' among 'prediction_output_error' datasets) found, calculation of real error not possible
2022-10-11 11:04:56,388 [344025] INFO     esmvaltool.diag_scripts.shared.io,211 Wrote /scratch/b/b382109/esmvaltool_output/recipe_schlund20jgr_gpp_change_rcp85_20221011_110207/work/diag_mmm/postprocess/mmm_GPP_change_prediction_error_lower_bound.nc
2022-10-11 11:04:56,388 [344025] INFO     postprocess.py,128    Lower bound of error: 2.601388776252018e-13 1
2022-10-11 11:04:56,403 [344025] INFO     esmvaltool.diag_scripts.shared.io,211 Wrote /scratch/b/b382109/esmvaltool_output/recipe_schlund20jgr_gpp_change_rcp85_20221011_110207/work/diag_mmm/postprocess/mmm_GPP_change_prediction_error_upper_bound.nc
2022-10-11 11:04:56,403 [344025] INFO     postprocess.py,164    Upper bound of error: 0.5943484535002844 1
2022-10-11 11:04:56,418 [344025] INFO     postprocess.py,237    Estimating true error from covariance derived from 'prediction_input' dataset with shape (360, 68, 180) for errors ('prediction_output_error') with shape (68, 180)
2022-10-11 11:04:56,440 [344025] INFO     postprocess.py,668    Reshaped 'prediction_input' for covariance estimation to (360, 68, 180)

On Levante; memory intake is astronomical at 57GB - could you pls tell me what's the problem here? Is it memory limit reached? (should say that is that's the case). Is code -9 error starvation? Cheers, guys :beer:

righto! Same code -9 for schlund20jgr/recipe_schlund20jgr_gpp_abs_rcp85.yml - starting to suspect serious memory starvation - please advise! By mother of Thor, what do these recipes do, simulate the entire visible Universe? :rofl:

Yeah, they use lots of memory. On Levante (at least 256GB per node) you should be fine with 2 parallel processes. The problem is here that the ML algorithms I'm using here are really optimized regarding their memory consumption 😬

OK thanks, dude, will try with 2 procs, and close this as soon as I run. Jeez, I wonder how bad the mem consumption is when they're not optimized :grinning:

Sorry, there is a "not" missing here, they are not optimized 😬

I had a hunch, I was being passive-aggressive towards Skynet :robot:


⚠ Cioos National CKAN has degraded performance

In da6379b, Cioos National CKAN (https://catalogue.cioos.ca/) experienced degraded performance:

HTTP code: 200
Response time: 23935 ms

Resolved: Cioos National CKAN performance has improved in c38e2d8 after 5 minutes.


fix: 忽略名称条件，只要cpu,mem一致就不再同步

这个 PR 实现什么功能/修复什么问题:
忽略名称条件，只要cpu,mem一致就不再同步
是否需要 backport 到之前的 release 分支:

release/2.12

/area region
/cc @swordqiu @yousong

Codecov Report

Merging #4004 into master will increase coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master   #4004      +/-   ##
=========================================
+ Coverage    6.93%   6.94%   +<.01%
=========================================
  Files         505     505
  Lines       85073   85073
=========================================
+ Hits         5904    5906       +2
+ Misses      78667   78666       -1
+ Partials      502     501       -1

Flag
Coverage Δ

#aFlag
6.94% <0%> (ø)
:arrow_up:

Impacted Files
Coverage Δ

pkg/compute/models/skus.go
1.31% <0%> (ø)
:arrow_up:

pkg/appsrv/response.go
69.23% <0%> (+3.07%)
:arrow_up:

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 f3b0c83...8ef397c. Read the comment docs.

/lgtm

/lgtm

/approve


perf(cache): optimizations for the new cache implementation

Summary:
Optimizations for the new cache implementation
Build & Testing:

[x] I ran cargo test successfully.
[x] I have run ./lint.sh --mode=fix to fix all linting issues raised by ./lint.sh --mode=check.

Checklist:

[ ] I have added relevant unit & integration tests.
[ ] I have updated the documentation accordingly.
[x] I have performed a self-review of my code.
[x] PR follows the naming convention of <type>(<optional scope>): <title>

Closing this PR for now. @shashitnak Please feel free to reopen when you think we have a clear idea of what optimizations are possible.


add zeta_config_table

This PR has the following changes:
1.Save the zeta gateway information in aca_zeta_programming::_zeta_config_table.
I use another table to store zeta gateway information, and store zeta_id in vpc_table. This can reduce the full table query required when generating group_id, but I am not sure if this is appropriate?
2. add the punt rule of the group table. E.g:
ovs-ofctl add-flow br-tun table=22,priority=50,dl_vlan=100,actions="strip_vlan,load:0x1->NXM_NX_TUN_ID[],group:100"
3. Only a simple test has been performed so far, there should still be bugs, and some necessary comments have not been added yet.

my PR #176 has been merged and I resolved the conflicts in your branch. Please go ahead to do git pull to get the latest change. The git automatic merge may not work perfectly, so please review the merge and clean up your code if needed. The goal is to get this change ready to merge this week. So please get yourself familiar with running aca_tests and confirm passing when you are ready: https://github.com/futurewei-cloud/alcor-control-agent/wiki/How-to-run-the-full-suite-of-aca_tests


[kn] Kannada New Language

[kn] Kannada New Language Translations Added

Will Raise new PR with PHPCS fix


Add administration

Specific users (e.g. users with a specific role) should be able to administrate the users and groups of OSIAM. This should be done with the integration of the addon-administration. The key features of the integrated administration should be:

Turn the administration on and off
Manage users (CRUD)
Manage groups (CRUD)

Some things we need to think about:

What about the Geb integration tests?
Should the HTML templates be customizable or static content like before?
Integration of the documentation (README and screenshots)

I think as a first version the code should not be refactored, just integrated.

The key features of the integrated administration should be:

Turn the administration on and off
Manage users (CRUD)
Manage groups (CRUD)

Agreed.

Some things we need to think about:

What about the Geb integration tests?

We should integrate them into the upcoming integration tests for OSIAM. I think this issue here is kind of blocked by that.

Should the HTML templates be customizable or static content like before?

I'm in favor for static templates.

Integration of the documentation (README and screenshots)

They should be moved to https://github.com/osiam/osiam/tree/master/docs.

I think as a first version the code should not be refactored, just integrated.

I totally agree.

I would schedule this for 3.1 and remove the integration of the registration feature from 3.1.

If someone add the administration to OSIAM, please be aware of the open issues:

https://github.com/osiam/addon-administration/issues/71
https://github.com/osiam/addon-administration/issues/69
https://github.com/osiam/addon-administration/issues/45