Spin up b

Include my branch including CABLE updates intended for spin up run

:rocket: The latest prerelease access-esm1p6/pr28-2 at 4dbe6eb2f1f1a4214eeaaf55908cbd6b563d4041 is here: https://github.com/ACCESS-NRI/ACCESS-ESM1.6/pull/28#issuecomment-2558683824 :rocket:

! redeploy

!redeploy

In these 2 simulations we use the same code base for the LSM/UM7. The script run uses Leuning, no soil thermal fix etc. An older cable.nml. The payu run updates all of these plus includes wombat-lite, generic tracers, Dave's iceberg fix. There is also a difference in netcdf versions, compiler versions. So on the one hand this comparison is like weather from 2 different models. It's a daily average screen Temp after a year.  On the other hand, such a large difference scares me that we need to do a payu/script comparrson. Which I really dont want to do. BUT I recall that NRI have done this an it was all acceptable. is this your recollection @rml599gh?

@JhanSrbinovsky - maybe let me know where the output is and I can have a look in the morning. Is this a single time? Looking at some of my runs, I am typically seeing +/- 15 degrees in the December monthly mean, so if your plot isn't a time average, then maybe the differences you are seeing are plausible.

daily averages. honestly though its the 1st of the three Tscrn fields in the .pe file.. I'm assuming it goes mean, max, min. Either way its mean-mean, max-max or min-min

On the Payu vs script run question – For the ESM1.5 release with payu we regularly checked whether the output from short PI simulations matched the PI-02 simulation on p73. I just checked my longer simulation of the NRI release, and into year 20 the two match:

I hadn't looked this far into the simulation before, but surprisingly it starts diverging, with what looks like noise appearing in May of the 20th year:

The difference then grows from here. It's surprising to me that the two diverged after being exactly the same for so long.

That is strange. It doesnt sound like a physical difference, pretending the simulation is physically real. I suppose you would know if the script model had been restarted or perturbed on restart. BUT the results of the tiny perturbation we sometimes applied  would've caused even greater divergence than you've shown here. I can vaguely make out coast lines there, suggesting it was a land thing. I dont really want to chase it. Almost pointless as the version of CABLE is different. BUT there was (perhaps still is) a dodgy variable in CABLE canopy. This was 10 years ago but I remember running the same single-site simulation back-to-back 10 times (or more) in order to follow it. Was your simulation running in monthly chunks, annual? It very well might have only been 1 timestep in the 20th year that started the divergence. It just snowballs after that.


OXV

Token Address https://tomoscan.io/token/0x0CA32BB134C8308ebaa86af5d1588229FA17C35A
Token Symbol
OXV
Token Name
OMIDXCIIVISION
Token Logo

Token Description
My vision of creating OXV is to support artists in every country on the planet. I also try to make OXV accepted as a common currency among artists.
Website
Omidxciivision.medium.com
Social links
https://twitter.com/OXVofficial
https://github.com/OMIDXCII
https://t.me/OXVofficial
https://www.reddit.com/u/OXVofficial

0x622b01adda23752d14151eabea7a320c91b97140a0472bc76a9f7db3b1a1b8d3

When you update my token logo on tomochain?

Due to a server error, I could not send the address in Hex data, do I have to pay again?

@thanhson1085 can you answer?

Please wait, we will process soon


0.7.1 incompatible with RN0.32.0

Hey,
react-native-maps on iOS does not compile with RN0.32.0 as the RCTImageLoader interface no longer has loadImageWithTag.
It is already fixed on master, I think we only need to publish a new version :)

+1 on this.  Thanks!

Another thing I notice after upgrading to RN32 is that
MapMarker
MapPolyline
MapPolygon
MapCircle
MapCallout
MapView
Are all requiring NativeMethodsMixin from react-native but it is not exposed anywhere so it resolves to undefined and react-native gives the following warning:
ExceptionsManager.js:78Warning: ReactClass: You're attempting to include a mixin that is either null or not an object. Check the mixins included by the component, as well as any mixins they include themselves. Expected object but got undefined.
From what I could see NativeMethodsMixin is not used in the code at all.
@lelandrichardson Am I right?
Anyway I think we should remove this

While you guys are working on publishing a new version, is there a workaround so I can get my project to build?  I can't roll back React-Native or I would.

