[CP] New tooling for iOS 17 physical devices

Issue Link
https://github.com/flutter/flutter/issues/128531
Commit Hash
d631b26
Target
beta
PR Link
https://github.com/flutter/flutter/pull/132412
Changelog Description
Updates tooling to allow flutter run with physical iOS 17 devices.
Impacted Users
All iOS Flutter developers using iOS 17 physical devices
Impact Description
flutter run won't work for physical iOS 17 devices.
Workaround
Run Flutter app through Xcode (see for more detailed instructions).
Risk
medium
Test Coverage
no
Validation Steps
There are a couple use cases this fixes, but major one is flutter run:

Download Xcode 15
Run flutter run --release
Everything should work as expected

Download Xcode 15
Run flutter run
Project should open in Xcode and debugging should start

@christopherfujino could you take a look at this?

@itsjustkevin FYI: this one is for 13.4 beta


Consistency between vitest.config.ts and sonar-project.properties

Related to #10836

@pascalgrimaud is this what you thought when you created the ticket?

@pascalgrimaud is this what you thought when you created the ticket?

I didn't have a closed look, but the list of files in sonar should be really closed to the list in vitest config
If it's not the case, we can have 100% coverage with vitest but 90% coverage with Sonar for example


Add a sentence search page.

This page has pagination because rendering thousands of sentences is very slow.
Note that this does not add link to sentence capabilities, but it does show which story each block of results came from... hopefully that's enough for now.
It supports regex and searches all 4 sentence lines.
Try it out at: https://rawgit.com/sande570/Moro-database/sentence-search/#/search?search=comp[^+]*appl&skip=0
@psejenks / @sande570 Does this appear to fit your sentence searching needs? or should I do a few more fixups before merging?

I hear searching is good now, and more things are tagged as the root of a word.


Update scope-activity.md

As Issues feature is not in fabric-docs, raising this PR. Actual use case is Scope activity is no longer available in Data pipelines as it might be deprecated with ADLS Gen1 getting deprecated by Feb month end
So please delete these references/docs

@chez-charlie
Can you address the contributor's question?
If you decide to delete the article, you'll need to open a PR in the private repo to do so. Otherwise, you can close this PR. Thanks!
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Hi @jonburchel and @chez-charlie,
Looks like you're the owners of Use the Scope activity to execute a script in Azure Data Lake Analytics.
Can one of you please validate @NandanHegde15 statement: "Scope activity is no longer available in Data pipelines" and if needed make changes or delete the article?
Thanks,
Kesem

Thanks @NandanHegde15 for your help to improve and maintain the docs! I've removed this from TOC for now and pending PM confirmation will delete the entire article if necessary too. #please-close


Contracts should include relayerFee in the CallParams

Problem
If the relayerFee is not used to generate the transferId, it cannot be enforced across chains (i.e. in SponsorVault usage).
Ideas to solve this
Move the relayerFee into the CallParams
Acceptance Criteria

[x] CallParams includes relayerFee

Other
Should be included with #1320 because both break interfaces

Note: offchain work being separated out, closing as onchain work completed


Activate AdmittanceControllerTestParameterizedInvalidParameters

CI lately complains about a test suite which is not instantiated with any TEST_P
https://github.com/ros-controls/ros2_controllers/actions/runs/5583901991/jobs/10204704127
I added a TEST_P now, but had to deactivate one invalid parameter test because SetUpController() did not fail.

Flaky tests of JTC are addressed in #558


feature: support version.yml or version.yaml

Feature Request
The version.yml file should be able to be named version.yaml and work fine. In the case where both exist a warning should be emitted and the .yaml file should be used.

technically you can alter the path by setting the VERSION_CONFIG env var and pointing it to any file.


Fix incorrect output of skuba cluster images

Fix SUSE/avant-garde#1750
Because of v5 regsitry path change, the output of skuba cluster images
has incorrect info for v4 related images of previous cluster versions.
The original code has hard-coded registry path for v4, so this will be
fixed in this commit to have adaptive way to have right registry path
for the corresponding cluster version.
Also, --kubernetes-version will be kept to use w/o issues.
Why is this PR needed?
Does it fix an issue? addresses a business case?
add a description and link to the issue if one exists.
Fixes #
Reminder: Add the "fixes bsc#XXXX" to the title of the commit so that it will
appear in the changelog.
What does this PR do?
please include a brief "management" technical overview (details are in the code)
Anything else a reviewer needs to know?
Special test cases, manual steps, links to resources or anything else that could be helpful to the reviewer.
Info for QA
This is info for QA so that they can validate this. This is mandatory if this PR fixes a bug.
If this is a new feature, a good description in "What does this PR do" may be enough.
Related info
Info that can be relevant for QA:

link to other PRs that should be merged together
link to packages that should be released together
upstream issues

Status BEFORE applying the patch
How can we reproduce the issue? How can we see this issue? Please provide the steps and the prove
this issue is not fixed.
Status AFTER applying the patch
How can we validate this issue is fixed? Please provide the steps and the prove this issue is fixed.
Docs
If docs need to be updated, please add a link to a PR to https://github.com/SUSE/doc-caasp.
At the time of creating the issue, this PR can be work in progress (set its title to [WIP]),
but the documentation needs to be finalized before the PR can be merged.
Merge restrictions
(Please do not edit this)
We are in v4-maintenance phase, so we will restrict what can be merged to prevent unexpected surprises:
What can be merged (merge criteria):
    2 approvals:
        1 developer: code is fine
        1 QA: QA is fine
    there is a PR for updating documentation (or a statement that this is not needed)

@cclhsu please help try this to see if related cases get fixed. Thanks.

nice!

I had fixed the linter error and the output for production is
 skuba cluster images
** This is an UNTAGGED version and NOT intended for production usage. **
VERSION    IMAGE
1.15.2     registry.suse.com/caasp/v4/etcd:3.3.11
1.15.2     registry.suse.com/caasp/v4/coredns:1.3.1
1.15.2     registry.suse.com/caasp/v4/caasp-dex:2.16.0
1.15.2     registry.suse.com/caasp/v4/gangway:3.1.0-rev4
1.15.2     registry.suse.com/caasp/v4/cilium-operator:1.5.3
1.15.2     registry.suse.com/caasp/v4/hyperkube:v1.15.2
1.15.2     registry.suse.com/caasp/v4/pause:3.1
1.15.2     registry.suse.com/caasp/v4/kured:1.2.0-rev4
1.15.2     registry.suse.com/caasp/v4/cilium-init:1.5.3
1.15.2     registry.suse.com/caasp/v4/cilium:1.5.3
1.15.2     registry.suse.com/caasp/v4/skuba-tooling:0.1.0
1.16.2     registry.suse.com/caasp/v4/coredns:1.6.2
1.16.2     registry.suse.com/caasp/v4/cilium-init:1.5.3
1.16.2     registry.suse.com/caasp/v4/cilium:1.5.3
1.16.2     registry.suse.com/caasp/v4/gangway:3.1.0-rev4
1.16.2     registry.suse.com/caasp/v4/kured:1.3.0
1.16.2     registry.suse.com/caasp/v4/metrics-server:0.3.6
1.16.2     registry.suse.com/caasp/v4/hyperkube:v1.16.2
1.16.2     registry.suse.com/caasp/v4/etcd:3.3.15
1.16.2     registry.suse.com/caasp/v4/pause:3.1
1.16.2     registry.suse.com/caasp/v4/skuba-tooling:0.1.0
1.16.2     registry.suse.com/caasp/v4/cilium-operator:1.5.3
1.16.2     registry.suse.com/caasp/v4/caasp-dex:2.16.0
1.17.9     registry.suse.com/caasp/v4/coredns:1.6.5
1.17.9     registry.suse.com/caasp/v4/pause:3.1
1.17.9     registry.suse.com/caasp/v4/skuba-tooling:0.1.0
1.17.9     registry.suse.com/caasp/v4/caasp-dex:2.16.0
1.17.9     registry.suse.com/caasp/v4/kured:1.3.0-rev4
1.17.9     registry.suse.com/caasp/v4/gangway:3.1.0-rev4
1.17.9     registry.suse.com/caasp/v4/hyperkube:v1.17.9
1.17.9     registry.suse.com/caasp/v4/etcd:3.4.3
1.17.9     registry.suse.com/caasp/v4/metrics-server:0.3.6
1.17.9     registry.suse.com/caasp/v4/cilium-init:1.6.6
1.17.9     registry.suse.com/caasp/v4/cilium-operator:1.6.6
1.17.9     registry.suse.com/caasp/v4/cilium:1.6.6
1.18.5     registry.suse.com/caasp/v5/skuba-tooling:0.1.0
1.18.5     registry.suse.com/caasp/v5/controller-manager:v1.18.5
1.18.5     registry.suse.com/caasp/v5/scheduler:v1.18.5
1.18.5     registry.suse.com/caasp/v5/proxy:v1.18.5
1.18.5     registry.suse.com/caasp/v5/caasp-dex:2.23.0
1.18.5     registry.suse.com/caasp/v5/kured:1.4.3
1.18.5     registry.suse.com/caasp/v5/cilium-operator:1.7.5
1.18.5     registry.suse.com/caasp/v5/pause:3.2
1.18.5     registry.suse.com/caasp/v5/coredns:1.6.7
1.18.5     registry.suse.com/caasp/v5/api-server:v1.18.5
1.18.5     registry.suse.com/caasp/v5/gangway:3.1.0-rev4
1.18.5     registry.suse.com/caasp/v5/kucero:1.1.1
1.18.5     registry.suse.com/caasp/v5/cilium-init:1.7.5
1.18.5     registry.suse.com/caasp/v5/etcd:3.4.3
1.18.5     registry.suse.com/caasp/v5/cilium:1.7.5
1.18.5     registry.suse.com/caasp/v5/metrics-server:0.3.6

@cclhsu Please remember to backport to release-caasp-5.0.0 branch.


Update fio package

Currently PerfKitBenchmarker is using v2.1.14 fio, which occasionally hangs as reported in https://github.com/GoogleCloudPlatform/PerfKitBenchmarker/issues/499 due to deadlock in fio. We need to update fio version to include the fix.

https://github.com/GoogleCloudPlatform/PerfKitBenchmarker/pull/509#issuecomment-145594985


Add VSCode settings

Should at least have the following:
    "rust-analyzer.cargo.targetDir": true,
    "rust-analyzer.cargo.allTargets": false,

Is this the setting that makes rust-analyzer use a separate folder for its build cache? If so, doesn't that significantly increase disk usage?

Yes. The increase is not that significant and it's needed on Windows for it to not sometimes decide to yeet your previous compilation.

It's unfortunate that JSON doesn't support comments... the tradeoff could be mentioned in the README, though.

Agreed, a section about that on the readme seems wise

Huh, VS Code seems to allow // comments in tasks.json and settings.json, actually.


STORM-2548: Simplify KafkaSpoutConfig to avoid duplicating KafkaConsu…

…mer configuration parameters
This is put up for discussion, since a few people on the mailing list expressed a desire for a simpler KafkaSpoutConfig.
@harshach and @priyank5485 I think this is something like what you were asking for?

@srdo Overall, this looks good. But I also think we should get rid of the Builder class totally. Reason is that it does not work well with flux yaml since you create the object using builder.build(). Let me know your thoughts on that.

