Wrong assignment of MS3 to MS2

Again thanks for this wonderful package!
We noticed some MS3 scans being attributed to the wrong MS2 scan.
It seems to be a result of an unfortunate partial string match here:
https://github.com/compomics/ThermoRawFileParser/blob/6e133e47a814d43b5a7c988c71c93d530ff54936/Writer/MzMlSpectrumWriter.cs#L30
https://github.com/compomics/ThermoRawFileParser/blob/6e133e47a814d43b5a7c988c71c93d530ff54936/Writer/MzMlSpectrumWriter.cs#L2156-L2159
For example, we have an MS2 with the following filter string
MS2 scan=2298: FTMS + c NSI d Full ms2<EMAIL_ADDRESS>[166.0000-1257.0000]
It gets assigned these two MS3 spectra:
MS3 scan=2300: FTMS + c NSI d Full ms3<EMAIL_ADDRESS><EMAIL_ADDRESS>[100.0000-1000.0000]
MS3 scan=2334: FTMS + c NSI d Full ms3<EMAIL_ADDRESS><EMAIL_ADDRESS>[100.0000-1000.0000]
Note that the match occurs on 623.0950, but it is incorrect in the second case which actually belongs to this MS2 spectrum:
MS2 scan=2332: FTMS + c NSI d Full ms2<EMAIL_ADDRESS>[241.0000-2000.0000]

