ROX-12632: Add vulnerability report with collections E2E test

Description
Adds a simple E2E test that tests that vuln reports generates a report given a collection and sends an email with it the report attached as a CVE. For now it just verifies the email was sent and has an attachment to keep the BAT tests simple and fast. Another test (or more) can be added later that'll run only in the nightlies which would actually parse the CSV.
Checklist

[x] Investigated and inspected CI test results
[x] Unit test and regression tests added
~[ ] Evaluated and added CHANGELOG entry if required~
~[ ] Determined and documented upgrade steps~
~[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)~

If any of these don't apply, please comment below.
Testing Performed
this is a test
In addition to reviewing your code, reviewers must also review your testing
instructions and make sure they are sufficient.

/test ui-unit-tests gke-postgres-nongroovy-e2e-tests

/test gke-postgres-nongroovy-e2e-tests  gke-postgres-qa-e2e-tests gke-nongroovy-e2e-tests

/test gke-postgres-nongroovy-e2e-tests

/test gke-postgres-qa-e2e-tests gke-nongroovy-e2e-tests gke-postgres-nongroovy-e2e-tests

The test failure is a known unrelated one. Merging


NTRs: subclasses of blastoderm cell

...is_a ZFA:0009000 ! cell
....is_a ZFA:0000093 ! blastomere *** (same as blastoderm cell in CL)
.....is_a ZFA:0000541 ! marginal blastomere
.....is_a ZFA:0001473 ! deep blastomere
.....is_a ZFA:0001484 ! superficial blastomere

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 12 Feb 2013 at 11:42

WARNING: This issue has been automatically closed because it has not been updated in more than 3 years. Please re-open it if you still need this to be addressed – we are now getting some resources to deal with such issues.


[Service Bus] Throw error when Entity Path in connection string does not match what is passed to the builder that create senders & receivers

In v7 of the Service Bus package, the ServiceBusClient builder takes connection string which can have the EntityPath specified in it if the user copied it from the policy created for a queue or topic.
The builder to create senders and receivers expect the queue/topic/subscription name to be passed to them.
This issue is to ensure that if there is a mismatch between the two, we should throw an error.
The reason is that a connection string from a policy from one queue/topic when used with a different entity gives a not so helpful error today. We can pre-empt this and throw a meaningful error ourselves instead.
If we are already doing this, please add a comment here sharing the error message and close this issue

@hemanttanwar @ramya-rao-a @conniey
There is already validation with informative error message. Any concerns?

"Cannot build client without setting either a queueName or topicName."
"Cannot build client with both queueName (%s) and topicName (%s) set."
"queueName (%s) is different than the connectionString's EntityPath (%s)."
"topicName (%s) is different than the connectionString's EntityPath (%s)."

Code is here:
https://github.com/Azure/azure-sdk-for-java/blob/eaf0db9a7d387ed21a9e0476f25b543cf8a9bf02/sdk/servicebus/azure-messaging-servicebus/src/main/java/com/azure/messaging/servicebus/ServiceBusClientBuilder.java#L407

@YijunXieMS  Looks good , We can close this issue.

Agreed


toolbar with elevation?

Great Library!
It seems that is not possible to use a toolbar with shadow.

You need to give the MaterialSearchView an elevation greater than that of the Toolbar.


Fix of issues #3 and #2

I made two commits for issues #3  and #2
For the Hacktoberfest, I hope I have done the pull request well.

Well done @kevinah95 Sorry for replying so late.


Add an RVC example app

Fixes #28564.
This PR adds an example RVC app that integrates the RvcRunMode  and RvcOperationalState clusters with logic as would be expected it a typical RVC implementation. Below is a visual representation of the state machine of the RVC example app.

Closing this PR because there is something weird happening with the CI tests.


[Interviewing] Cracking the Coding Interview

Must-have book to prepare for your SWE/SDE interview

OMG, How did I miss this?


Bump required min SDK and add CI check

required due to meta: ^1.7.0 dependency
required due to invalid plugin registrations prior to Flutter 2.8.0

@kuhnroyal
It seems meta: 1.7.0 requires Dart 2.12, which is available since Flutter 2.0.
https://github.com/dart-lang/sdk/blob/5b39eb2f0183fde63ae7d7851e8bfaf6a4075998/pkg/meta/pubspec.yaml
Thus, I'm failing to understanding the objective of this PR. Could you clarify?

Flutter versions prior 2.8.0 use an older meta version (1.3.0 I think). Since this package forces 1.7.0, you have to use min Flutter 2.8.0.

Yeah, I didn't want to break this package for <3, but it seems it's already broken for <2.8. I will accept your PR and release a last 2 versions (2.2) with these changes. Then, release a new breaking for Flutter 3.


libhandy: 1.2.3 -> 1.4.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for libhandy is: "Building blocks for modern adaptive GNOME apps"
meta.homepage for libhandy is: "https://gitlab.gnome.org/GNOME/libhandy"
meta.changelog for libhandy is: "https://gitlab.gnome.org/GNOME/libhandy/-/tags/1.4.0"
Updates performed

Ran passthru.UpdateScript

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 1.4.0 with grep in /nix/store/4g1jaky464fg8wrc68jyqmcmf77i8y9j-libhandy-1.4.0

found 1.4.0 in filename of file in /nix/store/4g1jaky464fg8wrc68jyqmcmf77i8y9j-libhandy-1.4.0

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

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

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

312 total rebuild path(s)

105 package rebuild(s)

105 x86_64-linux rebuild(s)
99 i686-linux rebuild(s)
11 x86_64-darwin rebuild(s)
97 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
adapta-gtk-theme
apostrophe
banking
baobab
bottles
caerbannog
calls
chatty
chrome-gnome-shell
cozy
deja-dup
denemo
dropbox-cli
elementary-planner
epiphany
evince
fondo
fragments
giara
glom
gnome-feeds
gnome-passwordsafe
gnome-photos
gnome-podcasts
gnome-tour
gnome-usage
gnome.baobab
gnome.bijiben
gnome.epiphany
gnome.evince
gnome.file-roller
gnome.geary
gnome.gnome-books
gnome.gnome-boxes
gnome.gnome-calculator
gnome.gnome-calendar
gnome.gnome-characters
gnome.gnome-clocks
gnome.gnome-contacts
gnome.gnome-control-center
gnome.gnome-disk-utility
gnome.gnome-documents
gnome.gnome-font-viewer
gnome.gnome-maps
gnome.gnome-notes
gnome.gnome-photos
gnome.gnome-screenshot
gnome.gnome-session
gnome.gnome-shell
gnome.gnome-software

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/4g1jaky464fg8wrc68jyqmcmf77i8y9j-libhandy-1.4.0 \
  --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 libhandy https://github.com/r-ryantm/nixpkgs/archive/2d3dbbdc4d02e1e29916680883da19ba7fb56769.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/4g1jaky464fg8wrc68jyqmcmf77i8y9j-libhandy-1.4.0
ls -la /nix/store/4g1jaky464fg8wrc68jyqmcmf77i8y9j-libhandy-1.4.0/bin

Pre-merge build results
NixPkgs review skipped

Maintainer pings
cc @hedning @jtojnar @dasj19 @maxeaubrey for testing.

I will test this with #130380 later.

https://gitlab.gnome.org/GNOME/libhandy/-/blob/master/NEWS
Version 1.4.0

Translation updates.

Version 1.3.90

HdyAvatar:

Fix giving it the .image style class when needed - already in 1.2.3

HdyCarousel:

Add the allow-scroll-wheel property.
Fix the child size allocation.

HdyPreferencesGroup:

Add the use-markup property.

HdySqueezer:

Fix the size allocation, not swapping width and height - already in 1.2.3

HdyTabBar:

Fix leaking extra drag destination targets - already in 1.2.3

HdyDeck and HdyLeaflet:

Fix a crash when swiping and empty  - already in 1.2.3

Deprecate HdySwipeGroup and HdyTitleBar.
Fix translations by setting the translation domain in UI files.
Translation updates.

This could have gone to master.

Yeah, we need to be more vigilant until https://github.com/ryantm/nixpkgs-update/issues/253 is fixed.


Allow more than one value to be sent to append

I was about to write documentation for the new append DSL method, when I realized that in practice you would very likely want to append more than one value at a time. For example:
append :linked_dirs, ".bundle", "tmp"

/cc @kirs

:+1: thanks!


auto link metadata records to relevant terms in thesauri

This process runs at intervals, results are stored in a dedicated table in SoilwiseDB
Loop through records in the catalog and analyse existing keywords

if a keyword originates from a thesaurus, try to match the concept to the prefered soilwise thesaurus by identifier
for other keywords, match the keyword to a skos-preflabel or skos-altlabel in record language in a the prefered soilwise thesaurus

for the multilingual labels, we expect the soilwise vocabulary to have multilingual preflabels, imported from for example agrovoc or inspire registry

Using soil-health knowledge graph as the thesaurus, needs to be check if it is enough, by looking into mis-matched keywords.

Keywords not found matching labels from the sh knowledge graph: (English only)

soil types: 457
soil classification: 372
ejp country survey: 176
water: 163
wapor: 159
wapor_2: 140
africa and near east: 138
wapor_2_subnational: 119
esdac: 107
land use: 97
soil profiles: 94
digital soil mapping: 92
monthly: 89
world: 87
climate: 72
water accounting: 71
autogenerated: 69
land suitability: 68
opendata: 67
land: 66
africa: 66
dekadal: 61
geology: 57
adb: 52
tag_soil: 49
ph: 47
rwanda: 44
.all subjects: 43
india: 41
texture: 41
ethiopia: 40
electrical conductivity: 39
crops: 38
el salvador: 38
sudan: 36
groundwater: 35
calcium: 35
landscape: 34
uganda: 34
aquaculture: 33
karnataka: 33
krishna river basin: 33
salinity: 33
annual: 33
tunisia: 31
drainage: 31
indonesia: 30
egypt: 30
hih_soil: 28
evapotranspiration: 28
kenya: 28
agriculture: 28
harmonized world soil database: 27
philippines: 27
niger: 27
fish farming: 27
fishery: 27
tag_gislab: 27
climafrica: 26
tag_climafrica: 26
lebanon: 26
pakistan: 26
land resources: 25
indus plain: 25
management: 25
nigeria: 24
africover: 24
tag_africover: 24
mali: 23
land cover: 23
senegal: 23
burundi: 23
suitability: 22
soil type: 22
mozambique: 22
tag_aquastat: 20
tag_lus: 20
tag_land: 20
natural resources: 20
arab region: 19
climate change: 19
mena - middle east north africa: 19
benin: 19
wa+: 19
geomorphology: 19
bangladesh: 18
riccar: 17
solaw: 17
burkina faso: 17
ghana: 17
population: 17
germany: 16
hydrology: 16
growth period: 16
wapor_2_basin: 16
south sudan: 16
countrystat: 15
water retention: 15
congo, dem r: 15
soil moisture balance model: 14
root zone: 14
horn of africa: 14
brassica napus: 14
festuca arundinacea: 14
lolium perenne: 14
raphanus sativus var. oleiformis: 14
trifolium pretense: 14
trifolium repens: 14
cover crops: 14
stable isotopes: 14
tanzania: 14
landcover: 14
precipitation: 14
tag_water-management: 14
water management: 14
environment: 14
landform: 14
lithology: 14
dry farming: 13
soil moisture: 13
forestry: 13
rainfall: 13
bmgf-aws: 13
seasonal: 13
clay: 12
soil projects data: 12
global farming systems study: 12
slope: 12
cifa technical paper 32: 12
farmland: 12
cameroon: 12
coarse fragments: 12
ghataprabha sub-basin: 11
k3 sub-basin: 11
k2 sub-basin: 11
middle krishna sub-basin: 11
malaprabha sub-basin: 11
malaysia: 11
phosphorus: 11
sri lanka: 11
latin america: 11
awash basin: 11
cent afr rep: 10
japan: 10
k4 sub-basin: 10
rice: 10
thailand: 10
botswana: 10
yemen: 10
muvumba: 10
et: 10
costa rica: 10
copescal technical paper 10: 10
lamego: 10
zankalon: 10