(copying from the mailing list so it is easier to track the discussion if someone reads this later)
With regard to getting rid of the builder pattern, I think it is a pretty nice pattern for Java. It looks to me like it should be possible to declare and configure the builder with "component:", and then pass it to the KafkaSpoutConfig constructor with "ref:" after (which lets you avoid calling build()). Doesn't this work?

+1(NB) given that people prefer keeping the Builder class around.

+1

@hmcl Addressed your comments, and updated the README to reflect the API changes.

@srdo thanks for addressing the code comments. It LGTM, but I forgot to publish the following comment - sorry for the extra overhead. Do you want to address it in this patch as well ?
Most of the defaults in here are also in here. I think it would be better to have de defaults only in one place, ideally in the top level class. However, if it makes the code simpler to leave the defaults only in the Builder class, so be it. Neertheless, we should avoid having them in both places.
I also would like to ask if you can squash all the commits once we are done addressing everything, such that we can keep the git log clean. Thanks a lot.

@hmcl I'm not sure how we can do that. The fields in Builder are not static, so if we move the default definitions there, we'd have to create a builder and fish out the default values in the tests. The defaults also wouldn't be visible on the KafkaSpoutConfig class, which would probably be a minus. I don't see a way to have the defaults solely in the KafkaSpoutConfig class with no references to them from Builder, since the Builer fields have to get the default values if nothing else is set.
We could move the FirstPollOffsetStrategy default into a static field in KafkaSpoutConfig like the rest for consistency at least?

I'll squash the commits soon, I just wanted people to have a chance to review the changes without having to read the entire diff again

Yeah... I also think that's the ideal way to do it. Squash at the end and have a new commit addressing each batch of code review comments.

@srdo I agree with you concerning this comment. There is really not a much better way to do this. I agree with your comment "move the FirstPollOffsetStrategy default into a static field in KafkaSpoutConfig like the rest for consistency" - let's do it.
I think that what confused me is the way the whole class is organized. For instance, the state of KafkaSpoutConfig is not at the top of the class as one would expect. Perhaps we should follow the usual convention and move the constructor and fields to the top, after the constants, then put the static inner classes (here and here), then the factory methods, and at last all the accessor methods.
On a different note, I noticed that this PR now has the changes from the STORM-2541 PR included in its single commit. I think that this PR should have only one commit, with the refactoring of KafkaSpoutConfig. The STORM-2541 PR should have two commits, this PR's commit, and it's commit. That would make it easy to merge and the same time not loose Git history.

@srdo Thanks for your diligence and awesome work refactoring this code. It just made it much better.

@hmcl Added a field for the offset strategy default and reordered the KafkaSpoutConfig contents as you suggested.
I didn't mean for the branches to contain each other, I accidentally reset a branch to the wrong content, should be fixed now. Once this or STORM-2541 goes in I'll fix any conflicts that pop up in the other PR.

Actually you're right, building them on each other would be faster. I'll do as you suggested

There were some +1s earlier, but this branch has changed a bunch since. I'd appreciate if someone could give it a skim and check that it's still good.

+1. @harshach @priyank5485 can you please take one final look. If you don't have any objection, I suggest that we merge this patch in the next day or so.

still +1 .

Thanks for the reviews everyone.


feat: mention support for other directory in update configure-devops-…

…pipeline.md
This is a docs update to reflect the changes in azd discussed in this issue / pr:

https://github.com/Azure/azure-dev/issues/4034#issuecomment-2566012999
https://github.com/Azure/azure-dev/pull/4190

@alexwolfmsft
Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


chore: Enhance code documentation in 'cypress-install.js'

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 #NONE

Enhance code documentation in 'cypress-install.js'
In this commit, we've added comments to the 'cypress-install.js' script to enhance code documentation.
These comments serve to improve code readability and offer insights into the functionality of different sections of the script.
This contribution aims to assist developers in understanding the code's purpose and structure, making it more accessible for future contributors.
Excitingly, this marks my first open-source contribution, with many more to come!

Hi @Nadav23AnT
Welcome to contributing to open-source.
Sadly, we can't accept this PR as is because these comments are obvious and, hence, unnecessary. To elaborate, if we were to add comments on every line of code we wrote to explain it in plain English, we would have doubled the repo size in an unwanted way.
While we appreciate your efforts, I would request you to go over the issue tracker and look for issues that are labeled help-wanted OR first-timer-only. Such issues are up for grabs, and we would happily guide you in contributing to those.
But first, give the contributing guide a read and reach us in the Discord chat if you have any questions or feedback.
Happy coding!


Remove dependency on bower

Move away from bower, as it has been deprecated by its creators

https://github.com/rooey/chromeos-filesystem-onedrive/commit/074eb6fae5f2a6b85f5cd86fd5d4b9aaff0f6243
Done


Automated cherry pick of #6206: bm register use domain token

Cherry pick of #6206 on release/3.2.
#6206: bm register use domain token

Codecov Report

Merging #6207 into release/3.2 will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@               Coverage Diff               @@
##           release/3.2    #6207      +/-   ##
===============================================
- Coverage         7.04%    7.04%   -0.01%
===============================================
  Files              646      646
  Lines           110882   110884       +2
===============================================
  Hits              7813     7813
- Misses          102467   102469       +2
  Partials           602      602

Flag
Coverage Δ

#aFlag
7.04% <0.00%> (-0.01%)
:arrow_down:

Impacted Files
Coverage Δ

pkg/baremetal/manager.go
0.50% <0.00%> (+<0.01%)
:arrow_up:

pkg/baremetal/tasks/bm_register.go
0.00% <0.00%> (ø)

pkg/appsrv/response.go
67.64% <0.00%> (ø)

Continue to review full report at Codecov.

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


Weird tabs auto scrolling behaviour?

VSCode Version: 1.3.0 Insider
OS Version: OSX

Steps to Reproduce:

I have opened multiple tabs in one single group editor
Scroll to the last tab in this group
Move the cursor into the editor part
The tab headers will scroll to left most automatically, which makes it difficult to right click the tab header to open context menu.

It takes me minutes to record above gif as I can't right click the tab header successfully every time.

@rebornix I am puzzled by this bug, having a hard time to understand it. you say the tabs start to scroll (slowly even?) when you do some action? like what?

It's difficult for me to describe this issue as well. Please take a look at below gif, at first I opened more than 10 files in a single Group Editor. Then I double click subparser.ts from left side explorer, after that, I did nothing. Now if you look at the tabs header, you can see they are scrolling. I'm not sure if this a new feature or not, but to me, it's weird.

It's difficult for me to describe this issue as well. Please take a look at below gif, at first I opened more than 10 files in a single Group Editor. Then I double click subparser.ts from left side explorer, after that, I did nothing. Now if you look at the tabs header, you can see they are scrolling. I'm not sure if this is a new feature or not, but to me, it's weird.

Oh, it's a feature, you want to always reveal the active one and this feature is brought in by https://github.com/Microsoft/vscode/commit/3d469f88107cdde449af8c6a0f5907ea7871c493 . Now my problem is really easy to describe, I can not see the active tab title on the tab scrollbar.

@rebornix whenever an editor opens or gets active, we scroll the active tab into view in the tabs container. however, we are not doing this in the way you see it. it should move quickly. see my video:

This doesn't happen with master build and latest Insider (updated this morning), this issue is already obsolete.

@rebornix great, in fact I have seen it on a build that was not latest and cannot really explain it.

Not reproducible for me, marking verified.

@chrmarti yes it's no longer there from 1.3.1.

Looks like this has regressed. I am on version 1.7.2 and am experiencing the same bug. If I navigate to a tab outside of the visible tabs, the tabs immediately begin scrolling back to the first open tab in the group.

@cdodge89 do you have exact reproducible steps so that I can try to reproduce? Also do you still see this in our insider build where we updated Chrome from 52 to 53?  You can give our preview releases a try from: http://code.visualstudio.com/Download#insiders

@bpasero unfortunately no, I don't have reproducible steps. Once I got it into that state it would do it regularly, but I restarted vscode and can't get it back to that state again. I've several things, but no luck. It is now working as expected though.


Identify other snowflake training resources

From my research and Carlos from Snowflake's email response the 6-hour training still seems to be the best option for now.


My guess is that you may not have langchain installed in the same environment as your Jupyter Notebook. Try running

My guess is that you may not have langchain installed in the same environment as your Jupyter Notebook. Try running
!pip list

in a Notebook cell and see if langchain is listed. If not, try running:
!pip install -U langchain

Also, you have a typo:
from langchain.llms import ...

Originally posted by @oddrationale in https://github.com/hwchase17/langchain/discussions/4138#discussioncomment-5811210

Is -U replaced with anything else ?
also how can i check if langchain and jupyter or python are in same environment?


A TypeError is thrown when attempting to get the property descriptor of a wrapped array's length

Describe the bug
Trying to do the following results in a TypeError being thrown:
import { createStore } from "solid-js/store";

const [store] = createStore({ list: [] });
Object.getOwnPropertyDescriptor(store.list, "length");

This is unexpected, but more importantly causes issues when the array is passed to Object.keys and other functions which try to do the above. For example, the following in dev mode will throw, as Index calls createSignal with each of the list's items as the initial value. In dev mode, a name will be assigned to the signal using hashValue, which uses Object.keys and Object.getOwnPropertyDescriptors, which both result in a TypeError.
import { createStore } from "solid-js/store";

const [store] = createStore({ list: [[], []] });
<Index each={store.list}>{() => {}}</Index>

I believe this is caused by the length property descriptor on arrays not being configurable.
To Reproduce
See reproduction.
Expected behavior
No error should be thrown.
Reproduction
https://playground.solidjs.com/?hash=2011186416&version=1.2.2
Additional context
This bug was initially found here on discord: https://discord.com/channels/722131463138705510/722349143024205824/905493655874322462

Yeah I see it. This one is a bit annoying since it prevents us from doing some really nice stuff like translating reactivity from descriptors. It's built into the spec but it's really obnoxious. I think length is special which might be why it is affected and nothing else but I don't have many options here except to not wrap the descriptor. I tried a whole bunch of things and none of them worked. I hope length is the only thing like this.


Deprecated new Buffer(size) syntax still in use

Issue

mongodb-core spends a significant amount of time inside of Node's Buffer library creating a stack trace and determining if it is called from inside of the node_modules directory or not to know whether or not to display the deprecation warning for the deprecated new Buffer() syntax (https://nodejs.org/api/buffer.html#buffer_new_buffer_size).

From the profile above, there is about 5294ms of non-idle time used in our application (which queries MongoDB very frequently), of that, 315ms are from generating the stack traces for deprecation warnings by the new Buffer(size) invocations inside of mongodb-core/lib/connection/msg.js.
Proposal
Use the new Buffer.alloc(size) syntax to improve the performance of the library.

From looking through the repository, it looks like Buffer.alloc(size) is already used everywhere else, it is just the remaining two references that are off.
https://github.com/mongodb-js/mongodb-core/blob/master/lib/connection/msg.js#L93-L96
https://github.com/mongodb-js/mongodb-core/blob/master/lib/connection/msg.js#L113

Looks like the deprecation warning and associated performance hit will only happen 10000 times, but still potentially damaging as an application autoscales up and down and containers cycle.
https://github.com/nodejs/node/blob/master/lib/buffer.js#L148-L164

Regardless, there will be a performance benefit from this since new Buffer(size) ultimately just ends up calling Buffer.alloc(size) after performing a couple type checks.

Merged the fix into the repository for the node driver.


