PR with SSO keycloak documentation

Describe your changes :
I worked on the Security Documentation because i added the integration with Keycloak SSO

Type of change :

[ ] Bug fix
[ ] Improvement
[ ] New feature
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] Documentation

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] I have added tests that prove my fix is effective or that my feature works.
[x] All new and existing tests passed.

Reviewers

Thanks @luanpaschoal


Grammar improvements

Hi, I've made a few changes to some of the (English) grammar. Please let me know if you want any of these changed in any way.

Thanks!


verson 2.4 compatibility

><EMAIL_ADDRESS>postinstall /app/node_modules/protobufjs
> node scripts/postinstall

npm notice created a lockfile as package-lock.json. You should commit this file.
npm WARN notsup Unsupported engine for<EMAIL_ADDRESS>wanted: {"node":">=16.6.1"} (current: {"node":"14.19.0","npm":"6.14.16"})
npm WARN notsup Not compatible with your version of node/npm<EMAIL_ADDRESS>
npm notice created a lockfile as package-lock.json. You should commit this file.
npm WARN notsup Unsupported engine for<EMAIL_ADDRESS>wanted: {"node":">=16.6            .1"} (current: {"node":"14.17.5","npm":"6.14.14"})
npm WARN notsup Not compatible with your version of node/npm<EMAIL_ADDRESS>
Expected Behavior
Version 2.x of the package: Node versions 12.22.5+, 14.17.5+, or 16.6.1+. Version 2.x uses the pure JS implementation of the gRPC library, and requires a fix to the nghttp2 library in Node (See https://github.com/camunda-community-hub/zeebe-client-node-js/issues/201).

Unfortunately, the engines field is a semver range, not a list of ranges.
https://docs.npmjs.com/cli/v6/configuring-npm/package-json#engines


Plugins: Can't use Typescript Mixins to build a datasource

What happened?
When creating a DataSource using Mixins, grafana logs the following error :
Uncaught (in promise)
Object { message: "Datasource: fd8841fb-5d9c-4e1c-aeb7-8c5489e83032 was not found" }
[actions.ts:174:2](webpack://grafana/public/app/features/datasources/state/actions.ts)

I tracked the bug down to this line
Typescript Mixins constructors MUST have a single rest parameter, therefore dsPlugin.DataSourceClass.length will be 0 instead of 1, incorrectly triggering the angular loading path.
What did you expect to happen?
Datasources built with mixins should work.
Did this work before?
I suppose not.
How do we reproduce it?
class Query {}
class Options {}

interface FooConstructor<
  TFoo extends AbsFooAPI<TQuery, TOptions>,
  TQuery extends Query = Query,
  TOptions extends Options = Options,
> {
  new (...args: any[]): TFoo;
}

// DatasourcePlugin
class FooPlugin<
  TFoo extends AbsFooAPI<TQuery, TOptions>,
  TQuery extends Query,
  TOptions extends Options,
>{
  private foo: TFoo
  constructor(FooClass:FooConstructor<TFoo, TQuery, TOptions>){
    this.foo = new FooClass()
  }
  process(){
    console.log('Plugin')
    this.foo.doFoo()
    if (this.foo.doBar){
      this.foo.doBar()
    }
    if (this.foo.doBaz){
      this.foo.doBaz()
    }
  }
}

// DatasourceAPI
abstract class AbsFooAPI<TQuery extends Query, TOptions extends Options> {
  constructor(arg1:string){}
  abstract doFoo():void
  doBar?():void;
  doBaz?():void;
}

// DatasourceWithBackend
class Foo<
  TQuery extends Query = Query,
  TOptions extends Options = Options,
> extends AbsFooAPI<TQuery, TOptions>
{
  constructor(arg1:string){
    super(arg1)
  }
  doFoo():void {
    console.log("Done Foo");
  }
}

// My Datasource base
class FooBar extends Foo {
  constructor(arg1:string){
    super(arg1)
  }
  doBar():void{
    console.log('Done Bar')
  }
}

// My datasource feature mixin

type GConstructor<T = {}> = new (...args: any[]) => T;

function withBaz<T extends GConstructor<FooBar>>(Base : T){
  return class extends Base {
    // A mixin constructor must have only one rest parameter of type any[]
    // A constructor with a positional parameter breaks the mixin contract
    // constructor(first: any, ...args:any[]){
    //  super(first, ...args)
    // }
    doBaz():void{
      console.log('Done Baz')
    }
  }
}

class MyQuery extends Query {}
class MyOptions extends Options {}

const FooBarBaz = withBaz(FooBar)

const fbb = new FooBarBaz('settings')

const plugin = new FooPlugin<InstanceType<typeof FooBarBaz>,MyQuery,MyOptions>(FooBarBaz)

plugin.process()
console.log("AbsFooAPI LENGTH", AbsFooAPI.length)
console.log("Foo LENGTH", Foo.length)
console.log("FooBar LENGTH", FooBar.length)
console.log("FooBarBaz LENGTH", FooBarBaz.length)

This example replicates the structure of a DataSource plugin and showcases how the use of mixins breaks the assumption on the constructor's length.
Transposing it onto a real plugin should be trivial.
Is the bug inside a dashboard panel?
No response
Environment (with versions)?
Grafana: Grafana v10.3.3 (252761264e)
OS: Ubuntu 23.04
Browser: Firefox 122.0
Grafana platform?
Docker
Datasource(s)?
No response

@ddelemeny thanks for reporting this issue, we are currently planning to deprecate angular https://grafana.com/docs/grafana/latest/developers/angular_deprecation/. Once we have removed angular loading support this issue would resolve itself.
Since our standard way of creating datasources is not relying on mixins - we do not intend to invest time adjusting angular loading logic which will be removed at some point anyway.
Closing as it will resolve itself.


Error: Could not find a production build in the '/Users/nick/dev/handbook/.next' directory

I followed the instructions in the readme to run locally on a fresh checkout and got the following error:
nick@Nicks-MBP dev % git clone git@github.com:sourcegraph/handbook.git
Cloning into 'handbook'...
remote: Enumerating objects: 26624, done.
remote: Counting objects: 100% (1702/1702), done.
remote: Compressing objects: 100% (882/882), done.
remote: Total 26624 (delta 994), reused 1268 (delta 784), pack-reused 24922
Receiving objects: 100% (26624/26624), 7.81 MiB | 12.29 MiB/s, done.
Resolving deltas: 100% (15579/15579), done.
nick@Nicks-MBP dev % cd handbook
nick@Nicks-MBP handbook % yarn
yarn install v1.22.11
[1/4] 🔍  Resolving packages...
warning Resolution field<EMAIL_ADDRESS>is incompatible with requested version<EMAIL_ADDRESS>warning Resolution field<EMAIL_ADDRESS>is incompatible with requested version<EMAIL_ADDRESS>warning Resolution field<EMAIL_ADDRESS>is incompatible with requested version<EMAIL_ADDRESS>warning Resolution field<EMAIL_ADDRESS>is incompatible with requested version<EMAIL_ADDRESS>warning Resolution field<EMAIL_ADDRESS>is incompatible with requested version<EMAIL_ADDRESS>warning Resolution field<EMAIL_ADDRESS>is incompatible with requested version<EMAIL_ADDRESS>[2/4] 🚚  Fetching packages...
[###############################################################################################################################################################-] 797/[###############################################################################################################################################################-] 798/[#############info<EMAIL_ADDRESS>The CPU architecture "x64" is incompatible with this module.
info "<EMAIL_ADDRESS>is an optional dependency and failed compatibility check. Excluding it from installation.
info<EMAIL_ADDRESS>The platform "darwin" is incompatible with this module.
info "<EMAIL_ADDRESS>is an optional dependency and failed compatibility check. Excluding it from installation.
info<EMAIL_ADDRESS>The platform "darwin" is incompatible with this module.
info "<EMAIL_ADDRESS>is an optional dependency and failed compatibility check. Excluding it from installation.
[3/4] 🔗  Linking dependencies...
warning " ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "next > styled-jsx ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning " ><EMAIL_ADDRESS>has unmet peer dependency "@types/node@*".
[4/4] 🔨  Building fresh packages...
✨  Done in 15.69s.
nick@Nicks-MBP handbook % yarn start
yarn run v1.22.11
$ next start
ready - started server on <IP_ADDRESS>:3000, url: http://localhost:3000
Error: Could not find a production build in the '/Users/nick/dev/handbook/.next' directory. Try building your app with 'next build' before starting the production server. https://nextjs.org/docs/messages/production-start-no-build-id
    at Server.readBuildId (/Users/nick/dev/handbook/node_modules/next/dist/server/next-server.js:1573:23)
    at new Server (/Users/nick/dev/handbook/node_modules/next/dist/server/next-server.js:92:29)
    at NextServer.createServer (/Users/nick/dev/handbook/node_modules/next/dist/server/next.js:107:16)
    at async /Users/nick/dev/handbook/node_modules/next/dist/server/next.js:119:31
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Ah, I would guess for yarn/next start (production testing) you first need to run yarn/next build.
What I've been using all week is yarn dev, which just needs that one command. Maybe we should update the instructions to point to yarn dev (I wrote these really quickly without fully understanding NextJS myself yet 😄 ).


Refresh tests button causing error

On both a java and a python project when I click the refresh button the following error pops up:

https://github.com/microsoft/vscode/assets/26030610/d2c2f822-0872-4987-b258-2c7812307c9a

any thoughts on this @connor4312? Seeing a few people commenting about it and if it ends up being on my side I investigate more but I think it might be core. Thanks

Great thank you! Will test on insiders next week!


tighten ALPN for proxy connections

mod_ssl: tighten the handling of ALPN for outgoing (proxy)
connections. If ALPN protocols are provided and sent to the
remote server, the received protocol selected is inspected
and checked for a match. Without match, the peer handshake
fails.
An exception is the proposal of "http/1.1" where it is
accepted if the remote server did not answer ALPN with
a selected protocol. This accomodates for hosts that do
not observe/support ALPN and speak http/1.x be default.

NOTE: the first 11 commits listed in this PR have already been merged. This stems from the fact that httpd github is a read-only mirror of Apache's subversion repository. In the future, I will start a new local branch for such a change, so it will be easier to follow the history of a PR. Thanks.

Merged in r1890693. Thanks!


Fix conn_param leaking issue which discovered by wangha

Codecov Report
Patch coverage: 32.14% and project coverage change: -0.04 :warning:

Comparison is base (e5b65e9) 53.11% compared to head (f22d596) 53.08%.

: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
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #553      +/-   ##
==========================================
- Coverage   53.11%   53.08%   -0.04%
==========================================
  Files         120      120
  Lines       36576    36594      +18
==========================================
- Hits        19429    19427       -2
- Misses      17147    17167      +20

Impacted Files
Coverage Δ

src/mqtt/protocol/mqtt/mqttv5_client.c
0.00% <0.00%> (ø)

src/supplemental/mqtt/mqtt_msg.c
47.45% <22.22%> (-0.48%)
:arrow_down:

src/mqtt/protocol/mqtt/mqtt_client.c
66.52% <40.00%> (-2.53%)
:arrow_down:

src/sp/protocol/mqtt/nmq_mqtt.c
45.80% <55.55%> (+0.76%)
:arrow_up:

... and 18 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


my claims have slicie members,and they can not be parse

type Claims struct {
	Username string `json:"username"`
	Password string `json:"-"`

	UserID     uint       `json:"userID"`
	Roles      []Role     `json:"roles"`
	Businesses []Business `json:"businesses"`

	ClientIp string `json:"clientIp"`
	jwt.StandardClaims
}

the Roles and Business's value will be []empty

Without more code, it's difficult to diagnose what your issue might be. My guess is the json decoder doesn't know how to decode into the Role and Business types you are using. Those types need to either be directly supported by the json package or they need to implement the JSON Unmarshaler interface.

Closing this ticket. Feel free to reopen if you have more information and still need help.


fix perf metrics calculation for genai llm pipeline

Fixing the regression introduced in https://github.com/openvinotoolkit/openvino.genai/pull/1118
CVS-156870

Please, fix code style


Fix SelectFieldBase.iter_choices docstring

This fixes one of the out-of-date docstrings mentioned in #811 but not fixed in #812.

Thank you.


Improve AnnotationUtil

Motivation:
Follow-up work after #1560.
Modifications:

AnnotationUtil will lookup meta-annotations of a meta-annotation to support the following case:

@ProducesJson
@Produces("text/plain")
@interface MyPostServiceSpecifications {}

@ProducesJson will also be found with this PR, which wasn't found before.
Add ElementType.ANNOTATION_TYPE to Target annotation for @Order and @StatusCode so that a user can use them as a meta-annotation for his or her custom annotation.

Thanks, @hyangtack!


feat: add typescript support

Hi, this PR would add better intellisense/code-complete for JavaScript developers and support TypeScript.
If you don't want to merge this it can instead be published to DefinitleyTyped, so no worries if you want to keep it separate.

Hello there!
Thanks for this PR. I am not a JS expert (anymore) and dont have the chance to test the TS version as well. Therefore wont be merging this.
Please push it in any other repo as you see fit and share it here.
Cheers,
Doruk

Thanks @dorukeker, I've added them to DefinitelyTyped. The PR is here and they can be installed via @types/gyronorm


feat(connlib): handle internet resource

It's working :) just want to do some manual testing but tests are passing!
Appreciate some early reviews, though.
This PR tries to handle the internet resource as simply as possible.
In the gateway, it's handled like any other resource save for the address which is both ipv6 and ipv4.
In the client it's handled mostly like any other resource but with some exceptions for DNS forwarded packets, because we want to work as a CIDR resource for the matter of forwarding packets to the gateway.
Fixes: #6313.

@jamilbk @thomaseizinger Are we okay to merge this and then test this after #6299 ? it's hard to test these changes without the portal since even if we "fakely" add the resource in the client the gateway needs to recieve it.
The tests are passing so it's a good indication that everything should be working and we can make any PR after #6299 if something should not work


Add cross-build wheel jobs for ppc64le and s390x

Summary
This commit adds a new CI job at release time to build precompiled
binaries for s390x and ppc64le linux platforms.  These platforms do not
have precompiled wheels for any upstream dependencies (except for
retworkx and tweedledum, which both publish binaries for both platforms).
Since the only publically available CI service that has native s390x and
ppc64le environments available is travis which doesn't provide
sufficient quota to open source projects anymore to make it usable we
have to rely on either cross compilation or emulation. Cross compiling
with python wheels while possible is quite tricky to setup and configure
in practice so emulation is used here. This is a much simpler path
configure, especially because cibuildwheel has support for emulating
non-x86 architectures via QEMU to build wheels. While QEMU emulation of
other architectures is exceedingly slow, we have a 6 hour job time limit
with github actions which should hopefully be sufficient to compile the
binary and build wheels for all our supported python versions.
This was previously attempted in #5844 but we hit a timout issue
reliably every time trying to build scipy from source. Compiling scipy
is slow on a native system and it's an order of magnitude slower under
qemu emulation. In #5844 we exceeded the 6 hour job time limit just
compiling scipy once for testing. To avoid this issue (since scipy
doesn't publish s390x or ppc64le wheels) this commit skips the test
runs on these jobs. This means we're solely building the wheels and not
testing if they're valid. This also means installing scipy and other
dependencies is still an exercise for the s390x and ppc64le users. But,
if this works it will mean that at least for Qiskit there will not be a
need to compile anything from source.
Details and comments

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.005%) to 83.145%

Files with Coverage Reduction
New Missed Lines
%

qiskit/pulse/library/waveform.py
3
89.36%

Totals

Change from base Build<PHONE_NUMBER>:
-0.005%

Covered Lines:
51998

Relevant Lines:
62539

💛  - Coveralls

Ci worked, you can see the logs (and download the wheels) from here: https://github.com/Qiskit/qiskit-terra/actions/runs/1724756938
I'll spin up an s390x vm in a bit to test the wheel works

In principle this looks good to me, but I don't really have the experience with virtualisation to meaningfully test the output wheels (nor the right software on my laptop, as far as I know!).

You could install QEMU locally for emulation like we do in CI to build these but I don't really recommend this as it's super slow. I normally test s390x via the LinuxONE community cloud which let you provision s390x VMs for open source work:
https://developer.ibm.com/gettingstarted/ibm-linuxone/
(my account was stale so I'm waiting on a new one to be provisioned now). For ppc64le I don't have a good test mechanism short of knowing someone with a server.

Do we want a release note to talk about the new wheels?

Yeah a release note is a good call, I'll add one in a bit.

Approved, provided you're happy with your latest VM tests.  It might not worth backporting this to 0.19.2, unless you reckon it'd be good to do a test release with the new platforms before the next minor.

I've tested this on s390x (with the python 3.8 wheel) and it works fine (after compiling numpy and scipy from source):
Python 3.8.10 (default, Nov 26 2021, 20:14:08)
[GCC 9.3.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import platform
>>> platform.processor()
's390x'
>>> import qiskit
>>> qiskit.__version__
'0.20.0'
>>> qc = qiskit.QuantumCircuit(2)
>>> qc.h(0)
<qiskit.circuit.instructionset.InstructionSet object at 0x3ff7e6bc080>
>>> qc.cx(0, 1)
<qiskit.circuit.instructionset.InstructionSet object at 0x3ff7e6bc340>
>>> qc.measure_all()
>>> from qiskit.test.mock import FakeRome
>>> print(qiskit.execute(qc, FakeRome()).result().get_counts())
/home/linux1/test_wheels/lib/python3.8/site-packages/qiskit/test/mock/fake_backend.py:156: RuntimeWarning: Aer not found using BasicAer and no noise
  warnings.warn("Aer not found using BasicAer and no noise", RuntimeWarning)
{'11': 521, '00': 503}

Approved, provided you're happy with your latest VM tests. It might not worth backporting this to 0.19.2, unless you reckon it'd be good to do a test release with the new platforms before the next minor.

Yeah, I think we can just hold this until 0.20.0. There's no pressing need here. It's not like either of these platforms have a lot of users. Looking at the CPU stats from PyPI over the last 30 days:
| cpu      | download_count |
| -------- | -------------- |
| x86_64   |         52,446 |
| None     |         25,158 |
| AMD64    |          9,037 |
| arm64    |            465 |
| aarch64  |            198 |
| armv7l   |            138 |
| i686     |            136 |
| s390x    |             58 |
| iPad13,4 |             24 |
| iPad8,5  |             20 |
| Total    |         87,680 |

s390x clients are only 0.06% of our downloads and 0 ppc64le downloads.

I've tested this on ppc64le (with python 3.7, 3.8, and 3.9) and they work fined.
$ python
Python 3.7.11 (default, Jul 27 2021, 15:02:35)
[GCC 7.3.0] :: Anaconda, Inc. on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import platform
>>> platform.processor()
'ppc64le'
>>> import qiskit
>>> qiskit.__version__
'0.20.0'
>>> qc = qiskit.QuantumCircuit(2)
>>> qc.h(0)
<qiskit.circuit.instructionset.InstructionSet object at 0x7dd35f50caa0>
>>> qc.cx(0, 1)
<qiskit.circuit.instructionset.InstructionSet object at 0x7dd35f50cdc0>
>>> qc.measure_all()
>>> from qiskit.test.mock import FakeRome
>>> print(qiskit.execute(qc, FakeRome()).result().get_counts())
/home/hhorii/anaconda3/envs/qiskit202201_3.7/lib/python3.7/site-packages/qiskit/test/mock/fake_backend.py:156: RuntimeWarning: Aer not found using BasicAer and no noise
  warnings.warn("Aer not found using BasicAer and no noise", RuntimeWarning)
{'00': 502, '11': 522}


Question about adding function to airline status

Hi
I was wondering is it possible to add this function to airline status bar.
function! NearestMethodOrFunction() abort
  return get(b:, 'vista_nearest_method_or_function', '')
endfunction

set statusline+=%{NearestMethodOrFunction()}

I got this from here but it seems to be for lightline only ?
https://github.com/liuchengxu/vista.vim#show-the-nearest-methodfunction-in-the-statusline
Thanks

have a look at the tagbar extension. It already provides such a feature out of the box.

hi @chrisbra I used tagbar however tagbar doesn't support LSP while vista does, so I switched to vista.
is there a way integrating that with airline ?

If you just want to display the function/method name, I fail to see why tagbar can't be used?
Have a look at the tagbar extension, it should be fairly easy to translate to vista. And then you might want to create a PR for vista.

thanks, will git it a go.

Thanks @chrisbra I will give it a go, just to confirm when you said  "I fail to see why tagbar can't be used" did you mean as a vim plugin or extension to airline ?

feel free to add as PR. Closing then.


fix: resolve the accessibility issues pointed out for header component

Pre-Flight checklist

[x] Screenshots and design review for any changes that affect layout or styling

[x] Desktop screenshots
[x] Mobile width screenshots

[x] Testing

[x] Code is tested
[x] Changes have been manually tested

What are the relevant tickets?
#154
What's this PR do?

Fixes accessibility issues with header
Fixes dropdown button unique id issue highlighted by LightHouse
Fixes area-label texts
Fixes alt for MIT logo in the header

How should this be manually tested?

Check the accessibility on any page and there should be no complaints regarding the header part.

Screenshots (if appropriate)
After

Before

@arslanashraf7 Before you merge, can you change the "What are the relevant tickets?" section from #154 to Closes #154? Please do that for all future PRs too

@arslanashraf7 Before you merge, can you change the "What are the relevant tickets?" section from #154 to Closes #154? Please do that for all future PRs too

@gsidebo I used to do this in the past and I know it will close the associated ticket and move it to Done once the PR is merged. I don't remember the place exactly but I believe there has been some conversation about when we consider a ticket to be Done which is once a ticket has made it to the end-user(production).
However, I can start following that again if that's the standard practice we're still following.
@pdpinch what's your opinion on the above?

I have the same question as @arslanashraf7 raised, Thanks for asking.

Codecov Report

Merging #168 (22dc55a) into main (b06acf2) will increase coverage by 6.75%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #168      +/-   ##
==========================================
+ Coverage   90.54%   97.30%   +6.75%
==========================================
  Files         214      122      -92
  Lines        6156     2780    -3376
  Branches      295        0     -295
==========================================
- Hits         5574     2705    -2869
+ Misses        498       75     -423
+ Partials       84        0      -84

Impacted Files
Coverage Δ

static/js/components/TopAppBar.js
100.00% <ø> (ø)

static/js/components/AnonymousMenu.js
100.00% <100.00%> (ø)

static/js/components/UserMenu.js
100.00% <100.00%> (ø)

courses/api.py

openedx/api.py

openedx/tasks.py

users/urls.py

openedx/exceptions.py

cms/templatetags/feature_img_src.py

cms/urls.py

... and 85 more

Continue to review full report at Codecov.

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


refactoring!(rpc/cmd): rework rpc commands

Overview

reworked p2p/das/header/share/admin/state cmds
extracted cmds to the respective packages in nodebuilder
added rpc flag that allows to specify a path for the node's config
removed deprecated endpoints

Checklist

[ ] New and updated code has appropriate documentation
[ ] New and updated code has new and/or updated testing
[ ] Required CI checks are passing
[ ] Visual proof for any user facing features like CLI or documentation updates
[ ] Linked issues closed with keywords

Codecov Report

Merging #2809 (d8a4e96) into main (8ca777e) will decrease coverage by 0.08%.
Report is 42 commits behind head on main.
The diff coverage is 15.09%.

@@            Coverage Diff             @@
##             main    #2809      +/-   ##
==========================================
- Coverage   51.38%   51.31%   -0.08%
==========================================
  Files         159      167       +8
  Lines       10697    10752      +55
==========================================
+ Hits         5497     5517      +20
- Misses       4722     4754      +32
- Partials      478      481       +3

Files
Coverage Δ

cmd/celestia/bridge.go
100.00% <100.00%> (ø)

cmd/celestia/rpc.go
100.00% <100.00%> (+82.14%)
:arrow_up:

cmd/flags_node.go
0.00% <ø> (ø)

api/gateway/endpoints.go
0.00% <0.00%> (ø)

cmd/celestia/full.go
88.00% <0.00%> (ø)

cmd/celestia/light.go
88.00% <0.00%> (ø)

api/gateway/state.go
0.00% <0.00%> (-9.05%)
:arrow_down:

cmd/auth.go
0.00% <0.00%> (ø)

cmd/rpc.go
0.00% <0.00%> (ø)

cmd/util.go
18.29% <18.29%> (ø)

... and 41 files with indirect coverage changes

oh wait, @vgonkivs can you make this go into rpc-breaks?
we will try to merge rpc-breaks today


Patched Hunspell to Lucene Version 4.10.4

See https://github.com/apache/lucenenet/issues/419#issuecomment-783585364
This also adds a "Lucene compatibility version" to the top of each code file in Lucene.Net.Analysis.Common and Lucene.Net.Tests.Analysis.Common.

This also adds a "Lucene compatibility version" to the top of each code file in Lucene.Net.Analysis.Common and Lucene.Net.Tests.Analysis.Common.

A nice touch that helps a lot.


Top level function for migrations

Pull request checklist

[x] All tests pass. Demo project builds and runs.
[x] I have resolved any merge conflicts.
[x] I have rebased on develop squashed my commits into 1 commit.
[x] I have followed the coding style, and reviewed the contributing guidelines.

This fixes issue #46 .
What's in this pull request?
Added a top level function that performs progressive migrations.
The function is a bit large, it could be broken down but not sure what's your preference @jessesquires .

Thanks @marius-serban ! :smile: :tada:
At first glance, this looks great! Before I review in-depth, let's do some re-organization:

Move new code from CoreDataExtensions.swift to a new file, Migrate.swift
Pull the inner functions out, try to breakup the remaining parts of the function into smaller ones
All helper functions should be internal and tested if possible/if it makes sense, otherwise let's make them private

Thanks @marius-serban ! :smile: :tada:
At first glance, this looks great! Before I review in-depth, let's do some re-organization:

Move new code from CoreDataExtensions.swift to a new file, Migrate.swift
Pull the inner functions out, try to breakup the remaining parts of the function into smaller ones
All helper functions should be internal and tested if possible/if it makes sense, otherwise let's make them private

@marius-serban - Is this ready to go? :smile:

yup! made those changes
--
Marius

On 28 Mar 2016, at 04:23, Jesse Squires<EMAIL_ADDRESS>wrote:
@marius-serban - Is this ready to go?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub

:tada: Thanks so much @marius-serban !
I'll be merging soon. here's the plan:

Release 2.2.1
Merge this and release 3.0 with Swift 2.2

+1

@marius-serban - just hand another look at this after merging. And I'm like:

Please keep contributing! :smile:

hey @marius-serban !
i'm going through PRs to add contributors as collaborators on this project. just sent you an invite! 😄


adapted script to use fixed errai version

MERGE this only after https://github.com/kiegroup/droolsjbpm-build-bootstrap/pull/943 and https://github.com/kiegroup/appformer/pull/673 was merged.

we could eliminate also all errai builds from daily and PR builder jobs when we have it set to fixed version.
Do you want to do it in this PR too or in the follow up PR?

follow up :) - if not I am afraid to loose the "overview"

ok

@mareknovotny btw. in dailyBuilds we won't have any more the errai build (fixed with this PR), the master PRs are copied manually - so they are not generated by an groovy script. Means the two PRs we have for errai we can ignore - or remove them. The next time we branch from master they are not generated and not copied. I think we can leave it now like it is.


Simplify Marker.toString

Description
Marker's toString was unnecessarily long and likely only used for printing debug info in the flowView example. This moves the old implementation to a tree-shakeable free function.

baseline commit on bundle report doesn't seem correct, explaining strange differences. The draft PR I made into main earlier did demonstrate this reduces bundle size by 600b-1kb.


set fuopt-=maxhorz doesn't work anymore in non-native FS

Make macvim set to do non-native fullscreen (NNFS) mode.  Then do mvim -u NONE -U NONE -N then do:
:set co=80
:set fuopt-=maxhorz
:set fu

What should happen is that MacVim enters NNFS, but &co is still 80.  What actually happens is that MacVim acts as if the -=maxhorz didn't happen.
It might be related to one of the additional difficulties from #488.

@chdiza Do you actually use fuopt? I barely remember that option exists and curious what the use cases for this option is. Is it just to focus or avoiding having too large an edit area?

@ychin I used to use &fuopt very frequently, until it broke.  It's been broken for a long time, which is too bad.  I miss this feature.  I can fake an approximation by having one MacVim window in (non-native) fullscreen and a second MacVim window on top of it that isn't in fullscreen, and make sure they have the same colorscheme.
I don't know what others use it for, but I use it so that I can be in (non-native) fullscreen but have a more normal number of columns (e.g., 80).  That way the lines don't get too long to read comfortably, while at the same time I fit as many lines as possible on the screen, and everything is easier visually to deal with because other things are blocked by the rest of the fullscreen-ness.
I don't think I ever made use of the maxvert piece, myself.  (I never had a monitor that was "too tall", even before the unfortunate widescreen revolution.)
Perhaps getting this to work correctly causes too many problems, I have no idea.  It's much more important to have a properly functioning non-native fullscreen than it is to have a half-working &fuopt.
Oh, also I never personally used the background piece of &fuopt.


Fix appimage run for bitwarden

Description of changes
Add  libsecret to appimage-run. The Bitwarden AppImage requires this to function correctly.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

rebase and squash them into one commit.

Done

Commit message don't fix CONTRIBUTION.md

Commit message updated

Result of nixpkgs-review pr 343845 run on x86_64-linux 1

2 packages marked as broken and skipped:

fspy
golden-cheetah-bin

1 package blacklisted:

appimage-run-tests

128 packages built:

altair
alvr
anki-bin
anytype
appimage-run
archipelago
archipelago-minecraft
arduino-ide
badlion-client
bazecor
beekeeper-studio
beeper
betterdiscord-installer
bloomrpc
bootstrap-studio
buttercup-desktop
caido
caprine-bin
chatzone-desktop
chrysalis
cider
clash-nyanpasu
clickup
clockify
code-cursor
codux
cozy-drive
crypto-org-wallet
davinci-resolve
davinci-resolve-studio
dbgate
deskreen
devdocs-desktop
dopamine
electron-mail
electronplayer
everdo
fflogs
firefly-desktop
flexoptix-app
fluent-reader
framesh
gearlever
hamrs
hamsket
heptabase
hifile
hoppscotch
httpie-desktop
immersed-vr
insomnia
irccloud
jan
jbrowse
jetbrains-toolbox
joplin-desktop
kchat
keet
kmeet
lbry
ldtk
ledger-live-desktop
lens
librewolf-bin
listen1
lmstudio
losslesscut-bin
lunar-client
lunatask
marktext
mathpix-snipping-tool
mendeley
miru
mobilecoin-wallet
mockoon
molotov
motrix
mqttx
muffon
museeks
mycrypto
neo4j-desktop
nosql-workbench
notable
notesnook
notion-app-enhanced
nuclear
osu-lazer-bin
p3x-onenote
paperlib
parsify
pinokio
plexamp
polypane
protonup-qt
pureref
quba
railway-wallet
rambox
raven-reader
remnote
requestly
rquickshare
saleae-logic-2
session-desktop
simplex-chat-desktop
snipaste
sonixd
sozi
ssb-patchwork
station
status-im
steam-rom-manager
sylk
timeular
todoist-electron
trezor-suite
tusk
tutanota-desktop
upscayl
via
vial
wootility
wowup-cf
xlights
zecwallet-lite
zettlr
zulip

I ran nix-build -A appimage-run.tests manually.

evaluating derivation 'path:/etc/nixos/.config/home-manager#homeConfigurations."adminy".activationPacerror: builder for '/nix/store/f429rwc897phs97f27f0w0gkrv74103r-libsecret-0.21.4.drv' failed with exit code 1;
       last 10 log lines:
       > 20/24 libsecret:libsecret / test-collection       FAIL            3.96s   killed by signal 5 SIGTRAP
       >
       > Ok:                 23
       > Expected Fail:      0
       > Fail:               1
       > Unexpected Pass:    0
       > Skipped:            0
       > Timeout:            0
       >
       > Full log written to /build/libsecret-0.21.4/build/meson-logs/testlog.txt
       For full logs, run 'nix log /nix/store/f429rwc897phs97f27f0w0gkrv74103r-libsecret-0.21.4.drv'.
error: 1 dependencies of derivation '/nix/store/prw4lp7q7djig3n8383ynqd4527f94la-appimage-run-usr-multi.drv' failed to build
error: 1 dependencies of derivation '/nix/store/spi5fdbybp9lsbwqql6pfy8wll8jv8md-appimage-run-fhs.drv' failed to build
error: 1 dependencies of derivation '/nix/store/cwy8vd8dm59mk336kzz8bfbnvn35nvj2-appimage-run-init.drv' failed to build
error: 1 dependencies of derivation '/nix/store/9r93ys2rhya1yn6zskl6y6cqcx1vpd6b-appimage-run.drv' failed to build
error: 1 dependencies of derivation '/nix/store/b1hh6z7k6hm0q0cv6x256373dsfpbf39-davinci-resolve-19.0.1.drv' failed to build
error: 1 dependencies of derivation '/nix/store/wf8fka6ff0l0vbk6fardaiyvnj5rhx5h-davinci-resolve-19.0.1.drv' failed to build
error: 1 dependencies of derivation '/nix/store/a1dw2x2dgd9igdsfzv9y7q3m0jks0px9-home-manager-path.drv' failed to build
error: 1 dependencies of derivation '/nix/store/qsbgwnqafmnny3lbzlpgq3kvm5i1n5dl-home-manager-generation.drv' failed to build


Exposes core_id in the configuration of an HttpServer

Also if you could also update the Changelog as the new PR template suggests? In section Breaking, as you are adding a new field to an existing struct which is not marked with non_exhaustive.


Raw tag editor - no horizontal scroll bar

No possibility to copy a long link in raw tags editor if it's too long - there is no horizontal scroll bar (but it used to be in the previous versions).

Just to note: there is a horizontal scrollbar on the raw tag editor of the V3 Preview: https://preview.ideditor.com/master/

But I mean normal, current version.

@maro-21 Thanks for reporting, this was definitely a regression.


[de] add missing json strings

This adds the missing json strings for the German language files.

Danke


feat: bump yaml and support complex types

The main work of this PR was actually tracking down where the remaining case mentioned in #6 broke. It turned out that the yaml parser was being overly generic and always parsing into map[any]any, when map[string]any would suffice.
So bumping yaml to v3 and some minor tweaking of the loader are enough to enable pretty much all the missing use cases. Including the one mentioned in https://github.com/alecthomas/kong/issues/145.
I did some testing on real-world projects and didn't seem to break anything. Did I overlook anything?
WDYT?

The lint failure seems to be related to hermit using an old version? Not sure.

@alecthomas friendly ping, just in case this went under the radar :innocent:

It's not passing lint ^^

Oh something Hermit related? Lemme check.

Actually nm I'll just rebase it onto master, should be fine.

awesome! Thanks for taking the time! :pray:


Fixed iOS bug where resend = no body

On iOS, copying the stream a second time resulted in no body because the
stream position would have been advanced. Using ReadAsByteArray prevents
this problem, it's more parallel with Android, and the implementation is
simpler overall.

So I updated to version 2.3, and it still has this issue. If I resend the same HttpRequestMessage (because the last time I sent it it timed out), the content doesn't get sent properly.


Provider of custom RKE2 cluster identified as Unknown on Cluster Management page in Explorer

What kind of request is this (question/bug/enhancement/feature request): bug
Steps to reproduce (least amount of steps as possible):

Deploy custom RKE2 cluster from Cluster Management in Explorer UI
Go to Cluster Management and see Provider type of the newly created cluster

Result:
The provider is detected as Unknown/RKE2 on Cluster Management page:

Other details that may be helpful:

in Ember UI the RKE2 cluster is correctly detected as Rke2
onCluster Dashboard page within Explorer UI the provider is correctly detected as RKE2:

Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): rancher/rancher:master-head 351f7ca
Installation option (single install/HA): any

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported): custom RKE2

https://github.com/rancher/dashboard/issues/2985


Removed django_extensions from example app settings.

Problem
Whilst testing #1329, I added django_extensions to settings.py so that the runscript command could be used during bulk testing.  However, this causes a crash (see issue #1355) if the test.txt dependencies are not used.
Solution

Removed django_extensions from settings.
Updated bulk testing readme.md to clarify that this needs to be installed if testing.

Acceptance Criteria

Tested locally in a new virtualenv.
No new tests necessary.
Ran the existing test suite.

Coverage remained the same at 98.073% when pulling 98739b45083560d2373ebd8c74154ad6bb415b61 on matthewhegarty:remove-django_extensions-from-settings into 1c32c8e446b0c82ffa6109e6c43a5755d37fffdb on django-import-export:main.


view php page in browser xampp and sublime text 3

Hi,
Try to view a .php page in the browser with sublime-text 3 and xampp.
This is my SideBarEnhancements.json:
{
"folders":
[
{
"path": "C:/xampp/htdocs/mywebsite-folder"
}
],
"settings": {
"sublime-view-in-browser": {
"baseUrl": "http://localhost",
"basePath": "C:/xampp/htdocs/mywebsite-folder"
}
}
}
Some help would be much appreciated!
Thanks and regards,
David

What is happening when you go to view your files? Any errors? Is the browser opening? Is XAMPP running?


Including  breaks build

Hello,
I wanted to test some code that uses C++11 regex.  A minimal example:
#include "CppUTest/TestHarness.h"
#include <regex>

int main(int argc, char** argv) {
    return 0;
}

If I comment out the first line, the code compiles. If I keep it, I get these errors:
 % g++ regex.cxx -std=c++11 -o regex
In file included from /usr/include/c++/10/map:60,
                 from /usr/include/c++/10/regex:52,
                 from regex.cxx:2:
/usr/include/c++/10/bits/stl_tree.h: In member function ‘void std::_Rb_tree<_Key, _Val, _KeyOfValue, _Compare, _Alloc>::_M_construct_node(std::_Rb_tree<_Key, _Val, _KeyOfValue, _Compare, _Alloc>::_Link_type, _Args&& ...)’:
/usr/include/c++/10/bits/stl_tree.h:617:14: error: ‘__node’ does not name a type
  617 |        ::new(__node) _Rb_tree_node<_Val>;
      |              ^~~~~~
In file included from /usr/include/c++/10/regex:56,
                 from regex.cxx:2:
/usr/include/c++/10/bits/std_function.h: In static member function ‘static void std::_Function_base::_Base_manager<_Functor>::_M_clone(std::_Any_data&, const std::_Any_data&, std::true_type)’:
/usr/include/c++/10/bits/std_function.h:152:11: error: ‘__dest’ does not name a type
  152 |    ::new (__dest._M_access()) _Functor(__source._M_access<_Functor>());
      |           ^~~~~~
/usr/include/c++/10/bits/std_function.h:152:17: error: expected ‘)’ before ‘.’ token
  152 |    ::new (__dest._M_access()) _Functor(__source._M_access<_Functor>());
      |          ~      ^
      |                 )
