Fix TableWriteTest compression selection

The test was reading the wrong bits representing the compression value.
Instead it was reading the multiDriver (0 or 1) setting which resulted in either no compression or ZLIB compression (1) instead of ZSTD (4).

Unrelated test failure in the pipeline.


Minimize the uniqueness of bytecodes

Duplicate of the https://github.com/FuelLabs/fuels-rs/issues/1267, but for the TS SDK. It is better to have only one issue with the description since both SDKs have the same problems.

@xgreenx Contextually, we used to have a single multi-call .sw script that we used for everything. One of the problems was that it could handle a limited number of calls. Then, we replaced it with its dynamic counterpart using WASM fuel-asm and following how the Rust SDK was doing it.
Do you think we should return to having a statically compiled .sw script in this sense?
Or just change how we assemble it dynamically before calling it?

@arboleya I'd like to reassign this to library. IMO this will likely be handled by a script loader in a similar way to the new loader contract bytecode generation, that should live inside the script or program packages.

@arboleya Should we move this back to the backlog? It seems we still lack some information about how to proceed here


Implement TLOAD and TSTORE

Description
[PR description]
Issue Link
[link issue here]
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

Contents

[item]

Rationale
[design decisions and extended information]
How Has This Been Tested?
[explanation]

How to fill a PR description
Please give a concise description of your PR.
The target readers could be future developers, reviewers, and auditors. By reading your description, they should easily understand the changes proposed in this pull request.
MUST: Reference the issue to resolve
Single responsability
Is RECOMMENDED to create single responsibility commits, but not mandatory.
Anyway, you MUST enumerate the changes in a unitary way, e.g.
This PR contains:
- Cleanup of xxxx, yyyy
- Changed xxxx to yyyy in order to bla bla
- Added xxxx function to ...
- Refactored ....

Design choices
RECOMMENDED to:

What types of design choices did you face?
What decisions you have made?
Any valuable information that could help reviewers to think critically

i think you forget to modify get_rev_op_by_ref.   Strange that the tests cannot find this.

I have added the changes by @lispc
fix EIP1153: is_static check and reversion to the PR upstream here: https://github.com/privacy-scaling-explorations/zkevm-circuits/pull/1811


how to use?

idk how to use it, I have read the instruction, so what should we do next?
if I open the Act As A Discord Bot file, it couldn't find the deps folder, so it couldn't open.

what files do you have in the program's folder besides what was already there?


Allow CI to assume s3_scala_releases_read_role_arn

What does this change?
This change is required to allow Buildkite to assume to s3 read role. This reduces the permissions Buildkite uses when running certain operations. See https://github.com/wellcomecollection/catalogue-api/pull/766 for a discussion of why this is now required.
How to test