synthv1: 0.9.15 -> 0.9.20

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/synthv1/versions.
meta.description for synthv1 is: "An old-school 4-oscillator subtractive polyphonic synthesizer with stereo fx"
meta.homepage for synthv1 is: "https://synthv1.sourceforge.io/"
meta.changelog for synthv1 is: ""
Updates performed

Version update

To inspect upstream changes
Impact

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 0.9.20 with grep in /nix/store/xcfqg2r16wj33wq1x3dmbhzkdfgb2j6w-synthv1-0.9.20

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

du listing: https://gist.github.com/d570c2be8fc414544fc8b60b058efa39

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

3 total rebuild path(s)

1 package rebuild(s)

1 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
synthv1

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/xcfqg2r16wj33wq1x3dmbhzkdfgb2j6w-synthv1-0.9.20 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A synthv1 https://github.com/r-ryantm/nixpkgs/archive/6f5937ee92ebe8c999f4b640cd5f5021d3f56274.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/xcfqg2r16wj33wq1x3dmbhzkdfgb2j6w-synthv1-0.9.20
ls -la /nix/store/xcfqg2r16wj33wq1x3dmbhzkdfgb2j6w-synthv1-0.9.20/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

synthv1

Maintainer pings
cc @cillianderoiste for testing.

new release available


Fix typo in Layout/FirstArrayElementIndentation

Little fix for a typo I found while reading the docs. 🎈

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[ ] 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.
[ ] Added tests.
[ ] 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 bundle exec rake default. It executes all tests and RuboCop for itself, and generates the documentation.

Thanks!


Opening Modal View on a Modal

I tried to open calendar modal view in a popup modal but I am seeing inline view i.e. not able to see close button and modal view in a modal .
Is it some Issue or do I need to include some attribute ? Please Suggest .
Please find attached Screenshot which shows date-picker modal in a popup modal .

Sorry !! Forget to upload attachment .
Attaching !!

Please Find attachment

I do not see the attached screenshot

Please Find attached Screenshot

Hi ,
I am not able to attach files here by selecting them from my computer . when I hover on ' selecting them' link ,I can see attached file but not able to see when I finally comment .

Go to http://eureka2.github.io/ab-datepicker/ and click on modal:

The script is then:
$(document).ready(function() {
	$('#date').datepicker({
		modal: true,
	});
});

You can see the close button ....

Hi ,
That is Fine .I am using modal attribute and this is working fine in all locations in our application but the issue is when I already have a modal/popover say which is opening in an iframe and I am then implementing datepicker on that modal with this modal attribute set to true its not working and is showing me the same inline view .
If you have checked the screenshot I attached its a modal which is opening in an iframe and I am trying to open a datepicker modal within that modal .
let me know if you still have questions/concerns/suggestions

I will try to reproduce this issue. Which component do you use for your modal dialog?

Hi ,
we are using iframe to open modal dialog and rendering our JSP in that div which contains this frame .
You can find code here :

Ok, but where are the scripts defined? In the page? in the iframe?
Do you use bootstrap modal.js : https://getbootstrap.com/docs/3.3/javascript/#modals ?
Please, give me more elements so that I can test!
Best regards

Hi ,
Scripts are defined in the page . No we are not using anything like bootstrap modals . we are just setting modal attribute to true and we are done and this is working as intended on normal pages but issue is with iframe .
what types of elements are you asking for ?

The title of this issue is

Opening Modal View on a Modal

So, there are two modal windows. the first is your modal dialog box, the second is the datepicker.
To my knowledge, an iframe or a JSP can not popup  a modal dialog box on the browser, only a Javascript code can do that.
In my opinion, there is a conflict between this Javascript code and the datepicker.
If you do not say what you use as Javascript code to display your modal dialog box, I will not be able to test.

I partially reproduced your dialog box here and there is no problem

Can you share the code or exactly what you used ?

The HTML page:
modal-on-modal.html.txt
The content of the IFRAME:
modal-on-modal-iframe.html.txt

If I close main modal without selecting any date after opening datepicker and again open modal from ' add remainder ' button ,datepicker remains open .
can you please check on this ?

The datepicker does not know that it's in a modal and that this modal has been hidden. It is the responsibility of the app to handle this.
To illustrate, I added this in the iframe code:
window.parent.$('#myModal').on('hidden.bs.modal', function (e) {
   $('#send-date').datepicker('hide');
})

Okay it fixes the problem I guess .
Thanks for working on this .

You're welcome !


Not very accurate results with my images

Hello everyone and, first of all, many congratulations, it is truly a wonderful project!
I've been using DeepForest for some time and I have a problem that unfortunately I can't solve in any way.
DeepForest is very accurate with the images that come with it, unfortunately I can't get the same excellent results with my images, and the quality of the forecasts is much lower.
My images have the same size (400x400) and resolution 0.2m/px very similar to the example images.
I can't figure out if the problem is that the model is trained with images of non-European trees, or if the quality of the images is bad, or if there is some other kind of problem.
For example, I attach one of these images with the relative results.
Can anyone help me and give me some indication on how to proceed to have better recognition accuracy?
Thanks in advance to anyone kind enough to help me, and congratulations again.

Hi @Caio-Giulio-Cesare! Start by checking out the docs page on "How do I make the predictions better?". One of the things it mentions is fine tuning the model using a small number of labeled images from your own dataset. Information on how to do that is available in the "Training" section of the "Getting Started" page. If neither of those solves your problems then describe what you've tried from those pages and what isn't working and we'll see what we can do to help.

Hi @Caio-Giulio-Cesare! Start by checking out the docs page on "How do I make the predictions better?". One of the things it mentions is fine tuning the model using a small number of labeled images from your own dataset. Information on how to do that is available in the "Training" section of the "Getting Started" page. If neither of those solves your problems then describe what you've tried from those pages and what isn't working and we'll see what we can do to help.

Thank you very much, I will try this way. Bye...CIAO

Any update here, can we close?

Yes, thanks, it can be closed.
Unfortunately, however, I can not do the training.
On Colab it works, but on my PC (Windows - CPU and venv), unfortunately not works, and I don't understand why.
If I put workers >0 (for example workers =1), the function code at a certain moment restarts from the first instruction of the function.
If I put workers =0 and model.config["epochs"] = 5, training starts but does not complete all the foreseen epochs but only the first one, at the end of the first epoch it gives me the message
Trainer.fit stopped: max_epochs=1 reached.
even though I set 5 epochs.
In any case it can be closed, most likely it's my fault and it depends on the fact that unfortunately I'm not very good at programming and I can't understand something.
Thanks again so much.
PS: I'm also making a GUI (TkInter) that uses DeepForest, both to view the results and to make changes after the prediction and add manual annotations for a new training. If I can fix the issues I'm having with the training I'll let you know.
Thanks again... Bye...CIAO.

Error log detail:
Model from DeepForest release https://github.com/weecology/DeepForest/releases/tag/1.0.0 was already downloaded. Loading model from file.
Loading pre-built model: https://github.com/weecology/DeepForest/releases/tag/1.0.0
No validation file provided. Turning off validation loop
GPU available: False, used: False
TPU available: False, using: 0 TPU cores
IPU available: False, using: 0 IPUs
HPU available: False, using: 0 HPUs
| Name  | Type      | Params
0 | model | RetinaNet | 32.1 M
31.9 M    Trainable params
222 K     Non-trainable params
32.1 M    Total params
128.592   Total estimated model params size (MB)
C:#######\test_addestramento_modello\venv\lib\site-packages\pytorch_lightning\trainer\connectors\data_connector.py:432: PossibleUserWarning: The dataloader, train_dataloader, does not have many workers which may be a bottleneck. Consider increasing the value of the num_workers argument(try 8 which is the number of cpus on this machine) in theDataLoaderinit to improve performance.   rank_zero_warn( C:\##########\test_addestramento_modello\venv\lib\site-packages\pytorch_lightning\loops\fit_loop.py:280: PossibleUserWarning: The number of training batches (1) is smaller than the logging interval Trainer(log_every_n_steps=50). Set a lower value for log_every_n_steps if you want to see logs for the training epoch.   rank_zero_warn( Epoch 0: 100%|██████████| 1/1 [00:04<00:00,  4.37s/it, v_num=27]Trainer.fitstopped:max_epochs=1` reached.

code: the same in the manual
code_and_config_fit_problem.zip

I should have solved the training problem. In addition to setting in the code
model.config["epochs"] = 5
I set the parameter in the .yml configuration file (train section).
epochs: 5
Now the training is OK for 5 epochs (with 1 worker).
Thanks and bye

Glad you got it sorted out. When you set model.config["epochs"] = 5 will determine whether it overrides the value in the config file or not, but generally setting it in the config file is the right way to go. Since you've got it changed in the config you can drop the model.config["epochs"] = 5 line entirely.
I'll go ahead and close this issue, but definitely feel encouraged to open new issues as you proceed if you have questions or run into problems.

Glad you got it sorted out. When you set model.config["epochs"] = 5 will determine whether it overrides the value in the config file or not, but generally setting it in the config file is the right way to go. Since you've got it changed in the config you can drop the model.config["epochs"] = 5 line entirely.
I'll go ahead and close this issue, but definitely feel encouraged to open new issues as you proceed if you have questions or run into problems.

OK, thanks again and again many compliments for DeepForest... truly a great job.
Bye.


Export Fine-Tuned LLM after Trainer is Complete

We discussed here: https://github.com/kubeflow/website/pull/3718#issuecomment-2096619898 that our LLM Trainer doesn't export the fine-tuned model.
So user can't re-use that model for inference or other purposes.
We should discuss how user can get the fine-tuned artifact after LLM Trainer is complete.
/cc @kubeflow/wg-training-leads @deepanker13
Would be nice to see integration with Kubeflow Model Registry as well. cc @kubeflow/wg-data-leads

Would be nice to see integration with Kubeflow Model Registry as well. cc @kubeflow/wg-data-leads

If there is a tutorial of the part specific to this project that exhibit the metadata we want to capture on Model Registry, I would be very happy to complement that example with indexing those metadata on MR ! 🚀👍

@andreyvelich I may have misunderstood the initial context of this API because I was under the impression that you could serve the model once fine-tuned. Can you elaborate on this?

So user can't re-use that model for inference or other purposes.

@andreyvelich I may have misunderstood the initial context of this API because I was under the impression that you could serve the model once fine-tuned. Can you elaborate on this?

So user can't re-use that model for inference or other purposes.

I think, right now the only way is to use output_dir for model checkpoints.
In that case, user can get the model from PVC that we attach to the PyTorchJob.
Like in this example: https://github.com/kubeflow/training-operator/blob/master/examples/pytorch/language-modeling/train_api_hf_dataset.ipynb
Right @johnugeorge @deepanker13 ?

/remove-lifecycle stale

per https://github.com/kubeflow/training-operator/issues/2101#issuecomment-2097204327
is there a tutorial/demo about this, please?
I would be very happy to integrate a demo/blueprint for the documentation, I just need a "seed" to get started on the training operator :) thanks!

/remove lifecycle/stale


Graphical designs for first alert workflow

Develop a graphical design layer on top of the wireframes developed in #25

High-fidelity explorations of the main interface:

Map as full-screen background with floating chat panel

Map as panel with floating chat panel

Pattern sheet (in progress)

cc @yellowcap @01painadam

