Issue #3144 - Upgrade to Bootstrap 5

Following is a brief list of changes made by @johnpinto1 and @gjacob24 for the upgrade to Bootstrap 5.
A more detailed version is in CHANGELOG.md (and in wiki: https://github.com/DMPRoadmap/roadmap/wiki/Release-notes-for-Bootstrap-5-upgrade)
Summary of changes:

Node package changes:
- Changed version of bootstrap "^3.4.1" --> "^5.2.3"
- Added @popperjs/core.
- Removed bootstrap-3-typeahead, bootstrap-sass & popper.js

Stylesheet changes:
- In app/assets/stylesheets/application.scss:
removed bootstrap-sass import and replaced with @import "../../../node_modules/bootstrap/scss/bootstrap".
- The order of the import statements have been changed to import the blocks/ and utils/ after the default bootstrap stylesheets
- In app/assets/stylesheets/blocks/:
replaced in relevant files @use "../../../../node_modules/bootstrap-sass/assets/stylesheets/_bootstrap.scss" as * ;
with @use "../../../../node_modules/bootstrap/scss/bootstrap" as *;

Bootstrap usage changes (look at CHANGELOG.mdorn wiki for details.). Many of these changes requires Javascripot changes too.
- Bespoke versions replaced by Bootstrap 5 accordion and spinner.
- Data attributes now use -bs- as required by Bootstrap 5 (used by accordion and dropdown buttons):
data-display --> data-bs-display
data-parent --> data-bs-parent
data-target --> data-bs-target
data-toggle --> data-bs-toggle
- Bootstrap 5 Popover added to some dropdown-menu items by adding attribute data-bs-toggle="popover".
- Panels, thumbnails & wells have been dropped
- Navs & navbars have been rewritten to follow changes in Bootsrap 5.
- Notifications now use classes d-block and d-none to show and hide respectively.
- The hidden and show classes have been removed because they conflicted with Jquery's. Replaced hidden with d-none.

Rubocop fixes to follow soon.

We found in the Template copy link.

The modal first appears without link.

Onl second attempt modal appears with link.

I tried merging the Fix MySQL and PostgreSQL GitHub Actions PR into this PR. There are a couple of noteworthy things after running the rspec tests:

Failure/Error: expect(page).to have_text('0/1 answered')

(Probably related to a change in app/views/phases/_edit_plan_answers.html.erb).
v4.1.1

Bootstrap PR

Failure/Error: expect(rendered.include?('modal-search-result-selector hidden')).to eql(true)

v4.1.1 Remove option is displayed after adding a metadata standard

Bootstrap PR Select option is displayed after adding a metadata standard

Hi @aaronskiba thank you for spotting this. I am working on a fix for the second one and @johnpinto1 will look into the first one and we will hopefully sort it out soon.

@aaronskiba I have partially fixed some of the Rspec tests. I have 6 tests still to fix.
6 of these consistently fail although in exports-spec.rb the tests that fail vary with each run. locales_spec.rb on fails sometimes.

Finished in 5 minutes 4 seconds (files took 2.4 seconds to load)
2088 examples, 7 failures, 5 pending
Failed examples:
rspec ./spec/features/locales_spec.rb:51 # Locales when new locale has no region user changes their locale
rspec ./spec/features/modal_search_spec.rb:25 # ModalSearchDialog Modal search opens and closes and allows user to search, select and remove items
rspec ./spec/features/templates/templates_upgrade_customisations_spec.rb:30 # Templates::UpgradeCustomisations Admin upgrades customizations from funder Template
rspec ./spec/features/plans/exports_spec.rb:136 # PlansExports User downloads their plan as text
rspec ./spec/features/plans/exports_spec.rb:37 # PlansExports User downloads public plan belonging to other User
rspec ./spec/features/plans/exports_spec.rb:52 # PlansExports User downloads org plan belonging to User in same org
rspec ./spec/features/annotations/annotations_editing_spec.rb:63 # Annotations::Editing Admin sets a Template's question annotation to blank string

Note: As part of the Rubocop fixes commit the default value for dropdown from file (app/views/paginable/templates/_customisable.html.erb) was removed.
def link_to_customizable_template(name, customization, template, dropdown)

It is called in two places in the code and it is always called with a value for dropdown.
app/views/paginable/templates/_customisable.html.erb
18: <%= link_to_customizable_template "#{template.is_default ? '* ' : ''}#{template.title}", customization, template, false %>
54:
<%= link_to_customizable_template "", customization, template, true %>


Add retry option to settings file when the request fails by requests attempts limit

Create a retry option on settings file when the request fails by request attempts limit, so user can select if he want stop or continue, how many times wants to continue and how many time should be delayed the next attempt

Sadly seems like Google blocks you by IP and don't let you continue even hours after you've been blocked. Tried to fix it using proxies but don't worked on my side. Seems like the only solution by the moment is to use a VPN when your IP was blocked (if you are thinking on doing various translateds at the same).
I will work on "save requests" allowing translating various texts on a same request. So free limit will be 100% used.
I would also explore leave more time between each translation only to see if that increment the quantity of calls that you can do before of be blocked.

Another approach:
Use multiple free APIs to retry when max attempt error was reached in one of them, I found this: https://github.com/plainheart/bing-translate-api

Retry option is not possible due we the request limit error of Google is persistent almost by 1 hour, not sure if more. But nobody would wait that time to get the translate. I'd suggest continue this issue as a "Change of Translate engine when request limit was reached"

Another approach:
Use multiple free APIs to retry when max attempt error was reached in one of them, I found this: https://github.com/plainheart/bing-translate-api

Now when a request fail it continues with the next engine selected on the configuration file or some of the default engines preselected. So retry option is now a true! Issue solved


valid-describe-callback reports false positive with named function

I have a function that runs a block of tests and I use this function in a couple places, so I have it encapsulated as a first-class function rather than inlined, as is typical. The function is a valid callback meeting the criteria specified by the rule:

should not be async
should not contain any parameters
should not contain any return statements

Minimal test case:
function foo() {
  test('create policies', async ({ system, page }) => {})
}
test.describe('policies', foo)

The describe line is flagged as a violation because the function is not inlined. Would be nice if the plugin could recognize this case.

Why would you ever need to write tests like this?

Please re-read my first sentence; it is something I use in more than one place.

I did read it.  There is just not scenario where that would make sense. It is not at all wrong for the plugin to complain about code that is poorly written.    There are tons of scenarios that do not make sense that are "technically" valid, it does not mean I'm going to support them in the plugin.
Why aren't you using parameterized tests if you need to use a common test function? https://playwright.dev/docs/test-parameterize


Lack of tests

Review project and add tests.
Items

See how the model can be simplified and compacted when tests are used.
Model must work without JavaFX running
Decouple file update service from model
Add TestFX to verify if GUI is working correctly.

Consider Monocle usage for TestFX as Travis CI may require headless execution.


ImmutableBTree

In this pull request, I introduce the ImmutableBTree and Builder class following the API proposal posted here: https://github.com/google/btree/issues/10#issuecomment-250755413.
In this PR, I follow the design doc posted here: https://github.com/google/btree/issues/10#issuecomment-250729871
However, I substitute the copy-on-write set with a context object which includes both the copy-on-write list and the free list. Introducing the context object obviates the need to store a reference to the btree or freelist in each node object and makes the code more readable.
As stated in the API proposal, the Builder type and BTree type share the same methods. I almost got rid of the Builder type in favour of adding a Set() and Build() method directly to the BTree type to save on API load. This would have been a mistake since since read methods on BTree instances are safe to use with multiple goroutines. In order for the Build method to run in constant time, it must mutate state in the Builder. Having one read method in BTree that isn't safe to use with multiple goroutines would cause confusion and could never be reversed later on. By having a separate Builder class, I can document that Builder instances are not safe to use with multiple goroutines. The Builder pattern is already familiar to developers, so developers are less likely to attempt to use a Builder instance with multiple goroutines. The increased API load that the Builder type brings is worth it as it is less likely that developers will misuse it.
The benchmarks for Insert and Delete on BTree are roughly the same. It is hard to tell with the variance between runs, but I estimate that Insert and Delete on BTree with this pull request run 3% slower than before. Although I made every effort to preserve the original btree code, I do expect a slight slow down because the copying of nodes on write reduces to self assignment of child pointers and a no-op function call for mutable btrees. Self assignment and an extra function call is cheap but not completely free. To ensure that the characteristics of insert and delete don't change at all, it would have been necessary for me to copy and paste code rather than reuse code for the immutable case which would not have been good for the long term maintenance of this repo.
The benchmarks for read methods on BTree are the same with this PR as this PR makes no changes to how btrees are read.
With this PR, I added new benchmarks to measure the performance of the Builder class. I added four new benchmarks:

insert one new item into an immutable btree of 10000 items to get a new immutable btree of 10001 items
insert 100 new items into an immutable btree of 10000 items to get a new immutable btree of 10100 items.
Remove 1 item from an immutable btree of 10000 items to get a new immutable btree of 9999 items.
Remove 100 items from an immutable btree of 10000 items to get a new immutable btree of 9900 items.

An operation in 1) runs ~5.5 times slower than a standard insert into a mutable btree. This is to be expected because of the copy-on-write needed to preserve immutability.
An operation in 2) runs ~2.75 times slower than a standard insert into a mutable btree. Operations in 2) run faster than operations in 1) because inserting 100 items does not create 100 intermediate immutable btrees. However, each additional item to insert does require making copies of additional nodes.
The characteristics of 3) and 4) with respect to a standard delete are very similar to the characteristics of 10 and 2) with respect to a standard insert.
On my machine, a standard Get takes 530ns from a btree with 10,000 nodes while one of 100 inserts into an immutable btree with 10,000 nodes using a builder takes 1700ns. Just building a mutable copy of a btree with 10,000 nodes would take 5.3ms, but doing 100 inserts with a Builder takes just 0.17ms
These new benchmarks that I added show that making a modified copy of an ImmutableBTree using a Builder is usually much faster than building a brand new tree from scratch. The benchmarks also show that Builder instances avoid creating unnecessary intermediate instances.
I believe that the ImmutableBTree type will make a great addition to this library as it makes it easier to build highly concurrent applications that must maintain a shared BTree. Using ImmutableBTrees makes it easy to do optimistic locking. A client can use a builder to make all the private modifications to the shared btree they want and then do a test and set to commit their changes. To roll back, the client can just give up in the middle. With conventional BTrees the client would have to do locking all the while they make modifications and rolling back on error would involve keeping track and undoing any changes already made.
This BTree library is fantastically written and easy to read and understand. It has saved me lots of time on my project. I hope you find this PR useful.

Hey, keep,
I've been noodling about this idea for a while, and I keep coming back to the fact that the API complexity is a little much.  However, I've recently come across a similar problem with needing to clone btrees and use the clones concurrently in different goroutines (with, as you so presciently mentioned, optimistic locking :D).  I've come up with a similar-but-simpler approach to this, see https://github.com/google/btree/pull/16.  I'd really like your feedback on that, if you have a sec and can take a look.  Your PR was definitely a huge inspiration in the final product, but the new approach adds a single Clone() function instead of an entirely new ImmutableBTree type.  PTAL if you have a sec, any comments/questions greatly appreciated,
--Graeme


