[Bug]: Pulling from CDN gives latest 0.x.x version

Current and expected behavior
I would expect a version available from CDN for latest V1+
Reproduction
n/a
Chartist version
v0.11.x
Possible solution
No response

Hi, which CDN are you referring to? Cheers

I am using https://cdn.jsdelivr.net/chartist.js/latest/chartist.min.js referenced from the old docs.
The links you sent work.  It would be great to add the new docs. Thanks for your help!
Best,
Kevin


fuel_gauge: Add fuel_guage_get_prop/s()

The fuel_gauge_get_prop() function prototype declares a function that retrieves multiple fuel gauge properties at once. The naming suggests it ought to fetch a singular property at a time. Moreso, some clients may just want to fetch properties one at a time and may feel uncomfortable using a prototype for fetching multiple properties when wanting to fetch them one at a time.
Modify fuel_gauge_get_prop() to fetch a single property and add fuel_gauge_get_props() to support fetching multiple properties. Modify existing tests/drivers/samples.
This is part of #61818 work.

Doing this before https://github.com/zephyrproject-rtos/zephyr/pull/62366 - SBS Property Moving.


host_app_feedback.js inconsistent with other platforms (outdated?)

The host_app_feedback.js included with the launcher is inconsistent with the one in the iOS/OSX launchers.
For example:
It includes a hook into ReadiumSDK.Events.CONTENT_DOCUMENT_LOADED (the others do not) and the handler is missing the right callback signature:
this.onContentLoaded = function() {
should be something like
this.onContentLoaded = function($iframe, spineItem) {

Thanks @JCCR !
I agree about cleaning-up.
Currently though, this is only used to print a debug message, so the parameters are unused:
@JavascriptInterface
public void onContentLoaded() {
if (!quiet)
Log.d(TAG, "onContentLoaded");
}

https://github.com/readium/SDKLauncher-Android/blob/develop/SDKLauncher-Android/src/org/readium/sdk/android/launcher/WebViewActivity.java#L751
To compare:
SDKLauncher-Android:
https://github.com/readium/SDKLauncher-Android/blob/develop/SDKLauncher-Android/assets/readium-shared-js/host_app_feedback.js#L75
SDKLauncher-OSX:
https://github.com/readium/SDKLauncher-OSX/blob/develop/LauncherOSX/ReaderScripts/host_app_feedback.js#L82
SDKLauncher-iOS:
https://github.com/readium/SDKLauncher-iOS/blob/develop/Resources/host_app_feedback.js#L81
SDKLauncher-Windows:
https://github.com/readium/SDKLauncher-Windows/blob/feature/feature_revised_launcher/SDKLauncher-Win/ReaderScripts/host_app_feedback.js#L24

Note that this will be addressed when we merge the feature/plugins branch to develop:
https://github.com/readium/SDKLauncher-Android/blob/feature/plugins/SDKLauncher-Android/assets/readium-shared-js/host_app_feedback.js

Need to review this now that the restructuring and plugins work has been merged.

As expressed in the above comment ( https://github.com/readium/SDKLauncher-Android/issues/49#issuecomment-71539027 ), the CONTENT_DOCUMENT_LOADED Readium event is not actually used for anything other than a debug message in SDKLauncher-Android. Other than that, the host_app_feedback.js has been revamped significantly since the RequireJS / Plugins merge to the develop branch. I suggest we close this issue now.


Feature: Allow deleting of local Git branches

Resolved / Related Issues

[x] Were these changes approved in an issue or discussion with the project maintainers? In order to prevent extra work, feature requests and changes to the codebase must be approved before the pull request will be reviewed. This prevents extra work for the contributors and maintainers.
Closes #13592

Important

Backup your repo before testing this feature
Check this PR doesn't affect performance in Git folders

I had to revert a change made in #13608 because Ahead / Behind wouldn't be update after Pulls (see BaseShellPage.cs)

Validation
How did you test these changes?

[x] Did you build the app and test your changes?
[ ] Did you check for accessibility? You can use Accessibility Insights for this.
[ ] Did you remove any strings from the en-us resource file?

[ ] Did you search the solution to see if the string is still being used?

[x] Did you implement any design changes to an existing feature?

[ ] Was this change approved?

[x] Are there any other steps that were used to validate these changes?

Open a git repo
Click on the branch name in the status bar
Click on Delete to delete a local branch
Click on Cancel and make sure the branch is still there
Repeat but click on Continue and check that the branch is deleted

Screenshots (optional)

We still haven't finalized the design for this feature.


[Question] What is the main difference between OpenCvSharp for C# & For mobile?

I've seen some differences in OpenCvSharp folder.
Or, basically my question is what is needed to make OpenCvSharp work in mobile platform?
thanks

OpenCvSharp is a wrapper of OpenCV to .NET, the problem is that is is not compatible with the ARM architecture, which means it will not run on smartphones. In other words, it all needs to be "adapted" to run on ARM devices.

I agree w/ @craftindo request. Is it possible to have the code changes documented in the Wiki or somewhere to show differences?
I would love to help contribute to this repository, however, the unfamiliarity with the areas that have been modified limits one's effectiveness.  Thank you for making this fork!


draw point different in firefox and chrome/ie

using line tool, set width to 3px, click on canvas, in firefox i draw a circle with r=3px.
export the json and import in chrome or ie, there is no such circle. to import in firefox is well.

Is this the same as #255 or #254? Can you provide the string you're importing, with just the one circle in it?

not same.
it is nothing about importing. just drawing is defferent.
acctually i am drawing a point by one click with the Line tool with width = 3px. in firefox i can draw a big point like a circle in this way, while draw nothing in chrome or ie.
it seems to draw a line (( x0, y0, ), ( x1, y1 )), if x0 == x1 and y0 == y1, in firefox it draws a point and in chrome not.
you may test by these steps:
1, select line tool and set width to a big value.
2, single click on cavas, do not drag.
if you are using firefox you get a bit point like a circle. but in chrome you get nothing.

by the way, to draw a rectangle or ellipse with x0 == x1 or y0 == y1, in chrome get nothing while in firefox get a line.

Fixed for v0.4.5.


[Question][datasource] How to configure impala data source

Describe the question
Currently, the Impala data source is configured with port 21050. The point test connection is always in the loading state. No error is found in the log of the background API module
Which version of DolphinScheduler:
-[1.3.1-release]

如何配置impala数据源。
目前配置impala 数据源，端口21050，点测试连接一直处于加载状态，后台API模块日志内没有发现错误
DolphinScheduler的哪个版本：
-[1.3.1-release]

This issue has been resolved,Impala can support hive-jdbc/impala-jdbc connection, currently dolphin supports hive-jdbc connection


JSON Schema from JSON-Data

What feature or new tool do you think should be added to DevToys?
We already have some JSON tools in the DevToys, all of which I consider to be really useful. What I personally am still missing is a converter that creates a JSON schema from JSON data. An example of how this works can be found here: like: https://www.liquid-technologies.com/online-json-to-schema-converter
Why do you think this is needed?
I use JSON schemas primarily for testing endpoints. This allows me to verify that all properties are still being returned as expected and that no one has changed the data types or other things.
Solution/Idea
Sample-Input:
{
  "name": "Alice",
  "age": 24,
  "address": {
    "street": "Main Str.",
    "no": 12,
    "postcode": "1234"
  },
  "gender": "FEMALE"
}

Sample-Output:
{
  "type": "object",
  "properties": {
    "name": {
      "type": "string"
    },
    "age": {
      "type": "integer"
    },
    "address": {
      "type": "object",
      "properties": {
        "street": {
          "type": "string"
        },
        "no": {
          "type": "integer"
        },
        "postcode": {
          "type": "string"
        }
      },
      "additionalProperties": false,
      "required": [
        "street",
        "no",
        "postcode"
      ]
    },
    "gender": {
      "type": "string"
    }
  },
  "additionalProperties": false,
  "required": [
    "name",
    "age",
    "address",
    "gender"
  ]
}

Comments
No response

HI,
I wonder if this could be added to the JSON Schema extension made by @benyaa ?

I believe it can, although I'm not sure it should be in the same tool as the Json schema validator as it is a tester and this feature is a generator.

HI, I wonder if this could be added to the JSON Schema extension made by @benyaa ?

I believe it can, although I'm not sure it should be part of the Json schema validator as it is a tester and this feature is a generator. But it shouldn't take that long to make one, it would have the same GUI and use the same library to accomplish this.

Indeed. I would suggest to make it a separated tool, and either in the same extension, or in a new one.
If both use the same library to handle the json schema, it may be better to keep it in the same extension.

okay, took the last hour to do it and added another tools on top of it to generate classes out of json schemas. NJsonSchema is just wonderful

okay, I spent the last hour implementing it and added another tools on top of it to generate classes out of json schemas. NJsonSchema is just wonderful

Wow, this is really awesome and exactly what I've been looking for! Thank you so much!

okay, I spent the last hour implementing it and added another tools on top of it to generate classes out of json schemas. NJsonSchema is just wonderful

Wow wow wow that was quick! 😁👏 Thank you so much!


Changed Testplatfrom to Testplatform

While trying to get coded UI Tests to work on my system, I saw this in a log file and figured, I can fix that.  :)

Looks like this file no longer exists


initial

No changes... Apparently duplicate the files, I don't understand why...

Yeah that's because you added two repositories that are not supposed to be on the git (only generated locally when building the module): omegapy.egg-info and build/lib/omegapy/

You can remove them from your Git forked repo.

New try, with update of omega_data only ! For the moment read all the NAV the plane into the out_geom dict and only put phase in self

Looks much better, thanks! Indeed it's probably better to read all the NAV data.
Just a question about the L-geom: is there an actual difference with the C-geom? I know that the V channel is not co-aligned but I thought the C and L ones have the exact same fov?

Ok, thanks for the information!
Yes I can see your modifications, it looks good to me. I'm gonna merge them into the branch to make some tests, then include them into the main branch and upgrade the module to v3.2. ;)


Update parser

Update the general parser to decide which parser should handle the given command.

Merged, #45


libreoffice-fresh: Fix shortcuts

Closes #4208
Closes #5059

extract_dir added to libreoffice-fresh.json, per request.


Plug-ins can be installed multiple times which causes issues

Ensure we check for plugins before running the install process. If a plugin exists it should be updated rather than installed from scratch.

This should be fixed in the latest main, also I added a update-plugin.sh script


Sorry, an error occurred while trying to access this service.

Hi
After adding the author app to Teams I am getting this error when I access the company communicator tab:
Sorry, an error occurred while trying to access this service.
I was able to add the app with no issues at all, but quite simply cannot figure out why it is not working.
I went through the whole configuration two times, but it seems to be in order according to the deployment guide.
Let me know if you need more info
UPDATE
I looked into application insights for the app and it seems there is and issue with some failed dependencies:
Type | Azure table |
Result code | 409 |
Call status | false |
Duration | 566 ms |
Remote dependency name | POST ruyljbyaow5m2/Tables |
Id | cdce30dc7054d047 |
Base name | ruyljbyaow5m2.table.core.windows.net |
Performance | 500ms-1sec |
Telemetry type | dependency
There is also a command related to this:
https://ruyljbyaow5m2.table.core.windows.net:443/Tables()
Finally I am also seeing a response code 401 in relation to GET /api/sentnotifications and GET /api/draftnotifications
I have no idea what any of this means, so I am hoping some in here can point me in the right direction.
Regards,
Michael

Hi
I had the same problem. I found the following worked:
In the application configuration, make sure all the urls are pointing to the uri for the service and not the frontdoor url.
I noticed that when the bit is deployed by the arm template it uses the azurefd.net address

Hi
But it should be possible to use the azurefd.net address right?
Or I am just not sure what the url for the service would like then.
Are you able to give me an example?

@Miclarsen1 : It must be working with azurefd.net URL.
Please double check Author app registration authentication configuration and make sure correct values entered in ARM template deployment:

Hey @SubbaReddi,
Ok thanks for the hint.
The Issue was the Application settings in Azure. For some reason the AllowedTenants and AzureAd:TenantId were both wrong, but after correcting that it all worked as intended.
Thank you again for the assist, you are a gentleman and a scholar :-)
Best regards,
Michael


Quat CreateRoll Error

Spotted a potential minor mistake while browsing,
https://github.com/Remedy-Entertainment/binderoo/blob/master/binderoo_client/d/src/binderoo/math/quaternion.d#L96

There's a few problems with the maths stuff...
Any 'lookat' or 'pyr' (ie, this) functions require specification of 'meanings' to the axiis. Of course, sensible developers would roll with RH, Z-Up, but most games actually don't, instead choosing either OpenGL or DX's default spaces... which are Y-Up, and Z-Unsure ;)
I wonder if it would be best to remove these functions from the API, or supply template args to the functions stating which axiis are 'up' and' forward'...? (I've never heard if a working space where X is not 'right', but I guess it could theoretically exist...)

Or alternatively, keep these modules in the game-specific code, and not in here?

Yep, that's an error sprinkle.
The main motivation for pushing the math code in to binderoo was so that I could have it in one place where I wouldn't have to write it again (it's a D port of the C++ code I've written, and it's clearly a poor port since this is not the first mistake I've found in the porting).
I've also been thinking of compartmentalising functionality more - rather than specialising functions within the mathematical constructs, make external constructs that know how to pass the correct values to the basics for the selected coordinate systems. Thus, the "remove the functions from the API" would be the route I'd go and have supplemental APIs that can handle different coordinate systems.
It needs a proper think through. Notice the angle module, which was designed to solve the problem of programmers being used to working with radians while content guys are far more comfortable with degrees. I'm certain there's a better overall solution somewhere for handling different spaces of all sorts.


feat: Add process metrics using psutil

Requires https://github.com/DSD-DBS/capella-collab-manager/issues/1946 before merging.


[Question] : Skins Application

This is a fantastic piece of code. I found it when I was googling problems from another template that I used. I am having a difficult time figuring out how the skins and style are applied in this design. I see the calls to the skins in the wwwroot when I'm looking in the logs. I just don't see where the calls to the skins are being made in code. Any help would be greatly appreciated.

So there's bootstrap sass being used, so you could completely theme the site in anyway you want using the bootstrap variables and simply overriding them (or even picking some sass themed options online and changing it from there). Thanks for the kind words 🙏


Updating version for release

Bumping to 0.1 since minor version is traditional for pre-1.0 releases.

Thanks for that.


Filtered rebreather has no resistance in modifiers

What item is incorrect/missing?
Please enter its exact name as it appears in the compendium (or the book if the item is missing)
Filtered rebreather
What book is the item from? What page is it on? If possible, include an Archives of Nethys link
CRB 205
https://aonsrd.com/ArmorUpgrades.aspx?ItemName=Filtered Rebreather&Family=None
SFRPG system version
0.26.1
(If the item is incorrect):
How is it incorrect?
A clear and concise description of what's wrong. Including direct quotations is appreciated.
Has no acid resistance in modifiers
What correction should be made?
A clear and concise description of how to fix it. Again, using direct quotations is appreciated.
Add Acid resistance 5 as a modifier

This has been fixed by #1354  on the development branch


[Feature]: Collect measurements of useful things

Contact Details
No response
Your request
As a bot owner I want to see some interesting measurements regarding the bot's performance.
A few examples:

Total songs played;
Currently joined servers;
Amount of concurrent songs currently playing;
???

More suggestions are welcome.

Is relevant to #82 in somewhay


Charmik/add collections support  to vuln reporting

Description
A detailed explanation of the changes in your PR.
Feel free to remove this section if it is overkill for your PR, and the title of your PR is sufficiently descriptive.
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
TODO(replace-me)
Use this space to explain how you tested your PR, or, if you didn't test it, why
you did not do so. Valid reasons include, for example, "CI is sufficient",
"No testable changes". Feel free to attach JSON snippets, curl commands,
screenshots.
In addition to reviewing your code, reviewers must also review your testing
instructions and make sure they are sufficient.

/test gke-postgres-nongroovy-e2e-tests

While these changes are being done, can you add the collection id search tag to the scope_id in the proto of report configurations?

While these changes are being done, can you add the collection id search tag to the scope_id in the proto of report configurations?

Created ROX-14220 to make report configs searchable by scope_id. Adding  it in a separate PR so that this can be merged to unblock the unit tests PR.

/retest


WIP: Approximate convex hulls

This work in progress PR aims to produce ε-apporixmate convex hulls using the algorithm of 1603.04422, with CVXOPT being used to solve the quadratic program at the innermost loop. Unfortunately, this approach results in CVXOPT taking up a truly absurd ratio of the runtime, so the current implementation is far too slow to be useful at the moment. That said, it seems to work reasonably well in local testing, even if its too slow to be usable.

Coverage decreased (-1.0%) to 66.722% when pulling 1c6be1e9928a7979128b5f78d2d7bbb14d4fbdc0 on feature-approx-chull into ddac8218f5367f303a7e98e191fbf4b7731e2af5 on master.

Coverage decreased (-1.0%) to 66.722% when pulling 1c6be1e9928a7979128b5f78d2d7bbb14d4fbdc0 on feature-approx-chull into ddac8218f5367f303a7e98e191fbf4b7731e2af5 on master.

Coverage decreased (-1.2%) to 66.556% when pulling 9aa4394034ce360a54b1a4cba8fb2b4648130dfa on feature-approx-chull into ddac8218f5367f303a7e98e191fbf4b7731e2af5 on master.


unhandled exceptin during ng update

🐞 Bug report
Command (mark with an x)

[ ] new
[ ] build
[ ] serve
[ ] test
[ ] e2e
[ ] generate
[ ] add
[x] update
[ ] lint
[ ] extract-i18n
[ ] run
[ ] config
[ ] help
[ ] version
[ ] doc

 Yes, the previous version in which this bug was not present was: ....
Description
 A clear and concise description of the problem...
ng update fails with an error.
🔬 Minimal Reproduction
ng update

Installed packages: (npm list)

├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>└──<EMAIL_ADDRESS>

🔥 Exception or Error

🌍 Your Environment

Your global Angular CLI version (12.0.0) is greater than your local version (11.2.13). The local Angular CLI version is used.

OS: Pop!_OS 20.04 LTS x86_64

I had the same. It was because my global installation of @angular/cli was not on the same version (already bumped to v12 while my local @angular/cli was in version 11).
Maybe make sure you have @angular/cli installed globally and that your local and global installations are on the same version ?

Please update your global CLI to the latest version.
npm i -g @angular/cli

The issue was caused by a defect in 12.0.0 which was fixed in 12.0.1.  Installing the latest version of the CLI globally first will solve the issue.
The local and global versions do not need to be the same.  Installing the latest global first will actually remove the need for the local version to download and install a newer version during the update initialization process.

Oh thank goodness there is a fix for this. It's been driving me nuts trying to debug this for the last hour+.


darwin.xcode: add automated download helper

Description of changes

Adds a script to semi-automate fetching Xcode. Companion to #245205
Personally, I'd be quite interested in working towards having Xcode as a supported build tool internally in nixpkgs, especially for packaging open-source macOS applications. However the experience right now for builders is understandably too clunky. I'd love to hear your thoughts if improving the experience might open up the possibility @NixOS/darwin-maintainers
Things done

Built on platform(s)

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

[x] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

The script here seems fine but I don't think I feel good about trying to use Xcode blobs as a build tool within Nix(pkgs). It'll never work with cross builds and it goes against the current migration to using as pure an LLVM toolchain as possible. Ideally we will have fewer binary blobs and bitrotting Apple OSS dumps in the future and move closer to the kinds of Clang toolchains used by Firefox and Chrome to make their macOS builds. The deprecation of xcbuild is admittedly a spanner in the works of this though.
Can you explain the advantages you think using Xcode as a build environment would have over the way the Darwin toolchain has been evolving lately?

@emilazy  The primary motivation would be for open-source projects, such as macOS desktop apps, that use the Xcode build system. Currently there's no way to build them from source and we have to package binary releases. I definitely agree that it doesn't feel great to pull in 20GB of binary blobs just for glorified make. I think if a modern evolution of xcbuild turns up that would be incredible, but right now I'm not seeing a way for nixpkgs to build a sizeable ecosystem of tools from source :(

That's fair. I guess xcbuild has bitrotted enough that it's not good enough to build some contemporary things? (Did it never implement the new build system Xcode added a while ago? Maybe llbuild being open source would help there, though of course that's a bit of a "just write a whole thing" kind of idea.)
Another thing to consider if we want to build substantial Mac GUI apps in Nix is that we also need a story that makes sense for code signing and entitlements, which is another can of worms with potential security model implications :(

Coincidentally I've recently been looking into how we do codesigning. We are able to sign with ad-hoc signatures, but don't currently specify entitlements. There's a certain subset of entitlements that we can grant even with an ad-hoc signature that we currently don't have infrastructure to apply. I was able to successfully test that --entitlements does allow granting com.apple.security.cs.allow-unsigned-executable-memory with this experiment. There's some room for nixpkgs to evolve there and I plan to work on it :)

I’m in alignment with @emilazy. One thing I think would be interesting is if it is possible to use xcodebuild with the nixpkgs toolchain.
My biggest fear with making it too easy (or possible) to use Xcode in derivations is people will default to that instead of using the toolchain provided by nixpkgs.

I think getting xcodebuild to with with a nixpkgs toolchain would be an interesting problem to work on, and it makes sense for compiling packages in nixpkgs. Assuming (based on very cursory research) that it works, you'd at least need to get the xcodebuild binary into hydra.

I don't think we can legally redistribute binary Xcode tools from Hydra.

We definitely could not redistribute xcode tools, but getting a copy into hydra /might/ be less impossible? Appreciate y'all for humouring me on this train of thought :D

AIUI every derivation output Hydra builds is available via cache.nixos.org, so it'd at least require some kind of rearchitecting to have private builds. I could be wrong, though.

Having xcodebuild available as a sort of escape-hatch would be nice, maybe it should be an external (to Nixpkgs) project first?
Are there any lessons we can take from Firefox and Chrome's build systems? I assume they're very tailored to their respective codebases and not using an xcbuild-replacement utility of sorts?

Yeah, I don't think Firefox and Chrome have any reason to involve the Xcode build tools as they're massive cross-platform applications with their own build systems anyway. I'm not sure what they do about SDKs.


Repository views

This PR introduces:

DefaultRecordBasedRepository, which is RecordBasedRepository with default entity construction and conversion behaviour.
Index iterators for storages
Filtering iterator for repositories

@armiol PTAL


build(ci): avoid concurrent deploys

See:

https://github.blog/changelog/2021-04-19-github-actions-limit-workflow-run-or-job-concurrency/
https://docs.github.com/en/actions/reference/workflow-syntax-for-github-actions#jobsjob_idconcurrency

This should fix this problem:

@roschaefer Here’s one thing to think about, but I don’t think it’s a problem. Right now, only the deploy job is limited to one concurrent run. However, the docker_push job will still run concurrently. I was wondering if we should may add the concurrency policy to the whole workflow, not just the deploy job.
The following could happen:

Start workflow at commit 123
Start workflow at commit abc
docker_push job runs for commit 123
docker_push job runs concurrently for commit abc and completes shortly after the first run, i.e. now the Docker image for commit abc is published.
deploy job for commit 123 runs and completes. This job uses the latest Docker image, i.e. the image built at commit abc.
deploy job for commit abc runs and completes.

I don’t really think this would be a problem, but the behavior might be unexpected. For example, imagine you’ve just merged commit abc and then notice a bug in that commit, so you cancel the workflow run. You’d expect that the Docker image at commit 123 would be deployed, but that’s not actually the case. As we’re only deploying to staging on push, consequences are limited though.


Add pull-request data to Bitbucket server collector.

Resolves https://github.com/Hygieia/Hygieia/issues/2662
This PR adds pull-request data to Bitbucket server collector.
Description of changes,

Added com.capitalone.dashboard.collector.PullRequestCollector which is invoked from com.capitalone.dashboard.collector.GitCollectorTask
Introduced com.capitalone.dashboard.bitbucketapi package which contains classes to construct Bitbucket REST API endpoints
Added package-info.java where ever found it missing
Added Hibernate validator dependency which spring boot was complaining when executing Application class from IDE.
Added com.capitalone.dashboard.collector.PageMetadata which encapsulates Bitbucket Pagination information

cc: @Sbrenthughes @meetgds

@shishirchaturvedi - I’ve spoken with legal. Have you signed the two CLA’s? If so I’ll pull this in.

Also do you mind if I move the copyright to a NOTICE.txt file specifically referencing your work?

@chtompki Are you referring to CLA for Hygieia project. I am not aware of it. Could you please share me CLA links/document. I think mentioning copyrights in Notice.txt is a great suggestion. After you share details about CLA, I will get an opinion from my legal team on this.
I really appreciate your effort on moving this topic in a positive direction.

The CLA forms are google forms linked to from here: http://hygieia.github.io/Hygieia/contribute.html

@chtompki I have filled up CLA forms but I haven't received a web response email. Could you please hold on to merging this PR. I would like to review the final CLA agreement with my team.

@shishirchaturvedi -That's fair. I was going to let @Sbrenthughes do the merging as he's more familiar with the application. But we can hold on it for now. Keep us posted

@shishirchaturvedi are you waiting on us for anything?

@Sbrenthughes Yes, please see this comment: https://github.com/Hygieia/Hygieia/issues/2662#issuecomment-490998502

@Sbrenthughes Please let me know how can I help here to get this PR merged.

From my understanding the below is what is left that needs to be done for the headers.

@Sbrenthughes I saw that LICENSE. If you read the very last section of that license, It is recommended that developers should include the boilerplate copyright comment in the source file. What's the recommendation here?
APPENDIX: How to apply the Apache License to your work.
  To apply the Apache License to your work, attach the following
  boilerplate notice, with the fields enclosed by brackets "{}"
  replaced with your own identifying information. (Don't include
  the brackets!)  The text should be enclosed in the appropriate
  comment syntax for the file format. We also recommend that a
  file or class name and description of purpose be included on the
  same "printed page" as the copyright notice for easier
  identification within third-party archives.

Copyright [yyyy] [name of copyright owner]
Licensed under the Apache License, Version 2.0 (the "License");
you may not use this file except in compliance with the License.
You may obtain a copy of the License at
   http://www.apache.org/licenses/LICENSE-2.0

Unless required by applicable law or agreed to in writing, software
distributed under the License is distributed on an "AS IS" BASIS,
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
See the License for the specific language governing permissions and
limitations under the License.

@Sbrenthughes I have update copyrights on this PR. Please let me know if anything else is required in this PR.

Thank you @Sbrenthughes @rvema @chtompki @SanthiSridharan for your support and guidance on this PR/OSS process.

You're welcome. Hope all is well down there in Atlanta these days.


Append a table prefixer

Everything is in the title.
Can we have a table prefixer. For example, I have an entity User. So the generated table is user. But in postgres for example, user is a reserved name so we can't have a user table.
I can of course specify change the table name but is more logical to have a automatic table prefixer for all entity (project prefix for example).
What do you think ?

see: http://docs.doctrine-project.org/projects/doctrine-orm/en/latest/reference/basic-mapping.html#quoting-reserved-words
see: http://docs.doctrine-project.org/projects/doctrine-orm/en/latest/reference/limitations-and-known-issues.html#identifier-quoting-and-legacy-databases
Also, what you want to do doesn't need to be in core: you can use a ``` event listener (see http://docs.doctrine-project.org/projects/doctrine-orm/en/latest/reference/events.html#lifecycle-events) to prefix the table name of each entity before the metadata is cached.


If there any way how to give live authentication access?

I want to make some js widget to add on site "Contact with me" button.
It simply should send email via google API. But I dont want the some user will have got pop up with access request for an app. If there any way to make it works permanently without access request each time for new users, sites, domains?

Hi @jack-fdrv this is not possible to authenticate the user without any popup, it goes against the design of OAuth. Closing this issue as working as intended.


Browse/Filesystem function sorts lowercase letters after all uppercase, not alphabetically

What version of Music Assistant has the issue?
2022.7.2
The problem
A very minor issue, Marcel, but I wanted to report it.
When in Browse/Filesystem the folders are displayed in alphabetical order as it should, unless there is a mix of folders starting with uppercase letters and lowercase. MA will first display all the folders starting with "A" to "Z" then will display folders "a" - "z".

It is an issue that is easy to work around... Simply rename your lowercase folders with uppercase letters, and all is good.
How to reproduce
To reproduce change a few folder names in the filesystem to start with lowercase letters, then Browse/filesystem again.they will immediately show up after all of the uppercase entries.
Relevant log output
No response
Additional information
Low priority for sure.
What version of Home Assistant Core are your running
2022.7.6
What type of installation are you running?
Home Assistant OS
On what type of hardware are you running?
Generic x86-64 (e.g. Intel NUC)

Yes Marcel did some work previously with sorting and my instinct was that what you are observing is correct as in ASCII order that is what you would see. However, the yardstick we used at the time of the previous fix is what would Excel do. I just tried it and indeed Excel sorts as you are suggesting so I will mark this as a bug and see what Marcel thiks.


localhost:8888

When I use "bash scripts/bootstrap.sh" to star the HMI,I get the informatation like this:
Started supervisord with dev conf
Start roscore...
voice_detector: started
Dreamview is running at http://localhost:8888
However, I can't open the http://localhost:8888, the chrome says:
This site can’t be reached
localhost refused to connect.
Try:
Checking the connection
Checking the proxy and the firewall
I have no idea to control the question.

@riskeverything you could check these 2 documents: Dreamview FAQs and
debugging the dreamview start problem

Please check FMA is supported by your CPU. If not, there should be the problem.
Please refer my essay for more info.
https://zhuanlan.zhihu.com/p/51160743


Signal handling may be delayed, which leads to dropped samples

I've previously mentioned this issue in the comments of PR #1, but now that I understand what the problem is, I think it's worth creating a separate issue, not tied to any particular PR.
djdt-flamegraph uses interval timers that periodically generate signals, and registers a signal handler to sample the current stack frame.  Now, the way CPython implements signal handlers is that they set an internal flag, which is checked the next time the CPython interpreter enters the main eval loop.
A consequence of this is that if the main thread is blocked in some C code, signal handling gets delayed for an unbounded time, and you're getting a skewed profile picture because you're missing a significant number of samples.  Example: executing an SQL query via psycopg2 may delay stack sampling for hundreds or thousands of milliseconds.

I've implemented a workaround in my fork that synthesizes fake stack samples for all the missed sampling intervals.  Now all I need to do is clean it up and create a fresh PR, without the extra debugging gunk.
I seem to be taking unreasonable amounts of time doing this, because I'm busy with other things. :/

Thank you for investigating this and the detailed explanation. Can't wait to see the PR!

any progress on this one?

I'm sorry, I drifted away to other projects without finding the time to clean this up.  :(

Just for the records, I asked here to find a solution: https://stackoverflow.com/questions/49030629/statistical-profiling-in-python


docker-compose does not work out of the box

Running docker-compose up -d does not work. It fails with permission denied in many places.
dstslave_1  | [Errno 13] Permission denied: '/home/steam/.klei/DoNotStarveTogether/settings.ini'

I've been meaning to get back around to updating this.  But unfortunately docker is not running on my machine with windows pro 1709.  I'll fix this asap.


Add flags to readme

I've added a table of flags to the readme file pointing to the appropriate translated page as well as a folder of svg files containing flags from all countries. Unfortunately however, adding these in has made the source code for the readme a bit messy and i am not sure if there is a cleaner way of doing it. Also because of the white background of the page some of the white from certain flags is blending in. I attempted changing the background color for the table rows but it doesn't look like it made a difference. Maybe you can help me with fixing it?
example:

@santanamichael211, as per the discussions we had in #4966, we decided to just have a set of flags linking to corresponding translations. Like this
Read this in other languages:
🇮🇳 🇲🇲 🇮🇩 🇫🇷 🇪🇸 🇳🇱 🇷🇺 🇯🇵 🇻🇳 🇵🇱 🇮🇷 🇮🇷 🇱🇹 🇰🇷 🇰🇵 🇩🇪 🇨🇳 🇹🇼 🇬🇷 العربية 🇺🇦 🇧🇷 🇵🇹 🇮🇹 🇹🇭 🏴 🇵🇰 🇧🇩 🇲🇩 🇷🇴 🇬🇧 🇹🇷

Closing this as this has already been taken care of in #5866


Fixing superfluous download of pre-built target rust-std when build-s…

This fix prevents the unnecessary download of of the target's pre-built rust-std when the target specified in Cross.toml has build-std = true. When this configuration is set, pulling the targets pre-built rust-std is unnecessary and is not used during the build.
An example Cross.toml file could look like:
[target.x86_64-unknown-freebsd]
build-std = true

with respective build commands:
cross build -Z build-std=std,panic_abort -Z build-std-features=panic_immediate_abort --target x86_64-unknown-freebsd --release

cross build -Z build-std=std,panic_abort -Z build-std-features=panic_immediate_abort --target x86_64-unknown-netbsd --release

You will note, the freebsd build will not pull the pre-built freebsd rust-std and the build will succeed, while the netbsd will still pull the pre-built std i.e:
info: downloading component 'rust-std' for 'x86_64-unknown-netbsd'
info: installing component 'rust-std' for 'x86_64-unknown-netbsd'

Sure thing! added the changelog entry! :)


Fix login/signup redirect and flashes

Logging in and signing up does not redirect back to the splash screen
Email confirmation flashes properly

This is great, thanks!


Update sbt-scalafix to 0.9.27

Updates ch.epfl.scala:sbt-scalafix from 0.9.18 to 0.9.27.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "ch.epfl.scala", artifactId = "sbt-scalafix" } ]

