Account creation fails on email providers with no auth

Central v2023.2.1
https://docs.getodk.org/central-install-digital-ocean/#using-a-custom-mail-server says we don't need user/pass in the .env.
Given an .env file with the following...
EMAIL_HOST=test.example.com
<EMAIL_ADDRESS>
Account creation fails with a 500. The error in the service log is "535 Authentication failed".
If I remove the auth section from https://github.com/getodk/central/blob/master/files/service/config.json.template#L18, account creation works great.

@lognaturel I think you tested this configuration, so I'm not sure if it wasn't a good test or maybe this is an issue with this specific email server. How did you verify that removing user/pass in the .env worked?

I tested on dev. I think the mail server we use for that doesn’t use username/pass? Or maybe it was the default container. For whichever one of those doesn’t require credentials, I removed them from .env but left in the template and that worked. Did you remove the vars from .env?

Yes, I removed them from .env and left them in the template. I'll try on dev when I have a moment (but you can too if you have a moment).

I bet https://github.com/getodk/central/pull/478 explains this problem, but I can't reproduce it.
I propose we update the docs to say we require username and password. It's not a great idea to have an SMTP server, even an internal one, without a password.

With nodemailer, you can't have blank/null strings. If you have that auth section in the transportOpts, it will try to auth with blank/null strings which won't work.
To support SMTP with no auth, we either need to do some sed magic to strip that auth section out or change Central's code to check for empty username/password when calling nodemailer and skip auth. Neither of these options sound sensible to me.
I think it's much more sensible to update our docs to ask people to setup SMTP servers with auth.

Works for me, thanks!


Update formatting

