TransitionToScene (and variants) only unload the current active scene.

When using TransitionToSceneAsync() to transition between scenes, I expected that the scene loader would unload all scenes under it's management when transitioning. Instead, no scene was unloaded at all, because I didn't make the scene I wished to unload the active scene.
This is a problem.
Unloading only the active scene is extremely limiting, as there is currently no way to transition from:

A scene that is not the active scene
A collection of scenes

What I propose is that ISceneLoader.TransitionToScene() and ISceneLoader.TransitionToScenes() would accept a scene or an array of scenes to unload, and that the implementations of these methods in SceneLoaderAsync, SceneLoaderCoroutine, and SceneLoaderUniTask would unload all managed scenes by default.
I know this will break backwards compatibility, but it doesn't seem like much of a problem, seeing as this package already broke some backwards compatibility in the past.

Hi @Clozent, thanks for your report.
I agree with you, simply unloading the active scene only covers a simple use case. It does not need to break compatibility though, we can just add new method alternatives for scene transition, such as:

Passing scenes to be unloaded

void TransitionToScenesAsync(ILoadSceneInfo[] targetScenes, int setIndexActive, ILoadSceneInfo[] unloadScenes, ILoadSceneInfo intermediateSceneReference = null)

Option to unload all managed scenes

void TransitionToScenesAsync(ILoadSceneInfo[] targetScenes, int setIndexActive, bool unloadAllScenes, ILoadSceneInfo intermediateSceneReference = null)

With the new methods, you'd have three options for transition:

Transition and unload the active scene (current implementation)
Transition and unload a group of scenes
Transition and unload all other managed scenes

What do you think? If you agree, I'll start working on it right away.

Hi, overall, I think these new methods are a good way to solve this problem.
One thing is bugging me, though: the method void TransitionToScenesAsync(ILoadSceneInfo[] targetScenes, int setIndexActive, bool unloadAllScenes, ILoadSceneInfo intermediateSceneReference = null) changes it's behavior depending on the unloadAllScenes boolean, which is, from my understanding, something a method should avoid.
Instead, I think it would be best to make the behavior a separate method, such as TransitionToScenesUnloadAll(...) (I don't think this is a good name for it, but I can't come up with anything better).

You're right, it'd be better to introduce a new method for unloading all scenes.
So to summarize, two new methods will be added:

TransitionToScenesFromScenes - unloads multiple provided scenes
TransitionToScenesFromAll - unloads all managed scenes

Not so sure about the names yet, but that's the scope. I'll start working on it today.

Adding a development note for my future self:
Check whether including #44 improves the experience when using TransitionToScenesFromScenes.


Single buttons on modals should be half width

What package(s) are you using?

[ ] carbon-components
[x] carbon-components-react

Detailed description

Describe in detail the issue you're having.

A single primary button inside a modal takes up the entire width of the modal. We've confirmed that the screenshot attached shows the correct styling of the button and the documentation on full-length buttons is incorrect.

Is this issue related to a specific component?

Modals and Buttons

What version of the Carbon Design System are you using?

"carbon-components": "^10.7.4",
"carbon-components-react": "^7.7.4",

Additional information

Our overriding styling:
.bx--modal-footer {
  justify-content: flex-end;
  button.bx--btn--primary {
    max-width: 50%;
  }
}

The button description and @asudoh comment above contradicts the provided screen shot and “acknowledgment Modal” guideline which says “span 50%“: https://www.carbondesignsystem.com/components/modal/usage#acknowledgment-modal


Saving time is too long. (1785 ms > 1000 ms),

While I was saving  workspace，It always comes up ：Saving time is too long. (1785 ms > 1000 ms),

I've also been running into this problem a lot with Volar. Initially, it saves quickly, but over time (~1-2 hours) it eventually crawls to a halt and takes a long time to save. I have tried disabling other extensions like ESLint but no luck. I've probably had this issue for about a month now.

I've also been running into this problem a lot with Volar. Initially, it saves quickly, but over time (~1-2 hours) it eventually crawls to a halt and takes a long time to save. I have tried disabling other extensions like ESLint but no luck. I've probably had this issue for about a month now.

Yes, sometimes I wonder if my computer is too bad, it never used to be the case, maybe there are too many lines of code in one file

#2740


New option --unique available in command write_gsb_to_xml

Motivate of the pull request

[x] To address an existing issue. If so, please provide a link to the issue: #456
[ ] Breaking new feature. If so, please describe details in the description part.

Describe the technical details
What is currently done? (Provide issue link if applicable)

Address issue #456

What does this pull request change?

This PR improves in the following aspects:

[x] Added a new option --unique to command write_gsb_to_xml as requested in #456
[x] Updated documentation about the new option
[x] Added 4 test cases to validate the command write_gsb_to_xml
[x] Deployed the 4 test cases to regression tests on CI

Which part of the code base require a change

[ ] VPR
[ ] Tileable routing architecture generator
[ ] OpenFPGA libraries
[ ] FPGA-Verilog
[ ] FPGA-Bitstream
[ ] FPGA-SDC
[ ] FPGA-SPICE
[x] Flow scripts
[ ] Architecture library
[ ] Cell library
[x] Documentation
[x] Regression tests
[ ] Continous Integration (CI) scripts

Impact of the pull request

[ ] Require a change on Quality of Results (QoR)
[ ] Break back-compatibility. If so, please list who may be influenced.

@ganeshgore Waiting for your confirm to merge this PR.


Permission Denied for git access

Everything works fine but it gives me this issue when I try to pull from my git repository.

Permission denied, please try again.
Permission denied, please try again.
Permission denied (publickey,password).
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.

It doesn't give me this error when I exit out of cash, how would I go about to fix this?

Could you paste the exact command you're using? e.g.
$ git pull origin master
Permission denied... etc.


refactor(crypto/hash): crypto hash to allign with the signature of WebCrypto

Codecov Report
Attention: Patch coverage is 91.30435% with 4 lines in your changes are missing coverage. Please review.

Project coverage is 90.31%. Comparing base (c8f4984) to head (7af9522).

Files
Patch %
Lines

crypto/hash.ts
89.47%
4 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main       #3      +/-   ##
==========================================
+ Coverage   90.05%   90.31%   +0.26%
==========================================
  Files           5        7       +2
  Lines        5228     5165      -63
  Branches       41       48       +7
==========================================
- Hits         4708     4665      -43
+ Misses        520      500      -20

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


[RFC] Svelte 5 (@next) support

How do you envision this feature/change to look/work like?
Svelte 5 (svelte@next) brings major productivity and code unification improvements, among other things allowing to use reactivity in any js/ts file.
I tried using svelte@next with plasmo, but plasmo errors out saying that the frontend is not supported.
What is the purpose of this change/feature? Why?
Add svelte@next to list of allowed frontends to be used with plasmo.
(OPTIONAL) Example implementations
No response
(OPTIONAL) Contribution

[X] I would like to contribute to this RFC via a PR

Verify canary release

[X] I verified that the issue exists in plasmo canary release

Code of Conduct

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

Same here. Tried to upgrade my extension to Svelte 5 and was getting a ton of issues. Eventually, I realized that even a skeleton project that is Svelte 5 rather than Svelte 4 would give the following error:
🔴 ERROR  | No supported UI library found.  You can file an RFC for a new UI Library here: https://github.com/PlasmoHQ/plasmo/issues


Fixed Issue 2

Resolved a minor problem where the callback was not being passed through
when a locator was supplied to find a build queue.

Hi @dcopestake, thanks for the PR :heart:!
As you can see I haven't bumped the version on this project in a while. In fact, there's a large set of changes post 0.0.7 (1caa6a0bc85a012bf54b29be3e5ec0d53cde8bde, I should have put these changes in a separate branch from the beginning) in master that I haven't bumped yet.
I'm going to merge this PR now, but I'm not ready to bump the version yet as my changes break a few things.
I'm currently using 0.0.3 and 0.0.6 in my projects that use this one.


update maturity models for use with VRS

Move Entity, IRI, and Extension to trial use maturity for anticipated use with VRS 2.0 pre-release classes.

I spoke this over with @larrybabb last week so feel confident he would also approve. Merging.


Compiler forces call super() even if it could be done automatically

TypeScript Version:
1.8.10
Code
class Base {
    constructor(name?:string){
         console.log(name);
    }
}

class Child1 extends Base {
      constructor(){
            super(); //wont compile without super call
            console.log('do not need name');
      }
}

class Child2 extends Base {
      //this code is ok...
}

Expected behavior:
It should be possible to skip super() call if there are no required parameters (just like Java does).
Actual behavior:
Will not compile without super() call.

It's according to the ES spec. If an explicit constructor is present, then it must include a super() call.

super() could be added to generated code automatically by compiler,  es spec would not be violated.

While access this is prohibited before the super() call, other actions can take place.  Auto-magic features are dangerous for little value, cause confusion and aren't in the spirit of TypeScript as a superset of ECMAScript.

This issue has nothing about this access. Any valid ECMAScript code would still be valid TypeScript code. Seriously, blindly typing super() for each inherited class does not add any value to the application code.  I still do not see any reason against generating super() call.

Seriously, blindly typing super() for each inherited class does not add any value to the application code.

What if you want to do something before super()?  What if you want to pass arguments to super()?  You might then argue that is super() is in the body of the constructor then don't emit the super() but that becomes auto-magical then, which is surprising.  You are correct, it is blindly emitting, which blindly emitting anything will cause issues.

It's not unreasonable to ask for an implicit super() call at the beginning of derived class constructor if it's not explicitly called anywhere in the body (and, in the case of TS, if a no-arg constructor actually exists on the base class). If the user wants explicit control over its placement they would call it explicitly. If the user wants to call it with arguments they would call it explicitly.
But this decision should come from TC39.

It's not unreasonable to ask for an implicit super() call at the beginning of derived class constructor if it's not explicitly called anywhere in the body (and, in the case of TS, if a no-arg constructor actually exists on the base class). If the user wants explicit control over its placement they would call it explicitly. If the user wants to call it with arguments they would call it explicitly.

Exactly.

But this decision should come from TC39.

So it will not happen any time soon.

This really comes down to preserving the ES6 class semantics. ES6 could have said there was an implicit super call in constructor bodies that did not contain a super call, but they didn't. We can agree or disagree with that decision but it's basically immaterial in a language that isn't intentionally diverging from the ES6 behavior.

Obviously, i'm not going to argue. Just want to know, what is the scope for new features in TypeScript? Is it limited to type system?

https://github.com/Microsoft/TypeScript/wiki/TypeScript-Design-Goals is a good overview of what we generally consider to be in scope.
In general things that aren't related to the type system have to clear a much higher bar. Changing the runtime behavior of TS relative to its "vanilla" JS counterpart is something we will not do, even if the behavior would be broadly preferable. For example, we could easily decide to have more sane behavior runtime around this, but haven't because maintaining JS compatibility is an absolute constraint.


Fixed error causes by migrating to null safety

@EddieKamau
I have been rewriting the Liquidart project from scratch, so I am don't think I can merge this PR.


How to build it on windows