labels: sbt-plugin-update, semver-patch

Superseded by #51.


Fix IR291

This PR fixes IR291.

I don't understand how using the timestamp is the correct solution.


Handle ErrUserProhibitLogin in http git

Fix #7558

Codecov Report

Merging #7586 into master will decrease coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #7586      +/-   ##
==========================================
- Coverage   41.25%   41.25%   -0.01%
==========================================
  Files         469      469
  Lines       63618    63621       +3
==========================================
  Hits        26246    26246
- Misses      33948    33951       +3
  Partials     3424     3424

Impacted Files
Coverage Δ

routers/repo/http.go
35.06% <0%> (-0.27%)
:arrow_down:

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 bcbc9f3...439b241. Read the comment docs.


ctkVTKDataSetArrayComboBox: Mask for arrays with different semantics

Within a VTKDataSetArray there are several types of arrays that can hold information and have a precise semantic meaning:
Scalars, Vectors, Tensors, Normals, TCoords, GlobalIds, PedigreeIds, EdgeFlag
Some times it doesn't make sense to show the one or various of these in a list. The requested feature is to have a mask such that if an array is associated with this particular semantic
(for instance arrayID = dataSet->GetTensors()) it would not be shown in the combobox list

See below a preview of the AttributeTypes property in QtDesigner ..

