c-api: component model

Feature
Expose the component model in the C API for other languages than Rust.
Benefit
Components in all the languages! (on the host side)
Implementation
I have a POC I created here with some feedback I will incorporate.
https://github.com/bytecodealliance/wasmtime/pull/7801
However prior to making further progress on this task, there needs more work on the test infrastructure for the C API. There was a rough plan in Zulip discussed about using GoogleTest and C++.
Alternatives

I am planning on getting to this, but progress will be slow, so if someone else wants to pick this up, I am happy to take it off my plate

Hi,
This is really great to see!
We (NGINX Unit) wrote a WebAssembly language module using the Wasmtime C API last summer.
However when it came to supporting the component model, we discovered this was totally lacking on the C side of things.
In fact Alex actually ended up writing us a Rust based wasm language module to support components.
However being a C guy, I am naturally chomping at the bit to have a NGINX Unit wasm language module in C that supports the component model.
In terms of core functionality is your PoC in a testable state? I.e would it be able to actually run say a WASI 0.2.0 component?
While I am a C programmer, I can only sort of kludge bits of Rust (and maybe C++, C++ and C have really been distinct languages from each other for quite some time) together, but don't really know what I'm doing there...
Cheers,
Andrew

Hey Andrew, glad you're interested in this as well! I have not yet had time to invest in this. I believe my PoC had all the needed bits to invoke a component, but there is more needed (exposing host functions to components for example), and things like resources were entirely unsupported. I have alligator energy here, and will eventually get around to this, starting with a test suite (using Google Test).

run say a WASI 0.2.0 component?

Even once I get the component model working, I don't personally have plans to expose WASI 0.2 in the C API (at least the implementation that Wasmtime provides), we have our own WASI implementation we plug in ourselves (that uses our scheduler, allocator, etc). I'm not sure of the state of WASI p1 in the C API, but we can open a followup ticket after this is completed.

Any update/plan on this? C API is a must-have for components.
Is it possible to enable loading a component as plain WASM module?
I mean: component is just  a WASM module + metadata in special file format.
I  understand that loading component properly requires additional works.
But what is the problem to extract  plain WASM from module file and load it as usual module?

Would my understanding that, without the Component in C APi, it would also not be available into a .NET Host ?
We were interesting to explorer the component model with our .Net application for extensions but we are blocked with a simple Module, not Component

Updates

Unfortunately I will not be able to get to this anytime soon. I don’t know if this is something the Wasmtime core team is considering taking up. You can ask on Zulip.

it would also not be available into a .NET Host

Correct, the component model host bindings for any other language than Rust is blocked on getting this done.

I've been using the C-api for component model for a while (in this branch, starting from #7801).
This works by limiting a lot the set of types allowed in host-provided functions, and needing to declare in advance all the allowed signatures, two very serious limitations.
But I'm finally giving a go to a more general implementation, especially now that, after rebasing, LinkerInstance::func_new no longer needs a Component parameter.


Add user ID tracking

As per https://trello.com/c/kO8rLR3R/963-use-imgix-tracking-parameter

@OliverJAsh also, just noticed, can you update the README with the userId?

@lukechesser I added a test to check this. It does work without user ID.


Add dashbot info

dashbot.logIncoming -> User typed messages
dashbot.logOutgoing -> Watson responses
Message text and UserId required (userID must the same to communicate in the same channel)

Done with latest commit.


Implemented tests for Subscription backend

This change is

Codecov Report

Merging #103 into master will increase coverage by 3.23%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #103      +/-   ##
============================================
+ Coverage     83.85%   87.09%   +3.23%
- Complexity     1236     1326      +90
============================================
  Files           108      108
  Lines          3531     3587      +56
  Branches        152      153       +1
============================================
+ Hits           2961     3124     +163
+ Misses          479      363     -116
- Partials         91      100       +9

Impacted Files
Coverage Δ
Complexity Δ

...g/oscm/rest/subscription/SubscriptionResource.java
100% <ø> (ø)
6 <0> (ø)
:arrow_down:

...g/oscm/rest/subscription/UsageLicenseResource.java
100% <ø> (ø)
5 <0> (ø)
:arrow_down:

...a/org/oscm/rest/identity/OnBehalfUserResource.java
100% <0%> (ø)
6% <0%> (+3%)
:arrow_up:

...rg/oscm/rest/service/TechnicalServiceResource.java
100% <0%> (ø)
12% <0%> (+5%)
:arrow_up:

...in/java/org/oscm/rest/service/ServiceResource.java
100% <0%> (ø)
13% <0%> (+6%)
:arrow_up:

...ain/java/org/oscm/rest/identity/RolesResource.java
100% <0%> (ø)
6% <0%> (+3%)
:arrow_up:

...java/org/oscm/rest/operation/SettingsResource.java
100% <0%> (ø)
12% <0%> (+6%)
:arrow_up:

...c/main/java/org/oscm/rest/event/EventResource.java
100% <0%> (ø)
4% <0%> (+2%)
:arrow_up:

...g/oscm/rest/service/CompatibleServiceResource.java
100% <0%> (ø)
5% <0%> (+2%)
:arrow_up:

.../org/oscm/rest/account/BillingContactResource.java
100% <0%> (ø)
11% <0%> (+5%)
:arrow_up:

... and 12 more


Don't reference DCs in system ks replication settings before they exist (fixes #829)