/usr/include/c++/10/bits/std_function.h: In static member function ‘static void std::_Function_base::_Base_manager<_Functor>::_M_init_functor(std::_Any_data&, _Functor&&, std::true_type)’:
/usr/include/c++/10/bits/std_function.h:233:11: error: ‘__functor’ does not name a type
  233 |  { ::new (__functor._M_access()) _Functor(std::move(__f)); }
      |           ^~~~~~~~~
/usr/include/c++/10/bits/std_function.h:233:20: error: expected ‘)’ before ‘.’ token
  233 |  { ::new (__functor._M_access()) _Functor(std::move(__f)); }
      |          ~         ^
      |                    )
In file included from /usr/include/c++/10/regex:59,
                 from regex.cxx:2:
/usr/include/c++/10/bits/regex_automaton.h: In constructor ‘std::__detail::_State<_Char_type>::_State(std::__detail::_Opcode)’:
/usr/include/c++/10/bits/regex_automaton.h:136:9: error: expected type-specifier before ‘this’
  136 |    new (this->_M_matcher_storage._M_addr()) _MatcherT();
      |         ^~~~
/usr/include/c++/10/bits/regex_automaton.h:136:9: error: expected ‘)’ before ‘this’
  136 |    new (this->_M_matcher_storage._M_addr()) _MatcherT();
      |        ~^~~~
      |         )