Cool!
Looks great!
How was the ski-week?
Dem
Demian Wassermann, PhD
<EMAIL_ADDRESS>LMI / PNL / SPL Labs
Harvard Medical School
Brigham and Women's Hospital
1249 Boylston, Boston, MA, USA
On Jan 25, 2012, at 4:16 PM, Jean-Christophe Fillion-Robin wrote:

See below a preview of the AttributeTypes property in QtDesigner ..

Reply to this email directly or view it on GitHub:
https://github.com/commontk/CTK/issues/130#issuecomment-3659198

Hi Demian,
Ski was great :) We got some very good snow Wednesday last week
(15/20inches of powder).
The best came on Saturday night .. but I had to fly out on Sunday morning
(27inches) .. arfff
Let me know if you have any issue with the updated widget, you should be
able to configure the attribute types from within QtDesigner directly.
Jc
On Wed, Jan 25, 2012 at 11:27 PM, Demian Wassermann <
<EMAIL_ADDRESS>
wrote:

Cool!
Looks great!
How was the ski-week?
Dem
Demian Wassermann, PhD
<EMAIL_ADDRESS>LMI / PNL / SPL Labs
Harvard Medical School
Brigham and Women's Hospital
1249 Boylston, Boston, MA, USA
On Jan 25, 2012, at 4:16 PM, Jean-Christophe Fillion-Robin wrote:

See below a preview of the AttributeTypes property in QtDesigner ..

Reply to this email directly or view it on GitHub:
https://github.com/commontk/CTK/issues/130#issuecomment-3659198

Reply to this email directly or view it on GitHub:
https://github.com/commontk/CTK/issues/130#issuecomment-3664196

--
+1<PHONE_NUMBER>


Avoid JSON parsing error in get_ansible_inventory

Avoids Ansible bug https://github.com/ansible/ansible/issues/59973 which
prevents testinfra from parsing inventory by disabling ansible verbosity.
Fixes: #470

Has someone validated that this does the job?

Asked in https://github.com/ansible/molecule/issues/2188#issuecomment-517958005.

#471
with fresh pip install -U molecule and applied proposal molecule verify passed successfully.
I've tested configurations:

ansible 2.5, 2.6, 2.7, 2.8.
centos and macos OSs
python 2.7 and 3.7.
commands molecule test and molecule verify

#472
Unfortunately, this proposal will not work.
Even though it is more clear in reading user option --output appears to be added in ansible 2.8. Testing with 2.5, 2.6, 2.7 returned error: no such option: --output
Full error:
==================================== ERRORS ====================================
    ____________________ ERROR collecting tests/test_default.py ____________________
    tests/test_default.py:6: in <module>
        os.environ['MOLECULE_INVENTORY_FILE']).get_hosts('all')
    ../../../test/lib/python3.7/site-packages/testinfra/utils/ansible_runner.py:131: in get_hosts
        inventory = self.inventory
    ../../../test/lib/python3.7/site-packages/testinfra/utils/__init__.py:44: in __get__
        value = obj.__dict__[self.func.__name__] = self.func(obj)
    ../../../test/lib/python3.7/site-packages/testinfra/utils/ansible_runner.py:149: in inventory
        return get_ansible_inventory(self.ansible_config, self.inventory_file)
    ../../../test/lib/python3.7/site-packages/testinfra/utils/ansible_runner.py:62: in get_ansible_inventory
        local.check_output(cmd, *args)
    E   AssertionError: Unexpected exit code 2 for CommandResult(command=b'ansible-inventory --list --output=/var/folders/mb/ckw0hwgs00bc763q92gn6h9r0000gq/T/tmpdpl1xezo -i /var/folders/mb/ckw0hwgs00bc763q92gn6h9r0000gq/T/molecule/molecule_test_ansible_verbose/default/inventory/ansible_inventory.yml', exit_status=2, stdout=None, stderr=b'Usage: ansible-inventory [options] [host|group]\n\nansible-inventory: error: no such option: --output\n')
    E   assert 2 == 0
    E    +  where 2 = CommandResult(command=b'ansible-inventory --list --output=/var/folders/mb/ckw0hwgs00bc763q92gn6h9r0000gq/T/tmpdpl1xezo...one, stderr=b'Usage: ansible-inventory [options] [host|group]\n\nansible-inventory: error: no such option: --output\n').rc
    !!!!!!!!!!!!!!!!!!! Interrupted: 1 errors during collection !!!!!!!!!!!!!!!!!!!!
    =========================== 1 error in 0.60 seconds ============================

Merged, thanks ! I'll release this in a new version today.

:fire: :fire: :fire:

@philpep I am waiting for the new release to bump it in molecule. If you have concerns about the risk of breaking others you can tag it as a pre-release (3.0.6rc1) so normal users would not get it. I would still be able to add testinfra>=3.0.6rc1 in molecule and consume it.


When I run the NDP sample, the following scenario occurs, and the dataset is present

It seems that the data is not properly loaded.

I meet the same problem ,do you fix it?

This is because the path of the dataset is wrong, modify the data root in the corresponding yaml file to solve.

This is because the path of the dataset is wrong, modify the data root in the corresponding yaml file to solve.

thank you so much! yes I found this mistake , I made a mistake in the file hierarchy.


cleanup(libsinsp): remove unreachable code in utils.cpp

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind cleanup
Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area libsinsp

Does this PR require a change in the driver versions?
No

What this PR does / why we need it:
Removes some code that is not used anywhere in the libs or falco codebase.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

NONE

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: LucaGuerra
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [LucaGuerra]

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

/milestone 0.18.0

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, LucaGuerra
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP,LucaGuerra]

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

LGTM label has been added.  Git tree hash: f56592d7f86fb3e2ea633b9c179f26a948bca63c

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, FedeDP, LucaGuerra
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP,LucaGuerra]

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

/hold

/unhold


hydy100 version

hey , i saw that  @hydy100 made a fork of the project , but when i checked the releases of its fork , i dont really get what it saying , is it only working in china ?

there's a one called riot version it works on all region not china only

it says i might lead to a ban ,  any1 got one here before?  because i cant find some1 in issues that has got banned

not banned, been using for 1 week now!

it says i might lead to a ban , any1 got one here before? because i cant find some1 in issues that has got banned

And where is was ?

it says i might lead to a ban , any1 got one here before? because i cant find some1 in issues that has got banned

Where you have find banned because of hydy100 R3nzSkin.

not banned, been using for 1 week now!

Okay thanks for sharing your experience

it says i might lead to a ban , any1 got one here before? because i cant find some1 in issues that has got banned

Where you have find banned because of hydy100 R3nzSkin and the ban was on china version or global version.

Do u understand  english ? , i said i didnt find one , it just says on the release that u might get banned

it says i might lead to a ban , any1 got one here before? because i cant find some1 in issues that has got banned

Where you have find banned because of hydy100 R3nzSkin and the ban was on china version or global version.

Do u understand  english ? , i said i didnt find one , it just says on the release that u might get banned

it says i might lead to a ban , any1 got one here before? because i cant find some1 in issues that has got banned

