[Enhancement] Optimize exception handling for merge commit

Why I'm doing:
optimize exception handling for merge commit
What I'm doing:

check db/table existing when FE receives the merge commit request from BE. Fix #8895
stop retry if BE gets an error rpc response from FE for merge commit request, such as db/table not existing, BE dropped/not alive. Fix #8895  #8941
make the load response json more clear.  Fix #8850

What type of PR is this:

[ ] BugFix
[ ] Feature
[x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[x] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

[ ] This is a backport pr

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.4
[ ] 3.3
[ ] 3.2
[ ] 3.1
[ ] 3.0

@Mergifyio backport branch-3.4.0-rc01


Config option to show shaders in other menus

Most of the time you're on other menus than the main menu. It would be great if this was also behind the community and robtop level menus.

added in commit 701356a3 :3

will release in like a few minutes

tysm :)


Testing Implementation

Design and set up better long term testing structure/method for all types of testing

Decide on a testing framework
Create designated test folder with necessary persistent data

find/create/include persistent test data

Create a separate test target for pipeline builds and CI/CD pipelines
Create Unit tests for all code as well as implemented types and new symbolic arrays
Begin work on integration tests

including fetching and building the initial database and any benchmarks created and shared

Begin work on regression testing
Begin work on system tests (for entire existing MVP arvg-v)

Tests for @rubalgoyal new yml code should also be created
Also this issue and tasks should be broken up into smaller pieces/issues


Package location variables point to Mitaka instead of Liberty

Operating System
N/A
OpenStack Release
Liberty
Description
The following files reference Mitaka instead of Liberty, for package installation:
playbooks/vars/agent_deploy_vars.yaml
playbooks/vars/agent_driver_deploy_vars.yaml
playbooks/vars/driver_deploy_vars.yaml
Versions will move forward over time, but should at least point to a version from the correct branch (i.e. 8.1.0)

Repository Archived


Added link to refresh-tokens.md

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

#please-close


Ceph RBD not creating needed configuration files

What kind of request is this (question/bug/enhancement/feature request):
bug
Steps to reproduce (least amount of steps as possible):
Add Ceph RBD storage class to your local rancher cluster(i.e. not workloads cluster).
Enable monitoring with Persistent Volume.
Result:
The Persistent Volume Claims complain.
Type: Warning
Reason: ProvisioningFailed
Message:

(combined from similar events): Failed to provision volume with StorageClass "ceph-hdd": failed to create rbd image: exit status 1, command output: 2019-07-14 09:56:37.061822 7fce277c1100 -1 did not load config file, using default settings. 2019-07-14 09:56:37.073879 7fce277c1100 -1 auth: unable to find a keyring on /etc/ceph/ceph.client.rancher.keyring,/etc/ceph/ceph.keyring,/etc/ceph/keyring,/etc/ceph/keyring.bin: (2) No such file or directory rbd: create error: (1) Operation not permitted 2019-07-14 09:56:37.083930 7fce277c1100 -1 librbd: Could not tell if kubernetes-dynamic-pvc-ab594aab-a61d-11e9-9f8e-0200ec7e0767 already exists

So it appears it's looking for /etc/ceph/ceph.client.rancher.keyring which I assume would have been generated by the secret when adding Ceph RBD as a storage class?
Other details that may be helpful:
Rancher running on Ubuntu 18.04 (Bioinic)
Got Ceph RBD to work on the workload cluster by modifying extra_binds for kubelet (#21189).
Environment information

Rancher version: 2.2.4
Installation option: HA
Docker version: 18.09.5

Still relevant for Rancher 2.3.3 with Docker 19.03.5 on Debian 10

Okey, I actually got it to work. To bad documentation is non-existent.
I suspect that error message I got above might be a fallback, so when it fails it goes looking for keyring on disk as a fallback.
Here is how I did it, don't forget to adjust the commands for your environment, i.e. don't just copy-paste.
I add this to my cluster.yml file for rke and setup/update rancher cluster. Similar to https://github.com/rancher/rancher/issues/21189
services:
  kubelet:
    extra_binds:
      - '/lib/modules:/lib/modules:ro'
      - '/usr/sbin/modprobe:/usr/sbin/modprobe:ro'

Create Ceph pools for rancher cluster
Example
ceph osd pool create rancher_hdd 64 64 replicated replicated_hdd
ceph osd pool application enable rancher_hdd rbd
ceph auth get-or-create client.rancher mon 'allow r, allow command "osd blacklist"' osd 'allow class-read object_prefix rbd_children, allow rwx pool=rancher_hdd'
# [client.rancher]
#   key = XXXXXX==

add storage class in rancher/kubernetes
First create api key in the UI under your account -> API & Keys.
Export these:
export ENDPOINT='https://rancher.deltaprojects.io/v3'
export CATTLE_ACCESS_KEY='token-xxx'
export CATTLE_SECRET_KEY='xxx'
export BEARER_TOKEN='token-xxx:xxx'

add ceph client key to kubernetes secrets
kubectl create secret generic ceph-rancher-secret --type="kubernetes.io/rbd" \
  --from-literal=key='XXXXX==' \
  --namespace=kube-system

Create storage class in kubernetes
curl -u "${CATTLE_ACCESS_KEY}:${CATTLE_SECRET_KEY}" \
-X POST \
-H 'Accept: application/json' \
-H 'Content-Type: application/json' \
-d '{"allowVolumeExpansion":true, "allowedTopologies":[], "annotations":{"storageclass.beta.kubernetes.io/is-default-class":"true", "storageclass.kubernetes.io/is-default-class":"true"}, "labels":{"cattle.io/creator":"norman"}, "mountOptions":[], "name":"ceph-hdd", "parameters":{"adminId":"rancher", "adminSecretName":"ceph-rancher-secret", "adminSecretNamespace":"kube-system", "fsType":"ext4", "imageFeatures":"layering", "imageFormat":"2", "monitors":"mon01:6789, mon02:6789, mon03:6789", "pool":"rancher_hdd", "userId":"rancher", "userSecretName":"ceph-rancher-secret", "userSecretNamespace":"kube-system"}, "provisioner":"kubernetes.io/rbd", "reclaimPolicy":"Delete", "volumeBindingMode":"Immediate"}' \
'https://rancher.deltaprojects.io/v3/clusters/local/storageclasses'

Even that I got this to work, I'm leaving this issue open since instructions needs to be added to documentation.


how can I select DB by random

how can I select some item from DB by random , mysql source query can do it, but how can do it by knex?

with knex.raw.


Directory structure is not documented

Hi Shubham,
Could you please add the directory structure information to the README file like what we have for ProFuzzBench: https://github.com/profuzzbench/profuzzbench
The current structure is hard to follow.
Thanks.
Thuan

Hi Thuan,
I have added a directory structure to the Readme file.
Thanks


Crystal metadata change

Code extracted from https://github.com/DiamondLightSource/dodal/pull/722/files
I don't quite understand why do we set this as a soft signal - that is one failing test at the moment

        # Soft metadata
        # If supplied include crystal details in output of read_configuration
        with self.add_children_as_readables(ConfigSignal):
            if crystal_1_metadata.usage is not None:
                self.crystal_1_usage, _ = soft_signal_r_and_setter(
                    str, initial_value=crystal_1_metadata.usage
                )
            else:
                self.crystal_1_usage = None

And I deleted one test too

the error has to do with crystal spacing values

What is the reason for this change?

@callumforrester this is a refactor as d_spacing can be directly traced from the reflection values and the material properties.
Neater for the i18 lookup service

thanks @rtuck99 , merged

<<<<<<< HEAD
async def test_crystal_metadata_not_propagated_when_not_supplied():
    async with DeviceCollector(mock=True):
        dcm = DoubleCrystalMonochromator(
            prefix="FOO-MO",
            temperature_prefix="FOO-DI",
            crystal_1_metadata=None,
            crystal_2_metadata=None,
        )

    configuration = await dcm.read_configuration()
    expected_absent_keys = {
        "crystal-1-usage",
        "crystal-1-type",
        "crystal-1-reflection",
        "crystal-1-d_spacing",
        "crystal-2-usage",
        "crystal-2-type",
        "crystal-2-reflection",
        "crystal-2-d_spacing",
    }
    assert expected_absent_keys.isdisjoint(configuration)

=======
>>>>>>> e8ac54553 (add to the beamline files)

@DominicOram  all is fixed besides this test. I am not sure what is the purpose of the behavior here to have absent keys. For it to pass the metadata param in the DCM init would need to be CrystalMetadata | None and not CrystalMetadata.
I'm happy to delete the test, git blame points last modification to the test, for May 2024 by @callumforrester  - what is the reason for the test? I'm equally happy to keep the test and change the behavior provided I note the reason for allowing None  in the code.

all is fixed besides this test...

Ok, I'm happy to retract https://github.com/DiamondLightSource/dodal/pull/843/files#r1820621900 so that we can keep this usecase. I'm still not 100% sure when it would be necessary but that's scope creep.

@DominicOram  I'm afraid I don't understand your last comment

 self.crystal_metadata_reflection, _ = soft_signal_r_and_setter(
                Sequence[int],
                initial_value=list(cm.reflection),  # type: ignore
            )

I think this might be due to ophyd-async change


MAISTRA-1963 skip privileged ports when building listeners for non-root gateways

This is a cherry-pick of https://github.com/istio/istio/pull/28268

/hold
This can wait for 2.0.1

Removed the hold as 2.0.0-2 has gone out


instantiate certain aggregation dataypes on node start, not on first use

which has caused a NullPointerException when one or more nodes in the cluster
have not used the datatype before

ack


Setting multiple headers

I'm having some trouble setting multiple headers using webb's HTTP library and I don't know if it's because I'm dense or I can't figure out how to cast types or something.
Here's my code:
Webb webb = Webb.create();
webb.setDefaultHeader( "User-Agent", "Java/8 by <APPLICATION_ID>" );
webb.setDefaultHeader( "Authorization", "bearer " + token );

JSONObject result = null;

try {
    result = webb
        .get( ABOUT_ME )
        .ensureSuccess()
        .asJsonObject()
        .getBody();
} catch ( WebbException ex ) {
    ...
}

I'm fairly certain that the second setDefaultHeader() is just overwriting the first statement because I found:
header(String name, Object value)
Set (or overwrite) >>a<< HTTP header value.

In the API docs, under Request. I tried just inserting .header( "header", "value") after get() and before ensureSuccess(), but then my IDE, NetBeans, said this symbol was not found.
Since header( String, String) is in the Request class, I just thought I could switch out the "Webb webb" declaration for "Request webb" as is shown here, but inserting either Request or Request<String> for the webb's variable type produced an error:

Also, a little sidenote: In the header()'s Javadocs, there's a grammatical error. It should be "Set (or overwrite) an HTTP header value."

It should work in 3 different ways:

Webb.setGlobalHeader sets an HTTP-header for the whole JVM (regarding DavidWebb)
webb.setDefaultHeader for the instance used to create HTTP-requests
webb.get/post/....header() for a single request

Example code (setting header for a single webb instance):
    webb = Webb.create();
    webb.setBaseUri(mSyncPrefs.getEndpoint());
    webb.setDefaultHeader(Webb.HDR_USER_AGENT, Const.UA);

    // take userId and secret to obtain access-token
    accessToken = obtainAccessToken(mSyncPrefs);
    webb.setDefaultHeader(HDR_ACCESS_TOKEN, accessToken.token);

Example setting header for a single request:
    Calendar cal = Calendar.getInstance(TimeZone.getTimeZone("GMT"));
    cal.set(2013, Calendar.NOVEMBER, 24, 23, 59, 33);

    Response<Void> response = webb
            .get("/headers/in")
            .header("x-test-string", SIMPLE_ASCII)
            .header("x-test-int", 4711)
            .header("x-test-calendar", cal)
            .header("x-test-date", cal.getTime())
            .param(Webb.HDR_USER_AGENT, Webb.DEFAULT_USER_AGENT)
            .asVoid();

    assertEquals(200, response.getStatusCode());

Like almost all calls to methods of Request, it's returning the instance (this) so you can chain the calls. I suppose there is something wrong with your IDE or your code has errors.
Thanks for pointing out the grammatical error!

Thank you very much for the quick response,
This is just a note for anyone who encounters the same problem:
Using the Response<Void> as the variable's type did allow me to add headers( "", "" ). However, it created a conflict with my try-catch statement below because a Response<JSONObject> variable is not the same as a JSONObject. But adding ensureSuccess(), getBody(), and changing the variable to JSONObject solved my problem as shown below:
JSONObject result = webb
    .get( "https://www.reddit.com/r/all/new/.json" )
    .param( "limit", 1 )
    .header( "User-Agent", "Java/8 by <<APP_ID>>" )
    .header( "Authorization", "bearer " + token )
    .ensureSuccess() // << added
    .asJsonObject()
    .getBody(); // << added

System.out.println( "OAUTH: Got response: "
    + result.toString().substring( 0, 32 ) + "..." );

String data;

try {
    data = result
        .getJSONObject( "data" )
        .getJSONArray( "children" )
        .getJSONObject( 0 )
        .getJSONObject( "data" )
        .getString( "id" );
} catch ( WebbException ex ) {
    ...
}

Upon reflection, I do now see how dense I was. Thank you, David, again for helping me with this.


Publish 7.0.1 release with bumped dependencies

@sungam3r how do I configrm that sourcelink is still working?

Use 7.0.1 in some project and step into its method.

Release has been started

Another option to check SourceLink - NugetPackageExplorer


Update scalafmt-core to 3.0.8

About this PR
📦 Updates org.scalameta:scalafmt-core from 3.0.0-RC7 to 3.0.8
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalameta", artifactId = "scalafmt-core" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.scalameta", artifactId = "scalafmt-core" }
}]