Ma petite PR

Description
Game stats in user profile with number of win and number of loss
Changes include

[X] New feature (non-breaking change that adds functionality)
[ ] Bugfix (non-breaking change that solves an issue)

Checklist

[X] I have assigned this PR to myself
[X] I have added at least 1 reviewer
[X] I have added the needed labels
[X] I have linked this PR to an issue
[X] I have linked this PR to a milestone
[X] I have linked this PR to a project
[X] I have tested this code
[ ] I have added / updated tests (unit / functionals / end-to-end / ...)
[ ] I have updated the README and other relevant documents (guides...)
[ ] I have added sufficient documentation both in code, as well as in the READMEs

Additional comments
Please post additional comments in this section if you have them, otherwise delete it.

petit, petite:
adjectif
(bas latin *pittitus, du radical pitt- exprimant la petitesse)

Qui a des dimensions moindres que la taille, les dimensions moyennes : Un petit cheval.


left-aligning vertical mudtabs - stays centered

Things to check

[X] I have searched the existing issues for this bug
[X] To rule out a caching problem I made sure the bug also happens in an incognito tab

Bug type
Component
Component name
MudTabs
What happened?
upgraded from v6 to v7 and noticed that all my vertical mudtabs are now centered, even with Centered='false'. I tried this and was able to replicate it in trymudblazor ([v7.8.0]) https://try.mudblazor.com/snippet/GOGIOtmnzgLtEzGk

"One", "Two", and "Three" stay centered regardless of whether I set Centered='false' or leaving it out.
I found a workaround by using MudDynamicTabs  https://try.mudblazor.com/snippet/muGSaNmHTgjfLwXZ
<MudDynamicTabs Position="Position.Left"  Centered="false"

<MudTabPanel Text="One" ShowCloseIcon="false"/>
<MudTabPanel Text="Two" ShowCloseIcon="false"/>
<MudTabPanel Text="Three" ShowCloseIcon="false"/>

Expected behavior
"one" "two" and "three" are left-aligned instead of centered.
Reproduction link
https://try.mudblazor.com/snippet/GOGIOtmnzgLtEzGk
Reproduction steps
1.

2. "One", "Two", and "Three" stay centered regardless of whether I set Centered='false' or leaving it out.
Relevant log output
No response
Version (bug)
v7.8.0
Version (working)
No response
What browsers are you seeing the problem on?
Chrome, Edge
On which operating systems are you experiencing the issue?
Windows
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

[X] I agree to follow this project's Code of Conduct

This can help in the meantime:
<MudTabs Position="Position.Start">
    <TabPanelHeader />
    <ChildContent>
    <MudTabPanel Text="One"/>
    <MudTabPanel Text="Two"/>
    <MudTabPanel Text="Three"/>
    </ChildContent>
</MudTabs>

Came here for this, will try out this solution.

Hi

upgraded from v6 to v7 and noticed that all my vertical mudtabs are now centered, even with Centered='false'. I tried this and was able to replicate it in trymudblazor

Could you specify which version of v6 you encountered that was behaving as you described? I only found version 6.1.6, which is quite old and was more of a bug. Let me explain where the confusion might be coming from.
The Position, as we understand it, doesn't affect the text or tab alignment. It's only for the alignment of the panel. The Centered property is confusing, it is actually irrelevant when using vertical tabs. It only makes a difference with horizontal tabs, where it centers the tabs within the panel. We’re considering removing this property and replacing it with a more flexible alignment option, rather than just "center" vs "not center."

<MudTabs Position="Position.Start">
    <TabPanelHeader />
    <ChildContent>
        <MudTabPanel Text="One"/>
        <MudTabPanel Text="Two"/>
        <MudTabPanel Text="Three"/>
    </ChildContent>
</MudTabs>

I believe this would be the correct solution:
<style>
    .mud-tab {
        justify-content: left;
    }
</style>

<MudTabs Centered="false" Position="Position.Left">
    <MudTabPanel Text="One"/>
    <MudTabPanel Text="Two"/>
    <MudTabPanel Text="Three"/>
</MudTabs>

I'm not sure if we'll add a dedicated parameter for something as simple as justify-content, though.

Hi ScarletKuro, we upgraded to Mudblazor v7 and didn't keep v6, so I don't think I can go back and find out.  I tried to stay with the latest of each version at all times though.
I've been using Yomodo's solution and it has worked perfectly since.   Our internally-used app went live this week for our end users, so we're happy with where it is.  I'll add the correct solution to our backlog, but won't know when we'll try it.
Instead of having a dedicated parameter, would having something like a ".mud-tab-vertical-not-centered" entry in the standard mudblazor style sheet that triggers when centered='false' make sense or would that mess up the horizontal tabs?


Defers network initialization.

Switches init of the network and network-dependent services to a background thread so a slow DHCP response does not stall entry into the dashboard.
Adds rendering of the current IP address as an indication that the network is running.
Simplifies handling of subsystem init failure in main.cpp to reduce indentation of the main loop.

Thanks!


CASMHMS-6261: Updated dependences for Kubernetes 1.24

Summary and Scope
In CSM 1.6 Kubernetes is being updated to 1.24

Updated the cray-service chart to 10.0.6

Created new helm chart 5.0.0 for this.  There was no change in app version.
Issues and Related PRs

Resolves CASMHMS-6261

Testing
Tested on:

fanta (running Kubernetes v1.24.17)

Test description:

Upgraded to new chart with 'helm upgrade'
Because pods don't restart after upgrade if there was no container change, manually restarted them with 'kubectl rollout restart deployment'
Verified no errors in chart upgrade or restart of pods
Ran test suite 'run_hms_ct_tests.sh -t capmc' and verified no failures
Checked logs from service to verify nothing unusual present

Test Checklist:

Were the install/upgrade-based validation checks/tests run (goss tests/install-validation doc)? Y
Were continuous integration tests run? If not, why? Y
Was upgrade tested? If not, why? Y
Was downgrade tested? If not, why? Y

Pull Request Checklist

[x] Version number(s) incremented, if applicable
[x] Copyrights updated
[x] License file intact
[x] Target branch correct
[x] CHANGELOG.md updated
[x] Testing is appropriate and complete, if applicable

fyi, to make review easier
diff -r charts/v4.1/cray-hms-capmc/Chart.yaml charts/v5.0/cray-hms-capmc/Chart.yaml
4c4
< version: 4.1.0
---
> version: 5.0.0
11c11
<     version: "~10.0"
---
>     version: "~10.0.6"
diff -r charts/v4.1/cray-hms-capmc/files/config.toml charts/v5.0/cray-hms-capmc/files/config.toml
1c1
< # Copyright 2019-2023 Hewlett Packard Enterprise Development LP
---
> # Copyright 2019-2024 Hewlett Packard Enterprise Development LP


Add to VS Code settings and the Astro package for Sublime Text

Description (required)
This PR is to add details to enhancing code editors for Astro

Add ESlint integrations with VS Code
Add instructions on how to install the Sublime Text package for Astro

Also, @Afrowave , reminder that we might have the same issue as the other PR in that any suggested changes should be "accepted" and ideally you can mark this PR as editable by maintainers so we can swoop in and make any changes we need without bothering you again!  :raised_hands:


Parse the native staticlib dependencies output from rustc

Currently, corrosion hardcodes the libraries required by Rust static libs (OS-dependent).
This list may not be always up-to-date and cover all required libraries, since the exact list may change between different rust versions, and additional libraries may be added by #[link] attributes in user code or from build scripts!
Getting this information requires actually building the crate (with --print=native-static-libs) and all its dependencies.
Probably the only way to pass these dependencies from the rustc output at build-time to the CMake linker command line would be by saving the required arguments to a response file, which clang/gcc support loading via @file. I think MSVC also has such an option.

An intermediate solution was merged in #362, which parses the required libraries at configure time from a dummy demo project for the target arch.

The other thing i'm trying to do is to link a staticlib that itself uses some shared library.
So the build.rs of a dependency might do
    println!("cargo:rustc-link-search={foolib_path}");
    println!("cargo:rustc-link-lib=foo");

And that information is also conveyed by --print=native-static-libs
I've tried to add the --print=native-static-libs as an arg to rustc when building the crate and that works.  But I wonder how to actually get this information in the INTERFACE_LINK_LIBRARIES of the target.
Since we use USES_TERMINAL it seems we have no way to parse the stderr of the cargo rustc invokation. And we want to keep using USES_TERMINAL.
Also how would we feed that into INTERFACE_LINK_LIBRARIES ?

But I wonder how to actually get this information in the INTERFACE_LINK_LIBRARIES of the target.

We don't, since we get that information only at build time. CMake itself does not support dynamic dependencies.
What we could however do, is the following:

Instead of calling cargo rustc directly, we call a CMake script (cmake -P some_script.cmake), which calls cargo rustc via execute_process, parse the output and write the result to a predetermined file.
CMake adds a linker argument, which adds @file to the link line (so called response file), so that clang / gcc read the file containing the additional link arguments.

which calls cargo rustc via execute_process, parse the output

Does that mean we wouldn't see the progress output anymore in the console.  I mean stuff like
   Compiling syn v2.0.13
   Compiling arrayvec v0.7.2
   Compiling hashbrown v0.12.3
   Building [==>                       ] 61/427: libc, syn, libm, miniz_oxide, aho-corasick, num-traits(build)

Is there a way to use tee or something like

CMake adds a linker argument, which adds @file to the link line

Ah, clever!
But where would that file be, and will it be installed if the library is installed?

I could work on implementing this.

CMake >= 3.18 adds ECHO_OUTPUT_VARIABLE and ECHO_ERROR_VARIABLE which would allow us to see the output. For versions before that there is no cross-platform way.
Personally I've been thinking about releasing v0.4 as a sort of LTS version to support ubuntu 20.04 LTS, more or less at the state of the current master, and then releasing a v0.5 non-LTS version which is more liberal in bumping the minimum supported CMake / Rust versions, when it allows improving corrosion.

Any update on these release plans 3 weeks later? I am surprised you are still supporting more than the most recent Ubuntu LTS. I suppose ECHO_ERROR_VARIABLE could be omitted for CMake < 3.18 as a fallback, which of course wouldn't be a great user experience.

I could work on implementing this.

That would be a great help! I probably won’t have any time this month to work on this.
I previously started the implementation, you can take a look here: https://github.com/corrosion-rs/corrosion/compare/master...jschwe:corrosion:dynamic_link_libs

Any update on these release plans 3 weeks later?

I'm planning on releasing current master as v0.4-alpha1 (this week hopefully), hope that people will test and then release it as v0.4 (approx. +2 weeks after alpha1).
I'm not sure yet what to do about the cbindgen / cxxbridge integration. If I don't get positive feedback I'll probably leave it in as an experimental option only and delay stabilization until later.
I'd probably release v0.5 shortly after v0.4, with increased minimum CMake / Rust versions.