nice, thanks @LanesGood! Is there a figma file you could share?
I'm conflicted on the full screen vs. panel map. On the one hand having a full screen map make it feel like the map is important, font-and center. Plus (I know it's rough right now) but I actually like the translucent chat panel used in the FE

Again, it makes the map feel central.
The problem I have with it is, how do we transition from this view when the map isn't the centrepiece (e.g. when we are more interested in say a dahsboard-view). I feel that the second option makes more sense as we can bring in and resize panels based on whats coming back from the LLM. Might be interesting to explore some of these transitioning in the next wireframes. e.g.

Given that the user has prompted for an analysis, when the data resolves, the map panel minimises and a dashboard (charts) panel appears and expands
Given that the user has asked for relevant datasets, when the LLM returns the response, the map panel closes and the bat panel fills the screen to show detailed text descriptions of suggested datasets

Thoughts?

I do feel that we should allow for the expansion/addition of charts, dashboard widgets, tables, etc. For this reason, I think containing the map is the right choice. It also helps make the application (the prototype at least) feel more distinct from GFW. Further exploration of expansion/contraction of these components could also be interesting, and continues in the figma file.


Missing Jupyter support packages when install Jupyter using IJulia.notebook()

I am trying to run plotting functions via @sglyon's PlotlyJS.jl package on a 0.7 Jupyter notebook using Julia 1.1.0. The functions run but do not display the plot. They can be saved to file.
I installed Jupyter using IJulia.notebook() and believe my problem is that I am missing a python package.
After looking at my js errors, Spencer has told me that I should install widgetsnbextension or ipywidgets 4.0 which are usually installed as python packages.
Is it possible to install these when installing Jupyter notebook from IJulia.notebook()?

If IJulia.notebook() installed jupyter for you, then it did so via Conda.add("jupyter"), which should automatically install ipywidgets as that is a dependency of jupyter.
Or are you using some other jupyter installation?  What operating system are you on, and what is IJulia.JUPYTER?

@stevengj  I may have used another installation method. I've forgotten. I'm on High Sierra OS X.
IJulia.JUPYTER
returns

"/Users/George/.julia/conda/3/bin/jupyter"

What happens if you do
using Conda
Conda.add("jupyter")

?

When I run this in the Julia 1.1.0 REPL, it does not find the package Conda. I installed Jupyter when I had only Julia 0.6.4 installed and haven't reinstalled Juypter since.
Here is the output in the REPL:

julia> using Conda
ERROR: ArgumentError: Package Conda not found in current path:

Run import Pkg; Pkg.add("Conda") to install the Conda package.

Stacktrace:
[1] require(::Module, ::Symbol) at ./loading.jl:823
julia> Conda.add("jupyter")
ERROR: UndefVarError: Conda not defined
Stacktrace:
[1] top-level scope at none:0

I ran this in Julia 6.5.0 (I no longer have 0.6.4) and this is the output:

julia> using Conda
julia> Conda.add("jupyter")
Info: Running conda install -y jupyter in root environment
Collecting package metadata: done
Solving environment: done
Package Plan
environment location: /Users/George/.julia/test/v0.6/Conda/deps/usr
added / updated specs:
- jupyter
The following packages will be downloaded:
package                    |            build
---------------------------|-----------------
ca-certificates-2019.1.23  |                0         126 KB
conda-4.6.7                |           py37_0         1.7 MB
openssl-1.1.1b             |       h1de35cc_0         3.4 MB
------------------------------------------------------------
                                       Total:         5.2 MB

The following packages will be UPDATED:
ca-certificates                               2018.12.5-0 --> 2019.1.23-0
conda                                        4.6.1-py37_0 --> 4.6.7-py37_0
openssl                                 1.1.1a-h1de35cc_0 --> 1.1.1b-h1de35cc_0
Downloading and Extracting Packages
ca-certificates-2019 | 126 KB    | ##################################### | 100%
openssl-1.1.1b       | 3.4 MB    | ##################################### | 100%
conda-4.6.7          | 1.7 MB    | ##################################### | 100%
Preparing transaction: done
Verifying transaction: done
Executing transaction: done

I have tried running the ploting functions again in a Julia 0.7 Jupyter notebook and the plots still do not display.
Should I try uninstalling Conda Jupyter and reinstalling via Julia 1.1.0?