What this PR does:
Only reference the first DC in the k8ssandra.io/initial-system-replication annotation. This avoids referencing non-existing DCs ahead of time (which doesn't work with Cassandra 4.1). We alter system keyspaces after each DC gets added.
Which issue(s) this PR fixes:
Fixes #829
Checklist

[x] Changes manually tested
[ ] Automated Tests added/updated
[ ] Documentation added/updated
[x] CHANGELOG.md updated (not required for documentation PRs)
[x] CLA Signed:  DataStax CLA

Codecov Report

Merging #833 (047bf93) into main (779eced) will decrease coverage by 0.16%.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@            Coverage Diff             @@
##             main     #833      +/-   ##
==========================================
- Coverage   56.80%   56.64%   -0.16%
==========================================
  Files          91       91
  Lines        9154     9079      -75
==========================================
- Hits         5200     5143      -57
+ Misses       3487     3470      -17
+ Partials      467      466       -1

Impacted Files
Coverage Δ

controllers/k8ssandra/dcconfigs.go
77.19% <ø> (-1.15%)
:arrow_down:

pkg/cassandra/config.go
39.34% <ø> (-11.98%)
:arrow_down:

pkg/cassandra/datacenter.go
68.90% <ø> (ø)

pkg/cassandra/util.go
75.00% <ø> (+8.33%)
:arrow_up:

controllers/k8ssandra/datacenters.go
73.40% <100.00%> (+0.52%)
:arrow_up:

controllers/k8ssandra/schemas.go
75.42% <100.00%> (-0.28%)
:arrow_down:

controllers/k8ssandra/reaper.go
60.60% <0.00%> (-1.82%)
:arrow_down:

controllers/control/k8ssandratask_controller.go
64.28% <0.00%> (-1.79%)
:arrow_down:

controllers/replication/secret_controller.go
65.43% <0.00%> (+1.00%)
:arrow_up:

... and 1 more

Hi, @olim7t @adejanovski, could you pls clarify, is it will be released as 1.5.1?

Hi, @olim7t @adejanovski, could you pls clarify, is it will be released as 1.5.1?

Hi @dnugmanov,
Yes, this will be included in 1.5.1.


UI Simplification: Unlock corners of rectangles and images when resizing

To confirm: how can the user keep ratio then while resizing?
Feature Description
Currently, the ratio of rectangle shapes and images are locked when resizing from the corners.

Alternatives Considered

Additional Context

The user can either press shift or press the lock in selection to lock the ratio.

@aaskedall Currently, the lock in the design panel and the way how the selection box work are not related.
The lock in the Selection panel is for the inputs only and does not influence anything else, it only influences how the user can interact with these specific inputs it's tied to.
Is the suggestion that we should change this? In this case, it would be good to review and change this for all the elements, not just media and rectangles, to make it consistent.

Ah, I did not test this out because I assumed it's how it worked based on using other applications. I think they should be related for all selectable objects. If we do that and turn off ratio lock by default I think that will make the experience a little ore intuitive.

@andrew-askedall Alright, confirming just in case:
If the aspect ratio is locked in the Design panel, it will now mean that resizing from any side of the Selection box (including the edges, not only the corners) will still keep the ratio, is that correct? (Previously corners always kept the ratio but edges not). And if the ratio is unlocked in the design panel, ratio is not kept, no matter where it's resized, corners or edges.

yes, that is correct. I believe this pattern is quite common in other editors. If it's unlocked by default this should not be too difficult. Holding shift while resizing should maintain the ratio of a shape or media element.


[Android] Cannot log screenshot because of Javax.imageio is not available for Android

A sample project for Kotlin does not work correctly on the last step. If you try log a screenshot test app throws an exception
.NoClassDefFoundError: Failed resolution of: Ljavax/imageio/ImageIO; What should I do to resolve the issue?
Thank you

Please inspect your libraries, because the example works for me well without changing anything:


Support Arabic letters in Images Link (Solved)

The URL passed to the kf is not handled if contained arabic letters
So I tried many solutions but the one that worked with me to encode the url with this line
urlName.addingPercentEncoding(withAllowedCharacters: CharacterSet.urlQueryAllowed)
So I think you can support that or add method for it in case there no one for it

no I pass a Valid URL like this one :
https://www.marraa.com/wp-content/uploads/2018/06/٢٠١٨٠٦٢٤_٠٠٢٠١٨.jpg

When you do let url: URL? = URL(string: "https://www.marraa.com/wp-content/uploads/2018/06/٢٠١٨٠٦٢٤_٠٠٢٠١٨.jpg"), you may get a nil.

I already tested it and it's not nil
my code was a collectionView and having data from api
the data is backed fully from the api
but failed to load with kf
only fully English link (that came from the same service for another photo) loaded
And when I made the encoding I mentioned above the link with arabic letters worked

May I have some code snippet or a sample URL string? The link you gave me above results to a nil, if the URL.init(string:) initializer used.

May I have some code snippet or a sample URL string? The link you gave me above results to a nil, if the URL.init(string:) initializer used.

so I made extension for UIImageView as follow :
func setImageWith(_ linkString: String?)  {         guard let linkString = linkString,             let url = URL(string: linkString) else { return }         self.kf.setImage(with: url)      }
Then my using for it as follow :
self.adPictureImageView.setImageWith(adModel.picture)
the adModel.picture is the link above
this made it not to download
the code that worked with me was :
self.adPictureImageView.setImageWith(adModel.picture.addingPercentEncoding(withAllowedCharacters: CharacterSet.urlQueryAllowed))

I believe your code returns early since the guard let url = URL(string: linkString) not pass.

I think it's not the problem since it pass with the English letters and when encoded too


2293 doi reg pending queue

Pull Request Checklist:

[ ] Positive Test Case Written by Dev

[ ] Automated Testing

[ ] Sufficient User and Developer Documentation

[ ] Passing Jenkins Build

[ ] Peer Code review and approval

Positive Test Case

[Enter positive test case here]

This is not ready for PR yet, just putting it here to see what jenkins thinks about it

Build finished.

Build finished.

Build finished.

Build finished.

Build finished.

Closing this PR - @engineSound will have another one ready at the end of the month


fix!: move readiness and healthz endpoints under api path

To avoid conflict with the sentry integration context.res.end can not be used inside getServerSideProps. This can be resolved by moving the endpoints under api which allows us to respond text/plain OK without getting hacky.
refs: LINK-2170
co-pr: https://dev.azure.com/City-of-Helsinki/linkedevents/_git/linkedevents-pipelines/pullrequest/9465

I guess these should be updated as well https://github.com/City-of-Helsinki/linkedregistrations-ui/blob/069bcabb22fe723406be30316d580e3262e0f2f9/jest.config.mjs#L24-L25

LINKEDREGISTRATIONS-UI branch is deployed to platta: https://linkedregistrations-pr172-ui.dev.hel.ninja :rocket::rocket::rocket:

e2e tests result is success for https://linkedregistrations-pr172-ui.dev.hel.ninja :laughing::tada::tada::tada:

I guess these should be updated as well
https://github.com/City-of-Helsinki/linkedregistrations-ui/blob/069bcabb22fe723406be30316d580e3262e0f2f9/jest.config.mjs#L24-L25

Right-o, sonarcloud would still require it's own file I guess. Looks like the old endpoints were at 0% coverage too.


can not read custom fonts

Hi this is a link which client want to read. PdfRenderer cannot read it.
https://igenbucket.s3.ap-south-1.amazonaws.com/1505737193910_1505737193728History-hin-1A.pdf

Any updates on this? I am having the same issue.

I am closing this issue/PR, as it has been migrated to the new repo linked above in the comments. Thank you!


Update Portal meta.json for 10.x

Hi @ivy-rew we received feedback from the Portal team that they want to keep the "ch.ivyteam.ivy.project.portal" groupId for app and guide for LTS 10.x to 10.1 - not only under 10.0.19 like the current config.
Could you help review this PR and give your feedback?
Thank you so much for all!

Hi @ivy-rew we received feedback from the Portal team that they want to keep the "ch.ivyteam.ivy.project.portal" groupId for app and guide for LTS 10.x to 10.1 - not only under 10.0.19 like the current config. Could you help review this PR and give your feedback? Thank you so much for all!

Hi @ivy-rew, @nqhoan-axonivy
Portal team prefers not change groupId and artifactId for LTS10 because it is LTS. Thanks for your support!

fyi @alexsuter


Update VSCode Compile Task problem matcher

#2 introduced .vscode folder for compiling directly in VSCode. Update the problemMatcher attribute to allow for better parsing of error messages.

@rimblas do you have a good problemMatcher that works? The one I have wasn't very useful so for now I've removed. Not I'm still working on #2 so I haven't committed code to github yet. If you have one please paste code in here and I'll add it in later.

I don’t use vsc (there I said it).
But looking here https://ora-00001.blogspot.com/2017/03/using-vs-code-for-plsql-development.html
Morten has one defined: https://gist.github.com/mortenbra/9d7bd2147b02ae8fa2e14dbc38935e5e#file-vscode_tasks-json
Will that help?


MeltanoHub Launch Content

Migrated from GitLab: https://gitlab.com/meltano/hub/-/issues/59
Originally created by @tayloramurphy on 2021-05-25 14:42:12

[x] Slack Announcements

[x] Meltano Slack
[x] Singer Slack
[x] dbt/LO Slack (tools-meltano)

[x] Meltano Blog Post - what is the hub, why are we doing this, what are our plans https://docs.google.com/document/d/1sJ5ZdJjIhk-Q-rSGlZFRNp3tHaUW7qIGxyy6pBfUkFU/edit?usp=sharing

[ ] We should see if any community members are willing to post on HN or Reddit

[x] Twitter Thread with similar content
[x] Email Newsletter

View 21 previous comments from the original issue on GitLab


Setup Authentication and Authorization

Description
In this PR, I created routes for users to signup and login. Both routes returns a token containing the user's Id, role, and roleId. On login, the user's role is verified and saved in the token.
I also added middleware that checks the roles of all users and takes appropriate actions.
I also made changes to the DB structure so running the new migrations will be destructive.
Fixes #23  #3
How Has This Been Tested?
I was able to signup and login with the routes. Will write tests later
Screenshots (if applicable, else remove this line / section)
Checklist:

[x] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have added necessary inline code documentation
[ ] I have added tests that prove my fix is effective and that this feature works
[ ] New and existing unit tests pass locally with my changes

Tests are failing because we've not setup a database for tests. Also codacy appears to be drunk.

Great job so far.
Problem is with that url undefined. It expects a string as shown.
If we merge it, we'll still run into the same problem or more when merging our Develop branch into Master.  And will still be shown when our code is tested after finishing our product.
I know our secrets must remain hidden but I think we somehow have to find a way to resolve it.
Is there no way to rework the code so that we only have
export const sequelize
on that Export instead of
export const sequelize = new....

Tests are failing because we've not setup a database for tests. Also codacy appears to be drunk.

IF you have VS CODE editor you need to install EditorConfig for VS Code then before push you commit changes you need right-click and select format code in edited files to avoid Codeacy issues. lol

Great job so far.
Problem is with that url undefined. It expects a string as shown.
If we merge it, we'll still run into the same problem or more when merging our Develop branch into Master. And will still be shown when our code is tested after finishing our product.
I know our secrets must remain hidden but I think we somehow have to find a way to resolve it.
Is there no way to rework the code so that we only have
export const sequelize
on that Export instead of
export const sequelize = new....

can you set secrets ?

Great job so far.
Problem is with that url undefined. It expects a string as shown.
If we merge it, we'll still run into the same problem or more when merging our Develop branch into Master. And will still be shown when our code is tested after finishing our product.
I know our secrets must remain hidden but I think we somehow have to find a way to resolve it.
Is there no way to rework the code so that we only have
export const sequelize
on that Export instead of
export const sequelize = new....

can you set secrets ?

It comes with most backend work. I could try if its the issue.


[Question] Flagger without Prometheus

Hi there,
I was wondering if I could use flagger without prometheus. Only using webhooks for example.
I have some hard requirements in one environment.

Sure, you can just delete the canaryAnalysis.metrics  and use only the webhooks.

Awesome thank you @stefanprodan !

But what does this test actually do ? It hey doesn't fail.

hey generates traffic, then Flagger looks at the metrics to see if the app returns HTTP errors.

if i'm not using the canaryAnalysis.metrics I won't be able to use all the webhooks ?

No, you can use all webhook types, just that a load test doesn't help much if no metrics are used. A load test could crash the app and Flagger would detect that.


Animated shapes remain after fading away but only in Chrome

Hi there,
I found this little bug/issue while working on a simple to-do app.
While running it on Firefox, the 'task completed' check animated circles after reaching opacity 0 aren't in DOM anymore, where as in chrome, they remain there and the whole screen because clickable and its causes to press that specific check button.
Codepen here

Codepen demo is broken.
Closing the issue.

Just a note in case anyone else read this. If the animation is purely visual, I would suggest always setting pointer-events: none to it in CSS. It's cheaper and easier then removing DOM elements (up to a certain limit of course).


WIP - Add Java support for metacp

This PR extends metacp to support Java class files. Opening PR now to get feedback on the diff (which should be mostly polished) but still need to clean up the commit history.

Alright, I'm done reviewing.

Super impressive work! Lots of value, which brings up lots of questions. Very happy about the momentum behind the project these days 😍

Superseded by https://github.com/scalameta/scalameta/pull/1361.

@ShaneDelmore I failed to reproducing the missing newline at EOF in scalafmt when trying to fix it. I've encountered the bug in the wild but in my tests scalafmt seems to insert a newline at EOF when it's missing. Do you have a minimal repro?

@olafurpg I don’t have a reproduction, but if I see it again I will file one.  I may just be remembering back from before it was implemented.


AttributeError: module 'pycolmap' has no attribute 'Rigid3d'

Hi !
my environment :
win11
anaconda
python=3.10.4
firstly, my pycolmap version is 0.4.0, i meet this error, then, i uninstalled it and use conda install -c conda-forge pycolmap.
the pycolmap version is 3.10.0, also has this error.
How can I slove it?

Please update pycolmap:
pip install pycolmap==0.6.1

thank you！It‘s ok


FEAT - Add WeightedGroupL2_plus_L1 penalty

Context of the PR
This PR adds a sparse weighted group L2 penalty enabling for instance fitting a sparse group Lasso estimator.
Closes #198
Contributions of the PR

add WeightedGroupL2_plus_L1 penalty

Checks before merging PR

[x] added documentation for any new feature
[ ] added unittests
[ ] edited the what's new

For unittest, we need to add support of fix-point distance strategy to build working sets in the GroupBCD solver (later, add it to GroupProxNewton)

Added thanks to https://github.com/scikit-learn-contrib/skglm/pull/267


Make modal hack docs reflect implementation of hack in demo

The implementation of the hack for setting the ApplicationRef in the modal demo was different from the documentation of how this should be done. This commit fixes this issue and brings the documentation inline with the implementation of the hack.

@tzierbock thank you for your help
but I am deprecating this hack with new dynamic component loader implementation:
https://github.com/valor-software/ng2-bootstrap/pull/1318

I hope you will like how components will look like: (this is popover)
https://github.com/valor-software/ng2-bootstrap/pull/1318/files#diff-47917cb7752d9ff4c3eddfa99fba920c
:D


Saving untitled document corrupts MRU order

I haven't been able to nail down solid repro steps for this yet, but I've seen it at least twice...
Symptom 1: messages like /_brackets_26272468/Untitled-1 duplicated in mru list appear in the console
Symptom 2: unsaved changes that you choose to drop when switching projects sometimes come back if you return to the same project without restarting Brackets in between

Found repro steps:

Open any file in your project in the working set
Create two new untitled files in a row (working set will now show: , Untitled-1, Untitled-2)
Switch back to Untitled-1
Save the file anywhere

Result: ...Untitled-1 duplicated in mru list message in console

Press Ctrl-Tab -- switches to Untitled-2 as expected (but prints warning again)
Press Ctrl-Tab again -- switches to the file formerly called Untitled-1 as expected
Press and hold Ctrl, and hit Tab twice -- gets stuck on Untitled-2 and does not continue to the 3rd MRU entry, )... and prints warning a 3rd time

This doesn't repro in 1.1.  That coupled with the involvement of MRU stuff makes me suspicious that PR #10327 broke something -- CC @JeffryBooher
Adding to 1.3 milestone since this is a recent regression and not that hard to hit