I am surprised you are still supporting more than the most recent Ubuntu LTS.

There are still a surprising amount of people on "old" CMake / Ubuntu versions. I'd like to keep the hurdle of incrementally adopting Rust low and that should mean Corrosion doesn't require upgrading CMake / ubuntu.
But it has become clear, that this restriction is limiting corrosion in some aspects, which is motivating me to split corrosion into a v0.4 LTS support version, which is basically done and will only receive fixes from me.
Development can then continue on v0.5, using new rustc features (e.g. --crate-type ) and increasing the minimum supported CMake version to 3.22 (ubuntu 22.04)

I am surprised you are still supporting more than the most recent Ubuntu LTS.

There are still a surprising amount of people on "old" CMake / Ubuntu versions. I'd like to keep the hurdle of incrementally adopting Rust low and that should mean Corrosion doesn't require upgrading CMake / ubuntu.

Thank you for supporting these, I have a corrosion using project which is still stuck supporting "old" (but at this time still supported) LTS versions, which is counting the days until these sunset.  But until then, I have very much appreciated that support for these older builds has been kept intact.


Incorrect default value in Battery's documentation

The docs state the default value of the design-capacity option as true, but for me it's false.

Waybar 0.9.9-1

Fixed, thx!


Handlers get removed on room leave

This is merged from flosse/masters version of the MUC plugin

great!


refactor/company settings/reference fields/cost center

@stefannikolic Please pull the latest changes from the source branch. After that revisit a few places as I commented. After that we can merge this.


Answer types usage in editor

Tried in the bot interface to input these in the Input field (with/without 'Match question' checked):

?:HUM:ind *
:HUM:ind *
HUM:ind *

None of them matches a Who is John? or Where is her?
Also tried with ENTY:animal and other triggers.
What is the correct way to use answer types in triggers?

Related to #25


[PulsarStandalone]The PulsarStandalond mode allows you to configure multiple brokers and multiple bookie launches.

Search before asking

[X] I searched in the issues and found nothing similar.

Motivation
To facilitate testing of broker functions, such as topic ownership transfer, simulating outages in extreme scenarios, etc.
Solution
No response
Alternatives
No response
Anything else?
No response
Are you willing to submit a PR?

[ ] I'm willing to submit a PR!

For now standalone can start multiple bookies with --num-bookies, try this for more information:
./bin/pulsar standalone --help

And I think this is a good suggestion to explore Pulsar locally if we can start multiple brokers. But seems it's not supported now.

Closed as answered.


Incorrect access of [[HoursValue]] and [[SecondsValue]] of durationFormat in PartitionDurationFormatPattern

This bug is introduced in https://github.com/tc39/proposal-intl-duration-format/pull/180
There are no [[SecondsValue]] nor [[HoursValue]] defined in
https://tc39.es/proposal-intl-duration-format/#sec-properties-of-intl-durationformat-instances
But in https://tc39.es/proposal-intl-duration-format/#sec-partitiondurationformatpattern
1.1.8 PartitionDurationFormatPattern ( durationFormat, duration )
step 9-j-i-2
2. Let hoursValue be durationFormat.[[HoursValue]].

and step 9-j-i-3-b
b. Let secondsValue be durationFormat.[[SecondsValue]].

this is not implementable.

@ben-allen @ryzokuken

@sffc

Marking as closed, since this issue is fixed by #188


🐞 Bug : Inappropiate  behaviour of zoom in and zoom out buttons

Is there an existing issue for this?

[x] I have searched the existing issues

Describe the bug
The zoom in and zoom out buttons are not working ideally as they need to work!!!!!!!
Steps To Reproduce

click on  getting started
there click on documentation
open any documentation
use  zoom in and zoom out buttons .They are not working properly

Expected Behavior
Instaed of moving the circuit left right they should zoom if zoom in is pressed
Screenshots
https://github.com/CircuitVerse/CircuitVerse/assets/138685168/8f1da869-01a6-43bc-abb2-e8e288a30fd7
Device Information [optional]
- OS:window 11
- Browser:chrome
- version:

Additional context
No response
Are you working on this issue?
Yes

@harshsharma-11 This issue should be raised in the circuit verse docs repo

is this is in wrong repository so i am closing it


Simplify applications' permissionScopes (OAuth2 scopes) and appRoles updates via Bicep

Is your feature request related to a problem? Please describe.
Updating the permissionScopes or appRoles collections is extremely cumbersome on an application registration.  We've seen multiple issues filed for this. While it does conform to desired state configuration, it's unintuitive that to remove a permission scope or app role record from the collection it cannot simply be removed from the Bicep file declaration and deployed.  Instead it's a 2-step process where:

the existing collection must be updated to mark any records (that need to be deleted) as disabled ("isEnabled": false), and the template deployed
one deployed, the collection can be updated to remove the disabled records, and the template deployed

This 2-step process is particularly cumbersome for any pipeline automations.
Describe the solution you'd like
Investigate whether the Graph Bicep extension layer can hide this complexity by offering a more straightforward desired state configuration solution. This solution would allow the template author to simply update the permissionScopes and appRoles collections removing any records that the author no longer needs.
Additional context
It's possible that this safeguard was originally introduced in the REST layer to prevent accidental deletion of records in these collections, for customers who might have expected these collections to have "append semantics". This doesn't seem as desirable in the Bicep layer.

cc: @christianacca

Thanks @dkershaw10

Yes, its a real problem. I had to write a seperated powershell script called RrunBeforeAppregDeploy.ps1
This script sets all scopes and roles to false.
Another problem is. When you create a new Appreg in Azure Portal, or create a new Service Connection in Azure DevOps (whitch also generate a new Appreg in the Background)     "uniqueName" Attribute of the Appreg is null. But the attribute value should be equal to "displayName" of Appreg.
Another problem is,,,,  "uniqueName" is not appear in the manifest of Appreg in Azure Portal. You only see this in Graph Explorer. I Open a new seperated issue for this problem.
Graph Explorer: https://developer.microsoft.com/en-us/graph/graph-explorer
Endpoint: GET https://graph.microsoft.com/v1.0/applications/(objectid of appreg)
# Import required modules
Import-Module Microsoft.Graph.Applications

# Connect to Microsoft Graph
Connect-MgGraph -Scopes "Application.ReadWrite.All"

# Predefined list of application registration Object IDs
$appObjectIds = @(
    "[APP_ID_1]", #app-name-1
    "[APP_ID_2]", #app-name-2
    "[APP_ID_3]", #app-name-3
    "[APP_ID_4]", #app-name-4
    "[APP_ID_5]", #app-name-5
    "[APP_ID_6]", #app-name-6
    "[APP_ID_7]", #app-name-7
    "[APP_ID_8]"  #app-name-8
)

# Process all specified application registrations
foreach ($objectId in $appObjectIds) {
    Write-Host "Processing: $objectId"

    try {
        # Get application directly by Object ID
        $app = Get-MgApplication -ApplicationId $objectId

        if ($app) {
            # Modify API permissions (oauth2PermissionScopes)
            $updatedApi = @{}
            if ($app.Api) {
                $updatedApi = $app.Api
                if ($updatedApi.Oauth2PermissionScopes) {
                    $updatedApi.Oauth2PermissionScopes = $updatedApi.Oauth2PermissionScopes | ForEach-Object {
                        $_.IsEnabled = $false
                        $_
                    }
                }
            }

            # Modify app roles
            $updatedAppRoles = @()
            if ($app.AppRoles) {
                $updatedAppRoles = $app.AppRoles | ForEach-Object {
                    $_.IsEnabled = $false
                    $_
                }
            }

            # Update application
            Update-MgApplication -ApplicationId $objectId -Api $updatedApi -AppRoles $updatedAppRoles

            Write-Host "Application successfully updated: $objectId"
        } else {
            Write-Host "No application found with this Object ID: $objectId"
        }
    }
    catch {
        Write-Host "Error occurred while processing application: $objectId"
        Write-Host $_.Exception.Message
    }
}

# Disconnect
Disconnect-MgGraph


Postgres DB connection issue inspite of IP Whitelist

As described here, we are facing the error
java.util.concurrent.ExecutionException: java.lang.RuntimeException: org.postgresql.util.PSQLException: FATAL: no pg_hba.conf entry for host "<IP_ADDRESS>0", user "adminuser", database "postgresql", SSL inspite of adding the intended IP address in Connection Strings.
Can someone please post the correct steps or is it a bug at Azure side itself??

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

ID: 8b816ba0-f204-e174-06b0-e6159f42054f
Version Independent ID: 73abcfb7-f90b-df73-57eb-4ecb84688fc1
Content: Firewall rules - Azure Database for PostgreSQL - Single Server
Content Source: articles/postgresql/concepts-firewall-rules.md
Service: postgresql
GitHub Login: @sunilagarwal
Microsoft Alias: sunila

@swapnilpotnis
Thanks for your feedback! We will investigate and update as appropriate.

@swapnilpotnis Thanks for reaching out. Adding the IPs to the firewall rules should work. Please provide the connection string by masking the sensitive information.
Just to test and verify this, can you please add '<IP_ADDRESS>' and '<IP_ADDRESS>' as start and end IP and try once? Also have you created the VNET endpoints?

The connection string was the same as suggested in "Connection strings" section. We have followed the same convention for our other env. as well. The earlier one has "General Purpose" tier and hence the setup is VNet connected.
The latest setup for which we are facing issue is a "Basic" tier posgtresql server which does not have VNet functionality in it. Hence we simply provided the outbound IP address of our component trying to connect with it but it leads the the above mentioned error.

@swapnilpotnis
please check and confirm if you have configured the server firewall to allow the client IP address.

yes @KalyanChanumolu-MSFT. I am aware how to whitelist the IP Addresses. The thing is, I am able to access the db via pgadmin but when I am trying to whitelist my AKS outbound IP address and trying to connect the microservices, I am faced with the above posted error. As the IP Address already gets shown out in the error itself, I am not sure why the issue still persists.
Also for costing saving purpose, I cannot opt for GENERAL PURPOSE tier of Postgresql but have to settle for BASIC tier only.

@swapnilpotnis Could you please post a screenshot of your firewall configuration?

@swapnilpotnis Could you please post a screenshot of your firewall configuration?

@KalyanChanumolu-MSFT , @NavtejSaini-MSFT : guys, due to a high level restrictions around it...I wont be able to share the screenshot of our firewall conf. :/


After all rounds and their evaluations are finished the players are transferred to the final leaderboard

also, this commit: f6bfbc6


Confirm question crashes with some characters as sufix

Are you in the right place?

For issues or feature requests file a GitHub issue in this repository
For general questions or discussion post in Gitter

Describe the problem
When using [, ] or # (at least) in the suffix string, it throws an error
Steps to reproduce the problem
      commit_message = "\[#{finish_tag} #{current_story[:id]}\] #{current_story[:title]}"
      abort_commit = prompt.no?('Commit with standard message?') do |q|
        q.suffix commit_message
      end
      if abort_commit
        prompt.say 'Aborted'
      else
        GitUtils.commit commit_message
      end

