[FEATURE REQUEST] Disable automatic updates for local apps and system components

Before submitting the issue

[x] I have checked that the requested feature is not a duplicate or, if it is, it needs to be reopened
[x] I have checked that I am running the latest version of WingetUI
[x] I have checked the FAQ section
[x] I have read the contributing guidelines and I agree with the Code of Conduct

Is your feature request related to a problem? Please describe.
WingetUI, every now and then, tries to update system / windows related components.
Like: .NET, C++, Vcredist, various KB123456, etc...
And various system apps, like: Edge, Xbox, Windows security, etc...
All stuff that usually get updated via Windows Update anyway.
Describe the solution you'd like
An option in the settings, to ignore all the updates of system components automatically. And leave them to Windows Update.
OR (even better probably)
An option in the settings, to ignore all updates for apps and programs that were not directly installed via WingetUI.
So, all the packages in the "Installed packages" tab wich are tagged as "Local PC" and "Microsoft store" as the source. (since as the time of writing this issue WingetUI does not support the MS store as a source, but it could change in the future)

For what you are proposing, the best thing you could do would be ignoring the packages you don't want WingetUI to update; since there's no way to determine whether a package is a system component or not. Furthermore, Detecting which packages were installed through WingetUI and which not would need an important code rebase, and wouldn't be backwards compatible.

since as the time of writing this issue WingetUI does not support the MS store as a source, but it could change in the future

The latest beta actually does support ms store

Oh yeah, my actual wrokaround is just to ignore them manually in fact, it's not a major issue, just an inconvenience.
I thought since many system utilities, like the Nirsoft's ones, have the option to "show/hide Windows programs from the list" thing, it would have been a simple thing to implement.

The latest beta actually does support ms store
That's nice to hear!

Oh yeah, my actual wrokaround is just to ignore them manually in fact, it's not a major issue, just an inconvenience.
I thought since many system utilities, like the Nirsoft's ones, have the option to "show/hide Windows programs from the list" thing, it would have been a simple thing to implement.

A simple way to achieve that would be simply typing Microsoft on the search bar and then blacklist the packages that appear.
I could see being annoying to manually right click each package. Maybe the uninstall section could have a button to blacklist multiple packages like the update section?
another thing, @marticliment in the latest beta, is it possible to update MS Packages through WingetUI? because there is no way to blacklist them.
Now that I think about it, a problem with adding a button like I proposed is that there would need to have some check to see if the package is not local pc.

MS Packages

No. The only Microsoft Store upgradable packages will be listed as Winget: msstore, and are not the UWP applications, but rather the Win32 apps available through the store. This is how Winget works.

This issue had no pending code fix


Extended Architecture

There are some problems that, in order to be addressed, may require a new architecture.  They are:

Bidirectional sync from local platform instances to a remote platform instance.
Consumption of mumuki content from an IDE - we need to be able to download mumuki guides, and submit solutions, and be able to query user's progress.
Preferences handling, that may be consumed by IDEs linked with mumuki. For example, preferences regarding code style that may be set by teacher
Adding or removing a guide to multiple instances

I am submitting this isue in order to freely discuss alternatives

@faloi @javierfernandes @npasserini

Firebase?
El jue., oct. 15, 2015 18:06, Franco Leonardo Bulgarelli <
<EMAIL_ADDRESS>escribió:

@faloi https://github.com/faloi @javierfernandes
https://github.com/javierfernandes @npasserini
https://github.com/npasserini
—
Reply to this email directly or view it on GitHub
https://github.com/mumuki/mumuki-platform/issues/354#issuecomment-148521382
.

This will probably require adding new nodes and publishing new APIs, that extend the current ones:

submissions web hook, which notifies submissions
guides and users endpoints, which retrieves guides and users

After some offline discussion, we agree this will require some refactors and additions.
They are:
Sync Component
Create a component that is polymorphic with a platform but just adds a cache in order to allow bidirectional sync of an offline platform/ide submissions with an online platform. This component might be replaced by a firebase  connection.
Content Server
This should a be a multitenant server that allows content -  guides - sync among platforms and IDE's, preferences distribution, centralized storage of guides, and a simple api for a content editor for storing guides, so that it does not need to deal with git complexities.

Extract a service that converts github repos to a guide json forth and back. This code should be extracted from current platform.
Create a service that stores guides in a json DB, notifies platforms when there are updates, and exposes an api for consuming them
Create a service that stores preferences per course and tenant
Create a service that allows association of a guide to one or more tenants and courses
Manage permissions

Some of this parts may be partially replaced by firebase, too.
Auth server
We need to maintain a consistent user - students and teachers - across multiple platforms and IDEs, and allows offline login.

Now that we have introduced Auth0 as SSO solution, and that bibliotheca is our working content-server, we can consider this issue closed by now.
A sync component should be discussed separately


Response Always Nothing

Hello,
I don't understand why every time I run a function, the response of that function equals nothing.
I do have a connection and a SID but never any data in the response.
On the other hand, I have an exception thrown: "'System.Runtime.Serialization.SerializationException' in RestSharp.dll" but without details.
Can you tell me what's going on?
I am in DSM 6.2.4-25556
Thx

I downloaded it through Nuget but had the same symptoms.
After downloading the source, I attached the dll and it worked normally.


Updated the supported column to fix #4325

Updated the supported column to fix #4325
Previous condition
Operation =~ "Set-Mailbox"
Current Condition
| where OfficeWorkload =~ "Exchange"

@petebryan - Could you please on this review as the original author.

@aprakash13 can you please check and approve?


supports-color breaks coloring

This line over here defines an ANSI extended color set for bright color which doesn't display correctly: (by incorrectly I mean it's just bold white color)
https://github.com/visionmedia/debug/blob/5c7c61dc0df0db4eb5de25707d8cd1b9be1add4f/src/node.js#L169
It shouldn't(?) have a traling ;1 after ${colorCode}. So the line should like:
const prefix = `  ${colorCode}m${name} \u001B[0m`;
This fixes the bug (for me :) )
EDIT: This supports my observation 256 colors
Also from the article:

Note that the bright versions of the background colors do not change the background, but rather make the foreground text brighter. This is unintuitive but that's just the way it works.