/usr/include/c++/10/bits/regex_automaton.h: In copy constructor ‘std::__detail::_State<_Char_type>::_State(const std::__detail::_State<_Char_type>&)’:
/usr/include/c++/10/bits/regex_automaton.h:142:9: error: expected type-specifier before ‘this’
  142 |    new (this->_M_matcher_storage._M_addr())
      |         ^~~~
/usr/include/c++/10/bits/regex_automaton.h:142:9: error: expected ‘)’ before ‘this’
  142 |    new (this->_M_matcher_storage._M_addr())
      |        ~^~~~
      |         )
/usr/include/c++/10/bits/regex_automaton.h: In constructor ‘std::__detail::_State<_Char_type>::_State(std::__detail::_State<_Char_type>&&)’:
/usr/include/c++/10/bits/regex_automaton.h:149:9: error: expected type-specifier before ‘this’
  149 |    new (this->_M_matcher_storage._M_addr())
      |         ^~~~
/usr/include/c++/10/bits/regex_automaton.h:149:9: error: expected ‘)’ before ‘this’
  149 |    new (this->_M_matcher_storage._M_addr())
      |        ~^~~~
      |         )

Similar with clang++ or other (newer) standards.
That's what I use:
% g++ --version
g++ (Debian 10.3.0-9) 10.3.0
% dpkg -l cpputest
ii  cpputest       4.0-2        all          C/C++ based unit test framework — main package
Can you tell me what's wrong, please?
cu
Markus