@kcfgl I'm currently using my fork, you can use it too
in your package.json change:
"react-native-maps": "0.7.1" ->
"react-native-maps": "https://github.com/kfiroo/react-native-maps.git#1d1758fbf95c91e5b6e57d4dcac12344c2fc60a5"

The loadImageWithTag issue was fixed here: https://github.com/lelandrichardson/react-native-maps/pull/456
The NativeMethodsMixin issue was fixed here: https://github.com/lelandrichardson/react-native-maps/pull/511
I'll be releasing 0.8.0 soon.


Synchronize render with vsync

When I moved everything to FFI I implemented a very hacky frame timer that just waits for the given interval (e.g. 16ms) after processing all tasks on the queue.
This means the actual frame interval won't be correct if we're submitting a lot of jobs to the queue. We should have a nicer way of handling this.

Per this and this, the render loop now waits for a signal before requesting a frame.
When using Thermion via Flutter, ThermionWidget adds a frame callback to request a new frame, meaning the Filament scene will be rendered and the render target/texture updated before Flutter draws its new frame. This addresses the frame stutter.


Fix github workbook in the GH solution

Required items, please complete
Change(s):

Uses TimeRange parameter in the queries

Reason for Change(s):

The workbook has a TimeRange picker but the queries are not using it

Version Updated:

No Need

Testing Completed:

No

Checked that the validations are passing and have addressed any issues that are present:

Yes

@wanpeng-cds Please provide preview images. thanks!!!

This PR changes the value of Time Range in the red circle to 'TimeRange' which is the parameter defined previously.

@wanpeng-cds : We could see same workbook changes were there in another 5247 because of that we are closing this PR.


What is the purpose of the local symbol applicable accessibility ?

I can see in Roslyn that it corresponds to Accessibility.NotApplicable.  Is this the behaviour when omitting applicable_accessibilities ? If so should it be in the docs ?

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

ID: b402d2e0-a53a-b61a-4da8-b5ee766d9301
Version Independent ID: 612695df-5934-d967-d709-500ce02b1395
Content: .NET Naming Conventions For EditorConfig files - Visual Studio
Content Source: docs/ide/editorconfig-naming-conventions.md
Product: visual-studio-windows
GitHub Login: @gewarren
Microsoft Alias: gewarren

@sharwell Can you clarify what an applicable_accessibilities value of local should be used for in a naming style?

It's used for items defined within a method, where accessibility cannot be specified. For local variables and local functions, the new accessibility doesn't really make a difference, but for local constants it means users can separate the naming conventions of constants defined in a type from constants defined in a method.
class TypeName
{
  const int X = 3; // This constant is defined in the type

  void Method()
  {
    const int Y = 4; // This constant is defined in a method, with local accessibility
  }
}

I don't think that this is the best example of a use case for accessibility local as it is possible to differentiate using applicable symbol kinds instead.
It is important to clarify what symbols have local accessibility and under what circumstances  so that developers can target appropriately.  For instance the symbols for a local variable and local function are always local accessibility ( local function has private accessibility in Rolsyn but gets changed to local ).  Symbols for parameters and parameter types in the context of editor config do have accessibility levels that come from their containing symbol and as such can have local, public etc .  see here  https://github.com/dotnet/roslyn/blob/5b05de681c46ae58c9d5e05b85bd736c4986cd3c/src/Workspaces/Core/Portable/NamingStyles/Serialization/AccessibilityExtensions.cs#L25. This is contrary to what this document states '... For example, parameters do not have accessibility levels.'
The document is also incorrect in stating 'If you specify an accessibility level for a parameter naming convention, your naming rule will not function correctly.'  It is possible to apply one rule to parameters in a public method and a different rule to parameters in an anonymous function.
he document is incorrect here too: 'Do not specify an accessibility level as part of your naming convention if accessibility is not applicable to the kind of symbol you are targeting'.   Although this works as the default behaviour is * ( which is not mentioned in the documentation ) it is not necessary to do so.  It may also result in developers writing multiple rules that are not necessary.
Lastly developers might want to know that a local function symbol is static for purpose of modifier matching.  See here https://github.com/dotnet/roslyn/issues/27719

Lastly developers might want to know that a local function symbol is static for purpose of modifier matching.

This will have to change, as C# 8 introduces the notion of a static local function which is different from a non-static local function.

I've updated the doc and it will go live tomorrow morning. I used Sam's example of 'local' accessibility, but if you have a better example please let me know. I guess you are saying that a naming convention for a local constant could be specified in the following way, without an accessibility?
applicable_kinds = local
required_modifiers = const