Following the recent changes in Transformers regarding quality tools (https://github.com/huggingface/transformers/pull/21480 and https://github.com/huggingface/transformers/pull/21493)

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Fix vkTexture being disposed early

Unless the reference count is in incremented immediately there is a chance that a referenced buffer gets destroyed before use.
Related: https://github.com/ppy/osu/issues/23056

Please open a pull request against the upstream repository rather than this fork, and include proper details on how the early disposal can happen so it's clear to the developers what's going on.

Thanks. As above ^^. We'll pull in your change separately.


Next step of Windows UWP support

Continue the discussion on https://github.com/openssl/openssl/pull/8917.
We have some arguments to solve.

What about taking UWP target support back to 1.1.1 as well? (I hope OpenSSL can take it back because there are a lot of projects that are waiting on this.)
What about the Windows 10 SDK Build 17134 or earlier support? (Need to write "You also should add "no-asm" option when you use the Windows 10 SDK Build 17134 or earlier." on the note)
What progress about the "Configure" script improvement? (Because we don't need to specify "no-uplink" at configure time.)
What about provding a batch script for helping people compile?

Kenji Mouri

@levitte @mattcaswell @jdm

What about taking UWP target support back to 1.1.1 as well? (I hope OpenSSL can take it back because there are a lot of projects that are waiting on this.)

Adding new platforms is a new feature. New features are not added to stable branches because of the very fact that they are stable. Adding features is a destabilising action. Very often we find when new features are added we don't get it right straight away and there is some churn in the code as we fix stuff. IMO this cannot be backported to 1.1.1.,

What about the Windows 10 SDK Build 17134 or earlier support? (Need to write "You also should add "no-asm" option when you use the Windows 10 SDK Build 17134 or earlier." on the note)

We should not be advocating no-asm for production use. Not only are there significant performance implications, there are security implications too. IMO we should not support earlier builds if we cannot get asm to work on them. My preferred route here would be to fix whatever the underlying cause of the asm failures is.

Regarding no-asm and no-uplink, I have an idea that would solve the problem with specifying them in a disable attribute.
See, the information they affect are currently in form of config attributes and special code in Configure. However, that information really belongs in build.info files, just like all other file and macro information.
Now that we have variables in build.info files, it's not even a hard task to move that information... Just tedious.

Adding new platforms is a new feature. New features are not added to stable branches because of the very fact that they are stable.

I wonder what is the next LTS version after the OpenSSL 1.1.1?

See, the information they affect are currently in form of config attributes and special code in Configure. However, that information really belongs in build.info files, just like all other file and macro information. Now that we have variables in build.info files, it's not even a hard task to move that information... Just tedious.

Well, I will try. But I am also curious about your experimental branch. (You have said "This can be remedied with late evaluation (I have an experimental PR that I didn't finish), maybe now's the time to introduce it...." before.)

For the idea I mentioned here, that experimental branch won't be needed.

#9166

What about the Windows 10 SDK Build 17134 or earlier support? (Need to write "You also should add "no-asm" option when you use the Windows 10 SDK Build 17134 or earlier." on the note)

If someone tells me how to detect the SDK version, I might be able to craft something.

@levitte
In https://stackoverflow.com/questions/2665755, we can know the NTDDI_WIN10_RS5 macro has been defined in Windows 10 SDK Build 17763 (RS5) or later. So, write code like this:
#include <sdkddkver.h>
#if !defined(NTDDI_WIN10_RS5)
    #error Windows 10.0.17763.0 SDK is required
#endif

In https://github.com/Chuyu-Team/MINT/blob/565dc12ef9d3fd9c725187588c680b12cc403df4/MINT.h#L7539, you will notice something like the following thing.

#if (NTDDI_VERSION >= NTDDI_WIN10_RS5)

I hope these can help you.
Kenji Mouri

@levitte any update on this? Any ETA?
Thanks

@levitte are you alive? It has been 22 days now...

Hey, sorry for the delay, there's much going on.
@MouriNaruto, thanks for the details.  I worry that sdkddkver.h might not always be available, would you (or @illera88?) care to shed some light on that?

sdkddkver.h has been present since  Visual C++ 2010 Express so unless you support a 10 year old VC version I don't thing you will have any problem.
Do you need anything else?

I think sdkddkver.h has been present since at least Windows 8 SDK.
Windows Universal Platform has been present since Windows 10.
We should use Windows 10 SDK and Visual Studio 2015 or later if we need to develop an UWP app.
So we don't need to worry about that.

Ah, so as long as I only check this for UWP builds, we're safe.  Thanks.

I ended up checking for the registry key that holds the SDK version.  Please try #9440 to see if it works for you.

#9440 has been merged (bcc0025).  I find it a little unfortunate that it went completely uncommented by those who wanted it, but feel free to try it out now.

So, I've seen no comment here since the fix.  I assume all is fine.  If not, raise a new issue.

I have been successfully using UWP-enabled openssl, so thank you!


TestID for Automated UI Testing

I'm using XCodes Automated UI Testing for Fastlane Snapshots in RN.
Is it currently possible to add a accessibilityTraits="button" and testID prop to each tab button so I can reference it?

Sure. Feel free to send a PR

cool. will do 👍


MediaQueryList undefined, potentially caused by server side rendering?

I'm trying to add a LeftNav (from material-ui) to the page and make it responsive so that when browser window shrinks or expands, it automatically undock/dock the nav menu. What I did was to get a MediaQueryList object in componentWillMount() as below.
  componentWillMount() {
    if (typeof window !== 'undefined') {
      const mql = window.matchMedia(`(min-width: 800px)`);
      mql.addListener(this.mediaQueryChanged);
      this.setState({
        mql: mql,
        docked: mql.matches,
        leftNavOpen: mql.matches
      }, () => {
        console.log(this.state.mql); // verifies that state.mql has been set
      });
    }
  }

Then in mediaQueryChanged(),
  mediaQueryChanged() {
    console.log(this.state.mql); // Uncaught TypeError: Cannot read property 'mql' of undefined
    this.setState({
      leftNavOpen: this.state.mql.matches
    });
  }

It looks like when the page is loaded, leftNavOpen does get proper value. However, when windows is resized, I got Uncaught TypeError: Cannot read property 'mql' of undefined. How come state.leftNavOpen became undefined? Note that I didn't use Redux for storing/updating the state, as this all within the same component.
BTW, I got the code from react-sidebar, which seems to be working (I didn't verify it, though). Could this be caused by server-sider rendering, or I'm missing something obvious? I'd love to hear other options to create a responsive LeftNav.
Thanks.

I think you should use componentdidmount, since it guaranteed to run only
on client side (better not use the client)
On Sun, Feb 28, 2016, 12:52 AM Peng Han<EMAIL_ADDRESS>wrote:

I'm trying to add a LeftNav (from material-ui
http://www.material-ui.com/#/) to the page and make it responsive so
that when browser window shrinks or expands, it automatically undock/dock
the nav menu. What I did was to get a MediaQueryList object in
componentWillMount() as below.
componentWillMount() {
if (typeof window !== 'undefined') {
const mql = window.matchMedia((min-width: 800px));
mql.addListener(this.mediaQueryChanged);
this.setState({
mql: mql,
docked: mql.matches,
leftNavOpen: mql.matches
}, () => {
console.log(this.state.mql); // verifies that state.mql has been set
});
}
}
Then in mediaQueryChanged(),
mediaQueryChanged() {
console.log(this.state.mql); // Uncaught TypeError: Cannot read property 'mql' of undefined
this.setState({
leftNavOpen: this.state.mql.matches
});
}
It looks like when the page is loaded, leftNavOpen does get proper value.
However, when windows is resized, I got Uncaught TypeError: Cannot read
property 'mql' of undefined. How come state.leftNavOpen became undefined?
Note that I didn't use Redux for storing/updating the state, as this all
within the same component.
BTW, I got the code from react-sidebar
https://github.com/balloob/react-sidebar/blob/master/example/src/responsive_example.js#L23-L39,
which seems to be working (I didn't verify it, though). Could this be
caused by server-sider rendering, or I'm missing something obvious? I'd
love to hear other options to create a responsive LeftNav.
Thanks.
—
Reply to this email directly or view it on GitHub
https://github.com/erikras/react-redux-universal-hot-example/issues/968.

I tried to change it to componentDidMount but ran into the same problem. It looks like storing MediaQueryList object in state, or in Redux store, will cause issue anyway. But I think I finally figured it out (at least to certain extent) with a callback in mql.addListener(). Here is the code
componentWillMount() {
    if (typeof window !== 'undefined') {
      const mql = window.matchMedia(`(min-width: 800px)`);
      this.setState({
        _isMounted: true, // this is to keep tracking if the component is unmounted to avoid the callback updating state on an unmounted component
        leftNavOpen: mql.matches
      });
      mql.addListener(() => {
        if (this.state._isMounted) {
          this.setState({
            leftNavOpen: mql.matches
          });
        }
      });
    }
  }

and update _isMounted to false when the component is unmounted.
  componentWillUnmount() {
    this.setState({
      _isMounted: false
    });
  }

I still get the warning message
Warning: setState(...): Can only update a mounted or mounting component. This usually means you called setState() on an unmounted component. This is a no-op. Please check the code for the undefined component.
but I should be fine with _isMounted keeping track of whether or not the component is unmounted.
I also did the same with Redux, essentially storing the state variable in Redux store. I think it's cleaner. However, I got a side effect that I couldn't solve by myself. In App.js, there is @asyncConnect to check if Info and Auth are loaded.
@asyncConnect([{
  promise: ({store: {dispatch, getState}}) => {
    const promises = [];

    if (!isInfoLoaded(getState())) {
      promises.push(dispatch(loadInfo()));
    }
    if (!isAuthLoaded(getState())) {
      promises.push(dispatch(loadAuth()));
    }

    return Promise.all(promises);
  }
}])

When I add my action creators to @connect as below

@connect(
  state => ({
    user: state.auth.user,
    leftNavOpen: state.uiStates.leftNavOpen, // new line I added
  }),
  dispatch => bindActionCreators({uiLayoutLeftNavUpdate, uiUpdateMql}, dispatch), // new line I added
  {logout, pushState: routeActions.push})

I got this nasty error message (with bright red background :( ). It looks like the actionCreators in @asyncConnect and @connect are not happy with each other? It would be REALLY appreciated if someone can help me on getting rid of this, as I do prefer to handle UI states in Redux store. Thanks.
TypeError: finalMergeProps is not a function
computeMergedProps
/home/ubuntu/repos/tutorials/redux-hot-example/node_modules/react-redux/lib/components/connect.js:83:23
Connect.updateMergedPropsIfNeeded
/home/ubuntu/repos/tutorials/redux-hot-example/node_modules/react-redux/lib/components/connect.js:176:31
Connect.render
/home/ubuntu/repos/tutorials/redux-hot-example/node_modules/react-redux/lib/components/connect.js:275:41
[object Object].ReactCompositeComponentMixin._renderValidatedComponentWithoutOwnerOrContext
/home/ubuntu/repos/tutorials/redux-hot-example/node_modules/react/lib/ReactCompositeComponent.js:587:34
[object Object].ReactCompositeComponentMixin._renderValidatedComponent
/home/ubuntu/repos/tutorials/redux-hot-example/node_modules/react/lib/ReactCompositeComponent.js:607:32
[object Object].wrapper [as _renderValidatedComponent]
/home/ubuntu/repos/tutorials/redux-hot-example/node_modules/react/lib/ReactPerf.js:66:21
[object Object].ReactCompositeComponentMixin.mountComponent
/home/ubuntu/repos/tutorials/redux-hot-example/node_modules/react/lib/ReactCompositeComponent.js:220:30
[object Object].wrapper [as mountComponent]
/home/ubuntu/repos/tutorials/redux-hot-example/node_modules/react/lib/ReactPerf.js:66:21
Object.ReactReconciler.mountComponent
/home/ubuntu/repos/tutorials/redux-hot-example/node_modules/react/lib/ReactReconciler.js:37:35
[object Object].ReactCompositeComponentMixin.mountComponent
/home/ubuntu/repos/tutorials/redux-hot-example/node_modules/react/lib/ReactCompositeComponent.js:225:34

OK, I figured it out. I was just blindly inserting
dispatch => bindActionCreators({uiLayoutLeftNavUpdate, uiUpdateMql}, dispatch),
to @connect without knowing that the object in the last line
{logout, pushState: routeActions.push}
is already the [mapDispatchToProps] argument. I fixed it by combining them together. Here is how it looks now.
@connect(
  state => ({
    user: state.auth.user,
    leftNavOpen: state.uiStates.leftNavOpen
  }),
  dispatch => bindActionCreators({uiLayoutLeftNavUpdate, uiUpdateMql, logout, pushState: routeActions.push}, dispatch))

No TypeError: finalMergeProps is not a function anymore. Closing the issue.


glosbe cleaning

Texts scraped from glosbe have English artifacts that reoccur in text. Which includes:

"Translation of $Page_name into $lang_name"
"$word is the translation of $page_name into $lang_name"
"Sample translated sentence:"
"\n$Page_name\nnoun\ngrammar\n+\nAdd translation\nAdd"
......
......
And Much more.

Being a dictionary it provides translation from a language to the other. For example "https://glosbe.com/en/ada/yard" would translate yard from English to Adangme. Another page "https://glosbe.com/fi/ady/koskea" would translate Finnish word "koskea" to the language "Adyghe". Furthermore, they provide example sentence translated.
Possible Options:

Don't include glosbe text for language modeling.
Clean up. Might be extremely convoluted due to the inclusion of other languages in a way which is hard to separate.

Perhaps a general text cleaner can be and should be written that would clean high resource languages out of scraped text rather that a site specific cleaner.

Beta cleaner made.

https://github.com/fedelopez77/langdetect


Upgrdae OpenSuse 12.1 To 12.2 - Issue at Reboot

Hello When I Upgrade OpenSuse from 12.1 to 12.2 , I am having an Issue

The server keeps shutdown
it stucks at GRUB

Actually I am planning to to Upgrade from OpenSuse 12.1 to Opensuse 15.4 , but itseems it needs to go one level at time.
Any Proper Docuemntation to Upgrade from OpenSuse 12.1 to OpenSuse15.4 would be great.
Thanks

Hi! This is issue tracker isn't for openSUSE but for a website to improve documentation about the end of life of many products, openSUSE just one of dozens.
You might be better asking at Stack Overflow or checking out the links at https://en.opensuse.org/Portal:Support


[Macros] Fix type-checking local pattern bindings in macro-expanded closures.

PreCheckExpr, ConstraintGenerator, and other walkers do not walk into macro expansions. However, the implementation of this macro walking behavior in ASTWalker would skip any declaration that appears inside any macro expansion buffer. This is incorrect for cases where the parent is in the same macro expansion buffer, because the local declaration is not inside a new macro expansion. This caused bogus errors when type checking expanded macro expressions containing closures with local declarations, because pre-check and constraint generation mistakenly skipped local pattern bindings.
Resolves: rdar://108148075

@swift-ci please smoke test


Pipeline parameters are not reported as span attributes by Elastic APM

Version report
Jenkins and plugins versions report:

Jenkins opentelemetry-plugin 0.10-beta
Elastic Observability 7.12.1

Reproduction steps

Create a parameterized pipeline and execute it
Visualise the trace in Elastic APM, the fields ci.pipeline.parameter.name,  ci.pipeline.parameter.sensitive, and ci.pipeline.parameter.value are not reported by Elastic APM as span attributes when Jaeger reports them.

pipeline {
    agent any

    parameters {
        string(name: 'PERSON', defaultValue: 'Mr Jenkins', description: 'Who should I say hello to?')
        text(name: 'BIOGRAPHY', defaultValue: '', description: 'Enter some information about the person')
        booleanParam(name: 'TOGGLE', defaultValue: true, description: 'Toggle this value')
        choice(name: 'CHOICE', choices: ['One', 'Two', 'Three'], description: 'Pick something')
        password(name: 'PASSWORD', defaultValue: 'SECRET', description: 'Enter a password')
    }
    stages {
        stage('Example') {
            steps {
                echo "Hello ${params.PERSON}"
                echo "Biography: ${params.BIOGRAPHY}"
                echo "Toggle: ${params.TOGGLE}"
                echo "Choice: ${params.CHOICE}"
                echo "Password: ${params.PASSWORD}"
            }
        }
    }
}

Results
Expected result as visible with Jaeger 1.22:

Actual result with Elastic APM 7.12.1:

Being fixed by https://github.com/elastic/apm-server/pull/5286

It works in 7.13.1

Pipeline -> https://beats-ci.elastic.co/job/e2e-tests/job/e2e-testing-mbp/job/master/1054/parameters/


after prolong usage it give error "use of closed network"

after prolong usage it give error "use of closed network"
any idea what might cause this?
is it somehow the connection might get forced closed internally?
and if possible any way to mitigate this?

Do you know where the error happened? I searched my GOPATH, can't find the error message, and googled mysql with the error message, also doesn't help

sorry, the alert is sent to the wrong place.
apparently its the error from my redis library.
the GCE will terminate idle tcp after 10 minutes and there is no way to prevent it.


Add Confetti to Donation page

I would like to add confetti in the donation page, [when the user clicks submit button].

If This issue is still open , I would love to work on this issue :)


CMP first load failing with no error

On first load of the Teams app, Champion Leader board tile shows up and a spinning circle.
It spins for ever and stays stuck.
App is loaded in Teams Windows client.
Tried removing all components and reloading the cmp package several times. Did not make a difference.
Tried loading the app in Teams Web client after removing the following assets from the SharePoint site:

Digital Badge Assets
Event Track Details list
Events List
Member List
It now prompt with a message saying "Admin permission have been granted to user X. Please refresh the app"
All is now functional.

Hi @olivjat,
Thanks for trying this app.
I understand that you are now able to load the app successfully. Is there anything that i can help you with. Please let me know.
Thanks.


Optimize getGlamorClassName

By shaping the string class names beginning with 'css-' we can
just give the resulting object to glamor. This offloads almost the
entire workload to glamor from glamorous. Another benefit is that
now there will be at most one css-[hash] class name on an element
The only failing test after the modifications was an element that had 2 css class names only having one now. The snapshot was updated to reflect this bug fix.

Checklist:

[ ] Documentation N/A
[x] Tests
[x] Ready to be merged
[x] Added myself to contributors table

Codecov Report

Merging #254 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master   #254   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          10     10
  Lines         144    142    -2
  Branches       37     35    -2
=====================================
- Hits          144    142    -2

Impacted Files
Coverage Δ

src/get-glamor-classname.js
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

Oh, and if you could run your fancy benchmarks on this, I'd love to include it in my blogpost if there's any significant perf improvement here...

I didn't see that much of an improvement in rerenders as I thought I would see.
Comparing the 2 fastest runs.
Before
- Rerender Duration:<PHONE_NUMBER>6666664 ms
- Mount Duration: 180.5808 ms

After
- Rerender Duration: 196.3391
- Mount Duration:<PHONE_NUMBER>9999998 ms

That's about an 8% speedup in mount time. I would be careful to say that this is an 8% speed up though.  It uses 1000 random numbers to generate elements with different styles on the page. You rarely hit the cache in that situation. On a real site with 100's of repeat classes and 1000's of elements, I could see this being a real perf win.

That's great! I'll just say something like: "And @tkh44 may or may not have made glamorous 50x faster"


Fixed Sensor Module - Add Equipment scale switch

Sweet Idea. I like it @tarcadius

``asdfjklsdhasdfsda

Wouldn't we want a generic switch that could be used for any module? For example by specifying the scale of measurement? Or am I not understanding this one ... is there something specific to fixed sensors?

Uncluded in some previous version


udocker support for github packages?

Hi there,
recently github started support for hosting docker images for repos.
I can use them in docker:
docker pull docker.pkg.github.com/dolmades/dolmades-docker/runtime:1.2

or in the Dockerfile
FROM docker.pkg.github.com/dolmades/dolmades-docker/runtime:1.2
I wonder if it is possible to use those in udocker:
udocker pull docker.pkg.github.com/dolmades/dolmades-docker/runtime:1.2
Error: no files downloaded

I also played with the --registry option but this does not work either:
udocker pull --registry=https://docker.pkg.github.com dolmades/dolmades-docker/runtime:1.2
Error: no files downloaded

I see there is also an option called --index but I don't understand what it does.
If I alter the URL of the registry I get also
Error: image not found
so I take this as hint that the URL is correct.
Does someone know what prevents udocker from working with github?

Fixes have been added to the devel branch, please check.

From current devel:
udocker pull --registry=https://docker.pkg.github.com dolmades/dolmades-docker/runtime:1.2
Error: layers section missing in manifest
Error: no files downloaded

Am I using wrong syntax?

You need to login with token. Then
udocker pull docker. pkg. github. com/dolmades/dolmades-docker/runtime:1.2
or specify both index and registry as arguments.
Cheers

Sorry to bother again but I don't seem to get the login right. Whenever I use --password, --username, --registry it ignores it. What would be the right syntax to call udocker login?

Hi this works for me,
udocker.py  login  --registry=https://docker.pkg.github.com  --login=USERNAME  --password=TOKEN
udocker.py  pull  docker.pkg.github.com/indigo-dc/udocker/test
Make sure you are using the latest version from devel.

I'm afraid it doesn't work:
./udocker login --registry=https://docker.pkg.github.com --login=katakombi --password=TOKEN
username: katakombi
Error: syntax error at: --login=katakombi

It keeps asking me for my username. This is devel(head) from your curl cmd.

Sorry is not --login is --username

./udocker login  --username=katakombi --password=VERYLONGTOKENWORKSWITHDOCKER
Info: installing 1.1.3-3
Info: installation of udockertools successful
stefan@khaleesi:/tmp$ ./udocker pull docker.pkg.github.com/dolmades/dolmades-docker/runtime:1.2
Error: manifest not found or not authorized
Error: no files downloaded

I don't see packages published under dolmades/dolmades-docker/runtime
Do you have 1.2 published ?

can you access this link? https://github.com/dolmades/dolmades-docker/packages/22887
I\ve just copypasted the commands for docker
docker pull docker.pkg.github.com/dolmades/dolmades-docker/runtime:1.2

and those work

Ok I see it and it is working for me:
./udocker.py login --registry=docker.pkg.github.com --username=jorge-lip --password=XXXXXXX
./udocker.py  pull docker.pkg.github.com/dolmades/dolmades-docker/runtime:1.2
Downloading layer: sha256:7e1eb4178be530d773f519ab3008be22fa02e6a13e9dba2d6bfc189c607e2ad2
Downloading layer: sha256:b809a35dfd3612833139cad73db0a5926dd17e035ad2c75d4ff306cbf7ce40f5
Downloading layer: sha256:5f164ab6670990fdb3d56ac75bc8ba4a3c3aa222541fb77d5bdb5ec8a720c922
Downloading layer: sha256:2d4971c608a492592f0faff0fbf83209e1a7c36221d587c900b329d3d325d2d1
Downloading layer: sha256:6001e1789921cf851f6fb2e5fe05be70f482fe9c2286f66892fe5a3bc404569c
Downloading layer: sha256:8e829fe70a46e3ac4334823560e98b257234c23629f19f05460e21a453091e6d
Downloading layer: sha256:251f5509d51d9e4119d4ffb70d4820f8e2d7dc72ad15df3ebd7cd755539e40fd
Downloading layer: sha256:35c102085707f703de2d9eaad8752d6fe1b8f02b5d2149f1d8357c9cc7fb7d0a
Can you please delete your udocker keystore and try again ?
rm  $HOME/.udocker/keystore

Still the same. Can I PM you my access token? I can revoke it once you got it working...
./udocker pull docker.pkg.github.com/dolmades/dolmades-docker/runtime:1.2
Error: manifest not found or not authorized
Error: no files downloaded

Ooookay, I've created a personal access token with RO permissions and it works now! Seems like I copy&pasted what was in ~/.docker/config.json but this token is not the format which is expected by udocker.
There is another caveat I've run into: once I use https:// and omit it the other time it doesn't use the stored credentials. Actually I could find two entries, maybe udocker could print a warning about that, or if it doesn't actually succeed in logging in.
Sorry for the hassle, and thanks for the support! Udocker is a great tool to me. Closing as of now...


[FR] Port web/directedlabelled.html to WebCola

https://ialab.it.monash.edu/webcola/index.html

https://observablehq.com/@d3/force-directed-graph

https://observablehq.com/@mbostock/hello-cola


working Payment service from swagger as well as all tests pass

@max-shapiro32 Payment service now works with default strategy that just validates credit card expiration date and if expired returns a status of card_expired or succeeded + random confirmation guid
strategy can be set in env vars as PAYMENT_PROVIDER valid values are PINGPONG or STRIPE ( not yet implemented at this stage, but plumbing in place to add it or other payment providers )

@Grant-Steinfeld I'm closing this, reopen a PR to master branch. What you have looks good, just switch to using "swagger-jsdoc instead of the yaml


Mondo Pattern Library 2.0

All our logical patterns can be found here. We want to achieve two things:

Standardise the patterns a bit better
Create beautiful, user friendly documentation for each of them

Standardisation
Standardisation can be a bit tedious, and once we have a schema, @nicolevasilevsky and I can help with this.

[X] Standardising the pattern order. (I have done that, the order should be ["pattern_name", "pattern_iri", "description", "contributors", "classes", "relations", "annotationProperties", "vars", "name", "annotations", "def", "equivalentTo"])
[x] Standardising the pattern variable names. As you can see here, some patterns have ugly variable names, we really want to standardise. Standardise means: if the slot is a disease the variable should be named disease (example). If the variable refers to a gene, the variable should be called "gene".
[ ] Standardising the classes, relations, annotationProperties fields. Standardising means: the slot should use the appropriate id as a value. Good example here.
[ ] Every pattern should have a good definition. Basically you paste a pattern in slack, and @nicolevasilevsky will come up with a description you can add.

Shiny Pattern Documentation System (dp2docs)
Here, we want to have a beautiful markdown based page in our readthedocs. We don't know yet the perfect way to do this, but in essence, we want:

[ ] A way to generate a great overview for patterns like README.md, which is generated by this script in our read the docs (a new sub page).
[ ] An individual page for each pattern which lays out the pattern in a beautiful way. The goal here is to have something really user friendly, that clinicians can be pointed to and that makes sense.
[ ] Everything should be based on markdown. So ideally the input is a directory of patterns, the output is a a set of markdown files that can be fiddled in to read the docs and mkdocs or github pages directly.

https://mondo.readthedocs.io/

Step 1: Create automatically
/docs/

create subdir /docs/editors-guide/patterns
for every pattern, generate a nice markdown view of pattern in patterns dir
generate a nice overview for all patterns

Step 2:

Group related patterns.
Option 1: category based (all patterns in cancer category are grouped together)
Option 2: pattern relation based : patterns (in overview) are rendered in a tree according to their respective sub/super patterns.

also replace all instances
disease: owl_thing
disease: MONDO:0000001
and adjust the variable names.

This is awesome
I'd like to discuss naming conventions
Naming vars after relations is better than ranges
On Fri, Feb 5, 2021, 03:23 Nico Matentzoglu<EMAIL_ADDRESS>wrote:

All our logical patterns can be found here
https://github.com/monarch-initiative/mondo/tree/master/src/patterns/dosdp-patterns.
We want to achieve two things:

Standardise the patterns a bit better
Create beautiful, user friendly documentation for each of them

Standardisation
Standardisation can be a bit tedious, and once we have a schema,
@nicolevasilevsky https://github.com/nicolevasilevsky and I can help
with this.

Standardising the pattern order. (I have done that, the order should
be ["pattern_name", "pattern_iri", "description", "contributors",
"classes", "relations", "annotationProperties", "vars", "name",
"annotations", "def", "equivalentTo"])
Standardising the pattern variable names. As you can see here
https://github.com/monarch-initiative/mondo/blob/baa5a0310510f841ebe077191b3922233504ab77/src/patterns/dosdp-patterns/adenocarcinoma.yaml#L25,
some patterns have ugly variable names, we really want to standardise.
Standardise means: if the slot is a disease the variable should be
named disease (example
https://github.com/monarch-initiative/mondo/blob/baa5a0310510f841ebe077191b3922233504ab77/src/patterns/dosdp-patterns/allergic_form_of_disease.yaml#L31).
If the variable refers to a gene, the variable should be called "gene".
Standardising the classes, relations, annotationProperties fields.
Standardising means: the slot should use the appropriate id as a value.
Good example here
https://github.com/monarch-initiative/mondo/blob/baa5a0310510f841ebe077191b3922233504ab77/src/patterns/dosdp-patterns/allergic_form_of_disease.yaml#L19
.
Every pattern should have a good definition. Basically you paste a
pattern in slack, and @nicolevasilevsky
https://github.com/nicolevasilevsky will come up with a description
you can add.

Shiny Pattern Documentation System (dp2docs)
Here, we want to have a beautiful markdown based page in our readthedocs
https://mondo.readthedocs.io/en/latest/editors-guide/e-design-patterns/#design-patterns-for-neoplasms.
We don't know yet the perfect way to do this, but in essence, we want:

A way to generate a great overview for patterns like README.md
https://github.com/monarch-initiative/mondo/blob/master/src/patterns/dosdp-patterns/README.md,
which is generated by this
https://github.com/monarch-initiative/mondo/blob/master/src/scripts/patterns_create_overview.py
script in our read the docs (a new sub page).
An individual page for each pattern which lays out the pattern in a
beautiful way. The goal here is to have something really user friendly,
that clinicians can be pointed to and that makes sense.
Everything should be based on markdown. So ideally the input is a
directory of patterns, the output is a a set of markdown files that
can be fiddled in to read the docs and mkdocs or github pages
directly.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/monarch-initiative/mondo/issues/2619, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAMMOLOAE2QMVFLMYDZUCDS5PIKFANCNFSM4XES6Y5Q
.

We will discuss at next qc call.

[ ] need to make sure synonym pattern is not equivalent to the label

in discussion with @matentzn we decided to close this ticket in favor of smaller tickets.


Fix/cli disable

No ticket. Fixes bug that occurs when trying to disable a service via the command line that's already disabled in the task config.

Codecov Report

Merging #75 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master    #75   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          10     10
  Lines         297    298    +1
=====================================
+ Hits          297    298    +1

Impacted Files
Coverage Δ

src/services.js
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

🐐:dash:, just needs conflicts resolved


Roadmap suggestion: Suggest automation enhancement by adding sensors

I suppose some automations can be improved in accuracy based on some additional inputs.
As an example, I have an automation that triggers based on the wifi BSSID my phone is connected to. Actually what I'm trying to achieve is to understand in which room I am. Obviously, the trigger won't work if I'm without my phone, or it might need some "massaging" to get it to be more exact.
If the AI could suggest some extra sensors that might give a better indication: add a camera, add a mmWave presence sensor, etc... that might be great (you might even monetize the affiliate purchase links)

Thank you for your great ideas! I was wondering if we could create a small chatbot within HASS that would allow users to suggest the types of automations they would like to bring together. One challenge we face is associating each of the possible entities, which will require comprehensive intelligence across all entities.
Additionally, I've noticed that there are very few comments on each of the entities, so I might need to explore the repository for more information.
I'll take some time to think about how we can integrate these ideas. My initial thoughts involve using AI agents to gather information into a local repository, allowing AI to analyze the collected data. While this is a larger project, it is not impossible.


Support google.api.HttpBody in gRPC/JSON transcoding

Motivation:
#5311
Trying to make this work.
Modifications:

Parse media type and content dynamically if method descriptor is a google.api.HttpBody.

Result:

Closes #5311

Need to do some testing, I couldn't fully understand how to do this. The way I am doing it is a bit tricky bit I guess it should work

Oops, approved by mistake 😅


If token not set, waits for the input and set it into config

It no longer waits for #23 but https://github.com/prgres/clickup-tui/issues/45 was an enabler for it. From now it should be rather easy to add that


Add Salary Calculator

Create calculator settings page
Create salary calculation page
Add calculation logic to code
Add 'calculation settings' table to DB using user_id and category_id as composite primary key
Possible addition: monthly salary quick view widget on home page

#156


Menu order is not alphabetical in Settings and Documentation

Microsoft PowerToys version
0.66.0
Installation method
Chocolatey
Running as admin
No
Area(s) with issue?
General
Steps to reproduce
Open Settings. The File Locksmith and File Explorer add-ons are not alphabetical like the rest of the items (excluding "General"):

Also, the documentation site does not match the order of the Settings either:

I understand this is very minor. I am simply wanting consistency for how these items are listed both in the UI and on the documentation.
✔ Expected Behavior
Consistent alphabetization
❌ Actual Behavior
Inconsistent alphabetization
Other Software
n/a

This has been fixed!


Update interop and stress tests to GA API

The only code in the repository that should call the Beta API should be the tests of the Beta API.
This is currently blocked on issue 6799 because the oauth2_auth_token, jwt_token_creds, and per_rpc_creds interop tests are implemented in terms of the oauth2client library and the stress tests are implemented in terms of the interop tests.

Resolved by pull request 7702.


Missing Commit

Hi,
Our composer install is failing (Using prefer-dist on our build server) and a commit / tgz seems to have disappeared?
  - Installing zendframework/zend-stdlib (2.3.0)
    Downloading: 100%
    Failed to download zendframework/zend-stdlib from dist: '/Users/rawkode/Development/TeamRock/Blackbird/symfony2-application/application/vendor/zendframework/zend-stdlib/Zend/Stdlib/b6a780f333e513e358f7df0ebdf9c6cb' is not a zip archive.
    Now trying to download from source
  - Installing zendframework/zend-stdlib (2.3.0)
    Cloning d1c481b8a43f2f079b16d3567960ba539e9dacaa
    d1c481b8a43f2f079b16d3567960ba539e9dacaa is gone (history was rewritten?)

Please run composer update update your composer.lock

Unfortunately another package we use is looking for this specific version

@rawkode Does not matter. Just update so your composer.lock takes the new reference (426b5396e89e7da2db9678bc9a0b57865f84fe0f)

OK - Thanks.

I'm getting the same issue but composer update doesn't seem to shift it
Installing zendframework/zend-stdlib (2.3.0)
    Downloading: 100%
    Invalid zip file, retrying...
  - Installing zendframework/zend-stdlib (2.3.0)
    Downloading: 100%
    Invalid zip file, retrying...
  - Installing zendframework/zend-stdlib (2.3.0)
    Downloading: 100%
    Failed to download zendframework/zend-stdlib from dist: '/var/www/connect/vendor/zendframework/zend-stdlib/Zend/Stdlib/62f20877145837f45f6036f29d063c52' is not a zip archive.
    Now trying to download from source
  - Installing zendframework/zend-stdlib (2.3.0)
    Cloning d1c481b8a43f2f079b16d3567960ba539e9dacaa
    d1c481b8a43f2f079b16d3567960ba539e9dacaa is gone (history was rewritten?)

  [RuntimeException]
  Failed to execute git checkout 'd1c481b8a43f2f079b16d3567960ba539e9dacaa' -- && git reset --hard 'd1c481b8a43f2f079b16d3567960ba539e9dacaa' --
  fatal: reference is not a tree: d1c481b8a43f2f079b16d3567960ba539e9dacaa

Honestly, I'm not sure what's going on here; the sha1 you're attempting to get does not exist anywhere in the tree, and, more interestingly, we imported the old component repository to the legacy branch to ensure all sha1's that existed previously would remain available.
I'd try doing the following:

Run composer clearcache to remove the local composer cache.
Remove your composer.lock file and run a fresh composer install.

This should grab the tag correctly at this point. If it still doesn't, I'm out of ideas.

It's not a cache issue, this occurs during the build of a docker container from scratch.
Looking at my dependancies, I see two packages with zend dependencies

ob/highcharts requires "zendframework/zend-json": "2.3.0" https://github.com/marcaube/ObHighchartsBundle/blob/1.2/composer.json
guzzle/guzzle requires  "zendframework/zend-cache": "2.*,<2.3" and "zendframework/zend-log": "2.*,<2.3"

I'm not using that older version of guzzle, that's been pulled in by geoip2/geoip2 - geoip2 has a beta which doesn't have a guzzle dependancy, so I've temporarily gone with dev-master. That eliminates the zend-cache/zend-log dependancy but that wasn't really the source of my problem.
The master branch of ob/highcharts requires "zendframework/zend-json": "~2.3", rather than "2.3.0", so I went with dev-master there too (at least until that is released). That means I'll pull in zend-json and zend-stdlib 2.5.1. Sure enough, that solved the problem.

This problem is very annoying and it is very recent. I have seen it on multiple machines repeatedly. It worked on my machine and a day later crashed on a colleague's one... That is so annoying in fact that im seriously considering arguing to remove the zend dependency completely and switch to different libraries.
Btw: Recommending composer update is a poor recommendation and not always an option. I have come to the conclusion that the repeated occurrence if this is related to zend (because I never ever saw this error with anything else than Zend), not to composer. So fixing this should be up to you! Here the most recent occurrence I encountered (where the composer.lock's age is around 1 day):
[exec]     Failed to download zendframework/zend-stdlib from dist: 'stdlib/Zend/Stdlib/9c5999b1f89b37acb6bd83698b3b86c3' is not a zip archive.
 [exec]     Now trying to download from source
 [exec]   - Installing zendframework/zend-stdlib (2.3.1)
 [exec]     Cloning c1f4830018b5d4f034d32fa01a9e17ea176f56f6
 [exec]     c1f4830018b5d4f034d32fa01a9e17ea176f56f6 is gone (history was rewritten?)

@lmanzke Honestly, I cannot, and never have been, able to reproduce it. I can verify that neither sha1 you reference is present, but I can also verify that neither was ever present in the original Component_ZendStdlib repository; the legacy branch of this repo has that entire history, and neither sha1 is present in that branch.
At this point, I've hit a wall; I cannot help you further, as (a) I can verify that those sha1 values are not present, so of course they will not install, and (b) I cannot verify that those sha1 value were ever valid.
What I can note is that it makes no sense that composer is mapping the 2.3.1 release to c1f48300, since that sha1 does not exist; that release tag currently maps to 6079302. This leads me to believe that you have a stale $HOME/.composer/cache and/or a MITM is seeding composer with invalid values. One step you can take is to execute composer clear-cache and attempt to install and/or update after that.
I recognize that you've already said "Recommending composer update is a poor recommendation and not always an option," however, I have no other options to present to you. We do have the entire original history in the legacy branch of this repository (which was done to ensure dist and source installs referenced by sha1 would continue to work); the installs you've shown are referencing sha1 values we do not have, and never have.

Not sure this helps, but…
The composer.lock that's checked into this zf2_sample_app  contains the d1c481b8a43f2f079b16d3567960ba539e9dacaa hash.
The composer.lock that's checked into Piwik-LdapConnection  contains the c1f4830018b5d4f034d32fa01a9e17ea176f56f6  hash.

@weierophinney I'm quite sure that the error was not on the composer side, as zend is the only library making me constantly hit that error message (and wanting to chuck something out the window because of it). I believe you are right that the hash is not present, but on the same note, you are probably mistaken in saying it never was there. As @akrabat points out, even composer.lock's in the zf2_sample_app contain these hashes.
I think composer is quite right in asking "history was rewritten?", because to my mind, that is the most likely explanation. I am not sure about your processes, but this seems like a history rewrite to me. If that is the case, then of course at some point git will remove certain commits like the ones missing as a part of its garbage collection (As no branch / tag / anything points to the commit anymore, making it look like it was abandoned and not needed anymore). This would explain why you came to the conclusion that it never was there, because your legacy branch would naturally also not contain it.
Please investigate that. Do you have anything rewriting history by any chance? (Maybe automatically?)

@Imanzke There's a ton of history behind this.
Prior to 2.5, we were using a manual process to sync the various Component_Zend* repositories from the main zf2 repo at each release. The process would literally go commit by commit on the zf2 repo, sync in the files for the given component, see if any change happened, and, if so, write a new commit referencing the commit on the main zf2 repo; tags were then done based on a combination of sha1 +  date. All commits were done only on the master branch; there was only one branch in any of these repositories. Furthermore, we only synced the library code for any given component. As you might imagine, the process was error-prone and untenable long term.
One of the big pushes after 2.4 was to create first-class repositories for each component, complete with tests and full history (not just references to the original repo). I've detailed that process elsewhere. We did this split into new repositories, dropping the Component_ prefix, and normalizing the names. Once the split was complete, we re-pointed packagist at the new repositories.
When we finished, I received a number of reports of people's composer.lock files breaking due to the fact that they were pointing at the Component_Zend* repositories, and updates were grabbing from the new zend-* repos, where the existing sha1 no longer existed. At that time, I copied in the master branch of each Component_Zend* repo into the corresponding zend- repository as a legacy branch to ensure all previously existing sha1s were available. The reports I had after doing this indicated it resolved the issues observed.
So, yes, history was rewritten, but the sha1s from the original component repositories are theoretically all present. I literally have no additional paths to explore at this time; we deleted the Component_* repos to prevent confusion from developers looking for the various component repositories, but only after the importing their master branches into their sibling repositories. We should have all the history; your report indicates we don't, but I have nowhere else to look.


Switch to pnpm

Will the switch to pnpm a problem if the package should be published to npm?
I understood that in order to publish the adapter in ioBroker, it needs to be on npm.

I am actually just playing around with pnpm. But I do think that it would be possible to have it on npm, even when this repo uses pnpm ...

Stopping this idea now ;)


(gnucash) Update required: latest stable version is 2.6.19

As of 2018-02-09, the latest stable version is 2.6.19. Thanks in advance for upating the package.
Expected Behavior
up-to-date with stable release
Current Behavior
not up-to-date with stable release
Possible Solution
update to latest stable release
Context
In v2.6.18 online banking isn't available when installed through chocolatey (probably has nothing to do with choco)

Thanks for letting us now, seems like the package wasn't updated due to the incorrect update of the 2.7.1 version in the past, and the version in this repository was forgotten to be rolled back.
A new version should be available in a few hours if everything goes according to plan.


Manage branches in UI

It would be cool, if we can manage the branches in the web ui like https://github.com/go-gitea/gitea/branches. List branches, create branch from branch and delete :)

Duplicate of #307


Add to comfyui manager

Would it be possible to add this to comfy ui manager?

The project is now on ComfyUI Manager.

Thank you!


update the SoLoader SDK to version 0.10.4 or higher

I just saw this from the Play console:

Your version of the SoLoader SDK is causing your app to crash for some users
Your app uses a version of the SoLoader SDK that is causing crashes on 64 bit-only devices. To resolve this, update the SoLoader SDK to version 0.10.4 or higher.

I also see this in app/build.gradle:
 testImplementation 'com.facebook.soloader:soloader:0.10.1'

Hi @nicolas-raoul,
It seems easy to fix this issue by changing the SoLoader SDK version in dependencies.
Did I understand this issue correctly?
If so can you assign this issue to me?
Thanks

Yes potentially easy, just meeds some testing afterwards. 🙂 It is yours!

Hi @nicolas-raoul, sorry for updating late for such an easy issue.
I changed the dependencies to testImplementation "com.facebook.soloader:soloader:0.10.5"
But when I ran the unit tests,   fun testCreateCameraPictureFile() failed caused by
java.lang.IllegalArgumentException: Failed to find configured root that contains C:\Users\Jasso\AppData\Local\Temp\robolectric-Method_testCreateCameraPictureFile17231003190764080051\fr.free.nrw.commons.beta-dataDir\cache\CommonsContributions\e6115ab3-969e-4ac7-aeda-a83b559a80267402240706843156903.jpg
But it seems caused by my local environment and OS system.
Can I open a pull request to run the CI test?

Yes, the CI will run the unit tests. 🙂


SIGSEGV compiling 1.54.0 librustc_mir on OSX

I've used a local root ca1096aafb2ccb56eed92cf3eecc2a4906471583 which crashes as:
Exception Type:        EXC_BAD_ACCESS (SIGSEGV)
Exception Codes:       KERN_INVALID_ADDRESS at 0x0000000000000050
Exception Codes:       0x0000000000000001, 0x0000000000000050

Termination Reason:    Namespace SIGNAL, Code 11 Segmentation fault: 11
Terminating Process:   exc handler [4891]

VM Region Info: 0x50 is not in any region.  Bytes before following region:<PHONE_NUMBER>39440
      REGION TYPE                    START - END         [ VSIZE] PRT/MAX SHRMOD  REGION DETAIL
      UNUSED SPACE AT START
--->
      shared memory            7ffffff08000-7ffffff09000 [    4K] r-x/r-x SM=SHM

Thread 0 Crashed::  Dispatch queue: com.apple.main-thread
0   mrustc                        	       0x10f044004 (anonymous namespace)::Visitor::visit_path_params(HIR::PathParams&) + 30
1   mrustc                        	       0x10effb47c HIR::Visitor::visit_trait_path(HIR::TraitPath&) + 62
2   mrustc                        	       0x10f043ed2 (anonymous namespace)::Visitor::visit_trait_path(HIR::TraitPath&) + 1378
3   mrustc                        	       0x10effacd4 HIR::Visitor::visit_params(HIR::GenericParams&) + 220
4   mrustc                        	       0x10effa56d HIR::Visitor::visit_struct(HIR::ItemPath, HIR::Struct&) + 25
5   mrustc                        	       0x10f03feed (anonymous namespace)::Visitor::visit_struct(HIR::ItemPath, HIR::Struct&) + 99
6   mrustc                        	       0x10eff89ff HIR::Visitor::visit_module(HIR::ItemPath, HIR::Module&) + 2125
7   mrustc                        	       0x10eff88e3 HIR::Visitor::visit_module(HIR::ItemPath, HIR::Module&) + 1841
8   mrustc                        	       0x10eff88e3 HIR::Visitor::visit_module(HIR::ItemPath, HIR::Module&) + 1841
9   mrustc                        	       0x10eff88e3 HIR::Visitor::visit_module(HIR::ItemPath, HIR::Module&) + 1841
10  mrustc                        	       0x10eff7ce4 HIR::Visitor::visit_crate(HIR::Crate&) + 158
11  mrustc                        	       0x10f03f93d ConvertHIR_LifetimeElision(HIR::Crate&) + 77
12  mrustc                        	       0x10ee0d3bb void CompilePhaseV<main::$_13>(char const*, main::$_13) + 36
13  mrustc                        	       0x10ee0bbae main + 6528
14  dyld                          	    0x7ff80192341f start + 1903

(for my reference) See https://github.com/thepowersgang/mrustc/pull/316#issuecomment-1732278118
Crash was in 1.54 rustc_mir

@catap  I don't have access to a mac to reproduce that error.
It looks like it's a null reference being passed through to one of the visitors (Address is 0x50), but I can't see quite where that could be coming from, there's no values that could be non-null near enough in that call tree.
Could you run with debugging enabled for that phase (MRUSTC_DEBUG=Lifetime Elision) and maybe with something like valgrind to try and narrow down where the bad reference is coming from?

@thepowersgang I've scrolled up terminal history and discover that line near the crash:
rustc-1.54.0-src/compiler/rustc_mir/src/borrow_check/region_infer/mod.rs:1864:50-58 warn:0:Unexpected attribute inline on expression

which seems quite related to the issue.
If it doesn't help I'll retry with MRUSTC_DEBUG but in a couple of hours.

Nah, that's completely unrelated - it's from the expand stage, and the crash is after HIR generation.

I spent maybe too long trying to reproduce the error by cross-building for OSX, and got a successful build of that crate.
It's likely some undefined behavior somewhere in that call chain being triggered by the compiler you're using.

@thepowersgang I'm using stock compiler from macOS 13 here. Nothing unusual. I just run the build on the machine and as soon as it fails will share with you an update

@thepowersgang I still have all build artifacts on that machine and was able to
make it quite fast. Here the output which you've required.
librustc_mir.rlib_dbg.txt.gz

I've used:
Apple clang version 14.0.0 (clang-14<IP_ADDRESS>)
Target: x86_64-apple-darwin22.4.0
Thread model: posix
InstalledDir: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin

to build everything

Iterator invalidation :(
Inherited bounds resized the parameter list currently being enumerated.

:(

The above commit (417f29394bd8948af407eb48be31750a5a62a7f5) has a potential fix.


Javadocs not always working for -SNAPSHOT versions

What happened?
Hi, when I mvn deploy one of my things as a -SNAPSHOT version, the javadocs link in the browser is sometimes not working. I don't really know when or why this happens, but it only seems to happen for multi-module maven artifacts that use -SNAPSHOT in the version.
Example: https://repo.jeff-media.com/#/public/com/jeff-media/jefflib/jefflib-core/13.0.0-SNAPSHOT
If you go there, you'll notice there's a javadoc .jar called jefflib-core-13.0.0-20230817.140707-1-javadoc.jar. When I click on the "book" button to open the javadoc browser though, it tries to open jefflib-core-13.0.0-20230817.140707-2-javadoc.jar instead, which in turns correctly tells me that this doesn't exist:
{"status":404,"message":"Cannot find 'com/jeff-media/jefflib/jefflib-core/13.0.0-SNAPSHOT/jefflib-core-13.0.0-20230817.140707-2-javadoc.jar' in local and remote repositories"}
Reposilite version
3.x
Relevant log output
No response

Hi, I have reproduced this bug with the steps you provided.
Currently, Reposilite uses the following code to determine the latest snapshot version:
https://github.com/dzikoysk/reposilite/blob/dc0373199ecf028a0096228e8c3c9025c408f336/reposilite-backend/src/main/kotlin/com/reposilite/javadocs/JavadocContainerService.kt#L66-L73
I had a look at the maven-metadata.xml file for the artifact that was used in your example (https://repo.jeff-media.com/public/com/jeff-media/jefflib/jefflib-core/13.0.0-SNAPSHOT/maven-metadata.xml) and noticed:
    <snapshot>
      <timestamp>20230817.140707</timestamp>
      <buildNumber>2</buildNumber>
    </snapshot>

This issue seems to be caused by the build number here being 2, whereas the artifacts have the build number 1.
This causes Reposilite to look for a javadoc .jar file with the build number 2, which does not exist.
I am currently unsure of what is causing the build numbers to differ, and as soon as I have some more spare time, I will investigate further.

Hi, thanks for the quick reply. I am unsure why maven produces this "2" buildNumber - maybe it helps if I link the repo for that artifact? (You probably won't be able to build it as it relies on many minecraft libraries that are not available in any public repository): https://github.com/JEFF-Media-GbR/JeffLib However the pom doesn't do any funny things. Hmmm...

I was finally able to reproduce this issue in Maven yesterday.
I ran into a configuration issue (that I am still yet to resolve) that caused the Maven deploy plugin to run twice.
When this happens, it appears that each run bumps the buildNumber in the metadata - however only the first one increments the build number in the artifact names, thus causing this problem.

It sounds quite strange, I wonder if it's intended behavior of Maven deploy plugin 🤔


refactor: consistent naming of serialization method

Naming of to fields methods was incosistent. This PR fixes it.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

Current dependencies on/for this PR:

master

PR #4335

PR #4351

PR #4360

PR #4364

PR #4365

PR #4379   👈

This stack of pull requests is managed by Graphite.

Benchmark results
Metrics with a significant change:

note_trial_decrypting_time_in_ms (8): 18.7 (-83%)

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
Values are compared against data from master at commit 16158031 and shown if the difference exceeds 1%.
L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
8 txs
32 txs
128 txs

l1_rollup_calldata_size_in_bytes
45,412
179,524
716,068

l1_rollup_calldata_gas
221,728
866,908
3,448,216

l1_rollup_execution_gas
314,215
984,648
3,667,873

l2_block_processing_time_in_ms
1,151
4,340
17,219

note_successful_decrypting_time_in_ms
307 (+3%)
977 (+2%)
3,637 (+1%)

note_trial_decrypting_time_in_ms
:warning: 18.7 (-83%)
111 (+15%)
139 (+1%)

l2_block_building_time_in_ms
17,748 (-2%)
70,189 (-2%)
281,639 (-2%)

l2_block_rollup_simulation_time_in_ms
13,084
51,630
207,645

l2_block_public_tx_process_time_in_ms
4,638 (-8%)
18,425 (-9%)
73,722 (-8%)

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 16 txs.

Metric
5 blocks
10 blocks

node_history_sync_time_in_ms
14,008 (+2%)
27,666 (+6%)

note_history_successful_decrypting_time_in_ms
2,384 (-4%)
4,720 (-1%)

note_history_trial_decrypting_time_in_ms
97.5 (-45%)
198 (-11%)

node_database_size_in_bytes
17,338,448
33,239,120 (+1%)

pxe_database_size_in_bytes
29,923
59,478

Circuits stats
Stats on running time and I/O sizes collected for every circuit run across all benchmarks.

Circuit
circuit_simulation_time_in_ms
circuit_input_size_in_bytes
circuit_output_size_in_bytes

private-kernel-init
310 (+1%)
44,481
26,193

private-kernel-ordering
194 (+1%)
44,513
14,929

base-rollup
1,405
128,970
881

root-rollup
82.7 (+1%)
4,088
677

private-kernel-inner
416 (+1%)
71,640
26,193

public-kernel-private-input
240
32,615
26,193

public-kernel-non-first-iteration
240
32,657
26,193

merge-rollup
7.67 (+1%)
2,608
881

Tree insertion stats
The duration to insert a fixed batch of leaves into each tree type.

Metric
0 deployed contracts
1 deployed contracts

tx_size_in_bytes
15,575
37,974

Transaction processing duration by data writes.

Metric
0 new commitments
1 new commitments

tx_pxe_processing_time_ms
548 (+1%)
1,218 (-11%)

Metric
0 public data writes
1 public data writes

tx_sequencer_processing_time_ms
0.478 (-17%)
573 (-9%)


Pass string instead of Path object to open

What is the error
A path object cannot be directly opened like a filename string.
How to reproduce
>>> from pathlib import Path
>>> p = Path('xdf_sample.xdf')    # it's a path object
>>> open(Path('xdf_sample.xdf'))  # but open only accept str / bytes
TypeError: invalid file: PosixPath('xdf_sample.xdf')

Same as gzip.open
>>> import gzip
>>> gzip.open(Path('foo'))
TypeError: filename must be a str or bytes object, or a file

How to fix
>>> p = Path('foo')
>>> open(str(p))

BTW, I suggest NOT to mix using os.path and pathlib. So all os.path functions are replaced.
Please consider merging.

Python 3.5.2
Ubuntu 16.04

This is a 3.5 backwards compatibility patch. In Python 3.6 and up, open accepts Pathlike objects.
It looks good to me. Thanks for the PR!

@hankso would you like your real name to be shown in the changelog for your contribution?


Is it possible that this plugin can work with floating window?

I'm the author of NeoZoom.lua, which is a plugin that will open the current buffer into a floating window. I will be happy to see that it can work with codewindow.nvim :) Currently it looks like this:
Before zoom-in:

After zoom-in: As you can see the minimap is not shown (or maybe it's shown but covered by the floating window created by NeoZoom.lua)

If there is possible to adjust the z-index of the minimap (considering it's also a floating window?), maybe it would be a good idea for users to customize the z-index.

Currently it has a z-index of 1, specifically for the reason of not showing the minimap if somehow it ended up being displayed on a floating window, but sure, this seems simple enough.

https://github.com/gorbit99/codewindow.nvim/commit/ad9ec2402307911e9df2f1802f93c9a952e28de2 should introduce this feature, give it a try and see if it works with your usecase

@gorbit99 Oops, I was trying to solve it before I saw your update. Maybe you could accept my PR since z_index can be omitted from the config = {...}.

It can, but it has a default value of 2, if it is. I closed the pr because it didn't bring anything else to the table https://github.com/gorbit99/codewindow.nvim/commit/ad9ec2402307911e9df2f1802f93c9a952e28de2 wasn't already

OK, it's up to you :) But any non-existing keys in a lua table are actually set to nil by default. i.e. I didn't mean to let users set to nil explicitly, but instead users can omit z_index when they just want the default.

It can, check out setup in codewindow.config. I merge the given config table and the defaults, so if anything isn't present, it will be set to the default.


Method 'addPostFrameCallback' cannot be called on 'WidgetsBinding?' because it is potentially null.

Flutter version 2.10.5
Dart version 2.16.2
Channel Stable
I am currently facing this issue whenever I try to import the package

Any compilation error ?
Or these warnings only.

Any compilation error ?
Or these warnings only.

It happens when I'm trying to hot reload after importing the package


will JedisClusterInfoCache.java cache READONLY slaves slots  ?

@carlvine500 no, it will not.
in JedisClusterInfoCache.java, line 43 and 44:

ClusterNodeInformation clusterNodeInfo = nodeInfoParser.parse(nodeInfo, new HostAndPort(jedis.getClient().getHost(), jedis.getClient().getPort()));
in nodeInfoParser.parse will parse command cluster nodes result, for example:

e499ce8577471779d43eb699197c693d9a20de65 <IP_ADDRESS>:6381 master - 0<PHONE_NUMBER>063 2 connected 5461-10922
517388f1d7f1edcaad07598035a4fad6a369a610 <IP_ADDRESS>:6382 slave e499ce8577471779d43eb699197c693d9a20de65 0<PHONE_NUMBER>042 5 connected
a73fb4f53cc132efec6be0f7681c7ad39d5dfbdf <IP_ADDRESS>:6379 myself,master - 0 0 1 connected 0-5460
da6b53b8386ebf7cdbdd426a9376f7370faed507 <IP_ADDRESS>:6380 slave a73fb4f53cc132efec6be0f7681c7ad39d5dfbdf 0<PHONE_NUMBER>575 4 connected
b3903cd4a1fd0af78ac7e12bc6ac615b440494c0 <IP_ADDRESS>:6383 master - 0<PHONE_NUMBER>063 3 connected 10923-16383
bcab0414b31948fccf3c7bc813aadd0830242c3f <IP_ADDRESS>:6384 slave b3903cd4a1fd0af78ac7e12bc6ac615b440494c0 0<PHONE_NUMBER>533 6 connected

u can see, in e499ce8577471779d43eb699197c693d9a20de65 <IP_ADDRESS>:6381 master - 0<PHONE_NUMBER>063 2 connected 5461-10922, 5461-10922 are the slots on this instance, and this instance is a master, and in 517388f1d7f1edcaad07598035a4fad6a369a610 <IP_ADDRESS>:6382 slave, this instance is a slave, and no slots info.
so JedisClusterInfoCache  will not cahce slaves slots.

will jedis support READONLY slaves in the future ?

@carlvine500 in redis cluster, u can't read data from slave, if u do this thing, MOVED error occurred.

With READONLY you can read data from slave, but AFAIK Redis Cluster doesn't treat slave down so I don't want to handle slave.
Please correct me if I'm wrong!

thands @HeartSaVioR . May antirez will support it . We will monitor slave , and skip reading from it after down.


Failure rendering DDBMuncher

Foundry Version: 0.8.9 (Build 102)
Module Version: 2.7.7
When clicking on the DDB Muncher button, it throws the following error:
foundry.js:2040 Foundry VTT | Rendering DDBMuncher

foundry.js:2013 TypeError: Cannot read properties of undefined (reading 'sources')
[Detected 1 package: ddb-importer]
    at getSourcesLookups (main.js:22871:65)
    at getMuncherSettings (main.js:22632:23)
    at DDBMuncher.getData (main.js:23234:24)
    at DDBMuncher._render (foundry.js:2049:29)
    at B.🎁call_wrapped [as call_wrapped] (libWrapper-wrapper.js:466:22)
    at DDBMuncher.🎁Application.prototype._render#lib-wrapper (listeners.js:110:11)
    at DDBMuncher.🎁Application.prototype._render#0 (libWrapper-wrapper.js:161:52)
    at DDBMuncher.render (foundry.js:2011:9)
    at HTMLDivElement.<anonymous> (main.js:34482:30)

The error happens both in the client and in chrome. The rest of the features in the module seem to be working find, including every other import feature.

Please can you try updating DDB Importer to the latest version 2.9.30, it sounds like you are a bit out of date with some changes.


Update NFC

A few nfc updates

Once again, the Flipper Team achieves what was once deemed impossible! I know there's been even more NFC updates since this PR was made, I'll try to make sure they all get accounted for in this table.


Has this been abandoned?

Has this repo been abandoned? Is there a newer alternative?

open PRs from 2012 and 2014, no commits to master since 2015 and the other open branches are even older? I think so


Update for Xcode 9.3/Swift 4.1

Basically I just did 2 things:

Automatic Xcode upgrade of the project
Replaced flatMap with compactMap where said map returned optionals

Note: Travis Build shall fail until it's updated to use Xcode 9.3 image

Was going to introduce a PR for the same.
But the tests are failing because compactMap is only implemented in Swift 4.1+.
Hence, the tests fail on Swift 4.0 and earlier. The solution should be to conditionally use compactMap/flatMap based on the Swift compiler version used.

@edwardmp Hey! Yes that makes sense, I could do those changes. How would you suggest doing this? Using a pre processor macro like this:
#if swift(>=4.1)
// .compactMap
#else
// .flatMap
#endif

I confess I'm not used to use pre processor macros, as such my only concern is that I would probably duplicate a lot of code (at least the whole block inside the map function).

@pcantrell I would normally agree, but since flatMap is used in just 5 places it might make sense to do this the "easy" way. Although I do have to say that a lot of my dependencies cause warnings for the same issue, so I guess I'll be stuck with those warnings for a while!

@joaomvfsantos Hi. In order to avoid code duplication as you said, you can store the closure in a variable. This way you only need one line inside the conditional cases.
For Example:
let stringNumbers = ["1", "2", "3", "x", "4", "5"]
let closure: (String) -> Int? = { Int($0) }

#if swift(>=4.1)
    let numbers = stringNumbers.compactMap(closure)
#else
    let numbers = stringNumbers.flatMap(closure)
#endif

print(numbers)

PS. I was about to make a PR about these annoying warnings lol, good think I checked first.

@Linnk
Good idea! Can you submit a PR implementing this, at least, if @pcantrell  is willing to settle for this?

@edwardmp Well, I was doing it… but then I read the comment about the siesta and swift versions and stuff. And I get it, I'm not fan either of those #if conditions.

@Linnk Thanks for taking the time to give a concrete example. I started to doing an approach like that but after @pcantrell comment I did not proceed. His view also makes sense to me.

If we were going to simultaneously support Swift 4.0 and 4.1, I’d prefer to do it with a conditional extension, which can live in its own separate file and doesn’t clutter up usage sites:
#if !swift(>=4.1)
    extension Sequence
        {
        func compactMap<ElementOfResult>(_ transform: (Self.Element) throws -> ElementOfResult?) rethrows -> [ElementOfResult]
            { return try flatMap(transform) }
        }
#endif

However, as I said above, I don’t see any particular reason not to just require 4.1 in the next Siesta release. If somebody does find a compelling reason to keep 4.0 support, we could add the above.
For now, if you want to get this PR passing CI, try updating the .travis.yml to require the newer Xcode.

Also P.S. I expect to be pulling together Siesta 1.4 in early May. There are some awesome new caching and custom request building features in the works, mostly complete; just have to survive the end of the semester first! If anybody wants a sneak preview, all that work is on the file-cache branch.

@pcantrell I changed travis to use the Xcode 9.3 image but it's failing only on iPhone 4s simulator. Any ideas why?


5.3.2: Property or method "then" is not defined on the instance but referenced during render

Version
5.3.2
Browser and OS info
Mac OS 10.13.6 / Version 0.69.135 Chromium: 77.0.3865.120 (Build officiel) (64 bits)
Steps to reproduce
Just load any vue app in dev mode.
What is expected?
The console should be empty.
What is actually happening?
The console is full of this kind of errors Property or method "then" is not defined on the instance but referenced during render for every component.

This wasn't occurring with 5.3.1.

I also get this error, but only when i enable "New Vuex backend" in settings.
When i disable this setting, the error does not appear.

Please provide a runnable minimal reproduction.

Just wanted to say I've been debugging my app for 2 hours so thank you @nicooprat for this issue! 😞 🙌

Glad it's not just me. I was going crazy this morning with it. I'm going to have to restore old dev tools between this and consuming all of my CPU the latest version seems a bit buggy.

Just came across this as well today. Turning off Vuex backend removed all those warnings.

I noticed it this morning after I updated to Chrome 78.
Toggling off "New Vuex Backend" in the Vue Dev Tools settings has prevented these warnings as others have suggested.

AAAAAAAAgggggh I just spent 30 minutes debugging my app and turning off things! I was about to open this issue too. Definitely a bug

#metoo
FYI, exception in console refers to this line in backend.js:
https://github.com/vuejs/vue-devtools/blob/30f15164aabf216b4fcd4b147c2377e598380912/packages/app-backend/src/clone.js#L379
Vue.js devtools 5.3.2

I can reproduce the error, if "new vuex backend" and recording on the vuex tab are enabled.
disabling any of them stops the error from happening

I am also experiencing this exact same issue. Turning off "new vuex backend" fixes it for me as well.

I think this might be related to using import to load components. I'll try to add a minimal example.

Hello!
My case for reproduce
You need app that uses following packages:

Vue Router
Vuex
vuex-router-sync

App should have minimum two routes =)
Steps:

Open your app in browser
Open Vue developer tools
Navigate to other app page (route)
vuex-router-sync runs mutation route/ROUTE_CHANGED with payload {from, to} where will have references to components and their instances
Vue developer runs cloning of mutation payload and instances of Vue components
When devtools try to clone payload.to.matched.0.instances.default._renderProxy Proxy checks the availability of method then and shows warning

Spend an Hour to fix this. Thank again for posting this on the forum!!!

it,s was there being two different version of vur-devtools on my chrome

同时存在两个不同版本的vue-devtools，禁用调其中一个就OK了

Getting this error in version 6.0.0 beta 19


main: Pass config_dir as &str

Generally ownership transfer by passing String parameters is unidiomatic.  By passing the config dir as a borrowed &str we avoid the need to heap allocate a copy of the string /tmp.
(No particular motivation, I just happened to be looking
at this code)
Signed-off-by: Colin Walters<EMAIL_ADDRESS>

Are we good to rebase + merge?

Code LGTM for reference


Lower sklearn requirement to 0.17

I am currently limited to using scikit-learn 0.17 due to the fact that model persistence isn't guaranteed to work when models were trained/written to a file in one version and then loaded from another version (https://scikit-learn.org/stable/modules/model_persistence.html#security-maintainability-limitations).
I don't see any reason mleap requires scikit-learn >= 0.18.dev0 as opposed to 0.17, so would it be possible to relax that constraint slightly?

0.17 is quite old (released around 2015), in fact, I'd like us to try to keep updating the dependencies so that we use the newer/latest versions.


minimal logging

I noticed Ian removed his Logging.h, not sure of the rationale, shouldn't we still use some sort of minimal logging system instead of errs() << ....
I had introduced these changes to my fork while trying to understand Occam, it uses LLVM's DEBUG macro to only log when -debug switch is on.
Please feel free to discard if I am being the new guy who doesn't understand crap and is only trying to get attention :-)

I removed the logging because it was redundant.
  export OCCAM_LOGFILE={absolute path to log location}
  export OCCAM_LOGLEVEL={INFO, WARNING, or ERROR}

does the job (have not tested the level stuff, but the errs() stuff gets redirected to the logfile.

errs() stuff is still compiled into the pass by LLVM, the advantage of using DEBUG macro is that it strips the code for release builds.
just my two cents.

Yes  I am sure we can optimize.  Might do so once the thing is in a more polished state.

The commits related to debugging, logging, and counting instructions (why?) should be in separate pull requests. Since the Python frontend has been rewritten, the changes may no longer be usable.


Move car harness information into openpilot

So we can add another check to the "Actively Maintained" star and to keep the website up to date by requiring all new car ports add harness information

Closed with https://github.com/commaai/openpilot/pull/24085


Addon crashes my Playnite entirely

For some reason, when I install your Playstation integration addon, then go into the settings to configure it, moments later the entire Playnite app will crash. This was not an issue a few months ago, I then recently updated to latest version, and now the crashing began.
I have tried fully uninstalling the addon, and deleting the addon data, then reinstalling it. Crashing still occurs.
The crash happens within 2-3 seconds of opening the addon settings window. I can see the addon settings, and if I hover over any tickbox it seems responsive for about 1 second. Then it becomes unresponsive for another second or two, then the entire app crashes.
Playnite does not give me a crash diagnostic, so this is a full diagnostic from About Playnite window:
dd9d2731-e255-4397-bdd4-7f08453d5828
Windows 11 (latest)
Playnite 10.32 (latest)
Your addon 2.31 (latest)

Uh this reminds me I need to update the bug template, I don't actually have access to diagnostics, this is copied over from Playnite maintainers repo. In Playnite data folder (you can access it trough menu->about->data or smt like that) there's and extension.log file, let's start by looking at that - I'll be able to see where it fails and compile a version with some extra debugging info.
If there's no crash dialogue, then chances are there might be no logs too.
One other thing you can try is to delete plugin data and then also clear cache (menu/settings/advanced/clear web cache). That would also trigger a restart, and all PSN authentication data should be completely gone this way.

The crashing no longer occurs. I'm not sure why. I put off diagnosing it for the past two weeks, and I saw there was an update a few days ago. I also completely removed and reinstalled the addon (I tried doing that before and it didn't help). Anyway I am glad it works. I'll close the ticket. Thanks


3D Scatter Plot animation

Hi
Is there a way to update the data in a 3D scatter plot in order to produce an animation? (My intention is to plot 3D data in real time). I have checked both the animation.cpp and update.cpp examples, but had no luck. Any help would be appreciated.
Many thanks.

Hi
Have you found a solution? if have, Can you share with me...

It looks like they haven't considered 3D animation. 2D animation works fine but things will get strange when I try to plot 3D figures.

Hi, I figured out a way to solve this problem. To make an animation of the 3d plot, you have to specify the figure handle. The header get it done by default for the 2d plot, but not for 3d. So here is a possible way for doing this:


"Get-WindowsOptionalFeature: Class not registered" on W11 with PowerShell 7.4.1

Describe the issue
I'm trying to create a new container, from VSCode running as an administrator (yes, I know I can do that while not running as Admin, but I got other permissions issues that I didn't have time to dive into right now).
In New-BcContainer there is a call to Get-HypervState if process is running as Admin.
In that function, the state of Hyper-V is checked via Get-WindowsOptionalFeature -FeatureName Microsoft-Hyper-V -Online
But on my W11 machine, running PowerShell 7.4.1, that results in the error (after a loooong time)
Get-WindowsOptionalFeature: Class not registered
>$PSVersionTable

Name                           Value
----                           -----
PSVersion                      7.4.1
PSEdition                      Core
GitCommitId                    7.4.1
OS                             Microsoft Windows 10.0.22631
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Scripts used to create container and cause the issue
Anything that calls New-BcContainer in an elevated mode.
Full output of scripts
Below is after I removed Write-Host "HyperV is $(Get-HypervState)" from New-BcContainer, otherwise I got stuck after the second line...
BcContainerHelper is version 6.0.7-preview1150
BcContainerHelper is running as administrator
UsePsSession is True
Host is Microsoft Windows 11 Enterprise - 10.0.22631.3155
Docker Client Version is 24.0.7
Docker Server Version is 24.0.7

Additional context

does it happen all the time?

Always if running as admin (unless uncomment the Write-Host line mentioned above 😉)

did it use to work?

Yes, before I updated to PowerShell 7

Strange

Do you have Hyper-V installed?

Yes, strange indeed.
It seems as if this issue is from Get-WindowsOptionalFeature, and not related to BcContainerHelper.
See https://github.com/PowerShell/PowerShell/issues/13866 for more info.
Even this fails:
PowerShell 7.4.1
PS C:\Users\nabjohwik> Get-WindowsOptionalFeature -Online
Get-WindowsOptionalFeature: Class not registered

As a workaround this comment seems to help: https://github.com/PowerShell/PowerShell/issues/13866#issuecomment-1519066710
PowerShell 7.4.1
PS C:\Users\nabjohwik> Get-WindowsOptionalFeature -FeatureName Microsoft-Hyper-V -Online
Get-WindowsOptionalFeature: Class not registered
PS C:\Users\nabjohwik> Import-Module DISM -UseWindowsPowerShell
WARNING: Module DISM is loaded in Windows PowerShell using WinPSCompatSession remoting session; please note that all input and output of commands from this module will be deserialized objects. If you want to load this module into PowerShell please use 'Import-Module -SkipEditionCheck' syntax.
PS C:\Users\nabjohwik> Get-WindowsOptionalFeature -FeatureName Microsoft-Hyper-V -Online

FeatureName      : Microsoft-Hyper-V
DisplayName      : Hyper-V Platform
Description      : Provides the services that you can use to create and manage virtual machines and their resources.
RestartRequired  : Possible
State            : Enabled
CustomProperties :

But I do not know if the side-effects mentioned is something that will affect other stuff... 😕
Would it make sense to try to catch this (and only this) error inside Get-HypervState function and default to blank, as is the default behavior if not running as admin?
And when this eventually is solved in PowerShell, this would just start to work.
Since you cannot repro this, I can create a PR for that try/catch, if you're open to that approach?

Yes, I think resilience here is the right path. Please create a PR with this

PR #3382 created.

PR #3382 created.

Thanks


feat: Changed color type to integer

Closes #34

:tada: This PR is included in version 1.0.0-alpha.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Update master

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@ad067f7). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master       #1   +/-   ##
=========================================
  Coverage          ?   95.45%
=========================================
  Files             ?        2
  Lines             ?       22
  Branches          ?        0
=========================================
  Hits              ?       21
  Misses            ?        1
  Partials          ?        0

Continue to review full report at Codecov.

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


[v3] Working version and guide on VPS

Needs steam login, should compare to the bots admins array
Will need to assume they’re running with pm2/ecosystem
Because of that, needs support for multiple bots

Will edit if I find out more stuff it needs

Verify user accesing website
maybe make comunication with sockets not steam chat

Verify is done using the steam login, isn't it

Split vue into component
maybe use webpack
maybe babel for better browser compatibility

I've made a work in progress guide on VPS installation
https://github.com/ZeusJunior/tf2-automatic-gui/wiki/Installation-on-VPS

Maybe make a login page, that requires user to input username and password/pin that he/she set in config.json.

@ZeusJunior wanted to do login with steam oauth. And it might be better that way, because it should not need SSL.

I see... 👍🏻

Temporary method to run tf2-automatic-gui on multiple bots or if you run the bot on a shared VPS:

set custom port.
https://github.com/idinium96/tf2-automatic-gui/blob/custom-port/app/index.js#L16
I can do a PR if it's not a bad idea.

set custom port.
https://github.com/idinium96/tf2-automatic-gui/blob/custom-port/app/index.js#L16
I can do a PR if it's not a bad idea.

Not yet, as I don’t recommend running it on VPS currently.

alright no problem

@idinium96 If you'd like to follow progress, its currently on this branch https://github.com/ZeusJunior/tf2-automatic-gui/tree/dev-z
(If that link ever stops working it means it was deleted and merged with regular development)

@ZeusJunior Thank you 👍🏻👍🏻👍🏻

Suggestion:

add a button to restart the bot, which will run command pm2 restart <appName or appID> on the top right of the GUI page.

If my tf2-automatic modification will work, this should not be needed


Table 39 Purchase Line [External] InitQtyToInvoice() Function

Kindly add below code in Table 39 Purchase Line - InitQtyToInvoice() Function
OnBeforeInitQtyToInvoice(Rec,CurrFieldNo,IsHandled);
IF IsHandled THEN
EXIT;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


SyntaxError: Unexpected token import (import {AotPlugin} from '@ngtools/webpack')

Bug Report or Feature Request (mark with an x)
- [x ] bug report -> please search issues before submitting
- [ ] feature request

Versions.

npm -v
4.6.1

node -v
v6.10.0

webpack version 3.4.1

Repro steps.

Following the documentation readme instructions, Adding this to webpack config file causes immediate compilation error
import {AotPlugin} from '@ngtools/webpack';

The log given by the failure.

import {AotPlugin} from '@ngtools/webpack'
^^^^^^
SyntaxError: Unexpected token import
    at Object.exports.runInThisContext (vm.js:76:16)
    at Module._compile (module.js:542:28)
    at Object.Module._extensions..js (module.js:579:10)
    at Module.load (module.js:487:32)
    at tryModuleLoad (module.js:446:12)
    at Function.Module._load (module.js:438:3)
    at Module.require (module.js:497:17)
    at require (internal/module.js:20:19)

The import keyword is not supported by Node.  You need to use require('@ngtools/webpack').

Ok but why doesn't it say that clearly in the documentation page? Its too much to assume people will know this. Please update docs with this explanation. Thanks

Same problem. Changed it to require and then I get
new AotPlugin({
^
TypeError: AotPlugin is not a constructor

Try - new AotPlugin.AotPlugin({})


DOC: Update Footer Section Link Buttons

What's wrong with the existing documentation
https://github.com/Suchitra-Sahoo/AgriLearnNetwork/assets/168094736/ac904c2c-25bc-485d-8c0d-b7a050349069
we can remove underline on hovering and add color on it
Add ScreenShots
No response
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@Suchitra-Sahoo please assign this project to me i did 4 projects like this
and add GSSoC level on it


fix: InterfaceType#element -> InterfaceType#element2

fixes: #14

Okay, I'll review the previous PR and make sure that I change them. This was the only one I distinctly remember changing, but I might've changed others. I'll have this PR ready by the morning

Hey!

Oh, I understand what happened. element was deprecated in 4.7.0, removed in 5.0.0 and restored in 5.2.0. So sometime element2 will be removed. In this case, do we need to support it? What do you think?

If we update the dependencies to support min 5.2.0, do we still need to use element2? It might be resolved if we just bump the version.
In what use case does this error come up? It'd be great to have a test for this to prevent breaking it in the future


OverrideWithProviderMixin not working on web

Error on master flutter channel using web
Uncaught TypeError: framework.OverrideWithProviderMixin$ is not a function
at :44719/packages/riverpod/src/state_provider.dart.lib.js:539
I'm guessing this is a dart issue?

I actually am getting this on flutter stable as well.

Sounds like a Dart issue. There's no native code after-all. So if something doesn't work on web, that's not Riverpod's fault

I'm going to close this, as I'm unable to reproduce on other machines. I'll restart the one I had issues with

This seems to be the same issue as: https://github.com/rrousselGit/river_pod/issues/889

I'm guessing it's a dart issue with web. I can see if I can create a simple reproducible issue for the dart team.

Anyone able to establish flutter and riverpod versions that work together to continue with development before this is getting fix ?

@jbremmer  flutter_riverpod: ^1.0.0-dev.11

@metafounder I'm on
flutter --version
Flutter 2.7.0-3.1.pre • channel beta • https://github.com/flutter/flutter.git
Framework • revision fc7015e35a (13 days ago) • 2021-10-27 15:18:41 -0700
Engine • revision a75e3b80f9
Tools • Dart 2.15.0 (build 2.15.0-178.1.beta)

flutter pub deps
|-- flutter_riverpod 1.0.0
|   |-- collection...
|   |-- flutter...
|   |-- meta...
|   |-- riverpod 1.0.0
|   |   |-- collection...
|   |   |-- meta...
|   |   '-- state_notifier...
|   '-- state_notifier 0.7.1
|       '-- meta...

with the version
flutter_riverpod: ^1.0.0-dev.11
this will lead me to same situation with the error Uncaught TypeError: framework.OverrideWithProviderMixin$ is not a function
then switching to
flutter_riverpod: 1.0.0-dev.11
yields bunch of build time errors.

oh ok, mine is (which is working together with flutter_riverpod: ^1.0.0-dev.11 ):
Flutter 2.5.2 • channel stable • https://github.com/flutter/flutter.git
Framework • revision 3595343e20 (6 weeks ago) • 2021-09-30 12:58:18 -0700
Engine • revision 6ac856380f
Tools • Dart 2.14.3

So the people who are still facing issue with the same problem.
Problem: It occurs to be dart sdk problem with web on stable channel version : 2.5._ and above. There is nothing wrong with riverpod itself.
Workaround: You need to switch from stable channel to master channel.


do the plugin work now?

i can not create wallet now
do the plugin work now ?
thanks

Plugin is working now - thanks!


[BUG]: colorScheme doesn't get passed to input variants

Description
When I pass colorScheme to the Select component. I expect to be able to use in my theme file. But the colorScheme comes back as undefined.
If I copy exactly what it here:
https://github.com/chakra-ui/chakra-ui/blob/main/packages/components/theme/src/components/input.ts
Let's console.log the colorScheme in variantOutline.
const variantOutline = definePartsStyle((props) => {
  const { colorScheme, theme } = props
  const { focusBorderColor: fc, errorBorderColor: ec } = getDefaults(props)

  console.log({ colorScheme });

  return { ... }
})

When I do this for a normal Input it correctly shows the colorScheme that I passed to my component. This works for other libs that use Chakra's input under the hood too.
But with this library it returns undefined.
chakra-react-select Version
4.6.0
Link to Reproduction
No response
TypeScript?

[X] Yes I use TypeScript

Steps to reproduce
No response
Operating System

[X] macOS
[ ] Windows
[ ] Linux
[ ] iOS/iPadOS
[ ] Android

Additional Information
I think it's just getting cleared somewhere. I tried to dig through the codebase to find it but everything is pretty much abstracted away.

For extra context I'm referring to the global colorScheme prop:
E.g. if it is used like this:
<Select colorScheme="red" />

Hmm I see what you mean. I think this is due to a difference in the way I'm using color scheme from how you're expecting it to be. As per the docs:

You can pass the colorScheme prop to the select component to change all of the selected options tags' colors. You can view the whole list of available color schemes in the Chakra docs, or if you have a custom color palette, any of the custom color names in that will be available instead.
Alternatively, you can add the colorScheme key to any of your options objects and it will only style that option when selected.

This prop was originally added for the Tag, component used to display the selected options for a multi-select. It was added a long time ago when this package was just a gist I had shared with some people who were interested, and I was just playing around with some customizations. Realistically I should probably have called it tagColorScheme, but it was hard to predict how far this project would come, and I didn't plan ahead very well. At this point, switching that around would be a breaking change that I'm not sure if I'd like to make, due to the fact that the Input component doesn't do anything with the colorScheme prop by default.
And on that note, what exactly are you trying to accomplish with this? What are you using the colorScheme prop for? Maybe there is some alternative approach?

I've managed to make a work around using the chakraStyle props like this:
chakraStyles={{
control: (provided, _state) => ({
...provided,
borderColor: isMaterial ? ${colorScheme}.200 : undefined,
color: isMaterial ? ${colorScheme}.500 : undefined,
_hover: {
borderColor: isMaterial ? ${colorScheme}.300 : undefined,
},
}), }}
It would be nice though to at least have something like chakraColorScheme
as a prop that would be passed down to that...
On Fri, 7 Jul 2023 at 21:41, Chris Sandvik @.***> wrote:

Hmm I see what you mean. I think this is due to a difference in the way
I'm using color scheme from how you're expecting it to be. As per the docs:
You can pass the colorScheme prop to the select component to change all
of the selected options tags' colors. You can view the whole list of
available color schemes in the Chakra docs
https://chakra-ui.com/docs/components/tag/props, or if you have a
custom color palette, any of the custom color names in that will be
available instead.
Alternatively, you can add the colorScheme key to any of your options
objects and it will only style that option when selected.
This prop was originally added for the Tag, component used to display the
selected options for a multi-select. It was added a long time ago when this
package was just a gist I had shared with some people who were interested,
and I was just playing around with some customizations. Realistically I
should probably have called it tagColorScheme, but it was hard to predict
how far this project would come, and I didn't plan ahead very well. At this
point, switching that around would be a breaking change that I'm not sure
if I'd like to make, due to the fact that the Input component doesn't do
anything with the colorScheme prop by default.
And on that note, what exactly are you trying to accomplish with this?
What are you using the colorScheme prop for? Maybe there is some
alternative approach?
—
Reply to this email directly, view it on GitHub
https://github.com/csandman/chakra-react-select/issues/267#issuecomment-1625983620,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABDVHNUPEW6BSIF2HS3NRI3XPBQ7ZANCNFSM6AAAAAA2BUDZHI
.
You are receiving this because you authored the thread.Message ID:
@.***>

As of v5.0.0, I renamed the attribute to be tagColorScheme to prevent confusion. I am also considering adding back the original colorScheme prop though, and making it apply to the select itself, but haven't decided whether to do so or yet.

As of v5.0.0, I renamed the attribute to be tagColorScheme to prevent confusion. I am also considering adding back the original colorScheme prop though, and making it apply to the select itself, but haven't decided whether to do so or yet.

that would be nice if i could control the color of the outline (border color) on the field.


Missing global namespace for class 'Exception' causing 'Class not found' exception

Probably, a typo, but classes always resolve to the current namespace, so we need to use fully qualified name here:
https://github.com/KyranRana/cloudflare-bypass/blob/5fde39d4c3b9e4a9670747bacc9e18663818208e/src/CloudflareBypass/CFBypasser.php#L182

Change new Exception to new \ErrorException

v3.0.0 out now.


SyntaxError: JSON.parse

Hi,
I get an error even when returning a newly created empty response.
POST Response:
SyntaxError: JSON.parse: unexpected character at line 2 column 1 of the JSON data
Firefox Devconsole (JS error):
SytaxError: expected expression, got '<'
I can see the JSON array (when sending data e.g. innerHTML) but it breaks afterwards.
Specs:
Apache: 2.4
PHP: 7.0

What happens if you trim the newlines from the HTML code you are assigning?

Any feeback on this issue?

I'm sorry for this issue. Was my personal fault. My Response included some HTML header an therefore was invalid. Nothing wrong with Jaxon.


Add a description of the return value to the docstring of the apply method.

Fix issue #4499 .

Codecov Report

Merging #4500 into master will decrease coverage by 2.45%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #4500      +/-   ##
==========================================
- Coverage   83.58%   81.12%   -2.46%
==========================================
  Files         349      349
  Lines       64978    64978
  Branches    10047    10047
==========================================
- Hits        54311    52715    -1596
- Misses       7827     9370    +1543
- Partials     2840     2893      +53

LGTM but you have PEP8 problems
https://travis-ci.org/mne-tools/mne-python/jobs/265639919#L3223

@larsoner I think the PEP8 is fixed.

Thanks for the quick fix @aweinstein


Switch from Travis to Github actions

Trello: https://trello.com/c/cnNEUjhz/2126-epic-migrate-travis-ci-usages-to-github-actions
Since Github actions can provide the same functionality as Travis CI it makes sense to migrate to it in order to:

Reduce 3rd party dependency
Benefit from faster build times due to lack of alphagov queue

Following the example of supplier-frontend

Once this has been merged, I'll remove Travis from the list of required statuses and add Github actions.

@bjgill can we remove the coveralls graphic from the readme?

Good point. Let me merge this now to get Brett's PR unblocked, and then I'll sort that out.