The same with <unordered_map> instead of <regex> ...

I tried to use the current git, build with c++20 support instead of the debian version, no improvement.

Try swapping the includes, the CppUTest header should be last.

Well, D'oh!
That worked, thank you! Have been using cpputest on C projects for years, but I didn't know that...


[WIP] Cleanup after all tests

Not much, but as discussed in #100 I've added class tearDown method, which is executed after all tests have passed (or not 😅). Thanks to this little change we create .marathonTests folder only once (and it's lazy), plus, we remove it at the end of running our test suite, so there are no artifacts in user's file system.

Looks like there's a problem with running class func tearDown() in --parallel mode... Strange, cause it does what it should (removes folder), but looks like it can't handle parallelism... 😓


upgrade to babel 6

#17

With this PR, I was able to build react-select without the broken ES6 module output, so it can be consumed from other babel 6 projects that use ES6 module imports

Is this going to be merged ? Would like to use it.

👍

Lovely - just tested this on a project and am getting clean builds. Babel :smile_cat:  6++ :smile_cat:


ImportError: attempted relative import with no known parent package

Traceback (most recent call last):
File "/Users/malcolm/Projects/autoanki/src/autoanki/AutoAnki.py", line 6, in
from .BookCleaner import BookCleaner
ImportError: attempted relative import with no known parent package

This should be fixed in v1.1.9. Thanks!


Update Observable to match latest Observable Proposal

The following signature is what has come back from the TC-39 (per @jhusain):
class Observable<T> {
  constructor(_subscribe:(observer: Observer) => Subscription);
  [Symbol.observer](observer: Observer): void;
  observe(observer: Observer): Subscription;
  subscribe(nextFn: (value:T) => void, errorFn?: (error:any) => void, completeFn?: () => void, startFn?: (subscription: Subscription) => void): Subscription;
  forEach(nextFn: (value: T) => void): Promise;
  last(): Promise;
  static of(...items:Array<any>): Observable;
  static from(observableLike:any): Observable;
}

interface Subscription {
  unsubscribe(): void
}

interface Observer {
  next(value: any): any;
  error(error: any): any;
  complete(): any;
  start(subscription: Subscription): void
}

I think this looks really promising, and it doesn't really constitute a lot of change on our part.

:-1: from me for right now because until it hits stage 2 or stage 3, I think we should stay as is.

for right now

I agree, but I want to keep this on the radar, because it's one of the goals for the library.

Is there not an issue in the current implementation to be resolved? Specifically the inability for a consumer to sync unsubscribe.
JH