labels: library-update, early-semver-pre-release, semver-spec-pre-release, commit-count:n:3

@scala-steward
In master I updated these versions:
Update scalafmt-core to 3.8.3
Update sbt-scoverage to 2.2.2
Update sbt, scripted-plugin to 1.10.4
Update scalameter to 0.21
related to these PR:
https://github.com/gekomad/itto-csv/pull/6
https://github.com/gekomad/itto-csv/pull/7
https://github.com/gekomad/itto-csv/pull/8
https://github.com/gekomad/itto-csv/pull/9


Update TypeScript to v4.4

See the release announcement for more details about v4.4: https://devblogs.microsoft.com/typescript/announcing-typescript-4-4-rc/
The main breaking change that affected us was the change in the default type of the catch error parameter from any to unknown. This was resolved in some places by changing the expected type to unknown. In cases where we need it to be an Error or where we use an Error property, the type is now checked at runtime.
A minor change was needed in BaseControlelrV2 as well, because TypeScript wasn't happy with the fact that it was never initialized for some reason, even though it wasn't supposed to be initialized (it was set to never). But initializing it as undefined serves the exact same purpose and resolved the error.

This is blocked by test coverage improvements, which are blocked by the various test related PRs that are currently open.


Feature request: support for custom allocator or pointer wrapper for parse_tree

In my parser which uses parse_tree, I found that most of the CPU cycles are going to allocation, deallocation, and cache misses due to the parse tree AST nodes being allocated far from each other in memory. I'd like to use an arena allocator instead, so that allocations are done by "bump-the-pointer" and all deallocations can happen in batch when the parse tree is no longer needed.
Currently, parse_tree enforces the use of std::unique_ptr and doesn't allow customization of the creation or deletion of nodes. It would be great to have a customization point to be able to override the use of std::unique_ptr<> to use a custom deleter instance. It's possible that this can be done by specializing std::default_delete but seems like a cleaner solution might be available (that solution would need to rely on threadlocal storage to pass in the pointer of the arena)

Any feedback on the above idea and the modified parse tree example? Can we close this issue?


Can't generate json for p5.js library using condense

****@**** MINGW64 ~/AppData/Roaming/Brackets/extensions/user/ternific/node_modules/tern/bin $ ./condense --name p /c/wamp/www/libraries/p5.js
{
  "!name": "p"
}

Can't generate json for p5.js library using condense. I'm using tern in Brackets editor and Im trying to get p5.js autocomplete. Ive ran the same command with jquery and it works fine.

I guess Tern has problems with the bundled output in p5.js. I don't know exactly why, but if you feel like simplifying the file to reduce the testcase in order to find out what aspect causes the problem, I'd be interested in the result. As it is, I don't have time to poke at random libraries to figure out which aspect of them breaks Tern.

I've also been attempting this. However, I'm running it on the source files.
If you run condense on each file individually, it works just fine...  but running this command:
$ ./node_modules/tern/bin/condense --plugin node --plugin modules --plugin requirejs --plugin doc_comment src/app.js
only produces:
{
  "!name": "src/app.js"
}

Any news about this matter ? I would love to have autocompletion with p5js

There are presumably other ways to do this now. I think this can be closed.


Refresh all tools/workflows in an organization

Might be a useful idea to allow users to be able to refresh organizations/groupings of entries instead of just an individual entry or all entries.  For example the UI could have refresh buttons beside the name of all the organizations on the lefthand side of my tools and my workflows which would refresh a grouping of entries.

This would also require adding a new API call for both tools and workflows which refreshes all the tools/workflows in the given org.

The refresh message has a partial name for tools which can be ambiguous.
For example refreshing "quay.io/cancercollaboratory" will say "Refreshing cancercollaboratory..."

If a message appears during fresh, dismissing it, browsing to a different page and then coming back, the message may return.

The refresh for workflows is worse and just displays a generic message "Refreshing Organization ..."

Not sure what message is being dismissed.  The bootstrap alert is currently set to not be dismiss-able.

An error message like a 500 or a gateway timeout

Ran into #977 and #975 , splitting those guys out. Otherwise this works in general.


Regular expressions don't work in the URL

Isue Closed.
Just forgot the . before the asterix 😊

ust forgot the . before the asterix 😊

ust forgot the . before the asterix 😊


chore: run type tests using old version of TS