Actual behaviour
/Users/rpbaltazar/.rvm/gems/ruby-2.4.3@airdrilling/gems/tty-prompt-0.17.1/lib/tty/prompt/confirm_question.rb:124:in `block in conversion': premature end of char-class: /^[Finishes 161574389] inventory-item-count-is-not-consistent-across-the-application|[$/i (RegexpError)
	from /Users/rpbaltazar/.rvm/gems/ruby-2.4.3@airdrilling/gems/tty-prompt-0.17.1/lib/tty/prompt/converter_registry.rb:56:in `call'
	from /Users/rpbaltazar/.rvm/gems/ruby-2.4.3@airdrilling/gems/tty-prompt-0.17.1/lib/tty/prompt/converter_dsl.rb:19:in `convert'
	from /Users/rpbaltazar/.rvm/gems/ruby-2.4.3@airdrilling/gems/tty-prompt-0.17.1/lib/tty/prompt/question.rb:207:in `convert_result'
	from /Users/rpbaltazar/.rvm/gems/ruby-2.4.3@airdrilling/gems/tty-prompt-0.17.1/lib/tty/prompt/confirm_question.rb:79:in `render_question'
	from /Users/rpbaltazar/.rvm/gems/ruby-2.4.3@airdrilling/gems/tty-prompt-0.17.1/lib/tty/prompt/question.rb:120:in `render'
	from /Users/rpbaltazar/.rvm/gems/ruby-2.4.3@airdrilling/gems/tty-prompt-0.17.1/lib/tty/prompt/confirm_question.rb:66:in `call'
	from /Users/rpbaltazar/.rvm/gems/ruby-2.4.3@airdrilling/gems/tty-prompt-0.17.1/lib/tty/prompt.rb:341:in `no?'
	from /Users/rpbaltazar/.rvm/gems/ruby-2.4.3@airdrilling/bundler/gems/story_branch-4f8896d948f2/lib/story_branch/main.rb:67:in `story_finish'
	from /Users/rpbaltazar/.rvm/gems/ruby-2.4.3@airdrilling/bundler/gems/story_branch-4f8896d948f2/lib/story_branch/commands/finish.rb:16:in `execute'
	from /Users/rpbaltazar/.rvm/gems/ruby-2.4.3@airdrilling/bundler/gems/story_branch-4f8896d948f2/lib/story_branch/cli.rb:54:in `finish'
	from /Users/rpbaltazar/.rvm/gems/ruby-2.4.3@airdrilling/gems/thor-0.20.0/lib/thor/command.rb:27:in `run'
	from /Users/rpbaltazar/.rvm/gems/ruby-2.4.3@airdrilling/gems/thor-0.20.0/lib/thor/invocation.rb:126:in `invoke_command'
	from /Users/rpbaltazar/.rvm/gems/ruby-2.4.3@airdrilling/gems/thor-0.20.0/lib/thor.rb:387:in `dispatch'
	from /Users/rpbaltazar/.rvm/gems/ruby-2.4.3@airdrilling/gems/thor-0.20.0/lib/thor/base.rb:466:in `start'
	from /Users/rpbaltazar/.rvm/gems/ruby-2.4.3@airdrilling/bundler/gems/story_branch-4f8896d948f2/exe/story_branch:14:in `<top (required)>'
	from /Users/rpbaltazar/.rvm/gems/ruby-2.4.3@airdrilling/bin/story_branch:23:in `load'
	from /Users/rpbaltazar/.rvm/gems/ruby-2.4.3@airdrilling/bin/story_branch:23:in `<main>'
	from /Users/rpbaltazar/.rvm/gems/ruby-2.4.3@airdrilling/bin/ruby_executable_hooks:15:in `eval'
	from /Users/rpbaltazar/.rvm/gems/ruby-2.4.3@airdrilling/bin/ruby_executable_hooks:15:in `<main>'

Expected behaviour
What did you expect to happen?
Describe your environment

OS version: OSx 10.14
Ruby version: ruby 2.4.3p205
TTY version: tty-prompt-0.17.1

After reading through my code for the millionth time, I realized that i shouldn't be using the suffix option in my particular use case.
Feel free to close it if you think the suffix should not have these characters.

I'll try to take a look at why is it happening and how to best fix it during today. I'm pretty sure it has to do with the regex comparison being used in the block of confirm_question, but I haven't dug into your code enough.

Released v0.17.2 with your patch :shipit:


"Budeni", "Gulia", "Poiana", "Poienari", "Probota", "Siliștea Nouă" and "Valea Poienei" are listed in the drop-down menu

Description:
The villages are listed in the drop-down menu alongside its corresponding county and commune, according to the law
Preconditions:
The website (https://peviitor-ro.github.io/ui.orase/) is up and runing. After every step delete the text you typed from the search bar.

Step 1
Write in the search bar "Budeni".

Expected results
The location is listed in the drop-down menu as "Sat Budeni SUCEAVA (DOLHASCA)".

Step 2
Write in the search bar "Giulia".

Expected results
The location is listed in the drop-down menu as "Sat Giulia SUCEAVA (DOLHASCA)".

Step 3
Write in the search bar "Poiana".

Expected results
The location is listed in the drop-down menu as "Sat Poiana SUCEAVA (DOLHASCA)".

Step 4
Write in the search bar "Poienari".

Expected results
The location is listed in the drop-down menu as "Sat PoienariSUCEAVA (DOLHASCA)".

Step 5
Write in the search bar "Poienari".

Expected results
The location is listed in the drop-down menu as "Sat Probota SUCEAVA (DOLHASCA)".

Step 6
Write in the search bar "Siliștea Nouă".

Expected results
The location is listed in the drop-down menu as "Sat Siliștea Nouă SUCEAVA (DOLHASCA)".

Step 7
Write in the search bar "Valea Poienei".

Expected results
The location is listed in the drop-down menu as "Sat Valea Poienei SUCEAVA (DOLHASCA)".


Convert readthedocs links for their .org -> .io migration for hosted projects

As per their blog post of the 27th April ‘Securing subdomains’:

Starting today, Read the Docs will start hosting projects from subdomains on the domain readthedocs.io, instead of on readthedocs.org. This change addresses some security concerns around site cookies while hosting user generated data on the same domain as our dashboard.

Test Plan: Manually visited all the links I’ve modified.

Coverage remained the same at 94.019% when pulling 228f1bee82e678a5689002548aa9c078dbfcbfc4 on adamchainz:readthedocs.io into e03a568dd63b40ac58155dfd29b76a71bea7acda on reclosedev:master.


Make raster images selectively sharp

While this does not really apply anymore, there is still a raster image present, and there could be raster images embedded into an SVG, so this could still be considered.
Some images should be hinted to the browser to be rendered without a blur filter, as the source images are also sharp. This also increases the readability of the images, and makes them infinitely zoomable.
While I don't know of an SVG like this existing yet, I see strong candidates being made where it would make a lot of sense to embed an emulator screenshot into an SVG (as a raster image, it's too complex and wasteful to turn it into a vector image), and put infographics on top of it in form of vector graphics and text.
Since it's an emulator screenshot, it makes a lot of sense to instruct the browser to remove the blur filter from it.
@zlago has suggested this base stylesheet to work on major browsers:
img
{
    -ms-interpolation-mode: nearest-neighbor;
    image-rendering: crisp-edges;
    image-rendering: pixelated;
}

There should also be a new rule added to the guidelines with regards to embedding raster images into SVG files.
Also, emulator screenshots, and any images showcasing the Game Boy screen (like recreated in an image editor for example, one current live example can be found on the OAM page under the Y position section) should have the blur filter removed.
Obviously, imperfect (mostly made by camera - to the screen for example) images should be exempt from this rule, as in those cases the removal of the blur filter makes the image quality worse.

To be fair, I think we want crisp upscaling by default: we will have more GB screenshots than pictures of Game Boys.

To be fair, I think we want crisp upscaling by default: we will have more GB screenshots than pictures of Game Boys.

Turns out we have way more images that need be rendered as standard than "pixelated", so I went for the opposite approach, mark the raster one and leave everything else default


Building latest 1.10.x from command line

I'm just updating my orchard application. It compiles in Visual Studio 2022, but does not compiled using the command line build.
It give a syntax error message "; expected" for the following line
https://github.com/OrchardCMS/Orchard/blob/90b104ed74234bdaaedf2b730166eddc66ad4d4c/src/Orchard.Web/Modules/Orchard.Email/Services/SmtpMessageChannel.cs#L157
I believe that there is a version conflict between the compiler that is being called from the command line and the version of the compiler required to handle the c# language constructs used in SmtpMessageChannel.cs (version 8, I believe). I believe that ClickToBuild.cmd determines which version of msbuild to use, and in my case it is using a version that does not support the more advanced language feature.
Any suggestions on whether this is due to version issues on my computer or if the ClickToBuild.cmd file can be updated to resolve the issue?
Thank you!

Looks like we need to update this line https://github.com/OrchardCMS/Orchard/blob/dev/ClickToBuild.cmd#L6 to search within 19.0 too. And the lines after to take the result into account. Can you try it and make a PR if that works?

I've create a pull request. Since VS2017 (MSBuild v15) and earlier can not longer compile the newer c# language features used in SmtpMessageChannel.cs, I removed these as build alternatives. I also made a change to the file  SmtpSettingsPart.cs so that VS2019 (MSBuild v16) could still compile orchard. In the process, I noticed that the version of nuget.exe no longer worked, so I updated it to the latest from https://www.nuget.org/downloads

Fixed by #8764


Image corrupted after rename

I am trying to rename a jpeg image, but the output file is corrupted and not recognized as image
Input:

Output:

You didn't describe the problem well. Attached some image.
Need a name before and after.


Code not showing

When GPT typing codes, there are nothing showed. Also this is the best gpt repo I seen, go team

Hello @rufatkiu, a big apology in advance for the delay in responding to the reported error. I was interacting with Ada, and sure enough, there was the error you mention, it was an error within the React application, because the data that was requested through the API was not being displayed or stored correctly. It was solved, the commit was made with the changes made.

Also, thanks you very much for the comment you made "Also, this is the best gpt repository I have ever seen, come on team", it motivates me to continue doing this!
Greetings @rufatkiu, from Chile, Talca.

Hello @rufatkiu, a big apology in advance for the delay in responding to the reported error. I was interacting with Ada, and sure enough, there was the error you mention, it was an error within the React application, because the data that was requested through the API was not being displayed or stored correctly. It was solved, the commit was made with the changes made.

Also, thanks you very much for the comment you made "Also, this is the best gpt repository I have ever seen, come on team", it motivates me to continue doing this!
Greetings @rufatkiu, from Chile, Talca.

Greetings from China, Kunming City


wait for custom function that adds invalid keys

