Add a basic MockServer integration test for Python

Reproducer for #3533
and cleaning the camp, since I'm here 🙂

I think something went wrong during the rebase, the history contains a lot of commits and there's currently a conflict.

yup, fixed now 👍 I'm targeting the bugfix/snake-case-properties branch with this PR, so that they can be merged together to main.
I hope this is fine for you and @samwelkanda

HOLD merge

@baywet I changed the target branch of this PR to be the one in #3524 , it's safe to be merged IMHO, should I keep it targeting main instead?

Sounds good, thanks for explaining! 👍

@samwelkanda @baywet I think that something went wrong here ...this should have been rebased, should I open it again?

interesting, the behaviour between PRs from the same repo and PRs across forks is different. Sorry about the mess here. Yes please do

There is no "button" to re-open it from UI, I'll open a new PR from a different branch if this works for you


pull request for e2e test at 12/16/2016 02:15:20

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


.Net 7

When do plan to release for .Net 7 resp. .Net 8? Thank you.

The library works fine for .NET 7 and .NET 8, you should be able to install it without any problems.
The target frameworks are more like "minimum versions".

Closing this issue. Feel free to open a new one in case you have more information (e.g. errors you're experiencing).


(feat): [ENGEN-638] User docs on how to build docker images that contain Kong gateway

We expect slim images to resolve most of the issues around security scans that pick up non-executable dependencies in our docker images. However, some users will need or want to build Docker images that contain Kong Gateway using their own base images.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

This document is okay for now.
Its existence fulfills the spirit of the original request: a document that directs customers on how to use/customize our Dockerfiles while we introduce slimmer images.
There's 4 things I would want changed in the future:

some information on how to get sha256s for use here (hopefully available on docs or download site)
no literal Dockerfile contents in the doc (I know this was brought up but the likelihood for bit rot feels too high)
cross-references to & from this document to docs.konghq.com's Install Kong Gateway on Docker page
references for how to use customize to create a "full" image: https://github.com/Kong/docker-kong/tree/master/customize


GraphQL Response Caching  throws error : options.session is not a function

Response cache plugin is throwing an error.
Below is the error seen :
{
"errors": [
{
"message": "options.session is not a function"
}
]
}
The error can be reproduced in the example sandbox editor (the same shared in the document page)
https://codesandbox.io/p/devbox/kind-leaf-sc5f59?embed=1&file=%2Fexample-query.graphql%3A1%2C1-12%2C1

Thanks for the reproduction.
Could you try with this alpha version?
@graphql-mesh/plugin-response-cache | 0.99.8-alpha-20240806131758-1cc296717c61d0b8eec150b84d500ca1d637c932

The above one was not working, but with the latest alpha build it is working.
@graphql-mesh/plugin-response-cache | 0.100.0-alpha-20240807040201-338274ed9f5e522ced3b57bb0f3cd9c5874483d9


Is there a way to clear the input value? I DO NOT mean clearing value on select.

So I know how to clear the input value on item select, but is there a way how to do it as a reaction for some "form clear" event?

Hey @sauspi, assuming you are holding the value of the input in the state object, you should be able to simply clear the state from an event handler. There should be no need to interact with the react-autosuggest api to accomplish this. Here's a code pen that should demonstrate what you're wanting.

Clearing the value in the state fixed it for me.


anchor the rover to the right edge so it works on tablet too

banged my head against this for a while, and then discovered that anchoring to the right edge instead of the left is a very easy fix that (i think) looks indistinguishably great.

phone should be unchanged

@cjb isPhone should work in platform styles as of https://github.com/keybase/client/pull/22425


Feature/usb head cam

includes https://github.com/ipa320/cob_robots/pull/497

we can close this one as soon as #497 is merged


change: [M3-7232] Rename isPropValid and update codebase & documentation

Description 📝
In our styled components, we have the ability to prevent the forwarding of custom props that could end up being passed as an invalid html attribute in the dom. To that effect we have a utility to use in shouldForwardProp that was named isPropValid but this name caused a bit of confusion since it made some assumption about the validity of the prop itself, while its intent is to only build an array of props to exclude from forwarding regardless of their semantic validity.
Note: the omittedProps name has been discussed and vetted with @jaalah-akamai
Changes  🔄

Rename isPropValid to omittedProps
Add unit test to the utility
Update imports and instances with new name through the codebase
Update documentation

Preview 📷
There is no visual change brought by this PR, nor should there be any functionality change
How to test 🧪
Prerequisites

Pull and run code locally

Verification steps

Verify there's no regression with the forwarding of our props
Confirm documentation reflects our new standards and the usage of omittedProps

As an Author I have considered 🤔
Check all that apply

[x] 👀 Doing a self review
[x] ❔ Our contribution guidelines
[x] 🤏 Splitting feature into small PRs
[x] ➕ Adding a changeset
[x] 🧪 Providing/Improving test coverage
[x] 🔐 Removing all sensitive information from the code and PR description
[x] 🚩 Using a feature flag to protect the release
[x] 👣 Providing comprehensive reproduction steps
[x] 📑 Providing or updating our documentation
[x] 🕛 Scheduling a pair reviewing session
[x] 📱 Providing mobile support
[x] ♿  Providing accessibility support

Commit message and pull request title format standards

Note: Remove this section before opening the pull request
Make sure your PR title and commit message on squash and merge are as shown below

<commit type>: [JIRA-ticket-number] - <description>
Commit Types:

feat: New feature for the user (not a part of the code, or ci, ...).
fix: Bugfix for the user (not a fix to build something, ...).
change: Modifying an existing visual UI instance. Such as a component or a feature.
refactor: Restructuring existing code without changing its external behavior or visual UI. Typically to improve readability, maintainability, and performance.
test: New tests or changes to existing tests. Does not change the production code.
upcoming: A new feature that is in progress, not visible to users yet, and usually behind a feature flag.

Example: feat: [M3-1234] - Allow user to view their login history

Just a heads up, I merged in the MUIv5 StackScripts pt1 PR, and it uses the old 'isPropValid' in this files:

StackScriptTableHead.styles.ts

Pt2 also uses isPropValid, but if your PR gets merged in first, I'll just resolve those conflicts on my end :D

@coliu-akamai thx for the heads up! for pt2, may the fastest merge win


Fix excess space before dot, change "-" to "--"

Due to the content block spanning three lines in the if hasLong condition, there is excess whitespace present when no description is given.
Moreover, the single dash "-" does look a bit too tiny so I extended it to "---".
Before:

Now:

will include this in futur release


sumo-bandstats

Hi,
what is the function of sumo-bandstats -s
I read that the default is samples 3 k-points when determining the effective masses. what does it mean?  if I change that number I got different effective mass values.
Best regards,
ibra

Also, for the example:
Hole effective masses:
m_h: -0.641 | band 34 | [0.00, 0.00, 0.00] (\Gamma) -> [0.50, 0.50, 0.50] (L)
m_h: -0.516 | band 34 | [0.00, 0.00, 0.00] (\Gamma) -> [0.50, 0.00, 0.50] (X)
m_h: -0.641 | band 35 | [0.00, 0.00, 0.00] (\Gamma) -> [0.50, 0.50, 0.50] (L)
m_h: -0.516 | band 35 | [0.00, 0.00, 0.00] (\Gamma) -> [0.50, 0.00, 0.50] (X)
m_h: -1.234 | band 36 | [0.00, 0.00, 0.00] (\Gamma) -> [0.50, 0.50, 0.50] (L)
m_h: -15.158 | band 36 | [0.00, 0.00, 0.00] (\Gamma) -> [0.50, 0.00, 0.50] (X)
Electron effective masses:
m_e: 0.207 | band 37 | [0.00, 0.00, 0.00] (\Gamma) -> [0.50, 0.50, 0.50] (L)
m_e: 0.211 | band 37 | [0.00, 0.00, 0.00] (\Gamma) -> [0.50, 0.00, 0.50] (X)
There are degenerate bands for the hole with different masses and paths (G-L and G-X).
Which band and path could be selected?
Best regards,
Ibra

Sorry one thing more about k-point indices.
In my example the band gap is direct at gamma and I have 3 gamma zero weight points, however sumo give indices for 5 points and even I can not follow the indices (other things are OK, band indices, bandgap values and so on). same like the previous example at 0 , 113 however, when I checked the kpoints it is 0 and 110.
here my example:
Kpoint
0.00000000000000    0.00000000000000    0.00000000000000     0 \Gamma
0.00000000000000    0.12500000000000    0.00000000000000     0
0.00000000000000    0.25000000000000    0.00000000000000     0
0.00000000000000    0.37500000000000    0.00000000000000     0
0.00000000000000    0.50000000000000    0.00000000000000     0 X
0.00000000000000    0.50000000000000    0.00000000000000     0 X
0.12500000000000    0.50000000000000    0.00000000000000     0
0.25000000000000    0.50000000000000    0.00000000000000     0
0.37500000000000    0.50000000000000    0.00000000000000     0
0.50000000000000    0.50000000000000    0.00000000000000     0 M
0.50000000000000    0.50000000000000    0.00000000000000     0 M
0.42857142857143    0.42857142857143    0.00000000000000     0
0.35714285714286    0.35714285714286    0.00000000000000     0
0.28571428571429    0.28571428571429    0.00000000000000     0
0.21428571428571    0.21428571428571    0.00000000000000     0
0.14285714285714    0.14285714285714    0.00000000000000     0
0.07142857142857    0.07142857142857    0.00000000000000     0
0.00000000000000    0.00000000000000    0.00000000000000     0 \Gamma
0.00000000000000    0.00000000000000    0.00000000000000     0  \Gamma
0.00000000000000    0.00000000000000    0.12500000000000     0
0.00000000000000    0.00000000000000    0.25000000000000     0
0.00000000000000    0.00000000000000    0.37500000000000     0
0.00000000000000    0.00000000000000    0.50000000000000     0 Z
Sumo output:
Direct band gap: 1.021 eV
k-point: [0.00, 0.00, 0.00]
k-point indices: 0, 21, 22, 23, 24
Band indices: 285, 287

Hi,
what is the function of sumo-bandstats -s
I read that the default is samples 3 k-points when determining the effective masses. what does it mean? if I change that number I got different effective mass values.
Best regards, ibra

Hi Ibra,
the effective mass is related to the curvature of the band structure. VASP give us gradient information directly, so we have to estimate the second-derivative from the points we have. The approach used in bandstats is to fit a quadratic function to a few of the calculated points, then use the second-derivative of this function. The default (3 points) is the minimum required so the fit line will pass through all of them. If you increase the range with -s, more points will be considered. This could have the benefit of averaging out some noise, but will also bring in data that is further away from the turning-point and so less representative. I would only recommend to increase -s when employing a very fine k-point sampling. (And at that point it would also be worth looking at the sensitivity to DFT convergence parameters...)
The feature is mostly there for testing sensitivity/convergence; if switching from 3 to 5 points makes a negligible difference, you can be more confident that your k-points sampling is dense enough.

Also, for the example:
...
There are degenerate bands for the hole with different masses and paths (G-L and G-X).
Yes

Which band and path could be selected?
Selected for what?

In my example the band gap is direct at gamma and I have 3 gamma zero weight points, however sumo give indices for 5 points and even I can not follow the indices (other things are OK, band indices, bandgap values and so on). same like the previous example at 0 , 113 however, when I checked the kpoints it is 0 and 110.
...
Sumo output:
Direct band gap: 1.021 eV
k-point: [0.00, 0.00, 0.00]
k-point indices: 0, 21, 22, 23, 24
Band indices: 285, 287

That is odd, it refers to index 24 but your KPOINTS list doesn't even have 24 points in it. Is this the full band structure path you used? The k-point spacing is too coarse, you need to have much finer steps than this to get sensible results from bandstats. To get more k-points, try varying the value of the --density argument to sumo-kgen.
You mentioned in the other post that there was a defect involved. Is this a huge unit cell? That would explain the large default step size. (Large cells are small in reciprocal space.) Band stuctures of supercells are quite difficult to interpret, you may be better-served by a band-unfolding approach (e.g. BANDUP or easyunfold) that projects onto the underlying lattice.

Thanks.

Hi,
what is the function of sumo-bandstats -s
I read that the default is samples 3 k-points when determining the effective masses. what does it mean? if I change that number I got different effective mass values.
Best regards, ibra

Hi Ibra,
the effective mass is related to the curvature of the band structure. VASP give us gradient information directly, so we have to estimate the second-derivative from the points we have. The approach used in bandstats is to fit a quadratic function to a few of the calculated points, then use the second-derivative of this function. The default (3 points) is the minimum required so the fit line will pass through all of them. If you increase the range with -s, more points will be considered. This could have the benefit of averaging out some noise, but will also bring in data that is further away from the turning-point and so less representative. I would only recommend to increase -s when employing a very fine k-point sampling. (And at that point it would also be worth looking at the sensitivity to DFT convergence parameters...)
The feature is mostly there for testing sensitivity/convergence; if switching from 3 to 5 points makes a negligible difference, you can be more confident that your k-points sampling is dense enough.

Thanks

In my example the band gap is direct at gamma and I have 3 gamma zero weight points, however sumo give indices for 5 points and even I can not follow the indices (other things are OK, band indices, bandgap values and so on). same like the previous example at 0 , 113 however, when I checked the kpoints it is 0 and 110.
...
Sumo output:
Direct band gap: 1.021 eV
k-point: [0.00, 0.00, 0.00]
k-point indices: 0, 21, 22, 23, 24
Band indices: 285, 287

That is odd, it refers to index 24 but your KPOINTS list doesn't even have 24 points in it. Is this the full band structure path you used? The k-point spacing is too coarse, you need to have much finer steps than this to get sensible results from bandstats. To get more k-points, try varying the value of the --density argument to sumo-kgen.
You mentioned in the other post that there was a defect involved. Is this a huge unit cell? That would explain the large default step size. (Large cells are small in reciprocal space.) Band stuctures of supercells are quite difficult to interpret, you may be better-served by a band-unfolding approach (e.g. BANDUP or easyunfold) that projects onto the underlying lattice.

I posted the part of KPOINTS that contains the gamma point (because the code detected the bandgap at that point, so I think k-point indices are for the gamma point, is it true?), however the total is 51 zero kpoints including the high symmetry kpoints.
The code detects everything correct including the bandgap, band indices, and so on.
Only my concern about k-point indices, what does it mean?
In my case the bandgap at gamma, and I have three gamma points with zero weight as I posted, however I got five indices.

Also, for the example:
...
There are degenerate bands for the hole with different masses and paths (G-L and G-X).

Yes

Which band and path could be selected?

Selected for what?

In case of different effective masses at degenerate bands or different paths (anisotropic), which value could be used and presented? could be the lightest or average.

I posted the top part of KPOINTS file with zero weight and contains the gamma point (because the code detected the bandgap at that point, so I think k-point indices are for the gamma point, is it true?), however there more kpoints with zero weight.

Unless you deleted lines from inside the block shown, there is still reason to be concerned about the sampling density.

The code detects everything correct including the bandgap, band indices, and so on.
Only my concern about k-point indices, what does it mean?
In my case the bandgap at gamma, and I have three gamma points with zero weight as I posted, however I got five indices.

Have a look at the k-point values at the corresponding indices, and the corresponding place in the band structure. Probably you will find that the band edges at that point are also equal to the VBM/CBM? You may find that due to band-folding there are Gamma-point features repeated at all the high-symmetry points.

In case of different effective masses at degenerate bands or different paths (anisotropic), which value could be used and presented? could be the lightest or average?

The degenerate bands and various directions (including as-yet-uncalculated directions) are all valid conduction pathways if the doping can be achieved. I would initially present all the calculated directions as a table; anisotropy is an interesting property. Then perhaps refer to effective masses "as low as X" when discussing potential of material. It depends on context.

Thanks.
but why k-point indices do not match with the KPOINTS file?

Please can you post the full KPOINTS and vasprun.xml

OK, because this is SOC calculation, vasprun.xml is very big so I will try to send it.

While you are uploading the vasprun, please can you upload the full k-points file.

Looking at sumo/tests/data/Cs2SnI6/bandstructure there does indeed seem to be something odd going on. I make it 111 zero-weighted k-points split over the two files, but a CBM is found at k-point 113. Is there some duplication going on in the data processing before bandstats are calculated?

The duplicating is done by this function: https://github.com/SMTG-UCL/sumo/blob/7a7e44f0d91a51e827d68a8e9cc6322c9ac69a2c/sumo/electronic_structure/bandstructure.py#L250
I'm not sure of the best way to account for this in band stats!

I figured out a simple fix! #169

OK. Thank you.

Thank you. It works.


Error: Positional queries with start == 1 remove parameter

If the start value is set to 1 it is being removed during query creation. This may be due to start = start -1 (transformation from 1-base interface to 0-based query parameter), and then the 0 being treated as missing value during query generation. However, the form value of start: 1 should translate into &start=0.

@ptoussai I've fixed this by casting the (first) start value to string. This is the only one that possibly would convert to 0:
  if (start) {
    const match = INTEGER_RANGE_REGEX.exec(start)
    if (!match) throw new Error("incorrect start range")
    const [, start0, start1] = match
    var s0 = start0 - 1
    s0 = s0.toString()
    starts.push( s0 )
    start1 && starts.push( start1 )
  }

... closing this since seems fixed (though maybe better solutions possible?).


How to use useResetAtom in atomEffect?

You shouldn't basically (re)set atoms in effect.
Though, RESET symbol should work.

https://codesandbox.io/s/xenodochial-ully-3pjgyj

https://codesandbox.io/s/xenodochial-ully-3pjgyj

666，thx


feat(Context-Based Restrictions): add Service Ref Target

PR summary
Add support for new endpoint
Update examples to match the API
PR Checklist
Please make sure that your PR fulfills the following requirements:

[X] The commit message follows the Angular Commit Message Guidelines.
[X] Tests for the changes have been added (for bug fixes / features)
[X] Docs have been added / updated (for bug fixes / features)

Current vs new behavior
Adds new endpoint for singular service ref target
Does this PR introduce a breaking change?

[ ] Yes
[X] No

@padamstx this is good to be merged

Integration Test Output:

Examples Output:

:tada: This PR is included in version 0.53.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Watson Sync prompts "No JSON object could be decoded"

Hello!
I am using Watson on Ubuntu Bash installed on Windows 10 Subsystem for Linux. Everything works great except when I try to "watson sync" with my crick api, I get this message:

Appears to be something with JSON and/or python and/or pip. I've tried installing and re-installing these things to no avail. Any thoughts on what might be going on here? Any help is much appreciated!

How have you configured the Crick backend? Are you using your own instance of Crick?

I have an account set up on crick.io and have "https://api.crick.io/watson" and my token set up under [backend] in my watson config file. It works fine when I sync on iMac, but I get the aforementioned error when I try to sync on my new Windows laptop. My guess is that I've somehow configured something wrong somewhere in setting up Watson and/or Ubuntu Bash, but my trial and error so far has accomplished nothing.

I'm getting this error as well on Elementary OS Luna (Ubuntu 18.04 LTS), as well as Windows 10 set up with Anaconda (pip 18.1)


Can't get a clear picture how to trigger on OneDrive file with JS.

The documentation is sparse on this issue. It is quite different compared to triggering on BlobStorage. Here we have subscriptions, renewal, authentication\authorization and then hopefully triggering. Another problem is how to tie this all to the Azure Portal. I have tried to collect the pieces of information like a detective but after 2 days of work still no positive result.

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

ID: 81cb62b5-7d3d-82ac-cbf9-cf8ec72da034
Version Independent ID: 69f0b334-acba-8dde-51d5-eb6bc9f85d5e
Content: Microsoft Graph bindings for Azure Functions
Content Source: articles/azure-functions/functions-bindings-microsoft-graph.md
Service: azure-functions
GitHub Login: @craigshoemaker
Microsoft Alias: cshoe

Related to #18614. The question is what to do (where to ask) in the meantime while the docs get updated and improved for beginners.

@StoyanStAtanasov Thanks for the question! We will investigate on this issue and get back to you soon.

@craigshoemaker I believe you have an open work item for this request as mentione in the refrenced issue #18614. Can you please share any ETA on the doc update?

@StoyanStAtanasov Have you reviewed Azure Logic Apps to trigger on One Drive File ?
https://docs.microsoft.com/en-us/azure/connectors/connectors-create-api-onedriveforbusiness
https://docs.microsoft.com/en-us/azure/connectors/connectors-create-api-onedrive

@KetanChawda-MSFT Thanks for the answer. I have reviewed the docs in the links.
I had tried OndeDrive trigger for Azure Logic Apps before and I had problems. I had synchronization problems - files and folders not visible until much later. I prefer to avoid using Azure Logic Apps if possible as they add another layer of complexity, monitoring, support, research, etc. I short they seem unreliable to me at this time.
I will give them another try though, but I can do that next week.

@StoyanStAtanasov Functions and Logic Apps are Azure services that enable server less workloads. Azure Functions is a server-less compute service, whereas Azure Logic Apps provides server-less workflows. You can mix and match services when you build an orchestration, calling functions from logic apps and calling logic apps from functions. Choose how to build each orchestration based on the services' capabilities or your personal preference.
Refer this document for comparison.
I have assigned the issue to the content author @craigshoemaker to evaluate and update as appropriate.

@KetanChawda-MSFT From my understanding, serverless is supposed to simplify development, but what I am experiencing with orchestrating 2 apps (with all the reading, experimenting monitoring and debugging) to run 10 lines of code is the direct opposite. I may be wrong but please note that the way the system is set up it defeats the purpose of being efficient and quick for me.
Maybe I am missing the point.

Apologies to all for the difficulty you've each had surrounding using the Graph bindings and for the delayed response. This binding is now officially deprecated, but you can find updated guidance on how to work with Azure Functions and Microsoft Graph in the recently-published Build Azure Functions with Microsoft Graph tutorial.
#please-close


Test Task/Batch workloads on Skipper mode

Update documentation to remove classic-mode references and use skipper mode as the only (composedly) option.
Merge the skipper doc section into the main documents.
Enable by default the Skipper URI property in the server-deployment.yml

Resolves #357

@chrisjs: I will give it a go.

merged in b23ae42


Do not discard top 8 significant bits of timestamp values in ranging calculation

Instead of casting the 5-byte timestamps into 4-byte values, perform
as much of the arithmetic as 64-bit values instead. Also, since
apparently the GCC compiler for the ESP32 fails to preserve the sign
across division of int64_t values, remove the negative sign before
division and add it later. The sign preservation is required for
later timestamp debugging.

Q
A

Bug fix?
yes

New feature?
no

Doc update?
no

BC breaks?
no

Deprecations?
no

Relative Issues
#

Thank you for this interesting pull request. Can you please change variable names in english and change variable "n" to a more explanatory name?

Thank you for this interesting pull request. Can you please change variable names in english and change variable "n" to a more explanatory name?
Can you also please add some comments to the portion of code that switches signs for future reference on the why it was written (you can use what you wrote in the pull request)

I have made two commits that address these points.

@Sonic0 It is a bit hard to say what is happening without access to both your version of the library and your example program. What is the output of your sample program after reverting my patch (and ONLY my patch)?

@Sonic0 Also, what MCU is being used by your example programs? ESP32? ESP8266? Other? I only have access to the ESP32, so I cannot check behavior under other MCUs.

@Sonic0 It is a bit hard to say what is happening without access to both your version of the library and your example program. What is the output of your sample program after reverting my patch (and ONLY my patch)?

Hi @avillacis, I have used StandardRTLSAnchorMain_TWR and StandardRTLSTag_TWR axamples. Test has been made with this PR tag code.
Switching to the master code there aren't any ranges problem and the output range is corrent or at least stable (no antenna delay).

@Sonic0 Also, what MCU is being used by your example programs? ESP32? ESP8266? Other? I only have access to the ESP32, so I cannot check behavior under other MCUs.

NodeMCU 1.0, then ESP8266.
Thus, no problem @avillacis, I will do other tests with Arduino Pro Mini 3.3V 8MHz as Anchor (We haven't an ESP32). Maybe the cast to uint32 has been made to prevent this problem (sorry, I don't remember that).

Thus, no problem @avillacis, I will do other tests with Arduino Pro Mini 3.3V 8MHz as Anchor (We haven't an ESP32). Maybe the cast to uint32 has been made to prevent this problem (sorry, I don't remember that).

Assuming a sane compiler for your platform, I cannot see how 48->32 bit truncation could be beneficial in smoothing out samples. That was one of the things I fixed in my fork of the library.

We are closing pull requests with merging conflicts or potential unstable commits as we are archiving the repo, feel free to create your own fork if you intend to support the project, we will be happy to add a link on this project.


Sample object type hierarchy broken on expand

It looks like the sample object type hierarchy view filter is broken when you click it.  This is 100% reproducible.  Investigating.

Pushed to iSamples Central and closed.


Built-in Minio bucket names must be prefixed by "neon-"

Minio is a shared cluster resource and we expect to make this available to user applications as well as for cluster services like Harbor.  We're currently naming our internal buckets like "harbor", "loki", "cortex", "alert-manager", "cortex-ruler", and "tempo".  These should all be prefixed by "neon-" and our guidance to users will be that buck names prefixed by this are reserved for neonKUBE services.
We also need to create a new KubeMinioBucket static class that defines these buckets as constants.

@marcusbooyah: I've created the KubeMinioBucket class with the current bucket names (without the prefix) and I updated cluster setup to use these constants when creating the Minio buckets.  I looked briefly at adding the prefixes myself but updating the related deployment setup looks like it'll be more than a search/replace.
Can you please add the neon- prefix to all of these and complete the cluster setup update to reference these.

Never mind: we'll have user create custom Minio tenants in their own namespaces.


example.txt no longer generates example.svg

For the debian packaging i'm testing aasvg to ensure that it produces example.svg from example.txt (see for example the build-time test and the runtime test which both do the same thing).  this test fails as of 4694a38e366eaf0f2a000683108fb54a140eb5d2.
4694a38e366eaf0f2a000683108fb54a140eb5d2 claims that all it does is "remove translate transforms" but it actually also adds the --source argument.  that's a bit confusing, but i think the removal of the translate transforms is actually what is causing the breakage.
i could make the diff more sophisticated, e.g. by converting them both to PNGs and seeing whether there is in fact any difference, for example, something like:

convert the reference image and the re-generated image, e.g. using gm convert example.{svg,png}
compare the resulting png's with diffimg from graphviz

(note: when i do this, the resulting image diff appears to be all-black, indicating no changes at the moment, so that's good!)
But if i do that for the test, then my choice of resolution and other parameters for the svg→png conversion tool might introduce spurious errors unrelated to the changes in aasvg, or mask changes in aasvg output.
or i could compare them with ImageMagick's compare utility which is supposed to return a distance metric between the images, and appears to think it can work on svg directly.  but it returns that the two files are different, even though the resulting output does not differ.
Can you keep example.svg up-to-date with example.txt?

This might just be skew, but I updated it shortly after fixing #7.  I see no difference now.  I'll add an npm test command and pre-commit to ensure that things work (or that example.svg is updated if the code changes).  I can use this to track that.

npm test should work now.

Thanks!  Looks like my problem was that the commits from 68feba537f7b40a1b32184f9b4cd5e2a00974046 on were not published here on github.  Now that they've been pushed i see the updated example. :+1:
Looks like dcf222c3d2c9b6de89f7fb507caaa0d3207b4fbb changed the version to 0.1.8.  Would you be up for pushing a tag to github when a new version is released?  or should i be looking at the npmjs.org registry for new releases?

(i also note that example.txt and example.svg are not shipped in the version on npmjs.org, so for now i'll be building the debian package from git, not from the npm tarball)

Happy to push tags if that makes it easier for you.


bump mincer version

@OverFlow636
bump mincer version to use autoprefixor, old version of mincer was using autoprefixor-core, which is deprecated.

unit test is failing, need to fix the unit test....

unit test should be fine now

tested on freshpair, dollargeneral, jamesperse, and rue21


chore(main): release 1.0.5

:robot: I have created a release beep boop
1.0.5 (2024-07-26)
Bug Fixes

using roverlib + rovercom (e51cb38)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v1.0.5
:sunflower:


german translation for semantics regarding issue #239

Did you test with Talkback or VoiceOver?

yeap and it was working without a problem


Highlighter module highlightSelection() function overrides selection option when provided

When using the Highlighter module, if the function highlightSelection function is called with the selection option - e.g.
highlighter.highlightSelection('highlight', { selection: currentSelection });```

the `currentSelection` object will be overridden by the `api.getSelection(this.doc)` call in the `highlightSelection` function (rangy-highlighter.js line 463).

The `api.getSelection(this.doc)` call only needs to happen if the selection option is empty. Currently, this is being called every time to create the default options to be used in the `createOptions` function, in which it will be tested if there are any options.

In conclusion: if you call  `highlightSelection` with the selection option, the selection option will be ignored.

From my perspective, this can be solved by changing the `createOptions` defaultOptions argument to be lazy evaluated (only if it's needed), or by changing the `highlightSelection` function to not use the `createdOptions`.

I stumbled upon the same problem and would love to read what's the correct change I would need to make to the highlightSeleciton function in order to make it work.


Date parameters for the CLI

Danny, I think date parameters would be helpful.  For example, -create_date>1/1/2020.  What do you think?   Also, does -pr-state=OPEN also support =pr-state=* to grab both open and merged?

Hello @jameschung2019!
In your date example would input be: >1/1/2020? Which date queries would you like to see? I'm assuming:

create date
close date

GitHub supports querying multiple states in a single API call;  I can add support for it sometime in the next week.

Sorry, still intend to make this change, will look at adding the states this week (famous last words)


parallel #89098

This is a parallel set of changes to #89098 - the duplicate text suggests a shared include could work here like for the compute include, but I'm not 100% sure that's the case.

@MikeBazMSFT : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off Looks good. Please merge.


Go: Embed wit-bindgen version

Unlike the C and Rust code generators, the Go code generator does not seem to embed the wit-bindgen version in the producer custom section in the "processed-by" field.  The field should be in a format like the other languages with the key "wit-bindgen-go"  and the version "0.8.0"

Related to this issue but in a different context, I believe go generator does not generate "// Generated by wit-bindgen 0.8.0. DO NOT EDIT!" message to the generated Go source code files.

I guess the my quesiton is how can we embed wit-bindgen version to the component? C bindgen generates a object file that does so. What's the mechanism for Golang to achieve the same thing?

I've opened #621 to address the generated code preamble.
We really need to add the entire component-type section not just the producer information. I'll adjust the issue to reflect that.

We really need to add the entire component-type section not just the producer information. I'll adjust the issue to reflect that.

I agree!


Add package augmentation extensions

and use them to guid in task capture extension selection

Codecov Report

Merging #4 (e8cb563) into master (2af0195) will not change coverage.
The diff coverage is 0.00%.

@@          Coverage Diff           @@
##           master      #4   +/-   ##
======================================
  Coverage    0.00%   0.00%
======================================
  Files          11      13    +2
  Lines         338     367   +29
======================================
- Misses        338     367   +29

Impacted Files
Coverage Δ

colcon_snapshot/event_handler/store_lockfile.py
0.00% <0.00%> (ø)

colcon_snapshot/package_augmentation/default.py
0.00% <0.00%> (ø)

colcon_snapshot/package_augmentation/git.py
0.00% <0.00%> (ø)

colcon_snapshot/subverb/capture.py
0.00% <ø> (ø)

colcon_snapshot/task/capture/dirhash.py
0.00% <0.00%> (ø)

colcon_snapshot/task/capture/git.py
0.00% <0.00%> (ø)

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 2af0195...e8cb563. Read the comment docs.


Update index.md

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Congrats @GoldbergData on your first pull request (PR)! 🎉
Thank you for your contribution to the page! 👍
We're happy to accept these changes, and look forward to future contributions. 📝


there any way to add a new option to selected options?

i wan to add a new option to the select, but when i using under code
var val='something'
$("#search").val([val]).trigger("change");
the old selected option were clear, so  how to add a new option  .no clear and create new one .

You could do like this add the new value without loose the old : https://jsfiddle.net/m0muaqet/1/
var oldValue = $('.select2').val() || []; // retrieve old value selected
oldValue.push($('.threeOpt').val()); // add the new val
$('.select2').val(oldValue).trigger('change'); // update the select and notify

If you're just adding a new option, you can create a new Option object and append it to your control: https://select2.org/programmatic-control/add-select-clear-items
The last two parameters of new Option(... control the selected status of the option.


Fix 2 broken hyperlinks in nginx.md

Hyperlinks to templates of nginx configurations pointed to
wrong pathes - fixed.

@alces This is actually correct for the canonical source of this repo at https://gitlab.com/gitlab-org/omnibus-gitlab , GitLab doesn't support ../.. syntax for relative links. Thanks for taking an effort to make a change, I appreciate it :+1:

So, there's no syntax for hyperlinks correct both for gitlab and github?


Add "udt_name" attribute to the columnInfo() method

This is useful when the field is set to array.

Looks reasonable. Any idea why this (and all the others) were omitted?

Maybe for consistency between databases, but in my tests the other informations, apparently, is not so relevant.
I don't know if is interesting add a parameter in the method to show all column informations.

So long as a minimum set of values is shared between all implementations then consistency shouldn't be an issue. However, if my understanding of the docs is correct, might it be more appropriate to do this?
columns[val.column_name] = {
  type: val.data_type === 'USER-DEFINED' ? val.udt_name : val.data_type;
  maxLength: val.character_maximum_length,
  nullable: (val.is_nullable === 'YES'),
  defaultValue: val.column_default
}

I agree with you and I forgot about User Types. However you suggestion won't solve the array problem.
I make same changes in my repository and  now I believe that is right.

Closing due to inactivity.


Multiple workers remove themselves but keep running

I am trying to use the COUNT=n env to run multiple workers. I'm okay with the parent process terminating immediately as long as I have a way to find the children later to stop them.
I am taking the approach of querying the resque:workers set in Redis to find all running workers that match my queue, and sending the QUIT signal to each. My code looks something like this:
$members = Resque::redis()->smembers('workers');
foreach($members as $m) {
  $parts = explode(':', $m);
  $pid = $parts[1];
  $queue = $parts[2];
  if($queue == getenv('QUEUE')) {
    echo "Sending QUIT to pid: $pid ";
    posix_kill($pid, SIGQUIT);
  }
}

However, I immediately noticed that after starting 10 workers, only 2-4 of them would be quit by the script. Digging into it, it appears that some of the workers remove themselves from the "running" list but their pids stick around. Here's the sequence of Redis commands that are run when I start 10 workers with COUNT=10 (inspected via "monitor"):
1428064566.774837 [0 <IP_ADDRESS>:55649] "SELECT" "0"
1428064566.775186 [0 <IP_ADDRESS>:55649] "SMEMBERS" "resque:workers"
1428064566.776244 [0 <IP_ADDRESS>:55649] "SADD" "resque:workers" "pk.local:74911:hedgy"
1428064566.776648 [0 <IP_ADDRESS>:55649] "SET" "resque:worker:pk.local:74911:hedgy:started" "Fri Apr 03 12:36:06 UTC 2015"
1428064566.777629 [0 <IP_ADDRESS>:55649] "BLPOP" "resque:queue:hedgy" "5"
1428064566.809484 [0 <IP_ADDRESS>:55652] "SELECT" "0"
1428064566.809920 [0 <IP_ADDRESS>:55652] "SMEMBERS" "resque:workers"
1428064566.810406 [0 <IP_ADDRESS>:55652] "SISMEMBER" "resque:workers" "pk.local:74911:hedgy"
1428064566.810994 [0 <IP_ADDRESS>:55652] "SREM" "resque:workers" "pk.local:74911:hedgy"
1428064566.811232 [0 <IP_ADDRESS>:55652] "DEL" "resque:worker:pk.local:74911:hedgy"
1428064566.811460 [0 <IP_ADDRESS>:55652] "DEL" "resque:worker:pk.local:74911:hedgy:started"
1428064566.811968 [0 <IP_ADDRESS>:55652] "DEL" "resque:stat:processed:pk.local:74911:hedgy"
1428064566.812179 [0 <IP_ADDRESS>:55652] "DEL" "resque:stat:failed:pk.local:74911:hedgy"
1428064566.812779 [0 <IP_ADDRESS>:55652] "SADD" "resque:workers" "pk.local:74910:hedgy"
1428064566.813247 [0 <IP_ADDRESS>:55652] "SET" "resque:worker:pk.local:74910:hedgy:started" "Fri Apr 03 12:36:06 UTC 2015"
1428064566.814364 [0 <IP_ADDRESS>:55652] "BLPOP" "resque:queue:hedgy" "5"
...

You can see it started pid 74911 and then almost immediately cleaned itself up from Redis. I don't have any jobs on the queue, so nothing should be actually doing any work. After I get into this state, about 2-4 of the workers end up appearing in Redis as "active" but the others are gone, but all 10 processes are still actually running, and I can see the other workers trying to pull jobs from the queue still. At this point I have no way of tracking which pids were started by the parent and I have to go clean things up manually.
I unfortunately don't have a solution to this yet but was hoping this would make sense to someone more familiar with php-resque.

Your workers are being spawned too quickly is what's up here.  When a new worker starts, it goes through the workers set and clears out any "stale" PIDs.  It does that by looping through the list and removing any PID it can't see as running on the system.  So there's a race condition when a worker adds itself to the set, but the other workers can't actually see it yet.
Potential fixes are fairly straightforward.  One, introduce a short (quarter-second?) delay between forking new workers in the startup script.  Two, modify the startup script to not exit, but to stick around and coordinate its child processes (I've done this in my own fork, if you want an example of that approach).  This second approach would likely replace your management code, above.
There may be other fixes for this, but the fact is that it is a race condition causing the issue you're seeing here.


Fix for failing builds: unique filenames

@jcavat After some digging into these half-passing/half-failing tests, I discovered the issue is simply in the lp model & solution filenames. Since cargo runs tests in parallel, this creates possibilities for race conditions and loading the solution file from another test (when all called "sol.sol" by default). The previous PR simply exposed this issue by adding another test which interacted with the solver.
This PR changes the example code to use the problem name as a filename for the lp model and solution. A more trivial way to mask this would be confining cargo tests to using only 1 thread. Ideally in the future each model should use a UUID filename to interact with the solver.
I believe this should fix the flaky builds.

Good point, I create an issue for this : #19. Thanks for the fix


chore: Test linter config

Add window popUp to the ptoject

I have conflict issues with the former pull request, so I made a new pull request to resolve the merging conflicts.

STATUS: CHANGES REQUIRED 🟡
Hi team,
:thumbsup: This is good to make it this far, your project is really nice, congratulations on the good work you have done so far. The popup behaves as expected. Kudos on your JS 👏🏻👏🏻👏🏻
Keep on coding, good job so far! :clap:
There are some issues that you still need to work on to go to the next project mostly to do with the styles.

Required Changes  :recycle:

[ ] According to project requirements, The popup follows the design for both desktop and mobile version from the chosen template in Figma (e.g., font, colors, images, tests, margins). Please try to meet the Figma design to comply with all the requirements, currently the buttons content is not properly aligned:

Yours
Figma

[ ] Fonts, margins and paddinds should also match the figma design,

Yours
Figma

[ ] I suggest you to meet the styles in mobile view as well:

Yours
Figma

!

Cheers and Happy coding! :clap: :clap: :clap:
Code review by Shoira
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.

Hi @ShoiraTa
I've modified those areas and everything Is working perfectly now.


feat: upgrade to Go 1.17 and expose KeyType in kms.ExportPubKeyBytes()

Signed-off-by: Baha Shaaban<EMAIL_ADDRESS>

Codecov Report

Merging #81 (8cc6363) into main (f7440a8) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main      #81   +/-   ##
=======================================
  Coverage   85.30%   85.30%
=======================================
  Files          13       13
  Lines        2151     2151
=======================================
  Hits         1835     1835
  Misses        183      183
  Partials      133      133

Impacted Files
Coverage Δ

pkg/controller/command/command.go
83.19% <100.00%> (ø)

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 f7440a8...8cc6363. Read the comment docs.


Drawing error

Why did I use your code to draw the picture in a mess?  I don't know where it is wrong because I just copy your code on github.

Can you share which code you used here? Based on the images it looks like the step counter is going to 170 and then going back and starting over, but can't be sure without seeing the code.

Note that if you use epochs in your loop (i.e. loop over epochs and in it a loop over batches) then pass the step value as a tuple. For example, hl.log( (epoch, batch), ....).

I only copied your code on github.

The code looks good, and I tested it again to be sure and it's working correctly. I'm really puzzled by the drawing that you got. Did you change anything in the library itself?

I didn't change anything in the library . I'm really strange, too.
But I find that it will be nice  if I change figure in the following .

I check carefully and I do not set any global Settings. I'm also surprised that the x-coordinate is normal at first when drawing the picture at the beginning (10,20,...,70), and then it gets messy afterwards ( jump to 100,110,...)

And I find a interesting thing- the code in the following
if step % num ==0:
when the variable num>=100, the picture will be nice and when num<100, the picture will be in mess.
I like your resource very much and hope my problem will be solved, meanwhile, thank you for taking the time to answer my questions.

In the process of drawing, the X-axis gets messy.

Yes, your last comments confirm my suspicion that the x-axis is being sorted as a string rather than integer.
For example, 10, 20, 30, 40, 50, 60, 70, 80, 90. These will be sorted corrected both as string and integer.
However, once you add 100, it becomes 10, 100, 20, 30, 40, ...etc. Because 10 and 100 both start with "1", so they get sorted before 20, which starts with "2". Then you add 110, and the order becomes 10, 100, 110, 20, ....etc.
So we know the x-axis is being treated like a string. Now we need to know why. There are two possibilities:

Some code is changing the step from integer to string.
2.Or, some global setting in matplotlib that causes the x-axis to be treated as a string.

Why can’t I get a dynamic picture in the same code? .... I only can get the saved .png images...

For the searchers:
It's highly probable, that you have matplotlib < 2.2.0 which produces a bug with axis sorting.
@waleedka it could be useful to add matplotlib >= 2.2.0 as pre-requisite for your library. And BTW, cool library, thanks!


"find: '>     bin [': No such file or directory" on Windows running update-packages

I've not used Flutter before but I'm investigating a bug in Dart Code and was asked to clone this repo and run ./bin/flutter update-packages. When I do this on Windows (10), there are lots of errors like the one in the subject which I presume are unexpected:
M:\Coding\Applications\flutter [master ≡]> ./bin/flutter update-packages
Checking Dart SDK version...
Downloading Dart SDK 1.23.0-dev.10.0...
Unzipping Dart SDK...
Updating flutter tool...
find: '>     bin [': No such file or directory

  ╔════════════════════════════════════════════════════════════════════════════╗
  ║                 Welcome to Flutter! - https://flutter.io                   ║
  ║                                                                            ║
  ║ The Flutter tool anonymously reports feature usage statistics and crash    ║
  ║ reports to Google in order to help Google contribute improvements to       ║
  ║ Flutter over time.                                                         ║
  ║                                                                            ║
  ║ Read about data we send with crash reports:                                ║
  ║ https://github.com/flutter/flutter/wiki/Flutter-CLI-crash-reporting        ║
  ║                                                                            ║
  ║ See Google's privacy policy:                                               ║
  ║ https://www.google.com/intl/en/policies/privacy/                           ║
  ║                                                                            ║
  ║ Use "flutter config --no-analytics" to disable analytics and crash         ║
  ║ reporting.                                                                 ║
  ╚════════════════════════════════════════════════════════════════════════════╝

Downloading Material fonts...
Downloading package sky_engine...
Downloading windows-x64 tools...
Downloading android-arm-profile/windows-x64 tools...
Downloading android-arm-release/windows-x64 tools...
Downloading android-x86 tools...
Downloading android-x64 tools...
Downloading android-arm tools...
Downloading android-arm-profile tools...
Downloading android-arm-release tools...
Running 'flutter packages get' in automated_tests...
find: '>     bin [': No such file or directory
Running 'flutter packages get' in complex_layout...
find: '>     bin [': No such file or directory
Running 'flutter packages get' in microbenchmarks...
find: '>     bin [': No such file or directory
Running 'flutter packages get' in bots...
find: '>     bin [': No such file or directory
Running 'flutter packages get' in devicelab...
find: '>     bin [': No such file or directory
Running 'flutter packages get' in manual_tests...
find: '>     bin [': No such file or directory
Running 'flutter packages get' in tools...
find: '>     bin [': No such file or directory
Running 'flutter packages get' in flutter_gallery...
find: '>     bin [': No such file or directory
Running 'flutter packages get' in flutter_view...
find: '>     bin [': No such file or directory
Running 'flutter packages get' in hello_world...
find: '>     bin [': No such file or directory
Running 'flutter packages get' in layers...
find: '>     bin [': No such file or directory
(snip)...

@goderbauer

@DanTup I've seen that error before when running flutter from a third-party shell (like Git Bash or Cygwin or MinGW) on Windows. It's a problem with the underlying Dart infrastructure (see https://github.com/dart-lang/pub/issues/1540). Unfortunately, until that is resolved, Flutter won't be able to support third-party shells on Windows (https://github.com/flutter/flutter/issues/8860).
Are you running Flutter from a third-party shell? Do you see the same error when running from PowerShell or Cmd?

@goderbauer It was just a standard PowerShell shell in Windows 10. I've not seen the issue using Dart or Pub before; though I'm not a heavy Dart user so that might not mean much.

Interesting... Unfortunately, I cannot reproduce this on my machine. Can you post the output of flutter doctor here and can you also tell me which version of PowerShell you are using?
Do you see the same issue when running flutter update-packages in the regular Cmd prompt?

Actually, I was wrong about the shell! :( I just tried it again and got Error: Unable to find git in your PATH.. That reminded me that when I did that before I just launched PoSh from within GitHub Desktop (which sets the path to include the version of Git it shipped with). So, it seems to be caused by using GitHub for Windows shell (which is likely why I never say it with Pub; I only ever launch PoSh this way for Git stuff).
So, this is certainly a dupe of the other issue and I've confirmed it does indeed work from a "standard" PoSh shell (once Git is available in the path).
Apologies for the misleading info; probably makes sense to close this as a dupe of the Dart case.

No worries. Closing as dup of https://github.com/flutter/flutter/issues/8860.


Add access to pods to generic gateway role

This PR grants the generic gateway component access to pods in the configured namespace. This is a workaround for https://github.com/Azure/brigade/issues/564.
The change results in the following Kubernetes Role:
# Source: brigade/templates/gateway-generic-role.yaml
---
kind: Role
apiVersion: rbac.authorization.k8s.io/v1beta1
metadata:
  name: release-name-brigade-generic-gateway
  labels:
    app.kubernetes.io/name: release-name-brigade
    helm.sh/chart: "brigade-0.21.0"
    app.kubernetes.io/instance: "release-name"
    app.kubernetes.io/managed-by: "Tiller"
rules:
- apiGroups: [""]
  resources: ["secrets", "pods"]
  verbs: ["get", "list", "watch", "create", "update", "patch", "delete"]

Now, if the component doesn't have access to pods, it continues to work properly - however, the logs are completely unusable, with the following error everywhere:
E0222 01:35:31.658531       1 reflector.go:205] github.com/Azure/brigade/pkg/storage/kube/apicache/liststore.go:36: Failed to list *v1.Pod: pods is forbidden: User "system:serviceaccount:brigade:brigade-brigade-generic-gateway" cannot list resource "pods" in API group "" in the namespace "brigade"

I tested that builds are created, but the logs are unusable - if we think we don't actually need the logs, we can simply close this PR.
cc @vdice, @dgkanatsios

Hey @radu-matei - apologies, we could/should have merged this prior to migration as it is a straightforward change.  When you get a chance, would you recreate the PR in the new brigadecore/charts repo?


move api.effechecka.org to api.guoda.bio

as discussed with @jhammock @godfoder @mjcollin and @unwashedmeme

same for archive.effechecka.org -> archive.guoda.bio

http://archive.guoda.bio and http://api.guoda.bio are now active.


Whisper

Whisper large v3 should be able to run in realtime on M1 Pro

Completed in examples/whisper

I'm getting an error when trying to run it on CPU.
thread 'main' panicked at C:\Users\user\.cargo\registry\src\index.crates.io-6f17d22bba15001f\tinyvec-1.6.1\src\arrayvec.rs:646:5:
ArrayVec::push> capacity overflow!

Is this expected? Thanks.


Computing normalized score

Hi,
Thanks again for this repository and data.
I was wondering are REF_MIN_SCORE and REF_MAX_SCORE (here) used in score normalization described in  section 5.1?
normalized_score  = 100* (score - REF_MIN_SCORE) / (REF_MAX_SCORE - REF_MIN_SCORE )
where
REF_MIN_SCORE == random_score
REF_MAX_SCORE == expert_score

If not would you please provide random_score and expert score [section 5.1 in the paper] to make comparison correct ? otherwise, it would be very hard to do correct and fair comparison.
Thanks for your help.

I have been also interested in this issue. Why did you close it? Was it somehow resolved?

Hi, you can use these two functions:
https://github.com/Farama-Foundation/D4RL/blob/5ef61940a5cbee3146d5de6fab06351e41553788/d4rl/offline_env.py#L71
https://github.com/Farama-Foundation/D4RL/blob/5ef61940a5cbee3146d5de6fab06351e41553788/d4rl/ope.py#L40

From these functions, we can see that the REF_MIN_SCORE and REF_MAX_SCORE are indeed the random_score and expert_score, respectively.


Single container bug

Solves several problems related to double-starting containers, more graceful container shutdown. Should address #30, and #31 at least.
As always, I'm optimistic.

Already tested manually and the code worked as expected!


ERR_SSL_PROTOCOL_ERROR when opening https://springdoc.org

This site can’t provide a secure connection
springdoc.org sent an invalid response.
ERR_SSL_PROTOCOL_ERROR
The site can't be opened both in Safari and in Chrome.

Thank you for the notification @Aloren.
The domain is now available using https as well.


bug: neodev not working with mason-lspconfig

Did you check docs and existing issues?

[X] I have read all the neodev.nvim docs
[X] I have searched the existing issues of neodev.nvim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
NVIM v0.9.5
Operating system/version
macOS 14
Describe the bug
I cannot seem to get neodev working in my config directory, even with a super minimal config that installs lua_ls with mason-lspconfig... I get Undefined global "vim" warnings whenever I try to use the vim object...
Steps To Reproduce

Copy the repro below into ~/.config/nvim/repro.lua
Run nvim -u ~/.config/nvim/repro.lua ~/.config/nvim/repro.lua
Wait for Lazy to finish installing the plugins
Wait for Mason to finish installing lua-language-server (use :Mason to see the progress)
Close Neovim
Do step 5 again
Enter insert mode
Exit insert mode
See warnings...

Expected Behavior
This simple config to work with Neodev.
Repro
-- DO NOT change the paths and don't remove the colorscheme
local root = vim.fn.fnamemodify("./.repro", ":p")

-- set stdpaths to use .repro
for _, name in ipairs({ "config", "data", "state", "cache" }) do
  vim.env[("XDG_%s_HOME"):format(name:upper())] = root .. "/" .. name
end

-- bootstrap lazy
local lazypath = root .. "/plugins/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
  vim.fn.system({ "git", "clone", "--filter=blob:none", "https://github.com/folke/lazy.nvim.git", lazypath, })
end
vim.opt.runtimepath:prepend(lazypath)

-- install plugins
local plugins = {
  "folke/tokyonight.nvim",
  "folke/neodev.nvim",
  "neovim/nvim-lspconfig",
  "williamboman/mason-lspconfig.nvim",
  "williamboman/mason.nvim",
}
require("lazy").setup(plugins, {
  root = root .. "/plugins",
})

vim.cmd.colorscheme("tokyonight")

require("mason").setup()
require("mason-lspconfig").setup {
  ensure_installed = { "lua_ls" }
}
require("neodev").setup()
require("lspconfig").lua_ls.setup {}

I also recently (aka up until just now) had the same/a similar problem.
In my case the lua language server incorrectly set the project root to / which obviously disables neodev as that is not a neovim config directory.
Creating a barebones .luarc.json with only the schema (from the lua language server docs) and lua runtime version set to LuaJIT seems to have fixed my problem.

I also recently (aka up until just now) had the same/a similar problem. In my case the lua language server incorrectly set the project root to / which obviously disables neodev as that is not a neovim config directory. Creating a barebones .luarc.json with only the schema (from the lua language server docs) and lua runtime version set to LuaJIT seems to have fixed my problem.

Could you share how to set Luajit:)

Sure! All I put into the .luarc.json file sitting next to my init.lua was this:
{
    "$schema": "https://raw.githubusercontent.com/sumneko/vscode-lua/master/setting/schema.json",
    "runtime.version": "LuaJIT"
}

The "$schema" came from the official lua language server documentation regarding the config files and the runtime.version key came from the same page while the value was my best guess at how the server likely stored/referred to the LuaJIT runtime.

I've had the same issue recently and the above .luarc.json worked perfectly.

I have this exact issue, however the .luarc.json doesnt seem to be making a difference

for me neodev works only if i specify it as a dependency for lspconfig, mason, and mason-lsp
	"neovim/nvim-lspconfig",
	dependencies = {
		"williamboman/mason.nvim",
		"williamboman/mason-lspconfig.nvim",
		{ "folke/neodev.nvim", opts = {} },
	},

if i do anything else, for some reason neodev does not work.
If you want, give it a try

It started working for me after I put neodev.nvim as a dependency of nvim-lspconfig and then also trigged nvim-lspconfig on LspAttach. However, I subsequently installed the luacheck linter with Mason and using with nvim-lint which brought the accessing undefined variable 'vim' back.


ListView - Added ability to apply header styling

Q
A

Bug fix?
[ ]

New feature?
[x]

New sample?
[ ]

Related issues?
Partially adds #410

What's in this Pull Request?
(This is my 1st open source contribution and I'm still fresh at TS/JS... bear with me.)
I added the ability to apply custom styles to the header of a ListView via optional prop headerClassName.
Something I was not able to implement was the ability to modify the font styling of the header text (size, weight, etc), so I set the header min-height to 17px which seemed small enough to scrunch everything without clipping any text - I tried testing as many scenarios as I could. Theoretically, to modify the font styling, I think you could add another styling prop to ListView, and apply that prop to '.ms-DetailsHeader-cellTitle' in ListView.tsx... I can experiment with this at another time.
For my use case, although likely unconventional, I'm using a ListView in a tight spot on a page and need all the real estate being taken up by the default padding (padding-top of 16px) and line height (42px iirc).
Let me know if I'm, at all, doing this incorrectly (lol).

i have no idea how the other 2 commits got in here. Oops?

Ah, I understand your reasoning about the class names. I'm not sure of a good alternative at the moment, I'll have to do some digging and tinkering.

Hi @bradleypregon,
Are you still working on this PR? Do you need help?

Hello @michaelmaillot,
Apologies for the late reply. Yes, I am still working on the PR, although I have not looked at it for some time, and I'm still stuck.
I tried a few different approaches, one involving the use of mergeStyleSets from the office-ui-fabric-react package, to no avail. If memory serves, every approach I found relied on utilizing the styling class name directly. I'm just not sure of a solid and practical way to access/modify the list header styling.


Update use-access-teams-owner-teams-collaborate-share-information.md

Just a simple correction

@pierodamafalda Thanks for catching that wording error.

Anytime @phecke


Pantheon test broken on master

Describe the bug
Looks like there were some pantheon breakage that passed through staging-next:
https://hydra.nixos.org/build/104001348
cc @worldofpeace

See https://github.com/elementary/files/issues/1081, worked around in https://github.com/NixOS/nixpkgs/commit/404f148e086db81316d8ca3b4a41ae38c53af535.
Vala has had some more release as well that should probably get queued into staging.


Invalid monitor handle error after waking computer

Microsoft PowerToys version
0.73.0
Installation method
PowerToys auto-update, WinGet
Running as admin
No
Area(s) with issue?
General
Steps to reproduce
I woke my computer and was greeted with a "Something went wrong." error screen with a log.
I did nothing but login and see the message on screen.
The error message seems to be "Invalid monitor handle".
Log file
I'm not sure how to reproduce this, but the window advised me to report the issue.
I selected General on the "Area(s) with issue" drop-down because I don't know what this is related to.
The utilities I have enabled are:

PowerToys Run
Various File Explorer add-ons
File Locksmith
Always On Top

In case this helps in debugging.
✔ Expected Behavior
N/A (Nothing to go wrong?)
❌ Actual Behavior
The error
Other Software
No response

Not able to repro this, going to close. Let us know if you see this again!


docs: updated document for 'Return Source Documents' Functionality

Description: updated the outdated code in the document that was generating the error,
Issue: #15086 ,
Dependencies: N/A,
Twitter handle: @vardhaman722

even easier - and better - fix would be to just call bot.invoke({})


Registry Key/Value: Result inconsistency

In reviewing the Registry Key/Value classes I noticed the requirements on reg_value and reg_value_result (s/value/key for the other class, same holds true) should be changed to an either or required constraint.
Proposal
Change the requirement on these event classes to reg_value OR reg_value_result MUST be set.
Examples
Using Windows APIs as an example as some tools still rely on hooking (and sysmon outputs data in this format)...
On the subject of Getting a Registry Value:
Documentation: RegGetValueW
OCSF would expect activity_id to be set to Read with reg_value set. Inside reg_value path would be set to a combination of hKey + lpSubKey, name set to the value of lpValue, and pvData would be data
On the subject of Deleting a Registry Value:
Documentation: RegDeleteKeyValueW
OCSF would expect activity_id to be set to Delete with reg_value set. Inside reg_value path would be set to a combination of hKey + lpSubKey and name set to the value of lpValueName.
On the subject of Modifying a Registry Value:
Documentation: RegSetKeyValueW
OCSF would expect activity_id to be set to Modify with reg_value_result set. Inside reg_value path would be set to a combination of hKey + lpSubKey, name set to the value of lpValueName, and pvData would be data

@tankbusta Currently, reg_key and reg_value are required and reg_key_result and reg_value_result are optional. The reg_key_result and reg_value_result should be only used with Modify to specify the new value/key for registry.
I am not sure what exactly is the issue? Is activity_id = Modify the problem? Because the current values are unknown/not available?

From an analyst perspective, having the field you want to access change depending on the operation type (reg_value for Get/Set/Delete and reg_value_result for Modify) is not optimal.
Would it be preferable to have reg_value_before (or similar) as an optional field for the Modify case, and have reg_value always set for Get/Set/Delete/Modify?

Good point @awhite456 - To summarize

Constraints remain the same
Drop reg_value_result
Create reg_value_before which would indicate the previous key/value before the mutation

This was one of the suggestions from the Slack thread as I recall (I may have made it myself in fact).  @tankbusta added a constraint and some clarification that will be useful in documentation without changing attribute names (or meanings).  If we want a more surgical approach, then I would change the name of reg_key_result and reg_value_result to reg_key_prev as Optional and reg_value_prev as Optional, and only populate them in the Modify case.
In these cases, reg_key and reg_value would remain Required, as they MUST always be there for any operation.
I don't know how we constrain reg_key_prev and reg_value_prev to be populated for Modify but it's not clear the previous value will always be available so it may just be Optional.


JS logs containing [ and ] are failing to parse

When logging from JS code and the message contains [ and ] (such as when logging JSON that contains arrays) the log message is not being parsed nor its JSON content.
Repro:
Create a JS lambda with the following code:
module.exports.handler = async (event, context) => {
  console.log('Just a message')
  console.log(JSON.stringify({
    message: 'just a JSON message'
  }))
  console.log(JSON.stringify({
    message: 'A message and array',
    array: ['with', 'data'],
  }))
  console.log(`Message with [brackets]`)

  return {
    statusCode: 200
  }
}

And the messages that contain brackets aren't being parsed:

Compare

with

Hi @ShlomiAltostra
We managed to reproduce your issue.
We trying to figure why it's happening . Will let you know shortly

Hi @ShlomiAltostra we found the bug the cause this issue and we are working on a fix

https://github.com/logzio/logzio_aws_serverless/pull/51

Hi @ShlomiAltostra
We released a new version to support this issue.
This version is only for python 3 as python2 is depreacted
Please check if it solve your problem and close this issue
Thanks

@Doron-Bargo
Tested
The above function is fixed, by logging a message that contains error, info etc in bracket would fail as before (maybe split the message on '\t' before the check?)

Anyway, as our issue is fixed, I'm closing


Mock Airtable

Need to mock the Airtable results in the Cypress tests.
See
https://github.com/cypress-io/cypress/issues/920
https://github.com/cypress-io/cypress-example-recipes/blob/master/examples/stubbing-spying__window-fetch/cypress/integration/spy-stub-clock-spec.js
Could possibly just mock the fetchTableFromAirtable or fetchFromAirtable functions in utils/airtable.js

Included in #139


PHP Fatal error:  Maximum execution time of 30 seconds exceeded in openid.php on line 188

Periodically in logs php there is such a mistake. - PHP Fatal error:  Maximum execution time of 30 seconds exceeded in ...openid.php on line 188
Any ideas how to fix?

This is just the Steam OpenID-Server timing out.
While you could increase the execution time for openid.php, you shouldnt if it doesnt happen too often.
Your users will only see a PHP error wich will most likely make them go back and try logging in again.


Provide stdout for zabbix sender failure

When there is a failed item for zabbix_sender when used as a binary the output is presented when x number for processed and x number for failed items and exits with status 2. When using zabbix-promise it just fails with no stdout even if one of the items succeeds.
Binary Example:
zabbix_sender -z zabbix-dev -s testhost -k testing2 -o 1 ; echo $?
info from server: "processed: 0; failed: 1; total: 1; seconds spent: 0.000031"
sent: 1; skipped: 0; total: 1
2

zabbix-promise Example:
    zabbix.sender({
        'path': '/usr/local/bin/zabbix_sender',
        'server': 'zabbix-dev',
        'host': 'testhost',
        'values': '- testing 444\n- testing2 111\n',
    })
    .then((value) => console.log(value))
    .catch((reason) => console.log(reason));

Error: /usr/local/bin/zabbix_sender process exited with code 2
    at ChildProcess.cmd.on (/Users/user/test/zabbix-promise-test/node_modules/zabbix-promise/lib/utils.js:61:20)
    at emitTwo (events.js:126:13)
    at ChildProcess.emit (events.js:214:7)
    at maybeClose (internal/child_process.js:925:16)
    at Socket.stream.socket.on (internal/child_process.js:346:11)
    at emitOne (events.js:116:13)
    at Socket.emit (events.js:211:7)
    at Pipe._handle.close [as _onclose] (net.js:567:12)

@alexraddas The zabbix-promise project released a major version v2.0.0. The package is written to support JavaScript Promise and Async/await interfaces. Zabbix-promise implements Zabbix sender protocol in pure JavaScript code, and there are no other package dependencies, just the Node.js runtime.
Check it out, and it should resolve the issue.


Can i map a dpad-stick to analog-stick?

Hi,
by the way: thanks for the great tool!
I have an "pc-modded" arcade cab (Legends Ultimate) and try to map the dpad Sticks i. e. P1 to the emulated analog controlstick, but without luck. The emulated stick is not centered. I think cause of the mapping -> "one dpad direction" to one axsis.. Maybe I missed something?

It happens to me too. As i remember, what i did is edit the .ini file and set the value of axis to -1.0 and 1.0 or -100 and 100. Can't clearly remember the value but that is the fix i found

@lesliehu16 Thanks for the hint.
Finally, i've managed to map a dpad-stick to an emulated analog one by looking into the config file "settings.json" and some trial and error:

This is just a work-around. The possibility of mapping d-pad to analog stick in the UI is still malfunctioning.

for me with keyboard it work well.
    "LX": {
      "Mappers": [
       {
           "InputDevice": "Keyboard",
           "InputType": "60",
           "MinValue": 0.5,
           "MaxValue": -0.5,
           "Deadzone": 0.0
       },
       {
           "InputDevice": "Keyboard",
           "InputType": "47",
           "MinValue": 0.0,
           "MaxValue": 2.0,
           "Deadzone": 0.0
       }
     ],
     "CenterPoint": 0.0
    },
    "LY": {
      "Mappers": [
       {
           "InputDevice": "Keyboard",
           "InputType": "62",
           "MinValue": 0.5,
           "MaxValue": -0.5,
           "Deadzone": 0.0
       },
       {
           "InputDevice": "Keyboard",
           "InputType": "69",
           "MinValue": 0.0,
           "MaxValue": 2.0,
           "Deadzone": 0.0
       }
     ],
     "CenterPoint": 0.0
    },


[charts/csi-powerstore] Centralize all images to one place

Is this a new chart?
No
What this PR does / why we need it:
Which issue(s) is this PR associated with:

KRV-12738

Special notes for your reviewer:
Configure the latest official CSI sidecars : https://kubernetes-csi.github.io/docs/
Checklist:
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[ ] Chart Version bumped
[x] Variables are documented in the chart README.md
[x] Title of the PR starts with the chart name (e.g. [charts_dir/mychartname]) if applicable

Can the base branch be updated to merge into release-v1.9.0 for these changes?


Rename master to main

See https://chat.hyperledger.org/channel/perun?msg=eLeA7iSZF6eboYqk9

done


Fix creating executable JAR command in basic.adoc

Gradle docs suggest no space after -P:
https://docs.gradle.org/current/userguide/build_environment.html#sec:project_properties


Force dependencies to work on PHP 5.3

:-1:
this is not a good solution IMO, that way, regardless of env, you require the packages to be compatible with old PHP, even if we run composer under PHP7.
This flag may be used for build, but not for regular usage.

@keradus This flag is not inherited, so it doesn't affect anything when php-cs-fixer is used as a dependency. It only affects building it, or running it's own tests.

exactly, it disallow us for testing newest packages that we claim to support, eg Symgony 3.x

exactly, it disallow us for testing newest packages that we claim to support, eg Symgony 3.x

Yes it does. That is unfortunate. This PR is just a suggestion though. ;)

It may be partial solution until we will attaching the phars for github releases.
In the meantime, could you remove that added part in Travis jobs ?

For me the best option is to require at least an LTS version of every symfony component, an then execute "composer update --prefer-lowest" before building the phar.

which cannot be done untill we will modify way of building and storing phars, as I mentioned.

@GrahamCampbell
The current problem is that we will only test 5.3.6 compatible version of packages, eg never test Symfony 3.x components.
Please, for build matrix whenever we don't use --prefer-lowest add --ignore-platform-reqs.
That way, on 5.3 we will check lowest deps compatible with 5.3.6, and on all other we will test highest deps possible for given PHP version (ignoring 5.3.6 requirement, as it was tested in first job)

The current problem is that we will only test 5.3.6 compatible version of packages, eg never test Symfony 3.x components.

Yeh. I know. D'no what to recommend here I'm afraid, other than having two composer.json files, which would really suck.

If it works, sure. I don't know how the phars are built.

ping @GrahamCampbell

?

@GrahamCampbell  ?

 I only put it here to motivate something better. :P

I thought we agreed on solution and we don't see anything better so far, so why closing ?


lunar-client: 3.2.19 -> 3.2.24

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for lunar-client is: Free Minecraft client with mods, cosmetics, and performance boost
meta.homepage for lunar-client is: https://www.lunarclient.com/
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 3.2.24 with grep in /nix/store/y67l6lipl3krlcssq37z9s8n8s549lb6-lunarclient-3.2.24
found 3.2.24 in filename of file in /nix/store/y67l6lipl3krlcssq37z9s8n8s549lb6-lunarclient-3.2.24

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

lunar-client

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/y67l6lipl3krlcssq37z9s8n8s549lb6-lunarclient-3.2.24 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A lunar-client https://github.com/r-ryantm/nixpkgs/archive/79578a172732a1d40af9bc371e464862b011891e.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/79578a172732a1d40af9bc371e464862b011891e#lunar-client

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/y67l6lipl3krlcssq37z9s8n8s549lb6-lunarclient-3.2.24
ls -la /nix/store/y67l6lipl3krlcssq37z9s8n8s549lb6-lunarclient-3.2.24/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review

x86_64-linux

:white_check_mark: 1 package built:

lunar-client

Maintainer pings
cc @Technical27 @surfaceflinger for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

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

@NixOS/nixpkgs-merge-bot merge


cli: create-plugin should handle build and type checking failures more gracefully

Expected Behavior
When using the backstage-cli create-plugin some build step tasks like yarn tsc might fail. These aren't required to be run and we could handle failures in a more graceful way, simply warning the user that the build task failed.
Current Behavior
If for example yarn tsc fails, the following error is logged:
It seems that something went wrong when creating the plugin 🤔
We are going to clean up, and then you can try again.

The new plugin is removed as well.
Possible Solution
If any of the tasks following the creation step fails, yarn install and beyond, we should just warn the user about the failure but not remove the plugin.
Steps to Reproduce

Introduce for example a TypeScript issue that makes yarn tsc fail
Try to create a new plugin using yarn backstage-cli create-plugin

Context
https://discordapp.com/channels/687207715902193673/687235481154617364/773303667675758643

Thanks @Rugvip .

Hey. I would like to give it a try and fix this bug!

Thanks @tudi2d, assigned you! :grin:

I decided for a straightforward solution by replacing the throw new Error(...) with Task.error('Warning: ...) and it would look something like this:

This fixes the bug and should give the user enough information. Nevertheless it might be more elegant and longer lasting to add a Task.warn(...) to the class Task and use that instead.
Is this solution fine for this bug or should I solve it more generally?


Wrong version number

The github release says it's version  0.1.3, but running ./mpris-discord-rpc --version shows 0.1.1

Well, I forgot to update the version number in the Cargo.toml file here:
https://github.com/patryk-ku/mpris-discord-rpc/blob/8119dd1273c2ce06f78a90a56384405fdbb51229/Cargo.toml#L3
I will fix this in the next version, which will come out soon. Thanks for pointing this out.

Fixed in v0.1.4


Problem with upper case column names #22

To prevent identifiers from being lower cased by SansOrm, you can now enclose them in quotes, e. g. @Column(name = "\"FIELD_NAME\"") SansOrm will than behave like most databases and will leave them as is.  The usage of double quotes for this seems to be common practice.

Codecov Report

Merging #23 into master will increase coverage by 5.83%.
The diff coverage is 91.42%.

@@             Coverage Diff              @@
##             master      #23      +/-   ##
============================================
+ Coverage     60.27%   66.11%   +5.83%
- Complexity      184      218      +34
============================================
  Files            15       15
  Lines           881      900      +19
  Branches        154      161       +7
============================================
+ Hits            531      595      +64
+ Misses          261      225      -36
+ Partials         89       80       -9

Impacted Files
Coverage Δ
Complexity Δ

...in/java/com/zaxxer/sansorm/internal/OrmWriter.java
77.41% <100%> (+10.31%)
31 <0> (+7)
:arrow_up:

...main/java/com/zaxxer/sansorm/internal/OrmBase.java
77.58% <85.71%> (+26.67%)
20 <0> (+8)
:arrow_up:

...java/com/zaxxer/sansorm/internal/Introspected.java
66.81% <91.66%> (+9.67%)
56 <11> (+19)
:arrow_up:

Continue to review full report at Codecov.

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

Hello Brett, when do you think that you can review my PR, so we know, if we can utilize SansOrm in production?

SansORM v3.3 has been published.

Thanks a lot!

@brettwooldridge There is still inconsistent behaviour wird @Id columns. Especially
@Id String Id2
will result in "id2", not "Id2". See CaseSensitiveDatabasesTest#getIdColumnNames(). This seems not to comply with sergiorussia's explanation "all entities in my project are annotated using only standard JPA annotations, and rely on the documented behavior that maps empty Table and Column names to entity and field names accordingly" (see PR #12).
sergiorussia's work "making Introspector case-insensitive regarding column names" (PR #12) is a big improvement. Beyond that I would propose to take the column and table names as is when building the SQL statements, that is not to lower case them. So in case of case sensitive databases (and in our case ;-)) one could avoid the double quotes at all. And with case insensitve databases it does not matter.

Is an @Id without a @Column valid?

I am just a JPA novice, but found these examples in "JBoss Enterprise Application Platform 4.2
Hibernate Annotations Reference Guide":
@Id @GeneratedValue(strategy=GenerationType.SEQUENCE, generator="SEQ_STORE")
public Integer getId() { ... }

@Entity
public class Flight implements Serializable {
    Long id;

    @Id
    public Long getId() { return id; }

    public void setId(Long id) { this.id = id; }
}


Fix severity when report contains no vulnerabilities

fixes: https://github.com/unioslo/harborapi/issues/91

@pederhan can you look ? After upgrade to newest harbor this issue is a blocker for me

Bumping to 0.25.2 does not help

Sorry for the delay. I will take a look ASAP.

No problem , i assume this is normal during summer on university :)


Use full lodash

It is very likely to already be in the dependency tree, this saves requests during npm install. I couldn't see the benefit of installing the separate modules instead of the full package.

Coverage remained the same at 98.361% when pulling 441f6f1a1b8c33064df0f2381ee530b0bb4e26c1 on oliversalzburg:fix/lodash into 5a417cf54cf90e8592e950e8721f87033e85f4fc on gulpjs:master.

No, sorry. This package is being deprecated and should no longer be used. Use lodash directly.

How disappointing :( Because people still use the package and it's being pulled in as dependency all the time, causing needlessly many requests.


Uploads are very slow compared to previous versions

Storage Explorer Version: 1.4.1
Platform/OS Version: Windows Server 2016
Architecture: ia32
Regression From: 1.2 IIRC
Steps to Reproduce:

Import a typed csv file into a table with only half a dozen columns
It takes many seconds before the upload is listed in the Activities section of the window
It takes a long time to load 100 entities.  I've watching it now for over a minute and it is stuck on 1700 entities in a table that has around 6000 rows.

Expected Experience:
Responsive feedback about progress, entities are uploaded quickly.
Actual Experience:
It's slower than a wet week.

This is a fairly old issue and many changes have been made to uploads & downloads since it was open. If you're still having problems with uploads & downloads, please open a new issue against the latest version of Storage Explorer. Thanks.


Heatpump jpegs are not available

Cannot find the files:
vp_base.jpg
vp_base_hgw_on.jpg
vp_base_hw.jpg
They should be included in the component.

Fixed v1.1.2
Needs to be moved into correct place...


Login glitch

Prerequisites

[x] I am running the latest version
[x] I checked to make sure that this issue has not already been filed

Expected Behavior
We are supposed to be logged in or signed up when we click the button of signing/log up with facebook/google.
Current Behavior
Currently, it does nothing and the button turns white like this

Failure Information (for bugs)
This is a bug as it fails to connect us to facebook or google but it enables to connect to twitter when logged in and when we click the about button. So this is a resolvable issue
Steps to Reproduce
Please provide detailed steps for reproducing the issue.

Running the project fact-bounty on localhost your system
Try to login via Facebook/google
It doesn't get us signed up or logged in if not done manually by adding name and email and password

Failure Logs

is this open?

do you add the relevant keys in the .env file? @kunalbhatia18

@wdevon99 I added REACT_APP_OAUTHD_KEY and REACT_APP_OAUTHD_URL to set up the project.

@kunalbhatia18 is this issue resolved?


LED Strip won't light up

Hello everyone, I am using WS2813 lights the strand test is working and the visualzation.py is working I am using a microphone for the input audio. However, once I run visualization.py and give it some sound the lights do not light up.

Are you running it on Raspberry Pi or Arduino?

ip in Config.py and in codes should be matched but you also have to change ip in config.py to the ip of your esp8266..by default it have different ip


Doesn't start!

Short description of the problem:
Fresh clone of the app and I can't seem to get it to work.  Chrome opens and I see the three windows but they're just white and there is an error in the console
What behavior are you expecting?
It to work.
Steps to reproduce:

git clone
npm install
ionic serve --lab

Gets the following console errors and the app is just white:
reflective_injector.js:689Uncaught No provider for Token Platform Pipes! (CompilerConfig -> Token Platform Pipes)

Other information: (e.g. stacktraces, related issues, suggestions how to fix, stackoverflow links, forum links, etc)
Run ionic info from terminal/cmd prompt: (paste output below)
$ ionic info
Your system information:
Cordova CLI: 6.3.0
Ionic Framework Version: 2.0.0-beta.11
Ionic CLI Version: 2.0.0-beta.36
Ionic App Lib Version: 2.0.0-beta.19
ios-deploy version: Not installed
ios-sim version: Not installed
OS: Mac OS X El Capitan
Node Version: v6.2.2
Xcode version: Not installed

I believe this is definitely related to the release of angular 2.0.0-rc.5 which occurred today. The application hasn't been updated to support the changes made in rc.5 which is causing the error.

Yeah; I was thinking the same after some googling... I'll try locking the dependencies and reporting back...

Sorted it, stuck in a pull request to lock it for the time being as I don't have the skills to fix this yet (just embarked on learning angular.js today :-))


How integrate stt without cognitive services? [Question]

Hi, I want to integrate BotFramework-Emulator with external services for stt like watson without cognitive services. I can’t find documentation about that

This is outside the scope of this forum.  This repository is for Issue and bugs within the emulator.  For a question like this please ask over on StackOverflow where the community can probably help you.


Add buildroot setting for rotating the sensor

Please add a setting to allow the inclusion of flip=3 into imp.conf

Adding shvflip=1 to BR2_SENSOR_PARAMS in the camera config should do it now.


Update to support rxjs 6.0

This is a forced requirement in new Angular >= 6.0 rc0

Actually it just need to support rxjs 5.6+, there's a document coming up soon that explains that.
I'll make the update once ngpackagr is updated to support the new Angular package format, it will probably be updated this weekend

Have some issues here. As the the new material2 6.0 beta package requires angular 6.0 rc, and angular 6.0 rc requires rxjs6.0 beta.
When all beta packages installed, the project will failed at compile:
ERROR in node_modules/@ngx-translate/core/src/translate.directive.d.ts(2,10): error TS2305: Module '"C:/Users/XXX/node_modules/rxjs/Subscription"' has no exported member 'Subscription'.
node_modules/@ngx-translate/core/src/translate.loader.d.ts(1,10): error TS2305: Module '"C:/Users/XXX/node_modules/rxjs/Observable"' has no exported member 'Observable'.
node_modules/@ngx-translate/core/src/translate.service.d.ts(2,10): error TS2305: Module '"C:/Users/XXX/node_modules/rxjs/Observable"' has no exported member 'Observable'.

Version 10.0.0 has been released with support for Angular 6 and RxJS 6, can you try and tell me if it works for you?

@ocombe Thanks so much for the next release.
It works! There is no more error now. Thanks!

Thanks for the release!!! I don't know why but since I upgraded I get Http failure response for http://localhost:4200/assets/i18n/en.json: 404 Not Found

Sorry, the 404 Not found is a problem with the Angular-Cli 6 RC01 and RC02, not a problem with this library.


Timings

added a timing collector to monitor how long each request took.
fixes #112

ping @egeloen ?

What does this change in comparison to using the stopwatch subscriber (https://github.com/egeloen/ivory-http-adapter/blob/master/doc/events.md#stopwatch) ?

This is a known limitation of the library and will not  be fixed as this repository is deprecated in favor of https://github.com/php-http The good fix is to upgrade following http://docs.httplug.io/en/latest/httplug/migrating/


Improve documentation

This is really a draft. Locally, I still have some notes with questions to continue with this documentation. I didn't commit these but we should probably discuss them before moving on.
Some other commits in this PR fix small quickbook and doxygen problems in passing. We could cherry-pick some of these but there's nothing critical in there.

Can we please rebase and squash this down?

Can we please rebase and squash this down?

I'm rebasing it but I wasn't planning to squash it down while this is still a draft. The reason is this PR has a lot of unrelated stuff and it was my understanding that you'd rather have that go into smaller PRs.

So the PR has been rebased. The commits were squashed into fewer commits containing each of the unrelated changes in this PR:

Include the C++11 requirement in the CMake target
Fix lots of small quickbook and doxygen warnings
Updated guidelines in javadoc.hpp
Javadoc for string_value, params_encoded, and params (The javadoc is better than before but there are still some guidelines from javadoc.hpp missing)

If any of these changes are considered useful while this PR is still a draft, I can cherry-pick them into another branch and open another PR. If everything is considered useful, I can squash them into a single commit.
(This PR contains all these unrelated changes mostly because I wanted to take advantage of the common documentation preview while this is still a draft.)

Changes to CMakeLists.txt should be in a separate pull request, in a single commit

Lets not put backticks in the commit message briefs because it makes them longer and less uniform

Lets not put backticks in the commit message briefs because it makes them longer and less uniform

Done.

I'm closing this since the draft preview is not very useful at the moment and everything else is either useless or wrong.


Release 0.2.0

Version 0.2.0 - released December 28, 2018

Geometry: Labeled polygon angles
Internal: Image storage refactor
Internal: Rollbar support
Internal: Cypress integration tests

0.2.0 to be precise 😉


migrate kured from helm2 to helm3

Hi, we have kured installed in our AKS cluster with helm2 from stable/kured(https://github.com/helm/charts/tree/master/stable/kured).
We are trying to migrate this chart to use helm3. Seems now the repo has been changed to https://weaveworks.github.io/kured. Can I use the helm2to3 plugin to migrate this chart or I need to remove this chart which is installed by helm2 and then reinstall with helm3 by
helm repo add kured https://weaveworks.github.io/kured  helm install my-release kured/kured

Generally, you should be able to migrate the chart with this utility to helm3. But this does only migrate the release-information to the new storage format helm3 uses. (I didn't test it for kured)
In the second step you can upgrade the chart to use the new repository. This should work without a clean reinstall. However this shouldn't be a problem, since kured does not store and persistent data.

Sounds like there are two steps:
1.use helm2to3 plugin to migrate this chart to helm3
2.do "helm upgrade" on this chart to update this chart to use new repository

exactly!

Shall we document this somewhere in our docs?

We can document, that the migration to the new chart is only a simple "helm upgrade" command with the new repo and parameters....
I wouldn't document the helm2to3 thing, since this is not related to kured and is generally useable for any helm-chart.

You can use the guide https://helm.sh/blog/migrate-from-helm-v2-to-helm-v3/ to migrate. It has some trickiness depending on how charts have been designed (I had some conflicting labels).
On the other side the kured chart is stateless (no ingresses to be reached, no volumes, no services exposed which will break anything) - so the easiest could be a simple reinstall.


How to tell shaka player to call a custom byte range fetcher function instead of directly doing HTTP range requests

What version of Shaka Player are you using?
4.7.7
What browser and OS are you using?
Chrome, MacOS
Please ask your question
I have a WASM function that is capable to fetch byte range of a video. My webapp cannot do HTTP range requests directly to the server. But the wasm function does a lot of complex things to fetch the exact byte range
How can I tell shaka player to fetch the required byte range by calling the wasm function instead of making HTTP range requests by itself. What options do I have?

I have implemented a feature in which a custom fetcher can be set. For additional information, please refer to the linked pull request (PR).
I am relatively new to object-oriented programming in JavaScript, and I have done my best to create code that I've tested and found to work effectively within my project.
If this feature aligns with the goals of Shaka Player, I am open to receiving your code reviews and suggestions on the PR.
However, if this feature is not considered desirable for the Shaka Player project, please feel free to close the pull request.


try analytics

Builds started to failed because one of the Next builds were misconfigured. (related to next-ga)
Then, that build failed, but somehow got aliased. (NOTE THIS FOR LATER)
Then, subsequent build failed when their Next builds were trying to reach {URL}/api/companies.
Current fix is to allow getInitialProps to fail, and perform the data fetching client-side instead when platformsData prop is default ([]).
@resir014