Where you have find banned because of hydy100 R3nzSkin and the ban was on china version or global version.

Do u understand english ? , i said i didnt find one , it just says on the release that u might get banned

It was in the china version.

it says i might lead to a ban , any1 got one here before? because i cant find some1 in issues that has got banned

Where you have find banned because of hydy100 R3nzSkin and the ban was on china version or global version.

Do u understand english ? , i said i didnt find one , it just says on the release that u might get banned

It was in the china version the ban issue when it was released.

When u check for release it says (it might lead to a ban ) in global


Add support for Capacitor's Haptics plugin

Feature Request
Ionic version:

[x] 4.x
Describe the Feature Request
Support Capacitor's built-in Haptics plugin for ionic components.
Describe Preferred Solution
Add support for Capacitor Haptics in haptic.ts. This should be supported since it is included out of the box with capacitor to avoid needing another Cordova plugin to accomplish the same thing.

Support for Capacitor's Haptics plugin has been added: https://github.com/ionic-team/ionic/blob/33be1f061ebbe27ee22e357c394f112af42ec360/core/src/utils/native/haptic.ts. Thanks!


Suggestion: can you support light versions also?

for ex. operator mono light

Yes, I plan on adding all the weights. I'm working on Operator Mono Book (non-ScreenSmart) now. I'll then start on Light (ScreenSmart and regular).

wonderful, Thank you so much.. waiting for it. :) 👍

@kiliman  How do i build the fonts for just the Book and Book Italic .By default its only building the Medium and Medium Italic fonts

@vikky49 Did you pull the latest version of the code?
If you run the build command it will try to build a font for every font that's defined in the ligature folder that you have in the original folder.
What's probably happening is that you don't have the ScreenSmart version of the Book weight. I haven't created the ligatures for the regular Book weight yet. I'm working on it now.

Thanks for the tool @kiliman! It's awesome!
Is there any progress with the light version? Maybe we can help you somehow?

I've started on Operator Mono SSm Light. Here's the regular version. Need to finish the italic.
Here's Operator Mono SSm Light at 14px in Visual Studio Code.
Note: I also added the F# fish operators >=> and <=<

I pushed Operator Mono SSm Light. I'll push the italic version out as soon as I can.
Enjoy!

Wow, that was fast. Thanks a lot!

Would also like the non-smartscreen version of light, when you get a chance.

@kiliman Any progress on the italic version?

@kiliman Looking forward to the light and extra light versions of the non-smartscreen versions. These should have been in the original font from the start! Thanks a bunch for doing these!

Just letting you know there's still interest in the light versions!
Using the book version right now and it's great with ligatures. :tada:

Can't say enough about how great this tool is! Fine work. 🚀
I too would love the light non-smartscreen versions. It must be good deal of work. I wish I could help out.

Starting work on Operator Mono Light (the #2 weight on the poll). Sorry for the delay everyone.
These are all redone from scratch with the new process (see features/contextualsubst) branch.
These ligatures have more consistent spacing and better hints for smaller sizes.
Ignore the italics in the screenshot. Since I haven't started the italic version, VS Code is synthesizing italics from the regular style.

@kiliman
./build.sh does not appear to do anything.

pulled branch features/contextsubst
npm install for good measure
used both node v8.10.0 and v9.8.0

Running ./build.sh in terminal completes sub-second and produces no output.

The new branch only contains Operator Mono SSm Book for now.
I should be finished with Operator Mono Light this weekend. I just wanted to let you know that I was finally back to working on the ligatures.

Pushed initial version of Operator Mono Light to branch. It doesn't have all the ligatures but I wanted to verify the extract and build process for a second font.
Let me know what you think.

I don't have the Linux subsystem installed, but I'm able to build from Windows 10.
Can you tell me what versions you have for Python and fonttools?

Not using the subsystem, it's a little too flakey for me at this point.
What I am using is the really good optional Unix tools integration in the latest git-scm builds for Windows.
Python 2.7.14 ,fonttools 3.25.0, node 8.11.1
I'll be upgrading python to 2.7.15 and if that doesn't work I'll move over to 3.6.5 and let you know if anything changes.

First of all, thank you for working on the ligature support for Operator Mono! Super cool!
I look forward to the completion of Operator Mono Lig Light with support for === !== 😀

light version is not build


Work items without dates get Start/Target dates assigned by the extension if selected

Describe the bug
Pick several features, create portfolio with them. Features without Start/Target dates get them auto assigned by the tool.
Browser
Choose a browser
[ ] IE11
[x ] Chrome
[ ] FireFox
[ ] Safari
Version
[x ] Azure DevOps/VSTS
[ ] TFS 2017 Update 2/3
[ ] TFS 2018
Screenshots
If applicable, add screenshots to help explain your problem.

Sorry, but this code in this repo is not currently being maintained. We recommend you use the Delivery Plans feature that is core to Azure DevOps.


async validate

check async validate function

Codecov Report

Merging #3371 into dev will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev    #3371   +/-   ##
=======================================
  Coverage   99.09%   99.09%
=======================================
  Files          18       18
  Lines        1108     1108
  Branches      298      298
=======================================
  Hits         1098     1098
  Misses         10       10

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 7f9d16c...06c5e34. Read the comment docs.

Thanks for the PR.
It's a good idea, but like @clarkdo said, it's better to run the validate in series since it will avoid making useless call.

@Serjlih97 Thanks for this good idea. I love it.
BTW shouldn't we apply the same patch to the client.js file?
https://github.com/nuxt/nuxt.js/blob/dev/lib/app/client.js#L272
Also running in series would be a better idea :)

@pi0 How to add an additional file to this Pull Request?

@Serjlih97 For client.js just switch to pr-1 branch :)
https://github.com/Serjlih97/nuxt.js/edit/patch-1/lib/app/client.js

How to add an additional file to this Pull Request?

Add them and just git push to the same branch again, GitHub will automatically update the PR. Very interested in seeing this come through @Serjlih97, thanks!

Need tests before merging it :)


Eliminate tabs for now, and put all information on one page.

UX Bug
There have been cases of secretariats that normally fill out records impeccably now forgetting to add authors and the full text. The chance that datasets will be forgotten is derived from these cases as well.
Temporary enhancement
Eliminate the tabs:

Full text
Authors & Editors
Datasets

And put the blocks in the first tab as one big form.
Not in scope
We will not be able to guide people from one tab to another right now. We will solve this at a later stage.

Context: Matthias mentioned that the tabs have just been refactored. Keep in mind; if we have to put them back – there will be triple work...

We will first test with the new validation, and revisit after.

Forgetting to fill in things is not just due to the existence of the tabs.
If you have to long list of files/authors, then you also not see certain things,
like a missing ugent author.
Why not a number next to author and file tab that indicates absence of authors/files?
Maybe in orange when there are none?
Description | Full text (0) | Authors (1)
Adding some sort of summary at the start for some stuff can be helpful for this purpose.
I do this also in my dashboards ("tiff files (2), jpeg files (1)")

@nicolasfranck definitely a nice improvement once we have the plumbing! For now we don't have a good structural way to guide people to those tabs. If we have proper guidance, those indications in the tabs would be very useful.
The summary page for now unfortunately is not meaningful because the rules are not defined, which results into lying to the researchers (e.g. "everything is fine" or "everything is broken" with little nuance). Once those rules are defined, we can bring it back from the deprecated.

Probably fixed by #331

Not fixed. We still get reports. Need to get a new design for it. Analysis is being made.

Redesign in progress.


Unable to migrate OctoberCMS on Postgres database

Winter CMS Build: 1.1.3 (currently latest)
PHP Version:   7.3.18
Database Engine: PostgreSQL server version 11.6

Description:
I performed the migration from OctoberCMS build 472 using composer update. On the development system with MySQL everything was ok. On the stage system with Posgres database the upgrade process was ok (no errors), however, I cannot open any page. The exception I get is
SQLSTATE[42883]: Undefined function: 7 ERROR: operator does not exist: character varying = integer LINE 1: ...ystem_files" where "system_files"."attachment_id" in (1, 2) ... ^ HINT: No operator matches the given name and argument types. You might need to add explicit type casts. (SQL: select * from "system_files" where "system_files"."attachment_id" in (1, 2) and "system_files"."attachment_type" = RainLab\Blog\Models\Post and "field" = featured_images order by "sort_order" asc)

Looking at the table the reason is:  on the database the attribute system_files.attachment_id is of type varchar and this type cannot be directly compared to integer:
select * from "system_files"
 where "system_files"."attachment_id" in (1, 2)
       and "system_files"."attachment_type" = "RainLab\Blog\Models\Post"
       and "field" = featured_images
order by "sort_order" asc

returns on the plain Postgres database
ERROR:  operator does not exist: character varying = integer
LINE 2:  where "system_files"."attachment_id" in (1, 2)

Probably Winter issue because with October everything worked fine.

A note to the error - it occurs whenever blog entries are listed. So it looks like a bug related to the RainLab.Blog plugin.  Anyway, it occurs while accessing the system_files table, which belongs IMO to the Winter core.

Looks like a duplicate of wn-blog-plugin issue

Hi @envas,
Yes, I can confirm this issue is a duplicate of https://github.com/wintercms/wn-blog-plugin/issues/16 .
As said in the other thread, I fixed it in the past but somebody mentioned a side effects.
Atm a quick fix is to disable the eager loading of featured images into the Posts component. This will be properly fixed soon.


Fix package ArobasMusic.GuitarPro.7 version <IP_ADDRESS>9

Microsoft Reviewers: Open in CodeFlow

Hi @marco-eckstein,
This manifest adds Apps and Features entries that aren't present in previous PR versions. Should these entries be added to the previous versions also?
(Automated response - build 489.)


Final css refactor branch

Should be good, given we talked about all this in a call.

Pulling the trigger!


Egui 0.22.0

Closes https://github.com/emilk/egui/issues/THE_RELEVANT_ISSUE.

This is a WIP to upgrade egui to 0.22.0, modifications have been made on eframe, egui-winit, egui-wgpu to make use of tao.
This currently fully works on Windows 11, macOS Ventura and is able to build egui_demo_app that works properly.
I'm currently having issue to make it run on both X11 and Wayland, which accidentally leads to the discovery and fix of https://github.com/tauri-apps/tao/pull/741 but it's irrelevant to the problem I'm having.
On Wayland, the build can be completed for egui_demo_app, but the app does not show properly.
On X11, the build can also be completed for egui_demo_app, but an error shows up that indicates Requested usage is not supported when configuring surface within WGPU.

I'm sad that this was not completed. Thank for attempting @dklassic.