On Jul 31, 2015, at 11:18 AM, Ben Lesh<EMAIL_ADDRESS>wrote:
for right now
I agree, but I want to keep this on the radar, because it's one of the goals for the library.
—
Reply to this email directly or view it on GitHub.

Speaking with @trxcllnt and @jhusain ... this change is minimal to what we already have and I think it has high value. Leaving it as a priority for Alpha release.

I think this issue covers some of what we discussed here: #70  ... at least in terms of adding a start method to the Observer interface.

I'm closing this for now, because the es7 observable spec is a moving target that moves just a little too fast. I'm not sure which branch in the spec's repository current work is living in. As soon as @jhusain and @zenparsing let us know when things are settled down (stage 2 perhaps?), we'll try to match the spec. As it stands now, I think we're very close.

@blesh FYI I deleted all of those old branches on the es-observable repo.


Use full target spec for cargo rustc --print --target

cargo::ops::cargo_compile::print() was using the shortname for the --target flag to rustc, but should be using the full target spec;  otherwise a JSON-specified target (e.g. --target /path/to/some-custom-target.json) will cause a rustc error such as
error: Error loading target specification: Could not find specification for target "some-custom-target". Run rustc --print target-list for a list of built-in targets

@bors r+

:pushpin: Commit cb1b2ac4f18a8c87cca5f7e1a582ed1f694ac757 has been approved by weihanglo
It is now in the queue for this repository.

:hourglass: Testing commit cb1b2ac4f18a8c87cca5f7e1a582ed1f694ac757 with merge e6aabe8b3fcf639be3a5bf68e77853bd7b3fa27d...

:sunny: Test successful - checks-actions
Approved by: weihanglo
Pushing e6aabe8b3fcf639be3a5bf68e77853bd7b3fa27d to master...


⚠ Bofrost.at has degraded performance

In ef649f0, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1255 ms

Resolved: Bofrost.at performance has improved in 49eb424 after 8 minutes.


DIP tables with complex data

Make a solution for DIP table blocks with following columns:
id int
desc str
size float cm
points[3] int
1 "Some long text" 23.3 [2,34,4]

implemented in the newest commit


DatePicker does not fire change event

Hi !
I just noticed that no "change" event is fired when a date is selected via the picker. Can please you correct this ?

ng-change is fired when the view value changes from user input - if you want to detect when the date has been selected, I recommend watching the model for changes.


figure filename in series_per_group

The series_per_group function has an option to save the figure. The filename is equal to name of the last plotted Observation. For GroundwaterObs the name is splitted.
https://github.com/ArtesiaWater/hydropandas/blob/b60fb6cff8593ae875382bb5416196a7d9122281/hydropandas/extensions/plots.py#L649
I would like to change that to a more meaning full name. The suggestion is to combine the value of by (i.e. the column name) and the value of it (i.e. the value in the selected group).
HydroPandas 0.11:
for _, group in tqdm(gr, desc="Plotting series per group", total=len(gr)):
[..]
if isinstance(row.obs, GroundwaterObs):
    name = name.split("-")[0]
f.savefig(
    os.path.join(outputdir, f"{name}.png"), bbox_inches="tight", dpi=150
                )

New:
for this_group, group in tqdm(gr, desc="Plotting series per group", total=len(gr)):
[..]
filename = f'by_{by}_plot_{this_group}.png'  # add something for groupby on multiple columns
f.savefig(
  os.path.join(outputdir, f"{filename}.png"), bbox_inches="tight", dpi=150
)

Examples

by (column name)
this_group (value of current group)
filename
result

monitoring_well
GMW000000051273
by_monitoring_well_plot_GMW000000051273.png
meaning full

tube_nr
1
by_tube_nr_plot_1.png
meaning full

x, y
106930_457550
by_x_y_plot_106930_457550.png
Meaning full if you want the groupby x,y. You will get it in your filename.    Not very meaning full, when you want to plot a monitoring well [I think author of current code had that in mind, becuase there is a fixed replace for GroundwaterObs]. Use then column monitoring_well instead of grouping on x, y.

What are your ideas about this update?

Closed by #201


Simplify a >> 31 to 0 for integer a?

When we have val >> 31, with val as int, coming from i.e the x86 SHR instruction
c1 ee 1f           SHR   ESI,0x1f
IDA Pro simplifies it to 0, i.e. discards it (see below). Would it be formally correct to implement in Ghidra?
Here what Ghidra does:
    uVar7 = this->a / 3 + (this->a >> 31);
    iVar12 = (uVar7 >> 31) + uVar7;
    uVar7 = *(int *)&this->b / 3 + (*(int *)&this->b >> 31);
    iVar10 = (uVar7 >> 31) + uVar7;

IDA:
  v1 = this;
  v2 = this->a / 3;
  v3 = this->b / 3;
  v30 = this->a / 3;
  v32 = this->b / 3;

The vars ar all int
EDIT
Done some experiments, it looks like it gives 0 if the number is positive, -1 if negative or if overflows.
So it must be a valid simplification. It would be appliable if var is a positive constant and if unsigned?

you can fix it yourself in ia.sinc
:SHR rm32,imm8  is vexMode=0 & opsize=1 & byte=0xC1; rm32 & check_rm32_dest ... & reg_opcode=5 ... ; imm8
{
local count = imm8 & 0x1f;
if ( count == 0x1f) goto ;
local tmp = rm32;
rm32 = rm32 >> count;
build check_rm32_dest;
shrflags(tmp, rm32,count);
shiftresultflags(rm32,count);

}

Is it correct though? The Intel manual says it is to be set to the LSB (CF ← LSB(DEST)), but I don't know if doing so it prevents optimizations. I've created a PR for this

I think not. It is rather a fix in haste, while we wait for a normal fix from developers.

It should be better implemented with a decompiler Rule, I guess. We need to specify this is valid only for unsigned/positive constants or stuff like that

Done some experiments, it looks like it gives 0 if the number is positive, -1 if negative or if overflows.

This statement is confusing because (with MSB==1 and count==0x1f) SAR is signed divide by 2, imm8 times, would result in -1.  While SHR is unsigned divide by 2, imm8 times, would result in 1.  How are you getting -1?
I don't see how you can just categorically if reduce count=0x1f to result in 0 (in either case).
Also your current fix changes this instruction to a NOP if 0x1f, removing any changes to flags and the destination register

@mumbel yep, probably I've made a mistake, I'm a bit confused. For which range does it work?

A shift of a 4-byte value by 31 bits is definitely not identically zero.  The '>> 31' artefact you are seeing in the original example is more likely something to do with the decompiler's simplification of optimized division, there is some work going on with the rule. We would need to see the bytes of the function to be sure, preferably via the "Debug Function Decompilation" action.

I've dumped two versions of the same function, from a PPC binary and from an x86 one, and they both have that issue. The function is not the one of the reported snipped above, but the (mis)behavior is exactly the same.
dump.zip

Thanks, I'm duplicating this, working on a fix.

A fix for this issue has been committed to master.

@caheckman @ryanmkurtz  Just checking if this got moved over to github or still queued to move, expected changes in decompile/cpp I guess, but if fix was elsewhere sorry for the noise.

GitHub is up to date with master.  Will have to check with @caheckman, but I'm guessing it was done someone near the top of this list of commits.  Maybe closing this ticket just fell through the cracks.

Yup you're right, sorry I had only looked at the 20 days ago


Linux新机器布署提示运行错误

./wei-hardware: error while loading shared libraries: libssl.so.1.1: cannot open shared object file: No such file or directory

add Cargo.toml
[dependencies.openssl-sys]
version = "0.9"
features = ["vendored"]


Test LTS 8 Travis GHCJS build

DON'T MERGE
This is a test pull request, see https://github.com/commercialhaskell/stack/issues/2607

This might not work ;) http://tolysz.org/ghcjs
You need to use a different ghcjs for different resolver

@tolysz Ahh, got it.


loss nan

请问下，我在训练centernet的时候，设置lr为0.0001，batch_size为4，第一个epoch时运行到三分之一不到就显示 loss nan，何故？先谢过！

me too! FCOS is the same..

https://github.com/Stick-To/CenterNet-tensorflow/issues/20
could this help you?

thanks a lot, I'll try it!

@Andy457 did u fix it? I get the same thing and i follow this issue #20 but nothing changes.

@DLH06 yeah,it works for me.


Docs updates

Pre-requisites
Please ensure you have done the following:

[x] I have read the CONTRIBUTING.md document.
[x] If my change requires a change to the documentation, I have updated the documentation accordingly.
[x] I have added tests to cover my changes.

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Describe changes
Docs updates as discussed.

@htahir1 should be better now


Use init generated files, clean up repo

Replaces the contents of the todo folder in this repo with the output of the init command from https://github.com/elastic/synthetics/pull/525
This also adds examples using the request object filling a gap identified in the documentation of that feature (CC @colleenmcginnis )
It also removes the e-commerce demo temporarily till we have steady hosting for it. This blocks #25 as a result.
Finally, this adds inline examples, and removes the heartbeat folder, since as of 8.4 we want to discourage running browser monitors directly with heartbeat.
DO NOT MERGE THIS this until 8.4 is publicly available and beta 32 of the synthetics CLI is out

Skipping POST FF Testing !!


fix: improve embedding documentation to use encoders

Description of change
Updating embedding documentation to prioritise native encoders.
Pull-Request Checklist

[x] Code is up-to-date with the master branch
[x] npm run format to apply eslint formatting
[x] npm run test passes with this change
[x] Documentation has been updated to reflect this change
[x] The new commits and pull request title follow conventions explained in pull request guidelines (PRs that do not follow this convention will not be merged)

Thanks for the PR!
I don't think that a comment inside a code example is the right place to guide on choosing the right embedding model.
I've opened PR #396 to enhance the Choosing a Model guide to include a paragraph about embedding models.


Publish "Angular basics" article

See https://docs.google.com/document/d/1T4s2R_aLefiNWxeC4g-dbEXXfBw_tGw5NdSOS6OCRvM/edit#heading=h.8x0bvrottdip for documentation spec for this article.
Author: Darius
Acceptance criteria

[ ] First/second draft reviewed and sent back with feedback.

[ ] Final draft agreed upon.

[ ] Final draft published.

Author did not deliver, so cancelling this work.


empty response from run()

What is wrong?
Maybe I did not undestand how it work. I try to get a match from boy and girls based on how much is beautiful a people.
Where does it happen?
run() return an empty object
How do we replicate the issue?
var boys = [
    { name: 'diego', beauty: 1},
    { name: 'tommy', beauty: 2},
    { name: 'davide', beauty: 3},
    { name: 'valerio', beauty: 4}
];

var girls = [
    { name: 'lodi', beauty: 1},
    { name: 'marina', beauty: 2},
    { name: 'gloria', beauty: 3},
    { name: 'bea', beauty: 4}
];

var inputs = [
    { boy: boys[0], girl: girls[0] },
    { boy: boys[0], girl: girls[1] },
    { boy: boys[1], girl: girls[0] },
    { boy: boys[1], girl: girls[1] }
];