Keywords not found matching labels from the soil health knowledge graph: (english keywords only, count >= 10). @wbcbugfree But 'water' already exists in the knowledge graph, not sure why it failed to match. I will try to find out.

soil types: 457
soil classification: 372
ejp country survey: 176
water: 163
wapor: 159
wapor_2: 140
africa and near east: 138
wapor_2_subnational: 119
esdac: 107
land use: 97
soil profiles: 94
digital soil mapping: 92
monthly: 89
world: 87
climate: 72
water accounting: 71
autogenerated: 69
land suitability: 68
opendata: 67
land: 66
africa: 66
dekadal: 61
geology: 57
adb: 52
tag_soil: 49
ph: 47
rwanda: 44
.all subjects: 43
india: 41
texture: 41
ethiopia: 40
electrical conductivity: 39
crops: 38
el salvador: 38
sudan: 36
groundwater: 35
calcium: 35
landscape: 34
uganda: 34
aquaculture: 33
karnataka: 33
krishna river basin: 33
salinity: 33
annual: 33
tunisia: 31
drainage: 31
indonesia: 30
egypt: 30
hih_soil: 28
evapotranspiration: 28
kenya: 28
agriculture: 28
harmonized world soil database: 27
philippines: 27
niger: 27
fish farming: 27
fishery: 27
tag_gislab: 27
climafrica: 26
tag_climafrica: 26
lebanon: 26
pakistan: 26
land resources: 25
indus plain: 25
management: 25
nigeria: 24
africover: 24
tag_africover: 24
mali: 23
land cover: 23
senegal: 23
burundi: 23
suitability: 22
soil type: 22
mozambique: 22
tag_aquastat: 20
tag_lus: 20
tag_land: 20
natural resources: 20
arab region: 19
climate change: 19
mena - middle east north africa: 19
benin: 19
wa+: 19
geomorphology: 19
bangladesh: 18
riccar: 17
solaw: 17
burkina faso: 17
ghana: 17
population: 17
germany: 16
hydrology: 16
growth period: 16
wapor_2_basin: 16
south sudan: 16
countrystat: 15
water retention: 15
congo, dem r: 15
soil moisture balance model: 14
root zone: 14
horn of africa: 14
brassica napus: 14
festuca arundinacea: 14
lolium perenne: 14
raphanus sativus var. oleiformis: 14
trifolium pretense: 14
trifolium repens: 14
cover crops: 14
stable isotopes: 14
tanzania: 14
landcover: 14
precipitation: 14
tag_water-management: 14
water management: 14
environment: 14
landform: 14
lithology: 14
dry farming: 13
soil moisture: 13
forestry: 13
rainfall: 13
bmgf-aws: 13
seasonal: 13
clay: 12
soil projects data: 12
global farming systems study: 12
slope: 12
cifa technical paper 32: 12
farmland: 12
cameroon: 12
coarse fragments: 12
ghataprabha sub-basin: 11
k3 sub-basin: 11
k2 sub-basin: 11
middle krishna sub-basin: 11
malaprabha sub-basin: 11
malaysia: 11
phosphorus: 11
sri lanka: 11
latin america: 11
awash basin: 11
cent afr rep: 10
japan: 10
k4 sub-basin: 10
rice: 10
thailand: 10
botswana: 10
yemen: 10
muvumba: 10
et: 10
costa rica: 10
copescal technical paper 10: 10
lamego: 10
zankalon: 10

Insightful. And besides "water", "climate“ is also in the knowledge graph. Might give you a clue to the bug.

matching status: 28/11/2024
works to be done:

[ ] improve matching mechanism (test different threshold, and maybe other packages other than thefuzz)
[ ] fixing code (currently not able to retrieve multiple labels from KG)
[ ] group missed terms for follow-up activities
[ ] records syntax error check, and check why 80% records do not have keywords (dcat:keyword, dcterms:subject, dcat:theme)

If you're also experiencing problems with British English and American English, this uk2us mapping may help you.

The missed keywords list after fixing the matching bug: @wbcbugfree
missed_keys.txt


Distance - Wrong error message is shown when creating an invalid distance request

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

Version Number: <IP_ADDRESS>
Reproducible in staging?: y
Reproducible in production?: n
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:

Navigate to staging.new.expensify.com.
Go offline.
Go to workspace chat > + > Request money.
Go to Distance.
Enter vague addresses - SF for Start and LA for Finish point.
Create the Distance expense.
Go online.

Expected Result:
The error message that will be shown is "Unexpected error requesting money, please try again later"
Actual Result:
The error message is "The receipt didn't upload. Download the file or dismiss this error and lose it" instead of the usual "Unexpected error requesting money, please try again later".
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

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

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/93399543/7d25a8ea-a9c6-4f86-9426-848f7477c7a3
View all open jobs on GitHub

Looks like a regression of https://github.com/Expensify/App/pull/29790

It indeed looks like a regression of that PR.. @Gonals can you take a look? Otherwise we can revert...
Going to wait till monday since most people are ooo today.

TBH I think we should revert given the comments on the PR after it was merged. That being said, a clean revert is no longer possible

Yeah, going to wait for monday so that Alberto can take a look. It's not like we want to deploy today...

Haha I was trying to deploy or at least leave the checklist clear for monday morning deploy, but there are so many open blockers 😅

I was trying to deploy
You crazy man, it's friday and most of the company is OOO and/or traveling 😄

Anyways, I got the revert PR ready if we decide to go that way - https://github.com/Expensify/App/pull/31854

@ishpaul777 no, it's local file download. Not from server. Just to help user not re-scan receipt in case upload failed for some reason.
Here's scenario:

user scans receipt using device camera
upload to server and forget the receipt
after a while, failed to upload
user has no way to upload receipt again

With this local download feature, user can download scanned receipt to device and re-upload importing from gallery.
So first time user scans from camera, 2nd time imports from device gallery.
Makes sense?

For distance receipts, there's nothing to download, so that message does not make sense.

yeah, I am talking about scan receipt, not distance receipt. So this is bug anyway

I don't think the implementation of this error is correct. If there was an error uploading the receipt, that's a server side error and we should be sending an errorField: receipt Onyx update from the server and checking that instead. I don't think we should not optimistically set this on the client if a money request fails because it could have failed for other reasons other than a receipt upload.

In any case a request fails, we want the user to have option to download receipt if he added otherwise it will be lost no matter the reason request fails as far as i understand, so i think its fine to use failuredata here

Correct. If the user uploaded a file, we want to ensure the file is not lost and they can download it.

@luacmartins For receipt money requests, shouldn't we allow the user to download the file irrespective of the error, as we will remove the request after clicking the close button?
I think the error message structure can be:
Error message from the backend
Due to the above error, the receipt didn't upload. You can just download the file or dismiss this error and lose it.
Something on the line of this.

Another thing, backend doesn't send anything except error status code when I send corrupted file as receipt. I think backend can send error for onyx in this case.

This is the issue https://github.com/Expensify/App/issues/28965. Here, we will replace the placeholder image to optimistic map component.

Okay, as i skimming through issue we will use a snapshot image of route as initial receipt unless request is successful,still the question is same why we show download file message when user haven't added one?

Nope we are not using an image there

Assigning @0xmiroslav as he was C+ on the PR and created a follow up fix

This is fixed in staging, given @0xmiroslav has been C+ on the offending PR which will have to be completed I think payments will be handled there and we can close this one
https://github.com/Expensify/App/assets/36083550/fee29b4d-e49d-480e-a2f4-7cc5a264b153


added simple example of how to use $timeout

I signed it!

I don't think something like $timeout that also exists in vanilla JS needs an example, and if so a runnable example would be better. Closing.


repositories.txt

@ArduinoBot

@ArduinoBot

Hello @HackerNowful. The automated system here only operates on open pull requests. Since your submission was already accepted and the pull request merged, the bot won't pay any more attention to this pull request.
But this human is happy to provide assistance. Was there something you needed from the bot?


Updated required windows version

Network protection works on Windows Pro and Enterprise. I have upgraded this information. Problem Link: https://github.com/MicrosoftDocs/windows-itpro-docs/issues/2776

@TokyoScarab - kindly do the copy editor review for this PR. Thanks.

Looks good

@andreabichsel - can you please assist to check, approve and merge the content for this PR based on issue #2776?
Thank you.

@andreabichsel - I would like to follow up on this. Can you please assist to check, approve and merge the content for this PR based on issue #2776?
Thank you.

@andreabichsel - I would like to follow up on this. Can you please assist to check, approve and merge the content for this PR based on issue #2776?
Thank you.

@DaniHalfin - would you be able to merge the content for this PR based on issue #2776? Thank you.

@andreabichsel - I would like to follow up on this. Can you please assist to check, approve and merge the content for this PR based on issue #2776?
Thank you.
cc: @kenwith

@jdeckerms - would you be able to help check and merge the content of this PR? Thanks.

@kenwith - would you be able to merge the content for this PR based on issue #2776?
Thanks.

This looks correct but I need to find someone that can sign off on this one to confirm.

I'm checking on this.

@joinimran sorry for delay! Thanks for the correction!


feat(ingress) Experimental Native Ingress

Adds LoadBalancer to builder if we are using EXPERIMENTAL_NATIVE_INGRESS
Requires deis/controller#1243
Requires deis/workflow#732
Requires https://github.com/deis/router/pull/316

Codecov Report

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

@@           Coverage Diff           @@
##           master     #495   +/-   ##
=======================================
  Coverage   56.18%   56.18%
=======================================
  Files          29       29
  Lines        1221     1221
=======================================
  Hits          686      686
  Misses        497      497
  Partials       38       38

Impacted Files
Coverage Δ

