NEXT-12173 - Add customer from admin: sales channel and customer number required

Description:
When adding a new customer in the admin, the fields "Sales channel" and "Customer number" are required, but not marked as such (*).
Steps to reproduce:

Admin > Customers > Overview > Add customer
Fill all fields marked with "*"
Save

Expected result:
Customer saved and created.
Actual result:
"Sales channel" and "Customer number" are required and need to be filled too. They should have an "*" behind their label.
not_empty.jpg

fixed with https://github.com/shopwareBoostDay/platform/pull/252

fixed with https://github.com/shopwareBoostDay/platform/pull/252

duplicates https://github.com/shopwareBoostDay/platform/issues/237

duplicates https://github.com/shopwareBoostDay/platform/issues/237


on('stateChange') sometimes not triggered

Hi,
as in the title, I have to point out that sometimes (1/20 ca), event 'onStateChange' don't be triggered.
In that cases in which event is not triggered, I can run manually sudo service bluetooth start and then the event is triggered.
Does it happen to someone else?
Luca

@velthune do you have steps (and code) to reproduce this?


The pop option doesn't work

The append option works but I think that with the new --rev option available we can dispense with append and pop.

Fixed by removing append and pop options.


useAsyncFn returned asyncFn should throw an error

What is the current behavior?
When I call the wrapped asyncFn, the promise returned will not throw the error. However, sometimes it is common to do some sync jobs after an async job. What is the best practice for this situation?
Reproduce Link
What is the expected behavior?
Should throw an error in useAsyncFn
A little about versions:

OS:  MacOS 10.15.3
Browser (vendor and version): Chrome 79.0.3945.130
React:  16.12.0
react-use: v14.0
Did this worked in the previous package version? No

I don't think asyncFn should throw an error. The purpose of hooks is to update the UI status, and the success or failure of asynchronous function calls is reflected by the status. Throwing errors is imperative and may be you should use native JS function calls.

I also agree that the promise should resolve with the error instead of returning it, for a couple of reasons:

Returning the error can make it difficult to disambiguate between the actual resolved object and a possibly unknown error type.
A callsite that invokes the the wrapped asyncFn absolutely may care if the promise rejected and may need to handle that appropriately.

Will you consider a PR for changing this behavior?

While I do believe the default behavior of useAsyncFn should be to not throw errors as @whinc said. But there are instances that I have faced where as @chardydev says in point 2, that the caller may want to distinguish whether an error was thrown to decide what actions to take after that.
I would propose modification to the options of the useAsyncFn or introduce a new hook altogether. The alternate hook/option would behave such that:

Errors and loading states work as in the current useAsyncFn (for the UI).
Errors should also be thrown so that the callers may catch it themselves (for the caller).

If, presuming that this is what the lib wants to pursue, we go via the options path, then this is what I was thinking it could look like:
type UseAsyncFnOptions<T> = {
  // The default value to use before a value is resolved from the promise.
  defaultValue?: T;

  // If true, the call function will return the value after executing [fn] and rethrow
  // error if caught.
  doNotConsume?: boolean;
};

I think it shoud return { value?: T, error?: any}  for the caller to handle the error correctly

I think it shoud return { value?: T, error?: any} for the caller to handle the error correctly

Typescript cannot infer that an error may be returned, which may lead to incorrect use of the return value, which did happen to me

I just hit this as well! Returning an error object is breaking the contract in Typescript. As such, Typescript is actually preventing me from checking if the function returned an error, because it's not allowed to do that according to its signature.