and the ;1 option only applies to \u001b[44 ie background colors, while the \u001b[38 option for foreground colors does not have such thing

Sorry for the late reply, here are some screenshots.
The result is the same when using git bash(with cmd console and powershell):

Here is the result when I use the version from my fork:

Could you do me a favor and change this line:
https://github.com/visionmedia/debug/blob/5c7c61dc0df0db4eb5de25707d8cd1b9be1add4f/src/node.js#L168
To this:
-const colorCode = '\u001B[3' + (c < 8 ? c : '8;5;' + c);
+const colorCode = '\\e[3' + (c < 8 ? c : '8;5;' + c);

Re-run and post the results here?

Yes of course.
First run is when ;1 is still in the code, the second run is without it:

Those look fine, they should be working. This is a bug in CMD.exe.
I'll open a ticket with Microsoft later.

You are in fact correct, I tested it on a mac and it works just fine

Bold has always been quite broken on Windows. Here's an open issue that probably has to do with this bug.
https://github.com/Microsoft/console/issues/109
For posterity, here's a bug where bold-off (21m) was completely missing that I filed in 2017.
https://github.com/Microsoft/WSL/issues/2174
cc @zadjii-msft is this a new problem that I should file, or is this the same as microsoft/console#109?

So, microsoft/console#109 is mostly just tracking the rendering of bolded text, which is still somewhere down the backlog and unsupported.
However, the bug here is that combining 256-color values (38;5;210m for ex) and boldness (1m) doesn't work correctly. I don't know when for a fact that was fixed, but I do believe it's fixed in Insider's now, and will be generally available with the next Windows update after 1809:

Wonderful, thanks for the quick response as always @zadjii-msft :)
I'm going to close this as fixed. Thanks for the report @NaNraptor, good find.

@Qix, what if I make a PR that removes boldness based on environment variable? I'm working in a corporate environment and very likely that windows updates that will bring this feature will take a while :(. And it will depend on each of my coworkers to update their machine 😢. Thanks

@webuniverseio I wouldn't approve that, unfortunately. I don't want to introduce more environment variables when we're working to remove them.

@Qix- Thank you so much for your response. Unfortunately I'm trying to archive the opposite - get different colors with supports-color. I guess for now I have to use a workaround.
I have a different proposal - what if everything will be in normal font when color is used? Even on windows 1809 I don't see bold text & even if was bold, not sure how much value would it add. I guess it can be useful when there is no color & perhaps it really helps to visually separate logs on other OS...
Please let me know what you think. Thank you!


RHOAIENG-14811 web-user-id for Sandbox is not correctly set.

Closes https://issues.redhat.com/browse/RHOAIENG-14811
We now get the userID from the status api endpoint if set.
This reverts parts of RHOAIENG-9480/PR #3107
Description
The previous code assumes that the (sso/web-)userId is passed in an annotation in the
get user object. This seems not be true in the real world. The underlying Kube is exposing
that id in the response to the /status call
The code reverts the try to fetch from the annotation and instead takes it from the Kube
object if present .
How Has This Been Tested?
Manual test in dev-mode running npm run start:dev:ext against an instance on Dev-Sandbox and a non-Sandbox one.
Then looking at the console output for the the Identify event
Identify event triggered: {... "userID":"b09067a69...99"}

If the userID looks like this (whith chars) it is an anonymous one. If it is numeric (may be negative) and short (probably < 8 digits), then it is a web-user-id.
Request review criteria:
Self checklist (all need to be checked):

[X] The developer has manually tested the changes and verified that the changes work
[X] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[ ] The developer has added tests or explained why testing cannot be added (unit or cypress tests for related changes)

After the PR is posted & before it merges:

[ ] The developer has tested their solution on a cluster by using the image produced by the PR to main

cc @andrewballantyne

/retest

Superseded by #3387


move repeated run of test suite (without and with MIR optimizations) out of compiletest

Just like we already pass the target architecture to our compiletest wrapper via MIRI_TEST_TARGET (and then have CI run compiletest with different values for that env var), we now do the same for running the tests with and without MIR optimizations.
This has the advantage of speeding up local ./miri test, where in the vast majority of cases running tests both with and without optimizations is just a waste of time.

Looking good on CI.
@bors r=oli-obk

:pushpin: Commit a32e25677f43d161db8fdb923cb80a7dc7c4ae48 has been approved by oli-obk

:hourglass: Testing commit a32e25677f43d161db8fdb923cb80a7dc7c4ae48 with merge 5e7fd2b44d802d0a3c0aa8f5e2598b9a3c10fa51...

:sunny: Test successful - checks-travis, status-appveyor
Approved by: oli-obk
Pushing 5e7fd2b44d802d0a3c0aa8f5e2598b9a3c10fa51 to master...


Upgrade to Babel 6.4.5

This includes support for __moduleName wrapping in the transpilation output itself.

Added in https://github.com/systemjs/plugin-babel/commit/b439eb6ecc402bc50adf8d220f2ade6d16d648f4.


Investigate solutions for handling long text in components

Issue Description
As part of this spike we want to explore ways we can handle long text in terra components.
Three options we've discussed with UX are:

Text truncation
Word-break
Horizontal scroll overflow

As part of this spike we want to determine:

Is there a default behavior we can apply to all components?
Can we make it possible to opt-in to one of the other long text handling behaviors?
Does this make sense as its own component or as utilities which can be applied to any component?

Issue Type

[ ] New Feature
[ ] Enhancement
[ ] Bug
[x] Other

Expected Behavior
Solution(s) found for handling long text in components.
Current Behavior
N/A

I've been looking into some of these possibilities, and here are some possible scenarios.
For default behavior, I believe our text should not be doing any truncation by default. We should have truncation as an opt-in behavior as opposed to opt-out.
The first level of text truncation I recommend is having the text truncated with ellipsis on it without a way to expand it. To get around this scenario, we could have an ellipsis css class in the utilities package that applies the appropriate width and overflow styles to the text container. This wouldn't be the first recommended approach, as there wouldn't be a way to expand the container if the screen was small. This would be for situations where text is added for extraneous reasons, and the user doesn't need to know what it says, of which I can't envision this happening very often.
The second approach we can do is having a horizontal scroll on text-overflow. To accomplish this, you would need a container who's max-width is 100% and has an overflow-x set to scroll. Given that we would need this outside container, I think having a component called OverflowingComponent would be a good solution to this problem. It is a very simple opt-in and opt-out strategy. We would just need to ensure that our components allow for inputs of react nodes and not just strings when building components.
For another possible solution, we can work with react-truncate to allow for show more functionality. This would take in a number of lines that are visible before truncation occurs, and then add in functionality for toggling the truncation on and off. This would be the most recommended approach for applying text truncation, as it shrinks accordingly on various devices, but also has the appropriate behavior to display additional information.
Overall, there are solutions we can provide to consumers, but I think it's better overall to ensure our components have the appropriate PropTypes set to ensure we can easily provide this functionality to consumers. When we provide a string prop, we should see if it's possible to make that a ReactNode instead if the string is used to display information to users. Not only will this allow for an easy use of truncation, but allow for consumers to add in their own customizations as well.

Of interest: https://www.sassmeister.com/gist/8939c65e0774c36d595ad64475b62ecb

What are those solutions

Essentially, we are looking at two terra components. The first one is an overflowing text element that always hides the overflow. This is to be used for when the text overflows, but you don't care about the user needing to view all of it. This is for elements such as navigational elements. I've seen this example across several different websites and applications.
For the next terra component, essentially we want to introduce a ScrollableOverflow container. This container allows for content to be horizontally scrolled for when it overflows. This strongest use case for this is with tables on small devices.
For the final use case of having text that flows multiple lines, the best solution we have for this case is react-truncate. This allows you to set the text to a set number of lines, and then expand it with a toggle click.
These solutions are opt-in approaches that consumers would need to implement when they are needed. For our team to ensure we can implement solutions like this, we need to make sure that we are setting our prop types appropriately. Instead of taking in string for titles for example, we need to also accept react nodes so that the appropriate wrappers can be applied to the element.

Of interest: https://assortment.io/posts/handling-long-titles-with-truncation

I don’t recommend using character count as a text truncation solution. It’s doesn’t handle responsiveness very well, as large monitors may leave a huge amount of white space when text can fill it. I can also see issues with it on smaller screens and having the truncation look odd potentially, such as at line 4 vs line 9 in a tablet vs mobile display.
Nevertheless, I do understand that there are instances where it can be beneficial, so I’d be fine with having this as another possible option. Good find!


Official support for self hosted Text Generation Inference server by Huggingface.

Feature request
Official support for self hosted Text Generation Inference which is a Rust, Python and gRPC server for generating text using LLMs.
Motivation
Expanding the langchain to support the Text Generation Inference server.
Your contribution
Implemented HuggingFaceTextGenInference class to add this support.

Opened PR #4244
Appreciate some review comments.


long service names should be wrapped

Similar to https://github.com/openzipkin/zipkin/issues/2183 logged against classic UI, Lens could wrap long service names where needed.
See the gif with an example of a really long service name, included below for testing convenience.

[
  {
    "traceId": "77047bf9cf04ed5d",
    "id": "77047bf9cf04ed5d",
    "name": "get",
    "timestamp":<PHONE_NUMBER>945904,
    "duration": 156470,
    "annotations": [
      {
        "timestamp":<PHONE_NUMBER>945904,
        "value": "sr",
        "endpoint": {
          "serviceName": "com.alipay.sofa.tracer.demo.facade.rpccallservice.okay.we.can.go.a.bit.further.to.see.how.lens.copes.with.this"
        }
      },
      {
        "timestamp":<PHONE_NUMBER>102374,
        "value": "ss",
        "endpoint": {
          "serviceName": "com.alipay.sofa.tracer.demo.facade.rpccallservice.okay.we.can.go.a.bit.further.to.see.how.lens.copes.with.this"
        }
      }
    ]
  },
  {
    "traceId": "77047bf9cf04ed5d",
    "id": "40ef4e93ab3b1507",
    "name": "get",
    "parentId": "77047bf9cf04ed5d",
    "annotations": [
      {
        "timestamp":<PHONE_NUMBER>100852,
        "value": "cs",
        "endpoint": {
          "serviceName": "com.alipay.sofa.tracer.demo.facade.rpccallservice.okay.we.can.go.a.bit.further.to.see.how.lens.copes.with.this"
        }
      }
    ]
  }
]

@openzipkin/ui anyone have some time to contribute? This should be an easier one

There's probably an overlap with https://github.com/openzipkin/zipkin/issues/2291 which says to scroll instead of wrap, but i'll let the UX guys do the debate on that.

Thank you for your report.
I fixed this bug in #3061 .

@tacigar thanks, looks good to me. Do you think anything could (should) be done for the service names in the trace diagram itself ?

Yes, I know the problem, but I haven't fixed it yet.
Currently this trace diagram written by SVG.
Since SVG is not good at wrapping text and doing text ellipsis, I am thinking of re-implementing the trace diagram with regular html tags instead of SVG.
Regular html can style with powerful css.
I'll do it as soon as possible.
FYI: This is a PR that tried to solve this problem before. (I once merged it by mistake, but soon reverted ...) https://github.com/openzipkin/zipkin/pull/2949


Release 0.23.0

Description
Release 0.23.0

Review Checklist

[ ] Code contains useful comments
[ ] (Integration-)Test cases added (or not applicable)
[ ] Documentation added (or not applicable)
[ ] Changelog updated (or not applicable)
[ ] Cargo.toml only contains references to git tags (not specific commits or branches)

Once the review is done, comment bors r+ (or bors merge) to merge. Further information

bors r+


Docs: await Completions on Promises truncated paragraph in typescript 3.6 release noted

Page URL: https://www.typescriptlang.org/docs/handbook/release-notes/typescript-3-6.html#await-completions-on-promises
Issue: The paragraph is truncated/incomplete. It is

In TypeScript 3.6, completions on an

and then nothing.

Thanks, I've removed it


Incremental learning algorithm pre trained for regression

Hi all
Phd student from Algéria
I use machine learning in my research in oil drilling in problem of regression i use Random forest
but I need to use Incremental learning algorithm pre trained for regression for generalization i try to use Adaptive random forest with MOA framework
Can you give other algorithm better than ARF in python for example
thank you
Djamil Rezki
Phd student Batna 2 University Algeria
<EMAIL_ADDRESS>tel : 213 698 36 21 15 (viber + whatsApp)

Hey, you've opened the same issue twice, so I'll close this copy and answer you on the other one.


Move CXShim and CXTest to a separate repo

Use git submodule to avoid code duplicate as we still need CXTest for test targets.
resolve #117

@luoxiu I'd like to move CXCompatible to CXShim repo, what do you think?

@luoxiu would you please review this pr?


RemoveVertex from graph

I want to be able to remove nodes from the graph without having to make a full copy of the graph and copy over existing edges and vertices.

Ah, sorry, was looking at 0.16 godocs. Thanks!


Correctly handle JDK 9 version

JDK9 changed the naming scheme for Java versions. Instead 1.9 the JRE returns "9" and subsequently "9.0.1", "9.1.2" and so forth.
See http://openjdk.java.net/jeps/223
Fixes #63

@chrisphe Could you please take a look at this pull request?

@chrisphe Would be possible to make a Jadira release with this change? In [Dropwizard]/https://github.com/dropwizard/dropwizard) we use Jadira as a dependency and we would like to compile our project with Java9.

@chrisphe ping?


MAC Randomization

Oi Luandro,
have been thinking about Pirania and heard about something that we need to be aware of: MAC randomization.
It looks like in order for devices not to be tracked in corporate wifi environments (like Google's city wifi, or facebook's, or other corporations or state owned wifi hotspot networks), what devices started doing is doing MAC randomization, meaning that each time they connect to a new wifi network they will use a different MAC address.
We need to see how this would affect Pirania... I have seen that what some do is to have a separate wifi password per device... like an open wifi that gets captured by the hotspot, and a closed wifi that requires unique wifi passwords per device.
this is the article about MAC randomization: https://source.android.com/devices/tech/connect/wifi-mac-randomization
An implementation of that strategy: https://www.youtube.com/watch?time_continue=118&v=CxFx9PbkP8A&feature=emb_title

AFAIK recent Network Manager versions do it by default

We've had a girl in Moinho who seemed to have that, not by option but as a bug in her hardware. I never thought there would be a way to deal with that. I like the idea of having a password protected wifi for such cases.


Changes to connect page

The new connect page for community growing still has some issues and needs some work:

[ ] Double name will be changed to 3bot name -> no . in between just like 'jdelrue' (will then refer to jdelrue.3bot.me as 3bot address)
[ ] Every one registering in the  registration app should be checked if the name does not already exists on eg https://login.threefold.me/api/users/jdelrue.3bot
[ ]  If they are already registered, a message should be shown 'You already registered through the 3bot connect app, thanks for being member of our community'
[ ] We an API to check if a user already exists on the connect thing when signing up a user, so can force the user to validate the same email address
[ ] Flow will have some changes, @Sacha96 will get back on this later today

This is a relic of the old referral flow which was abandoned. Closing


Ability to set frame X and Y position

Thanks for a cool encoder - much faster than the others I've tried for Node so far, and very nice that it is as flexible as it is! :)
You can already set a frames width and height independently of the other frames, but the (x,y) position of a frame is hardcoded to (0,0). If would be nice to be able to specify the x and y positions for writeFrame. (for optimization reasons - it becomes faster to generate a GIF where only a small part of the image changes from frame to frame)
It could for example be part of the "opts" parameter, something like:
        encoder.writeFrame(pixels8bit, width, height, {
            ...
            x = 123,
            y = 456
        });

Good call ! Will accept a PR for that. :)