Allow server to restart after stopping

A server should be allowed to auto-restart after being told to stop.
This should occur after a delay (if configured) and should be disabled.
This is independent of the auto-start feature, which only happens when pufferd starts.

Hey. How can i make my Sponge server restart after stopping or crashing? There is no auto start property in server .json file.
PufferPanel version v1.2.4


Unable to run gulp serve watch

https://github.com/Swimlane/angular-systemjs-seed/blob/master/build/tasks/watch.js#L9
There is no build-dev task in your gulpfile, hence gulp task such as watch or serve will fail.
How can I actually get this up and running?

Fixed


PieChart flash with animation

When I used PieChart with animation, there are times when the chart is seen for a moment and then it does the animation. Gives the sensation of flash. It is random, sometimes it is done well and sometimes it is not.
My config:
  chart.animateY(500);
  chart.setHoleColor(new Color("transparent"))
  chart.setTransparentCircleRadius(10)
  chart.setDrawEntryLabels(false)
  chart.setHoleRadius(50)

@vallemar you need to share a full repro sample

My code:
<template>
<GridLayout width="160">
            <PieChart
              id="chart"
              width="160"
              height="160"
              @loaded="loadChart"
            />
          </GridLayout>
</template>

....script

export type CurrencyChart = {
  label: string;
  value: number;
  color: string;
  percentage: string;
  stake: string;
};

const transparentCircleRadius = 45;
const drawEntryLabels = false;
const holeRadius = 70;

export const drawPieChart = (
  currencies: Currency[],
  chart: PieChart
) => {
  chart.animateY(500);
  chart.setHoleColor(new Color("transparent"));
  chart.setTransparentCircleRadius(transparentCircleRadius);
  chart.setDrawEntryLabels(drawEntryLabels);
  chart.setHoleRadius(holeRadius);

 const data = new ObservableArray(
        new Array(5).fill(0).map(function (v, i) {
            const value = Math.random() * 1;
            return { index: i, value: value , color: getColor(value.toString()), };
        })
    );

  const sets = [];
  const set = new PieDataSet(data, "Dataset Label", "value");
  set.setColors(data.map((d) => d.color));
  set.setDrawValues(true);
  sets.push(set);

  // Create a data object with the data sets
  const pd = new PieData(sets);

  // Set data
  chart.setData(pd);
  chart.highlightValues(null);
  chart.invalidate();
  return data;
};

function getColor(stringInput: string) {
  let stringUniqueHash = [...stringInput].reduce((acc, char) => {
    return char.charCodeAt(0) + ((acc << 5) - acc);
  }, 0);
  return `hsl(${stringUniqueHash % 360},70%,70%)`;
  /*  return `hsla(${~~(360 * (Math.random() * Math.random() * Math.random()))},70%,70%,0.8)`*/
}

@vallemar your code is weird. You animate before setting the data. The consequence would indeed be a flash during animation

I don't think I get it. Am I animating the graph before the data? I'm not sure where this code would go: chart.animateY(500);

@vallemar move the chart.animateY(500); to the end of drawPieChart . Where is drawPieChart called by the way? What s the difference with loadChart?

@farfromrefug Ok I'll move this, I thought the setting was before .setData. The loadChart method makes the server call and when I have the data I call drawPieChart.
Anyway it seems that the flash does not appear now. I've put another pieChart as background and it doesn't seem to happen now. It usually happened on the first load of the chart.
I'm going to give you access to the project in case you want to look at it.

@vallemar see animateY as animate in N. The animation starts right away.
So it is fixed now ?

@farfromrefug  Oh, okay! then that must be the problem yeah! if it has stopped happening when putting the other graphic in the background! Thank you!

I think this problem has to be opened, the same happens with a BarChart, if you uncomment these lines of code when you start the app you will see a flash
https://github.com/vallemar/freqtrade-app/blob/master/app/utils/charts/BarChart.ts#L55

Sorry I have moved animateY and animateX above setData and it no longer does the flashing, disregard the comment above.


Permission Denied Error When Configuring test1-glb_15min.sh

I am encountering a "Permission Denied" error when attempting to configure or run the test1-glb_15min.sh test case for the CaMa-Flood model. also showing some files are missing which is not attached to the downloaded folder(log_CaMa.txt)

GitHub repository only contains "codes". To execute simulation, please download package from the CaMa-Flood webpage.

Thanks for the response @bigasmountain it works fine, I just want to inquire whether I will be able to calculate 100,75,... flood return periods for both depth and inundation


add: adithya sharma

Description

Related Issues

Changes Proposed

Checklist

[x] I have read and followed the Contribution Guidelines.
[x] All new and existing tests passed.
[x] I have updated the documentation to reflect the changes I've made.
[x] My code follows the code style of this project.
[x] The title of my pull request is a short description of the requested changes.

Screenshots
Note to reviewers

Hi @Adithyasharma1162 , thanks for your interest in contributing to DevDisplay!
Before we proceed, we noticed you adding your profile on #DevDisplay but forgot to star the repo. It's like making an amazing sandwich and forgetting to take a bite! -- I think our repo's feeling a little lonely without your 🤗 So, be sure to star the DevDisplay GitHub Repository. 🌟
💌 This is a mandatory step for all contributors. & It's is the first priority of contribution in DevDisplay 💌
🚀 We look forward to your contribution once these steps are completed. 🚀
Thanks 👍🏻


Add lervag/vimtex

Checklist:

[ ] The plugin is specifically built for Neovim.
[x] The lines end with a .. This is to conform to awesome-list linting and requirements.
[x] It's not already on the list.
[x] If it's a colorscheme, it supports treesitter syntax.
[x] The title of the pull request is Add `username/repo`  when adding a new plugin.

It is not specifically built for Neovim but it works seamless on it :)

Sorry but closing this because it's not specific to neovim.


docs: Move docs/build.sh to bazel

Commit Message: docs: Move docs/build.sh to bazel
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue] Fix #13229
[Optional Deprecated:]
[Optional API Considerations:]

ping back when this is ready for review!

/retest

coverage is here https://storage.googleapis.com/envoy-pr/15229/tooling/index.html

ping back when this is ready for review!

@alyssawilk ping!
@htuch this is the final part in the move to building docs with bazel - should be ready for review!

Still looks like a draft?  per multiple requests I'm going to switch our tooling to ignore them, so maybe de-draft?
I'll let @htuch take a look either way since he's reviewer and Matt's out.

Still looks like a draft?

oops - done


tomli not found in 3.10

I ran a pre-commit autoupdate on my project and now pre-commit won't run. The log is below:
auto-walrus..............................................................Failed
- hook id: auto-walrus
- exit code: 1

Traceback (most recent call last):
  File "/home/karlwooster/.cache/pre-commit/repo1ulum2zu/py_env-python3.10/bin/auto-walrus", line 5, in <module>
    from auto_walrus import main
  File "/home/karlwooster/.cache/pre-commit/repo1ulum2zu/py_env-python3.10/lib/python3.10/site-packages/auto_walrus.py", line 17, in <module>
    import tomli as tomllib
ModuleNotFoundError: No module named 'tomli'
Traceback (most recent call last):
  File "/home/karlwooster/.cache/pre-commit/repo1ulum2zu/py_env-python3.10/bin/auto-walrus", line 5, in <module>
    from auto_walrus import main
  File "/home/karlwooster/.cache/pre-commit/repo1ulum2zu/py_env-python3.10/lib/python3.10/site-packages/auto_walrus.py", line 17, in <module>
    import tomli as tomllib
ModuleNotFoundError: No module named 'tomli'
Traceback (most recent call last):
  File "/home/karlwooster/.cache/pre-commit/repo1ulum2zu/py_env-python3.10/bin/auto-walrus", line 5, in <module>
    from auto_walrus import main
  File "/home/karlwooster/.cache/pre-commit/repo1ulum2zu/py_env-python3.10/lib/python3.10/site-packages/auto_walrus.py", line 17, in <module>
    import tomli as tomllib
ModuleNotFoundError: No module named 'tomli'
Traceback (most recent call last):
  File "/home/karlwooster/.cache/pre-commit/repo1ulum2zu/py_env-python3.10/bin/auto-walrus", line 5, in <module>
    from auto_walrus import main
  File "/home/karlwooster/.cache/pre-commit/repo1ulum2zu/py_env-python3.10/lib/python3.10/site-packages/auto_walrus.py", line 17, in <module>
    import tomli as tomllib
ModuleNotFoundError: No module named 'tomli'
Traceback (most recent call last):
  File "/home/karlwooster/.cache/pre-commit/repo1ulum2zu/py_env-python3.10/bin/auto-walrus", line 5, in <module>
    from auto_walrus import main
  File "/home/karlwooster/.cache/pre-commit/repo1ulum2zu/py_env-python3.10/lib/python3.10/site-packages/auto_walrus.py", line 17, in <module>
    import tomli as tomllib
ModuleNotFoundError: No module named 'tomli'

I'm not really sure how to debug this in a pre-commit environment.
I'm using poetry and my python version is 3.10.12.

thanks @WoosterInitiative for the report
@cclauss do you know how to fix this?  https://github.com/MarcoGorelli/auto-walrus/pull/36/files

Create a new release to PyPI that contains the changes in

#36

Perhaps a test in CI for different versions of Python? Would have caught this before release.

@WoosterInitiative Are you able to create a pull request that would add Python 3.8 and 3.10 to our GitHub Actions?