Hi @MatthewThe,
thank you for reporting this issue. The way spectra are assigned to parents has been changed completely in a recent beta (see #126 ). Could you check if the error still exist in the version I have shared there? You can also share the problematic raw file with me if it is more convenient.

Yes, that seems to have fixed it. Thanks a lot!


Update test cases for OS Integ Jenkins file

Description
Update test cases for https://github.com/opensearch-project/opensearch-build/pull/3489
Issues Resolved
List any issues this PR will resolve, e.g. Closes [...].
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #3491 (4b21cf0) into main (7bc2286) will not change coverage.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##             main    #3491   +/-   ##
=======================================
  Coverage   91.16%   91.16%
=======================================
  Files         181      181
  Lines        5340     5340
=======================================
  Hits         4868     4868
  Misses        472      472


Replace autorefresh with advanced control

There is a button that toggle autorefresh in Diagnostics. It's needed to add additional button to refresh current Diagnostics tab content.
Possible solution - store fetch data functions added with useAutofetcher in autofetch class and trigger them on button click. Manual autorefresh should also postpone next passive autorefresh if it's enabled.

What intervals should we use for automatic updates?

15s
1m
2m
5m


Unable to remove all of the tokens in the selected frame

Describe the bug
Release 1.38.1 is unable to remove all of the tokens in the selected frame. Pressing the "Remove selected" button makes all of the selected objects with tokens applied invisible.
To Reproduce
Steps to reproduce the behavior:

Select any object in a Figma file with tokens applied.
Launch Tokens Studio for Figma.
Go to the "Inspect" tab or simply find any token applied.
Remove the token by pressing the "Remove selected" button or by pressing the token name.
If you are removing the color token, then see how the object disappears because of it's colour gets deleted totally. If you remove the typography token, then see nothing happens.
On the "Inspect" tab see the token still applied.
Press "Apply to selection" button.
If you have multiple objects selected, then see some of the tokens applied again. If you have a single object selected, then see the token applied again.

Expected behavior

Select any object in a Figma file with tokens applied.
Launch Tokens Studio for Figma.
Go to "Inspect" tab or simply find any token applied.
Remove token by pressing the "Remove selected" button or by pressing the token name.
See the selected tokens removed, but styles still applied and all of the objects remained visible.

Screenshots or Screencasts
Not applicable.

@kostyaflyphant I am unable to reproduce this. Is this issue still persisting in 1.38.2?

Now I'm also unable to reproduce this. The issue seems fixed.

I am closing this issue as it cannot be reproduced anymore.


Add a quiet option

This won't actually make initialising package.json quiet yet due to an npm issue:
https://github.com/npm/npm/issues/15425

Coverage increased (+0.2%) to 92.814% when pulling 949b6da7278d51a11407f63f3f1451703aaa0d33 on insin:quiet-option into 23af5551f35fdd22bae8afe45f716452361a168e on ericclemmons:master.

Coverage increased (+0.2%) to 92.814% when pulling 949b6da7278d51a11407f63f3f1451703aaa0d33 on insin:quiet-option into 23af5551f35fdd22bae8afe45f716452361a168e on ericclemmons:master.

Should we set stdio to "ignore" for now when running npm init -y when quiet is enabled?

Coverage increased (+0.2%) to 92.814% when pulling b454aaa4a740e4780b2c634c571672db4d87d933 on insin:quiet-option into 23af5551f35fdd22bae8afe45f716452361a168e on ericclemmons:master.

Example of output with this in place:

Coverage increased (+0.3%) to 92.857% when pulling 0f05e1170b20b66501c22432eb610ac024887d45 on insin:quiet-option into 23af5551f35fdd22bae8afe45f716452361a168e on ericclemmons:master.

Coverage increased (+0.3%) to 92.857% when pulling 0f05e1170b20b66501c22432eb610ac024887d45 on insin:quiet-option into 23af5551f35fdd22bae8afe45f716452361a168e on ericclemmons:master.

Coverage increased (+0.3%) to 92.857% when pulling 7ce725368e115c0a664df9362611fb5c5e4bcedf on insin:quiet-option into 23af5551f35fdd22bae8afe45f716452361a168e on ericclemmons:master.

One thing missing from this - what shold we do if we detect that npm install failed for whatever reason?


Prompt users to provide feedback

Given we want valuable feedback from users of the Nitric CLI, perhaps we could show a text output after the first use of nitric new (or another appropriate command?) which prompts with a message like this:
Thanks for trying nitric!

If you have feedback you can raise issues on GitHub https://github.com/nitrictech/nitric or come talk with us directly on Discord https://our.discord.link.example

Show it when the user runs Nitric run for the first time, ask me again later?

This is blocked until a discord invite link is created.


Pre-trained models

Hello, I have a question for your pre-trained models. Are these pre-trained models you have given the best result?

Hello! Limited by computing resources and time, there is a possibility of not getting the best trained model.

I got it, thank you!


Bitbox command line -r, --restURL tag says defaults to mainnet "rest" cloud. should be "trest"

bitbox-javascript-sdk/index.js
line 28)
.option(
"-r, --restURL ",
"The rest URL to use. default: https://rest.bitcoin.com/v1/" [SHOULD BE trest.bitcoin.com/v1/]

Thanks for pointing this out. It's been fixed in v2.2.1.


gats.io

Hi, have had a few requests for this site so can be logged into and used on our proxies. Per alexa https://www.alexa.com/siteinfo/gats.io it is a BIG site.
thanks

Too much scripting on that site. No web proxy will ever work perfectly with JavaScript heavy sites. For this, you should just use an actual HTTP proxy:
https://www.proxynova.com/proxy-software/

Not a problem, appreciated.


No Preferred Line Length

It took me a little while to realize this package was setting this attribute which overrides my editor defaults. This is very irritating and honestly seems like a flaw of Atom that this can even happen like this.

This is actually by intention since Rust's recommended style is to wrap at a line length of 99. We intentionally included this setting to encourage the use of this formatting convention. You can explicitly override it by setting a different line length in the settings for the language-rust package (see also #28).
Since you're not the first who likes to remove this setting, you might want to start an issue to discuss if this is actually a good idea or not. I'm still thinking that the default setting should reflect the Rust code formatting convention.


Client x.509 certificates possible?

Is it possible to present a client certificate to a server for authentication with nodemailer? I couldn't seem to find any documents on it.

Oh. it seems like this is probably setup in the TLS options Going to close this ticket.


Clean-up test and doctests datasets

I suggest we review and update our test datasets.  The NRCan datasets that we currently use will be unfamiliar to many users, and have a lot of grid points, which slows down testing.
I suggest we discuss what datasets should be included in the test, doctest and documentation.
For example, we could use recent CMIP6 simulations.

@sbiner I suggest we include some of the observational datasets that caused you issues.
@Zeitsperre We could create a xclim-data repository playing the same role as xarray-data, then add an xclim.tutorial wrapping xarray.tutorial.
@aulemahal Comments ?

We had the same discussion in #351, where we evaluated the intake option. While it did take care of the machinery part, it did not solve the storage.
xclim-data seems a good idea indeed. Also the code of  xarray.tutorial is super simple, it would be easy to adapt and that solution adds no dependency.
Are there any challenges to creating a data-only repo?  I think I remember from earlier discussions, that we didn't want to do that, but I have no memory why...

Are there any challenges to creating a data-only repo? I think I remember from earlier discussions, that we didn't want to do that, but I have no memory why...

Interestingly enough, the folks at roocs have done precisely that with our existing data via sub-modules: https://github.com/roocs/xclim-testdata/tree/master. When the source code for one of their projects is cloned, they also pull this testdata repo using git clone --recursive <link>. If anything I suggest we use this repo (and maybe add in some data license texts and Apache 2 License stating that this was forked from xclim).
Integrating this into xclim would be extremely simple:
cd tests/
git submodule add https://github.com/roocs/xclim-testdata/
git submodule init

+1 for using xclim-testdata, but -1 for the submodule approach. I would use the xarray.tutorial access mechanism to simplify access for users.

After today's discussions, we discussed the following partial fixes:
1 - Create a xclim-testdata github repo and use xr.tutorial.open_dataset to read data from it.
2 - Using the implementation suggested in #558, create a large dataset with a sample of all kinds of variables used in xclim. Cycle through all indicators (with the registry) and call them with the dataset (all other parameters default).
Simple as this, it would at least test that the computation works for most indices. Moreover, we will be able to test if attributes are carried on or not.
Bonus 1: A dictionary (json file?) could be created to store expected outputs of these simple calls.
Bonus 2: The same dict could list different combination of arguments (other than variables) to test a larger set. This would reduce the number of hand-written tests, but might also be less intuitive to work with...

xarray.tutorial access mechanism

@huard
Can I have more details?  Would like to plan https://github.com/Ouranosinc/raven/issues/185.

@tlvu xarray.tutorial.open_dataset is a method that retrieves a dataset (by name) from a git repository and stores it in a hidden folder on the local machine. It has the flexibility to check in the repository before hand, as not to redownload the same things more than once. It also performs a md5 checksum to ensure data integrity, but that's it, nothing much fancier.
Our goal is to put all test data in a Ouranosinc/xclim-testdata repo and pull it on demand when testing or making examples.

One step toward this would be to wrap xr.tutorial.open_dataset inside raven, so we have raven.tutorial.open_dataset that connects to raven-testdata. As Pascal explained, in the original, files are cached locally. For raven I think we need to be able to get a DAP URL that can be passed to remote services.

Steps:
[x] - Transfer test data to a seperate repo
[ ] - Replace heavy datasets by smaller ones and adapt tests.

[ ] Update doctests, tutorials and notebooks

@aulemahal We good to close this with #620 ?

@aulemahal We good to close this with #620 ?

Are the files referenced in conftest/add_example_file_paths taken from xclim-testdata ?

Are the files referenced in conftest/add_example_file_paths taken from xclim-testdata ?

@huard Yep. This trick relies on the fixture just above (add_imports) where xr is mapped to xclim.testing, so we can trick the reader when doing xr.open_dataset.
@Zeitsperre  We could, but there is the "Update docstests, tutorials and notebooks" check box that is unchecked. I think that to complete the task we want to use xclim-testdata everywhere needed in the notebooks (I'm not sure it is the case currently).

@huard Yep. This trick relies on the fixture just above (add_imports) where xr is mapped to xclim.testing, so we can trick the reader when doing xr.open_dataset.
@Zeitsperre  We could, but there is the "Update docstests, tutorials and notebooks" check box that is unchecked. I think that to complete the task we want to use xclim-testdata everywhere needed in the notebooks (I'm not sure it is the case currently).

@huard Yep. This trick relies on the fixture just above (add_imports) where xr is mapped to xclim.testing, so we can trick the reader when doing xr.open_dataset.

Yes, it's very sneaky at first but very obvious if you take a look at it. Very happy with the xdoctest namespace injection approach.

@Zeitsperre We could, but there is the "Update docstests, tutorials and notebooks" check box that is unchecked. I think that to complete the task we want to use xclim-testdata everywhere needed in the notebooks (I'm not sure it is the case currently).

For conftest/add_example_file_paths, everything but the .json files map to the right folder on xclim-testdata. I can fix this, run the doctests and notebooks and go from there.

@huard Yep. This trick relies on the fixture just above (add_imports) where xr is mapped to xclim.testing, so we can trick the reader when doing xr.open_dataset.

Yes, it's very sneaky at first but very obvious if you take a look at it. Very happy with the xdoctest namespace injection approach.

@Zeitsperre We could, but there is the "Update docstests, tutorials and notebooks" check box that is unchecked. I think that to complete the task we want to use xclim-testdata everywhere needed in the notebooks (I'm not sure it is the case currently).

For conftest/add_example_file_paths, everything but the .json files map to the right folder on xclim-testdata. I can fix this, run the doctests and notebooks and go from there.

Looks as though our atmos checks are scrutinizing our test data pretty deeply. This test file:
ns["path_to_tas_file"] = str(cmip3 / "tas.sresb1.giss_model_e_r.run1.atm.da.nc")

isn't passing our Validation checks any more:
xclim.core.utils.ValidationError: Variable has a non-conforming cell_methods. Got `time: mean`, expected `['*time: mean within days*']`

And our artificial data needs much more in terms of standards (cell_methods, standard_name, etc.). The artificial data is easy enough to fix, but since cmip3 data probably doesn't follow modern CF conventions, I may need to replace it in doctests. More to come.

Looks as though our atmos checks are scrutinizing our test data pretty deeply. This test file:
ns["path_to_tas_file"] = str(cmip3 / "tas.sresb1.giss_model_e_r.run1.atm.da.nc")

isn't passing our Validation checks any more:
xclim.core.utils.ValidationError: Variable has a non-conforming cell_methods. Got `time: mean`, expected `['*time: mean within days*']`

And our artificial data needs much more in terms of standards (cell_methods, standard_name, etc.). The artificial data is easy enough to fix, but since cmip3 data probably doesn't follow modern CF conventions, I may need to replace it in doctests. More to come.


Event request: justification and additional info

Warning about possibility of loosing text.

@shoaibsufi Where the warning should go?


envoy.features.fail_on_any_deprecated_feature Runtime Key Doesn't Prevent Envoy Startup

While upgrading to Envoy 1.18.2 I tried to use the new envoy.features.fail_on_any_deprecated_feature runtime key to prevent Envoy from starting up at all if one or more deprecated fields are found to be in use. However, even after adding this runtime key, Envoy continued to start up even when logging deprecation warnings - in my case due to having an access_log_path defined under admin, which is newly deprecated in 1.18. This was unexpected, since the Envoy 1.18.0 changelog includes this entry:
config: add envoy.features.fail_on_any_deprecated_feature runtime key, which matches the behaviour of compile-time flag ENVOY_DISABLE_DEPRECATED_FEATURES, i.e. use of deprecated fields will cause a crash.
To narrow down the problem and rule out any misconfiguration on my part, I attempted a much simpler test case by purely pulling down envoyproxy/envoy:v1.18.2 and making minor edits to its /etc/envoy/envoy.yaml, then trying to run envoy against this updated configuration to see if I could get envoy.features.fail_on_any_deprecated_feature to work in this context. I could not, so the simplified test case here is just to swap out the following lines in /etc/envoy/envoy.yaml...
admin:
  address:
    socket_address:
      protocol: TCP
      address: <IP_ADDRESS>
      port_value: 9901

...with this larger config that utilizes the now-deprecated access_log_path field and enables the envoy.features.fail_on_any_deprecated_feature runtime key:
admin:
  access_log_path: /dev/stdout
  address:
    socket_address:
      protocol: TCP
      address: <IP_ADDRESS>
      port_value: 9901
layered_runtime:
  layers:
  - name: static-layer
    static_layer:
      envoy.features.fail_on_any_deprecated_feature: true

To execute these steps directly against the envoy:v1.18.2 image we can use these commands (not passing "--bootstrap-version 3" prevents Envoy from spotting the deprecated fields at all):
docker pull envoyproxy/envoy:v1.18.2
docker exec --entrypoint /bin/bash -it envoyproxy/envoy:v1.18.2
  apt-get update && apt-get install -y vim.tiny # To help us make a quick edit for the test.
  vi /etc/envoy/envoy.yaml  # Make the edit defined above.
  envoy --bootstrap-version 3 -c /etc/envoy/envoy.yaml # Observe deprecation warnings, but Envoy starts up.

The actual deprecation warning logged is the one we'd expect if we'd never turned on envoy.features.fail_on_any_deprecated_feature at all:

[2021-05-02 17:27:39.585][385][warning][misc] [source/common/protobuf/message_validator_impl.cc:21] Deprecated field: type envoy.config.bootstrap.v3.Admin Using deprecated option 'envoy.config.bootstrap.v3.Admin.access_log_path' from file bootstrap.proto. This configuration will be removed from Envoy soon. Please see https://www.envoyproxy.io/docs/envoy/latest/version_history/version_history for details. If continued use of this field is absolutely necessary, see https://www.envoyproxy.io/docs/envoy/latest/configuration/operations/runtime#using-runtime-overrides-for-deprecated-features for how to apply a temporary and highly discouraged override.

Looking in PR history, it certainly appears that https://github.com/envoyproxy/envoy/pull/14749 introduced support for this new runtime key with the expected semantics, and this code is indeed present in 1.18.2, specifically at https://github.com/envoyproxy/envoy/blob/v1.18.2/source/common/protobuf/utility.cc#L210.
Is this expected behavior when the envoy.features.fail_on_any_deprecated_feature is set to true?

My suspicion is that unfortunately there is an init order issue and the deprecated field in question is actually parsed prior to runtime being initialized. This prevents the failure. This feature probably should have been a CLI option or we need to do some type of double pass over the bootstrap to look for deprecated fields and whether we need to fail to start. cc @htuch @alyssawilk who might have further ideas.

Yeah, I think this assessment is correct. We need to have already parsed the entire bootstrap and validated prior to the availability of bootstrap based runtime fields.


Nexrad fixes

Better test coverage with more test files
Implemented support for XDR parsing
Fix is_precip_mode()

Looks good to me - ship it!


lsp.update_ is deprecated

the lsp.update_ capabilities is deprecated , default_capabilities should be used instead.

Maybe had to before doing a pull request, but the error message   was already in an earlier pull request. But nothing changed since than. Unless you want to keep it in there as teachable moment?

Thanks :)


Typography font size increasing with changing device orientation

I am using cards to view a list of simple items which is working fine on all devices and browsers. However, on iOS whenever I place the web app to home screen using "Add to Home Screen" from Safari, the cards are getting zoomed in or font sizes are increasing somehow whenever I change the device orientation.

[ x] The issue is present in the latest release.
[ x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
Font sizes are increasing for the cards whenever I change the orientation from portrait to landscape and then again to portrait.

Expected Behavior 🤔
Font sizes should remain the same for the cards whenever I change the orientation from portrait to landscape and then again to portrait.

Steps to Reproduce 🕹

Steps:

Open Safari and go to https://eservices.baladia.gov.kw/
Add to Home Screen
Change device orientation from portrait to landscape and then again to portrait multiple times.
Notice the font size increasing on the cards list when changing the orientation from portrait to landscape and then again to portrait.

Context 🔦

Your Environment 🌎

Tech
Version

Material-UI
latest 5.xx and 4.xx

React
latest

Browser
latest

TypeScript
No

etc.

Never mind. It was a safari issue where I had to do the following:
html {   -webkit-text-size-adjust: 100%; }

@fadijabi We fixed it in #22089


firecracker: send logs to a vsock port

fixes #652

/test

folks, what do you think about this enhancement? can we merge this?
cc @jcvenegas @egernst @amshinde @jodh-intel @GabyCT

Hmm, that's distressing. I can't merge this due to the codecov failure.
@devimc - since it's only dropped a tiny amount, you could try adding a test to resolve this?


seems issue in j.me

these 2 keys should be different?
the verify key should be the one which is public and can go everwhere
check the j.myidentities.test()

fixed in commit: https://github.com/threefoldtech/jumpscaleX_core/commit/f4be728416645ba72320e1fe281e88d661ccb1da

verified.


Add "gaslimit" option to CLI (just like existing "gasprice")

Is your feature request related to a problem? Please describe.
Case:
I have an "OpenZeppelin Transparent Proxy" contract with upgradeAndCall method. Sometimes the upgrading is failing due to low estimated gas limit.
Describe the solution you'd like
I want to control it from CLI options.
Describe alternatives you've considered
To hardcode it to a deploy script, but it's inconvenient way I think because sometimes it's needed, sometimes not.

hmm, if it is in the cli, this would have to apply to every tx made during the execution of that script, would that not be weird ?

@wighawag How can I change gas Limit for a transaction that I'm trying to make with my contract? My transaction is failing with Out of gas error. I want to increase gas limit Unable to find any documentation for that?


[docs] Add AdaptingHook TypeScript demo

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

Added typescript version of styles/basics/AdaptingHook

Follow-up to #15546

@merceyz Much appreciated, thanks.


Add example for changing the debug output state.

Created this as I struggled to understand it myself. Since someone came along with the same question I had, I figure an example might help others too.
I had a version where the button handling was more complicated, with one system to handle all button clicks. That version didn't require a second parameter for .add_button, but then ends up kind of complicated because the only distinguishing element is the text itself. Even with a simplified version from my initial try, I feel like it adds more complication that isn't really relevant to the example. If you want, though, I could take another look at that.

Looks great, thanks!


Benchmarks.

How does this library compare to the original math.hs in MSVC, MinGW, and Linux CSTDLibs? It looks like you used many bit tricks and optimized this library extensively yourself, but I wonder if it's stopping compiler optimizations or if it can really replace native CSTD math.h libraries in places where performance matters.

Modern compilers are indeed has faster functions than my own functions. I agree with that. But it is just a fun project. Or at least if you're writing a kernel, you MUST use codes that written in zero like these instead of using system or compiler functions to create your kernel. Like fdlibm does.
So again, it is a fun project. And plus, it was written for École 42 school to show students that they also can create stuff like these.

That is cool, fun is all you can ask for. I still wonder if this library can actually stack up performance-wise, and if you should use platform specific instructions when they exist, or builtins for that matter. While it's true that you can't use stdlib's math.h for writing kernels, you would use a whole C STDLIB implementation instead of just specific parts like this math library, or just port glib.


vue add doesn't work with pnpm 6,report Error command failed:pnpm install --reporter silent --shamefully-hoist vue-cli-plugin-windicss -D

Version
4.5.13
Environment info
Environment Info:

  System:
    OS: macOS 11.3.1
    CPU: (12) x64 Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz
  Binaries:
    Node: 14.16.1 - ~/.nvm/versions/node/v14.16.1/bin/node
    Yarn: 1.22.10 - /usr/local/bin/yarn
    npm: 6.14.12 - ~/.nvm/versions/node/v14.16.1/bin/npm
  Browsers:
    Chrome: 90.0.4430.93
    Edge: 90.0.818.56
    Firefox: 87.0
    Safari: 14.1
  npmPackages:
    @vue/cli-plugin-babel: ~4.5.0 => 4.5.13
    @vue/cli-plugin-eslint: ~4.5.0 => 4.5.13
    @vue/cli-service: ~4.5.0 => 4.5.13
    eslint-plugin-vue: ^6.2.2 => 6.2.2
    vue: ^2.6.11 => 2.6.12
    vue-cli-plugin-windicss: ~0.2.0 => 0.2.0
    vue-eslint-parser:  7.6.0
    vue-template-compiler: ^2.6.11 => 2.6.12
  npmGlobalPackages:
    @vue/cli: 4.5.13

Steps to reproduce
1.vue create tailing
2、rm -fr node_module
3、pnpm i
4、vue add windicss
What is expected?
add windicss successfully
What is actually happening?
ERROR  command failed: pnpm install --reporter silent --shamefully-hoist vue-cli-plugin-windicss -D

I have similar error
Version
4.5.13
Environment info
Environment Info:

  System:
    OS: Linux 5.0 Ubuntu 19.04 (Disco Dingo)
    CPU: (4) x64 Intel(R) Core(TM) i7-4500U CPU @ 1.80GHz
  Binaries:
    Node: 14.17.3 - ~/.nvm/versions/node/v14.17.3/bin/node
    Yarn: 1.21.1 - /usr/bin/yarn
    npm: 7.21.1 - ~/.nvm/versions/node/v14.17.3/bin/npm
    pnpm: 6.14.6 - ~/.nvm/versions/node/v14.17.3/bin/pnpm
  Browsers:
    Chrome: 91.0.4472.114
    Firefox: 72.0.1
  npmPackages:
    @vitejs/plugin-vue: ^1.6.1 => 1.6.2
    @vue/compiler-sfc: ^3.2.6 => 3.2.11
    vue: ^3.2.6 => 3.2.11
  npmGlobalPackages:
    @vue/cli: 4.5.13

Steps to reproduce

vue add @vue/cli-plugin-eslint

What is expected?
add eslint plugin successfully
What is actually happening?
ERROR  command failed: pnpm install --reporter silent --shamefully-hoist<EMAIL_ADDRESS>-D
Try to run pnpm command manually
$ pnpm install --reporter --shamefully-hoist<EMAIL_ADDRESS>-D

ERROR This modules directory was created using a different public-hoist-pattern value.
Run "pnpm install" to recreate the modules directory.

$ pnpm i
Lockfile is up-to-date, resolution step is skipped
Already up-to-date

similar too.
Version
4.5.15
Environment Info:

  System:
    OS: Windows 10 10.0.22000
    CPU: (16) x64 Intel(R) Core(TM) i7-10700 CPU @ 2.90GHz
  Binaries:
    Node: 14.6.0 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.11 - C:\Program Files\nodejs\yarn.CMD
    npm: 6.14.6 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Chrome: Not Found
    Edge: Spartan (44.22000.120.0), Chromium (96.0.1054.34)
  npmPackages:
    @vue/cli-plugin-babel: ^3.8.0 => 3.12.1
    @vue/cli-plugin-eslint: ^3.8.0 => 3.12.1
    @vue/cli-service: ^3.8.0 => 3.12.1
    @vue/eslint-config-standard: ^4.0.0 => 4.0.0
    eslint-plugin-vue: ^5.0.0 => 5.2.3
    vue: ^2.6.10 => 2.6.14
    vue-cropper: ^0.4.9 => 0.4.9
    vue-eslint-parser:  5.0.0
    vue-router: ^3.0.3 => 3.5.3
    vue-template-compiler: ^2.6.10 => 2.6.14
    vuex: ^3.0.1 => 3.6.2
  npmGlobalPackages:
    @vue/cli: Not Found   // installed by pnpm, pnpm list -g show @vue/cli 4.5.15

Steps to reproduce
git clone project
pnpm i
vue upgrade

What is expected?
upgrade successfully.
What is actually happening?
got :
 ERROR  command failed: pnpm install --reporter silent --shamefully-hoist<EMAIL_ADDRESS>
Any additional comments? (optional)
add .npmrc file
shamefully-hoist=true

same as vue-cli v5
Upgrading @vue/cli-service from 4.5.15 to 5.0.0-rc.1
 ERROR  Error: command failed: pnpm install --reporter silent --shamefully-hoist<EMAIL_ADDRESS>Error: command failed: pnpm install --reporter silent --shamefully-hoist<EMAIL_ADDRESS>    at ChildProcess.<anonymous> (/Users/jianhuaren/.nvm/versions/node/v16.9.1/pnpm-global/5/node_modules/.pnpm/@vue+cli@5.0.0-rc.1/node_modules/@vue/cli/lib/util/executeCommand.js:138:16)
    at ChildProcess.emit (node:events:394:28)
    at maybeClose (node:internal/child_process:1064:16)
    at Process.ChildProcess._handle.onexit (node:internal/child_process:301:5)

any ideas why this does not work?

As a workaround I renamed node_modules/.modules.yaml to node_modules/.modules.yaml_bak
and then rerun the command
Related: https://github.com/pnpm/pnpm/issues/3600

This is still a problem

This is still a problem

package.json with pnpm is incompleted...

Same problem happened to me when I run vue create **project-name**
I switched from pnpm to npm by appending -m npm option

pnpm 7.0.0 Same problem
[I] ➜ vue create xxxxxx
Vue CLI v5.0.4
? Please pick a preset: Default ([Vue 2] babel, eslint)
Vue CLI v5.0.4
✨  Creating project in /home/hens/work-space/xxxxxxx.
🗃  Initializing git repository...
⚙  Installing CLI plugins. This might take a while...
ERROR  Error: command failed: pnpm install --reporter silent --shamefully-hoist
Error: command failed: pnpm install --reporter silent --shamefully-hoist
at ChildProcess. (/home/hens/.nvm/versions/node/v16.15.0/lib/node_modules/@vue/cli/lib/util/executeCommand.js:138:16)
at ChildProcess.emit (node:events:527:28)
at maybeClose (node:internal/child_process:1092:16)
at Process.ChildProcess._handle.onexit (node:internal/child_process:302:5)

vue create vue-demo -m npm

I set up a test environment in a container
node v16
Vue/cli 5.04
Using the PNPM 7.x version causes an error,
PNPM 6.32.14  can successfully create a project

I am having same issue with apple m1 macbook air, 8 gb ram

vue create PROJECT -m npm

Works fine

Really thanks https://github.com/vuejs/vue-cli/issues/6476#issuecomment-1137174677
Fixed via
npm i pnpm@6 -g
vue create PROJECT

Works great for me.

Any update for pnpm@7?

Really thanks #6476 (comment) Fixed via
$ npm i pnpm@6 -g
$ vue create PROJECT

Works great for me.

@quyctd

vue create vue-demo -m npm
good solution, also recommend

cd your_project_path
rm -rf .npmrc && rm -rf node_modules
pnpm install
pnpm add @vue/vue-cli-plugin-windicss -D
pnpm add -D yorkie webpack @babel/core @types/node @types/webpack-env

More:

https://github.com/vuejs/vue-cli/blob/b154dbd7aca4b4538e6c483b1d4b817499d7b8eb/packages/%40vue/cli/lib/Creator.js#L168-L176
https://github.com/vuejs/vue-cli/blob/b154dbd7aca4b4538e6c483b1d4b817499d7b8eb/packages/%40vue/cli/lib/util/ProjectPackageManager.js#L48-L59

I have vue-cli v5.0.8 and pnpm 7.8.0 and I can confirm this issue. I want to be able to use pnpm on my project because I have an old ADSL which speeds are concerning, and I was relived to see vue-cli has compatibility for pnpm. However it seems broken?
Error: command failed: pnpm install --reporter silent --shamefully-hoist
   at ChildProcess.<anonymous> (/usr/lib/node_modules/@vue/cli/lib/util/executeCommand.js:138:16)
   at ChildProcess.emit (node:events:513:28)
   at maybeClose (node:internal/child_process:1093:16)
   at Process.ChildProcess._handle.onexit (node:internal/child_process:302:5)

This issue was fixed in #7265 and should be closed.

This issue was fixed in #7265 and should be closed.

Same issue still occurs

Same issue still occurs

有人已经解决了这个问题，是因为pnpm v7默认的配置导致的，vue 官方人员已经merge该commit到dev分支了，应该会在下一个版本发布时修复

@linghengqian @cl1107
This error is still happening.
Vue CLI v5.0.8
pnpm 7.11.0

@linghengqian @cl1107
This error is still happening.
Vue CLI v5.0.8
pnpm 7.11.0

Make sure you are using the master branch, not 5.0.8.

@linghengqian @cl1107
This error is still happening.
Vue CLI v5.0.8
pnpm 7.11.0

Make sure you are using the dev branch, not 5.0.8.

That was fast. I assume this is because it isn't released yet?

@linghengqian @cl1107
This error is still happening.
Vue CLI v5.0.8
pnpm 7.11.0

Make sure you are using the dev branch, not 5.0.8.

That was fast. I assume this is because it isn't released yet?

It has not been published to npm yet.  You need to build the dev branch and hook into your local repo or private repo.

@linghengqian @cl1107
This error is still happening.
Vue CLI v5.0.8
pnpm 7.11.0

Make sure you are using the dev branch, not 5.0.8.

and how do you use dev branch?

@linghengqian @cl1107
This error is still happening.
Vue CLI v5.0.8
pnpm 7.11.0

Make sure you are using the dev branch, not 5.0.8.

That was fast. I assume this is because it isn't released yet?

It has not been published to npm yet. You need to build the dev branch and hook into your local repo or private repo.
same error in Vue CLI v5.0.8. How to hook into my repo?

@Benleie You need to complete yarn link according to https://github.com/vuejs/vue-cli/blob/dev/.github/CONTRIBUTING.md .


Health service

Use a health service that gets notified of health updates by the system
instead of querying the system constantly. This is useful because if a
user hasn't moved for awhile, we can cache the current steps, heart
rate, etc. instead of constantly asking the system.

Excellent! I will try to get this integrated when I have time!


Implement language popup

Do you want @wojpawlik to review this PQ or view the merged result?

@KnorpelSenf @wojpawlik prefers ready tables.


Why is the 'cnn_module_kernel' set to 15 for the U2 recipe but 8 for the U2++?

I am trying to reproduce the results of U2++. However, I'm confused about the cnn_module_kernel's default setting for U2++.
U2++:

U2:

The default setting is 8, while 15 for U2.
It will be appreciated if your could explain a little bit about your consideration in U2++.
Is that because U2++ finds small kernel size is beneficial to the performance or for any other reason?
Thanks a lot.

U2++ uses causal convolution, it only takes left context and current frame, so it's 7+1 = 8 in U2++.

Thanks for your timely response.
U2 also uses causal convolution as a streaming and non-streaming unified solution,  see the figure beblow.

Is there any problem for U2 with the setting of 15?

Sorry for that, maybe just because 8 get better result.


Hide the icon

Discussed in https://github.com/HandyOrg/HandyControl/discussions/779

Originally posted by AlphaNecron April 12, 2021
How do i hide the app's icon in the titlebar?

You can put this in your .csproj :
favicon.ico
Your project will have an Icon in the taskbar and so, you don't need to declare it in


Should have a flake for NixOS

This way it's easy for NixOS users to use this.

PACKAGING.md might be helpful for anyone who wants to make packages for distributions besides Arch Linux

Looks like hyprshade is now in nixos-unstable, so a flake isn't needed.

Alright; in that case, I'll close the issue for now.


Writing Calorie data to Google Fit using googleapis package (Fitness API) with Flutter

Is there a way to write calorie as a Session to Google Fit using Fitness API with Flutter? I tried Google API package for that but the documentation is for native android app and not Flutter. Is there an example for that in Flutter?

We only have wrappers around the existing API.
Look here https://developers.google.com/fit/scenarios/add-nutrition-data#rest or similar

I already checked all documentation but it's not very clear how to add data to Google Fit. For example after creating a data source and adding data points to it, I couldn't find a way in Flutter to add the datasource to a session in Google Fit app.

Sadly, I don't know either. We just expose the APIs as-is!
On Sat, Dec 18, 2021 at 2:16 PM Adeebullah Adab @.***>
wrote:

I already checked all documentation but it's not very clear how to add
data to Google Fit. For example after creating a data source and adding
data points to it, I couldn't find a way in Flutter to add the datasource
to a session in Google Fit app.
—
Reply to this email directly, view it on GitHub
https://github.com/google/googleapis.dart/issues/370#issuecomment-997292194,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAEFCVQNTFNC5BDTS5HTQ3URUB2HANCNFSM5KLDZNJA
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

Ok, thanks


Request for maintainer rights

This looks like an awesome lib that I'd love to use in one of my projects. However, there are a few things I'd like to see changed:

Port to TypeScript
Add Deno support
Add test coverage analysis tools, and property-based testing
Add performance benchmarking
See #4 merged
Add support for custom constructors so that library users are able to pass their own object constructors along with deconstruct and reconstruct functions
Experiment with storing object shapes in order to further reduce the size of the serisalised data (I have a use case where I serialise 10 almost identical objects but one small string deep inside changes, it would be great (but probably very hard) to reduce the redundancy here)

However, it looks like this project is no longer a priority for you. Are you willing to transfer this project to me so that I can take over and perform the changes I'd like to see?
If you disagree with the direction that I'd like to see this project take, that's fine. I can rewrite this in TS under a different name and credit your project for the excellent ideas it has.
I hope you have time to quickly respond. If I don't hear back from you, I'll just accept that this thing died and go my own way :)

I have now implemented my own library from scratch. It uses a number of ideas from here, but also applies almost all of the above suggestions. Property-based testing and object shapes are missing so far, but everything seems to work well.
I also need to write a better README, it does not really explain anything yet.
In case anyone wants to try it, the repo is here: https://github.com/KnorpelSenf/oson
I will leave this issue open in case anyone is looking for a maintained version of this library.


⚠ Aither has degraded performance

In 9d35976, Aither ($AIH) experienced degraded performance:

HTTP code: 200
Response time: 1408 ms

Resolved: Aither performance has improved in 57bee62 after 7 minutes.


[question] how to turn off key-binding suggestion?

After pressing prefix keys like C-x or C-h and stay still for seconds, then auto-complete(?) list of key-bindings are displayed on the bottom command line buffer (?) like below:

But in my machine, it tooks 2~3 secs to show the list and it freezes emacs meanwhile.
a) How can I disable this suggestion feature?
b) Or how can I feature out which operation is freezing emacs?
Thanks in adavance!

Hello. I believe that this behavior comes from which key, which was introduced with PR #168 . You can disable it by setting exordium-help-extensions to nil in your before-init.el file. Let me or @pkryger know if you have questions about it.

@philippe-grenet  Thanks a lot for fast reply! I adjusted which key setting using its document: https://github.com/justbur/emacs-which-key#manual-activation


Provide reasons using lambda

Sometimes it would be handy to compose the reason message just in case of a failing assertion.
For example currently, it seems not to be possible to get a corresponding behaviour to this obviously flawed code:
customer.Should().NotBeNull("there is no matching customer, but found customer \"{0}\".", customer.FullName);

Suggestion (overload taking Func<T, string>):
customer.Should().NotBeNull(c => string.Format("there is no matching customer, but found customer \"{0}\".", c.FullName));

Although a nice idea, I think the amount of work will be huge.


Make travis faster

This makes deployment Travis faster
Also address some tests errors when controllers are in HA giving back 409s conflict errors on entities

Closes #651


feat: policy optimizer

This PR introduces a new cli tool called policy-optimizer. This binary will download the policies and optimize them. The goal is to implement what is described inside of this RFC.
Currently the code doesn't do any download/optimization. It just leverages the Lease primitive declared by Kubernetes to ensure only one process can have write access to the directory where the optimized policies are going to be written.
The actual code similates the download & optimize work with a simple sleep.
The change is pretty invasive as you can see, I don't want to merge it in the main branch yet.

Closing in favor of https://github.com/kubewarden/policy-server/pull/519, which is open against a feature branch of policy-server


CURL parsing is not working as expected

Expected Behavior
This curl should be parsed correctly same as Postman
curl --location 'https://example.com' \
--header 'Content-Type: application/json' \
--header 'Authorization: Bearer xxxx' \
--data '{
    "test_prop": true
}'

With the headers in the Curl being respected, the request body, and method should be POST
Actual Behavior
Parsed insomnia request has the following issues:

Method is GET not POST
No request body
No headers

Reproduction Steps

Add a new request using "From Curl"

Use this

curl --location 'https://example.com' \
--header 'Content-Type: application/json' \
--header 'Authorization: Bearer xxxx' \
--data '{
    "test_prop": true
}'