Fix various warnings for C++11, C++14, and C++17 builds

There's also a fix for parallel builds in test/, and the generation of filenames in autocutee.mk when building from out of tree.


Adding URI serializer with tests

Adds serialization and de-serialization for URIs. Adds the appropriate JUnit tests. Resolves #28

Actually the object should be cast as a URI before calling to string(). I'll fix this when I get home


Update docs: aws-lambda plugin according to PR#2823

@akh00 Thank you. I'm :+1:.

Thanks! Merged to #508 which is the release/master branch and holds documentation updates for the changes currently accepted in Kong's master but not released.

Haha poor @akh00 everything he does gets ignored


Add new About page with project details and team members

Created an "About" page with data from our contributors. Also that item was added to the main menu and spacing was standardized for menu items.
TODO:

[x] Add page intro text
[x] Add big list of contributors text
[x] Add Diversity section text
[x] Decide what to do about the remaining content and layout suggestions. I would like @ccordoba12's input on this.

I don´t have any issue with your suggestions CAM, but we will need to wait for @ccordoba12 to give his input

>be Andres
>go to Files Changed tab

>click Add suggestion to batch

>repeat on other suggestions
>click Commit Suggestions

>write banger commit message
>click Commit changes

>whole dev team cheered

be Andres >go to Files Changed tab

click Add suggestion to batch

repeat on other suggestions

click Commit Suggestions

write banger commit message

click Commit changes

whole dev team cheered

Got it, thank you for your detailed tutorial.

I'm still waiting for the missing data of contributors, but the infrastructure to handle it is already there, so maybe this can be reviewed @CAM-Gerlach? check the tooltip on @ccordoba12's profile in the About page.

@conradolandia, the suggestions left by @CAM-Gerlach were for me to review since I was the one that came up with the initial text (fortunately I was ok with most of them, except one that I already fixed). So, for next time please don't be so eager to apply them. Thanks!
In addition, you can add multiple suggestions in a single commit for next time (instead of applying them one by one). That's described here (see Add suggestion to batch), which is also the way we prefer to apply suggestions in all our repos.

Thanks @ccordoba12 for the heads up. I did the first one, and then tried the batch, but for some reason, the buttons appeared disabled for me. I will wait for your comments before approving anything.


Adding deploymentModel Fields in azureresource properties in order to…

… be consumed to decide on what error message should be shown to user

verified the fix


WIP: Cookie storage session

With cookie_store support landed in reqwest, I'd like to retire user_agent which has some shortcomings due to being externally implemented. For my needs, the current support in reqwest for cookie_store is only lacking being able to provide an initial cookie storage and being able to retrieve the cookie storage at the end of a session.
I implemented a couple of different ideas to this end; this PR is the approach that seemed cleanest to me.
Approach
We introduce a simple trait CookieStorage which represents the actions reqwest needs for cookie storage, storing and retrieving cookies. We then introduce cookie::Session, which simply holds an instance implementing CookieStorage for use for the duration of a set of requests. The idea of the usage being:

Initialize a session with a CookieStorage implementer, (cookie_store::CookieStore::default(), loaded from disk, etc.). Session::new(cookie_store) mimics Client::new() in that the session will contain a default Client. Session::from_builder(cookie_store, client_builder) may be used to specify Client configuration.
Session provides a Deref<Target = Client> implementation, so it can be used as a stand in for Client.
During a series of requests, the storage may be manipulated via session.modify_cookie_store(|cookie_store| { ... }). The cookie_store here is a & mut to the concrete implementation provided at Session creation.
At the end of a series of requests, the concrete implementer may be retrieved via session.end(self), allowing the user to serialize the final state, etc.

To implement the above, the Session holds an Arc<RwLock<Box<dyn CookieStorage>>> alongside the Client instance. To provide access to and recover the original concrete type, the downcast crate is pulled in as a new dependency.
FIXMEs/TODOs
There are a handful of un-addressed issues:

CookieStorage is defined here directly in reqwest, along with the impl for cookie_store::CookieStore. It could instead be provided as a separate crate, and cookie_store could implement it for CookieStore; this would allow reqwest to remove the dependency on the cookie_store crate in lieu of some minimal cookie_storage crate defining the trait.
This implementation is a breaking change w.r.t. the API added when CookieStore was initially added; we could choose to implement this as a non-breaking change.

The cookie_store fn is now marked pub(crate) to prevent external usage and hide implementation details such as the Arc<RwLock<...>> of the new API.

For the downcast crate, the code generated by the downcast! macro call does not include documentation, so causes a failure of the deny(missing_docs) directive. This PR disables this check for now; I am not sure what the elegant way to deal with this, perhaps there is a simple workaround. We could implement by hand the functionality of downcast, avoiding this issue as well as the new dependency.
This implements both reqwest::cookie::Session and reqwest::async::cookie::Session. I am not sure it's a benefit to provide an async version of a Session; it may be non-obvious to e.g. attempt to modify_cookie_store() when requests are in flight, etc.
Also implemented is a fn Session::replace_cookie_store(). I am inclined to drop this functionality, especially if async support is retained. It seems like niche usage, and would be better to encourage users to end() their current session and start a new one.

Alternatives

An initial implementation is present in this branch, which simply exposes cookie_store::CookieStore directly. However, per prior discussion, a goal is to avoid exposing this in reqwest public API.
Another approach using the CookieStorage trait is this branch, which instead opts to implement it via a generic type. This avoids introducing the downcast dependency and type casting shenanigans, but ends up needing to thread the generic parameter through many types (Client, ClientBuilder, Request, etc.). The implementation works, but IMO clutters the API/typescape.

Thanks for all the hard work exploring options here! I have another alternative:

Add Client::cookies() -> impl Iterator that returns instances of reqwest::cookie::Cookie. This should allow you to export all the cookies when you're done with a Client.
Add some sort of Client::set_cookie(arg), such that you can import your cookies at the beginning. The trickier part here is that reqwest::cookie::Cookie doesn't have many setters, but we could probably add them.

@seanmonstar thanks for the feedback. I like your suggestion, it seems like a less intrusive approach (although exploring the above was a useful education in Any). I'd be happy to provide another PR implementing this approach, but some thoughts/questions:

For cookies coming out of the cookie store, there are properties/metadata that are derived or created by the store when receiving a Set-Cookie header. For example, the Max-Age property specifies a time of expiration from when the cookie is stored; so the cookie store would have an expiration date associated with the cookie that isn't explicitly stated in the actual Set-Cookie header. Similarly for other properties such as a cookie lacking a Domain attribute having metadata that it is a host-only cookie (and the source url). So, it seems the cookie we would want to return from any Client::cookies() fn would need to carry slightly more data; presumably this would be best implemented as either new fields on reqwest::cookie:Cookie or another struct (reqwest::cookie::StoredCookie). I do not think it would be appropriate to try and encode this sort of metadata by encoding it as a different cookie property (e.g. change a cookie received via Set-Cookie with a Max-Age attribute into one with a concrete Expires attribute when returning it from the store).
It would probably make sense then to modify the current API, Client::cookie_store(bool) fn to be something akin to Client::cookie_store(Option<Vec<Cookie>>); passing None would disable cookie storage (the default), Some(vec![]) initializing an empty cookie store and Some(cookies) initializing a store with some initial set of cookies.
I would still like to have a means of modifying the cookies in the store mid-session; a set_cookies or merge_cookies fn would suffice; removing cookies would be accomplished by merging cookies with an appropriate Expire attribute. What to do if a user were to call such a method without having actually initialized a cookie store (calling cookie_store on the builder) is less clear; should we create a store at that time? Silently ignore as a no-op? Or return a Result<(), Error> from the method, allowing us to signal a "no cookie store initialized" Error to the user?
A cookie store itself can potentially have some configuration, namely in the case of cookie_store it can be configured with a public suffix list. There are not a lot of configuration aspects here, so perhaps this is simply another parameter to Client::cookie_store(Option<Vec<Cookie>>, Option<publicsuffix::List>) (exposing another dependency, the publicsuffix crate).