ah thanks, looks like I merged the PR but forgot to cut a release 🤦
There's a new release now, could you try it out please? thanks!
(I'm still amazed that people are actually using auto-walrus 😄 I'm tempted to try rewriting it in Rust and call it auto-wal.rs 🦀 Then it'll be just as silly as before, but blazingly fast)

i presume it works  now, so closing, but lmk if there's still an issue pls

@WoosterInitiative Can you please run pipx run --python 3.10 auto-walrus -h and tell us what the result is?


Broken link

404: https://www.visualstudio.com/explore/msft-android-emulator-vs
200: https://www.visualstudio.com/vs/msft-android-emulator/

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

ID: 0546ef4d-4237-66c4-d7b0-f86b57dc91b3
Version Independent ID: 85fa9e0e-cf82-e63b-2557-c3869a3888b4
Content: Visual Studio Android Emulator - Xamarin
Content Source: docs/android/deploy-test/debugging/visual-studio-android-emulator.md
Service: unspecified
GitHub Login: @mgmclemore
Microsoft Alias: mamcle

That's for reporting this. A fix for this will be published shortly.


Build 34 error.

http://pastie.org/pastes/10846019/text?key=vtu1sbtnc3qahdj0bm8wa

Now getting spammed with http://pastie.org/pastes/10846021/text?key=lvf0qi8mtgkloj2zzzeww

Nvm: looks like buycraft is down.

Not much we can do about transient service failures. This usually clears itself up, however.


🍒 Cherry pick PR #4358 to staging 🍒

🍒 Cherry pick https://github.com/Expensify/Expensify.cash/pull/4358 to staging 🍒

🚀 Deployed to staging         in version: 1.0.82-1🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

🚀 Deployed to production         in version: 1.0.82-7🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


clarify CLI usage hints about CSV values

resolves #1048

@plorenz handful of usage hint tweaks here to correct a common point of confusion about valid input for options that expect comma-separated values as a string argument


id="identifiers" in HTML

Describe the issue
I now see id="identifiers", which clashes with document IDs:
https://github.com/ietf-wg-jsonpath/draft-ietf-jsonpath-base/issues/291
(I first mentioned  this in https://github.com/ietf-tools/xml2rfc/issues/875#issuecomment-1302606581 -- that issue is otherwise unrelated.)
Code of Conduct

[X] I agree to follow the IETF's Code of Conduct

id="identifiers" is used to identify the document header section.
https://www.rfc-editor.org/js/metadata.min.js uses this id and there are CSS rules specific to this id.
I don't see a good solution to this issue other than a xml2rfc warning because,

Changing the header id will require regenerating all v3 HTML RFCs.
New id="identifiers" is defined by authors in the XML file, so xml2rfc cannot change that.

I'm not sure I follow.
This bug only affects documents where the author has used an anchor "identifiers".
So there should not be an issue with existing RFCs.
For new documents (I-D or RFC), this "system" id should be chosen so that it doesn't conflict with "user" ids in the XML.
4NtsVrQEvSIvP6PsBImQocc8 would work for me.  Maybe there is a better one.
Since existing documents use id="identifiers" in a specific context, even a shared CSS file shouldn't have a problem.

(And I'm trying to find the place where specific anchor prefixes are reserved....)

(And I'm trying to find the place where specific anchor prefixes are reserved....)

https://datatracker.ietf.org/doc/html/rfc7992#section-6.5

I'm not sure I follow.
This bug only affects documents where the author has used an anchor "identifiers". So there should not be an issue with existing RFCs. For new documents (I-D or RFC), this "system" id should be chosen so that it doesn't conflict with "user" ids in the XML. 4NtsVrQEvSIvP6PsBImQocc8 would work for me. Maybe there is a better one. Since existing documents use id="identifiers" in a specific context, even a shared CSS file shouldn't have a problem.

The issue with existing RFCs is they have the following JavaScript:
<script src="https://www.rfc-editor.org/js/metadata.min.js"></script>

metadata.min.js has logic based on identifiers id.
So I think any changes to this will require regenerating RFCs.

The JS could look whether there is a 4NtsVrQEvSIvP6PsBImQocc8, and only if that is not there, use identifiers.


Revert boat physics

1.6 completely destroyed boat physics, which should be reverted. 1.5's boat physics are not great, but at least aren't a disaster.

Some notes from beta testing on boat issues caused by this:
Frost:
riding them is kinda gitchly, you stop randomly
and when you exit, you teleport back where you first entered the boat
it also seems like i can't break a boat i placed down


Fix some tests

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (7044f92) to head (108ccec).

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #213   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           33        33
  Lines          634       634
  Branches        41        41
=========================================
  Hits           634       634

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


added phar info & link to README.md

It should be downloaded as composer package, as in README: https://github.com/symplify/easy-coding-standard-prefixed
Link to the repository would be better

hmm but if someone, like me, only wants to have the phar file? it's easy to miss in the prefixed repo.
to keep it simple a call of wget https://github.com/symplify/easy-coding-standard-prefixed/blob/master/ecs.phar is easier than to require it via composer.
what about:

Install
Regular Version
composer require symplify/easy-coding-standard --dev

Prefixed Version
composer require symplify/easy-coding-standard-prefixed --dev

Phar file Version
ecs.phar is also included in the prefixed version
wget https://github.com/symplify/easy-coding-standard-prefixed/blob/master/ecs.phar

Standalone phar can report false positive.
Without access to autoload file, the coding standard night not work correctly.
One way or another, the phar related install should be in repository, where the phar is. So are the tests for it, etc.

i run phpstan, phpcsfixer and phpcs as phar version which work really fine.
so what is the text you want to see in this readme? or should i close the MR?

Thanks


All releases/podspecs after 2.0.2 haven't been pushed to cocoapods

Despite of changes to podspec files for all releases(after 2.0.2) have been updated in repository it seems that they haven't been pushed to cocoapods master spec repo so they aren't available to be used through Cocoapods
Latest podspec that is available right now - https://github.com/CocoaPods/Specs/blob/master/Specs/9/2/2/Sourcery/2.0.2/Sourcery.podspec.json

I pushed 2.1.1 just now

hey @krzysztofzablocki ,
would be nice for you to push 2.1.2 as well. I suspect you need to be an owner to do the push to blob in cocoapods (i.e. execute pod trunk push NAME.podspec)? Please 🐻 in mind that recently CocoaPods have reset all of the access keys due to RCE vulnerability.


[FEATURE REQUES] acme-client-portable as library (libacp?)

Hello,
I've studied acme-client-portable sources and noticed that it could be a very useful library to be used in other languages like Python, JS, Pascal etc.
So, what do you think about to refactore acme-client-portable in a near future to make it a library? :-) With the library in hands, the acme-client-portable tool could use it. The curl tool follows a similar approach, it uses its libcurl library, and the new features are implemented in the library.

It's pretty straightforward to library-ise (see main.c), but I don't have a use case for that, so it's unlikely to happen without somebody else doing so. ;)

Indeed. I totally agree!


I can start. install WSL error Wsl/0x80070422 (still not working after tried  sc.exe config LxssManager start=auto

Windows Version
Microsoft Windows [Version 10.0.22631.3374]
WSL Version
<IP_ADDRESS>
Are you using WSL 1 or WSL 2?

[X] WSL 2
[ ] WSL 1

Kernel Version
No response
Distro Version
No response
Other Software
WSL version: <IP_ADDRESS>
Kernel version: <IP_ADDRESS>-2
WSLg version: 1.0.60
MSRDC version: 1.2.5105
Direct3D version: 1.611.1-81528511
DXCore version: 10.0.25131.1002-220531-1700.rs-onecore-base2-hyp
Windows version: 10.0.22631.3374
Repro Steps

Expected Behavior
it should work
Actual Behavior
see the screenshot above
Diagnostic Logs
No response

collect-wsl-logs.zip
It was recovered somehow (more interestingly, my bluetooth got disabled / recovered  with wsl at the same time).

WslLogs-2024-04-23_20-31-16.zip


Calling "complete()" in execute workflow step takes longer than 3 seconds to finish when running in AWS Lambda

Reproducible in:
pip freeze | grep slack
python --version
sw_vers && uname -v # or `ver`

The slack_bolt version
slack-bolt==1.9.0
slack-sdk==3.10.1
Python runtime version
Python 3.8.11
OS info
Lambda Python 3.8
Steps to reproduce:

Code below is configured as Lambda function with API gateway in front.
Execute the workflow.

import os
import logging
from slack_bolt import App
from slack_bolt.workflows.step import WorkflowStep
from slack_bolt.adapter.aws_lambda import SlackRequestHandler

logging.getLogger().setLevel(logging.INFO)

app = App(
    token=os.environ.get("SLACK_BOT_TOKEN"),
    signing_secret=os.environ.get("SLACK_SIGNING_SECRET"),
    process_before_response=True,
)

def edit(ack, step, configure):
    ack()

    blocks = [
        {
            "type": "input",
            "block_id": "task_name_input",
            "element": {
                "type": "plain_text_input",
                "action_id": "name",
                "placeholder": {"type": "plain_text", "text": "Add a task name"},
            },
            "label": {"type": "plain_text", "text": "Task name"},
        },
        {
            "type": "input",
            "block_id": "task_description_input",
            "element": {
                "type": "plain_text_input",
                "action_id": "description",
                "placeholder": {"type": "plain_text", "text": "Add a task description"},
            },
            "label": {"type": "plain_text", "text": "Task description"},
        },
    ]
    configure(blocks=blocks)

def save(ack, view, update):
    ack()

    values = view["state"]["values"]
    task_name = values["task_name_input"]["name"]
    task_description = values["task_description_input"]["description"]

    inputs = {
        "task_name": {"value": task_name["value"]},
        "task_description": {"value": task_description["value"]}
    }
    outputs = [
        {
            "type": "text",
            "name": "task_name",
            "label": "Task name",
        },
        {
            "type": "text",
            "name": "task_description",
            "label": "Task description",
        }
    ]
    update(inputs=inputs, outputs=outputs)

def execute(step, complete, fail):
    try:
        inputs = step["inputs"]
        task_name = inputs["task_name"]
        complete(outputs={})
        logging.info(f"{task_name} complete!!")
    except Exception as err:
        fail(error={"message": f"Error: {err}"})

def handler(event, context):
    """Lambda handler"""
    # Create a new WorkflowStep instance
    ws = WorkflowStep(
        callback_id="alert",
        edit=edit,
        save=save,
        execute=execute,
    )
    # Pass Step to set up listeners
    app.step(ws)

    slack_handler = SlackRequestHandler(app=app)
    result = slack_handler.handle(event, context)
    return result

Expected result:
"complete()" call in "execute" step should finish in less than 3 seconds so Slack doesnt retry the step.
Things I have tried:

Have put timers around the "complete()" and can see that it takes 3+ seconds.
Enabled provisioned concurrency to rule out cold starts.
Provisioned more memory for the function.

Is there any way around this?
Actual result:
The "complete()" call takes longer than 3 seconds for the first event but eventually succeeds. Because the complete call takes longer than 3 seconds Slack resends the event and the second run fails with:
"ERROR:slack_bolt.App:Failed to run listener function (error: The request to the Slack API failed. (url: https://www.slack.com/api/workflows.stepFailed)
The server responded with: {'ok': False, 'error': 'trigger_exchanged'})"
Requirements
Please read the Contributing guidelines and Code of Conduct before creating this issue or pull request. By submitting, you are agreeing to those rules.

Thanks for writing in @mcroft246 - Let me try to reproduce what you're seeing and get back to you on this issue.

@mcroft246 - Would the workaround using a lazy listener mentioned here work for your situation?

Thanks for the response. I saw this example of a lazy listener for workflow steps however it seems to be the "complete()" call that takes up the time so i was unsure of how the lazy listener would assist as that call is still in the execute step?

Hi @mcroft246, the following example should work for you!
def run_the_step(logger: logging.Logger, step: dict, complete: Complete, fail: Fail):
    logger.info("execute called")
    try:
        complete(
            outputs={
                "taskName": step["inputs"]["taskName"]["value"],
                "taskDescription": step["inputs"]["taskDescription"]["value"],
                "taskAuthorEmail": step["inputs"]["taskAuthorEmail"]["value"],
            }
        )
    except Exception as err:
        fail(error={"message": f"Something wrong! {err}"})

@copy_review_step.execute(lazy=[run_the_step])
def execute():
    # If you don't have anything to do within 3 seconds,
    # Just having `pass` or logging works here
    pass

Hi @seratch. I tested the above and it works. Thanks for your help!

Hi @seratch, how do I import @copy_review_step?

Hi Seratch, we had same issue, but the solution is not working because we used different workflow builders like below, could please provide an alternative way to overcome the issue
the primary issue is with ""copy_review_step = WorkflowStep.builder("copy_review")"", when we used the copy review step the Workflow builder steps trying to find execute command but changed from "execute " to "kendra_step_execute" that is not in workflow builder step.
The execute step are in Kendrastep.py we are importing in this module
from .kendrastep import *
def run():
"""
Entry and starting point for execution of the slack app steps execution where we configure the steps with ids
and starts the slack app in socket mode to access across the network.
:return: None
"""
secret_string = SecretManagerProxy.get_secrets()
SLACK_BOT_TOKEN = secret_string['SLACK_BOT_TOKEN']
SLACK_APP_TOKEN = secret_string['SLACK_APP_TOKEN']
helpcentral_step = WorkflowStep(
    callback_id=os.environ.get('HELPCENTRAL_STEP_ID'),
    edit=helpcentral_step_edit,
    save=helpcentral_step_save,
    execute=helpcentral_step_execute,
)

kendra_step = WorkflowStep(
    callback_id=os.environ.get('KENDRA_STEP_ID'),
    edit=kendra_step_edit,
    save=kendra_step_save,
    execute=kendra_step_execute,
)
bot_app = App(token=SLACK_BOT_TOKEN)
bot_app.step(helpcentral_step)
bot_app.step(kendra_step)
bot_app.event("app_mention")(app_mention)
handler = SocketModeHandler(bot_app, SLACK_APP_TOKEN)
handler.start()

Hi Seratch, we ran into similar issue were we called two API's using web socket connection like below in AWS Kubernetes The init.py consist of below code
from slack_bolt.workflows.step import WorkflowStep
from secret import SecretManagerProxy
from .2_step import *
from .1_step import *
from slack_bolt import App
from slack_bolt.adapter.socket_mode import SocketModeHandler
from logger import log
from logger.cloudwatchlogging import cloudwatches_logstream
def app_mention(body, say):
pass
def run():
"""
Entry and starting point for execution of the slack app steps execution where we configure the steps with ids
and starts the slack app in socket mode to access across the network.
:return: None
"""
secret_string = SecretManagerProxy.get_secrets()
SLACK_BOT_TOKEN = secret_string['SLACK_BOT_TOKEN']
SLACK_APP_TOKEN = secret_string['SLACK_APP_TOKEN']
1_step = WorkflowStep(
    callback_id=os.environ.get('1_STEP_ID'),
    edit=1_step_edit,
    save=1_step_save,
    execute=1_step_execute,
)

2_step = WorkflowStep(
    callback_id=os.environ.get('2_STEP_ID'),
    edit=2_step_edit,
    save=2_step_save,
    execute=2_step_execute,
)
bot_app = App(token=SLACK_BOT_TOKEN)
bot_app.step(1_step)
bot_app.step(2_step)
bot_app.event("app_mention")(app_mention)
handler = SocketModeHandler(bot_app, SLACK_APP_TOKEN)
handler.start()

once we receive the API call the below 2_step.py  code triggered
def 1_step_execute(step, complete, fail):
"""
this step for executing the step code below once edit and save step completed
:param step: receiving step configuration
:param complete: completing the step
:param fail: hadling failures
:return: None
"""
details = {
"question": step["inputs"]["question"]["value"],
"noanswer": step["inputs"]["noanswer"]["value"],
"channel": step["inputs"]["channel"]["value"],
"accountid": step["inputs"]["accountid"]["value"],
"category": step["inputs"]["category"]["value"],
"environment": step["inputs"]["environment"]["value"],
}
try:
    answer = getAnswer(details)

except Exception as error:
    if str(error).find(" The character count must be less than or equal to 1000.")!=-1:
        answer = "Please refine your Question to a more basic format. Use Additional Details to add more information"
    else:
        log.info("Error_character", error)
        answer = error

cloudwatches_logstream(
    {"timestamp": int(round(time.time() * 1000)),
     "message": str({"question": details['question'],
                     "channel_name": details['channel'], "account_id": details['accountid'],
                     "category": details['category'], "environment": details['environment']})})
outputs = {"answer": answer}

try:
    start = time.time()
    #log.info("Complete step start time" + str(start))
    complete(outputs=outputs)
    end = time.time()
    #log.info("Complete step end time" + str(end))
    #time_taken = end - start
    time_taken = start - end
    pass
except Exception as e:
    log.info("Error in complete step:- " + str(e))

@copy_review_step.execute(lazy=[run_the_step])
def execute():
# If you don't have anything to do within 3 seconds,
# Just having pass or logging works here
pass


Update Product API Definitions

Purpose
Related 4.0.0 PR : https://github.com/wso2/docs-apim/pull/3970
Goals

Describe the solutions that this feature/fix will introduce to resolve the problems described above

Approach

Describe how you are implementing the solutions. Include an animated GIF or screenshot if the change affects the UI (email<EMAIL_ADDRESS>to review all UI text). Include a link to a Markdown file or Google doc if the feature write-up is too long to paste here.

User stories

Summary of user stories addressed by this change>

Release note

Brief description of the new feature or bug fix as it will appear in the release notes

Documentation

Link(s) to product documentation that addresses the changes of this PR. If no doc impact, enter “N/A” plus brief explanation of why there’s no doc impact

Training

Link to the PR for changes to the training content in https://github.com/wso2/WSO2-Training, if applicable

Certification

Type “Sent” when you have provided new/updated certification questions, plus four answers for each question (correct answer highlighted in bold), based on this change. Certification questions/answers should be sent to<EMAIL_ADDRESS>and NOT pasted in this PR. If there is no impact on certification exams, type “N/A” and explain why.

Marketing

Link to drafts of marketing content that will describe and promote this feature, including product page changes, technical articles, blog posts, videos, etc., if applicable

Automation tests

Unit tests

Code coverage information

Integration tests

Details about the test cases and coverage

Security checks

Followed secure coding standards in http://wso2.com/technical-reports/wso2-secure-engineering-guidelines? yes/no
Ran FindSecurityBugs plugin and verified report? yes/no
Confirmed that this PR doesn't commit any keys, passwords, tokens, usernames, or other secrets? yes/no

Samples

Provide high-level details about the samples related to this feature

Related PRs

List any other related PRs

Migrations (if applicable)

Describe migration steps and platforms on which migration has been tested

Test environment

List all JDK versions, operating systems, databases, and browser/versions on which this feature/fix was tested

Learning

Describe the research phase and any blog posts, patterns, libraries, or add-ons you used to solve the problem.

@msm1992 , please apply the new review comment changes to 4.0.0 as well.

@msm1992 , please make sure that these changes are in 400 as well


Initial Binning fails

Hi there,
I just tried Nanophase, both with one of my datasets and with the example dataset.
The assembly using flye --meta works fine, but the pipeline keeps terminating at the initial binning step. The logfile of MetaBat2 shows only this:
MetaBAT 2 (v2.12.1) using minContig 2500, minCV 1.0, minCVSum 1.0, maxP 95%, minS 60, and maxEdges 200.
I tried the version 0.2.2 and 0.2.3 but both versions did not work with mine datasets or the example dataset.
The NanoPhase check shows this information:
Check software availability and locations
The following packages have been found
#package             location
flye                 /home/xxx/anaconda3/envs/nanophase0.2.2/bin/flye
metabat2             /home/xxx/anaconda3/envs/nanophase0.2.2/bin/metabat2
maxbin2              /home/xxx/anaconda3/envs/nanophase0.2.2/bin/run_MaxBin.pl
metawrap             /home/xxx/anaconda3/envs/nanophase0.2.2/bin/metawrap
checkm               /home/xxx/anaconda3/envs/nanophase0.2.2/bin/checkm
racon                /home/xxx/anaconda3/envs/nanophase0.2.2/bin/racon
medaka               /home/xxx/anaconda3/envs/nanophase0.2.2/bin/medaka
polypolish           /home/xxx/anaconda3/envs/nanophase0.2.2/bin/polypolish
POLCA                /home/xxx/anaconda3/envs/nanophase0.2.2/bin/polca.sh
bwa                  /home/xxx/anaconda3/envs/nanophase0.2.2/bin/bwa
seqtk                /home/xxx/anaconda3/envs/nanophase0.2.2/bin/seqtk
minimap2             /home/xxx/software/ont-guppy/bin/minimap2
BBMap                /home/xxx/anaconda3/envs/nanophase0.2.2/bin/BBMap
parallel             /home/xxx/anaconda3/envs/nanophase0.2.2/bin/parallel
perl                 /home/xxx/anaconda3/envs/nanophase0.2.2/bin/perl
samtools             /home/xxx/anaconda3/envs/nanophase0.2.2/bin/samtools
gtdbtk               /home/xxx/anaconda3/envs/nanophase0.2.2/bin/gtdbtk
fastANI              /home/xxx/anaconda3/envs/nanophase0.2.2/bin/fastANI
blastp               /home/xxx/anaconda3/envs/nanophase0.2.2/bin/blastp
All required packages have been found in the environment. If the above certain packages integrated into nanophase were used in your investigation, please give them credit as well :)
grep: warning: stray \ before /
Warning: [flye metabat2 maxbin2 metawrap checkm racon medaka polypolish POLCA bwa seqtk BBMap parallel perl samtools gtdbtk fastANI blastp minimap2] has not been installed in the [nanophase] env. Strongly recommend intalling all packages in the nanophase env, or it may result in a failure
This message is confusing, since the required packages are installed and found, but the pipeline keeps warning about missing software.
Anyway, I would love to test your pipeline. Please let me know, if i can provide any additional information for this issue.

Hi, could you provide the command that you used?

Sure:
For the example dataset, I used this command:
nanophase meta -l lr.fa.gz -t 24 -o ont-nanophase-out
for my own datasets, i used the same command, bud modified the output folder, obviously.

I guess the confusing message you mentioned before is due to an installation issue. for example, the name of conda env should be nanophase0.2.2, but somehow, as I can see from the log file, you activated nanophase env using a command like conda activate nanophase, but the nanophase command invoked was under the  nanophase0.2.2 env. Because they are only warning messages, so no need to worry about this.
Before I can identify the potential issues, could you use the following command (after activation of the nanophase package) to see what exactly has happened for metabat binning:
metabat2 -t 16 -i ont-nanophase-out/01-LongAssemblies/assembly.fasta -o ont-nanophase-out02-LongBins/INITIAL_BINNING/metabat2/metabat2-bins/bin -a ont-nanophase-out/02-LongBins/INITIAL_BINNING/metabat2/metabat2_abun.txt --cvExt

Thanks for your answer.
I performed the analysis again, using the correct environments for nanophase 0.2.2 and nanophase 0.2.3 but still got the same error. The metabat2 command you suggested results in no additional results, but gives a "segmentation fault".

It is weird for me, I can't reproduce this error using the example dataset. I would suggest removing the whole package of nanophase 0.2.2 and re-install it to see if this problem could be resolved.

I am having this exact same issue with the exact same results as this thread.  Winterlich did you ever solve the problem?

Hmm, what are the general size of your reads?  Mine are admittedly kind of small for nanopore and its possible that flye is filtering too many so as that metabat2 does not have enough information to work with.

Thats a good idea, my read sets are also rather small. I will try another, larger dataset these days and will report on this...

Thank you both for your contributions!
If only a small long-read dataset was provided, it would be pretty challenging to perform genome binning. If you wanted to try nanophase with a long-read dataset, we had sequenced a mock community (you can find more details about the mock community in our paper) using nanopore sequencing and uploaded it to NCBI. The dataset can be downloaded via the following command: (you may need to install sra-tools)
fastq-dump SRR17913199

Please don't hesitate to let me know if I can help.
Best

So I ran it using the provided practice data set from your setup page and this was my result:
All required packages have been found in the environment. If the above certain packages integrated into nanophase were used in your investigation, please give them credit as well :)
[2023-06-14 12:08:04] TASK: Long-read assembly starts (be patient)
[2023-06-14 12:16:40] DONE: long-read assembly finished sucessfully: detailed log file is miniconda3envsNanophasedir/01-LongAssemblies/flye.log
[2023-06-14 12:16:40] TASK: Initial binning::metabat2 binning starts
/root/miniconda3/envs/nanophase-v0.2.2/bin/nanophase.meta: line 245:  3997 Segmentation fault      metabat2 -t $N_threads -i $OutDIR/01-LongAssemblies/assembly.fasta -o $OutDIR/02-LongBins/INITIAL_BINNING/metabat2/metabat2-bins/bin -a $OutDIR/02-LongBins/INITIAL_BINNING/metabat2/metabat2_abun.txt --cvExt > $OutDIR/02-LongBins/INITIAL_BINNING/metabat2/bin.log
[2023-06-14 12:16:40] ERROR: Something wrong with metabat2 binning, please also check miniconda3envsNanophasedir/02-LongBins/INITIAL_BINNING/metabat2/bin.log, terminating...
So the I would guess that the issue is going beyond just the data we are providing, though as of yet/what I am not sure.  I'm not running this on the world's most powerful computer either, is it possible I'm hitting a CPU bottleneck?  I'm running it on a laptop with an i7 1360p (12 cores, 5Ghz) with 32 GBs of RAM.  The ram is definitely not the bottleneck but I'm noticing my CPU is hitting 100% utilization during this run.

Did you mean the lr.fa.gz in the [Example dataset]? (https://github.com/Hydro3639/NanoPhase/blob/main/Usage_tutorial.md)

Yes! Is there a better one I should run?

I am still unsure what happened, I would expect the command to exit at the semibin stage rather than metabat2 if you use the provided lr.fa.gz. If you want to try v0.2.3, you can download the long-read dataset: SRR17913199, as I mentioned earlier. Is that possible for you to run it on a Linux workstation?

I actually did run this on ubuntu on a windows subsystem, I don't have a workstation though.  I did originally do this on v0.2.3 and I had gotten the same output with my data, I didn't try it with the practice data though.  I can try it with the specific long-read data set as well though.


Allow setting our own fallback currency names

This is useful for setting custom names for currencies that aren't listed on CMC like Komodo asset chains or smaller crypto currencies.

I also think our names should take priority over Coinlist. Right now CHIPS still get's listed as CHIP (CHIPS) since it's actually on Coinlist. So maybe we should treat it as an override instead? And only define the name when it doesn't already exists or we want it different?

I also think our names should take priority over Coinlist. Right now CHIPS still get's listed as CHIP (CHIPS) since it's actually on Coinlist. So maybe we should treat it as an override instead? And only define the name when it doesn't already exists or we want it different?

Yeah, I wasn't sure the best way to prioritise these. I decided to prioritise coinlist because it then we can rely on CMC to get up to date names. For example sometimes currencies rename, like Raiblocks => Nano, but they'll never change their ticker symbol. That way we don't have to update our app every time a coin updates. Although in practice, that data isn't live, it's just archived from CMC, and I don't intend to update coinlist too often.
Also, I just checked, and CHIPS is actually the official name as far as I can tell.
But yeah, I think I agree it probably does make more sense to have local overrides. Although if we do that, it seems like it might be better to just enforce adding names for each currency and not use coinlist for names at all. Thoughts?

I think you misunderstood me:
We first check our local overrides, then coinlist, then use the symbol name.
That way we only have to define local overrides if we know it's missing from coinlist or if we want to use a custom name (Like with CHIPS), otherwise it will use coinlist.

Yeah, I agree that makes more sense.

@sindresorhus Ideas on what helper methods to add to supported-currencies.js.
getCurrencySymbols();
// Array

getCurrencyName(symbol)
// String

Anything else?

Those are the ones I know we need right now. We can add more later if needed.

Can you also rebase from master and use getCurrencySymbols and getCurrencyName` here: https://github.com/lukechilds/hyperdex/commit/c7ca564b6e9c6356d4ff39dc55f19ed89ef5abb5#diff-b104ba320844f7609baac0cec3b95728R32 ? So they get correct names.

@sindresorhus done

Looks great 👌


Merge from upstream

This PR should appear in my fork.

Closed because of accidental action.

You might want to just merge instead of a PR on your local fork


document how to do local container testing

podman build --target build -t test .
podman run --rm -it test
bash /project/.github/workflow/container_tests.sh

done!

BUT - should I add this into tox -p ??
An issue with doing so currently is that we dont have DinD (but that's my next thing).

Since the review described in #75 we no longer run the tests in a container so this is a non-issue.


Update integration test suite

I have implemented your test in local code, and is testing against two different groups
Just before the code
        const memberAttr = results[0].attributes.find(a => a.type === 'member')
        t.ok(memberAttr)
        t.ok(memberAttr.vals)
        t.type(memberAttr.vals, Array)
        t.is(memberAttr.vals.length, 2000)

I did add the following
results[0].attributes.forEach(attr => {
                    console.log(attr.type)
                })

And the output from a small group is

objectClass
cn
description
member
distinguishedName
instanceType
whenCreated
whenChanged
uSNCreated
uSNChanged
name
objectGUID
objectSid
sAMAccountName
sAMAccountType
groupType
objectCategory
dSCorePropagationData

and the output from the large group is

objectClass
cn
description
member
member;range=0-1499
distinguishedName
instanceType
whenCreated
whenChanged
uSNCreated
uSNChanged
name
objectGUID
objectSid
sAMAccountName
sAMAccountType
groupType
objectCategory
dSCorePropagationData

and for the large group the test failed as memberAttr.vals.length was 0
but using
const memberAttr2 = entry.attributes.find(a => a.type === 'member;range=0-1499')

did return the first 1500 users in the group

⚠ This issue has been locked due to age. If you have encountered a recent
problem that seems to be covered by this issue, please open a new issue.
Please include a minimal reproducible example
when opening a new issue.


Client Consistently Crashes

I've tried uninstalling the RuneLite client multiple times and deleting any corresponding folders like .runelite and even the main RuneLite folder. There was a crash log for every time I tried launching the folder but I'm including the most recent one. I have installed the latest Java version but I don't think this was the issue since before clicking the "Login to RuneLite" I was able to use the client just fine. Taking any suggestions for my stupidity.
jvm_crash_pid_2964.log

The issue was resolved by successfully reinstalling Java and removing any old version. I also made sure to uninstall any files associated with RuneLite and run the installation as an administrator.


windows 2022 vm is really slow and cpu is 100% pinned

Operating system
ubuntu 24.04
Description
the windows vm is really slow and the cpu cores of the vm are 100% all the time even im idle
Docker compose
GNU nano 7.2                                       docker-compose.yml                                                 services:
windows:
image: dockurr/windows
container_name: windows
environment:
VERSION: "2022"
VMX: "Y"
DISK_SIZE: "50G"
RAM_SIZE: "6G"
CPU_CORES: "2"
devices:
- /dev/kvm
cap_add:
- NET_ADMIN
ports:
- 8006:8006
- 3389:3389/tcp
- 3389:3389/udp
stop_grace_period: 2m
volumes:
- /root/windows:/storage
Docker log
❯ Starting Windows for Docker v4.00...
❯ For support visit https://github.com/dockur/windows
❯ CPU: AMD Ryzen 7 2700X Eight Core Processor | RAM: 7/8 GB | DISK: 42 GB (ext4) | HOST: 6.8.0-48...
❯ Warning: your configured RAM_SIZE of 6 GB is very close to the 7 GB of memory available, please consider a lower value.
❯ Booting Windows using QEMU v9.1.0...
BdsDxe: loading Boot0004 "Windows Boot Manager" from HD(1,GPT,7CCD9614-2FC1-4769-8343-4017DD677380,0x800,0x40000)/\EFI\Microsoft\Boot\bootmgfw.efi
BdsDxe: starting Boot0004 "Windows Boot Manager" from HD(1,GPT,7CCD9614-2FC1-4769-8343-4017DD677380,0x800,0x40000)/\EFI\Microsoft\Boot\bootmgfw.efi
❯ Windows started succesfully, visit http://localhost:8006/ to view the screen...
root@ubuntu:~/windows#
Screenshots (optional)

not sure if passed, but there could be the possibilty to add the hyperv extensios to the -cpu call that often seem to help: https://www.qemu.org/docs/master/system/i386/hyperv.html

Hey sorry for the late response..
Ill try it out.. thank you

Hey sorry for the late response..
Ill try it out.. thank you

@MindlessGamingDE Did it work?


Hotfix #471

I preemptively merged #471 without letting the CI run and it turns out, many things started breaking. This is an attempt to address those issues.

The current concerns are the example API 7 test, the one for subscriptions, and a type mismatch here between the CLI and the main library.

here

The package that is used in the test is actually archived and not maintained anymore, should migrate to use:
https://github.com/enisdenjo/graphql-ws

This fixes the test, not all required props are passed down.
        subscriptionServer = new SubscriptionServer(
          {
            execute: (
              schema,
              document,
              rootValue,
              contextValue,
              variableValues,
              operationName
            ) =>
              execute({
                schema,
                document,
                rootValue,
                contextValue,
                variableValues,
                operationName
              }),
            subscribe: (
              schema,
              document,
              rootValue,
              contextValue,
              variableValues,
              operationName
            ) => {
              return subscribe({
                schema,
                document,
                rootValue,
                contextValue,
                variableValues,
                operationName
              })
            },
            schema,
            onConnect: (params, socket, context) => {
              // Add pubsub to subscribe context
              return { pubsub }
            }
          },
          {
            server: wsServer,
            path: '/subscriptions'
          }
        )

@Pagebakers Thanks for your suggestions! I'll make a new release now. We definitely need to upgrade the other modules but I don't want to delay things any further.

Nice! I will see if there are other low hanging fruits this week.


Using Polymer with Webjars

To use JavaScript packages from NPM/Bower in Java apps, Webjars provides an on-request automatic JAR packaging of NPM/Bower packages and publish them to Maven, which makes it very straightforward for a Java developer to use these with a single package manager.
Unfortunately Webjars do no work with Polymer web components, since they put web files in a structure like this:
/webjars/polymer/2.2.0/polymer-element.html
/webjars/my-component/1.0.0/my-component.html

but Polymer assumes a certain directory structure to import dependencies, e.g. <link rel="import" href="../polymer/polymer-element.html">, assuming to find them in the parent directory and a specific path.
Will this be addressed in Polymer 3?

Assuming a cross-component directory structure is certainly bad design.

Hm, this setup is not something we were considering. Since Polymer 3 is simply Polymer 2 but using ES modules instead of HTML imports, we will not be changing our URL destinations either. I am surprised that this is an issue specific with Polymer though. As I understand it, other packages rely on their dependencies and URLs as well. Could you maybe point to other packages that you have no problems with installing so that we can take a look? If there is an obvious fix, we could apply it, but I fear it will be very hard to not break everyones flow right now.
Open to any opportunity to improve adoption, if we can make it work for everyone 😄 Let us know!

I think that if this is in JS then we at least have the opportunity to override defaults.  That is possible with RequireJS.  So maybe we will be ok.

Polymer 3 will be using ES modules, which means we will be fully in JS context. All conventional JS tools should be usable with Polymer, thus I think RequireJS should be able to deal with it 🤞

With our reliance on ES modules with name syntax, this issue should be resolved. We no longer assume a particular directory structure 😄


Flaky test: Timeout in test of find-and-replace:copy-path

https://github.visualstudio.com/Atom/_build/results?buildId=42708&view=logs&jobId=6de74252-975b-522d-7a3d-227ca79ca6c1&taskId=b7ded238-9370-50c9-8500-e02f948495a4&lineStart=468&lineEnd=473&colStart=1&colEnd=78

We had hoped that this issue was resolved by #19505, but we saw this flake again last night:
https://github.visualstudio.com/Atom/_build/results?buildId=43605&view=logs&jobId=6de74252-975b-522d-7a3d-227ca79ca6c1&taskId=b7ded238-9370-50c9-8500-e02f948495a4&lineStart=473&lineEnd=477&colStart=1&colEnd=9
That build includes the changes from #19505 (i.e., the upgrade to find-and-replace v0.218.13), so I think that means that this flake is still an issue. With that in mind, I'm going to re-open this issue. If I'm interpreting these results incorrectly and this issue should remain closed, just let me know.


Kehao addch9ex

Ke Hao Chen
participation_grade

Chapter # 9
Practice

Repeat the Problem Title, Chapter #, and Section Name format for as many problems as you have.
Optional: Brief description of changes or any other notes you want to leave to the Curriculum Development team

@Citrus716 Make sure you run the solution to your program before submitting a PR. I caught a logic error in your code that would have thrown a RunTime error. It was that you were iterating over the dictionary itself, instead of dict.keys(). Just be a bit more carfeul :)


Error setting up channel X

Hello,
Once a python script using CHIP_IO has exited, when I try to run it again, even with using GPIO.cleanup() before setting up the IO, I'm getting an error:
GPIO.setup(set, GPIO.OUT)
RuntimeError: Error setting up channel CSID2, maybe already exported? (gpio_export: could not write '134' to /sys/class/gpio/export (Device or resource busy))

Is there any workaround to be able to run my script again, without rebooting?
Thanks.

So, it looks like your script isn't unexporting CSID2, so when you start it again, it's throwing the error.
What does your full script look like?  I think you might not be cleaning up the GPIO properly.
There is a way to deal with this, the Utilities module.  You can do the following at the beginning of your script:
import CHIP_IO.Utilities as UT
UT.unexport_all()

The above will work, but it doesn't solve the error of the GPIO not being cleaned up.

Thank you, I will try this, and also handle correctly the signal supervisord is sending to my script to gracefully shut it down... :)

@nio101, you still have issues with cleanup?  My latest commits here fix it for good now (hopefully).  Just seeing if there is an issue still, otherwise I'll close this.


add subdomains for name prefix mismatches

It would be nice to have something like prefix.mismatch.badssl.com serving a certificate that is only valid for mismatch.badssl.com (and vice-versa: mismatch.badssl.com serving a certificate that is only valid for prefix.mismatch.badssl.com).

Sounds reasonable; we even have special logic for a www mismatch in Chrome now.
Note that subdomain.preloaded-hsts.badssl.com is already linked from the front page, and wrong.host.badssl.com is also a prefix mismatch.
However, both of those are a mismatch of ..badssl.com subdomains with a *.badssl.com cert.
I feel like there's a better word than mismatch we could use; got any ideas?

Does Chrome allow www.subdomain.badssl.com if the cert is for subdomain.badssl.com? That would let us test that piece of functionality.

Does Chrome allow www.subdomain.badssl.com if the cert is for subdomain.badssl.com?

It already redirects! (Try it.)


Example for Existing Exchanges and Queues

Can an example be provided on how to publish messages to an existing exchange and listen for messages on an existing queue associated to the exchange? I have tried multiple combinations but can't get it. The existing examples dynamically generate queue, which unfortunately I cannot use.
I appreciate any assistance on this.

I have been able to create an example. Took me a few days:
client.js
`import { AMQPChannel, AMQPClient, AMQPConsumer, AMQPQueue } from '@cloudamqp/amqp-client';
export class AmqpClientService {
static #Instance: AmqpClientService;
#client: AMQPClient;
#channel: AMQPChannel;
#connection;
#exchangeName: string;
#routingKey: string;
#queueName: string;
#queue: AMQPQueue;
#consumer: AMQPConsumer;
private constructor(
client: AMQPClient,
connection,
channel: AMQPChannel,
queue: AMQPQueue,
queueName: string,
exchangeName: string,
routingKey: string
) {
this.#client = client;
this.#channel = channel;
this.#exchangeName = exchangeName;
this.#routingKey = routingKey;
this.#queueName = queueName;
this.#queue = queue;
this.#connection = connection;
}
public static async init(
uri: string,
queueName: string,
exchange: string,
routingKey: string
) {
if (this.#Instance) {
return this.#Instance;
}
const client = new AMQPClient(uri);
const conn = await client.connect();
const ch = await conn.channel();
const qu = await ch.queue(
queueName,
{ durable: true },
{
'x-dead-letter-exchange': 'EBINDER_DEAD_LETTER_EXCHANGE',
'x-dead-letter-routing-key': 'DOCUMENT_DRIVEN_STUDY_STARTUP',
'x-max-in-memory-length': 1,
'x-delivery-limit': 3,
'x-queue-type': 'quorum',
}
);
await qu.bind(exchange, routingKey);
return (this.#Instance = new AmqpClientService(
  client,
  conn,
  ch,
  qu,
  queueName,
  exchange,
  routingKey
));

}
async subscribe() {
try {
  this.#consumer = await this.#queue.subscribe(
    { noAck: false, },
    async (msg) => {
      console.log(msg);
    })

  await this.#consumer.wait();
} catch (e) {
  console.error('ERROR', e);
  e.connection.close();
  setTimeout(this.subscribe, 5000); // will try to reconnect in 1s
}

}
async publish(message: string) {
try {
await this.#channel.basicPublish(
this.#exchangeName,
this.#routingKey,
Buffer.from(message, 'utf-8'),
{ deliveryMode: 2 }
);
} catch (e) {
console.error('ERROR', e);
e.connection.close();
setTimeout(this.publish, 5000, message);
}
}
async close() {
await this.#consumer.cancel();
await this.#client.close();
await this.#connection.close();
}
}
`
bootstrap.js - publish a message and consume a message
`import { AmqpClientService } from '../shared/services/amqp-client-service.js'
import { ConfigurationService } from '../shared/services/configuration-service.js';
const location = 'QAUS'
const configuration = new ConfigurationService(
[
'AMQP_SERVICE_URI',
'AMQP_SERVICE_QUEUE',
'AMQP_SERVICE_EXCHANGE',
'AMQP_SERVICE_ROUTING_KEY'
],
.env.${location}
);
const {
AMQP_SERVICE_URI,
AMQP_SERVICE_QUEUE,
AMQP_SERVICE_EXCHANGE,
AMQP_SERVICE_ROUTING_KEY
} = configuration.environmentVariables;
const amqpClientService = await AmqpClientService.init(AMQP_SERVICE_URI, AMQP_SERVICE_QUEUE, AMQP_SERVICE_EXCHANGE, AMQP_SERVICE_ROUTING_KEY);
console.log(Waiting for Messages from ${AMQP_SERVICE_QUEUE});
amqpClientService.subscribe();
while(true) {
const str = randomCharacters(24);
console.log(Sending a Message - ${str})
amqpClientService.publish(JSON.stringify({ message: This is a test for ${str}}))
await wait(5000);
}
function wait(milliSeconds){
return new Promise(res=>setTimeout(res,milliSeconds));
}
function randomCharacters(length){
const chars =
'AaBbCcDdEeFfGgHhIiJjKkLlMmNnOoPpQqRrSsTtUuVvWwXxYyZz1234567890';
const randomArray = Array.from(
{ length: length },
(_, __) => chars[Math.floor(Math.random() * chars.length)]
);
const randomString = randomArray.join('');
return randomString;
}`

If you expect the amqp resource to already exist you can set the passive flag when declaring the resources.


Allow DelegatedIdentity API clients to subscribe by PID

This is designed to provoke discussion around the DelegateIdentity API changes required for https://github.com/spiffe/spire/issues/5019
This is following the "pass PIDs" approach settled on in the RFC doc: https://docs.google.com/document/d/1A1oQHuR6z3bvQtXN17r2EwBr5lazGGPbUPkxoURAAh4/edit
This assumes that the caller/client is ensuring that the PID is not recycled for the duration of the stream/call, using pidfd or similar.
Looking for the following feedbacks:

Is this additional functionality a good fit for the DelegateIdentity API or should we add a new API? I think we should just extend DI.

Do we object to supporting stream/subscriber based APIs? I think we should support them, even though the PID recycling protection requirement puts more onus on the client.

Do we want to do this for both JWT and x509 SVIDs? I assume so, and added new calls for both of the ones that previously required selectors.

Rather than make currently-required fields optional (selectors), I added parallel calls with new, parallel messages/fields. Do we want to do it like this, or do we want to use a single message/call and deprecate the old fields?

The impl side of this is here: https://github.com/spiffe/spire/pull/5272

Thanks for this @bleggett. We discussed this a bit and I think at this time the preference is to:

Extend the existing RPCs instead of creating new ones
Use a oneof to either provide selectors or the pid

Thanks for this @bleggett. We discussed this a bit and I think at this time the preference is to:

Extend the existing RPCs instead of creating new ones
Use a oneof to either provide selectors or the pid

Okay, I'm fine with that. Will redo it along those lines.
Thanks for getting back!

Use a oneof to either provide selectors or the pid

@azdagron There's one gotcha with this - you cannot use repeated fields in a oneof. And the existing selectors are repeated.
This could be fixed by creating a new message for the selectors, nesting the repeated selectors in that message, and using oneof {message, pid}
The problem with this is that it will break back compat for people using  this API that already use the selectors as-is.
Thoughts?

Oh that's right. We need to preserve backcompat. I think the best option is to not leverage oneof at this point and just check for mutual exclusivity in the handler implementation.

@azdagron all right, suggestions applied.
I also updated https://github.com/spiffe/spire/pull/5272/
Note that one consequence of sharing the same message and not using oneof is that an "unset" PID is naturally protobuf'd to 0: https://github.com/spiffe/spire/pull/5272/files#diff-1007e43f9ac81915b4fd0f98b5a937ae07f2d12c5d20fa69a2919788663c868eR121
0 is technically a valid PID, and with this approach we basically cannot attest a PID of 0, but I cannot imagine how or why SPIRE might need to do that given that PID 0 is special, so I don't think it's a big deal.

Yeah, AFAIK, no userspace process will have pid 0, so I think that's a practical tradeoff.

Ty @azdagron and others!


COMA and MIND complexes- New terms in Component intology

Hi,
I would like to add the following two complex terms to
the component ontology as children of GO:5828
(Kintechore microtubule) and GO:5875 (microtubule
associated complex)
COMA complex
A multiprotein complex in yeast consisiting of Ctf1p,
Okp1p, Mcm1p, Ame1p. This complex bridges the subunits
that are in contact with centromeric DNA and the
subunits bound to microtubules during kinetochore assembly
definition_reference: SGD:se
definition_reference: PMID:14633972
Synonym (exact): Ctf1p-Okp1p-Mcm1p-Ame1p
MIND complex
A multiprotein complex in yeast consisting of Mtw1p and
includes Nnf1p, NSl1p and Dsn1p. This complex bridges
the subunits that are in contact with centromeric DNA
and the subunits bound to microtubules during
kinetochore assembly
efinition_reference: SGD:se
definition_reference: PMID:14633972
Synonym (exact): Mtw1p Including nnf1p-Nsl1p-Dsn1p
Reported by: rbalakri
Original Ticket: "geneontology/ontology-requests/1750":https://sourceforge.net/p/geneontology/ontology-requests/1750

thanks www.movied.org


hubot - slack integration

when i issue HUBOT_ADAPTER=slack HUBOT_SLACK_TOKEN=*****  ./bin/hubot
getting below error
info: unable to RTM start, attempting reconnect: Error: self signed certificate
but when i give ./bin/hubot then it's working fine
@hubot list all server hardware
getting the JSON response.
$ node --version
v7.2.0
$ npm --version
3.10.9

@reddypramod85 are you connecting to Slack from behind a proxy?

Connected from HPE network

Does your Hubot instance without this plugin connect to Slack successfully?

@danielreed
I tried Hubot instance without this plugin and the connection was unsuccessful. But when I logged in from Home network I was able to connect to slack. Can you please help me with work around for proxy?

You probably need to set http_proxy and https_proxy variables in the shell where you are starting Hubot.  You’ll need to use the correct proxy host for the network you are on.

@danielreed
Thanks you Daniel, after setting the proxy able to list all server hardware from slack channel.

Ok that's great news.  I'm going to close this issue.


User.Identity.IsAuthenticated is False after upgrading to .Net Core 2

After I call
var result = await _signInManager.PasswordSignInAsync(model.Email, model.Password, model.RememberMe, lockoutOnFailure: false);
which results in successful.
User.Identity.IsAuthenticated
is always false.
I have this in ConfigureServices:
    services.AddAuthentication(sharedOptions =>
	        {
				sharedOptions.DefaultAuthenticateScheme = CookieAuthenticationDefaults.AuthenticationScheme;
		        sharedOptions.DefaultSignInScheme = CookieAuthenticationDefaults.AuthenticationScheme;
			});

			services.AddSession();
	        services.AddMemoryCache();

			services.AddMvc()
		        .AddRazorPagesOptions(options =>
		        {
			        options.Conventions.AuthorizeFolder("/Account/Manage");
			        options.Conventions.AuthorizePage("/Account/Logout");
		        });

and this in Configure:
app.UseSession();
app.UseAuthentication();

_signInManager is part of the Identity system. You shouldn't need AddAuthentication if you have AddIdentity.

See https://github.com/aspnet/Security/issues/1338

And https://github.com/aspnet/Identity/issues/1364

@Tratcher You are right. When I removed AddAuthentication it works fine. Thanks


geographiclib x64

h:\Repos\vcpkg-master>vcpkg.exe install geographiclib:x64-windows
The following packages will be built and installed:
geographiclib[core]:x64-windows
Starting package 1/1: geographiclib:x64-windows
Building package geographiclib[core]:x64-windows...
-- CURRENT_INSTALLED_DIR=H:/Repos/vcpkg-master/installed/x64-windows
-- DOWNLOADS=H:/Repos/vcpkg-master/downloads
-- CURRENT_PACKAGES_DIR=H:/Repos/vcpkg-master/packages/geographiclib_x64-windows
-- CURRENT_BUILDTREES_DIR=H:/Repos/vcpkg-master/buildtrees/geographiclib
-- CURRENT_PORT_DIR=h:/Repos/vcpkg-master/ports/geographiclib/.
-- Downloading https://jaist.dl.sourceforge.net/project/geographiclib/distrib/GeographicLib-1.47-patch1.zip...
-- Downloading https://jaist.dl.sourceforge.net/project/geographiclib/distrib/GeographicLib-1.47-patch1.zip... OK
-- Testing integrity of downloaded file...
CMake Error at scripts/cmake/vcpkg_download_distfile.cmake:87 (message):
File does not have expected hash:
      File path: [ H:/Repos/vcpkg-master/downloads/geographiclib-1.47-patch1.zip ]
  Expected hash: [ d8fdfd7ae093057ec1a4ab922457fe71a3fb9975df5b673c276d62a0e9c4f212dc63652830b9d89e3890bc96aafd335992943cf6a1bce8260acf932d1eb7abfd ]
    Actual hash: [ 3b53ccf0243cd4a6a731a898bfdba95c3d22342e7e64f9e9a4a2bce5f74ac809943f570ed92c9077101a178f82f49ece1d1c4bfee1f288ea61d99fb36f6a243e ]

The file may have been corrupted in transit.
Call Stack (most recent call first):
scripts/cmake/vcpkg_download_distfile.cmake:129 (test_hash)
ports/geographiclib/portfile.cmake:15 (vcpkg_download_distfile)
scripts/ports.cmake:72 (include)
Error: Building package geographiclib:x64-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with .\vcpkg update, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
Package: geographiclib:x64-windows
Vcpkg version: 0.0.105-unknownhash
Additionally, attach any relevant sections from the log files above.

Thanks for reporting this issue!
I am unable to repro the bug -- my download matches the recorded hash. It is possible that you are behind a proxy that is causing issues during the download.
If this is the case, please set the HTTPS_PROXY variable to configure curl to use your proxy [1].
[1] https://stackoverflow.com/questions/9445489/performing-http-requests-with-curl-using-proxy


Reimplement it in Rust

A Rust rewrite could have quite a few advantages, including memory safety, blazing fast speed, and easy concurrency. I got some code to get you started:

Just make sure to include a Cargo.toml, and then you can run it with cargo run

This would only run on nightly

Ah, you're right, but that's not bad with rustup. Then you just add the feature:
#![feature(no_code)]


BracketPairColorization doesn't match well after copy&paste some code into current code section

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

VS Code Version: 1.60.0
OS Version: Windows_NT x64 10.0.19043

Before copy&paste some code:

After paste copied code to it:

/assign @hediet

Thanks for reporting this bug! Can you provide steps for how to reproduce this?

@hediet The screenshot is a demo,  just copy some code into a code block.

``

@jeffreyyjp I cannot reproduce:

Please add some more steps!

Can barely reproduce it in .vue files, but looks like issue is Vetur-related and gets triggered by tab switching; Bracket colorization seems to work fine with extension being disabled
fyi: brackets sometimes even flashes to red for a moment and then get back to normal color

And always stuck after tab switching

@hediet The copied code is from another file, but I need to make sure that it's occurred for any copy$paste situations

@HardKullez

Bracket colorization seems to work fine with extension being disabled

Even in case of the Vetur extension enabled, you disabled/uninstalled the bracket pair colorization extension, right?

@HardKullez

Bracket colorization seems to work fine with extension being disabled

Even in case of the Vetur extension enabled, you disabled/uninstalled the bracket pair colorization extension, right?

Not exactly, i meant disabling Vetur; No Bracket pair colorization extension installed at all

I have the same issue in simple

@lorand-horvath I think this might be caused by a tokenization race condition that has already been fixed. Can you try to reproduce it with the VS Code Insiders build?


re-save test data with compression

use the compression parametrer of saveRDS to compress the stored objects

@hechth I think this can be closed as with the updated testdata this is not needed anymore


Adds about command

Codecov Report

Merging #123 into master will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #123   +/-   ##
=======================================
  Coverage   99.67%   99.68%
=======================================
  Files           1        1
  Lines         923      938   +15
  Branches      184      185    +1
=======================================
+ Hits          920      935   +15
  Misses          1        1
  Partials        2        2

Impacted Files
Coverage Δ

src/turbot/__init__.py
99.68% <100.00%> (+<0.01%)
:arrow_up:

Continue to review full report at Codecov.

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


Update Configuring.md

Add regular expression support in Virtual Hosts section

Is this how it works by fact? Or it is a feature request?

Hi Andrey,
I believe so and we are using it in our testing environment to capture any
missing service mapping in an effort to remove our existing mocks and
external dependencies (many individual ones).  It's supported by Toronto
webserver in hostname match.
Also I've tested the exact configuration on the doc to make sure it's
working.
Regards,
Wenyuan song
On Thu, 25 Nov 2021, 5:24 pm Andrey Pokhilko, @.***>
wrote:

Is this how it works by fact? Or it is a feature request?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/up9inc/mockintosh/pull/157#issuecomment-978874535,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABJBB5EPK77SNIRRBBRBTYDUNXJA7ANCNFSM5IXRUZRQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Great! Thanks for your contribution!


migrate some final FromPyObject implementations to the Bound API

This PR picks up a few small bits around the codebase which were calling / implementing FromPyObject::extract and moves them to extract_bound.
I've split them into individual commits to help list what happened.

Thanks both for the review!