Notice that parsed insomnia request has the following issues:

Method is GET not POST
No request body
No headers

Is there an existing issue for this?

[X] I have searched the issue tracker for this problem.

Additional Information
If I change the Curl to this
curl --request POST \
  --url https://example.com \
  --header 'Authorization: Bearer xxxx' \
  --header 'Content-Type: application/json' \
  --data '{
	"test_prop": true
}'

It works as expected now, but I think it would be much better if the parsing was as robust as Postman
Insomnia Version
8.3.0
What operating system are you using?
macOS
Operating System Version
Darwin arm64 23.0.0
Installation method
https://insomnia.rest
Last Known Working Insomnia version
No response

This is a dupe of https://github.com/Kong/insomnia/issues/6755
and there's a PR for that here https://github.com/Kong/insomnia/pull/6774
HTH

Working well with the latest version, thanks


Carthage support

Could Carthage support be added?

I will added tomorrow. :)

Done. see 1.2.0 version. Thanks :)

Thanks for the fast response!

:)


Added Opposite Number challenge to CodeWars

Hey thanks for your contribution ❗. Keep up the good job and leave a star if you liked the repo .
Happy hacking 😉


[SMALLFIX] Replace compare with integer.compare in alluxio.master.meta.AlluxioMasterRestServiceHandler#getTierAliasComparator #310