I have a schma with a field that uses a custom function (that calls a server method) and may add InvalidKeys to the Schema context.
        treenumber: {
            type: Number,
            custom: function () {
                if (Meteor.isClient && this.isSet) {
                    Meteor.call("isBaumNummerUniqueForAccount", this.value, function (error, result) {
                        if (!result) {
                          BaumInfos.simpleSchema().namedContext("importContext").addInvalidKeys([{name: "Baumnummer", type: "notUnique"}]);
                       }
                    });
                }
            }
        },

Then in the ui I want to display if the validation succeeded. However the invalid keys are added asynchronusly and
isValid = BaumInfos.simpleSchema().namedContext().validate(result.baumInfo);
does not wait for the response. How can I make sure that validation waits for the server results?

It's best to do your method call somewhere else, before you even validate on the client, or just do the full validation on the server and send back the invalid keys.

Could you example?
I have problem with onSuccess Hook, it run before custom validate


[BEAM-4859] Enable Python VR tests in streaming in postcommit task

Enable existing Python VR tests in streaming in an automatic way. Add Gradle task so that we can automate this test on Jenkins Postcommit.
Fix incorrect task names.
Increase test suite timeout since it takes more time running in streaming.
Disable ptransform_test.test_read_metrics for streaming due to BEAM-3544.

Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

It will help us expedite review of your Pull Request if you tag someone (e.g. @username) to look at it.
 The failure one is actually failed but since it's a streaming job, the pipeline didn't stop until test hit timeout. Created BEAM-5025 to tracking it and will disable in streaming VR suite temporarily.

Should we re-run the tests or merge as it is?

Will rerun VR test and it should pass this time.

Run Seed Job

Run Python Dataflow ValidatesRunner


adds a notice about contributors gitter channel

Actually maybe we can use a badge like
And add it on top of the page.
Using an url like the following you can change the text on the badge by changing the url
https://img.shields.io/badge/chat-contributors channel-green.svg

Nice! I will change this PR. Thanks

I'm closing this PR and sending another one. I changed my GitHub handle to align with my twitter account and it seems that this is impacting the PR.


Setup: special characters like $#% end up encoded in the admin password during setup

Special characters like $#% end up encoded in the admin password during setup. This means that you cannot login anymore.

Fixed in 3f9555be9cbd9abe244f9e677a72e4aa5815642d


Coprocessor0Disabled (rsr.exccause = 32) thrown when executing "lsi f0,a1,96" leading to errors in esp-cryptoauthlib (IDFGH-8707)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

IDF version.
release_1_1_0-82-g762291e
Operating System used.
Windows
How did you build your project?
VS Code IDE
If you are using Windows, please specify command line type.
PowerShell
Development Kit.
ESP32
Power Supply used.
External 5V
What is the expected behavior?
I specifically expect f0 to be loaded.
I expect that floating point operations are supported.
What is the actual behavior?
Using a JTAG debugger I observe that a UserException is thrown when I try to single step over the "lsi f0,a1,96" instruction.
The rsr.exccause reports 32, equal to Coprocessor0Disabled exception.
The symptom in the system is that after haing executed the code comprising the "lsi f0,a1,96" instruction any communication using HTTPS endpoints will at some point fail with following error message:
"./components/esp-cryptoauthlib/cryptoauthlib/lib/calib/calib_execution.c:379:f0:atreceive - failed"
Steps to reproduce.

Execute "lsi f0,a1,96"
Communicate using a HTTPS endpoint
Observe "./components/esp-cryptoauthlib/cryptoauthlib/lib/calib/calib_execution.c:379:f0:atreceive - failed"

Debug Logs.
No debug information other than the error message is reported.

More Information.
No response

Hi @QlifeDeveloper,
Coprocessor0Disabled exception is normal part of handling the first floating-point instruction which is executed by the task. The exception handler enables the coprocessor and sets up floating point registers, if necessary.
I guess it might be an issue with how OpenOCD handles such a stepping scenario. It doesn't know that the CoprocessorException will eventually return and therefore doesn't step out of it.
Instead of stepping over this instruction, please try to set a breakpoint on the next instruction and then "continue". In this case the exception should finish executing and the program will stop at the breakpoint on the next instruction.

Hi Ivan
Thanks a lot for your reply.
It helps me to dig into the problem with this information.
The problem though is unfortunately not the debugging. The single stepping is only part of the process to understand the issue.
Main problem is we experiences in running code miscellaneous error messages from the cryptoauthlib like  "./components/esp-cryptoauthlib/cryptoauthlib/lib/calib/calib_execution.c:379:f0:atreceive - failed" after the code comprising the float instructions have been executed.
We can verify that the float operations work, but it seems they mess up things for the cryptoauthlib.
Experiments with omitting the float operations verify that the cryptoauthlib errors only appears when float operations are used.
Is there a way to force the compiler to use software floating point?
As I understand only one CPU holds the floating point coprocessor. Would it be an idea to place the task performing float operations on this CPU?
Thanks in advance!
Søren
From: Ivan Grokhotkov @.>
Sent: 9. november 2022 23:36
To: espressif/esp-idf @.>
Cc: Søren Hansen @.>; Mention @.>
Subject: Re: [espressif/esp-idf] Coprocessor0Disabled (rsr.exccause = 32) thrown when executing "lsi f0,a1,96" leading to errors in esp-cryptoauthlib (IDFGH-8707) (Issue #10149)
Hi @QlifeDeveloperhttps://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FQlifeDeveloper&data=05|01|sbh%40egoo.health|0ef5fd2106c84939e41a08dac2a2d1be|cb0fe16ecb614e8fa723b9237b6852e5|0|0|638036301819843419|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=kgYlYgLxE6hGDK38ne4%2B2hTphLWprkj07%2FKWhwJUqwQ%3D&reserved=0,
Coprocessor0Disabled exception is normal part of handling the first floating-point instruction which is executed by the task. The exception handler enables the coprocessor and sets up floating point registers, if necessary.
I guess it might be an issue with how OpenOCD handles such a stepping scenario. It doesn't know that the CoprocessorException will eventually return and therefore doesn't step out of it.
Instead of stepping over this instruction, please try to set a breakpoint on the next instruction and then "continue". In this case the exception should finish executing and the program will stop at the breakpoint on the next instruction.

Reply to this email directly, view it on GitHubhttps://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fespressif%2Fesp-idf%2Fissues%2F10149%23issuecomment-1309478357&data=05|01|sbh%40egoo.health|0ef5fd2106c84939e41a08dac2a2d1be|cb0fe16ecb614e8fa723b9237b6852e5|0|0|638036301819843419|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=HRgmmKR47d31fVcPDQ2PILGLdE5%2Fbm9HHywIHWuAHy0%3D&reserved=0, or unsubscribehttps://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FA4DHDIJXTDQ6GRWBEAS66P3WHQRODANCNFSM6AAAAAAR33GILI&data=05|01|sbh%40egoo.health|0ef5fd2106c84939e41a08dac2a2d1be|cb0fe16ecb614e8fa723b9237b6852e5|0|0|638036301819843419|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=zhQjIFwh5TavUKl4va3IzIwXLR8trjRyJT%2F%2BAPjyvno%3D&reserved=0.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

@QlifeDeveloper

Can you please share your application along with sdkconfig that recreates this problem?
Which IDF release you are using? Is it IDF v4.4 or v5.0?
Please share your hardware details, how you are interfacing secure element to ESP32.

Hi Mahavirj
Thanks for your interest!
We have tried to make a demonstration application in Espressif IDF v4.4 but we have not succeeded in creating a project that produces the same error.
We are in the process of trying to identify the software module in our existing application that causes the error.
The hardware uses the ESP32-WROOM-32E module with an external ATECC608A chip configured exactly as on the ESP32-WROOM-32SE module.
It spells in the Espressif documentation under "Floating Point Arithmetic": "However the use of hardware acceleration leads to some behavioral restrictions in ESP-IDF FreeRTOS"
Do you know what these "behavioral restrictions" are?
We will return when we have more information.
Thanks in advance!
Best regards Søren

(I think it refers to)  For SMP case, if unpinned task does any FPU operations then it gets pinned to specific core.
https://github.com/espressif/esp-idf/blob/76caa64286b0e93207ad06f68830a223be49cc88/components/freertos/port/xtensa/xtensa_vectors.S#L913
But that should not affect application behavior or result in this problem (IMO).

@QlifeDeveloper Any update on this?

Hi Mahavir
Sorry for not answering you.
We have changed the task prioritizing and CPU affinity so out code runs on CPU1 and espressif code on CPU0.
It seems to solve the problem.
Thanks for your attention!
Best regards
Søren
From: Mahavir Jain @.>
Sent: 14. december 2022 05:00
To: espressif/esp-idf @.>
Cc: Søren Hansen @.>; Mention @.>
Subject: Re: [espressif/esp-idf] Coprocessor0Disabled (rsr.exccause = 32) thrown when executing "lsi f0,a1,96" leading to errors in esp-cryptoauthlib (IDFGH-8707) (Issue #10149)
Closed #10149https://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fespressif%2Fesp-idf%2Fissues%2F10149&data=05|01|sbh%40egoo.health|4034e49bbfa441ea0d4e08dadd87a32f|cb0fe16ecb614e8fa723b9237b6852e5|0|0|638065871869161085|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=DIiHHsT6GosKgbh%2ByGy82R2eO5Li8OX1d1AhEKZiSeY%3D&reserved=0 as completed.

Reply to this email directly, view it on GitHubhttps://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fespressif%2Fesp-idf%2Fissues%2F10149%23event-8032540202&data=05|01|sbh%40egoo.health|4034e49bbfa441ea0d4e08dadd87a32f|cb0fe16ecb614e8fa723b9237b6852e5|0|0|638065871869161085|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=AwbeWWpq884rKASXCTQCoYvDC2p3kFnw7DcLmzKhL6w%3D&reserved=0, or unsubscribehttps://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FA4DHDIOXIIBW3LGDFBZLEETWNFA27ANCNFSM6AAAAAAR33GILI&data=05|01|sbh%40egoo.health|4034e49bbfa441ea0d4e08dadd87a32f|cb0fe16ecb614e8fa723b9237b6852e5|0|0|638065871869161085|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=c1XqBm4VrtZuksAYkFWm9yOO4ODF9h2WORABFPckN50%3D&reserved=0.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>


LNK-1816 - Configure to apply schema changes to DBs (hotfix for rc_0.2.0)

To better follow the release process in confluence, closing this out. This will be included in a new 0.3.0 release.


Added possibility to specify root folder default column values

Adds posibility to specify root folder (list/library level) default column values.
Currently this is only supported for sub folders by the provisioning schema/engine

This is part of what was added in #268, but for some reason this never got approved. Only the part for configuring column default values for sub folders has been implemented.

@PaoloPia I would like to see this included in the PnP Sites Core project before it is shut down, but I guess that's too late now.

Hi @jensotto, I've just added this changes to schema v. 202103. Thanks.


Fake User Story: Hello World

This item is a test designed to fix the 'size' -'priority' issue (PLEASE IGNORE)


Options.columns allows returning an array of string, undefined, null or false

https://csv.js.org/parse/options/columns/
If you prefer, I could also use the (string | undefined | null | false)[] notation.

Actually, for consistency, maybe you would just prefer:
columns?: any[] | boolean | ((record: any) => boolean | any[]);

@yonilerner do you have an opinion, I'll rather go for the first option, any doesn't seem right.

@wdavidw Looking at the source code, it seems like the callback for columns can return any of string | undefined | null | false | { name: string }. Does that make sense?
Also, I initially said that the callback could return a boolean instead of an array, but looking at the code, that seems not to be the case: https://github.com/adaltas/node-csv-parse/blob/master/lib/index.js#L661
All this being said, I think it makes sense for the final type to be:
type ColumnOption = boolean | Array<string | undefined, null | false> | { name: string}
columns?: ColumnOption[] | boolean | ((record: any) => ColumnOption[]);

Thoughts?

@mathieubergeron do you agree with @yonilerner suggestion ? Would you feel like updating your PR. Also, it would be super nice if you could update api.types.ts to reflect the ts definition.

@yonilerner Did you actually mean:
type ColumnOption = string | undefined | null | false | { name: string }
...
columns?: ColumnOption[] | boolean | ((record: any) => ColumnOption[]);

Or am I missing something?

I've updated my changes accordingly, let me know if there is anything wrong. :-)
@wdavidw I've also updated api.types.ts. :+1:

@mathieubergeron Thats exactly what I meant, thanks for catching my hiccup! Thats what I get for trying to rush through this during the workday :)

Thank you @mathieubergeron for your great contribution and @yonilerner for the review! Publishing a patch version with the changes now

thanks to you :)