You have to do add Conda at the package prompt in the Julia 1.1 REPL to make it user-visible, since it was only installed as an internal dependency of IJulia.
(If you only installed Jupyter years ago, you might have an obsolete Conda installation.  Note also that at this point you might have multiple Conda distributions installed — updating the Conda installation from Julia 0.6 won't affect the jupyter you are running when you run IJulia.notebook() in Julia 1.1.)

Ok. In Julia 1.1 I've just added Conda, then ran using Conda Conda.add("jupyter") which ran successfully with the following output:

Julia> using Conda
[ Info: Recompiling stale cache file /Users/George/.julia/compiled/v1.1/Conda/WZE3U.ji for Conda [8f4d0f93-b110-5947-807f-2305c1781a2d]
julia> Conda.add("jupyter")
[ Info: Running conda install -y jupyter in root environment
Collecting package metadata: done
Solving environment: done
Package Plan
environment location: /Users/George/.julia/conda/3
added / updated specs:
- jupyter
The following packages will be downloaded:
package                    |            build
---------------------------|-----------------
conda-4.6.7                |           py37_0         1.7 MB
openssl-1.1.1b             |       h1de35cc_0         3.4 MB
------------------------------------------------------------
                                       Total:         5.1 MB

The following packages will be UPDATED:
conda                                        4.6.4-py37_0 --> 4.6.7-py37_0
openssl                                 1.1.1a-h1de35cc_0 --> 1.1.1b-h1de35cc_0
Downloading and Extracting Packages
conda-4.6.7          | 1.7 MB    | ##################################### | 100%
openssl-1.1.1b       | 3.4 MB    | ##################################### | 100%
Preparing transaction: done
Verifying transaction: done
Executing transaction: done

Then I opened Jupyter with IJulia.notebook(). I opened a new Julia 0.7 notebook (it does not give a Julia 1.1 option in the "New" notebook dropdown menu.)
Though I rebuiltThe same problems persists. Plots do not appear and the same js warning is issued:

failed to load extension "nbextensions/webio/main"

and

main.js:230 Widgets are not available.  Please install widgetsnbextension or ipywidgets 4.0

Does this look like an incomplete Jupyter installation?

Then I opened Jupyter with IJulia.notebook(). I opened a new Julia 0.7 notebook (it does not give a Julia 1.1 option in the "New" notebook dropdown menu.)

Something is wrong here if it doesn't give a Julia 1.1 option.  You were running IJulia.notebook() in Julia 0.7 or 1.1?  Is IJulia installed in Julia 1.1?   What is IJulia.JUPYTER?  Do you have ENV["JUPYTER"] set to something else?

ERROR: SystemError: opening file "C:\Users\Dell\.julia\registries\General\Registry.toml": No such file or directory

Just FYI, with Julia v1.7.2 on linux, installing jupyter with the above method fails completely:
julia> Conda.add("jupyter")
[ Info: Downloading miniconda installer ...
[ Info: Installing miniconda ...
PREFIX=/home/jena/miniconda3/envs/julia
Unpacking payload ...
Extracting "pyopenssl-21.0.0-pyhd8ed1ab_0.tar.bz2"
Extracting "conda-4.11.0-py39hf3d152e_0.tar.bz2"
Extracting "six-1.16.0-pyh6c4a22f_0.tar.bz2"
Extracting "ruamel_yaml-0.15.80-py39h3811e60_1006.tar.bz2"
Extracting "pip-21.3.1-pyhd8ed1ab_0.tar.bz2"
Extracting "tzdata-2021e-he74cb21_0.tar.bz2"
Extracting "idna-3.1-pyhd3deb0d_0.tar.bz2"
Extracting "ld_impl_linux-64-2.36.1-hea4e1c9_2.tar.bz2"
Extracting "libzlib-1.2.11-h36c2ea0_1013.tar.bz2"
Extracting "tk-8.6.11-h27826a3_1.tar.bz2"
Extracting "setuptools-59.4.0-py39hf3d152e_0.tar.bz2"
Extracting "brotlipy-0.7.0-py39h3811e60_1003.tar.bz2"
Extracting "xz-5.2.5-h516909a_1.tar.bz2"
Extracting "libffi-3.4.2-h7f98852_5.tar.bz2"
Extracting "wheel-0.37.0-pyhd8ed1ab_1.tar.bz2"
Extracting "tqdm-4.62.3-pyhd8ed1ab_0.tar.bz2"
Extracting "python-3.9.7-hb7a2778_3_cpython.tar.bz2"
Extracting "libgomp-11.2.0-h1d223b6_11.tar.bz2"
Extracting "certifi-2021.10.8-py39hf3d152e_1.tar.bz2"
Extracting "conda-package-handling-1.7.3-py39h3811e60_1.tar.bz2"
Extracting "_libgcc_mutex-0.1-conda_forge.tar.bz2"
Extracting "ncurses-6.2-h58526e2_4.tar.bz2"
Extracting "zlib-1.2.11-h36c2ea0_1013.tar.bz2"
Extracting "ca-certificates-2021.10.8-ha878542_0.tar.bz2"
Extracting "charset-normalizer-2.0.9-pyhd8ed1ab_0.tar.bz2"
Extracting "_openmp_mutex-4.5-1_gnu.tar.bz2"
Extracting "requests-2.26.0-pyhd8ed1ab_1.tar.bz2"
Extracting "python_abi-3.9-2_cp39.tar.bz2"
Extracting "pycosat-0.6.3-py39h3811e60_1009.tar.bz2"
Extracting "urllib3-1.26.7-pyhd8ed1ab_0.tar.bz2"
Extracting "sqlite-3.37.0-h9cd32fc_0.tar.bz2"
Extracting "pycparser-2.21-pyhd8ed1ab_0.tar.bz2"
Extracting "openssl-1.1.1l-h7f98852_0.tar.bz2"
Extracting "cryptography-36.0.0-py39h95dcef6_0.tar.bz2"
Extracting "colorama-0.4.4-pyh9f0ad1d_0.tar.bz2"
Extracting "cffi-1.15.0-py39h4bc2ebd_0.tar.bz2"
Extracting "readline-8.1-h46c0cb4_0.tar.bz2"
Extracting "yaml-0.2.5-h516909a_0.tar.bz2"
Extracting "libgcc-ng-11.2.0-h1d223b6_11.tar.bz2"
Extracting "pysocks-1.7.1-py39hf3d152e_4.tar.bz2"

                                           __
          __  ______ ___  ____ _____ ___  / /_  ____ _
         / / / / __ `__ \/ __ `/ __ `__ \/ __ \/ __ `/
        / /_/ / / / / / / /_/ / / / / / / /_/ / /_/ /
       / .___/_/ /_/ /_/\__,_/_/ /_/ /_/_.___/\__,_/
      /_/

Encountered problems while solving:
  - package libnghttp2-1.46.0-ha19adfc_0 requires openssl >=3.0.0,<4.0a0, but none of the providers can be installed
  - package python-3.9.7-hb7a2778_3_cpython requires openssl >=1.1.1l,<1.1.2a, but none of the providers can be installed
  - package cryptography-36.0.0-py39h95dcef6_0 requires openssl >=1.1.1l,<1.1.2a, but none of the providers can be installed
  - package charset-normalizer-2.0.9-pyhd8ed1ab_0 requires python >=3.6, but none of the providers can be installed
  - package colorama-0.4.4-pyh9f0ad1d_0 requires python, but none of the providers can be installed
  - package idna-3.1-pyhd3deb0d_0 requires python, but none of the providers can be installed
  - package pycparser-2.21-pyhd8ed1ab_0 requires python =2.7|>=3.4, but none of the providers can be installed
  - package python_abi-3.9-2_cp39 requires python 3.9.*, but none of the providers can be installed

ERROR   Could not solve for environment specs
ERROR: failed process: Process(`/home/jena/miniconda3/envs/julia/installer.sh -b -f -p /home/jena/miniconda3/envs/julia`, ProcessExited(1)) [1]

Stacktrace:
 [1] pipeline_error
   @ ./process.jl:531 [inlined]
 [2] run(::Cmd; wait::Bool)
   @ Base ./process.jl:446
 [3] run
   @ ./process.jl:444 [inlined]
 [4] _install_conda(env::String, force::Bool)
   @ Conda ~/miniconda3/envs/julia/share/julia/packages/Conda/1403Y/src/Conda.jl:207
 [5] _install_conda(env::String)
   @ Conda ~/miniconda3/envs/julia/share/julia/packages/Conda/1403Y/src/Conda.jl:193
 [6] runconda(args::Cmd, env::String)
   @ Conda ~/miniconda3/envs/julia/share/julia/packages/Conda/1403Y/src/Conda.jl:128
 [7] add(pkg::String, env::String; channel::String)
   @ Conda ~/miniconda3/envs/julia/share/julia/packages/Conda/1403Y/src/Conda.jl:223
 [8] add (repeats 2 times)
   @ ~/miniconda3/envs/julia/share/julia/packages/Conda/1403Y/src/Conda.jl:222 [inlined]
 [9] top-level scope
   @ REPL[7]:1

I have Julia installed inside a conda environment, but I don't think this makes a difference. I also have jupyter in the base env of conda, which IJulia can open, but the kernel keeps dying, so I'm trying other things...


Spotify Player

https://github.com/Sayak-Bhunia/mystory/assets/153416455/92c24a55-86cb-40d8-8f04-16fbd7560e51
This is how it looks.


Separating OPA installation from policy loading

This PR is based on the discussion #420. This partially addresses #238.
The following changes are incorporated in this PR.
(1) OPA installation is separated from OPA policy loading.
(2) Now all user created policies have to use dataapi.authz as the package name. An example is provided here.
Detailed commands for deploy opa and the corresponding policies are given below.
Deploy OPA
make deploy

Add a policy
make loadpolicy ARGS=<POLICYFOLDER>

Example: make loadpolicy ARGS=data-and-policies/user-created-policy-1

Remove a policy
make unloadpolicy ARGS=<POLICYFOLDER>

Example: make unloadpolicy ARGS=data-and-policies/user-created-policy-1

Add a policy data folder
make loaddata ARGS=<POLICYDATAFOLDER>

Example: make loaddata ARGS=data-and-policies/meshfordata-external-data

Remove a policy data folder
make unloaddata ARGS=<POLICYDATAFOLDER>

Example: make unloaddata ARGS=data-and-policies/meshfordata-external-data

UnDeploy OPA
make undeploy

LGTM
We would need in separate PRs to update the deployment of opa in the m4d chart and then also to update the documentation in the website on how to work with OPA.
Please rebase this PR on master.

Rebased on master. I will raise the other PRs as mentioned by @roee88 . Thanks.

Anyone else plans to review?


Opposed to commenting on the diff when a build fails, is there any chance we could get Jenkins to to reject/'request changes to' the diff. This is for two reasons;
1, we use 'arc list' to get a list of open diffs from the command line. When jenkins only posts a comment on build failure the status from the list command will still show as "Needs Review". Having jenkins handle the failure differently would show the diff as "Changes Requested" in the list command.
2, better visual representation of build failure when browsing diffs from the UI. Failure comments could be easily scrolled passed due to the standard grey colouring of the comment box (plus the build status icon on the page is tiny so easily missed).
Thanks

Another thought in the total opposite direction would be to have Jenkins act as one of X reviewers on a diff (jenkins added as reviewer through herald).
So:

Author hacks some code, runs 'arc diff' and submits diff to be reviewed by Reviewer1.
Herald adds Jenkins to reviewers and triggers jenkins build
Jenkins accepts/rejects review based on build status.
Author can't land until Reviewer1 has also accepted diff.
Author can't land if Reviewer1 has accepted diff, but jenkins build fails.

This would add an additional layer to the automated part of the code review/test, which prevents authors landing untested/unapproved code (with the exception of a forced land).


Validity check not respecting §2 Absatz 3 Buchstabe b SchAusnahmV (Single vaccination after positive test)

Avoid duplicates

[x] Bug is not mentioned in the FAQ
[ ] Bug is specific for Android only, for general issues / questions that apply to iOS and Android please raise them in the documentation repository
[x] Bug is not already reported in another issue

Technical details

Device name: Motorola One
Android version: 10
App version: 2.7.1

Describe the bug

As of §2 Absatz 3 Buchstabe b SchAusnahmV a person that was previously tested positive and got a single vaccination shot, is considered fully vaccinated from the day of the shot. Currently the CWA does not respect this and shows that the certificate is valid in 14 days. It also says this if you check the validity for Germany:

Steps to reproduce the issue

Expected behaviour

Possible Fix

Additional context

https://www.gesetze-im-internet.de/schausnahmv/__2.html

The business rule update is being worked on to the extent possible with the current DCC spec, see e.g. https://github.com/eu-digital-green-certificates/dgc-business-rules-testdata/blob/56a783f388a6a57d240d8d2cf6c17636f664e90c/DE/VR-DE-0003/rule.json. I cannot comment on ETA though.


Update ballerina.lang.version

Purpose

$subject

closing since this is actually irrelevant for the release build


Documentation doesn't specify how to add web_target_link_libraries

Hello!
First off, thanks for the awesome work on this project.
One issue I had while starting off is that none of the examples seem to include web_target_link_libraries anywhere, so it was not immediately clear what I needed to do to set up a proper CMakeLists.txt. I ended up adding target_link_libraries in the wrong place until eventually finding the answer in one of the tests: https://github.com/InsightSoftwareConsortium/itk-js/blob/1fe62d437549e057de42b98fa70706cc96ec1d13/test/MedianFilterPipeline/CMakeLists.txt#L14-L21
I realize that this may be because I am not a C++ expert, but this feels like a missing piece in the documentation.
Would you accept a PR to add something like NodeHelloInsight, which would be essentially the same as NodeHelloWorld, but include itkImage.h. Basically just showing how to cross-compile this: https://itk.org/Wiki/ITK/HelloInsight
Here is what I was thinking: https://github.com/InsightSoftwareConsortium/itk-js/compare/master...swederik:node-hello-insight
I can add a docs page if you want and send a PR. Alternatively, we could change NodeHelloWorld instead.
Thanks!

@swederik great point and suggestion -- yes a PR to clarify how to use web_target_link_libraries with the NodeHelloInsight example is welcome and appreciated! :+1:

Awesome, I'll send a PR when I get a chance.

And following https://github.com/InsightSoftwareConsortium/itk-js/issues/246 I guess we should also add a specific example for usage of VTK with ITK.

Recent itk.js was improved so the regular target_link_libraries can be used -- web_target_link_libraries is no longer necessary.


Load PDB file from query on standalone instance

I can't seem to find this information anywhere, so I assume this feature is not implemented.
It would be awesome if someone navigating to http://nglviewer.org/ngl/ could also submit a PDB file to be fetched in the query, e.g. http://nglviewer.org/ngl/?pdb=https%3A%2F%2Ffiles.rcsb.org%2Fdownload%2F6EI5.pdb (I understand that I can fetch PDB files in the UI via id but that's just for the purpose of the example).
I definitely have a use case for that and I can imagine I'm not the only one :)

try http://nglviewer.org/ngl/?load=https%3A%2F%2Ffiles.rcsb.org%2Fdownload%2F6EI5.pdb

Thanks @arose , your example works! Whenever I give it in my PDB files, though it doesn't like them. I assume it's because it expects to have the PDB extension at the end of the URL to then get a unique identifier out for the structure?
The error I get is:
Uncaught (in promise) error loading file: 'Error: autoLoad: ext '' unknown'

If I download the PDB file and File > Open it, it works.
My URL looks like this:
https://api.evcouplings.org/api/runs/download?q=817403fed3bed98f457130388638c0fa_b0.2&i=5b6771da9a79d2c3276fd8e0

thus:
http://nglviewer.org/ngl/?load=https%3A%2F%2Fapi.evcouplings.org%2Fapi%2Fruns%2Fdownload%3Fq%3D817403fed3bed98f457130388638c0fa_b0.2%26i%3D5b6771da9a79d2c3276fd8e0

Any news on this? :)


Fixes 64-bit build on Windows

Close #397

Codecov Report

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

@@          Coverage Diff          @@
##             dev    #400   +/-   ##
=====================================
  Coverage   68.8%   68.8%
=====================================
  Files         22      22
  Lines       9359    9359
=====================================
  Hits        6439    6439
  Misses      2920    2920

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 ce6d9f3...b403814. Read the comment docs.


Why was alternateAllele in TranscriptEffect removed?

I can't remember the part of the discussion around #90 where alternateAllele in TranscriptEffect was dropped.  As far as I can tell, this is still necessary when reading in variants to associate an effect with a specific alternate allele.

TranscriptEffect is going to nest under VariantSiteAnnotation, right? If so, VariantSiteAnnotation should contain the Variant.


[BUG] Naming of initialAdminPassword doesn't comply with convention

What is the bug?
I found https://github.com/opensearch-project/security/blob/e69831505a13fc79dab14c0743c8a12864cdb94d/src/main/java/org/opensearch/security/tools/democonfig/SecuritySettingsConfigurer.java#L97 that relies on an environment variable initialAdminPassword. This feels at least "unusual". Most variables I've seen are OPENSEARCH_SOMETHING_SOMETHING.
What is the expected behavior?
The variable should be OPENSEARCH_ADMIN_PASSWORD or OPENSEARCH_INITIAL_ADMIN_PASSWORD.

Thanks for filing @dblock - I agree with this suggestion.  I'd vote for OPENSEARCH_INITIAL_ADMIN_PASSWORD

Ty @dblock for filing this. I've created a PR to update the name: https://github.com/opensearch-project/security/pull/3843
Please feel free to review this.

Thanks for filing @dblock - I agree with this suggestion. I'd vote for OPENSEARCH_INITIAL_ADMIN_PASSWORD

And looking at https://github.com/opensearch-project/opensearch-py/pull/631/files#diff-a3c4761b16aa51b10e83377d99fec8879c360931ae22d98f2181cac72e207d29, we don't want it to be something like users.admin.password=...?


always create a memento regardless of resource state

JIRA Ticket: https://jira.lyrasis.org/browse/FCREPO-3465
This PR depends on https://github.com/fcrepo/fcrepo-storage-ocfl/pull/19
What does this Pull Request do?
Changes the memento creation behavior so that it always creates a memento on a resource regardless of whether or not that resource has outstanding, unversioned changes. This is inline with the behavior in F5.
How should this be tested?
# 1. Create a resource
curl -u fedoraAdmin:fedoraAdmin http://localhost:8080/rest/box -H'Link: <http://www.w3.org/ns/ldp#BasicContainer>;rel="type"' -v -H "Content-Type: text/turtle" -X PUT --data "<> <http://purl.org/dc/elements/1.1/title> 'Box'"

# 2. Inspect its mementos
curl -u fedoraAdmin:fedoraAdmin http://localhost:8080/rest/box/fcr:versions

# 3. Create a new memento
curl -u fedoraAdmin:fedoraAdmin -X POST http://localhost:8080/rest/box/fcr:versions

# 4. Verify the new memento exists
curl -u fedoraAdmin:fedoraAdmin http://localhost:8080/rest/box/fcr:versions

Interested parties
@fcrepo/committers

I get the same error when building locally:
https://travis-ci.com/github/fcrepo/fcrepo/jobs/428716694#L4045

I get the same error when building locally:
https://travis-ci.com/github/fcrepo/fcrepo/jobs/428716694#L4045

Did you build the dependent PR locally too?
https://github.com/fcrepo/fcrepo-storage-ocfl/pull/19
This PR won't work without it


XRI3 migration step09

Added XRI3+ versions of BasicInputTests InputBindingSmoketest, HandMovingSmoketest, InteractableDisabledDuringInteraction, SpawnInteratableOnHand, and UntrackedHandNearInteractions.

/azp run


308 Resume Incomplete

In the past day, I have had a few videos go to a 308 Resume Incomplete when uploading.  If I attempt the upload again, I then get that the video is a duplicate, because it actually made it up the first time - with the below exception.  What I am not getting back when the exception is happening is the ID assigned to the upload.  If I had the ID I could check on the video to see if it completed - I need this ID for my app so I can react to it.
If I upload the same video a second time, YouTube says it is duplicate, but I don't see any information about the video that it matches in its response.  Either way, I am seeking the ID of the first video.
Is it possible to turn off resumable?  Thoughts about this error?
{"request_curl":"curl -X PUT -H \"transfer-encoding: chunked\" -H \"x-upload-content-length: 42066390\" -H \"x-upload-content-type: video/*\" -H \"user-agent: Yt::Request (gzip)\" -H \"authorization: Bearer xxxxxxxxxxxx\" -H \"host: www.googleapis.com\" -H \"content-type: application/x-www-form-urlencoded\" \"https://www.googleapis.com/upload/youtube/v3/videos?part=snippet%2Cstatus\u0026uploadType=resumable\u0026upload_id=AEnB2Uq5iicvetrvHJ-3xQ2jzubwVZjvUi0tIWDz5TDrOinya_mQZATQA5uZkKxRG6z7qfZmPv5SnoHNfMhIXTd4Ola5XKk1dw\"","response_body":"#\u003cNet::HTTPRedirection 308 Resume Incomplete readbody=true\u003e"}

Appreciate any help.

Thank you for using yt gem. We recently opened this "Issues" tab back. We are also working on the prioritization process, to maintain yt better. As a part of maintenance plan, we decided to close all issues from the past, because they might be stale and outdated. Please feel free to reopen if you think necessary.


use broad name for certificate view, school admin dashboard, etc.

Fix places where lastName shows undefined


Allow capturing a JPEG into a bytes object

While capture_into_bitmap() exists, it produces a bmp too large for sending over MQTT . This PR factors out the capture into memory buffer code from capture_jpeg() as capture_into_jpeg(), which returns a bytes object if the capture was successful otherwise None (return type must be checked from user-code).

we might want to make the mode switching lazy someday, or re-use capture_into_jpeg from capture_jpeg, but this is a good start with a handy feature

Yeah, I was considering that


Some prerequisites should be optional

I noticed there are dependencies that are not needed for correct operation of the package. For example, tqdm is just a cool way of visualizing, but is not critical. I would consider making non-critical parts optional.

Hi Zafar,
Good seeing you at BARC. You're absolutely right about the package deps. We'd like to remove that one as well as some of the preprocessing dependencies that could probably be integrated without too much effort. If you have a pull request, I'd be happy to test it.
Thanks!

I will definitely look into it -- next week (pulling allnighters for some deadlines right now :)


Alone in the Dark 1/2

Signed-off-by: Ronnie Sahlberg<EMAIL_ADDRESS>

One is stretched, one is from the 2008 game. Switch to these:


fix: add compatibility logic for empty array when unmarshal json string to route

Please answer these questions before submitting a pull request

Why submit this pull request?

[x] Bugfix

[ ] New feature provided

[ ] Improve performance

[ ] Backport patches

Related issues

Bugfix

Description
unmarshal json string to routeItem error when route info contains "vars":{}.

How to fix?
add compatibility logic for empty array when unmarshal json string to routeItem

New feature or improvement

Describe the details and related test reports.

Backport patches

Why need to backport?

Source branch

Related commits and pull requests

Target branch

@okaybase Now you can re-factor this PR.

Codecov Report

Merging #374 (0efe3f7) into master (2da94e9) will decrease coverage by 0.05%.
The diff coverage is 71.42%.

@@            Coverage Diff             @@
##           master     #374      +/-   ##
==========================================
- Coverage   40.04%   39.98%   -0.06%
==========================================
  Files          38       37       -1
  Lines        3109     3111       +2
==========================================
- Hits         1245     1244       -1
- Misses       1719     1721       +2
- Partials      145      146       +1

Impacted Files
Coverage Δ

pkg/apisix/route.go
33.33% <0.00%> (-0.51%)
:arrow_down:

pkg/apisix/resource.go
57.77% <83.33%> (+6.34%)
:arrow_up:

pkg/kube/translation/ingress.go
91.44% <0.00%> (-0.22%)
:arrow_down:

pkg/kube/translation/apisix_route.go
46.26% <0.00%> (-0.15%)
:arrow_down:

pkg/kube/translation/apisix_ssl.go
0.00% <0.00%> (ø)

test/e2e/e2e.go

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 2da94e9...0efe3f7. Read the comment docs.

@okaybase Now you can re-factor this PR.

@tokers i have refactor this PR base on the #373

@okaybase Merged, many thanks!


remove duplicate attachment url check

ref #596 https://github.com/jointakahe/takahe/pull/596#issuecomment-1633014161, remove check which skipped new ActivityPubFormatError support.

Thanks!


Reorder includes in _foundation_and_overrides.scss

I reordered them to match https://github.com/zurb/foundation-sites/blob/develop/scss/foundation.scss.
Out of the box, I was getting a seriously broken dropdown menu (issue #234) but it was fixed after this change.

Ping?

@kball can you take a look?

Good luck @andmej :) See - #237


Using keyof or index with a class type in a table only gets a single property of a class

-- props is inferred as { Shape: Enum.PartType } when it should be more akin to { ["Shape" | "CFrame"]: CFrame | Enum.PartType }
local function create_part(
	props: { [keyof<Part>]: index<Part, keyof<Part>> }
): Part
	local part = Instance.new("Part")

	for key, value in props do
		(part :: any)[key] = value
	end
	return part
end

I've merged in fixes to keyof and index internally, and they should both land in the next release! Thanks for the report! 😸


libsass bindings not found. Try reinstalling node-sass? (Windows)

can't run the demo

I have wrote a patch to solve this issue:
https://github.com/ferrybrouwer/node-sass-patch

The problem can also be reproducible with node versions > 0.11.1 and is related to dlmanning/gulp-sass#185.
I'm closing the issue but it can be reopened if needed


DNS Fails to be ready, possibly because node can't access master

I'm running a 2-node microk8s cluster using ubuntu server 20.04 on two raspberry pis.  Domain resolution wasn't working for a pod running on the node, so followed the microk8s troubleshooting information and enabled/disabled the dns extension. ufw/firewalld are both enabled, and I applied the IPtables policy change with no affect.  My node can successfully ping the master over the network.  However, even pinging the above ip within a container on the node fails.   I looked at the logs you attached but could not figure out why the connection between master and worker is blocked. Did you manage to get it working?

I have the same issue. Something happened and all my nodes became unreachable (no idea why...) as a quick fix I tried restarting each node. All my nodes are now reachable, but DNS is failing from within my pods, and my coredns pod has almost exactly the same logs as OP.

fixed by rebooting controller node again, so something must have come up "unhappy"


Removed "restart" reference.

There is no daemon to restart. SSMTP emulates the /usr/sbin/sendmail command. Next time the ssmtp command is run, the new settings are applied.

 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.

What if we changed it to say "Rerun ssmtp to apply the changes" ?

How about. "Changes will come into effect as soon as the next message is sent via ssmtp"
That would apply to the mail command or other processes that directly hook into the "/usr/bin/sendmail" symlink. Honestly, I don't care... I just wanted to point out that there is no server-process to restart.

I will get the change in and merge this right away. Thank you!!


The anchor links weren't working

Hi @donkristo, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

@donkristo : Thanks for your contribution to the Azure documentation! The author, @szarkos, has been notified to review your proposed change.

Sorry, missed this.  This LGTM.

#sign-off


[moor_ffi] Error occurs while using REGEXP in select/where statement (expected two strings as parameters to regexp)

Hello,
I want to report an issue I have observed when trying to use REGEXP in the where/select statement.
Error message: SqliteException(1): Expected two strings as parameters to regexp, SQL logic error (code 1)
moor: "3.1.0"
moor_ffi: "0.6.0"
moor_generator: "3.1.0"
Exception is thrown while using Android simulator (x86, Android 10.0)
Code generation built successfully, with no errors.
The problem is with the only query listed below.
my code snippet (provides/rewards/objectives fields contains json with identifiers):
Future<List<QuestModel>> getByObjects(List<String> objects) async {
    final regex = '"' + objects.join('"|"') + '"';

    final query = select(questTable)
      ..where((table) => table.provides.regexp(regex)
          | table.rewards.regexp(regex)
          | table.objectives.regexp(regex));

    return (await query.get())
        .map((entry) => QuestModel.fromMap(entry.toJson()))
        .toList()
      ..sort((a, b) {
        var sort = (a.level ?? 0).compareTo(b.level ?? 0);

        if (sort == 0) {
          sort = a.title.toString().compareTo(b.title.toString());
        }

        return sort;
      });
  }

build.yaml:
targets:
  $default:
    builders:
      moor_generator:
        options:
          sqlite_modules:
            - moor_ffi

pubsec.yaml:
dependencies:
  flutter:
    sdk: flutter

  cupertino_icons:
  english_words:
  url_launcher:
  sembast:
  path_provider:
  flutter_bloc:
  equatable:
  shared_preferences:
  sprintf:
  sticky_headers:
  in_app_purchase:
  flutter_focus_watcher:
  flutter_markdown:
  flutter_speed_dial:
  intl:
  moor:
  moor_ffi:
  json_annotation:

dev_dependencies:
  flutter_test:
    sdk: flutter
  test:
  mockito:
  pedantic:
  build_runner:
  moor_generator:
  json_serializable:

flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, v1.17.2, on Microsoft Windows [Version 10.0.18363.836], locale en-US)

[√] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
[√] Android Studio (version 4.0)
[√] VS Code, 64-bit edition (version 1.45.1)
[√] Connected device (1 available)

• No issues found!

stack trace
E/flutter ( 8821): [ERROR:flutter/lib/ui/ui_dart_state.cc(157)] Unhandled Exception: Unhandled error SqliteException(1): Expected two strings as parameters to regexp, SQL logic error (code 1) occurred in bloc Instance of 'SourceBloc'.
E/flutter ( 8821): #0      PreparedStatement.select (package:moor_ffi/src/impl/prepared_statement.dart:54:7)
E/flutter ( 8821): #1      _VmDelegate.runSelect (package:moor_ffi/src/vm_database.dart:97:25)
E/flutter ( 8821): #2      _ExecutorWithQueryDelegate.runSelect.<anonymous closure> (package:moor/src/runtime/executor/helpers/engines.dart:41:19)
E/flutter ( 8821): #3      BasicLock.synchronized (package:synchronized/src/basic_lock.dart:32:26)
E/flutter ( 8821): #4      _ExecutorWithQueryDelegate._synchronized (package:moor/src/runtime/executor/helpers/engines.dart:22:26)
E/flutter ( 8821): #5      _ExecutorWithQueryDelegate.runSelect (package:moor/src/runtime/executor/helpers/engines.dart:39:26)
E/flutter ( 8821): #6      LazyDatabase.runSelect (package:moor/src/utils/lazy_database.dart:64:17)
E/flutter ( 8821): #7      SimpleSelectStatement._getWithQuery.<anonymous closure> (package:moor/src/runtime/query_builder/statements/select/select.dart:40:22)
E/flutter ( 8821): #8      QueryEngine.doWhenOpened.<anonymous closure> (package:moor/src/runtime/api/query_engine.dart:98:64)
E/flutter ( 8821): #9      _rootRunUnary (dart:async/zone.dart:1192:38)
E/flutter ( 8821): #10     _CustomZone.runUnary (dart:async/zone.dart:1085:19)
E/flutter ( 8821): #11     _FutureListener.handleValue (dart:async/future_impl.dart:141:18)
E/flutter ( 8821): #12     Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:682:45)
E/flutter ( 8821): #13     Future._propagateToListeners (dart:async/future_impl.dart:711:32)
E/flutter ( 8821): #14     Future._completeWithValue (dart:async/future_impl.dart:526:5)
E/flutter ( 8821): #15     _AsyncAwaitCompleter.complete (dart:async-patch/async_patch.dart:36:15)
E/flutter ( 8821): #16     _completeOnAsyncReturn (dart:async-patch/async_patch.dart:298:13)
E/flutter ( 8821): #17     BasicLock.synchronized (package:synchronized/src/basic_lock.dart)
E/flutter ( 8821): <asynchronous suspension>
E/flutter ( 8821): #18     DelegatedDatabase.ensureOpen (package:moor/src/runtime/executor/helpers/engines.dart:238:25)
E/flutter ( 8821): #19     LazyDatabase.ensureOpen.<anonymous closure> (package:moor/src/utils/lazy_database.dart:43:49)
E/flutter ( 8821): #20     _rootRunUnary (dart:async/zone.dart:1192:38)
E/flutter ( 8821): #21     _CustomZone.runUnary (dart:async/zone.dart:1085:19)
E/flutter ( 8821): #22     _FutureListener.handleValue (dart:async/future_impl.dart:141:18)
E/flutter ( 8821): #23     Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:682:45)
E/flutter ( 8821): #24     Future._propagateToListeners (dart:async/future_impl.dart:711:32)
E/flutter ( 8821): #25     Future._completeWithValue (dart:async/future_impl.dart:526:5)
E/flutter ( 8821): #26     Future._asyncComplete.<anonymous closure> (dart:async/future_impl.dart:556:7)
E/flutter ( 8821): #27     _rootRun (dart:async/zone.dart:1184:13)
E/flutter ( 8821): #28     _CustomZone.run (dart:async/zone.dart:1077:19)
E/flutter ( 8821): #29     _CustomZone.runGuarded (dart:async/zone.dart:979:7)
E/flutter ( 8821): #30     _CustomZone.bindCallbackGuarded.<anonymous closure> (dart:async/zone.dart:1019:23)
E/flutter ( 8821): #31     _microtaskLoop (dart:async/schedule_microtask.dart:43:21)
E/flutter ( 8821): #32     _startMicrotaskLoop (dart:async/schedule_microtask.dart:52:5)
E/flutter ( 8821):
E/flutter ( 8821): #0      Bloc.onError.<anonymous closure> (package:bloc/src/bloc.dart:146:7)
E/flutter ( 8821): #1      Bloc.onError (package:bloc/src/bloc.dart:147:6)
E/flutter ( 8821): #2      _rootRunBinary (dart:async/zone.dart:1204:38)
E/flutter ( 8821): #3      _CustomZone.runBinary (dart:async/zone.dart:1093:19)
E/flutter ( 8821): #4      _CustomZone.runBinaryGuarded (dart:async/zone.dart:995:7)
E/flutter ( 8821): #5      _BufferingStreamSubscription._sendError.sendError (dart:async/stream_impl.dart:358:15)
E/flutter ( 8821): #6      _BufferingStreamSubscription._sendError (dart:async/stream_impl.dart:376:16)
E/flutter ( 8821): #7      _BufferingStreamSubscription._addError (dart:async/stream_impl.dart:275:7)
E/flutter ( 8821): #8      _SyncBroadcastStreamController._sendError.<anonymous closure> (dart:async/broadcast_stream_controller.dart:393:20)
E/flutter ( 8821): #9      _BroadcastStreamController._forEachListener (dart:async/broadcast_stream_controller.dart:327:15)
E/flutter ( 8821): #10     _SyncBroadcastStreamController._sendError (dart:async/broadcast_stream_controller.dart:392:5)
E/flutter ( 8821): #11     _BroadcastStreamController._addError (dart:async/broadcast_stream_controller.dart:294:5)
E/flutter ( 8821): #12     _rootRunBinary (dart:async/zone.dart:1204:38)
E/flutter ( 8821): #13     _CustomZone.runBinary (dart:async/zone.dart:1093:19)
E/flutter ( 8821): #14     _CustomZon

Statement looks OK: SELECT * FROM quest_table WHERE provides REGEXP ? OR rewards REGEXP ? OR objectives REGEXP ?;

The function is implemented in Dart: https://github.com/simolus3/moor/blob/b5cc9b974d34d6227d50a0b1b21c88e91c628736/moor_ffi/lib/src/impl/moor_functions.dart#L80-L83
My guess is that any of the arguments is null (probably the column). I've changed the implementation to just return null in that case.

I believe that was it - some of mentioned columns are nullable indeed.
Thank you very much for your awesome work!


Fail to update vscode to latest version

I have version 1.22.1 of VSCode, on windows 10, and was trying to install the latest update.
Got an error with instructions to report the issue and attach the file I have attached.
Steps to Reproduce:

Open visual studio code
Click gear at the bottom left
Click "install updates"
Click "Update vs code" popup at the bottom right

vscode-inno-updater.log

@froron Did an admin prompt come up at any point? Can you retry again and check what those Code.exe processes are doing (their full command line)?


Troubles with youtube playback

Hi.
I'm trying to play a youtube video and I need the controls to be hidden when the playback is started (After mejs.play() call). However when hideVideoControlsOnLoad setting is set "true" I'm facing an error with youtube playback in fullscreen. When the controls are hidden in the fullscreen I can't show them back.
Also, there is a second minor issue with youtube: Click on the volume slider, holding mouse button move it to video aria, and then release it. After that, if you move the mouse back to volume slider it will follow you.
Thank you in advance!

Do you have a CodePen or URL you can share where I can see your issues and check them please?

I see the issue with the volume; I'll check it I just need you to show me the first one

Just add hideVideoControlsOnLoad: true to player settings in demo.js

So I add that and select the YouTube video, play it and click the fullscreen button? Or do I need to do something else?

Yes. After that do not interact with the player until the controls panel will not be hidden. And then try to show it back.

Perfect I'll check this and keep you posted on both issues.

I know the root of both issues: when moving inside an iframe, there's no way to detect those events since it's a cross-origin issue. So I'm working on adding a layer next to each iframe, so it can overlay the iframe and the events are executed properly. I'll let you know when it's ready

@anomaly-stalker Ready. Download the latest version and you shouldn't get any of the errors you listed above. Let me know

Thank you. The player behavior become better! However in the fullscreen mode, after the controls were hidden one should click on the player to show them back. And such click pauses playback. It would be great if controls will show after the mouse move. But this is minor.
Thank you again!

I'll check this since I tested it last time and by just moving the mouse in full screen you could see the controls; maybe I forgot to push something but I'll keep you posted

Yeah I tested them just now and it displays the controls by moving the mouse. Closing this issue now. Thanks for testing this

I found that was my mistake. When I'm initializing player via setSrc method everything works perfectly!

Perfect! I'm glad this is sorted out.


WIP: declare "off_cluster_minio"

for an External minio server using s3.accesskey and s3.secretkey,
will add s3.regionendpoint later
WIP: to resolve deis/workflow#481
More changes will go in deis/builder deis/controller deis/database
deis/registry, all of which have dependencies on s3/swift storage
In PoC I overrode DEIS_MINIO_SERVICE_HOST and DEIS_MINIO_SERVICE_PORT
I'd like to use that shortcut and it seems safe, but I'm open to doing it another way if it's cleaner.  I would decompose host and port from value in s3.regionendpoint, reorganizing values.yaml so the off-cluster minio configuration is very obvious.
I could not reliably use Deis Workflow on any of my (relatively not-top-of-the-line) portable machines via minikube or minishift without adding a remote storage server for minio, and adding some persistent disk.  Can we get the configuration for this mode worked out in 2.19 final release?

Values.global.storage "minio" <-- is this value read and used anywhere other than in charts/minio?
If not, we can just pick a different value instead of "minio" so that it will be distinct from internal minio.
https://gist.github.com/kingdonb/845945908336cbeefb3e8cce73ec5dc2
These changes were all I needed to enable off-cluster minio on my local/minikube Workflow instance.

I thought I had better get some feedback before I went to the trouble and opened four separate pull requests on deis/builder, deis/registry, deis/database, and deis/controller... please take a look at my patch and suggest alternatives if you think it is not in good shape to merge.  The better approach is probably not to try overwriting kubernetes service host and port variables... but I am new at this.

This will be better if I follow the guidance in #481 more closely, namely the part about setting s3.regionendpoint

Jenkins, add to whitelist.

Related to deis/monitor#191
If you want to support a cloud-neutral storage for swift objects, to be complete would also need to provide an answer for block storage or at least for the storage requirements of Grafana and InfluxDB


add configure-cloud-routes = false to aws configuration

After installing aws cluster with default kubeone config
apiVersion: kubeone.io/v1alpha1
kind: KubeOneCluster
name: cluster-dev
versions:
  kubernetes: '1.14.5'
cloudProvider:
  name: 'aws'

there are many annoying messages in  kube-controller-manager pod logs
kubectl -n kube-system logs kube-controller-manager...
route_controller.go:120] Couldn't reconcile node routes: error listing routes: unable to find route table for AWS cluster: kubernetes

Please add configure-cloud-routes: "false" as  controllerManager extraArgs config to default configuration aws cluster
controllerManager:
  extraArgs:
    cloud-config: /etc/kubernetes/cloud-config
    cloud-provider: aws
    configure-cloud-routes: "false"

Maybe better make new "feature" as the extraArgs to configure kubernetes components ?
apiVersion: kubeone.io/v1alpha1
kind: KubeOneCluster
name: cluster-dev
versions:
  kubernetes: '1.14.5'
cloudProvider:
  name: 'aws'
features:
  extraArgs:
    controllerManager:
    - configure-cloud-routes=false
    - node-monitor-period=2s
    - node-monitor-grace-period=16s
    - pod-eviction-timeout=30s
    kubelet:
    - node-status-update-frequency=4s
  metricsServer:
    enable: false
machineController:
  deploy: false

view my commit
https://github.com/maksim-paskal/kubeone/commit/db23ba89b83440101436638fa1a74a1b1fc8681e

Very soon, when in-tree cloud providers will be kicked out of the tree, flags like configure-cloud-routes will became obsolete, and they will be required to be passed down to provider specific CCM (cloud controller manager deployed as dedicated workload).
We opted out from exposing all possible flag combinations to the end user, as it's confusing and super easy to misconfigure. Instead, we bundle "preconfigured packs" of flags as dedicated features that we can easily control and avoid misconfiguration. Obviously "extraArgs" is not a feature :)
As of original issue unable to find route table for AWS cluster it looks like kubernetes is unable to find tagged routing table.
https://github.com/kubernetes/kubernetes/blob/946087b422e38dba37a8418860b0f2ede8597394/staging/src/k8s.io/legacy-cloud-providers/aws/aws_routes.go#L52

In our example aws terraform config we (re)use all defaults that are available, including subnets and routing tables. And that's the reason you see those warnings.
In more advanced example awe-private terraform config we create everything from scratch and tag everything.
I suppose we can count this issue as bug, related issue: https://github.com/kubermatic/kubeone/issues/502 .

Kubeone not using native aws route table to route traffic to pods, instead it use other network implementations like canal/flannel (by default) and this option configure-cloud-routes: "false" as controllerManager extraArgs is required for not to logging annoying messages...
Simplicity is very good idea - but i think for advanced usage, this tool need parameters that cat be tuned on cluster setup...

Solution would be: start creating own resources (subnets/routing-tables/etc) and tag everything.


ensure file is not in cache before calling require

Description of the Change
Fixes https://github.com/mochajs/mocha/issues/3084.
Alternate Designs
Could have modified require.cache directly, but seems easier to add the decache dependency and it handle modifying the require.cache for us.
Why should this be in core?
Mocha makes an assumption that when loading files, require(file) will parse a file and encounter mocha directives. For this to be valid, mocha should ensure the file is not in the require cache.
Benefits
Allows mocha to be used programmatically in scripts/environments where modules mocha is going to run tests on have already been loaded (perhaps by a previous invocation of mocha itself).
Possible Drawbacks
None that come to mind immediately, other than risk by having the decache dependency, though I think having that dependency makes sense here.
Applicable issues
bug fix

How do I fix Error: Cannot find module 'decache' in travis CI?

Coverage increased (+0.05%) to 89.86% when pulling 8618e7fb9c07e78913505ea02932107c25134ebe on finnigantime:finnigantime/decache into da901da1af3d911670da7c6df6f9818d89c39ae1 on mochajs:master.

Coverage increased (+0.05%) to 89.86% when pulling 7c3e89d54f3362af08fa9dfc24d6770fb4d55214 on finnigantime:finnigantime/decache into da901da1af3d911670da7c6df6f9818d89c39ae1 on mochajs:master.

thanks.  as @ScottFreeCode wrote in #3084, we can't just blast the cache wholesale.  if we're going to have this capability baked in to core, the solution is going to be much more nuanced.


Typo Error in contributing file

Under the section project setup local.
There is a small typo error.
In the commit changes section there is a typo error of messagge.
Should be message.

This was already fixed in #2304


Issue 4

Url and Data to be submitted in the post request has been obtained

@Kunalgarg2100 Does this work properly?


Display the Jfrog Xray info the the Jfrog Artifactory

Describe the solution you'd like
Jfrog does provide a security scan with Xray. It would be great to see the number of vulnerabilities (and the severity) in the Jfrog Artifactory history table. If an image has an issue, a link should redirect to the Jfrog Artifactory page with all the information.

Being able to visualize a sbom report would be nice - https://jfrog.com/help/r/jfrog-security-documentation/xray-sbom-report

Addressed by https://github.com/janus-idp/backstage-plugins/pull/431


update expected failures line numbers for core branch 'spaces-apiLocks2'

Description

part of https://github.com/owncloud/ocis/issues/3098
test pr for https://github.com/owncloud/core/pull/39759

:boom: Acceptance test Core-API-Tests-ocis-storage-7 failed. Further test are cancelled...


chore: update dependencies to remove vulnerabilities

Published in +<EMAIL_ADDRESS>Note that I needed to add some files to npmignore: 3fd9f406c3b919cfdcf98982c4a4fcc0af47ee3f
@crgwbr @jimjenkins5 @yokuze


Admin dashboard infinite loop in ListView when filtering on relation

Link to reproduction
No response
Describe the Bug
Refreshing the page after filtering on a relation field via the "Filter" pill causes an infinite loop on the page. It will continuously request the query until the server errors out with a 429 error.  My main project experiencing this issue is using v2.22.0.
To Reproduce

Use npx to create a new Payload v2 project using one of the templates (website, ecommerce, etc...)
Install deps and run dev
Navigate to /admin, login, and select a collection with a relationship field
Open devtools, click the filter pill and filter on the relation
Refresh the page with devtools open to the network tab

I used the eCommerce template in the video below.
https://github.com/payloadcms/payload/assets/36972061/84926005-c830-4b38-ab80-80d04460c9c4
Payload Version
v2.22.0
Adapters and Plugins
db-postgres, bundler-webpack, plugin-nested-docs, plugin-redirects, plugin-seo, richtext-slate

What is interesting is if I fork the payload project and run the community version with pnpm to reproduce - the issue doesn't occur... However, if you use npx to create any of the templates, I can reproduce the issue every time.

@akhrarovsaid the reason you were unable to repro this in the _community folder is by default it is using MongoDB, but this issue is PG specific.


write unit tests

The project needs some unit testing to maintain functionality between builds.

Rewriting the modules to use pub/sub in the crossbrowser branch. It will be easy to write unit tests after that. Adjusting milestone.


Marketplace CRUD API Testing & Optimization

This issue was raised to track the backlog for the Beta Deployment.

Line 7 & 14 must be taken care in between or after beta testing @jckamaicha

Line 7 & 14 must be taken care in between or after beta testing @jckamaicha

Well, this still needs to be done.


typo in ljspeech/sd/run.sh

91 # stop when error occurred
92 set -euo pipfail
should be 'pipefail' shouldn't it?

Thanks! I fixed!


Figure out how to sync image data via firebase

solution: https://github.com/yoha/Ventori/commit/5f71adaec5f407c18a0efd4e9348cd87b682ad92


I have trash in my poetry installation

I have Artix Linux & I have trash in my poetry installation (from curl)
$ find .poetry/ -iname .DS_Store
.poetry/lib/poetry/.DS_Store
.poetry/lib/poetry/packages/constraints/.DS_Store
.poetry/lib/poetry/packages/.DS_Store
https://en.wikipedia.org/wiki/.DS_Store

It should be fixed as of release 0.12.6


Name collision with LogError

My project already has a LogError method, and it seems like that would be a common name.  Is there an easy way to avoid this that I'm not aware of?  If not, how about prefixing or otherwise "uniquifying" your method?  I'd be glad to do a pull request.

Hmm, I'll have to think about that. If I change the name it'll break anyone's code depending on it if they upgrade. The best way to avoid it is to qualify your code with the module name (e.g. YourModule.LogError ... and WebHelpers.LogError "abc") I've tried to do this internally with VBA.FnName... to avoid colliding with any user-defined functions that may have the same name. This is an unfortunate issue with Modules in VBA and I've tried to limit the amount of global functionality in WebHelpers (and actually trimmed out a bunch in the v4 release), but there are at least 40-50ish things in VBA-Web that could collide with globals and I'm not sure there are any good ways to deal with that other than qualifying your code when using global (i.e. Module) functions.

OK, that will work.  You do have one unqualified call though: https://github.com/VBA-tools/VBA-Web/blob/master/src/WebClient.cls#L720

Ah, good catch. I'll fix that and dig around to make sure there aren't any others.

(reopening to track issue with non-qualified internal library calls)


Add mergeMapConcurrently

:warning: Experiment :warning:
Add mergeMapConcurrently, which lazily maps and merges.  Convert concatMap to use it, which allows concatMap to be lazy.
While the end result is almost indistinguishable from eager concatMap, it does change when the mapping function is called, which can change the timing of I/O, etc.  See the example in #217.  It eagerly fetches all of the items in parallel with the current eager concatMap, but fetches them in a non-overlapping sequence when concatMap is lazy.

Looks great, but I see one test failing (windows, Node v4.4.1):
Failure: switch | when input contains many streams | should switch when new stream arrives
  [expect.toEqual] [1, 1, 1, 2, 2, 3, 3, 3, 4, 4] expected to be equal to [1, 1, 1, 2, 2, 2, 3, 3, 3, 4]

Interesting, didn't fail for me locally or in travis!  I'll look into it.

False alert, found kind of race condition in tests:
it('should switch when new stream arrives', function() {
	var i = 0;
	var s = map(function() {
		return constant(++i, periodic(10));
	}, periodic(25));

	return sequenceEqual([1,1,1,2,2,2,3,3,3,4], take(10, switchLatest(s)));
});

On 50th ms both streems based on periodic(10) and periodic(25) triggers

Same issue on master, made PR #219

@standy Could you try the latest on this branch?  I ported the switch tests to use the virtual timer infrastructure, to reduce quirks due to differences in platform timers, poor timer resolution, etc.

I'm :+1: on this, and we've gotten positive feedback on it from the community on gitter.  So, I think we're a go on this for 0.19.0.  Next step is to review the concatMap unit tests to see if we can verify the laziness of the mapping function.  We should also make a note of the behavior in the docs.

Rebased on latest master.  Gonna try to add a unit test that (somehow) verifies the laziness, then merge this.

Ok, added a test and tried to include some info in the docs.  Feedback very welcome, especially on the docs.

Everything looks in order to me :+1:

Thanks @TylorS!


Improvements on Question and Choice Events

= This Improvement that may be related to #452 =
Since Emilio Is going to be adding the "Focus style for buttons" on 1.4, (#597 ), it would be nice to add this.
-When the Question Event is called then it looks like this:

Note here that you do not have the focus thingy yet, therefore, if you press the action key, you wont select any choices.
-That's untill you move any of the arrow keys. Then you get the focus image and now can choose between choices:

This will help the players not to choose a choice they didn't want to choose.
ALSO! dont know if I must create another Improvement to say this
When you press a button (In the example below is when the text is yellow)...
https://user-images.githubusercontent.com/95051206/145942590-7f63d43f-1288-49dc-bd48-af7f5fd46b99.mp4
...I think it would be nice that you can't move between choices anymore.
Dialogic v1.3

This kind of UI should be doable with the new system, but I'll leave it open to make sure there is an easy way of doing it


When I use vite as host to realize "shared-directory" case of  "https://github.com/module-federation/module-federation-examples" can not run after build

Repreduction
https://github.com/kehuay/shared-directory

pnpm --filter=shared-directory-* build
pnpm --filter=shared-directory-* preview

Results

rust host

vite host

Hi @kehuay thanks for you feedback. We need to investigate.


Changed update_entity to update_token

Fixes #114
Every instance of the method update_entity has been renamed or is calling update_token now.

HI @neilong31 Thanks for the changes, could you just look at @kousikmitra comment and then i can merge your excellent work in, thank you.

Thanks @neilong31 for the good work.