Patch release <IP_ADDRESS>?

As #7529, which is quite an issue, got fixed and also a lot of docs have been written since the release of 2.0.3, could you release a patch version <IP_ADDRESS>?

Docs as updated automatically independent of releases.
It's @qiangxue decision to release or not to release <IP_ADDRESS>.

This time we will still follow our original release plan which is to release 2.0.4 about a month from today. Starting from 2.0.4, for each 2.0.x release we will create a maintenance branch 2.0.x so that it can be used as release vehicles to make patch releases like this.


update to eap 2020.1, update of gradle plugin required as well

This updates the IntelliJ platform to 2021.1 EAP. The problem can be replicated using this setup.
closes #53

I don't know how to fix, this but I managed to reproduce the issue. Some insight in the JavaScript parts of diagrams.net would be required. // cc: @rdmueller + @hediet
The error message as displayed in the browser:

The error message as displayed in the browser's console (right-click on preview to open the Open the DevTools)

It seems that due to the new schema "drawio-plugin" the browser doesn't allow access to cookies any more. I searched the Internet, but didn't find a solution for that.
I tried to change "drawio-plugin://main..." to "https://main..." and changed also in registerSchemeHandlerFactory the "drawio-plugin" to "https" and "*" to "main". The the JavaScript initializes, and show the following dialog, and it seems that it doesn't load the diagram:

Any help would be appreciated. The branch already contains the changes regarding IntelliJ EAP, but doesn't include the changes about the schema (as I wasn't sure if this is a path to explore). Please let me know if I should push these changes as well.

I tried to change "drawio-plugin://main..." to "https://main..." and changed also in registerSchemeHandlerFactory the "drawio-plugin" to "https" and "*" to "main". The the JavaScript initializes, and show the following dialog, and it seems that it doesn't load the diagram

I don't think overriding the default http/s handler is a good idea.
This line should avoid this issue:
Object.defineProperty(document, "cookie", { value: "" });

I don't know why it does not work anymore.

I debugged and found that at least locally, maybe even in the latest version of the plugin the index.html custom file was not used, instead the one from the submodule was used. Maybe it depended on file timestamps? Anyway, explicitly excluding the submodule one solved it for me.
Once the right index.html file was used, the cookie issue disappeared.
I then received an error that due to CORS policy Cross origin requests would not be allowed for the new protocol scheme. Once I changed the protocol, it started to work again without issues, see 2dd2f91bec0ff89a804e070cf5fc5f7dd382b65d
This is now ready for review. If no-one has a different solution to this, I'd like to publish this to support the latest 2021.1 EAP version (final 2021.1 should be out soon I suppose).

Thanks for the quick review, PR updated, merging to main.


Add 'update' function

See update. So far this is the only functionality available in the API that is unsupported.

Fixed in https://github.com/bbarker/Hypothesis-API/releases/tag/0.09


[ConstraintSystem] Favor TypeChecker::compareDeclarations over isDecl…

…AsSpecializedAs.
Use the higher level function compareDeclarations rather than two
direct calls to isDeclAsSpecializedAs as it better captures the intent
here.
NFC.

@swift-ci Please smoke test

In retrospect this isn't actually NFC since we now enter the code related to both declarations being as specialized as the other even when neither is as specialized as the other.


Updates to Polish translation

This commit changes the Gemfile.lock file as well. Please re-submit a new pull request that doesn't contain these Gemfile.lock changes.

@radar should we just remove Gemfile.lock from this project?

Sure thing, based on Yehuda's advice: http://yehudakatz.com/2010/12/16/clarifying-the-roles-of-the-gemspec-and-gemfile/

@radar make it so


feat: add lazy animations for better bundle size

Thanks Enea!

@all-contributors please add @eneajaho for code bug


[UI] Show additional deployment information for apps deployed via Github

Covers step 2 & 3 from https://github.com/epinio/ui/issues/63#issuecomment-1164180440
Persist the information used when creating an app from github.
Use the information to show additional detail in the app detail page
User should be able to view commits from the branch and select one to deploy from

This will probably mean using the app progress component in a new page

@scures Not sure where we got to on this one, is https://github.com/rancher/dashboard/pull/6852 ready to review (after conflicts updated)?

https://github.com/epinio/ui/issues/171

Checked:
To test I used this repo with 5 commits: https://github.com/epinio/example-go/commits/main and deployed the latest
Once the app is deployed I checked the following:

Additional details is shown in detail page so  information is persisted and works as displayed here:

Deployed commit from which app is running is displayed. When I click on Revision I am taken to the code in Github of the specific commit where I deployed from.

@richard-cox , I am not sure how to check this:

User should be able to view commits from the branch and select one to deploy from
This will probably mean using the app progress component in a new page

I am seeing only 1 commit, but there are several in this example (https://github.com/epinio/example-go/commits/main) . Am I perhaps missunderstanding what should happen here?

ok, thank you @scures for the clarification.
I can see indeed the commits on the new tab on the left just the selection is not ready at this point. Ok noted that for further release. Closing ticket with that in mind.


Fix sentences which end with double full stop

Fixes sentences in XML comments which end with double full stops.

@pascalberger thanks for fixing these up!


CMakeLists: fix optflags for PowerPC case

@JohanMabille Should I rebase to main branch?

Ah yes, sorry for this I missed your PR. I am currently updating this project to the latest release of xeus and other packages, and I took the opportunity to clean it a bit. If you could open the PR again, I would be happy to merge it!

Ah yes, sorry for this, I missed your PR. I am currently updating this project to the latest release of xeus and other packages, and I took the opportunity to clean it a bit. If you could open the PR again, I would be happy to merge it!

Done: https://github.com/jupyter-xeus/xvega/pull/48


Main

this merge fixed the logo and cleaned it, rewrote the front page, listed the appropriate redirection links in the website, changed the BG page.

here goes nothing


Vendor dependencies

It'd be nice to have a way to get a reproducible builds, so vendored dependencies are needed.

I've vendored with dep. Can you check?

Thanks for verification!


[html] html.autoCreateQuotes is not instant

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.67.2
OS Version: Windows_NT x64 10.0.22000

Steps to Reproduce:

Open a HTML document
Inside of a tag where the attributes are located, type id=test quickly

I would expect id="test" to be typed but if it is typed too quickly then only id=test is typed.
It's quite frustrating to pause after every equals sign as most auto insertions such as quotes and parentheses are always inserted at any typing speed.

Agree with @jf908 that this behaviour is extremely frustrating for fast typists.
Worth noting that fixing #140698 might solve both issues (if quotes have already been added by typing, don't add them again).

There's currently a 100ms wait to see if more character are typed and the quick completion request should be cancelled.

If this is intended, then why does it still insert an ending quote which moves the cursor past the quote? I keep getting class=""my-cursor-is-here.


App Runner quickstart

Summary
A concise description of the changes being introduced. If you are submitting a PR for a
new quickstart please review the pre merge checklist section to validate this PR is ready for review.

Pre merge checklist

[ ] Did you check you NRQL syntax? - Does it work?
[ ] Did you check your dashboard image quality? -  Do they look good?
[ ] Did you check that your alerts actually work?
[ ] Did you include an InstallPlan and Documentation reference?
[ ] Did you check your descriptive content for voice, tone, spelling and grammar errors?
[ ] Did you attach images of your dashboards to the PR so we can see them working?

Screenshots
Attach images of any visual changes, such as a dashboard here.

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

The reference for install plan needs to map to an install plan id, and you will have to provide one (check install/aws/aws-infrastructure-monitoring for an example).
not sure I quite understood how to "provide one". I've created a file install/aws/aws-apprunner/install.yml with id: aws-apprunner (matching one in config.yaml)  but still have the error. Is install/aws/aws-apprunner/install.yml  required for AppRunner case? AppRunner is not an installable package as such. install.yml  just has a reference to the docs.
The final version of the docs should be available from @bradleycamacho

@4demos I believe the id is correct, but there is an issue with the white spacing. Try replacing the config.yml with this code and see if it resolves the issue:
`slug: aws-apprunner
description: |
Monitor your Apps running in App Runner coutainers on New Rerilc Instant Observability Platform!
With New Relic's AWS App Runner integration, you now can see all your App Runner data on New Rerilc Instant Observability Platform.
Get started!
Check out the documentation to learn more about New
Relic monitoring and integration with AWS App Runner.
title: Amazon App Runner
summary: |
Use New Relic - AWS App Runner Integration to Monitor Your Conteinerized Apps and Services with New Relic
icon: logo.png
level: New Relic
authors:

New Relic
documentation:
name: Amazon App Runner integration docs
description: |
Faster and comprehensive telemetry from Amazon App Runner Service for new and existing New Relic Customers
url: >-
https://docs.newrelic.com/docs/integrations/amazon-integrations/aws-integrations-list/aws-apprunner
keywords:
aws
app runner
logs
installPlans:
aws-apprunner
dashboards:
aws-apprunner`

@aswanson-nr  , this one looks good to me. The only issue is that the url does not look like it's live yet.

Hey @4demos and @jcountsNR, I had a few comments on the NRQL used in the dashboard. If those aren't a concern, then we should be good to merge this in. I'm going to wait for a confirmation from you both before I move anything along.


🛐 Worship the OCaml module system

This pull request contains two changes:

Replace the handler records with modules. I found (experimentally) while hacking algaett that it is a lot less painful to program the handlers with modules than it is to write out the records. Features like include and open are useful, and I would not discount reducing the stress of being several parenthesis/brace delimiter levels into a gnarly piece of code.

I have refactored the modules so that the main signatures are declared (and documented) just once. This removes a lot of duplication that is annoying when refactoring code. It also means that if documentation is generated for the private/internal modules, the documentation comments will appear for these as well. Sometimes we wish to expose a different public interface than the private interface (but I didn't notice that in this library); that too can be incorporated, through judicious use of include.

Let me know if you have any feedback about these changes, or if you don't like it for some reason. Thanks!

Just wanted to add that, under the records=modules Thought, these two approaches should be isomorphic. I hope we can fix this problem at least in the languages/tools we created. :-)


Fix the sortBy old button

Requires a javascript function to sort the president's profile and view them from old to new presidents

The sorting has to be performed on the basis of "From" year field right ?

The sorting has to be performed on the basis of "From" year field right ?

💯 you are on point 👍

Opened a PR #9


Add otree

otree is a command line tool to view objects (json/yaml/toml) in TUI tree widget.

Mainly based on https://github.com/EdJoPaTo/tui-rs-tree-widget.
I developed this to better view yaml files, since they usually have very deep levels.

Sexy! 😍


SSRF vaulnerability in hydrating schema refs

currently we have a minor vulnerability where we do a GET on whatever a bundle author puts in a schema ref.
probably not the worst attack surface, but could be an opportunity to check "hey does this look like a legit schema ref url" could perhaps have a value of allowed domain names or something.
I found this while trying to enable gosec linter https://securego.io/docs/rules/g107.html
https://github.com/massdriver-cloud/massdriver-cli/blob/911a7ac116b4c9b24966385087e762d6141fbc3d/pkg/jsonschema/hydrate.go#L68

End users can manage their own libraries of “types” to ref into their schemas. What’s the best approach  if we are agnostic about where the json comes from?

hmm, I think we could have an org level setting captured in the API that could allow that domain (in addition to the list of domains we allow for everyone).


refactor: yaml stream format

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

[ ] N
[ ] Y

2. What is the scope of this PR (e.g. component or file name):

3. Provide a description of the PR(e.g. more details, effects, motivations or doc link):

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Other

4. Are there any breaking changes?(Y/N) and describe the breaking changes(e.g. more details, motivations or doc link):

[ ] N
[ ] Y

5. Are there test cases for these changes?(Y/N) select and add more details, references or doc links:

[ ] Unit test
[ ] Integration test
[ ] Benchmark (add benchmark stats below)
[ ] Manual test (add detailed scripts or steps below)
[ ] Other

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 10 (100.0%)  changed or added relevant lines in 1 file are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.4%) to 63.309%

