Default presentation of Explorer: Copy Relative Path Separator in Setting editor is confusing

Issue Type: Bug
Would be noticeable on Windows because of (implicit) default \\.
Steps to reproduce:
0. Make sure you are using Windows and reset explorer.copyRelativePathSeparator to default value.

Open Settings view and search for explorer.copyRelativePathSeparator - dropdown has / displayed (and setting not changed - no blue border).

Perform Copy Relative Path - path copied with \\ in it (while dropdown displays /)

Change setting to \\ and then "back" to / - setting now marked as changed

Perform Copy Relative Path - path copied with / in it as expected

I think it's because this setting doesn't have default value and dropdown will display first from list, however implementation has different logic.
VS Code version: Code - Insiders 1.59.0-insider (87d692b7bf1eb5a663b3e62620a6e7e436e1838c, 2021-07-09T05:13:26.890Z)
OS version: Windows_NT x64 10.0.19041
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8700 CPU @ 3.20GHz (12 x 3192)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: enabled_onvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
31.72GB (15.15GB free)

Process Argv
--crash-reporter-id cf52dbde-2c94-45c3-9bf3-45d8b9966b04

Screen Reader
no

VM
0%

Extensions: none
A/B Experiments
vsliv695:30137379
vsins829:30139715
vsliv368cf:30146710
vsreu685:30147344
python383cf:30185419
pythonvspyt602:30291494
vspor879:30202332
vspor708:30202333
vspor363:30204092
vswsl492cf:30256198
pythonvspyt639:30291487
pythontb:30258533
vspre833:30321513
pythonptprofiler:30281269
vsdfh931cf:30280410
vshan820:30294714
pythondataviewer:30285072
vscus158:30321503
pythonvsuse255:30319630
vscorehov:30301224
vscod805cf:30301675
pythonvspyt200:30323110
vscextlangct:30310089
vsccppwt:30312692
bridge0708:30335490

I added a auto default.
//cc @rzhao271 maybe this is a bug for the settings editor: if a setting is a enum without a default, shouldn't the dropdown indicate this by not showing the setting?


Add Sass support, closes #65

Adds Sass support via the assets/sass/ directory.

@net awesome man!  Thanks for doing this!  Out of curiosity, why the removal of yarn.lock (and outside a PR no less :-} )?  I think it should stay checked in, it is akin to a Gemfile.lock and helps ensure repeatability and conssitency of npm dependencies, half the reason I prefer yarn over npm :-)

Otherwise happy to merge this as is, I know @komizutama and others including myself prefer scss over plain css, so this was definitely timely!

I deleted the project root yarn.lock, not the one in assets/. :smile:


[Bug][Issues-2374]Modify the StreamPark application flink taskmanger configuration

What changes were proposed in this pull request
Issue Number: close #2374

Brief change log

Verifying this change

This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

Does this pull request potentially affect one of the following parts

Dependencies (does it add or upgrade a dependency): (yes / no)

#2374


[Feature Request] add variable-type ‘hostgroup’

Hi,
I have noticed that on grafana there is currently no hostgroup filter, which makes dashboard management by host complicated.
Currently in the field ‘Query - Object type to query’ only the filter for ‘Site, Host, Service’ is available. is it possible to add the hostgroup field as well?
Thanks and BR,
Vincenzo

Context: https://forum.checkmk.com/t/template-variable-grafana/37091/13

add variable type, only allow site filter for this variable
make sure hostgroup filter accepts variables

servicegroup as well pls :)

Hi Folks, any news on this?
Thanks!


Adapt Crowdsale contract to use Dai ERC-20 token

Some previous discussion:
Reddit discussion on Dai crowdsale contract:
https://www.reddit.com/r/ethdev/comments/8iith6/what_would_a_crowdsale_raise_in_dai_look_like/
BlankDAO evidently did a crowdsale using Dai:
https://github.com/BlankDAO/smart-contracts/blob/master/contracts/BlankCrowdsale.sol

Nice find. Looks clean.
So it...

Has a whitelist (the ‘referrers’)
Has math for calculating dollar values
Trades DAI for BlankToken, which would be the debt token

Closed


How can we provide certificates for DGCG

Hi everyone,
Please advice how to send TLS, Upload and CSCA certificates for the testing environment. We would like to test our implementation against it.
In the documentation is said that we have to send them over an email, but I can not find this email anywhere.
Thank you

@stamo A functional mailbox will be announced this week together with the test environment. After this announcement you can send your CRT files/PEM strings to this mailbox.

Functional mailbox was announced to member states today.


Change constructor of CreatorOutput

I propose a change to the signature of the constructor of CreatorOutput that allows to pass script &
blinding public key separately, this way users can have more freedom in the type of script we want to
pass (namely OP_RETURN and future segwit versions not yet supported)

@tiero LGTM but can you add assertions to the unit test to make sure everythin's ok?


Updated Starter Pack

Update to Webpack 4
Use webpack-dev-server as it automatically reloads html/css changes
Use html-webpack-plugin
Strip webpack config to the bare minimum

Thanks for this @knoopx! 🙌 Question for you; I didn't realize Webpack supported CSS entry files (like mentioned in #10), what do you think makes more sense for this project, HTML or CSS entry? Main goal is just a quick way to get the CSS compiling with Webpack so people who want to mess around with the framework can take advantage of the built-in watch functionality in Webpack.

Webpack 4 does not yet support HTML entry points, so CSS entry point is the only option right now.
webpack --watch is not the "cool" way to develop for the web, webpack-dev-server is the way to go as it hot-reloads changes to html/css/javascript without a full-page refresh (so the state is not lost).

Thanks for this, sorry it took so long to merge!


imageToImage inpainting?

Hello. using imageToImage inference nodejs library, is it possible to use a model for inpainting? did some tests but get an error that the model pipeline is not supported. but could be the models I tried didn't support it. is this possible? is the mask, the transparent pixels of the image sent? Thanks

for example https://huggingface.co/runwayml/stable-diffusion-inpainting under inference says Text To Image... is this because is not supported? tested sending mask in different ways but it doesn't work. get back after some time, it is doing something... "Model type not found or pipeline not implemented"

looking at the source code of JS HF library it seems this would not be img2img really? is there a way to do a custom call to the API to send the mask? thanks

ok, it works using a space, but if someone is paying for HF... that person should be able to use the model without being in a queue in the gradio space. is it possible to add a task "inpaint" or something like that? thank you!

Well, @xenova added a sub-task yesterday in this PR: https://github.com/huggingface/huggingface.js/pull/357/files#diff-8b3b4279b1381c00a2825a978eca266b637c34a17eced482517ad9156ab5655aR440
Not sure what else is needed to make it work, happy to take a look.


Schema urgent fixes to main: type => set type and definitions -> $defs

Opening the official push of the last schema urgent fixes to main, just to take the initiative. We will need to find in which milestone we suit this and tag it before merging.

Just a note here that #155 will wreak havoc on existing implementations using the (non-versioned) schema references to framework components, as I learned the hard way. For instance, there are instances of

https://raw.githubusercontent.com/ga4gh-beacon/beacon-v2/main/framework/json/common/beaconCommonComponents.json#/definitions/Info