Is there anyone build it on windows. The project looks like a linux project. I am trying to build on windows, “unistd.h”: No such file.
should i use cygwin to build it?

You can put the OPENCL_3_0, NVIDIA_PLATFORM, and WINDOWS into the preprocessor definition. Then that "unistd.h" error will be gone .


Updated dependencies and fixed security issues

Improved/updated things:

Updated Visual Studio Code dependencies using Dependabot.
Improved Visual Studio Code security using Dependabot.

Sorry, I don't want to take a PR like this


⚠ Bofrost.fr has degraded performance

In 5868728, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 4302 ms

Resolved: Bofrost.fr performance has improved in 2cb395b.


sql migration error while upgrading to LoRa App Server 0.20.0 / LoRa Server 0.26.0 / LoRa Gateway Bridge 2.4.0

Dear Brocaar,
Thanks for your excellent work. Unfortunately last application update did not work out.
Please find all the information below, thanks ahead for your help.
Is this a bug or a feature request?
Bug I guess
What did you expect?
The db to be migrated from last version to current one and all the servers to restart
What happened?
Lora App Server never restarted
What version are your using?
LoRa App Server 0.20.0 / LoRa Server 0.26.0 / LoRa Gateway Bridge 2.4.0
Could you share your log output?
Apr 13 08:43:56 dev01.post-iot.lu lora-app-server[4228]: time="2018-04-13T08:43:56+02:00" level=fatal msg="applying migrations error: pq: operator class "gin_trgm_ops" does not exist for access method "gin" handling 0027_global_search.sql"
Apr 13 08:43:56 dev01.post-iot.lu systemd[1]: lora-app-server.service: Main process exited, code=exited, status=1/FAILURE
Apr 13 08:43:56 dev01.post-iot.lu systemd[1]: lora-app-server.service: Unit entered failed state.
Apr 13 08:43:56 dev01.post-iot.lu systemd[1]: lora-app-server.service: Failed with result 'exit-code'.
Apr 13 08:43:56 dev01.post-iot.lu systemd[1]: lora-app-server.service: Service hold-off time over, scheduling restart.
Apr 13 08:43:56 dev01.post-iot.lu systemd[1]: Stopped LoRa App Server.
Apr 13 08:43:56 dev01.post-iot.lu systemd[1]: lora-app-server.service: Start request repeated too quickly.
Apr 13 08:43:56 dev01.post-iot.lu systemd[1]: Failed to start LoRa App Server.

Thanks for helping! Instructions in changelog worked as charm


[lsp-server] 🐞 The vscode-graphql server crashed 5 times in the last 3 minutes

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
I just updated the GraphQL: Language Feature Support VS Code Extension to v0.7.12 in VS Code v1.72.2 and since then I am getting this error:

Error: No cache available for handleWatchedFilesChanged
at Vte.handleWatchedFileChangedNotification (c:\Users\bn.vscode\extensions\graphql.vscode-graphql-0.7.12\out\server\index.js:736:4341)
at c:\Users\bn.vscode\extensions\graphql.vscode-graphql-0.7.12\out\server\index.js:742:5066
at Array.map ()
at Gte.handleWatchedFilesChangedNotification (c:\Users\bn.vscode\extensions\graphql.vscode-graphql-0.7.12\out\server\index.js:742:4967)
at c:\Users\bn.vscode\extensions\graphql.vscode-graphql-0.7.12\out\server\index.js:770:102695
at ie (c:\Users\bn.vscode\extensions\graphql.vscode-graphql-0.7.12\out\server\index.js:745:1591)
at Te (c:\Users\bn.vscode\extensions\graphql.vscode-graphql-0.7.12\out\server\index.js:744:6743)
at Immediate. (c:\Users\bn.vscode\extensions\graphql.vscode-graphql-0.7.12\out\server\index.js:744:6621)
at process.processImmediate (node:internal/timers:466:21)
[Error - 11:45:59] The vscode-graphql server crashed 5 times in the last 3 minutes. The server will not be restarted. See the output for more information.
[Info  - 11:45:59] The graphql server has stopped running

Expected Behavior
No error pop up in VS Code

Steps To Reproduce
1. Update GraphQL VS Code extension
2. Wait 3 minutes for VS Code to bring up this error

Environment
- LSP Server Version: 0.7.12
- OS: Windows 11
- LSP Client: vscode-graphql-0.7.12

Anything else?
No response

I have re-opened #2421 to track this, thank you for reporting!

@bennycode re-opening, I have a fix incoming!

Does 0.8.1 fix this for you?


Revert "Implement Double Tap Handling in TextField and Editable"

Reverts flutter/flutter#21264

Landing on red to fix the build


Rules are deleted by 1.1.6 when using redirect actions

I'm using a host-based redirect action as documented here. Specifically, I'm redirecting from one domain to another based on the host-header.
When I upgraded to 1.1.7, I found that all of the rules except for the redirect were deleted from my ALB. After some testing, I found that the rules are created correctly in 1.1.5 and previous versions, but not in 1.1.6+.
Here's the relevant excerpts from my config:
alb.ingress.kubernetes.io/actions.domain-redirect: '{"RedirectConfig":{"Host":"target-host.example.com","Port":"443","Protocol":"HTTPS","StatusCode":"HTTP_301"},"Type":"redirect"}'
alb.ingress.kubernetes.io/conditions.domain-redirect: '[{"Field":"host-header","HostHeaderConfig":{"Values":["test-redirect.example.com"]}}]'