I'm sad that this was not completed. Thank for attempting @dklassic.

No worries, I'm still working on it!
gtk3 does not support Vulkan context so we're reevaluating our options here.

Reworked the upgrade using glutin, have it working on Windows and macOS, egui_demo_app runs just fine.
I'm still figuring out a weird behavior on X11 that leads to an Glutin(Error { raw_code: Some(12299), raw_os_message: None, kind: BadNativeWindow })
Full log:
[2023-06-12T04:33:27Z DEBUG eframe] Using the glow renderer
[2023-06-12T04:33:27Z DEBUG eframe::native::run] Entering the winit event loop (run_return)…
[2023-06-12T04:33:27Z DEBUG eframe::native::file_storage] Loading app state from "/home/wusyong/.local/share/eguidemoapp/app.ron"…
[2023-06-12T04:33:27Z DEBUG eframe::native::run::glow_integration] trying to create glutin Display with config: ConfigTemplateBuilder { template: ConfigTemplate { color_buffer_type: Rgb { r_size: 8, g_size: 8, b_size: 8 }, alpha_size: 8, depth_size: 0, stencil_size: 0, num_samples: None, min_swap_interval: None, max_swap_interval: None, config_surface_types: WINDOW, api: None, transparency: false, single_buffering: false, stereoscopy: None, float_pixels: false, max_pbuffer_width: None, hardware_accelerated: None, max_pbuffer_height: None, native_window: None } }
[2023-06-12T04:33:27Z DEBUG eframe::native::run::glow_integration] using the first config from config picker closure. config: Egl(Config { inner: Config { raw: EglConfig(0x55fb9896c7e0), display: Khr(0x55fb989db2a0) } })
[2023-06-12T04:33:27Z DEBUG eframe::native::run::glow_integration] successfully created GL Display with version: EGL 1.5 and supported features: CONTEXT_ROBUSTNESS | CONTEXT_NO_ERROR | FLOAT_PIXEL_FORMAT | SWAP_CONTROL | CREATE_ES_CONTEXT | MULTISAMPLING_PIXEL_FORMATS | SRGB_FRAMEBUFFERS
[2023-06-12T04:33:27Z DEBUG eframe::native::run::glow_integration] creating gl context using raw window handle: Some(Xlib(XlibWindowHandle { window: 0, visual_id: 0 }))
[2023-06-12T04:33:27Z DEBUG eframe::native::run::glow_integration] running on_resume fn.
[2023-06-12T04:33:27Z DEBUG eframe::native::run::glow_integration] creating surface with attributes: SurfaceAttributes { srgb: None, single_buffer: false, width: Some(1320), height: Some(855), largest_pbuffer: false, raw_window_handle: Some(Xlib(XlibWindowHandle { window: 0, visual_id: 0 })), native_pixmap: None, _ty: PhantomData }
[2023-06-12T04:33:27Z ERROR eframe::native::run] Exiting because of error: Glutin(Error { raw_code: Some(12299), raw_os_message: None, kind: BadNativeWindow }) on event NewEvents(Init)

Tracing this issue leads to https://github.com/emilk/egui/issues/2520, which indicates the possibility of driver issue instead, but I'm not sure if this is the case here.

The upgrade seems to be working now!
I'll be doing a revamp to the repository structure before reading for review.


Tabs bar covers page-content on Firefox

Framework7 version: 4.5.2, 5.5.3
Platform and Target: Firefox 70, 71, 72

Describe the bug
On firefox, tabs bar covers page-content
To Reproduce
Steps to reproduce the behavior:

Run framework7 create
Choose all default options except starter template (Simple web app, Framework7 Core)
Choose starter template: Tabbed Views (Tabs)
Run npm run start
Open app in small Firefox window

Expected behavior
Tab bar covers page content (left: safari, right: firefox):

https://github.com/framework7io/framework7/issues/2053#issuecomment-355756625


Upgrade into Spring Boot 2.7.7 version

Upgrade into Spring Boot 2.7.7 version

Hey sure, what are the improvements? Would you like to contribute a diff for this?

I'll be pushing my PR tomorrow


Don't require rake in gemspec