uncaught at rootSaga   TypeError: (0 , _effects.all) is not a function

Am getting this in the browser console:
uncaught at rootSaga
 TypeError: (0 , _effects.all) is not a function

In thie tutorial...
https://redux-saga.js.org/docs/introduction/BeginnerTutorial.htm
'all' is defined as coming form the effects module:
import { put, takeEvery, all } from 'redux-saga/effects'
Is this correct?

What version of the redux-saga do u use? all was introduced in v0.15

Thank you - have upgraded and error now gone.


[TBDGen][Test] Check implied Obj-C symbol generation based on a stable swift version

I previously thought the coupled OBJC_CLASS | OBJC_METACLASS was specific to iOS, but it actually seems tied to compiling for deployment versions lower than when swift was shipped with darwin OSes. The test has been updated to use a recent-ish target triple across all CI configurations, where both symbols get emitted.

@swift-ci please test

@swift-ci please test

@swift-ci please test Linux


ExecutionActivity has negative duration?

News-MBP:.ideaflow janelle$ cat active_messages.log
ExternalActivity={"taskId":5,"durationInSeconds":41,"comment":null,"endTime":"2016-11-09T14:25:03"}
ExecutionActivity={"taskId":5,"durationInSeconds":-30220,"processName":"Plugin","exitCode":137,"executionTaskType":"Plugin","endTime":"2016-11-09T14:25:30","debug":false}

have not seen recently, possible it's fixed; re-open if it happens again


expand Gaia dataset to include full Gaia source tables and RVS spectra

now includes:

Gaia Source, full tables (1TB, 1.8B rows), primarily photometry and astrometry, but also some astrophysical parameters
XP coefficients, as before, but now with the complete tables including the full covariance matrices (2.6TB, 220M rows)
RVS spectra, (18GB, 1M rows), R~11,500 spectra

Test for this dataset passes but fails in the GH action because the runner is out of disk space. Would be nice if https://github.com/MultimodalUniverse/MultimodalUniverse/pull/57 was merged in so that only the test for this dataset runs.


examples/shader: Add an example for lighting with a normal map

I can probably do this if you want.
Do you have anything special in mind?
This is pretty cool:
https://blog.selfshadow.com/sandbox/normals.html

That'd be a great help! I don't have any special requests. Do we need handling multiple images?

I think multiple images is not required, but would be make the example a bit cooler.
I didn't know normal maps existed before today, so I'll take a while to familiarize myself with the concept :-)

First go

This is fantastic :-)
http://mattdesl.github.io/kami-demos/release/normals-pixel.html