Files with Coverage Reduction
New Missed Lines
%

pkg/kcl/api.go
3
50.64%

Totals

Change from base Build<PHONE_NUMBER>:
0.4%

Covered Lines:
3149

Relevant Lines:
4974

💛  - Coveralls


[BUG] msvc link error about imath

Environment
**Operating System:win11 22h2
**Version / Commit SHA:10.0.1
**Other:latest vs2022 with its msvc (e.g compiler, C++ standard etc.)
Describe the bug
(A clear and concise description of what the bug is.)
build the openvdb x64-windows by vcpkg, run the cookbook's example code, showed as follows:
EXEC : error : main.cpp.obj : error LNK2001: unresolved external symbol imath_half_to_float_table
build\windows\x64\debug\OpenVDBTest.exe : fatal error LNK1120: 1 unresolved externals
To Reproduce
Steps to reproduce the behavior:

Build with vcpkg install openvdb --triplet=x64-windows
Run code from https://www.openvdb.org/documentation/doxygen/codeExamples.html in vs2022 debug mode
See error

Expected behavior
(A clear and concise description of what you expected to happen.)
no link error
Additional context
(Add any other context about the problem here.)
found a solution in forum to skip this error: https://groups.google.com/g/openvdb-forum/c/51664ugGfIs/m/pfrO462vAAAJ
add "#define IMATH_HALF_NO_LOOKUP_TABLE" before "#include <openvdb/openvdb.h>", the cookbook's code can link and run successfully.

Hey @freestriker, thanks for this, this should now be fixed on master - we always explicitly disable the inbuilt half lookup table now


Muter cleans up after itself in response to UNIX signals

Right now, if you interrupt Muter while it's running, it's most likely that a user's codebase will still contain a mutant. This issue is to improve Muter's cleanup process by registering handlers for UNIX signals.

Relevant: https://stackoverflow.com/questions/45713819/trapping-signals-in-a-swift-command-line-application

That’s a great point. It’s less complex to run the testing in an
environment where you’re not concerned with unintentionally leaving bugs
behind. After thinking about the work involved with adding signal support
for this issue, I’m thinking there’s a new (and simpler) issue around
changing how Muter ensures the integrity of the codebase it’s working with.
On Tue, Dec 25, 2018 at 22:24 Zev Eisenberg<EMAIL_ADDRESS>wrote:

This could be made much easier and safer by having Muter work on a copy of
the user’s code base in /tmp. And with APFS, copies should be nearly free.
And even on HFS+, I don’t think you’re going to run into serious
performance issues by copying the repo before mutating. You could have a
flag like --no-copy to disable this for CI builds.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/SeanROlszewski/muter/issues/3#issuecomment-449895960,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACFi8r8fobVIrvspQQ-dsCRDdy0aLIDfks5u8uvdgaJpZM4ZgjYP
.


Cumulative Flow Diagrams

How awesome would cumulative flow diagrams be for individual projects? Possibly with filters (e.g. by colors).
This is rather a feature proposal than a request.

Currently this groups data to 24 hour perioids. It would be cool to see even more accurate flow diagram. I should see difference between tasks logged now and tasks logged 2 hours ago.


does this work with AWS lambda yet?

Hi,
Can't wait to get stuck into this, does this work with aws lambda yet?
Bests,
Sean.

Hey @JuniorDEVed! We've got some stuff coming out in the next couple of weeks that will make working with AWS Lambda a lot better. Stay tuned!

Hi @JuniorDEVed,
We just shipped this: https://planetscale.com/blog/connect-any-mysql-client-to-planetscale-using-connection-strings
For Lambda we recommend creating a password and then using a direct connection string (rather than this library).


Taking too much memory, and  app crashes

Thanks for the beautiful library.
I am making a news app that contains news feed and every news article will have an Image to it. I am using this library to show images. Everything looks good but, the FastImage taking too much memory sometimes it is crossing 200MB and reaching 500MB upon using for few minutes.  I don't know if it is kind of memory leak. Kindly assist me to resolve this isssue.
info Fetching system and libraries information...
System:
    OS: Linux 4.15 Ubuntu 16.04.6 LTS (Xenial Xerus)
    CPU: (4) x64 Intel(R) Core(TM) i7-6500U CPU @ 2.50GHz
    Memory: 286.99 MB / 7.68 GB
    Shell: 4.3.48 - /bin/bash
  Binaries:
    Node: 10.15.3 - /usr/local/bin/node
    Yarn: 1.22.4 - /usr/bin/yarn
    npm: 6.14.4 - /usr/local/bin/npm
    Watchman: 4.9.0 - /usr/local/bin/watchman
  SDKs:
    Android SDK:
      API Levels: 14, 18, 19, 20, 21, 22, 23, 24, 25, 26, 27, 28, 29
      Build Tools: 28.0.3, 29.0.2, 30.0.1
      Android NDK: Not Found
  IDEs:
    Android Studio: Not Found
  Languages:
    Java: 1.8.0_252 - /usr/bin/javac
    Python: 2.7.12 - /usr/bin/python
  npmPackages:
    @react-native-community/cli: Not Found
    react: 16.13.1 => 16.13.1
    react-native: 0.62.2 => 0.62.2
  npmGlobalPackages:
    *react-native*: Not Found

this is how I am using the FastImage in my app.
  getSource = () => {
    if (this.props.item.node.featuredImage) {
      return {
        uri: this.props.item.node.featuredImage.node.sourceUrl,
      }
    } else {
      return require('../../assets/e-logo.png')
    }
  }

<FastImage
       source={this.getSource()}
       style={{
              borderRadius: 5,
              width: Dimensions.get('window').width - 20,
              height: ((Dimensions.get('window').width - 20) * 2) / 3,
     }}
   // resizeMode={FastImage.resizeMode.contain}
 />