So, it seems the cookie we would want to return from any Client::cookies() fn would need to carry slightly more data; presumably this would be best implemented as either new fields on reqwest::cookie:Cookie or another struct (reqwest::cookie::StoredCookie).

That makes sense.

I would still like to have a means of modifying the cookies in the store mid-session.

The tricky part is that the Client assumes all configuration happens during the builder, and then is relatively stable and put into an Arc. I realize internally the cookie store is in an RwLock, but I suspect the cookie store changing as responses arrive is less surprising than suddenly changing the store through a Client handle...
Would you be able to expand the why part?

A cookie store itself can potentially have some configuration, namely in the case of cookie_store it can be configured with a public suffix list.

I could imagine adding some reqwest::cookies::PublicSuffixList trait that a user could implement. Looking in cookie_store, seems only 1 method is needed, so we don't need to duplicate the entire API...

FYI, this week I've been working on a new cookies crate, to replace the cookie dependency in reqwest. It's to solve the annoyance of it relying on ring. Having used the cookie crate a bit in cookie_store, you may have thoughts on parts that could be designed different...
For mostly research reasons in the beginning, I settled on exposing a trait Cookie instead of specific type, since parsing from a string has different optimizations vs a user building one programmatically...

As an update, here's the cookies I've been working on...

Just touching base here as I've been quiet on this issue; I am interested in your latest comments and the trait-based cookie effort, I simply have not had time to push on the ideas for an updated PR.

I'm going to close for now, since it seems dormant.


Dependecies in master brach

Hi I pulled last master revision but I have problems with the following recipes in pyro.
Maybe it is time to add a pyro branch that tracks the current pyro.
Would this help you?

yes thanks because I found a configuration quite stable using pyro on the rpi3, if you want, I can help you to maintain updated the pyro branch with new backports.

@persuader72 It would be nice if you could take over maintenance of the pyro branch. If you like to discuss you can join the chat on https://matrix.to/#/#meta-homeassistant:matrix.org or if you prefer I can bridge the channel to IRC.


Add shell32 library

This was included by default for older Rust versions but is now needed explicitly as of 1.33.0.
Fixes #14

See the Reddit thread on the 1.33 release for more info about why this is necessary.

@raphlinus thanks, merging


Linter rule for http that check Accept header and response content-type are consistent

Now that we have the new linter system this seems like a good candidate
op getPng(@header accept: "image/png"): {
  @header contentType: "image/jpeg";
  @body image: bytes;
};

Example playground

@timotheeguerin Can you clarify why this would be invalid?

can you please clarify?

The response content type and the request accept should be in sync. It doesn’t make sense that you expect an Accept header with a value that is not in the potential response content types.
I think if you omit the accept header in the request it shouldn’t complain but if you do provide it it should make sure it match the response content types.

This scenario cannot/should not happen in OpenAPI because it is not allowed to specify an "accept" header parameter:

If in is "header" and the name field is "Accept", "Content-Type" or "Authorization", the parameter definition SHALL be ignored.

Perhaps we should have a linter rule that flags any "Accept", "Content-Type", or "Authorization" header parameters. I think that would also cover the case above.


Updated formatting for multi workspaces

Just wanted to do a little housekeeping as I've been frequenting this project's docs recently. In the documents you end up seeing !Simplest multiworkspace, which makes it seem like some markdown leaked into the final doc. Updated it with a better heading markdown item.
Feel free to reject if not important.

Hi @SpikePuppet! Thank you so much for your pull request.
Unfortunately, this documentation is generated from upstream docs - with a few edits and additions, which means that the next time we update this provider's schema, your changes would be overwritten.
The correct way to edit these docs would be to do it before generating the schema, in provider/resources.go via a targeted docsEditRule. Here is an example of such a pre-schema edit - basically you target the file ingested from upstream, look for the offending string, and replace/edit the content.
However - I do believe you have actually found a more general bug! In this case, there's leftover markdown because an asset was linked in the upstream doc. For the Pulumi registry, we stripped the link itself, but failed to remove the content of the square brackets. The correct fix would be to remove the !Simplest multiworkspace altogether, but because it looks like this is related to a docs processing step not giving us the desired results, I'm going to file an issue in pulumi/pulumi-terraform-bridge to track this.
In the meantime, I'll close this PR - feel free to open if you'd like to play around with DocEditRules - there is zero expectation here. :) Thanks again!


no reverse connections

thanks for the tool but all steps finished but i didn't recieved a reverse connection
i used msfconsole -r backdoor-apk.rc also i used the third party applications

thanks for the tool but all steps finished but i didn't recieved a reverse connection
i used msfconsole -r backdoor-apk.rc also i used the third party applications

Need more information. Are you attempting to receive a reverse connection on the same LAN/subnet or across the Internet for example?


Mainnet Replay Integration Test

In order to optimize the contract runtime in a backward compatible manner, we need to replay blocks on mainnet as an integration test.

@dwerner could you provide the status update on this work?

If I recall correctly, this is no longer possible because the auction changed to no longer use re-staking purses. This meant that at every switch block we produce a different hash from what we did before that change, and so re-executing blocks from before that change occurred results in a hash mismatch. @xcthulhu

This is an older ticket and the comment above indicates it is no longer relevant.   @fizyk20 can you please confirm Dan's comment and close this out if it is no longer possible.  Thank you

If I understand correctly, this ticket was about benchmarking block execution times using data from a real network. If so, then as Dan said at the end - it is possible provided that we use recent enough blocks. I think the ticket still has merit and we might want to do that at some point.


Added check-if-form route

added a new route and funtion to test if a Pdf as AcroForm fields. returns true or false

looks good


disabling features is edition dependent

Not all features may be disabled depending on the SKU being used.

Docs Build status updates of commit 490ab05:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/winserver2019-ps/mmagent/Disable-MMAgent.md
:white_check_mark:Succeeded
View (WindowsServer2019-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit bbbdbca:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/winserver2019-ps/mmagent/Disable-MMAgent.md
:white_check_mark:Succeeded
View (WindowsServer2019-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Add fieldlist to docsum request.[run-systemtest]

Only return fields requested from the backend.

@bratseth @geirst @toregge PR

Test added, and conversion done.
Will look into streaming search separately.


[HUDI-7350] Make Hudi reader and writer factory APIs Hadoop-independent

Change Logs
Abstract io reader and writer to de-hadoop
Impact
de-hadooping for trino
Risk level (write none, low medium or high below)
low
Documentation Update
N/A
Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

passed ci:


ci: fix release workflow

Resolves #123.

Updates the knope version
Per @dbanty, --prerelease-label calls a workflow but as a prerelease. So to deduplicate the knope.toml (and avoid issues like with missing the Release step), the file now contains only a release workflow, and the CI calls it with --prerelease-label for prereleases.

One potential argument against this is that the prerelease commit message now says "release" instead of "prerelease". Since it includes the release tag, including a rc.x part, I think that conveys "prerelease" appropriately.
That said, I am open to arguments for restoring a "prerelease" workflow.

We're probably going to have a problem when we get to 0.15.0, since there is already a 0.15.0-rc.0 on crates.io. Do you think we should just tag 0.15.1 and go from there?

We're probably going to have a problem when we get to 0.15.0, since there is already a 0.15.0-rc.0 on crates.io. Do you think we should just tag 0.15.1 and go from there?

I am hoping that, once we reach that point, knope will detect the prerelease and automatically skip it.
If I read this function correctly, it will ignore the latest prerelease if the major/minor/patch versions do not match the current stable.
... Which made me wonder if we'll run into an issue where we keep hitting 0.14.3-rc.0. It looks like yes, so we might need a new PR to knope :/

Updated the knope version to 0.7.0 and fixed some clippy lints so that CI passes.

@Shadow53 Is this PR still relevant?


Elm UI PoC

Did a little UI for testing without having to do everything manually through postman. Used Elm because it seemed like a good excuse to do more Elm things :smile:
You'll have to create a User through postman before you'll be able to login and actually do things.
Adding instructions to README for getting everything up and running.

This closes #25


I can run Demo

When I execute react-native run-ios I get that error:

Somebody know what happens?

@icastillejogomez I've updated the example project to RN 57.5 and just tested this, it's working now. Sorry about that! Thanks


Support TailwindCSS VScode extension

Hello ! Thanks for this awesome compiler/interpreter :) !
I'm trying to have a minimal dev environment, and your tool does a wonderful job trimming a bunch of dependencies, but I also like using Tailwind's VSCode extension, which seems to rely on the main npm package
Here's the error
Failed to initialise: Error: Cannot find module 'tailwindcss/package.json'
Require stack:
- c:\Users\leo\Projets\cervo-svelte\noop.js
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:972:15)
    at a (c:\Users\leo\.vscode-oss\extensions\bradlc.vscode-tailwindcss-0.5.9\dist\server\index.js:13:38328)
    at s (c:\Users\leo\.vscode-oss\extensions\bradlc.vscode-tailwindcss-0.5.9\dist\server\index.js:13:38437)
    at e.exports (c:\Users\leo\.vscode-oss\extensions\bradlc.vscode-tailwindcss-0.5.9\dist\server\index.js:13:38459)
    at resolve (c:\Users\leo\.vscode-oss\extensions\bradlc.vscode-tailwindcss-0.5.9\dist\server\index.js:13:156730)
    at c:\Users\leo\.vscode-oss\extensions\bradlc.vscode-tailwindcss-0.5.9\dist\server\index.js:13:161602
    at c:\Users\leo\.vscode-oss\extensions\bradlc.vscode-tailwindcss-0.5.9\dist\server\index.js:13:156672
    at L (c:\Users\leo\.vscode-oss\extensions\bradlc.vscode-tailwindcss-0.5.9\dist\server\index.js:13:156738)
    at L (c:\Users\leo\.vscode-oss\extensions\bradlc.vscode-tailwindcss-0.5.9\dist\server\index.js:13:156957)
    at c:\Users\leo\.vscode-oss\extensions\bradlc.vscode-tailwindcss-0.5.9\dist\server\index.js:13:161544 {
  code: 'MODULE_NOT_FOUND',
  requireStack: [ 'c:\\Users\\leo\\Projets\\cervo-svelte\\noop.js' ]
}
(node:3132) UnhandledPromiseRejectionWarning: TypeError: Cannot read property 'classNames' of undefined
    at Te (c:\Users\leo\.vscode-oss\extensions\bradlc.vscode-tailwindcss-0.5.9\dist\server\index.js:13:116588)
    at Ge (c:\Users\leo\.vscode-oss\extensions\bradlc.vscode-tailwindcss-0.5.9\dist\server\index.js:13:124048)
    at c:\Users\leo\.vscode-oss\extensions\bradlc.vscode-tailwindcss-0.5.9\dist\server\index.js:13:124397
    at Array.forEach (<anonymous>)
    at c:\Users\leo\.vscode-oss\extensions\bradlc.vscode-tailwindcss-0.5.9\dist\server\index.js:13:124368
    at Array.forEach (<anonymous>)
    at c:\Users\leo\.vscode-oss\extensions\bradlc.vscode-tailwindcss-0.5.9\dist\server\index.js:13:124333