pkg/healthsrv/circuit_state.go
77.77% <0%> (-11.12%)
:x:

pkg/healthsrv/buckets_lister.go
90% <0%> (+10%)
:white_check_mark:

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 3b29d47...12dc3e1. Read the comment docs.

@bacongobbler nit has been fixed 😄

So this is all rebased and ready to go methinks..
The only thing it looks like we are missing is the fact that test coverage went down here. But I am not really sure what we would do in a situation like this as we are only really adding YAML configuration and not really any code here.
@mboersma thoughts?

test coverage went down here

Probably just the vagaries of codecov--the YAML files aren't relevant obviously, and there isn't a unit test we could write to cover the LoadBalancer change anyway.


Unable to resume CodeActivity when re-launching app from launcher

I understand that users can use the recent app switcher to change between activities to type in their code after receiving it via text. However, what about when users decide to tap the app icon again to go back to the launcher?
Currently, as far as I can tell, the only way to get back to the "Enter Code" screen is through the app switcher.
Is there a way to bring the CodeActivity back to the front using something like mDigitsCallbackManager.onResume(this) we can put in our on onResume()?

Thank you for your feedback, We really appreciate it.
The Digits auth flow can't be started on the "EnterCode" screen, because confirmation codes expire. Moreover we are working on new features that won't work in that case.

I understand codes expire but I am talking about the case where someone is trying to enter the code within seconds of getting it. Currently, it doesn't appear that there is a way to bring the user back to the "Confirm Code" screen if they decide to open their SMS app to read the code and then click on our app's launcher icon instead of using the app switcher in Android to back to the "Confirm Code" screen.
Maybe I am implementing the Digits.authenticate() call wrong but it appears this is the case.


Missing security attributes when generating session cookies

Hi,
in my opinion, the WG-Portal is the best solution so far for managing Wireguard in enterprise networks with LDAP/AD connection. In addition, some features are solved very performant. Great work!
The cookies that WG-Portal creates, on the other hand, need a better implementation. From a security point of view, they are not secured against possible XSS attacks (cross-site scripting) and against insecure http connections. The attribute "Secure" and "HttpOnly" therfore must be added.
To add this attributes to the cookies the following line of code at /internal/server/server.go must be replaced by the following part or something like this:
replace line 119:
s.server.Use(sessions.Sessions("authsession", memstore.NewStore([]byte(s.config.Core.SessionSecret))))

with something like this:
store := memstore.NewStore([]byte(s.config.Core.SessionSecret))
store.Options(sessions.Options{
	Path: "/",
	MaxAge: 86400, // 1 Day
	Secure: true,
	HttpOnly: true,
})
s.server.Use(sessions.Sessions("authsession", store))

@bstnbuck Activating the "Secure"  flag only makes sense if WG-Portal is used behind an SSL proxy right?

@h44z Yes that is true. But I think that this is mostly the case. Maybe you could create a configuration option for this, which sets this accordingly?

This issue should be fixed now. The "Secure" flag is set if the EXTERNAL_URL config option starts with "https".

Great work!


Using 'up:all'  errors with a zero-length delimiter

When I run the following command it fails:
db-migrate up:all