spec:
  rules:
  - http:
      paths:
      - path: /*
        backend:
          serviceName: domain-redirect
          servicePort: use-annotation
      - path: /v1/*
        backend:
          serviceName: backend
          servicePort: 8080
      - path: /ws/*
        backend:
          serviceName: websocket
          servicePort: 8000
      - path: /*
        backend:
          serviceName: frontend
          servicePort: 80

Here's the resulting rules, before and after upgrading to 1.1.6.
Rules in 1.1.5:

Rules in 1.1.6

It's my belief that this is a bug introduced by #1162, which adds behavior to detect unconditional redirects and delete any rules after them.

In the meantime, I've been able to avoid this bug by adding a separate HTTP Ingress rule for that specific hostname, which works for our purposes, but might not be a valid workaround for everyone.

thanks for reporting this, indeed a bug 👍
fixed it in https://github.com/kubernetes-sigs/aws-alb-ingress-controller/pull/1286, will cut a release soon.


Cannot access renamed ports via wirelink

If ports are renamed, as in #1053, wires get reassigned to the new port name.
However, it is no longer possible to use wirelink to access a renamed port via the old name.
This might break legacy E2 chips.

Fixed by 5adcd53784507e05268f208f03a6f89a46834bdf


Update Elden Ring

Game Title
Elden Ring
Status of the Anti-Cheat
🎉 Confirmed
Proof of Mention
https://steamdb.info/app/1245620/history/?changeid=14059872

Work flawlessly

@Starz0r Should be "Supported", the game is already working with EAC.

It still says confirmed rather than supported


spi_bus_free crashes if interface was uninitialized (IDFGH-7024)

IDF 4.4
This is another regression from 3.3.x.
Previously, you could call spi_bus_free if the bus hadn't been initialized, and it would just silently do nothing.
I use this ability frequently - my device needs to deinit and reinit the SPI bus often.  So I would just do a spi_bus_free and then spi_bus_initialize.
You have to do the free first, since spi_bus_initialize will crash if it has already been initialized.  However, now spi_bus_free crashes if it wasn't initialized (IE, the first time the routine is called).
I'm not seeing a way to check if the bus was initialized or not, so I will need to add my own state to track this.
I tried to get to it in the debugger to see the exact state when it dies, but I can't because of #8640 :-/
Sorry to be filing so many bugs so quickly, but I'm about 20 hours in so far and I'm seriously reconsidering this upgrade.  I've barely touched 1/10 of the functionality I need to verify and it's just a mess all across the board.

Hi @readmodifywrite ,
Thanks for reporting this issue.
Though we don't encourage you rely on its returning, crash is obviously bad behavior... Will fix it.
Michael

+1
https://github.com/platformio/platform-espressif32/issues/944

+1


http needs to be changed to https in avatar of data/AnanjayGurjar.json

Description
While in the development run, the next/image is causing an error for getting the profile picture of Ananjay Gurjar as the next.config.mjs uses https protocol for github.com
Changes to be done:
...
"avatar": "http://github.com/AnanjayGurjar.png",
...

to
...
"avatar": "https://github.com/AnanjayGurjar.png",
...

in data/AnanjayGurjar.json
Screenshots

Additional information
No response

Found 4 files with http as the protocol for the avatar.

Sure let me change the config...


Added a travis file

A Travis file allows us to use the Apache's Travis account to do automatic testing on pull requests and pushes on the repo which is mirrored to github. Currently the file is only set for oracle Java 8 for limiting testing time but multiple versions of Java can be included. The testing runs ./gradlew assemble which packages the project and then tests it. This is not meant to replace the Jenkins config of creating distribution builds.
Note: to enable this infra would have to add our repo.

LGTM


Did anyone successfully export onnx?

code as follows, but export nothing:
weight_path = 'weights/hybridnets.pth'
device = 'cuda' if torch.cuda.is_available() else 'cpu'
params = Params(os.path.join(Path(__file__).resolve().parent, "projects/bdd100k.yml"))
model = HybridNetsBackbone(num_classes=len(params.obj_list), compound_coef=3,
                           ratios=eval(params.anchors_ratios), scales=eval(params.anchors_scales),
                           seg_classes=len(params.seg_list), backbone_name=None)
model.load_state_dict(torch.load(weight_path, map_location=device))
model.eval()
inputs = torch.randn(1, 3, 384, 640)
print("begin to convert onnx")
torch.onnx.export(model, inputs, 'HybridNetsBackbone.onnx',
                  verbose=False, opset_version=12, input_names=['images'])
print("done")

shell log:
HybridNets/utils/utils.py:673: TracerWarning: torch.from_numpy results are registered as constants in the trace. You can safely ignore this warning if you use this function to create tensors out of constant variables that would be the same every time you call this function. In any other case, this might cause the trace to be incorrect.
  anchor_boxes = torch.from_numpy(anchor_boxes.astype(dtype)).to(image.device)
Warning: Constant folding - Only steps=1 can be constant folded for opset >= 10 onnx::Slice op. Constant folding not applied.
Warning: Constant folding - Only steps=1 can be constant folded for opset >= 10 onnx::Slice op. Constant folding not applied.
Warning: Constant folding - Only steps=1 can be constant folded for opset >= 10 onnx::Slice op. Constant folding not applied.
Warning: Constant folding - Only steps=1 can be constant folded for opset >= 10 onnx::Slice op. Constant folding not applied.
Warning: Constant folding - Only steps=1 can be constant folded for opset >= 10 onnx::Slice op. Constant folding not applied.
Warning: Constant folding - Only steps=1 can be constant folded for opset >= 10 onnx::Slice op. Constant folding not applied.
Warning: Constant folding - Only steps=1 can be constant folded for opset >= 10 onnx::Slice op. Constant folding not applied.
Warning: Constant folding - Only steps=1 can be constant folded for opset >= 10 onnx::Slice op. Constant folding not applied.
Warning: Constant folding - Only steps=1 can be constant folded for opset >= 10 onnx::Slice op. Constant folding not applied.

...
ONNX export failed: Couldn't export Python operator SwishImplementation

code as follows, but export nothing:
weight_path = 'weights/hybridnets.pth'
device = 'cuda' if torch.cuda.is_available() else 'cpu'
params = Params(os.path.join(Path(__file__).resolve().parent, "projects/bdd100k.yml"))
model = HybridNetsBackbone(num_classes=len(params.obj_list), compound_coef=3,
                           ratios=eval(params.anchors_ratios), scales=eval(params.anchors_scales),
                           seg_classes=len(params.seg_list), backbone_name=None)
model.load_state_dict(torch.load(weight_path, map_location=device))
model.eval()
inputs = torch.randn(1, 3, 384, 640)
print("begin to convert onnx")
torch.onnx.export(model, inputs, 'HybridNetsBackbone.onnx',
                  verbose=False, opset_version=12, input_names=['images'])
print("done")

shell log:
HybridNets/utils/utils.py:673: TracerWarning: torch.from_numpy results are registered as constants in the trace. You can safely ignore this warning if you use this function to create tensors out of constant variables that would be the same every time you call this function. In any other case, this might cause the trace to be incorrect.
  anchor_boxes = torch.from_numpy(anchor_boxes.astype(dtype)).to(image.device)
Warning: Constant folding - Only steps=1 can be constant folded for opset >= 10 onnx::Slice op. Constant folding not applied.
Warning: Constant folding - Only steps=1 can be constant folded for opset >= 10 onnx::Slice op. Constant folding not applied.
Warning: Constant folding - Only steps=1 can be constant folded for opset >= 10 onnx::Slice op. Constant folding not applied.
Warning: Constant folding - Only steps=1 can be constant folded for opset >= 10 onnx::Slice op. Constant folding not applied.
Warning: Constant folding - Only steps=1 can be constant folded for opset >= 10 onnx::Slice op. Constant folding not applied.
Warning: Constant folding - Only steps=1 can be constant folded for opset >= 10 onnx::Slice op. Constant folding not applied.
Warning: Constant folding - Only steps=1 can be constant folded for opset >= 10 onnx::Slice op. Constant folding not applied.
Warning: Constant folding - Only steps=1 can be constant folded for opset >= 10 onnx::Slice op. Constant folding not applied.
Warning: Constant folding - Only steps=1 can be constant folded for opset >= 10 onnx::Slice op. Constant folding not applied.

...
ONNX export failed: Couldn't export Python operator SwishImplementation

Solved:
modify backbone.py
class HybridNetsBackbone(nn.Module):
    ......
    if backbone_name:
        self.encoder = timm.create_model(backbone_name, pretrained=True, features_only=True, out_indices=(2,3,4))  # P3,P4,P5
    else:
        # EfficientNet_Pytorch
        self.encoder = get_encoder(
            'efficientnet-b' + str(self.backbone_compound_coef[compound_coef]),
            in_channels=3,
            depth=5,
            weights='imagenet',
        )
            # Slove cannot export onnx, added by Andy, 2022-07-01
        if onnx_export:
            self.encoder.set_swish(memory_efficient=False)

@aimuch

weight_path = 'weights/hybridnets.pth'
device = 'cuda' if torch.cuda.is_available() else 'cpu'
params = Params(os.path.join(Path(__file__).resolve().parent, "projects/bdd100k.yml"))
model = HybridNetsBackbone(num_classes=len(params.obj_list), compound_coef=3,
                           ratios=eval(params.anchors_ratios), scales=eval(params.anchors_scales),
                           seg_classes=len(params.seg_list), backbone_name=None)
model.load_state_dict(torch.load(weight_path, map_location=device))
model.eval()
inputs = torch.randn(1, 3, 384, 640)
print("begin to convert onnx")
torch.onnx.export(model, inputs, 'HybridNetsBackbone.onnx',
                  verbose=False, opset_version=12, input_names=['images'])
print("done")

Solved: modify backbone.py
class HybridNetsBackbone(nn.Module):
    ......
    if backbone_name:
        self.encoder = timm.create_model(backbone_name, pretrained=True, features_only=True, out_indices=(2,3,4))  # P3,P4,P5
    else:
        # EfficientNet_Pytorch
        self.encoder = get_encoder(
            'efficientnet-b' + str(self.backbone_compound_coef[compound_coef]),
            in_channels=3,
            depth=5,
            weights='imagenet',
        )
            # Slove cannot export onnx, added by Andy, 2022-07-01
        if onnx_export:
            self.encoder.set_swish(memory_efficient=False)

@aimuch that's great, please feel free to open a pull request if you succeeded

Now, I can export onnx, but snpe cannot support asymmetric pads. So I need to change the backbone to retry.
Encountered Error: ERROR_ASYMMETRIC_PADS_VALUES: Asymmetric pads values is not supported


New release?

When do you plan to make new release? I think it will be help full for us.
For example, I am waiting for PR https://github.com/facebook/php-webdriver/pull/560

Same issue

I too am waiting for the 1.8.0 release but I know that there are a number of issues still pending. Most of them now have patches up, or are in-progress.
So far it seems that only a couple of people are actively helping with the testing effort. Perhaps you’re both able to help and identify any other issues?

As @andrewnicols wrote. There is lot of changed stuff in upcoming release, so we'd like to make sure there aren't any issues.
Also, you are more than welcome with testing the new release - you can already use it by requiring the development version, see https://github.com/facebook/php-webdriver/issues/469#issuecomment-554979855 .


open page in a new tab

fixes: #253
Signed-off-by: Yixin Qian<EMAIL_ADDRESS>

Thanks @OscarQQ


:chat hides vanilla chat with a custom chat, which is fine. But :unchat doesn't return the vanilla chat.

What are the steps that lead to the bug occurring? Or, how frequently does the bug occur?
Always when we use :chat
What would be the normally expected or desired result?
That it would return the vanilla chat
What is the actual result (the bug)?
When you do :unchat the vanilla chat doesn't re-appear
What is the impact of this bug?
Breaks gameplay badly, requires rejoin or manuially running a local script

This is very problematic for our game which uses radios with the adonis chat under the hood. We had to remove radios fully due to this bug and its hugely game breaking.

This is very problematic for our game which uses radios with the adonis chat under the hood. We had to remove radios fully due to this bug and its hugely game breaking.

L

This is intended behavior based on user feedback. I do understand this is a major concern of cause, but please vote for this in our feedback portal.


rebar3: 3.6.1 -> 3.8.0

Motivation for this change

Update package
Simplify build

Things done

Updated rebar3 package

Removed incompatible hermetic patch (nix-build runs with network isolation, hence the hermetic patch is not needed anymore)

Started eliminating the need of hex pac

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

Built on platform(s)

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"

[x] Tested execution of all binary files (usually in ./result/bin/)

[ ] Determined the impact on package closure size (by running nix path-info -S before and after)

[ ] Assured whether relevant documentation is up to date

[x] Fits CONTRIBUTING.md.

Unfortunately hashes used by hex are different from what could be used by nix. They concatenate the source tarball together with some metadata, which is unknown in advance [1]. It is still possible to use hex-style hash in a fixed-output derivation, but via a hack so dirty I don't even want to talk about. So I had to resort to a fixed-output derivations containing downloaded sources. It is used by Rust people, however this practice is disputed [2]
[1] https://github.com/hexpm/hex_core/blob/bfd012f7200091cac2901850b2f7a828f7bf6301/src/hex_tarball.erl#L244
[2] https://github.com/NixOS/nix/issues/2270

@GrahamcOfBorg build hex2nix relxExe

@Mic92 Sorry for the delay. Both hex2nix and relxExe build now and nox doesn't report any errors.

Since this is still open, would you mind bumping ot 3.9.0?

@ankhers Done, also squashed commits to reduce noise

I'm good with this. My only other suggestion would be to drop rebar3-open since it is effectively useless. But we can do that in another PR if preferred.

Someone may use it. Is there a good way to deprecate a package?

Based on a conversation in IRC, there doesn't seem to be a way to deprecate things. We may just need to change some docs the refer to it. Lets keep it for now and we can clear it out in a different PR.

If possible, I would like to update docs through a separate PR after the rest of planned changes are proven viable and implemented. After that both old hex2nix packages and rebar3-open can be nuked.

@ankhers May I have this one approved so I can move on to the next step?

I can't actually merge. Will need to wait for someone else to do that.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/state-of-the-beam-ecosystem-in-nix/4202/2

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/state-of-the-beam-ecosystem-in-nix/4202/2

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/state-of-the-beam-ecosystem-in-nix/4202/2


Disable the password grant type

The password grant type should be disabled. Enabling it just encourages the development of insecure clients that are doing who-knows-what with your login info. Clients should be required to use the authorization_code grant type. The only location users should be entering their username and password is on the q.daskeyboard.com domain. Best practices are outlined in RFC 8252.

What grant do you suggest for a shell script / node script?

https://tools.ietf.org/html/rfc8252#section-8.11
From what I understand, the shell script / node app should open the users browser, where they should use standard oauth flow. Users should never put their credentials into an app other than the das website running on https.

The flow would look like this.

The script sets up temporary, lightweight HTTP server that listens at a loopback URI.
The script opens a browser to the authorization endpoint to obtain an authorization code grant. The redirection endpoint (redirect_uri) would be the loopback URI.
The user enters their q.daskeyboard.com credentials and signs in.
q.daskeyboard.com authenticates the user and redirects the browser to the loopback redirect_uri with the authorization code added to the query string.
The script's HTTP server accepts the HTTP request at the loopback redirect_uri and retrieves the code from the query string.
The script uses the code to make an authorization code grant type request to obtain an access_token.
The script responds to the browser's HTTP request notifying the user that authentication was successful and that they may close the browser.

rclone's OneDrive implementation is a good example of this flow, and it even accounts for cases where the computer does not have a browser installed, like on a Linux server.

In the case where a computer does not have browser, you could also provide an alternative endpoint where users would enter their credentials and receive an authorization code that they enter into the script's prompt. Here's how that might look.

The script instructs the user to open a browser (perhaps on their phone) to the alternative endpoint.
The user enters their q.daskeyboard.com credentials and signs in.
q.daskeyboard.com authenticates the user and provides an authorization code.
The user enters that code into the script's prompt.
The script uses the code to make an authorization code grant type request to obtain an access token.

There's also device flow. This is the same type of flow that's used for devices with limited input capabilities, like smart TVs, and it's similar to the "alternative endpoint" flow I described in my last comment. Here's how it works:

The script makes a request to q.daskeyboard.com at the device flow endpoint and includes its client_id.
q.daskeyboard.com responds with a device_code, user_code, and verification_uri.
The script instructs the user to visit the verification_uri in a browser and enter the user_code.
The user visits the verification_uri, signs in if necessary, and enters the user_code.
In the meantime, the script repeatedly polls q.daskeyboard.com to see if the user has entered the user_code.
Once the user has entered the user_code q.daskeyboard.com responds to the client with an access_token.


training on Linemod

Hi, I try to reproduce your experiments on Linemod.
Since you have different models for different objects, does that imply you split dataset to multiple and train for different objects?
e.g., there are 10000 records in total, and 1000 records for ape. So do you train the ape mode on this 1000 subset?

what's more, acc-0.1d for nearly 100 percents. That is really amazing. Since diameter of those objects are around 10-20cm, that means almost all add are in 1-2cm. Really impressive.


Close tags as you open them

VSCode Version: 1.4
OS Version: Windows 10

Steps to Reproduce:

HTML tags should be automatically closed as I open them. This used to be available before, don't know what happened since I updated.
Also I should get snippets based on the CSS and JS files linked to the document. For example I included bootstrap.css in my html, I should get suggestion for all the style classes included in that css file.

duplicate of #3000


1.3.4 in OS X 10.11.2 can't access preferences.

I apologize for the length of this issue: I don't have the vocabulary to make it shorter.
OS X 10.11.2
QS 1.3.4
If I try to open Quicksilver preferences, the QS preferences window pops up but then the window for the last open app gets the focus.
So for example, if I am using Firefox after the Finder and I try to open QS preferences, the QS preference window appears, but the Finder window is in the foreground.
Then if I click on the QS preferences window that is showing behind the Finder window, the Firefox window jumps to the foreground. If I try to click back on the QS preferences, it goes back to Finder. It's as if clicking on the preferences window acts as a control-tab.
It actually gets stranger. If I try to open Dropbox preferences while QS is running, the Dropbox preference window appears in the foreground, but clicking on it (the Dropbox preference window) switches between Finder and Firefox. I cannot make changes to Dropbox preferences because, even though that window is in the foreground, the focus is on Firefox or Finder.
Other preferences windows (for exampopened from the menu bar exhibit the same behavior when QS is running.
I can drag the windows around, but cannot focus on them to make changes.
I can close the preference window (Dropbox or Quicksilver or whatever) by clicking the close button in the top left corner of the window.
If QS is not running, other preferences windows (including Dropbox) work fine.
When I quit QS everything seems to work fine.
When I quit everything but QS the problem persists.

My guess is that you’ve somehow enabled the “Hide other applications when switching applications (shift prevents)” preference.
If you open Quicksilver’s preferences and go to Extras, you should see it. It sounds like you want it disabled. If you’re having trouble getting the window to stay in front, holding Shift when you click it should help (as the name of the setting implies).

Your guess is correct. Brilliant, thanks.


Create the user dashboard and display some preliminary data regarding their enabledModules

Set up enabledModules in the user document (don't worry about the collections for each different module yet)
Adjust the seed.js file to account for this and initialize one or two users with different enabledModules
Edit the dashboard.handlebars file to render some text for each enabledModule (this will simulate the module's partial later on)

Closed as I've created a new, more updated: issue #15


[New Theme Request] MovieMatch

This app lets people swipe left/right on the movies in your Plex server to find a movie to watch together. It would look so good with a Plex skin.
https://github.com/LukeChannings/moviematch

Nice, added it now. Super simple UI so it was a quick job.


"Argument list too long" error on some repos

I'm getting the following error when scanning some repos:
/usr/local/bin/git-secrets: line 56: /usr/bin/grep: Argument list too long

I'm guessing that the repos in question are two large.  The two that fail have a size-pack of 205,724 and 106,999 (generated with this command: git count-objects -v)

I'm guessing this is the result of running git secrets --scan with no directory? When run with no directory, we set the path to each file in the repository to a variable and then pass that to grep. I'm guessing that is tripping up grep. We have another issue related to this behavior in at https://github.com/awslabs/git-secrets/issues/5. I think we could either iterate over the files and scan them individually, or use git-grep when scanning repository files. I'll need to do some research to see what's possible.
Thanks for reporting.

Nope, this happens with repos that are very, very large, with lots of directories and files.

Is there any resolution for this?

Confirmed behaviour when scanning openssl on OSX.
$ git clone https://github.com/openssl/openssl.git
$ cd openssl
$ git-secrets --scan-history
/opt/homebrew/bin/git-secrets: line 105: /usr/bin/git: Argument list too long


Mattermost 4.4.0 not starting on Linux

I confirm (by marking "x" in the [ ] below: [x]):

[x] This is not a troubleshooting question. Troubleshooting questions go here: http://www.mattermost.org/troubleshoot/.
[ X ] This doesn't reproduce on web browsers (such as in Chrome). If it does, issue reports go to the Mattermost Server repository.
[ X ] I have read contributing guidelines.

Summary
Installation of client 4.4.0 for Linux doesn't work, while 4.3.2 does work just fine
Environment

Operating System: Fedora 30 Linux
Mattermost Desktop App version: 4.4.0
Mattermost Server version: 5.19.1

Steps to reproduce

Download client from Mattermost web site
Extract to a new directory
Run "mattermost-desktop

Even though I extracted the binaries to a clean directory, I was prompted with a pop-up window indicating that there were existing files that would be replaced. Odd.
Expected behavior
Have the client start, just like I'm still able to with the 4.3.2 version
Observed behavior
$ ./mattermost-desktop
internal/modules/cjs/loader.js:630
    throw err;
    ^

Error: Cannot find module '/home/martin/bin/mattermost-desktop-4.4.0-linux-x64/resources/electron.asar/browser/init.js'
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:627:15)
    at Function.Module._load (internal/modules/cjs/loader.js:531:27)
    at Function.Module.runMain (internal/modules/cjs/loader.js:850:10)
    at internal/main/run_main_module.js:17:11 {
  code: 'MODULE_NOT_FOUND',
  requireStack: []
}

Possible fixes
:man_shrugging:

@martinmarques what package did you download from the releases page? Just the regular .tar.gz?
I'm also running Fedora and would like to confirm this.

It's the tarball from https://docs.mattermost.com/install/desktop.html#linux
More specifically, this one: https://releases.mattermost.com/desktop/4.4.0/mattermost-desktop-4.4.0-linux-x64.tar.gz

It's the tarball from https://docs.mattermost.com/install/desktop.html#linux
More specifically, this one: https://releases.mattermost.com/desktop/4.4.0/mattermost-desktop-4.4.0-linux-x64.tar.gz

Interesting, I could start this without any issues.
@martinmarques can you try the rc4 release and see if that one works?
https://releases.mattermost.com/desktop/4.4.0-rc4/mattermost-desktop-4.4.0-rc4-linux-x64.tar.gz

Same issue. I'm going to restart my laptop later and see if it's something that's left in cache

Restarted the graphical session. No joy.
Rebooted Linux, same issue, even with the RC4 version.

tested on ubuntu, it works. I currently don't have fedora, but I can install it on a vm
@martinmarques  Do you have any especial config setup? different DE or anything away from the defaults?

I use the plasma environment. I don't believe I have anything special setup on this laptop.

I was able to make it work on a desktop computer with Fedora 30 and plasma desktop. What I did see there was that the look&feel is completely different from what I see here with mattermost 4.3.2. This looks very much like plasma desktop, while on the desktop computer it has more of a chrome look&feel

Maybe this helps. It's part of the strace output, at the end:
madvise(0x3f8774394000, 36864, MADV_DONTNEED) = 0
futex(0x3f8773f99d68, FUTEX_WAKE_PRIVATE, 1) = 1
mmap(0x3017e3ac0000, 520192, PROT_NONE, MAP_PRIVATE|MAP_ANONYMOUS|MAP_NORESERVE, -1, 0) = 0x3017e3ac0000
munmap(0x3017e3b00000, 258048)          = 0
mprotect(0x3017e3ac0000, 262144, PROT_READ|PROT_WRITE) = 0
getpid()                                = 14793
readlink("/proc/self/exe", "/home/martin/mattermost/mattermo"..., 8191) = 42
getuid()                                = 1000
geteuid()                               = 1000
getgid()                                = 1000
getegid()                               = 1000
getuid()                                = 1000
geteuid()                               = 1000
getgid()                                = 1000
getegid()                               = 1000
mprotect(0x2da2d9f82000, 249856, PROT_READ|PROT_WRITE) = 0
mprotect(0x2da2d9f82000, 249856, PROT_READ|PROT_EXEC) = 0
mprotect(0x2da2d9f82000, 249856, PROT_READ|PROT_WRITE) = 0
mprotect(0x2da2d9f82000, 249856, PROT_READ|PROT_EXEC) = 0
stat("/home/martin/mattermost/resources/electron.asar", 0x7fffb64743d8) = -1 ENOENT (No existe el fichero o el directorio)
openat(AT_FDCWD, "/home/martin/mattermost/resources/electron.asar", O_RDONLY) = -1 ENOENT (No existe el fichero o el directorio)
openat(AT_FDCWD, "/home/martin/mattermost/resources/electron.asar", O_RDONLY) = -1 ENOENT (No existe el fichero o el directorio)
openat(AT_FDCWD, "/home/martin/mattermost/resources/electron.asar", O_RDONLY) = -1 ENOENT (No existe el fichero o el directorio)
openat(AT_FDCWD, "/home/martin/mattermost/resources/electron.asar", O_RDONLY) = -1 ENOENT (No existe el fichero o el directorio)
openat(AT_FDCWD, "/home/martin/mattermost/resources/electron.asar", O_RDONLY) = -1 ENOENT (No existe el fichero o el directorio)
ioctl(2, TCGETS, 0x7fffb6473e90)        = -1 ENOTTY (Función ioctl no apropiada para el dispositivo)
fstat(2, {st_mode=S_IFREG|0664, st_size=220080, ...}) = 0
madvise(0x3f87743b3000, 36864, MADV_DONTNEED) = 0
madvise(0x3f87743a2000, 69632, MADV_DONTNEED) = 0
madvise(0x3f8774399000, 36864, MADV_DONTNEED) = 0
madvise(0x3f8774399000, 36864, MADV_DONTNEED) = 0
mprotect(0x2da2d9f82000, 249856, PROT_READ|PROT_WRITE) = 0
mprotect(0x2da2d9f82000, 249856, PROT_READ|PROT_EXEC) = 0
mprotect(0x2da2d9f82000, 249856, PROT_READ|PROT_WRITE) = 0
mprotect(0x2da2d9f82000, 249856, PROT_READ|PROT_EXEC) = 0
write(2, "internal/modules/cjs/loader.js:6"..., 466internal/modules/cjs/loader.js:630
    throw err;
    ^

Error: Cannot find module '/home/martin/mattermost/resources/electron.asar/browser/init.js'
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:627:15)
    at Function.Module._load (internal/modules/cjs/loader.js:531:27)
    at Function.Module.runMain (internal/modules/cjs/loader.js:850:10)
    at internal/main/run_main_module.js:17:11 {
  code: 'MODULE_NOT_FOUND',
  requireStack: []
}
) = 466
madvise(0x3f8774037000, 4096, MADV_DONTNEED) = 0
madvise(0x3f8774032000, 4096, MADV_DONTNEED) = 0
madvise(0x3f8774356000, 12288, MADV_DONTNEED) = 0
madvise(0x3f8774381000, 16384, MADV_DONTNEED) = 0
madvise(0x3f87742f2000, 20480, MADV_DONTNEED) = 0
madvise(0x3f877431e000, 24576, MADV_DONTNEED) = 0
madvise(0x3f87742cd000, 32768, MADV_DONTNEED) = 0
madvise(0x3f8774341000, 12288, MADV_DONTNEED) = 0
close(29)                               = 0
close(30)                               = 0
madvise(0x3f8773e21000, 4096, MADV_DONTNEED) = 0
madvise(0x3f8773e22000, 4096, MADV_DONTNEED) = 0
madvise(0x3f8773e0d000, 4096, MADV_DONTNEED) = 0
madvise(0x3f8773e0e000, 4096, MADV_DONTNEED) = 0
madvise(0x3f8773e0f000, 4096, MADV_DONTNEED) = 0
madvise(0x3f8773e10000, 4096, MADV_DONTNEED) = 0
madvise(0x3f8773e11000, 4096, MADV_DONTNEED) = 0
madvise(0x3f8773e12000, 4096, MADV_DONTNEED) = 0
madvise(0x3f8773e13000, 4096, MADV_DONTNEED) = 0
madvise(0x3f8773e14000, 4096, MADV_DONTNEED) = 0
madvise(0x3f8773e15000, 4096, MADV_DONTNEED) = 0
madvise(0x3f8773e16000, 4096, MADV_DONTNEED) = 0
madvise(0x3f8773e18000, 4096, MADV_DONTNEED) = 0
madvise(0x3f8773e19000, 4096, MADV_DONTNEED) = 0
exit_group(1)                           = ?
+++ exited with 1 +++

given that you saw this warning:

Even though I extracted the binaries to a clean directory, I was prompted with a pop-up window indicating that there were existing files that would be replaced. Odd.

I'm guessing you wrote over a previous installation of the app, can you try using a new directory or removing the original one and see if you still get the warning and where it is actually unarchiving the files? and also see if the problem reproduces

When I say clean directory I meant a newly created directory.

yes, i understood, but that pop up window should not happen, right? I don't know of a way to unarchive into a new directory and get it (which doesn't mean there isn't a way, it just never happened to me). So i'd recomend double checking just to ensure we are not trying to debug something that is solved by a clean install.

Oh dear. It seems there was something wrong with where ark was placing the target files (seems with the years I've become too lazy and keep using these GUI).
I downloaded the binary to disk and manually used tar from the console (what I should have done to start with, or at least after the first failure) and now it works just fine.
Sorry for the noise and the time wasted.


Update license to BSD?

I like this remark taken from here

Illustrating how subtle language variations matter, we recently decided to change our pick to the BSD 3-clause License. It is very similar to the MIT License, but there is a small difference in the wording about attribution (which we learned about from a comment by A. Scopatz on a GitHub discussion thread).

The MIT License says:

The above copyright notice and this permission notice shall be included in all copies or substantial portions of the Software.

While BSD 3-clause says:

Redistributions of source code must retain the above copyright notice, this list of conditions and the following disclaimer.

As Scopatz noted, this language implies that a user could copy some of an MIT-licensed code (as long as the portion is deemed "not substantial") without attribution. In academia, we always prefer full attribution of any portions of copied works, and BSD 3-clause is more precise in this.

I propose we go to BSD license.
What do you think @jbarnoud @HubLot ?

I'm not fully competent to have a critical comparison between MIT and BSD so I let you decide guys :)

I have nothing against any of these licenses. The 3-clauses BSD seems slightly less ambiguous, but I doubt the difference actually matters at the scale of the project.


Completed recipes folder is missing in the download

Hi, in th video, Getting started - for teachers, you show  a folder named "completed" in src for the teachers which is missing in the download, i browsed the code here as well, it is also missing
thanks for your help

This is by design.  The 'working' download (for the classroom and students) does NOT include the answers.
We have linked the answers for each recipe/lesson in the Penflip (teacher lesson plans) - see http://teachingkidsprogramming.org/courses/ > course01... > 'download lesson plan'
Alternatively, if you, as a teacher, would like to use the recipe code answers to help for your own preparation they are here on Github.


Extract Predicted Covariance Matrix from gpvar.GPVAREstimator & deepvar.DeepVAREstimator

Hi,
Does the current implementation of gpvar.GPVAREstimator & deepvar.DeepVAREstimator allow to extract the predicted Covariance Matrix. I am asking because it is mentioned in the paper
High-Dimensional Multivariate Forecasting with Low-Rank Gaussian Copula Processes: Figure 1: Top: covariance matrix predicted by our model.
Would be really great to have that information available.
Thanks!

Hi @StatMixedML,
you can get the covariance matrix using the code here.
One note: There is no "the covariance matrix" because the covariance is estimated using the the current data context. Thus, every time you pass different data into the network the covariance matrix is estimated with respect to that data. You can also refer to the code above how to do that.
Hope that helps.

Thanks @mbohlkeschneider! Appreciate your help.
I close the issue now.

Hi! @mbohlkeschneider
I hope that you are here, sorry to bother you.
The code seems to be a code for training, and it will unroll for context_length on future data, do you have codes for prediction? I tried with InferenceDataLoader and Predictor.prediction_net. It doesn't seem to work (It throws me a dim*num_samples ndarray, which I cannot comprehend and extract distr_args). I also tried with DistributionForecast. It also threw me an error.

I have not written code to do this during inference. Can you share a reproducible example?


Create base_controller.dart

Added a file base_controller.dart

Good going @SharviE29 💯
For your next pull requests, make sure you comment on the issue that you're working on.
This helps new contributors understand that the issue is taken by you.
Everything else is done pretty well✨ If you ever need any help, feel free to reach out to me or other mentors on discord :)


Note where one should expect BS j/js string extension syntax to work

The documentation is going to be split between BuckleScript  and OCaml Native related Reason, otherwise mixing all those repls / playgrounds and different runtime characteristics will furthermore confuse our newcomers,... so I'd refrain from adding this section in the String docs.
That said the differences between nit and sketch could probably just be mentioned in the "Extra Goodie" part: https://reasonml.org/docs/manual/latest/extra-goodies#repl

We are closing this for now since there will be some big change coming up to the whole docs structure. Feel free to reopen when things are still unclear later on!


Add Travis

Description
Add travis for CI
Possible Solution
Travis CI is a hosted continuous integration service used to build and test software projects hosted at GitHub and Bitbucket. Travis CI provides various paid plans for private projects, and a free plan for open source. TravisPro provides custom deployments of a proprietary version on the customer's own hardware.
Checklist

[X] I have read and follow the CONTRIBUTING.md document.
[X] There are no other issues similar to this one.
[X] The issue title and template are correctly filled.

We use GitHub Actions for CI.


ICC profile support

Not that this should be added for first release, or indeed soon, but just so I can close some browser tabs :)

JavaScript source code to parse ICC profile, version 4

Javascript notes for the above

node-transicc which is a node wrapper around

Little-CMS

To be clear, this is a future "nice to have". For additive colorspaces, our code does the same thing. The main benefits would be:

Cross checking the JS code against the ICC result, for existing supported spaces
Add support for non-additive colorspaces (CMYK, CMTKOGV) which require multidimensional interpolation of nearby measured color values.

To be clear, this is a future "nice to have". For additive colorspaces, our code does the same thing. The main benefits would be:

Cross checking the JS code against the ICC result, for existing supported spaces
Add support for non-additive colorspaces (CMYK, CMTKOGV) which require multidimensional interpolation of nearby measured color values.

Indeed.
For anyone interested in embarking on this:

I’m envisioning a ColorSpace.load() async factory method, that would read a path/URL to an ICC profile and return a ColorSpace object that behaves like normal ColorSpace objects. We will likely need to introduce a few constructor options to make this possible.
ICC profile parsing is quite involved, but there are also many libraries on it (mostly in other languages) that can be used as a base.

I found this library https://github.com/glennwilton/jsColorEngine and result is quite similar compare to Little-CMS and written in 100% Javascript.


How to install this as intellij-plugin

Hi,
How to install this as intellij-plugin? I can only clone the source code, where is the zip file for installed? Thanks!

plugins.jetbrains.com
or when you are trying to open a YANG file, your IDE will suggest installing this plugin. Assuming i did not forget to update it to the next major release of course :)


Add -ObjC flag to fix static linking on macOS

Description
Fixes regression introduced in #2174 of a bug fixed in #1485.
I think this made it through CI since it only appears when running certain Window module code and our tests don't run that since such tests would fail on Linux.

SFMLBuildMaster: Build this please.

Seems a good opportunity to add a test to catch this regression? Even if it's disabled on linux (what is it that fails?)

Seems a good opportunity to add a test to catch this regression?

Once #2302 is done, that may be possible. I hope by adding some basic tests for RenderWindow it will implicitly catch this bug.


TPU Colab fails with AttributeError: module 'jax.tree_util' has no attribute 'register_pytree_with_keys_class'

On a fresh Colab TPU runtime, we have:
>>> import flax
AttributeError                            Traceback (most recent call last)
[<ipython-input-2-f5b294e0faf0>](https://localhost:8080/#) in <cell line: 2>()
      1 # Verify we can import everything.
----> 2 import flax
      3 from flax.training import (checkpoints, dynamic_scale, early_stopping, lr_schedule,
      4                            orbax_utils, prefetch_iterator, train_state, common_utils)
      5 from flax.metrics import tensorboard

2 frames
[/usr/local/lib/python3.10/dist-packages/flax/core/frozen_dict.py](https://localhost:8080/#) in <module>
     48
     49
---> 50 @jax.tree_util.register_pytree_with_keys_class
     51 class FrozenDict(Mapping[K, V]):
     52   """An immutable variant of the Python dict."""

AttributeError: module 'jax.tree_util' has no attribute 'register_pytree_with_keys_class'

Note the versions:
!pip freeze | egrep 'jax|flax'
flax==0.7.0
jax==0.3.25
jaxlib==0.3.25

This is due to Colab TPU runtimes being pinned to an old jax==0.3.25 (because TPU runtime does only support legacy "TPU Node" setup).
Flax has been requiring jax>0.3.25 for a while, but recent changes make the incompatibility visible on a simple import.
Ideally we should pin flax==0.6.4 (most recent version that supports jax<=0.3.25).

This was resolved by pinning Flax version as well on TPU runtimes; we now have:
>>> import jax
>>> jax.__version__
'0.3.25'
>>> import flax
>>> flax.__version__
'0.6.4'


Change all URIFromDestinationV1 to AddressableFromDestinationV1

Fixes #3109

Proposed Changes

URIFromDestinationV1 and AddressableFromDestinationV1 are 2 function do the same thing, but their return types are different. URIFromDestinationV1 will only URL string, however, AddressableFromDestinationV1 will return a Destination object, which contains more information (e.g CAcert)

Release Note

Docs

/ok-to-test

/do-not-test

/hold

maybe @pierDipi knows the answer, afaik, tests are run on every pr everytime a change is made, but that should not be a problem. even hold won't stop testing (lgtm will remove the hold)

I don't think we can stop it :)
it will run the tests when you push new changes

/unhold
/lgtm

Yes
/approve

I don't think we can stop it :)

it will run the tests when you push new changes

Perhaps "DRAFT" PRs do not have that? But I am not sure, and would not too much worry about that

I pulled up a draft PR here. It seems like some tests are suspended, some tests are still running. @matzew

Usually the Github actions run always. Only the Prow jobs will not run on draft PRs.

I'm also not sure if you start with ready PR and then convert to draft (which was the case in this PR), does it stop?


[puz2xd] support block-style cryptics

I tested this with the Dec 2009 puzzle here: https://kegler.gitlab.io/Block_style/

++ for linking to a sample cryptic!


chore: scalex-jenkinsx-rollout-app to 0.0.2

chore: Promote scalex-jenkinsx-rollout-app to version 0.0.2

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@JoshuaCWebDeveloper: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

promotion-build
83ff6e1899ebc9a01eb5b1b4cd0bdfe696ed5cfa
link
/test this

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


[QUESTION] Mem usage is the same for 2 gpus and 4 gpus.

Description
Hi team, I’m trying the a deepspeed project on two gpus, and I see that the mem usage per gpu is about 13G, but using the estimate_zero3_model_states_mem_needs function estimator, they mem usage should be about 0.77G.
And futher, if I use 4 gpus, the mem usage is the same as 2 gpus.
I think if we use zero-3, the mem usage should decrese linearly right.
Does anyone know why this is the case? Thanks a lot!
Model: gpt-j-6b
mem estimate:

Actual usage on 2 gpus:

On 4 gpus(batch size is the same as 2 gpus):

Config I use:
{
"fp16": {
"enabled": "auto",
"loss_scale": 0,
"loss_scale_window": 1000,
"initial_scale_power": 16,
"hysteresis": 2,
"min_loss_scale": 1
},
"optimizer": {
    "type": "AdamW",
    "params": {
        "lr": "auto",
        "betas": "auto",
        "eps": "auto",
        "weight_decay": "auto"
    }
},

"scheduler": {
    "type": "WarmupLR",
    "params": {
        "warmup_min_lr": "auto",
        "warmup_max_lr": "auto",
        "warmup_num_steps": "auto"
    }
},

"zero_optimization": {
    "stage": 3,
    "offload_optimizer": {
        "device": "cpu",
        "pin_memory": true
    },
    "offload_param": {
        "device": "cpu",
        "pin_memory": true
    },
    "overlap_comm": true,
    "contiguous_gradients": true,
    "sub_group_size": 1e9,
    "reduce_bucket_size": "auto",
    "stage3_prefetch_bucket_size": "auto",
    "stage3_param_persistence_threshold": "auto",
    "stage3_max_live_parameters": 1e9,
    "stage3_max_reuse_distance": 1e9,
    "stage3_gather_16bit_weights_on_model_save": true
},

"gradient_accumulation_steps": "auto",
"gradient_clipping": "auto",
"steps_per_print": 2000,
"train_batch_size": "auto",
"train_micro_batch_size_per_gpu": "auto",
"wall_clock_breakdown": false

}

Not sure if this is whats happening to you, but atleast what i observe in my projects (im not a dev for deepspeed)
Unless you are clearing the cache between each step (you probably shouldn't), DS3 will gladly use the remaining vram space and fill it up with the some weights in other GPUs if possible - which im guessing to avoid cross GPU communication - which can be a bottleneck

Not sure if this is whats happening to you, but atleast what i observe in my projects (im not a dev for deepspeed)
Unless you are clearing the cache between each step (you probably shouldn't), DS3 will gladly use the remaining vram space and fill it up with some weights from other GPUs if possible - which im guessing is to avoid cross GPU communication - which can be a bottleneck
However if you do test it with GPUs with less vram, you get to see if it fits with less vram

Thanks for the information! I got your idea, so you're saying that, if we didn't set a large batch size, and there is extra space after plain DS3 is initialized, then DS3 will automatically fill these space with weights to accelerate the communication?

yup, if you are looking at a DS2 vs DS3 vram usage.
Assuming the model fit when in DS2 mode.
I dun think it has anything to do with batch size. Because what DS3 does is that it removes duplicate copies of the weights across the GPU's, but this forces the GPU's to read from one another instead for those weight values.
This is great for super large models that dun fit into a GPU. But is a slow process, hence when possible it will try to cache some of those weights, all the way up to effectively the same as DS2.
PS: this is my understanding from my observation, we might still want someone official to confirm

Hi team, I’m trying the a deepspeed project on two gpus, and I see that the mem usage per gpu is about 13G, but using the estimate_zero3_model_states_mem_needs function estimator, they mem usage should be about 0.77G.

@ilml, thanks for this question.
First, looking closely at the estimator output, you will see that 0.77G per GPU is for the case of offloading both parameters and optimizers.
Secondly, these estimates are only for model and optimizer state memory consumption and does not account for activation and gradient memory consumption that would occur during training. And so, the above screenshots are not appropriate comparison points.  One way to get a more precise memory consumption is to call the see_memory_usage() utility after model and optimizer initialization (e.g., deepspeed.intialize()). This utility leverages torch.cuda memory management routines.

Hello, I have a question for ZeRO3 memory usage too. When I'm training a 13B model with 8 machines of 8 * A100, the maximum seq length can be set to 25k. However, when the machines num come to 12, 25k seq length will lead to OOM. As you say, the activation seems to be a bottleneck here, but don't know why the seq length can't be the same.

Hi team, I’m trying the a deepspeed project on two gpus, and I see that the mem usage per gpu is about 13G, but using the estimate_zero3_model_states_mem_needs function estimator, they mem usage should be about 0.77G.

@ilml, thanks for this question.
First, looking closely at the estimator output, you will see that 0.77G per GPU is for the case of offloading both parameters and optimizers.
Secondly, these estimates are only for model and optimizer state memory consumption and does not account for activation and gradient memory consumption that would occur during training. And so, the above screenshots are not appropriate comparison points. One way to get a more precise memory consumption is to call the see_memory_usage() utility after model and optimizer initialization (e.g., deepspeed.intialize()). This utility leverages torch.cuda memory management routines.

Hello, I have a question for ZeRO3 memory usage too. When I'm training a 13B model with 8 machines of 8 * A100, the maximum seq length can be set to 25k. However, when the machines num comes to 12, 25k seq length will lead to OOM. As you say, the activation seems to be a bottleneck here, but don't know why the seq length can't be the same.


Run test on windows in GitHub Actions.

We have Travis
The badge reports errors
The link errors: Project not found or access denied.

appveyor is supposed to test Windows, but I think we can probably try out the new Windows support of travis: https://blog.travis-ci.com/2018-10-11-windows-early-release

Updated the title and removed the stale label since this is still relevant for GitHub Actions we currently use.


scroll in tmux does not get the same result as ubuntu terminal does

this is ubuntu terminal get when scroll up in terminal ( it is true history result of terminal)

this is terminal get

Just to be certain: this is what happens when you use your mouse to scroll, and tmux scrolls off the bottom of the screen instead of staying in place and scrolling inside it?

(Note to self: i think this is probably related to not supporting alt buffer at the terminal level (outside of conpty))

what do you mean scroll inside it ?

Sorry, I was attempting to determine whether you were scrolling the actual terminal history or whether your mouse events were going into tmux and tmux was implementing its own scrolling.
Judging by the tmux indicator in the top right of the "working" screenshot, this is caused by a lack of mouse support in Windows Terminal and ConPTY. #545 #376.


Clean up #if SILVERLIGHT

PR Summary
Cleanup unused #if SILVERLIGHT code.
Discovered during psl code review.
PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Change is not breaking
[x] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: to the beginning of the title and remove the prefix when the PR is ready.

User-facing changes

[x] Not Applicable
OR
[ ] User-facing Documentation needed

[ ] Issue filed - Issue link:

Testing - New and feature

[x] Not Applicable or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

[ ] Add [feature] if the change is significant or affects feature tests

@iSazonov there are a couple other places where SILVERLIGHT is called. Do you want tor clean them in this PR or track in an issue?
Also, could you push [feature]? I'd feel more at ease with engine changes if the full test suite runs, even when one is as obvious and simple as this.


KV pair ir stream (IR_v2)  --> clp_s archive format

Description

This PR:

Adds IR V2 to archive format conversion.
Exposes the JSON to IRV2 parsing to the user through the command line
Enables users to write the IRV2 format to a file.

Validation performed

Validated IR conversion, compression, and extraction to JSON

Generated IR V2 format for all 5 JSON public datasets

ex) ./clp-s r elasticsearch_ir elasticsearch/

Compressed those IRs into Archive

ex) ./clp-s i elasticsearch_archive elasticsearch_ir/

Extracted those archives back to JSON

ex) ./clp-s x elasticsearch_archive elasticsearch_out/

Compress and Extract Json using clp-s for comparison

./clp-s c elasticsearch_clp-s_archive elasticsearch/
./clp-s x elasticsearch_clp-s_archive elasticsearch_clp-s_out/

Sorted and compared the spark, elasticsearch, and postgresql datasets' JSON received to original and to clp_s for exact match

ex)

jq -S -c '.' elasticsearch_out/original | sort > elasticsearch_sorted.json
jq -S -c '.' elasticsearch_clp-s_out/original | sort > elasticsearch_clp-s_sorted.json
diff elasticsearch_clp-s_sorted.json elasticsearch_sorted.json | diffstat

Spot checked match of momgodb and cockroach database
Notes:

I noticed some differences in numbers of significant digits for floats between both clp-s and the IRv2 to archive, and the original JSON.
ex.) original: 1.0; clp_s: 1.00000; IRv2: 1.00000
ex.) original: 0.9758767; clp_s: 0.9758767; IRv2: 0.975877

@coderabbitai review

Discussed offline: The current plan is to create a new PR with the same functionality, but based off of main.

Can we close this now that #618 & #630 were merged?


Cassandra/Kafka usage question

Hello. Not sure it's a good place to ask questions, but you might move this Q/A to the wiki section if you find it relevant.
What is the best way to get connected to the services like Cassandra/Kafka from other applications (Java code, for example)? I need to give my developers IP addresses of the service endpoints, but since the EC2 instances might be terminated and started new ones at any time, that IP addresses will be changed and an application will lose access to the service.

Every service member will get a unique dns name. For example, the cluster is testcluster, Cassandra service name is mycas with 3 replicas. FireCamp will assign the dns name, mycas-0.testcluster-firecamp.com, mycas-1.testcluster-firecamp.com, and mycas-2.testcluster-firecamp.com, to the 3 Cassandra containers. The applications could simply access the Cassandra cluster via these 3 dns names.
When the EC2 instance goes down, let's say the EC2 of mycas-1.testcluster-firecamp.com, AutoScaleGroup will start a new EC2. ECS will schedule the Cassandra container to the new EC2. FireCamp will attach the original EBS volume that belongs to mycas-1.testcluster-firecamp.com, and update the address of mycas-1.testcluster-firecamp.com to the new EC2's privateIP in Route53.
If the applications use JVM, you will need to set JVM TTL. Please check Cassandra/Kafka readme, catalog/cassandra/README.md. "By default, JVM caches a successful DNS lookup forever. If you use Cassandra Java CQL driver, please set JVM TTL to a reasonable value such as 60 seconds. So when Cassandra container moves to another node, Java CQL driver could lookup the new address."

added the explanation to readme.


Graph going blank randomly after a while

Graph goes blank after a while randomly .  Console logs show the following error .

We currently have async rendering setting on for the graph .
Expected Behavior
Graph shouldn't show any errors or handle the rendering by itself .

Current Behavior
Graph goes blank , and user needs to refresh to load objects .

Possible Solution
Is this a problem with async rendering ? Unsure on the fix as it seems native to the library.
Steps To Reproduce

...
...
...

Error Message & Stack Trace

```txt

```

Additional Context

Your Environment

x6: ^1.21.5
OS:  Not OS specific
Browser: All browsers

Please provide a demo that can be reproduced.

无法重现，先关闭。


update to new gemma

actual fix is rather simple but this is untested in it's actual context

Seems like it should work in context to me. Will solve #12


Add force redirect

Allow to do something like given().redirects().alwaysFollow(true) to get around RFC 2616:

If the 301 status code is received in response to a request other
than GET or HEAD, the user agent MUST NOT automatically redirect the
request unless it can be confirmed by the user, since this might
change the conditions under which the request was issued.

This is how one can configure the http client instance:
DefaultHttpClient client = new DefaultHttpClient();
client.setRedirectStrategy(new DefaultRedirectStrategy() {
    @Override
    protected boolean isRedirectable(String method) {
        return true;
    }
});

Hi,
yes, such functionality would be very useful. I ended up here after searching for why many of my RestAssured tests are completely broken, as RestAssured is not automatically following permanent redirects (301).
Few points to consider: for HTTP 1/1 the RFC 2616 (from 1999) is obsolete, and should not be used. It has been replaced by RFC 7231 (and others) in 2014. This latter RFC does not seem to have that constraint from 2616.
Problem is, even if a I have
 RestAssured.config().getRedirectConfig().followRedirects(true);
(which should be the default, isn't it?) then there is no redirect for DELETE (which is my problem). Unless I misunderstand something here, this seems quite confusing. I mean, RestAssured seems configured to do redirects, but then it does not do it. As far as 7231 (the current standard) is concerned, this looks like a bug, isn't it? Or am I missing something here?
Btw, I tried to do the following, but still it does not work (ie no automated redirect):
        final DefaultHttpClient client = new DefaultHttpClient();
        client.setRedirectStrategy(new DefaultRedirectStrategy() {
            @Override
            protected boolean isRedirectable(String method) {
                return true;
            }
        });
        RestAssured.config().httpClient(
                RestAssured.config().getHttpClientConfig().httpClientFactory( () -> client )
        );

Few points to consider: for HTTP 1/1 the RFC 2616 (from 1999) is obsolete, and should not be used. It has been replaced by RFC 7231 (and others) in 2014. This latter RFC does not seem to have that constraint from 2616.

Thanks for pointing this out, I didn't know this. But then doesn't Apache Http Client follow these kinds of redirects automatically!?

Btw, I tried to do the following, but still it does not work (ie no automated redirect)

I've haven't tried it myself, just found it somewhere so I just wrote it down for myself.

Hi,
the specs of 3xx redirects are quite a "mess", and highly confusing. I would strongly recommend to read the following blog.
Regarding Apache Http Client, I am not sure (I didn't check).

Thanks for the link. If what you're saying is true (which I have no reason to doubt :)) I think this can be treated as a bug. If you manage to find out if and how you can configure HttpClient to always follow redirects it would be great if you could share it.

Thanks for the link. If what you're saying is true (which I have no reason to doubt :)) I think this can be treated as a bug. If you manage to find out if and how you can configure HttpClient to always follow redirects it would be great if you could share it.

Hi,
I tried again with 3.1.0, and now the redirection takes place. But it is totally wrong :(
If I return a 301, HttpClient has this great idea to change a DELETE into a GET (wtf!?!).
I tried a 308 (which shouldn't allow changing verb), but that code is ignored (i.e., no redirection takes place)

REST Assured just follows the default behavior of Apache Http Client when
it comes to redirects. Redirects are actually very complicated and defined
in various RFC's. See for example this
https://stackoverflow.com/a/42138726/398441 SO answer. You're always free
to extract the location header yourself and do whatever you want with it in
a subsequent request (and even write a custom filter that does this
automatically if apache http client doesn't do it for you).
On Mon, Jul 16, 2018 at 1:57 PM Andrea Arcuri<EMAIL_ADDRESS>wrote:

Hi,
I tried again with 3.1.0, and now the redirection takes place. But it is
totally wrong :(
If I return a 301, HttpClient has this great idea to change a DELETE into
a GET (wtf!?!).
I tried a 308 (which shouldn't allow changing verb), but that code is
ignored (i.e., no redirection takes place)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/rest-assured/rest-assured/issues/750#issuecomment-405223823,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AALalTJcxX-gHRLX-YjdspNFJuBghyRAks5uHH-jgaJpZM4KOvWC
.


Sort order

It would be useful to indicate in the documentation that sorting is done A-Z, then a-z.

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

ID: e1753278-9dd5-96b0-1f41-8c24f96b8863
Version Independent ID: 00ff26c1-7e1e-4c97-6c27-4f93e9c5c9f0
Content: Sort columns - Power Query
Content Source: powerquery-docs/sort-columns.md
Service: powerquery
GitHub Login: @ptyx507
Microsoft Alias: miescobar

Hi @HairyDrumroll
Thanks for the feedback! The Sort columns article has been completely updated, including adding A-Z, a-z ascending sort order and z-a, Z-A descending sort order information.


TP API token protected

Marks TP's API token as a protected field to be encrypted in the database.


progress on wsl --set-version, or at least time elapsed

Is your feature request related to a problem? Please describe.
wsl --set-version <name> <1|2> this is one of the most time-consuming commands, but there is no progress, nor time elapsed. It is really confusing sometimes if it is working or not.
Describe the solution you'd like

Show progress if possible
Show time-elapsed at least
Some additional info like, system load maybe be helpful

Describe alternatives you've considered
nAn
Additional context
And finally, show the total time elapsed.

We've added animated loading dots for this in the latest WSL versions! If you use WSL from the Microsoft Store you can see this in action. Thanks for filing!


[CDN] az cdn name-exists: Fix type argument missing issue

Description

Fix type argument missing issue
Testing Guide

az cdn name-exists --name contoso
History Notes

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

[x] I adhere to the Error Handling Guidelines.

Fix https://github.com/Azure/azure-cli/issues/22066


About page added

I have added about page. Please review my page. If everything is all right then merge my repo. Thank you

final about page


Make facets work like in LibreCat to make filtering manageable

Make facets work like in LibreCat to make filtering manageable

user/librarian facets now have the same order and visibility as in librecat now


Suggestion for API change to remove more boilerplate

I like that you don't have a subclass like in Parceler and other similar libraries.
How about changing the code you have to write to something like code below?
@ParcelablePlease
public class Model implements Parcelable {

  int id;
  String name;
  OtherModel otherModel;

  @Override public int describeContents() {
    return 0;
  }

  @Override public void writeToParcel(Parcel dest, int flags) {
    ModelParcelablePlease.writeToParcel(this, dest, flags);
  }

  public static final Creator<Model> CREATOR = ModelParcelablePlease.creator();
}

good idea


go vet issues

Hi there. I am using go vet in my repo. I get an error in the case of getting signature data at the final round of signing.
assignment copies lock value to data: github.com/binance-chain/tss-lib/common.SignatureData contains google.golang.org/protobuf/internal/impl.MessageState contains sync.Mutex
Also if you use your codes, you will get the same error in https://github.com/bnb-chain/tss-lib/blob/30c5ee412a0ef9096ae37f67d6e0d5adcfdd551e/ecdsa/resharing/local_party_test.go#L209
ecdsa/resharing/local_party_test.go:209:20: copylocks: assignment copies lock value to signData: github.com/binance-chain/tss-lib/common.SignatureData contains google.golang.org/protobuf/internal/impl.MessageState contains sync.Mutex (govet)

This PR could solve it. #167


most times I cannot download any image and it is really time consuming, more than one minute for one keyword.

@ruclijy Image urls crawling on Google & Bing engines are browser based, it is more time-consumed than Baidu engine, which is api based.


Add Bold clock app, Pedometer and Digital clock widgets

Add Bold clock app
Add Pedometer widget
Add Digital clock widget

Looks awesome - thanks! I'll add some code to ensure that it's easy to continue step counting when changing apps soon as well.


Conflicting advice on using Graph API over MS Graph

Hi,
On this article, below is mentioned
"
Consider using the Graph API for user migration ..."
However, when I clicked the link provided below
https://docs.microsoft.com/en-us/azure/active-directory-b2c/active-directory-b2c-devquickstarts-graph-dotnet
"You must use the Azure AD Graph API to manage users in an Azure AD B2C directory"
"we recommend the use of Microsoft Graph over Azure AD Graph, as Microsoft Graph is where we are investing for Microsoft cloud services."
https://blogs.msdn.microsoft.com/aadgraphteam/2016/07/08/microsoft-graph-or-azure-ad-graph/
My question is that which one to use?

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

ID: fc5e148d-b7cf-92e7-4cc7-a65d20d037b6
Version Independent ID: 2ba6e31b-5899-f35a-0f08-d280110ded13
Content: Frequently asked questions for Azure Active Directory B2C
Content Source: articles/active-directory-b2c/active-directory-b2c-faqs.md
Service: active-directory
GitHub Login: @davidmu1
Microsoft Alias: davidmu

@PingPongSet
Thanks for the feedback! We are currently investigating and will update you shortly.

@PingPongSet It is recommended from Microsoft to use the latest Microsoft Graph as which will support all functionality that currently Azure AD Graph offers to access directory data.

@SaurabhSharma-MSFT
If MS recommends MS graph, shouldn't this document and other doc under B2C be updated based on what you said?
Where is the equivalent document for MS Graph for B2C? At the moment I can only see Azure AD Grap doc under B2C, especially for user migration etc.

@PingPongSet It is recommendation to use Microsoft Graph however, I do acknowledge your feedback on the same as on improving the experience of these documents with the latest technology updates. Also, our document content author team do monitor each of these feedback and use them as they author new contents.
@davidmu1 FYI.

@SaurabhSharma-MSFT
I understand you recommand MS Graph for B2C. But I cannot find the document. Can you please advise?
Thanks in advance.

@PingPongSet Apologies as I was not clear on B2C path for Microsoft Graph.  Currently, If you want to manage users in B2C then you have to use Azure AD Graph API as Microsoft Graph still doesn't support features that are used by Azure AD B2C.

Agreed.
So this line: "No, Azure AD Connect is not designed to work with Azure AD B2C. Consider using the Graph API for user migration. See the User migration guide for details."
Should read:
"No, Azure AD Connect is not designed to work with Azure AD B2C. Consider using the Azure AD Graph API for user migration. See the User migration guide for details."

@PingPongSet I believe Graph API meant Azure AD Graph API in the document however, I feel it is better to mention Azure AD Graph API and I have started a pull request to update the document, the PR for the fix has been created and it will be merged soon. Thanks again for pointing this out, I am closing the issue based on this update, please let us know if there is something else that we can help you with.


[BUG] upgrade will stuck without any error meessage if the checksum is invalid

Describe the bug
The upgrade will stuck if we input the checksum like v
To Reproduce
Steps to reproduce the behavior:

Install Harvester with any nodes
Create upgrade version
Perform upgrade

Expected behavior
Upgrade should failed with error message
Environment:

Underlying Infrastructure (e.g. Baremetal with Dell PowerEdge R630): Baremetal DL360G9 3 nodes
Harvester ISO version: v1.2.1
ui-source Option: Auto

Additional context
It would similar  #3123 but the check sum is invalid, in #3123, we will detect that the inputted is mismatch to the ISO's, but this issue is, the inputted checksum is not a valid checksum, maybe length is not enough, or something else.

For detailed reproduced steps:

Create the Version CR with a malformed checksum string

$ cat <<EOF | kubectl apply -f -
apiVersion: harvesterhci.io/v1beta1
kind: Version
metadata:
  namespace: harvester-system
  name: master
spec:
  isoURL: http://<IP_ADDRESS>:8000/harvester/master/harvester-master-amd64.iso
  isoChecksum: 'asdf'
EOF

Check the created Version CR

$ kubectl -n harvester-system get versions master -o yaml
apiVersion: harvesterhci.io/v1beta1
kind: Version
metadata:
  annotations:
    kubectl.kubernetes.io/last-applied-configuration: |
      {"apiVersion":"harvesterhci.io/v1beta1","kind":"Version","metadata":{"annotations":{},"name":"master","namespace":"harvester-system"},"spec":{"isoChecksum":"asdf","isoURL":"http://<IP_ADDRESS>:8000/harvester/master/harvester-master-amd64.iso"}}
  creationTimestamp: "2024-03-28T08:18:45Z"
  generation: 1
  name: master
  namespace: harvester-system
  resourceVersion: "44990"
  uid: c70b4396-9c6c-47b2-b452-56c6eabdba14
spec:
  isoChecksum: asdf
  isoURL: http://<IP_ADDRESS>:8000/harvester/master/harvester-master-amd64.iso

Click the "Upgrade" button on the dashboard to start the upgrade
Check the created VMImage CR

$ kubectl -n harvester-system get vmimages hvst-upgrade-r2c77 -o yaml
apiVersion: harvesterhci.io/v1beta1
kind: VirtualMachineImage
metadata:
  creationTimestamp: "2024-03-28T08:19:25Z"
  finalizers:
  - wrangler.cattle.io/vm-image-controller
  generation: 1
  labels:
    harvesterhci.io/upgrade: hvst-upgrade-r2c77
  name: hvst-upgrade-r2c77
  namespace: harvester-system
  ownerReferences:
  - apiVersion: harvesterhci.io/v1beta1
    kind: Upgrade
    name: hvst-upgrade-r2c77
    uid: 122e89b1-67f7-45ba-82ca-db9680cb971a
  resourceVersion: "45581"
  uid: 52014bce-2e14-479a-b13d-3f401a7170af
spec:
  checksum: asdf
  displayName: hvst-upgrade-r2c77-master
  pvcName: ""
  pvcNamespace: ""
  retry: 3
  sourceType: download
  storageClassParameters:
    migratable: "true"
    numberOfReplicas: "3"
    staleReplicaTimeout: "30"
  url: http://<IP_ADDRESS>:8000/harvester/master/harvester-master-amd64.iso
status:
  failed: 0

Check the Harvester Pod's log

$ kubectl -n harvester-system logs harvester-7bf5959cb6-jpwj7
time="2024-03-28T08:19:25Z" level=info msg="Initialize upgrade harvester-system/hvst-upgrade-r2c77"
time="2024-03-28T08:19:25Z" level=info msg="Upgrade observability is administratively disabled"
W0328 08:19:25.342616       7 warnings.go:70] unknown field "type"
time="2024-03-28T08:19:25Z" level=info msg="Creating upgrade repo image"
time="2024-03-28T08:19:25Z" level=info msg="handle upgrade harvester-system/hvst-upgrade-r2c77 with labels map[harvesterhci.io/latestUpgrade:true harvesterhci.io/upgradeState:LoggingInfraPrepared]"
time="2024-03-28T08:19:25Z" level=error msg="error syncing 'harvester-system/hvst-upgrade-r2c77': handler vm-image-controller: admission webhook \"validator.longhorn.io\" denied the request: invalid checksum asdf, requeuing"
time="2024-03-28T08:19:25Z" level=error msg="error syncing 'harvester-system/hvst-upgrade-r2c77': handler vm-image-controller: admission webhook \"validator.longhorn.io\" denied the request: invalid checksum asdf, requeuing"
...

This is because Harvester tries to create the BackingImage CR derived from the VMImage CR but gets denied by the Longhorn validating admission webhook:
https://github.com/longhorn/longhorn-manager/blob/103bcaaae5e080dc93af66bf0f8355c31d2f1b0e/util/util.go#L288-L291
func ValidateChecksumSHA512(checksum string) bool {
    validChecksum := regexp.MustCompile(`^[a-f0-9]{128}$`)
    return validChecksum.MatchString(checksum)
}

If you provide it with a valid but mismatched SHA512 hex string, the backing-image-manager will show you the following log lines:
$ kubectl -n longhorn-system logs backing-image-ds-harvester-system-hvst-upgrade-7ht7c -f
time="2024-03-28T08:25:09Z" level=debug msg="Sync Service: initialized" component=sync-server listen="<IP_ADDRESS>:8001"
time="2024-03-28T08:25:09Z" level=info msg="Started sync server at <IP_ADDRESS>:8001"
time="2024-03-28T08:25:10Z" level=debug msg="DataSource Service: initializing" component=data-source-service diskUUID=6c3d7ea2-00ca-49e4-bc12-90cac83d5a35 expectedChecksum=2ee2abb2952fb7d29eaa9b3ce776ff094e56d087e860ac737d0b7cd572f454b2a8e7aadc1634faf0a9b6d4155d036765da7772e2550aab9ad1df7f1f9083dd93 filePath=/data/tmp/harvester-system-hvst-upgrade-7ht7c-4ee3904c name=harvester-system-hvst-upgrade-7ht7c parameters="map[url:http://<IP_ADDRESS>:8000/harvester/master/harvester-master-amd64.iso]" sourceType=download uuid=4ee3904c
time="2024-03-28T08:25:10Z" level=debug msg="Sync Service: initializing sync file /data/tmp/harvester-system-hvst-upgrade-7ht7c-4ee3904c" component=sync-server listen="<IP_ADDRESS>:8001"
time="2024-03-28T08:25:10Z" level=info msg="SyncingFile: cannot to reuse the existing file, will clean it up then start processing: stat /data/tmp/harvester-system-hvst-upgrade-7ht7c-4ee3904c: no such file or directory" component=sync-file filePath=/data/tmp/harvester-system-hvst-upgrade-7ht7c-4ee3904c uuid=4ee3904c
time="2024-03-28T08:25:10Z" level=debug msg="DataSource Service: initialized" component=data-source-service diskUUID=6c3d7ea2-00ca-49e4-bc12-90cac83d5a35 expectedChecksum=2ee2abb2952fb7d29eaa9b3ce776ff094e56d087e860ac737d0b7cd572f454b2a8e7aadc1634faf0a9b6d4155d036765da7772e2550aab9ad1df7f1f9083dd93 filePath=/data/tmp/harvester-system-hvst-upgrade-7ht7c-4ee3904c name=harvester-system-hvst-upgrade-7ht7c parameters="map[url:http://<IP_ADDRESS>:8000/harvester/master/harvester-master-amd64.iso]" sourceType=download uuid=4ee3904c
time="2024-03-28T08:25:10Z" level=info msg="Started data source server at <IP_ADDRESS>:8000"
time="2024-03-28T08:25:11Z" level=info msg="SyncingFile: start to download sync file from URL http://<IP_ADDRESS>:8000/harvester/master/harvester-master-amd64.iso" component=sync-file filePath=/data/tmp/harvester-system-hvst-upgrade-7ht7c-4ee3904c uuid=4ee3904c
time="2024-03-28T08:25:11Z" level=debug msg="DataSource Service: datasource file is state starting, the processing is not begin yet" component=data-source-service diskUUID=6c3d7ea2-00ca-49e4-bc12-90cac83d5a35 expectedChecksum=2ee2abb2952fb7d29eaa9b3ce776ff094e56d087e860ac737d0b7cd572f454b2a8e7aadc1634faf0a9b6d4155d036765da7772e2550aab9ad1df7f1f9083dd93 filePath=/data/tmp/harvester-system-hvst-upgrade-7ht7c-4ee3904c name=harvester-system-hvst-upgrade-7ht7c parameters="map[url:http://<IP_ADDRESS>:8000/harvester/master/harvester-master-amd64.iso]" sourceType=download uuid=4ee3904c
time="2024-03-28T08:26:20Z" level=debug msg="SyncingFile: failed to get the checksum from a valid config during processing wrap-up, will directly calculated it then"
time="2024-03-28T08:26:30Z" level=error msg="SyncingFile: failed to process sync file: the expected checksum 2ee2abb2952fb7d29eaa9b3ce776ff094e56d087e860ac737d0b7cd572f454b2a8e7aadc1634faf0a9b6d4155d036765da7772e2550aab9ad1df7f1f9083dd93 doesn't match the file actual checksum f2d29671b33f1bdfafca198e2515e75c13ba43130b0c68a06025c79af312f7f3d633322fecfc2a2cec22c6ccd653e250a267dccf8d144dcfbe8db1af3162613c" component=sync-file filePath=/data/tmp/harvester-system-hvst-upgrade-7ht7c-4ee3904c uuid=4ee3904c

IMO we have at least two ways to address this, either

Adding a validating admission webhook for the Version CRD in Harvester's webhook, or
Handle such a BackingImage CR creation failure to also fail the entire Harvester upgrade

@lanfon72 should we close this issue? since its backport issues are all closed.

Close as the issue has been verified in backport issues.


Frame using script setup

Hi, thanks for this library!
I'm trying to render some content in a Frame.
I see some components use <script setup>, but not this one:
https://ownego.github.io/polaris-vue/?path=/docs/components-frame--frame
Would it be possible to update it to use  <script setup> too?
Thanks!

I don't have an full app example but I'm using the library for all of my apps, and I see it works well.
I will update or fix bug if there is something wrong.


Version Path Item Object does not use Host Object

This spec introduces Host Object which is used in OpenAPI object. However, this can be overridden now in Path Item Object but there the fields are flattened instead of reusing the Host Object

See #700.