net.train([
    {input: inputs[0], output: { match: 'no'  } },
    {input: inputs[1], output: { match: 'no'  } },
    {input: inputs[2], output: { match: 'yes' } },
    {input: inputs[3], output: { match: 'yes' } }
]);

var output = net.run({ boy: boys[0], girl: girls[0] });
console.log(output);

How important is this (1-5)?
1
Expected behavior (i.e. solution)
return a object like: { match: 'yes' } instead:
{ match: NaN }
Other Comments

What net are you using for this problem?
Provide jsfiddle if possible.

Convert your strings to numbers. E.g. 'no' = 0

And what you probably will want to do is pass in only the beauty property (normalize it as well)

If you still need help, feel free to re-open this issue.

Has anybody found a solution?


Add extra links to README and improve description

Adds badges to README for latest stable version of documentation and GMD journal article (will need to be updated once the accepted version is available online). Also tweaks the description a little bit to mention thread-based parallelism and add a link to ARC website. I also noticed there was a unit_test.h5 HDF5 file we don't appear to use anywhere that was added in #218, I assumne accidentally, so have removed this.

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

Project coverage is 94.05%. Comparing base (3619473) to head (858a548).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #264      +/-   ##
==========================================
+ Coverage   92.48%   94.05%   +1.56%
==========================================
  Files           7        7
  Lines         386      353      -33
==========================================
- Hits          357      332      -25
+ Misses         29       21       -8

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

Looks like there is some implicit reference to master branch in benchmarks workflow from the fact it fails with
fatal: couldn't find remote ref refs/heads/master

when running
using PkgBenchmark, BenchmarkCI

BenchmarkCI.judge(
    PkgBenchmark.BenchmarkConfig(
        env = Dict(
            "JULIA_NUM_THREADS" => "2",
        ),
    ),
)

but not sure how to fix - @giordano any ideas?


There is no option to add a shape in table cells

Category

[ ] Enhancement

Version
Please specify what version of the library you are using: [   Latest    ]
Please specify what version(s) of PowerPoint you are targeting: [         ]
Expected / Desired Behavior / Question
I need to add a progress bar in one of my table cells, I can achieve that by adding shape(line) to table cell but there is no option in table to add shapes.
Please provide an option to add shapes in a table cell this will provide more flexibility in using this library.
This has been raised by many, so please provide an urgent solution for the same.

You can place a circle on a table, but you cant "insert" it into a cell in PowerPoint table that i'm aware of.

Can you provide some work around for adding progress bar instead of values in table cell ???

You can place a circle on a table, but you cant "insert" it into a cell in PowerPoint table that i'm aware of.

Can you provide some work around for adding progress bar instead of values in table cell ???


Enchancement crash on startup

Mod Name
Enchancement
Mod Homepage
https://modrinth.com/mod/enchancement
Minecraft version
1.21.1
Describe the bug
Crashes on startup
Steps to reproduce

Install Connector, FFAPI, Connector Extras and Enchancement
Launch the game
Crashed
...

Logs
https://mclo.gs/jRMoxda
Additional context
No response

Duplicate of #1068


Store isArchived on repository and show on package page

GraphQL query already contained isArchived so pulled it into the Repository model and then routed through the PackageShow model.
Closes #552
(Also updated the directory of one of the migrations just to have a leading zero - consistency and all that...!)
I feel like archived is not a word any more

LGTM - thanks, James! Will merge after CI passes.

Thanks @Sherlouk! This is a great little feature.


JackOpenError when (re)creating clients with identical name

I have recently noticed some very annoying behavior with the Jack server "clogging up" on client names. The critical application is that a client is created (usually does something) and is destroyed afterwards. This is then repeated for a considerable amount of times with an identical name of the client. Although this seems like a pretty bonkers use case, we actually rely on doing it (in a current implementation at least). Nevertheless, I would expect this to be possible and handled by the server without problems. Right?
I created the a simple script which provokes the behavior.
import jack

i = 0
while True:
    i += 1

    # name = f'Client{i:d}'  # this runs infinitely
    name = f'Client'  # this ALWAYS fails for the 99th instance ???

    print(f'Test {i:d}: creating "{name}" ...')
    client = jack.Client(name=name)
    client.activate()
    client.deactivate()
    client.close()
    del client

Executing the script results in the following trailing output and error:
...
Test 97: creating "Client" ...
Test 98: creating "Client" ...
Test 99: creating "Client" ...
Cannot open Client client
JackShmReadWritePtr1::~JackShmReadWritePtr1 - Init not done for -1, skipping unlock
JackShmReadWritePtr::~JackShmReadWritePtr - Init not done for -1, skipping unlock
JackShmReadWritePtr::~JackShmReadWritePtr - Init not done for -1, skipping unlock
Traceback (most recent call last):
  File "jack_test.py", line 11, in <module>
    client = jack.Client(name=name)
  File ".../jack.py", line 213, in __init__
    raise JackOpenError(name, self._status)
jack.JackOpenError: Error initializing "Client": <jack.Status 0x21: failure, server_error>

The surprising thing (to me at least) is that this fails at the 99th client. And this is always reproduced in the exact same way.
On very annoying aspect is that this client name is now blocked indefinitely. An continues to be blocked even after restarting the Jack server. I did not find a possibility to instantiate a client with the given name again. This is indeed a problem when using tools like ecasound which do not let you determine the instantiated Jack client name (this is also how I encountered the problem). The easiest way to reset the behavior is to log out and in of the OS session (or restart), which is still very disruptive. Is there an easier option to get this reset?

Can somebody else confirm the behavior? My testing was done on macOS 10.15.7, jackd 1.9.16 (via the new installer) and jack-client 0.5.3.
I very much assume that this error has nothing to do with the Jack Python API, since I also encountered this when instantiating ecasound via shell scripts. However, this was the most straight forward way to test the behavior to me. Maybe we can help to resolve the issue upstream by narrowing it down first.
I did not encounter this problem (at least so systematically) before jackd 1.9.16. Maybe the provided binaries are not perfect yet and are causing the issue here. I think testing the latest version, but built locally in the traditional way, would be the first case to check whether it behaves differently. Doing this on my dev system is inconvenient though.
@mgeier do you have a convenient way of testing this on different Jack versions and OS?!

This is reproducible upstream without the Python binding. It's a problem I've faced on Windows since at least 1.9.11.
I reported this at https://github.com/jackaudio/jack2/issues/658 together with a reproducer.

Resolved with https://github.com/jackaudio/jack2/pull/788.


Make question.order a StringType

As much as I want this to be an integer, it is a string from the API

@aroder thanks for the PR! Does this mean that the order can be a non-numeric string or just that the API actually responds with numeric strings instead of integers?

@aroder I've published https://pypi.org/project/tap-jotform/0.0.6/ with the right data type for question.order. Let me know if that solve the same problem. I'm closing this, but feel free to reopen.


Internal KUBERNETES_SERVICE_HOST IP address disclosure in /config endpoint

Affected endpoint:
GET /config
Details:
While the delivery platform is running, the application periodically sends configuration requests (GET /config) and health checks (GET /clusters/main/healthz). While sending a GET /config request application reveals the KUBERNETES_SERVICE_HOST's internal IP address in the responses, intended for internal use within the cluster and backend systems. This exposure could allow attackers, in the event of a compromised pod, to target the system (for example, with DOS/DDOS attacks) or to learn the internal IP addressing schema.

Done.


Make use of part of speech(pos)

Hi, @seominjoon
Do you have try to use the part of speech(pos)?
Because the same word may have different pos, it may increase the performance of bi-att-flow.
If i want to make use of pos besides  word embedding and character embedding, could you give me some advise? Thank you so much.

Hi @rubby33 ,
POS can be helpful, but I have never used it.
probably the best way to do it is create a new input placeholder (in addition to 'x', 'cx', etc.) that accepts these POS tags (you will need to preprocess them of course), and then use it in the model as a word with embedding_lookup.


Set the Transfer-Encoding header instead of adding

Motivation:
HttpUtil.setTransferEncodingChunked could add a second Transfer-Encoding
header if one was already present. While this is technically valid, it
does not appear to be the intent of the method.
Result:
Only one Transfer-Encoding header is present after calling this method.

Cherry-picked into 4.1 (9bec25a6eb491a3aa2b680d098113581bbc29bf1) and 4.0 (2539aa7)

@cconroy thanks !


Add note about $router in navigation

Closes #1136

Thanks @kazupon !
It's so cool you always review many PR 🙂


TabBarIndicator custom color

Current behaviour
The "border" indicating active tab is fixed to:
backgroundColor: '#ffeb3b',

https://github.com/satya164/react-native-tab-view/blob/main/src/TabBarIndicator.tsx
Expected behaviour
I would like to customize it,
preferably on TabBar level
Code sample
<TabBar
    {...props}
    style={{backgroundColor: bgColor}}
    tabBarIndicatorStyle={yourStyle}
    ...
    }}
  />

What have you tried
Checking docs and code
Your Environment

software
version

ios or android
android

expo
no

react-native
0.63.3

react-native-tab-view
2.15.2

react-native-gesture-handler
1.9.0

react-native-reanimated
2.0.0-alpha.9.2

node
14.15

npm or yarn
6.14

https://github.com/satya164/react-native-tab-view#indicatorstyle


Align with templates

Not strictly necessary, but will make the examples more uniform.

i'm pretty sure that @pat-s created a ropensci repo for tic.covrpage, which is more current than this one.

Got you. Thanks for the heads up.


Include native artifacts for unit testing

Strange, that fix worked locally but CI still have the same issue: no jnitensorflow in java.library.path
@saudet , any tip how we can fix this? including tensorflow-core-platform in the test scope should allow us to run unit tests using tensorflow, no?
CC: @dhruvrajan

Yeah, that gets a bit tricky. We need to tweak this a bit to make it work correctly.
I've added the proper changes to pull #54.

Changed were pushed in #54


Add enforcement of minimum # of idle VMs to HTCondor autoscaler

Enable the HTCondor autoscaler to maintain a minimum number of idle HTCondor execute points by tracking the machines known to be idle by HTCondor itself and the number of machines still being created by the Managed Instance Group.
Migrate all per-MIG settings to a SystemD override file that is loaded when the MIG autoscaler is triggered

Submission Checklist

[x] Have you installed and run this change against pre-commit? (pre-commit install)
[x] Are all tests passing? (make tests)
[ ] Have you written unit tests to cover this change?
[x] Is unit test coverage still above 80%?
[x] Have you updated all applicable documentation?
[x] Have you followed the guidelines in our Contributing document?

@nick-stroud I have discovered a bug in the method by which idle nodes are calculated. I'm resolving it and will update this branch when I have the fix.


Task not found case not explicitly handled in CFE_ES_RunExceptionScan

Is your feature request related to a problem? Please describe.
Not clear if task not found case is handled correctly:
https://github.com/nasa/cFE/blob/e80aae94e0f56b868657daba965c590766a4dc57/modules/es/fsw/src/cfe_es_erlog.c#L360-L362
Describe the solution you'd like
Analyze and explicitly handle if needed or document why not needed for future reference
Describe alternatives you've considered
None
Additional context
Code Review
Requester Info
Jacob Hageman - NASA/GSFC