Summary
This currently fails, /cc @mrazauskas
 FAIL   types  packages/expect/__typetests__/expect.test.ts
  ● Test suite failed to run

    SyntaxError: ',' expected.

      at packages/expect/__typetests__/expect.test.ts:11:8

 PASS   types  packages/jest-types/__typetests__/config.test.ts
 PASS   types  packages/jest-expect/__typetests__/jest-expect.test.ts
 PASS   types  packages/jest-types/__typetests__/globals.test.ts
 PASS   types  packages/jest-types/__typetests__/expect.test.ts
 FAIL   types  packages/jest-types/__typetests__/jest.test.ts
  ● tsd typecheck

    Argument of type '((a: string, b: number) => boolean) & Mock<unknown, unknown[]>' is not assignable to parameter of type 'Mock<boolean, [a: string, b: number]>'.

    > 130 |   expectType<Mock<boolean, [a: string, b: number]>>(maybeMock);
          |   ^

      at packages/jest-types/__typetests__/jest.test.ts:130:3

    Parameter type 'Mock<boolean, [a: string, b: number]>' is declared too wide for argument type 'Mock<boolean, [a: string, b: number]> & Mock<unknown, unknown[]>'.

    > 143 |   expectType<Mock<boolean, [a: string, b: number]>>(surelyMock);
          |   ^

      at packages/jest-types/__typetests__/jest.test.ts:143:3

    Argument of type 'Mock<boolean, [a: string, b: number]>' is not assignable to parameter of type 'never'.

    > 150 |   expectType<never>(surelyMock);
          |   ^

      at packages/jest-types/__typetests__/jest.test.ts:150:3

    Parameter type 'string & Mock<unknown, unknown[]>' is not identical to argument type 'string & Mock<unknown, unknown[]>'.

    > 160 |   expectType<string & Mock<unknown, Array<unknown>>>(stringMaybeMock);
          |   ^

      at packages/jest-types/__typetests__/jest.test.ts:160:3

    Parameter type 'Mock<unknown, unknown[]>' is not identical to argument type 'Mock<unknown, unknown[]>'.

    > 170 |   expectType<Mock<unknown, Array<unknown>>>(anyMaybeMock);
          |   ^

      at packages/jest-types/__typetests__/jest.test.ts:170:3

    Parameter type 'Mock<unknown, unknown[]>' is not identical to argument type 'Mock<unknown, unknown[]>'.

    > 180 |   expectType<Mock<unknown, Array<unknown>>>(unknownMaybeMock);
          |   ^

      at packages/jest-types/__typetests__/jest.test.ts:180:3

    Argument of type 'SpyInstance<string, []>' is not assignable to parameter of type 'SpyInstance<Date, [value: string | number | Date]>'.

    > 271 | expectType<SpyInstance<Date, [value: string | number | Date]>>(
          | ^

      at packages/jest-types/__typetests__/jest.test.ts:271:1

    Argument of type 'string' is not assignable to parameter of type '(...args: any[]) => unknown'.

    > 155 | if (!jest.isMockFunction(stringMaybeMock)) {
          |                          ^

      at packages/jest-types/__typetests__/jest.test.ts:155:26

    Argument of type 'string' is not assignable to parameter of type '(...args: any[]) => unknown'.

    > 159 | if (jest.isMockFunction(stringMaybeMock)) {
          |                         ^

      at packages/jest-types/__typetests__/jest.test.ts:159:25

    Argument of type 'unknown' is not assignable to parameter of type '(...args: any[]) => unknown'.

    > 175 | if (!jest.isMockFunction(unknownMaybeMock)) {
          |                          ^

      at packages/jest-types/__typetests__/jest.test.ts:175:26

    Argument of type 'unknown' is not assignable to parameter of type '(...args: any[]) => unknown'.

    > 179 | if (jest.isMockFunction(unknownMaybeMock)) {
          |                         ^

      at packages/jest-types/__typetests__/jest.test.ts:179:25

    Expected an error, but found none.

    > 133 |   expectError(maybeMock.mockReturnValueOnce(123));
          |   ^

      at packages/jest-types/__typetests__/jest.test.ts:133:3

    Expected an error, but found none.

    > 146 |   expectError(surelyMock.mockReturnValueOnce(123));
          |   ^

      at packages/jest-types/__typetests__/jest.test.ts:146:3

    Expected an error, but found none.

    > 255 | expectError(jest.spyOn('abc', 'methodA'));
          | ^

      at packages/jest-types/__typetests__/jest.test.ts:255:1

Test Suites: 2 failed, 4 passed, 6 total
Tests:       14 failed, 447 passed, 461 total
Snapshots:   0 total
Time:        2.958 s

Closes #12444

Test plan
Green CI

First one SyntaxError: ',' expected. is clear. Fixed it in #12448
The rest.. hm.. At first glance it feels like types from @types/jest are under test. Many of these errors I saw while working on #12442. Must be fixed. Will take a better look.

Oh, nice! Merged it, will rebase


Ctrl+Space does not work on archlinux (endeavouros)

UnoCSS version
0.22.5
Describe the bug
Issue Type: Bug

Launch vscode
Open a *.svelte or *.astro file
Place your cursor anywhere code completion would be available
Press Ctrl+Space
Expected: IntelliSense popover should appear
Actual: Nothing happens
Hover Preview
Expected: See the complete CSS for a class name by hovering over it.
Actual: Working, all good.

Used Extensions: Unocss, Svelte, Astro.
VSCODE > settings.json (emty)
Reproduction
https://open-vsx.org/extension/antfu/unocss
System Info
Version: 1.75.0
Date: 2023-02-04T10:31:04.465Z
Electron: 19.1.9
Chromium: 102.0.5005.167
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Linux x64 6.1.10-arch1-1

I don't think this is an Unocss bug. It should be that some software occupies your shortcut keys?

I am try install WindiCSS  extensions and working all good..
If be that some software problem, then will does not working WindiCSS..

Came here because of the same issue. After looking for a while it looks like autocompletion isn't fully functional yet.
As antfu said on #1363:

For now, the autocomplete is only triggered by : and -.
https://github.com/unocss/unocss/blob/b727d55121e7b4946a951c6654a48e542f13caaf/packages/vscode/src/autocomplete.ts#L128-L132
I do not yet know a better way to do that. If you got any idea, PR would be greatly welcome. Thanks.

So basically, it'll work after typing something like bg- or hover:

yep. here's the real reason: currently autocompletion is only triggered by certain delimiters.

I am fix it problem!
uninstalled code (vscode) on archlinux - endeavouros:
yay -S code
and instead :+1:
yay -S visual-studio-code-bin
Press Ctrl+Space
Expected: IntelliSense popover should appear
Actual: All working, fix it


Images are lost when no local docker exists

I was doing some testing to see if I could use ko to build without docker and save the artifact (tar) somehow. I know the documentation says that ko "doesn't require docker to be installed", however, in my testing, when using the option --push=false, the resulting image is lost. The go-build cache remains, but there is no container tar file or other artifact.
It would be nice for there to be a feature where the tar file could be output as an artifact without being pushed to a registry.

--push=false instructs ko not to push the image, so if there's nowhere else the image is written to, it's just gone. This is normally like a dry-run flag in a lot of cases, to prove the build works without any side effect.
If you want to write to a local file, you can use ko build --tarball out.tar or ko build --oci-layout-path=path/to/somewhere to write the resulting image(s) in an OCI layout format.

oh nice, sorry I missed that option in the --help output for ko build.


CLI generate-struct-layouts fix

Depending on the platform, "write" function may fail if the full directory path does not exist. So we have to create it before.
Description
According to this doc https://doc.rust-lang.org/std/fs/fn.write.html "write" function may fail if the full directory path does not exist. And it does on macOS with an error "No such file or directory (os error 2)". I added a create_dir_all call before, to create dir first.
Test plan
build --generate-struct-layouts flag must work on all platforms. It did not work on macOS Sequoia

Release notes

[ ] Protocol:
[ ] Nodes (Validators and Full nodes):
[ ] Indexer:
[ ] JSON-RPC:
[ ] GraphQL:
[x] CLI: Fixed an issue on the sui move command regarding writing into a non existing directory.
[ ] Rust SDK:
[ ] REST API:

Nevermind, just tested it, this should be good as is. Thanks!


FilterState: WriteOnce and WriteMany support

Based on the discussions in this design doc:
https://docs.google.com/document/d/101Rcggh7flniYKgsr2T07yHXApywbnXiKj1hIpxX6yQ/edit#
Adds support for storing/retrieving mutable filter state objects, in addition to existing support for immutable filter state objects.
Signed-off-by: Shriram Rajagopalan<EMAIL_ADDRESS>

I am mainly looking for feedback on whether these interfaces are acceptable. Will add the tests etc. once we agree on the interfaces.

The getreadonly can get both mutable and immutable (so enum doesn't help here - stateType::ReadOnly looks odd). Also, setReadOnly and getReadOnly are not counter parts because I can get mutable as a readOnly object. The enum leaves the door open for more types in future IMO.


All category sections should be collapsable

This functionality exists on the inventory page. It should also be applied to create pack and view pack.

I'll do this one.


Can't launch a .net core 3.0 WPF app from another .net core 3.0 app

I have a .net core 3.0 WPF application, called client.exe.
I have a second .net core 3.0 application.
I'm trying to launch client.exe from the other application. I get the error,
A fatal error was encountered. The library 'hostpolicy.dll' required to execute the application was not found in 'C:\Program Files\dotnet'.

The code is
var filename = @$"bin\Debug\netcoreapp3.0\Client.exe";
var child = Process.Start(new ProcessStartInfo()
{
  WorkingDirectory = Path.GetDirectoryName(filename),
  FileName = filename,
});

@jeffschwMSFT does it belong to core-setup repo?


fix(curriculum): Update handleEnter to accept a message argument

Checklist:

[x ] I have read and followed the contribution guidelines.
[ x] I have read and followed the how to open a pull request guide.
[x ] My pull request targets the main branch of freeCodeCamp.
[x ] I have tested these changes either locally on my machine, or GitPod.

Closes #57423

The current implementation of the handleEnter function appends a hardcoded message ('You pressed the enter key! ') to the message state repeatedly. While this works for the current requirement, it limits the flexibility of the function.
If we anticipate future features that involve appending different types of messages based on user interactions, we can improve the code's maintainability and reusability by generalizing handleEnter to accept a dynamic message as an argument.

Thank you, for opening this pull request.
This is a standard message notifying you that we've reviewed your pull request and have decided not to merge it. We would welcome future pull requests from you.
Thank you, and happy coding.


Update CLI to print more user-friendly help string if no argument is provided

Before:
st2
usage: shell.py [-h] [--version] [--url BASE_URL] [--auth-url AUTH_URL]
                [--api-url API_URL] [--stream-url STREAM_URL]
                [--api-version API_VERSION] [--cacert CACERT]
                [--config-file CONFIG_FILE] [--print-config] [--skip-config]
                [--debug]
                {action,action-alias,auth,apikey,execution,key,login,pack,policy,policy-type,rule,run,runner,sensor,trace,trigger,trigger-instance,webhook,whoami,timer,rule-enforcement,role,role-assignment}
                ...
shell.py: error: too few arguments

After:
st2
usage: Usage: st2 [options] {command} [options]

For example:

    st2 action list --pack=st2
    st2 run core.local cmd=date
    st2 --debug run core.local cmd=date

CLI for StackStorm event-driven automation platform. https://stackstorm.com

positional arguments:
  {run,action,action-alias,auth,login,whoami,apikey,execution,key,pack,policy,policy-type,rule,webhook,timer,runner,sensor,trace,trigger,trigger-instance,rule-enforcement,role,role-assignment}
    run                 A command to invoke an action manually.
    action              An activity that happens as a response to the external
                        event.
    action-alias        Action aliases.
    auth                Authenticate user and acquire access token.
    login               Authenticate user, acquire access token, and update
                        CLI config directory
    whoami              Display the currently authenticated/configured user
    apikey              API Keys.
    execution           An invocation of an action.
    key                 Key value pair is used to store commonly used
                        configuration for reuse in sensors, actions, and
                        rules.
    pack                A group of related integration resources: actions,
                        rules, and sensors.
    policy              Policy that is enforced on a resource.
    policy-type         Type of policy that can be applied to resources.
    rule                A specification to invoke an "action" on a "trigger"
                        selectively based on some criteria.
    webhook             Webhooks.
    timer               Timers.
    runner              Runner is a type of handler for a specific class of
                        actions.
    sensor              An adapter which allows you to integrate StackStorm
                        with external system.
    trace               A group of executions, rules and triggerinstances that
                        are related.
    trigger             An external event that is mapped to a st2 input. It is
                        the st2 invocation point.
    trigger-instance    Actual instances of triggers received by st2.
    rule-enforcement    Models that represent enforcement of rules.
    role                RBAC roles.
    role-assignment     RBAC role assignments.

optional arguments:
  -h, --help            show this help message and exit
  --version             show program's version number and exit
  --url BASE_URL        Base URL for the API servers. Assumes all servers use
                        the same base URL and default ports are used. Get
                        ST2_BASE_URL from the environment variables by
                        default.
  --auth-url AUTH_URL   URL for the authentication service. Get ST2_AUTH_URL
                        from the environment variables by default.
  --api-url API_URL     URL for the API server. Get ST2_API_URL from the
                        environment variables by default.
  --stream-url STREAM_URL
                        URL for the stream endpoint. Get ST2_STREAM_URLfrom
                        the environment variables by default.
  --api-version API_VERSION
                        API version to use. Get ST2_API_VERSION from the
                        environment variables by default.
  --cacert CACERT       Path to the CA cert bundle for the SSL endpoints. Get
                        ST2_CACERT from the environment variables by default.
                        If this is not provided, then SSL cert will not be
                        verified.
  --config-file CONFIG_FILE
                        Path to the CLI config file
  --print-config        Parse the config file and print the values
  --skip-config         Don't parse and use the CLI config file
  --debug               Enable debug mode

Resolves #3709.

With this change I believe we don't even need to sort command names alphabetically - I think it's better to keep them "grouped" together the way they are now.
@dzimine wdyt?

Even if we later decide to move them to alpha order this is still a positive change.


[SPARK-44199] CacheManager refreshes the fileIndex unnecessarily

What changes were proposed in this pull request?
The CacheManager refreshFileIndexIfNecessary logic checks if the fileIndex root paths starts with the input path. This is problematic if the input path and root path share the prefixes but the root path is not a subdirectory of the input path. In such cases, the CacheManager can unnecessarily refresh the fileIndex which can fail the query if it does not have access to the rootPath for that SparkSession.
Why are the changes needed?
Fixes the bug where the queries on cached dataframe APIs can fail if the cached path shares prefix with the different path.
Does this PR introduce any user-facing change?
NO
How was this patch tested?
Unit test

@cloud-fan Do you know why the github action is failing for me? I fails due to "Unhandled error: TypeError: Cannot read properties of undefined (reading 'head_sha')"

@HyukjinKwon @Yikun do you know why github action doesn't work here?

This might related to github incidents: https://www.githubstatus.com/incidents/ws2l6203cs7g , retry should help.

This might related to github incidents: https://www.githubstatus.com/incidents/ws2l6203cs7g , retry should help.

Thanks for checking. The problem was with my permissions to the github actions on my fork. I think it should work now.

The tests ExpressionImplUtilsSuite and RandomDataGeneratorSuite look flaky. They work for me locally. Let me rebase the PR and retrigger the tests.

thanks, merging to master!


Notification authorization status is not always up-to-date when app comes to foreground

Hi guys,
I am testing Notificare v3.2.0, and noticed that sometimes the SDK doesn't trigger event func notificare(_ notificarePush: NotificarePushKit.NotificarePush, didChangeNotificationSettings granted: Bool) when an App returns to foreground (and a user changed permissions for notifications via Settings). Is it something I should count on, or should update on my side and push the SDK to re-register/unregister?
Kind regards

The scenario i have is the following:

Launching Notificare SDK: notificare.launch() + registering delegates
Waiting for SDK to be ready (listening to delegate be called)
A user is registering for notifications: notificare.push().enableRemoteNotifications()
A user is registering a device: notificare.device().register(userId: some_id, userName: some_name)
Getting an event that authorization status has been changed to Granted.
Now notifications are successfully received, all works fine.
The App goes to background, and a user changes authorization status to Denied.
The App comes back to foreground.
Delegate is called to notify that authorization status has changed, and it is now Denied.
I do unregistering device and unregistering from notifications: notificare.push().disableRemoteNotifications() and notificare.device().register(userId: nil, userName: nil).
Now as expected I don't receive notifications.
The App goes to background, and a user changes authorization status back to Granted via Settings.
The App goes to foreground.

Expected behaviour: the app is notified about authorization status change and registers the user again.
Actual behaviour: the app doesn't get any notification about this change. And delegate is assigned and is not accidentally deallocated (to answer your first assumption). There are no settings switching twice. The network is stable. The issue is replicable, which also excludes network issues.

Good question. So if I just register a device for notifications, Notificare SDK will take of permissions and actual sending? And a logical reason to unregister a device will be when a user logs out?

That's right. When you call enableRemoteNotification(), we take care of registering with APNS, triggering the permission prompt and reporting the response.
Depending on your use case, when the user logs out, you can register(userId: nil, userName: nil) to remove the association with the user. This doesn't disable remote notifications as you may want to engage anonymous devices. However, nothing prevents you from doing so.

Thank you, Helder. Now it is clear to me!


Send latest change note as string

With the work completed in 729eff we were incorrectly sending a hash to
Rummager for the latest change note. This commit updates it to send a
string structed like so "Title of section changed - description of
change."

:100:


Use DimAnalysis in lowering MatMul

This patch uses DimAnalysis to detect broadcasting when lowering N-D x N-D ONNXMatMul to krnl dialect. If there is no broadcasting, SIMD path will be used for the lowering.

Jenkins Linux amd64 Build #10900 [push] Use DimAnalysis in lower... started at 00:04

Jenkins Linux ppc64le Build #9957 [push] Use DimAnalysis in lower... started at 01:08

Jenkins Linux s390x Build #10910 [push] Use DimAnalysis in lower... started at 01:04

Jenkins Linux amd64 Build #10900 [push] Use DimAnalysis in lower... passed after 1 hr 22 min

Jenkins Linux ppc64le Build #9957 [push] Use DimAnalysis in lower... passed after 1 hr 51 min

Jenkins Linux s390x Build #10910 [push] Use DimAnalysis in lower... passed after 1 hr 53 min


DOC: Issue with Agent Names (for Notebook Example: multi-agent-collaboration.ipynb)

Issue with current documentation:
After invoking the agent workflow in the final cell, we receive the following error:
BadRequestError: Error code: 400 - {'error': {'message': "'Chart Generator' does not match '^[a-zA-Z0-9_-]{1,64}$' - 'messages.5.name'", 'type': 'invalid_request_error', 'param': None, 'code': None}}
Idea or request for content:
The way I resolved it is by changing the names of the agent from "Chart Generator" to "Chart". Appearently, the issue is with spacing in agent names when collaborating.

Should be updated now


[Fix #6250] Skip useless access modifier for new method

Closes https://github.com/rubocop-hq/rubocop/issues/6250

[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.
[x] Run rake default. It executes all tests and RuboCop for itself, and generates the documentation.

I feel like it's fair not to spend a ton of time keeping up to date on Ruby 2.2 since it's now officially EOL - https://www.ruby-lang.org/en/news/2018/06/20/support-of-ruby-2-2-has-ended/

Also, I feel like it's weird that the Ruby 2.3 branch is using Ruby 2.2

#6535 was solved similar issue with a solution that does not specialize to new method. Sorry for my late reply and thank you for opening a PR.


RLP encoding for single byte between 0x80 and 0xff is incorrect

RLP Encoding of a single byte between 0x80 and 0xff is incorrect due to the missing boundary check for a single byte RLP encoding. Thus it is right now impossible to compute correctly bytes for the transaction with nonce 0x80 to 0xff.

The job #100 assigned to @biboran/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and thisWe should be aware that biboran is on vacation; this ticket may be delayed

Job gh:blockstoreapp/web3swift#100 already assigned to @biboran, can't assign to @biboran

Bug was reported, see §29: +15 point(s) just awarded to @biboran/z

Manual assignment of issues is discouraged, see §19: -5 point(s) just awarded to @biboran/z

It is strongly discouraged to assign jobs to their creators, see §19: -15 point(s) just awarded to @biboran/z

@0crat waiting

@0crat waiting (here)

@biboran The impediment for #100 was registered successfully by @biboran/z

Order was finished: +30 point(s) just awarded to @biboran/z

The job #100 is now out of scope


Add gfx1035, gfx1036 to GFXLIST

APUs that appear in Ryzen 6000 & 7000 CPUs

This is so people with the new Ryzen 7900X/7950X can run OpenMP on the gfx1036 APU in their dev systems.
Sergio has verified that OpenMP runs on the gfx1036.
Currently has to manually set GFXLIST, this change just builds it by default.
Sounds like there are at least 3 systems in our group now with this APU (Sergio, Jon, JP?)

Absolutely.   Lets get this in release notes for AOMP 16.0.3.


test: run e2e test pod without framework setup

Added a simplified version of the current test/pod to reduce the number of configurations.
Modified the e2e test framework to skip the operator setup if no test image is provided.
This way the test-pod can be used to run e2e tests in a namespace with an already deployed etcd-operator.
The default cmd is set to run-e2e which only runs the e2e tests for now.
The e2e-slow tests are not run for now because they require aws creds or modify the deployed operator which requires the operator image env.
The Dockerfile for the simplified test pod is 2 staged. The first stage builds the test binary and the second stage adds the test binary to be run inside a smaller image.
/cc @colhom @fanminshi

LGTM

lgtm

Thanks haseeb


Use :connection-type pipe when creating local processes

This is helpful because some formatters, rather than having an option
to enable reading from stdin, simply check whether their input comes
from a pipe or terminal. I don't really think this is a wise thing for any code formatter/application to do, but unfortunately it seems somewhat widespread, so we should support it.
In particular, this fixes prettier, so should close #118.
I hope this suits you code and documentation-style wise, wasn't really how to best handle adding a new but non-changing make-process argument with the keyword-argument forwarding the existing code does.

The long argument lists seem to be causing lint errors. Please instruct me on how you want this fixed, as it seems like this mostly depends on your personal taste.

It's a fantastic solution, @leotaku, I missed that option of make-process somehow.

All formatters should be able to do stdin->stdout via pipes. If a formatter requires a tty, that's a bug in the formatter.

@lassik It's actually the other way around. The prettier command tests if its input is connected to a pipe, and if this is the case, waits for data from stdin. If it thinks its input is connected to a terminal, it requires a filename as an argument to read input from.
While, as I've said, I don't think this is a good decision, it is in fact deliberate and not a bug as evidenced by prettier/prettier#4559.

I agree the new behavior is brittle, but still it seems using pipes will always work.


State machine emits too generic errors

Calling h2.reset_stream(...) on an already closed streams gives me an ProtocolError, but since there is a StreamClosedError in hyper-h2, I would have expected to get this.
Trace of error:
  File ".../hyper-h2/h2/connection.py", line 680, in reset_stream
    frames = stream.reset_stream(error_code)
  File ".../hyper-h2/h2/stream.py", line 730, in reset_stream
    self.state_machine.process_input(StreamInputs.SEND_RST_STREAM)
  File ".../hyper-h2/h2/stream.py", line 98, in process_input
    "Invalid input %s in state %s" % (input_, old_state)
ProtocolError: Invalid input StreamInputs.SEND_RST_STREAM in state StreamState.CLOSED

Is it possible to handle/check all errors for the correct exception class and emit the best fit?
Otherwise I need to catch multiple error classes, where a single one would have done the job.

I think this is an oversight in the state machine. I think all SEND_* inputs in the CLOSED state should raise the StreamClosedError. Should be an easy fix. =)

See #118.


docs: Beautify README

I sent the original README to claude.ai and asked it to fix grammatical errors and add some emojis.

Thanks a lot for this contribution, that will helped me a lot.


#5 : Integrate Basic Styling and Theme

Description

Establish a consistent color scheme and font style for the UI.
Ensure the design is visually appealing and accessible.

I'd like to work on this!

So my plan is to use colors from Material 3 (not widgets just the colors)
This will allow us to have a complete pallete for colors for both light and dark mode, and also make sure its accessible

Do you have a specific color in mind, which can be used as the seed color?

Do you have a specific color in mind, which can be used as the seed color?

Something that has a good combo b/w fluorescent green and Grey. Other than that, I'd leave it entirely upto you.

I tried going with a saturated blue, here is how it looks (added some more info too)

Based on the seed color, the surface, background colors will differ,
for eg with a flouroscent green type color:

The second look better !!

what about the outlines?

LGTM !! As I said, the final submission ultimately depends on you. I'll proceed with the best build you rovide. Outlines are necessary in cards


metaRepoLink redirects to 404 page

Should this link https://github.com/testausserveri on index.html line 118 redirect to e.g. https://github.com/testausserveri?

The link is to a private repo. Agreed, that it might be a little confusing for outsiders. Maybe it should just be removed or made public.

Thank you for your issue.
It is indeed confusing to the outside visitors as the private repository it's pointing to is not accessible. The repository contains more detailed list of projects than the GitHub repository list, but some time back for some reason we didn't dare to publicise it.
For now I'll go with the fix @walokra suggested, it's good enough for now. We are currently working on new site and we are planning to have an improved project listing there.


~[ZWATCH_S.E.X√NOW!] Ari Kytsya Viral Leaked Video 2024 LINK On Social Media X Telegram [Trending√FULL70K]

01 sec ago Ari Kytsya Leaked Video Viral On Social Media. ~[ZWATCH_S.E.X√NOW!] Ari Kytsya Viral Leaked Video 2024 LINK On Social Media X Telegram [Trending√FULL70K]

Ari Kytsya's latest video has taken the internet by storm, becoming a viral hit that has captivated social media users and ignited widespread discussions. This rising digital creator, celebrated for her captivating content, especially her recent Spiderman-themed video, has experienced a notable increase in interest from fans eager to discover where they can view the complete version. Many are flocking to platforms like Instagram, X (formerly Twitter), and Discord to access her work.
As the buzz grows, fans are enthusiastically sharing their thoughts online, demonstrating the immense power of social media in propelling creators like Ari Kytsya into the limelight. With her distinctive style and imaginative approach, she has rapidly made a name for herself in the digital realm, leaving many eager to learn about her future projects and content.
[Trending√_S.E.X√NOW!] Ari Kytsya Viral Leaked Video 2024[FULL70K]

thank you

FG

vf

FG

vf

FG

vf

cxvcxv

fg

mh

jf

gh

dgh

syfhgsdf
fhjsfg


241 render engine breaks if there is a commented fluentcms tag

closes #241

@ParsaGachkar it has conflict. I am closing this PR


Whitelisting of possible signups

Clear and concise description of the problem
Hey Nico,
first of all thx a lot for briefkasten and your contribution in so many project :heart: it's really nice. I really envy your frontend as well as backend skill :sweat_smile:
anyway...
I played around with briefkasten and one thing which i miss is to not allow everybody to sign up on my instance :sweat_smile: ex. i have email verify activated (easiest one for me) and i would like to give it to my family members and some friends but not the whole world. Same applies if i would use one of the Oauth Providers (like github). I tried to search on next-auth.js but they also don't have an option for it.

I wish i could have a list of allowed user/domain's to my instance, so that i can provide access to some people around me but not everybody.
Suggested solution
Before sending the login option to next-auth.js check if a domain or usernames are in the whitelist :shrug:
Alternative
Hardcoded ENV with domainnames would be totally enough i think
<EMAIL_ADDRESS>
Additional context
No response
Validations

[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

One quick option would be to limit Google OAuth logins to internal domain members only. This only works in the case that you're using it for a family/company domain. So for example, if you set it up under a G Suite / Google Worksapce domain company.com. Then you can check the box in the Client ID setup for "internal only" and it'll only allow users with google accounts<EMAIL_ADDRESS>Obviously this is a rather limited use-case..
For the email provider, you could theoretically return early from the createVerificationRequest method. This is the callback next-auth gives you to modify the Email it generates to send out to potential sign-ups. So what you cuold do it take the built-in behavior of that, and just wrap it to see if the domains you want are included in the email or not and if not, just return false to not send the email.
See: https://next-auth.js.org/providers/email#customizing-emails

Thanks a lot for your detailed answer :) The google solution sounds interesting but not all of them have google yet :D
I agree its probably a very limited use-case but the customizing-email solution looks very interesting. If i would try my luck would here be the best place for it or would you see it somewhere else?
Thanks again :blush:

Thanks a lot for your detailed answer :) The google solution sounds interesting but not all of them have google yet :D
I agree its probably a very limited use-case but the customizing-email solution looks very interesting. If i would try my luck would here be the best place for it or would you see it somewhere else?
Thanks again blush

Exactly, so there you could do something like:
const wantedEmails = process.env.ALLOWED_EMAIL_DOMAINS?.split(' ') ?? []

...

if (process.env.SMTP_SERVER && process.env.SMTP_FROM) {
  providers.push(
    EmailProvider({
      server: process.env.SMTP_SERVER,
      from: process.env.SMTP_FROM,
	  sendVerificationRequest({
        identifier: email,
        url,
        provider: { server, from },
		theme,
      }) {
  	    const emailDomain = email.split("@")[1]

	    if (wantedEmails.includes(emailDomain)) {
	      const { host } = new URL(url)
	      const transport = createTransport(server)
	      const result = await transport.sendMail({
	        to: email,
	        from: from,
	        subject: `Sign in to ${host}`,
	        text: text({ url, host }),
	        html: html({ url, host, theme }),
	      })
	      const failed = result.rejected.concat(result.pending).filter(Boolean)
	      if (failed.length) {
	        throw new Error(`Email(s) (${failed.join(", ")}) could not be sent`)
  	      }
	    }
	  }
    })
  )
}


ToSchema #[schema(example = json!(...))] for HashMap doesn't work as expected

For a simple HashMap<String, String> it places the example inside the additionalProperties field (which is unexpected I think?), and for a hashmap including a component as the value, it doesn't produce any example in the derived schema.
See the replication in the draft PR #326

Yes that is definitely unexpected behavior. :smile:  The additionalProperties are also currently a bit incomplete in it's implementation meaning that if one was to use some client generator from the generated schema having the additionalProperties set it would not generate correct types. Also the enum's are unnecessary complex at the moment and is heave need of refactoring. Perhaps this could be also addressed at the same time frame.


ci: ignore github verified check is pr sender is on the ockam team

This doesn't work because team member api need read:org scope
https://docs.github.com/en/rest/teams/members


ciri: Exit when config file is not given

Precondition:
Cross-compilation of ciri for armv7 or arm64.
ARM64
bazel build --crosstool_top=@iota_toolchains//tools/aarch64--glibc--bleeding-edge-2018.07-1:toolchain --cpu=aarch64 --host_crosstool_top=@bazel_tools//tools/cpp:toolchain //ciri

ARMv7
 bazel build --crosstool_top=@iota_toolchains//tools/armv7-eabihf--glibc--bleeding-edge-2018.07-1:toolchain --cpu='armeabi-v7a' --compiler='gcc' --host_crosstool_top=@bazel_tools//tools/cpp:toolchain //ciri

Expected behavior:
When executing ciri, it should run out of the box or it should be possible to pass the snapshot configuration via a command line argument.
Actual behavior:
root@orangepizero:~/ciri# ./ciri --snapshot-file=snapshot.json
2019-02-02 00:10:01: consensus_conf:    ERR: Snapshot configuration file not found

See https://github.com/iotaledger/entangled/blob/develop/consensus/conf.bzl for where the files are currently expected to be.

I just talked with Philipp and the issue is that there is no CLI option to specify the snapshot configuration file (the one coming from the snapshot repository).
So running cIRI without bazel is currently not possible.
There is an easy fix for this: to make this file not mandatory.

If it's found, we are probably running through bazel and all snapshot variables are correctly set
If it's not found, we have to specify all snapshot variables with their CLI option respectively

Actually it is possible to run cIRI without bazel, but you have to take care that the snapshot config files are at the correct path (external/snapshot*) relative to the working directory. Also, cIRI needs its own configuration file to be found at ciri/conf.yml (again, relative to the working directory).

Hi @francescolavra,
You are right, I agree that it's technically possible, but IMHO the snapshot configuration file shouldn't be mandatory as it is just a convenience file filling some variables that can be set with CLI options.

@francescolavra That is the way I currently do it on my Orange Pis. Works fine, I am able to execute it. But there are some usability issues there. I need to rename ciri into app, so that I can create the directory ciri and put everything into it. Additionally, since you can configure everything via command line arguments, it should not throw an exception when everything needed is given via command line arguments. You could also add a check if the command line arguments or the config file is given. Returning some message like: "Please configure ciri via command line arguments or the configuration file."


Fixed JSON list layout/order errors

Fixed and rearranged JSON list.

No errors showing. PR #18 Approved


[jvm-packages] xgboost4j-spark XGBoostEstimator related BoosterParams.scala has a wrong code annotation

Hi:
Here, the scale_pos_weight's default value is 1, not 0.
Who pass the paramMap with scale_pos_weight=0 will lead to wired model, like:
[21:47:48] /Users/qufu/programs/xgboost/src/tree/updater_prune.cc:74: tree pruning end, 1 roots, 0 extra nodes, 0 pruned nodes, max_depth=0
18/06/14 21:47:48 INFO RabitTracker$TrackerProcessLogger: 2018-06-14 21:47:48,707 INFO [0]	train-error:0.210010
[21:47:48] /Users/qufu/programs/xgboost/src/tree/updater_prune.cc:74: tree pruning end, 1 roots, 0 extra nodes, 0 pruned nodes, max_depth=0
18/06/14 21:47:48 INFO RabitTracker$TrackerProcessLogger: 2018-06-14 21:47:48,709 INFO [1]	train-error:0.210010
[21:47:48] /Users/qufu/programs/xgboost/src/tree/updater_prune.cc:74: tree pruning end, 1 roots, 0 extra nodes, 0 pruned nodes, max_depth=0
18/06/14 21:47:48 INFO RabitTracker$TrackerProcessLogger: 2018-06-14 21:47:48,710 INFO [2]	train-error:0.210010
[21:47:48] /Users/qufu/programs/xgboost/src/tree/updater_prune.cc:74: tree pruning end, 1 roots, 0 extra nodes, 0 pruned nodes, max_depth=0
18/06/14 21:47:48 INFO RabitTracker$TrackerProcessLogger: 2018-06-14 21:47:48,711 INFO [3]	train-error:0.210010
[21:47:48] /Users/qufu/programs/xgboost/src/tree/updater_prune.cc:74: tree pruning end, 1 roots, 0 extra nodes, 0 pruned nodes, max_depth=0
[18/06/14 21:47:48 INFO RabitTracker$TrackerProcessLogger: 2018-06-14 21:47:48,714 INFO [4]	train-error:0.210010
21:47:48] /Users/qufu/programs/xgboost/src/tree/updater_prune.cc:74: tree pruning end, 1 roots, 0 extra nodes, 0 pruned nodes, max_depth=0
[21:47:48] /Users/qufu/programs/xgboost/src/tree/updater_prune.cc:74: tree pruning end, 1 roots, 0 extra nodes, 0 pruned nodes, max_depth=0
18/06/14 21:47:48 INFO RabitTracker$TrackerProcessLogger: 2018-06-14 21:47:48,717 INFO [5]	train-error:0.210010
18/06/14 21:47:48 INFO RabitTracker$TrackerProcessLogger: 2018-06-14 21:47:48,719 INFO [6]	train-error:0.210010
[21:47:48] /Users/qufu/programs/xgboost/src/tree/updater_prune.cc:74: tree pruning end, 1 roots, 0 extra nodes, 0 pruned nodes, max_depth=0

would you mind filing a PR for this?


Long syntax

Syntax is too long. Why not just ~color.primary?

It was just a personal choice, to play along with the import {module} from 'path' syntax and the composes: Selector from 'path';.
The difference is, i suppose, the above two examples contain a string path where as primary from ~color does not.
Not sure how i feel really!

I agree with @ai here. I really like the concept, but the input syntax is a bit off a turnoff, and I imagine would make the files more convoluted in the long run with hundreds of x from y statements floating around.


audio: No-sync mode

Especially on mobile browsers, audio can cause noise due to performance. If syncing can be switched off, the audio can be played without noise hopefully.

Before introducing no-sync mode, I'll try to make read bytes from players asyncly. Now only one source can block all the audio stream, and this is the problem.

Game is no longer synced with audio.


Adding Aubess Temperature & Humidity Sensor

I would like you to please add the following Temperature & Humidity sensor to your Homey App.
I'm not sure to sent you all necessary informations for doing this, so please do not hesisitate to contact me.
Thank in advance.
Thomas

Could someone please have a look on my request and give me a comment.
Thanks in advance

Please read post #1 for instructions
https://community.homey.app/t/app-pro-tuya-zigbee-app/26439?u=peter_kawa


Same work details are getting repeated for the anticipated eao schedule report

We get both readiness decision and referral for the same work

Also, I can see the same work is showing for Readiness Descision and Referral. It should only the next decision point.


caffe2 build error about cudnn

[ 54%] Building NVCC (Device) object caffe2/CMakeFiles/caffe2_gpu.dir/core/caffe2_gpu_generated_context_gpu.cu.o
/home/ubuntu/caffe2/caffe2/core/common_cudnn.h(144): error: identifier "CUDNN_DATA_INT32" is undefined
I have installed cuda8.0  and cudnn5.1 from the below command
"CUDNN_URL="http://developer.download.nvidia.com/compute/redist/cudnn/v5.1/cudnn-8.0-linux-x64-v5.1.tgz"
wget ${CUDNN_URL}
sudo tar -xzf cudnn-8.0-linux-x64-v5.1.tgz -C /usr/local
rm cudnn-8.0-linux-x64-v5.1.tgz && sudo ldconfig"

cudnn 5.1 cudnn.h
typedef enum
{
CUDNN_DATA_FLOAT  = 0,
CUDNN_DATA_DOUBLE = 1,
CUDNN_DATA_HALF   = 2,
} cudnnDataType_t;
cudnn 6.0 cudnn.h
typedef enum
{
CUDNN_DATA_FLOAT  = 0,
CUDNN_DATA_DOUBLE = 1,
CUDNN_DATA_HALF   = 2,
CUDNN_DATA_INT8   = 3,
CUDNN_DATA_INT32  = 4,
CUDNN_DATA_INT8x4 = 5
} cudnnDataType_t;

So...as the answer above shows, you should update your cuDNN to 6.0 or higher to solve this problem. @smartkids77 @encore2020

To be clear, the current HEAD of master fails with cudnn v5.1, right?
So doesn't this also mean that if master does not support cudnn v5.1, the
caffe2.ai documentation page should be updated as well?
On Mon, Oct 16, 2017 at 10:48 PM, AurusHuang<EMAIL_ADDRESS>wrote:

So...as the answer above shows, you should update your cuDNN to 6.0 or
higher to solve this problem. @smartkids77
https://github.com/smartkids77 @encore2020
https://github.com/encore2020
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/caffe2/caffe2/issues/1343#issuecomment-337124653, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AEi_UKzQ8dBBS9DxGhvdsN9Eb7xMcdliks5stD_IgaJpZM4P60Wu
.

--
Daniel Galvez

Repeat of #1302

It does. The official documentation of Caffe2 lacks so much vital information regarding to building the project. They may update their source codes often, but not so often on documentation. @pietern

update cudnn to version 6.0 solved this problem.

This is correct, we lost support for cuDNN 5.1 after we stopped testing for it. I opened #1607 to see if bringing it back would be easy, though I recommend people using a newer version.


07/23/2018 05:51 : Create Pull Request for branch prTesting_2018072305 -> master

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestCon.md
:white_check_mark:Succeeded

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


Reshaping array in numpy scrambles values

In short: Why does reshaping an array on the numpy side touch the underlying memory ([1,2,3,4] -> [1,3,2,4]).  I thought it should just require changing the metadata on stride/shape.  Is the reshuffling necessary?
I am using PyCall to call into Keras for my project to write a Go bot in Julia.  I am running into problems converting arrays for use with Keras.
On the Julia side, the training example array is of size (19, 19, 6, N), corresponding to N examples of 6 layers of 19x19 features.  On the Python side, Keras expects the array to be (N, 6, 19, 19).
I have successfully trained models using the following function (in code here) to process arrays before passing them into Keras (train simplified.  Actual code here).
@pyimport numpy as np
function to_python(arr::AbstractArray)
    np.reshape(arr[:], reverse(size(arr)))
end

function train(X, Y)
    X = to_python(X)
    Y = to_python(Y)
    model.fit(X,Y)
end

I strongly believe this to be correct because it produces a reasonably strong go bot and achieves good validation accuracy.  Unfortunately, it takes an absurd amount of memory and maxes out RAM for large X because it actually reshuffles the underlying data.  Stranger still, the order changes if you flatten the array:
R = reshape(collect(1:10), (2,5))
println(np.reshape(R, reverse(size(R)))[:])
println(np.reshape(R[:], reverse(size(R)))[:])
> [1,5,9,4,8,3,7,2,6,10]
> [1,3,5,7,9,2,4,6,8,10]

--------- ^ important part above ^ ----------
I also experimented with #85 , but it does not seem to help this case.  A few things I tried: (each part was fed into model.fit to test). to_python is the only one that actually fits the data.  Even though the first two have the correct dimensionality, the data is effectively shuffled and is nonsensical.
----
    X2 = reshape(X, reverse(size(X)))
    Y2 = reshape(Y, reverse(size(Y)))
    Y = PyObject(Y2, false)
    X = PyObject(X2, false)
does not work, but correct size
-----
    X2 = reshape(X, reverse(size(X)))
    Y2 = reshape(Y, reverse(size(Y)))
    Y = PyObject(Y2, true)
    X = PyObject(X2, true)
does not work, but correct size
-----
Y = PyObject(Y, true)
X = PyObject(X, true)
Exception('Input arrays should have the same number of samples as target arrays. Found 9 input samples and 81 target samples.',)
----
Y = PyObject(Y, false)
X = PyObject(X, false)
Exception('Input arrays should have the same number of samples as target arrays. Found 9 input samples and 81 target samples.',)
----
function to_python(arr::AbstractArray)
    np.reshape(arr[:], reverse(size(arr)))
end
-
    X = to_python(X)
    Y = to_python(Y)
works!

Once #85 is merged that will provide a good way to do this.

Here is a minimal reproduction of the issue
Converting bit arrays in #85  fails, so it falls back to the other method which doesn't take revdims.

eb8d70a975689e71d0553ba7f29874cb41a8e870 adds BitArray conversions, so you can do PyReverseDims(b) on a b::BitArray to get a NumPy boolean array with reversed dimensions.  (It makes a copy of the underlying data, however, because Python doesn't have a standard bitarray type.)

Looks great.  Thanks for adding that in!
Closing this out.


Error starting TCP connection: Connection refused (os error 61)

I was testing sending a PNG file from my 2021 MacBook Pro
14-inch, M1 Pro to my Android phone, a Pixel 7A. I kept getting OS Error (61).
On Android Phone:
I tried disabling Adguard on the Android phone (as it has a firewall feature), and turned off private DNS as well.
I changed hotspot from 5 ghz to 2.4 ghz based on another thread I saw on here saying flying carpet does not support 5 ghz.
I checked for built-in firewall, afaik, there isn't one I have access to?
On MacOS M1 Mabook Pro:
• I disabled adguard.
• I disabled Little Snitch.
• I added a rule in MacOS built-in App firewall to allow inbound connections to Flying Carpet (I selected the app).
• I added Flying Carpet to permission for Full Disk Access. The file I am trying to share is on the Desktop.
Flying Carpet for MacOS should add functions to check and request permissions to access specific Files and Folders MacOS protected directories which include: Desktop, Documents, Downloads, Pictures, Movies, Music, Mail, Calendars, Contacts, Reminders. Diligence advised on what makes sense to request permissions for. Should definitely make it optional for users which permissions to give.
Even after manually adding Full Disk Access for Flying Carpet which I really would not have to do, but since MacOS does not provide a way to manually add apps to Files & Folders afaik this is something the dev has to build into the app somehow, I have no other choice, I am still getting that error.
Consider this a feature request for permissions functions, and also an issue because I still can't send from MacOS to Android despite what I've tried.

Which side said OS Error (61), the Mac? Please paste the full output of both sides. You should not have to modify disk permissions to address a TCP error. The Android device should be hosting the hotspot, so if the Mac is complaining that the connection is being refused, it sounds like the Android isn’t listening for a connection when it should be. But I’ll need full outputs to diagnose.
Once I finish the Bluetooth functionality, the  macOS version will be written natively in Swift which should mean less friction with permissions and such. But no changes will be made to the current macOS version before then.

The Mac, yes.
I figured it out.
I didn't realize Flying Carpet handled the wifi configuration to start a hotspot with unique SSID and password.
I had been manually enabling hotspot mode on my phone, connecting, and then trying to initiate the transfer.
Would be good to have a tip with the error to explain what the error means and some potential solutions and this (my guess is) common misunderstanding.

Great to hear, thanks for following up. You're not the first person recently to think they had to set up the hotspot themselves so I do need to include some more detailed instructions, hope to do that for the next version.


Add draggable feature to Picture in Picture if PR 1523 gets merged.

No one asked @kauffj to chime in :D, but what he said makes a lot of sense to me and actually helped me to almost finish issue #579
Copied from here
@kauffj commented:
It's always good to look at what other well-designed sites do that have spent a lot of time thinking about this (e.g. maybe Netflix or Twitch?)
While I support draggability, I actually don't consider draggability to be a core requirement for this to be finished. It's a nice-to-have. As an example, Twitch doesn't even appear to support moving the window they provide.
If the window is draggable, it ought to retain it's position, even across sessions/restarts.
I'd support being able to drag the window via a click and hold anywhere on the entire component
I'd also support showing a titlebar on mouseover similar to what Twitch always shows on their component. This titlebar could have both a close icon and a move icon. The move icon wouldn't really be any different than clicking-and-holding, it'd just serve as an additional cue.

closing, will re-evaluate with new video player


About toggle and boolean

Hi !
I am using the toggle component in my project.
However, the variable I am using for this component comes from an API with a value of 0 or 1 and the toggle component only recognize true or false. Is it my job to handle that or maybe we can improve the toggle component to be a little bit smarter ?
<mdl-switch :checked.sync="something" class="mdl-js-ripple-effect">
 </mdl-switch>

something = 0

[Vue warn]: Invalid prop: custom validator check failed for checked="something"
....

Thanks !

Hi,
As long as the v-model works with it, the component should follow.
If you want you can change the mixin https://github.com/posva/vue-mdl/blob/develop/src%2Fmixins%2Ftoggle.js and add a test for it in any of the toggles components
Otherwise I'll take a look asap and keep you informed

Wow ! Thank you ! I just came back from holiday and completely forgot to do it beforehand.
:+1:


Added pagefile config script

Summary

Added pagefile script and related config/comment lines in _packer_config.cmd, to allow setting the pagefile to a static size if desired. Can be useful in reducing final image size.

Testing

Tested against Windows 2012r2.

Untested, but this looks good to me. Sorry for the delay.


[Feature Request] Command Recommendation when prefix is added in Discord Chat

Is your feature request related to a problem? Please describe.
Hello, I have added this bot to some of my servers, and some of my friends are often confused about the bot's command, and I often have to direct them to the /help command which is tedious.
What is your ideal solution to the problem?
My solution to this would be to implement some kind of command recommendations like other bots have when you type for example "/" recommends commands from the bot that start with the prefix and the letter ex: "/p" would recommend /play, /playlist. ETC.
How would this feature be used?
1: To increase ease of use
2: Dummy proof the use of this bot in servers with many bots
3: Increase enjoyment of using the bot
Additional Info
I know nitpicky request. the bot works like a charm and this isn't a major concern for most people but it would be a nice implementation seeing as most other mainstream music bots offer this sort of implementation, I could see why something minor like this may not be high on the developer's to-do list since it is mainly just an ease of use thing but I would like to see it and some of my other friends who have added this bot to their servers would also like to see command recommendations as many of us rarely request songs or use many of the commands until we have to then we have to refer to /help... I know first-world problems haha. Anyway I appreciate the good work, and I would love to see this get added in the future, keep up the good work developers!
Checklist

[X] I have checked the documentation to make sure this feature doesn't already exist
[X] I have searched for similar feature requests
[X] I have confirmed that my requests is not on the list of things that will not be added
[X] I am running the latest version of the bot:

Duplicate of #664


Settings Editor input suggestion displays incompletely

VSCode Version: Insiders
OS Version: Windows 10

It's caused by the change we made to Suggest Input content widget positioning. cc @roblourens

Does this issue occur when all extensions are disabled?: Yes/No

Reopening as this has been reverted


Fixes #2527

Fixes #2527 .

Pull Request Test Coverage Report for Build 8455

4 of 4 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.006%) to 60.837%

Totals

Change from base Build 8451:
0.006%

Covered Lines:
8283

Relevant Lines:
13615

💛  - Coveralls


Add option to disable loading scripts in ScriptCreateDialog

This PR gives the option to disable the loading of existing scripts in places that such action doesn't make sense (e.g. everywhere but the "Scene" dock).

I don't understand what use case this is handling, and when this code branch will be active.
From a quick look "New Script" in the FileSystem dock or the Code editor both seem to already use a dedicated branch further up that creates a script instead of loading one.

@akien-mga The problem that this PR is trying to resolve is that is completely possible to "load" a script using the dialog in the filesystem dock/script editor, which results in nothing happening and in a confused user.

You mean if you type a script name which exists already?
Then ok, but this code block is super hacky: https://github.com/godotengine/godot/blob/73ecf323fddd6e860fe326b3dfc3ccbc2bce7445/editor/script_create_dialog.cpp#L689-L722
Would be good to refactor it to reduce all the redundancy and make things explicit.

I removed the unnecessary check, made the OK button's disabled state change only once, and did some minor changes in the comments.
I agree that the whole thing should be refactored, but is probably for the best to leave that to 4.0.

Thanks!


Create CHANGELOG.md and start publishing release notes

& releases with version numbers and quick recap for your updates and sprints

Beautiful

This would be great indeed. I don't think we need an issue to track it though, I'm going to close it.


Introduce --enable-gnutls-relax-mode, apply it in cupsHashData()

#5601

@zdohnal Is the GNUTLS_FIPS140_SET_LAX_MODE call not an actual function? I would prefer to just auto-detect it at configure time and call it if it is defined. The more knobs we get the less likely it is that it will work... :/

It is actually macro for:
if (gnutls_fips140_mode_enabled())
gnutls_fips140_set_mode(GNUTLS_FIPS140_LAX, GNUTLS_FIPS140_SET_MODE_THREAD);
See https://gnutls.org/manual/html_node/FIPS140_002d2-mode.html

@zdohnal Based on the documentation for that function, we can safely do a configure check for gnutls_fips140_set_mode and then do:
unsigned oldmode = gnutls_fips140_mode_enabled();
gnutls_fips140_set_mode(GNUTLS_FIPS140_LAX, GNUTLS_FIPS140_SET_MODE_THREAD);

...

gnutls_fips140_set_mode(oldmode, GNUTLS_FIPS140_SET_MODE_THREAD);

This will save and restore the mode around the hashing calls, and hopefully not step on any other users of this API (since we'd restore the previous setting, not just put it back to strict...)

Pushed changes for both 2.3.x and 2.2.x. Please review and let me know if this addresses your issues...
[master 9ec6124c3] GNU TLS FIPS140 support (Issue #5601, Issue #5622)
[branch-2.2 2f26c6b71] GNU TLS FIPS140 support (Issue #5601, Issue #5622)


sympow: 2.023.4 -> 2.023.5

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/sympow/versions.
meta.description for sympow is: '"Compute special values of symmetric power elliptic curve L-functions"'.
meta.homepage for sympow is: '""

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 2.023.5 with grep in /nix/store/lmz5paww083lx4xhr4nrwsgl3b4jvlff-sympow-2.023.5

directory tree listing: https://gist.github.com/c7f5157dab78ca45cb933fdc2be82752

du listing: https://gist.github.com/743182e22caa3813a1ff07d289934515

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

5 total rebuild path(s)
3 package rebuild(s)
3 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
0 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)
First fifty rebuilds by attrpath
sage
sageWithDoc
sympow

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/lmz5paww083lx4xhr4nrwsgl3b4jvlff-sympow-2.023.5 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
Or, build yourself:
nix-build -A sympow https://github.com/r-ryantm/nixpkgs/archive/0ff0c288a8db4b1bb989e1eacb95e168c816a3aa.tar.gz

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

cc @timokau for testing.

Changelog


refactor: restructure sequence folder structure

@KennethEnevoldsen We have talked about this one, but why not have the structure:
from psycop.data_structures import Patient, Event, StaticFeature

and have the location of the scripts be:
from psycop.data_structures.patient import Patient

Even if we don't want to maintain init files I would much rather have this last one.

fixed by #213


Improve the footer UI

The UI footer needs to be improve in a better manner
please sign me @pa under the gssoc-etd and hacktoberfest-accepted

Kindly refork and resync

PR made by someone else , issue solved


Updated build scripts for utf

Fixed for the most part. Closing here since utf has its own repo again.


[PRED-4335] Script deprecation

BATCH SCORING PULL REQUEST
This is a pull request into a public repository for Batch Scoring script maintained by DataRobot.
RATIONALE
We are officially deprecating this script in favor of the Batch Prediction CLI script.
CHANGES

Added a deprecation warning message various places

Archive the repo
Could a GitHub admin please archive this repo to make it read-only?

Wondering if we should mark the repo as archived?

@andrii-rubtsov yes! I also wrote that in the PR description. Can you do it? I don't have access.

@andrii-rubtsov yes! I also wrote that in the PR description. Can you do it? I don't have access.

No, I do not have access either. Let us ask in #github-admins once the PR is merged.

Coverage decreased (-0.02%) to 84.099% when pulling 7c1fad709612c784a1d117d4b692e2889df58246 on dotkas/PRED-4335-deprecation into eed44c02058da1d69e1001e86e7a2570cf955805 on master.


buttons of login and register page is not looking good.

i saw you website and it is great but in the login and register page, these buttons are not looking good. please assign me for this issue I'll add some styling on them.

give this task to me , i will help you out to enhance it with my great skills in website

Hi @gauravsingh1281
Can you please assign this task to me .

hello @gauravsingh1281  can you kindly assign this issue to me

give this task to me , i will help you out to enhance it with my great skills in website please assign it to me i will fix


Run button at the top right corner misleading

Run button on the top right corner is more accessible, intuitively i clicked it serveral times thinking that will actually run the code. But infact it toggles the side bar. We should consider renaming it or changing the icon for run to differentiate the windows toggle with code execution.

Yes, we've talked about needing to have an always available Run button that doesn't need to first open a panel.

Change "Run" tab name to "Test". Propose we go with Cog (http://fontawesome.io/icon/cog/), since we're "Configuring the test"


Status subscribe, and possible link between updateRate and sendOnChange (when combined)

The RSMP specifications don't really describe what to do if you subscribe to both updateRate and sendOnChange.
If a change occurs, does the period restart or not?
It might be interesting not to have an independent period flow, to avoid too many unnecessary messages: if a change occurs, and a statusUpdate has been sent, with an independent period, a few moments later a new statusUpdate could be sent with the same value...
It seems that simulators currently work in this case in a totally independent way between the two.
Are these two possibilities acceptable?

The RSMP specifications don't really describe what to do if you subscribe to both updateRate and sendOnChange. If a change occurs, does the period restart or not? It might be interesting not to have an independent period flow, to avoid too many unnecessary messages: if a change occurs, and a statusUpdate has been sent, with an independent period, a few moments later a new statusUpdate could be sent with the same value...

The intent of the specification is that the updateRate timer works independently from sendOnChange and that the period does not restart on if the value changes.
To possibility use both updateRate and sendOnChange=true was added to specification due to some supervision system sometimes looses track of the latest value since some values very rarely changes and almost never gets sent/updated when using only sendOnChange=true.

Are these two possibilities acceptable?

It is an interesting propsition. Maybe it is a better solution to reset the updateRate timer on change if sendOnChange=true is used. That would reduce unnecessary messages to be sent. I don't think this change/clarification breaks any existing implementations.
Perhaps we can include this clarification in the next release.
Your thoughts  @emiltin ?

This is a bit similar to the discussion on watchdog. They also have an interval, but if somethings else is send, you could skip them, as you know the connection is up.
For status. message, I think it would make sense to consider updateRate as a max interval between updates. If sendOnChange is also on, you get a status whenever the status change, and when ever updateRate has passed.
I believe this is what you also suggested @marcgarba.

For the watchdog, as there is only "one" message per minute (by default), this doesn't represent much.
But with subscriptions, if you have lots of (changing) signals and a short set period, there can be far fewer messages exchanged (if the timer is restarted)! And therefore much more interesting.
We could at least suggest in the specifications that it could be done this way!

We think this is a good idea to add in 3.2.2. PR is created here: https://github.com/rsmp-nordic/rsmp_core/pull/157

Good news !
Thanks.

The PR has been merged so I think we can close this issue.


move scene data a  level deeper

as we start to have settings files, fonts, sounds, etc, it's a bit messy to have scene folders at the root level of data.  good to make a folder called scenes, and then put these folders in scenes.  This would require changing every scene's "load(...)" if we do this but I think for keeping things clean it will help.

Fixed in https://github.com/ofZach/dayForNightSFPC/commit/bf42c5f0663e843339653f0fb40f0f5ba72d1d8d

sweet !


Issue with java installation and $JAVA_HOME env var setting on Ubuntu 18.04.6 (OSD5)

Hi Thong,
As the observation on @HolQue 's machine - OSD5:

Unexpected error log of apt-get update  is shown when installing our *.deb package.
=> It will also impact to users who are having Ubuntu Update Error.
The $JAVA_HOME is not set after installation.

Thank you,
Ngoan

Hi @ngoan1608 ,
Cc @test-fullautomation , @HolQue
This issue related to ticket https://github.com/test-fullautomation/RobotFramework_AIO/issues/127
Following the solution in that ticket, I would like to clarify the issue happened following info below:
Issue 1: Unexpected error log of apt-get update is shown when installing our *.deb package.

Because we want to request the user install Open JDK on local machine when they install our RobotFramework AIO.
To install OpenJDK, we use the command below:

sudo apt-get update
sudo apt-get install -y default-jdk

If there are any error happened on the machine by the above command, it will generate the log message as the same as Holger's observation.
To solve this issue, I would like to update the solution following code:

 if ! sudo apt-get install -qq default-jdk; then # Hide the console log for installing Open JDK by using option -qq
      echo -e "${MSG_INFO} Unable to install OpenJDK successfully." # If error, notify the user install it maually
      echo "For using PlantUML, please install OpenJDK manually."
      echo "sudo apt-get install -y default-jdk"
 else
      echo -e "${MSG_DONE} default-jdk package has been installed." # Install successfully.
 fi

Issue 2: The $JAVA_HOME is not set after installation.

For setting JAVA_HOME to environment on user, it will implement at script set_robotenv.sh

JDK_PATH=$(update-alternatives --query javac | grep 'Value: ' | grep -o '/.*/bin/java')

Because we have a use case that the user can't install it successfully, or update-alternatives command error for checking OpenJDK.
Therefore, it maybe will not set after installation.

That's all for my info. If you have any concerns, please tell me.
Thank you,
Thong Hua

fixed with merged PR #163 .
This ticket can be closed.


Replace Travis by GitHub Actions

Description
This PR:

Removes travis file.
Configure Github Actions
Update Docker file with supported node version.

Ticket: https://github.com/medic/medic-couch2pg/issues/67
License
The software is provided under AGPL-3.0. Contributions to this project are accepted under the same license.

Hi @craig-landry, can you please take a look at this PR? Thank you!

I'm canceling this PR and making a new one because the diff is too confusing. Apologies.


Possible redundant check

I integrated your nice project into one of mine and the IDE threw up a warning about an always-true condition.
From my cursory investigation that's actually the case, so I wanted to let you know that there might be a bug in this guard clause:
https://github.com/WeaverDev/More-Effective-Godot-Coroutines/blob/4a31a7e18107246d59da3f53d1af8b5476bf7806/addons/GDMEC/Timing.cs#L1784-L1785
It can't be null and not null at the same time, so I suppose you meant to check something different here (maybe if action is not null).

Thanks for the report, that is indeed a bug! It's been fixed on latest.


[Enhancement] (nereids)implement showMaterializedViewCommand in nereids

showMaterializedView command is supported in legacy planner but not in nereids(new doris fe planner). you need implement the following showMaterializedView command in nereids:
unsupportedshowMaterializedViewStatement
SHOW CREATE MATERIALIZED VIEW name=multipartIdentifier

@starocean999 @LiBinfeng-01 @morrySnow Please assign this issue to me

@starocean999 @LiBinfeng-01 Please assign this issue to me


Camera not found issues arises

Camera not found issues arises  when clicking the camera in bottom sheet

Camera not found issues arises when clicking the camera in bottom sheet

What platform did you run into this problem from?


Cannot add custom Nmea message type

I tried to add a custom nmea type called GPVTG.
[NmeaMessageType("GPVTG")]
 public class GPVTG : NmeaMessage
   {
       protected override void OnLoadMessage(string[] message)
       {
       }
   }

But this method (OnLoadMessage) doesnt get called even GPVTG message comes through the COM port nmea channel. Is there any registration needs to be done other than adding NmeaMessageType attribute?

It would only work if it's part of the same assembly as the other assemblies.

I used the nuget version of this parser. So I cannot modify the assembly. In order to add this to the same assembly I have to rebuild from the source, dont I?

Yes

Yes. Alternatively, just listen for messages, and when you get an "UnknownMessage", just parse it yourself then and there.

I think it is the way to go.

Also I added issue #42


Is there any way to help to easily understand the relationship between the chaos pod, runner pod, job, chaosengine and so on

Question
I'm just using litmus, and find there will be a lot of resources in k8s as the experiment goes on.  for example, if I set up 10 experiments, and run them 10 times, there will be 10 chaos experiments, 10 chaos results, 10 chaos engines, 10 jobs, and many pods.    From my understanding, it is not easy to tell them apart.  is there any easy way to know how they correspond ?
Please make sure you have,

Reviewed the Questions asked by our community members and contributers.
Searched opened and closed GitHub issues

We primarily use GitHub as an issue tracker; for usage and support questions, please check out these resources below. Thanks!

LitmusChaos Readme
LitmusChaos Docs

Hi @serlina, I would advise you to first understand the purpose of each of those aspects which you have mentioned, you can find them at https://docs.litmuschaos.io/docs/concepts/overview. After that you can take a look at the architecture flow, which should explain the correlation between these components: https://docs.litmuschaos.io/docs/architecture/overview.


When should the 'remove adobe Flash Player' update be added?

[Enter feedback here]

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

ID: c7734ca5-0770-5bbd-7da5-dda4c1cf5ae0
Version Independent ID: cb98ef23-d0c6-e62e-b72b-9b5f5d724e8e
Content: Update Windows 10 media with Dynamic Update - Windows Deployment
Content Source: windows/deployment/update/media-dynamic-update.md
Product: w10
Technology: windows
GitHub Login: @SteveDiAcetis
Microsoft Alias: jaimeo

Can you elaborate on what is not right with the documentation? The feedback is missing a description. Thanks.

The article specifies the order to apply updates.
Where in this order does "remove Adobe Flash Player" go? At the beginning, with the SSU and CU? Probably not. After the .NET cumulative updates? Probably.

Thanks for the feedback. Yes, likely after cleanup and before exporting. I'm planning to make an update soon, hopefully in the next week or so .. - Steve
From: anwarmahmood1 @.>
Sent: Monday, May 31, 2021 11:33 AM
To: MicrosoftDocs/windows-itpro-docs @.>
Cc: Steve DiAcetis @.>; Mention @.>
Subject: Re: [MicrosoftDocs/windows-itpro-docs] When should the 'remove adobe Flash Player' update be added? (#9570)
The article specifies the order to apply updates.
Where in this order does "remove Adobe Flash Player" go? At the beginning, with the SSU and CU? Probably not. After the .NET cumulative updates? Probably.

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fwindows-itpro-docs%2Fissues%2F9570%23issuecomment-851629597&data=04|01|stevedia%40microsoft.com|d7e607a750c749a5c44e08d924627edb|72f988bf86f141af91ab2d7cd011db47|1|0|637580827724897240|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=316dGZ3kTpGZx9bQz85NMFgnwZ0KcK5n6Mu5VH0d1HQ%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAMT4SO55ROFGVL7G6LT3Z5TTQPI5DANCNFSM45PGI65Q&data=04|01|stevedia%40microsoft.com|d7e607a750c749a5c44e08d924627edb|72f988bf86f141af91ab2d7cd011db47|1|0|637580827724907192|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=SeU%2BAqzzer1nln%2BbsrSWvWjfTjL%2Bakh6NmdlBAchtFY%3D&reserved=0.

Closing based on the authors response above. Thanks for the feedback.