(node:3132) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 1)
(node:3132) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

I'm also opening an issue on the VSCode extension issue tracker, since I guess it might need cooperation between the two projects to make things work : https://github.com/tailwindlabs/tailwindcss-intellisense/issues/264

Well, this issue is very tricky. It seems that tailwind IntelliSense forces you to install tailwindcss during development, and it cannot be installed globally. I tried some methods to skip, for example, only keeping package.json still failed.
I believe adding tailwindcss to your dependencies should go against your original intention of having a minimal dev environment, because it will add about 50 dependencies. . .
So, we probably only have one choice now, to create a windicss intellisense.

tailwindcss intellisense belongs to tailwindlabs. They don't want to see a compiler that does not depend on tailwindcss, so I am sure they will not fix this problem with a high probability.

Yeah, I haven't dug enough in tailwind IntelliSense internals to understand what they does, but maybe , with a bit of help, they could adapt it easily ? (And, who know, maybe base their plugin on windicss for tinier deps 😇) ?
We could at least see if they straight-up don't want it... I mean, Tailwindlabs's income source is from Tailwindui, not tailwindcss compiler itself :) with your compiler you even allow them to get more potential customers eheh

Well, what you said makes sense. Wait for them to reply. If it doesn't work, I will start this project. I don't think it will take a long time.

We will wait for the last day, if there is still no reply, I will post a replacement for windicss.

Well, the ticket I opened on Tailwind's side have been closed, since it's not their goal to support things that aren't Tailwind CSS.
https://github.com/tailwindlabs/tailwindcss-intellisense/issues/264#issuecomment-782048491

Don’t worry. I basically finished windicss-intellisense. Will be published soon.

@voorjaar do you have a preview version out there, I think VSCode has this option? Think the community will be good testing audience

Could we have Webstorm WindiCSS Intellisense? I use Webstorm, and I am very excited about it.

https://github.com/windicss/windicss-intellisense is here. It's still a beta version, you guys can test it and create some issues, and then I will fix it. Thank you.

Could we have Webstorm WindiCSS Intellisense? I use Webstorm, and I am very excited about it.

I will take a look, I like meeting people's needs.


8-beta.4 fails to load under SystemJS

Error: (SystemJS) XHR error (404 Not Found) loading http://foo.baz.com/appd/app1/node_modules/rxjs/operators.js
...
Error loading http://foo.baz.com/appd/app1/node_modules/rxjs/operators.js as "rxjs/operators" from http://foo.baz.com/appd/app1/node_modules/ng2-ui-auth/bundles/ng2-ui-auth.umd.js

Seems you have an older version of rxjs. the beta uses Rxjs 5.5.0+ that have the new "lettable" operators that seems missing from your version

