Debug production environment with Office 365 Authentication

Our Customer has a BC Onprem installation.
The NST and users authentication has been setup with AAD and Office 365.

But we're unable to connect to this server with VSCode for debugging.
Below you find the VSCode Output:
[2019-10-15 17:18:49.10] Using reference symbols cache path: c:\Users\<username>\Documents\AL\ALProject1\./.alpackages
[2019-10-15 17:18:49.10] Authenticating...
[2019-10-15 17:19:35.53] Authenticated as user<EMAIL_ADDRESS>Please note that these credentials are cached. Clear the credentials cache to authenticate as another user.
[2019-10-15 17:19:35.53] Targeting Dynamics 365 Business Central.
[2019-10-15 17:19:35.53] Sending request to https://api.businesscentral.dynamics.com/v1.0/sandbox/dev/packages?publisher=Microsoft&appName=Application&versionText=<IP_ADDRESS>
[2019-10-15 17:19:35.53] Sending request to https://api.businesscentral.dynamics.com/v1.0/sandbox/dev/packages?publisher=Microsoft&appName=System&versionText=<IP_ADDRESS>
[2019-10-15 17:19:35.72] The request for path /v1.0/sandbox/dev/packages?publisher=Microsoft&appName=Application&versionText=<IP_ADDRESS> failed with code Forbidden. Reason: Forbidden
[2019-10-15 17:19:36.91] The request for path /v1.0/sandbox/dev/packages?publisher=Microsoft&appName=System&versionText=<IP_ADDRESS> failed with code Forbidden. Reason: Forbidden
[2019-10-15 17:19:37.04] Could not download reference symbols. Please ensure that:
1. The correct server name and instance are specified in the launch.json file.
2. The correct application version is specified in the app.json file.
3. The dependencies are correctly specified in the app.json file.

Is there a method that allows debugging with Office 365 Account Authentication?
Kind regards,
Job Louage

Hi! Thank you for raising this issue. As mentioned in our online documentation https://docs.microsoft.com/en-us/dynamics365/business-central/dev-itpro/developer/devenv-json-files
authentication | Yes | Specifies the server authentication method and can be set to "UserPassword", "Windows", or "AAD". Currently, AAD authentication is supported only for Dynamics 365 Business Central sandboxes. AAD authentication cannot be used for on-premise servers.

Please post this to our Ideas forum at https://aka.ms/BusinessCentralideas, or vote up the idea if its already there. We're constantly monitoring top Ideas and will consider them for a future release.


[FancyZones] Picture-in-picture mode