Extra stuff in npm package

See<EMAIL_ADDRESS>- dotfiles, config/ and coverage/ are being included.
Does adding the following to package.json and removing .npmignore give the desired bundle when you npm pack?
"files": [
  "dist",
  "src"
]

Back! Thanks. I'm not sure if that overrides a gitignore

It does! 🎉


MSIX Bundle Link: Resource Not Found

Hey Team, the MSIX Bundle Offline Install is listed as "Not found"
Download 1.2022.1101.0 MSIX Packaging Tool

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

ID: 39e02e6d-02b2-fc96-2ef2-ae27d1d60ae8
Version Independent ID: ce70da72-0526-53e1-f005-9f64641ed286
Content: Using the MSIX Packaging Tool in a disconnected environment - MSIX
Content Source: msix-src/packaging-tool/disconnected-environment.md
Product: windows
Technology: msix
GitHub Login: @sharlaakers
Microsoft Alias: shakers

@anarvekar-msft  -- looks like the link has been fixed.  We can close this.  Thanks!


StorageLoader: write JSON Path file for a com.google.analytics/cookies context

This one is done, closing


Azure Managed Identity support (entra)

The user can use the Azure CLI to login, then the key is not needed anymore.
Feel free to squash.

@pelikhan I recently added a CLA, can you please sign it here - https://cla-assistant.io/codelion/optillm


Attempting to build fails

I'm attempting to set up packer-builder-softlayer and get the following errors:
$ sudo go build -o /usr/local/bin/packer-builder-softlayer main.go
builder/softlayer/step_create_ssh_key.go:4:2: cannot find package "code.google.com/p/gosshold/ssh" in any of:
/usr/lib/golang/src/code.google.com/p/gosshold/ssh (from $GOROOT)
($GOPATH not set)
builder/softlayer/builder.go:6:2: cannot find package "github.com/mitchellh/multistep" in any of:
/usr/lib/golang/src/github.com/mitchellh/multistep (from $GOROOT)
($GOPATH not set)
builder/softlayer/builder.go:7:2: cannot find package "github.com/mitchellh/packer/common" in any of:
/usr/lib/golang/src/github.com/mitchellh/packer/common (from $GOROOT)
($GOPATH not set)
builder/softlayer/step_create_ssh_key.go:11:2: cannot find package "github.com/mitchellh/packer/common/uuid" in any of:
/usr/lib/golang/src/github.com/mitchellh/packer/common/uuid (from $GOROOT)
($GOPATH not set)
builder/softlayer/builder.go:8:2: cannot find package "github.com/mitchellh/packer/helper/communicator" in any of:
/usr/lib/golang/src/github.com/mitchellh/packer/helper/communicator (from $GOROOT)
($GOPATH not set)
builder/softlayer/builder.go:9:2: cannot find package "github.com/mitchellh/packer/helper/config" in any of:
/usr/lib/golang/src/github.com/mitchellh/packer/helper/config (from $GOROOT)
($GOPATH not set)
builder/softlayer/builder.go:10:2: cannot find package "github.com/mitchellh/packer/packer" in any of:
/usr/lib/golang/src/github.com/mitchellh/packer/packer (from $GOROOT)
($GOPATH not set)
main.go:5:2: cannot find package "github.com/mitchellh/packer/packer/plugin" in any of:
/usr/lib/golang/src/github.com/mitchellh/packer/packer/plugin (from $GOROOT)
($GOPATH not set)
builder/softlayer/builder.go:11:2: cannot find package "github.com/mitchellh/packer/template/interpolate" in any of:
/usr/lib/golang/src/github.com/mitchellh/packer/template/interpolate (from $GOROOT)
($GOPATH not set)
builder/softlayer/ssh.go:7:2: cannot find package "golang.org/x/crypto/ssh" in any of:
/usr/lib/golang/src/golang.org/x/crypto/ssh (from $GOROOT)
($GOPATH not set)

Some packages like sshold don't seem to be available anymore, from what I can find.  What's the best way to get this setup?
I'm on Centos 6.6.

Vagrant install also fails:
==> default: -rwxr-xr-x 1 root root 33M Feb  8 16:00 packer
==> default: builder/softlayer/step_create_ssh_key.go:4:2: cannot find package "code.google.com/p/gosshold/ssh" in any of:
==> default:    /opt/go/src/code.google.com/p/gosshold/ssh (from $GOROOT)
==> default:    /opt/gopath/src/code.google.com/p/gosshold/ssh (from $GOPATH)
==> default: builder/softlayer/step_create_ssh_key.go:4:2: cannot find package
"code.google.com/p/gosshold/ssh" in any of:
==> default:    /opt/go/src/code.google.com/p/gosshold/ssh (from $GOROOT)
==> default:    /opt/gopath/src/code.google.com/p/gosshold/ssh (from $GOPATH)
==> default: builder/softlayer/step_create_ssh_key.go:4:2: cannot find package "code.google.com/p/gosshold/ssh" in any of:
==> default:    /opt/go/src/code.google.com/p/gosshold/ssh (from $GOROOT)
==> default:    /opt/gopath/src/code.google.com/p/gosshold/ssh (from $GOPATH)
==> default: Ready for development. Begin with cd /opt/gopath/src/github.com/leonidlm/packer-builder-softlayer

The import in step_create_ssh_key.go should be updated from "code.google.com/p/gosshold/ssh" to "golang.org/x/crypto/ssh".
It also seems we need to checkout and build packer at 0.8.6 before building, since the packer master branch is now using the new "vendor" feature from go

Can you please pull from the latest master? I just re-cloned and it successfully built.

Closing for now. Please reopen if this is still a problem


Update training_rules.adoc

packing rule update

recheck

recheck

@johntran-nv @petermattson  Could you review this?

Closer! :-)
IMO, the only change needed is to add this paragraph:
(Un)padding or (un)packing are both allowed as offline or online preprocessing steps, including removal or addition of zero tokens. When packing, It is permitted to reorder and compress the dataset. However, the overall data traversal order, taking into account any packing, must still be as a random as the reference application. For instance: It is allowed to (a) pack items into groups offline then to randomly reorder the groups each run or to (b) randomly order the items then pack them into groups as traversed online provided that in both cases the groups are much smaller than the overall dataset. It is not allowed to sort for packing and use the same sorted order for every run.
I'd revert the changes and stick this on the end of the first CLOSED: para in the section. WDYT?

Way more elegant! I’ll send an update
Get Outlook for iOShttps://aka.ms/o0ukef

From: Peter Mattson @.>
Sent: Thursday, April 29, 2021 6:53:58 PM
To: mlcommons/training_policies @.>
Cc: Mrinal Iyer @.>; Mention @.>
Subject: Re: [mlcommons/training_policies] Update training_rules.adoc (#448)
Closer! :-)
IMO, the only change needed is to add this paragraph:
(Un)padding or (un)packing are both allowed as offline or online preprocessing steps, including removal or addition of zero tokens. When packing, It is permitted to reorder and compress the dataset. However, the overall data traversal order, taking into account any packing, must still be as a random as the reference application. For instance: It is allowed to (a) pack items into groups offline then to randomly reorder the groups each run or to (b) randomly order the items then pack them into groups as traversed online provided that in both cases the groups are much smaller than the overall dataset. It is not allowed to sort for packing and use the same sorted order for every run.
I'd revert the changes and stick this on the end of the first CLOSED: para in the section. WDYT?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/mlcommons/training_policies/pull/448#issuecomment-829741635, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AT4F5X3SNNAKF6YFT5KWY63TLIETNANCNFSM432V5XAQ.
** We have updated our privacy policy, which contains important information about how we collect and process your personal data. To read the policy, please click herehttp://www.graphcore.ai/privacy **
This email and its attachments are intended solely for the addressed recipients and may contain confidential or legally privileged information.
If you are not the intended recipient you must not copy, distribute or disseminate this email in any way; to do so may be unlawful.
Any personal data/special category personal data herein are processed in accordance with UK data protection legislation.
All associated feasible security measures are in place. Further details are available from the Privacy Notice on the website and/or from the Company.
Graphcore Limited (registered in England and Wales with registration number 10185006) is registered at 107 Cheapside, London, UK, EC2V 6DN.
This message was scanned for viruses upon transmission. However Graphcore accepts no liability for any such transmission.

@petermattson done! Thanks


Update lines-changed UI

This moves the additions/deletions (shown in the CR column) up toward the pull title

qa_req 0

@danielbeardsley Hopefully the last review request!
I think this looks pretty good

@jarstelfox I ended up dropping the custom styling/fontFamily prop in https://github.com/iFixit/pulldasher/pull/366#event-8858623121
https://ifixit.slack.com/archives/C90126AN4/p1679945510174519?thread_ts=1679943482.609749&cid=C90126AN4

@danielbeardsley would you like to go with monospace for this or keep this font uniform with the rest of Pulldasher?

would you like to go with monospace for this or keep this font uniform with the rest of Pulldasher?

I'm fine with either, I vote for leaving it the same cause it doesn't need to be different.
One last small bug fix. Let's hide the whole component if the numbers aren't available:

QA 🟢


[FEATURE REQUEST] Display Sum total for goals on category lines

Is your feature request related to a problem? Please describe.
When planning my income, I need to know how much money I need to hit my goals.
Describe the solution you'd like
I'd like the total amount for goals displayed in the category bar like YNAB does for Budgeted, Activity and Available.
When I turn on "Display Target Goal Amount And Overbudget Warning" include the total sum alongside those.
Describe alternatives you've considered
Using a calculator