Checked this, its fine as-is.
The default behavior after exception is to do a restart of some form  (processor or poweron).  Only if the exception is specific to a certain task and the app owning that task was configured with "RESTART_APP" option, then only that app will be restarted instead of the whole system.
So in this case if the task was not found it will just fall through as intended to do the full processor restart.
Most likely if this occurs its because data structures are out of sync, which in an exception handling context is most likely because the data structures were possibly clobbered as part of whatever went wrong, before the RTOS actually caught the exception and stopped the task.  This can have far reaching effects/unpredictable behavior, so the best bet is to restart ASAP.
I suppose we could add a SysLog if that happens, but even that involves taking a lock which if the data structures are corrupted, could deadlock.  All hypothetical though....

Recommend a comment to document this intended behavior.


Add git pull & git submodule update --checkout

Adds an additional guideline for people having the repo already cloned.
Further, the PR tries to mitigate the risk of failed reproducible builds due to wrong commit checked out at the drongo submodule, see problem description https://github.com/sparrowwallet/sparrow/issues/619#issuecomment-1228697555 or https://stackoverflow.com/questions/1899792/why-is-git-submodule-not-updated-automatically-on-git-checkout
For reference of  git submodule update --checkout see https://stackoverflow.com/a/41920893
For a permanent solution on the users side https://stackoverflow.com/questions/4611512/is-there-a-way-to-make-git-pull-automatically-update-submodules provides additional hints.

Thanks - this is a good improvement!


[Improvement] Use kebab-case syntax instead of camelCase for emitted events

Is your feature request related to a problem? Please describe
Vue JS conventions: It is recommended that you always use kebab-case to name custom events. It is not recommended to use camelCase to name events.
<ChatWindow
   @fetchMessages="doSomething"
/>

CF second point
Describe the solution you'd like
Use
this.$emit('fetch-messages', data)

So:
<ChatWindow
   @fetch-messages="doSomething"
/>

Yep, that's already on my todo list

Fixed https://github.com/antoine92190/vue-advanced-chat/commit/26861535298aa8cca444dc301924f7fc35c2b17a


Not compatible with 39.1

I'm on a older version of sketch and I found that 2.5 is not compatible with Sketch 39.1. I am reverting back to 2.4 for now.

@utom It's likely that this issue can be safely closed now.


Support for standard dnd5e rolltables

In most of the dnd5e books as well as Adventurer's League adventures tables are in this format (at least, this is how it looks when copied and pasted):
Alien Devices
d8 Device
1 A hollow cylinder with a purple crystal embedded inside
and a sliding switch. When activated, it emits a 3-footlong beam of warm light for a fraction of a second
before shorting out.
2 A fist-sized metal orb with three jagged blades
fixed to one side. It vibrates with a soft humming
when touched.
3 A green glass statuette of a dragonborn woman in
robes, with a tentacle where her head should be. There’s
a place to insert a Far Realm battery crystal, but the
statuette has no obvious function.
4 A seven-inch-wide, green metal disk with a handle. The
disk has a magnet-like attraction to stone, but can’t
support more than half a pound of weight.
5 A lidded box with a crank. A number of blades inside
the box whirl and chop when the crank is turned.
6 A blank book made of thin sheets of the glassy, green
stone. A dragonborn with faceted eyes is embossed on
the cover.
7 A coil of copper wire wrapped around a metal rod with
an upside-down bell at one end and two prongs at
the other.
8 A thin box embedded with pieces of colored glass and a
dozen buttons. The box fits easily in one hand, but has
no discernable function.

It would be nice to be able to copy and paste these directly in and have them imported correctly to lower prep time.

Oh, and sometimes it says 1-10 at the beginning instead of a single number

Thanks for this - I'll take a look at adding it!

Feature should be added - closing this issue, open another one if you have an example that isn't working. Thanks so much for opening the PR!


Need some help hiding the Achievements section

Hello.
I am trying to hide some sections on Steam and I managed to hide almost all of the ones I wanted to, except for the Achievements section. I followed your guidance on using the dev mode to highlight the section and change the appropriate CSS, but I wasn't able to find the code to put display: none to. I am not familiar with programming in general so I was hoping to turn to you for advice. Thank you making this awesome tool, my experience on Steam would not be nearly as pleasant without it.

I believe off the top of my head that if you open up:

SteamUI-Oldglory - >

CSS Options - >