Replace compare with integer.compare in alluxio.master.meta.AlluxioMasterRestServiceHandler#getTierAliasComparator #310

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/574/
Test PASSed.


Simplify removeAttachedFiles function

Coverage remained the same at 81.661% when pulling 75f3ed79817c14849a690c8149f9c998003aec7e on SimePel:master into 5732ff5e4af905696e6d028da76559b76b9973f9 on hhrutter:master.

Coverage remained the same at 81.661% when pulling 75f3ed79817c14849a690c8149f9c998003aec7e on SimePel:master into 5732ff5e4af905696e6d028da76559b76b9973f9 on hhrutter:master.

I understood. I will open this PR later.

Thanks 💚


can't send ADA back to Exchange!!!

I have sent ADA to Binance exchange but nothing happen, There was " Transaction submitted and then no any transaction ID or anything showing in history. The full quantity of ADA still showing left there in Nami wallet. I've been trying for several time and waiting for hours!.  What should i do to transfer ADA out of Nami plss?
Thank you.

Got it done!! ..just have to be patient and try again many times and took times few hours to wait for.


TypeError: reader.skipField is not a function

Since 3.21.1 I receive
TypeError: reader.skipField is not a function
on
deserializeBinary(data).toObject()
with 3.21.0 it is working
Node 16.17.1

Same with
reader.readUint64String is not a function
Working with 3.21.0

This definitely looks like the closure optimizations taking out too much. In the deeper details, all of our tests pass closure deps side, but it looks like our jasmine config for commonjs doesn't include most of our binary/ tests.

I just published a hotfix release (3.21.2) to npm and will close this out now.  Please re-open and ping me if the problem persists.


Fix #10: Rename ri to sample-implementation

@Emily-Jiang maybe we can this sorted before the call

don't forget to git clean -fd your local repo after this change has been merged