https://gyazo.com/71694373d534a1e1e6e57a14197a3630
I found a tool (https://www.codeandweb.com/spriteilluminator) and succeeded to create a normal map.
@kyeett I'll be working on adding a new example with this. Sorry if you've already started, but is that fine to you?

@hajimehoshi no problem! Glad you got it to work!


How to use DragRotateAndZoom interaction of map?

Describe the bug
Upgrade to latest version of 0.1.39, and couldn't find any information about using DragRotateAndZoom interaction of map
To Reproduce
Just run npm run serve in project directory, then use shift key and drag to rotate, the rotation of map never change
Expected behavior
Could change the rotation of map by shift+drag

thanks for your request. I added  for DragRotateAndZoom interaction. Please update 0.1.40.
props
condition : A function that takes an module:ol/MapBrowserEvent~MapBrowserEvent and returns a boolean to indicate whether that event should be handled. Default is module:ol/events/condition.shiftKeyOnly.
you can get all conditions with
        const conditions = inject('ol-selectconditions')
        const pointerMove= conditions .pointerMove;

duration(Number): Animation duration in milliseconds.


How to change logo font

Excuse me. I'm not an expert of CSS and so on.
How can I change the font in the a class="logo", that is Chancery by default. Let's say I want to use Comfortaa https://fonts.google.com/specimen/Comfortaa
Thanks

The easy way:

Create a custom.css file under your site ./static/css/ folder.
Modify the custom.css file like this:

@import url(https://fonts.googleapis.com/css?family=Comfortaa);

.logo {
  font-family: Comfortaa !important;
}

Modify your site config file customCSS option : customCSS = ['custom.css']

Thanks.

IMPORTANT:  Make sure that you are using Hugo-extended since standard Hugo does not support SASS fully.  This took me a long file to figure out.


[NEW UI] Icon rotation direction

Questions
Answers

Description?
Fix icon log progress icon rotation animation on old BO themes

Type?
bug fix

BC breaks?
no

Deprecations?
no

Fixed ticket?
-

Sponsor company
@PrestaShopCorp

How to test?
Check the progress animation direction in Storybook; it should turn clockwise, following the direction of the arrow.

https://github.com/user-attachments/assets/62e04584-c810-4ab7-bc1e-929540c124bd

Checked on Storybook :heavy_check_mark:


emit psa_packet_paths

I think we said we aren't merging to ours, please leave a comment if you think its ok. Its just a copy and paste from simple switch, it will let us use the packet path values in bmv2. This is also blocking our bmv2 PR.

lgtm


Simplify workflow

The workflow can be generalized by adding the os to the matrix. Using the include entry we can specify which download link to use for each os, just like the compiler. This might look something like:
    strategy:
      matrix:
        compiler_c: [gcc, clang]
        os: [ubuntu-latest, macos-10.15]
        include:
          - compiler_c: gcc
            compiler_cpp: g++
          - compiler_c: clang
            compiler_cpp: clang++
          - os: ubuntu-latest
            download_url: <url>
          - os: macos-10.15
            download_url: <url>

After some more thought it seems like this would only complicate the workflows further, closing issue


Pick a bootstrap theme

@matenwego since I referenced this in my last commit it'll close once we merge dev into master right?


tools: ./tools/renumber-migrations to ask only 1 input question from user as the second one can be inferred from the first.

Refer: http://zulip.readthedocs.io/en/latest/schema-migrations.html

Numbering conflicts across branches: If you've done your schema change in a branch, and meanwhile another schema change has taken place, Django will now have two migrations with the same number. To fix this, you can either run ./tools/renumber-migrations which renumbers your migration(s) and fixes up the "dependencies" entries in your migration(s), and then rewrite your git history as needed, or you can do it manually.

On running ./tools/renumber-migrations
It asks for 2 input questions from the user:

Conflicting migrations:  1.  2.  Enter the order in which these migrations should be arranged: On giving input say, 2 1
Enter the name of last correct migration file:

The second point above should be implicit from the first question.

@abhijeetkaur if you have time, it'd be great if you could take a look at this.

@timabbott Sure!
@zulipbot claim

@zulipbot claim

I got annoyed by this and just fixed it, since it's just a couple lines of changes.  Thanks @abhijeetkaur!


Fix small typo in handlers.rst

What do these changes do?
do duplicating side-effects -> no duplicating side-effects
Description

Issues/PRs

Issues:

Related:

Type of changes

Refactoring (non-breaking change which does not alter the behaviour)
Mostly documentation and examples (no code changes)

Checklist

[x] The code addresses only the mentioned problem, and this problem only
[ ] I think the code is well written
[ ] Unit tests for the changes exist
[x] Documentation reflects the changes
[ ] If you provide code modification, please add yourself to CONTRIBUTORS.txt

Thank you very much for your contribution.
:+1:


Add apns-collapse-id header in notification

Hi,
please provide me how to add "apns-collapse-id" in header of request. so that i can replace notification content.

Hello!
This sounds more like a request for general support than it does like a bug report or a feature request. We generally ask people to move these kinds of questions to the Pushy mailing list because:

The mailing list has a wider audience, and you're more likely to find somebody who has experience with your particular circumstances on the mailing list.
Posting your question to the mailing list can help other people who are having the same problem.
Because there isn't a bug to fix or a feature to add, we can't reasonably solve your problem by making any kind of engineering change.

I emphasize that we'd be happy to help, but let's move this conversation to a different venue.
In the meantime, I'll point out that support for the collapse-id header is already in the ApnsPushNotification interface, and SimpleApnsPushNotification provides a concrete implementation.

thanks sir for support.
you save my day.


AsciiStr::to_ascii_lowercase returns an uppercase copy of the string instead

https://tomprogrammer.github.io/rust-ascii/src/ascii/ascii_str.rs.html#396
whoops :)

Closed and tested in 7ca810c and released in version 0.8.3
Thanks for the bug report!


Treemap isnt displayed

Hello.
I am back on treemaps with d3treeR. Thanks to you, i solved two problems i had previously (loops and white space between two treemaps).
Now, similarly, i have another problem, working in Rmarkdown environment.
I use {.tabset} to create a better reporting (see code below) and would like to have two treemaps separated this way.
The first one is displayed (treemap1) but not the second one ((treemap2)
Répartition {.tabset}
Volume d'affaires nouvelles
treemap1

Volume de primes sur affaires nouvelles
treemap2

I hope it is clear...
thank you in advance.
Pat

Hello.
I am back on treemaps with d3treeR. Thanks to you, i solved two problems i had previously (loops and white space between two treemaps).
Now, similarly, i have another problem, working in Rmarkdown environment.
I use {.tabset} to create a better reporting (see code below) and would like to have two treemaps separated this way.
The first one is displayed (treemap1) but not the second one ((treemap2)
Répartition {.tabset}
Volume d'affaires nouvelles
treemap1

Volume de primes sur affaires nouvelles
treemap2

I hope it is clear...
thank you in advance.
Pat

Hey Pat, I had the same problem as you. Did you solve it? Thanks!


Mvmds Doc Updates

Reference Issues/PRs

What does this implement/fix? Explain your changes.
Any other comments?

Pycodestyle errors blocking
$ pycodestyle mvlearn/
mvlearn/embed/mvmds.py:52:1: W293 blank line contains whitespace
mvlearn/embed/mvmds.py:54:80: W291 trailing whitespace
mvlearn/embed/mvmds.py:55:70: W291 trailing whitespace
The command "pycodestyle mvlearn/" exited with 1.

Since you have a PR for MVMDS docs up anyway, can you also make this quick change?
In the Examples section, replace the lines

print(Xs[0].shape[0])  # number of samples in each view
2000

with these lines

print(len(Xs))  # number of samples in each view
6
print(Xs[0].shape)  # number of samples in each view
(2000, 76)


Enhance CI tests for TensorRT supported bundles

Currently, all bundles use the same pipeline to do pre-merge tests. For each changed bundle (within a pull request), its required libs (such as MONAI version, Torch version, ...) will be extracted from metadata.json, and installed with pipenv during the testing workflow.
However, to enable TensorRT for a part of bundles (see also: https://github.com/Project-MONAI/model-zoo/issues/324), we need to add related tests. Install TensorRT requires more works than pipenv install, thus we decided to use a MONAI docker container (https://docs.monai.io/en/latest/installation.html#from-dockerhub) to do the test.
A limitation is that we need to align the libs (versions) used for all TensorRT supported bundles. As the first stage, three bundles will support TensorRT (see also: https://github.com/Project-MONAI/model-zoo/pull/325), and they will unify MONAI and Torch versions.
The expected new CI tests logic is like:

@YanxuanLiu will help to build the new pipeline, and I will prepare the new test scripts.

Hi @yiheng-wang-nv @YanxuanLiu ,
In my simple mind, I think for the first step we can enable weekly CI job to run it on dev branch or some specific branch.
For the TensorRT environment, let's stick with MONAI LTS version.
What do you think?
CC @wyli .
Thanks.

Hi @yiheng-wang-nv @YanxuanLiu ,
In my simple mind, I think for the first step we can enable weekly CI job to run it on dev branch or some specific branch. For the TensorRT environment, let's stick with the released docker of latest MONAI LTS version. What do you think? CC @wyli .
Thanks.

Hi @Nic-Ma ,

In my simple mind, I think for the first step we can enable weekly CI job to run it on dev branch or some specific branch

Here do you mean use dev branch to test all TensorRT bundles? Yes, I think we can maintain all these bundles to work with latest MONAI dev branch.

For the TensorRT environment, let's stick with the released docker

Agree.

I mean use the latest MONAI LTS version docker to run the dev branch of this repo.
Thanks.


Additionall webiste

I love this application. Unfortunately, my municipality changed the provider of the API to "Mashie". Is it possible to add this API also as optional in the application? The mashie-app/API is on this url; https://mpi.mashie.com/public/app

Hi, yes sure. I'll let you know when it up. Seems this one needs to be scraped off the page :/
/Anders

Hi, try updating to 1.4 and let me know how it works.
/Anders

Works like a charm. Thank you so much!

Well, I mean, it grabs the menu for a couple weeks and stores it as an attribute but from what I can tell there's no sensor data available telling me what's being served today??
My TTL morning briefing just says "ingen mat" for the school I'm tracking using mashie.

Well, I mean, it grabs the menu for a couple weeks and stores it as an attribute but from what I can tell there's no sensor data available telling me what's being served today??
My TTL morning briefing just says "ingen mat" for the school I'm tracking using mashie.

My bad, check the update

Well, I mean, it grabs the menu for a couple weeks and stores it as an attribute but from what I can tell there's no sensor data available telling me what's being served today??
My TTL morning briefing just says "ingen mat" for the school I'm tracking using mashie.

My bad, check the update

Nice one mate. Saved me from delving in myself :) I may still reach out with a PR at some point though.


Use pipenv sync instead of pipenv install

Install is an alias for lock + sync, which can update packages
unexpectedly. We should always use sync when installing packages on
CI.

ping @AbletonDevTools/gotham-city

The unit tests must be adapted.

@ala-ableton indeed, fixed.

Please adapt the following:
https://github.com/AbletonDevTools/python-pipeline-utils/blob/caf0192de6e3143547a3ff59840fd01a892db0e3/src/com/ableton/Pipenv.groovy#L21
https://github.com/AbletonDevTools/python-pipeline-utils/blob/caf0192de6e3143547a3ff59840fd01a892db0e3/test/com/ableton/PipenvTest.groovy#L48

@ala-ableton Done!

LGTM 👍


Unable to build with markdown images. Getting killed or segmentation fault

After reaching "Completed in.." my pc fans go Brrrr for a while, then the build gets killed or gets a segmentation fault.
Reproduced in this repo as a basic example: https://github.com/MrTeapot/astro-imagetools-build-killed-reproduction
Try cloning, "npm i", then "npm run build".
Running inside WSL2, Ubuntu 20.04 LTS.
CPU and memory usage becomes extremely high during the build.

@MrTeapot confirming the issue. The build is getting killed and exits with error code 137. That means the process ran out of memory. It's right that the images are large but I just tested and astro-imagetools is working fine with this 11.92MB image https://wallpaperaccess.com/download/16k-resolution-82311. There may be some other issue.

The ram usage is constantly increasing until it runs out of memory.

Looks like 15gb is available in WSL2, although I have 32gb installed. But 15 should be more than enough.
I've tried running this in node 14.17.1 and 18.2.0 with no difference.

@MrTeapot a fix for the performance regression has been merged and released in 0.6.9. https://github.com/MrTeapot/astro-imagetools-build-killed-reproduction is now building fine after upgrading to<EMAIL_ADDRESS>

My builds are working now when using 0.6.9.


Claim State Change triggering state change of application - DO NOT MERGE

Before creating a pull request make sure that:

[X] commit messages are meaningful and follow good commit message guidelines
[X] README and other documentation has been updated / added (if needed)
[X] tests have been updated / new tests has been added (if needed)

Please remove this line and everything above and fill the following sections:
JIRA link (if applicable)
https://tools.hmcts.net/jira/browse/CIV-2801
Change description
This ticket talks about changing the application status in accordance with the main claim. Currently, the implementation has only been made for CASE_DISMISSED triggering events for APPLICATION_CLOSED state. Since the remaining second part of the ticket is changing state for PROCEEDS_IN_HERITAGE_SYSTEM, which is yet not implemented, please DO NOT MERGE this PR.
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[X] No

Closing this PR as the design has changed. PLEASE DO NOT DELETE THIS BRANCH.


Grammatical error on the Value Conversions page

I believe I've spotted a grammatical error in the following paragraph in the 'Column facets and mapping hints' section on the Value Conversions page:
Now any time this converter is used, the database column will be non-unicode with a max length of 20. However, these are only hints since they are be overridden by any facets explicitly set on the mapped property.
I suspect "are be overridden" should be "could be overridden"?

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

ID: 03dfe9f6-e831-ac3f-0b90-acb9b4c0f828
Version Independent ID: a26156d1-a4b3-7df0-e63e-b8e9f804abbf
Content: Value Conversions - EF Core
Content Source: entity-framework/core/modeling/value-conversions.md
Product: entity-framework
Technology: entity-framework-core
GitHub Login: @ajcvickers
Microsoft Alias: avickers

Thanks @metaljase, I've submitted #3651 to fix this.


[stable/traefik] Add support for ACME KeyType

Signed-off-by: Kaden Nelson<EMAIL_ADDRESS>
What this PR does / why we need it:
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #14884

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

/assign @geraldcroes

/assign Juliens

/assign maorfr

/ok-to-test

/lgtm


Delete a few functions

I believe these functions could be deleted, but before doing so I'd like to check to be sure they aren't used in any current PITcleanr functions:

writeSpwnPaths
writeFishPaths
writeCapHistOutput
summarizeValidTagsLGR
processCapHist_TUM
processCapHist_PRO
processCapHist_PRD
processCapHist_LGD

These functions have been deleted


Possible information leakage from pretrained model

Dear author,
Thank you for your excellent work!
I am a little curious about the pretrained model, it is trained on the entire ImageNet-21k dataset, and is fixed during training. But will this lead to information leakage?
Take the class incremental setting as an example, I think all 100 classes of CIFAR100 can be found in ImageNet-21k so it is possible that the model has already learned all the features necessary for CIFAR100. But in practice, the model is expected to learn new features. We can not assume the classes in new tasks have already been observed by the backbone, right?
Have you tried to remove CIFAR100 classes in ImageNet and pretrained a model or evaluate the model on some datasets disjoint with ImageNet?
Thank you very much!

Dear author,
Thank you for your excellent work!
I am a little curious about the pretrained model, it is trained on the entire ImageNet-21k dataset, and is fixed during training. But will this lead to information leakage?
Take the class incremental setting as an example, I think all 100 classes of CIFAR100 can be found in ImageNet-21k so it is possible that the model has already learned all the features necessary for CIFAR100. But in practice, the model is expected to learn new features. We can not assume the classes in new tasks have already been observed by the backbone, right?
Have you tried to remove CIFAR100 classes in ImageNet and pretrained a model or evaluate the model on some datasets disjoint with ImageNet?
Thank you very much!

I am also very curious about the pre-train part as previous incremental baselines train from the scratch by default.

Great insight!
Actually we have not tried your suggested experiments, but it is definitely something worth trying. Regarding the "information leakage", I think we do make the assumption that we have a "well-pretrained" model, and we use the same pretrained model for all competitors, so the comparison is actually fair. Another thing I would like to highlight is that the idea of prompting is actually leveraging learned knowledge in the model, and trying to "instruct" the model to selectively use learned knowledge for coming tasks. Since large-scale pretrained model is prevalent these days, leveraging them is quite natural.
On the other hand, thinking about the extreme case that the pretrained model is totally off (e.g. trained on a totally different dataset, though we will not do it in practice), L2P will probably fail if the backbone is frozen. Thus, it will be interesting to see how and when to adapt the model backbone as a future direction.
Thanks again for your question and suggestion!
Best,
Zifeng


Fix duplicate image entries in k8s.io namespaces

fix3702

This should be opt-in via a config entry like --kube-hide-dupe: https://github.com/containerd/nerdctl/blob/main/docs/config.md

Okay, I'll try to implement it. Initially, I wanted to implement it separately, but this is a preliminary version for the sake of testing and discussion.

@fengwei0328
Maybe we could have an extra test for the following case?
nerdctl pull busybox
nerdctl tag busybox foo
id="$(nerdctl images busybox -q)"

Then
nerdctl rmi -f "$id"

Versus
nerdctl --kube-hide-dupe rmi -f "$id"

And confirm that in both cases, all images with that id are correctly removed?
And the same test, this time with id=$(nerdctl images debian -q --no-trunc)
wdyt?

Maybe we could have an extra test for the following case?

Ok, that's also I do manually every time I'm locally

debian

Sorry, I definitely meant busybox, not debian.
The point is to verify that force-deleting by short id, and digest, is still working as expected, and will delete images with multiple tags.

short id, and digest

short id and digest tests added

debian

Sorry, I definitely meant busybox, not debian.
The point is to verify that force-deleting by short id, and digest, is still working as expected, and will delete images with multiple tags.

short id and digest tests added

Thanks a lot @fengwei0328
One last comment on making sure the root test is NoParallel, LGTM otherwise.
Then, to maintainers.

Thanks a lot @fengwei0328
One last comment on making sure the root test is NoParallel, LGTM otherwise.
Then, to maintainers.

Thanks a lot @apostasie


In graph hash, use direction

formally, we shouldn't ignore edge labels here but also add the voltage

will be considered via #196


Add trust_remote_code for loading datasets in the audio classification example

What does this PR do?

With the release of Datasets v2.20, the default of the arg trust remote code in the method load_dataset is now False while it used to be True.
This PR makes sure that the 8x audio classification example still works properly by enabling users to set this value to True.
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

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


Cannot Run Mac Executable.

I used
pkg package.json
to create a Mac Executable for my project --example.js. But when I double click on that executable, it executes from the root directory instead of executing from it's own directory.
Therefore, I'm getting module not found error.
Error: Cannot find module '/Users/*******/debug.js'
I have to run it through terminal by moving into that directory. I want it to work by just clicking on the executable.
I'm using

MacOS Mojave 10.14.5
Node version 10.13.0

is it works on others platform?
have you mentions this in package.json file?
"pkg": {
"scripts": "build//*.js",
"assets": "views//*"
}

You need `path.dirname(process.execPath)'

Thanks @igorklopov for your reply. Yes I did try that one also, but it creates another issue when running directly with node, then it outputs my current node directory, i.e.
HOME/.nvm/versions/node/v12.16.1/bin.
Any ideas to have a solution or do I need to make extra checks and call the right one?

@erjanmx  Right. You need to check process.pkg. If true then my line above, if false then your line you want from direct node run, like __dirname (not sure tho).

Just tested and everything seems to be working as expected with your proposed solution, @igorklopov, both on macOS and Windows (didn't test on linux tho).
Thanks a lot for your help!


Deactivate dependency on pylzma

It's hard to install and we're not using it.

This is done now. Removal of library import and all references.

Resolved by #50.


[Feature] CustomRP config compatibility

Is your feature request related to a problem? Please describe.
As a user I would like the application to be a drop-in replacement for CustomRP.
Describe the solution you'd like
I would like to import/convert my existing CustomRP configuration file. I would also like to be able to convert my Genzai config to CustomRP.
Describe alternatives you've considered
Only way for this to be possible is by making a converter between the file types.
Additional context
CustomRP uses an XML format to store its configuration files. Converting should be as easy as just running a function that maps the xml values to the json format we're using.

I agree, I'll patch out the json in favor for the xml format.


release 3.1 and add  assets

@choldgraf, @betatim  I'm going to suggest we move that to the JupyterHub org. In particular that will prevent me from being the bottle neck and pushing here by mistake.

Actually we need a 3.2 as now chrome require 128px wide icons.

I'm all for it!

Moving sounds good to me. I tagged the revision I used to make v0.3.0 just now. Hadn't seen this PR until after.
One thing I am not clear on: this talks about 3.1 but I would have said the next release is 0.3.1? I don't care much either way just that we pick one way :)


Synchronous loading content breaks animation and 'Loading...' message

I was figured out, that when I use functionReady (A custom function to be fired when the tooltip and its contents have been added to the DOM) - then when I call instance.content() inside it with using content loading by AJAX in functionBefore - instance.content() return 'Loading...'.
I thinks that's a bug.
I tried to do workaround and make content loading synchronously, then animation and preloading message doesn't show... vicious circle :-)

    $('.fa-language').closest('a').tooltipster({
        content: 'Loading...',
        theme: ['tooltipster-noir', 'tooltipster-noir-customized'],
        functionPosition: function (instance, helper, position) {
            position.coord.top += 0;
            position.coord.left += -200;
            return position;
        },
        trigger: 'custom',
        interactive: true,
        side: 'bottom',
        triggerOpen: {
            click: true
        },
        triggerClose: {
            click: true,
            scroll: false
        },
        contentAsHTML: true,
        functionBefore: function (instance, helper) {
            var $origin = $(helper.origin);
            $.ajax({
                url: $origin.data('url'),
                method: 'get',
                dataType: 'html',
                async: false,
                success: function (response) {
                    instance.content($(response));
                }
            });
        },
        functionReady: function (instance, helper) {
            $('.close-toolstipster', instance.content()).on('click', function () {
               instance.close();
            });
        }
    });

It's most probably an issue in your code. Please provide a basic jsFiddle to demonstrate your issue and I'll reopen if there actually was a bug.

smth like this:
https://jsfiddle.net/georzal/u8eungkc/
see console, functionReady invoked before content-ready.. All is okey and clear why, but documentation say 'A custom function to be fired when the tooltip and its contents have been added to the DOM'.
So, you should add some note to the functionReady doc or somehow wait functionBefore completion..

The displayed content is the string "Loading...", and functionReady is called after that string has been appended to the DOM, as it should. What you do is a mere update of the content at a later timer. So there is no documentation issue here. Thank you though.