@sandeshnaroju I used this library a few months ago, and I believe the memory issue I encountered was because the Images I were displaying were large file sizes. So when I was hitting, 4mb, 5mb, 8mb images it was literally using an equivalent amount of memory and the memory usage soared. I had to go and compress all the extremely large images I was pulling. Perhaps inspect the size of the files you're displaying.

I am also facing this issue. using React Native 0.63.1 and     "react-native-fast-image": "^8.3.2",

We have noticed in our app that for displaying local images, react native's default Image actually performs better :)
Apart from that I don't completely agree with @jordangrant, IMHO,  it's not file size that matters, but image dimensions.
So, for example, if you have 2MB image and 100kB  image and both are of dimensions 1000 x 1000 px, they will take same amount of memory. And memory allocated is easily calculated as follows:
ImageWidth * ImageHeight * BytesUsedToRepresentEachPixel
Regarding  BytesUsedToRepresentEachPixel  - this can vary from image to image, eg. if you are serving JPEG this number is usually 24 -  8 bits for each of  R-G-B, and for example, for transparent PNGs, this number is usually 32 (24 bits for RGB + 8 bits for alpha channel).
Also if you are going to render 1000 x 1000 px image into 100 x 100px container, app will still allocate memory for 1000 x 1000 px image. So, it is super important to serve Images from server which are as close as possible to container you are going to display them in.
They can be resized before caching and displaying them - which would imply smaller memory footprint, but fast-image doesn't support this yet :/  (I have plans to open PR for this feature, but not sure when 😕 nor that it will be merged)
References:

SDWebImage docs: link
This youtube talk: The Life of an Image on iOS

Use this pr https://github.com/DylanVann/react-native-fast-image/pull/425 and then apply with patch-package

Guys,
My codebase dependencies needed to be upgraded and this versions are used:
react-native 0.65.0
react-native-fast-image 8.5.11
After this upgrade, I've face issues with FastImage blocking the screen loading, I guess it happens while cache population phases, but I'm not sure, after cache population, works better.
I've tried to use priority low, but not changes in this behavior.
Another point is, I also need to calculate the width and height to better fit in various screens.
Do you have any suggestions for better performance or is it an issue related to this issue?


improvements at README.md

Updated README from hash like :sym => value to ruby 1.9+ new form: sym: value
Added the highlight in symbol words, when I founded one.

Thanks!

:+1:


[Kernel] Re-tune Mixtral MoE configurations for FP8 on H100

Re-tune FP8 MoE configs based on https://github.com/vllm-project/vllm/pull/4921
Results for ITL of Mixtral-8x7B-v0.1, prompt-len 1000, gen-len 50:
TP2:
qps = 1: 10.5 ms (was 11.6 ms)
qps = 2: 10.2 ms (was 11.6 ms)
qps = 4: 11.8 ms (was 11.9 ms)
qps = 6: 12.6 ms (was 12.7 ms)

TP4:
qps = 1: 10.9 ms (was 11.7 ms)
qps = 2: 11.1 ms (was 12.0 ms)
qps = 4: 12.0 ms (was 12.2 ms)
qps = 6: 12.9 ms (was 13.0 ms)

As you can see, for Mixtral 8x7B and FP8 one should be running TP2 and not TP4 (unlike for FP16).

 PR Checklist (Click to Expand)
Thank you for your contribution to vLLM! Before submitting the pull request, please ensure the PR meets the following criteria. This helps vLLM maintain the code quality and improve the efficiency of the review process.
PR Title and Classification
Only specific types of PRs will be reviewed. The PR title is prefixed appropriately to indicate the type of change. Please use one of the following:

[Bugfix] for bug fixes.
[CI/Build] for build or continuous integration improvements.
[Doc] for documentation fixes and improvements.
[Model] for adding a new model or improving an existing model. Model name should appear in the title.
[Frontend] For changes on the vLLM frontend (e.g., OpenAI API server, LLM class, etc.)
[Kernel] for changes affecting CUDA kernels or other compute kernels.
[Core] for changes in the core vLLM logic (e.g., LLMEngine, AsyncLLMEngine, Scheduler, etc.)
[Hardware][Vendor] for hardware-specific changes. Vendor name should appear in the prefix (e.g., [Hardware][AMD]).
[Misc] for PRs that do not fit the above categories. Please use this sparingly.

Note: If the PR spans more than one category, please include all relevant prefixes.
Code Quality
The PR need to meet the following code quality standards:

We adhere to Google Python style guide and Google C++ style guide.
Pass all linter checks. Please use format.sh to format your code.
The code need to be well-documented to ensure future contributors can easily understand the code.
Include sufficient tests to ensure the project to stay correct and robust. This includes both unit tests and integration tests.
Please add documentation to docs/source/ if the PR modifies the user-facing behaviors of vLLM. It helps vLLM user understand and utilize the new features or changes.

Notes for Large Changes
Please keep the changes as concise as possible. For major architectural changes (>500 LOC excluding kernel/data/config/test), we would expect a GitHub issue (RFC) discussing the technical design and justification. Otherwise, we will tag it with rfc-required and might not go through the PR.
What to Expect for the Reviews
The goal of the vLLM team is to be a transparent reviewing machine. We would like to make the review process transparent and efficient and make sure no contributor feel confused or frustrated. However, the vLLM team is small, so we need to prioritize some PRs over others. Here is what you can expect from the review process:

 After the PR is submitted, the PR will be assigned to a reviewer. Every reviewer will pick up the PRs based on their expertise and availability.
 After the PR is assigned, the reviewer will provide status update every 2-3 days. If the PR is not reviewed within 7 days, please feel free to ping the reviewer or the vLLM team.
 After the review, the reviewer will put an  action-required label on the PR if there are changes required. The contributor should address the comments and ping the reviewer to re-review the PR.
 Please respond to all comments within a reasonable time frame. If a comment isn't clear or you disagree with a suggestion, feel free to ask for clarification or discuss the suggestion.

Thank You
 Finally, thank you for taking the time to read these guidelines and for your interest in contributing to vLLM. Your contributions make vLLM a great tool for everyone!

@WoosukKwon can you have another look / approve?


[BUG] No easy way to change (or remove) pull-secret

When the pull-secret is stored in secure storage, it is difficult to change this to another one. This can happen when the pull-secret is revoked or has to be changed. However when the pull-secret API is used to store a copy&pasted pull-secret, it is not possible to use crc config to reset or unset it. Only crc cleanup allows for the removal, but this command has too many side-effects.

A possible solution would be to use crc config set/unset pull-secret as an abstracted (intercepted) config option to deal with the pull secret API.

Should we instead document how the user can remove it by using the os provided tools (keychain in macos and credential manager on windows? my concern is having both pull-secret and pull-secret-file as config options would be confusing.

Should we ... document how the user can remove it by using the os provided tools...?

Yes, please, including Linux.


Feat:  finalize message component

This is not ready yet, I'll ping ya when it is.

Kk Lets walk through this tomorrow!
This is gonna make you 🤤🤤🤤


Error improvement wishlist for Beta

New things to implement for all public APIs:

https://github.com/Eventual-Inc/Daft/issues/464
https://github.com/Eventual-Inc/Daft/issues/465

Some low hanging fruit:

https://github.com/Eventual-Inc/Daft/issues/469
https://github.com/Eventual-Inc/Daft/issues/470
https://github.com/Eventual-Inc/Daft/issues/507

Closing as most of them are done - to reopen if necessary


Show pins in a special folder

Is your feature request related to a problem? Please describe.
No
Describe the solution you'd like
It would be interesting if we could show the user pinned pins (yeah!) on the Files section inside the Web UI. I imagine it as special folder with different icon and some explanation.
It'd be a read-only folder and would only be shown to users that have one or more recursive (and direct?) pins. The folder would allow to navigate the pinned directories/files.
It wouldn't, of course, show indirect pins.
Describe alternatives you've considered
Use the CLI. I'm thinking about something visual.
Additional context
N/A

Yes, I want to show a page above the MFS root, that lists out yout pins, and some interesting CIDs, so you can see things in the blockstore that aren't in the MFS, and parhaps a way to ipfs.get an arbitrary CID.

It'd be a read-only folder and would only be shown to users that have one or more recursive (and direct?) pins.

It could support context actions such as "Copy to MFS/Files"  and "Unpin" :)

@olizilla so what would the structure of the root be? I did thought about something like this:

MFS/Home (we'd need to give it a good name)
Pins ? or should we list the pins alongside MFS? Or should we list the pins + interesting stuff, and the pins would have a tag telling it's a pin and a way to unpin?
Also add some way to navigate to an arbitrary CID/IPFS/IPNS/path

Some questions for things to decide:

On the root /, what should we should besides the MFS?

Should we show our PINS?
Or should we show something else such as ipfs and ipns as they are the two main namespaces. Then, inside ipfs, we'd have the pins.

This might be out of scope and perhaps should be moved to another issue: I noticed the files page is being re-rendered like crazy for no reason. I've also noticed that we could really simplify our code if we kept most of the action code and state on the redux bundle. That way, we could connect each component to the specific bits of data they need, simplifying the code. And since it would be a "shared" state, it could help us with functionalities that involve the selected files, for instance.

In general it's better to keep new features and refactors as seperate tasks. Let's make that a seperate issue.
For the "what should we show where" the goals are:

make something useful for developers
stay consistent with what IPFS does.

So, when you click on the files tab, ideally we'd show a few of your most recently pinned CIDs and the current CID for the MFS root, that you could click through to.
Alas, we don't have a way to get "recent pins"... it's all or nothing. We could get the full list and show it as a count like "58 blocks pinned / 2988 blocks stored", along with a link to list the recursively pinned root CIDS...
We could also change the "expore a CID" feature to show a preview of the content along with a little metadata, rather than have it open things in the IPLD explorer... In general I think folks will probably want to see the things rather than see the IPLD data behind the things.
We could show /ipfs and /ipns as special folders inside your MFS... but it's weird. With an empty MFS, the root dir would have the CID of the empty dir QmUNLL... but we'd show it as having 2 dirs in it... Not a deal breaker, but it makes it that bit harder to explain CIDs and how IPFS works to people. We could side step the issue by showing /ipfs and /ipns as visually seperate from /everything-else, but it needs some experimenting.

Some ideas taken from a call today with @olizilla:

On the main files page show the MFS root contents, a button (separated from the files list) to the pins. That "button" would have the number of pinned blocks.
Also, show the number of blocks and perhaps some other interesting info.
Do something like Github bar:

Oli also had an idea to let users browse the blocks (could you give more details on that one?).
Another interesting topic that came in mind is to update the main bar at the top (Explore) to Browse so it opens IPFS paths on Files page instead of explore page by default.
This thoughts need to be improved. In the meanwhile, you can take a look at #1027. It already lets you browse any files you want (/files/mfs..., /files/ipns..., /files/ipns/... and /files/pins are the main URLs).
Ideas for the navigation breadcrumbs (we'd still need a way to get back to MFS when viewing IPNS/IPFS):

Ideas for the right side actions:

Do something like Github bar

I like this idea a lot!

Oli also had an idea to let users browse the blocks (could you give more details on that one?).

ipfs refs local will list all local references (CIDs of blocks physically present at your node's repo)
There is also
ipfs files stat --with-local /ipfs/{cid} shows what part of data behind a CID is local (kB and %)

Another interesting topic that came in mind is to update the main bar at the top (Explore) to Browse so it opens IPFS paths on Files page instead of explore page by default.

:+1: for Files to be the default view.
IPLD Explorer makes sense as an opt-in "inspector/debug view".

Ideas for the navigation breadcrumbs (we'd still need a way to get back to MFS

What if we avoid using a name for MFS and display the home icon as the MFS root crumb?
 / my / path

Ideas for the right side actions

Personally I'd like to keep icon+text for Adding files and Folders, as those are key features and text makes them even more prominent. Other less often used features could be behind icons alone.

Personally I'd like to keep icon+text for Adding files and Folders, as those are key features and text makes them even more prominent. Other less often used features could be behind icons alone.

Agreed. On pins page, perhaps a 'Pin' button and also add the 'Pin' option to the drop down. We will be able to navigate any files we want :O

@hacdias i'm currently distracted with camp design, but maybe this design which i began in collab with @olizilla can help advance the conversation. To be cont


Generate scaffolding with storybook demo and typescript, no stories found by default

I generated the scaffolding for a web component application, and told it to include storybook for demo purposes (including adding the default story).  I also told it to use typescript.  With these options, upon building or running storybook it does not find the default storybook.
It works if I update main.js to look for stories in
'../out-tsc/**/stories/.stories.{js,md,mdx}'
instead of
'../**/stories/.stories.{js,md,mdx}'

This seems to be working for me, the only thing is that you need to run tsc once before starting storybook. Maybe we should change the storybook script to "storybook": "tsc && concurrently --kill-others --names tsc,storybook \"npm run tsc:watch\" \"start-storybook --node-resolve --watch --open\"",

Oh, duh moment for me.  That does work, thanks!


[#469]fix spelling exception.

Make sure that:

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

Codecov Report

Merging #478 (2b2986e) into master (dff886f) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #478   +/-   ##
=========================================
  Coverage     14.05%   14.05%
  Complexity       56       56
=========================================
  Files            52       52
  Lines          1572     1572
  Branches        228      228
=========================================
  Hits            221      221
  Misses         1310     1310
  Partials         41       41

Continue to review full report at Codecov.

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


ipad style CupertinoActionSheet widget

Use case

When building UI for iOS there are differences how ActionSheet UI is presented on iPhone and iPad.
The current implementation of the CupertinoActionSheet is rendered as the iPhone-style action sheet UI as shown in the Apple's UIKit documentation at:
https://developer.apple.com/documentation/uikit/windows_and_screens/getting_the_user_s_attention_with_alerts_and_action_sheets#2938395

Proposal

New widget is needed or an option in the CupertinoActionSheet widget so render action sheet in the iPad-stylelike this:
https://developer.apple.com/documentation/uikit/windows_and_screens/getting_the_user_s_attention_with_alerts_and_action_sheets#2940105

This style widget is actually used in some native implementations for iOS plugins. E.g. the "share" plugin's method share(..) has a sharePositionOrigin parameter needed for the iOS native iPad-style action sheet used by the native iOS Share UI.

Hi @ekuleshov
I see there's an open issue addressing the case you described. https://github.com/flutter/flutter/issues/79236
Please follow up on that issue, I'm closing the current one as a duplicate.
If you disagree, please write in the comments and I will reopen it.
Thank you


[2.10] Properly format windows env-vars, redeploy windows plans if they fail

Issue(s): https://github.com/rancher/rancher/issues/42458, https://github.com/rancher/rancher/issues/45386, https://github.com/rancher/rancher/issues/45385, https://github.com/rancher/rancher/issues/47544
Problem
Windows provisioning currently suffers from instability to due to multiple issues. For instance, environment variables utilized by the system-agent-installer-rke2 image are not properly formatted by Rancher and as such cannot be used. This results in rke2 being stopped and reinstalled unnecessarily, which has the potential to result in a plan failure. Additionally, there are cases where non-Rancher managed Windows services (as as vSphere vmtools) may temporarily prevent rancher-wins from properly applying plans. In these cases, Rancher should automatically reattempt the plan to ensure that it is successfully applied.
Solution
Implement a new helper function for properly formatting windows environment variables, FormatWindowsEnvVar.  FormatWindowsEnvVar will convert a corev1.EnvVar into the proper string representation for use in either a PowerShell script (like the rancher-wins install script) or a planner variable (which is later passed to an exec.Command).
Deliver two new environment variables when creating the system-agent-installer-rke2 install plan, WINS_RESTART_STAMP and INSTALL_RKE2_VERSION. These environment variables will be utilized by the system-agent-installer-rke2 to ensure that the rke2 service is only reinstalled or restarted when needed. Other preexisting environment variables have also been updated to use FormatWindowsEnvVar.
Update the Rancher planner to respect the max-failures and failure-threshold plan fields. Previously, -1 and 1 were always passed, which prevents plans from being reattempted after a failure. Now, Rancher will attempt to reapply the windows system-agent-installer-rke2 plan up to 5 times before declaring a plan failure. All Linux plans continue to use -1 and 1. Additionally, a new field has been added to the OneTimeInstruction struct, ResetFailureCountOnServiceRestart. When this field is set to true, the system-agent will clear the failure-count and failed-checksum values on the plan when force applying the instruction after a restart. This ensures that the plans max-failure value will be respected if the node or system-agent is restarted.
Note that an associated system-agent change and rancher-wins bump is needed to fully address this issue, so this PR shouldn't be merged until that occurs
Testing
Engineering Testing
Manual Testing
I have built a custom version of Rancher with these changes and tested them against modified versions of rancher-wins and the system-agent-installer-rke2 image. I was able to confirm that the required environment variables are now properly formatted and that the plan will be reattempted up to five times. I also tested these changes against vSphere and saw improved stability post vm reboot
Automated Testing

Test types added/modified:

Unit
Summary:

I've added unit testing for the new FormatWindowsEnvVar function to ensure that it works as expected. Existing unit tests covering the relevant environment variables have also been updated to cover these changes.
QA Testing Considerations
While some aspects of this PR will be testable, such as reattempting plans multiple times, other aspects relating to the new environment variables will only be testable using recent versions of the system-agent-installer-rke2 image (i.e. recent versions of rke2).
Regressions Considerations
Existing / newly added automated tests that provide evidence there are no regressions:

Thanks for the reviews, I'm going to hold off on merging this until https://github.com/rancher/wins/pull/248 and https://github.com/rancher/system-agent/pull/201 are merged. I'll bump both versions as an update to this PR

This PR has been updated based off of the changes made in https://github.com/rancher/system-agent/pull/201, which places the new ResetFailureCountOnSystemAgentRestart field on the Plan struct as opposed to the OneTimeInstruction. This simplifies some parts os this PR

I've approved but realistically @Oats87 should also give this a :+1: before merging.

Merging with a single approval as the latest change request has been addressed and no other major changes have been made since Jakes last approval. Both the system-agent and rancher-wins have been bumped to the latest RC versions.


Links to submission and distribution api_v1.json files

Your Question
I'm wondering about the api_v1.json links in the https://github.com/corona-warn-app/cwa-server/blob/main/README.md file.
They were recently changed in https://github.com/corona-warn-app/cwa-server/pull/1061 by @hilmarf
from
https://github.com/corona-warn-app/cwa-server/raw/master/services/submission/api_v1.json and
https://github.com/corona-warn-app/cwa-server/raw/master/services/distribution/api_v1.json
to
https://github.com/corona-warn-app/cwa-server/raw/HEAD/services/submission/api_v1.json
https://github.com/corona-warn-app/cwa-server/raw/HEAD/services/distribution/api_v1.json
where master was replaced by HEAD.  I would have expected master to be replaced by main (as the new default branch of the repository). Why wasn't "main" used in the links?
In fact, I'm also wondering why the links aren't using "/blob/main" instead of "/raw/HEAD" such as:
https://github.com/corona-warn-app/cwa-server/blob/main/services/submission/api_v1.json
https://github.com/corona-warn-app/cwa-server/blob/main/services/distribution/api_v1.json

Hi @MikeMcC399,
thanks for your question. The reason I chose HEAD instead of main is the flexible automated way git keeps updating it to your currently working branch. See https://git-scm.com/book/en/v2/Git-Branching-Branches-in-a-Nutshell
This way it's easier to work on multiple different branches in parallel, but always reference within the actual branch from the documentation, instead of 'jumping' to another branch.
I hope that clarifies the idea.
Regards,
Hilmar

Hi @MikeMcC399,
thanks for your question. The reason I chose HEAD instead of main is the flexible automated way git keeps updating it to your currently working branch. See https://git-scm.com/book/en/v2/Git-Branching-Branches-in-a-Nutshell
This way it's easier to work on multiple different branches in parallel, but always reference within the actual branch from the documentation, instead of 'jumping' to another branch.
I hope that clarifies the idea.
Regards,
Hilmar


[base-ui][Select] The macOSVoiceOver doesn't read current value correctly on Safari

Steps to reproduce
https://mui.com/base-ui/react-select/
Steps:

On MacOS run VoiceOver.
Open https://mui.com/base-ui/react-select/ in Safari.
Tab to the demo control in the Introduction section.

Current behavior
When VoiceOver reads the Select control (without it being opened), it announces its value with the last character missing and with a comma added to the beginning. For example for the demo on the MUI site, it announces the three values like so:

Documentation:   ", Documentatio"
Components:   ", Component"
Features:   ", Feature"

We have used it in an implementation (unfortunately I can't provide access) and it is sometimes misses out the last two characters and sometimes adds "misspelt" before the value.
This only happens in Safari, not when using Chrome.
Expected behavior
VoiceOver should announce the value as it is displayed.
Context
No response
Your environment
MacOS 14.1.1 Sonoma
Safari 17.1
Excuse my ignorance but I don't know how to run npx @mui/envinfo. Where should I do this?
Search keywords: Select VoiceOver

@danilo-leal Can I suggest you use "VoiceOver" in the summary rather than changing it to "voice over". This will help users who are searching for VoiceOver issues. Thanks.

Hey, @markmagennis, thanks for the report. I can confirm the behavior, accepting it as a bug.
cc: @michaldudak to prioritize.


goog.editor.Field Safari Mobile on IOS focus broken

This issue was imported from Closure Library's previous home at http://closure-library.googlecode.com
The link to the original issue is:
https://code.google.com/p/closure-library/issues/detail?id=554

This issue is being closed as a preparation step before archiving the repository.  See the README for more detail.


[WIP] Accept all input via params

Firstly, this is an exploration of ideas in the form of code changes so please do not actually merge this. It is intended for discussion only. If any of these sound good, I'm happy to work them into separate PRs.
Purpose: I'm working on adapting this to be able to create a web UI version. Ideally I would like to be able to reuse the entirely of the script but if that isn't possible then it would be nice to reuse as much of the logic as possible to allow for improvements to be synced.
Proposed Changes:

Adding param for the resource account/auto attendant/call queue to visualize. Apart from the fact that Out-GridView isn't supported on all platforms, this allows me to pull the desired object outside of the script and pass it in.
Adding params for all auto attendants/call queues. Similar to 1 above, namely being able to fetch it external to the script, I think using something like the GetAaForAppInstanceId or GetCqForAppInstanceId functions could speed up your script for deeply nested cases since it will pull them once instead of every time.
Refactor logic into functions. This is a bit of an example for how I think it might be possible to reduce the line count (CreateAfterHoursCheckNode is a good example) and simplify some of the logic (CreateHolidaySubgraph could actually be refactored again into a more generic CreateCallFlowSubgraph function) but also a bit of a plan B for me if 1 & 2 don't fit with your vision.

Questions/Outstanding Items:

Was there a reason that you weren't using functions or didn't want to have a generic CreateCallFlowSubgraph type function?
My API can't access the MsOnline api, which impacts the calls to Get-MsolUser and Get-MsolGroup. I was thinking I could replace the Get-MsolUser calls with Get-CsOnlineUser and then also pass them in as a param with a lookup function similar to Get{Aa,Cq}ForAppInstanceId (maybe GetUserForObjectId) but that doesn't cover the groups which I will need to still thinking about.

Hi @pvaillant . Thanks a lot for the feedback and the suggestions. I am not a pro dev, so please excuse me if there are some rookie mistakes. I also do not know how PRs etc. work on GitHub since I only ever uploaded my own code.

I agree that adding a param for a specific resource account makes sense.
I don't really understand what you mean here. Could you please try to explain it to me in a different way?
No, there was no reason. I've written functions before but I'm only doing PowerShell since about 3 years so I'm still learning. I've updated the script in my repo. It can now create call flows for either AAs or CQs at the top level. I've also put all the stuff for the CQs in a function.

The whole thing was merely an experiment before. I wanted to see if I can pull it off. So, I started scripting and kept adding stuff to the script to capture more and more different config types. It got a little out of hand and overcomplicated.
I will look into creating more functions for the different operations inside the script. My vision is that you can also specify how deeply you want to go on nested call flows via a parameter.
Can your API use AzureAD Module? For a CQ you can have individual agents, a Teams Channel or an M365 Group. The M365 group doesn't necessarily need to be teams enabled or can even be a security group. I believe that you won't be able to find those groups via MicrosoftTeams Module alone.

I think it's a great concept and the hardest part is definitely getting that first version done. I'm happy to help where ever I can, whether it's with GitHub-isms or with the actual coding, so let me know if there's something you'd like me to do.
RE 2, if you take a look at https://github.com/pvaillant/M365CallFlowVisualizer/blob/accept-all-input-via-params/CallFlowVisualizer.ps1#L43 the function abstracts the lookup of the auto attendant so that it can use the -allAutoAttendants parameter (if provided) or call Get-CsAutoAttendant a maximum of once ever rather then possibly multiple times depending on if there's a menu that forwards in the holiday, after hours or default call flow. You can see it in action at https://github.com/pvaillant/M365CallFlowVisualizer/blob/accept-all-input-via-params/CallFlowVisualizer.ps1#L88
For my purpose, you can imagine that I'm calling the script something like this:
$autoAttendants = Get-CsAutoAttendant
$callQueues = Get-CsCallQueue
$appInstances = Get-CsOnlineApplicationInstance
$ra = Get-CsOnlineApplicationInstance $requestedResourceAccount
$mermaid = ./CallFlowVisualizer.ps1 -resourceAccount $ra -allAutoAttendants $autoAttendants -allCallQueues $callQueues -allApplicationInstances $appInstances

Which actually reminds me, it would be great to also have a param to control whether to write the diagram to a file or return it instead. It would also be nice to be able to pass a single Auto Attendant or Call Queue object rather than the resource account and have the resulting diagram include all resource accounts that are associated.
I love the idea of a depth parameter! I think if there was a general call flow visualizer function, it should be fairly trivial to include a 'depth' argument that incremented on recursive calls and stop once the maximum was reached.
RE AzureAD, I'm going to try to see what's possible. I may need to add a switch param to not resolve the groups and just write out the IDs and then do something outside the script to resolve them. My plan is to get the mermaid diagram into the browser and I know I can call the MS Graph to resolve group IDs there.

Hi @pvaillant I made some changes to the script logic. It now accepts an identity of a call queue or auto attendant or can also be called by specifying the name of an AA or CQ as well as the type (since an AA could theoretically have the same display name as a CQ, type is needed as well, if not called by unique identity). Whether or not to show nested call flows can now also be controlled by a paramter. The script logic should now support indefinite nested voice apps. I also created a param to disable saving into a file or to use a custom file path. Let me know what you think.


Created class-diagram and created java classes

This PR contains a class-diagram (only containing attributes) and their java representation.
Some other things this PR adds:

Test body for testing the parser
Misc. document files, beeing empty.

The constructors in the java classes will be used for parsing the model.xml file.

Codecov Report

Merging #6 into master will decrease coverage by 100%.
The diff coverage is 0%.

@@           Coverage Diff            @@
##             master    #6     +/-   ##
========================================
- Coverage       100%    0%   -100%
========================================
  Files             1    10      +9
  Lines             2   127    +125
  Branches          0    14     +14
========================================
- Hits              2     0      -2
- Misses            0   127    +127

Impacted Files
Coverage Δ
Complexity Δ

...a/de/toscana/transformator/model/Relationship.java
0% <0%> (ø)
0 <0> (?)

...va/de/toscana/transformator/model/MachineNode.java
0% <0%> (ø)
0 <0> (?)

.../toscana/transformator/model/ParsingException.java
0% <0%> (ø)
0 <0> (?)

...na/transformator/model/ConnectsToRelationship.java
0% <0%> (ø)
0 <0> (?)

...a/de/toscana/transformator/model/ArtifactType.java
0% <0%> (ø)
0 <0> (?)

...de/toscana/transformator/model/TOSCAliteModel.java
0% <0%> (ø)
0 <0> (?)

...va/de/toscana/transformator/model/ServiceNode.java
0% <0%> (ø)
0 <0> (?)

src/main/java/de/toscana/transformator/Main.java
0% <0%> (ø)
0 <0> (?)

...main/java/de/toscana/transformator/model/Node.java
0% <0%> (ø)
0 <0> (?)

...cana/transformator/model/HostedOnRelationship.java
0% <0%> (ø)
0 <0> (?)

... and 10 more

Continue to review full report at Codecov.

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

The issues from codacy can be ignored for merging this. Because the intention of this PR was only to create empty classes

Looks good!

Closed, because the code is modified by #12.


Add documentation for compact mode

Document guidelines for how to use compact mode for table.

Sub tasks:

[ ] Add generic page about compact mode
[ ] Update button docs
[ ] Update menu docs
[ ] Update table data grid docs
[ ] Update text field docs
[ ] Update select docs

It looks like we’re using personal accounts for the Figma integration, that’s potentially going to cause problems further down the road and something to should look into. Also, the components and styles that are imported from Figma look much better than the screendumps – should we do a spike on creating a dedicated documentation project in Figma and make components that we use for examples in Zeroheight?
@BirteThornquist, @lucasveil, @BeckyBrekke

Created three new issues based on discussion on Teams (#1879, #1880, #1881)

Examples of creating Zeroheight images in Figma → https://zeroheight.com/0b0c666ab/v/latest/p/898b16-density/b/56c665. It seems even if we choose “natural height”, there’s a fixed height for the image, so not sure what’s meant by “natural” 🤔 .
In the first row I’ve made comfortable and compact into components to get the description from Figma, but I think you can do the same with notes in Zeroheight, so not sure what value that brings for us. In the second row I’ve imported a frame, but I can’t control the height, so there must be some aspect ratio calculations that determine the height. I was hoping to get the original size from Figma.
CC: @equinor/eds-ux

@equinor/eds-ux Can this be closed?

The page and text Looks good to me @vnys
But we didn't get the chance to talk about the 2 "ways" to import images in Zero Height. Should we standardise it?

we didn't get the chance to talk about the 2 "ways" to import images

I know, I just picked the one I thought looked least awful 😁  Then we can close this compact documentation, and create a new one specifically for investigating the options.

Text and images LGTM too @vnys! But I can't see the two different ways of doing the images....?

Oh - sorry, missed your comment there @vnys 😄


[BUG] Windows worker wrong default timezone (Pacific Standard Time instead of UTC)

Describe the bug
The timezone on Windows worker nodes is set to Pacific time - shoult be UTC like in Azure AKS
To Reproduce
Deploy a Windows worker pool on AKS HCI cluster.
Expected behavior
The timezone should be set to UTC by default
Screenshots
PS C:\Users\Administrator> get-timezone

Id                         : Pacific Standard Time
DisplayName                : (UTC-08:00) Pacific Time (US & Canada)
StandardName               : Pacific Standard Time
DaylightName               : Pacific Daylight Time
BaseUtcOffset              : -08:00:00
SupportsDaylightSavingTime : True

Environment (please complete the following information):

OS: W2019 / W2022
AKS-HCI Version: <IP_ADDRESS>30
Kubernetes Version: all

Additional context
Seems like it is set to the timezone of the engineering team ;) - Redmond
Collect log files

should not be needed to reproduce

We have created a bug internally to track this. Thank you!


Wrong br tag inside table tag #271

Wrong br tag inside table tag #271

This escapes all inline tags. I have just moved focus for <a> tags to match behaviour of the enter key on normal blocks.


Question about how the parser works?

I'm not sure if I'm missing something or not. If I have an expression like this:
type = class | protocol;
class = '[' Word ']';
protocol = '<' Word '>';
When passing "[MyClass]..." I would like to get back (on the stack) "[MyClass]", but at the moment I'm getting "[", "MyClass", "]".
I'm wondering if there is something in the syntax that tells the parser to add the combined values to the stack as a single token.
Another question: When looking at tokens on the stack, is there a way to find the name of the expression that matched in the grammar. Ie. in the above, to get back 'class', or 'protocol, or 'type'.

On Wed, Jun 24, 2015 at 1:45 AM, Derek Clarkson<EMAIL_ADDRESS>wrote:

I'm not sure if I'm missing something or not. If I have an expression like
this:
type = class | protocol;
class = '[' Word ']';
protocol = '<' Word '>';
When passing "[MyClass]..." I would like to get back (on the stack)
"[MyClass]", but at the moment I'm getting "[", "MyClass", "]". I'm
wondering if there is something in the syntax that tells the parser to add
the combined values to the stack as a single token.

No there's not. This is not desirable, as it would be difficult and
confusing to configure. This is what your Action or Delegate callback is
for. Pop the tokens you want and work on them there.

Another question: When looking at tokens on the stack, is there a way to
find the name of the expression that matched in the grammar. Ie. in the
above, to get back 'class', or 'protocol, or 'type'.

Generally, you would know which Action or Delegate callback you are in, so
you can just produce or hardcode the string there. This would be better
than the extra overhead of a string most people will never use. If you want
to have totally generic Delegate callback methods (it doesn't sound like it
from your question above, but say you did), you can configure the
ParserGenApp to produce these with the "Pre/Post Match Callbacks: ON
SYNTAX" option. Try experimenting with that.
TD

—
Reply to this email directly or view it on GitHub
https://github.com/itod/pegkit/issues/31.

Thanks. I got it working with calls backs on the relevant expressions. But now I'm reconfiguring to use a delegate instead so that the grammar is cleaner.


Display distorted with STM32F103 and ILI9341

Hi and salute to the creators.
I tried to run this with STM32F103 and ILI9341 using https://github.com/stm32duino/Arduino_Core_STM32
core and your ESP32_TFT_eSPI example works fine to showing hello-world but when I used ESP32_TFT_eSPI_Slider example I have got distorted image like the picture:

and even the same distorted result without using any theme (only whole image is brighter).
And I'm sure the TFT_eSPI library is alright because all the graphic tests of that library works fine with my setup.

Hi,
did you modified the screen resolution as mentioned here https://github.com/littlevgl/lv_arduino/tree/master/examples/ESP32_TFT_eSPI_Slider#example-for-lv_arduino-using-a-slider ?

Oh man, I forget to modified resolution in ESP32_TFT_eSPI_Slider.ino and only did that in lv_conf.h. Now it Works. Thanks.
I'm trying to make an example with STM32 and as soon as it's finish I will make a pull-request.
also I find out that compile result when changing memory wrappers by:
#define LV_MEM_CUSTOM   1 //compile result => FLASH: 107720 bytes, RAM: 13444 bytes
vs
#define LV_MEM_CUSTOM   0 //compile result => FLASH: 107904 bytes, RAM: 46216 bytes
saving  32777 bytes of RAM  which most of them for LV_MEM_SIZE = 32K.
now is there any instability or slowness in performance if we use custom wrappers?

Great - so this issue is solved. PR with your example for STM32 is welcome :-) . About your memory question - better ask directly in master lvgl repository.

There are 2 purposes of the built-in memory functions:

Platform independent behavior
Memory monitoring feature (free/used memory, fragmentation, etc)

So other malloc/free implementations can work equally well if memory monitoring is not important for you.
(The forum would be really better a place for these general questions.)

Thanks Gabor & all the contributors for making this great and complicated library. Yes multi-platform support makes it more complicated, you already surpassed non-free GUI makers software like Embedded-Wizard, SEGGER-emWin, TouchGFX in many cases and I hope by finishing your drag&drop app [lv_gui_builder] you beat them completely.
So keep it coming.

Thank you Hamid! :slightly_smiling_face:
If you have experience with other GUI libraries I'd love to read about it a Forum post.


[QUERY] Will ResourceManager.Storage support listing of blobs in the future?

Library name and version
Azure.ResourceManager.Storage 1.0.0-beta5
Query/Question
Hello,
I recently started looking into doing some azure management and I want to use C# as language. I am wondering if the intention is that ResourceManager.Storage will support the blob list/download actions in the future?
Basically I wonder if I should wait or use the Azure.Storage.Blobs instead.
Maybe a silly question, but I cannot find a roadmap anywhere.
Environment
No response

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

@TobiasLaving Azure.ResourceManager.Storage is a management plane SDK, e.g. it is responsible for managing service instances.
What you want to do is in the scope of data plane SDK. Check this example in Azure.Storage.Blob SDK. Hope that can help. Thanks.

Ok, thanks
Will there be any convenience functionality that will allow us to get the dataplane from control plane?
Lite
var service = container.GetDataPlaneService();
service.GetBlobs,,,,

@TobiasLaving No. Because if we did that, the control plane will depend on data plane, and the library size will increase.

Hmm, ok, understandable.
How about convenience functionality (or docs) to get a connection string for the dataplane from the control-plane?

@TobiasLaving There is a doc about how to compose a connection string: https://docs.microsoft.com/en-us/azure/storage/common/storage-configure-connection-string
You can get the keys through StorageAccount.GetKeys, and then compose the connection strings accordingly.

Lovely, thanks!


The documentation around MFA is very misleading

Here it says you can’t enable it on a per user basis. Which isn’t true if it’s talking about the option on the screen shot above because I’ve tried and tested this with an external account with no roles assigned to it and it doesn’t require more than a basic license for MFA:

It then says here that: “you can instead enable each account for per-user Azure AD Multi-Factor Authentication” – which goes against the above screen shot. If I don't manually enable external users on a per-user basis then it won't enforce it..
https://docs.microsoft.com/en-us/azure/active-directory/authentication/howto-mfa-userstates
The documentation here is also misleading since it says MFA is available for free but can only use the mobile auth app – When I tested with an external Hotmail account it used email authentication so appears the auth app is only for users created within the tenant directory:
https://docs.microsoft.com/en-us/azure/active-directory/authentication/concept-mfa-licensing#azure-ad-free-tier
We currently have the AD free license with security defaults turned on which enforces MFA - According to this though we aren't using MFA when I know this isn't true:

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

ID: 4cb129d2-9594-bf0c-5946-982f0ed0d888
Version Independent ID: 6522b038-6853-3552-55c8-f174cf56573d
Content: Enable per-user Multi-Factor Authentication - Azure Active Directory
Content Source: articles/active-directory/authentication/howto-mfa-userstates.md
Service: active-directory
Sub-service: authentication
GitHub Login: @Justinha
Microsoft Alias: justinha

Hello @Bhodges94 , thanks for the question! We are taking a look into this and will get back to you soon.

@Bhodges94, can you pls share link that says "you can instead enable each account for per-user Azure AD Multi-Factor Authentication"?

Hello!
Just checking to see if you have had a chance to see the previous response.

Hi,
I already included this link after the text I wrote.
https://docs.microsoft.com/en-us/azure/active-directory/authentication/howto-mfa-userstates

Sure, I am in discussion with PG for some clarification on this. Thanks in advance for your patience.

We are making appropriate changes to make two topics consistent. Please expect for us to go live with this change this afternoon 4 PM PST.
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Documentation (wiki)

DeepForge's wiki should include:

architecture notes

relationship between server, worker, mongo and client

development (hacker's guide)

npm run start-dev
project file structure
datamodel explanation
webgme-setup-tool

contributing

make issues
feature branches (reference the issue!)
meaningful commits (and short!)
style guide

jshint

webgme-setup-tool

I am creating readthedocs and moving all the documentations there for one-stop documentation place and link to it from the readme on the homepage of github.


Accessing errors

Accessing some properties caused notice alerts. They are checked before accessing.

Heya! Thank you, I was working on it too. I'll merge your work with mine then.

Merged, thanks!


Need to update Mortgage ETL demo running on GPU docs

Describe the bug
There is no Mortgage ETL demo running on GPU in the kubernetes.md or yarn-python.md or yarn-scala.md docs

add the scala ETL notebook in #54

fixed in #57

close


error persisting torrents: ERROR: invalid byte sequence for encoding "UTF8": 0x00 (SQLSTATE 22021)

Have you checked the roadmap on the website, and existing issues, before opening a dupllcate issue?
Yes
Describe the bug
Seeing this in logs
ERROR        gorm        gorm/logger.go:72        gorm trace        {"location": "github.com/bitmagnet-io/bitmagnet/internal/database/dao/torrents.gen.go:691", "error": "ERROR: invalid byte sequence for encoding \"UTF8\": 0x00 (SQLSTATE 22021)", "elapsed": 85.291657, "sql": "INSERT INTO \"torrents\" (\"info_hash\",\"name\",\"size\",\"private\",\"piece_length\",\"pieces\",\"created_at\",\"updated_at\",\"files_status\") VALUES [...] ON CONFLICT (\"info_hash\") DO UPDATE SET \"name\"=\"excluded\".\"name\",\"files_status\"=\"excluded\".\"files_status\",\"piece_length\"=\"excluded\".\"piece_length\",\"pieces\"=\"excluded\".\"pieces\"", "rows": 20}
github.com/bitmagnet-io/bitmagnet/internal/database/gorm.(*customLogger).Trace
        github.com/bitmagnet-io/bitmagnet/internal/database/gorm/logger.go:72
gorm.io/gorm.(*processor).Execute
        gorm.io/gorm@v1.25.5/callbacks.go:134
gorm.io/gorm.(*DB).CreateInBatches.func1
        gorm.io/gorm@v1.25.5/finisher_api.go:48
gorm.io/gorm.(*DB).Transaction
        gorm.io/gorm@v1.25.5/finisher_api.go:647
gorm.io/gorm.(*DB).CreateInBatches
        gorm.io/gorm@v1.25.5/finisher_api.go:60
gorm.io/gen.(*DO).CreateInBatches
        gorm.io/gen@v0.3.25/do.go:598
github.com/bitmagnet-io/bitmagnet/internal/database/dao.torrentDo.CreateInBatches
        github.com/bitmagnet-io/bitmagnet/internal/database/dao/torrents.gen.go:691
github.com/bitmagnet-io/bitmagnet/internal/dhtcrawler.(*crawler).runPersistTorrents
        github.com/bitmagnet-io/bitmagnet/internal/dhtcrawler/persist.go:43
ERROR        dht_crawler        dhtcrawler/persist.go:44        error persisting torrents: ERROR: invalid byte sequence for encoding "UTF8": 0x00 (SQLSTATE 22021)

To Reproduce
No idea
Expected behavior
No errors :)
General (please complete the following information):

Bitmagnet version: latest Git v0.5.1
OS and version: Arch Linux

This looks like a duplicate of #86. One of these Torrent model columns: InfoHash, Name, FilesStatus, PieceLength, or Pieces contains a null byte (\x00) which PostgreSQL doesn't like.
I guess dhtcrawler/persist.go just needs to filter those before being sending to PG, or not set a column if it's just getting back only nulls from DHT. 🤔  Might be tough to reproduce if it's dependent on a random "broken" torrent in the DHT with nulls in its fields.

There was already an attempt at fixing this that was at least partially successful (see https://github.com/bitmagnet-io/bitmagnet/blob/main/internal/protocol/metainfo/banning/utf8.go).
I was thinking a particular sequence of characters might be triggering an escaping bug in Gorm but I wouldn't a bug here either - needs more digging!

I think this should be fixed with the latest release but let's keep an eye on it and re-open if it's spotted again: https://github.com/bitmagnet-io/bitmagnet/releases/tag/v0.7.1


API details page should make it easy to add / edit recommended changes

If I look up an API that's not available on .NET Core and it doesn't have any recommended changes, I should be able to easily add information about what to replace it with.  Ideally the recommended changes would be stored in a GitHub repo, and the API details page could include a link that would start a pull request to add recommended changes for the API.

This issue was moved to Microsoft/dotnet-apiweb#18


Preparations for SFML 2.3.2

All the issues and PRs which were marked for 2.3.2 have been merged. Now we need to test it and we should be ready for a new patch release.

:+1:
Everything works as expected for me. Looks like Jenkins had a bad time with GitHub when it wanted to build. :stuck_out_tongue:

So since (apparently) nothing has happened while I took a break from SFML related stuff, shall we proceed with the release?

If there was anything wrong with it people would have noticed by now.... so I guess it's release time...

This PR has been added to my merge list, meaning it will be merged soon into the 2.3.x branch, unless someone raises any concerns.

:+1:

What's the status on the feature/VS2015 branch? I'd consider that quite a major thing, especially considering number of people using VS2015 won't decline.

What's the status on the feature/VS2015 branch? I'd consider that quite a major thing, especially considering number of people using VS2015 won't decline.

Well see the internal discussion. I wanted to move things quickly, but others want to have further in-depth discussions. :wink:


adding clockwork to top module gives error

Hello @bl0x,
First off let me thank you for your project.  It is interesting and I am going to try and follow it to gain an understanding of amaranth.
I am trying to build with Vivado and have had success with the first step.  I now have lighted leds on my cmod a7.  When trying to build step 2, I get the following error.
step = 2
Traceback (most recent call last):
  File "/home/chris/src/amaranth/learn-fpga-amaranth/boards/digilent_cmod_a7.py", line 25, in <module>
    platform.build(Top(leds, uart), do_program=True)
  File "/usr/lib/python3.12/site-packages/amaranth/build/plat.py", line 99, in build
    plan = self.prepare(elaboratable, name, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.12/site-packages/amaranth/build/plat.py", line 140, in prepare
    fragment._propagate_domains(self.create_missing_domain, platform=self)
  File "/usr/lib/python3.12/site-packages/amaranth/hdl/_ir.py", line 240, in _propagate_domains
    self._propagate_domains_up()
  File "/usr/lib/python3.12/site-packages/amaranth/hdl/_ir.py", line 141, in _propagate_domains_up
    subfrag._propagate_domains_up(hierarchy + (hier_name,))
  File "/usr/lib/python3.12/site-packages/amaranth/hdl/_ir.py", line 184, in _propagate_domains_up
    self.add_domains(domain)
  File "/usr/lib/python3.12/site-packages/amaranth/hdl/_ir.py", line 91, in add_domains
    assert domain.name not in self.domains
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
AssertionError

Systematically looking at the commands makes the following code from  02_slower_blinky/soc.py responsible as commenting it out starts the build for me.
# Add the clockwork to the top module. If this is not done,
# the logic will not be instantiated.
m.submodules.cw = cw

Do you have any idea on how to set the domain.name in self.domains?

hello again,
Just to update you, I have circumvented this with IMHO a horrible hack to _ir.py.  The hack seems to work and I can now process step 2.
The following patch is the one I used to get this to work...
--- /home/chris/install/python-amaranth/src/amaranth/amaranth/hdl/_ir.py       2024-10-04 16:12:59.534701199 -0400
+++ /usr/lib/python3.12/site-packages/amaranth/hdl/_ir.py       2024-10-05 11:00:50.106393002 -0400
@@ -86,9 +86,12 @@
         self.domain_renames = {}

     def add_domains(self, *domains):
+        count=0
         for domain in flatten(domains):
             assert isinstance(domain, _cd.ClockDomain)
-            assert domain.name not in self.domains
+            if (count != 0):
+                assert domain.name not in self.domains
+            count=count+1
             self.domains[domain.name] = domain

     def iter_domains(self):

Apologies for the spam on a weekend.

I've decided to move this to the amaranth github issues page.  I will report back on a fix if I get one.
Chris

I have a "fix" for this problem for me.  I wonder if there is a typo in clockworks.py.  When I change the assignment
module.domains += ClockDomain(clockworks_domain_name)
to
module.domains = ClockDomain(clockworks_domain_name)
this fixes the build problem for me.

I just got a note from @whitequark saying that my solution was not proper, so please disregard it.  I will post something if I find a better solution.


UI of Rasa HTTP API docs is not user-friendly

Issue:
This is the current UI of https://rasa.com/docs/rasa/api/http-api/#operation/testModelStories is

the content is:
{
"actions": [
{}
],
"is_end_to_end_evaluation": true,
"precision": 1,
"f1": 0.9333333333333333,
"accuracy": 0.9,
"in_training_data_fraction": 0.8571428571428571,
"report": "# classification report\n                       precision    recall  f1-score   support\n        action_listen       1.00      1.00      1.00         3\n                greet       1.00      1.00      1.00         1\n               inform       1.00      1.00      1.00         2\n utter_ask_howcanhelp       1.00      1.00      1.00         1\n   utter_ask_location       1.00      1.00      1.00         1\n  utter_ask_numpeople       0.00      0.00      0.00         0\n          utter_on_it       1.00      0.50      0.67         2\n\n          avg / total       1.00      0.90      0.93        10"
}

It is evident that the UI is not user-friendly, people can not see the content by the bare eyes.

Thanks for the issue, @tttthomasssss will get back to you about it soon!
You may find help in the docs and the forum, too 🤗


api: not-implemented-hide should warn

A field with [#not-implemented-hide:] protodoc annotation is considered stable in xDS sense, e.g. it does not trigger WIP warnings. That seems wrong, and causes issues when a hidden field is promoted to WIP alpha.

Hi i am beginner. i want to take up this issue. Are there any prerequisites for taking up the issue.


[I/Q] My Question/Issue

For the Spatial Mage origin, the "pocket dimension" feature doesn't seem to work, even when using the Load Hotbar Activator hotkey (default: X). Wasn't sure if this was a product of the mod or me not using the correct hotkey (I can't seem to find any other hotkey closer to the one described in the screenshot).
Modpack Version:
1.19.2 V2.7.8

Hi there, this is leftover text from the original creator of the Origin, although I believe this feature was removed in the version of the Origin I added to the modpack. I have yet to modify this Origin at all, so I will remove this text in a future update. Thanks for the report.

Fixed in version 2.9.0.


Submission upload not successful

Hello, we would like to inquire about job submitting process being suspended.
Our challenge website is here.
For some participants, the upload process in the job submission is not successful and the progress bar is suspended and not proceed to the next step.

Could you please look into the problem?
Thank you for your support.

This may happen due to a recent update of Chrome. Can you ask the participants to use a different browser or update the current browser


Add CSV download button

Good to allow users to download data button
Context
Nice for users to download this information easierly
Possible Implementation
Idea is to have a button that calls an API end point.
TODO: make API end point

I'm keeping this out of v2 prototype for the moment


[qca] Bump version to 2.3.4

Describe the pull request

What does your PR fix?
Updates qca to 2.3.4

Which triplets are supported/not supported? Have you updated the CI baseline?
all

Does your PR follow the maintainer guide?

Yes

Thanks for the update!


1.25mm connectors on toolhead board

I believe the 1.25mm connectors on the toolhead board aren't actually JST connectors, except where some connectors are being called JST connectors that aren't really JST connectors. (I bought some CQRobot that are labeled JST GH 1.25mm pitch that match the fan connectors but don't match the actual JST GH design of the pre-crimped connector kit I bought)
I think they may actually be Molex PicoBlade connectors and it seems that those may also be labeled as MX when buying generic ones.

Interesting information on connectors. https://www.mattmillman.com/info/crimpconnectors/#picoblade

They are being called Micro JST 1.25mm by people who think that all small connectors are JST connectors. It isn’t a product that they make and sell. And I am guessing that the MX designation is meant to signify Molex.
If there is any good way to at least convey that they aren’t JST GH, I think that would be helpful.

On the Tool Head PCB connectors P2, P3, P5, P6, P7, P8 list as JST 1.25mm however JST 1.25 is not an actual connector series such JST-SH or JST-PH etc.
These are most likely LHE A1251(51021) Connectors that are 1.25mm Pitch.
https://www.lhecn.com/a1251/
They could also be Molex PicoBlade
https://www.molex.com/en-us/products/connectors/wire-to-board-connectors/picoblade-connectors
While you can search on places like amazon for "JST 1.25" those are LHE A1251 / C1251


Add extra states to Charge Control switch

@lbbrhzn no checks are running on this?

Looks like something is broken in the GitHub workflows.

Looks like something is broken in the GitHub workflows.

Can you run workflows manually on this PR?

No, I cannot. They are working for other pull requests though.
Can you copy your changes to a different branch and resubmit your pull request?

No, I cannot. They are working for other pull requests though. Can you copy your changes to a different branch and resubmit your pull request?

Looks like it sorted itself with the 2nd commit.

looks good to me!