[ ] Can Buildkite perform the same jobs it has before without error (see: https://buildkite.com/wellcomecollection/catalogue-api/builds/2980 for examples of before & after).

How can we measure success?
This change follows https://github.com/wellcomecollection/catalogue-api/pull/763, and is part of resolving subsequent permissions issues. See:

https://github.com/wellcomecollection/aws-account-infrastructure/pull/18
https://github.com/wellcomecollection/catalogue-api/pull/766

Have we considered potential risks?
This changes the permissions available to Buildkite, but practically it reduces the scope of permissions used so long as the S3 read role permissions are not expanded in the future.

Part of https://github.com/wellcomecollection/platform-infrastructure/issues/431


ErrImagePull on actions-runner-metaphor-frontend pod

actions-runner-metaphor-frontend pod is falling on localhost:
github-runner               actions-runner-metaphor-frontend-j9g7r-qjm7c                      0/1     ErrImagePull       0               27m
Pod describe:
  Type     Reason     Age                From               Message
  ----     ------     ----               ----               -------
  Normal   Scheduled  27m                default-scheduler  Successfully assigned github-runner/actions-runner-metaphor-frontend-j9g7r-qjm7c to k3d-kubefirst-agent-2
  Warning  Failed     25m                kubelet            Failed to pull image "summerwind/actions-runner-dind": rpc error: code = Unknown desc = failed to pull and unpack image "docker.io/summerwind/actions-runner-dind:latest": failed to copy: read tcp <IP_ADDRESS>:35464-><IP_ADDRESS>:443: read: connection reset by peer
  Warning  Failed     16m                kubelet            Failed to pull image "summerwind/actions-runner-dind": rpc error: code = Unknown desc = failed to pull and unpack image "docker.io/summerwind/actions-runner-dind:latest": failed to read expected number of bytes: unexpected EOF
  Normal   BackOff    16m (x2 over 25m)  kubelet            Back-off pulling image "summerwind/actions-runner-dind"
  Warning  Failed     16m (x2 over 25m)  kubelet            Error: ImagePullBackOff
  Warning  Failed     56s                kubelet            Failed to pull image "summerwind/actions-runner-dind": rpc error: code = Unknown desc = failed to pull and unpack image "docker.io/summerwind/actions-runner-dind:latest": failed to copy: httpReadSeeker: failed open: server message: invalid_token: authorization failed
  Warning  Failed     56s (x3 over 25m)  kubelet            Error: ErrImagePull
  Normal   Pulling    41s (x4 over 27m)  kubelet            Pulling image "summerwind/actions-runner-dind"

issue occurred due to network intermittence


stringifyObjects replacement

Sometimes you pass objects into a child component or HTML element and when serialized it just produces:

  <my-child-stub someProp="[object Object]" />

In jest-serializer-vue-tjw there was an experimental feature that would loop over the VDOM nodes and get the original values, then pass them through a complex stringification function to display them in the snapshots:

  <my-child-stub someProp="{ food: 'burrito', quantity: 2 }" />

Much of this problem space overlaps with addInputValues, so this issue is dependent on what happens in #1.

So I attempted the 3rd solution, and everything maps perfectly, but one little problem....

Text

vnode.attributes('title') // '[object Object]'
vnode.element.getAttribute('title') // '[object Object]'

I can't find a way to access the original value ({ x: 2 }), the attribute seems to always be a string representation in Vue 3 (unlike Vue 2).
This may be different for child components, or stubbed child components, I haven't experimented with that yet, so there may still be some value in this feature if one of those works, but would need to rename it to be more specific.
Also potentially there could be a way to loop over the Virtual DOM to find the internal value a different way, but not sure how possible that is.
Created branch for current experiement:

https://github.com/tjw-lint/vue3-snapshot-serializer/pull/7/files

After digging through the vnode object for over an hour I found it.

vnode.wrapperElement.__vnode.props.title; // { x: 2 }
It works! So this feature is actually possible in a safe and reliable way in Vue 3. Still need to do real world testing in actual repos, but looks good so far.
Final implementation boils down to

If feature is enabled, loop over all VDOM nodes and add an attribute with a unique key on each element.
Create the cheerio object like normal from wrapper.html()
If the feature is enabled, loop over all cheerio elements and get their unique key, match it against the VDOM node.
Loop over all non-key attributes on a cheerio element and replace their string value with the true value from the VDOM
Then delete the unique key attribute off of both the VDOM node and the Cheerio element.
Finally, convert the cheerio object back to a string of html with $.html()
Format the string and store the snapshot


ENH: Configure pytest to use DTParser & DTFinder

overrode the DoctestModule class to use the DTFinder which utilizes the DTParser when creating a DoctestItem
overrode the DoctestTextfile to use an instance of DTParser
added a test case to ensure pytest goes through the DTParser to filter out stopwords and pseudocode in modules.
update the CI to include a test for the stopwords_cases module through the plugin.

Closes #73

Oops. I didn't mean to create two separate jobs, I think I will use a for loop instead.

LGTM. Can be tweaked later, if needed.

Great!
Will proceed to merge this. 🚀


Memory leak in Java_sun_lwawt_macosx_CInputMethod_nativeNotifyPeer

zcash4mac 1.0.4-208
Steps:
Launch zcash4mac.
What happened:
Memory leak in Java_sun_lwawt_macosx_CInputMethod_nativeNotifyPeer
https://www.dropbox.com/s/pf54fy2ge7yx5o9/zcash4mac.trace.zip?dl=0
Expected result:
No leaks.
Notes:
Unzip and open file with Apple Instruments (part of Xcode tools).

This project is dead so retracting.


Ensure methods like dispose are not async by accident

Tiny PR, just ensure nobody will accidentially write sth like Future<void> didChangeDependencies() or Future<void> dispose
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Since this is not super-critical I am going to close this for now before it just goes stale. Feel free to reopen it with the feedback from above addressed when you have time to work on this again.

Sure! I will do so when I get back to Flutter.


Feat: AI plugins

Description
Docs for the new AI plugins in Gateway 3.6.
https://konghq.atlassian.net/browse/DOCU-3606
The plugin schemas (configuration reference pages, eg see AI Proxy) are all prefilled via docs-plugin-toolkit.
To do:

[x] Icons for plugins
[x] Short descriptions for hub landing page
[x] Full descriptions for each plugin
[ ] Basic example config for each plugin
[x] Short description of the AI plugin category (for top nav/menu)
[x] Kong Gateway feature table entry
[x] Icon for top nav
[ ] (Optional for beta, especially since we have a quickstart available) Detailed how-to guide for each plugin

Testing instructions
Gateway feature table
AI category on the hub
AI Proxy
AI Request Transformer
AI Response Transformer
AI Prompt Template
AI Prompt Decorator
AI Prompt Guard
Checklist

[x] Review label added
[x] PR pointed to correct branch (main for immediate publishing, or a release branch: e.g. release/gateway-3.2, release/deck-1.17)

I'll fill in ALL of this tomorrow! Then nearly good to go.

Preview build is currently broken because I don't think we planned for this use case on dev.
@fabianrbz - I'm running into an issue where the new plugins, added only for 3.6, are causing the build to break (because there's no release version? I'm not 100% sure.)

@lena-larionova I pushed a fix for it.
The problem was that for plugins that have minimum_version set to dev a latest release didn't exist.
I fixed that now.
Bear in mind that plugins that have minimum_version set to dev will not show up in /hub/ and are only accessible by entering the URLs, e.g. /hub/kong-inc/ai-proxy/dev/.

Bear in mind that plugins that have minimum_version set to dev will not show up in /hub/ and are only accessible by entering the URLs, e.g. /hub/kong-inc/ai-proxy/dev/.

That's perfect. I was concerned that new plugins would show up on the plugin hub landing page if they're in dev state, which would be false advertising. Thanks for fixing so quickly 🙏

@cloudjumpercat I wanted to point you to this companion PR meant to extend this PR which includes a get-started page for this capability.  This will include some additional context on the "why" that might help you and docs readers.
https://github.com/Kong/docs.konghq.com/pull/6882

I wanted to point you to this companion PR meant to extend this PR which includes a get-started page for this capability. This will include some additional context on the "why" that might help you and docs readers.

+1 on this, and all of the "broken" links in the AI plugin docs that point to the quickstart go to Rick's doc.
I can also add a sentence that's something like "learn more about our AI Gateway" early on in each plugin's intro, then link to the quickstart. That should at least help set the context.


Update rest-backend and rest-lib-utils to Spring 2.2.4

Update spring services from version 1.5.20 to 2.1.4

We cannot use a migration to fix the checksum as (of course) spring/flyway will not do anything (apply new migrations or  create a baseline) if the validation does not go through.
I think the checksum issue may be caused by the fact that the latest flyway release does not support Postgres 11.  They added support in flyway 6.0 but it only exists as a beta [1]. On top of the fact that it's not recommended to use a beta in production, we cannot even test it as we get the same error of [2] as "Spring Boot is not yet compatible with Flyway 6.0". As far as they say it will be once they release 6.0 (no beta).
As it does not make sense to downgrade postgres, i think we need to wait for the release of 6.0 to finally move to spring2.
[1] https://flywaydb.org/blog/flyway-6.0.0-beta
[2] https://github.com/flyway/flyway/issues/2333

In Spring2 the default for Id generator changed:
"The spring.jpa.hibernate.use-new-id-generator-mappings property is now true by default to align with the default behaviour of Hibernate. If you need to temporarily restore this now deprecated behaviour, set the property to false."
The new default was causing our test to run awfully slow (e.g. testGetAppVulnerabilitiesForBundledLibs was running for 14 minutes instead of 2), spending most of the time in the saving of the libraries whose debug showed calls to an hibernate sequence to get the next value.
For the time being we reverted to the configuration used in Spring 1.5 (false), we will need to investigate if the new default also causes conflicts with the existing data and how to migrate.

Seems like Flyway 6.0 is not yet compatible with Spring 2.1
https://github.com/flyway/flyway/issues/2333

When using spring 2.2.M6, which, according to https://github.com/spring-projects/spring-boot/pull/17993 should fix the issue with flyway, we get another compatibility issue between the latest version of swagger (2.9.2) and spring 2.2, see https://github.com/springfox/springfox/issues/2932

Restbackend Latest changes:

updated springboot to latest version 2.2.4 (and related code changes)
left flyway version unspecified (to get the one defined in the springboot parent pom) [Note that springboot does not work with the very latest flyway version. "Our" compatibility issues among postgres11-flyway-springboot  were solved with springboot 2.2.x,
Updated swagger to latest version (even if it's a SNAPSHOT) to solve compatibility issues. That was the easiest change even if it means that we depend on a SNAPSHOT that may never be released as swagger is not maintained anyloger (?) [ see springfox/springfox#2932]. We may decide to switch to springdoc (https://www.baeldung.com/spring-rest-openapi-documentation).
Explicitly set UTF-8 in HTTP responses as it's not the default anylonger
hateos.Resource was renamed into EntityModel (see https://docs.spring.io/spring-hateoas/docs/current/reference/html/#migrate-to-1.0)

The new constraint on the 4-tuple in Dependency.java is not supported by H2:
Caused by: org.h2.jdbc.JdbcSQLFeatureNotSupportedException: Feature not supported: "Index on BLOB or CLOB column: ""RELATIVE_PATH"" TEXT"; SQL statement:
alter table app_dependency add constraint UKnueog86fts45j2wcql6idbqwn unique (lib, app, parent, relative_path)

Review comments are addressed


Check that c++14 is fully supported before compiling

Setting CMAKE_CXX_STANDARD only add the flag, but maybe that the compiler is
too old and does not fully support it.
This way, it will fail before compiling and avoid understandable errors during
compilation.

-- The C compiler identification is GNU 4.8.5
-- The CXX compiler identification is GNU 4.8.5

and the CMake step didn't give an error. The build failed though:
[ 26%] Building CXX object coreneuron/CMakeFiles/coreneuron.dir/apps/corenrn_parameters.cpp.o
In file included from /gpfs/bbp.cscs.ch/home/olupton/CoreNeuron/coreneuron/apps/corenrn_parameters.cpp:9:0:
/gpfs/bbp.cscs.ch/home/olupton/CoreNeuron/coreneuron/apps/corenrn_parameters.hpp:43:28: warning: missing terminating ' character [enabled by default]
     unsigned spikebuf = 100'000;           /// Internal buffer used on every rank for spikes
                            ^
/gpfs/bbp.cscs.ch/home/olupton/CoreNeuron/coreneuron/apps/corenrn_parameters.hpp:43:5: error: missing terminating ' character
     unsigned spikebuf = 100'000;           /// Internal buffer used on every rank for spikes
     ^

I hadn't actually seen this error before, but it looks like missing support for digit separators.
I thought with this PR there should have been an error at CMake execution time?

C++14 support is broken/incomplete for GCC4.x: https://gcc.gnu.org/projects/cxx-status.html
We should add a check for it (>5).

So it is useless to check it in the way I did here, right?
This cxx_14_support only tell us that the flag is available, right?
should we check version for gcc / clang / icc / vscode?

Thinking about it, it's not that straightforward.
I remember reading about this: https://cmake.org/cmake/help/latest/prop_tgt/CXX_STANDARD_REQUIRED.html
but not sure that will work out.
Maybe the only valid way would be to have a test.cpp with all C++14 features and try to compile that with whatever compiler comes in.  But I wouldn't go that far, especially if we can rely on CXX_STANDARD_REQUIRED for most of the time.

In the case of GCC 4 then it's enough to change it to
if(NOT "cxx_std_14" IN_LIST CMAKE_CXX_COMPILE_FEATURES OR NOT "cxx_digit_separators" IN_LIST CMAKE_CXX_COMPILE_FEATURES)

but for later C++ standards they don't have all of these specific feature macros.
I would vote for doing the best we can with CMAKE_CXX_COMPILE_FEATURES and not writing our own table of compilers and minimum versions.

Failures should be fixed by #592, but I'm not sure we need to re-run the CI on this branch (it only touches CMake files...)

I rerun only failed jobs


Issue/6612 fix mypy error on delete cascade method

Description
Fixes the following mypy error:
$ make mypy-diff
- src/inmanta/data/__init__.py:2843: error: Signature of "delete_cascade" incompatible with supertype "BaseDocument"  [override]
- src/inmanta/data/__init__.py:2843: note:      Superclass:
- src/inmanta/data/__init__.py:2843: note:          def delete_cascade(self, connection: Optional[Connection] = ...) -> Coroutine[Any, Any, None]
- src/inmanta/data/__init__.py:2843: note:      Subclass:
- src/inmanta/data/__init__.py:2843: note:          def delete_cascade(self, only_content: bool = ..., connection: Optional[Connection] = ...) -> Coroutine[Any, Any, None]

closes #6612
Self Check:
Strike through any lines that are not applicable (~~line~~) then check the box

[x] Attached issue to pull request
[x] Changelog entry
[x] Type annotations are present
[ ] Code is clear and sufficiently documented
[x] No (preventable) type errors (check using make mypy or make mypy-diff)
[ ] Sufficient test cases (reproduces the bug/tests the requested feature)
[ ] Correct, in line with design
[ ] End user documentation is included or an issue is created for end-user documentation (add ref to issue here: )
[ ] If this PR fixes a race condition in the test suite, also push the fix to the relevant stable branche(s) (see test-fixes for more info)

Added the clear method for Environment. I didn't add it for Project since there was no existing use case for it.

Processing this pull request

Pull request rejected by merge tool. Failed to parse change entry file: Change entry /tmp/tmpyk91j133/changelogs/unreleased/6612-fix-mypy-error-on-delete-cascade-method.yml does not match pydantic schema: 1 validation error for ChangeEntry
sections
none is not an allowed value (type=type_error.none.not_allowed)

Processing this pull request

Merged into branches master in 5169cc90e284358b4e2ad5987978f017aed38e08

Processing #6666.


dlib cross-platform boilerplate with web-camera (Qt based)

Hi all,
Hope you are all well !
Is it possible to create a cross-platform boilerplate for testing examples, maybe based on qt5 for limited computational resource, embedded systems and IoT devices ?
Why ?
Easy to test and boostrap, so we can improve performances more easily.
How ?
Potential base #1
I would suggest to use maybe a project like gatherer as a boilerplate base; it is a cross platform gpgpu opengl (es) shader pipeline framework: ogles_gpgpu + qt (UI). For anything related to vision, web camera, It would provide a demo for linux, ios, android and windows.
Also, it uses hunter, a CMake-driven cross-platform package manager for C++. Linux, Mac, Windows, iOS, Android, Raspberry Pi, and Polly, a toolcahain helper,
They already have integrated DLIB [here], or opencv for ios, android, desktop, windows (link).
It is really convenient for caching,  customizing some packages build. For eg, (dlib shared, dlib static)
Potential base #2
Or your have this Vuforia employee project, Yarrar, stopped because he is working full-time now (ref discussion) , examples (iOS,Android,Desktop and Unity)
**Potential base #3 **
Mixing both above, I can do that.
Nb. I saw tiny-dnn using these helpers for iOS; https://github.com/tiny-dnn/tiny-dnn/tree/master/examples/ios
Questions:

Is there any simple example with MMOD, or the normal facial landmark detector for iOS/Android available ? (would be funny to test this repo, @grisevg,  with facial landmarks for sheep, it made me laugh so much :-) )
Are NEON instructions able to improve performances for MMOD (eg. dlib for arm) ?
Shall I start a new issue ? I just commented as it was related to the fact to test the dog hipsterizer on iOS and to bundle a default client boilerplate for dlib cpu or mobile devices.

Hope I was clear enough about this suggestion for dlib, meanwhile I wish you all an awesome day !
Nb. it was inspired/related to a previous issue: https://github.com/davisking/dlib/issues/284
Cheers,
Richard

Yes, NEON instructions make things faster if they are used.
I have no idea what your suggestion is about.  Making some desktop application that uses dlib?  That's super.  We aren't going to put desktop applications into the dlib repository though.  If you want to do something like that then it should be its own separate project.


Bump Swift version from 4.1.2 to 4.1.3

This PR bumps the Swift version to 4.1.3 on the swift-4.1-branch.

Resolves rdar://problem/42669362

@swift-ci please smoke test

Mac OS Smoke test is expected to fail due to infrastructure.


Importing config from parent folder

Forgive me if this is a beginner question!
In app/main/init.py
you import config from the parent folder.
I've been running the code as I go and it won't import from the parent - am I missing something?
I've seen a work-around as:
`import sys
sys.path.append('..')
Thanks, Doug.

This works if the current directory is the parent, because the current directory is in the import path. It will not work if the current directory is somewhere else.

Thanks for the quick response.
So I guess I have to move config.py down to the /main folder, so that when init.py is run (which is in the /main folder), it can locate config.py, also in the /main folder.

Thanks for the quick response.
And I was confusing myself, but my main point was still relevant.
I was actually referring to the file: app/init.py
This file was trying to import config from the parent folder - flasky
So if I understand you, I need to either move config.py down to the app folder, so that when init.py runs, then both it and config.py are in the same import path, OR amend the import path by this code, so that it looks in the parent folder too?
`import sys
sys.path.append('..')
Thx, Doug.

No, I think I did not make myself clear.
When you have import config in app/__init__.py, you are seeing an import from a parent directory, but this is an incorrect interpretation. This is an import from the current directory (which happens to be the parent directory of this particular file, but that is irrelevant). Basically any Python file in your project regardless of location can do import config and it will work, as long as the current directory is the directory where config.py is located.

Ah, ok. Thx for the explanation. I think I see now.


lamp_area - shadow issues (build 14 rc0)

See attached video.
shadow_issues.mp4.zip

Will move this issue to the repository at https://github.com/armory3d/armory_examples, marking this one as deprecated and intended for b27 usage.


Feature Request:  keep_source_var to be applicable across appropriate family of functions

Feature Idea
https://github.com/pharmaverse/admiral/discussions/2010#discussioncomment-6659822
https://pharmaverse.slack.com/archives/C028SJ83KM1/p1692633891115359
In our slack post, we have a proposal to make the argument keep_source_var consistent across all selection functions which add new observations.
Relevant Input
No response
Relevant Output
No response
Reproducible Example/Pseudo Code
No response

Didn't we implement it in #1697?

Think we missed some, I have a draft for derive_param_tte() for example

It was decided to not implement this feature.  See here for discussion - https://github.com/pharmaverse/admiral/pull/2215#discussion_r1388401569


I have one more question about stride.

I have one more question about stride.
When I read the reference paper, "Image Super-Resolution Using Deep Convolutional Networks" on 6 page,
it mentioned about 24,800 sub-images, which are extracted from original images with a stride of 14 and 33.
In you program, what do you use a stride? When I see your code in "SRnet.m", you used like
conv2_1Block = dagnn.Conv('size',[9 9 1 64],'hasBias',true,'stride',[1,1],'pad',[0,0,0,0]);.
I think you use 'stride',[1,1].
I would like to use a stride of 14. Did you use the stride of 14?
I am so sorry every time, I am bother you.
Thank you very much for your attention.

It is unreasonable to use a too large stride step like 14.
It means every time you sample a pixel then you need to skip 14 pixel to sample the next pixel.
If the size of input is 2828 and the stride is 14, the size of output is 22.
In super-resolution problem, we do not want to lose the information from the original image so we use the stride 1 to do dense sampling.
You can find more detail information about CNN in http://cs231n.github.io/convolutional-networks/
If you are not clear with the code in the convolution layer, you may refer to http://www.vlfeat.org/matconvnet/mfiles/vl_nnconv/.

Or do you mean padding 14??
You can do it by change the pad setting  like [7,7,7,7].
It will pad 14 pixels in width and height.

I got it! Thank you for your help!
I confused about stride. stride will make more approximation.


problem when using sql bulk copy

Dear sir,
Hope you are well,
i am deleting multiple records then inserting using  sql bulk copy, all change events raised are delete.
problem solved when i set SqlBulkCopyOptions.FireTriggers
Regards

Thanks. I will put your solution in home page (check bottom page)
Thanks.


Category theory cleanup

Some more examples/description on the category theory.
I do intend to move some stuff around if possible. Particularly, a discussion of these might help:

initial and terminal objects
products
duality
adjunctions

I have a branch with some of those that I can try to merge in later.
I'm not sure if my mention of monoidal categories would close #9
Also, I think #14 can be closed?

Congratulations @hemangandhi on your first PR to PureRust / Functional Programming Jargon in Rust! Hope to see more contributions like this! this is really cool


Define Cython language_level explicitly.

Description
I saw some warnings while building cudf with Cython 3.0b1 that could be fixed by specifying the language level as the default value (3str). Since we only care about one language level (3str) in RAPIDS projects, we can specify that directive.
Checklist

[x] I am familiar with the Contributing Guidelines.
[ ] New or existing tests cover these changes.
[x] The documentation is up to date with these changes.
[x] The cmake-format.json is up to date with these changes.
[ ] I have added new files under rapids-cmake/

[ ] I have added include guards (include_guard(GLOBAL))
[ ] I have added the associated docs/ rst file and update the api.rst

/merge


Show the kind of complement was done with JHUData.subset_complement()

Summary of this new feature
JHUData.subset_complement() was created to complement recovered data, but it is difficult for users to know what kind of complement was done for the dataset. It is necessary to show the kind of complement with stdout or the title of figures.
(Optional) Solution

Stdout the kind of complement.
Show the kind of complement in the title of Scenario.records() and Scenario.records_diff().

Then, developers may need to monitor the list of countries where complement performed as mentioned in #380.

Dear @Inglezos ,
I'm writing a pull request for the second solution "Show the kind of complement in the title of Scenario.records() and Scenario.records_diff()".
Titles of Scenario.reocrds() figures could be

"Cases over time (with fully complemented recovered data)"
"Cases over time (with partially complemented recovered data)"
"Cases over time (with recovered data complemented to show monotonic increasing)"

What do you think?

The second solution is the ideal one, showing the complement kind directly into the plot name. Yes I think these figure names are appropriate for informing the user.

Thank you, the third title will be revised.
OK, we will update figure names, not standard outputs.
P.S.
If you have time, please check the statuses of the other issues to select issues which will be fixed in version 2.13.0.

Dear @Inglezos ,
With #390, titles of Scenario.records() with complemented data were updated.
With Netherlands records,

FYI:
With #391, methods for complement in JHUData class were moved to new class covsirphy.cleaning.jhu_complement.JHUDataComplementHandler class for refactoring.
When we update complement tools, we will

create/update methods of this class,
update ._protocol method and
update class variable .STATUS_NAME_DICT, if necessary.

Great thank you, this is enough I think, now we can close this issue.

I found that

confirmed value decreased from 2020-06-18 to 2020-06-19 and monotonic increasing was broken
fatal value decreased from 2020-06-23 to 2020-06-24 and monotonic increasing was broken
https://gist.github.com/lisphilar/258ddea8f0f2604fdc58b9145fb528ab

I do not know why the values were changed in the raw data, but the behavior of complement method is correct.

However, \n should be added to the title.

Oh I thought that we would inform the user only about the recovered complement/monotonic handling, not for the other two.

Sorry... I omitted monotonic handling for confirmed/fatal, refering the titles.
Monotonic handling for confirmed/fatal was included before this change.

Becuase complement of confirmed and fatal reocrds impacts on the result of trend analysis and so on, users should be notified when confirmed/fatal values were complemented.


BUG: create landing pages

At the very least we need a Wiki landing page and a pointer to it from the repository's README file so it is easily found.


Closing token account bug

Bug description

After closing token account, the rpc will return error with message address does not exist. Currently, getting error does not delete the token account, so the deletion does not work proplerly.
To solve this problem, we should delete when receiving address does not exist error.

It is due to import snapshot and not the soljuno bug.


TextField types are not allowing spreading props over pre-set variant value

TextField types are not allowing spreading props over pre-set variant values like so:
const CustomTextField: FunctionComponent<TextFieldProps> = props => {
  return (
    <TextField
      variant="outlined"
      {...props}
    />
  );
};

I think this has to do with how the type union for TextFieldProps includes types that specify variant as a required prop, despite the fact that it's an optional prop.

[x] This is not a v0.x issue.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior 🤔
No type error.
Current Behavior 😯
I get the error:
Type '{ variant: "standard" | "outlined"; InputProps?: Partial<InputProps> | undefined; inputProps?: InputBaseComponentProps | undefined; autoComplete?: string | undefined; autoFocus?: boolean | undefined; ... 274 more ...; classes?: Partial<...> | undefined; } | { ...; } | { ...; }' is not assignable to type '(IntrinsicAttributes & StandardTextFieldProps) | (IntrinsicAttributes & FilledTextFieldProps) | (IntrinsicAttributes & OutlinedTextFieldProps)'.
  Type '{ variant: "standard" | "outlined"; InputProps?: Partial<InputProps> | undefined; inputProps?: InputBaseComponentProps | undefined; autoComplete?: string | undefined; autoFocus?: boolean | undefined; ... 274 more ...; classes?: Partial<...> | undefined; }' is not assignable to type '(IntrinsicAttributes & StandardTextFieldProps) | (IntrinsicAttributes & FilledTextFieldProps) | (IntrinsicAttributes & OutlinedTextFieldProps)'.
    Type '{ variant: "standard" | "outlined"; InputProps?: Partial<InputProps> | undefined; inputProps?: InputBaseComponentProps | undefined; autoComplete?: string | undefined; autoFocus?: boolean | undefined; ... 274 more ...; classes?: Partial<...> | undefined; }' is not assignable to type 'OutlinedTextFieldProps'.
      Types of property 'variant' are incompatible.
        Type '"standard" | "outlined"' is not assignable to type '"outlined"'.
          Type '"standard"' is not assignable to type '"outlined"'.ts(2322)

Steps to Reproduce 🕹
Link: https://codesandbox.io/s/suspicious-fog-g416x

Make a function component which takes TextFieldProps and returns a customized TextField with variant set to some valid value, spreading the props in after the variant is set

Context 🔦
I'm just trying to make a customized TextField that has some props already defined to avoid code duplication (this example is fairly trivial, but I'd like to include classes overrides as well).
Your Environment 🌎

Tech
Version

Material-UI
v3.9.3

React
16.8.6

Browser
n/a

TypeScript
3.5.2

tsconfig.json:
{
  "compilerOptions": {
    "downlevelIteration": true,
    "esModuleInterop": true,
    "allowSyntheticDefaultImports": true,
    "jsx": "preserve",
    "lib": [
      "es2018",
      "dom"
    ],
    "module": "esnext",
    "moduleResolution": "node",
    "target": "es5",
    "outDir": "./dist",
    "rootDirs": ["./src", "./.storybook"],
    "sourceMap": false,
    "strict": true,
    "baseUrl": "./",
  },
  "exclude": [
    "server",
    "node_modules"
  ]
}

Duplicate of #15697

Looks like I entered the wrong search terms, apologies.

No worries. That's what we are for. Even helps visibility.


Feature Request: Volume Bar

a volume bar added to the menu pop up for quicker volume adjustments for users that use tvs that do not support hardware volume control. (such as older tvs before 2009, HDMI1.0)

Neeeeeed <3
I use a lot Spotify Connect and need to change volume quickly only for Spotify, not my computer.
Thanks

Yes, please! Mostly playing Spotify on my Sonos speakers, so the macOS volume controls are useless.
I'm currently using https://github.com/dbilgili/Ses, which controls Sonos' volume directly, but that's missing album art and the slider to navigate within a track.
I'd love to finally see a menubar player that offers album art, volume control and in-track navigation...


Windows install error (Detected that the setup program has been called inside a Cygwin or MSYS2 environment)

Trying to install on windows, getting this error:
Detected that the setup program has been called inside a Cygwin or MSYS2 environment. In particular, C:\msys64\usr\bin\cygpath.exe was available in the PATH. Since the setup program uses its own MSYS2 environment, and since MSYS2 does not support one MSYS2 or Cygwin environment calling another MSYS2 environment, it is highly probable the installation will fail. Please rerun the setup program directly from a Command Prompt, PowerShell, the File Explorer or directly from the browser Downloads if you downloaded it. Do not use Git Bash (part of Git for Windows) or anything else that contains an MSYS2 environment.

dkml-package.bc: [ERROR] FATAL [5f927a8b].
                         A transient failure occurred.

                         Root cause: The command had exit code 20:
                                       C:\Users\reedj\AppData\Local\Temp\7z69CCF908\bin\dkml-install-user-runner.exe
                                         install-user-ocamlcompiler-network --verbosity=info
                                         --color=always --prefix
                                         C:\Users\reedj\AppData\Local\Programs\DkMLNative
                                         --staging-files
                                         C:\Users\reedj\AppData\Local\Temp\7z69CCF908\sg

                         >>> A t

What do you see when you type:
echo $env:PATH
in PowerShell?
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Reed Jones @.>
Sent: Monday, December 25, 2023 8:58:36 PM
To: diskuv/dkml-installer-ocaml @.>
Cc: Subscribed @.***>
Subject: [diskuv/dkml-installer-ocaml] Windows install error (Detected that the setup program has been called inside a Cygwin or MSYS2 environment) (Issue #93)
Trying to install on windows, getting this error:
Detected that the setup program has been called inside a Cygwin or MSYS2 environment. In particular, C:\msys64\usr\bin\cygpath.exe was available in the PATH. Since the setup program uses its own MSYS2 environment, and since MSYS2 does not support one MSYS2 or Cygwin environment calling another MSYS2 environment, it is highly probable the installation will fail. Please rerun the setup program directly from a Command Prompt, PowerShell, the File Explorer or directly from the browser Downloads if you downloaded it. Do not use Git Bash (part of Git for Windows) or anything else that contains an MSYS2 environment.
�[0;1m�[0m
dkml-package.bc: [ERROR] �[4m�[1m�[31mFATAL [5f927a8b].�[0;4;1m�[0;4m�[0m
�[1m�[31mA transient failure occurred.
                     Root cause: The command had exit code 20:
                                   C:\Users\reedj\AppData\Local\Temp\7z69CCF908\bin\dkml-install-user-runner.exe
                                     install-user-ocamlcompiler-network --verbosity=info
                                     --color=always --prefix
                                     C:\Users\reedj\AppData\Local\Programs\DkMLNative
                                     --staging-files
                                     C:\Users\reedj\AppData\Local\Temp\7z69CCF908\sg

                     >>> A t

—
Reply to this email directly, view it on GitHubhttps://github.com/diskuv/dkml-installer-ocaml/issues/93, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AREG4PJDD7YQAIMGPGMUEGTYLJKPZAVCNFSM6AAAAABBC4NHWWVHI2DSMVQWIX3LMV43ASLTON2WKOZSGA2TMMBSHEZDIOI.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

I have a bunch of stuff on path, why do you ask something specific you suspect I'm missing or?
One thing relevant I found was I had 'AppData\Local\Programs\DkMLNative\usr\bin'
which I had added this to path, because whenever I run anything from 'AppData\Local\Programs\DkMLNative\bin' (which I had added onto path before as well)  I got some error about missing: 'dklmvars-v2. or something' I never got anything to work

The error message you got is that you have a conflicting program in your PATH. Kinda hard to help you identify it if you don't tell me your PATH though.
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Reed Jones @.>
Sent: Monday, December 25, 2023 9:17:20 PM
To: diskuv/dkml-installer-ocaml @.>
Cc: Jonah Beckford @.>; Comment @.>
Subject: Re: [diskuv/dkml-installer-ocaml] Windows install error (Detected that the setup program has been called inside a Cygwin or MSYS2 environment) (Issue #93)
I have a bunch of stuff on path, why do you ask something specific you suspect I'm missing or?
One thing relevant I found was I had 'AppData\Local\Programs\DkMLNative\usr\bin'
which I had added this to path, because whenever I run anything from 'AppData\Local\Programs\DkMLNative\bin' (which I had added onto path before as well) I got some error about missing: 'dklmvars-v2. or something' I never got anything to work
—
Reply to this email directly, view it on GitHubhttps://github.com/diskuv/dkml-installer-ocaml/issues/93#issuecomment-1869262167, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AREG4PP5KMOWZA67NVA7QODYLJMVXAVCNFSM6AAAAABBC4NHWWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNRZGI3DEMJWG4.
You are receiving this because you commented.Message ID: @.***>

here's the full output of path:
C:\Program Files\NVIDIA GPU Computing Toolkit\CUDA\v10.0\bin;C:\Program Files\NVIDIA GPU Computing Toolkit\CUDA\v10.0\libnvvp;C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\wbin;C:\Program Files\ImageMagick-7.1.1-Q16-HDRI;C:\PROGRA~1\MICROS~2\JDK-11~1.7-H\bin;C:\PROGRA~1\MICROS~1\Bin\;C:\PROGRA~2\COMMON~1\Intel\SHARED~1\redist\intel64\compiler;C:\msys64\usr\bin;C:\windows\system32;C:\windows\System32\Wbem;C:\windows\System32\WindowsPowerShell\v1.0\;C:\windows\System32\OpenSSH\;C:\PROGRA~3\COMPOS~1\bin;C:\PROGRA~1\Docker\Docker\RESOUR~1\bin;C:\PROGRA~1\MI35E1~1\bin\;C:\PROGRA~2\MICROS~3\110\DTS\Binn\;C:\tools\msys64;C:\src\CHROME~1;C:\PROGRA~2\WI3CF2~1\10\WINDOW~1\;C:\Program Files\Docker\Docker\resources\bin;C:\Program Files\GTK3-Runtime Win64\bin;C:\Program Files\Amazon\AWSCLIV2\;C:\Windows;C:\Users\reedj\AppData\Local\Yarn\bin;C:\Program Files (x86)\Tailscale IPN;C:\Program Files\Microsoft SQL Server\150\Tools\Binn\;C:\Program Files\dotnet\;C:\Users\reedj\AppData\bin\nim-2.0.0\bin;C:\Program Files\Vagrant\bin;C:\Users\reedj\.nimble\bin;C:\Users\reedj\AppData\Local\pnpm;C:\Users\reedj\AppData\Local\Programs\Python\Python310\Scripts;C:\Users\reedj\AppData\Local\Programs\Python\Python310\;C:\Users\reedj\AppData\Local\Programs\Python\Launcher\;C:\Users\reedj\AppData\Local\MICROS~1\WINDOW~1;C:\Users\reedj\AppData\Roaming\npm;C:\PROGRA~3\CHOCOL~1\lib\ghc\tools\GHC-81~1.2\bin;C:\tools\msys64;C:\src\CHROME~1;C:\Users\reedj\LOCAL~1\bin;C:\Users\reedj\AppData\Roaming\nvm;C:\MinGW\lib\gcc\mingw32\92F99B~1.0\include\C__~1;C:\PROGRA~1\OpenVPN\bin;C:\PROGRA~1\nodejs;C:\Users\reedj\AppData\Local\Programs\MICROS~1\bin;C:\PROGRA~1\TESSER~1;C:\PROGRA~1\CMake\bin\;C:\Users\reedj\SURREA~1;C:\PROGRA~1\PostgreSQL\12\bin;C:\Users\reedj\DOTNET~1\tools;C:\Users\reedj\AppData\Local\.bvm;C:\Program Files\Git\cmd;C:\Users\reedj\.cargo\bin;C:\Users\reedj\AppData\Local\Programs\Microsoft VS Code\bin;C:\Users\reedj\AppData\bin;C:\Users\reedj\AppData\bin\poppler\Library\bin;C:\Users\reedj\scoop\shims;C:\Users\reedj\AppData\Local\Programs\Azure Dev CLI\;C:\Program Files\ImageMagick-7.1.1-Q16-HDRI;C:\Users\reedj\.dotnet\tools;C:\Users\reedj\AppData\bin\nim-2.0.0\bin;C:\Users\reedj\AppData\Local\Programs\DkMLNative\usr\bin;

You have both the following in your PATH:

C:\msys64\usr\bin
C:\tools\msys64

Having the first one in the PATH will definitely conflict with DkML (and other programs that use MSYS2). I'm not sure about the second but I think that will conflict as well.
The MSYS2 installer does not add itself to the PATH, so I'm guessing you did it manually or some other program did it automatically. Regardless, remove both of them from your PATH.
Root cause: Only one version of MSYS2 is allowed to be accessible to programs on your machine, so placing MSYS2 in the PATH is almost always "wrong". You can always launch MSYS2 using the MSYS2 recommended https://www.msys2.org/docs/terminals/ if you need your own private copy.

That worked, thanks for the help.


fixed zulipchat domain

The ZulipChat application is a program that can be run on local servers and under your own domain. Therefore, mandating the use of the .zulipchat.com domain name hinders connecting to local servers. Making this field editable would be more beneficial.

#119


Using Crystal Workbook fails to create SHAINet model

I was trying to play with SHAINet using crystal play and creating a workbook similar to Jupyter:
require "shainet"

diabetes = SHAInet::Network.new

I'm getting a JSON parsing error and wanted to know if this is something you have encountered this before or if I'm doing something wrong.
Exception

Error in line 2: instantiating 'Crystal::Playground::Agent#i(Int32)'

in /usr/local/Cellar/crystal-lang/0.23.1_3/src/compiler/crystal/tools/playground/agent.cr:25: instantiating 'send(String)'

    send "value" do |json|
    ^~~~

in /usr/local/Cellar/crystal-lang/0.23.1_3/src/compiler/crystal/tools/playground/agent.cr:53: instantiating 'JSON:Module#build()'

    message = JSON.build do |json|
                   ^~~~~

in /usr/local/Cellar/crystal-lang/0.23.1_3/src/json/builder.cr:356: instantiating 'String:Class#build()'

    String.build do |str|
           ^~~~~

in /usr/local/Cellar/crystal-lang/0.23.1_3/src/string.cr:269: instantiating 'String::Builder:Class#build(Int32)'

    String::Builder.build(capacity) do |builder|
                    ^~~~~

in /usr/local/Cellar/crystal-lang/0.23.1_3/src/string.cr:269: instantiating 'String::Builder:Class#build(Int32)'

    String::Builder.build(capacity) do |builder|
                    ^~~~~

in /usr/local/Cellar/crystal-lang/0.23.1_3/src/json/builder.cr:356: instantiating 'String:Class#build()'

    String.build do |str|
           ^~~~~

in /usr/local/Cellar/crystal-lang/0.23.1_3/src/json/builder.cr:357: instantiating 'build(String::Builder, Nil)'

      build(str, indent) do |json|
      ^~~~~

in /usr/local/Cellar/crystal-lang/0.23.1_3/src/json/builder.cr:367: instantiating 'JSON::Builder#document()'

    builder.document do
            ^~~~~~~~

in /usr/local/Cellar/crystal-lang/0.23.1_3/src/json/builder.cr:367: instantiating 'JSON::Builder#document()'

    builder.document do
            ^~~~~~~~

in /usr/local/Cellar/crystal-lang/0.23.1_3/src/json/builder.cr:357: instantiating 'build(String::Builder, Nil)'

      build(str, indent) do |json|
      ^~~~~

in /usr/local/Cellar/crystal-lang/0.23.1_3/src/compiler/crystal/tools/playground/agent.cr:53: instantiating 'JSON:Module#build()'

    message = JSON.build do |json|
                   ^~~~~

in /usr/local/Cellar/crystal-lang/0.23.1_3/src/compiler/crystal/tools/playground/agent.cr:54: instantiating 'JSON::Builder#object()'

      json.object do
           ^~~~~~

in /usr/local/Cellar/crystal-lang/0.23.1_3/src/compiler/crystal/tools/playground/agent.cr:54: instantiating 'JSON::Builder#object()'

      json.object do
           ^~~~~~

in /usr/local/Cellar/crystal-lang/0.23.1_3/src/compiler/crystal/tools/playground/agent.cr:25: instantiating 'send(String)'

    send "value" do |json|
    ^~~~

in /usr/local/Cellar/crystal-lang/0.23.1_3/src/compiler/crystal/tools/playground/agent.cr:27: instantiating 'JSON::Builder#field(String, (File::PReader | HTTP::Server::Response::Output | IO::FileDescriptor | Int32 | OpenSSL::SSL::Socket | String | Nil))'

      json.field "value", safe_to_value(value)
           ^~~~~

in /usr/local/Cellar/crystal-lang/0.23.1_3/src/json/builder.cr:226: no overload matches 'JSON::Builder#scalar' with type (File::PReader | HTTP::Server::Response::Output | IO::FileDescriptor | Int32 | OpenSSL::SSL::Socket | String | Nil)
Overloads are:
 - JSON::Builder#scalar(value : Nil)
 - JSON::Builder#scalar(value : Bool)
 - JSON::Builder#scalar(value : Int | Float)
 - JSON::Builder#scalar(value : String)
 - JSON::Builder#scalar(string = false, &block)
Couldn't find overloads for these types:
 - JSON::Builder#scalar(File::PReader)
 - JSON::Builder#scalar(HTTP::Server::Response::Output)
 - JSON::Builder#scalar(IO::FileDescriptor)
 - JSON::Builder#scalar(OpenSSL::SSL::Socket::Client)
 - JSON::Builder#scalar(OpenSSL::SSL::Socket::Server)

    scalar(value)
    ^~~~~~

If I run the same code using crystal run, I do not encounter the error.

It looks like I need to specify the type when using playground otherwise it doesn't know which type is being returned.
require "shainet"
diabetes : SHAInet::Network = SHAInet::Network.new

I think this is a playground issue.  Closing this.

@drujensen Cool :) let me know what do you think about the network's performance


Update to getbin.io

Migrate the deployed domain to getbin.io to make it clear this is a general tool.

Done!


CLI and API separation

Basically the same thing, split in two files and wrapping most of it in a module.exports function. This way we can batch process sprites creation from node, without having to spawn audiosprite itself, also making easier to write things like a gulp plugin.
I ain't proud of wrapping everything in a function, but I wanted to change it slightly as possible.

Thanks! Made a small fix in eab9916cadf6d05ed923307446de745289dd05e3
Feel free to clean this up in future if you want. Take the functions that doesn't share common state out of the closure etc. Just make as many separate commits as you can to keep the review easier.

Oh, great. I'll be using it in production, so I'll improve it them if I find any issues. Thanks.


Add ability to pass children to Icon

I use bloomer and also the react fontawesome library.
I was using an older version of bloomer that accepted children for the Icon component.  I noticed that it was changed in a previous commit and was hoping it could be added back.  Let me know what you think.
Thanks for this fantastic library.  :)
Here is an example of how I use it.
<Icon isAlign="left">
  <FontAwesomeIcon icon={faEnvelope} />
</Icon>

Coverage remained the same at 100.0% when pulling f9ec190e5abcf6289b2058d89b1996bd3858296e on dusty:master into b9af8271840c9a52ecf493f3b1f919d8cd3e5fc1 on AlgusDark:master.


ARM64 support + Version bumps

Addresses feature request #80 and fixes issue #90 in the process. First time contribution so I could use help figuring out how to test this.
Changes

Split the amazon-linux-2-ami AMI into amazon-linux-2-amd64-ami and amazon-linux-2-arm64-ami
Build ARM64 AMIs for Amazon Linux 2 for both docker + without docker
Set server instance_type to a variable server_instance_type to allow for ARM64 support
Nomad 0.9.1 -> 1.1.1
Consul 1.5.1 -> 1.9.6
Fix clean_ami_name deprecation with clean_resource_name

Thank you for your submission! We require that all contributors sign our Contributor License Agreement ("CLA") before we can accept the contribution. Read and sign the agreement
Learn more about why HashiCorp requires a CLA and what the CLA includes
Have you signed the CLA already but the status is still pending? Recheck it.

Ubuntu AMIs have failed building, trying to investigate. https://gist.github.com/nepeat/2c28bcf31c6bbda65e2fed243fe1aff2
Unsupported: The requested configuration is currently not supported. Please check the documentation for supported configurations

Odd issues that resolved themselves with no new changes to the files. Logs and notes for those single failed tests below.
TestNomadConsulClusterSeparateWithUbuntu16Ami
Passes when rerun as a single test with major caveats. Test command had to be run twice to make this build and test successfully.
Log Attempt 1: https://gist.githubusercontent.com/nepeat/fb662121cd9a830a031f55641075b8bc/raw/3795cd1122b095293ce445ac29e8384f96878726/attempt_1.txt
Log Attempt 2: https://gist.githubusercontent.com/nepeat/fb662121cd9a830a031f55641075b8bc/raw/06af23cb06a882be01369b6de898f5466398b108/attempt_2.txt
TestNomadConsulClusterSeparateWith18UbuntuAmi
Passes when rerun as a single test with same major caveat as TestNomadConsulClusterSeparateWithUbuntu16Ami. I had to run the command 2 times to make this work.
Log:  https://gist.github.com/nepeat/29f25412e290ad0139698b0faa7cce08
TestNomadConsulClusterColocatedWithUbuntu16Ami
Passes when rerun as a single test, no changes.
Log: https://gist.github.com/nepeat/0240912afaec916e4fc456677bd1df3f

Some tests failed on the first run, but I think it was due to some transient issues. They passed on re-run, so I'm going to merge!

https://github.com/hashicorp/terraform-aws-nomad/releases/tag/v0.9.1


Rank selection in KCCA

Currently no great way of telling the best number of components to use from KCCA. Currently the user specifies a number. Consider otherwise calculating all the eigenvalues and thresholding a % of the variance, using the implemented Zhu-Godise elbo selector, or return the eigenvalues so that the user can examine a scree plot. GCCA does the first two

i like this idea. will look into too


Out of date docs or incorrect types?

🐛 Bug Report
When using typescript there are two issues. Once when using the syntax from the prefetching docs: https://www.smooth-code.com/open-source/loadable-components/docs/prefetching/ and the other when trying to preload.
To Reproduce
// Argument of type '() => string' is not assignable to parameter of type '(props: {}) => Promise<DefaultComponent<{}>>'.
// Type 'string' is not assignable to type 'Promise<DefaultComponent<{}>>'.
export const User = () => loadable(() => './User')

and
export const User = () => loadable(() => import('./User'))

// Property 'preload' does not exist on type '() => ComponentType<{ fallback?: Element | undefined; }>'.
User.preload()

Expected behavior
Typescript should not error and docs should be updated
Details

## System:
 - OS: Linux 4.4 Ubuntu 16.04.2 LTS (Xenial Xerus)
 - CPU: (16) x64 AMD Ryzen 7 1700X Eight-Core Processor
 - Memory: 8.81 GB / 15.95 GB
 - Shell: 4.3.46 - /bin/bash
## Binaries:
 - Node: 10.15.0 - ~/.nvm/versions/node/v10.15.0/bin/node
 - Yarn: 1.9.4 - ~/.yarn/bin/yarn
 - npm: 6.4.1 - ~/.nvm/versions/node/v10.15.0/bin/npm

Hello @bitttttten, thanks for your report.
Help is needed here, could someone please update TypeScript types ? Thanks!

@theKashey I guess?

:working:

Ah I didn't realise the preload method was reasonably new too. I was actually on 5.5.0 at the time.

That's quite strange, but there are type definitions?
Nor here, nor in DefinitelyTypes. All I found is https://github.com/DefinitelyTyped/DefinitelyTyped/tree/f37d7b008ce2c4ecbbbc4bee009fb3f03d639268/types/loadable__component, which is for older versions.

There are type definitions but they have not been updated since for the release that was 2 weeks ago.

@bitttttten - so where is that there

Looks like this can be closed now: https://github.com/DefinitelyTyped/DefinitelyTyped/commit/062bc01af259fdf566bb5922482cb9d4af7426e8#diff-f3a10f9ffcf2e48ce15ac6338d265145


Link to frontend from admin product list or product detail page

Hi,
is possible to add a link inside admin product list and  admin product detail able to jump to front end product page ?
regards.

Hi @ghena
Can you please elaborate ??

Hi,
is possible to add a link inside admin product list and admin product detail able to jump to front end product page ? regards.

@ghena
Thanks for your suggestion. we will implement it and will reflect it in the upcoming release.

@jitendra-webkul
Now, this feature is implemented. so I am closing this.
And I have attached a screenshot Here. for future reference.


Revert "bump github.com/DataDog/go-sqllexer to v0.0.18"

What does this PR do?
Reverts DataDog/datadog-agent#32315.
Motivation
This PR broke a KMT test (pkg/network/protocols/postgres.TestExtractTableFunction/single_table_name_with_mixed_caps).
#incident-33427

Force-merging the PR to fix main


hddtemp displays same value as during startup

Hi,
there seems to be a bug with hddtemp fetching. I am not sure if it is a retry issue or something else. But seems the temperatures get stuck on the position they were when sensor-exporter starts.
I have hddtemp running on my host machine and sensor-exporter in docker.
[root@loovsys ~]# docker logs sensors
2018/11/19 15:45:24 error readding hddtemps: error connecting to hddtemp address '<IP_ADDRESS>:7634': dial tcp <IP_ADDRESS>:7634: getsockopt: connection refused
[root@loovsys ~]# docker exec -it 31c4388720e8 bash
root@31c4388720e8:/go# curl <IP_ADDRESS>:7634

curl: (56) Recv failure: Connection reset by peer
|/dev/sda|Samsung SSD 840 EVO 500GB|45|C||/dev/sdb|WDC WD40PURX-64GVNY0|42|C||/dev/sdc|WDC WD40EZRZ-00GXCB0|41|C|root@31c4388720e8:/go#
root@31c4388720e8:/go# ps -ef
UID        PID  PPID  C STIME TTY          TIME CMD
root         1     0  0 Nov19 ?        00:00:00 /bin/sh -c /go/bin/sensor-exporter -hddtemp-address=<IP_ADDRESS>:7634
root         8     1  0 Nov19 ?        00:00:17 /go/bin/sensor-exporter -hddtemp-address=<IP_ADDRESS>:7634
root        23     0  0 11:02 pts/0    00:00:00 bash
root        31    23  0 11:07 pts/0    00:00:00 ps -ef

It has one line about failed to fetch. From grafana the only change in temperature happend when I restarted sensors-exporter.

From screenshot you can see that it should show different temperatures.
For example hda temperature is 45C, but sesnors-exporter reports it as 38 (as it was when sensor-exporter started up)

Hi, so I had time to debug this. The issue is that the connection is not done again...so the same connection buffer is used every time.
By re-initializing h.Init() and closing connection every time readTempsFromConn is called. Also needed to wipe the h.buf to insert the returned values from new connection.
`diff --git a/sensor-exporter/main.go b/sensor-exporter/main.go
index 1641ec1..986fa39 100644
--- a/sensor-exporter/main.go
+++ b/sensor-exporter/main.go
@@ -161,16 +161,17 @@ func (h *HddCollector) Init() error {
}
func (h *HddCollector) readTempsFromConn() (string, error) {

  if h.conn == nil {

  //if h.conn == nil {
          if err := h.Init(); err != nil {
                  return "", err
          }

  }

  //}

   h.buf.Reset()
  _, err := io.Copy(&h.buf, h.conn)
  if err != nil {
          return "", fmt.Errorf("Error reading from hddtemp socket: %v", err)
  }

   h.conn.Close()
  return h.buf.String(), nil

}
`
The function in main.go now looks like this and it works properly:
func (h *HddCollector) readTempsFromConn() (string, error) {         //if h.conn == nil {                 if err := h.Init(); err != nil {                         return "", err                 }         //}         h.buf.Reset()         _, err := io.Copy(&h.buf, h.conn)         if err != nil {                 return "", fmt.Errorf("Error reading from hddtemp socket: %v", err)         }         h.conn.Close()         return h.buf.String(), nil }

For me this issue is now resolved :-)

@mardicas Please create a pull request with this fix!

Be gentle, it is my first pull request on github.

Now the HDD graph looks correct , instead of a straight line after startup it actualy changes.


Rename project file name to be consistent with the scenario

We should rename:

WarmStorage to AdhocExploration
ColdStorage to LongTermStorage
ScenarioSimulator to Simulator

In addition,

WarmStorage.EventProcessor.ConsoleHost.Template.config to settings.template.config (with the real file being settings.config)

ColdStorage.EventProcessor.ConsoleHost.Template.config to settings.template.config (with the real file being settings.config).

ScenarioSimulator.ConsoleHost.Template.config to settings.template.config (with the real file being settings.config).

I definitely agree with WarmStorage  and ColdStroage, but I am not sure about ScenarioSimulator. I thought that including "Scenario" in the name helps to clarify the intent. Thoughts?

I discussed more with Christopher and we feel that it's OK to leave the project name be different from the parent scenario name. The reason is the each scenario may consist multiple projects in the future, so each project will have different names and they may not necessarily be the same as their parent scenario name. Actually, making them the same may cause confusion.

:+1:


Test

Thank you for contributing to Harbor!
Comprehensive Summary of your change
Issue being fixed
Fixes #(issue)
Please indicate you've done the following:

[ ] Well Written Title and Summary of the PR
[ ] Label the PR as needed. "release-note/ignore-for-release, release-note/new-feature, release-note/update, release-note/enhancement, release-note/community, release-note/breaking-change, release-note/docs, release-note/infra, release-note/deprecation"
[ ] Accepted the DCO. Commits without the DCO will delay acceptance.
[ ] Made sure tests are passing and test coverage is added if needed.
[ ] Considered the docs impact and opened a new docs issue or PR with docs changes if needed in website repository.

Codecov Report

Merging #3 (3141101) into main (eac647c) will decrease coverage by 0.17%.
The diff coverage is 48.96%.

@@            Coverage Diff             @@
##             main       #3      +/-   ##
==========================================
- Coverage   67.32%   67.14%   -0.18%
==========================================
  Files         968      979      +11
  Lines       81124    81777     +653
  Branches     2550     2604      +54
==========================================
+ Hits        54619    54913     +294
- Misses      22823    23109     +286
- Partials     3682     3755      +73

Flag
Coverage Δ

unittests
67.14% <48.96%> (-0.18%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

src/chartserver/handler_repo.go
83.33% <0.00%> (ø)

src/chartserver/redis_sentinel.go
0.88% <0.00%> (-0.02%)
:arrow_down:

src/common/rbac/project/rbac_util.go
63.63% <ø> (ø)

src/controller/gc/callback.go
6.25% <0.00%> (-0.42%)
:arrow_down:

src/controller/scanner/options.go
33.33% <0.00%> (ø)

src/controller/user/controller.go
21.05% <0.00%> (ø)

src/core/api/base.go
30.13% <0.00%> (-1.75%)
:arrow_down:

src/core/controllers/authproxy_redirect.go
0.00% <ø> (ø)

src/core/controllers/base.go
22.05% <0.00%> (-1.02%)
:arrow_down:

src/core/controllers/oidc.go
0.00% <0.00%> (ø)

... and 133 more


Excel Sheet Formats

I have installed all the dependencies and with custom dataset and now trying to run camera.py file .
However it is throwing "Please Open Excels Files" .
How to proceed ahead in this case ?

@RykerLe

Hi @bhargavravat, thank you for your interest in our project. There is something wrong in your Application GUI, a File button designated to located in the top left corner was missing.
Here is the right GUI:

I do not know the reason why it is missing, we are under intensive discussion on this issue. You may try  running this application on another computer and tell me if it still happens.

Hi @bhargavravat, thank you for your interest in our project. There is something wrong in your Application GUI, a File button designated to located in the top left corner was missing.
Here is the right GUI:

I do not know the reason why it is missing, we are under intensive discussion on this issue. You may try running this application on another computer and tell me if it still happens.

Hi @RykerLe : I tried running in different system as well , however the result is one and the same.

@bhargavravat Can you describe more about your system (OS version, Python version)?

@AntiAegis : I am presently working on Ubuntu 16.04 LTS (64 bit ) system with Anaconda Environment which has Python 3.6.4 installed in it. Do I need to change python version ?

My system is Ubuntu 18.04 LTS (64bit), Python 3.6.5, and Pip.
Can you try to create a virtual environment using Pip and install packages with Pip?

My system is Ubuntu 18.04 LTS (64bit), Python 3.6.5, and Pip.
Can you try to create a virtual environment using Pip and install packages with Pip?

@AntiAegis  : I have installed it with the same configuration mentioned by your side , still File tab is missing in it.


Add Healthchecks.io start ping before sync

Adds a /start ping to Healthchecks.io to help debug stuck jobs and measure execution time.
I've tested this both with and without  $CHECK_URL set and it worked fine. Hopefully it can be useful for someone else.
Thanks @bcardiff for the great image!

Thanks @felixbouleau, in order to keep users unlocked from healthchecks.io, WDYT about

using a START_URL argument
setting START_URL to CHECK_URL/start by default in the entrypoint (if START_URL is not set and CHECK_URL it is set)

something like:
  if [ -z ${START_URL+x} ]
  then
    if [ -z ${CHECK_URL+x} ]
    then;
    else
      START_URL="${CHECK_URL}/start"
    fi
  fi

why -z ${var+x}?

use the same code for CHECK_URL and START_URL. (I'm not sure if checking with -n is better than -z here)

  if [ -z "$START_URL" ]
  then
    wget $START_URL -O /dev/null
  fi

Add a description in the README

?
The behavior should be the same and the user can opt-out the START_URL or define it's own.
FYI #7 attempt to implement FAIL_URL, I need to do another round and merge that functionality. But is related and you might be interested.

@bcardiff thanks for the quick feedback!
Since the goal was to be agnostic towards ping providers I've taken a slightly different approach than #7 for defaults. Here's a shortcut to the logic.
This way, if you use something else than HC you don't need to override START_URL with something else.
I'm happy to switch over to the #7 approach if you prefer it - just let me know. It's a super easy change. :)

It's fine as is. Thanks!
The duplicated wget bugs me a little. But is fine.
Bash is always cryptic to me.

Was there any reason to drop the proposed +x? Just curious.
The other checks use [ -z "$VAR" ] while the PR is using [ -z ${VAR} ] (as a consequence of dropping +x.
"${VAR}/start" is the same as "$VAR/start"

I'm not asking for additional changes, but highlighting the difference with the other checks for comparison and eventual unification.

@bcardiff Dropping the -x was a knee jerk fix for a bug I introduced. I solved it in a better way now. Basically I included START_URL as an ENV in the Dockerfile which sets it to an empty string. So the [ -z ${VAR+x} ] evaluated to x and broke the if statement. Instead of dropping the -x I dropped the ENV in the Dockerfile.


yquake2-ref-vk: Add version 1.0.2

This is the vkQuake2 vulkan renderer library ported to Yamagi Quake II.
What package release type is this?

[x] stable
[ ] beta
[ ] dev/nightly/canary

I followed the bucket's manifest standards?

[x] I have read the Contributing Guide.
[x] properly named and capitalized shortcuts?
[x] autoupdate and checkver entry
[x] persist defined with config/data/user/portable/textures/saves folder(s) specific for the app?
[x] a pre_install script to auto-enable portable mode (if needed)?

[x] a pre_install script creates runtime created files specified in persist (ensures symlinks properly get created)

[x] license identifier and url
[x] a short terse description matching existing style.
[x] url to the release along with its sha256 hash
[x] bin entry for main binary

[x] if beta, dev, etc: does bin shim have variant appended to the name? (e.g appname-dev)

[x] passes bin/checkver.ps1 and bin/checkurls.ps1
[x] tested install, checked persist, no errors.
[x] manifest is sorted to spec
[x] manifest follows spec: appname or appname-variant (e.g citra-canary or dolphin-dev)
[x] I tested the package and it runs in portable mode. I have verified the symlinks are correct in the persist folder and working.
[x] I will maintain this manifest and promptly fix it in the event it breaks.

/verify

Shouldn't we modify the yquake2 manifest to automatically install the Vulkan renderer instead? I believe it'll still default to OpenGL out of the box, so it'll keep working on GPUs that don't support Vulkan.

How do we do that in the original manifest?
I thought we'd to create this first and then make a suggest?

How do we do that in the original manifest?

I guess we can add a second download URL, similar to openttd.

I thought we'd to create this first and then make a suggest?

That can be done, but since the download size is just a few megabytes, I wouldn't bother.

How do we do that in the original manifest?

I guess we can add a second download URL, similar to openttd.

I thought we'd to create this first and then make a suggest?

That can be done, but since the download size for the Vulkan renderer ZIP is only 300 KB, I wouldn't bother.

Now I get it, didn't know that possibility - on it.

It looks like this needs to be its own manifest still since I can't find a way to make version, extract_dir, checkver and autoupdate take care of the vulkan library as well.

Alright, I guess we can merge this as-is then.
Feel free to open a PR to add a suggest to yquake2 for yquake2-ref-vk.


Added Clean Code Presentation Slides for Davids Presentation

During the Session of Robter, there was also mine.
This PR adds my slides.
Oh and: Great work! I really like how the community starts evolving around the Camp :+1:

Thanks!


A fatal code error

Inrelational-proxies.py
Line 93 mean_logits = global_logits + summary_logits + relation_logits / 3. Do you need to calculate the average of these three logits? A parenthesis is missing here. This value directly affects the prediction results, and the experimental results of the entire article are based on wrong code.  Or is this the original calculation method?

Many thanks for pointing this out. This bug had crept in during the code clean-up and refactoring for public release. I have now fixed it in the following commit:
https://github.com/abhrac/relational-proxies/commit/8d0b08f99ee2f0db76a2ce54b0b8db1841fe851c


DataModel as mixin

Hi everyone,
I was wondering if there is a way to let flutter_data work with model that are defined in some other packages. I've imagined something like that:
class A extends B with DataModel<A>
where B is model defined in another package. But since DataModel is a class and not a mixin that simply can't work.
I tried to find similar issues here on GitHub but the only referring to this argument was #165 . Was DataModelMixin introduced? Maybe with a different name?

I'm interested too because I'm stuck in the same issue with generated models from GraphQL

Renamed DataModel to DataModelMixin
Re-added DataModel as an abstract class mixing in DataModelMixin
This should not break anything, but make DataModelMixin available - see the Library model in tests for an example
Important: Models using the mixin MUST call init() after being initialized, this includes using copyWith in freezed classes

Try out that commit and let me know if it helps.

Hi @frank06 ,
just tried the new version (1.5.12) with the mixin and I can confirm that is seems to work reliably.
Great work and many thanks!


Puting csv as default argument

This PR amends PR #426. The default option is now to save the csv under the name 'model_comparison.csv'. Few changes were made for better readability.
Doc was modified in #425

Pull Request Test Coverage Report for Build 233492307

-1 of 5 (80.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at ?%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

ivadomed/scripts/compare_models.py
4
5
80.0%

Totals

Change from base Build 233255204:
0%

Covered Lines:

Relevant Lines:
0

💛  - Coveralls

Pull Request Test Coverage Report for Build 233492307
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

4 of 5 (80.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.08%) to 90.882%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

ivadomed/scripts/compare_models.py
4
5
80.0%

Totals

Change from base Build 233255204:
0.08%

Covered Lines:
5472

Relevant Lines:
6021

💛  - Coveralls


Convert to SCSS

It would be nice to have the colours available as scss variables. Scss also compiles to css, so the resulting package would give better support for everyone.

Closed by https://github.com/ipfs-shipyard/ipfs-css/pull/39 - thanks @bluelovers!


Need to do better on image comparison

Investigate https://github.com/Huddle/Resemble.js

Turns our Resemble is not really node compatible and needs canvas for working on the images.
Upgraded mirror-mirror to use https://github.com/yahoo/blink-diff which seems simple and light.


Wrong PICO_FLASH_SIZE_BYTES for Tiny2350

Pimoroni's Tiny2350 has 4MB of flash room, not 8, as they describe in their website:
https://shop.pimoroni.com/products/tiny-2350?variant=42092638699603
I have a Tiny2350 and I can confirm that picotool reports 4MB.
https://github.com/raspberrypi/pico-sdk/blob/efe2103f9b28458a1615ff096054479743ade236/src/boards/include/boards/pimoroni_tiny2350.h#L89
I spoke to them but I did not get any answer and not sure how to proceed or who is the responsible for maintaining the board's header definition.

I already fixed this as part of https://github.com/raspberrypi/pico-sdk/pull/1814 :grinning:
Fixed on develop branch.


Clicking "Morphic World" on M1 Max running Mac OS X Sonoma 14.0 causes system crash

Clicking on "Morphic World" in the latest gtoolkit as of 8/15/2023 causes a crash of the application on Mac OS X Sonoma 14.0 (running on M1 Max).

Can verify this behavior as of Release v1.0.184 as well.

It seems some people can open it once, I cannot open it at all, Gt crashes immediately on Ubuntu 20.04 and Gt v1.0.226 with the error below:
[2023-11-03T23:09:42Z ERROR value_box::error] Error: There was an error
[2023-11-03T23:09:42Z ERROR value_box::error]  - No suitable `wgpu::Adapter` found.

Maybe different error entirely but haven't been able to open Morphic World for longer than this ticket has been open.

Think there should be some CI gate that blocks releases if something like this breaks. I understand that philosophically Gt would want to eventually not depend on Morphic at all, and I rarely go into Morphic world (otherwise I would have been noisier when I noticed it broke), but a hard crash is never fun, even for patient users, new users might be more put off by seeing something like this.

Ugh, I asked around on Discord for Sonoma problems and, hearing of none, upgraded yesterday. This is disappointing as GT is not quite feature complete enough for me to ditch Morphic. Any fix on the horizon?

Apparently they are getting rid of Morphic world completely

Morphic was working for me on M1 Mac Sonoma on GT v1.0.192 and broken on GT v1.0.233. While Morphic may go away at some point, hopefully that is not imminent as there are currently things that are impossible or difficult to do in GT that one can do in Morphic.
From GT Discord:

The first things missing that come to mind are:
easy multi-select/all/none
Drag and drop
For example, in Pharo’s Iceberg tool, if you have a lot of changes that you want to split between several commits, you can deselect all and then cherry-pick. Vs GT’s git tool, you have to start each commit by unselecting each item individually
Another example: in Pharo’s browser, you can select half a dozen methods and drag them into another class or trait in another browser. In GT the same op is fairly tedious
Another: while Coder is moving towards selecting a portion of the hierarchy for which to show methods, in Pharo you simply select from a list.
In GT’s git tool, it seems you can only add an https repo, not an ssh one
Small thing but feels significant: GT has save, and quit, but not “save and quit” like Pharo
Another small thing: Lepiter is amazing, but sometimes I just want a simple old school playground that e.g. is not saved anywhere to disk
Also I feel like GT’s git tool doesn’t do everything I need it to, like creating and checking out branches, but maybe I just don’t know how to use it. Also repairing repos is not fully implemented i.e. in Pharo there are several options re how that gets handled
And finally many of us have lots of time invested in Morphic UIs so in any event it would be great to at least remain loadable if only for migration purposes


:unicorn: Spike: Confirm that iiif print split works

Summary
Now that adventist has been valkyrized, try importing a multi page pdf with iiif print configurations turned on. See if it properly splits the works.
Acceptance Criteria

[ ] Create separate tickets if there are issues found.

Screenshots or Video
Testing Instructions
Notes

this needs to be tested on staging

Issues found:

[ ] Derivative Rodeo splitter expects to receive file_set


Define DB schema for bundles and subscriptions

Relates to #2543
Define the subscriptions and bundles table. Add a foreign key reference to subscriptions in both the profiles and rule type tables.
Summary
Provide a brief overview of the changes and the issue being addressed.
Explain the rationale and any background necessary for understanding the changes.
List dependencies required by this change, if any.
Fixes #(related issue)
Change Type
Mark the type of change your PR introduces:

[ ] Bug fix (resolves an issue without affecting existing features)
[x] Feature (adds new functionality without breaking changes)
[ ] Breaking change (may impact existing functionalities or require documentation updates)
[ ] Documentation (updates or additions to documentation)
[ ] Refactoring or test improvements (no bug fixes or new functionality)

Testing
Outline how the changes were tested, including steps to reproduce and any relevant configurations.
Attach screenshots if helpful.
Review Checklist:

[x] Reviewed my own code for quality and clarity.
[x] Added comments to complex or tricky code sections.
[ ] Updated any affected documentation.
[ ] Included tests that validate the fix or feature.
[ ] Checked that related changes are merged.

coverage: 46.733% (+0.007%) from 46.726%
when pulling<PHONE_NUMBER>8e67c7a189a8f26c1e6141792262d0 on bundle-db-structure
into f15700994c63b1cc8ce72a323e90cb8ab40be3ce on main.

This was indeed the behavior we wanted


ssh error while resolving hostname

Pass port to ssh using -p option to avoid error " Could not resolve hostname" if the Url is not parsed correctly.

Thanks for all your contributions!


[bug] Unable to switch between earpiece & speaker

There is a critical issue where the audio is only coming through the speaker. Additionally, Hardware.instance.canSwitchSpeakerphone is always false, and Hardware.instance.setSpeakerphoneOn(false) has no effect on iOS.
Similarly, On Android, setting room.setAudioOutputDevice('earpiece') also has no effect.
This functionality used to work in version 2.1.3
To Reproduce
This issue is easy to reproduce by running the example app and attempting to switch to the earpiece on Android. On iOS, the button to switch to the speaker is disabled.
Expected behavior
Should be able to switch between earpiece & speaker.
Platform information
iOS - 17.6.1, Android - 13

Flutter version:
Flutter  3.24.3
Plugin version:
livekit_client: ^2.2.6

We are also having same issue with flutter version 3.22.0 and livekit_client: ^2.2.6

Hi @cloudwebrtc Are there any plans to fix this, as it's such a critical issue?

Sorry, I just saw this, trying to fix it.

@rahulraj-idt fixed it, tested and can switch the handset and speaker correctly.

@cloudwebrtc It fixes the issue on android. But this still doesn't work on iOS.

ah, okay, let me check it.

Hello @cloudwebrtc
I'm curious to know whether you had an opportunity to check this iOS speaker issue.

fixed https://github.com/livekit/client-sdk-flutter/pull/617

Yes, you are right, so I guess this is the final solution.
  void setSpeakerphoneOn(bool speakerOn) async {
    if (lkPlatformIs(PlatformType.iOS)) {
      if (!Hardware.instance.speakerOn && Hardware.instance.preferSpeakerOutput) {
        await Hardware.instance.setPreferSpeakerOutput(false);
      }
    }
    await Hardware.instance.setSpeakerphoneOn(speakerOn);
  }

https://github.com/livekit/components-flutter/blob/main/lib/src/context/media_device_context.dart#L285-L290

Hi, your demo still has this problem on iOS.

hi @std-s, please try v2.3.4+hotfix.1, I believe the issue has been fixed


Pornhub - HTTP Error 404: Not Found

Pornhub link not supported :
youtube-dl -t http://fr.pornhub.com/view_video.php?viewkey=ph55ca2f9760862
[generic] view_video: Requesting header
WARNING: Falling back on generic information extractor.
[generic] view_video: Downloading webpage
[generic] view_video: Extracting information
ERROR: unable to download video data: HTTP Error 404: Not Found

This issue has been fixed and will be available in the next version. Thanks for the report.


Document remaining IDExxxx rules

All IDExxxx rules currently implemented in Roslyn master should now be documented with a dedicated doc page.

@gewarren Seeing the following warning:

docs/fundamentals/code-analysis/style-rules/index.md | Warning | View | Line 96: [Warning] Cross reference not found: 'System.HashCode.Combine'.

@gewarren this should be ready now.


PWM frequency

How do I select the PWM frequency, please?

The PWM clock on the PI5 is not documented but I was able to reverse engineer  and I found that the base clock is around 30MHz.
The frequency of the PWM is set by the range.
ex: On my code  you will see that the  total frequency is 50Hz and the default ON pulse rate is 1.5ms which is the center of any R/C servo
Frequency for R/C servo is 50Hz (20ms) then  range = 30303030 (30Mhz) / 50 = 606060 (I know on my code is 615318)
pwm0[PWM_CHANNEL[currentPWM]+PWM_CHAN_RANGE]= 615318; // it should be 606060
Duty for 1.5ms On    duty=30303030  *1.5e-3 =. 45454.5 => 45455 (NOt sure if I need to remove 1)
pwm0[PWM_CHANNEL[currentPWM]+PWM_CHAN_DUTY]= 45455;


[COMMAND] Channel Slow Mode Command

Description
This command should activate/deactivate slow mode in the current channel.
Use-case
There was a raid recently in which no admins were present, and us moderators had to manually ban/delban each account. This command can be useful in situations where admins are asleep, and will help moderating the server without having to add more admins. Of course this has to be a mod only command.
Proposed syntax
!slowmode <channel?(optional)> <time?(optional)>

don't think we need the channel there, but otherwise, sounds good!


Consistent Document IDs in NotionReader.ts

Problem
When using the NotionReader Document objects do not have the same ID as the Notion document they represent. This looks to simply be an oversight, rather than intended behavior. The Notion document ID is available.
Solution
Use the Notion document ID as the ID of the Document object. This ensures consistent id_s for each ingestion run.

I'm guessing the current behavior was not intended, but let me know if a random UUID is actually what is desired for Notion docs.


Simplify Moving Data lab

Right now we load a day of data and then a year.  Walking through that Cypher improvement should probably stay.  However given the change in GDS approach, we don't need the two data models any more.

Done!


fix: clamp diagnostic line number to 0

Current dependencies on/for this PR:

main

PR #116   👈

PR #117

This comment was auto-generated by Graphite.


Cache busting on assets

Currently we do not version static assets. This of course needs doing before going live. And ideally before (second) release to uat.

asset versioning implemented using the gulp-rev module: https://github.com/gbif/portal16/commit/4a3e140558e06adb6cbbc26ffe1ce41761f7bfe4

Images fail to load as the image urls have been versioned but the referencing url hasn't
http://internal.gbif-dev.org/search?q=argentina
Same for page specific scripts
http://internal.gbif-dev.org/tools/species-lookup

page specific script versioning should be fixed with last commit.
The country flags are dynamically included URLs, thats indeed a little tricky.
I suggest to create a nunjucks filter that will apply the manifest.json to convert asset filenames on server side page rendering for dynamic content

Implemented a "flag" and a "rev" nunjucks filter to use the rev-manifest file versions and simplify the generation of a country flag image path

We see 404s for http://demo.gbif.org/favicon.ico
I suspect that some browsers don't care much what we say about the favion but simply look for that specific filename.

I have excluded the favicon.ico from asset versioning: https://github.com/gbif/portal16/commit/5f39cb0c423728f6a4454aba4abbee01dd1848ae

Dev environment still have cached assets leading to errors after each build.

logs show that this is still an issue.

We just had an issue with a build that all assets weren't versioned leading to a site without css. Doing a new build solved it. Maybe async calls that just happen to work most of the time? If it happens again we should revisit the asset versioning. Could also be one of those things that we test for in smoke tests?

Closed with https://github.com/gbif/portal16/commit/371ee74b38fa921ca6e3a709bab6271cb4c0074e
We might consider simplifying this process to also make it easier to not cache stuff. But for now it will do


fix: correctly detect changed data

fixes #8157. this really needs some tests but my brain is melted right now
Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[x] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[x] This message body should clearly illustrate what problems it solves.
[ ] Ideally, include a test that fails without this PR but passes with it.

Tests

[x] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

Changesets

[x] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpm changeset and following the prompts. All changesets should be patch until SvelteKit 1.0

You said you wanted to add some tests - still up for it? I could take a look if you want, too.

@dummdidumm yeah, I was planning to (but don't let me stop you from getting there first if you're feeling inspired 😛 )

While writing a test I discovered that this doesn't fix the issue yet; we have to do a Math.max(branchLengthOld, branchLengthNew), else a now shorter branch length with the old one containing a data point at a higher index would give a false negative (didn't change when it actually did)

While writing a test I discovered that this doesn't fix the issue yet; we have to do a Math.max(branchLengthOld, branchLengthNew), else a now shorter branch length with the old one containing a data point at a higher index would give a false negative (didn't change when it actually did). Pushing a fix.


Make css pseudo functions receive any argument

Currently this is not working:
input:not(:checked)

workaround is using quotes:
input:not(':checked')

+1 to make it works without the workaround

Is this really happening!? The issue is implemented :)))


Crash at startup

Describe the bug
When I try to start application it crashes immediatly and I have no see UI, but in tray there is icon with balloon message that there is no valid token.
To Reproduce

Install 1.6.4
Start.
Crash.

Expected behavior

Install 1.6.4
Start.
UI Appeared.

Screenshots

Hi! Thank you for your message. This is a local problem. First of all, you can try to turn off your antivirus or other programs, which can block the tracker.

Check if there's error log file in app installation folder

There are only 2 files in application folder: MTGApro.exe, MTGApro.pdb. There are no logs and no cards DB files.

Unfortunately, we cannot help you with this local problem. The only one reason of it is the  block of your own system (for example, Windows Brandmauer or other programs can block the start of install process).

Downgrading to .NET Framework 4.5.2 might help you as this could be the same as https://github.com/HearthSim/Hearthstone-Deck-Tracker/issues/3662.


feat(BrowserFetcher): support downloading content_shell

This patch adds a "product" option to BrowserFetcher that can be
either 'chromium' or 'content_shell'.
ContentShell might be useful as a stripped-down version of Chromium.

NOTE: Puppeteer API is not guaranteed to work with content_shell; some of the methods don't actually work. Use on your own risk.

It turned out content_shell doesn't pass a lot of pptr tests:

browser.close() is not implemented in content/; instead, the implementations are in chrome/ and headless/ layers
javascript dialog support is not implemented in content/ as well
most of screenshot tests fail with opacity and other issues

There's no intent to invest into content_shell to fix these incompatibilities, but it's good we've explored this.


RFE: Add meson build

Feature Request
I'd like to request to add meson as a build system for hashmap.c to easily integrate it into other (meson) projects.
If this RFE is considered to be an improvement for the project, I'd be happy to provide a PR for it.
Background
In the project hirte we are currently using a copy of hashmap.c. However, we want to be able to easily upgrade the library in our project if a new hashmap.c version is released. Since we are using meson as a build system we consider integrating it as a meson subproject via a wrap-git. This, however, only works for projects using meson (ignoring some workarounds).

+1

@tidwall Hey Josh, could you please review this one and give as an ack?
Thanks!

Adding the Meson build system is outside of the scope of this project.

@tidwall oh no, can we add least send a PR and help maintain the meson build? The only required action is review/merge. We would prefer to stay upstream as much as possible and skip forks.
Thanks!

Adding the Meson build system is outside of the scope of this project.

That is unfortunate. Although I can understand your argument, I think this project would benefit from providing its source via a build system like meson to ease the integration into using projects and stay up to date.
Closing this issue.


Signature's dir

All signatures should be in signature/ folder

thanks for noticing, fixing up right now.

fix completed.


Update manifest.json

bump version so it will update in home assistant after adding this repo

sorry, didnt realise that this wasnt the ongoing fork.


Temporarily disable bundle size analysis CI

Summary
We are going back to board to further think how we can best add bundle analysis for PR. Currently the analysis skips for forked prs. It also doesn't show the correct diff. Hence we are reverting this.
Ticket Link
n/a
Related Pull Requests
PR which added the action https://github.com/mattermost/mattermost-webapp/pull/10969
Screenshots
n/a
Release Note

NONE

/cherry-pick release-7.3

Cherry pick is scheduled.


remotecfg: add some basic metrics

New metrics:

remotecfg_hash
remotecfg_last_load_successful
remotecfg_load_failures_total
remotecfg_last_load_success_timestamp_seconds
remotecfg_load_attempts_total
remotecfg_request_duration_seconds

Component loading info should already be exposed via controller metrics.
This does not give any insight into the fallback local files, and is focused mainly on the remote fetch.

Are we always injecting these metrics even if someone is not using remote config?

Are we always injecting these metrics even if someone is not using remote config?

Good point. Not everything is very good about deferring registration, but no reason this couldn't.


Move nyc_driver_demo notebook to sample directory

Description

This PR moves nyc_driver_demo notebook to samples directory so all notebooks are orginazed under same directory.
How was this PR tested?

Verified existing referrence path are updated.
Does this PR introduce any user-facing changes?

[ ] No. You can skip the rest of this section.
[x] Yes. Make sure to clarify your proposed changes.

The link check tool will pass once this PR is merged.

@blrchen , Shall we resolve the conflicts and merge this in for release 0.8.0?

@Yuqing-cat xiaoyong told me if this one is moved, then cli users won't see any samples under default workspace. I am abandoning this one right now.


working on a windows OS

hi,
is there a way to run YAMALE on windows?
I tried it but it did not work.
Thanks,
Itamar

If you're still interested, I'd need to see what the error was.

thanks but I'm no longer working on YAMALE.
If no one else experienced the same problem, then I guess, the issue is closed.
thanks,
Itamar


HDDS-6566. [Multi-Tenant] Fix a permission check bug that prevents non-delegated admins from assigning/revoking users to/from the tenant

What changes were proposed in this pull request?
A permission check bug prevents non-delegated admins from assigning users to the tenant or revoking users from the tenant.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-6566
How was this patch tested?

[x] Add a new test case that verifies a non-delegated admin have the permission to assign or revoke user accessIds in the tenant.
[x] All existing test cases shall pass.

@errose28 No problem! Thanks for the review and comments.


Fix Organization Name

Everywhere in the code servicecomb was used instead of ServiceComb because of which go get fails.

Codecov Report

Merging #41 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #41   +/-   ##
=======================================
  Coverage   70.33%   70.33%
=======================================
  Files           8        8
  Lines        3594     3594
=======================================
  Hits         2528     2528
  Misses        859      859
  Partials      207      207

Impacted Files
Coverage Δ

server/service/microservice/microservice.go
4.61% <ø> (ø)
:arrow_up:

server/service/query_rule_service.go
69.51% <ø> (ø)
:arrow_up:

server/service/resource_assembler.go
100% <ø> (ø)
:arrow_up:

server/service/services_dependency.go
69.15% <ø> (ø)
:arrow_up:

server/service/governservice.go
66.02% <ø> (ø)
:arrow_up:

server/service/instances.go
65.68% <ø> (ø)
:arrow_up:

server/service/microservices.go
74.75% <ø> (ø)
:arrow_up:

server/service/microservice/versionrule.go
100% <ø> (ø)
: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 7510be8...867995f. Read the comment docs.

Coverage remained the same at 76.159% when pulling 867995fe88afc8587753703675c0f824fdb6cf23 on asifdxtreme:master into 7510be871c1dd533406103cd80cab36ffcfacba0 on ServiceComb:master.

Coverage increased (+0.08%) to 76.722% when pulling a52289cacc89a10bcd84e723e79e35a4525a26a9 on asifdxtreme:master into a19f1882e38b2d90e873742b364316d1eb5d034a on ServiceComb:master.

Coverage decreased (-0.06%) to 76.582% when pulling a52289cacc89a10bcd84e723e79e35a4525a26a9 on asifdxtreme:master into a19f1882e38b2d90e873742b364316d1eb5d034a on ServiceComb:master.


No metrics shown in batch explorer and app insights

Batch configured, but metrics are not collected
start\stdout.log
SUCCESS: Specified value was saved.
Starting App insights background process in D:\batch\tasks\startup\wd

TaskPath                                       TaskName                          State
--------                                       --------                          -----
\                                              batchappinsights                  Ready
\                                              batchappinsights                  Running

startup\wd\nodestats.log
WARN Using postional arguments for Node ID, PoolID, KEY and  Process names is deprecated. Use --poolID, --nodeID, --instKey, --process
WARN It will be removed in 2.0.0
User configuration:
   Pool ID: psl1
   Node ID: tvmps_******
   Instrumentation Key: xxxxx
   Aggregation: 1
   Disable: []
   Monitoring processes: [--poolID]
BatchInsights configuration:
   Pool ID: psl1
   Node ID: tvmps_966f3398e2dcc0f5778aa0650e26720476375e2eebd4dddbb14d1bbd4bc80110_d
   Instrumentation Key: xxxxx
   Aggregation: 1m0s
   Sampling rate:<PHONE_NUMBER>
   Disable: {"diskIO":false,"diskUsage":false,"networkIO":false,"gpu":false,"cpu":false,"memory":false}
   Monitoring processes: [--poolID]
System information:
   OS: windows
No GPU detected. Nvidia driver might be missing. Error while initializing NVML NVIDIA driver is not loaded

Batch configuration

Statistics were collected and reported properly but not displayed in Batch Explorer due to firewall issues.


rkt list: Retrieve Image name from treestore when Image has been removed.

Issue #1628 pointed out that when an image was removed before a pod that referenced had been garbage collected, rkt list would crash. In that ticket a simple fix was made that uses the PodManifest to get the Image name. However,  the Image name is not a required field in the PodManifest.
The image manifest is, however, still present after image removal. The location is dependent on whether one uses overlayfs or not. In the case of overlayfs it's found here:
/var/lib/rkt/pods/run/UUID/

if running with --no-overlay it's located here:
/var/lib/rkt/pods/run/UUID/stage1/rootfs/opt/stage2/APP_NAME/

To check if the pod has been executed with the --no-overlay flag one can check for the presence of ~/var/lib/rkt/pods/run/UUID/overlay-prepared`
Thus, we're going to need to implement some logic to determine all of this.
For this to be considered complete, tests must be created.

SGTM :)

btw, the first path should be /var/lib/rkt/cas/tree/deps-sha512-{IMAGE_W_DEPS_HASH}/manifest

It turns out we didn't need to much around with figuring out which directories the ImageManifest was in because there should always be an ImageManifest in the app's appinfo directory.
I've opened issue #1702 to followup on that.