Added current document should be shown in the accordion [#170250414]

Listens for primary and comparison document changes and auto expands the accordion menu for the section that the document exists in and then scrolls to the item in the list.
NOTE: this uses React hooks as the DocumentSection component is a functional component.

@kswenson @DALove1025 I figured out would both want to look at this as it contains the first usage of React hooks in CLUE.

I figured out would both want to look at this as it contains the first usage of React hooks in CLUE.

Actually, we're already using hooks in some of the Rete controls implementation because the Rete controls are functional components also. We've even got our own custom hooks! 😉

@kswenson I've updated the PR based on your recommendation and I've made the scroll calculation more explicitly use the list parent container.


fogify.deploy()

with such errors:
gaierror                                  Traceback (most recent call last)
/opt/conda/lib/python3.7/site-packages/urllib3/connection.py in _new_conn(self)
159             conn = connection.create_connection(
--> 160                 (self._dns_host, self.port), self.timeout, **extra_kw
161             )
/opt/conda/lib/python3.7/site-packages/urllib3/util/connection.py in create_connection(address, timeout, source_address, socket_options)
60
---> 61     for res in socket.getaddrinfo(host, port, family, socket.SOCK_STREAM):
62         af, socktype, proto, canonname, sa = res
/opt/conda/lib/python3.7/socket.py in getaddrinfo(host, port, family, type, proto, flags)
751     addrlist = []
--> 752     for res in _socket.getaddrinfo(host, port, family, type, proto, flags):
753         af, socktype, proto, canonname, sa = res
gaierror: [Errno -2] Name or service not known
During handling of the above exception, another exception occurred:
NewConnectionError                        Traceback (most recent call last)
/opt/conda/lib/python3.7/site-packages/urllib3/connectionpool.py in urlopen(self, method, url, body, headers, retries, redirect, assert_same_host, timeout, pool_timeout, release_conn, chunked, body_pos, **response_kw)
676                 headers=headers,
--> 677                 chunked=chunked,
678             )
/opt/conda/lib/python3.7/site-packages/urllib3/connectionpool.py in _make_request(self, conn, method, url, timeout, chunked, **httplib_request_kw)
391         else:
--> 392             conn.request(method, url, **httplib_request_kw)
393
/opt/conda/lib/python3.7/http/client.py in request(self, method, url, body, headers, encode_chunked)
1251         """Send a complete request to the server."""
-> 1252         self._send_request(method, url, body, headers, encode_chunked)
1253
/opt/conda/lib/python3.7/http/client.py in _send_request(self, method, url, body, headers, encode_chunked)
1297             body = _encode(body, 'body')
-> 1298         self.endheaders(body, encode_chunked=encode_chunked)
1299
/opt/conda/lib/python3.7/http/client.py in endheaders(self, message_body, encode_chunked)
1246             raise CannotSendHeader()
-> 1247         self._send_output(message_body, encode_chunked=encode_chunked)
1248
/opt/conda/lib/python3.7/http/client.py in _send_output(self, message_body, encode_chunked)
1025         del self._buffer[:]
-> 1026         self.send(msg)
1027
/opt/conda/lib/python3.7/http/client.py in send(self, data)
965             if self.auto_open:
--> 966                 self.connect()
967             else:
/opt/conda/lib/python3.7/site-packages/urllib3/connection.py in connect(self)
186     def connect(self):
--> 187         conn = self._new_conn()
188         self._prepare_conn(conn)
/opt/conda/lib/python3.7/site-packages/urllib3/connection.py in _new_conn(self)
171             raise NewConnectionError(
--> 172                 self, "Failed to establish a new connection: %s" % e
173             )
NewConnectionError: <urllib3.connection.HTTPConnection object at 0x7fbc13636f50>: Failed to establish a new connection: [Errno -2] Name or service not known
During handling of the above exception, another exception occurred:
MaxRetryError                             Traceback (most recent call last)
/opt/conda/lib/python3.7/site-packages/requests/adapters.py in send(self, request, stream, timeout, verify, cert, proxies)
448                     retries=self.max_retries,
--> 449                     timeout=timeout
450                 )
/opt/conda/lib/python3.7/site-packages/urllib3/connectionpool.py in urlopen(self, method, url, body, headers, retries, redirect, assert_same_host, timeout, pool_timeout, release_conn, chunked, body_pos, **response_kw)
724             retries = retries.increment(
--> 725                 method, url, error=e, _pool=self, _stacktrace=sys.exc_info()[2]
726             )
/opt/conda/lib/python3.7/site-packages/urllib3/util/retry.py in increment(self, method, url, response, error, _pool, _stacktrace)
438         if new_retry.is_exhausted():
--> 439             raise MaxRetryError(_pool, url, error or ResponseError(cause))
440
MaxRetryError: HTTPConnectionPool(host='controller', port=5000): Max retries exceeded with url: /annotations/ (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x7fbc13636f50>: Failed to establish a new connection: [Errno -2] Name or service not known'))
During handling of the above exception, another exception occurred:
ConnectionError                           Traceback (most recent call last)
 in
----> 1 fogify.deploy()
/opt/conda/lib/python3.7/site-packages/FogifySDK/FogifySDK.py in deploy(self, timeout)
84     def deploy(self, timeout=120):
85         url = self.get_url("/topology/")
---> 86         self.clean_metrics()
87         self.clean_annotations()
88         response = requests.post(url, files={"file": self.upload_file}, headers={}).json()
/opt/conda/lib/python3.7/site-packages/FogifySDK/FogifySDK.py in clean_metrics(self)
188
189     def clean_metrics(self):
--> 190         self.clean_annotations()
191         if hasattr(self, 'data'):
192             del self.data
/opt/conda/lib/python3.7/site-packages/FogifySDK/FogifySDK.py in clean_annotations(self)
435
436     def clean_annotations(self):
--> 437         return requests.delete(self.get_url("/annotations/")).json()
438
439     def get_annotations(self):
/opt/conda/lib/python3.7/site-packages/requests/api.py in delete(url, **kwargs)
159     """
160
--> 161     return request('delete', url, **kwargs)
/opt/conda/lib/python3.7/site-packages/requests/api.py in request(method, url, **kwargs)
59     # cases, and look like a memory leak in others.
60     with sessions.Session() as session:
---> 61         return session.request(method=method, url=url, **kwargs)
62
63
/opt/conda/lib/python3.7/site-packages/requests/sessions.py in request(self, method, url, params, data, headers, cookies, files, auth, timeout, allow_redirects, proxies, hooks, stream, verify, cert, json)
528         }
529         send_kwargs.update(settings)
--> 530         resp = self.send(prep, **send_kwargs)
531
532         return resp
/opt/conda/lib/python3.7/site-packages/requests/sessions.py in send(self, request, **kwargs)
641
642         # Send the request
--> 643         r = adapter.send(request, **kwargs)
644
645         # Total elapsed time of the request (approximately)
/opt/conda/lib/python3.7/site-packages/requests/adapters.py in send(self, request, stream, timeout, verify, cert, proxies)
514                 raise SSLError(e, request=request)
515
--> 516             raise ConnectionError(e, request=request)
517
518         except ClosedPoolError as e:
ConnectionError: HTTPConnectionPool(host='controller', port=5000): Max retries exceeded with url: /annotations/ (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x7fbc13636f50>: Failed to establish a new connection: [Errno -2] Name or service not known'))

It is a similar issue like #1. Please, check my advise at https://github.com/UCY-LINC-LAB/fogify-demo/issues/1#issuecomment-927178342
Thank you


Decide which terminals run in admin mode

Would be grand if we could determine which terminal tabs ran in elevated admin mode. Or is it that the whole WT process itself is the admin elevation? At least to clarify that part would be acceptable.
Might also be awesome to allow for command line configuration, i.e. run a Visual Studio command prompt from within Windows Terminal, i.e. replete with CLI configuration, parameters, etc, i.e. running the VS variables scripts and so forth. Thanks!

/dup #5000 :smile:


Clicks browser console view throws Cannot read property '_id' of undefined

I clone https://github.com/pipefy/doc-api-general and run $ apiary preview --output="index.html". Then I tried to click console view. It gave me this error. I can duplicate this error in my own repo too.
Uncaught TypeError: Cannot read property '_id' of undefined
    at a.initialize (docs-e7b71b96.js:1)
    at a.c (docs-e7b71b96.js:1)
    at a (docs-e7b71b96.js:1)
    at new a (docs-e7b71b96.js:1)
    at a.renderErrorView (docs-e7b71b96.js:1)
    at a.render (docs-e7b71b96.js:1)
    at a.renderConsoleView (docs-e7b71b96.js:1)
    at a.render (docs-e7b71b96.js:1)
    at a.initConsoleView (docs-e7b71b96.js:1)
    at a.renderConsoleView (docs-e7b71b96.js:1)

dup of https://github.com/apiaryio/apiary-client/issues/175


Implement unnecessary-literal-within-dict-call (C418)

Closes #3964.

Got to re-generate the schema :)


Using git in Ubuntu Bash does not Sync with Native Windows git

When executing git status inside Ubuntu Bash on Windows, it lists many files incorrectly. A git pull on an up-to-date repo can fail with collisions.
To reproduce:

Using a native windows git (e.g. in GitBash):

$ git clone https://github.com/Microsoft/BashOnWindows.git
Cloning into 'BashOnWindows'...
remote: Counting objects: 109, done.
remote: Total 109 (delta 0), reused 0 (delta 0), pack-reused 1Recei09
Receiving objects: 100% (109/109), 18.02 KiB | 0 bytes/s, done.
Resolving deltas: 100% (35/35), done.
Checking connectivity... done.

Guyon@Toruk MINGW64 /e/dev/git
$ cd BashOnWindows/

Guyon@Toruk MINGW64 /e/dev/git/BashOnWindows (master)
$ git status
On branch master
Your branch is up-to-date with 'origin/master'.
nothing to commit, working directory clean

Start Ubuntu Bash on Windows and navigate to the repo directory

guyon@TORUK:/mnt/e/dev/git/BashOnWindows$ git status
On branch master
Your branch is up-to-date with 'origin/master'.

Changes not staged for commit:
  (use "git add <file>..." to update what will be committed)
  (use "git checkout -- <file>..." to discard changes in working directory)

        modified:   CONTRIBUTING.md
        modified:   ISSUE_TEMPLATE.md
        modified:   README.md

no changes added to commit (use "git add" and/or "git commit -a")

Expected Result
There should be no modified files
Actual Result
There are modified files
My Windows Build Number: Version 1607 (OS Build 14393.447)

Hi @guyonroche -- if you type git config core.autocrlf at the two different git prompts, do you get the same value at each, or is it different?
Windows and Linux represent newlines in text files differently.  So, text files checked out on Windows will, by default, deliberately look different on disk than text files checked out on Linux.  My guess is that what you're seeing here is Linux-git reporting that the files have been modified.  It doesn't know that you have Windows-git installed and that Windows-git made this change.
If you have a single checkout that you want to share between Windows and Linux, my recommendation would be that you reinstall Windows git and, when prompted during installation, tell it to not modify newlines.  (The default is to automatically convert them on checkout and convert them back on checkin.)  Most Windows programs (with the notable exception of notepad.exe) can handle Linux newlines just fine, so this shouldn't affect your workflow.

Maybe also has to do with file modes?

Interesting - in Gitbash I get an output of "true". In Ubuntu bash I get no output

@Herst - file modes looks likely...
In Gitbash:
$ ls -al package.json
-rw-r--r-- 1 guyon<PHONE_NUMBER> Nov  3 07:48 package.json

In Ubuntu bash (same file):
$ ls -al package.json
-rwxrwxrwx 1 root root 1632 Nov  3 07:48 package.json

Are there any workarounds for this?

The core.fileMode setting of git (run git config core.fileMode false).

Found it - I ran:
git config core.autocrlf true

in the Ubuntu bash. I guess no setting at all defaults to false. Now it ignores the \r\n differences

Sorry for bumping this thread. But just a small note, you could do
git config --global core.autocrlf true on ubuntu bash. This way you don't have to do this for every local repo.
Let me know if anyone finds a problem doing this, that is if it breaks anything. Thanks


Executing Stored Procedure in SQL Server without a full accounting of Errors and Queries

Environment
To diagnose, we usually need to know the following, including version numbers.  On Windows, be
sure to specify 32-bit Python or 64-bit:

Python: 3.9.2 (64 bit)
pyodbc: 4.0.31
OS: Microsoft
DB: MS SQL Server 2019
driver: ODBC Driver 17 for SQL Server

Issue
I have the following stored procedure which I am trying to execute.  The commented 'SELECT 0;' is somewhat important as adding or removing it has what I think is a relevant change to the output but I wanted to leave it out initially.
Stored Procedure

CREATE PROCEDURE [dbo].[MultipleReturnMultipleError]
AS
BEGIN

    SET XACT_ABORT OFF;

    --SELECT 0;

    SELECT 1/0;

    SELECT 1;
    SELECT 2;
    SELECT 3;
    SELECT 1/0;

    SELECT 4;
    SELECT 1/0;

    SELECT 5;
    SELECT 1/0;

    SELECT 6;

END;

Procedure Call in Python

import os
import pyodbc

from dotenv import load_dotenv

from sqlalchemy import create_engine
from sqlalchemy.orm import sessionmaker

def test_handle_multiple_errors():

    DATABASE = os.getenv('METADATA_DATABASE_NAME')
    ENGINE = create_engine(CONNECTION.format(
        SERVER, DATABASE, DRIVER
    ), encoding='utf8')
    raw_connection = ENGINE.raw_connection()

    cursor = raw_connection.cursor()
    sql = "EXECUTE [DW_Staging].[dbo].[MultipleReturnMultipleError];"

    try:
        cursor.execute(sql)
        print("RESULT")
        result = cursor.fetchone()
        print(result)
    except Exception as ex:
        print("\n\nINITIAL EXECUTION ERROR")
        print(ex.args)

    next_set = True
    while next_set:

        try:
            print("\nMOVE TO NEXT RESULT")
            next_set = cursor.nextset()
            print("RESULT")
            result = cursor.fetchone()
            print(result)
        except Exception as ex:
            print("\n\nSUBSEQUENT ERROR")
            print(ex.args)

This has the output

INITIAL EXECUTION ERROR
('22012', '[22012] [Microsoft][SQL Server Native Client 11.0][SQL Server]Divide by zero error encountered. (8134) (SQLExecDirectW)')

MOVE TO NEXT RESULT
RESULT
(1, )

MOVE TO NEXT RESULT
RESULT
(2, )

MOVE TO NEXT RESULT
RESULT
(3, )

MOVE TO NEXT RESULT

SUBSEQUENT ERROR
('22012', '[22012] [Microsoft][SQL Server Native Client 11.0][SQL Server]Divide by zero error encountered. (8134) (SQLMoreResults)')

MOVE TO NEXT RESULT
RESULT

SUBSEQUENT ERROR
('No results.  Previous SQL was not a query.',)

I expected the 'nextstep' function to continue moving through subsequent items with output for the SELECT statements and errors down until 'SELECT 6' where it would exit.  Unfortunately this is not the case.  I believe it may be associated with the 'SQLMoreResults' error that comes with the second Divide by Zero error.
If the original 'SELECT 0' is uncommented and the procedure updated then even less information is returned as shown in the output below.
RESULT
(0, )

MOVE TO NEXT RESULT

SUBSEQUENT ERROR
('22012', '[22012] [Microsoft][SQL Server Native Client 11.0][SQL Server]Divide by zero error encountered. (8134) (SQLMoreResults)')

MOVE TO NEXT RESULT
RESULT

SUBSEQUENT ERROR
('No results.  Previous SQL was not a query.',)

It appears that there is a difference in an immediate execution error and an error thrown after a successful SELECT statement.  I'm not sure if this is relevant to the underlying issue.
I am not sure why the SQLMoreResults error is thrown on the first encounter here however it exits immediately without moving on to the 'SELECT 1;', 'SELECT 2;' and then exiting on the second encounter of Divide by Zero as it did previously.
Essentially I am looking for the full output of all steps both successful statements and errors so if multiple errors are encountered then they can all be logged at the Python level rather than needing to move between Python and SQL for debugging.
I think 'SET NOCOUNT ON;' has been suggested for similar errors which I have tried with no success.

Could you post an ODBC trace of the two cases?

ODBC Trace Attached.
First File
initial_error.log
Contains trace for the first instance where we hit the Divide by Zero immediately.
Second File
initial_success.log
Contains trace where we initially receive the 'SELECT 0;'

This issue happens because inside cursor.nextset() there is a call to SQLMoreResults and if it returns an error, pyODBC will free the statement handle and getting more results becomes impossible. The problem is that there is no obvious way to tell if SQLMoreResults returned an error because it itself failed (in which case the statement should be freed), or because the result that it got contains an error (in which case it shouldn't).
I'm going to try to think of a solution to his, but if anyone else has suggestions feel free to share.

I am able to obtain the output when running SQLCMD on the same procedure (same driver, SQL Version etc.) so have attached trace of this executing the 'initial error' stored procedure.
Not sure if it is of use at all but it is as close as I have to a suggestion.  Possibly a starting point though...
sqlcmd_initial_error.log

sqlcmd is designed specifically for msodbcsql so it can make assumptions on how SQLMoreResults behaves; not so with pyODBC which is generic and has to work with other ODBC drivers too.

try this, it's built on top of ideas from @gordthompson

def sql_execute_pyodbc(self, sql_script, as_pdf=False):
    pyodbc.pooling = False
    connection_url = sqlalchemy.engine.URL.create(
        "mssql+pyodbc",
        username = self.login_user,
        password = self.login_pass,
        host = self.login_server,
        database = self.login_db,
        query={
            "driver": "ODBC Driver 17 for SQL Server",
            "autocommit": "True",
        },
    )
    engine = sqlalchemy.create_engine(connection_url).raw_connection()
    if sqlalchemy.__version__>='2.0.0': sql_script = sqlalchemy.text(sql_script)
    sql_cols, sql_result =[], []
    with engine.cursor() as cursor:
      cursor.execute(sql_script)
      if cursor.description and cursor.rowcount:
          sql_cols, sql_result = [col[0] for col in cursor.description], cursor.fetchall()
      while cursor.nextset(): pass
      if len(sql_result)>0:
        result = pandas.DataFrame(np.array(sql_result), columns=sql_cols)
      else:
        result = pandas.DataFrame({"sql_script":[sql_script],"result":["True"]})
    return result if as_pdf else spark.createDataFrame(result)

How bad would it be to assume the SQLMoreResults call did not fail?  What if we assume it indicates that there is an error to retrieve?  In the super rare case that is wrong, the next API call should fail also.
Is this a driver bug?  Should it really be returning an error status?  Not that it helps any.

I think my suggestion above might be the way to go:
https://learn.microsoft.com/en-us/sql/odbc/reference/develop-app/multiple-results?view=sql-server-ver16

If one of the statements in a batch fails, SQLMoreResults will return either SQL_ERROR or SQL_SUCCESS_WITH_INFO. If the batch was aborted when the statement failed or the failed statement was the last statement in the batch, SQLMoreResults will return SQL_ERROR. If the batch was not aborted when the statement failed and the failed statement was not the last statement in the batch, SQLMoreResults will return SQL_SUCCESS_WITH_INFO. SQL_SUCCESS_WITH_INFO indicates that at least one result set or count was generated and that the batch was not aborted.

Since it is documented to return SQL_ERROR, do not assume the call itself failed.


Add jdk8-with-graal build for Windows

I'd love to see GraalVM-enabled builds on Windows. Ideally, I'd go for JDK8 on Windows 64-bit.
I don't know any details, but there might be a general compatibility issue, as even Oracle does not provide such a build (see bottom of https://www.oracle.com/technetwork/oracle-labs/program-languages/downloads/index.html). We might expect limitations to the Windows versions similar to the ones mentioned on this page.
Might relate to #369

@neomatrix369 - I think you have some Graal builds working already correct?  Would be good to integrate them into our pipelines and see what other platforms we can add.

@neomatrix369 What do you mean by "graal compiler"? Unfortunately, the Graal project is a bit inconsistent in its naming.
To my understanding, the main parts of GraalVM are:

Graal(VM): JIT-Compiler + required VM infrastructure
Polyglot / Truffle: Runtime for language interoperability and interpreters (effectively Java libraries)
Interpreters for several languages on top of Truffle (Graal.js, FastR, TruffleRuby, Python; effectively also Java libraries)
SubstrateVM: Ahead-of-time compiler
Sulong: LLVM bitcode interpreter

(See also https://www.graalvm.org/docs/reference-manual/glossary/)
If I understand the comments on the Oracle download page correctly, they exclude SubstrateVM (and possibly Sulong) from their Windows build; I'd expect the same limitations apply to this endeavor here.
This leaves us with Graal(VM) (which you mentioned to build already), and Polyglot / Truffle + interpreters. The latter are "only" Java libraries, so building seems pretty straightforward (although I never tried it).
In this sense, I don't see that much missing compared to the Oracle preview.


Adding a JSON renderer for PHPMD.

XML is a common output format, especially for Java. But in order to make this easy to use inside other PHP applications (like one I'm building), JSON is really simple.
This renderer simply compiles all the violations and errors into an array, and then JSON serializes them into something that can be output and used. I wanted to contribute it back since I found it useful, and I bet others would as well.

Closed without any comments? Why may I ask? this is actually quite useful.

@brandonsavage just came across this old issue. Just wanted to let you know that PHPMD will have a JSON renderer in the upcoming version 2.7.0.


Unable to compare DATETIME and TIMESTAMP[BUG]

Describe the bug
Unable to compare DATETIME and TIMESTAMP for logical operators and CASE.
Or for query issues:'

Run the query 'SELECT TIMESTAMP(DATE("2010-04-12")) = SUBDATE(TIMESTAMP(DATE("2010-04-12")), INTERVAL 1 HOUR)'
See error "ExpressionEvaluationException"

Expected behavior
The SQL plugin should be able to compare DATETIME and TIMESTAMP or SUBDATE and ADDDATE should have output of type TIMESTAMP.
Screenshots

The root cause seems implicit conversion between DATETIME and TIMESTAMP is not supported. Code: https://github.com/opensearch-project/sql/blob/main/core/src/main/java/org/opensearch/sql/data/type/ExprCoreType.java#L58


Raise Error when data would be ignored.

A fix for ingest-central#35
Not included the idea of warnings yet, I think that might need to wait until after the UI can show many more errors.
This change will raise an error if:

the concrete type of the tab is not included in the schema list
or the module type of the tab is not within the properties of the concrete schema
Any data is not converted to JSON, usually because the column name does not match the module name.

Codecov Report

Merging #52 into master will increase coverage by 0.69%.
The diff coverage is 89.88%.

@@            Coverage Diff             @@
##           master      #52      +/-   ##
==========================================
+ Coverage   74.03%   74.73%   +0.69%
==========================================
  Files          37       37
  Lines        3131     3249     +118
==========================================
+ Hits         2318     2428     +110
- Misses        813      821       +8

Impacted Files
Coverage Δ

ingest/importer/conversion/metadata_entity.py
100% <100%> (ø)
:arrow_up:

ingest/template/schema_template.py
83.05% <100%> (ø)
:arrow_up:

ingest/importer/spreadsheet/ingest_workbook.py
78.57% <50%> (ø)
:arrow_up:

ingest/importer/conversion/template_manager.py
79.84% <60%> (+0.15%)
:arrow_up:

ingest/importer/importer.py
80.61% <94.28%> (+6.4%)
:arrow_up:

ingest/importer/conversion/conversion_strategy.py
98.46% <0%> (+0.08%)
: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 945ecf1...c6d2040. Read the comment docs.


Update kernel for odroid-ux4

@agherzan @floion @michal-mazurek
Connects to resin-os/resinos#92

Closing as obsolete


getMessageByID Always Returns Failure

So I'm trying to get a deleted message's content, and every time I am getting an answer from the RestAction<Message>, it is a failure, even though I know the message exists in the specified channel.
@Override
    public void onGuildMessageDelete(GuildMessageDeleteEvent e) {
        if (!e.getChannel().equals(main.log)) {
            //Check to make sure it's not a message auto removed by the bot, causing an infinite loop
            if(!deletedMsgs.contains(e.getMessageId())) {
                //Check to make sure we know which channel it is in for less work
                if(messages.containsKey(e.getMessageId())) {
                    String channel = messages.get(e.getMessageId());
                    String id = e.getMessageId();
                    System.out.println("Found the message! " + id);
                    MessageChannel origin = e.getGuild().getTextChannelById(channel);
                    System.out.println("Found the channel! " + channel);
                    //Code executes fine up to here, we know the message exists in the specified channel
                    //This should return success, which is then a Message,
                    //so I should be able to success.getContent()
                    origin.getMessageById(id).queue(success -> System.out.println("Successfully found the message!"),
                         failure -> System.out.println("Could not find the message!"));
                } else {
                    System.out.println("Message was not saved in the map!");
                }
            }
        }
    }

If the message is deleted, most likely that the message object is also removed from JDA's cache.

You can't get a deleted message.

How can I can get a deleted message then? I see other bots that do it

You have to keep track of messages and access your own cache. Discord does not provide this.

If you need any more help please consider to join our discord server.
JDA does not provide a message cache to keep a lower memory footprint thus it is unable to provide deleted message.
If you want to keep track of messages that are deleted you will have to make your own cache or database which stores messages for your own use.


How to predict one response map for each vehicle key point

请问在代码当中是如何从一张训练集图片预测得到含有20个关键点的车辆响应图？

keypoint_test.txt和keypoint_train.txt是如何得到的？

The key points provided in this repo are manually annotated. We then use the annotations as supervision to train an Hourglass-like Network[1] for automatically predicting the key points, given an input vehicle image.
[1].A. Newell, K. Yang, and J. Deng. Stacked hourglass networks for human pose estimation. In ECCV, 2016


update engine name from sqlops to azdata

sqlops is no longer recognized and will be ignored, update the with azdata in case we need to restrict the ADS versions for these extensions.

Realistically we should also update vscode to * as well right? None of our extensions are actually dependent on the vscode version AFAIK

Realistically we should also update vscode to * as well right? None of our extensions are actually dependent on the vscode version AFAIK

IIRC, the vscode engine must be listed otherwise there will be an error.

@alanrenmsft - A number of extensions already defined vscode engine version as *, e.g. https://github.com/Microsoft/azuredatastudio/blob/a59d1d3c05ee4c67b08f7b6fff83367c97b6b73f/extensions/notebook/package.json
	"engines": {
		"vscode": "*",
		"azdata": "*"
	},

So I think we should be fine.

@alanrenmsft - A number of extensions already defined vscode engine version as *, e.g. https://github.com/Microsoft/azuredatastudio/blob/a59d1d3c05ee4c67b08f7b6fff83367c97b6b73f/extensions/notebook/package.json
	"engines": {
		"vscode": "*",
		"azdata": "*"
	},

So I think we should be fine.


publishing mixture of live and extract connections

https://github.com/tableau/server-client-python/blob/b7b544e5ef405ca76f9e0c5681b001373811c6b2/samples/publish_workbook.py#L73
I have a workbook with a live connection to an Excel file in the Box cloud, and I have attached it to the workbook via the Box connector. There are 3 other datasources in Teradata embedded as extracts and all of them are on the same database with the same credentials. The Box API makes me click on a "grant permission to application" in my browser when I open the workbook on Tableau Desktop. This setup works seamlessly through the desktop, I can refresh the db extracts and saved changes to the Excel file in Box populate onto the server version of the dashboard as long as I refresh the browser.
I am using the server-client-python package to trigger an extract refresh, and having to make a few other changes and publish the downloaded .twbx file using the following variants with no luck:
version 1
def publishWorkbook(twbWorkbook, tAuth, tServer, filePath):
    with tServer.auth.sign_in(tAuth):
        req_option = TSC.RequestOptions()
        req_option.filter.add(TSC.Filter(TSC.RequestOptions.Field.Name,
                                         TSC.RequestOptions.Operator.Equals,
                                         twbWorkbook))
        matching_workbooks, pagination_item = tServer.workbooks.get(req_option)
        cc = TSC.ConnectionCredentials(td_user, td_pass, embed=True)
        uploadWB = matching_workbooks[0]
        wbItem = TSC.WorkbookItem(project_id=uploadWB.project_id,name=twbWorkbook, show_tabs=True)
        wbItem = tServer.workbooks.publish(wbItem,
                                               file_path=filePath +
                                               twbWorkbook.replace(":", '') +
                                               '.twbx',
                                               mode='Overwrite',
                                               connection_credentials=cc)

Result: 400011: Bad Request
There was a problem publishing the file
version 2
        uploadWB = matching_workbooks[0]
        wbItem = TSC.WorkbookItem(project_id=uploadWB.project_id,name=twbWorkbook, show_tabs=True)
        wbItem = tServer.workbooks.publish(wbItem,
                                               file_path=filePath +
                                               twbWorkbook.replace(":", '') +
                                               '.twbx',
                                               mode='Overwrite')

Result: 400011: Bad Request
There was a problem publishing the file
version 3
        uploadWB = matching_workbooks[0]

        tServer.workbooks.populate_connections(uploadWB)
        conns = uploadWB.connections

        wbItem = TSC.WorkbookItem(project_id=uploadWB.project_id,name=twbWorkbook, show_tabs=True)
        wbItem = tServer.workbooks.publish(wbItem,
                                               file_path=filePath +
                                               twbWorkbook.replace(":", '') +
                                               '.twbx',
                                               mode='Overwrite',connections=conns)

Result: 400000: Bad Request
Payload is either malformed XML/JSON or incomplete
How can I handle this mixture of live / extract datasources that have different types of credentials using the REST API?
Tableau Server 10.5
server-client-python 0.7
python 3.5 on Ubuntu and macOS

Is there a solution for this yet? Can't we publish the reports with live connections yet?


Sort variables in VisualScriptEditor

Sorts the script variables in alphabetical order to display them in VisualScriptEditor.
The variables are sorted in 3.x because get_variable_list returned the list already sorted but this was changed in #39649.
This PR sorts the variables after retrieving them in VisualScriptEditor instead of directly in get_variable_list so this works like the functions (also retrieved unsorted with get_function_list and then sorted by VisualScriptEditor).
Closes #58509

Question, do we want to use Unicode sorting? @bruvzg Any ideas?

Question, do we want to use Unicode sorting? @bruvzg Any ideas?

There's no easy solution. We can use ICU collation API. But it will depend on a specific text server implementation, and probably will need more data. I need to re-check it, but I'm pretty sure I have stripped it from the data file currently used.

Sounds difficult, but Unicode sorting doesn't block this.

Thanks!


pyuac support

requirements
pip install pyuac pywin32
code

import pyuac

if not pyuac.isUserAdmin():
    pyuac.runAsAdmin()

when compiled we get this

since it tries to find python.exe

Any progress on this one @KevinRodriguez777 it seems easy I think, just needs to identify the function that does the elevation and use change_function on it. If you feel the docs are lacking for that to be actually easily doable, let me know.


Deprecated NSDictionaryExtensions

Checklist

[ ] New Extension
[ ] New Test
[ ] Changed more than one extension, but all changes are related
[x] Trivial change (doesn't require changelog)

Related to #281

          8 Warnings

:warning:
EZSwiftExtensionsTests/DictionaryTests.swift#L94: ‘formatJSON()’ is deprecated
        let jsonString = jsonDic.formatJSON()

:warning:
EZSwiftExtensionsTests/DictionaryTests.swift#L96: ‘init(json:)’ is deprecated
        let secondJsonDic = NSDictionary(json: jsonString!)

:warning:
EZSwiftExtensionsTests/NSDictionaryTests.swift#L16: ‘formatJSON()’ is deprecated
        let json = dic.formatJSON()

:warning:
EZSwiftExtensionsTests/NSDictionaryTests.swift#L16: ‘formatJSON()’ is deprecated
        let json = dic.formatJSON()

:warning:
EZSwiftExtensionsTests/NSDictionaryTests.swift#L24: ‘init(json:)’ is deprecated
        let dic = NSDictionary(json: str)

:warning:
EZSwiftExtensionsTests/DictionaryTests.swift#L96: ‘init(json:)’ is deprecated
        let secondJsonDic = NSDictionary(json: jsonString!)

:warning:
EZSwiftExtensionsTests/DictionaryTests.swift#L94: ‘formatJSON()’ is deprecated
        let jsonString = jsonDic.formatJSON()

:warning:
EZSwiftExtensionsTests/NSDictionaryTests.swift#L24: ‘init(json:)’ is deprecated
        let dic = NSDictionary(json: str)

          1 Message

:book:
Executed 142 tests, with 0 failures (0 unexpected) in 4.425 (4.686) seconds

  Generated by :no_entry_sign: danger

@Khalian is this good to go before we can approve #334?

Yeah this is fine.


Problems with unicode characters

First: this is a great plugin which I use every day, thanks for making it available!
I noticed I can no longer work with (German, in my case) Unicode-characters after loading zsh-hist, here is what happens:
Without this plugin:
❯ echo 'ß'
ß

Same command with zsh-hist loaded:
❯ echo 'Ã<bf>'
Ã

...which then breaks programs like grep searching for Umlauts or similar.
This behavior is reproducible even in a clean zsh-environment as described here

Works now, thanks!


Reformat all the code to match the latest SQ codestyle

Most of the noise is coming from the change to indent spaces 2 instead
of the Kotlin standard of 4.
https://github.com/square/java-code-styles/pull/51
This is causing a lot of local diff noise when developing, so figured
one giant change was better.

I only reformatted the KT and java files
~/Development/misk rhall-reformat git diff HEAD^1..HEAD --name-only | grep -v ".kt" | grep -v ".java"
~/Development/misk rhall-reformat

I'd prefer to do this as a git history rewrite to avoid breaking git blame. It's more work but it avoids having a giant gap in the (short) history of the project.

Oh I've never done that before!  I did a quick google search and couldn't find any tooling to support that

Here’s a branch with the rewrites:
https://github.com/square/misk/commits/master_reformatted

And here’s an example of a rewritten commit on master and master_reformatted

... and the program that recursively “reformats” a directory of Kotlin files. This one just halves the leading spaces on .kt files and nothing else.
https://gist.github.com/swankjesse/7952d0c54efc2701fe6fc0bc184549cc

@mmihic @nbroyles if y’all give me the green light, I’ll run this on master. I don’t think I can do it with a pull request though. It’ll just be a push to master with a new rewritten history.
The only hassle is that you’ll need to pull master and fixup in-flight PRs. Possibly by running the gist above.

@swankjesse sgtm!

@swankjesse I'm not 100% sure, but I think you probably want to keep a tag or branch which points to the master-before-reformatting to keep old commits reachable, especially for those commits mentioned in GitHub Issues.


Add collocation method for optimal control problems

This PR changes the default way that optimal control problems are solved from using a shooting method to using collocation.  This results in more robust (and faster) solutions, especially for trajectory generation for unstable or marginally stable systems (where shooting is very sensitive to small parameter variations).
The following new functionality is included:

The solve_ocp() function and OptimalControlProblem class have new parameters trajectory_method that be set to either shooting (default for discrete time systems) or collocation (default for continuous time systems).
When collocation is used, the initial_guess parameter can either be an input trajectory (as before) or a tuple consisting of a state trajectory and an input trajectory.
The documentation (docstrings and manual) have been updated to reflect the new functionality.
New unit tests are included that tease out the functionality of the various possibilities.
Benchmarks comparing shooting and collocation will be added in a bit (after another PR updating the benchmark suite is added).

This PR also addresses issue #782 (see also discussion in #781) which were showing some of the weakness of the shooting methodology.

Coverage decreased (-0.03%) to 94.806% when pulling 614f2a7cb012d79607d08b5156e76fafd7b87b0f on murrayrm:fix_horizon-19Nov2022 into 832527db6f8198cbe15b4414a85e8d61af795835 on python-control:main.


Learning Mode (Without Intelissense, Auto-complete and Other Automations)

Learning Mode (Stop Easily Automations)
The intelissense / autocompletion provided by VsCode and related plugins is nice at work, but as a college student i could say, when education is the case  (mainly in classes at colleges), these are rather damning for learning.
I haven't found in vscode docs and in the application no way to deactivate almost everything so that i would need to type everything (without any automation to be seen or made). I just found this script here (to add in settings.json:
 // TURN OFF AUTOCOMPLETION
    // Controls if quick suggestions should show up or not while typing
    "editor.quickSuggestions": false,

    // Controls the delay in ms after which quick suggestions will show up
    "editor.quickSuggestionsDelay": 90,

    // Enables parameter hints
    "editor.parameterHints": false,

    // Controls if the editor should automatically close brackets after opening them
    "editor.autoClosingBrackets": false,

    // Controls if the editor should automatically format the line after typing
    "editor.formatOnType": false,

    // Controls if suggestions should automatically show up when typing trigger characters
    "editor.suggestOnTriggerCharacters": false,

    // Controls if suggestions should be accepted 'Enter' - in addition to 'Tab'. Helps to avoid ambiguity between inserting new lines or accepting suggestions.
    "editor.acceptSuggestionOnEnter": "off"

It'd be good to have an option that could be named as 'turn on learning mode' or something else, which would deactivate everything related to intelissense and autocomplete (automation in general) in VsCode, independently of plugins turned on at the moment.

Thank you for the interesting idea and use case!

How are they "damning for learning"? If anything, to me, they are helpful for learning.


update prevent plant notebook with 2020 changes

Main changes:

kick off 2020 in-season predictions
source updates and replacements: precipitation is read from GPM instead of TRMM; Two weeks extended precipitation forecast from GFS is an option.
some bug fixes to account for a county in the corn belt with no data.

Looks ok on high level, i.e. assuming various new concat and merge operation does not screw up alignment. I think merge sometimes changes row order (even though it is not supposed to), so I tend to insert sort_index all over the place for safety. These potential alignment issues can't really be  checked without looking at actual dataframes, so you are better positioned to do this.
I think interpolation I used will extent last loaded values as constants (or maybe even set them to 0) beyond what's loaded (all the way till the end of the year), so you might be able to generate some model projections. But I am not sure you want to use such constant extrapolations them in actual projection computation. Last year all data needed were already in, so it was not an issue, but this year you probably want to ensure that 2020 projection is only generated for model ending week 18 or 19 - whenever last GFS precipitation projection ends. Going beyond that is probably not warranted.
Finally, I am a bit surprised that INS (2011-2019) accuracy of final GBT models appears to be rather poor (much worse than what we saw last year, I think) but, again assuming there is no alignment issues,  I guess it is what it is. It could indeed be that 2019 year being so huge relative to others causes the model to overestimation other years to get closer to the 2019 number, but I don't like that it seems to overestimate 2019 as well ...
On the missing counties for geopandas query - I don't think this was an issue last year. Were new counties introduced sinc ethn?

Update: pushed a fix for df_all, which did not have lat/lon filled in properly at the beginning.
@ash-gro regarding your questions:

alignment is the exact reason why i switched to merge and concat in specific cases, namely to create one single df to include both features and target, to make sure features and targets are on one-to-one correspondence.
I was concerned about interpolation at the beginning as well, especially the extrapolation in the end of the time series. As you said we can make sure only models prior to a time period (e.g. models of week 14 through week 16 if not including weather forecast, or through 18 if including weather forecast) are presented to minimize the impact of extrapolations. I can make an automatic cut-off is that's desired.
the bug fix above have solved the problem for bad in-sample performance as in the second chart for each period. out-of-sample performances should not change though after the bug fixes. they are even slightly better than last year if you look at the numbers

Oh, so THAT's what this warning was about.  The dataframe simply did not have lon/lat coordinates in it but, when asked for them, pandas in its misguided wisdom simply filled in with something and gave this obscure warning:___________/home/yiqing/.local/lib/python3.6/site-packages/pandas/core/indexing.py:979: FutureWarning:Passing list-likes to .loc or [] with any missing label will raiseKeyError in the future, you can use .reindex() as an alternative. See the documentation here:https://pandas.pydata.org/pandas-docs/stable/indexing.html#deprecate-loc-reindex-listlikereturn self._getitem_nested_tuple(tup)______________ I just hate these pandas internal warnings without any reference to the user code which caused them - these are next to impossible to trace without removing potentially offending lines one-by-one. Moreover, they are often benign, so natural tendency is to just ignore them if some result is generated ... And yes, INS results look much good now.

View entire conversation on ReviewNB

>>out-of-sample performances should not change though after the bug fixes. they are even slightly better than last year if you look at the numbers I assume you refer to the skip-a-year CV-like results. These should indeed be the same as before, but the 2020 full xgb model projection will change, I think.

View entire conversation on ReviewNB

yes it did. you can see the changes from the diff in the notebook

View entire conversation on ReviewNB

done on all cases

View entire conversation on ReviewNB

On using list of regions instead of getting one-by-one, I think there is a limit on how long the list can be (for example, I don't think one can't get entire world on county level in a single request). But I guess list of counties for a single US state should be ok.
For the code
if not in_season_run:
df_interp.interpolate(inplace=True, limit_direction='both')
else:
....
df_interp.interpolate(inplace=True, limit_direction='both')
wouldn't it be easier to put interpolation outside the if (and have only one if-branch)
Finally, comments (in data loading cells)
pd.interpolate will do linear interpolation ignoring index for multiindexed frames
which is exactly what we need in this case
and
internal concat is to add index layer, external one to assemble different frames into one
is out of date and does not correspond to the code now (interpolated frame is not multiindex and there is only one concat)

On using list of regions instead of getting one-by-one, I think there is a limit on how long the list can be (for example, I don't think one can't get entire world on county level in a single request). But I guess list of counties for a single US state should be ok.

Regarding lists of regions, there is an upper limit: ~300
For districts per US state, you should be okay - the max is Texas with 254.
for state in client.get_descendant_regions(1215, 4):
    print(state['name'], len(client.get_descendant_regions(state['id'], 5)))

There's a api.client.utils.list_chunk helper function I've been using as a standard way of doing it (it defaults to 50 to be safe, but if you really want to, you can play with the chunk_size kwarg). For example:
for region_chunk in list_chunk(region_ids):
    get_data_points(**other_selections, region_id=region_chunk)

I really hope we can refactor that once for all when we are ready to expose some reading utility functions from bitbucket such as https://bitbucket.org/grointelligence/gro/src/e236a3377842eb708fd11dd9500a9dc1b9f4441a/xm/prevent_plant/read.py#lines-64, so that we dont have to maintain the two versions separately.

View entire conversation on ReviewNB

same here. we simplified the logic by a lot in https://bitbucket.org/grointelligence/gro/src/e236a3377842eb708fd11dd9500a9dc1b9f4441a/xm/prevent_plant/read.py#lines-27 by using our API reader. once we expose that part to the client, we can reduce the amount of redundancy here by a lot. Maybe we should just make this a TODO to refactor all the readings once for all after our internal reader is added to the API client as a utility?

View entire conversation on ReviewNB

Updated. I Sill have them in two separate cells to clearly separate the cases for soil moisture and precipitation, since precip needs special treatment from weather forecast, but the data collection is now through add_single_data_series  and get_df

View entire conversation on ReviewNB

TODO added.

View entire conversation on ReviewNB

confirmed with @tolothy that it was tested with py3. shippable tests are passing despite "unstable"


Simplify applying autocomplete suggestions

Today the autocomplete toolbar has a standard function for applying a suggestion. However, this should be an even more foundational function.

UITextDocumentProxy has a new insertAutocompleteSuggestion function to make it easier to call this programatically.

StandardKeyboardActionHandler has a new tryApplyAutocompleteSuggestion that it will call to try inserting any existing autocomplete suggestions.


Users can delete series with episodes when series.hasEvents.delete.allow=false

Describe the bug
As reported by @uzk-rrzk in the opencast group, if the setting series.hasEvents.delete.allow is set to false, a user owner of the series still can delete the series when the series has events.
The routine can you find it in  SeriesEndpoint.java
To Reproduce
Steps to reproduce the behavior:

Set series.hasEvents.delete.allow=false in org.opencastproject.adminui.endpoint.SeriesEndpoint.cfg.
With a user owner of a series with events, try to delete the series.
The series is deleted

Environment Information

OS: CentOS 7
Opencast branch 8.x

I'm looking into this and I found something interesting.
If you are in the admin user, it calls the endpoint that confirms if the user is allowed to delete the series with events, but if its a non-admin user is not called this API endpoint in the admin-ui
 @GET
  @Path("configuration.json")
  @Produces(MediaType.APPLICATION_JSON)
  @RestQuery(name = "getseriesconfiguration", description = "Get the series configuration", returnDescription = "List of configuration keys", reponses = {
    @RestResponse(responseCode = SC_BAD_REQUEST, description = "The required form params were missing in the request."),
    @RestResponse(responseCode = SC_NOT_FOUND, description = "If the series has not been found."),
    @RestResponse(responseCode = SC_OK, description = "The access information ") })
  public Response getSeriesOptions() {
    JSONObject jsonReturnObj = new JSONObject();
    jsonReturnObj.put("deleteSeriesWithEventsAllowed", deleteSeriesWithEventsAllowed);
    return Response.ok(jsonReturnObj.toString()).build();
  }

  public Boolean getOnlySeriesWithWriteAccessSeriesTab() {
    return onlySeriesWithWriteAccessSeriesTab;
  }

  public Boolean getOnlySeriesWithWriteAccessEventsFilter() {
    return onlySeriesWithWriteAccessEventsFilter;
  }
}```

I suspect this problem comes from the frontend.

Yes, it was the frontend, some endpoints were blocked. With these lines will fix the issue
<sec:intercept-url pattern="/admin-ng/series/*/hasEvents.json" method="GET" access="ROLE_ADMIN, ROLE_UI_SERIES_VIEW" />
<sec:intercept-url pattern="/admin-ng/series/configuration.json" method="GET" access="ROLE_ADMIN, ROLE_UI_SERIES_VIEW" />


[FEATURE] Allow to add a [skip ci] when cog create commit

Is your feature request related to a problem? Please describe.
I use cocogitto in my CI system, and I trigger jobs when commits are pushed on my develop and main branch. When I use cog bump, it will create a commit and the CI is triggered when I push it.
Describe the solution you'd like
It would be nice to have an option that allowed me to say "Add [skip ci] pattern at the end of the commit".
Like all CI system use a different pattern to skip build, having a parameter to say which pattern to use seems to be a clever idea
$ cog bump --auto --skip-ci "[skip-ci]"

Thanks for your time

Hey @AlexisMtr and @oknozor
I took the liberty of implementing this feature because I think It would be an awesome addition :)
If you guys wanna have a look : https://github.com/cocogitto/cocogitto/pull/274
So far you can only specify the skip-ci pattern using the CLI but it would be great to have it specified globally through the cog.toml file :)
Wassim.

PR Merged and released in 5.5.0 -> https://github.com/cocogitto/cocogitto/releases/tag/5.5.0
The PR for the documentation -> https://github.com/cocogitto/website/pull/57

will this work only with for bump commands? e.g., cog bump --skip-ci <string>
I like to use this for normal commits as well.  for example, don't want my CI to run jobs for commits I know it is a waste of time.
# assume I have  skip_ci = "[skip ci]" in cog.toml
cog commit docs "added cli options" cli --skip-ci

question
if I set skip_ci = "[skip ci]" in cog.toml
will it add "[skip ci]" for every bump ?
what if I want to add skip_ci for selective bump commits?
would be nice if we can enable/disable skip_ci flag via command-line but have the skip_ci = "[skip ci]" in cog.toml
cog bump --auto --skip-ci