This can cause breakage when installing via bundler:
$ bundle install --without development --path=${BUNDLE_PATH:-vendor/bundle}
Fetching https://github.com/michaeldv/awesome_print
There was a LoadError while loading awesome_print.gemspec:
cannot load such file -- rake from
/home/travis/build/vendor/bundle/ruby/2.3.0/bundler/gems/awesome_print-d8aeb66b4b03/awesome_print.gemspec:6:in
`<main>'
Does it try to require a relative path? That's been removed in Ruby 1.9.

Rebuilding this on my own fork, due to Travis-ci not letting me completely restart the build on this PR: https://travis-ci.org/gerrywastaken/awesome_print/builds/129531246

OK it passed. Thank you for the PR and I'm very sorry about it taking so long.

Your change went live in version 1.7.0. Thank you for the fix. 👍


Removed storage estimate from PPM dates/locations page; enabled on summary page

Description
We're experiencing some confusion around the storage estimate that is being shown on the dates/locations page in the PPM flow.  Initially, the storage estimate is based upon the user's max entitlement weight since we don't have an actual weight estimate at this point in the flow.  However, if the user goes forward to the point of getting a weight estimate in the PPM record, then goes back via the wizard to the dates/locations page, the estimate changes to using the estimated weight.  So the user may see significantly different numbers without any change in how the number is presented.  Furthermore, if they change a date/zip/days, the estimate reverts back to using the max entitlement weight.
To add to the problem, a different bug fix (#2015) was setting the PPM weight to zero when picking a move size, so going back to the dates/locations page caused a drastically different amount to show up when a true estimate had not even been entered by the user.
There was also a bug on the PPM move summary page that was keeping the storage estimate (based on the actual weight estimate) from showing up there, so the user may not be seeing the true storage estimate in the normal flow.
This PR does two things: 1) removes the storage estimate and API calls from the initial dates/locations page to avoid showing changing numbers (were we adding any value showing it there?), and 2) fixes the bug on the PPM summary page so the true estimate shows up.
Reviewer Notes
There is a separate "edit" page for dates/locations that I did not modify.  It is reached from the PPM summary page, so you always have a weight estimate on it.  It does show the storage estimate and recalculates based upon the weight estimate (i.e., max entitlement doesn't play a role there, so it's not confusing).
Setup
Create a PPM.  When on the initial dates/locations page, you should no longer see any storage estimate box when entering a number of days in storage.  You should see the estimate, however, on the PPM summary page later in the flow.  You can also see/change the estimate when editing the dates/locations (see "edit" button on the summary page).
Code Review Verification Steps

[ ] There are no aXe warnings for UI.
[ ] This works in Supported Browsers and their phone views (Chrome, Firefox, IE, Edge).
[ ] User facing changes have been reviewed by design.
[ ] Request review from a member of a different team.
[ ] Have the Pivotal acceptance criteria been met for this change?

References

Pivotal story for this change

Screenshots
On initial dates/locations page (note no storage estimate box):

At PPM summary page (note storage estimate):

On edit dates/locations page (note storage estimate):

Looks good. I'm just wondering if we should be adding some cypress tests around the edit behavior. Maybe a check that the estimate on the edit page matches what's on the summary page.


MAISTRA-1491: generate a sanitised version for the directory and CSV …

…name

does it need to be changed from __VERSION__ to __NAME_VERSION__?

No, it should still be the 1.1.2+2 version since the '+' is the build identifier

Merging to rebuild the metadata


Search SDK: Merge AutoRest changes and add missing descriptions

Corresponding spec change: https://github.com/Azure/azure-rest-api-specs/pull/1086
FYI @Yahnoosh @chaosrealm

Description

This checklist is used to make sure that common guidelines for a pull request are followed.

[X] I have read the contribution guidelines.
[X] The pull request does not introduce breaking changes.

General Guidelines

[X] Title of the pull request is clear and informative.
[X] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[X] Pull request includes test coverage for the included changes.

SDK Generation Guidelines

[X] If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
[X] The generate.cmd file for the SDK has been updated with the version of AutoRest, as well as a link to the swagger spec, used to generate the code.
[X] The project.json and AssemblyInfo.cs files have been updated with the new version of the SDK.

FYI @chaosrealm @Yahnoosh Most of this is a big merge commit; Please review only the final commit.

🎯


Raspberry Pi 3B+ Errors

Please tick all applicable boxes.

[x] I am using Python 3.5 or higher (run python --version on the command line)
[x] I have followed the official guides to install the bot for my system
[x] I have updated my dependencies to the latest version using the appropriate update_dependencies script

Which version are you using?

[x] The latest master version (1.9.7)
[ ] The latest review version

What type of issue are you creating?

[x] Bug
[ ] Feature request
[ ] Question

Description of issue
The installation works fine; besides the part where I can't install the
sudo add-apt-repository ppa:deadsnakes -y sudo add-apt-repository ppa:mc3man/trusty-media -y sudo add-apt-repository ppa:chris-lea/libsodium -y part. After the complete installtion, the bot works; but the chat messages are duplicated, so when I pause something, it says my vote has been ackowledged... then it says in another message Error "Can't skip! The player is not playing." So, the message is doubled; the first one is successful, but the second one is an error.
Steps to reproduce

Install the bot all the way and skip sudo add-apt-repository ppa:deadsnakes -y sudo add-apt-repository ppa:mc3man/trusty-media -y sudo add-apt-repository ppa:chris-lea/libsodium -y, then attempt to use any command, and it will be doubled. One is successful, and the second one is errored.

Log file
Please attach your MusicBot log file (located at logs/musicbot.log) to this issue. You can do so by dragging and dropping the file here. If you do not include your log file, you WILL be asked to provide one.
https://pastebin.com/qvBBEAv9
I couldn't drag and drop the file here; it's on a different OS.

I also had al lot of issues on my pi3.
Download Python-3.6.1.tar.xz from https://www.python.org/
Unzip the file and keep the folder in home directory.
Open terminal in that directory and perform the following commands:
./configure
make
make test
sudo make install
This will install Python 3.6 but pip3 may not be working.Install necessary modules using:
sudo apt-get install libreadline-gplv2-dev libncursesw5-dev libssl-dev libsqlite3-dev tk-dev libgdbm-dev libc6-dev libbz2-dev
Now write the following to re run the installation:
sudo make
sudo make install
Now this, even if its again... (If pip3 is not working try pip)
sudo pip3 install numpy
sudo apt-get install libreadline-gplv2-dev libncursesw5-dev libssl-dev libsqlite3-dev tk-dev libgdbm-dev libc6-dev libbz2-dev
If not working, do not uninstall a thing but follow the ubuntu 16 guide.
Then try some of my codes again.
I do not have a really good guide because i tried installing the bot and it tooks more than 3 days. Some of these codes fixed it for me.

@XFlawless1 I just skipped that step, and the bot seems to be correctly working. So, I guess I'll just wait for it to break before I come back and try to get those repos to work.


add partial expression and collation options

@washu Pull requests have been merited to master and a new branch called version-0.2.1 has been generated
Thank you for supporting to improve.


Fix #263

Codecov Report
Base: 75.52% // Head: 75.52% // No change to project coverage :thumbsup:

Coverage data is based on head (9174d2e) compared to base (9365c66).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #264   +/-   ##
=========================================
  Coverage     75.52%   75.52%
  Complexity      714      714
=========================================
  Files           108      108
  Lines          2051     2051
  Branches        251      251
=========================================
  Hits           1549     1549
  Misses          439      439
  Partials         63       63

Impacted Files
Coverage Δ

src/main/java/tuthub/model/tutor/Name.java
80.00% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


chore(settings): allow settings autocompletion

I don't really know why but I got "unknown" setting when typing "jest.pathToJest" in vscode. While with other extensions like flow it works perfectly.
It seems you are missing a "type": "Object" prop and also have some extra properties like titles for every option. Compared to flow that's the only difference.
vscode doc also says we may want to add type information (https://code.visualstudio.com/docs/extensionAPI/extension-points#_contributesconfiguration).

Great - awesome 👍


Uncaught RangeError: Maximum call stack size exceeded

Operating System: win10
Node Version: 8.10.0
NPM Version:5.7.1
webpack Version:4.1.1
webpack-serve Version:0.2.0

This issue is for a:

[x] bug
[ ] feature request
[ ] modification request

Code
CLI Command
  $ webpack-serve  --config ./webpack.config.js --content=./dist

webpack.config.js
  const HtmlWebpackPlugin = require('html-webpack-plugin');
const webpack = require('webpack');
module.exports = {

	entry: './src/index.js',
	output: {
		filename: '[name].js',
		path: require('path').resolve(process.cwd(), 'dist')

	},
	plugins:[
		new HtmlWebpackPlugin(),
		new webpack.HotModuleReplacementPlugin()
	]
}

Expected Behavior
compile successful, then my browser reload with new content
Actual Behavior
the termical show me that webpack has compiled successful, but  browser doesnot reload with new content,but with an err in the console pane ,tell me that  Uncaught RangeError: Maximum call stack size exceeded
How Do We Reproduce?
it's easy to reproduce with 0.2.0,or configure it like mine

@lifuzhao100 an error report is no good without the call stack.

@shellscape

That's an error in webpack itself, not something that webpack-serve is in control of. There's nothing strange about your config so it must be something in your app code or environment that isn't playing nice with webpack's HMR.

@shellscape thanks!!!i am sorry to find that this err occurs when i add HotModuleReplacement Plugin to configure files.once i rm it, it works well.


Remove system prompt from data generation

Remove system prompt from data generation and will be re-introduced in the mixing phase.
https://github.com/instructlab/sdg/blob/b28a12bb647ef72f2b152051fc73d55c5a30da98/src/instructlab/sdg/generate_data.py#L38

+1, would be good to introduce system role during the data mixing phase which prepares the dataset for training - this makes it a tad bit cleaner to understand - as the system role is only applicable to training

Unmarking as stale, because this is still relevant and as we more cleanly separate data generation from mixing that would be the time to ensure we aren't passing around system prompts during generation.


Propose change *_ERR FITS labels to, e.g., *_COMP_ERR

Data columns in the STIX FITS files labelled *_ERR (e.g. COUNTS_ERR) only include compression error.  While there are good arguments for this, it transpired that the current naming led members of the STIX team to assume that it was the total error.  This confusion is more likely among the wider community.
Proposed solution: Change label to *_COMP_ERR, e.g. COUNTS_COMP_ERR.
Cons

Longer.

Pros

Explicit.
If users don't know what COMP means, this makes them more likely to read the docs/comments and realise that it is only compression error.  Currently users may unwittingly think they already know what *_ERR means and so are less likely to read the docs and realise their mistake.

I will double check the changes is as trivial as discussed


[UWP] Unnecessary string copying

Platform

Android
iOS
UWP

Author or host
Author and host. Cortana in Windows - app is both an author and host.
Version of SDK
NuGet 1.2.8
Details
Many C++ functions accept std::string parameters by value when they should really be by const-ref or r-value ref. Updating the parameter types would help improve performance by reducing the number of string copies made during parsing and rendering.

@paulcam206 / @RebeccaAnne to help review this and shepherd it into main where it can bake for a bit until we push out the July refresh.


jsc.property missing stack trace

When running jsc.property tests with mocha, if an exception occurs it produces output like this:
Error: Failed after 6 tests and 0 shrinks. rngState: 84a5f3e91248d6bc70; Counterexample: {}; Exception: foobar is not defined

But no stacktrace is shown.
It would be really helpful if this could be fixed.
Thanks.

Fixed in 0.6.1. Sorry it took so long.


Flash to use gateway as zigbee router.

Can xiaomi multi mode gateway works as a zigbee router now?

Don't have time to support this feature


Add babel-plugin-transform-regex to devDependencies

Closes #704.
Per previous discussion with @fabian-hiller in #704, this adds package babel-plugin-transform-regex to devDependencies for improved DX.

Tests, lint, prettier, and build all pass.
Renamed library/src/regex.ts as library/src/regex.src.ts.

regex.ts is now the transpiled output.

In regex.src.ts, I updated only the first two regexes (BASE64_REGEX and BIC_REGEX) to use the regex tag with more readable syntax (using free spacing, subroutines, a subroutine definition group, and noncapturing (…)).

The emitted source in regex.ts for these two regexes is identical to before, which helps demonstrate that the Babel plugin is doing a great job with producing efficient and readable native regexes (since the regexes being replaced were hand-tuned regex literals). As a result, this PR's only change to regex.ts is an added header comment.
Other regexes and the rest of the code in regex.src.ts (apart from a hint comment at the top) is untouched.
I'll update the remaining regexes (many of which will benefit much more than these two, which are quite simple) after this PR lands.

Also added package regex to devDependencies so the types work in regex.src.ts.

The import { regex } from 'regex'; line in regex.src.ts is stripped from transpiled source by the Babel plugin (via its removeImport option which removes any import declarations with module name 'regex').
To make the types for regex work, I had to update library/tsconfig.json, changing "moduleResolution": "node" to "Bundler". Values "NodeNext" and "Node16" would also work for regex.src.ts, but those values conflict with other, current tsconfig.json settings.

This change doesn't seem to cause any issues elsewhere, but I'm not deeply knowledgeable about TS/CJS/bundling so it's worth examining.

Thank you for creating this PR. Since I have less time at the moment and other things have a higher priority, I will try to get back to you next week.

Is it not possible to do the transformation as part of our build step? I don't like that we need a regex.ts and regex.src.ts.

That sounds good, with the slight downside that you won't be able to review the emitted regex literals as easily (without looking at the output in dist).
I have a working solution that I just updated the PR with. It generates a regex.gen.ts file to use for the build, and then deletes it afterward. tsup (and esbuild which it relies on) doesn't have a built-in way to alias local file import paths, but I created a tiny esbuild plugin in tsup.config.ts that I'm using for this.
If this seems a little too magical, we could also use a build process like the following:

Copy (or rename) regex.ts to a temp file (lets say regex.orig.ts).
Run Babel on the temp file, with the output overwriting regex.ts.
Run the regular build.
Delete the generated regex.ts, then rename the regex.orig.ts copy back to regex.ts.

But my initial impression is that moving files around like that would not be preferable.

Thanks for the update! I am pretty busy at the moment and mainly focused on getting our v1 RC and after that our v1 release ready. I will come back to this PR when I have more time again or this feature becomes more important.

No problem, take your time. If you have a moment, though, I'm curious whether you have concerns with the current approach (or need time to think about it), or if you think the current PR is on the right track to pick back up from when you have more time.

In general, I don't like to complicate the setup because I like things simple, but of course I see the benefits of this PR.
Is it possible to run our tests with the compiled output? For security reasons this seems to be necessary in the long run.

In general, I don't like to complicate the setup because I like things simple, but of course I see the benefits of this PR.

I share your concern. This was actually the primary reason for my initial approach of having a separate "source" file, with regex.ts being the transpiled output. But I see the downside of that as well.
I've simplified things somewhat in the latest update to the PR, including getting rid of rimraf, and narrowing the aliasing plugin (which can always be generalized later if for whatever reason more paths need to be aliased).

Is it possible to run our tests with the compiled output? For security reasons this seems to be necessary in the long run.

I've updated the PR to do so. Vite/Rollup share the resolve.alias option, so this was easy to do. Their local aliasing support seems very robust (and allows regexes or functions, which I didn't use here).

Thank you for the changes! This PR is now going in a good direction.

Cool. In the meantime, I've made a couple more tweaks and I've gone ahead and updated the PR to cover the first 12 of 24 regexes in regex.ts. Hopefully this shows more substantively what the end result will be (I think it makes sense to still wait on the remainder until after this PR lands). Of the first 12 regexes, a couple (CUID2_REGEX and DECIMAL_REGEX) were so simple that they didn't benefit from regex, so I left them alone. I also included the following comment above IP_REGEX:
// Q: Would it be tree-shakeable if this used interpolation without adding any new variables, as
// ``new RegExp(`^(?:${IPV4_REGEX.source.slice(1, -1)}|${IPV6_REGEX.source.slice(1, -1)})$`, 'iu')``?

If the answer is yes, that would remove a lot of redundancy. That redundancy is much easier to verify now that I've rewritten the regexes in a readable way.
Important: None of the rewritten regexes have changed what they match in any way (existing bugs in IPV6_REGEX and IP_REGEX have been preserved).

I've rebased on top of recent conflicting Valibot configuration changes, and additionally bumped regex to v4.1.3. The latest version of regex avoids the need to update Valibot's library/tsconfig.json to use "moduleResolution": "Bundler". So I've undone that change.


Erlang and Elixir shell lexers don't recognize node names in prompts

(Original issue 1521 created by yuce on 2019-06-02T07:35:53.462037+00:00)
Erlang and Elixir shell prompts may have the node name as a prefix. lexers.ErlangShellLexer  doesn't highlight the prompt in the text below:
Erlang/OTP 22 [erts-10.4] [source] [64-bit] [smp:8:8] [ds:8:8:10] [async-threads:1] [hipe]

Eshell V10.4  (abort with ^G)
(e1@example.com)1> {any<EMAIL_ADDRESS>! {hello, world}.
{hello,world}
(e1@example.com)2> flush().
Shell got {hello,world}
ok

Updating lexers.ErlangShellLexer fixes the issue:
_prompt_re = re.compile(r'(?:\([\w@_.]+\))?\d+>(?=\s|\Z)')

lexers.ElixirConsoleLexer doesn’t highlight the prompt in the following text:
Erlang/OTP 22 [erts-10.4] [source] [64-bit] [smp:8:8] [ds:8:8:10] [async-threads:1] [hipe]

Interactive Elixir (1.8.1) - press Ctrl+C to exit (type h() ENTER for help)
iex(c1@ichi.cs)1> IO.puts("hello!")
hello!
:ok

Updating lexers.ElixirConsoleLexer fixes the issue:
_prompt_re = re.compile(r'(iex|\.{3})((?:\([\w@_.]+\))?\d+|\(\d+\))?> ')

(Original issue 1521 last updated on 2019-06-22T19:02:50.712884+00:00)
(Issue automaticaly closed due to status in Bitbucket: resolved)

(Original comment by yuce on 2019-06-02T08:21:32.731432+00:00)
PR at: https://bitbucket.org/birkenfeld/pygments-main/pull-requests/823/fixes-1521-erlang-and-elixir-shell-lexers/diff

(Original comment by Anteru on 2019-06-22T19:02:49.883160+00:00)
Merged in yuce/pygments-main/fix-1521 (pull request #823)
Fixes #1521, Erlang and Elixir shell lexers
→ <<cset 7b333c00cb64>>

(Original comment by Anteru on 2019-06-22T19:02:50.582509+00:00)
Fixes #1521, Erlang and Elixir shell lexers
→ <<cset 0f8d2e8ec84e>>

(Original comment by Anteru on 2019-06-22T19:02:50.704371+00:00)
Merged in yuce/pygments-main/fix-1521 (pull request #823)
Fixes #1521, Erlang and Elixir shell lexers
→ <<cset 7b333c00cb64>>


Add overview for the debugger when in maximized mode

Now that we can render more things when tools are maximized, we should add an overview to the debugger.

I think this is done. Should open new issues as they become apparent.


Removed Some Code From BoxUI that is maybe obsolete?

Can we get rid of this code yet?  There is a todo saying that it would be better to do this by promoting metadata, and it looks like metadata promotion is now working correctly.
I'm wanting to remove it because it sometimes causes problems.  If you promote a compound plug such as a TransformPlug ( which can't be done from the UI, but seems like a legitimate operation ), then this code screws things up.  It creates a widget for the internal TransformPlug, which is a LayoutPlugValueWidget.  This binds all the child plugs to widgets.  It then calls setPlug on the LayoutPlugValueWidget, but this is ignored, leaving all the internal child plugs visible in the UI.
If we can just ditch this code now that we've got metadata working, that's certainly the most elegant solution.

I don't think we're quite there yet I'm afraid - we still have a few spots where we're using PlugValueWidget.registerCreator() rather than metadata. This includes the RenderManShaderUI, so it's likely that we have plenty of existing promoted plugs which have no suitable metadata registered.
Although I agree it'd be great to finally remove the last few remaining registrations, it is actually a bug that LayoutPlugValueWidget doesn't implement setPlug(), so fixing that would certainly be a valid approach in the meantime. And a workaround for your specific situation would be to apply a suitable "plugValueWidget:type" metadata value to your promoted plug manually - this should bypass the old code path entirely.
I've marked this as revisionRequired, but I suspect our best course of action at present is to close this and open a new one with the bugfix.

I'll close this one then, since we're not quite ready for it...


Is there a RenderTexture-like widget in flutter?

I want use flutter implement a function like as PowerPoint, which thumbnail is syhc all change with stage area.But i did not find appropriate widget，finally i used RepaintBoundary to pick the image from stage area at the every frame.
Do you have any better suggestions?

This is a question better suited for StackOverflow, since there doesn’t seem to be anything wrong with Flutter itself. Closing, as this is not an issue report. See https://flutter.dev/community for support options.


fix!: Allow ConfigCat provider to be used in server applications

This PR
The conficat-js package that the ConfigCat provider is based on is for browser applications only as it uses XMLHttpRequest under the hood. However, the provider might be used in server applications (like it was attempted in a Node.js application here, at around 2:00:00).
For multiplatform scenarios, ConfigCat provides the conficat-js-ssr package (that uses Axios under the hood). This provider should use that instead of conficat-js to provide a more extensive support for various JS platforms.
This PR also fixes another serious shortcoming of the provider: currently it cannot evaluate feature flags without setting targetingKey in the evaluation context. Simple feature flags may not use targeting at all, so requiring a targetingKey to evaluate them is obviously not correct.
Related Issues
n/a
Notes
n/a
Follow-up Tasks
n/a
How to test
n/a

Hey @adams85 thank you this change looks good!
Maybe we should also consider developing a client-sdk provider, as this is a server sdk provider. Which makes you change even more important.
We can release this, but it has to be a breaking change as the API changed.

Thanks for the release, @lukas-reining. I did a quick test and found that the new version (v0.6.0) works fine.

Maybe we should also consider developing a client-sdk provider, as this is a server sdk provider. Which makes you change even more important.

Since our SSR SDK can handle both targets, I guess the client provider would be more or less just a duplicate of the server one (at least, in our case). I'd avoid maintaining another package if possible.
BTW, NPM packages kind of support multitargeting via the browser field. AFAIK, most of the bundlers support it nowadays. I have plans to look into the viability of this approach because it would be a godsend to us if we could merge our various JS SDK packages (or at least some) into one.


Batch correction as a fine-tuning procedure

Minor change was made, so this is expected to pass. Merging.


TFX Message exit handler

Implementation for  #101.

[x] Tests pass #TODO
[x] Appropriate changes to README are included in PR

LMK when this is ready to merge

Please also update the release notes. 😁

@rcrowe-google @casassg Can I please get a review and lgtm if things are ok?

@rcrowe-google @casassg This PR is now fully ready. Test are working again.

/lgtm
/merge

Missing files to be approved: /.github/workflows/ci.yml. Potential owners: @casassg and @vulkomilev

/lgtm for changes under CI

once pre-commit passes it should be merged automatically

Sounds good. I'll address the pre-commit issues probably tonight.


Update dependencies

Coverage remained the same at 86.275% when pulling 7f9d2bf84be379b00bbe933ce909a2f5328a1623 on develop into 5a9a951cd8671958f03315afbe537114798524ad on master.

Coverage remained the same at 86.275% when pulling c76f466264ffa9ca3206f500a7bd5425c0687287 on develop into 5a9a951cd8671958f03315afbe537114798524ad on master.


Creating a new tab from focussed tab on right group creates tab in left group

Azure Data Studio Version:
Version: 1.16.1 (system setup)
Commit: 16801c522df68e63218b39ccb5b4a7431f9bf667
Date: 2020-03-23T22:08:06.892Z
VS Code: 1.42.0
Electron: 7.1.11
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Windows_NT x64 10.0.17763

Steps to Reproduce:

Open two sql query tabs and split them into a left and a right group.
click on the tab in the right group
use ctrl+N to create a new tab
the new tab is created in the left group
expected: the new tab is created in the same group as the one of the focussed tab (right).

When creating new tabs/queries I now usually have to drag and drop the newly created tab to the other group before continuing my work.
Off topic: I love your azure data studio and use it all the time! Keep it up!

@FuegoArtificial when I try the repro my new query editor documents are being opened in the editor group with focus as expected (i.e. the right group in this example).  @anthonydresser are you able to repro this bug?

The issue still consists in my environment.
In addition a second empty tab is opened together with the initial one without any connection.
Am I doing something wrong?
Azure Data Studio Version:
Version: 1.18.0 (system setup)
Commit: 2acd37a1b1cff34b64fbb4ea2d40715d7e2accac
Date: 2020-05-19T23:54:30.554Z
VS Code: 1.45.0
Electron: 7.2.4
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Windows_NT x64 10.0.17763


[BUG] Asan failures in tests

Couldn't figure out right away so creating this issue.
doc.core.nth_prime.exe
Didn't even look
https://gist.github.com/DenisYaroshevskiy/2e51df3f4b499ef4453184f235186290
unit.api.memory.aligned_ptr.exe
Complains at the post increment/decrement test. I can't see why.
https://gist.github.com/DenisYaroshevskiy/5af28cd055568b4d7339eb8fab366084

nth_prime probably try to make a gather with not enough memory.
THe aligned_ptr is more strange.
Or it is a symptom of the lack of load_unsafe

nth_prime probably try to make a gather with not enough memory.
THe aligned_ptr is more strange.
Or it is a symptom of the lack of load_unsafe

reported nth_prime separately https://github.com/jfalcou/eve/issues/452 with more details.
I dunno how to fix it properly.

reported nth_prime separately https://github.com/jfalcou/eve/issues/452 with more details.
I dunno how to fix it properly.

Fix for algined_ptr problem: https://github.com/jfalcou/eve/pull/453

Fix for algined_ptr problem: https://github.com/jfalcou/eve/pull/453


issues running this implementation on google colab

i cannnpt seem to run some lines on google colab for example the import densenet does not work


rename ockam forwarder clap commands to ockam relay

Current behavior
Currently there are several ockam forwarder commands
ockam forwarder create
ockam forwarder list
ockam forwader show

As well as documentation, code examples, and bats integration tests that refer to these commands.
Desired behavior
Update the naming of these commands to ockam relay
ockam relay create
ockam relay list
ockam relay show

and surrounding documentation, code examples, and bats integration test

We love helping new contributors!
If you have questions or need help as you work on your first Ockam contribution, please leave a comment on this discussion.
If you're looking for other issues to contribute to, checkout this discussion and labels -  https://github.com/build-trust/ockam/labels/good first issue or https://github.com/build-trust/ockam/labels/help wanted

Do I need to be assigned to this before I can submit? I'd like to try this out.

@lvllvl it looks like there is already some work started on this in #4661 .. would you be interested in picking up a different issue?


Fix name resolution test in master considering micro block change

Delivers https://www.pivotaltracker.com/story/show/159008315

make test SUITE=test/name_resolution passed locally:
%%% name_resolution_SUITE ==> spend_to_name: OK
%%% name_resolution_SUITE ==> transfer_name_to_named_account: OK
All 2 tests passed.

No way this can affect other CI jobs or other tests, and master failed more than once: merging without waiting for static analysis and test job.