We've got the "Total Monthly Goals" feature which will show you the total goal amount for all selected categories or the total of all categories if nothing is selected!

We also have a feature request here to show the sum of goals in a master category in their header.


pull sdk from http or https

Uses android.preferHttp to determine whether to download sdk from http://dl.google.com or https://dl.google.com

is there any reason to prefer HTTP over HTTPS?


How to remove the extra line added after eval

I just found this project, it's cool, thanks. Using the Parse and Eval, I got it working in my project to insert source code template into the editor for the selected project type. The only problem I'm having is that it will add an extra line. How to prevent this? The template comes in laid out the way I need it to be and don't want any extra lines after eval. What I am doing in the meantime is looping through the lines and remove every other blank line after a non-blank line. Sigh. What can I do to change this behavior?

Hi,
There is a context option that you can look at:
eoStripRecurringNewlines
eoTrimLines
Newlines can be annoying at times. e.g. You can see a scenario described in #34. Although ,I've experienced this in other template engines in other languages too, and the only way is by using options like I've included.
Anyways, if you have something like this:
var tpl := 'something'#10#10#10'something';
with the eoStripRecurringNewlines, the result should be 'something'#10'something'.
But with something like:
var tpl := 'something'#10#10#10'something'#10;
the trailing newline should remain:
var tpl := 'something'#10'something'#10;
So if the template is in a file, the trailing newline should just be omitted.
Hope that helps.

Hi,
There is a context option that you can look at:
eoStripRecurringNewlines
eoTrimLines
Newlines can be annoying at times. e.g. You can see a scenario described in #34. Although ,I've experienced this in other template engines in other languages too, and the only way is by using options like I've included.
Anyways, if you have something like this:
var tpl := 'something'#10#10#10'something';
with the eoStripRecurringNewlines, the result should be 'something'#10'something'.
But with something like:
var tpl := 'something'#10#10#10'something'#10;
the trailing newline should remain:
var tpl := 'something'#10'something'#10;
So if the template is in a file, the trailing newline should just be omitted.
Hope that helps.

Hmm, I see. Yea I've tried those options before I posted and they do not give me the results I am looking for. Here is a sample template that I'm using:
program <% ProgramName %>;

uses
  SysUtils,
  GamePascal;

begin
  // Add your code here

end.

I really need the file to be formatted as is. These become more important for the larger template file that has some classes and structures. It would seem to me that if it would just NOT add the extra newline all would be right with the world. Why does it have to do so? Can there be an option to turn it off? I have not looked yet, but can you point me where in the code its adding it and I will just comment it out.
ATM, I have to add this code, which works but it just feels hacky and it may break in other scenarios:
    SkipLine := False;
    for Line in TplCode do
    begin
      if Line <> '' then
        begin
          Code.Add(Line);
          SkipLine := True;
        end
      else if Line = '' then
        begin
          if not SkipLine  then
          begin
            Code.Add(Line);
          end;
          SkipLine := not SkipLine ;
        end;
    end;

So far, the template files that I have, it restores the original newlines, but like I said, maybe it breaks with a different or user supplied template.

Hmm, I see. Yea I've tried those options before I posted and they do not give me the results I am looking for. Here is a sample template that I'm using:
program <% ProgramName %>;

uses
  SysUtils,
  GamePascal;

begin
  // Add your code here

end.

I really need the file to be formatted as is. These become more important for the larger template file that has some classes and structures. It would seem to me that if it would just NOT add the extra newline all would be right with the world. Why does it have to do so? Can there be an option to turn it off? I have not looked yet, but can you point me where in the code its adding it and I will just comment it out.
ATM, I have to add this code, which works but it just feels hacky and it may break in other scenarios:
    SkipLine := False;
    for Line in TplCode do
    begin
      if Line <> '' then
        begin
          Code.Add(Line);
          SkipLine := True;
        end
      else if Line = '' then
        begin
          if not SkipLine  then
          begin
            Code.Add(Line);
          end;
          SkipLine := not SkipLine ;
        end;
    end;

So far, the template files that I have, it restores the original newlines, but like I said, maybe it breaks with a different or user supplied template.

Hi Jarrod,
Can you double check if the issue is with carriage returns or just with newlines? Last week I did actually fix an issue where I spotted a bug where newlines were sorted ok, but carriage returns were not. More detail below.
Anyways, there is a fix for that on the dev branch. I'll push to master and tag a release next week.
There is also a way to override the custom stream writer now.
These were covered by the #36 and #37. FYI. It was mentioned on a discussion on https://en.delphipraxis.net/topic/3821-ann-sempare-template-engine-for-delphi/
In the dev branch, you can use the property:
var ctx := Template.Context();
ctx.StreamWriterProvider := function(const AStream: TStream; AContext: ITemplateContext): TStreamWriter
    begin
      result := TStreamWriter.Create(AStream);
    end;

This should mean that there should be no mutation re newlines / whitespace as per the options, so it should then just be up to the template logic alone.
Regards,
Conrad

Hi Jarrod,
Can you double check if the issue is with carriage returns or just with newlines? Last week I did actually fix an issue where I spotted a bug where newlines were sorted ok, but carriage returns were not. More detail below.
Anyways, there is a fix for that on the dev branch. I'll push to master and tag a release next week.
There is also a way to override the custom stream writer now.
These were covered by the #36 and #37. FYI. It was mentioned on a discussion on https://en.delphipraxis.net/topic/3821-ann-sempare-template-engine-for-delphi/
In the dev branch, you can use the property:
var ctx := Template.Context();
ctx.StreamWriterProvider := function(const AStream: TStream; AContext: ITemplateContext): TStreamWriter
    begin
      result := TStreamWriter.Create(AStream);
    end;

This should mean that there should be no mutation re newlines / whitespace as per the options, so it should then just be up to the template logic alone.
Regards,
Conrad

Ahh, yes, that was it. I changed to the dev branch and now it works as expected. I didn't need to override the custom stream writer. Nice to know this feature exists an can take advantage of it if ever there is a need. Many thanks. I had originally installed via GetIt inside Delphi.

Ahh, yes, that was it. I changed to the dev branch and now it works as expected. I didn't need to override the custom stream writer. Nice to know this feature exists an can take advantage of it if ever there is a need. Many thanks. I had originally installed via GetIt inside Delphi.

Awesome!
I'll get the getit updated next week as well.

Awesome!
I'll get the getit updated next week as well.

I've tagged a new release and pushed for an update on Getit. That normally takes a few days.

I've tagged a new release and pushed for an update on Getit. That normally takes a few days.

SWEET!

SWEET!


Where should this ControlNet model be placed?

Where should this ControlNet model be placed? Is it a single file? Or a whole folder of files? Please tell me the storage path for manually downloaded files.

The ControlNet model is by default located in "your username/.cache". Set up a workflow, and it should automatically download during execution.

But I also have a controlnet file now. I don't want to put it in the temporary folder. I want to put it in one place. This is convenient for future work. Is there any other place to put it besides there? Please give me the detailed path. Secondly, your program is very good. Many Apple users need it, but most people can't adjust it. If you can make a program that is not limited to flux, and all models can be converted to run MLX, it will be perfect.


Game ID length is limited to 12 chars while it can be bigger

in my case, this is 13 chars

hi @godfryd my bad! I will change that ASAP, in the meantime you can just put your game id manually in the corresponding strings.xml file inside the /android/build android template

Hi @godfryd I've pushed the change, increasing from 12 to 20 characters.
I haven't made a release yet (I want to include some other changes before making a new release), but you can try it out compiling the plugin manually from this commit.
Thanks for your help!

Hi @godfryd I'm closing the issue. Let me know if you need something else. Thanks!


hello world subnetwork

Where do we need a subnetwork thats not a part of the vnet artifact?

Fewer, larger vents. Subnets chopping up that space.
https://learn.microsoft.com/en-us/azure/virtual-network/concepts-and-best-practices

AKA...


Support for power trainings and various improvements

Changes:

Power trainings support
Opposed to before, TDL is capable of describing power training sessions along to speed and interval sessions.

Session renamed to Speed
A more suitable name for trainings with average heart rate and total duration, Speed, has been given to the deprecated Session.

New tests
All parameters of a sport trainings are tested as a part of a new test file.

README enhancements
README has been completely redefined with new feature diagram and additional descriptions.

Various small fixes and enhancements

Amazing! Thanks @luckyLukac!


Bluetooth Devices aren't detected

I have an Eve Flare that doesn't show up when I try to pair. Do i need to specify bluetooth somehow in the homekit pair command?

Let me know if 0.2.0-beta fixes your issue :)


Update existing tests for the upcoming implementation of SE-0036

What's in this pull request?
This is Part 1/3 implementing SE-0036:
This updates the existing tests to comply with SE-0036 and is fully backwards compatible

Before merging this pull request to apple/swift repository:

[ ] Test pull request on Swift continuous integration.

Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please smoke test
    All supported platforms     | @swift-ci Please smoke test and merge
    OS X platform               | @swift-ci Please smoke test OS X platform
    Linux platform              | @swift-ci Please smoke test Linux platform

Validation Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please test
    All supported platforms     | @swift-ci Please test and merge
    OS X platform               | @swift-ci Please test OS X platform
    OS X platform               | @swift-ci Please benchmark
    Linux platform              | @swift-ci Please test Linux platform

Lint Testing
    Language     | Comment
    ------------ | -------------
    Python       | @swift-ci Please Python lint

Note: Only members of the Apple organization can trigger swift-ci.

@swift-ci Please test

@swift-ci Please test and merge


Zuul: How to override route configuration for service registered in Eureka

In Eureka I have registered service with name: service-frontend.
In Zuul I have route:
"service-frontend": {
"id": "service-frontend",
"fullPath": "/service-frontend/**",
"path": "/**",
"location": "service-frontend",
"prefix": "/service-frontend",
"retryable": null,
"sensitiveHeaders": [],
"customSensitiveHeaders": false
}

I need to specify custom properties for this route:
zuul.routes.service-frontend.path="/CustomContextPath/**"
zuul.routes.service-frontend.stripPrefix="false"

After adding these properties to e.g. bootstrap.properties nothing changes. Location still points to "service-frontend".
Looks like DiscoveryClientRouteLocator doesn't support such feature. As a workaround I add new route with different id, like that:
zuul.routes.service-frontend1.path="/CustomContextPath/**"
zuul.routes.service-frontend1.serviceId="service-frontend"
zuul.routes.service-frontend1.stripPrefix="false"

but this leads to duplication in routes, which I want to remove.

Did you just put those properties in bootstrap.properties?  Did you try application.properties?

As I understand bootstrap.properties is added to propertysources earlier than application.properties, so it should not be an issue with it. But I will try, thanks.

there is a zuul.ignoredServices property which you can ignore the discovery client route

I've just checked. This is not an issue neither with application.properties, nor with bootstrap.properties. I was using custom info contributor for routes, that was adding routes the following way:
    @Override
    public void contribute(Info.Builder builder) {
        routeLocator.getRoutes().forEach(r -> builder.withDetail(r.getId(), r));
    }