CSS Sections - > (It's the top right!)

achievements_page (disable this!)

That should hide achievements without need for hand-editing. I can't test right now, sorry!

Hrmn...
I think I was a dummy, that just tells OldGlory not to import the separate achievements.css rather then exclude it entirely.
Apologies, juggling a bit today. Re-check that box, and let me mess around for a bit...
It looks like none of the settings currently available effect achievement visibility. Aggravating. I'll do my best to figure something out this weekend.

@Fizzboks I've tried every permutation of achievements_container I can find, and steam just doesn't want to hide the achievements segment. Unless there's something I'm missing that @Jonius7 knows about, I'm afraid you're stuck with them. I'm very sorry, and wish I had better news for you.

This should work for most if not all pages, since the Achievements section is usually div no.4
.appdetailsoverview_SeekTarget_vzLed:nth-of-type(4) {
    display: none;
}


Not working with HTC Vive

Currently I have a HTC Vive but it wont work. I try the HMD Track example and it says, "Exception has occurred: FormFactorUnavailableError". The other two examples give assertion errors. Also, when will pyglet be supported and do you have a discord server for this project? Thanks, Drake.

Did a little more digging and as it turns out its trying to use my disconnected HP headset and not my stream vr one. Is there a way to get different headsets?

I don't know how to change the active openxr runtime when you have more than one. If you figure it out please let us know.

Fixed it! I was just playing an openxr game and it had the same issue. I looked it up and apparently I never changed the openxr environment back to steamvr in the steamvr dev settings.


Alert manager stops responding under production load

Hi,
We've recently migrated to the new AM (0.1.0) and the service stops responding under the load.
The load at hand is about 2500 firing alerts.
What we're seeing is the alerts endpoint does not ever load and after a while the service itself stops sending alerts.
Nothing is visible in the logs.
Before we revert to the previous version what can we do to debug and learn here.
Thx

I don't think we can expect sqlite to handle any bigger load. I've seen similar issues in our setup. I'd love to have a plugable database backend, so that users can choose to use PostgreSQL, MySQL, etc.
The other issue is the amount of data we sent to the frontend, that needs to be capped.

That's roughly were I expected things to break down. SQLite is certainly no long term solution. Neither is any other *SQL in my opinion to be honest. I also don't want to get into the business of multiple backends. Especially not at this stage, but probably never. It comes with a lot of surface for bugs and had to predict variance in performance.
SQL also isn't a good fit for the general concerns we have and having well working storage seems superior to plugging a complex cache-everything layer in front of a database.
Currently investigating some options.
For now running multiple AMs that are fed by different Prometheus servers could be an option (of course no final solution).

All of the above options pointing for us reverting back,
Mainly because even just on  Prometheus server can generate 700 alerts and im not sure it will handle that.
Also going multiple AMs in our setup means i need to manage silence, and state across to many servers.
What other options were talking about that can be leveraged in the short term ?
Many thanks

Having a backend for alertmanager alerts would be nice. Especially if you're running it on Kubernetes in ephemeral pods.
Can we add an isolation layer of interfaces for the backend?

There's a simple set of interfaces abstracting the storage layer already: https://github.com/prometheus/alertmanager/blob/master/provider/provider.go
Something that implements these could replace SQLite.
Generally, I'm very much aware of the issue and working on a replacement as I find time but very wary to not throw in the next thing that "works for now".


Fix image reference for GitHub

@Springstone,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


Uncaught Error: Cannot read property 'object' of undefined TypeError

I encounter an error this morning and I thought its my responsibility to let community know about errors to help make better.
Below is the details of error.
If you have any questions feel free to comment.
Atom Version: 1.10.2
Electron Version: 0.37.8
System: Mac OS X 10.11.6
Thrown From: Atom Core
Stack Trace
Uncaught Error: Cannot read property 'object' of undefined
TypeError: Cannot read property 'object' of undefined
at ObjectsRegistry.get (/Applications/Atom.app/Contents/Resources/electron.asar/browser/objects-registry.js:44:28)
at EventEmitter. (/Applications/Atom.app/Contents/Resources/electron.asar/browser/rpc-server.js:309:31)
at emitMany (events.js:121:13)
at EventEmitter.emit (events.js:195:7)
at EventEmitter. (/Applications/Atom.app/Contents/Resources/electron.asar/browser/api/web-contents.js:141:25)
at emitTwo (events.js:100:13)
at EventEmitter.emit (events.js:185:7)
At /Applications/Atom.app/Contents/Resources/electron.asar/renderer/api/remote.js:178

Error: Cannot read property 'object' of undefined
TypeError: Cannot read property 'object' of undefined
    at ObjectsRegistry.get (/Applications/Atom.app/Contents/Resources/electron.asar/browser/objects-registry.js:44:28)
    at EventEmitter.<anonymous> (/Applications/Atom.app/Contents/Resources/electron.asar/browser/rpc-server.js:309:31)
    at emitMany (events.js:121:13)
    at EventEmitter.emit (events.js:195:7)
    at EventEmitter.<anonymous> (/Applications/Atom.app/Contents/Resources/electron.asar/browser/api/web-contents.js:141:25)
    at emitTwo (events.js:100:13)
    at EventEmitter.emit (events.js:185:7)
    at metaToValue (/Applications/Atom.app/Contents/Resources/electron.asar/renderer/api/remote.js:178:13)
    at undefined.remoteMemberFunction (/Applications/Atom.app/Contents/Resources/electron.asar/renderer/api/remote.js:112:18)
    at AtomEnvironment.module.exports.AtomEnvironment.getWindowDimensions (/Applications/Atom.app/Contents/Resources/app.asar/src/atom-environment.js:636:29)
    at AtomEnvironment.module.exports.AtomEnvironment.storeWindowDimensions (/Applications/Atom.app/Contents/Resources/app.asar/src/atom-environment.js:670:36)
    at WindowEventHandler.module.exports.WindowEventHandler.handleWindowBlur (/Applications/Atom.app/Contents/Resources/app.asar/src/window-event-handler.js:182:35)
    at /Applications/Atom.app/Contents/Resources/app.asar/src/window-event-handler.js:3:61

Commands
     -1:53 editor:newline (atom-text-editor.editor.is-focused)
  7x -1:51.1.0 core:save (atom-text-editor.editor.is-focused.autocomplete-active)
     -0:50.9.0 editor:select-to-first-character-of-line (atom-text-editor.editor.is-focused)
     -0:50.4.0 core:copy (atom-text-editor.editor.is-focused)
  2x -0:50.1.0 core:save (atom-text-editor.editor.is-focused)
     -0:42 core:copy (atom-text-editor.editor.is-focused)
     -0:41.7.0 core:save (atom-text-editor.editor.is-focused)
     -0:38.7.0 core:move-down (atom-text-editor.editor.is-focused)
     -0:38.4.0 core:move-up (atom-text-editor.editor.is-focused)
  3x -0:38 core:save (atom-text-editor.editor.is-focused)

Config
{
  "editor": {
    "fontSize": 15,
    "tabType": "soft"
  }
}

Installed Packages
# User
atom-beautify, v0.29.9 (active)
integrated-learn-environment, v1.0.0 (inactive)
ruby-slim, v0.2.0 (active)
terminal-plus, v0.14.5 (active)
atom-dark-syntax, v0.27.0 (inactive)
atom-dark-ui, v0.52.0 (inactive)
atom-light-syntax, v0.28.0 (inactive)
atom-light-ui, v0.44.0 (inactive)
base16-tomorrow-dark-theme, v1.1.0 (inactive)
base16-tomorrow-light-theme, v1.1.1 (inactive)
one-dark-ui, v1.5.0 (active)
one-light-ui, v1.5.0 (inactive)
one-dark-syntax, v1.3.0 (active)
one-light-syntax, v1.3.0 (inactive)
solarized-dark-syntax, v1.0.2 (inactive)
solarized-light-syntax, v1.0.2 (inactive)
about, v1.5.3 (active)
archive-view, v0.61.1 (active)
autocomplete-atom-api, v0.10.0 (active)
autocomplete-css, v0.11.2 (active)
autocomplete-html, v0.7.2 (active)
autocomplete-plus, v2.31.1 (active)
autocomplete-snippets, v1.11.0 (active)
autoflow, v0.27.0 (inactive)
autosave, v0.23.1 (active)
background-tips, v0.26.1 (active)
bookmarks, v0.41.0 (active)
bracket-matcher, v0.82.1 (active)
command-palette, v0.38.0 (inactive)
deprecation-cop, v0.54.1 (active)
dev-live-reload, v0.47.0 (active)
encoding-selector, v0.22.0 (active)
exception-reporting, v0.39.0 (active)
find-and-replace, v0.201.0 (active)
fuzzy-finder, v1.3.0 (active)
git-diff, v1.1.0 (active)
go-to-line, v0.31.0 (inactive)
grammar-selector, v0.48.1 (active)
image-view, v0.58.2 (active)
incompatible-packages, v0.26.1 (active)
keybinding-resolver, v0.35.0 (active)
line-ending-selector, v0.5.0 (active)
link, v0.31.1 (inactive)
markdown-preview, v0.158.0 (active)
metrics, v0.53.1 (active)
notifications, v0.65.0 (active)
open-on-github, v1.2.0 (inactive)
package-generator, v1.0.0 (inactive)
settings-view, v0.242.2 (active)
snippets, v1.0.2 (active)
spell-check, v0.67.1 (active)
status-bar, v1.4.1 (active)
styleguide, v0.47.0 (active)
symbols-view, v0.113.0 (inactive)
tabs, v0.100.2 (active)
timecop, v0.33.2 (active)
tree-view, v0.208.2 (active)
update-package-dependencies, v0.10.0 (active)
welcome, v0.34.0 (active)
whitespace, v0.33.0 (active)
wrap-guide, v0.38.1 (active)
language-c, v0.52.1 (active)
language-clojure, v0.21.0 (active)
language-coffee-script, v0.47.2 (active)
language-csharp, v0.12.1 (active)
language-css, v0.37.1 (active)
language-gfm, v0.88.0 (active)
language-git, v0.15.0 (active)
language-go, v0.42.1 (active)
language-html, v0.45.1 (active)
language-hyperlink, v0.16.0 (active)
language-java, v0.23.0 (active)
language-javascript, v0.119.0 (active)
language-json, v0.18.2 (active)
language-less, v0.29.5 (active)
language-make, v0.22.2 (active)
language-mustache, v0.13.0 (active)
language-objective-c, v0.15.1 (active)
language-perl, v0.35.0 (active)
language-php, v0.37.2 (active)
language-property-list, v0.8.0 (active)
language-python, v0.45.0 (active)
language-ruby, v0.69.0 (active)
language-ruby-on-rails, v0.25.0 (active)
language-sass, v0.56.0 (active)
language-shellscript, v0.22.4 (active)
language-source, v0.9.0 (active)
language-sql, v0.23.0 (active)
language-text, v0.7.1 (active)
language-todo, v0.28.0 (active)
language-toml, v0.18.0 (active)
language-xml, v0.34.9 (active)
language-yaml, v0.26.0 (active)

# Dev
No dev packages

This is a duplicate of #11692 - feel free to subscribe there for updates.

Ok


LowLatencyMrcSample only works on Debug mode.

Hi,
When I run the sample codes, it only works on Debug mode.
Please help me to fix the problem.
Thanks,
Woon

To help understand my case, I correctly deploy the hologram on HoloLens. Once I run the Viewer , I tried to connect to HoloLens. If I set the LowLatency sample on the debug mode of VS 2015, I works fine. But on release mode, the viewer fails to connect to HoloLens and then closed the window because of a connection problem to HoloLens.
Thanks,
Woon

there shouldn't be a difference between debug/release other than timing. Is there a timeout trying to connect or something else?
we have seen intermittent connection issues to the device. Restarting the device and starting the lowlatency app corrects that. Waht type of network wifi hardware(AC/bandwidth) are you using? Are both the viewer machine and HoloLens using wifi? x86 device and x86 viewer builds?

I think the speed of my network causes this problem. Now it works well.
Thanks Carmines,
Woon


[#140] Fixed parsing request body when path expression points to a si…

…ngle item instead of multiple items.

I can see tests are failing ... are this related to the change?

I can't see which tests are failing. How to check it?

Check the ci here: https://travis-ci.org/castlemock/castlemock/builds/594636819?utm_source=github_status&utm_medium=notification

Hi @mbilinainteca.
The problem here is most likely not due to @mbilinainteca commit. From what I can see, Travis CI seems to not support Oracle JDK 8 and 9. I working on some reconstructions right now for the project and will resolve this issue at the same time.
I will take a look at your contribution as soon as I can.
Best regards,
Karl Dahlgren

Hi again @mbilinainteca.
Just took a quick look. The commit looks great! I can merge your pull request as soon as the contribution license has been signed. All you need to do is to sign the following license contribution agreement and send it to us: https://www.apache.org/licenses/icla.pdf
Once this is done, we can merge your request.
Thank you very much!
Best regards,
Karl Dahlgren


Get-TssSecret -ForceCheckIn does not check secret in

Verified issue does not already exist?
Yes
What error did you receive
No error.
Please run the command using -Verbose
VERBOSE: Command invocation: Get-TssSecret -TssSession:TssSessionObject -Id:System.Int32[] -ForceCheckIn:True -Verbose:True
VERBOSE: Performing the operation POST https://xxx.xxx.xxx/api/v1/secrets/xxx/restricted with: {
"forceCheckIn":  true
}
Provide a test case or steps to reproduce
Get-TssSecret -TssSession $session -Id xxx -ForceCheckIn
Expected behavior
This should check in and check out a secret. Expected behavior is that this will provide a new secret. Using Close-TssSecret in conjunction with Get-TssSecret provides the expected behavior with a refreshed secret.
What Edition of Secret Server?
Other
What version of Secret Server
11.0
What PowerShell host was used when producing this error
Windows PowerShell (powershell)
PowerShell Host Version
Name                           Value

PSVersion                      5.1.19041.1320
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.19041.1320
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

@GnXeyTfY7zBWhzUw we just released a new version of the plug-in, can you confirm if this issue still exists?


[v2.7] Update Nutanix node driver to version 3.6.0 / rancher 2.7

Update Nutanix node driver to version 3.6.0 for rancher 2.7

Ticket #44983 - Test Results - ✅
Verified on Rancher v2.7-3def2db5944369c53249e7fe44f661734a625db4-head:

Fresh install of Rancher v2.7-head
Navigate to Cluster Management > Drivers > Node Drivers, and observe nutanix driver
Verified - v3.6.0 is available
Active nutanix driver
Verified - driver successfully activates and is available for use

Screenshot:

i just check with v2.7-head (bd0bb77) and driver is always in 3.4
@snasovich @Josh-Diamond

@tuxtof , latest v2.7-head images are no longer published to DockerHub and are only available from Rancher Prime registry.
https://github.com/rancher/rancher/commit/bd0bb772b60a447d352789b3d7c4368ffe50eecf is before https://github.com/rancher/rancher/pull/45003 was merged.

OK i don't have access to this registry


getUrl removes template markers

First of all, really great library!
I'm currently encountering an issue when using the getUrl function. Apparently the result that is passed to the callback, has all template markers removed.
So e.g. when the final URL should be http://something.tld/books{?page,size,sort,projection}, the url that is passed to the callback is http://something.tld/books.
For my usecase I'd actually like to have the template markers.
Is there a specific reason for this? Or maybe did I do something wrong?

The URI template processing happens before the URL is returned to the callback. The reason is kind of "that's just the way it works". If you know the proper substitions for the URI template parameters ahead of time you can just use Traverson's withTemplateParameters method and let Traverson handle the substitution for you.
If you want the full template and resolve it yourself later then I am afraid that there is no way to do this currently. I'm open to suggestions for an API that would allow that.

Thanks for the quick response!
Oh I actually missed that getUrl also works with withTemplateParamters. That actually works for my usecase, so there's no need for a change.
Thanks again!

I added a clarification to the user guide and the API docs for this.


How to stop using /tmp for optimisation process?

It seems that imagemin uses /tmp as temporary directory. Do we have a way to tel imagemin to make in-memory optimization? or at least to use an alternative location like /dev/shm (on GNU/Linux)?

Whether /tmp is memory supported tmpfs or actual file system depends on the OS/distro.
On ArchLinux /tmp has long been tmpfs.


Disable hex input auto completing

Description
If you type characters directly into the hex input field, it will autocomplete. For example if I type 123 it will autocomplete the field by setting the value to 112233. When you try and delete some characters to change the entry it just continues to autocomplete them. Do not run the setValue() function unless the field loses focus OR there are at least 6 characters in the field. This is driving users crazy. I think you fixed this in your previous version here: https://github.com/zefoy/ngx-color-picker/issues/84. Version 3.2 doesn't have the same fix.

Context

bootstrap-colorpicker version: 3.2
bootstrap version: 4
jQuery version: jQuery v3.5.1
Browser name and version: Firefox
Operative System name an version: Mac OsX

Live Example
You can see this in action on the example page for the plugin - try clearing out the hex value in the "Add complementary color swatches" example and start typing any number or letter and see what it does:
https://itsjavi.com/bootstrap-colorpicker/tutorial-p02_Advanced_Examples.html

Same issue here

Have you tried
...
$('#colorpicker').colorpicker({
  autoInputFallback: false
});
...

To be fair it still doesn't work for me, but you can try this setting, maybe it will somehow work for you..

I've fixed this Issue for the hex-format in the update-function (Line 4653):
var originalColor = this.colorpicker.color._original.color;
if (originalColor.length < 7 && originalColor.startsWith('#')) {
  return;
}

That's how the color parsing library works. I think there is a PR from @erehulka that will solve this with an option.
thanks for taking care of that.


Refactor how signatures/trampolines are stored in Store

This commit refactors where trampolines and signature information is
stored within a Store, namely moving them from
wasmtime_runtime::Instance instead to Store itself. The goal here is
to remove an allocation inside of an Instance and make them a bit
cheaper to create. Additionally this should open up future possibilities
like not creating duplicate trampolines for signatures already in the
Store when using Func::new.

@fitzgen would you be able to take a look at this?