No, I'm on 5.5.2. I think it's a SystemJS thing (though that's an internal rxjs issue). I added the following to my systemjs.config.js to get around it:
map: {
...
  'rxjs/operators': 'node_modules/rxjs/operators/index.js'
}

I'm guessing it will be common in any library using the new rxjs operators... I'm closing this issue. If you think there is a way I could mediate this in my end feel free to reopen this.


Update cif2cell feedstock to version 2.0.0a3

Current cif2cell package is outdated, PR to update it to latest version (2.0.0a3):
https://github.com/torbjornbjorkman/cif2cell/releases/tag/2.0.0a3
Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe:

License is not an SPDX identifier (or a custom LicenseRef) nor an SPDX license expression.

Documentation on acceptable licenses can be found here.

Thanks @anish-mudaraddi , this also fixes https://github.com/conda-forge/cif2cell-feedstock/issues/2
@npavlovikj Would you be able to have a look and merge this?

@npavlovikj just following up on this as it's been a couple of weeks, any chance you can take a look?

hey @elichad, would you be willing to be added as a maintainer of this feedstock?
Since @npavlovikj was last active on this repo in September 2020, this seems to me that either you are me are going to have to take care of this.
If you are willing to be added, there is a mechanism where you open a special issue and a corresponding PR is automatically created.
If we don't hear back from the maintainer of the feedstock, a conda-forge admin can also merge the PR.

@ltalirz It seems to me that you are more involved in cif2cell development, so you would probably make a better maintainer of this feedstock.

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

noarch: python recipes are required to have a lower bound on the python version. Typically this means putting python >=3.6 in both host and run but you should check upstream for the package's Python compatibility.

For recipe:

License is not an SPDX identifier (or a custom LicenseRef) nor an SPDX license expression.

Documentation on acceptable licenses can be found here.

@conda-forge-admin , please rerender

Thanks a lot @chrisburr !

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Thanks folks!


[DOC-1647] - Small troubleshooting article

Small troubleshooting article
Preview link:

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app


interfacing ch376s module with build in sd

hi @djuseeq,
thank you for this great library. i saw quite heavy mode required to interface the board with sd. i saw ch376s module sold on ali with build in sd as below:

i am not sure if this board use sd as storage and the usb port as com interface or both act as flash storage but i hope by any chance to have this board supported by your library. it would be great to have this library compatible with sd without require to do the mods mentioned for the current supported module. let me know how i can sent you a module. will try my best to ship this to you.
thank you.

Hi @tsctrl , thanks it's my pleasure. Yes it's need some tinkering with the board if someone want it to use for SD as well. I think the board on the photo is a developement board, so that should be ok. The library supports the SD card mode, with calling setSource(srcDrive); function, where srcDrive's value can be a 0 = USB(default), 1 = SD card mode. Please check the example file , and enable the ability to use SD card in line 59, but be careful with it if you use a normal CH376 module, in this case you must to do some modification on the board otherwise you risking to damage the CH chip.

hi @djuseeq, thank you for your reply. could you elaborate more which modification in the normal ch376 board that was required to avoid the ch376 chip damage?
i only see the modification shown is to hook the sd to connect to vcc, gnd and sd pins. do this modification is only relate to ch376 and not ch376s chip? i am not able to find the different between both chip version. i could see dedicated pin to connect the sd in schematic so wondering if the modification is to connect to this pins or is some other approach?
thank you!

CH376S can handle SD card, CH376T not. HERE is a little note how i made it with this "blue" modul. For some reason the SD_CS pin is connected to the ground on the pcb and that not make any problem as long as you use with USB only. So check with these pins with DMM against VCC and GND, is any of them is shorted to these lines then you have to isolate it otherwise you risk to damage the chip.


Steadily increasing memory usage

After a 2 hour run of all our mocks except gl* and mapbox* using test/image-make_baseline.js (351 mocks, taking about 55 seconds per run), against image-exporter running as an imageserver in a Docker container, memory usage grew steadily:
https://plot.ly/~JodyMcintyre/2209/

When the run was stopped, memory usage decreased a bit then leveled out.
Examination of ps results showed that two electron processes (probably the plot image and plot thumbnail processes) were responsible for the memory usage:

A similar issue was observed in a 12 hour run of 3 imageservers in our staging environment:

As a workaround for this issue we could restart Electron after a reasonably large number of requests (e.g. 1,000). I'm going to look into this but a fix for the root cause is needed at some point.
@etpinard @monfera FYI

I was able to reproduce the error feeding sequential request of a large json file ( 4.7M ) and every single time at 233 request the process crash. The larger the file is the quicker the process crash.

it's not a resources issue as no message appear in dmesg nor a segfault.
I've tested with current version of electron and 2.0.9 same result.

I've tested with the following file : success/ee151de8-d07e-4d19-a2b6-400a2292c609_200.json
from : https://github.com/plotly/streambed/issues/9865#issuecomment-360823004
 for i in {0..5000}; do curl -d @ee151de8-d07e-4d19-a2b6-400a2292c609_200.json http://localhost:9091/ ;echo $i;done
capture of orca's container when the 233rd requests happen:
12544ae95ac6        eager_banach          149.46%             2.447GiB / 4GiB       61.18%              0B / 0B             0B / 303kB          153
error message :
<--- Last few GCs --->
[44:0x2542baa0000]   286865 ms: Mark-sweep 2047.7 (2087.0) -> 2047.7 (2087.0) MB, 1668.0 / 0.0 ms  allocation failure GC in old space requested
[44:0x2542baa0000]   288532 ms: Mark-sweep 2047.7 (2087.0) -> 2047.7 (2087.0) MB, 1667.2 / 0.0 ms  last resort
[44:0x2542baa0000]   290208 ms: Mark-sweep 2047.7 (2087.0) -> 2047.7 (2087.0) MB, 1675.5 / 0.0 ms  last resort
<--- JS stacktrace --->
==== JS stack trace =========================================
Security context: 0x1783e7b2d681
0: builtin exit frame: parse(this=0x1783e7b13ec9 ,0x2a6c162ff499 <Very long string[1514304]>)
2: /* anonymous */ [/var/www/image-exporter/src/app/server/create-server.js:144] [bytecode=0x27ecb9d52d21 offset=51](this=0x631dc09bb61 <JSGlobal Object>,err=0x363dedf82201 <null>,_body=0x2a6c162ff499 <Very long string[1514304]>)
4: onEnd [/var/www/image-exporter/...


How to loop over JSON response if the response is a few levels deep

if let dict = response.responseObject as? Dictionary<String,AnyObject> {
        for (key, value) in dict {
            if key == "result" {
                print(value["videos"])
            }
        }

The videos property in the JSON contains a further level of video objects. How can I access the individual properties on each video object, preferably without resorting to using another library i.e JSONJoy-Swift
Apologies for the noobish question - it has taken me hours to get to this point via many cocoapods / command line tool related rabbit holes so any help much appreciated!

I'm not sure if I understand your question correctly, but you probably can use value["videos"].array and then loop over it.

@SebastianOsinski thanks for the reply. Maybe this will help - link expires in an hour
http://pastebin.com/tfYnjvhX
I want to loop over all the videos and extract e.g. the title

Sorry, I got confused, and my answer was valid if you would use SwiftyJSON. I spent couple of days working with SwiftHTTP and SwiftyJSON and somehow I mixed them. Sorry!

I sorted it with:
        for (key, value) in dict {
            if key == "result" {
                let videosArr = value["videos"] as! NSArray

                for vid in videosArr {
                    if let url = vid["videoFullURL"]! {
                        print (url)
                        //shell("/usr/local/bin/youtube-dl", "\(url)")
                    }
...


Might work on windows?

Hi. I've been asking myself.. What would happen if I'd try to compile and run it on Windows. Would I get a vac ban?

It wont run

Ok. Thanks m8


fix progress wheel after file deletion

Pull Request Checklist:

[x] Positive Test Case Written by Dev

[ ] Automated Testing

[ ] Sufficient User and Developer Documentation

[ ] Passing Jenkins Build

[ ] Peer Code review and approval

Positive Test Case

Create multiple folders and add some files to them.
Delete the folders.
After some time, the operation seems to complete and the pinwheel stops spinning as expected.

Resolves #4066

METRIC
VALUE

https://sonarqube.cuahsi-workstation.com:9000/dashboard?id=hydroshare-4529


ref: Rename ClientOptions.SendDefaultPii to ClientOptions.SendDefaultPII

re: https://github.com/getsentry/sentry-go/pull/485#discussion_r1011500255

I think it's ok to stay unified with the Go ecosystem, especially if we expand on better linting in the future.
In terms of DevEx, IDEs should autocomplete on this just fine.

Yes Sir! https://github.com/getsentry/sentry-docs/pull/5708


[Bug Report]: Missing properties for Compute/Gallery for Bicep when creating images

Describe the bug
When trying to create a Compute/Gallery with images in array, the properties "isAcceleratedNetworkSupported" and "isHibernateSupported" aren't passed.
They exist as parameters for Compute/Gallery/Image but when comes the moment to create images from Compute/Gallery the properties are just not passed.
It is a fairly simple bug to solve, as a lot of the stuff surrounding the solution is already in place.
To reproduce
Deploy a Compute/Gallery via Bicep with an image array specifying isAccelereatedNetworkSupported to true.
You will that the property is not applied!
I believe the explanation of this bug is quite simple. The Compute/Gallery main.json doesn't pass the property.

galleries_images is missing the params for "isAcceleratedNetworkSupported" and "isHibernateSupported".
Code snippet
param galImages = [
  {
    hyperVGeneration: 'V2'
    maxRecommendedMemory: 16
    maxRecommendedvCPUs: 8
    minRecommendedMemory: 4
    minRecommendedvCPUs: 2
    isAcceleratedNetworkSupported: true
    name: 'xrserver-devkit'
    offer: 'xrserver-devkit'
    osState: 'Generalized'
    securityType: 'TrustedLaunch'
    osType: 'Linux'
    publisher: 'example'
    sku: 'xrserver_devkit'
  }
  {
    hyperVGeneration: 'V2'
    maxRecommendedMemory: 16
    maxRecommendedvCPUs: 8
    minRecommendedMemory: 4
    minRecommendedvCPUs: 2
    isAcceleratedNetworkSupported: true
    name: 'xrserver-scaleset'
    offer: 'xrserver-scaleset'
    securityType: 'TrustedLaunch'
    osState: 'Generalized'
    osType: 'Linux'
    publisher: 'example'
    sku: 'xrserver_scaleset'
  }
]

module gallery '../../modules/compute/gallery/main.bicep' = {
  name: '${uniqueString(deployment().name, location)}-dev-cgcom'
  params: {
    name: galName
    location: location
    enableDefaultTelemetry: true
    images: galImages
    roleAssignments: [
      {
        principalIds: [
          galUserAssignedIdentity.outputs.principalId
        ]
        principalType: 'ServicePrincipal'
        roleDefinitionIdOrName: 'Reader'
      }
    ]
    lock: ''
  }
}

Relevant log output
No response

Hey @ReneHezser,
I just migrated this issue over from CARML. Please take a look and triage if still relevant :)


Changelog or New Feature List upon opening a version for the first time

Self-explanatory.

this has been added as a feature, so this has no need to be open anymore


[Feature Request]: Add caikit/tgis as a OOTB Custom Serving Runtime

Feature description
Based on the effort of https://github.com/opendatahub-io/caikit-tgis-serving/pull/49, we would need caikit as an OOTB Custom Serving Runtime for users to manage.
Describe alternatives you've considered
Users can already add caikit as a Custom Serving Runtime, but we would need to ship it as an Out Of The Box resource.
Anything else?
No response

Migrated to:  https://issues.redhat.com/browse/RHOAIENG-1268 - moving to closed


Explore other hooks

http://revelry.co/conditioning-yourself-with-git-hooks/

As this hook's purpose is only the pre-commit action, this issue belongs to another scope.


Add load pdf from url  use okhttp3

Add load pdf from url  use okhttp3

Please use only english language
Please reformat code
You cannot use Environment.getExternalStorageDirectory() as is, current state must be checked with Environment.getExternalStorageState()
You can extend existing sample, but not replace it
Your sample does not work, because you have to ask user for WRITE_EXTERNAL_STORAGE permission starting with Android 6.0
After requesting permissions it still does not work, PDFView remains white
What will happen when configuration change occurs while downloading file?

PdfViewCache directory is never cleaned
When you move app to background document stops downloading and becomes corrupted

@barteksc thanks

Is it library working??

@barteksc @shxdos any updates for this?

@shxdos @barteksc Any update? because the library doesn't work for URL.


Add http endpoint /index

Add an endpoint for /index that we can query the current index of each node.  This is primarily for testing.
This could initially return a uint64 as the body of the response.
@benbjohnson please add any additional requests you would like to see on this and I'll knock it out.

Would this be more useful if you could pass in an index number that it should block and wait for and the return after that?

I think that's fair.  If you pass in an id like /?id=10 the endpoint could poll.  I would think we would want some type of timeout, maybe 5s, or something, to force an error (maybe shorter).  Then, just by getting http.StatusOK you know the condition was satisfied and you can move on.  Would make for really simple testing.

yeah, +1


trigger

test plantuml trigger, @l3r8yJ take a look, please

PR-Codex overview
This PR adds a new sequence diagram to the diagrams directory.
Detailed summary

Added sequence.puml file to diagrams directory
Defined GitHub participant in the sequence diagram
Added user, bot, hub, and notifications participants to the sequence diagram
Shows communication flow between user, bot, hub, and notifications with tokens and chat IDs

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@rultor merge

@l3r8yj OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@l3r8yj Done! FYI, the full log is here (took me 1min)


feat: library calls

In this section, we show an example of how to perform a Library Call in Cairo using the high-levels LibraryDispatcher and LibraryDispatcherTrait which perform a library_call syscall (reference here https://docs.starknet.io/documentation/architecture_and_concepts/Contracts/system-calls/#library_call)
Library calls are used to invoke functions from classes directly, not contracts.
It is already covered in surface in the Cairo book but the section needs to be reworked.
This example will demonstrate the implementation of a very simple contract intended to be used as a library, and how we can use XXXLibraryDispatcher to make library calls to this contract from another contract.
A simple could be a contract named MathUtils with a add function and from our contract we could call IMathUtilsLibraryDispatcher.add(a,b)

Just updated the description @neelkamath, sorry for the confusion

Is this issue still open? I'd love to work on it @julio4 @enitrat

Is this issue still open? I'd love to work on it @julio4 @enitrat

Hello, yes I can assign you!

Is this issue still open? I'd love to work on it @julio4 @enitrat

Hello, yes I can assign you!

Okay, thank you

@OkoliEvans Do you need any help for this?

I started working on it this morning, I have not had any blocks yet. I will reach out if I need any assistance @julio4


Encountering page footer issue if margins are set on four sides

Hi Nathan,
There is a page footer issue if I set the margins on four sides of the page like this:
var report = new Report("test.pdf", {paper: "letter"});
...
report.margins({left: 50, bottom: 5, right: 0, top: 0});
...
The footer size is not being considered when rendering resulting to page footer being printed on the next page.
However, when I set the margins like this:
var report = new Report("test.pdf", {paper: "letter"});
...
report.margins(50);
...
I am not encountering any issue. I hope you can look into this. Thank you!

Please note this issue is now fixed in v1.00


ARC NFC Reader

Hello! What kind of NFC reader did you use for this? Do you have any links to one that would work?

Its this one I tested with:


Install problem: urgent help needed

I first met several problems when installing all required packages and then open the requirement file, installing each package separately.
However, later I met this problem:
(qstraderp3) bash-3.2$ python buy_and_hold_backtest.py
Traceback (most recent call last):
File "buy_and_hold_backtest.py", line 7, in
from qstrader.trading_session import TradingSession
File "/Users/pwl1972/venv/qstraderp3/lib/python3.7/site-packages/qstrader/trading_session.py", line 12, in
from .statistics.tearsheet import TearsheetStatistics
File "/Users/pwl1972/venv/qstraderp3/lib/python3.7/site-packages/qstrader/statistics/tearsheet.py", line 15, in
import seaborn as sns
File "/Users/pwl1972/venv/qstraderp3/lib/python3.7/site-packages/seaborn/init.py", line 17, in
from .widgets import *
File "/Users/pwl1972/venv/qstraderp3/lib/python3.7/site-packages/seaborn/widgets.py", line 8, in
from ipywidgets import interact, FloatSlider, IntSlider
File "/Users/pwl1972/venv/qstraderp3/lib/python3.7/site-packages/ipywidgets/init.py", line 20, in
from IPython import get_ipython
File "/Users/pwl1972/venv/qstraderp3/lib/python3.7/site-packages/IPython/init.py", line 48, in
from .core.application import Application
File "/Users/pwl1972/venv/qstraderp3/lib/python3.7/site-packages/IPython/core/application.py", line 24, in
from IPython.core import release, crashhandler
File "/Users/pwl1972/venv/qstraderp3/lib/python3.7/site-packages/IPython/core/crashhandler.py", line28, in
from IPython.core import ultratb
File "/Users/pwl1972/venv/qstraderp3/lib/python3.7/site-packages/IPython/core/ultratb.py", line 117,in
from IPython.utils import path as util_path
File "/Users/pwl1972/venv/qstraderp3/lib/python3.7/site-packages/IPython/utils/path.py", line 19, in
from IPython.utils.process import system
File "/Users/pwl1972/venv/qstraderp3/lib/python3.7/site-packages/IPython/utils/process.py", line 19,in
from ._process_posix import system, getoutput, arg_split, check_pid
File "/Users/pwl1972/venv/qstraderp3/lib/python3.7/site-packages/IPython/utils/_process_posix.py", line 24, in
import pexpect
File "/Users/pwl1972/venv/qstraderp3/lib/python3.7/site-packages/pexpect/init.py", line 75, in
from .pty_spawn import spawn, spawnu
File "/Users/pwl1972/venv/qstraderp3/lib/python3.7/site-packages/pexpect/pty_spawn.py", line 15, in
from .spawnbase import SpawnBase
File "/Users/pwl1972/venv/qstraderp3/lib/python3.7/site-packages/pexpect/spawnbase.py", line 218
def expect(self, pattern, timeout=-1, searchwindowsize=-1, async=False):
^
SyntaxError: invalid syntax
It's quite hard to believe that it fails because of invalid syntax. Anyone met similar problems before? I used Anaconda and python3.7

After creating a python 3.4 environment, this problem is perfectly solved. However, when run the test file, there is no tearsheet result. I can see the return and sharpe ratio though.

https://github.com/mhallsmoore/qstrader/issues/215

Hi @zf480,
I pushed up some changes to the codebase yesterday to ensure compatibility with Python 3.5, 3.6 and 3.7. In addition threquirements.txt file has been updated for the latest versions of the dependencies.
In terms of the tearsheet disappearing there was a slight design issue in the code that prevented the tearsheet from 'blocking' the rest of the backtest process. This has now been modified such that the tearsheet displays by default, allowing you to save a figure.
Regards,
Mike.


Add num_updates_per_train_step hyperparameter

Previously we did not have this hyperparameter. It's needed in the Atari100K benchmark for the OverTrainedRainbow. The reason for adding to the general algorithms is that I think it's kind of an important hyperparameter in RL algorithms in general.  I'll be glad to hear your comments on this.

I'd close this for now; we can pick it up later.


Add effectful-plugin (#54)

FWIW the best course of action is to port cleff-plugin instead which has much cleaner implementation.


biGraph is a misleading name

In conventional mathematical terminology, "bigraph" is short for "bipartite graph".  Within CertiGraph, "biGraph" appears to mean "binary graph", that is, a graph where every nontrivial node has 2 out-edges.   This is unnecessarily confusing.  I suggest you rename the module, and all the relevant files.  Perhaps binGraph and demo/mark_bin.c, et cetera.

It is done now.


Clarify instructional comment

There were two different years in these instructions, and it seemed confusing.

Thanks!


multiple npm installs can lead to unpredictable builds

We have provided a fix in v5.2.1 that skips calling npm install if the node_modules directory exists, but this should be seen as a temporary fix. Look into the underlying cause.
See #84

closing for now - if we change how we interact with the pie installation directory we can revisit


Startdate is not localized in the Editor

Thanks for reporting that. However, the Opencast Editor has its own repository (confusing, I know). Please report issue related to the editor over there: https://github.com/opencast/opencast-editor/issues


Webp support

Any update regarding webp support?

No. We hope to make it easier for users to add their own image decoding, but haven't had a chance to implement that. AlamofireImage will likely never include support for WebP, unless the OS adds it, but hopefully we can enable users to add it themselves.


Added small social meta snippets

Added in minimal meta for Twitter and Facebook. More could be added but I believe any omitted will fall back to page title and description
closes #14

Thanks for your PR @AaronLayton .
I had to augment the headers a bit because image URLs need to be absolute and title & description are required for Twitter cards.
Gonna be live in a few minutes :)

Good catch @surma , I didn't know what the bare minimum was for Twitter, I thought my may fallback to the document title, description.
Wasn't sure on the image as well if I needed a hard coded URL .Thanks for the merge


[Hotfix][Soap] ZF-11937

Added support for xsd:long is Zend_Soap_Wsdl

sync svn r24581

Thank you.


Issue when jquery ui is included with materialize

The tool-tips doesn't show up.
Please have a look into it asap

Please put more effort into describing your issue. Have you tried recreating this issue with a codepen or a  jsfiddle where you can add these external libraries?

Probably a naming conflict, you can change the name of the function to avoid this.

@acburst Am working on a project in Yii2, with materialize and recently i added jquery ui asset for sortable functionality. It works fine in development environment, if i run the project in production environment then the tooltips are not at all generated in the view files (even if i call the function from console it doesn't initialize the tooltips)
Difference between dev and prod environment is that, in prod environment the assets are compressed into one js and css file using http://www.yiiframework.com/doc-2.0/guide-structure-assets.html
So i don't think i can replicate using codepen or a jsfiddle

Try resolving the naming conflict as I said. We also don't provide support nor do we expect Materialize to work perfectly in conjunction with other UI libraries


Error: no parameter named 'block_devices'

Hi,
I'm new to puupet and trying create EC2 launch configuration using "ec2_launchconfiguration" resource.
but for some reason I keep getting below error.
Puppet version: 4.3.1
AWS module version: 1.4.0
config:
ec2_launchconfiguration{ $lc_name:
security_groups => $service_conf[lc_params][security_groups],
user_data => template($ud_path),
key_name => $service_conf[lc_params][key_name],
block_devices => [
{
device_name  => '/dev/sda1',
volume_size  => 8,
}
],
vpc => $service_conf[vpc],
region => $service_conf[region],
instance_type => $service_conf[launch_config][instance_type],
image_id => $service_conf[launch_config][image_id],
ERROR:
Error: no parameter named 'block_devices' at /etc/puppetlabs/code/environments/production/modules/dmi_dr_asg/manifests/init.pp:20 on Ec2_launchconfiguration[dr-ms-plb-nvirigina-lc-1479147342] at /etc/puppetlabs/code/environments/production/modules/dmi_dr_asg/manifests/init.pp:20 on node ip-10-30-51-13.ec2.internal
I am not sure why this is happening, any help will be appreciated.

I'll create another issue with proper details


Deploy a default budget on all subscriptions under the assigned scope fails when setting an Action group parameter

What happened? Provide a clear and concise description of the bug, including deployment details.
This custom policy deploys ok when an action group isnt set. When an action group is set in the parameters at time of assignment it fails.
Please provide the correlation id associated with your error or bug.
Resource
/subscriptions/xxxxxx/providers/Microsoft.Consumption/budgets/budget-set-by-policy
Operation name
Create and update budgets
Time stamp
Wed Apr 24 2024 10:24:23 GMT+0100 (British Summer Time)
Event initiated by
6223b87e-5c7e-4767-bf85-f2fbcfaa95f1
Error code
400
Message
Bad Request. Request ID: b082aee9-fc42-45f3-9a33-0e0fb4870334
What was the expected outcome?
The Action group should eithe rbe cerated or use an existing Action group tha has already been created in the Subscription and can be referenced by name
Relevant log output
No response
Check previous GitHub issues

[X] I have searched the issues for this item and found no duplicate

Code of Conduct

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

@gerrynicol Took me a bit of time to figure out what is going on here.
I think the issue may be related how you are assigning the policy. You are correct, we do not deploy an action group as part of the budget policy, so the action group must exist in order to use it with this policy.
Also, if you just provide the action group name, you will get that error (400 Bad Request), you need to provide the full resource ID of the action group like:
/subscriptions/mysubscriptionID/resourceGroups/myresourcegroup/providers/microsoft.insights/actiongroups/AGTest
Could you please try again with the full resource ID of the action group?

@Springstone. Nice one. Thanks for looking at that so quickly. Yip, that works a treat :)


Make docs compatible with old client versions

This changes the docs so that they mention the version from which certain bug fixes are available.

@Simn Do the documentation changes here look good to merge? They're required to advance with #567.

I'll merge this for now, we can always come back and change this if there are any issues.


Include fund/loan name/icon or (📁 Multiple) in account transaction list

ChatGPT suggests the following:
from t in Transaction,
join: l in assoc(t, :line_items),
join: f in assoc(l, :fund),
group_by: t.id,
select: %{
  id: t.id,
  name: fragment(
    "CASE
      WHEN count(l.id) = 1 THEN max(f.name)
      ELSE ?
    END",
    "multiple"
  ),
  line_item_count: count(l.id)
}

Closed by 4eeba1e5eb393e8c248073e887b137bafc84fa78


hub fork fails with "Not a git directory"

Hi,
I'm able to clone github repo, commit to my github repo using hub without any issues.
But when i try hub fork github/hub it fails with fatal: Not a git repository.
hub version shows
git version 2.1.4
hub version 2.3.0-pre8

I'm using 32 bit Debian GNU/Linux 8.5 (jessie) if that helps.
I've also tried by setting the verbose flag,
~/devel$ HUB_VERBOSE=1 hub fork github/hub
$ git config alias.fork
$ git rev-parse -q --git-dir
fatal: Not a git repository

Do i need to configure anything else for forking a repo? what are the steps to fork a repo? I would be happy if you could point me towards any documentation (sorry, i couldn't find any). So my knowledge is only restricted with whatever help the hub command showed me.
Thanks,
Hari

btw, thanks for making this tool :)

looks like we need to clone the git repo before we can fork the repo. Found the command usage at https://hub.github.com. Closing the issue.

@rdotdk I totally understand how you got confused about the hub fork command. Plenty of people expected that they can fork a repo directly on GitHub and clone that instead. I'm thinking of supporting an argument given to fork to specify the repo name.

@mislav that'll be very useful. Also I think the commands and examples should be easily accessible. It took me a while to find the example at hub.github.com

The documentation given by hub help fork mentions that you're working the
"current project". I'm gonna try to make the documentation clearer and more
accessible.

I only did hub with no arguments to check out what arguments are available and jumped on trying hub fork. I didn't look at hub help fork. I have become lazy by all those  github projects that have extensive examples either in the Readme or in the wiki.

I'm willing to help out with documentation :)


Installation - Error: sed: couldn't write 136 items to stdout: Broken pipe

I have been trying to install bcbio in the server that I have been using.  I encountered the following error message:
2018-05-22 11:51:29 (32,2 MB/s) - ‘variation/dbsnp-150-orig.vcf.gz’ saved [7554058973]
sed: couldn't write 136 items to stdout: Broken pipe
sed: couldn't write 139 items to stdout: Broken pipe
gzip: stdout: Broken pipe
/marconi_work/LI08p_EMUT/bcbio/genomes/Hsapiens/GRCh37/txtmp/ggd-run.sh: line 10: 42606 Exit 1                  zcat variation/dbsnp-$version-orig.vcf.gz
42607 Exit 4                  | sed "s/^chrM/MT/g"
42608 Exit 4                  | sed "s/^chr//g"
42609 Killed                  | bgzip -c > variation/dbsnp-$version.vcf.gz
Traceback (most recent call last):
File "/marconi_work/LI08p_EMUT/bcbio/anaconda/bin/bcbio_nextgen.py", line 222, in
install.upgrade_bcbio(kwargs["args"])
File "/marconi_work/LI08p_EMUT/bcbio/anaconda/lib/python2.7/site-packages/bcbio/install.py", line 98, in upgrade_bcbio
upgrade_bcbio_data(args, REMOTES)
File "/marconi_work/LI08p_EMUT/bcbio/anaconda/lib/python2.7/site-packages/bcbio/install.py", line 306, in upgrade_bcbio_data
cbl_deploy.deploy(s)
File "/marconi_work/LI08p_EMUT/bcbio/tmpbcbio-install/cloudbiolinux/cloudbio/deploy/init.py", line 63, in deploy
_setup_vm(options, vm_launcher, actions)
File "/marconi_work/LI08p_EMUT/bcbio/tmpbcbio-install/cloudbiolinux/cloudbio/deploy/init.py", line 109, in _setup_vm
configure_instance(options, actions)
File "/marconi_work/LI08p_EMUT/bcbio/tmpbcbio-install/cloudbiolinux/cloudbio/deploy/init.py", line 269, in configure_instance
setup_biodata(options)
File "/marconi_work/LI08p_EMUT/bcbio/tmpbcbio-install/cloudbiolinux/cloudbio/deploy/init.py", line 251, in setup_biodata
install_proc(options["genomes"], ["ggd", "s3", "raw"])
File "/marconi_work/LI08p_EMUT/bcbio/tmpbcbio-install/cloudbiolinux/cloudbio/biodata/genomes.py", line 340, in install_data
_prep_genomes(env, genomes, genome_indexes, ready_approaches)
File "/marconi_work/LI08p_EMUT/bcbio/tmpbcbio-install/cloudbiolinux/cloudbio/biodata/genomes.py", line 469, in _prep_genomes
retrieve_fn(env, manager, gid, idx)
File "/marconi_work/LI08p_EMUT/bcbio/tmpbcbio-install/cloudbiolinux/cloudbio/biodata/genomes.py", line 836, in _install_with_ggd
ggd.install_recipe(env.cwd, recipe_file, gid)
File "/marconi_work/LI08p_EMUT/bcbio/tmpbcbio-install/cloudbiolinux/cloudbio/biodata/ggd.py", line 31, in install_recipe
recipe["recipe"]["full"]["recipe_type"])
File "/marconi_work/LI08p_EMUT/bcbio/tmpbcbio-install/cloudbiolinux/cloudbio/biodata/ggd.py", line 63, in _run_recipe
subprocess.check_output(["bash", run_file])
File "/marconi_work/LI08p_EMUT/bcbio/anaconda/lib/python2.7/subprocess.py", line 223, in check_output
raise CalledProcessError(retcode, cmd, output=output)
subprocess.CalledProcessError: Command '['bash', '/marconi_work/LI08p_EMUT/bcbio/genomes/Hsapiens/GRCh37/txtmp/ggd-run.sh']' returned non-zero exit status 137
Traceback (most recent call last):
File "bcbio_nextgen_install.py", line 266, in
main(parser.parse_args(), sys.argv[1:])
File "bcbio_nextgen_install.py", line 44, in main
subprocess.check_call([bcbio, "upgrade"] + _clean_args(sys_argv, args))
File "/usr/lib64/python2.7/subprocess.py", line 542, in check_call
raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['/marconi_work/LI08p_EMUT/bcbio/anaconda/bin/bcbio_nextgen.py', 'upgrade', '--tooldir=/marconi_work/LI08p_EMUT/bcbio/tools', '--genomes', 'GRCh37', '--aligners', 'bwa', '--aligners', 'bowtie2', '--data']' returned non-zero exit status 1
Could you please help me resolve this

Thanks for the report and apologies about the issue. It looks like the problem is here:
42609 Killed | bgzip -c > variation/dbsnp-$version.vcf.gz

The bgzip process is getting killed by your operating system, causing the failure you're seeing. This could either be due to using too much memory on the machine, or if you're running on the head node of a cluster there may be processes in place that kill jobs that take too long (or too much memory/CPU).
If you're able to re-run the upgrade on a different machine or setup, it should pick up where it left off and hopefully finish cleanly with more memory/CPU available to the process. Hope this helps.

Thank you for the response. I tried installing again, and it worked.
Regards
Judith Hariprakash
On Tue, May 22, 2018 at 5:11 PM, Brad Chapman<EMAIL_ADDRESS>wrote:

Thanks for the report and apologies about the issue. It looks like the
problem is here:
42609 Killed | bgzip -c > variation/dbsnp-$version.vcf.gz
The bgzip process is getting killed by your operating system, causing the
failure you're seeing. This could either be due to using too much memory on
the machine, or if you're running on the head node of a cluster there may
be processes in place that kill jobs that take too long (or too much
memory/CPU).
If you're able to re-run the upgrade on a different machine or setup, it
should pick up where it left off and hopefully finish cleanly with more
memory/CPU available to the process. Hope this helps.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bcbio/bcbio-nextgen/issues/2403#issuecomment-391027827,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMgvWGJoCLUynECn1fY5hUB0R61L0Jy0ks5t1Cq9gaJpZM4UIXB_
.

--
Regards,
Judith Hariprakash


chore: bdd-spring-1583907846 to 0.0.1

chore: Promote bdd-spring-1583907846 to version 0.0.1

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

Needs approval from an approver in each of these files:

OWNERS

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


s3website: collections.Container removed from python3.10

this was added to collections.abc in python 3.3, and removed from collections in 3.10

AttributeError: module 'collections' has no attribute 'Container'

Fixes: https://tracker.ceph.com/issues/58960

@cbodley lgtm. Do you have a run of s3-tests that shows this passes?

@cbodley lgtm. Do you have a run of s3-tests that shows this passes?

not yet. i can batch it together with https://github.com/ceph/s3-tests/pull/503 when i'm ready to test https://github.com/ceph/ceph/pull/50504

@alimaredia cherry-picked to ceph-master and ceph-reef


Right side does not work

Hello, I collected everything, I asked for it, but the right side does not work, if I change the cable in places, the left side does not work. How do I fix the problem? I ran the firmware on both sides.

qmk setting

Try these
Replace TRRS cable.
Check the soldering of the TRRS jack.
Check ProMicro soldering.

Replace TRRS cable. Done, 100% worked.
soldering

TRPC cable needs mono or stereo? Could it be too long?

It looks like it's shorted out here

This is also suspicious

Yep, fixed all the problems and it worked!


Operations and admin cucumber tests

Contains operations and admin test features and steps

@Sharvaribarve

Make the steps concise to avoid future step collision
Write common steps for re-usability, see refactored operation's delete web server
Take due diligence in writing only the steps needed for the scenario e.g. in the delete web server scenario, you didn't need to the create JVM media, the apache tomcat, generate the web app and the JVM. You only needed apache media, generate and start the web server


Added type validation to Assertions.

Changes:

Kotlin test harness will validate types.

Blocked by https://github.com/openrewrite/rewrite-kotlin/issues/480.

@knutwannheden type validation is essentially ready for review. The failing tests are identifiers with aliases, and I'm looking into how to detect aliases.
import java.util.regex.Pattern.CASE_INSENSITIVE as i

class A {
    val f = arrayOf(i)
}

As we can disable validations on a test case basis, we can fix any remaining issues in follow up PRs.

Opened: https://github.com/openrewrite/rewrite-kotlin/issues/545 to track a fix for aliases. Tracking the declarations of a name will allow us to apply the appropriate field type comparison.


ADS1115

I was trying to connect the ADS1115 to the Pico using the libraries from circuitpython. I was able to initialize the sensor, but I always get the wrong result back. Anyone who can help?

This is probably better asked on the forums, you're more likely to get an answer there.