Both routes have the same id, so the one from Eureka was overwriting custom configuration and I was not able to see it.
Still I wonder is that correct behavior to create absolutely new route if I want to override settings for already existing?
What I see now:
"Route 2": {
"id": "service-frontend",
"fullPath": "/CustomContextPath/**",
"path": "/CustomContextPath/**",
"location": "service-frontend",
"prefix": "",
"retryable": null,
"sensitiveHeaders": [],
"customSensitiveHeaders": false
},
"Route 5": {
"id": "service-frontend",
"fullPath": "/service-frontend/**",
"path": "/**",
"location": "service-frontend",
"prefix": "/service-frontend",
"retryable": null,
"sensitiveHeaders": [],
"customSensitiveHeaders": false
}

Shouldn't I have only one route?

Did you try zuul.ignoredServices?

Thanks, @spencergibb. I've resolved my initial issue (please see my previous comment) without using zuul.ignoredServices property.
I was wondering if this is a correct behaviour for zuul to create 2 routes when configuration for eureka route is set in application.properties?

I guess whether it is "correct" is open for debate.  I can certainly see an argument for augmenting an existing route if the ids match.

The key into the routes map is the only thing that provides uniqueness.

@spencergibb I guess I would need to look into it some more but it sounded like even though the ids for the routes were the same (from a config perspective) there was an additional route created.  Again I have not tested it but if that is the case I could see why it would be confusing.

Can you have many properties files with zuul route configuration ? E.g:

/resources/routes/a-category-service.properties
/resources/routes/b-category-service.properties
/resources/routes/c-category-service.properties

I've tried to load all these fields using @Bean PropertyPlaceholderConfigurer, but it does not work. It looks like zuul load configuration files at the very early stage.
Is there any way to change routing paths in runtime ? E.g

zuul.routes.a.path=/a/**
zuul.routes.a.url=http://host/service

change to:

zuul.routes.a.path=/a/**
zuul.routes.a.url=http://host/a-service


Global SSL rejectUnauthorized=false for all requests to software catalog

Expected Behavior
I have self signed certificates on my bitbuckt onpremis repo.
Current Behavior
I get the following error message if I want to connect to a bitbucket instance with self signed certificates like:
reason: unable to verify the first certificate at Module.generalError (webpack-internal:///../../node_modules/@backstage/plugin-catalog-backend/src/ingestion/processors/results.ts:59:56)
 at UrlReaderProcessor.readLocation (webpack-internal:///../../node_modules/@backstage/plugin-catalog-backend/src/ingestion/processors/UrlReaderProcessor.ts:75:67)
 at processTicksAndRejections (internal/process/task_queues.js:93:5)
at async LocationReaders.handleLocation (webpack-internal:///../../node_modules/@backstage/plugin-catalog-backend/src/ingestion/LocationReaders.ts:142:13)
 at async LocationReaders.read (webpack-internal:///../../node_modules/@backstage/plugin-catalog-backend/src/ingestion/LocationReaders.ts:74:11)
at async HigherOrderOperations.addLocation (webpack-internal:///../../node_modules/@backstage/plugin-catalog-backend/src/ingestion/HigherOrderOperations.ts:80:26)
at async eval (webpack-internal:///../../node_modules/@backstage/plugin-catalog-backend/src/service/router.ts:115:22)

Possible Solution
I want to disable the checks for ssl communication.

Does adding NODE_TLS_REJECT_UNAUTHORIZED=0 to the environment solve the issue?

thats right thank you!

Is there a way to add our own CA certificates to the trust store, so we can keep verifications on?

Is there a way to add our own CA certificates to the trust store, so we can keep verifications on?

Will this https://stackoverflow.com/questions/29283040/how-to-add-custom-certificate-authority-ca-to-nodejs help?

Is there a way to add our own CA certificates to the trust store, so we can keep verifications on?

Will this https://stackoverflow.com/questions/29283040/how-to-add-custom-certificate-authority-ca-to-nodejs help?

Looks promising. I'll pass it on internally here, thank you.


Adds french translation and split policies into 2 different languages

My goal is to add complete translation of the site. For now, in this PR i've just splitted the policies into 2 different files but we still need to implement a mechanism for the loading.
i thought it could as simple as adding a new domain, something like notrepolitique.ca, default would be to load the english version and if we are on the french domain, the french translation would loaded.
Please note that i've changed the order of the topics by reindenting the json files.

Also, i'm not translating the references or the details. Elections is arriving very quickly, my goal would be to have something minimal working :)

Oh wow this is fantastic! I would love to have launched with french content, but alas I only know English :) Thanks for this contribution. I'll see about getting a proper french version up ASAP.

Please note that i've changed the order of the topics by reindenting the json files.

Sorry, not sure I follow why the order of the topics was changed. Was it automated?

Basically i've used a json formater to reident and reformat the json file but i've discovered afterward that it change the ordering of the different topics. What do you think ? Do you want to revert back to the previous ordering ?

Yeah, I'd like to keep the original order if possible. Also I tried your branch and it worked well except that we'll need to get rid of the trailing commas on the french summary. The browser chokes on parsing that for some reason. If you could update the JSON with those commas gone and the original ordering, I can merge this now :)
Thanks again for your effort! I spent some time yesterday making the code a bit more language-agnostic so it plays nicer.

Ok i've the trailing commas error AND the ordering of en and fr data.
The french one will not loading currently cause there is no mechanism for loading the right translated party names but i'm planning to do it in another PR :)


Rotation Compression Changes

Codecov Report

Merging #163 into mirror will not change coverage.
The diff coverage is 0%.

@@          Coverage Diff           @@
##           mirror    #163   +/-   ##
======================================
  Coverage    6.55%   6.55%
======================================
  Files          29      29
  Lines        6198    6198
  Branches      866     866
======================================
  Hits          406     406
  Misses       5779    5779
  Partials       13      13

Impacted Files
Coverage Δ

Mirror/Runtime/NetworkTransform.cs
0% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Assigning nan to int-dtype array converts nan to int

(I am almost sure this already exists as an issue, but I can't find the original)
What happened:
When assigning nan to a integer-dtype array, the nan gets incorrectly inverted to int.
What you expected to happen:
I expect to get a ValueError, like I get for pure numpy arrays.
Minimal Complete Verifiable Example:
import xarray
import numpy
da = xarray.DataArray(numpy.array([0,1], dtype="u8"))
da[0] = numpy.nan
print(da)

Gives:
<xarray.DataArray (dim_0: 2)>
array([9223372036854775808,                   1], dtype=uint64)
Dimensions without coordinates: dim_0

Anything else we need to know?:
In numpy the equivalent code raises ValueError
This is related but different from #2945.  In #2945, xarray behaves the same as numpy.  In #4612, xarray behaves differently from numpy.

One oddity is that by supplying dtype="u8" I see the assignment having no effect:

In [18]: import xarray
    ...: import numpy
    ...: da = xarray.DataArray(numpy.array([0,1], dtype="u8"))
    ...: da[0] = numpy.nan
    ...: print(da)
<xarray.DataArray (dim_0: 2)>
array([0, 1], dtype=uint64)
Dimensions without coordinates: dim_0

I see the same result without the dtype arg.

Thanks, I agree this looks like a bug.
One oddity is that by supplying dtype="u8" I see the assignment having no effect:

In [18]: import xarray
    ...: import numpy
    ...: da = xarray.DataArray(numpy.array([0,1], dtype="u8"))
    ...: da[0] = numpy.nan
    ...: print(da)
<xarray.DataArray (dim_0: 2)>
array([0, 1], dtype=uint64)
Dimensions without coordinates: dim_0

I see the same result without the dtype arg.


"Project" tab under a repository doesn't show up, even under dropdown

The entire issue is in the title!
Let me know if there's some way I can help fix. Thanks!

It only shows up if you have a project. This is documented in the readme.

Since we have a drop down now I suggest stuffing it in there instead. We already decided that More is full of stuff we don't use anyway.

@bfred-it 👎 I don't see how that changes anything. We shouldn't use it as a junk drawer. "Projects" was removed for other reasons. The ones in the "More" are just less useful, not completely useless.

@sindresorhus I thought my use-case (I don't have a project and I want to add one) was useful. But this is your project, of course; I'll just disable it so I can make projects from my repo pages if I can't find the button.

You can create Projects from your repo's settings page:

After creating one, the Project tab will re-appear.

Thanks for the tip, @bfred-it ! I didn't realize you could do that.
I don't think this project is actually for me, regardless. I guess I prefer having a common UI with my collaborators over the improvements from the extension. Thanks though!

Hey @fluffywaffles, thanks for dropping by! The extension is meant for power users who already know their way around GitHub fairly well. But even that said, it's quite opinionated. Any feedback on non-intuitive things is always welcome!
BTW the add project button in the settings is added by the extension specifically because we are hiding the projects tab when there aren't projects. This way users don't need to guys the URL.

@sindresorhus makes sense. How about just putting the Add a project button in the dropdown then? It doesn't affect you, nobody else but admins will see it and those who can see it and are bothered by it can also disable the projects altogether.

How about just putting the Add a project button in the dropdown then? It doesn't affect you, nobody else but admins will see it and those who can see it and are bothered by it can also disable the projects altogether.

👎 from me.

It doesn't fit with the other buttons. This is an action, the other buttons are for going to root pages.
It will clutter the menu for no good reason. It's not like you add a project to repos daily. Only things you use very often should be in the main UI. The rest fits in the Settings page.
It's easy to say that users can just disable "Projects", but most won't bother to do that, and the point of this extension is to improve the "default" experience for power users.
This extension is for power users.

From #596

Maybe it would be worth adding a section in the quick setup page that mentions projects aren't visible cuz 0, and supply a new project button ¯(ツ)/¯

(altho I'm not sure how often the quick setup gets read by pro users)


[UIView setScene:]: unrecognized selector sent to instance 0x101906830

hi,

I meet the same error , can you tell me, what can i do ? thanks

Your controller view should be of type SCNView but it just an ordinary view. You can correct it by editing your storyboard/xib

can you show me the code ？ you mean i must init a SCNView then use the button to other SCNView. thanks.

Please check Main.storyboard


SAKIII-4875, SAKIII-5027, SAKIII-5040, SAKIII-5058, SAKIII-5070

SAKIII-4875 Fixes per code review in inserter and inserterbar widget
https://jira.sakaiproject.org/browse/SAKIII-4875
SAKIII-5027 Line height in htmlblock widget is too cramped
https://jira.sakaiproject.org/browse/SAKIII-5027
SAKIII-5040 Format and Font size dropdown have blue text
https://jira.sakaiproject.org/browse/SAKIII-5040
SAKIII-5058 Clean up pageviewer
https://jira.sakaiproject.org/browse/SAKIII-5058
SAKIII-5070 Inserter looks odd after resize
https://jira.sakaiproject.org/browse/SAKIII-5070

Old pull request with code review can be found at https://github.com/sakaiproject/3akai-ux/pull/1705