... throughout the schemas were the updated framework will lead to external versions failing to build.
Writing & checking this - the Info links see ATM the only problematic one for the schema itself; all the other ones break only endpoints links which is maybe a lesser issue.
No good solution; the obvious one is to get rid of the deep model links to framework components, by changing referenced schemas into stand alone versions either in the model itself or in the framework. Still transition will break things.
(My own solution: I render schemas programmatically from our sources  at https://progenetix.org/services/schemas/{schemaName}, even for the ones doing deep references so I have control; e.g. https://progenetix.org/services/schemas/beaconCommonComponents/).


Feature request: Allow conditionals in extensions.yml files

Thanks for this project! Glad to have my settings and extensions being synced in my standard dotfiles repo!
I find myself using both VSCode and Codium at times. Some extensions are only available on one or the other (MS Remote tools), and some have mismatched IDs (eg. MJML, Theme by file). Allowing conditionals in the extensions.yml files would mean I don't have to manage seperate profiles for each.
I suspect this could be useful for OS specific extensions as well.

Hi, I see that you are already using profiles for different languages. So adding a different set by editors will be quite difficult.

Indeed, hence the request for conditionals similar to what is available in the settings.json files in the extension.yml files of each profile.
I'm hoping since YAML is related to JSON, it's possible. But an alternative could be to extend the YAML extension definition itself. For example:
enabled:
  - id: GitHub.codespaces
    uuid: 4023d3e5-c840-4cdd-8b54-51c77548aa3f
    editor: visual studio code
  - id: sebsojeda.vscode-apple-music
    uuid: 48c4d108-0dd6-4404-8c20-c6f62c59950c
    os: mac

I've just finished to reply to you other issue.
So I really think you need to use attributes (https://github.com/zokugun/vscode-sync-settings/blob/master/docs/attributes.md), then you can filter by os/editor/profile.
To do so, the extensions need to be manageable from the Settings

Closing in favour of keeping discussion in #32


Fix React example

Thanks for the awesome new website!
Just a quick PR to fix up the React example.

Thanks!


Move block restrictions out of .circleci/config.yml

Added an environment variable to set the CI_LEVEL ('checkin', 'merge', 'all')
To run merge level CI from the command line, run:
CI_LEVEL=merge pytest --maxerrors 10 -vv --runnightly -- tests/integration tests/pdks

This is the checkin level CI command:
CI_LEVEL=checkin pytest --maxerrors 10 -vv --runnightly -- tests/integration tests/pdks

@soneryaldiz @parijatm This is probably ready to go. We'll see if it passes.

@soneryaldiz Should be add the mock 22FFL tests to the 'merge' CI runs?

@soneryaldiz Should be add the mock 22FFL tests to the 'merge' CI runs?

There are very few nightly tests in tests/pdk/finfet_pdk and they are mostly unit tests that are not required in CI runs. Any test that is not marked nightly is already running in checkin, right?


Updated janno file

Here is a first draft to work in the changes that will be introduced by https://github.com/poseidon-framework/poseidon2-schema/pull/17
The validator should work already, enforce_types not yet. Should not be merged until this is done.

So - with this PR the package should be able to deal with https://github.com/poseidon-framework/poseidon2-schema/pull/17 if we decide to merge it.
Would be good to make a decision soon, @stschiff, @AyGhal & @wolfgangaroo.


Unable to edit journal entries when Polyglot is enabled.

When Polyglot is enabled clicking the edit button in the top right of the text field in the journal causes all text to disappear and the edit UI does not appear at the top of the text field.
Text can be typed still, but no edits save.
Running on a Pi4B, narrowed down using Find the Culprit.

What version of Foundry and which system (and version of the system) and which version of polyglot ?
Find the Culprit may have been able to identify the cause as being Polyglot but can you confirm that enabling the module breaks journal entries and disabling it fixes it ? Is it possibly an incompatibility with another module that also needs to be enabled (i.e: did you test this with nothing but polyglot enabled) ?
It all looks good from my side both with 0.6.6 and 0.7.1

Foundry is 0.6.6 running on a RasPi 4b 2GB, Polyglot 1.1.1
I've run with only Polyglot active and the problem persists.

I can't reproduce it and you're the first one to mention this issue.
What browser are you using and did you try a different browser? Although that shouldn't really matter since you say it works without the module...
Can you press F12 to open the dev console in Foundry and see if there's any errors in the console when you try to edit a journal entry ? If there is, please screenshot the console.

It occurs on both Firefox and Chrome and in both the OSE and 5e module.
 I'm able to reproduce this here now, by modifying the languages that Polyglot finds. I'm not sure though how you could have the situation where the bug happens.
That shouldn't affect, but do you have any custom languages set in the Polyglot settings ?
In the dev console, can you type the following commands and screenshot the result (make sure you click on the little arrow to the result) :
PolyGlot.languages

and then with both polyglot enabled and disabled :
CONFIG.DND5E.languages

Hopefully that helps me figure out what caused it. In the meantime, I can add a workaround so if it happens again, it won't affect the editor.

Will do!
PolyGlot.languages

CONFIG.DND5E.languages with

CONFIG.DND5E.languages without

Please let me know if this isn't right/you need something else. I'm new to helping debug this sort of thing!

Thanks... this is... I don't know what to think of this, lol. You've been very helpful though in figuring it out!
Since it doesn't work for DND5E, and I noticed you mentioned OSE earlier, I just checked with the OSE system and I'm now getting the problem. I don't know if OSE changed something recently to the system that caused it to act that way, but this is indeed quite weird.
Maybe @mesfoliesludiques knows why the languages is an Array instead of an Object, and whether it's a new change or it was a bug from the start.
Note that you said dnd5e also had the problem, can you confirm if that's true ?
Anyway, I've made appropriate fixes to prevent this and made a new release (v1.1.2) which works now with OSE.
Thanks for reporting and in your patience in debugging this issue!

I'll respond with a check on 5e in the morning. Happy to help!

Hi !
I am pushing an update to OSE that should solve the issue. Let me know.

Thanks for the quick reaction!
I've fixed it with this : https://github.com/kakaroto/fvtt-module-polyglot/commit/579a86691dc25ff72735e2b8f5e262a647a3aa2b
I'll wait for your release and see how I can make it compatible with both old and new version.

I had an issue, my translation logic was bad, you should not have to update anything !

Tested with both updates applied and it's working correctly now for me in both OSE and 5e!


Various cleanups

See commit messages

Codecov Report

Merging #5586 into master will decrease coverage by 0.19%.
The diff coverage is 95.12%.

@@             Coverage Diff             @@
##             master    #5586     +/-   ##
===========================================
- Coverage      82.9%   82.71%   -0.2%
+ Complexity    11729    11709     -20
===========================================
  Files          1390     1390
  Lines         45307    45258     -49
  Branches       5618     5616      -2
===========================================
- Hits          37563    37436    -127
- Misses         5141     5208     +67
- Partials       2603     2614     +11

Impacted Files
Coverage Δ
Complexity Δ

...rate/operation/reference/sys/check/SysChecker.java
75% <ø> (ø)
7 <0> (ø)
:arrow_down:

...main/java/io/crate/analyze/symbol/Aggregation.java
91.3% <0%> (-8.7%)
8 <0> (ø)

...crate/operation/collect/files/SummitsIterable.java
84.61% <100%> (-0.57%)
6 <1> (-1)

.../io/crate/planner/node/dql/RoutedCollectPhase.java
88.65% <100%> (-0.45%)
34 <0> (-2)

sql/src/main/java/io/crate/analyze/OrderBy.java
76.23% <50%> (-1.34%)
31 <5> (-3)

...operation/collect/sources/SystemCollectSource.java
94.28% <94.44%> (-0.09%)
26 <14> (-2)

...ormation/InformationSchemaExpressionFactories.java
95.83% <97.95%> (-1.03%)
26 <24> (+17)

...in/java/io/crate/lucene/match/CrateRegexQuery.java
43.47% <0%> (-34.79%)
4% <0%> (-4%)

...exceptions/GroupByOnArrayUnsupportedException.java
66.66% <0%> (-33.34%)
1% <0%> (-1%)

...r/transport/executionphases/SetBucketCallback.java
66.66% <0%> (-33.34%)
2% <0%> (-1%)

... and 38 more

Continue to review full report at Codecov.

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


Instruction for manual package installation in Windows is missing

Please note that the mentioned path
~/Library/Application Support/Sublime Text 3

works only in iOS
For Windows users it should be
%APPDATA%\Sublime Text 3

Source: http://docs.sublimetext.info/en/latest/basic_concepts.html#the-data-directory
Also: http://docs.sublimetext.info/en/latest/basic_concepts.html#the-user-package

I updated the readme to include this info.  See aac22c8deee3bc


Propagate nicely exceptions from domains

Currently, if a domain has an exception, it just shows the error and never terminate as we can see here:
https://github.com/robur-coop/miou/blob/0560a14ea848466ea99792d27138c47f71aab747/lib/miou.ml#L1039-L1047
From an user perspective, it's a bit annoying because the application does not stop nicely: dom0 was not informed that a domain terminated with an exception and when it tries to kill them, it will await forever... So it should not difficult and very nice to propagate the exception to dom0 and terminate nicely the program.

Probably fixed by 506c6ff.


Fix undefined variables

Minor fixes about undefined variable (typo), missing imports and globally scoped variable.
May I suggest the use of JavaScript Standard Style for two reasons:

Easier contributions
Standard eslinter may track down many little mistakes

(Anyway, great es6 refactoring ! Thanks)

Thanks for this, I will review this tonight if at all possible and merge in.

Hey, sorry for the delay. This is probably really pedantic, but I would probably declare let lastToken where it is first assigned here, rather than at the top of the parseTemplate scope? Or did I miss something?

Maybe you right! When I did this, I checked first the current master build first, here and lastToken was scoped outside the while block. But there is no need to do this thanks to let: I'll amend my PR

Ah yes, good old coffeescript transpiling... Thanks! I'll merge once you've committed this.


[FIX] Incorrect Lang variables

In raising this pull request, I confirm the following:

[ X ] I have read and understood the contributing guidelines?
[ X ] I have checked that another pull request for this purpose does not exist.
[ X ] I have considered, and confirmed that this submission will be valuable to others.
[ X ] Do the PHPCI tests pass?
[ X ] Does the StyleCI test pass?

Description of change
There were typos at descriptions of:
{{ deployer_email }}
{{ deployer_name }}
{{ committer_email }}
{{ committer_name }}

How did I never spot that? I must have been asleep 😳
Thanks


python312Packages.safety: 3.2.4 -> 3.2.5

Description of changes
Changelog: https://github.com/pyupio/safety/blob/3.2.5/CHANGELOG.md

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

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

10 packages built:

poetryPlugins.poetry-audit-plugin
poetryPlugins.poetry-audit-plugin.dist
python311Packages.safety
python311Packages.safety-schemas
python311Packages.safety-schemas.dist
python311Packages.safety.dist
safety-cli (python312Packages.safety)
python312Packages.safety-schemas
python312Packages.safety-schemas.dist
safety-cli.dist (python312Packages.safety.dist)


[Makefile][Readme] Add argument to make training

Example:
$ make training  # runs job training-project-name-base
$ make kill-training   # kills job training-project-name-base
$ make training  EXPERIMENT=second   # runs job training-project-name-second
$ make kill-training  EXPERIMENT=second  # kills job training-project-name-second
$ export EXPERIMENT=3
$ make training   # runs job training-project-name-3
$ make kill-training   # kills job training-project-name-3

@AlekseySh, @mariyadavydova, WDYT?

Resumé
Problem:

we need to be able to run multiple concurrent trains
no way to pass make-target arguments except env var
even if syntax make training EXPERIMENT=second is OK, we need to be able to kill all training jobs submitted

Possible solutions:

keep the same syntax make kill-training EXPERIMENT=second, but also keep track of all training jobs run to a file, and then during make kill-all read the job list from that file and kill jobs.
Use --description field as proposed in current version of this PR. The problem is that it's not enough to grep by project-id, to be sure not to kill other project's jobs, we need a randomized label. So let's add a random label to the description of all jobs run in current project? In this approach, we can transform make kill-all to neuro kill $(neuro -q ps --description="${PROJECT_NAME} label:${RANDOM_LABEL}).

All your comments considered, thank you!
Plus, I changed the logic: the file storing train-job-ids is now deleted only if no errors was during neuro kill. This is needed not to lose jobs running (otherwise make kill-all might forget about them).
Plus, added a test on invalid job name postfix: make train RUN=...


Schedule contrast limits calculation for dask arrays after first set_view_slice

Description
fixes #3422
Type of change

[ ] Bug-fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

References

How has this been tested?

[ ] example: the test suite for my feature covers cases x, y, and z
[ ] example: all tests pass with my change

Final checklist:

[ ] My PR is the minimum possible work for the desired functionality
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] If I included new strings, I have used trans. to make them localizable.
For more information see our translations guide.

Thanks for update test. I expanded other comment slightly, will merge after tests pass


Fix for Verio & Verio Flex on macOS Mojave

Unfortunately the diff does not show the actual code changes, as these are inside the .zip. For code review, you'll have to extract the.zip and compare against master.

@pazaan OK, our scripts are now outside of the .zip file


send on behalf

You have two different users in the 3rd paragraph about send on behalf, Alex and Donald.

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

ID: 784a196f-3e0e-0c0a-0b2f-af2105a774d9
Version Independent ID: 08300f82-3b1d-b96d-bf10-e401ac6ef2c0
Content: Allow members to send as or send on behalf of a Group
Content Source: Office365-Admin/create-groups/allow-members-to-send-as-or-send-on-behalf-of-group.md
Service: o365-administration
GitHub Login: @MikePlumleyMSFT
Microsoft Alias: mikeplum

@officedocsbot assign @mypil

@midnight2105 - Thank you for your efforts to clarify the docs.
Pull Request #620  has been merged and the Github page is updated. It may take a few days before the change migrates to the docs.microsoft.com page, but you can consider your feedback responded to and closed. If you feel it hasn't been resolved, please re-open this issue.
Thank you for engaging with the community here for the docs.


Workflow Implemented

TRAPI input has a workflow section with operations that must be completed in order specified by workflow.
Due: July 29, 2021
Details in architecture repo Git issue here.

Complete


Reorder strings in localizations

So there are some localizations missing. I can do the german one tomorrow maybe.

I guess since you already did a full rewrite I can close this. But I will look into the german localization (in another PR).

The moment I finish testing things on the develop branch, I will proceed to localization and you're more than welcome to help.


RNG-129: Improve performance of UnitSphereSampler

RNG-129:

Add a benchmark for specialisations for low dimensions.
Implement specialistions for 1D, 2D and 3D.
Use a delegate in a publicly constructed UnitSphereSampler.

RNG-130:

Fix 1 dimension sampling to only return vectors containing 1 or -1.

Coverage increased (+0.005%) to 99.521% when pulling e41bf6fc94b65c0c7ab6369ecda6dcc38e4d8ad2 on aherbert:improvement-RNG-129 into c3f7dd513bff7c22b104c186044182e883ac27f7 on apache:master.

¿it is normal that the same person does the PR make merge him? It doesn't seem ethical to me. it is my opinion.

By creating a PR it ensures the code is run through all the target environments by the CI system. It is a something I prefer for my workflow when I do not have access to the resources to personally run the build on multiple environments. I find this preferable to committing directly to master for larger changes.


Enforce memory limit for Cesium3DTilesetCache

Resolves #6226.
This PR deprecates the  Cesium3DTileset constructor option maximumMemoryUsage, and replaces it with two new options:

cacheBytes: The size (in bytes) to which the tile cache will be trimmed, if the cache contains tiles not needed for the current view.
cacheHeadroomBytes: The maximum additional memory (in bytes) to allow for cache headroom, if more than cacheBytes are needed for the current view.

If the tileset's totalMemoryUsageInBytes exceeds cacheBytes + cacheHeadroomBytes, processing of new tiles will be interrupted, and the target screen-space error used to drive level-of-detail refinement will be adjusted higher. When the camera moves to an area where totalMemoryUsageInBytes drops below cacheBytes, the target screen-space error will be adjusted back down toward the user-provided value in maximumScreenSpaceError.
The adjusted screen-space error is exposed as a new read-only property memoryAdjustedScreenSpaceError
Here is a hosted Sandcastle demonstrating the interaction between memory usage and adjusted screen-space error.

Thanks for the pull request @jjhembd!

:heavy_check_mark: Signed CLA found.
:grey_question: CHANGES.md was not updated.

If this change updates the public API in any way, please add a bullet point to CHANGES.md.

:grey_question: Unit tests were not updated.

Make sure you've updated tests to reflect your changes, added tests for any new code, and ran the code coverage tool.

Reviewers, don't forget to make sure that:

[ ] Cesium Viewer works.
[ ] Works in 2D/CV.

Instead of cacheHeadroomBytes,  perhaps maximumCacheOverflowBytes?

@jjhembd if it's not too much trouble would it be possible to move the cleanup changes into a separate PR? I have some minor comments there but don't want to distract from the main review.

would it be possible to move the cleanup changes into a separate PR?

@lilleyse I moved all the cleanup commits into the texture-cleanup branch. The changes here should now be only the memory limit changes.

Thanks @lilleyse! I think I've addressed all your feedback.
@ggetz can you give this a final look?

Thanks @ggetz! I think I addressed all your feedback. Let me know if there's anything else.

All looks good to me! Thanks @jjhembd!


Create accessibility tests for all default framework widgets

The entire Flutter SDK's widget catalog should be run against an accessibility test suite to verify they meet the minimum guidelines. Currently, there are Google tests that are run very late in the CI lifecycle that detect these breakages, but it would be good to catch it on GitHub.
For example, a select all semantics node was added in a widget, but it was too small to meet Android's accessibility guidelines for tappable nodes in https://github.com/flutter/flutter/pull/107193
Googlers, see b/258829761 for internal context, and references to the existing tests. We may able to port some of these externally to benefit the community.

\cc @justinmc


Fix online visitor values

Improve speed when having a lot of old connection data and use the time range value from the module configuration instead of a fixed value.
Should fix: https://github.com/PrestaShop/PrestaShop/issues/15128

Thanks @ckubitza


[docs-infra] Migrate docs to Next App Router

Summary
This issue is about moving our docs away from the legacy Pages Router of Next.js to App Router.
Motivation

It's critical as most new projects created by developers today are on the App Router,

https://analytics.google.com/analytics/web/#/analysis/p353089763/edit/IBVRQAiCT2mrgZwzpg9fFQ

How can we build a great DX for something we don't experience every day? It's impossible IMHO.

It's a great way to solve the Layout piece of https://github.com/mui/material-ui/issues/41117.

Context
It's related to https://github.com/mui/material-ui/issues/38966.
Search keywords: App Router

@flaviendelangle Thanks for adding this here. 👍 to measure this as we do our migration. Now, to me, this is part of the value, e.g. https://github.com/mui/material-ui/issues/39432.


Printing the command being run when encountering an error.

One suggestion: when running simpleaf, if we encounter an error, can we print the command being run? I had some path issues when running simpleaf quant. It just told me that mapping failed. To debug it, I had to go to quant_info.json and find the mapping command and remove all quotes between arguments and run the salmon alevin command.

The log /should does print the command being run. For example, I recently had one terminate with:
[2022-12-23T02:27:02Z INFO  simpleaf] cmd : "alevin-fry" "quant" "-i" "data/af_quant" "-o" "data/af_quant" "-t" "24" "-m" "t2g_3col.tsv" "-r" "cr-like"
Error: quant failed with exit status ExitStatus(unix_wait_status(512))

The problem is --- WTH is unix_wait_status(512)? It turns out the path to the t2g_3col.tsv was wrong. I think some of that can be avoided by better input and output validation between steps (e.g. check all provided input files exist, and when step i executes, ensure that all expected files for step i+1 exist when it finishes (and print an informative message otherwise).

Right, I also got weird error messages like this. Is there a way to copy and paste the error message returned by salmon and alevin-fry when quitting with errors?

It depends how they report the error, they may not report a message. The standard way to do this would be to capture the standard error buffer and dump that as well.  However, these errors (e.g. wait status 512) are usually due to missing files.  From that perspective, I think defensive checks for files being where the user expected they were would cover like 80% of the problems.

Basic input file checking has been implemented as of 034d5314467092349407deff7e79a48c15bf3f59.  It can still be improved (e.g. to make sure that things expected to be files are files and things expected to be directories are directories). Also, we can (if we choose) explicitly check for the expected files inside of directory arguments.  However, I'll consider what we currently have to be a decent solution, or at least the start of one.


Limit the number of log files generated by vDVS/vFile

Limit the number of log files generated by vDVS/vFile

Useful link - https://godoc.org/github.com/chasex/glog. Read support for log rotate.

we are using logrus for plugin side logging (https://github.com/sirupsen/logrus)


[Enhancement] Add "Try Cider" Button to Sidebar

      Special

                                            <-- Icon Here -->

                                                ✨Try Cider ✨

                    ```

create a PR.

This app will be sunseted very soon and we are NOT maintaining this app
unless of a security issue. We encourage you to try Cider, our new music app built from scratch. Issues filled on that repository will be seen and will be acted upon. Please join our discord and see the #nightly-builds channel for a current build.
Cider can be found here


connecting to db in docker container fails on windows

I'm trying to get my project working in dev mode on windows 11.
npm run backend:start creates the ps project with two containers: 1. postgres-1, 2. electric-1. This is same as on pop os and MacOS and they start without errors.
No other containers are running.
npm run db:migrate fails with:
PS C:\Users\alexa\ps> npm run db:migrate

><EMAIL_ADDRESS>db:migrate
> node ./db/migrate.js

 Electric appears not to be running for this app.
Docker container ps-electric-1 not running.

I'm probably too bad at understanding port access to solve this 😢

Hi @barbalex, npm run backend:start should start containers for postgres and electric that are named <app name>-postgres-1 and <app name>-electric-1. When running npm run db:migrate the script checks that those containers are running but it appears they are not. You said the containers seem to be named postgres-1 and electric-1 which is odd as they are missing the app name prefix.
A few things to check:

Can you check the container names with docker container ls to make sure the names are postgres-1 and electric-1 (i.e. are missing the app name prefix)
If the container names are missing the app name prefix, can you check the contents of the backend/compose/.envrc file. It is in this file that an environment variable is defined that contains the app name, e.g.:

export POSTGRESQL_IMAGE=postgres:14-alpine
export APP_NAME=bug_repro
export ELECTRIC_PORT=5134
export ELECTRIC_PROXY_PORT=65431
export ELECTRIC_IMAGE=electricsql/electric@sha256:4f21be46b836c6f3a59649cf6c8d03479c5df45ef28012c51d61706cfd8d2892

Notice the APP_NAME environment variable, this should be picked up by our compose file (cf. https://github.com/electric-sql/electric/blob/main/examples/starter/template/backend/compose/docker-compose.yaml#L2):
name: "${APP_NAME:-electric}"

The way how these environment variables are picked up is because when running npm run backend:start we invoke our dockerCompose helper function (https://github.com/electric-sql/electric/blob/main/examples/starter/template/util/util.js#L34) which passes the envrc file using the --env-file option.
Since you mentioned that you are on Windows, perhaps there is a problem with the paths that we define to the compose file and to the envrc file:
const envrcFile = path.join(__dirname, '../backend/compose/.envrc')
const composeFile = path.join(__dirname, '../backend/compose/docker-compose.yaml')

You may want to console.log them in your app and manually try to run the compose file as how our dockerCompose function would do, to see if there's a problem there.

@kevin-dp
Regarding container names: The problem was my container noobness:
PS C:\Users\alexa\ps> docker container ls
CONTAINER ID   IMAGE                         COMMAND                  CREATED        STATUS                    PORTS                                              NAMES
8f6966d545f5   electricsql/electric:latest   "/app/bin/entrypoint…"   19 hours ago   Up 30 seconds             <IP_ADDRESS>:5133->5133/tcp, <IP_ADDRESS>:65432->65432/tcp   ps-electric-1
dd15cedf6d46   postgis/postgis:16-3.4        "docker-entrypoint.s…"   19 hours ago   Up 30 seconds (healthy)   <IP_ADDRESS>:53411->5432/tcp                            ps-postgres-1

I didn't understand how they are named and saw this:

So they are named correctly. Sorry for the noise.
More in next comment.

To find out more, I added a console.log in db/util.js/fetchHostPort:
const envrcFile = path.join(__dirname, '../backend/compose/.envrc')
  const composeFile = path.join(
    __dirname,
    '../backend/compose/docker-compose.yaml',
  )
  console.log('util.js, fetchHostPort', {
    container,
    containerPort,
    service,
    envrcFile,
    composeFile,
    output: `docker inspect --format='{{(index (index .NetworkSettings.Ports "${containerPort}/tcp") 0).HostPort}}' ${container}`,
  })

the output part was copied from the shell.exec command of fetchHostPort itself.
This returns:
util.js, fetchHostPort {
  container: 'ps-electric-1',
  containerPort: 65432,
  service: 'Electric',
  envrcFile: 'C:\\Users\\alexa\\ps\\backend\\compose\\.envrc',
  composeFile: 'C:\\Users\\alexa\\ps\\backend\\compose\\docker-compose.yaml',
  output: `docker inspect --format='{{(index (index .NetworkSettings.Ports "65432/tcp") 0).HostPort}}' ps-electric-1`
}

When I manually run a text.js file with this content:
const shell = require('shelljs')
shell.exec(
  `docker inspect --format='{{(index (index .NetworkSettings.Ports "65432/tcp") 0).HostPort}}' ps-electric-1`,
)

I get this error:
PS C:\Users\alexa\ps> node test.js
template parsing error: template: :1: unclosed action

Oh, seems to be a windows quoting thing: https://stackoverflow.com/a/38886047/712005

I found the documentation on the docker inspect --format command: https://docs.docker.com/config/formatting/
So pasting this command directly into powershell works:
docker inspect --format='{{(index (index .NetworkSettings.Ports \"65432/tcp\") 0).HostPort}}' ps-electric-1

(returns: 65432)
But running this file does not:
const shell = require('shelljs')
shell.exec(
  `docker inspect --format='{{(index (index .NetworkSettings.Ports \"65432/tcp\") 0).HostPort}}' ps-electric-1`,
)

error:
PS C:\Users\alexa\ps> node test.js
template parsing error: template: :1: unclosed action

I wonder if the docker command could be written without using --format. That might be a way to prevent the error? Unfortunately that seems not to be documented: https://docs.docker.com/engine/reference/commandline/inspect/#find-a-specific-port-mapping

This is the command we execute (cf. https://github.com/electric-sql/electric/blob/main/examples/starter/template/db/util.js#L51):
`docker inspect --format="{{(index (index .NetworkSettings.Ports \\"${containerPort}/tcp\\") 0).HostPort}}" ${container}`

Note the double backslashes before the double quotes \\"${containerPort}/tcp\\"
Does that work?

Calling a test.js file containing this works:
const shell = require('shelljs')
shell.exec(
  `docker inspect --format="{{(index (index .NetworkSettings.Ports \\"65432/tcp\\") 0).HostPort}}" ps-electric-1`,
)

Pasting it directly into powershell gives: template parsing error: template: :1: unexpected "\\" in operand

So it seems that line has recently changed. In the new version of the app I built using create-electric-app today, that line is correct. Now it works on windows for me.
Well, actually, that means: I am now getting this error on windows too: https://github.com/electric-sql/electric/issues/750

So it seems that line has recently changed. In the new version of the app I built using create-electric-app today, that line is correct. Now it works on windows for me.
Well, actually, that means: I am now getting this error on windows too: #750

Great! Yes, we knew we had some issues with Windows but @samwillis fixed them recently. Glad that those fixes address this issue.


Props don't work

<Linkify properties={{style: {textDecoration: 'none', color: 'black'}, target: '_blank'}}>{text}</Linkify>
This is my code and the properties prop doesn't work. It seems to neither open the link in a new tab or style the link.

+1


Cisco ASA timestamps incorrect in HEC output after running for a few minutes

IHAC with SC4S BYOE on RHEL 7.  With Cisco ASA logs coming in over UDP, the timestamp parsing in SC4S fails after a few minutes.  The logs are coming in with times in EDT and using the HEC debug capture on SC4S, the output timestamp in the curl commands jumps back 4 hours after a few minutes.  I have both a tcpdump pcap for the input UDP traffic as well as the HEC debug file that shows this.
2020-08-28-hec-shortened.zip
cisco-asa-input.log-shortened.zip
In the hec log, between line 126,611 and 126,612, the time stamp jumps back 4 hours.  This corresponds to packets 126,611 and 126,612 in the pcap file.

The critical thing to ascertain here is what the incoming UDP packets' timestamps are compared to the sc4s server's reality.

Best way to troubleshoot this is write and use a template that emits both S_UNIXTME and R_UNIXTIME and compare them.  You will undoubtedly find that when your events pop back four hours, the difference between those two integers is > 30.
https://www.syslog-ng.com/technical-documents/doc/syslog-ng-open-source-edition/3.23/administration-guide/62#TOPIC-1268641

Ensure that your VM (or wherever you are running sc4s) is sync'd and within seconds of the device time and it doesn't drift.  This looks to me like VM clock drift of some sort...

The clocks on both the appliance and SC4S VM are synchronized with NTP.  I've check the time with ntpupdate and it is not more than a few milliseconds off from NTP time.
When I look at the times on the pcap file, there is less than a second between the UDP packets in the area where the problem occurred.  While that's the receive time of the packet, I don't see any delays in the inter-packet gap.

Please confirm fixed in current release

We are seeing this problem currently where the ASA data timestamps look correct but are showing up as being 4 hours behind, any idea? I (Bruce Penn) am working with the customer and can't seem to figure it out.


Find-DbaInstance not returning anything

I'm attempting to find all SQL instances on my ActiveDirectory but started first with a fixed list of only one server in a text file and doing this: Get-Content $PSScriptRoot\servers.txt | Find-DbaInstance
Nothing is being returned and the server has 2 instances on it.
What could I be doing wrong?
Thanks.

hy @AlreadyPicked , is this still valid ? sorry nobody replied here. Feel free to reopen if the issue still stands

Indeed. No one replied and I still have an issue with this. I will re-open.

Hey @AlreadyPicked - thanks for the note. can you slap a -Verbose on your command and paste it here?

Sure thing. Sorry, parts of it are in french. I also removed the names of the server\instance, IP address.
COMMENTAIRES : [16:02:38][Find-DbaInstance] Processing: [Server Name\Instance]
COMMENTAIRES : [16:02:59][Resolve-DbaNetworkName] Connecting to [Server Name]
COMMENTAIRES : [16:02:59][Resolve-DbaNetworkName] IP Address from [Server Name] is [IP Address]
COMMENTAIRES : [16:02:59][Resolve-DbaNetworkName] Getting computer information from dev-sql-010-v.domain.local
COMMENTAIRES : [16:02:59][Get-DbaCmObject] Configuration loaded | Cache disabled: False
COMMENTAIRES : [16:02:59][Get-DbaCmObject] [Server Name.domain.local] Retrieving Management Information
COMMENTAIRES : [16:02:59][Get-DbaCmObject] [Server Name.domain.local] Accessing computer using Cim over WinRM
COMMENTAIRES : [16:02:59][Get-DbaCmObject] [Server Name.domain.local] Accessing computer using Cim over WinRM - Failed!
COMMENTAIRES : [16:02:59][Get-DbaCmObject] [Server Name.domain.local] Accessing computer using Cim over DCOM
COMMENTAIRES : [16:02:59][Get-DbaCmObject] [Server Name.domain.local] Accessing computer using Cim over DCOM - Failed!
COMMENTAIRES : [16:02:59][Resolve-DbaNetworkName] Unable to get computer information from [Server Name]
COMMENTAIRES : [16:02:59][Resolve-DbaNetworkName] No WMI/CIM from [Server Name]. Getting HostName via .NET.Dns
COMMENTAIRES : [16:02:59][Resolve-DbaNetworkName] Resolving [Server Name.domain.local] using .NET.Dns GetHostEntry
COMMENTAIRES : [16:02:59][Get-DbaSqlService] Getting SQL Server namespace on [Server Name.domain.local]
COMMENTAIRES : [16:02:59][Get-DbaCmObject] Configuration loaded | Cache disabled: False
COMMENTAIRES : [16:02:59][Get-DbaCmObject] [Server Name.domain.local] Retrieving Management Information
COMMENTAIRES : [16:02:59][Find-DbaInstance] Computer [Server Name\Instance] could be contacted, but no trace of an SQL Instance was found. Skipping...

Sorry for the delay - what is the version of the sql servers that are missing?

All of them. I can't get any server to respond. The one above was a SQL Server 2012 SP3.

i wonder if this is a french thing. @niphlod do you have any Italian servers to test?
@AlreadyPicked - can you try again? I modified the command and it may have opened up more discovery possibilites. use -ScanType All

That seems to did the trick. Tried -ScanType All and without, both worked.
Thanks!

awesome news!! thanks for your report. This was a good and necessary fix.


Update to tokio 0.2

[ ] Blocked on h2: https://github.com/hyperium/h2/milestone/2

looks like the blocker is gone now? :)

The h2 change was just commited to the code repository and no new release uploaded to crates.io. :disappointed:

A follow up change is needed to h2 to restore all behavior: https://github.com/hyperium/h2/pull/429
Please keep in mind that in the US it's Thanksgiving weekend, so many folks will not be looking at github, instead spending time with friends and family and over eating.

hyperium/h2#429 has merged, though for the remaining blocker of publishing a new h2 release let's utilize threads in the relevant repo: https://github.com/hyperium/h2/issues/430

Yep, h2 updated its tokio dependency, there's one last remaing thing there: we need a new http release since tokio uses a newer bytes.
This is my highest priority this week!

Is an alpha release with this planned soon?


fix integrate-tests pulsar not enable broker entry meta

Codecov Report

Merging #96 (af46980) into main (06cbd8a) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #96   +/-   ##
=======================================
  Coverage   57.32%   57.32%
=======================================
  Files          14       14
  Lines        1427     1427
=======================================
  Hits          818      818
  Misses        507      507
  Partials      102      102

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 06cbd8a...af46980. Read the comment docs.


Matchmaking shows all players codenames as "0"

This was fixed a while ago, forgot to close the issue.


Update docs and scripts for beta.6 release

Is this change manual or automated?  Would it make sense (for a separate PR) to have the name of the release stored in a single file that is picked up by all these scripts?

This is a manual change. The file tools/download.sh probably will be deleted soon as we have moved to tools/install-powershell.sh. According to Travis, the file download.sh is kept for now just in case. As for the Dockerfiles, I'm not if it's doable to make them read the name from a file, @TravisEz13 could you please weigh in?

I submitted this PR targeting master branch by mistake. I have rebased the release branch against master.

I still have a case where download.sh is needed, but I plan to fix it and then remove it.

I take it back... I didn't need the script in the end, but I should still fix the issue.

Is a .spelling update needed as well?

No need to update spelling. The Travis CI passed (see CI run details by clicking 'View details').


aws-iam-authenticator

Support new Cluster.Features.AWSIAMAuthenticator

add flag --authentication-token-webhook-config-file to apiserver
mount /etc/kubernetes/aws-iam-authenticator into apiserver

docs: https://github.com/kubernetes-sigs/aws-iam-authenticator#3-configure-your-api-server-to-talk-to-the-server

This is out of the scope
/close


Fastlane subscription

Environment

OS: Ubuntu
Erlang/OTP:
EMQ: 2.4.2

Description
I am new to EMQ.Can anyone explain how Fastlane subscription works in Emqx with an example. I have thousands of micro controllers sending signals and I am trying to understand , how can I benefit from fastlane subscription.
Many thanks.

@bharat-mehta hi, the message delivery in Fastlane subscriptions should be much more efficient than the normal ones, although I have not got the benchmarks comparing of the two.
This is because, in contrast to the normal subscriptions, if a target client has subscribed to a "$fastlane" prefixed topic, the source client process would deliver the message to the target client process directly without having to go through the session process.
Where the "process" talked about is "erlang process". The client processes are to handle the TCP sockets of the clients, while the session process is for maintaining the message queues and subscription states. The erlang message passing would be a significant overhead under a high load.
But it thus loses all features the 'session process' provides, i.e. message queues, QoS1 and QoS2 related procedures. So it is mainly used for QoS1.

@terry-xiaoyu  Thanks

@terry-xiaoyu Also is $fastlane good for backend subscription with wildcards ?

No other limitations.
And yes, good for backend subscription, but use plugins with hook callbacks is a more efficient way.


Add archive files support like tar,zip

Related to https://github.com/datafuselabs/opendal/issues/1216


BSIP87: Test of force settlement fee with three actors

This test currently elicits three failures of which two are in database::verify_asset_supplies()

Supplanted by PR #4


Issues with CloudStack version number and checksum

Happy to try out the latest release; found the following:

The README example of this repo specifies the provider version to be "0.4.0". However, Terraform can't download it: indeed, lookup of the source URL in the Hashicorp Terraform Registry reveals it's "0.4.0-pre" and not "0.4.0". Maybe this should be changed in the docs. Also, in terms of repo tags, the latest one is just "0.3.0".
After changing the version, I get a checksum error (run twice, same wrong checksum).

Execution environment: Docker/Gitlab/GitLab's Terraform wrapper script.
╷
│ Error: Failed to install provider
│
│ Error while installing cloudstack/cloudstack v0.4.0-pre: checksum list has
│ unexpected SHA-256 hash
│ c031e5466360394787d1c408c853a853ed4d6e171a611ae30c0c5111f20e21b9 (expected
│ a36b77f35a0eeff9fbd5bb2fd68b986bae6023ac49e6f035ea43deec4043b3f4)

While trying same setup with the hard fork release by Orange (thanks @ArthurHlt for linking to this repo!),  I do not get the above checksum error.
- Installed orange-cloudfoundry/cloudstack v0.5.1 (self-signed, key ID B910DF52F90CB360)
Partner and community providers are signed by their developers.

After that though I get another error which is for sure not to consider in this context, but to prevent with a proper usage example as Terraform's error messages are sometimes not very helpful.
│ Error: Failed to query available provider packages
│
│ Could not retrieve the list of available versions for provider
│ hashicorp/cloudstack: provider registry registry.terraform.io does not have
│ a provider named registry.terraform.io/hashicorp/cloudstack
│
│ Did you intend to use orange-cloudfoundry/cloudstack? If so, you must
│ specify that source address in each module which requires that provider. To
│ see which modules are currently depending on hashicorp/cloudstack, run the
│ following command:
│     terraform providers

UPD - as it seems currently there is no possibility at all to use this provider?
In an old project:
provider "cloudstack" {
  config  = var.cloudmonkey_config_file
  profile = var.cloudmonkey_profile
}

Error, similarly:
Error: Failed to query available provider packages
│
│ Could not retrieve the list of available versions for provider hashicorp/cloudstack: provider registry registry.terraform.io does not have a provider named registry.terraform.io/hashicorp/cloudstack

Same issue here and mentioned already before.
It's possible to place the provider (binary terraform-provider-cloudstack_v0.4.0-pre) locally (create the folders):
~/.terraform.d/plugins/registry.terraform.io/cloudstack/cloudstack/0.4.0-pre/linux_amd64/
checksum is not compared online then and you can use the provider locally.
But it's only a workaround and the registration to the terraform provider list should be fixed.

yes @synergiator , @StepBee mentioned workaround can be applied. We are trying to push the binaries again to see if the issue still exists.

@synergiator @StepBee can you try again with:
https://registry.terraform.io/providers/cloudstack/cloudstack/latest

I am not sure if i am doing something wrong, but now i get the error
no available releases match the given constraints 0.4.0-rc1

@StepBee Can you please check if .tf file has the proper version mentioned. Or it could be caching issue, please try again.
I just did the terraform init and worked fine
harikrishna@harikrishna:~/go/src/github.com/apache/provider$ terraform init
Initializing the backend...
Initializing provider plugins...

Reusing previous version of cloudstack/cloudstack from the dependency lock file
Using previously-installed cloudstack/cloudstack v0.4.0-rc1

Terraform has been successfully initialized!
You may now begin working with Terraform. Try running "terraform plan" to see
any changes that are required for your infrastructure. All Terraform commands
should now work.
If you ever set or change modules or backend configuration for Terraform,
rerun this command to reinitialize your working directory. If you forget, other
commands will detect it and remind you to do so if necessary.

@harikrishna-patnala thanks for the hint - i tested on a different system and can confirm it's working.

Closing on your remark.


feat: port GroupTheory.Subsemigroup.Membership

The only things left are

the supr_induction' and
possibly fixing some names.

bors merge


Will this project support slot transfer data?

Will this project support slot transfer data?
For example, selectively display menu content when triggered/

I didn't get the idea from your context, could you provide more detail of your thought? Maybe a code snippet?
thanks!

如果我想在多个元素上右键打开同一个菜单呢，比如聊天列表的右键顶置功能，右键的时候分别要传一个id给菜单上的顶置按钮事件


Fix php version in composer json. Change process dependency

Fixed php version and use more restrictive constraint for symfony process. Otherwise php version requirement must be 5.5.9  at least

@muxx Please merge


Borg-backup mount bug for multiple servers

As discovered in PR #19 by @noplanman : the borg-backup mount command accepts 3 additional parameters: the name of the backup server, the backup id and the mount path. In the current implementation, the server name is ignored. In case of multiple servers, the backup will always be mounted from the last server in your list.
So: borg-backup mount whatever-you-type-here 20200318-0931 /mnt/
will also mount the backup from the last server in your server list.
This should be fixed.

Wouldn't it make more sense to have the server parameter at the end?
So leaving it away will take the default / last one. Basically: Required parameters first (i.e. backup ID and mount path)
Because at the moment, if only one backup server is defined, it is necessary to add some dummy text as the 2nd parameter, which is a bit weird, as (I assume) most people won't have multiple backup servers.
Or would the idea be, that one "has to" define the second parameter correctly always?
What would the value of the 2nd parameter with multiple backup servers be?
As theoretically, multiple servers can be defined and simply have different pools, so all values (like FQDN, etc.) would be the same.
Would a new id field or something need to be added?

I have mixed feelings about this. It feels wrong to put the server name after the mount directory.
On the other hand it's also strange to ignore the server parameter if you only have one backup server. Maybe we should just check for the server name and throw and error if it's incorrect to have consistent behaviour?

If I understand you correctly, the server should always be defined when calling the mount command, even if there is only 1 server?
Would an option be to have something like this:
borg-backup mount <backup-id>[@<backup-server>] <mount-directory>
If there is no string with @ it uses the default / last one.
Or does this complicate things?
Alternatively we fix the command as it is now and force the usage of the server.
Also, regarding the last part of my comment, how would we handle multiple backup repos (with different pools) on the same backup server?

Right, I have implemented this on my servers as follows:
Each backup server can have a new id field (which doesn't have to be unique), to make the selection via the commands easier, instead of needing the FQDN.
So commands have the following call signatures:

borg-backup init [<id|fqdn>]
borg-backup backup [<id|fqdn>]
borg-backup list [<id|fqdn>]
borg-backup check <tag> [<id|fqdn>]
borg-backup info <tag> [<id|fqdn>]
borg-backup mount <tag> <id|fqdn> <mount-path>

All parameters in [...] are optional and will result in querying all servers for the called command if left empty.
Note that I've switched the backup-tag and backup-server for the mount command, to make it consistent with the others.
Examples:
# ansible config
borgbackup_servers:
  - id: rsync
    fqdn: s010.rsync.net
    ...
  - fqdn: mybackupserver.net
    ...

# commands
# initialise both repos
$ borg-backup init

# backup to s010.rsync.net only
$ borg-backup backup rsync

# list backup tags from all servers
$ borg-backup list

# list backup tags from mybackupserver.net only
$ borg-backup list mybackupserver.net

# check backup 20200404-2000 on rsync
$ borg-backup check 20200404-2000 s010.rsync.net

# get info about backup 20200404-2000 from all servers
$ borg-backup info 20200404-2000

# mount backup 20200404-2000 from rsync to /mnt/borg-repo-mount
$ borg-backup mount 20200404-2000 rsync /mnt/borg-repo-mount

Shall I PR my changes?

Looks good and I'm ok to have this in a PR, but I'm trying to get the two other open PRs merged first. Especially 25 will bring much improved testing, what should make it easier to test future PRs. So you might want to hold on for a few days to avoid merge conflicts.

Oh, awesome progress :tada:
My changes only affect the templates, so there shouldn't be any conflicts.
Will wait anyway, to not have to rebase unnecessarily.


Refactor instantiation of test apps

Description

Ensure each app is instantiated (and middleware is applied) according to the framework-specific API.
Fix: update route paths so that Starlette-based apps don't trigger redirect responses (and send 2 spans).

Motivation
We weren't mounting the middleware the way users would actually do it in the wild, esp for Starlette and FastAPI. By doing so, we're better able to detect any UX issues for these frameworks.

Codecov Report

Merging #34 into master will not change coverage by %.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master       #34   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           10        11    +1
  Lines          362       377   +15
=========================================
+ Hits           362       377   +15

Impacted Files
Coverage Δ

tests/utils/fixtures.py
100.00% <0.00%> (ø)

Continue to review full report at Codecov.

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


Potentially Dangerous Function (unarchiveObjectWithFile) in Messaging Component

[REQUIRED] Step 1: Describe your environment

Xcode version: 13.2.1
Firebase SDK version: 8.13.0
Installation method: Swift Package Manager
Firebase Component: Messaging
Target platform(s): iOS

[REQUIRED] Step 2: Describe the problem
Our security team scanned our code and dependencies for vulnerabilities and found Use of Potentially Dangerous Function (CWE-676). Is there a plan to migrate to new api?
Source: FIRIAMClearcutLogStorage.m:171, FIRIAMActivityLogger.m:155
Attack Vector: NSKeyedUnarchiver.unarchiveObjectWithFile:
Description: Use of an unsafe function that are either deprecated due to security concerns, such as not conforming to secure coding practices, can introduce a vulnerability.
Most, if not all, of these functions have been documented as unsafe and should not be used, as mentioned in the WWDC session 'Threat Modeling', and can be replaced with more recent API calls.
Steps to reproduce:
Go to linked files, please note use of unarchiveObjectWithFile:
FIRIAMClearcutLogStorage.m:171
FIRIAMActivityLogger.m:155
Relevant Code:
... = [NSKeyedUnarchiver unarchiveObjectWithFile:filePath];

Thanks for the report and specific links, @grzegorzleszek! Sounds reasonable to update the code to use replacement calls where possible. Note that the recommended calls are only available on iOS 11+ so we'll still need to keep the old calls around since we still support iOS 10 for CocoaPods - it just won't be used on the newer platforms.


Test testItemAllowancesChargesImport() fails

Small issue:
$git clone https://github.com/ZUGFeRD/mustangproject.git
Cloning into 'mustangproject'...
...
$mvn package
WARNING: An illegal reflective access operation has occurred
...
Running org.mustangproject.ZUGFeRD.ZF2ZInvoiceImporterTest
Mar 31, 2022 12:01:45 PM org.mustangproject.ZUGFeRD.ZUGFeRDImporter <init>
SEVERE: null
java.nio.file.NoSuchFileException: ./target/testout-ZF2PushItemChargesAllowances.pdf
...
Tests in error:
  testItemAllowancesChargesImport(org.mustangproject.ZUGFeRD.ZF2ZInvoiceImporterTest): java.nio.file.NoSuchFileException: ./target/testout-ZF2PushItemChargesAllowances.pdf
  testAllowancesChargesImport(org.mustangproject.ZUGFeRD.ZF2ZInvoiceImporterTest): java.nio.file.NoSuchFileException: ./target/testout-ZF2PushChargesAllowances.pdf

steps to reproduce?
git clone https://github.com/ZUGFeRD/mustangproject.git
cd mustangproject
mvn package

works with me on windows.
Some background: some files the validator checks are previously generated by the unit tests of the library, so the unit tests of the validator also check those. I thought of explicitly making the unit tests depending on each other but it's a hassle and v(alidator) is anyway after l(ibrary) in the alphabet so why care? It's possible that I may have accidentally used windows path delimites, i.e. , or you may have tried to package the validator before running unit tests on the library, though...

Yes, those are the steps to reproduce.
I checked and there are apparently 3 tests that handle the file. The test ZF2PushTest.java creates it and the tests LibraryTest.java and the test ZF2ZInvoiceImporterTest.java expect the file to be present.
It seems however that the test ZF2ZInvoiceImporterTest.java is run before the test ZF2PushTest.java as a test shows:
$pwd
/home/alex/ZUGPFeRD/mustangproject
$mvn package
...
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 3.558 sec
Running org.mustangproject.ZUGFeRD.ZF2EdgeTest
Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.445 sec
Running org.mustangproject.ZUGFeRD.ZF2ZInvoiceImporterTest
Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.187 sec
Running org.mustangproject.ZUGFeRD.XRTest
Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.153 sec
Running org.mustangproject.ZUGFeRD.FXTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.032 sec
Running org.mustangproject.ZUGFeRD.MustangReaderWriterEdgeTest
Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.451 sec
Running org.mustangproject.ZUGFeRD.CalculationTest
Tests run: 4, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.07 sec
Running org.mustangproject.ZUGFeRD.ZF2PushTest
Apr 05, 2022 7:37:47 AM org.apache.pdfbox.cos.COSDocument finalize
WARNING: Warning: You did not close a PDF Document
Tests run: 11, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 1.523 sec

You correctly state that the validator tests are run after the library tests but it is the library test (ZF2ZInvoiceImporterTest) that is failing (not the validator test LibraryTest).
I am unfortunately not an expert on maven so that I can't tell why but it seems as if the execution order of these tests are different on my machine than on yours.


crypto com query unreceived packets borks

Crate
hermes

Summary of Bug
Trying to query unreceived packets on the crypto.com mainnet results in panic.
$ hermes query packet unreceived-packets crypto-org-chain-mainnet-1 transfer channel-10
Jun 21 17:32:05.062  INFO using default configuration from '/home/admin/.hermes/config.toml'
Jun 21 17:32:05.064 DEBUG Options: QueryUnreceivedPacketsCmd { chain_id: ChainId { id: "crypto-org-chain-mainnet-1", version: 1 }, port_id: PortId("transfer"), channel_id: ChannelId("channel-10") }
Jun 21 17:32:05.068 TRACE subscribing to query: tm.event = 'Tx' chain.id=crypto-org-chain-mainnet-1
Jun 21 17:32:05.069 TRACE subscribing to query: tm.event = 'NewBlock' chain.id=crypto-org-chain-mainnet-1
Jun 21 17:32:05.071 TRACE subscribed to all queries chain.id=crypto-org-chain-mainnet-1
Jun 21 17:32:05.071 TRACE getting counterparty chain chain_id=crypto-org-chain-mainnet-1 port_id=transfer channel_id=channel-10
Jun 21 17:32:05.071 DEBUG starting event monitor chain.id=crypto-org-chain-mainnet-1
Jun 21 17:32:05.081 TRACE counterparty chain: osmosis-1 chain_id=crypto-org-chain-mainnet-1 port_id=transfer channel_id=channel-10
Jun 21 17:32:05.081 DEBUG Fetched from source chain crypto-org-chain-mainnet-1 the following channel IdentifiedChannelEnd { port_id: PortId("transfer"), channel_id: ChannelId("channel-10"), channel_end: ChannelEnd { state: Open, ordering: Unordered, remote: Counterparty { port_id: PortId("transfer"), channel_id: Some(ChannelId("channel-5")) }, connection_hops: [ConnectionId("connection-12")], version: "ics20-1" } }
The application panicked (crashed).
Message:  called `Result::unwrap()` on an `Err` value: Error(Context { kind: Rpc(Url { inner: Url { scheme: "http", cannot_be_a_base: false, username: "", password: None, host: Some(Ipv4(<IP_ADDRESS>)), port: Some(26657), path: "/", query: None, fragment: None }, scheme: Http, host: "<IP_ADDRESS>", port: 26657 }), source: Some(Error { code: HttpError, message: "error trying to connect: tcp connect error: Connection timed out (os error 110)", data: None }), backtrace: Some(   0: <ibc_relayer::chain::cosmos::CosmosSdkChain as ibc_relayer::chain::Chain>::init_light_client
   1: ibc_relayer::chain::runtime::ChainRuntime<C>::spawn
   2: <ibc_relayer_cli::commands::query::packet::QueryUnreceivedPacketsCmd as abscissa_core::runnable::Runnable>::run
   3: abscissa_core::application::Application::run
   4: abscissa_core::application::boot
   5: hermes::main
   6: std::sys_common::backtrace::__rust_begin_short_backtrace
   7: std::rt::lang_start::{{closure}}
   8: core::ops::function::impls::<impl core::ops::function::FnOnce<A> for &F>::call_once
             at /rustc/53cb7b09b00cbea8754ffb78e7e3cb521cb8af4b/library/core/src/ops/function.rs:259:13
      std::panicking::try::do_call
             at /rustc/53cb7b09b00cbea8754ffb78e7e3cb521cb8af4b/library/std/src/panicking.rs:379:40
      std::panicking::try
             at /rustc/53cb7b09b00cbea8754ffb78e7e3cb521cb8af4b/library/std/src/panicking.rs:343:19
      std::panic::catch_unwind
             at /rustc/53cb7b09b00cbea8754ffb78e7e3cb521cb8af4b/library/std/src/panic.rs:431:14
      std::rt::lang_start_internal
             at /rustc/53cb7b09b00cbea8754ffb78e7e3cb521cb8af4b/library/std/src/rt.rs:34:21
   9: main
  10: __libc_start_main
  11: _start
) })
Location: relayer-cli/src/commands/query/packet.rs:212

Backtrace omitted. Run with RUST_BACKTRACE=1 environment variable to display it.
Run with RUST_BACKTRACE=full to include source snippets.

Version
special osmosis prod version as of Monday (after troubleshooting call)

Steps to Reproduce

Acceptance Criteria

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate milestone (priority) applied
[ ] Appropriate contributors tagged
[ ] Contributor assigned/self-assigned

Same with unreceived-acks.
Querying commitments work.

All right, this turned out to be that the GRPC port was not available for Hermes.
My issue is resolved, but it would be nice to have a more straightforward error message when a port is unavailable.


Better mobile navigation rendering

Fixes

Fixes #39  by @TimidRobot
Description
This PR depends on vocabularry-styles/pull/12 by @brylie
Test

Pull this PR for the vocabulary-styles project and compile a build version.
Enter the dist/css folder and start a dev server. You can use python by running this command in that folder python3 -m http.server 5100
Pull this PR here for cc-global-components project. Change this line https://github.com/creativecommons/cc-global-components/blob/main/dev/serve.vue#L26 to use the newly generated vocabulary-styles css. The new url should be something like http://localhost:5100/vocabulary-styles.css
run npm run serve

Screenshots

Checklist

[x] My pull request has a descriptive title (not a vague title like Update index.md).
[x] My pull request targets the default branch of the repository (main or master).
[x] My commit messages follow best practices.
[x] My code follows the established code style of the repository.
[ ] I added or updated tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no
visible errors.

Developer Certificate of Origin

For the purposes of this DCO, "license" is equivalent to "license or public domain dedication," and "open source license" is equivalent to "open content license or public domain dedication."

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

I wonder if we should include the changes to package-lock.json in a separate pull request? It's not a big deal in any case.

Approved. Will merge after we fix https://github.com/creativecommons/vocabulary-styles/pull/12


Updated SwiftPrivate.swift using String Interpolation

I updated /stdlib/private/SwiftPrivate/SwiftPrivate.swift using String Interpolations, so the code is more readable and more "Swifty". I also updated the copyright to 2018.

This PR doesn't resolve any bugs or issue.

@swift-ci Please smoke test

Build comment file:
No performance and code size changes

How to read the data
The tables contain differences in performance which are larger than 8% and differences in code size which are larger than 1%.
If you see any unexpected regressions, you should consider fixing the regressions before you merge the PR.
Noise: Sometimes the performance results (not code size!) contain false alarms. Unexpected regressions which are marked with '(?)' are probably noise. If you see regressions which you cannot explain you can try to run the benchmarks again. If regressions still show up, please consult with the performance team (@eeckstein).

Hardware Overview
  Model Name: Mac Pro
  Model Identifier: MacPro6,1
  Processor Name: 12-Core Intel Xeon E5
  Processor Speed: 2.7 GHz
  Number of Processors: 1
  Total Number of Cores: 12
  L2 Cache (per Core): 256 KB
  L3 Cache: 30 MB
  Memory: 64 GB

@xwu I applied your changes

@swift-ci Please smoke test


Davidkutu/scrapbook fix

For #9796

[x] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR)
[x] Title summarizes what is changing
[x] Has a news entry file (remember to thank yourself!)
[ ] Appropriate comments and documentation strings in the code
[ ] Has sufficient logging.
[ ] Has telemetry for enhancements.
[ ] Unit tests & system/integration tests are added/updated
[ ] Test plan is updated as appropriate
[ ] package-lock.json has been regenerated by running npm install (if dependencies have changed)
[ ] The wiki is updated with any design decisions/details.

Codecov Report

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

@@            Coverage Diff            @@
##           master   #9906      +/-   ##
=========================================
- Coverage   61.22%   61.2%   -0.02%
=========================================
  Files         563     564       +1
  Lines       30058   30061       +3
  Branches     4545    4545
=========================================
- Hits        18403   18399       -4
- Misses      10626   10632       +6
- Partials     1029    1030       +1

Impacted Files
Coverage Δ

src/client/common/utils/platform.ts
70.58% <0%> (-5.89%)
:arrow_down:

src/client/linters/pydocstyle.ts
86.66% <0%> (-2.23%)
:arrow_down:

src/client/datascience/debugLocationTracker.ts
76.56% <0%> (-1.57%)
:arrow_down:

src/client/common/process/proc.ts
14.49% <0%> (-0.73%)
:arrow_down:

...atascience-ui/data-explorer/globalJQueryImports.ts
0% <0%> (ø)

Continue to review full report at Codecov.

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


Release 0.16-M4

Bumps version;
Cleans up some warnings;

Codecov Report

Merging #863 into master will increase coverage by 0.68%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #863      +/-   ##
==========================================
+ Coverage   80.74%   81.43%   +0.68%
==========================================
  Files          50       50
  Lines         727      727
  Branches       33       33
==========================================
+ Hits          587      592       +5
+ Misses        140      135       -5

Impacted Files
Coverage Δ

core/src/main/scala/io/finch/Endpoint.scala
74.8% <100%> (ø)
:arrow_up:

core/src/main/scala/io/finch/Input.scala
100% <0%> (+12.5%)
:arrow_up:

Continue to review full report at Codecov.

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

Looks good to me!


Setting decimal column defaultTo to 0

I am creating a decimal price column in my migration file like this:
table.decimal('price', 19, 4).notNullable().defaultTo(0).index();

However, running migration, I always get this error:
 Command failed: error: null value in column "price" violates not-null constraint

What is the proper way to set the default on a decimal column to be 0? Thank you.

Can you turn debug on and grab the queries? Because I see no reason why this shouldn't work:
https://github.com/tgriesser/knex/blob/9777a1c3dea044666e3066b76935dd45ee8a6a84/lib/schema/columncompiler.js#L112-L128

This only happens when I have mixed inserts--some with price and some without. If I run them in isolation, then everything works as expected.
Here is an example of such mixed seed
    {// Without price property
     poster_type_id: 1, source_id: 1, status_id: 2, classified_id: 2, section_id: 1,
      category_id: 2, details: { condition: "Used", make: "iPhone 5c", serial:<PHONE_NUMBER>77442 },
      location: { city: "Newport Beach", state: "CA" },
      title: "This is a disabled ad #1 title", description: "This is a disabled ad #1" },

    {
     // With price property
     poster_type_id: 1, source_id: 1, status_id: 2, classified_id: 2, section_id: 1,
      category_id: 2, details: { condition: "Used", make: "iPhone 5c", serial:<PHONE_NUMBER>77442 },
      location: { city: "Newport Beach", state: "CA" },
      title: "This is a disabled ad #2 title", description: "This is a disabled ad #2",
      price: 232.34 }

that generates this error:
{ __cid: '__cid1',
  method: 'insert',
  options: undefined,
  bindings:
   [ 2,
     2,
     'This is a disabled ad #1',
     { condition: 'Used', make: 'iPhone 5c', serial:<PHONE_NUMBER>77442 },
     { city: 'Newport Beach', state: 'CA' },
     1,
     undefined,
     1,
     1,
     2,
     'This is a disabled ad #1 title',
     2,
     2,
     'This is a disabled ad #2',
     { condition: 'Used', make: 'iPhone 5c', serial:<PHONE_NUMBER>77442 },
     { city: 'Newport Beach', state: 'CA' },
     1,
     232.34,
     1,
     1,
     2,
     'This is a disabled ad #2 title' ],
  sql: 'insert into "ads" ("category_id", "classified_id", "description", "details", "location", "poster_type_id", "price", "section_id", "source_id", "status_id", "title") values ($1, $2, $3, $4, $5, $6, $7, $8, $9, $10, $11), ($12, $13, $14, $15, $16, $17, $18, $19, $20, $21, $22)',
  returning: undefined }

error: null value in column "price" violates not-null constraint
    at Connection.parseE (/Users/dmoore/projects/dimaslist/dimaslist-org-api/node_modules/pg/lib/connection.js:534:11)
    at Connection.parseMessage (/Users/dmoore/projects/dimaslist/dimaslist-org-api/node_modules/pg/lib/connection.js:361:17)
    at Socket.<anonymous> (/Users/dmoore/projects/dimaslist/dimaslist-org-api/node_modules/pg/lib/connection.js:105:22)
    at Socket.emit (events.js:95:17)
    at Socket.<anonymous> (_stream_readable.js:765:14)
    at Socket.emit (events.js:92:17)
    at emitReadable_ (_stream_readable.js:427:10)
    at emitReadable (_stream_readable.js:423:5)
    at readableAddChunk (_stream_readable.js:166:9)
    at Socket.Readable.push (_stream_readable.js:128:10)
    at TCP.onread (net.js:529:21)

[08:17:04] 'dbseed:dev' errored after 603 ms
[08:17:04] Error: Command failed: error: null value in column "price" violates not-null constraint
    at Connection.parseE (/Users/dmoore/projects/dimaslist/dimaslist-org-api/node_modules/pg/lib/connection.js:534:11)
    at Connection.parseMessage (/Users/dmoore/projects/dimaslist/dimaslist-org-api/node_modules/pg/lib/connection.js:361:17)
    at Socket.<anonymous> (/Users/dmoore/projects/dimaslist/dimaslist-org-api/node_modules/pg/lib/connection.js:105:22)
    at Socket.emit (events.js:95:17)
    at Socket.<anonymous> (_stream_readable.js:765:14)
    at Socket.emit (events.js:92:17)
    at emitReadable_ (_stream_readable.js:427:10)
    at emitReadable (_stream_readable.js:423:5)
    at readableAddChunk (_stream_readable.js:166:9)
    at Socket.Readable.push (_stream_readable.js:128:10)
    at TCP.onread (net.js:529:21)

    at ChildProcess.exithandler (child_process.js:658:15)
    at ChildProcess.emit (events.js:98:17)
    at maybeClose (child_process.js:766:16)
    at Socket.<anonymous> (child_process.js:979:11)
    at Socket.emit (events.js:95:17)
    at Pipe.close (net.js:466:12)

Ah okay so this is not a migration issue. You're actually running into #662 here. For now you can just manually set the price to 0.


feat: Partition Custom Queries

Hi,
When validating rows of large datasets, validations can run out of memory. Large datasets can be validating by partitioning them into smaller chunks. This feat extends partitioning to queries.
For Row and Column validations, DVT used a separate command validate custom-query row and validate custom-query column when validations are required on queries instead of table. This PR integrates the --source-query and --target-query into the existing generate-table-partitions command. A user may provide either --tables-list or --source-query and --target-query (or the corresponding file options), but not both. Due to the limitations of argparse, the check to see which option was chosen (tables or query) is done after parsing. This leads to less duplicated code.
Closes #1145.
This also includes code cleanup and enhancements including:

Closes #1282 and #1283
Removes duplicate code from build_conf_from_args in data_validation/__main__.py
Moves argument parsing/validation and default functions (for filters and threshold) as parameters to add_argument
Moves the testing of yaml file creation and storage to unit tests

Thanks.
Sundar Mudupalli

/gcbrun

/gcbrun

/gcbrun

/gcbrun

/gcbrun

/gcbrun

/gcbrun

/gcbrun

/gcbrun


"your situation is special"

I'm getting the following message when I follow the link to install keybase (I do not wish to). But no actual instructions on how to use GPG to sign the first device key or where to even get it.

You chose to manage your PGP private key yourself (nice!), and you don't have any Keybase devices yet. To prove your first install is really yours, you must use a computer (1) that has GPG installed, and (2) has access to your private key. Keybase will ask GPG to sign its first device key.
If you've lost your PGP private key or would rather not use that key, you can reset your account. However, you will need to prove your social identities again.

(I do not wish to)

Then don't install it. Then you don't need to fiddle with GPG.


Adding a class ".visible-XX-inline-table" or changing ".visible-*" entirely

Currently, it is impossible to use the .visible-XX classes along with input groups. Input groups require the display property to be set to inline-table. Currently, .visible-* classes are only capable of using the values

block
inline-block
inline
table
table-row
table-cell

Because of this, it is impossible to have a working element with both the classes .visible-* and input-group. Currently, as a workaround, the desired behavior can be achieved using three .hidden-* classes in place of one .visible-* class, but it would be more readable to have a .visible-XX-inline-table class. Here is an example of how one of these new classes could be used with an input group. Resize your browser to test the changes in behavior.
Alternatively, a more robust solution would be to change the .visible-* classes such that they did not override the display property. Essentially, the .visible-xs class would set display to none for sm/md/lg screen sizes, and change nothing else (and likewise for the other three classes). This shouldn't break any correct code, but it would render .visible-XX-* classes unnecessary.
I'd be happy to make a pull request once it's clear what would be a better decision for handling this bug.

Hi @MarcoGiancarli!
You appear to have posted a live example (http://fiddle.jshell.net/MarcoGiancarli/3teneete/4/show/light/), which is always a good first step. However, according to the HTML5 validator, your example has some validation errors, which might potentially be causing your issue:

line 41, column 3 thru line 41, column 8: Start tag body seen but an element of the same type was already open.
line 109, column 9 thru line 109, column 38: Duplicate ID form2.
line 134, column 1 thru line 134, column 7: Saw an end tag after body had been closed.

You'll need to fix these errors and post a revised example before we can proceed further.
Thanks!
(Please note that this is a fully automated comment.)

I revised the example and fixed the link in my original post.

Adding a class ".visible-XX-inline-table"

I'm going to call that an approximate duplicate of #14662. FYI @mdo.

Alternatively, a more robust solution would be to change the .visible-* classes such that they did not override the display property.

I'm pleased to say that we've already addressed this problem in our internal draft of Bootstrap v4 by only having hidden-* classes (which have been revamped and can be combined so as to have the same effect as the old .visible-* classes, except without the display override problem).
We're trying to minimize changes to Bootstrap v3 at this point (so as to focus on v4), so I believe we're going to leave this as "Fixed in v4, CantFix/WontFix in v3".


[Bug] WordCloud will cause crash in some small size

Version
latest
Link to Minimal Reproduction
none
Steps to Reproduce
none
Current Behavior
none
Expected Behavior
none
Environment
- OS:
- Browser:
- Framework:

Any additional comments?
No response

Need VGrammar's help, relate:
https://github.com/VisActor/VGrammar/issues/115
https://github.com/VisActor/VGrammar/issues/117


Merge convblock changes into main

The two differences are the biases (https://github.com/ZettaAI/zetta_utils/blob/3e0e2278e5760968f56f6c94ade3cef94abc6a72/zetta_utils/convnet/architecture/convblock.py#L71) and removing defaultdict in favour of a regular dictionary with handling logic (https://github.com/ZettaAI/zetta_utils/blob/3e0e2278e5760968f56f6c94ade3cef94abc6a72/zetta_utils/convnet/architecture/convblock.py#L100). I'm not sure I see the reason for either change - can you explain the reason for these changes?

Ugh, let me do this properly

So this type ignore comment thing has been showing up for me intermittently - each time this has happened, I just do git commit -n and then the next time I commit it doesn't throw an error. Any ideas?

Should be good now.


Invoking SkinnedMeshRenderer.BakeMesh() with tracked hand mesh in HoloLens 2 is creating an invalid mesh with OpenXR.

Describe the bug
Invoking SkinnedMeshRenderer.BakeMesh() with tracked hand mesh in HoloLens 2 is creating a mesh with its default mesh geometry rather than a mesh which represent the current deformation of the hand mesh.
The generated mesh also not at its intended position, it seems to spawn at world origin or head position.
We are facing this issue after transitioning our project to use MRTK OpenXR. (Mixed Reality OpenXR Plugin v1.7)
It is happening with latest MRTK version 2.8.3 and with Unity 2020.3.43
To reproduce
Steps to reproduce the behavior:
1)Open the "BasicSample" unity project with latest version of Unity 2020. (We used Unity 2020.3.43)
https://github.com/microsoft/OpenXR-Unity-MixedReality-Samples/tree/main/BasicSample

Upgrade the MRTK (Foundation & StandardAssets packages) and Mixreality OpenXR plugins to 2.8.3

3)Now attach some script to "RiggedHandLeft" or "RiggedHandRight" prefab which will capture the mesh of SkinnedMeshRenderer
component associated with the tracked hand prefab.
You can try using this mono script, just attached it to "RiggedHandLeft" or "RiggedHandRight" prefab and trigger "BakeMesh()" when  hand is tracking in HoloLens 2.
https://gist.github.com/krishx007/a645f77c1284b2f69479da8e3266fce1
Expected behavior
Invoking SkinnedMeshRenderer.BakeMesh() with tracked hand mesh in HoloLens 2 should generate a mesh geometry similar to the current deformation of the hand mesh at the exact same position where the tracked hand mesh was visible.
Screenshots
If applicable, add screenshots to help explain your problem.
Your setup (please complete the following information)
Unity 2020.3.43
MRTK version 2.8.3
Mixreality OpenXR plugins to 2.8.3
Target platform (please complete the following information)

HoloLens 2

Additional context
Add any other context about the problem here.

Hi @keveleigh, could you please have look at this bug. Our project progress has been stalled because of this bug.

#11399 is describing the root of the issue, with a work around.

@krishx007 Were you able to resolve this issue?

Hi @krishx007,
We appreciate your feedback and thank you for reporting this issue.
Microsoft Mixed Reality Toolkit version 2 (MRTK2) is currently in limited support. This means that Microsoft is only fixing high priority security issues. Unfortunately, this issue does not meet the necessary priority and will be closed. If you strongly feel that this issue deserves more attention, please open a new issue and explain why it is important.
Microsoft recommends that all new HoloLens 2 Unity applications use MRTK3 instead of MRTK2.
Please note that MRTK3 was released in August 2023. It features an all new architecture for developing rich mixed reality experiences and has a minimum requirement of Unity 2021.3 LTS. For more information about MRTK3, please visithttps://www.mixedrealitytoolkit.org.
Thank you for your continued support of the Mixed Reality Toolkit!


GSM8K Accuracy Gap on Jamba

Hi, thanks for this wonderful repo.
I have met a problem when running the evaluation task of GSM8K on the model Jamba. I ran both gsm8k and gsm8k_cot but got much lower accuracy than the released result.
I got:

Tasks
Version
Filter
n-shot
Metric
Value

Stderr

gsm8k_cot
3
strict-match
3
exact_match
0.2109
±
0.0362

flexible-extract
3
exact_match
0.2344
±
0.0376

But the result listed in the paper is 59.9 for 3-shot CoT. Could you please help to check? Thanks a lot.

I recommend asking the Jamba authors for more details on how they evaluated their models. Typically gaps like this are explained by undisclosed custom prompting in the paper.

@Mandy-77
could you share the logs / outputs for your 3-shot gsm8k_cot run? because we hardcode the original CoT paper's 8-shot prompt in gsm8k_cot (https://github.com/EleutherAI/lm-evaluation-harness/blob/main/lm_eval/tasks/gsm8k/gsm8k-cot.yaml) the setting of num_fewshot: 0 in its config file is meant to be un-overrideable since adding extra fewshots to this would mess with the formatting.
Aside from this, I'd recommend editing gsm8k_cot's prompt to be 3-shot and seeing what you get. As Stella mentioned, if you can't reproduce it, you probably just need to ask the Jamba authors for eval details.


Opacity reset to initial value when animation ends

When using opacity effect in Chrome, the opacity jumps back to the initial element opacity when the animation ends.
Tested with Chrome 40.0.2214.115 m, Windows 7, web-animations.js 1.0.6
In the following example, the initial opacity of the div is 0.
When clicking on the startAnimation button, the opacity is animated from 0 to 1.
When the animation ends, the opacity jumps back to 0.
I would expect the opacity to stay at 1.
Am I doing something wrong?
<!DOCTYPE html>

    <meta charset="utf-8">
    <title>web animations</title>

Hello world!

    <button onClick="startAnimation()">Start Animation</button>

This behavior is correct, you can use fill: 'forwards'
    var startAnimation = function() {
        var elem = document.querySelector('#elem');
        var player = elem.animate([
            {opacity: getComputedStyle(elem).opacity},
            {opacity: 1.0}
        ], {
            duration: 500,
            fill: 'forwards',
        });
    };

Thanks


Auto Start Break Timer: Right now the timer always auto starts regardless of this setting, need to add in support for it, so it either starts, or waits to be manually started

Support for this was added in commit e84d4a5


Responsible disclosure policy

Hello 👋
I run a security community that finds and fixes vulnerabilities in OSS. A researcher (@saharshtapi) has found a potential issue, which I would be eager to share with you.
Could you add a SECURITY.md file with an e-mail address for me to send further details to? GitHub recommends a security policy to ensure issues are responsibly disclosed, and it would help direct researchers in the future.
Looking forward to hearing from you 👍
(cc @huntr-helper)

Hi There @JamieSlome / @huntr-helper / @saharshtapi
We have to talk about a policy in the core team first but please shoot me a mail to soc AT mjmair DOT com about the issue. I will share it on backchannel with the rest of the team so we can get a fix out.
@SchrodingersGat thoughts on a security.md ?

@JamieSlome thanks for reaching out. We will certainly take this seriously. And your suggestion for adding a security policy to the project is well noted :)

Hello, @matmair @SchrodingersGat - thanks for your response and diligence 👍
Not sure if you are interested, but you can always point your community toward our platform for your security management, as an example:
https://huntr.dev/repos/inventree/inventree
You can also see the first report here if you sign-up:
https://huntr.dev/bounties/4cae8442-c042-43c2-ad89-6f666eaf3d57/
If signing up is out of the question, let me know and we will just wait for you to create a security e-mail, and we will get the details sent over :)


COBOL - Improve conditional compilation support (syntax highlighting)

Description of the enhancement requested
Currently the syntax highlighting of compilation directives, in particular conditional compilation directives, is not uniform.
On my VS Code configuration (with default options):

the >>if / >>else / >>end-if directives are colored in red (not bold)
the other directives in the format >>directive are colored in bold green
directives that do not use the >> prefix are colored blue, sometimes bold, sometimes not

We suggest that compilation directives, and in particular conditional compilation directives, use specific syntax highlighting that clearly distinguishes them from COBOL source code.
For example :

compilation directives could be highlighted by applying a background color to them
conditional compilation directives should all have the same syntax highlighting, if possible different from the syntax highlighting of other compilation directives.

Is it possible to change the syntax highlighting for some COBOL elements yourself? And if yes, how ?

@FALLAI-Denis Thanks for the feedback.  We are looking at how we can improve syntax highlighting, one of the issues we come against is VS Code uses Textmate for syntax highlighting which has some limitations for situations like this.  We will label this as a bug and continue our research.

Hi,
Regarding the colorization of conditional compilation directives, I found the origin of the problem in the textmate grammar.
for >>if, >>else, >>end-if:

for all other directives of the form >>xxxx:

I think this could be corrected by a rule of the form:

Ich bin derzeit nicht im Büro. Ich kehre zurück am  03.01.2022.
Während meiner Abwesenheit bitte bei Postkorb SCM-Admin melden!
Die Rückmeldung bezieht sich auf ein Mail mit folgendem Thema:
Re: [IBM/zopeneditor-about] COBOL - Improve conditional compilation support
(syntax highlighting) (#88)

Gesendet (c) GRZ/RSG Linz 2021 Agent 'Abwesenheit'

Raiffeisen Software GmbH
Firmenbuchnummer: 86804d, Landesgericht Linz
UID: ATU 36961105
Der Austausch von Nachrichten mit dem oben angeführten Absender via E-Mail
dient ausschließlich Informationszwecken. Rechtsgeschäftliche Erklärungen
dürfen über dieses Medium nicht ausgetauscht werden.
Correspondence with above mentioned sender via e-mail is only for
information purposes. This medium is not to be used for the exchange of
legally-binding communications.

Hi,
Problem still present in ZOE 3.0.0.
Also missing new compiler directive for IBM Enterprise COBOL V6R4: >>data
Correction suggestion:

Hi,
Problem still present in ZOE 3.1.1...  😢


Add new exercise pangram

We have anagrams and palindromes so we can't leave out pangrams. I think it's a nice short and simple task for beginners.

@rchavarria how about this?  I would be happy to implement this in the ES track.  I could do JS too at the same time I guess.  Let me know.  @behrtam thanks for doing this

@matthewmorgan It would be awesome to have both!

This is great, thank you!


Domain activation is Failed - [BRIEF DESCRIPTION]

App Id: d1igffntj9dkjc
Region: ap-northeast-1
Step: Domain activation
Status: Failed
Issue/question
Domain Activation fails.
Error message
One or more of the CNAMEs you provided are already associated with a different resource.
Additional information

no CNAME record in my domain
no CloudFront distribution yet

I re-created my domain and it begins to work.


Fixed #30231 -- Fixed FilteredSelectMultiple widget verbose_name usage

https://code.djangoproject.com/ticket/30231
I added the JS changes for updateSelectFilter made by Florian Schuler and added a Selenium test to verify the verbose_name was being showed correctly.

@carltongibson, if Teresa is not in the market for finishing this off, I'd happy to put my hand up to have a look.
That is, if you are happy with this proposal. If you think it would cost you more time, not less, then please tell me to leave this alone!

Hi @smithdc1 Great thanks. I don't think it needs much. Mainly the extra test case (and seeing if there's the opportunity for re-use) 👍

OK, let's finish off in #13436. Thanks for getting this one started @TeresaPartidaS. Sorry again for the delay in following up! 🙏