Added support for feature flags in opensearch.yml (#4959)

(cherry picked from commit 241bd42ab4d0bff0c6d232df0076d2568e326667)

Description
This change introduces a static store "settings"
in FeatureFlags.java file to enable isEnabled method to fetch flag settings defined in opensearch.yml.
Issues Resolved

Addresses this issue #4102

Check List

[x] New functionality includes testing.

[x] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff
[x] Commit changes are listed out in CHANGELOG.md file (See: Changelog)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #5707 (2e93b82) into 2.x (ee33c9e) will decrease coverage by 0.51%.
The diff coverage is 66.66%.

@@             Coverage Diff              @@
##                2.x    #5707      +/-   ##
============================================
- Coverage     71.02%   70.50%   -0.52%
+ Complexity    58984    58559     -425
============================================
  Files          4741     4742       +1
  Lines        280607   280629      +22
  Branches      40894    40897       +3
============================================
- Hits         199295   197871    -1424
- Misses        65099    66290    +1191
- Partials      16213    16468     +255

Impacted Files
Coverage Δ

...a/org/opensearch/test/OpenSearchIntegTestCase.java
54.51% <0.00%> (-2.11%)
:arrow_down:

...pensearch/common/settings/FeatureFlagSettings.java
50.00% <50.00%> (ø)

.../java/org/opensearch/common/util/FeatureFlags.java
80.00% <88.88%> (+30.00%)
:arrow_up:

server/src/main/java/org/opensearch/Version.java
83.40% <100.00%> (+0.07%)
:arrow_up:

...org/opensearch/common/settings/SettingsModule.java
84.45% <100.00%> (+0.32%)
:arrow_up:

server/src/main/java/org/opensearch/node/Node.java
83.33% <100.00%> (+0.02%)
:arrow_up:

...n/indices/forcemerge/ForceMergeRequestBuilder.java
0.00% <0.00%> (-75.00%)
:arrow_down:

.../indices/forcemerge/TransportForceMergeAction.java
25.00% <0.00%> (-75.00%)
:arrow_down:

...port/ResponseHandlerFailureTransportException.java
0.00% <0.00%> (-60.00%)
:arrow_down:

...a/org/opensearch/client/cluster/SniffModeInfo.java
0.00% <0.00%> (-56.25%)
:arrow_down:

... and 486 more

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

Gradle Check (Jenkins) Run Completed with:

RESULT: UNSTABLE ❕
TEST FAILURES:

      2 org.opensearch.snapshots.SegmentReplicationSnapshotIT.testSnapshotOnSegRep_RestoreOnSegRepDuringIngestion

URL: https://build.ci.opensearch.org/job/gradle-check/9023/
CommitID: b5084ee
Please review all flaky tests that succeeded after retry and create an issue if one does not already exist to track the flaky failure.

Another segment replication flaky test. There are multiple test failures which is worrysome. Let's wait on this PR until we identify cause of this flakyness. We are priortizing fixing these tests as part of https://github.com/opensearch-project/OpenSearch/issues/5669.

Another segment replication flaky test. There are multiple test failures which is worrysome. @tan31989 Let's wait on this PR until we identify cause of this flakyness. We are priortizing fixing these tests as part of #5669. Please feel free to pick some if you are interested.

Yes, that makes sense. I believe if there are any fixes to follow up on, I will keep an eye open for merge conflicts if any.

Another segment replication flaky test. There are multiple test failures which is worrysome. @tan31989 Let's wait on this PR until we identify cause of this flakyness. We are priortizing fixing these tests as part of #5669. Please feel free to pick some if you are interested.

Yes, that makes sense. I believe if there are any fixes to follow up on, I will keep an eye open for merge conflicts if any.

Hi @tan31989, thank you for helping on this issue. We analyzed the flaky tests, most of them are failing due to doc count mismatch. There is a PR on the way to fix this. I think we can go ahead and merge this PR. Can you please resolve the merge conflicts and rebase your changes.

Gradle Check (Jenkins) Run Completed with:

RESULT: FAILURE ❌
URL: https://build.ci.opensearch.org/job/gradle-check/9589/
CommitID: 272edee
Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green.
Is the failure a flaky test unrelated to your change?

Segrep test failure testSnapshotOnSegRep_RestoreOnSegRepDuringIngestion. This is already tracked in https://github.com/opensearch-project/OpenSearch/issues/5669. Refiring the gradle check!
REPRODUCE WITH: ./gradlew ':server:internalClusterTest' --tests "org.opensearch.snapshots.SegmentReplicationSnapshotIT.testSnapshotOnSegRep_RestoreOnSegRepDuringIngestion" -Dtests.seed=692E412BA80E078D -Dtests.security.manager=true -Dtests.jvm.argline="-XX:TieredStopAtLevel=1 -XX:ReservedCodeCacheSize=64m" -Dtests.locale=es-NI -Dtests.timezone=America/Kentucky/Monticello -Druntime.java=17

org.opensearch.snapshots.SegmentReplicationSnapshotIT > testSnapshotOnSegRep_RestoreOnSegRepDuringIngestion FAILED
    java.lang.AssertionError: timed out waiting for green state
        at __randomizedtesting.SeedInfo.seed([692E412BA80E078D:46A1E74C1F5EF990]:0)
        at org.junit.Assert.fail(Assert.java:89)
        at org.opensearch.test.OpenSearchIntegTestCase.ensureColor(OpenSearchIntegTestCase.java:1007)
        at org.opensearch.test.OpenSearchIntegTestCase.ensureGreen(OpenSearchIntegTestCase.java:938)
        at org.opensearch.test.OpenSearchIntegTestCase.ensureGreen(OpenSearchIntegTestCase.java:927)
        at org.opensearch.snapshots.SegmentReplicationSnapshotIT.testSnapshotOnSegRep_RestoreOnSegRepDuringIngestion(SegmentReplicationSnapshotIT.java:180)

Thanks @tan31989! This backport is pretty messy for a couple different reasons, so I went ahead and created another attempt here: https://github.com/opensearch-project/OpenSearch/pull/5941

#5941 has been merged in to 2.x. Thanks for your contribution @tan31989!
I am closing this PR. Please re-open if there is any change pending.


Patched Serverless

Removed redundant folder on GFS with incorrect case
-> https://github.com/harness/developer-hub/commit/eabf34c7cdda8831557d7167fc868adcaa7a96b4
Added tile

Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the maintainers.
What Type of PR is This?

[x] Issue
[ ] Feature
[ ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[x] Tested Locally
[x] Optional Screen Shoot.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app


fix: Fixed firehose s3 policy typos

Fixed some typos on kinesis-firehose-stream-role-s3-policy

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


Importing types from module with relative path does not work

Environment

Operating System: Linux
Node Version:     v18.18.0
Nuxt Version:     3.8.0-28292363.6141c08a
CLI Version:      3.9.1-1697114959.393b07e
Nitro Version:    2.7.0-28276432.ae25359
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      imports
Runtime Modules:  -
Build Modules:    -

Reproduction
https://github.com/manniL/nuxt-23733

pnpm i
pnpm dev
Check pages/index.vue and see that addMinutes is any. The code works though

Describe the bug
When using unimport to import exports from date-fns, it is successful but types aren't provided as expected. When trying to reproduce it in plain unimport (playground) though, it resolves as desired.
Apparently this also happens when using luxon 👀
To "not reproduce" in unimport:

Clone unimport
pnpm i
pnpm i -D date-fns
Change the vite config to:

export default defineConfig({
  plugins: [
    vue(),
    unimport.vite({
      dts: true,
      presets: [
        'vue',
        {
          from: 'date-fns',
          imports: ['addMinutes']
        }
      ],
      dirs: [
        './composables/**'
      ],
      addons: {
        vueTemplate: true
      }
    }),
    inspect()
  ]
})

Last step: use addMinutes and see that it is correctly typed
Additional context
unimport.d.ts
export {}
declare global {
  const EffectScope: typeof import('vue')['EffectScope']
  const PascalCased: typeof import('/home/alichter/Programming/unimport/playground/composables/PascalCased')['PascalCased']
  const addMinutes: typeof import('date-fns')['addMinutes']
  // ...
}

.nuxt/types/imports.d.ts
// Generated by auto imports
export {}
declare global {
  const abortNavigation: typeof import('../../node_modules/.pnpm/nuxt-nightly@3.8.0-28292363.6141c08a_nuxt-nightly@3.8.0-28292363.6141c08a_vite@4.4.11/node_modules/nuxt-nightly/dist/app')['abortNavigation']
  const addMinutes: typeof import('../../node_modules/.pnpm/date-fns@2.30.0/node_modules/date-fns/index')['addMinutes']
  // ...
}

It is interesting how the paths are resolved differently. When replacing the above const addMinutes: typeof import('../../node_modules/.pnpm/date-fns@2.30.0/node_modules/date-fns/index')['addMinutes'] with const addMinutes: typeof import('date-fns')['addMinutes'], the type imports work as expected.
Logs
No response

Adding autoimport for zod also does not provide the types.
I can create the zod schemas but I can not access the types.
I assume these are related?
export default defineNitroConfig({
  imports: {
    imports: [
      { from: 'zod', name: 'z' },
    ],
  },
}

// Expected input from the client
const cancellationInput = z.object({
  membershipId: z.string().length(24).cuid2(),
  reason: z.string().min(1).max(255),
  status: z.nativeEnum(statusType).optional(),
});

export type CancellationInput = z.input<typeof cancellationInput>;

Cannot find namespace 'z'.  ts(2503)

Is there any workaround for the zod types to be auto-imported?


Web Browser Scroll Hangs

Hi I have implemented the donut apex chart but due to this my browser window scroll doesn't scroll smoothly instead it feel like it's a forced scroll

Sorry, I have not experienced it in any charts/dashboards.
Can you please create a codepen demo of the behavior?


DllPlugin is exporting an "corrupted" manifest file

Do you want to request a feature or report a bug?
Bug
What is the current behavior?
webpack.DllPlugin is outputing an invalid Json file
If the current behavior is a bug, please provide the steps to reproduce.
i made a minimal repository to demonstrate:
https://github.com/milewski/webpack-dll-plugin-bug
run: npm run build and analyze the dist/vendor-manifest.json, if it doesnt seems broke at the first run , run the same command a few more times..
ultimately it will outputs something like this:
{
  "name": "d951ba6513e1658e7c3a",
  "content": {
    "../node_modules/jquery/dist/jquery.js": {
      "id": 7,
      "meta": {}
    }
  }
}de_modules/webpack/buildin/global.js":{"id":4,"meta":{}},"../node_modules/webpack/buildin/module.js":{"id":5,"meta":{}}}}

What is the expected behavior?
it should generate a valid manifest.. however it is "corrupted"
Please mention other relevant information such as the browser version, Node.js version, webpack version and Operating System.
Windows 10
Node v8.1.3
webpack 3.8.1

Okay my bad.. as Sokra pointed out on slack

This can happen if you write multiple times to a file in parallel. You need to have different manifest files for each entry

adding [name] to the path: [name].manifest.json options will outputs multiple outputs of course...


How do you select the .zim file ?

I tried to use the example shown in iter.py but how do I use the path to my .zim file ? It is not really clear for me.
I used the command : python iter.py path/to/zim and I got an error message. I tried to add the path inside the code but It doesn't work either. Do you have an exemple on how to clearly select the zim file to use the iter.py on ?

Hi, sorry for the late reply.

I used the command : python iter.py path/to/zim

That command should work. Could you share the exception?
This may sound dumb, but have you verified that the path is actually correct? The path argument is passed directly to pythons open function, so pyzim should be capable of opening any file that python can open. Perhaps it's a permission issue? Or something simple to overerlook - like using the wrong type of path separators (e.g. using / instead of \\ on windows)? You can test that python can open a file by running python -c "open('path/to/file.zim', mode='r').close()".
Or do you get some ZIM-related exception (e.g. file incompatible)?
Also, be sure you use python 3 (see python --version, though usually it is invoked via the python3 command).

Hi, no worries. Here's the output of the command.
C:\Users\TR3\Desktop\WIKI\pyzim>python iter.py 'wikipedia_en_100_maxi_2024-04.zim'
Traceback (most recent call last):
  File "C:\Users\TR3\Desktop\WIKI\pyzim\iter.py", line 97, in <module>
    main()
  File "C:\Users\TR3\Desktop\WIKI\pyzim\iter.py", line 36, in main
    with pyzim.Zim.open(zimpath) as zim:
  File "C:\Users\TR3\AppData\Local\Programs\Python\Python310\lib\site-packages\pyzim\archive.py", line 201, in open
    f = open(path, fmode)
FileNotFoundError: [Errno 2] No such file or directory: "'wikipedia_en_100_maxi_2024-04.zim'"

the zim file is inside the same directory as the iter.py
I tried to use python -c "open('path/to/file.zim', mode='r').close()" but it did nothing, using the full path or using \\ but it throwed some more errors.
It might be some dumb issue ^^' I don't really use windows for those kind of thing but I have some issues with Python on my Linux so I didn't have a choice.

File "C:\Users\TR3\AppData\Local\Programs\Python\Python310\lib\site-packages\pyzim\archive.py", line 201, in open
f = open(path, fmode)
FileNotFoundError: [Errno 2] No such file or directory: "'wikipedia_en_100_maxi_2024-04.zim'"

This indicates that python itself can indeed not find the file.

the zim file is inside the same directory as the iter.py

Did you remember to adjust the path so it respects your CWD (meaning that any relative path you pass to the program needs to be relative to the directory you are currently working in, not to iter.py)?
Just to be sure, try the following:

place the ZIM inside any directory (can be the same as iter.py)
ensure your CWD is the same as the directory the ZIM file is located in (on linux, use cd path/to/directory, but you probably already know this.
Confirm that the path is correct - when running ls (on linux, IIRC the windows command would be dir) it should list "wikipedia_en_100_maxi_2024-04.zim" (or whatever ZIM you use) as a file. If not, your CWD is wrong.
run python3 /path/to/iter.py wikipedia_en_100_maxi_2024-04.zim

I tried to use python -c "open('path/to/file.zim', mode='r').close()" but it did nothing, using the full path or using \ but it throwed some more errors.

If it threw some exception, that would indicate that this issue is unrelated to pyzim. If it did not raise any exception, then python can access the file and the problem must be with the way the path is being passed to open().

No such file or directory: "'wikipedia_en_100_maxi_2024-04.zim'"

Actually, I just noticed the that the file name in the error message uses both single and double quotation marks. This indicates that python believes the ' to be part of the filename. Have you tried it without the quotation marks?

I don't really use windows for those kind of thing but I have some issues with Python on my Linux so I didn't have a choice.

Have you tried a python virtual environment? These are very easy to setup isolated python environments that can avoid dependency related problems and may help in your case:

apt-get python3-venv (or similiar)
python3 -m venv path/to/some/directory
source /path/to/some/directory/bin/activate
Now python3 and pip use an isolated environment

I tried every step and this is what I got in the end :
(myenv) casper@casper-desktop:~/pyzim$ python3 iter.py wikipedia_en_100_maxi_2024-04.zim
Traceback (most recent call last):
  File "/home/casper/myenv/lib/python3.11/site-packages/pyzim/archive.py", line 862, in get_entry_by_full_url
    location = self._url_pointer_list.get(full_url)
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/casper/myenv/lib/python3.11/site-packages/pyzim/pointerlist.py", line 408, in get
    return self._pointers[self.get_index(key)]
                          ^^^^^^^^^^^^^^^^^^^
  File "/home/casper/myenv/lib/python3.11/site-packages/pyzim/pointerlist.py", line 428, in get_index
    raise KeyError("No pointer matching key '{}' found!".format(key))
KeyError: "No pointer matching key 'b'Xlisting/titleOrdered/v1'' found!"

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/casper/pyzim/iter.py", line 97, in <module>
    main()
  File "/home/casper/pyzim/iter.py", line 36, in main
    with pyzim.Zim.open(zimpath) as zim:
         ^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/casper/myenv/lib/python3.11/site-packages/pyzim/archive.py", line 202, in open
    archive = cls(f, mode=mode, policy=policy)
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/casper/myenv/lib/python3.11/site-packages/pyzim/archive.py", line 135, in __init__
    self._load_pointerlists()
  File "/home/casper/myenv/lib/python3.11/site-packages/pyzim/archive.py", line 322, in _load_pointerlists
    self.get_entry_by_full_url(constants.URL_ARTICLE_TITLE_INDEX).read(),
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/casper/myenv/lib/python3.11/site-packages/pyzim/archive.py", line 864, in get_entry_by_full_url
    raise EntryNotFound("No entry for full URL '{}'".format(full_url))
pyzim.exceptions.EntryNotFound: No entry for full URL 'Xlisting/titleOrdered/v1'

So, great news:

this is a different  (and unrelated) error than the one you've encountered first, it seems like we've sorted that problem out
this new error has already fixed seven months ago - seems like I've forgotten to push the newest version to pypi. Sorry about that, totally my fault. I'll try to push the new version sometime later. Until then, you can install the version directly from github (either clone the repo or use pip install git+https://github.com/IMayBeABitShy/pyzim.git). I've just tested it with wikipedia_en_100_maxi_2024-04.zim, so I am fairly sure that it should work.

Thank you so much ! I was almost going crazy.


RDK-33287: Stop Maintenance API

Reason for change: Abort the maintenance task if an
api call to thunder plugin.
Test Procedure: Build and verify
Run all end to end scenarios.
Risk: High

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 1 file pending identification.

Protex Server Path: /home/blackduck/github/rdkservices/1656/rdkcentral/rdkservices

Commit: 06e4ad482d05c7a86863caf8bd351ea01eb3e6fc

Report detail: gist

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 1 file pending identification.

Protex Server Path: /home/blackduck/github/rdkservices/1656/rdkcentral/rdkservices

Commit: b267455529c4d50f9fa00531dfc76c9b7dbf8f02

Report detail: gist

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 1 file pending identification.

Protex Server Path: /home/blackduck/github/rdkservices/1656/rdkcentral/rdkservices

Commit: 2a31def9fea5f92cbd92778390206efadd1cd45b

Report detail: gist

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 1 file pending identification.

Protex Server Path: /home/blackduck/github/rdkservices/1656/rdkcentral/rdkservices

Commit: 2a31def9fea5f92cbd92778390206efadd1cd45b

Report detail: gist

Hi @livinsunny519
MaintenanceManager/MaintenanceManager.cpp:findProcessPid is not original code.  I think the original source is http://proswdev.blogspot.com/2012/02/get-process-id-by-name-in-linux-using-c.html and this is a problem as there is no permission to use (open to correction).  Comcast Legal has prevented such code from being used before.  The code is a straight copy and must be rewritten.  However, this is a pretty standard way to read the proc file, so it is acknowledged that rewritten code will have to do similar actions.  But comments and variable names must not be identical, or layout as much as possible.
Thanks,
Martin Hughes RDK CMF Compliance Team

This is just for checking the code logic.. This will be removed.

Thanks Sunny


Add to collections sort by

Sort by

biggest collections by assets
most buys
most owners
volume
floor price

Hints are in series insight https://deploy-preview-1775--koda-nuxt.netlify.app/series-insight/
Probably would require a write resolver to https://github.com/kodadot/rubick
Ref

#1357

@yangwao, do you mean to remove the sorting functionality as a separate button instead of how it is currently implemented? Presently, you click on each column header to toggle the sort function.

@yangwao, do you mean to remove the sorting functionality as a separate button instead of how it is currently implemented? Presently, you click on each column header to toggle the sort function.

Probably you can remove that one that doesn't work well.
Seems interacted and long time no interaction works. old and new first as well
From most expensive and from cheaper doesn't work

@yangwao, do you mean to remove the sorting functionality as a separate button instead of how it is currently implemented? Presently, you click on each column header to toggle the sort function.

Probably you can remove that one that doesn't work well. Seems interacted and long time no interaction works. old and new first as well From most expensive and from cheaper doesn't work
Notice that #1357 is related issue so you can make it as one PR as you can check on that one

hello @yangwao can I do this issue partially? I was only able to remove "expensive" and "cheaper" from the collection page.
I have no idea how to exclude empty array (nodes) from graphql or how to update the API (resolver). need learning more about it

@yangwao, do you mean to remove the sorting functionality as a separate button instead of how it is currently implemented? Presently, you click on each column header to toggle the sort function.

Probably you can remove that one that doesn't work well. Seems interacted and long time no interaction works. old and new first as well From most expensive and from cheaper doesn't work
Notice that #1357 is related issue so you can make it as one PR as you can check on that one

hello @yangwao can I do this issue partially? I was only able to remove "expensive" and "cheaper" from the collection page. I have no idea how to exclude empty array (nodes) from graphql or how to update the API (resolver). need learning more about it

Sure, create an issue with backreference and I can update it here :)


WFLY-5312 component upgrade to Hibernate ORM 5.0.1.Final, change JPA2…

…LCTestCase to use unique timestamp, change ClassFileTransformerTestCase to use unique app name

Hibernate ORM 5.0.1 seems to pass the TCK tests


Bad Performance due to poor types — which can be done much better

The problem
When the list of apis gets bigger, TypeScript gets very slow, and as a result — intellisense becomes very slow. Here's where I think the issue lies:
This is due to the way the types are currently handled in fetchapi.ts. If you take a look, there's a big interface ResponseType, and in the bottom a single type; FetchApi.
This means, for every endpoint we have, it has to go through this giant type (FetchApi) and check statements. This is extremely bad for performance; especially at scale. When I wrote this, I lacked some understanding which resulted in the bad design.
Proposed solution
Using the zero-api functions ex. import { Ok } from 'sveltekit-zero-api/http', these functions should be typed instead.
Instead of checking whether an endpoint returns a specific status-code (as it's currently doing), the return type should just be typed. That way, it doesn't have to do any checking.

This is an urgent problem that halts productivity. I will get on to it as soon as possible. Unfortunately am caught up in some work for at least a week, so the fix will hopefully arrive early July.
On behalf of past-me, I apologize if this caused any trouble🕺

Bad health has prevented me to assess this issue and the August SvelteKit updates. I'm expecting to push an update within a couple of days.

Currently finishing up the Wiki to be updated for latest content.


Initialize method  does not work as it is

Initialize method  does not work as it is, as ID column in Course table is an identity column, it does not accept values when record is added. correct code block is attached here:
var courses = new Course[]
{
new Course{Title="Chemistry",Credits=3},
new Course{Title="Microeconomics",Credits=3},
new Course{Title="Macroeconomics",Credits=3},
new Course{Title="Calculus",Credits=4},
new Course{Title="Trigonometry",Credits=4},
new Course{Title="Composition",Credits=3},
new Course{Title="Literature",Credits=4}
};
...
var enrollments = new Enrollment[]
{
new Enrollment{StudentId=students[0].ID,CourseId=courses[0].ID,Grade=Grade.A},
new Enrollment{StudentId=students[0].ID,CourseId=courses[1].ID,Grade=Grade.C},
new Enrollment{StudentId=students[0].ID,CourseId=courses[3].ID,Grade=Grade.B},
new Enrollment{StudentId=students[1].ID,CourseId=courses[2].ID,Grade=Grade.B},
new Enrollment{StudentId=students[1].ID,CourseId=courses[3].ID,Grade=Grade.F},
new Enrollment{StudentId=students[1].ID,CourseId=courses[4].ID,Grade=Grade.F},
new Enrollment{StudentId=students[2].ID,CourseId=courses[1].ID},
new Enrollment{StudentId=students[3].ID,CourseId=courses[4].ID},
new Enrollment{StudentId=students[3].ID,CourseId=courses[3].ID,Grade=Grade.F},
new Enrollment{StudentId=students[4].ID,CourseId=courses[3].ID,Grade=Grade.C},
new Enrollment{StudentId=students[5].ID,CourseId=courses[4].ID},
new Enrollment{StudentId=students[6].ID,CourseId=courses[6].ID,Grade=Grade.A},
};

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

ID: 7a54c97f-ddd4-b041-5e5f-a2df009ace50
Version Independent ID: fa488196-1f98-08ea-adcc-17d9dac86727
Content: Razor Pages with Entity Framework Core in ASP.NET Core - Tutorial 1 of 8
Content Source: aspnetcore/data/ef-rp/intro.md
Product: aspnet-core
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

@PeterLuZC  did you miss this?
        [DatabaseGenerated(DatabaseGeneratedOption.None)]
        public int CourseID { get; set; }

using System.Collections.Generic;
using System.ComponentModel.DataAnnotations.Schema;

namespace ContosoUniversity.Models
{
    public class Course
    {
        [DatabaseGenerated(DatabaseGeneratedOption.None)]
        public int CourseID { get; set; }
        public string Title { get; set; }
        public int Credits { get; set; }

        public ICollection<Enrollment> Enrollments { get; set; }
    }
}


Add reference samples for full body-complex swagger

This includes references for the following groups:

Array
Basic
Dictionary
Flatten
Inheritance
Polymorphic recursive
Primitive
Readonlyproperty

This is mostly complete.  Flattening is not applicable and R/O is tracked in a separate issue.


self.health is updated too late after triggering a mine

This issue is described in this forum post:
http://discourse.codecombat.com/t/problem-with-self-healing-minesweeper/2835?u=j_f_b_m

Got it, thanks! Actually the mine was blowing the player up into the air into the coin for another mine, I think.


remove documentation about speciality of setupterm's descriptor, use os.devnull

It's actually perfectly fine to call curses.setupterm(kind, open(os.devnull).fileno()).
More importantly, this removes a chunk of documentation and comments, such as:
            Terminal initialization sequences will be sent to ``stream`` if it
            has a file descriptor and to ``sys.__stdout__`` otherwise. (``setupterm()`` demands to send them
            somewhere, and stdout is probably where the output is ultimately
            headed. If not, stderr is probably bound to the same terminal.)

and
            # (...) Explicit args make setupterm()
            # work even when -s is passed to nosetests. Lean toward sending
            # init sequences to the stream if it has a file descriptor, and
            # send them to stdout as a fallback, since they have to go
            # somewhere.

I'd like to give this more consideration if I ever return my attention to blessings.

oops, sorry to close, didn't see your message last time


Add API to get the Kubeconfig for an arbitrary UCP resource

What this PR does / why we need it
This PR adds API to get the kubeconfig for an arbitrary UCP resource given the UCP context
Which issue(s) this PR fixes

Fixes #
Describe testing done for PR
Unit tests and CI passed

Release note

Add API to get the kubeconfig for an arbitrary UCP resource

Additional information
Special notes for your reviewer

Also Could you add more details on PR description regarding the changes and new APIs. Thanks


bench code

@stjepang I didn't quite do the bench macro the way you said we should do it. Mostly because I forgot what it should be like. Could you maybe share again how it should be? Thanks!

This was addressed in https://github.com/async-rs/async-std/issues/70#issuecomment-528042987


Update build image to Go 1.22.10

What it says on the title. Go 1.22.10 has been released recently, let's go through the usual dance to update to it.

/cc @xrstf


When is the patch updated?

When is the patch updated? Thank you for the answer.

approximately ?

Have patience! 😄 Everything added, I am sure, has to be extensively play-tested before the release of a new patch update.

Thank you for information !

I'm just so excited about this patch that I can't wait for the next one. Sorry for the rush.


Add TCollection

Purpose: make any object work like jQuery object.
Related with or duplicate of #222

Not actual anymore, use functions defined in \Base\autoload.php.


KayakPro reads 2000+ W when I stop paddling

This must be some kind of no data magic value or some other interference of on stage / off stage / pause logic. As soon as I start to paddle the Watts normalize, but the high reading skews the average a little. And calories advance like crazy at 2000+W.

I tried to reproduce this but wasn't able to. I think this might only come if a heart rate monitor is also paired to the app! Need to test that!

I tried with the Wahoo TICKR HRM + KayakPro Genesis Port but I was not able to reproduce the issue. I'll try again next time with the Polar Vantage HRM + Genesis Port (as I experienced the original issue) at some point.


transport: conn, abort OpenShardConn faster

Currently OpenShardConn retries every time it encounters an error, regardless of whether it is context related or not, bloating the test log in case of TestContextsIntegration.
This PR changes the behavior of OpenShardConn, so it checks if context is done before trying to open a new connection, leaving immediately if it's done with appropriate error message.
Relates to #281, solving the context case, another PR will be made resolving the case of whether source port was busy or the error was something else.

I think that something different should be done. In general, we should return every error, unless it's either EADDRINUSE or EPERM. For other errors it doesn't really make sense to retry - for example, in case of a timeout it does not make sense because a different port is not better than any other.
Something like that is already done in gocql, see here: https://github.com/scylladb/gocql/blob/e3e27db4440c6a1f0029322f2bb0e266589d9f0f/scylla.go#L631-L633
I included EPERM because it turns out that on Windows some programs with administrator privileges can reserve ports for themselves and Windows returns a WinApi equivalent of EPERM if you try to use it; we had an issue like that in the rust driver: scylladb/scylla-rust-driver#205

Implemented that as part of #295 bugfix PR. Closing this one its favor.


更新完flutter最新版本，打包报错

../../../../../flutter/.pub-cache/hosted/pub.flutter-io.cn/pull_to_refresh-1.6.4/lib/src/smart_refresher.dart:434:9: Error: No named parameter with the name 'keyboardDismissBehavior'.         keyboardDismissBehavior:
需要回退到1.6.3才能解决

+1

+1

+2

+2

+1

+1

The same build error with flutter 1.22.6

The same build error with flutter 1.22.6

+1

+1

+1

+1

+1

+1

Duplicate #424,If your flutter version is >=2.0.0,use >= 1.6.4.Otherwise use 1.6.3


Whois for .ch domain is not working properly

Since 1 January 2021, personal data associated with registered domain names is no longer disclosed in standard WHOIS queries. This limitation affects many WHOIS clients, including python-whois.
To address this, could you add support for retrieving domain data through RDAP queries? For instance, using an RDAP query (e.g. https://rdap.nic.ch/domain/carini.ch) could provide access to publicly available domain data like Registrar name, Registration date, Nameservers.

That's awesome. I would welcome a PR adding support for RDAP.

I have really no experience in contributing to Github project and a very little proficiency in Python, I can try to bring in contribution but I need some tutoring.

If you create an initial pull request I can give feedback how to improve it.


New version: Stheno v0.2.0

Registering package: Stheno
Repository: https://github.com/willtebbutt/Stheno.jl
Created by: @willtebbutt
Version: v0.2.0
Commit: 3df659e042fff4525d8bf3f688f09426f8fc2f9a
Reviewed by: @willtebbutt
Reference: https://github.com/willtebbutt/Stheno.jl/commit/3df659e042fff4525d8bf3f688f09426f8fc2f9a#commitcomment-34654594

TagBot tag


Sparrow Whirlpool.. Fetching pools timeout

Hello,
there doesn't appear to be any docs/faq about this, or how whirlpool even connects to more easily self-diagnose this issue.
I am running Sparrow version 1.6.4 on linux (Kubuntu 22.04). I have tor running @ <IP_ADDRESS>:9050 with proxy option enabled under electrum server, connected to the public blockstream v3 onion. Tested successfully, everything in sparrow works as it should other than this mixing feature.
When attemping to mix my first time I walk through the introductory screens, set normal for the fee and next screen it'll say Fetching pools... and after some time it'll just give an error. Error fetching pools timeout of 3000ms. I couldn't see anything related to this in sparrow.log, unless I missed it but it's full of other info not releated to this.
Do the public servers such as blockstream have to have support for whirlpool for this feature to work, or is whirlpool an external source not connected through the server manually setup?
It appears as if one time when testing it with the public sever electrum.bitaroot.net, it fetched pools ok, but when I tried again later when atually going to start the mixing it gave timeout again. I haven't tried without tor, as I don't want to cause a leak of IP.
and again, everything else works as usual even after this timeout error for whirlpool.

Connecting to the Whirlpool server is unrelated to your Electrum server configuration, except that if you are connecting to a .onion Electrum server address, Sparrow will use the Whirlpool .onion address. It's likely your problem is Tor related, and I suggest restarting your proxy as a first step.
If you would like to try connecting manually (to test the connection), you can connect to
http://udkmfc5j6zvv3ysavbrwzhwji4hpyfe3apqa6yst7c7l32mygf65g4ad.onion/rest/pools/

@craigraw
I have done basic troubleshooting for this issue, such as restart tor more than just a few times and issue keeps occuring.

If you would like to try connecting manually (to test the connection), you can connect to
http://udkmfc5j6zvv3ysavbrwzhwji4hpyfe3apqa6yst7c7l32mygf65g4ad.onion/rest/pools/

Where do I put this url to test it? In tor browser it just gives 404 error.

When trying to Start Mixing (Postmix - UTXO), I get:
Error connecting to Wirlpool server, will retry soon.....
Sparrow 1.6.4
Ubuntu 20.04
using Private Electrum Server over tor (umbrel node) via internal tor, which works fine.

Where do I put this url to test it? In tor browser it just gives 404 error.

Typically I execute this using cURL configured with a Tor proxy as follows:
❯ curl -x socks5h://localhost:9050 http://udkmfc5j6zvv3ysavbrwzhwji4hpyfe3apqa6yst7c7l32mygf65g4ad.onion/rest/pools
{"pools":[{"poolId":"0.01btc","denomination":1000000,"feeValue":50000,"mustMixBalanceMin":1000170,"mustMixBalanceCap":1009690....

This is the same request that Sparrow makes to fetch the pools.

@craigraw
Executing that curl command outputs (I do have tor running @ localhost:9050):
curl -x socks5h://localhost:9050 http://udkmfc5j6zvv3ysavbrwzhwji4hpyfe3apqa6yst7c7l32mygf65g4ad.onion/rest/pools
{"pools":[{"poolId":"0.01btc","denomination":1000000,"feeValue":50000,"mustMixBalanceMin":1000170,"mustMixBalanceCap":1009690,"mustMixBalanceMax":1019125,"minAnonymitySet":5,"minMustMix":2,"tx0MaxOutputs":70,"nbRegistered":280,"mixAnonymitySet":5,"mixStatus":"CONFIRM_INPUT","elapsedTime":836976,"nbConfirmed":2},{"poolId":"0.001btc","denomination":100000,"feeValue":5000,"mustMixBalanceMin":100170,"mustMixBalanceCap":109690,"mustMixBalanceMax":119125,"minAnonymitySet":5,"minMustMix":2,"tx0MaxOutputs":25,"nbRegistered":302,"mixAnonymitySet":5,"mixStatus":"CONFIRM_INPUT","elapsedTime":74921,"nbConfirmed":2},{"poolId":"0.05btc","denomination":5000000,"feeValue":175000,"mustMixBalanceMin":5000170,"mustMixBalanceCap":5009690,"mustMixBalanceMax":5019125,"minAnonymitySet":5,"minMustMix":2,"tx0MaxOutputs":70,"nbRegistered":150,"mixAnonymitySet":5,"mixStatus":"CONFIRM_INPUT","elapsedTime":1359709,"nbConfirmed":2},{"poolId":"0.5btc","denomination":50000000,"feeValue":1750000,"mustMixBalanceMin":50000170,"mustMixBalanceCap":50009690,"mustMixBalanceMax":50019125,"minAnonymitySet":5,"minMustMix":2,"tx0MaxOutputs":70,"nbRegistered":44,"mixAnonymitySet":5,"mixStatus":"CONFIRM_INPUT","elapsedTime":24725574,"nbConfirmed":2}]}

Ok, so that establishes this machine can connect to the Whirlpool server over Tor. Sparrow is making the same call when it fetches the pools for mixing. Are you still getting the "Fetching pools timeout" error consistently in Sparrow?

@craigraw
Yes, the fetching pools timeout error continues to occur

It is strange that cURL succeeds in making a request that Sparrow cannot. Sparrow's HTTP client is a standard and very well tested software library.
I don't have any great ideas atm why this might be the case. I can suggest trying to see if it happens on another internet connection, or perhaps if it is resolved by using a VPN instead of Tor. You did indicate originally it was intermittent, which points to something outside of Sparrow itself.

@craigraw
Well, now it appears like it works just fine over tor & whirlpool's onion address through sparrow. Fetching pool's only take ~5 seconds. Last few day's, I tried almost every hour it seems so I am not sure why its working now, maybe an update to tor addressed this if it was directly related.
I just hope it stays this way, since I don't know what the issue was exactly.

Ok thanks for the feedback. I'll close this off for now as likely Tor related.


keptn presentation at tag-app-delivery meeting

As part of the review, keptn should be presented in one of the next TAG meetings

We would like to present on August, 4th. Is that possible?

Is there any update on this?

Notes from today's presentation (original):

Proposal for incubation: https://github.com/cncf/toc/pull/670
Links: https://keptn.sh, https://github.com/keptn/keptn
Motivation: Thousands of lines of automation code (shell scripts), different in each team, must be integrated into pipeline runner.
How does Keptn solve? Break big pipelines into many small tasks and abstract these tasks as opaque units; relate tasks and executions together via events and environments.

Opinionated set of common tasks - build, deploy, release, etc.
Abstracts the implementation of a task from its use in a declared pipeline
One task type is to check compliance to SLOs and SLIs - "Quality Gates"

Adopters:

Raiffeisen Software - release validation using quality gates
Vitality - deploy and test shared services in many tenants as quality gates
Kitopi - evaluate resiliency - availability under load and chaos - of deployment

Integration with other CNCF projects

cloud events - all events are structured per this standard
Artifact Hub - publish Keptn artifacts there
Tekton (CDF) - build and deploy with Tekton, coordinate other activities with Keptn
others: LitmusChaos, Argo, Helm, Crossplane, Prometheus, Slack, https://github.com/keptn-contrib, https://github.com/keptn-sandbox

Progress since sandbox (see deck):

implement multi-cluster support - separate control plane from execution plane
implement closed-loop remediation - reconciliation rather than one-off apply

Community

stats - see deck
graduated 3 mentees

resources - https://keptn.sh/resources
roadmap - https://github.com/orgs/keptn/projects/1

plan to integrate CDF SIG Events events (https://github.com/cdfoundation/sig-events)

Q's

API stability, version updates? A: Always provide an upgrade guide, try to avoid breaking changes. A: Deprecated APIs still operate for a while.
Is it for quality assurance, or general build/test/deploy tasks?

Keptn does orchestration based on events, can orchestrate many task runner systems like Tekton or a custom task runner by subscribing to events
Keptn does quality assurance tasks itself, QA is a first-class capability of Keptn

Is this simpler than Argo Workflows, Tekton?

Tekton is not opinionated, Keptn provides opinionated automation sequences
Clarify how to add an integration with a task runner service like Tekton. A: https://keptn.sh/docs/0.8.x/integrations/how_integrate/

Closing this as the meeting is done, ongoing review continues in #131.


fix: Could not find a declaration file for module '@jspm/generator'

Resolves https://github.com/jspm/generator/issues/370

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

Can you please sign the CLA as well?


How to add current date time by default on a table declaration?

Privileged issue

[X] I'm @tiangolo or he asked me directly to create an issue here.

Issue Content
how to add a field in the table with the current date by default?

We've been doing it like this
created_at: datetime = Field(default_factory=lambda: datetime.now())

I do the same as @antont but without the lambda
from sqlmodel import SQLModel, Field
import datetime

class Article(SQLModel, table=True):
    created_at: datetime.datetime = Field(
        default_factory=datetime.datetime.utcnow,
    )

lol I don't know why we had the useless lambda, i guess some mindless copy-paste, thanks, will get to clean those up.

This is the cleanest way I have found to do it
core_models.py
class SimpleIDModel(BaseModel):
    id: int

class UUIDIDModel(BaseModel):
    id: UUID = Field(default_factory=uuid4)

class TimestampModel(BaseModel):
    created_at: datetime = Field(default_factory=datetime.utcnow)
    updated_at: Optional[datetime]

class EventTimestamp(BaseModel):
    occured_at: Optional[datetime]

consuming.py
class Service(SimpleIDModel, TimestampModel, table=True):
    name: str = Field(nullable=False)
    description: str = Field(nullable=False)

I think having opinionated timestamping might not work.
Some people might regard update_at as when a new version of the record was created rather than when the record is updated later on

This is how I use updated_at:
from sqlmodel import SQLModel, Column, Field
from sqlalchemy import DateTime, func
from typing import Optional
import datetime

class Article(SQLModel, table=True):
    created_at: datetime.datetime = Field(
        default_factory=datetime.datetime.utcnow,
    )
    updated_at: Optional[datetime.datetime] = Field(
        sa_column=Column(DateTime(), onupdate=func.now())
    )

Some people might regard update_at as when a new version of the record was created rather than when the record is updated later on

Personally I believe this does not make much sense.

When the record is created at 5PM, it was created_at 5PM.
When the record is updated at 7PM, it was updated_at 7PM.

Just going to throw this out there - created at / updated at / soft deleted at are really common patterns and there might be a bit of value to just make them first class features

How would you do it? Maybe a mixin in some library module, that one can use to add features to base SQLModel?

Yes, I think a mixin might work. I do think it's worth having a discussion how created at / updated at / deleted at would fit with created by / updated by / deleted by (so user IDs, which may be an integer or string type) as well as versioned entities (does created / updated just get replaced with "updated", and "created" is implicit with version 1?)

It's common to delegate generating timestamps to the database rather than generating them in Python. Much like @FilipeMarch is showing in the update_at column.
In that case you would set the server_default attribute on the column. In the case of sqlalchemy it would look like this:
time = db.Column(db.Time, server_default=sqlalchemy.func.now())
I agree it's worth having a firstclass feature for this.

Is the solution in https://github.com/tiangolo/sqlmodel/issues/594#issuecomment-1578962030 the answer? Can the issue be closed?

The advantage of this is I don’t need to refer to an actual sa_column, and using CURRENT_TIMESTAMP adheres to ansi sql standards

wow, amazing, this is works pretty good with alembic's --autogenerate
def upgrade() -> None:
    # ### commands auto generated by Alembic - please adjust! ###
    op.add_column(
        "users",
        sa.Column(
            "created_at",
            sa.DateTime(),
            server_default=sa.text("CURRENT_TIMESTAMP"),
            nullable=False,
        ),
    )

This can be closed.

lol I don't know why we had the useless lambda, i guess some mindless copy-paste, thanks, will get to clean those up.

default_factory= need a callable, so you need to add lambda if you add parentheses at the end of datetime.now()

default_factory=datetime.now


Formatting while typing

This is gonna be a fun issue...
So a lot of editors try to do code formatting while you are typing, and to varying degrees they all kinda suck at it.
While it's an okay experience to format on save or after the fact, it would be nice if things were auto-formatting while typing.
There's a couple messages in the language-server-protocol for "formatting requests":

Document Formatting Request
Document Range Formatting Request
Document on Type Formatting Request

Each of this represents a different opportunity for prettier to fix the formatting of code.
One of the important pieces here will be that it will be possible to have an invalid program which Babylon will currently reject because the user hasn't finished making it valid. Will have to talk to Babylon folks about implementing a "loose" parser which can recover from mistakes (cc @hzoo @danez).
While the language-server-protocol doesn't work everywhere. If more tools like prettier embraced it, it would push more editors to formally implement it, which would be a nice effect.

Fwiw, this is the API that Visual Studio implements and is known to have extremely good formatting while typing facilities.

Actually... It's probably better to do this even before Babel does any error-recovery. If you tried to format an invalid program, you'd end up formatting it incorrectly in a lot of scenarios.
Example:
far (var i = 0; i < 10; i++) {
  // ...
}

This would be parsed as:
- ExpressionStatement
  - CallExpression(far, [], {})
- VariableDeclaration
  - VariableDeclarator(i, 0)
- ExpressionStatement
  - BinaryExpression(i, "<", 10)
- ExpressionStatement
  - UpdateExpression("++", i)
- BlockStatement []

Which would be printed something like this:
far();
var i = 0;
i < 10;
i++;
{

}

That would be really really annoying.
But there are other scenarios where you could get away with it:
try {
  // ...
}

function foo() {
 // ...
}

Here you get a TryStatement that is simply missing it's handler or finalizer. Invalid, but you could probably print it without any issues.
I actually made an exhaustive list of "raised" errors in the parser (this is not every possible error) the other day here: https://github.com/babel/babylon/issues/343. That list would probably also represent errors that prettier could print regardless of. They fall into these two categories:

Errors caused by syntax that needs to be wrapped in other syntax (i.e. return; – can be put inside function)

Not syntax which is invalid due to being wrapped in other syntax (i.e. ({a}) = 2; – would have to be unwrapped)

Errors caused by syntax that is missing a child node (i.e. try {} – catch {} can be added programmatically)

Not when missing a parent node, isn't useful/possible (i.e. class { @foo } – you can't attach the decorator to a method afterwards)

This is something that we should explore at some point but it is not very actionable right now so I'm going to close this issue. If someone is interested in building this, that would be awesome!


C++ date types depend on C++ standard

Yardl defines the yardl::Date type differently depending on the C++ standard used for compilation (std::chrono::local_days for C++20 and newer, date::local_days otherwise).
Yardl officially supports C++17 and C++20, but code generated from the same model and compiled separately for each standard is not compatible.
This becomes a problem when publishing a shared library for a Yardl model. Which C++ standard should be used to build the shared library? If we build and publish using C++20 or newer, users need to compile their client code with C++20 or newer. Likewise for C++17 or older.

Two potential solutions:

Publish two versions of your shared library, one for C++17 and older, the other for C++20 and newer. This is a terrible idea.
Change Yardl to always use HowardHinnant date.h, even if C++20 library features are available. It is already a required dependency in Yardl anyway.

For more context, the two underlying types std::chrono::local_days and date::local_days are actually compatible in-memory, so in many cases, there's no issue implicitly "casting" them if mixing code compiled with C++17 with code compiled with C++20.
Issues do manifest, however, when using a std::optional<Date> (and possibly others like std::variant):
Yardl model:
RecordWithOptionalDate: !record
  fields:
    dateField: date?

ProtocolWithOptionalDate: !protocol
  sequence:
    record: RecordWithOptionalDate?

C++ client executable:
#include "generated/binary/protocols.h"

int main(void) {
  test_model::binary::ProtocolWithOptionalDateWriter w("test.bin");
  test_model::RecordWithOptionalDate rec1;
  w.WriteRecord(rec1);
  w.Close();

  test_model::binary::ProtocolWithOptionalDateReader r("test.bin");
  std::optional<test_model::RecordWithOptionalDate> rec2_opt;
  r.ReadRecord(rec2_opt);
  r.Close();
  if (!rec2_opt.has_value()) {
    throw std::runtime_error("ERROR: Expected to read a valid record");
  }
  return 0;
}

The serialized record is valid, but when it is deserialized, it is a std::nullopt!!


ImportError: cannot import name ‘Sequence‘ from ‘collections‘

Error:
File "G:\research\code\MicroDE_py\plot_bcic_iv_4_ecog_trial.py", line 262, in
    from skorch.helper import predefined_split
  File "C:\Users\Lenovo.conda\envs\braindecode\lib\site-packages\skorch\helper.py", line 6, in
    from collections import Sequence
ImportError: cannot import name 'Sequence' from 'collections' (C:\Users\Lenovo.conda\envs\braindecode\lib\collections_init_.py)
Solution:
Sequence should be imported from collections.abc instead of collections since Python 3.3. A warning is printed starting with Python 3.7 and in Python 3.9 the new import location will be required.( https://github.com/wireservice/agate/pull/737 )
So  from collections import Sequence should be:   from collections.abc import Sequence since Python 3.10.

This is an issue in Skorch not here. Are you using an up to date version of skorch?

I recently ran into the same error and downgraded the python version of my environment to "solve" it.
Both times using the current 0.11 version.
Skorch fixed this issue but they haven't released it yet.

Hello @Guan-Smile!
I was wondering, did the information given by @martinwimpff solve your issue?

I think this bug is caused by unsupported version of pytorch.
the latest version of pytorch that skorch supported is 1.90, older than that in my pc.
The braindecode work perfect after pytorch=1.9 installed


[MM 7970] Support maintaining online status while the Desktop App is in the background ... and other things.

Summary
See Desktop App PR for details.
Related Pull Requests
Desktop: MM-7970
Webapp: MM-7970 (this PR)
Server: MM-14582

Docs covered by another PR


Training Time

Dear Authors,
How long does your method need to train?

We trained our method with resnet50 backbone on four NVIDIA GeForce RTX 3090 GPUs for two days and nine hours.

Thanks. Do you perform training on the work [7] (Towards open-vocabulary video instance segmentation)? If you train, how long does it take?

Sorry, we haven't trained work[7] completely.


Have you prepared for laravel 5.3

5.3 laravel upcoming new and changing many advantages laravel infyom packet matches.

this is the first package i like to see in laravel 5.3 .
waiting to see it working with L5.3

@thandonguocmo2020 @john5db yes, its already on plan as next. Right now, I am working on some major refactoring for relationships and supprot for vuejs templates.
Once, its done. I am planning to have a version for 5.3.

Closing in reference of https://github.com/InfyOmLabs/laravel-generator/issues/258


One more string manipulation library, with UTF-8 safe replacement methods.
-> https://github.com/voku/portable-utf8#class-methods

This change is

Any reason this is better/different from the existing one?

I have bundled many string functions into one simple class via static methods + added missing tests, fixes, documentation + performance improvements, so you can simply switch from an non UTF-8 safe function. Compatible and Tested with php >= 5.3 - 7.1 + HHVM.
Example:
var_dump(str_split('déjà', 2)); // https://3v4l.org/K78c8
var_dump(UTF8::str_split('déjà', 2)); // https://github.com/voku/portable-utf8/blob/master/tests/Utf8StrSplitTest.php

There is already another library we list right above yours. How is your library better than that one?

Nicolas Grekas has merged all logic from "Patchwork-UTF8" into symfony/polyfill + fixes. So I try to use the installed extensions if possible and as fallback we use symfony/polyfill.
Example: UTF8::substr()
-> 1. if "mbstring" is really installed we use it
-> 2. if "iconv" is available we use that
-> 3. otherwise we use a workaround or a polyfill

So then does this new library supersede the patchwork library?

Theoretically, this could be replaced, but there are other parameters, namespaces, class-name etc., so to much BC. And I still use "Patchwork-UTF8" for some projects, some other projects only need "symfony/polyfill" and in the most cases you can use php's native functions. It's only one more option for people who need some special UTF-8 functionality like `````

3 years later :) ... and I am still maintaining this lib. :+1:

Thanks, I've merged this.


unrecognized import path "golang.org/x/net/html"

Command:
 go get -u github.com/PuerkitoBio/goquery

Output:
package golang.org/x/net/html: unrecognized import path "golang.org/x/net/html"

I am confused about this, why it's not working for me?
It's not about my proxy. I can visit google.com, fb etc.

I found a way to work this out.
mkdir -p golang.org/x
cd golang.org/x
git clone https://go.googlesource.com/net


How to write an array of image views to a descriptor set?

Version of vulkano: 0.33

Hi, I'm trying to follow this guide on implementing descriptor indexing in order to bind a large number of textures. I have two questions:

I'm not sure how to bind my textures (which are image views) to the descriptor set.
I tried using  WriteDescriptorSet::image_view_array(2, 0, textures) with textures having the type Vec<Arc<ImageView<ImmutableImage>>>, but I'm getting this error:

How do I obtain the required type from my textures?

How do I enable partially bound descriptor sets in vulkan? The guide mentions the VK_DESCRIPTOR_BINDING_PARTIALLY_BOUND_BIT which you're supposed to add to your VkDescriptorSetLayoutBindingFlagsCreateInfo but I wasn't able to figure out how to set that with Vulkano. The closest I could find was using the closure in  GraphicsPipeline::with_auto_layout, which I'm already using to set some of the other required properties for variable descriptor sets, but I wasn't able to find any flag options there:

        .with_auto_layout(device.clone(), |x| {
            let binding = x[0].bindings.get_mut(&0).unwrap();
            binding.variable_descriptor_count = true;
            binding.descriptor_type = DescriptorType::UniformBufferDynamic;
            binding.descriptor_count = 32;
        })

Thanks in advance!

You can cast the image view: tex as Arc<dyn ImageViewAbstract>
Partially bound descriptor sets are not supported yet in Vulkano.

Thanks! Adding to this, how can you dynamically update the variable descriptor count in PersistentDescriptorSets?
I'm using this WriteDescriptorSet to store my textures:
WriteDescriptorSet::image_view_sampler_array(0, 0, textures)

Then, I create a PersistentDescriptorSet:
                PersistentDescriptorSet::new_variable(
                    descriptor_set_allocator,
                    descriptor_set_layout,
                    var_count,
                    write_desc_set,
                )

which I bind after initializing my pipeline. Now let's say we allow the user to import more textures. How can I update the variable descriptor count at runtime such that the descriptor can fit more textures? I'm a bit confused as to why this value has to be provided in the first place, given it's supposed to be a variable count. :sweat_smile:


go.mod: go.etcd.io/etcd/server/v3 v3.5.5

[x] follow-up to https://github.com/moby/swarmkit/pull/3070
relates to https://github.com/moby/moby/pull/44498 (https://github.com/moby/moby/pull/44498#discussion_r1026775528)

etcd v3.5.5 now uses otel v1.0.1; https://github.com/etcd-io/etcd/blob/v3.5.5/server/go.mod#L34-L37
(through commit https://github.com/etcd-io/etcd/commit/2d7e49002c72f4f83b85f9a472fceb31dc4a7940)
Which means we may be able to get rid of this replace rule 🎉
https://github.com/moby/moby/blob/8bb58153e75b4310f3ae9ec6d03924aa286fbf20/vendor.mod#L172
	// Resolve dependency hell with github.com/cloudflare/cfssl (transitive via
	// swarmkit) by pinning the certificate-transparency-go version. Remove once
	// module go.etcd.io/etcd/server/v3 has upgraded its dependency on
	// go.opentelemetry.io/otel to v1.
	github.com/google/certificate-transparency-go => github.com/google/certificate-transparency-go v1.0.20

go.mod: golang.org/x/sys v0.1.0
It's an indirect dependency, but updating other modules will cause it
to be updated, so updating it first.
go.mod: golang.org/x/time v0.1.0
go.mod: golang.org/x/text v0.3.7
It's an indirect dependency, but updating other modules will cause it
to be updated, so updating it first. This is an intermediate version,
which can be done in isolation.
go.mod: golang.org/x/mod v0.7.0
It's an indirect dependency, but updating other modules will cause it
to be updated, so updating it first.
go.mod: github.com/onsi/gomega v1.10.5
to get rid of the golang.org/x/xerrors dependency
go.mod: golang.org/x/text v0.4.0
Now that other dependencies are updated, we can update this one in
isolation (yay for circular dependencies).
go.mod: golang.org/x/net v0.1.0
go.mod: golang.org/x/crypto v0.1.0
go.mod: github.com/docker/go-units v0.5.0
github.com/cespare/xxhash/v2 v2.1.2
It's an indirect dependency, but updating other modules will cause it
to be updated, so updating it first.
github.com/modern-go/reflect2 v1.0.2
It's an indirect dependency, but updating other modules will cause it
to be updated, so updating it first.
go.mod: github.com/prometheus/client_golang v1.12.1
To match other projects and (upcoming) dependencies.
Full diff: https://github.com/prometheus/client_golang/compare/v1.11.0..v1.12.1
go.mod: go.etcd.io/etcd/server/v3 v3.5.5
etcd v3.5.5 now uses otel v1.0.1;
https://github.com/etcd-io/etcd/blob/v3.5.5/server/go.mod#L34-L37
https://github.com/etcd-io/etcd/commit/2d7e49002c72f4f83b85f9a472fceb31dc4a7940

One failure
--- FAIL: TestReadRepairWAL (0.22s)
    walwrap_test.go:258:
        	Error Trace:	walwrap_test.go:258
        	Error:      	Received unexpected error:
        	            	wal: max entry size limit exceeded, recBytes: 24, fileSize(200) - offset(184) - padBytes(0) = entryLimit(16)
        	            	irreparable WAL error
        	            	github.com/moby/swarmkit/v2/manager/state/raft/storage.ReadRepairWAL
        	            		/home/circleci/.go_workspace/src/github.com/docker/swarmkit/manager/state/raft/storage/walwrap.go:178
        	            	github.com/moby/swarmkit/v2/manager/state/raft/storage.TestReadRepairWAL
        	            		/home/circleci/.go_workspace/src/github.com/docker/swarmkit/manager/state/raft/storage/walwrap_test.go:257
        	            	testing.tRunner
        	            		/usr/local/go/src/testing/testing.go:1439
        	            	runtime.goexit
        	            		/usr/local/go/src/runtime/asm_amd64.s:1571
        	Test:       	TestReadRepairWAL
FAIL

Which comes from this line; https://github.com/moby/swarmkit/blob/f34bfc0e146ed785e69aa2db16315b981c2d7bc1/manager/state/raft/storage/walwrap_test.go#L257
And produced here; https://github.com/etcd-io/etcd/blob/v3.5.5/server/wal/decoder.go#L83-L89
	recBytes, padBytes := decodeFrameSize(l)
	// The length of current WAL entry must be less than the remaining file size.
	maxEntryLimit := fileBufReader.FileInfo().Size() - d.lastValidOff - padBytes
	if recBytes > maxEntryLimit {
		return fmt.Errorf("wal: max entry size limit exceeded, recBytes: %d, fileSize(%d) - offset(%d) - padBytes(%d) = entryLimit(%d)",
			recBytes, fileBufReader.FileInfo().Size(), d.lastValidOff, padBytes, maxEntryLimit)
	}

Looks like that failure is already reported in https://github.com/moby/swarmkit/issues/2479, and locally I ran into TestCreateOpenInvalidDirFails indeed to be broken. I wonder if those are broken altogether but CI not capturing the errors. make coverage uses a loop, and may not be capturing failures as long as tests for the the last package succeed; see https://github.com/moby/sys/pull/105

Since we call a few commands (under for) from the make target, intermediate failures are ignored unless -e shell option is set (or explicit && is used, which is not possible to do in for).
Example:
for cmd in false true; do $cmd; done

will succeed unless set -e is set, because the last command executed is true and its exit code is 0.
To fix, use explicit set -e where we execute more than a single command.

I see v3.5.6 was also released;  I hoped some fix for that part was included it that one (or at least a way to detect that specific error now that ErrMaxWALEntrySizeLimitExceeded was removed, but it looks like it isn't. Nevertheless we should update to that version as well as it's removing an unmaintained dependency with vulnerabilities (github.com/form3tech-oss/jwt-go); see

https://github.com/form3tech-oss/jwt-go/pull/17
https://github.com/form3tech-oss/jwt-go/pull/14
and https://github.com/distribution/distribution/pull/3459#discussion_r679139988 for more fun (had to go a bunch of rounds to get it done).

Flaky test?
--- FAIL: TestMixedFIPSClusterNonMandatoryFIPS (83.39s)
    integration_test.go:139:
        	Error Trace:	integration_test.go:139
        	            				integration_test.go:917
        	Error:      	Received unexpected error:
        	            	rpc error: code = DeadlineExceeded desc = context deadline exceeded
        	            	polling failed
        	            	github.com/moby/swarmkit/v2/testutils.PollFuncWithTimeout
        	            		/home/circleci/.go_workspace/src/github.com/docker/swarmkit/testutils/poll.go:28
        	            	github.com/moby/swarmkit/v2/integration.pollClusterReady
        	            		/home/circleci/.go_workspace/src/github.com/docker/swarmkit/integration/integration_test.go:138
        	            	github.com/moby/swarmkit/v2/integration.TestMixedFIPSClusterNonMandatoryFIPS
        	            		/home/circleci/.go_workspace/src/github.com/docker/swarmkit/integration/integration_test.go:917
        	            	testing.tRunner
        	            		/usr/local/go/src/testing/testing.go:1439
        	            	runtime.goexit
        	            		/usr/local/go/src/runtime/asm_amd64.s:1571
        	Test:       	TestMixedFIPSClusterNonMandatoryFIPS

@corhere @rumpl PTAL (it's ✅ now!) one more coming after this

Almost at the end of my queue 🎉


refactor: add drawer.canvasentry

This PR adds a CanvasEntry class used by the MultiCanvas renderer. Previously the renderer used simple objects to store the canvas and rendering context in, this PR creates a dedicated class for these canvases. This way it should be easier to eventually implement OffscreenCanvas support (#1600).
Also disabled eslint in webpack, because it's really annoying to fix lint errors while you're working on wavesurfer.js. eslint will run when you try to commit the changes, a much better moment to fix lint errors.

Coverage decreased (-0.4%) to 68.225% when pulling 5758811ed0474ef8f13a005011e9dd546c7f5264 on canvas-entry into b154dc4e4c152a7890e1f284972f7f8a6a3dbd93 on master.

Perhaps this should also be released as a v3.0 since it could break some existing custom renderers.

Btw, can the original drawer.js potentially also be implemented using an Entry?

not sure, since multicanvas extends drawer.. If I find a way to improve drawer, I'll open another PR.

Sorry I didn't get a chance to look at this before it was merged @thijstriemstra.
It looks great though, definitely a good idea to encapsulate the complexity of managing the canvas layers :+1:


support async SessionStore

I think that SessionStore should also have async api.
If I have 3 servers, and I want a user to have the same session, I need to keep the session in a central datastore, for example, Redis, and I use only the async api when working with Redis.
In order to properly implement the SessionStrore api, I will need to block, and break the async functionality.

Yes, this is something we have in mind more globally (make things async). See: https://github.com/pac4j/pac4j/issues/180
But this is definitely not easy.

See also https://github.com/millross/pac4j-async/tree/master/pac4j-async-core/src/main/java/org/pac4j/async/core/session - an experiment to determine what changes need to be made to offer pac4j a fully async API. This includes an Async session store API.
However, in your scenario, if your code calling pac4j is fundamentally expecting to call a blocking API, then an async SessionStore API won't help you - you''ll still need to block a thread somewhere (using CompletableFuture::get or join for example). Async either gets wrapped to block at some point or is async end to end.

One thing we've already learned during the experimental work is that there are going to need to be some fairly substantial changes in the core API to enable as much commonality as possible between async and sync APIs (e.g. common WebContext parents).

@millross my code is not directly calling the pac4j api, I use the CallbackController in play, which is using DefaultCallbackLogic which eventually calling the SessionStore.

Isn’t DefaultCallbackLogic sync and blocking?

On 24 Apr 2017, at 10:36, Tomer Praizler<EMAIL_ADDRESS>wrote:
@millross https://github.com/millross my code is not directly calling the pac4j api, I use the CallbackController in play, which is using DefaultCallbackLogic which eventually calling the SessionStore.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/pac4j/pac4j/issues/897#issuecomment-296595421, or mute the thread https://github.com/notifications/unsubscribe-auth/ABQfzlzQpkce9MhvnoX9yktxi2nqZfJSks5rzG0qgaJpZM4NF4XU.

Yes it is. I think we are talking about the same thing.
Even if the asyncStore will be used it will need to be blocked.

Yeah, it will be a while before I get the end-to-end async API into a viable shape unfortunately, and before it can be joined up with the sync API there will need to be significant change in the sync API to maximise concept reuse (and only differ where we have to diverge sync from async). There will, for example, need to be async versions of all logics, webcontext, authorizers etc. In some cases it will be possible to wrap non-blocking sync elements in an easy async wrapper, in others there will need to be specific non-blocking implementations, but at the moment I'm still (when time permits) teasing out the divergences in terms of APIs.

Feel free to look at the experimental repository - if you want you're welcome to submit PRs relating to asynchronous callback logic (it will certainly help me out, I get about 15-20 min a day at best, and the existing vertx-pac4j implementation gets priority, plus I also have other stuff I need to do with that time). My main focus right now is DefaultSecurityLogic's async version so we wouldn't necessarily overlap, but you might identify some of the other divergences from the sync API we need to prepare for.


Change all core models from pydantic models to classes

Core models (ie context, input, source, data, hooks, path) into classes. No need for pydantic here.
Perhaps keep it in models and pull out hook models into own file. Also might be a good place to put in methods that help set each of these objects.
Should be done after #177

Changing this yielded a massive improvement in startup time since now each hook doesn't have to import pydantic. After 177, we should have reasonable startup times


Fix message deletion

In some situations placeholder messages in K-9 Mail's database haven't been deleted.

Works for me :) Still need to remove fulltext info about purged messages ;)

I also noticed that emptying trash adds FTS4 entry (segdir one, specifically):
INSERT INTO messages_fulltext_segdir VALUES(0,8,0,0,'0 524',X'0005313737373502210000056164646f6e02210001086c77696e6573636802210001026e640221000102726502210000086264656c65746564022100010665636175736502210003046f6d65730221000001630221000114616c6c7265636f7264696e676d656e75686f6f6b02210002067465676f727902210001026f6d02210003066d656e74656402210004026974022100000764656c65746564022100010769726563746c7902210002087363757373696f6e02210001036f6e650221000005656d61696c0221000005676574696402210003076c6162656c69640221000105697468756202210000046861766502210001036f6f6b022100040464617461022100010474747073022100000769686f6f6b696402210001116c6f63616c697a6564737472696e67696402210001017402210000096d656e74696f6e6564022100030575686f6f6b02210000026f6e022100010172022100000470756c6c02210001027672022100000a7234313638323639353002210001086563656976696e6702210003066f7264696e6702210002056c6174656402210002056d6f7665640221000203706c790221000205717565737402210002047475726e0221000204776f726b0221000003746167022100010368697302210001016f02210002056164646f6e022100000476696577022100000477657265022100000478626d630221000003796f75022100');

which is left after the message gets purged.

Works for me :) Still need to remove fulltext info about purged messages ;)

I've never looked closely at how SQLite's FTS tables work. But I don't think we're supposed to directly modify that particular table. Can you look into this? I need to work on other things for a while.

Works for me :) Still need to remove fulltext info about purged messages ;)

I've never looked closely at how SQLite's FTS tables work. But I don't think we're supposed to directly modify that particular table. Can you look into this? I need to work on other things for a while.

tl;dr: Add the folliowing SQL statement at the end of message destroy operations
`"INSERT INTO messages_fulltext(messages_fulltext) VALUES('optimize')``
Detailed : I started looking into segdir issue and foundthis discussion on SQLite3 mailing list:
http://sqlite.1065341.n5.nabble.com/FTS3-4-merge-function-behaviour-when-deleting-rows-td75370.html
It led me to https://www.sqlite.org/fts3.html#mergecmd
Now I am testing the removal of messages using this technique and it looks like deleted messages are really purged form fts4 table.


Unexepected charges

I started the server with azd up, and it created the azure resources for cognitive search. It keeps on adding me more money to pay even though I deleted all the reseources created. How can i know if it will actually stop charging me??
This issue is for a: (mark with an x)
- [ ] bug report -> please search issues before submitting
- [ ] feature request
- [X] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce
make the auth login.
execute azd up
Any log messages given by the failure

Expected/desired behavior
The azure should stop charging me with a service which i am not using. I dont really understand how to fix this problem.
OS and Version?

Windows 7, 8 or 10. Linux (which distribution). macOS (Yosemite? El Capitan? Sierra?)
Windows

azd version?

run azd version and copy paste here.

Versions

Mention any other details that might be useful

Thanks! We'll be in touch soon.

Sorry to hear @Aloncarrasco, can you provide more details on where you're seeing the budget usage? Have you tried filing a support request in the Azure Portal? Sometimes they have more access to accounts and can provide more insight. (And if you're being charged accidentally, perhaps you can ask for a refund).

I am on the 'Cost Management + Billing' section. The 'cost analysis' section showed it was from cognitive search. I deleted the resources some time ago but the charges increased. If they keep on increasing i am probably leaving a ticket or contacting microsoft support.


Transactions and direct debit titles are dutch

Could add a name_nl and name_en field to a Transaction, or just only generate it in english

I see now that this text is based on your preference language, so I will close this issue as this is not an issue


Misuse of withRequestConifg()

When using withRequestConfig() using the lamda style, you need to use the client passed to the function.
Currently we use the original version of the client, which practically ignores the request config.

@jimmidyson: FYI


Android - Chat - User lands on inbox for a second when returning from a search to chat

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.75-0
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: No, reproducible on hybrid only
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/5338694&group_by=cases:section_id&group_order=asc&group_id=296775
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause Internal Team
Device used: Motorola MotoG60 - Android 12
App Component: Other
Action Performed:

Open the Expensify app.
Open any chat that already has messages on it.
Tap on the search icon on the top right corner.
Tap on "Search in this chat" button.
Write any search that will trigger results.
When navigated to the results, return to chat using the device back button.

Expected Result:
When returning from a search to the conversation, the user should be automatically redirected to the chat.
Actual Result:
User lands on inbox for a second, when returning from a search to a conversation using the device back button.
Workaround:
Unknown
Platforms:

[ ] Android: Standalone
[x] Android: HybridApp
[ ] Android: mWeb Chrome
[ ] iOS: Standalone
[ ] iOS: HybridApp
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
https://github.com/user-attachments/assets/b7600b04-7847-4f5f-8291-8f87eaf40b3c
View all open jobs on GitHub

This is technically correct in my opinion, and ultimately it's so small that I am inclined to close this bug report.


✨ Don't augment root:compute

Summary
Because we don't need the augmentations to export kube resources.
(The REAL goal of this PR is to hit 1400.)

/cc @ezrasilvera

/approve


Create the initial gradle project structure

Thanks for the code submission, Ive merged it, but we should probably look at using BUCK as we move forwards.


No error when you try to copy a blob that it is not in the container (also with a non existen container)

Which version of the AzCopy was used?
10.0.8-Preview
Which platform are you using? (ex: Windows, Mac, Linux)
Mac
What command did you run?
azcopy cp "https://account.blob.core.windows.net/container/nonexistnetfile.foo?XXXXX
What problem was encountered?
When a non-existent file is tried to be copy, azcopy gives a non-specific error or no error. The same happens with a non-existent container.
Scanning... failed to perform copy command due to error: cannot start job due to error: cannot download the enitre container / virtual directory. Please use --recursive flag.
Also, this spelling mistake it is killing me, "enitre" in cmd/removeBlobEnumerator.go and in cmd/copyDownloadBlobEnumerator.go
How can we reproduce the problem in the simplest way?
Just try to copy a file that is not in the container or in a non-existent container.
Have you found a mitigation/solution?
No

Hi @ToledoEM, thanks for reaching out and providing feedbacks!
We'll fix the error message in the next release.

Hi @ToledoEM, the output in 10.0.9 now looks like:
INFO: Scanning...
INFO: source is not validated as a single file: BlobNotFound
INFO: trying to copy the source as container/directory/list of files

failed to perform copy command due to error: cannot start job due to error: cannot download the enitre container / virtual directory. Please use --recursive flag.

Please let us know if you have any other concern.

This is fantastic !


Change the default style for windows from fruity to auto

Fixes #1266. The documentation already claims that 'auto' is the default style HTTPie follows, so no need to make any edits there.

Codecov Report

Merging #1268 (d88927f) into master (4d7d6b6) will decrease coverage by 0.83%.
The diff coverage is 94.75%.

:exclamation: Current head d88927f differs from pull request most recent head 1521e2c. Consider uploading reports for the commit 1521e2c to get more accurate results

@@            Coverage Diff             @@
##           master    #1268      +/-   ##
==========================================
- Coverage   97.28%   96.44%   -0.84%
==========================================
  Files          67       85      +18
  Lines        4235     5937    +1702
==========================================
+ Hits         4120     5726    +1606
- Misses        115      211      +96

Impacted Files
Coverage Δ

tests/test_binary.py
100.00% <ø> (ø)

httpie/compat.py
31.11% <27.90%> (-68.89%)
:arrow_down:

tests/conftest.py
77.14% <58.33%> (-9.82%)
:arrow_down:

httpie/output/lexers/http.py
59.09% <59.09%> (ø)

tests/test_ssl.py
91.01% <66.66%> (-3.93%)
:arrow_down:

httpie/output/lexers/common.py
75.00% <75.00%> (ø)

httpie/output/lexers/metadata.py
81.25% <81.25%> (ø)

httpie/manager/__main__.py
82.35% <82.35%> (ø)

httpie/models.py
93.93% <88.88%> (-3.43%)
:arrow_down:

tests/test_uploads.py
97.66% <92.15%> (-2.34%)
:arrow_down:

... and 67 more

Continue to review full report at Codecov.

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


Mbed os port

Problem
There is no support for Mbed OS in CHIP.
Summary of Changes
Add Mbed OS support to CHIP.
This is a revival of https://github.com/project-chip/connectedhomeip/pull/6355 now that master is stable again with the Bluetooth provisioning flow.
Few details about the port:

Our main development target is the PSOC6 WiFi-BT Prototyping kit from cypress:
The port only support WiFi and BLE connectivity; not Thread.
There is only one examples in this PR: The lock application. The shell application, lightning app, all cluster app and unit test app are ready and will be submitted when that PR is merged.
There is good integration with VSCode, build, flash and debug of the examples supported is provided.

Testing
How was this tested? (at least one bullet point required)

If unit tests existed, how were they fixed/modified to prevent this in future?

Chip unit tests are supported by Mbed OS but will be added in a subsequent PR. Reference

If integration tests were added, how do they verify this change?

We made integration tests for the lock app running on the mbed platform, it uses chiptool to connect on the board running the application and run pairing. It is not part of this PR but will be submitted in a subsequent PR. Reference

If new integration tests are not added, why not?

See above. Too much in one PR.

If manually tested, what platforms controller and device platforms were manually tested, and how?

PSOC6 WiFi-BT Prototyping kit + Linux with the chip tool app.

Two major changes compared to https://github.com/project-chip/connectedhomeip/pull/6355:

The Mbed OS POSIX socket addaptation layer has been offlowded to a separate repository which is now imported in third_party.
Support of minimal MDNS has been added to the mbed ports.

Other changes just reflect change made to upstream (event queue, select, ... )

This require integration of https://github.com/project-chip/connectedhomeip/pull/7927

@saurabhst @msandstedt @Damian-Nordic @jepenven-silabs @mspang ?


Enable user to provision shared memory for pipeline node

Is your feature request related to a problem? Please describe.
Using FARM to manage PyTorch KPI training and extraction, runs would deadlock: https://github.com/os-climate/aicoe-osc-demo/issues/174
I solved this problem by limiting the use of multiprocessing so that shared memory did not need to be allocated.  But a better solution would be to allocate sufficient shared memory to allow the runs to complete.  The Elyra Pipeline has an elegant way for users to specify how many CPUs and GPUs, as well as how much RAM should be allocated to a pipeline node.  That pattern could be applied to other resources, like shared memory.  Docker run supports the --shm=SIZE parameter, for example.
Describe the solution you'd like
I'd like to specify on a per-node basis a non-default amount of shared memory to allocate.  In my case, I'd like to see whether 512mb is enough, or whether 1gb or 2gb are needed.  I should have the freedom do specify an amount (possibly with units).
Describe alternatives you've considered
I have already implemented changes to disable multiprocessing, but this makes poor use of the powerful CPUs our cluster makes available.  Another possibility would be to write Operate First scripts to control the Kubeflow execution parameters outside of Elyra, but why not expose this parameter that is so critical to specific node tasks?
Additional context
My own project repo is here: https://github.com/MichaelTiemannOSC/aicoe-osc-demo/tree/cdp-fixups
@Shreyanand
@ptitzler

After quick research it appears that Kubernetes currently does not support setting the pod's shared memory size (https://github.com/kubernetes/kubernetes/issues/28272). Using an emptyDir as seen here https://docs.openshift.com/online/pro/dev_guide/shared_memory.html in combination with a size limit might be a possible approach that could be considered.

Thank you!  We are investigating and will report back.

If there's any follow-up required on our end please re-open the issue!

Hi @ptitzler for this issue, I tried the following:

Exported the pipeline yaml (it's a very useful and powerful feature!)
Edited the yaml as per this comment that essentially mounts a shm dir and allows for a larger shared memory for multiprocessing
Imported the yaml in a kubeflow pipeline run

The pipeline ran successfully without any deadlocks. The increased shm size would help deep learning workloads that use multi processing.  Could that be possibly added to node properties in the UI and the kf workload yaml in the backend?