with the following:
[INFO] Enter scope "/"
[INFO] No migrations to run
[INFO] Enter scope "migrations/<name of scope>/sqls"
[INFO] Done
[ERROR] error: zero-length delimited identifier at or near """"
    at Connection.parseE (/home/neikos/projects/webdev/<name>_backend/node_modules/pg/lib/connection.js:539:11)
    at Connection.parseMessage (/home/neikos/projects/webdev/<name>_backend/node_modules/pg/lib/connection.js:366:17)
    at Socket.<anonymous> (/home/neikos/projects/webdev/<name>_backend/node_modules/pg/lib/connection.js:105:22)
    at emitOne (events.js:115:13)
    at Socket.emit (events.js:210:7)
    at addChunk (_stream_readable.js:252:12)
    at readableAddChunk (_stream_readable.js:239:11)
    at Socket.Readable.push (_stream_readable.js:197:10)
    at TCP.onread (net.js:589:20)

This effectively makes it useless.

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Hey @TheNeikos,
unfortunately we can't help you if you don't provide us all informations necessary to reproduce your case.
This means, always post:

used node version
used npm version
used db-migrate version
used driver and its version
used plugins
full output of the verbose run
in complex cases a full gist, as an executable proof
if necessary used migrations and configuration details (replace sensible informations before posting configs!)
if necessary code samples

As soon as you provide us all information needed, we have an easy way to reproduce your problem and provide you with the help or bugfix you need.
Thank you.

This issue is being closed due to missing responses and insufficient information. Feel free to reopen when you're ready to provide those @TheNeikos


Modified the action.html and anime.html for flipping card problem

Fixes #521

Describe the changes you've made
modified the entire action.html and anime.html page
Describe if there is any unusual behaviour of your code(Write NA if there isn't)
NA
Additional context (OPTIONAL)
Add any other context or screenshots about the feature request here.
ex.

Checklist

[x] I have mentioned related issue number in PR description.
[x] I have added screenshots and related demo links of my changes..
[x] The title of my pull request is a short description of the requested changes.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] Open Source Program names(OPTIONAL: If you participated in any open-source program then please mention the program name here)

Reopening #521
Still, many cards are not behaving as expected. Verify all the cards.


Device found but no entities

Hi there,
I have a Tuya Neo NAS-AB02WT Siren w integrated temperature and humidity sensor.
When I add this integration, the device itself is found, but no entities are listed.
I can read the sensor values and control the alarm from the Smart Life app. I also see the sensor values in the Tuya IoT Platform.
Same thing happened when I've tried the "old" tuya integration.
The device is named as "unsupported", however it's category is "wsdcg", which appears in several places in the code.

smartlife-274cf8a08cfb6478192e440284f1524b-Siren1234-5023e25f76e3ee53d74049401edf8a4e.json.txt
I'm quite new in this topic, but if you point me to the right direction, I'm happy to contribute.
Thank you for the support in advance!
Péter

This product does not contain standard features. You can view the diagnostics file.

Hi there,
What does "standard feature" means in this context and what should I search for in the diagnostics file? Is there a public documentation regarding that?
I'm quite new in this topic, but if you point me to the right direction, I'm happy to contribute.
Péter


bug: Session Fails to Set When Exceeding Data Size Limit

Description:
While implementing session management with the nuxt-auth-utils library, I encountered a very hard to debug issue where the session fails to set/update with a slightly large amount of data.
How did I discover this?
One of the user entered a large amount of data to my users table field called "bio", the next time they logged in this data was getting included in the session, I have now removed this added a limit to this field. This kept breaking auth flow because the session was never getting set, and always returned a 401 when this specific user tried logging in, this can be handled in the setUserSession method
Steps to Reproduce:

Initialize a session for a user with a minimal amount of data.
The next time they login, set the session with additional data, gradually increasing the total size.
Observe the behavior of the session update functionality as the total session data size approaches and exceeds 4KB (max cookie size)

Suggested Remedies:

Throw an error stating the cookie is getting too large, instead of continuing, made this quite hard to figure out the issue

Environment:
nuxt-auth-utils Version: ^0.0.22
Nuxt Version: 3.11.2
Browser/Node.js Version: Chrome 123.0.6312.107
Platform: Macos Sonoma

I will try to work on this and raise a PR

Closing as duplicate of #41
One solution at the moment is to only keep the minimal & required data for the backend in the session.
And use the session hooks to extend for the frontend (bio is an example)


Turbolinks Support

Using a callback in lib/gmaps4rails/js_builder.rb adds support for the turbolinks gem. This should, thereby, solve Issue #320.
I'm sorry, I couldn't test if this breaks your tests, since the tests for the master branch are not green on my machine (before changing anything).
What I've changed:
If you can't use this pull request, here's what I did:
In lib/gmaps4rails/js_builder.rb, in create_js, within if load_map?, add the following line:
@js << "$(document).on('page:load',  function() { if( $('.gmaps4rails_map').length > 0 ) { Gmaps.loadMaps(); } });"

This will create a callback for turbolinks, which triggers Gmaps.loadMaps() when a page is loaded via turbolinks. The line also checks if there's a map displayed on the page. Because otherwise, if you first open a page with a map, and then go to another page without map, turbolink will still call this callback.
Edit: Include API in header

In order to include the api in the html head, one may use the new gmaps4rails_api_script_tags helper method.
Including the api in the header is turned on by:
a) including the turbolinks gem (then the constant Turbolinks is defined)
b) adding this configuration in an initializer: YourApp::Application.config.gmaps4rails = { include_api_in_header: true }

Test this on your project
To test this modification on another project or your test app, add my fork to the Gemfile of the test project, temporarily:
gem 'gmaps4rails', '>= 2.0.1', git: 'https://github.com/fiedl/Google-Maps-for-Rails.git'

Furthermore, add the gmaps4rails_api_script_tags to the html head, as shown in this gist: https://gist.github.com/fiedl/6600555

I'm doing nuts over here and can't figure out how to get this gem to work on first app load.  It works find after 1 single page reload and fine from then on.  But it's really annoying this first page problem. Is there no simple solution?

have all scripts in all pages, otherwise turbolinks screws you


'dotnet publish' command runs a restore even with the --no-build --no-restore flags.

We are running dotnet CLI tasks in an Azure DevOps build pipeline. We run a 'dotnet restore' and 'dotnet test' before the 'dotnet publish' command. Even with the --no-build and --no-restore flags, the command is still restoring packages as seen in the output:
C:\hostedtoolcache\windows\dotnet\dotnet.exe publish D:\a\1\s\src\WebUI\PROJECT.csproj --configuration Release --output D:\a\1\a\PROJECT --no-build --no-restore
Microsoft (R) Build Engine version 17.1.1+a02f73656 for .NET
Copyright (C) Microsoft Corporation. All rights reserved.

  added 1242 packages in 6m

This issue can be closed. This was due to us restoring NuGet packages beforehand but not restoring NPM packages. The 'added X packages in Xm' output is from npm install.


Tetrahedral mesh features

Tetrahedral data structure to be compared

All main concerns should be addressed by the current state of the PR. The remaining issues is, we use for part of the functionality VTK in the back. This might be too slow for e.g. the barycentric coordinate calculation. It might be worth replacing that later.


No weights

The links to weights are all dead, no "releases" section where they point. Is it supposed to be implemented in the future?

Uh, sorry, I changed the release name and forgotten the links... fixed.

Uh, sorry, I changed the release name and forgot the links... fixed.

Thanks, leondgarde!


Improvement to identification error in Speaker Diarization

Linked issue(s):
Fixes KOL-3800
What change does this PR introduce and why?
Changes Identification error to include FN (previously it was only FP)
Please check if the PR fulfills these requirements

[x] Include reference to internal ticket and/or GitHub issue "Fixes #NNNN" (if applicable)
[ ] Relevant tests for the changes have been added
[x] Relevant docs have been added / updated

Can you add a link to the latest results?

https://trunk.kolena.io/bill/results?modelIds=9&workflow=Speaker+Diarization&testSuiteId=76&evaluators=N4Ig7AbCBcoJwwIwBYBMBfdQ
Latest results can be found here


Quick Fix: Loading %

As it scrapes all 76000 We want it to print out the % it is at currently. Super easy just takes a few minutes.

Not super important or breaking but if the database is empty it takes 20-30 minutes to populate our data sources. Would be nice in console to have a percentae log of how many left.


⚠ Bofrost.es has degraded performance

In 0c99e0c, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1630 ms

Resolved: Bofrost.es performance has improved in d846f22 after 27 minutes.


Test against node v7

Adds node v7 to .travis.yml.

Coverage remained the same at 100.0% when pulling 381c5dc5b32f03bbf84bc0c104bc4288058116fc on wbreakell:update-travis into fb16b0289e3ab9db0316907d72f3d366bf47f634 on reactjs:master.

Current coverage is 100% (diff: 100%)

Merging #189 into master will not change coverage

@@           master   #189   diff @@
====================================
  Files           1      1
  Lines          12     12
  Methods         0      0
  Messages        0      0
  Branches        0      0
====================================
  Hits           12     12
  Misses          0      0
  Partials        0      0

Powered by Codecov. Last update fb16b02...381c5dc

Thank you


[Suggestion] Modal component, hope to click "Primary Button" to display the loading effect

Hope to click "Primary Button" to display the loading effect

Like this

@PrintNow
Thank you for your suggestion, we will work on it.


imshow not working from a script

I can use imshow in an interactive prompt:
rnice8-0207:/mntdirect/_users/wright % python
Python 2.7.9 (default, Jun 29 2016, 13:08:31)
[GCC 4.9.2] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> import numpy as np, silx.sx
>>> silx.sx.imshow( np.random.random( ( 10, 10) ))
<silx.gui.plot.PlotWindow.Plot2D object at 0x7f6b0e693b98>
>>>

But not when I run the same commands in a script:
rnice8-0207:/mntdirect/_users/wright % python test.py
QWidget: Must construct a QApplication before a QPaintDevice
Aborted
rnice8-0207:/mntdirect/_users/wright % cat test.py
import numpy as np, silx.sx
silx.sx.imshow( np.random.random( ( 10, 10) ))
raw_input("done yet?")

I was expecting it to behave in a way that is more similar to matlab/octave/pylab versions of imshow.

silx.sx initial goal was to be used from the console.
Currently it is not creating the QApplication when loaded from a script, but I don't see specific reason not to do it all the time.


Canceling report generation

Is there a way to cancel report generation? I cant seem to find any sollution.
Thanks in advance.

What is “report generation” in this context?

Lets say when i pass tons of data to pivotUi() and dont want to wait for it to finish processing all the data and want to cancel the execution.
The for loop in line 329 in pivot.coffee

Ah i see. No, there is feature for interrupting this process.

Too bad, maybe an idea for future implementations :)


style(SliceControl): Appease prettier

:tada: This PR is included in version 2.8.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


remove noarch: python

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[N/A] Reset the build number to 0 (if the version changed)
[N/A] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)  (done 24 hours ago)
[N/A] Ensured the license file is being packaged. (assume the package author did)

Fixes #23.

Wheels for Python 2/3 contained different source code, so the single (Python 3) tarball doesn't work for Python 2.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

Non noarch packages should have python requirement without any version constraints.
Non noarch packages should have python requirement without any version constraints.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin rerender

@conda-forge-admin rerender

This PR should add a constraint on python.

I'm not sure. It uses 2to3, which we typically don't make noarch. Admittedly we don't care about building the Python 2 part any more. Though I'm not sure how 2to3 behaves on different Python 3 versions. Since it is part of Python, it is possible that the 2to3 code changes in noticeable ways between Python minor versions.

cc @mariusvniekerk any thoughts here?

@conda-forge/nose ready to merge!

I ask, because I'm not getting build errors in a downstream package that tests against Python 2.7 still using nose. I'm guessing that a Python 3 only compatible version is being installed in my Python 2 conda environment.

Please see issue ( https://github.com/conda-forge/nose-feedstock/issues/23 )

That's the same error I'm now getting in my builds. So if I need nose in a Py 2.7 environment, how do I get that? I can't say nose<1.3.7 because the version hasn't changed in 5 years. Is there a prior build number that works with p27? Can I use that somehow?

Can you please file a new issue sharing what is being run and what the output is? We made a number of changes in the past few days and it's not clear how this issue is being run into since those changes.

Sure. Thanks.

I just reran my CI build where I was getting the failure and no longer do, so maybe I was pulling in nose prior to these new changes before. If anything else comes up I'll make an issue. Good for now.

Thanks for the update. Glad things are working again 🙂


minor formatting issues in tutorial

https://docs.microsoft.com/en-us/office/dev/add-ins/tutorials/word-tutorial?tutorial-step=6
has formatting issues on the page starting from
```js function

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

ID: 65a8acc8-a2b2-4601-d705-269b11d95d10
Version Independent ID: ec2b21d4-cae6-e1f4-a2ee-937af2ff5d06
Content: Word add-in tutorial
Content Source: docs/tutorials/word-tutorial.yml
Product: office
GitHub Login: @o365devx
Microsoft Alias: o365devx

Thanks. Fixed now.


refresh categoryMappedFilters after loading attributes in PLP

Related Issues

closes #4695
Short Description and Why It's Useful

More info about solution https://github.com/DivanteLtd/vue-storefront/issues/4695#issuecomment-671460740
Which Environment This Relates To
Check your case. In case of any doubts please read about Release Cycle

[ ] Test version (https://test.storefrontcloud.io) - this is a new feature or improvement for Vue Storefront. I've created branch from develop branch and want to merge it back to develop
[ ] RC version (https://next.storefrontcloud.io) - this is a stabilisation fix for Release Candidate of Vue Storefront. I've created branch from release branch and want to merge it back to release
[x] Stable version (https://demo.storefrontcloud.io) - this is an important fix for current stable version. I've created branch from hotfix or master branch and want to merge it back to hotfix

Upgrade Notes and Changelog

[x] No upgrade steps required (100% backward compatibility and no breaking changes)
[x] I've updated the Upgrade notes and Changelog on how to port existing Vue Storefront sites with this new feature

IMPORTANT NOTICE - Remember to update CHANGELOG.md with description of your change
Contribution and Currently Important Rules Acceptance

[x] I read and followed contribution rules

 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.


.net wont start

Problem encountered on https://dotnet.microsoft.com/learn/dotnet/hello-world-tutorial/install
Operating System: windows
Provide details about the problem you're experiencing. Include your operating system version, exact error message, code sample, and anything else that is relevant.

Won't start how? Do you get an error message?


[$500] [Distance] - Request money does not work with email & phone number

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

Action Performed:

Go to + > request money > distance > choose start and end point > next
Observe it says name, email and phone number
Attempt to add an email. Verify you cannot create a request money report
Attempt to change the amount. Verify that you are unable to.
Attempt to add a phone number. Verify you cannot create a request money report

Expected Result:
Since email and phone number are said to be accepted, request money reports should be created using both.
Actual Result:
Email and phone number does not have reimbursement detail and hence, a report cannot be created but mentioning email and phone number can be misleading
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/fdf8a1f3-096b-4652-b2f2-dae20b210519
https://github.com/Expensify/App/assets/93399543/621d8e52-358c-45dd-9d04-3346be289017
Expensify/Expensify Issue URL:
Issue reported by: @avi-shek-jha
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1693872152503239
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01784d0162b22d689f
Upwork Job ID:<PHONE_NUMBER>770354176
Last Price Increase: 2023-09-08

Proposal
Please re-state the problem that we are trying to solve in this issue.
Request money does not work with email & phone number
What is the root cause of that problem?
We already disable includeP2P in getRequestOptions which mean we can't send request to normal users.
https://github.com/Expensify/App/blob/f896f71a1ce00320a2ba029e44396d4e29b88223/src/pages/iou/steps/MoneyRequstParticipantsPage/MoneyRequestParticipantsSelector.js#L92-L93
But in OptionsListUtils.js, we still send userToInvite while includeP2P = false, and send it base on canInviteUser only
https://github.com/Expensify/App/blob/f896f71a1ce00320a2ba029e44396d4e29b88223/src/libs/OptionsListUtils.js#L1044
What changes do you think we should make in order to solve the problem?
We need to prevent send userToInvite when includeP2P is false here.
// userToInvite: canInviteUser ? userToInvite : null,
userToInvite: includeP2P && canInviteUser ? userToInvite : null,

What alternative solutions did you explore? (Optional)

hm, it looks like melv assigned sasha to this one first so i am going to un-assign myself from this one.

I think the issue here is that when you send a distance request to an email instead of to a workspace, there is no mileage rate set up. I'm going to post this in Slack to see if we've considered this case already.

This is a known limitation that we haven't built yet. I'm closing this in favor of https://github.com/Expensify/App/issues/23291 which is where we're building the functionality.


How to get pretrained weights of  yolo-nas on obj365 datasets

💡 Your Question
How to get pretrained weights of  yolo-nas_s on obj365 datasets? yolo-nas got a mAP of 47.5, is this the result of pre-training on the obj365 dataset and then fine-tuning on the coco dataset?
Versions
No response

Objects365 was part of pretraining of Yolo-NAS. However we do not provide pretrained weights for Objects365, since there wasn't any goal to reach best possible accuracy on that dataset.
So Yolo-NAS weights for Objects365 will not be released.

Are pre-trained weights of yolo-nas_s on coco (or any other dataset) released?


bevy_ui build error

Bevy version
bevy = { version = "0.11", features = ["dynamic_linking"] }
Relevant system information
cargo 1.76.0-nightly (9b13310ca 2023-11-24) or cargo 1.74.0 (ecb9851af 2023-10-18)
Linux  6.6.3-200.fc39.x86_64 #1 SMP PREEMPT_DYNAMIC Tue Nov 28 19:11:52 UTC 2023 x86_64 GNU/Linux
What you did
Tried to build a bevy program.
What went wrong
Build error, as below.
Additional information
error[E0063]: missing fields `grid_auto_columns`, `grid_auto_flow`, `grid_auto_rows` and 5 other fields in initializer of `taffy::prelude::Style`
  --> /home/martin/.cargo/registry/src/index.crates.io-6f17d22bba15001f/bevy_ui-0.10.1/src/flex/convert.rs:99:5
   |
99 |     taffy::style::Style {
   |     ^^^^^^^^^^^^^^^^^^^ missing `grid_auto_columns`, `grid_auto_flow`, `grid_auto_rows` and 5 other fields

This is occurring because you have duplicate incompatible versions of Bevy in your dependency tree. Use cargo tree to help root out the problem.


fix: update siblingNode

closes https://github.com/privatenumber/vue-frag/issues/16

:tada: This PR is included in version 1.2.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Error when picking Create new local app setting from local.settings.json when creating blob trigger (Error getSubscriptionPromptStep)

Repro steps:

Create new Function from the workspace
Create new project when prompted
Select JavaScript (happens on any language)
Select Azure Blob Storage trigger
Select "BlobTrigger1" as function name
On "Select setting from "local.settings.json" ", choose +Create new local app setting

Expected:
New value created and written to local.settings.json upon creation
Actual:
Error appears with "Cannot read properties of undefined (reading 'getSubscriptionPromptStep')

Thanks for reporting the issue, I'm able to reproduce this.
Notes:
Issue is with this line https://github.com/microsoft/vscode-azurefunctions/blob/411ece5f9453af075c1ff48c70aec349f5942a47/src/commands/addBinding/settingSteps/LocalAppSettingListStep.ts#L75

On "Select setting from "local.settings.json" ", choose +Create new local app setting

Doesn't reproduce if you pick an existing setting on this step.

@alexweininger thank you very much. Please how do one pick an existing setting on this step?
I am having the same issue

Repro steps:

Create new Function from the workspace
Create new project when prompted
Select JavaScript (happens on any language)
Select Azure Blob Storage trigger
Select "BlobTrigger1" as function name
On "Select setting from "local.settings.json" ", choose +Create new local app setting

Expected: New value created and written to local.settings.json upon creation
Actual: Error appears with "Cannot read properties of undefined (reading 'getSubscriptionPromptStep')

Please have you found a fix around this?

I downgraded the Azure Function extension to v1.6.2 as a work-around.

Until we release a fix for this issue, as dwoldo mentioned above, a valid workaround is to install the previous version of the extension.
To do this, go to the Azure Functions page in the Extensions explorer and using the dropdown on the "Uninstall" button, select "Install another version..."


Add support for Microsoft SQL Server 2016+

Hi All,
Very interested in this project, looks fantastic to me!.
I see a significant and probably quite common use case here where developers and the business will want to be reassured of a successful migration from an old legacy system to a cloud platform such as Snowflake.
As a consultant,  I see a huge number of clients on MS SQL Server, and this is the source of the data for their migration from on-prem to the cloud.
I'm excited to use this tool but unfortunately it does not support most of the use cases I would want to use it for, due to the lack of MS SQL Support. To this end, I would like to suggest support for this platform.
I'm also more than happy to contribute to this, and I'm wondering if anyone can point me in the right direction for contributing please? I see the developer environment  guide in the README, but what I mean is more specific information about creating a new adapter/adding new database support.
Thanks!

Hello. Thanks for reaching us!
MS SQL is one of the databases on the near-future roadmap. It has a few performance-related challenges though.
To make a new driver, take a look at the existing ones here: https://github.com/datafold/data-diff/tree/master/data_diff/databases — and do the same. Then also run the tests, as documented (CONTRIBUTING.md)[https://github.com/datafold/data-diff/blob/master/CONTRIBUTING.md].
If you have any questions, feel free to ask.

Hello. Thanks for reaching us!
MS SQL is one of the databases on the near-future roadmap. It has a few performance-related challenges though.
To make a new driver, take a look at the existing ones here: https://github.com/datafold/data-diff/tree/master/data_diff/databases — and do the same. Then also run the tests, as documented (CONTRIBUTING.md)[https://github.com/datafold/data-diff/blob/master/CONTRIBUTING.md].
If you have any questions, feel free to ask.

Thanks for your response!
I see there is a commented out mssql module. Anything I should know about this?

@DVAlexHiggs You can see more details on that in #51.

Not sure how this got closed. Think I mis-clicked

@DVAlexHiggs Any update on this? I also could help contribute, but don't want to pick it up if you are already actively working on it.

@cacondie ,
We are currently not pursuing this avenue, but we will be happy to accept contributions. However, be aware that the solution isn't likely to be simple.
SqlServer does support MD5 (which is what we used for hashing), but it is approx. 100 times slower than postgresql, which makes it unusable for practical purposes.
I think the solution would have to involve implementing our own checksum, probably using only simple arithmetic operations, since SQL isn't capable of much more.
Keep in mind that whatever checksum is used for SQLServer, it has to be supported by all the other databases, so that comparisons are possible. That probably means implementing this new checksum for each one. (or at least the major ones)
If you can think of a new creative solution, we'll be happy to consider it.

Probably not as many SQL Server implementations, including just about anything on cloud storage, disallow CLR procs.

I proposed a solution to this issue in #51


Alert Details broken after 23-04 Cumulative Update

When checking Quality updates and Update status alert in Windows Update Business report.

And click more details from AlertName link. It shows Alert Description and Recommendaions multiple times.

@microsoft/update-compliance-workbooks can you look at this?

@adeolau-ms is this still an issue?  trying to clean up old issues here.


Switch from jenkins-persistent to jenkins-ephemeral in order to avoid…

… NFS issues
CC: @ALRubinger

Closed by #10


master/slave → primary/replica (take 2)

This PR reimplements the changes made in #18102 in a definitely non-breaking way, as suggested by @rob006 (https://github.com/yiisoft/yii2/pull/18102#issuecomment-643847335).

The old master/slave properties are back, but deprecated.
The new primary/replica properties are now implemented by magic getter/setter methods, and documented with @property tags.
All Yii code is now referencing the deprecated properties/methods, to ensure that any overridden properties/methods are still used.

You can view the combined changes made by #18102 and this PR here: https://github.com/brandonkelly/yii2/compare/9a3093442c8f7a39d1fa5972b58127266d8504f7...61ead1d890b589db98834205cc92f5b01667c1db

Q
A

Is bugfix?
❌

New feature?
❌

Breaks BC?
❌

Tests pass?
✔

This change is against good coding practices, since now you have two names for things which are exactly the same.
For Yii2 you can not entirely deprecate it, since it would require a new major version. Also, we are in feature/code-freeze state for Yii2, but are adding useless new properties.
To my understanding a slave and a replica are two different things, even for db-servers.
PS: How do I call a primary master and secondary master btw... primary primary and secondary primary?
PS2: And, what's the new name for the master branch, btw?

@schmunk42 See #18102 for context.

This change is against good coding practices, since now you have two names for things which are exactly the same.

It’s not a bad practice as long as it’s clear which one should be used, which is precisely why you the old properties/methods are now deprecated.

For Yii2 you can not entirely deprecate it, since it would require a new major version. Also, we are in feature/code-freeze state for Yii2, but are adding useless new properties.

Only breaking changes warrant new major versions. Deprecations are not breaking changes; they are warnings of future breaking changes. If this is going to be a breaking change in Yii 3, then the earlier it can be deprecated in Yii 2, the better, so people have more time to prepare.

To my understanding a slave and a replica are two different things, even for db-servers.
PS: How do I call a primary master and secondary master btw... primary primary and secondary primary?

We are going with modern naming conventions here – see https://github.com/yiisoft/yii2/pull/18102#issuecomment-643662745 and the following comment.

PS2: And, what's the new name for the master branch, btw?

GitHub is switching to main by default, but that is obviously not something you can change via a PR, nor would it have any effect on the installable Yii package, so I’ll leave it up to the Yii team to decide whether they care to make that change.

Please also remove all whitespaces from the codebase.
The next thing will be to rename parent to super I guess, and inherited classes will need to check their privilege. :roll_eyes:

We can think about changes like that for Yii 3, if they make sense, but for Yii 2 it's just actionism.
My 2 cents.

It’s a pretty insignificant change that moves Yii along with modern conventions & makes the code/docs more inclusive, with zero to low impact on your project depending on your level of OCD. Seemed like an obvious v2 change to me. ¯\_(ツ)_/¯

If this is going to be a breaking change in Yii 3, then the earlier it can be deprecated in Yii 2, the better, so people have more time to prepare.

That could make sense if renaming was done for Yii 3, and then deprecations are added to Yii 2 to make migration smoother (assuming that there will be sane migration path from Yii 2 to Yii 3, which I highly doubt). But Yii 3 still uses "non-modern" terminology, and it probably will be another fresh start without migrations path (from Yii 2) in mind. So at this point this is deprecation for the sake of deprecation - it only brings more work for people to adjust their code to new naming, without any benefits from this change.

I want to get to that. One thing at a time.

I want to get to that.

You should start with that. How can you predict how these deprecations will simplify migration to Yii 3 if you don't know how these interfaces will look like in Yii 3. Most of deprecated properties does not exist in Yii 3.

BTW:

It’s not a bad practice as long as it’s clear which one should be used, which is precisely why the old properties/methods are now deprecated.

This is not that simple. If I need to override one of the changed methods, should I override original deprecated method (getSlave()), or new method (getReplica())? Old one is deprecated, so at first glance I should override the new method. But if I do that, old code and extensions which still use the old method will ignore my override - I need to override deprecated method to make sure everything works, which is counter-intuitive.
Similarly with properties - if I want to redefine property with new default value, I need to use deprecated property name. If I define property using new name (for example public $enableReplicas = false;) it will be ignored, since this property is never used directly (and it will also make related getter and setter useless). If I really need to use new interface to set default value for such property, I need to override constructor, which is much more tricky.
Redundant interfaces are confusing and create additional surface for mistakes. It is "necessary evil" if you want to provide migration path to new version with BC break, but if there is no migration path, "necessary" does not apply and only "evil" is left.

@samdark please read the above posting from @rob006 and reconsider this merge

You’re the one that suggested we go this route!

Seems both current and previous approaches to renaming are technically problematic, not fully backwards compatible and likely cause issues :(

@brandonkelly thanks for trying it but I'm going to revert these changes.
We can use new terms in Yii 3 where there's no backwards compatibility yet. In case there will be a way to do renaming in Yii 2 without causing issues, I'll gladly merge it.
@rob006 thanks for re-checking technical side of things.

Created a task in Yii 3 Trello board not to forget it: https://trello.com/c/GiGjG2y5/69-update-terminology

Alright, no worries. We will just make the same changes in our own extension of yii\db\Connection, and provide our own read-write splitting docs. Thanks for being responsive.

You’re the one that suggested we go this route!

I'm not sure if you're talking to me, but I was against this renaming in the first place. This PR is much better than the first one, since it is more BC and less magic, but it is still much worse than not touching anything and leaving old master-slave terminology.


nacos和spring-cloud-config混用报错，加载顺序有问题

Error creating bean with name 'configServerInstanceProvider' defined in class path resource [org/springframework/cloud/config/client/DiscoveryClientConfigServiceBootstrapConfiguration.class]: Unsatisfied dependency expressed through method 'configServerInstanceProvider' parameter 0; nested exception is org.springframework.beans.factory.NoSuchBeanDefinitionException: No qualifying bean of type 'org.springframework.cloud.client.discovery.DiscoveryClient' available: expected at least 1 bean which qualifies as autowire candidate. Dependency annotations: {}

refer https://github.com/alibaba/spring-cloud-alibaba/issues/708, it has been fixed.


[UI] Do not show pod security policy selection if not enabled

What kind of request is this (question/bug/enhancement/feature request):
bug
Steps to reproduce (least amount of steps as possible):

do not enable psp for a cluster
go to add project to cluster

Result:
user can set psp
Backend Issue:
https://github.com/rancher/rancher/issues/21548
Other details that may be helpful:
Once backend is merged, cluster will have a capabilities field called "pspEnabled". This will provide the necessary info for showing or not showing this option.

@vincent99 I cant really remember if we said 1) "dedicated UI for this" or 2) "there's already an annotations section on the cluster create page so we can just tell the user (and document) how to do it via the annotation"
Do you recall? I think 2 could be sufficient for now

had a commented and deleted it bc i was thinking of something else, but now im writing this bc i know vincent is going to see that ghost comment and woner what happened

closing and will combine with https://github.com/rancher/rancher/issues/23730


Compile error: taking the address of a temporary object

When trying to compile the engine with GodotSteam on Linux, I get the following error:

modules/godotsteam/godotsteam.cpp:1345:105: error: taking the address of a temporary object
of type 'CSteamID' [-Waddress-of-temporary]
...&serverIP, &serverPort, &(CSteamID)serverID);

The same happens when cross-compiling for Windows:

modules/godotsteam/godotsteam.cpp: In member function 'Dictionary Steam::getLobbyGameServer(uint64)':
modules/godotsteam/godotsteam.cpp:1345:116: error: taking address of temporary [-fpermissive]
->GetLobbyGameServer((CSteamID)steamIDLobby, &serverIP, &serverPort, &(CSteamID)serverID);
^~~~~~~~

Hey there.  Yeah, I noticed it when I was making the pre-compiles for Linux.  Oddly I do not get these errors when building in Windows, but they are a problem nonetheless.   I'll try to get them sorted out today.

OK, I have made changes that will allow compiling in Linux and -should- work with cross-compiling for Windows from Linux.  There are additional fixes to the Friends functions.

That has fixed it. Both compiling for Linux and cross-compiling for Windows work. Thank you!

Awesome!  Very welcome, glad it's all sorted!


Can I use Faster RCNN as a backbone Detector?

Sure! SIGMA works on the image-level feature and doesn't depend on specific detection heads, meaning that it can be extended to different detection algorithms, even for segmentation tasks. We had tried a quick (naive) implementation on the famous DAFRCNN (IJCV) framework and found it can still work. We may provide a Faster RCNN based implementation in our extended journal version. But for now, I think that you have to try it by yourself.
Actually, this framework contains an unfinished Faster RCNN implementation, and I hope to unify it with DAFRCNN (IJCV) in the future if time is available.


RuntimeException_1 {detailMessage: "Document had a mysterious sink.  Restoration is unsafe"

I am getting this in the console when refreshing the window. @pablojan do you know what can be happening?
RuntimeException_1 {detailMessage: "Document had a mysterious sink.  Restoration is unsafe", stackTrace: Array[33]}
detailMessage: "Document had a mysterious sink.
__proto__: seedTable.(anonymous function)
17: StackTraceElement_0
className_0: "Unknown"
fileName: "http://localhost:8000/js/app.min.js@19"
lineNumber: -1
methodName: "anonymous"
__proto__: seedTable.(anonymous function)
18: StackTraceElement_0
className_0: "Unknown"
fileName: "http://localhost:8000/js/app.min.js@7"
lineNumber: -1
methodName: "options.domOperation"
__proto__: seedTable.(anonymous function)
19: StackTraceElement_0
className_0: "Unknown"
fileName: "http://localhost:8000/js/app.min.js@19"
lineNumber: -1
methodName: "close"
__proto__: seedTable.(anonymous function)
20: StackTraceElement_0
className_0: "Unknown"
fileName: "http://localhost:8000/js/app.min.js@9"
lineNumber: -1
methodName: "closeAndReturnNoopAnimator"
__proto__: seedTable.(anonymous function)
21: StackTraceElement_0
className_0: "Unknown"
fileName: "http://localhost:8000/js/app.min.js@16"
lineNumber: -1
methodName: "init"
__proto__: seedTable.(anonymous function)
22: StackTraceElement_0
className_0: "Unknown"
fileName: "http://localhost:8000/js/app.min.js@22"
lineNumber: -1
methodName: "prepareRegularAnimation"
__proto__: seedTable.(anonymous function)
23: StackTraceElement_0
className_0: "Unknown"
fileName: "http://localhost:8000/js/app.min.js@13"
lineNumber: -1
methodName: "initDriverFn"
__proto__: seedTable.(anonymous function)
24: StackTraceElement_0
className_0: "Unknown"
fileName: "http://localhost:8000/js/app.min.js@24"
lineNumber: -1
methodName: "invokeFirstDriver"
__proto__: seedTable.(anonymous function)
25: StackTraceElement_0
className_0: "Unknown"
fileName: "http://localhost:8000/js/app.min.js@31"
lineNumber: -1
methodName: "triggerAnimationStart"
__proto__: seedTable.(anonymous function)
26: StackTraceElement_0
className_0: "Unknown"
fileName: "http://localhost:8000/js/app.min.js@13"
lineNumber: -1
methodName: "anonymous"
__proto__: seedTable.(anonymous function)
27: StackTraceElement_0
className_0: "Unknown"
fileName: "http://localhost:8000/js/app.min.js@20"
lineNumber: -1
methodName: "forEach"
__proto__: seedTable.(anonymous function)
28: StackTraceElement_0
className_0: "Unknown"
fileName: "http://localhost:8000/js/app.min.js@9"
lineNumber: -1
methodName: "anonymous"
__proto__: seedTable.(anonymous function)
29: StackTraceElement_0
className_0: "Unknown"
fileName: "http://localhost:8000/js/app.min.js@36"
lineNumber: -1
methodName: "$digest"
__proto__: seedTable.(anonymous function)
30: StackTraceElement_0
className_0: "Unknown"
fileName: "http://localhost:8000/js/app.min.js@24"
lineNumber: -1
methodName: "$apply"
__proto__: seedTable.(anonymous function)
31: StackTraceElement_0
className_0: "Unknown"
fileName: "http://localhost:8000/js/app.min.js@23"
lineNumber: -1
methodName: "<anonymous>"
__proto__: seedTable.(anonymous function)
32: StackTraceElement_0
className_0: "Unknown"
fileName: "http://localhost:8000/js/app.min.js@21"
lineNumber: -1
methodName: "eventHandler"
__proto__: seedTable.(anonymous function)

Hard to say without more research, but it might be that the editor or doc used after refresh hasn't been properly initialised again. I will try to reproduce. Do you load again all swellrt stuff in the refresh?
El 2/3/2016 3:36 p. m., Antonio Tapiador<EMAIL_ADDRESS>escribió:
I am getting this in the console when refreshing the window. @pablojan do you know what can be happening?
RuntimeException_1 {detailMessage: "Document had a mysterious sink. Restoration is unsafe", stackTrace: Array[33]} detailMessage: "Document had a mysterious sink.

Ok, it probably has to do with the way I am using angular in some point of the redesign. I will take a workaround
Thank you for the feedback


add nguy to maintainers

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Superseded by #14.


ENSCORESW-2553: retrieve correct accession for peptides

Requirements

Filling out the template is required. Any pull request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion;
Review the contributing guidelines for this repository; remember in particular:

do not modify code without testing for regression
provide simple unit tests to test the changes
if you change the schema you must patch the test databases as well, see Updating the schema
the PR must not fail unit testing

Description
Using one or more sentences, describe in detail the proposed changes.
Retrieve RefSeq accessions for peptide when it is not stored as a stable ID
Use case
Describe the problem. Please provide an example representing the motivation behind the need for having these changes in place.
For some species, the refseq_import format changes. The stable id is a randomly generated unique ID and the refseq accession is stored as an xref.
The code was updated to deal with this change but only transcript accessions were correctly retrieved. This change deals with the peptide accessions.
Benefits
If applicable, describe the advantages the changes will have.
We have RefSeq accessions for peptides.
Possible Drawbacks
If applicable, describe any possible undesirable consequence of the changes.
Testing
Have you added/modified unit tests to test the changes?
This was tested on the latest human refseq import in the new format and correctly stores the accessions and versions.
If so, do the tests pass/fail?
The results are now as expected
Have you run the entire test suite and no regression was detected?
NA

Coverage remained the same at 81.126% when pulling 84352fcfb01df6d6017231f6e76424328e4b07c4 on bugfix/refseq_coordinate into 09f9c7bd58a12c556abf122e7e5a944b7f8befe6 on master.


Improve sequence control messages

I think we can similarly improve these messages for other backends as well?

This is specifically for sequence control messages but yes I will look into the other backends as well

Look here for plan_backend: https://github.com/NVIDIA/tensorrt-inference-server/blob/18724376b845bfcc62344cf4a99e17696bb9e031/src/backends/tensorrt/plan_backend.cc#L520
It will be good to have a common format.


Bad display of the content of a station

Thank you for this great mod.
I have a problem with one of my game. In the list of station. one of the provider station "Iron 1" display the total inventory instead of its contents.
Here is a save of the game:
https://drive.google.com/open?id=1e1Z0ssUV0OYAL9p2HVBu3Izc4UMY0dvm

Hi, Raiguard!
I have the same problem. It seems that the first provider/requester (lowest station ID?) in every network ID displays the total contents of all providers/requesters in this network.

For example, i have 2 requesters in the network 524288. The first requests crushed stone, the seconds requests slag. But LtnManager displays that the first request both, while the second displayed ok.
The next 3 stations are providers. First provides slag, second - mineralized water, third - carbon dioxide. And again, LtnManger displays that the first provides all 3 sort of items.
Same issue with default network ID (-1). Not presented on screeshot.
P.S. I use Angel's and Bob's mods.
Thanks for your efforts!

So, funny story... I was actually experiencing this bug the entire time that I was developing this mod, and even noticed it a few times, but put it off for "fixing later". I thought the cause would be really hard to pin down, but it was actually quite easy.
Anyway, fixed for the next release!


TestingDB support nested field selection using dots ({"foo.bar": 1})

Currently this results in models.GetActiveProfilesWithOnMatch() returning an empty list.
The above makes the ScanCV route useless using the testing db

Fixed in 2657e3f2c46817bac5e7295a6ede20b5f4bbc1ae 2ced9e41a53861b24b45b18d5d46f4c82f4e476a and 42e8a673b30db363ef78be877d74ac67ba9cb309


Why my pool of yarn couldn't shows the same and DS queue?

*For better global communication, please give priority to using English description, thx! *
Please review https://dolphinscheduler.apache.org/en-us/docs/development/issue.html when describe an issue.
Describe the question
Why my pool of yarn couldn't shows the same and DS queue?
Which version of DolphinScheduler:
-[1.3.2-preview]
Which version of CDH:
6.2.1
Additional context
I have some questions about the queue of ds.The document shows the queue of ds is due to MapReduce,Spark and so on.But,I find that the logs of yarn(host:19888) shows the queue is tenant code of ds,not the queue of ds.Why ?
And the document shows the tenant code is the user of Linux.So,the queue of yarn is the tenant code?
Therefore,I test the tenant and queue of ds.Here are the results.
The following is the configuration of DS and Resources Pools of yarn.

The image is the queue of DS.

The image is the users of DS.

The image is the tenant of DS.

The image is all the pools of yarn.

The image is the resource pool placement rules of yarn.

The image is the resource pool placement rules of yarn also.

The image is the logs of yarn.
Requirement or improvement

Please describe about your requirements or improvement suggestions.

who can look the problem?

Priority of queue: queue from other args > user queue > tenant queue
Flink: 1.3.x has bug about queue, so that the job will be submitted to the default queue regardless of the existing user or tenant queue. The latest dev branch has fixed it.
Spark: No problem
MR: No problem
Can this information help you explain the current problem?

User
Tenant
User Queue   (Submitted)
Tenant Queue   (Ignored)
Yarn Queue   (Actual)

1
ddyw
root.hdfs.flink
root.hdfs.mapreduce
root.hdfs.ddyw

2
dd011
root.hdfs.mapreduce
root.hdfs.spark
root.hdfs.dd011

3
ddyw1
root.hdfs.spark
root.hdfs.flink
root.hdfs.ddyw1

I guess the reason is as follows:
The tenant ddyw submits a job with user queue root.hdfs.flink, not the tenant queue root.hdfs.mapreduce, so that root.hdfs.flink doesn't take effect. It may be caused by queue permissions.
So the job doesn't meet rule 1 in Dynamic Resource Pool Configuration, so the second rule takes effect by using root..[username] with middle pool name hdfs.
In summary, the job submitted by the tenant ddyw will use root.hdfs.ddyw queue in yarn.

Close the question since it has been replied


Radar repro for nondeterminstic optional binding

Closing this out since I don’t need it, but the PR and branch will remain available for reproducing the relevant radar.


Results aren't accurate for search

var lunr = require('lunr');
var documents = [{
  "name": "Lunr",
  "text": "this Like Solr, but much smaller, and not as #bright."
}, {
  "name": "React",
  "text": "A JavaScript library for building user interfaces."
}, {
  "name": "Lodash",
  "text": "A modern JavaScript utility library delivering modularity, performance & extras."
}]

var idx = lunr(function () {
  this.ref('name')
  this.field('text')

  documents.forEach(function (doc) {
    this.add(doc)
  }, this)
})

console.log(idx.search("#bright"));

Should return match for #bright

@sosharma1403 One way to fix this would be to add lunr.trimmer to this.searchPipeline in the lunr function invocation.  @olivernn Do you think that would make sense to have in the search pipeline in the general case, or is there a reason that lunr.trimmer is in the indexing pipeline but not the search one?


SPR-8974 Fix regression in UriUtils.java

@poutsma, please review the change in UriUtils. I reverted as little as possible from the prior behavior. It turned out slightly more than initially expected but still a reasonable compromise in my opinion.

Looks good to me.


Show terminal command changes size of terminal

VSCode Version: VSCode Insiders
OS Version: Windows 10 (fully updated)

Steps to Reproduce:

Hit CTRL+` to show the terminal
Click ^ to "maximize" the terminal (which now hides the editor)
Hit CTRL+` to hide the terminal
Hit CTRL+` to show the terminal, and the terminal is now a different size, no longer hiding the editor.

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

/duplicate #80366


Manifest merger never ends.

The manifest merger task stays running forever (when run by apm)
I need to CTRL-C to stop execution, but then the tasks are still running (and accumulate).
ps aux showed that I had 8 processes running like this one:
/usr/bin/java -jar "/Users/jeko/Library/Application Support/airsdk.apm/Local Store/tools/manifest-merger-30.0.1.jar" --main /tmp/xxx/.tmp538437/AndroidManifest.xml --property PACKAGE=air.com.my.app --remove-tools-declarations
When ran manually, it exits almost immediately and show the result on the console.
It occasionally succeeds though, might it be a race condition?

Also, apm should make sure it kills it child processes when it is killed.

Can you give me more information on what packages you are using?

No package installed, still testing from the empty project. It's a bit weird, when it happens it would always happen. Now it doesn't happen anymore, didn't anything apart adding some logs... I'll see if I can reproduce.

Hmmm odd... haven't seen that before. What version of java is at /usr/bin/java on your machine?

It version 1.8.0_271.
This had me thinking and got a little more info: it happens when I set JAVA_HOME=/usr
Run this way it "breaks" (never ends):
AIR_HOME=~/Fovea_AIR_Toolchains/air-<IP_ADDRESS>5 JAVA_HOME=/usr ~/Documents/github/apm/client/out/apm generate app-descriptor
Run this way it works fine:
AIR_HOME=~/Fovea_AIR_Toolchains/air-<IP_ADDRESS>5 ~/Documents/github/apm/client/out/apm generate app-descriptor
(still testing from the empty project)

Strangely enough, when NOT setting JAVA_HOME to anything, I confirmed that RunConfig::getJava() returns /usr/bin/java...

After digging a little bit deeper, I found out that running the manifest-merger process manually this way (adding JAVA_HOME=/usr) will make it hang:
JAVA_HOME=/usr /usr/bin/java -jar "/Users/jeko/Library/Application Support/airsdk.apm/Local Store/tools/manifest-merger-30.0.1.jar" --main /tmp/xxx/.tmp845059/AndroidManifest.xml --property PACKAGE=air.com.my.app --remove-tools-declarations
So this tool is not happy when JAVA_HOME is set.

More details: I seem to ONLY have the issue when JAVA_HOME is set to /usr.
When running the manifest-merger manually, I can set it to a valid or invalid location, it will work (except for /usr).
When running apm, if I set JAVA_HOME to something like /Library/Java/JavaVirtualMachines/jdk1.8.0_25.jdk/Contents/Home, it goes through.
No idea what's wrong with JAVA_HOME=/usr...
I tried to see how to remove this environment variable when running the child process, but it doesn't look that easy (NativeProcess functions doesn't have this kind of parameters).
Alternatively, there could be some logic similar to what you did for windows, to use the latest JVM located in /Library/Java/JavaVirtualMachines.

... or use the java_home utility, which has an option to return a specific version if available: /usr/libexec/java_home -v 1.8

Since this looks more like an issue with manifest-merger-30.0.1.jar, I guess it's OK to close here.

Hi,
This is actually an issue with incorrectly setting JAVA_HOME to be /usr. JAVA_HOME is expected to point to a JDK/JRE installation not just the java executable.
This causes issues with a lot of java programs, in the update I've removed this as a default and use the java_home executable instead.
If you are still setting it to /usr on the command line I suggest you correct that.

If you are still setting it to /usr on the command line I suggest you correct that.

I'm not, this was my mistake then.


numpy dependency on 1.19.2

https://github.com/tensorflow/tensorflow/blob/master/tensorflow/tools/pip_package/setup.py#L81
why is there a dependency on 1.19.2 when there are later version of numpy?

See https://github.com/tensorflow/tensorflow/issues/47691

@ganand1 ,
As mentioned in this link, there is a bound on numpy versions supported by TF2.4 ( ~=1.19.2). Currently numpy version should be >=1.19.2 and < 1.20.
Also for future releases please subscribe to #47691
Thanks!

Closing as duplicate


Need to add additional maven url for "merge" dependency

When adding the dependency to my project i got the following error which was based on the com.commonsware.cwac:merge dependency.
Error
Error:Failed to resolve: com.commonsware.cwac:merge:1.1.1
In order to fix it i had to add the maven repo for cwac merge
repositories {
    maven { url "https://jitpack.io" } // sectioned merge adapter
    maven { url "https://s3.amazonaws.com/repo.commonsware.com" } // had to add this guy
}

Thanks for pointing out! I'll add this to the README file.


Git actions

Added git actions that:
Lint and Compile Main branch project
Checks java style

Added Git Actions that build project w/ Gradle in Main branch


applied fragments

resolves #245
@minedeljkovic how does this look? see the example

Codecov Report

Merging #247 into master will decrease coverage by 0.47%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #247      +/-   ##
==========================================
- Coverage   83.85%   83.37%   -0.48%
==========================================
  Files         107      108       +1
  Lines        1251     1257       +6
  Branches       22       27       +5
==========================================
- Hits         1049     1048       -1
- Misses        202      209       +7

Impacted Files
Coverage Δ

modules/core/src/main/scala/AppliedFragment.scala
0.00% <0.00%> (ø)

modules/core/src/main/scala/Fragment.scala
90.90% <0.00%> (-9.10%)
:arrow_down:

.../core/src/main/scala/syntax/StringContextOps.scala
77.77% <0.00%> (-22.23%)
:arrow_down:

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 5d16349...0f085d4. Read the comment docs.

This works great!
As a general solution, it also solves another problem that I didn't mention - conditionaly setting columns in an UPDATE statement.


Update repo resources

General Support Request
Update dependencies and repo support
Support request
Update the repo to support the latest versions of the issue templates and repo settings.
Possible Solution
Copy over newer versions of settings and issue templates. Also, run npm update to bring dependencies up to date.

Duplicate issue


Fix changelog and API docs

Fixes #Issue https://github.com/envoyproxy/envoy/issues/25791

added to #25512 in order to include in next patch release


Something doesn't seem right with Info-nodes

In Info-mode, when you're in the Top node, using Go To Node Info-goto-node doesn't show any nodes, only this:
*
Top This is the top of the INFO tree

It makes it unable to navigate to selected nodes.
Trying to type ( Tab and then searching for a topic shows some files, but the completion (tested with Vertico) is broken. It still won't allow you to search for a topic.
I'd try to type (elisp) but it won't let me insert closing paren, throws:
Error in post-command-hook (vertico--exhibit): (wrong-number-of-arguments #[(filename &optional noerror)

I thought it maybe due to smartparens, disabled it - same results
It doesn't seem like Info-directory-list get anyhow modified, anyone has any ideas?
Steps to reproduce

Open Emacs
M-x info
Note that the Node indicator at the top should say: "Node: Top"
M-x Info-goto-node
Try to jump for example to (elisp) node

System Information
https://pastebin.com/H7H6KWmD

I'm having the same issue.

Perhaps you need M-x Info-menu (gm when evil is enabled) rather than M-x Info-goto-node (gG when evil is enabled).

@condy0919 No, these are two different operations. One is to open a given Info manual, another is to jump within a manual. I ended up adding info+ package and binding a key to info-display-manual provided in the package.

@jkroes Ah, I see what's happening. Unfortunately, I am unable to select a topic. Vertico complains: Error in post-comman-hook (vertico--exhibit) (wrong-number-of-arguments)....
I'll try to search upstream, maybe someone has reported it there.

@agzam There is nothing wrong with Vertico in my tests. Please compare how the default completion UI behaves and compare the behavior with Vertico.
The issues being discussed here are the following:

Duplicate candidates. This is minad/vertico#69 and one could indeed consider this a Vertico issue. For now I avoided to filter out duplicate candidates in Vertico since I consider the presence of duplicate candidates a bug in the backend. By not filtering I wanted to see which tables are actually affected by this, and as it turns out there are only a very small number of these buggy tables. Filtering in the frontend is of course not a big problem, except that it leads to unnecessary allocations and a small performance degradation in the frontend (at least as long as hashtable based delete-dups is used). As a compromise I could filter out consecutive duplicates, which does not incur a performance penalty.
The post-command-hook error is probably an upstream issue. I have not observed this before on Emacs 27.1. Maybe the bug has been introduced in a change in Emacs 28. You may want to submit a bug report to emacs-devel soon, such that this can be fixed before Emacs 28 is released. Note that you should be able to reproduce the bug with the default completion UI. The only difference is that you have to press TAB after entering a letter in order to trigger the bug.

Not specifically related to this issue, but I got the same error message and traced it back to the code from the vertico github wiki which caused the error message in vertico 0.15 and 0.16 (but not 0.14) in Emacs 27.1.


[Enhance] Better YOLOv3 head initialization

Better YOLOv3 Head Initialization
Motivation
The original YOLOv3 in mmdet initializes bias in convs_pred with 0, which results in a very large loss in the early training steps. And this will cause overflow during fp16 training.
Modification
Use bias_init_with_prob(0.01) to initialize the bias of class and use a prior of "8 objects per feature map" to initialize objectness bias. Inspired by https://github.com/ultralytics/yolov3 but not the same.
Effect
After initializing head with this method, the loss of classification and objectness reduced a lot at the early training steps.
Here are losses of epoch1 step50:
Before
loss_cls: 4499.7637, loss_conf: 94941.6641, loss_xy: 226.8941, loss_wh: 80.2181, loss: 45748.5430

After
loss_cls: 308.9337, loss_conf: 590.1021, loss_xy: 159.6315, loss_wh: 38.8394, loss: 1097.5066

The model is still in training. I'll post the final metrics a few days later.

Update

Model
Use New Init
Type
mAP
AP50
AP75
AP s
AP m
AP l

yolov3_d53_mstrain-608_273e_coco
No
FP32
33.4
56.3
35.2
19.5
36.4
43.6

yolov3_d53_mstrain-608_273e_coco
No
FP16(Overflow)
-
-
-
-
-
-

yolov3_d53_mstrain-608_273e_coco
Yes
FP32
33.7(+0.3)
56.6(+0.3)
35.3(+0.1)
19.4
36.8(+0.4)
44.3(+0.7)

yolov3_d53_mstrain-608_273e_coco
Yes
FP16
33.8(+0.4)
56.8(+0.5)
35.1
19.7(+0.2)
37.3(+0.9)
43.6

It seems that the results will be a little bit higher after using this initialization method.

Can be merged after fix unit tests.

Can be merged after fix unit tests.

Unit tests failed during testing onnx yolov3 neck, but I didn't change that part. I just re-run and passed. Seems to be a random bug.


Noosa

There doesn't seem to be a suburb called "Noosa" - only (QLD)

Noosa Heads
Noosa North Shore
Noosaville

All of which have been done.


Feature: Databricks compatibility

Confirm the following files were correctly updated automatically:

[x] hooks/pre-command line added
[x] pipeline.yml section added
[x] sample.profiles.yml catalog updated
[x] integration_tests/dbt_project.yml section added (check for dupes though)
[x] Version updates for dbt_project.yml and integration_tests/dbt_project.yml (automatically moved to next breaking change)
[x] Changelog (automatically moved to next breaking change, but check the PR links were added correctly!)
[x] greenhouse_source.yml database logic updated

Manual updates are:

[x] README installation and dependencies
[NA] Incremental Models
[x] Regen Docs
[x] Local Databricks run and test passing
[x] Buildkite passing

Standard updates.


Color of the extension's icon for easy identification

When we open multiple tabs, both Salesforce and Inspector, then the inspector tabs are not easily identifiable (screenshots below).
For fewer tabs it's fine, but it becomes slightly difficult when we've many tabs open in a single browser window.
Would it be possible to change the color of the icon please or give users an option to do so?
Thanks.

It was easy in the original inspector:

Same as this one https://github.com/tprouvot/Salesforce-Inspector-reloaded/issues/180

That's a good idea which has been requested by other members of the community 👍

Will be released in 1.23 🚀 and in the beta version sooner

Anyway to not have to set it for each org manually?
Perhpas it can match the color of this extension somehow
https://chromewebstore.google.com/detail/salesforce-colored-favico/peohlnebahcddpmfaplmilpkgbkkcdho?hl=en

Hi @RebbePod,
I prefer to keep the default favicon as it is for the users who don't want custom favicons.
I don't know how are you dev skills but I think you/we can code something in order to add a favicon for all the orgs you've visited.
Let me know if you're interested in this solution

@tprouvot Theres two parts to this, one the way I see it, have an option to auto set each org a different color. So every org visited is unique. Second part is copying the functionality of that other extension, giving an option for the favicon to be colored for the org pages not just inspector.
If we get part two complete the colors should match to part 1.
My skills are limited, but I can definitely try to take a look if you give me some direction

Part 2 is already developed and available in the beta version.
How many orgs are you working on ? Because to me it would be much more faster to configure each color on the org when needed

Over 30 😀

it's also about ease of use. A setting to change the color make sense, but why shouldn't auto assign one?

@RebbePod

Open data export page
Right click on the page and select inspect
Paste following code in chrome dev console

let colors = ['green', 'orange', 'pink', 'purple', 'red', 'yellow'];

let orgs = Object.keys(localStorage).filter((localKey) =>
    localKey.endsWith("_isSandbox")
);
orgs.forEach((org) => {
    let sfHost = org.substring(0, org.indexOf(("_isSandbox")));
    let randomFavicon = colors[(Math.floor(Math.random() * colors.length))]
    console.info(sfHost + "_customFavicon", randomFavicon);
    localStorage.setItem(sfHost + "_customFavicon", randomFavicon);
});

Will this keep the color of each one, so it's the same color everytime I reopen it?

@RebbePod yes

@tprouvot it's working great so far, not sure if you'd like to incorporate it in somehow

@RebbePod actually I documented the code in the how-to section and enhanced the feature by increasing the available colors with HTML codes


[5.1] Fix for tagged cache events

This is a possible way to fix #10423. The problem lies in the TaggedCache class which calls the stores directly, bypassing the Repository class entirely, which contains all the code for triggering events.
The approach I took in fixing this was to have the TaggedCache class extend the Repository class and then call the corresponding methods in the parent using the generated cache key for the tags used. This way the code for firing the events is not duplicated.
When an event is fired from a tagged cache an array of the tag names is also passed in the payload now.
This PR is meant to demonstrate what the issue is and a possible solution. This also has a set of new tests which determine whether the expected events are fired for both tagged and non-tagged caches.

I have also come up with a slight variation where a method called itemKey is added to the Repository class and is called on the key in the methods which interact with the cache stores. This means that the itemKey method can be overridden by the TaggedCache class. This variation would seem to be a more elegant solution and avoid the hackiness that is in the fireCacheEvent method of the TaggedCache class.

I just added a commit with the other variation.

@taylorotwell I still wonder if this is best targetted at 5.2.

Just let me know if I need to retarget this at 5.2.

Bit late for that now. Will either be 5.1, or 5.3 now.

If it's a true bug fix it could go on 5.2?
On Tue, Dec 22, 2015 at 9:26 AM, Graham Campbell<EMAIL_ADDRESS>wrote:

Bit late for that now. Will either be 5.1, or 5.3 now.
—
Reply to this email directly or view it on GitHub
https://github.com/laravel/framework/pull/11338#issuecomment-166646190.

Good luck with the branch merge. :trollface:

I got it merged. @willrowe one thing I'm kind of curious about is you have no idea from the event if the given "key" was a tagged key or not... is that OK and expected?

@taylorotwell a list of the tags that were used is appended to the event payload here. In 5.1 this works fine and will automatically pass the list of tags to the event listener. In 5.2 an additional parameter will need to be added to the event classes and passed to them here.

So can you show me how I would access the tags from the event in some
sample code?
On Thu, Dec 24, 2015 at 10:51 AM, Will Rowe<EMAIL_ADDRESS>wrote:

@taylorotwell https://github.com/taylorotwell a list of the tags that
were used is appended to the event payload here
https://github.com/laravel/framework/commit/679da7059801fb350f6a1443ad1e919355b06e3f#diff-f7407eeeb7ec42d3f5f6d341a44e3c03L38.
In 5.1 this works fine and will automatically pass the list of tags to the
event listener. In 5.2 an additional parameter will need to be added to the
event classes and passed to them here
https://github.com/laravel/framework/commit/679da7059801fb350f6a1443ad1e919355b06e3f#diff-f7407eeeb7ec42d3f5f6d341a44e3c03L38
.
—
Reply to this email directly or view it on GitHub
https://github.com/laravel/framework/pull/11338#issuecomment-167136203.

A listener class could look something like this:
namespace App\Listeners;

use Illuminate\Cache\Events\CacheHit;

class LogUserCacheHit
{
    /**
     * Create the event listener.
     *
     * @return void
     */
    public function __construct()
    {
        //
    }

    /**
     * Handle the event.
     *
     * @param  Illuminate\Cache\Events\CacheHit  $event
     * @return void
     */
    public function handle(CacheHit $event)
    {
        if (in_array('user', $event->tags)) {
            Log::info('User ' . $event->key . ' was retrieved from the cache.');
        }
    }
}

OK can you make a PR to actually add that parameter to the events and such?

Sure, I'll submit one over the weekend..


Clear telemetry signal for when a user is forced to upgrade

@anthony-murphy - Can you put some more details in here? So that someone unfamiliar with this area (like me!) could pick up this item. Thanks.


Added new functionality delete

Added the functionality to delete a single user at a time. Also made some changes in the root.go file related to the flags.

#1


Review requirements for Jupyter notebook

[x] In addition to the setup.py requirements when you install the package, the jupyter notebook requires pandas (maybe more - that's as far as I got).
[x] It also gives a warning about numba.
[x] %matplotlib qt doesn't seem to work for me

The numba warning come from the skan package so there isn't really anything I can do about that. The rest should now be fixed.

@gcapes oops! Sorry about that! There is something you can do though, submit a PR/issue to the skan repo!. 😉 It would be much appreciated!

https://github.com/jni/skan/issues/103