Description of the new feature / enhancement
In modern browsers the "Picture-In-Picture" mode can be founded while watching videos.
The small video with the video appears in the bottom-right angle and stay always on top. (Please do NOT confuse it with #13 )
Since I have made an appropriate layout (but it is worth to be among the "Templates"), the only problem is that the small windows does not stay on top.
Suggestion: let the small window that inside bigger one to be always on top / or add the "always on top" flag for each layout element.
Scenario when this would be used?

You have a monitoring tool and want to see the measures in real time
You have a media source without "Picture-in-picture" or "Always on top" mode
The Calculator, Terminal and so on... are at hand and in front of the eyes

Supporting information
The evidence of the feature popularity

New Feature: Built in Picture in Picture mode (PIP) is added to Microsoft Edge Global Media Controls and the same news for the other browsers
Picture-in-picture Mode - Microsoft Edge Addons and the same addons for Edge and other browser.
Android: Picture-in-picture Support
and many other things about the mode.

this would be handled by https://github.com/microsoft/PowerToys/issues/2774 or https://github.com/microsoft/PowerToys/issues/279

I misunderstood this.  I understand what you're asking but don't fully understand the scenario of wanting to pin an app to the top of another app. Web browsers are self contained which gives them a lot of power.  Having two totally different apps working correctly in this scenario could be trickly.
Honestly, it also could be leveraged as a bad actor for one app acting like a security dialog too

To be crisp, i don't understand how #13 doesn't solve what you're asking for.

Having two totally different apps working correctly in this scenario could be trickly.
Honestly, it also could be leveraged as a bad actor for one app acting like a security dialog too

The current implementation of FancyZones also allows the parallel working and such a bad behavior. The only difference is the pinning ability between current FancyZones and PIP mode.

To be crisp, i don't understand how #13 doesn't solve what you're asking for.

They have common internal details and the solution for one of these issues boosts the solution of another but...
It is two different products: while Always On Top allows user to pin an arbitrary window, PIP mode can be implemented by small tweaking of the zones intersection behavior from FancyZones.

@nickveld, lets take a step back outside tech.  I'm trying to understand real world scenarios where this would be needed.  pictures are super helpful here.  In my head i have an idea of what is being asked but it may be totally off which is why i'm asking for scenarios.
For the security threat statement:
FZ can't force one app to stay on top of another application and when that bottom app moves, that app doesn't have anythign on top of it anymore.  This is no different than how Window snapping / placement has worked in Windows.  For Always on top, same scenario. A user can move the bottom window out.  However if 1 app is pinned to another, one app can fake a login dialog for instance or other things.

I suppose you asks me to give more details for the examples in the head description of the issue.

Scenario when this would be used?

You have a monitoring tool and want to see the measures in real time
You have a media source without "Picture-in-picture" or "Always on top" mode
The Calculator, Terminal and so on... are at hand and in front of the eyes

FZ can't force one app...

To my mind if the malware is powerful enough to pin itself then it is powerful enough to catch window focus state changes and keep itself focused / do not let change the focus / etc.

Sorry, what I'm not seeing is how making any app be able to be always in top (this includes multiple apps believe it or not) doesn't enable your scenarios below.
All three examples, setting always on top for any of those apps would allow you to use the IDE.  You still could snap any app in that scenario as well.

doesn't enable your scenarios below

As I said: "They have common internal details and the solution for one of these issues can boost the solution of another". In other words: "yes, Always On Top is an option", but...

Pinning every windows is not such comfortable as just clicking on a PIP mode button that places windows with a predefined layout and pinning it.
If Always On Top is not implementing then the PIP mode can be implemented with changes to intersection behavior of zones

Wait, is the ask here have a FZ zone(s) that forces an always on top bit on the snapped window?  When it is unsnapped, that bit goes is removed?
correct?

It is only one of the options and acceptable enough

@NickVeld i'm trying to be sure i understand the scenario you're asking for.  Based on that response, i don't think i still have a crisp understanding then.
if that isn't correct, please tell me

There are several options to implement the PIP mode. You have written one of these options correctly: indeed, you can implement the PIP mode assigning a "Always on top" bit to the zone that is located over another zone.

#919 is the tracking item for this request

Original header:
[FancyZones] Picture-in-picture mode
It is more concrete task than all related that is mentioned here. So I am leaving original header if the general issue will not be implemented. (And sadly, there is no progress on them.)

@crutkas It is too general and only one of the many  implementations, moreover without any progress almost 2 years.

Original header:
[FancyZones] Picture-in-picture mode
It is more concrete task than all related that is mentioned here. So I am leaving original header if the general issue will not be implemented. (And sadly, there is no progress on them.)

As a team, we prioritize work. if you feel you can execute this work or a variant of this before we can, we are happy to accept a pull request.

Yes, I understand it, so I am thinking about some sort of decomposition. Instead of planning for distant future a big general feature, small more concrete feature with small test cases can be implemented.
I wish I was an expert in all programming languages but I am more Python-developer but not C++ :(


import --help usage example doesn’t mention file arg

The Usage example that’s output when you run nitro import --help doesn’t mention that there’s a required file argument:
$ nitro import --help
Import database into a machine

Usage:
  nitro import [flags]

Flags:
  -h, --help   help for import

Global Flags:
  -d, --debug            Show command output and do not execute.
  -m, --machine string   Name of a machine.

But it’s clearly required:

$ nitro import
Error: requires at least 1 arg(s), only received 0

We can add examples to each command and it would output like so:
$ nitro import -h
Import database into a machine

Usage:
  nitro import [flags]

Examples:
import my-backup.sql

Flags:
  -h, --help   help for import

Global Flags:
  -d, --debug            Show command output and do not execute.
  -m, --machine string   Name of a machine.

We should definitely update the usage on the command, but we would need to alter quite a bit of the commands to create a custom template. There is also a 1.0.0 release to the package we are using that has some nice upgrades. I'm going to look into that effort and see if they have made it easier to define usage.

This will be released in 1.0.0-beta.3!


Branch error handling update v1.4

This PR makes the following changes/fixes to the main repo:

Changed ...2359 format for time to Time should be in 24hr format. Input time should be from 0000 to 2359 inclusively.
Generalised error messages for index to use MESSAGE_INVALID_INDEX in ParseUtil.java
Updated error messages for enroll/expel error messages to mention both participants name and event name
Add more layers of checks in ParserUtil::parseBirthDate to check for formatting, if the date exists and if it is valid (i.e. not in the future) with different error messages for each case.
Add 1 more layer of checks in ParserUtil::parseEventDate to check for formatting with a different error message to be displayed

Other miscellaneous changes:

Update MESSAGE_USAGEs to remove instances of "address book" and replace with "managera"
Update MESSAGE_CONSTRAINTS of numerous classes
Update test cases

Codecov Report

Merging #210 (5d7420a) into master (cb4501e) will increase coverage by 0.10%.
The diff coverage is 62.85%.

@@             Coverage Diff              @@
##             master     #210      +/-   ##
============================================
+ Coverage     76.22%   76.33%   +0.10%
- Complexity      796      801       +5
============================================
  Files           115      115
  Lines          2246     2252       +6
  Branches        259      262       +3
============================================
+ Hits           1712     1719       +7
+ Misses          460      456       -4
- Partials         74       77       +3

Impacted Files
Coverage Δ

...du/address/logic/commands/AddNextOfKinCommand.java
100.00% <ø> (ø)

...va/seedu/address/logic/commands/DeleteCommand.java
100.00% <ø> (ø)

...edu/address/logic/commands/DeleteEventCommand.java
100.00% <ø> (ø)

...address/logic/commands/DeleteNextOfKinCommand.java
100.00% <ø> (ø)

...java/seedu/address/logic/commands/DoneCommand.java
100.00% <ø> (ø)

...java/seedu/address/logic/commands/EditCommand.java
87.34% <ø> (ø)

...seedu/address/logic/commands/EditEventCommand.java
89.09% <ø> (ø)

...ic/commands/RemoveParticipantFromEventCommand.java
87.50% <0.00%> (-3.81%)
:arrow_down:

...ddress/logic/commands/ShowEventDetailsCommand.java
91.66% <ø> (ø)

...s/logic/commands/ShowEventParticipantsCommand.java
95.23% <ø> (ø)

... and 14 more

Continue to review full report at Codecov.

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


Add the missing nodeSelectors for windows

Current config would only work in tainted windows nodes environment. Add nodeSelector to all subcharts to force them on linux machines

Ref: #51


tryCatch impl (rxjs parity)

Logically equivalent semantic change in the implementation of tryCatch for parity with the rxjs tryCatch https://github.com/gcanti/fp-ts-rxjs/pull/48 as per @OliverJAsh suggestion https://github.com/gcanti/fp-ts-rxjs/issues/31#issuecomment-760982846
Semantically, promise.then(a).catch(e) is closer to observable.pipe(catchError(e)) than promise.then(a, e). This should have no bearing on tests or behavior, since E.right will never throw an error.
Difference btw/n then's onRejected and catch documented here, though they are logically equivalent in this case.

It goes without saying, but I'd be perfectly willing to close this PR. I just wanted to be sure to follow up on the discussion we had here

It goes without saying, but I'd be perfectly willing to close this PR. I just wanted to be sure to follow up on the discussion we had here

Closing


Account for route-level host manipulation in auto-sni/auto-san-validation

Commit Message: Account for route-level host manipulation in auto-sni/auto-san
Additional Description:
Auto-SNI will now account for manipulation performed at route level when setting the SNI value.
Before this changes it always relied on the original host value ignoring all host header mutations that takes place at route-level.
A runtime flag has been added as part of the deprecation process
Risk Level: Medium
Testing: new integration tests
Docs Changes: None
Release Notes: Done
Runtime guard: envoy.reloadable_features.use_route_host_mutation_for_auto_sni_san
Fixes #35330

PR description still has it marked as draft - this ready for a review pass?
/wait-any

PR description still has it marked as draft - this ready for a review pass? /wait-any

Oops sorry I forgot to remove that. Yes the PR is ready for review

@ggreenway this was passing before a change-and-revert so is theoretically green and ready for a look
/retest in case that un-wedges CI.  Otherwise it'll need a main merge


Make derivation macros work for generic case classes

returnType doesn't work appropriately if the member type is generic.

I fracking hate sbt.
something in the build is somehow now wrong. The 2.12 build is looking for a non-existent 2.11 version of algebird.
I would love to get bids from people to see what I would have to pay to never have to worry about sbt +++ vs sbt ++ and get sane cross building support. Also, while this sbt expert is here, can they look at an scalajs build too.

going to merge assuming #648 fixes the build failure.
Thanks for fixing this!

this is published in 0.13.3: https://github.com/twitter/algebird/releases/tag/v0.13.3


Upgrade TS to 2.x

We will do this but can't evaluate till sometime next week.  Churning the task repo this week ...

Let's hold for 108.  I want to make sure 107 stabilizes and we consciously move to 2.0.  I will send communications to teams inline with Trevor's mail

since we started a 2.x preview task lib, we can consider taking this.  we have to carefully look at all the 2.0 implications but it's a good time.  The doc gen might also be impacted since we load the ts compiler for that.

This has been done in a separate PR. Closing.


Add option to treat docblocks as pre-formatted reST

javasphinx-apidoc attempts to mimic the Javadoc tool by converting docblocks from HTML + Javadoc tags into reST. Any pre-formatted reST placed into docblocks will be mangled by this process. An option to javasphinx-apidoc to treat all docblocks as pre-formatted would provide an option for people who would prefer to mark up their docblocks in reST.

I also think an option to javasphinx-apidoc should be enough.


Add ethnic group pattern

What is the context of this PR?
Repeat of PR #1621 which had to be reverted for more people to review the content due to the subjects sensitivity.
How to review

Pattern looks as it did in Census 2021 and documentation makes sense
Content is correct and the right people have signed it off

 Ethnic group [Ask users for...]

 Ethnic group [Ask users for...]

Discussions link url is wrong (it's linking to feedback discussion)
Browser metadata
Path:      /patterns/ethnic-group/
Browser:   Chrome 92.0.4515.131 on Mac OS 10.14.6
Viewport:  1920 x 978 @1x
Language:  en
Cookies:   Enabled

Open in BrowserStack

Open Deploy Preview · Mark as Resolved

Buttons on questions usually have a top margin class of u-mt-xl
Browser metadata
Path:      /patterns/ethnic-group/
Browser:   Chrome 92.0.4515.131 on Mac OS 10.14.6
Viewport:  1920 x 978 @1x
Language:  en
Cookies:   Enabled

Open in BrowserStack

Open Deploy Preview · Mark as Resolved

Typo:

show an error message if they have not selected a required radio button or entered something in when it is require for “Other ethnic group”

Browser metadata
Path:      /patterns/ethnic-group/
Browser:   Chrome 92.0.4515.131 on Mac OS 10.14.6
Viewport:  1920 x 978 @1x
Language:  en
Cookies:   Enabled

Open in BrowserStack

Open Deploy Preview · Mark as Resolved

Should this have a colon @LauraC119?

I think we should probably show examples of all the second and third level questions so users know exactly what each question and all answers should be (laura will discuss this with you)

Discussions link url is wrong (it's linking to feedback discussion)
Browser metadata
Path:      /patterns/ethnic-group/
Browser:   Chrome 92.0.4515.131 on Mac OS 10.14.6
Viewport:  1920 x 978 @1x
Language:  en
Cookies:   Enabled

Open in BrowserStack
Open Deploy Preview · Mark as Resolved

Oh yeah forgot about that, I was going to create a discussion when it was signed off, for now Ill set the link to "#"

No longer publishing pattern


UI 2.0: How should we serve the UI & API seamlessly?

/api, /ui, etc
Going to need to keep legacy UI views servable
Easy upgrading/migrating existing views that can easily adopt new UI

Per https://github.com/nautobot/nautobot/pull/3285#discussion_r1103060850, current plan is to group all UI-related API URLs under /api/ui/, e.g. /api/ui/get-menu/, /api/ui/filterset-fields/lookup-choices/ etc.

Per #3285 (comment), current plan is to group all UI-related API URLs under /api/ui/, e.g. /api/ui/get-menu/, /api/ui/filterset-fields/lookup-choices/ etc.

Can we ensure we don't needlessly spend time publishing/maintaining serializers or swagger docs for our UI? They are not needed and can and should break as they will be tightly coupled with our UI and are not meant for integrating externally.

It's absolutely possible to exclude endpoints from the schema, and I agree that we should do so for the API-UI endpoints.

It's absolutely possible to exclude endpoints from the schema, and I agree that we should do so for the API-UI endpoints.

Further, the UI API endpoints should be considered "private" despite being public, so that we don't have to support them for anything other than what we need until such time we feel it is sufficiently stable to say otherwise. This give us license to say "sorry, please use the primitives, calling the UI API endpoints directly is not supported".

Closed by #3385


Swift: missing array initializer for CCActionSequence, CCActionSpawn

There is no Swift equivalent for this code because the array initializer is missing from Swift:
id sequence = [CCActionSequence actionWithArray:@[move, wait, spawn, block, remove]];

The sequence (or spawn) action needs to be constructed via repeat use of the one:two: initializer:
let sequenceActions = [move, wait, spawn, block, remove]
var sequence = sequenceActions[0]
for var i = 1; i < sequenceActions.count; i++
{
    sequence = CCActionSequence(one: sequence, two: sequenceActions[i])
}

It wasn't a cocos2d problem, everything seems to work fine with latest version of Swift.


fix(RadialMenuController): Don't change the axis if the touchpad isn'…

…t being touched
SteamVR seems to sent a TouchpadAxisChanged event when the touchpad
is released. This is causing an OnOverEnter event to fire out of the
RadialMenu after the OnHoverExit event is fired. Because of this,
you'll always get an OnHoverEnter event at the end of picking your
finger up off of the touchpad.
Keep track of if the touchpad is currently being touched or not. Only
trigger the axis change if it is during an active touch.

There were the following issues with your Pull Request

Commit: 99aae3a364d5d1113613abca7f8c48043a818f51
Your subject line is longer than 72 characters

Guidelines are available at https://github.com/thestonefox/SteamVR_Unity_Toolkit

This message was auto-generated by https://gitcop.com

Updated to fix the GitCop error.
This fixes issue: https://github.com/thestonefox/SteamVR_Unity_Toolkit/issues/609

cool :) thanks for the fix ❤ 💚 💛 💙 💜

Ooh one more thing....
can you lowercase the commit subject line (using git commit --amend then force pushing so you don't get 2 commits)
so it reads:
fix(RadialMenu): only change the axis if the touchpad is being touched

Commit message updated.


Etcd load-balancer on control-plane-only server may select removed etcd cluster member as default server

K3s tracking issue for:

https://github.com/rancher/rke2/issues/5646

Validated on Version:
-$ k3s version v1.29.3+k3s-06b64449 (06b64449)

Environment Details
Infrastructure
Cloud EC2 instance
Node(s) CPU architecture, OS, and Version:
Ubuntu
AMD64
Cluster Configuration:
Split role:

3 cp only
3 workers
3 etcd only

Steps to validate the fix

start k3s on split role
get logs on cp only
Validate that the address of the server specified in the --server address is always reported as the server address in the etcd load-balancer log message

Reproduction Issue:

Validation Results:

$ cat /etc/rancher/k3s/config.yaml
write-kubeconfig-mode: "0644"
tls-san:
  - <IP_ADDRESS>
server: https://<IP_ADDRESS>:6443

ip-172-31-14-164.us-east-2.compute.internal   Ready    etcd                   53m   v1.29.3+k3s-06b64449   <IP_ADDRESS>   <IP_ADDRESS>

 :~$ sudo journalctl --no-pager -u k3s | grep "Adding" -A2
Apr 16 22:47:24 ip-172-31-1-64 k3s[2049]: time="2024-04-16T22:47:24Z" level=info msg="Adding server to load balancer k3s-etcd-server-load-balancer: <IP_ADDRESS>:2379"

Apr 16 22:47:24 ip-172-31-1-64 k3s[2049]: time="2024-04-16T22:47:24Z" level=info msg="Running load balancer k3s-etcd-server-load-balancer <IP_ADDRESS>:2379 -> [<IP_ADDRESS>:2379] [default: <IP_ADDRESS>:2379]"


machine parts + power cell resprite

About the PR

Old parts + cell + recharger were shiny TG sprites to the max. The old vg ones were damn good and the cells just needed some workshopping.
also fixes a logic error in recharger visualizer that caused some stuff to not display properly.
Media

parts are from vg and the cells are adaptations based off of some stuff flareguy did and i cooked into a stew.

new panel sprites, too

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

no cl no fun

pls.. the matter bin just fix it gahh

aren't we really close to just removing machine parts? I thought the lathes / heaters were kind of the final step in that. fine with changing them but it feels a little unnecessary if I'm correct on that timeline

aren't we really close to just removing machine parts? I thought the lathes / heaters were kind of the final step in that. fine with changing them but it feels a little unnecessary if I'm correct on that timeline

the original T1 parts are gonna remain just for construction/machine stuff. They're not functional beyond little doodads but i think that's a fine place to leave em.

sovl matterbin = merge

return to monke parts
tbh the new (old? v3) parts are a bit hard to look at compared to tg parts (v2), especially the matter bin being skinny as heck

cell charger looks way too matte to me imo

maintainers are brainwashed
these are fugly
actually the powercell is kinda cool

maintainers are brainwashed
these are fugly
actually the powercell is kinda cool

If you are just gonna leave garbage spam then do us all a favor and go away.

How did you even get hired into the development team?🤣
The sprites are just terrible

How did you even get hired into the development team?🤣 The sprites are just terrible

because there is no development team and every single person who works on this project is an unpaid volunteer doing every last bit of it in their free time.

both TG's team and WizDen's team, but yeah these sprite looks so depressing for lack of better terms, fitting the capitalist dystopia era that is SS13/14!

How did you even get hired into the development team?🤣 The sprites are just terrible

because there is no development team and every single person who works on this project is an unpaid volunteer doing every last bit of it in their free time.

let him better not do this, he will kill the game with his useless PRs

How did you even get hired into the development team?🤣 The sprites are just terrible

because there is no development team and every single person who works on this project is an unpaid volunteer doing every last bit of it in their free time.

let him better not do this, he will kill the game with his useless PRs

i gotta remind that that Emo brought what is currently Science and Cargo salvage content(that is getting slightly reworked by Emisse) he has been putting some of the most interesting job content the game saw in a good while

that said i think these are pretty bad sprites probably would have been better to just de-eris the current ones


Add text-regression and text-classification tasks

Similar to #100, but we should also make sure the config file and the README contain the right components for the widget to work.
This might also involve adding a text-classification alias on the api-inference-community side.

Once those are done, we should an entry for sklearn here.

That list seems quite misleading, as in, it doesn't seem to show what tasks are supported by each library, so we could already add ours there. But I'm in no rush.

Yeah, no rush, but it seems that with those tasks finished, we have covered the most important ones for sklearn.

Renamed since we now support text-classification.


[HOLD] Blazor WASM debug topic updates for 3.2p3

Fixes #17192
Internal Review Topic

@danroth27 This is updated now from your content.

@danroth27 Updates made.

We don't have approval for a tab (i.e., "Debug in the browser) to go along with the VS and VSC tabs, so I split all three out into sections.

I changed the title to include "WebAssembly."

For ...

Run the app with debugging support in Visual Studio and Visual Studio Code (F5 support).
Single-step (F10) through the code.
Resume code execution in a browser with F8 or F5 in Visual Studio or Visual Studio Code.

... does it need a bit more work?

@mkArtakMSFT See :eyes: how this looks :point_right: https://github.com/dotnet/AspNetCore.Docs/pull/17438/commits/ac0ceddec9f4d3f9f117ccbbcad24bd98a720e60

I'll patch it if it needs more work. Going ahead with the merge now, and I'll take this batch live immediately.


Avoid clearing objects in CiliumEndpoint conversion funcs

This removes the behavior of mutating the objects received from the client-go library. To begin with there isn't really any benefit from doing so, given we don't store the object afterwards, and it will be ready for gc when it leaves the scope inside client-go. client-go can possibly return the same pointer twice here, to trigger eg. both an object update delta and then a DeletedFinalStateUnknown delta with the same pointer.
For more info, see the issue 115658 in the kubernetes/kubernetes repo on github.
Follow-up of: 74307f175ceb ("Avoid clearing objects in conversion funcs")

/test

Also landed in v1.11.


Set hostNetwork: true in node-exporter ds

This commit configures node-exporter pods in the node-exporter daemonset to be set to the host network, allowing for the host's network metrics te be recorded rather than the node-exporter pod's network metrics to be recorded.
For more information: https://stackoverflow.com/questions/62606739/network-transfer-speeds-in-mb-s-using-prometheus-grafana
Signed-off-by: Ryan Drew<EMAIL_ADDRESS>

Simple PR, self-high-fiving :hand:


mixin: Add namespace to the alerts

Signed-off-by: Kemal Akkoyun<EMAIL_ADDRESS>
[x] I added CHANGELOG entry for this change.
[ ] Change is not relevant to the end user.

Changes

Add namespace to the alerting rules and annotations

Verification

make examples
make example-rules-lint

Merge after: https://github.com/thanos-io/thanos/pull/3854

Friendly ping @metalmatze @bwplotka

Didn't we just work on getting rid of kubernetes specific things?

Yes, definitely @brancz. I'll be introducing further changes to provide a more abstract solution for this https://github.com/thanos-io/thanos/issues/3867. It'll require more consideration. This would solve our problems in the short-term.

Ok, I'm convinced that we need to generalize this. Marking it WIP and jumping to fix these.

@bwplotka It's ready for another round. I've added missing aggregators.


try to simplify the graphics api (#63)

@jbmouret I took the liberty to improve a bit the representation and speed of images and conversions. Now it is much faster and cleaner. I also remove duplicate code from CameraOSR. For the BaseApplication, I think we should leave it as is because it'll be too much work to change it now: I should do it at some point but another time. For now, I think it only remains to remove the inconsistencies between returning pointers and references.

Merging..


Enhancement: Vehicle launch list.

The ability to click on a launch vehicle or orbiter and be brought to a "Launches" page (with upcoming and previous) for just that vehicle.
Source

Tracking on feature tracker.


Add static check bot

What is the problem you're trying to solve
I found some static check errors in code.
$ staticcheck ./...
cmd/nerdctl/apparmor_inspect_linux.go:44:11: printf-style function with dynamic format string and no further arguments should use print-style function instead (SA1006)
cmd/nerdctl/exec.go:257:2: should replace loop with pspec.Env = append(pspec.Env, strutil.DedupeStrSlice(env)...) (S1011)
cmd/nerdctl/info.go:234:34: strings.Title has been deprecated since Go 1.18 and an alternative has been available since Go 1.0: The rule Title uses for word boundaries does not handle Unicode punctuation properly. Use golang.org/x/text/cases instead.  (SA1019)
cmd/nerdctl/ipfs_linux_test.go:146:2: redundant return statement (S1023)
cmd/nerdctl/ipfs_registry_up.go:121:74: should use String() instead of fmt.Sprintf (S1025)
cmd/nerdctl/kill.go:58:17: containerd.ParseSignal is deprecated: Use github.com/moby/sys/signal instead.  (SA1019)
cmd/nerdctl/main_unix.go:47:2: should replace loop with candidates = append(candidates, nsList...) (S1011)
cmd/nerdctl/main_unix.go:68:2: should replace loop with candidates = append(candidates, snapshotterPlugins...) (S1011)
cmd/nerdctl/run.go:55:2: package "github.com/docker/cli/opts" is being imported more than once (ST1019)
        cmd/nerdctl/run.go:56:2: other import of "github.com/docker/cli/opts"
cmd/nerdctl/run.go:1071:3: should check returned error before deferring f.Close() (SA5001)
cmd/nerdctl/run_linux_test.go:120:11: should use fmt.Errorf(...) instead of errors.New(fmt.Sprintf(...)) (S1028)
cmd/nerdctl/run_network.go:125:6: should omit nil check; len() for []string is defined as zero (S1009)
cmd/nerdctl/run_network_linux_test.go:194:23: should use raw string (`...`) with regexp.MustCompile to avoid having to escape twice (S1007)
cmd/nerdctl/stats.go:317:5: ineffective break statement. Did you mean to break out of the outer loop? (SA4011)
cmd/nerdctl/stop.go:141:18: containerd.ParseSignal is deprecated: Use github.com/moby/sys/signal instead.  (SA1019)
cmd/nerdctl/stop.go:146:18: containerd.ParseSignal is deprecated: Use github.com/moby/sys/signal instead.  (SA1019)
cmd/nerdctl/stop.go:179:17: containerd.ParseSignal is deprecated: Use github.com/moby/sys/signal instead.  (SA1019)
cmd/nerdctl/top.go:148:8: should use raw string (`...`) with regexp.MustCompile to avoid having to escape twice (S1007)
pkg/formatter/formatter.go:49:11: strings.Title has been deprecated since Go 1.18 and an alternative has been available since Go 1.0: The rule Title uses for word boundaries does not handle Unicode punctuation properly. Use golang.org/x/text/cases instead.  (SA1019)
pkg/formatter/formatter.go:51:10: strings.Title has been deprecated since Go 1.18 and an alternative has been available since Go 1.0: The rule Title uses for word boundaries does not handle Unicode punctuation properly. Use golang.org/x/text/cases instead.  (SA1019)
pkg/formatter/formatter.go:56:10: strings.Title has been deprecated since Go 1.18 and an alternative has been available since Go 1.0: The rule Title uses for word boundaries does not handle Unicode punctuation properly. Use golang.org/x/text/cases instead.  (SA1019)
pkg/formatter/formatter.go:60:10: strings.Title has been deprecated since Go 1.18 and an alternative has been available since Go 1.0: The rule Title uses for word boundaries does not handle Unicode punctuation properly. Use golang.org/x/text/cases instead.  (SA1019)
pkg/formatter/formatter.go:72:10: strings.Title has been deprecated since Go 1.18 and an alternative has been available since Go 1.0: The rule Title uses for word boundaries does not handle Unicode punctuation properly. Use golang.org/x/text/cases instead.  (SA1019)
pkg/imgutil/pull/pull.go:76:3: containerd.WithSchema1Conversion is deprecated: use Schema 2 or OCI images.  (SA1019)
pkg/ipfs/registry.go:91:2: redundant return statement (S1023)
pkg/mountutil/mountutil.go:129:5: ineffective break statement. Did you mean to break out of the outer loop? (SA4011)
pkg/mountutil/mountutil.go:129:5: redundant break statement (S1023)
pkg/portutil/port_allocate_linux.go:89:15: this result of append is never used, except maybe in other appends (SA4010)
pkg/statsutil/stats.go:163:10: unnecessary use of fmt.Sprintf (S1039)
pkg/statsutil/stats.go:170:10: unnecessary use of fmt.Sprintf (S1039)
pkg/statsutil/stats.go:177:10: unnecessary use of fmt.Sprintf (S1039)
pkg/statsutil/stats.go:184:10: unnecessary use of fmt.Sprintf (S1039)
pkg/statsutil/stats.go:191:10: unnecessary use of fmt.Sprintf (S1039)
pkg/statsutil/stats.go:198:10: unnecessary use of fmt.Sprintf (S1039)

Describe the solution you'd like
We don't have much code at the moment and I think we should enable staticcheck bots now to prevent more problems like this.
Additional context
No response

Can this be assigned to me?
Thanks


[Idea]: Apriori Algorithm

Idea Title
Apriori Algorithm
Idea Description
The Apriori algorithm is a classic algorithm used for mining frequent itemsets and generating association rules in transactional databases. It is particularly useful in market basket analysis, where it helps identify sets of products that frequently co-occur in transactions. The algorithm operates on the principle of "if-then" rules, providing insights into customer purchasing behavior.
@ajay-dhangar kindly assign me this issue along with the labels
Potential Benefits

[X] Improves user experience
[X] Enhances learning
[X] Adds a new feature
[X] Improves code quality
[X] Other

Implementation Suggestions (Optional)
No response

@ajay-dhangar kindly assign me this isuue


fix(config): fix Github Actions config

:tada: This PR is included in version 1.0.1-next.1 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


recipes-test: Add new utilities to 'initramfs-test-image.bb'

Add following utilities to 'initramfs-test-image.bb':
-> For kexec/kdump analysis:

crash
makedumpfile
kexec
-> iozone3: for filesystem performance analysis.
-> cryptsetup: for crypto benchmarks.
-> util-linux: for 'gdisk', 'taskset', etc.
-> coreutils: mainly for 'dd' (as busybox 'dd' does not show performance
figures).

Addition of the above tools results in an approximate increase of
6M in the resulting rootfs size (cpio.gz archive)
Before:
$ ls -lah initramfs-test-image-qcom-armv8a-20210521045425.rootfs.cpio.gz
19M
After:
$ ls -lah initramfs-test-image-qcom-armv8a-20210521070020.rootfs.cpio.gz
25M
Signed-off-by: Bhupesh Sharma<EMAIL_ADDRESS>

we've discussed with @bhupesh-sharma , and agreed to introduce initramfs-test-full-image with 'more' utilities. So that we have:

initramfs-tiny
initramfs-test
initramfs-test-full
Where we incrementally add more packages, so that users can pick the appropriate image they need.

I am going to merge as is, since we really need it merged now. we will do further improvements later (ncurses and some cleanup such as building on top of initramfs-test instead of tiny).


BUG: Not all named parameters have two dashes

Working on a pack for fpm, which takes named arguments. (https://github.com/jordansissel/fpm/wiki)
Not all named parameters have two dashes, as assumed in the current documentation. For example, some of the condensed attributes (like -s or -t or even -n) have only a single dash, yet there does not appear to be a place for me to adjust how parameters are being passed.

It looks like documentation as well as examples of how to use this feature have been created and this issue can be closed - https://docs.stackstorm.com/actions.html?highlight=two dashes.


Should not focus window below cursor when focus mode is disabled

When cleaning up it looks like you have mistakenly removed the code that made sure osx focus was used when focus mode is disabled. (it worked very well, so thank you for implementing that :) )
I think line 496 and 512 of window.cpp should have && KwmFocusMode != FocusModeDisabled added to the if statements.

line 496 is a fix that has to be there for multiple monitor support I believe, 512 can be changed however.
The 496 should not impact how focus works when cycling through screens with winidow -f prev|next and config cycle-focus all is set.

Ok. I am not currently using the multi monitor support, so I did not test that. It just looked like it should be there to make sure it is the previously focused window that gets focus, and not the one with the mouse above it.
Sounds good with a fix for the space change thing.


Figure is small on phone screens

Considering making a two-row version of the interval plot to make the viz more readable on a small screen (phone).

Current two-row version looks like this:

Things to improve:

should the scales for the x-axis be repeated or not?
intervals could be bigger, lots of white space
mean should be bigger so that it shows up at this reduced scale


Misleading error "Unable to find Algorithm in CanonicalizationMethod element"

I'm currently working on some rather old systems and was getting the "Unable to find Algorithm in CanonicalizationMethod element" error while signing an XML document. This error implies that signedxml is not able to find the element's attribute, yet this is also triggered by unsupported algorithms.
In the Sing method, if the check for the canonAlgoURI is successful, but the URI is not present in the CanonicalizationAlgorithms map, the error will default to the said error message.
The error was triggered by trying to use the now outdated ExclusiveCanonicalization 1.0 as the attribute for SignedInfo.CanonicalizationMethod (http://www.w3.org/TR/2001/REC-xml-c14n-20010315).

CanonicalizationMethod
http://www.w3.org/TR/2001/REC-xml-c14n-20010315

@CamiloHernandez Can you use this algorithm？
http://www.w3.org/TR/2001/REC-xml-c14n-20010315


detect risc-v vector and xtheadvector

https://github.com/nihui/ruapu/pull/107

https://github.com/nihui/ruapu/pull/134


Trello responds with "Invalid return_url" when trying to authorize token

Question
Hi, i wanted to know if i'm doing something wrong. When i execute the app, trello responds with "Invalid return_url"
Relevant information
These are the steps i've taken:

Cloned the repo
Executed yarn command
Went to https://trello.com/app-key and copied the key
Pasted it into the config.js file along with the rest of the information.
Went to allowed origins and put localhost:2222
Went to localhost:2222, this redirects me to https://trello.com/1/authorize?...
Received a message from trello telling me "Invalid return_url"

Your Environment

Browser: Chrome
Browser version: 78
OS: Windows 10 x64
App Version: 0.2.1

Hi @goljavi,
first of all: I am sorry, must have missed the notification.
Now back to your problem: does it still exist? Have you tried it with localhost instead of localhost:2222 as "allowed origin" on trello?

Thank you for your answer!
Yes, the problem still exists. I have tried with localhost and localhost:2222

Hi @goljavi,
finally I had time to test it by myself. It still works for me. Not sure if the port is required, but these are my "Current Origins" settings:

Can you verify that this works?

Hey Stefan!
I was setting an https:// origin when it was supposed to be http://
It was a Trello bug, because it automatically appendend https:// even if you wrote an url beggining with http://
Now it is fixed and everything works fine!
Thanks!

Perfect! Happy to hear it works for you now @goljavi.

Hi! I'm working locally in ubuntu, using firefox. I'm running a simple .html file with client.js. What should I put on the origin field?


Fix the style and position of comment after replying

should fix #325

@adcpm thanks please try again


FS-1581: Add templates for Assessor Task List page

Summary:

Add templates outlined by jira story FS-1581
Consumes mapping configuration from:

https://github.com/communitiesuk/funding-service-design-assessment-store/pull/45

Add test for new scored/unscored macro components
There's some missing data from the eventual "Assessor Task List", for example scores & actual statuses, those parts are dependent on in-flight work so can't be done atm

FS-1581

Just on the fourth point, the config will be consumed from the assessment store

FS-1581

Just on the fourth point, the config will be consumed from the assessment store

Thanks, have updated the description to reflect that just now


#1725 Cannot read property 'fullName' of undefined

Description
Added catching errors and logging input data. To further understand which card is causing the problem
About #1725
Checklist:

[x] PR title matches follow: (Feature|Bug|Chore) Task Name
[x] My code follows the style guidelines of this project
[x] I have followed all the instructions described in the initial task (check Definitions of Done)
[x] I have performed a self-review of my own code
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] I have added reference to a related issue in the repository
[x] I have added a detailed description of the changes proposed in the pull request. I am as descriptive as possible, assisting reviewers as much as possible.
[ ] I have added screenshots related to my pull request ( for frontend tasks)
[ ] I have pasted a gif showing the feature.
[ ] @mentions of the person or team responsible for reviewing proposed changes

pls fix build


nvidia-kernel-oot: fix a wrong variable

Commit 9fa8701606dffc7153e7593f817e8ef8f56d335e:
[ nvidia-kernel-oot: fix a build issue ]
intended to pass ${STAGING_KERNEL_BUILDDIR} to make but mistakenly passed ${STAGING_KERNEL_DIR} which still referring to kernel source but not kernel builddir, fix that.

Hi, @madisongh, my previous MR has a wrong reference to a variable, it did not really fix the problem, fixed in this MR.


take_from_bytes_cobs not consuming trailing null bytes inserted by to_slice_cobs

take_from_bytes_cobs returns two things on success. The de-serialized structure and any leftover buffer. When de-serializing using take_from_bytes_cobs it would make sense for the trailing null byte to be consumed by the de-serialization since it is somewhat part of the data. It also is unintuitive that the user will end up with a buffer with a single null byte after de-serializing cobs data.
My specific use case is as follows:
Calling postcard::to_slice_cobs on my data returns [2, 1, 0] and calling postcard::take_from_bytes_cobs returns (<my data>, &[0])

Hey @DusterTheFirst, thanks for reporting this! I managed to track down the issue. take_from_bytes_cobs was returning the number of DECODED bytes, not the number of SOURCE BYTES used during decoding.
This was... very wrong, and not a useful number.
Thanks again!


LM4VisualEncoding

https://github.com/ziqipang/LM4VisualEncoding what is the difference

https://github.com/ziqipang/LM4VisualEncoding what is the difference

Thanks for the attention! LM4VisualEncoding alongside the paper Frozen Transformers in Language Models are Effective Visual Encoder Layers have been pivotal in shaping our approach. Drawing upon their insights, we adapted and expanded upon the 2D image classification framework to accommodate both 2D and 3D medical imaging data, experimenting with various configurations of residual blocks to enhance the model's performance on the medical image dataset. This repo mainly focuses on medical image applications.

Thanks for the attention! LM4VisualEncoding alongside the paper Frozen Transformers in Language Models are Effective Visual Encoder Layers have been pivotal in shaping our approach. Drawing upon their insights, we adapted and expanded upon the 2D image classification framework to accommodate both 2D and 3D medical imaging data, experimenting with various configurations of residual blocks to enhance the model's performance on the medical image dataset. This repo mainly focuses on medical image applications.


u_map_zoom?

Is it a thing? Can't seem to find/use it.
https://github.com/tangrams/tangram-docs/blob/gh-pages/pages/shaders.md#built-in-uniforms

We have it as 'u_zoom' set in style.cpp

It's u_map_position.z (.xy are the map center in meters).

https://github.com/tangrams/tangram-docs/commit/537db4f4e12f2546a20107f54f5ca9efff8bce47


HBox widget does not show with versions >6.0.0.beta3

I tried using the newest versions and noticed that in addition to a new error (the widgets show nevertheless)
The installed widget Javascript is the wrong version. It must satisfy the semver range ^2.0.9.
all versions after 6.0.0.beta3 do no show the simple HBox widget
import ipywidgets as widgets
items = [widgets.Label(str(i)) for i in range(4)]
widgets.HBox(items)

I think that you have a conflicting other version of the javascript hanging around in one of the jupyter directories.

Closing since I cannot reproduce in either jupyterlab or notebook.

Thanks for your answers! What can I try to fix this?

I recommend that you check out the data paths listed in in the output of jupyter --paths if you have an older version of the javascript hanging around.

In notebook 4.3.1, you can start the server with --debug and it will print out the path on disk it is serving any javascript files from. That helps tremendously in tracking down problems like this where you may have multiple installations of the javascript.
jupyter notebook --debug
Then go to a notebook page that loads the widget javascript and look in the server logs for where it is serving the widget javascript from.

Look for the line that says:
Path jupyter-js-widgets/extension.js served from ...
and see if that path is what you are expecting.


Courses seconds spent metric no results

seems like seeder is missing here

There isn't really anything to seed here... unless you want some false historical data to be seeded?

And seconds spent probably should be seeded in ProgressSeeder in Courses if it is not seeded already

And seconds spent probably should be seeded in ProgressSeeder in Courses if it is not seeded already

exactly 👍


Some inconsistencies

I was wondering why the following classes are not exported (i.e. part of sleepecg/io/__init__.py and therefore also in the API docs):

sleepecg.io.sleep_readers.Gender
sleepecg.io.sleep_readers.SubjectData

These are "public" classes, and should therefore be exported (or made "private" by appending an underscore).
Similarly, sleepecg.io.gudb.GUDB_MD5 is "public" (but not exported), I think here it would be consistent to rename to _GUDB_MD5.
@hofaflo WDYT?

I agree on all points!


Scroll to bottom programmatically

When I calling recyclerview.scrollToPosition(...) or chipsLayoutManager.scrollToPosition(...) with different arguments or something else, it`s not scroll correctly

+1

@kovac777 You found any work around ???
Please suggest me


Add blk_read_time to pg stat statements metrics collected

What does this PR do?

Adds blk_read_time to the list of pg_stat_statements columns to collect metrics for
Motivation

I would like to be able to see and track this value on a per query basis in Datadog
Additional Notes

I'm guessing this isn't all that needs to happen before the metric is usable for me as an end user, but hopefully it's a good first step! Let me know if there is anything else I can/should do as part of these changes. Thanks!
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached

Hey @tcpatterson! Thanks for the suggestion! I have a few questions which will help us prioritize this:

Can you give us some background on how you intend to use this metric?

Can you describe what your database workloads look like?

thanks @justiniso, sure thing

Can you give us some background on how you intend to use this metric?

We're looking to reduce our IOPS usage on our AWS RDS instance and hoping to be able to use the blk_read_time value to help determine queries that are contributing the most to read iops.

Yes

Can you describe what your database workloads look like?

The workloads are mostly OLTP based. about 3-4B queries a day. reads account for about 70% of the throughput

@justiniso any update here? I'd like to see this as well

@tcpatterson @mbarany thanks for the contribution and input! I spoke to our product managers about this change and we definitely want to move forward on this metric. However, we only want to send it when track_io_timing is enabled. There are some additional things we have to do on our side, so I am going to close this PR for now, but will follow up when we are ready to merge the required changes.

Hi @tcpatterson and @mbarany!  This feature has been merged in #12380 and will be available in agent version 7.38.


Plex app needs a newer tag in docker-compose.yml

What happened?

Installed Plex from new store.
Opened Plex - app said that it has some updates
Go to Casa - choose Plex - Check then update. Got message "In the process ....."
30 min later.. Nothing.. Same version and no updates.

Screenshots

Should be resolved in https://github.com/IceWhaleTech/CasaOS-AppStore/pull/176
Thanks to @WisdomSky


How to use this ?

hello, im noob in compiling, but i figured how. Now what ? how to use the .exe ?

This is a work in progress and the exe is not yet useful. Note that once
finished, one will have to type out the full matrix, which can take longer
than simply solving  it manually ;)
On Mon, Dec 14, 2020 at 4:29 AM iulicafarafrica<EMAIL_ADDRESS>wrote:

hello, im noob in compiling, but i figured how. Now what ? how to use the
.exe ?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/kyle-rader/breacher/issues/1, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABEI7XD3O6QLCBCIISXS7XDSUX5XFANCNFSM4U2UTMRQ
.

@iulicafarafrica I've completed the first version of this. It's now fully functional. Lots of room for improvement on visually displaying the solution but it can now be used to solve puzzles :). See the readme for instructions and feel free to leave suggestions!


(Server) Implement seed data mechanism for Roles collection

Currently we have seed data mechanism for Video collection. Follow this process  and implement seeding for Role.

Work in progress


Correction for raw affiliation Ryerson University Accounting Society, The Netherlands

Correction needed for raw affiliation Ryerson University Accounting Society, The Netherlands
raw_affiliation_name: Ryerson University Accounting Society, The Netherlands
new_rors: 05g13zd79
previous_rors:
works_examples: W2286591073;W2267874019
contact: 037484cd7f02c7ae86c248c61900af7e:eb976190 @ ourresearch.org

This issue was accepted and ingested by the OpenAlex team on 2024-10-10. The new affiliations should be visible within the next 7 days.


Using SunmiStyle doesn't work as expected

await SunmiPrinter.printText(
        'Payment receipt',
        style: SunmiStyle(
                fontSize: SunmiFontSize.XS,
                bold: true,
                align: SunmiPrintAlign.CENTER
        )
)},

This code dosn't print the expected styled string but print only bold string

example:
void printText() {
    SunmiPrinter.line();

    SunmiPrinter.startTransactionPrint(true);
    SunmiPrinter.printText("Test",
        style: SunmiStyle(bold: true, align: SunmiPrintAlign.RIGHT));
    SunmiPrinter.exitTransactionPrint(true);

    SunmiPrinter.lineWrap(2);
  }

the result is :
---------------------------------

Test

Test is not bold, not align, not XS
and is printed after the line

Are you using the V2s ?
my other device was working very good

Are you using the V2s ? my other device was working very good

Yes, i'm on V2s

I tried with the new package update but obviously it didn't change the result.
It appears as if styled printText calls are executed last, as if they were put on hold and executed last

ok I found the problem, that is, the lack of await in front of the function, so the code after calling printText() continued and did not wait for it to finish so that the printText function was apparently executed last
Code solution:
void printText() {
    SunmiPrinter.startTransactionPrint(true);
    await SunmiPrinter.printText("Test",  // there is the missing await
        style: SunmiStyle(
            fontSize: SunmiFontSize.XS,
            bold: true,
            align: SunmiPrintAlign.CENTER));
    SunmiPrinter.exitTransactionPrint(true);

    SunmiPrinter.line();
    SunmiPrinter.lineWrap(2);
}

Print result is :
Test

  ---------------------------------


MLDB-1954 clang

The latest instalment in making it possible to develop MLDB on a laptop on a plane...

+1


Principal.getName with federated cognito user not returning identityId

Hi,
I am using spring with Api gateway and lamba proxy.
When i Inject the principal on my controller it does not return the cognito identity with federated user.
The principale getName method now return the temporary user generated by the assume role call.
before the 0.5 my getAuthentificationScheme was AUTH_SCHEME_COGNITO, but now it fall into the AUTH_SCHEME_AWS_IAM
https://github.com/awslabs/aws-serverless-java-container/commit/4f1e773a667ada725590455793e5f47ebd5ea748#diff-dd23022884ed5861d345f0b5f36b2cc7
seen with @SAPessi in gittter

There was indeed a breaking change when we started handling user pool auth properly. You are now going to the AWS_IAM branch of the code - which is the correct behavior. However, you expect (rightly) the Cognito identity id as a principal since the call is authorized with credentials produced by Cognito Federated Identities.
This is caused by the fact that the library returns the userArn when the call is authorized by AWS credentials, regardless of the origin of the credentials: Code here.
We can make a simple change to the method to handle Cognito credentials as a special case:
if (event.getRequestContext().getIdentity().getCognitoIdentityId() != null) {
    return event.getRequestContext().getIdentity().getCognitoIdentityId();
} else {
    return event.getRequestContext().getIdentity().getUserArn();
}

The behavior in this case would be to return the Cognito identity id whenever the call was signed with credentials from Amazon Cognito or the user ARN whenever the request was signed with non-Cognito credentials. This is potentially another breaking change to the behavior of the library so I'm going to leave it here for a bit to gather some feedback before I implement it.
FWIW, I think it's the right thing to do.

I'm not very familiar with this use case, but if it's going to be a breaking change, perhaps it should be in 0.6? The proposed fix looks fine to me

Following the conversation on Gitter looks like we are settled on option 1. Committing a fix for this now.

Resolving now that the fix is merged into master


windows: Return exit status from test script

This is needed for automated testing.

The script runs fine, but I am seeing a test that appears new since my last SDK testing, WaitEventThenSet, crashes my Nvidia driver / Windows system.  Is anyone else seeing this?

WaitEventThenSet is passing for me on Skylake, but only because it bails early.  I added a printf:
"EXITING WaitEventThenSet due to queue_props[0].queueCount < 2"
We should consider instrumenting all the bailouts to get more accurate coverage analysis.
We do fail CreatePipelineTessPerVertex on Skylake, but I've snuffed that test until we have time to debug it.

I don't think we should hold up this series to understand existing test failures.  The last commit adds the ability to disable tests.  Once this is in, we can start testing Windows builds on our side.

GitHub is warning that there is no newline at the end of the file.

I agree that this series is fine and sorry to derail with the test failure

Newline added - thanks!  I'll merge it and this will be the basis for precommit validation against master.

cherry-picked to master


SqlServerDsc: Fix PSSA rule warnings

Pull Request (PR) description

Changes to SqlDatabase

Fix minor Script Analyzer warning.

Changes to SqlServerReplication

Fix Script Analyzer warning (issue #263).

This Pull Request (PR) fixes the following issues:
FIxes #263
Task list:

[x] Change details added to Unreleased section of CHANGELOG.md?
[ ] Added/updated documentation, comment-based help and descriptions in .schema.mof files where appropriate?
[ ] Examples appropriately updated?
[x] New/changed code adheres to Style Guidelines?
[ ] Unit and (optional) Integration tests created/updated where possible?

This change is

Reviewed 3 of 3 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Review status: :shipit: all files reviewed at latest revision, all discussions resolved, all commit checks successful.

Comments from Reviewable


fix:  show_gui and ui_mode value swap

closes https://github.com/ansys/pyfluent/issues/2773 https://github.com/ansys/pyfluent/issues/2754
show_gui and ui_mode values were swapping. It's resolved now.

@hpohekar @mkundu1
I have verified that all the following use cases are working properly with these changes:
>>> import ansys.fluent.core as pf
>>> s=pf.launch_fluent(show_gui=True)
>>> s2=pf.launch_fluent(ui_mode=pf.UIMode.GUI)
>>> s3=pf.launch_fluent(ui_mode=pf.UIMode.HIDDEN_GUI)


A few broken links

First of all, kudos on your repo as it's well thought out and I've discovered a few new tools that I will be reseachibg more.
Nothing major and didn't think opening an issue would qualify, but Flagger on your readme is actually linked to Falco,  I believe it should be their home page: https://falco.org/
Second one is when reviewing your page about giving feedback/contributing, most point back to a page on your repo that's not populated.
Once again, thanks for taking the time to make sure a decent repo and write up!

Thanks @osnabrugge I'll need to update and repair those links, appreciate that overall things have been helpful!
Feel free to also join the Discord community for more direct contact and appreciate the help :)

Those links and items should all now be resolved 👍


what folks were requesting of npm's search

Just forwarding a link to a issue/request made long ago at about npm's website which never went anywhere.
Very happy to see someone else has finally picked up the ball and run with this major issue...thx!
https://github.com/npm/www/issues/6
the score is a good idea but additional filtering, sorting and/or ability for visitor to tweak the scoring algorithm would be a plus.

Hey @dkebler
We have a good number of filters in place, you may find the documentation here: https://api-docs.npms.io/#api-Search-ExecuteSearchQuery. Let me know if you have a specific filter requirement not present there already.
Regarding additional sorting features, could you look at my comment at https://github.com/npms-io/npms-www/issues/133? We got that almost done, we just need feedback to see if it's good.
We don't allow sorting for a specific metric because that goes against how npms scores packages. Instead, the sorting options offered in that branch are based on quality, popularity and maintenance weights.

on npmjs.org After thinking I found a good package I have to jump to github repo and take a look at the latest commit.  Packages not being maintained (say no commits in last x months) or with a bunch of unclosed issues are not worth my time. Is that what your maintenance score tracks?   How to find tune that?  Seems I saw something in the api for weights?
It's pretty easy to find popular/used packages (which tend to be maintained) but what has been hard is to find are high quality packages that may not have a ton of use yet.
I suppose the "npms score" is a sorting but if the score is the only way to query without some override or filter then not as useful.     Such a query in the past would have yielded nothing but as npm db has grown some gems.
In general It seems as though github has some of the important stats for a package so does npms only access the npm db or the github db for the corresponding package repo as well?
I understand that via the api or the website search box one can tweak things "manually"  but down the road the webapp gui should give the visitor a little help with that.   Not sure I have time to memorize another Api :).   I do have several github search "templates" saved as bookmarks for searching github but kinda wish that I could just do some clicking for more advanced search or tweaking your algorithm.
Looks like you are using a rest api for the search so at least I can bookmark them and come back and change my keywords.
So some input...but really Thx for you time and efforts.  Even as it is it's way better the npms.org web.

@dkebler I definitely agree with you that's why we want to offer additional sorting features. The attached screenshot shows an upcoming feature that will allow to tweak the weights, so that you can tweak quality and maintenance over popularity.
This answers most of your points in a way that is simple and (hopefully) intuitive. What do you think?

Yea that's the idea.  perfect.
more suggestions then.
To expand on that if user clicks on a parameter (e.g. Quality) a modal could open that one would allow tweaking the  algorithm for that parameter (whatever has not been hard coded).  Might as well link/display relevant documentation on how that parameter is calculated (way important)
Could also include an "advanced" link/dialog where you could put in post weight query additional filters like I mentioned (e.g. first commit in last year).
So little by little the "front ender"  could add more gui enabled access to the API as it grows.
Down the road  I'd add the ability to name/save/recall search settings.  It would allow user to bring up some useful past advanced searches and for example swap the keywords (that's what I so manually at github now).  I wouldn't set up any internal user/storage system just use their github account (using a token if need be) and save the searches to a gist(s) or maybe a repo.  For example there's an Atom settings sync package that just does that.  Allows you to sync settings between Atom instances on different machines by writing and reading them to/from a gist.  Personally I'd write them out as mongodb style collection of search objects in their format of choice (json,cson,yaml,toml,honey... (mine is cson).   Then it's way easy for a user to manually copy/tweak etc and add comments to the file.  I'm doing doing all my project settings that way now.    There are packages that will load or "require" any of those simply based on extension.   So if you don't already have a search "object" in your code probably good time to think about all the keys it might contain.  In a collection format it's super easy to use something like Lowdb https://github.com/typicode/lowdb to manipulate the collection which is what I do.


is there mobilenet in caffe2?

https://github.com/Zehaos/MobileNet I found that the cpu speed of mobilenet is about 50ms!!!

+1

has caffe1 ,but i find not caffe2

I translated the model given by shicai in https://github.com/shicai/MobileNet-Caffe using the caffe2 translator. It works well on my computer and mobile device.
https://drive.google.com/file/d/0B-q03jmMkN8lV21RZkFNYTJjeE0/view?usp=sharing
https://drive.google.com/file/d/0B-q03jmMkN8lZzNjcmVSUmdyTVE/view?usp=sharing

@tianyoul how about the speed?  if no custom depthwise conv op, the mobilenet seem not so fast when implemented by group method

With batch norms in place with the convolutions even though the conversion creates the model files it gave me error while using them.
@tianyoul did you also had to change the batch norm layers not to be in place with the convolutions, as it was in the original deploy prototxt?

Hi,
I encountered the following error when I tried to run the  mobilenet model of Caffe2, exported from my server,
on Android.
08-11 09:17:57.012 14428-14498/facebook.f8demo E/F8DEMO: Couldn't parse net from data. 08-11 09:17:57.058 14428-14498/facebook.f8demo E/F8DEMO: done. 08-11 09:17:57.058 14428-14498/facebook.f8demo E/F8DEMO: Instantiating predictor... 08-11 09:17:58.000 14428-14498/facebook.f8demo A/native: [F given_tensor_fill_op.h:27] Check failed: output->size() == values_.size() output size: 2359296 given size: 1144986 08-11 09:17:58.000 14428-14498/facebook.f8demo A/native: terminating. 08-11 09:17:58.000 14428-14498/facebook.f8demo A/libc: Fatal signal 6 (SIGABRT), code -6 in tid 14498 (AsyncTask #1)
I implement the depthwise convolution via group_conv.
Have you guys met this problem? Any suggestion will be appreciated.
@zvadaszi @tianyoul

deploy.prototxt.txt
@lhCheung1991 , find attached the deploy.prototxt that worked for me to convert the caffemodel from here https://github.com/shicai/MobileNet-Caffe
The only difference to shicai's mobilenet_deploy.prototxt is that I had to change all batchnorms not to be in place with the bottom convolutions, otherwise it did not work in Caffe2 for me.

Get. Thank you @zvadaszi.
I wrote the code to build the mobilenet with Caffe2 brew.group_conv(). It seems work correctly on my cloud server but sucks on Android for batchnorms... Once I removed the batchnorms, it will work.

I have been trying various models in caffe2 and the speeds, across the board, are much poor than those achieved on caffe. For instance, if one tries different models at https://github.com/onnx/models , the timings are nowhere near what are reported in different papers. Could I be doing something wrong?

@austingg  I have the same problem.If I use group convolution,not depthwise convolution,it is so slow. I found implement of group convolution using for loop. So  how to solve this problem in caffe2? Thanks!


Exception when javascript awaits a Dart async function

I have a web app for which I'm implementing an API that can be used by independent js scripts. Using the recommended package:js/js.dart with @JS annotations, the calls from js to Dart all work fine except for async calls to await a Dart function. The await call throws an exception or doesn't wait.
I have written a small sample VScode project that easily reproduces the problem.
The Dart connect async function returns a Future but when the future is completed an exception is thrown. Is this to do with Dart returning a Future but the javascript expecting a Promise?
There are a lot of posts referencing Dart calls into javascript but I have found very little in the reverse direction (js to Dart) using async/await.  I expected the promiseToFuture and futureToPromise functions to perhaps throw some light on this problem but there's not much information out there in the js-to-dart context. The Dart documentation on this issue seems to be non-existent.
There is another odd issue that could just be a symptom. Using 'webdev serve' the await call throws an exception but, if it is caught in a try/catch, the await actually does wait and returns the completion value. Using 'webdev build' the await calls do not wait at all.
If I have missed the relevant documentation I would be very grateful to be pointed in the right direction. Aside from that, I'd like to hear any suggestions for a working solution!
All the Dart code is in main.dart, built and tested on:
Dart SDK 2.14.4 (stable) (Wed Oct 13 11:11:32 2021 +0200) on "windows_x64":
VS Code 1.62.3 (user setup) OS: Windows_NT x64 10.0.19043
Chrome 96.0.4664.45 (Official Build) (64-bit)
@JS()
library testawait;

import 'dart:async';
import 'dart:core';
import 'package:js/js.dart';

/// main.dart
///
/// This web app is an example of a javascript script await-ing a Dart async
/// function. The Dart async function returns a Future but when the future
/// is completed an exception is thrown. Is this to do with Dart returning
/// a Future but the javascript expecting a Promise?
///
/// The script is triggered by a button (Click me) in the web page (index.html).
///
/// When running with WEBDEV SERVE the await respects the Future.delay but throws
/// an exception and the returns to the catch.
///
/// When running with WEBDEV BUILD the awaits do not delay and the returns go to
/// the next statement.

@JS('connect')
external set _connect(void Function(String host) f);

void main() async {
    _connect = allowInterop(connect);
}

///
/// This causes an exception but the await does wait and the
/// completion is returned, but to the catch:
///
///     Uncaught TypeError: T.as is not a function
///     at _Future.new.[_setValue] (future_impl.dart:419)
///
///     The code at that location was seen in dart_sdk.sound.js:formatted:
///                 [_setValue](value) {
///                 T.as(value);    <<<<<<<<<<<<<<< exception here!!
///                 if (!!this[_isComplete])
///                     dart.assertFailed(null, I[32], 420, 12, "!_isComplete");
///                     this[_state] = 8;
///                     this[_resultOrListeners] = value;
///                 }
///
Future<dynamic> connect(String host) async {
    print('main: before connect');
    var completer = Completer<dynamic>();

    // This is just to simulate a connect event being processed
    // elsewhere that completes the completer. This future
    // is not used for anything else.
    Future.delayed(Duration(seconds:2), () {
        print('main: connect done after 2s');
        completer.complete('Future: Connect complete');
    });

    return completer.future;
}

And here is the html that includes the js script; click on the 'click me' button to trigger a call to scriptWaits:
<!DOCTYPE html>

    <meta charset="utf-8">
    <meta http-equiv="X-UA-Compatible" content="IE=edge">
    <meta name="viewport" content="width=device-width, initial-scale=1.0">
    <meta name="scaffolded-by" content="https://github.com/dart-lang/sdk">
    <title>testawait</title>

			<input type="button" value="Click me" id="script_trigger" onclick="scriptWaits()"></input>

and pubspec.yaml:
name: testawait
description: Testing javascript await a Dart function
version: 1.0.0

environment:
  sdk: '>=2.14.4 <3.0.0'

dependencies:
  http: ^0.13.3

dev_dependencies:
  build_runner: ^2.1.2
  build_web_compilers: ^3.2.1
  js: ^0.6.3
  lints: ^1.0.0

Hey @zapappa  did you ever find a fix for this? I'm experiencing the same issue, I'll appreciate any help.

Sorry to say that the only indication was that the reverse direction (js to Dart) was not supported.

If you have a Dart Future and you want to await it on the JS side then you need to convert to Promise first using some variant of futureToPromise helper.


[docs][joy] Add adjustments to the color inversion page

[x] I have followed (at least) the PR section of the contributing guide.

This PR adds some adjustments to the Joy UI Color inversion page. It includes both some slight copywriting tweaks, as well as information hierarchy and design tweaks (mostly simplifications) to the common examples.
https://deploy-preview-37143--material-ui.netlify.app/joy-ui/main-features/color-inversion/

@samuelsycamore these look great already, appreciate it! I think we could look back at the Joy docs before its stable launch to refine all of these rough edges 🤙

If any of you knows how to help: I've ran yarn docs:typescript:formatted twice, which returns "0 demos were unsuccessfully transpiled" but test_types keeps shouting the "Did you run yarn docs:typescript:formatted?". 🤔


Fixed issue 6612 by covering ibmq examples by lint check

Summary
Fixes #6612
Details and comments
The ibmq examples were not being covered by lint checks. This PR fixes lint errors in ibmq examples and covers them by the lint checks.

Added __init__.py files to convert qiskit examples to package to facilitate lint checking
Updated Makefile and tox.ini lint targets to cover ibmq examples as well
Updated Make.bat batch file to make it consistent with these and other recent changes
Fixed up lint errors found
The check C0114 (missing module docstring) is no longer disabled in examples, as it is good to have docstrings.

Hi, as some things are new to me, I'd like to discuss some points about this PR:

I got two lint errors while running pylint on ibmq examples: E0611: no name 'ibmq' in module 'qiskit.providers' and E0401: Unable to import 'qiskit.providers.ibmq'. These were appearing even after doing 'pip install qiskit-ibmq-provider'. When trying to run the ibmq examples locally, 'import qiskit.providers.ibmq' is able to locate it in 'virtualenvpath'/lib/site-packages/qiskit/providers/ibmq. However, I believe pylint is looking for it in qiskit-terra/qiskit/providers, where it is not present. Therefore, I think the pylint check is too strict in this case and I believe it is appropriate to disable these two checks in the lint targets. If there is a better solution to this, I'd love to hear. @levbishop

From what I could understand by reading the code, examples are not yet checked by pylint in the azure_pipelines.yml. Is there a reason for doing so? I think azure_pipelines.yml should be updated similarly too.

Thanks :)

Hi @dhruvbhq !
Sorry we missed this one. Is your first point ("Added __init__.py files to convert qiskit examples to package to facilitate lint checking") necessary? Im not sure if the examples are being packaged.

Hi @1ucian0!
I haven't looked at this in a while, but I think what I did/meant was this:
In the Makefile, the lint target runs pylint ... on examples/python/*.py . See here.
However, the IBMQ examples are present as examples/python/ibmq/*.py, so the above implementation didn't cover the ibmq/ *.py files.
I believe adding the __init__.py files allowed me to define the line target as
pylint -rn --disable='C0103, W0621, E0611, E0401' examples,
instead of individually specifying the *.py files.
I think I went for this implementation because if other example subdirectories get added in future, it would not be needed to add an extra dir/*.py to the lint.

Hmm... I think pylint is recursive in all the subdirectories by default. But I did not check it.

Hi,
Ah, I see. Those are some interesting consequences which I was unaware of. I will try to work on it as soon as I am able to.

Hi there - sorry to come back like this, but with #8080 (and general Qiskit direction), we're going to be removing the IBMQ examples from the Terra repository because qiskit-ibmq-provider is going to be deprecated in favour of a new package called qiskit-ibm-provider (ibm not ibmq).  We won't be replacing these examples, so covering them for lint isn't necessary any more.
Thanks for the interest in contributing, though, and sorry it came down like this.  If you're interested in finding something else, please have a look at our issues tagged "good first issue" or "help wanted".


[Bug]: GET request for files using wrong path in autogenstudio

Describe the bug
After executing the sine wave example, the sine_wave.png file does not display.  I then tried my own custom example and the same thing happened.  The terminal shows that the GET request for the file ended in a 404.
INFO:     <IP_ADDRESS>:53348 - "GET /api/files/user/user/198fb9b77fb10d10bae092eea5789295/sine_wave.png HTTP/1.1" 404 Not Found

Here is the actual path to the file:
\files\user\198fb9b77fb10d10bae092eea5789295\sine_wave.png

I'm not sure where the second user is coming from in the GET request.  It looks like it might be a concatenation issue.  I'm running this on Windows 11 with Mistral Instruct 7B running on LM Studio.  I tweaked the default system message slightly to remove the TERMINATE command, as this was causing autogen to terminate before the task was completed successfully.  Not sure if that would have any bearing on this though.
Steps to reproduce
No response
Expected Behavior
The GET request should use the correct URI and return a 200.
Screenshots and logs

Additional Information
Autogen version: 0.0.27a0
Operating System: Windows 11
Python Version: 3.10.6

Same on Windows 10 with standard installation using anaconda, no change of the default setup, just tried the same use case with exactly the same result as reported by tjb4578.

Hi,
I believe this is a windows issue where some code that works correctly on linux resolves incorrectly on windows.
This function is what determines the url of files that have been updated. There is change it resolves incorrectly on windows. I'll take a look

(Hopefully someone finds this useful) I managed to patch this by simply making a soft symbolic link loop to correct the behaver. (didn't feel like compiling a fix from source so a no-code solution is as follows)
launch an administration CMD
mklink /D "YOURPYTHONFOLDER(E.G. Anaconda\envs\YOUR ENVIROMENT e.g. (ag)\Lib\site-packages\autogenstudio\web\files\user\user" "YOURPYTHONFOLDER(E.G. Anaconda\envs\YOUR ENVIROMENT e.g. (ag)\Lib\site-packages\autogenstudio\web\files\user"
this creates a simple loop where the  /user/user folder points to the /user/ folder. ^_^
Incidentally I would recommend solving the OS path issues by implementing the 'pathlib' library. for example Windows uses "" instead of "/" which can be a major issue on cross platform use but pathlib just deals with that.
from what i can see the point of failure in the function is line 184 :: uid = dest_dir.split("/")[-1]
just replace with
dest_dir_as_path = pathlib.Path(dest_dir)
uid = dest_dir_as_path.parent.name
and remember to import the library at the top
import pathlib

@Starskimmer ,
Thanks for the note. I think you are right.
Adding a fix .
Will get my paws on a windows machine at some point and test this out.

This has been addressed and tested to work on windows.
Closing out for now. Feel free to reopen if still seeing a related issue.


Compilation error: po translation typo

There is a line break that should not be there:

qalculate/translations/po/plasma_applet_org.kde.plasma.qalculate_hr.po:306: end of line found within a string

I fixed it and compilation works fine

I had seen this already yesterday but apparently forgot to push the fix.


Hovering Delay issue with Responsive Navigation

How to reproduce this bug:
I've used "Drilldown Dropdown Responsive Menu" and I am facing this issue:

https://codepen.io/IamManchanda/pen/jmKPdM?editors=1010
Could anyone please tell me how to stop hover delay?

Couldn't reproduce!
See video screenshot => https://drive.google.com/file/d/0B4EN5uGMUjoZeUZJdTBLN3pNT1k/view
( PS: Video attached is getting processed right now, so you may need to check back 5 minutes later from now )

Thank you, @IamManchanda for your quick reply.
If you observe closely, there is a small delay, when you hover over Item1B the Item1A sub-menu still show, but I actually want to make Item1A submenu to hide more quickly, when hovering over Item1B.
Let me know in case, I am not conveying it correctly.

I understand... I am not sure if that can happen or not!
cc @kball

@kashifullahwebdeveloper try setting data-closing-time to a small (or 0) number on your menu. see https://codepen.io/kball/pen/vJzjwY?editors=1010
This is utilizing the options from dropdown menu (see http://foundation.zurb.com/sites/docs/dropdown-menu.html#js-options) as the responsive menu is made up of the individual menu pieces, so their options work.
Going to close this as I believe it resolves your issue, please reopen if you're still having trouble.

@kball, Thank you so much.
Yes, that fixed my issue. 👍


Search for private repositories

Description
I'de like to know if is possible to have an feature that allows to log-in and search for private repos. I struggle everey time that i do a git clone, i need to open github website, go to repos, find my repository and got the url. This addon will make it so much easier do that if that were possible.

Try setting up the access token in settings.


[BUG] Mac doesn't delete Vault shares after deletion of app

Describe the bug
When deleting the application on Macbook and reinstalling it after the old vault shares are still loaded and accessible.


⭐FEAT: Make the links text bolder

TODO:

Make the text a bit bigger/bolder

Shall I work on this issue @ArslanYM, please assign this to me.

@shaikahmadnawaz Assigned to you.

The above screenshot is of bold 👆

The above screenshot is of semi bold 👆
Which one gives better look? or shall we increase text size?

@shaikahmadnawaz I feel like the first one  will do


Remove initalizer requirement for Ruby 2.7+

Motivation
As discussed in github#93, Ruby 2.7 introduces #const_source_location to find the location of constants. This means that we no longer have to find the location of components through the location of their #initialize method, and we can remove the initializer requirement for Ruby 2.7+.
Changes

Use #const_source_location in ActionView::Component::Base::source_location, when it's available.
Move the initializer-less component test out of invalid, and in to a proper component test. Test it still throws the error for Rubies < 2.7.
Change eager-loading logic for Ruby 2.7+

Todo

[x] Update CI to test Ruby 2.7 once supported by actions (actions/setup-ruby#45, actions/virtual-environments#209).
[x] Rebase to pick up workflow changes in #200 when that's merged.

(The old todo, Test eager loading change, is no longer necessary.)

Thanks for taking this on! Let's keep this PR open until we can test these changes in CI.

Awesome, @bostanio , When the changes from #175 go in I'll refactor here

Do folks have a preference of using the Minitest skip versus conditionally defining tests with #define_method based on the Ruby version?
I see some instances of Rails's test suite making use of skip, like here. So I'm leaning towards that. But here's what changing to conditionally defined tests looks like:
diff --git a/test/action_view/component_test.rb b/test/action_view/component_test.rb
index 504ad03..e2e359d 100644
--- a/test/action_view/component_test.rb
+++ b/test/action_view/component_test.rb
@@ -353,11 +353,11 @@ class ActionView::ComponentTest < ActionView::Component::TestCase
     assert_html_matches "The Awesome post component!", render_inline(post).to_html
   end

-  def test_missing_initializer
-    skip unless RUBY_VERSION > "2.7.0"
-
+  if RUBY_VERSION >= "2.7.0"
+    define_method :test_missing_initializer do
       assert_html_matches "Hello, world!", render_inline(MissingInitializerComponent).text
     end
+  end

   private

diff --git a/test/action_view/integration_test.rb b/test/action_view/integration_test.rb
index 936ed1d..ad66d87 100644
--- a/test/action_view/integration_test.rb
+++ b/test/action_view/integration_test.rb
@@ -173,9 +173,9 @@ class IntegrationTest < ActionDispatch::IntegrationTest
     assert UnreferencedComponent.compiled?
   end

+  if RUBY_VERSION < "2.7.0"
     test "does not compile components without initializers" do
-    skip if RUBY_VERSION >= "2.7.0"
-
       assert !MissingInitializerComponent.compiled?
     end
   end
+end
diff --git a/test/action_view/invalid_components_test.rb b/test/action_view/invalid_components_test.rb
index be70de0..20d6906 100644
--- a/test/action_view/invalid_components_test.rb
+++ b/test/action_view/invalid_components_test.rb
@@ -1,15 +1,16 @@
 # frozen_string_literal: true

 class ActionView::InvalidComponentTest < ActionView::Component::TestCase
-  def test_raises_error_when_initializer_is_not_defined
-    skip if RUBY_VERSION >= "2.7.0"

+  if RUBY_VERSION < "2.7.0"
+    define_method :test_raises_error_when_initializer_is_not_defined do
       exception = assert_raises ActionView::Component::TemplateError do
         render_inline(MissingInitializerComponent)
       end

       assert_includes exception.message, "must implement #initialize"
     end
+  end

   def test_raises_error_when_sidecar_template_is_missing
     exception = assert_raises ActionView::Component::TemplateError do

@bostanio Refactored to include your changes and it seems like much less of a hack. Thanks!! 🙇

I like skip better than define_method. I think I'd make it a named check (like the Ruby example) skip if supports_..... We can define that check against a Ruby version or maybe with a similar check for the method we look for on Module.

@bostanio, Maybe I'm misunderstanding, but the checks should be named in both the tests and in base.rb.
Perhaps you were looking at an older version of the changes? See a9122edb12afae627af5ff10aba4e772a5288035 and 7f22b0eb3b59c69f89980ecedca03f79fa01922d

@dylnclrk Yup read and responded your larger comment without seeing the code. Exactly what I was thinking. Nice!

@bostanio oh duh, the diff I posted didn't have the predicate methods in it

@joelhawksley OK, I think with the extraction of the CI config, I think I addressed all of your comments. Let me know what ya think!

Thanks for the reviews everyone! I don't know about you, but I'm stoked to see the initialize requirement go away! 😄

Thanks! You got it, I added a little section to the bottom of the readme & a section to the changelog!

👏 🙇


Update Manifest files

Purpose
This PR updates to the latest dependencies.
Closes #356
Content

Solution implemented: Updated Manifest.toml files using ] up

Review checklist
I have:

followed the codebase contribution guide: https://clima.github.io/ClimateMachine.jl/latest/Contributing/
followed the style guide: https://clima.github.io/ClimateMachine.jl/latest/DevDocs/CodeStyle/
followed the documentation policy: https://github.com/CliMA/policies/wiki/Documentation-Policy
checked that this PR does not duplicate an open PR.

In the Content, I have included

relevant unit tests, and integration tests,
appropriate docstrings on all functions, structs, and modules, and included relevant documentation.

[x] I have read and checked the items on the review checklist.

CI passes and this PR is ready for review.
Cc: @LenkaNovak

LGTM! No behavioral changes from the previous merge! rocket

Great! I rebased and will merge

bors r+

bors r+


[ReadMe] Fixed Color Picker spacing.

Summary of the Pull Request
We refer to Color Picker (with spacing) throughout the PT app. Readme mentioned it as ColorPicker.
PR Checklist

[ ] Applies to #xxx
[ ] CLA signed. If not, go over here and sign the CLA
[ ] Tests added/passed
[ ] Requires documentation to be updated
[ ] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Info on Pull Request
What does this include?
Validation Steps Performed
How does someone test & validate?

Thanks @niels9001


Change current "angular-translate.min.js" to the one in the description

Change the current lib/angular-translate/angular-translate.min.js to this file http://cdnjs.cloudflare.com/ajax/libs/bower-angular-translate/2.0.1/angular-translate.min.js

Sorry, but wait with this!!!


Build task fails

After adding the nuget package to my project the build returns an error:

Fehler	MSB4113	Die angegebene Bedingung @(DsComTlbAliasNames->Count() > 0) wird zu @(DsComTlbAliasNames->Count() > 0) statt zu einem booleschen Wert ausgewertet.

According to the message the error originates in dSPACE.Runtime.InteropServices.BuildTasks.Tools.targets, line 47. It seems there is a typo in the condition phrase of _DsComAliasNames  (comparing it to the surrounding ones). I assume it should be Condition="@(DsComTlbAliasNames->Count()) > 0" instead of Condition="@(DsComTlbAliasNames->Count() > 0)" (wrong position of the closing paranthesis).

I won't manage to watch for this today. I'll come back to you next week.


CI: fix MacOS CI

Fixes #13108

carefully specify the absolute path to OpenBLAS
gcc lib runtime deps so that our MacOS CI is
able to run the testsuite again

in the future, we may want to restore an approach
that is not tied to a particular gcc toolchain
version; however, as this involved i.e., the use
of perl tricks, I suggest we open an issue to
do that later

MacOS CI is passing again so I'll self-merge and open an issue about restoring a potentially more general solution at a later time.


[Info] - Mock Snakebite to point to LocalFS

Is there a way to Mock Snakebite to point to LocalFS for the purpose of unit tests ?

Not right now. Should be trivial to implement though.


Add data assimilation notebook

Overview
This PR adds the data assimilation notebook, the final one in the tutorial series. Also updates the doc index to add it to the list.

From ReadTheDocs:
Traceback (most recent call last):
  File "/home/docs/checkouts/readthedocs.org/user_builds/pavics-raven/conda/421/lib/python3.10/runpy.py", line 196, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/home/docs/checkouts/readthedocs.org/user_builds/pavics-raven/conda/421/lib/python3.10/runpy.py", line 86, in _run_code
    exec(code, run_globals)
  File "/home/docs/checkouts/readthedocs.org/user_builds/pavics-raven/conda/421/lib/python3.10/site-packages/sphinx/__main__.py", line 13, in <module>
    from sphinx.cmd.build import main
  File "/home/docs/checkouts/readthedocs.org/user_builds/pavics-raven/conda/421/lib/python3.10/site-packages/sphinx/cmd/build.py", line 25, in <module>
    from sphinx.application import Sphinx
  File "/home/docs/checkouts/readthedocs.org/user_builds/pavics-raven/conda/421/lib/python3.10/site-packages/sphinx/application.py", line 32, in <module>
    from sphinx.config import Config
  File "/home/docs/checkouts/readthedocs.org/user_builds/pavics-raven/conda/421/lib/python3.10/site-packages/sphinx/config.py", line 23, in <module>
    from sphinx.util import logging
  File "/home/docs/checkouts/readthedocs.org/user_builds/pavics-raven/conda/421/lib/python3.10/site-packages/sphinx/util/__init__.py", line 35, in <module>
    from sphinx.util import smartypants  # noqa
  File "/home/docs/checkouts/readthedocs.org/user_builds/pavics-raven/conda/421/lib/python3.10/site-packages/sphinx/util/smartypants.py", line 33, in <module>
    from sphinx.util.docutils import __version_info__ as docutils_version
  File "/home/docs/checkouts/readthedocs.org/user_builds/pavics-raven/conda/421/lib/python3.10/site-packages/sphinx/util/docutils.py", line 31, in <module>
    from sphinx.util.typing import RoleFunction
  File "/home/docs/checkouts/readthedocs.org/user_builds/pavics-raven/conda/421/lib/python3.10/site-packages/sphinx/util/typing.py", line 34, in <module>
    from types import Union as types_Union
ImportError: cannot import name 'Union' from 'types' (/home/docs/checkouts/readthedocs.org/user_builds/pavics-raven/conda/421/lib/python3.10/types.py)

Looks as though someone pushed a bug to sphinx (Union is in typing, not `types).

@Zeitsperre I'm pretty sure I didn't do it though... possible someone else pushed that somehow? Or else I am at a loss how I could have modified the sphinx package!

Definitely not you, @richardarsenault,  but I can see that we're about a year behind on Sphinx versions and RtD. Still, that import is verifiably wrong. If the next build fails, I'll see if there's something I can do about it.

Edited, please review to make sure I did not screw anything.
Added an interactive graph at the end, keep only one of both.

@huard It seems I can't even import the data assimilation package because it can't find perturb_full_series (and I imagine the others that I wrote). I tried forcing raven version 0.7.8 to no avail, same issue. I was able to run it before though... so could it be on my pavics server instance?

Small update and more info :
This notebook worked on the server (pavics.ouranos.ca jputerlab instance) last week when I worked on and submitted it here for review, so I am pretty sure the server had ravenpy 0.7.8 installed -- either automatically or because I had forced it using:
!conda install -c conda-forge ravenpy>=0.7.8 -y
Today, I uploaded the new version after the commit from @huard (c2eeaaf) on the jupyterlab to test it out. But now at this line:
from ravenpy.utilities.data_assimilation import perturb_full_series, sequential_assimilation, assimilation_initialization
I get an error: cannot import name 'perturb_full_series' from 'ravenpy.utilities.data_assimilation', which I had in version 0.7.7 because the functions just didn't exist yet. But it should work in 0.7.8.
No big deal I thought, I'll just re-force the ravenpy 0.7.8 package using the command from above... but it still fails with the same error.
@tlvu Perhaps there is something I did to revert versions, but I doubt it. Any ideas? Has the server perhaps been reverted to 0.7.7 and blocking me from forcing 0.7.8? It is also probably a "user error - replace user" kind of thing so if you have any idea how this could be fixed I'd be glad to test them! Thanks!

Just as I finished typing my previous comment I thought of the fact that I can test for the ravenpy version...
import ravenpy
ravenpy.__version__

'0.7.5'

So I guess this is the problem, but no idea why I can't supersede it with 0.7.8?

@tlvu Perhaps there is something I did to revert versions, but I doubt it. Any ideas? Has the server perhaps been reverted to 0.7.7 and blocking me from forcing 0.7.8? It is also probably a "user error - replace user" kind of thing so if you have any idea how this could be fixed I'd be glad to test them! Thanks!

@richardarsenault
There has been a full server restart yesterday.  You Jupyter server has reset itself to the original state with raven 0.7.5.
We sometime have to restart for various reasons so do not be surprised next time if your customization seems to vanish.  We also have an "auto kill idle Jupyter server" to avoid resource exhaustion on our system so your Jupyter server could also vanish even if we did not intentionally restart the full system.
However, good news for you, there is currently 3 flavors of the Jupyter server to choose from, in production.  If you select the "beta" flavor right now, you'll get ravenpy 0.7.8.  This "beta" version will most likely become the "current" version next week.
If you do not get the prompt below, shutdown your currently Jupyter server and restart it again.

Thanks @tlvu ! It worked like a charm 👍


Warn on overwrite?

Currently, bundling will happily overwrite existing files. Maybe we should ask for confirmation in the CLI before overwriting for safety, but at the same time that would be a bit of a hassle and maybe not necessary since we would just be overwriting generated files. Should take a look at how other bundlers handle this...

Decided not to add a confirmation before overwriting. The default destination path is unique enough that it should never accidentally overwrite anything other than a generated bundle, and if a user specifies a destination path then a confirmation would just be obnoxious.


can we support IsolatedModules by default?

can we support IsolatedModules by default?
since upgrading to 1.0.1 I'm receiving build errors in my Typescript Next.js project, where I need to have the isolatedModules flag provided in our tsconfig.json
they look like:
Type error: Re-exporting a type when the '--isolatedModules' flag is provided requires using 'export type'.
we did some digging and discovered that simply updating the exports in packages/core/src/index.ts to explicitly export the types as types, will resolve the Type error and allow the module to build again.
for example, in the exports from './constants', we can do this:
export type {
  Chain,
  Config,
  FullConfig,
  MulticallAddresses,
  NodeUrls,
} from './constants'

export {
  ChainId,
  BNB,
  BUSD,
  DEFAULT_SUPPORTED_CHAINS,
  Dai,
  ERC20,
  ERC20Interface,
  ERC20Mock,
  ERC20MockInterface,
  Ether,
  KovanDai,
  KovanEther,
  MultiCall,
  MultiCall2,
  MultiCall2ABI,
  MultiCallABI,
  NATIVE_CURRENCY,
  TestBNB,
  TestBUSD,
} from './constants'

we'll need to apply this export change to all type exports, and I think this will resolve our issue.

I'm not sure I know how you would do that or perhaps I don't understand the question?
I'm happy to make the PR and have it merged, as it would really unblock my team-- but when I tried making the PR I got a permissions issue?
Any idea how we can move this forward?
Your help is greatly appreciated!

@rzadp is this file auto-generated? What do you mean by automatically?

By automation I meant that it is enforced that we do export type for all types - but it's as simple as enabling the IsolatedModules in our config.
https://github.com/TrueFiEng/useDApp/pull/762 is on the way.

@dmrusyniak @jemilezzet The dev version of usedapp containing this change is released on NPM under<EMAIL_ADDRESS>Let me know if that resolves your problems.