BI-1650 - Store and fetch genotype data

Description
Story: https://breedinginsight.atlassian.net/browse/BI-1650
Updating GitHub actions to work with changes introduced by BI-1650 (added Gigwa)
Dependencies
bi-docker-stack/feature/BI-1650
bi-api/feature/BI-1650
bi-web/feature/BI-1650
Testing
https://github.com/Breeding-Insight/taf/actions/runs/3500841631
This run shows the same errors as the daily run from 11/18/22
Checklist:

[x] I have performed a self-review of my own code
[x] I have tested my code and ensured it meets the acceptance criteria of the story
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to documentation

I reverted to the commit before this card. This should NOT be merged until the rest of BI-1650 on the other branches is merged due to dependencies.


Gradle is building url_launcher_macos, url_launcher_web when building android fat apk

Project has a dependency on url_launcher:
dependencies:
  flutter:
    sdk: flutter
  url_launcher: ^5.4.1

When building a fat apk using flutter build apk, I can see the following gradle logs:
✓ Built build/app/outputs/repo.
Building plugin url_launcher...
Running Gradle task 'assembleAarRelease'...
Running Gradle task 'assembleAarRelease'... Done                    1.7s
✓ Built build/app/outputs/repo.
Building plugin url_launcher_macos...
Running Gradle task 'assembleAarRelease'...
Running Gradle task 'assembleAarRelease'... Done                    0.6s
✓ Built build/app/outputs/repo.
Building plugin url_launcher_web...
Running Gradle task 'assembleAarRelease'...
Running Gradle task 'assembleAarRelease'... Done                    1.5s

This is not a blocking issue, but is it normal that web and macos variants are evaluated and compiled when building an APK?
Flutter doctor:
[✓] Flutter (Channel stable, v1.12.13+hotfix.7, on Mac OS X 10.15.3 19D76, locale en-CA)

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
[✓] Xcode - develop for iOS and macOS (Xcode 11.3.1)
[!] Android Studio (version 3.5)
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
[!] IntelliJ IDEA Ultimate Edition (version 2019.3.1)
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
[✓] VS Code (version 1.41.1)
[✓] Connected device (1 available)

Until https://github.com/flutter/flutter/issues/39657 is fixed, yes, this is normal.


🛠 Repo: remove util.inherits / convert to classes

Is your feature request related to a problem or a nice-to-have?? Please describe.
It would be nice to get rid of some NodeJS specific things that makes it better suited for browsers.
one such thing could be the option to remove util.inherits and replacing it with classes that can extends things.
resulting in fewer dependencies and smaller code bundle that won't require the browserified version of util that has an additional of 20 modules and 12 kb
NodeJS docs have marked it as legacy and has this to say about util.inherits

Usage of util.inherits() is discouraged. Please use the ES6 class and extends keywords to get language level inheritance support

Describe the solution you'd like
Change prototype based classes into es-classes

Fully agree!
Some classes that needs to be converted:

https://github.com/mochajs/mocha/blob/24560c1532b8e67d1254b489fcc6f84c4033c0e1/lib/hook.js#L29
https://github.com/mochajs/mocha/blob/24560c1532b8e67d1254b489fcc6f84c4033c0e1/lib/runnable.js#L55
https://github.com/mochajs/mocha/blob/24560c1532b8e67d1254b489fcc6f84c4033c0e1/lib/suite.js#L99
https://github.com/mochajs/mocha/blob/24560c1532b8e67d1254b489fcc6f84c4033c0e1/lib/test.js#L39
https://github.com/mochajs/mocha/blob/24560c1532b8e67d1254b489fcc6f84c4033c0e1/lib/reporters/dot.js#L79
https://github.com/mochajs/mocha/blob/24560c1532b8e67d1254b489fcc6f84c4033c0e1/lib/reporters/landing.js#L114
https://github.com/mochajs/mocha/blob/24560c1532b8e67d1254b489fcc6f84c4033c0e1/lib/reporters/list.js#L76
https://github.com/mochajs/mocha/blob/24560c1532b8e67d1254b489fcc6f84c4033c0e1/lib/reporters/min.js#L50
https://github.com/mochajs/mocha/blob/24560c1532b8e67d1254b489fcc6f84c4033c0e1/lib/reporters/nyan.js#L78
https://github.com/mochajs/mocha/blob/24560c1532b8e67d1254b489fcc6f84c4033c0e1/lib/reporters/progress.js#L102
https://github.com/mochajs/mocha/blob/24560c1532b8e67d1254b489fcc6f84c4033c0e1/lib/reporters/spec.js#L99
https://github.com/mochajs/mocha/blob/24560c1532b8e67d1254b489fcc6f84c4033c0e1/lib/reporters/tap.js#L80
https://github.com/mochajs/mocha/blob/24560c1532b8e67d1254b489fcc6f84c4033c0e1/lib/reporters/tap.js#L239
https://github.com/mochajs/mocha/blob/24560c1532b8e67d1254b489fcc6f84c4033c0e1/lib/reporters/tap.js#L291
https://github.com/mochajs/mocha/blob/24560c1532b8e67d1254b489fcc6f84c4033c0e1/lib/reporters/xunit.js#L117

If we can find a linting rule that enforces / helps with this, then we should

Bad news: I think this would technically be a breaking change. Out of the three PRs that split it up:

#5178: no changes needed for tests
#5179: no changes needed for tests
#5180: quite a few changes needed for tests

...indicating that the runtime differences noted in https://nodejs.org/api/util.html#utilinheritsconstructor-superconstructor do impact usage of these class(-like)s.
As much as we want to clean up this old code, we'll have to wait for the next major version of Mocha. 😞

I think we should get planning for a new major version fairly soon, as it eg is needed to update some outdated dependencies as well
We can keep a 10.x branch around for bug fixes and have main be for the next major?

get planning for a new major version fairly soon

Agreed, the more I dig into the legacy dependencies the more breaking changes I find...
For visibility: we'll talk about this in our next maintainer meeting to see what strategy we go with. 👍


Settings not saved when using new installer

We have written the needed data into your clipboard because it was too large to se
Issue Type: Bug
VSCode kept nagging me after the latest update to instead use the msi installer, and assured me it would not lose settings. On one machine I installed without removing the current version, and when it popped up a dialog warning me that I should first uninstall the old version, I chose to go ahead with the install, and ended up with both installed.
On a second machine, I chose to listen to the dialog and first uninstall, but lost all of my settings. (I mainly notice that my keyboard shortcuts are reset to default).  You might want to make your notifications a little more clear.
VS Code version: Code 1.26.1 (493869ee8e8a846b0855873886fc79d480d342de, 2018-08-16T18:38:57.434Z)
OS version: Windows_NT x64 10.0.17134

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-6820HQ CPU @ 2.70GHz (8 x 2712)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
31.61GB (22.65GB free)

Process Argv
C:\Users\justin.michel\AppData\Local\Programs\Microsoft VS Code\Code.exe

Screen Reader
no

VM
0%

Extensions (1)

Extension
Author (truncated)
Version

vetur
oct
0.12.6

nd. Please paste.

On a second machine, I chose to listen to the dialog and first uninstall, but lost all of my settings.

Code will not delete your settings and/or keybindings on uninstall. Something else must've gone broken. Please check %APPDATA%\Code\User\settings.json to see whether you still find them there.

Same happened to me too. I had created a data folder to make my setting portable, unfortunately the latest update has deleted all my extensions and settings from this folder.


Re-enable MySQL support

MySQL support was removed for the first official release because it was untested. Once tested, and fixed if broken, MySQL can be re-enabled.

@whimxiqal how to add Mysql support?

The commented-out code referencing "MySQL" just has to be uncommented and then it has to be tested with a real MySQL database, which I haven't done yet.
Do you need it for your server?

Added for v1.1


Crashlytics in tvOS for Apps without Fabric App

Is there a way to use Crashlytics in a new tvOS app.  Since now you can't create a new Fabric app, #2009 solution is not possible anymore.
I tried integrating it as a new iOS App but test crashes are not showing up in the console (yes, I did read/follow the Firebase Crashlytic integration instructions for new apps step by step, and the firebase integration is working since I can use FirebaseDatabase without issues)
Most issues in here seems to apply to older apps with a Fabric app already created, but how does it work for a new Firebase tvOS App?

Tracked internally at b/139003562

At this time you can still create new apps in the Fabric console. This is the recommended path. I have let the team know . :)

At this time you can still create new apps in the Fabric console. This is the recommended path. I have let the team know . :)
Do you mean the dashboard in https://fabric.io/settings/apps ? beacause if I click the add button it redirects me to the home page https://fabric.io/kits

Hey @bjimenezned, it sounds like you got the setup right. I took a look at this and think there is an issue in the backend. I'm gonna work on getting this fixed and then update this bug.

Hey @bjimenezned, can you try onboarding your app in Firebase again? The issue should be resolved now.
For future tv and macOS onboarders, my recommendation is to follow the onboarding steps for an iOS app, and include the following in your Podfile if you want Crashlytics:
  pod 'FirebaseCore'
  pod 'Crashlytics'

Seems to be working now, I managed to link and receive crash reports in firebase console.
Thanks!


move pixiedust to its own GitHub org

Changed references to old GitHub organisations to the new "pixiedust" GitHub organisation

Are you going to re-publish on pypi @glynnbird?


Moment Object returns to the model rather than formated string

My vue Component
<vue-monthly-picker
                  dateFormat="YYYY-MM"
                  v-model="date"
                ></vue-monthly-picker>

Expected behaviour:
the model date should be string of format YYYY-MM rather than moment object.

Hi @bedus-creation,
Thank you for your comment.
Unfortunately, this library only supports moment date object.
Please feel free to fork it.


[pt-br] Add docs/tasks/configure-pod-container/configure-pod-configmap.md

/remove-lifecycle stale

/retitle [pt-br] Add docs/tasks/configure-pod-container/configure-pod-configmap.md
/cc
/remove-lifecycle stale

/remove-lifecycle stale

/lgtm
/approve

Hi, there.
This PR was with LGTM, but due to a long time for reviewing got no more up-to-date,  was reviewed, is up-to-date with the latest version, and all changes requested were updated.

Thanks!
/approve


Encapsulate signatures validation logic in Extra

Description
This PR encapsulates signatures validation logic in a single place. Prior to this, we were having almost the same logic for ParentSignature validation spread to fsm and polybft. This way Extra is more encapsulated and it contains validation logic of its own state.
Changes include

[ ] Bugfix (non-breaking change that solves an issue)
[ ] Hotfix (change that solves an urgent issue, and requires immediate attention)
[x] New feature (non-breaking change that adds functionality)
[ ] Breaking change (change that is not backwards-compatible and/or changes current functionality)

Checklist

[ ] I have assigned this PR to myself
[ ] I have added at least 1 reviewer
[ ] I have added the relevant labels
[ ] I have updated the official documentation
[ ] I have added sufficient documentation in code

Testing

[ ] I have tested this code with the official test suite
[ ] I have tested this code manually

I am not sure, did we have unit tests for these functions? If we did not, it would be nice to add them, since these are now pretty isolated functions.

I am not sure, did we have unit tests for these functions? If we did not, it would be nice to add them, since these are now pretty isolated functions.

They were covered as part of fsm unit test, however not all unhappy paths are covered. Let me see if I can implement some more unit tests.


Can't delete leftovers for Facebook Gameroom

I've uninstalled Facebook Gameroom and tried to remove leftovers, but it doesn't work for some reason.
Being at this stage:

I press "Delete selected", window "Waiting for process to exit" blinks for 1-3 ms and nothing happens next:

After the window blinks, what exactly happens? Are you back to the Leftover removal window and can press Delete again?

Yes, exactly.
It just blinks and nothing happens next.

Could you test this again if I sent you a debug version?

Sure, I'll try to!

Try this version: Debug.zip
It should be fixed, but if you still have problems, close BCU and send me the BCUninstaller.log file.

I've just tried to reproduce the issue and looks like it doesn't happen anymore.
Instead, Facebook Gameroom doesn't uninstall at all looks like.
I Uninstall it, delete all found leftovers, everything looks fine, but it's not removing. It's still at the apps list and I can repeat while uninstallation procedure again and again and leftovers are found again, etc.
Please let me know how can I help debug this issue and I'll be glad to help.
Here is a log file after few such uninstallations.
BCUninstaller.zip

I installed Gameroom and found out a few things:

It uses an unusual install location that BCU can't detect, which means BCU can't see if it's running before uninstalling it. It probably isn't worth trying to detect it, as it would require a lot of HDD grinding = longer load times.
When uninstalling normally it launches itself for some reason, and the uninstall won't continue until you log-in.
Quiet uninstall works perfectly, Gameroom is closed automatically and leftover removal completes without issues.

I just tried to uninstall it quietly but it's still there, still doesn't uninstalls =\

Oh, sorry, I did tried it on that debug version you've sent.
I re-installed latest release version and quiet uninstall worked perfectly fine!

All right, thanks! I'm closing this as fixed, since the Kill dialog is fixed and there's not much else that can be done here.


Add option to modify value label before drawing

Added the possibility to modify the value label before drawing. This allows to e.g. change the kerning and make % sign smaller than the numbers.

Thanks, for all the work lately!
Could you give me an example of a use case for this?
This is adding a pretty hot code path which I feel people may misuse.

I use it to format the value label to have designer defined kerning and half size percentage sign (sorry for the objc syntax 🤣):
// change kerning to -5 from default 1
NSMutableAttributedString* attributedString = [label.attributedText mutableCopy];
[attributedString addAttribute:NSKernAttributeName value:@(-5) range:NSMakeRange(0, self.text.length)];
// make font half size from % sign, this could be done faster only taking the last char, but I actually have this in UILabel extension and use it for another labels like 10 / 20 to show the slash part in half size
NSUInteger locOfPerc = [label.text<EMAIL_ADDRESS>NSRange range = NSMakeRange(0, locOfPerc);
UIFont* font = [attributedString attribute:NSFontAttributeName atIndex:0 effectiveRange: &range];
UIFont* halfSize = [UIFont fontWithName:font.fontName size:font.pointSize/2];
[attributedString addAttribute:NSFontAttributeName value:halfSize range:NSMakeRange(locOfPerc, attributedString.length - locOfPerc)];
[label setAttributedText:attributedString];

Ah I see, alright this looks good!
Thanks again!


[23.09.2020]: UIDatePicker changed with iOS 14

Information
Device:
iOS version:
14.0
App Version:
1.2.5.
Environment:
Frequency (How often does the bug occur?):
Your picker now looks like this:

You most likely want to set the style to wheels.

Then it looks like this again:

https://github.com/karhoo/karhoo-ios-ui-sdk/pull/25 [fix]


chore: create executable tx creator for testing

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#672  👈
#666
#661
#660
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @ArniStarkware and the rest of your teammates on  Graphite


Fix atomic for architectures without support for lock-free atomics (e.g. armv6l)

Without this patch mimalloc fails to build on armv6l

I guess the MI_USE_LIBATOMIC will work, also commit 144b4a2d3 uses find_library to auto include it. So closing this for now. Thanks again!


Stuck on white screen when minimising chat window

I have not tested this on Android yet, but when running on iOS simulator / iOS device I seem to be unable to exit the chat window properly.
When I tap the minimise button in the top right corner, a white screen is displayed and I can not go back by any means.
Hot restarting does not help.
Only way to return to app is to kill the app and restart.

This has been fixed @halldorlogi .


Remove id field from Constellation and separate the constellation information from the solution count

Make Constellation always immutable. In ConstellationsGenerator, instead od looping through the sub constellations of an ijkl updating the startIjkl of each Constellation, just have a global variable currentIjkl which is used in the preplace-method.


BUG: No Vault check on joining a keysign

Describe the bug
It is possible to sign a transaction from any vault selected as currently the vault check is not working
To Reproduce
Steps to reproduce the behavior:

create tx with a vault
on the pairing device be on a different vault
scan QR
tx gets signed without wrong vault error
See error

Expected behavior
Getting the wrong vault error due to the verification
-v22

@yvebe I get where you are coming from. Personally I would rather have the user knowingly select the correct vault as a "approve" method before sending.
@johnnyluo what is you preference?

Let's keep current behaviour ,  user mostly approve the transaction


add Rustlang.Rust.GNU version 1.80.0

This PR is auto-generated. If there's something wrong, please file an issue at https://github.com/Rust-Winget-Bot/my-source-code/issues
Microsoft Reviewers: Open in CodeFlow

/azp run


add command for generating and staging certs

To make it easier to create secure clusters in roachprod, it would be nice to be able to do this:
roachprod create $CLUSTER
roachprod stage $CLUSTER release 2.0.5
roachprod gen-certs $CLUSTER # generate certs and deposit them on the cluster's nodes
roachprod start --secure # start the nodes with `--certs-dir=certs`

cc @mberhault

Are you aware of the existing start --secure option?
~ roachprod create local
Creating cluster local with 4 nodes
OK
~ roachprod start local --secure
local: checking certs 1/1
local: initializing certs 1/1
local: initializing certs 3/3
local: starting 4/4
local: initializing cluster settings 1/1
SET CLUSTER SETTING
~ roachprod get local:1 certs
local: getting certs certs
   1: done
~ ls certs/
ca.crt          ca.key          client.root.crt client.root.key node.crt        node.key
~ roachprod run local -- ls certs/ca.crt
local: ls certs/ca.crt 4/4
   1: certs/ca.crt
   2: certs/ca.crt
   3: certs/ca.crt
   4: certs/ca.crt

Wow, perfect. Didn't realize that would actually initialize the certs. Thanks!


Setup mui theme

Setup a mui theme for the project and organise the spacing between the buttons with this theme.

I like this colour scheme https://coolors.co/e8c547-30323d-4d5061-5c80bc-cdd1c4


Help: {"detail":"Not Found"} error

While I can interact with the web interface, I am unable to use any API requests as it returns {"detail":"Not Found"}. Refreshing the web interface also shows the same error. It's my first time working with API request sbut I looked up the proper formatting (i.e http://my-ip:7912/vendor?name=test). Thanks in advance.

The API endpoint is at /api/v1, so try something like
http://myip:7912/api/v1/vendor?name=test

Thank you


Project Reactor support

There are many users who'd like to use this, but cannot because this project is based on slow sync connections. I'd like to request to consider supporting Project Reactor \ async connections from the very start, so this project will be usable for many of us.
For those who'll find this issue useful, I recommend using the great reactive-pulsar meanwhile.

We already are discussing this https://github.com/spring-projects-experimental/spring-pulsar/issues/7

@rolkhas2 Thanks for your interest in the project. As @onobc suggested, at the moment, we are focussing on ensuring that the foundations are laid properly. However, do you mind elaborating on the slow sync connections you mentioned? At the moment, Spring Pulsar clients are using the PulsarClient provided by the Java client in Pulsar. Is that you were referring to? When you respond, feel free to comment on the other open issue.

Are there any plans for reactive spring pulsar implementation?

@lynch19 We are looking into this area of reactive use cases in general. In a few weeks, we hope to update you on this issue.


Remove Documentation caption from sidebar

List of Changes
See title.
Motivation
The non-clickable caption in the table of contents is confusing, I feel.
Explanation for Changes
No caption => no confusion.
Acknowledgement

[x] I have read the Contributing Guidelines

I'm approving this just because I have no time to review this massive PR


ManagedCluster doesn't report as ManagedClusterConditionAvailable during detach

$ oc get nodes -A
NAME                                  STATUS   ROLES           AGE    VERSION
ztpfw-edgecluster0-cluster-master-0   Ready    master,worker   154m   v1.23.5+b463d71
ztpfw-edgecluster0-cluster-master-1   Ready    master,worker   144m   v1.23.5+b463d71
ztpfw-edgecluster0-cluster-master-2   Ready    master,worker   154m   v1.23.5+b463d71

This is before detach, it's at the 'check_cluster' function:

Debugging issue seems to be related with some containers not being downloaded because of missing certificate information for the internal registry, so as soon as we move to the ICSP on the HUB, it fails

:tada: This issue has been resolved in version 1.8.4 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


sort_array_using_recursion

Aim
sort_array_using_recursion
Details
To sort a given array using recursion.
Programming language

[ ] C
[x] C++
[ ] Java
[ ] Python

@Ayush7614 bhiya, please assign me this issue .

I want to do this in C language
LGM_SOC

i wan to do this in java please assign me this issue

@Ayush7614  please assign me under I want to write the code in python .

/assign


Replaced broken links

Replaced broken links

@dat-a-man pleace build the website locally, the links are still broken (but seemed to be working before?) you'll see the error output of docusaurus.


EOS 1.3.0 - system newaccount - Asset's amount and symbol should be separated with space

I'm having an issue in bootstrapping a local testnet. I've created all the eosio.* accounts, deployed msig and system contracts. I'm now at the point of trying to create a new account with staked resources per the bios bootsequence guide .
Here's the error I'm seeing:
➜  build git:(08819aae0) ✗ cleos system newaccount --transfer --stake-net '100000.0000 SYS' --stake-cpu '100000.0000 SYS' --buy-ram 8192 eosio testmetestme EOS543VEy5TimeSba53Xx6aRMGb4nfY3h6uvB61tHiisPw2c6BnhX
2018-09-21T03:13:44.505 thread-0   asset.cpp:79                  from_string          ] 3010011 asset_type_exception: Invalid asset
Asset's amount and symbol should be separated with space
    {}
    thread-0  asset.cpp:42 from_string
2018-09-21T03:13:44.505 thread-0   asset.cpp:79                  from_string          ] from: 8192
Error 3010011: Invalid asset
Error Details:
Asset's amount and symbol should be separated with space
rethrow

I've tried formatting with double-quotes single quotes and no luck. Any insight into this?

if use --buy-ram, you should input  asset, no bytes type.

if use --buy-ram, you should input asset, no bytes type.

🤦♂
sooo I guess the next question is where do I submit issues to update the documentation. I'm guessing it used to be --buy-ram with bytes before and hasn't been updated? or just a typo in the documentation.
775292ms thread-0   main.cpp:419                  create_action        ] result: {"binargs":"0000000000ea30551082d4334f4d113200200000"} arg: {"code":"eosio","action":"buyrambytes","args":{"payer":"eosio","receiver":"accountnum11","bytes":8192}}
775295ms thread-0   main.cpp:419                  create_action        ] result: {"binargs":"0000000000ea30551082d4334f4d113200ca9a3b00000000045359530000000000ca9a3b00000000045359530000000001"} arg: {"code":"eosio","action":"delegatebw","args":{"from":"eosio","receiver":"accountnum11","stake_net_quantity":"100000.0000 SYS","stake_cpu_quantity":"100000.0000 SYS","transfer":true}}
executed transaction: fb47254c316e736a26873cce1290cdafff07718f04335ea4faa4cb2e58c9982a  336 bytes  1799 us
#         eosio <= eosio::newaccount            {"creator":"eosio","name":"accountnum11","owner":{"threshold":1,"keys":[{"key":"EOS8mUftJXepGzdQ2TaC...
#         eosio <= eosio::buyrambytes           {"payer":"eosio","receiver":"accountnum11","bytes":8192}
#         eosio <= eosio::delegatebw            {"from":"eosio","receiver":"accountnum11","stake_net_quantity":"100000.0000 SYS","stake_cpu_quantity...```

Either way... this was the answer. Thanks a bunch! 🙏

where do I submit issues to update the documentation

On the https://developers.eos.io site click on the "Suggest edits button" in order to suggestion documentation changes.


Restrict timesheet entries to be added to only one invoice

Notion card
https://www.notion.so/saeloun/Bug-If-a-timesheet-entry-has-been-added-to-an-invoice-in-draft-status-and-if-I-create-new-invoice--6255fd6ba598482eaa30e0363afcd093
Summary
If a timesheet entry was added to an invoice in draft status and if we create a new invoice then those entries were still available to be added to the new invoice. This pr fixes this bug and restrict entries to be displayed only if they are not a part of any other invoice.
Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
On local
Checklist:

[x] I have manually tested all workflows
[x] I have performed a self-review of my own code
[x] I have added automated tests for my code

Add test for change.

TODO: Add test for change.

@akhilgkrishnan  please review and approve.

@prasanthchaduvula Can you review this PR ?


the point cloud looks a bit plausible and noisy

PENet is efficient and the output depth image seems very nice, but the point cloud transformed from the depth inferred by PENet looks plausible and noisy. Is there something wrong with my matlab code used for transformation? Thank you.
depth2cloud.zip

Thanks for your interest! I don't think there is anything wrong in your code.
Actually depth results inferred by regular depth completion models inevitably suffers from artifacts when back projected to 3D space. In other words, not all pixels of the completed depth are reliable. One simple way to select reliable pixels is to predict an additional confidence map and remove pixels with low confidence. You may supervise the confidence prediction network with e^-|D_gt-D|. There also exists some more complex solutions like [DenseLiDAR: A Real-Time Pseudo Dense Depth Guided Depth Completion Network. RAL 2021.].

And one recent work claiming to address the smearing depth problem [Depth Completion with Twin Surface Extrapolation at Occlusion Boundaries. In CVPR 2021]

And one recent work claiming to address the smearing depth problem [Depth Completion with Twin Surface Extrapolation at Occlusion Boundaries. In CVPR 2021]

Thanks so much for your prompt reply.


Escaping of space character is ignored within markup extension

{Binding StringFormat={} {0}\ , Mode=OneWay} is collapsed into {Binding StringFormat={} {0}\, Mode=OneWay}
Note the space after \ is removed.
It's a good thing that the resulting XAML is invalid though, otherwise some regressions could have gone unnoticed.

I have made a beta release available at:
https://github.com/PolarbearDK/XamlStyler/releases/tag/2.0Beta


change align to flex-starts

@oscarotero Belén suggested to us change the icon align in Issue components.

Nice!


System.NotSupportedException: 'The binary operator 'Or' is not supported'

Hello all. I'm trying to do a simple query using my test table in DynamoDB. But for any OR query I'm getting the System.NotSupportedException: 'The binary operator 'Or' is not supported'. The same applies to OrElse.
Maybe I'm missing some premise in dealing with constraints in DynamoDB or even Linq in C#.
Here's the code:
// The Entity
    [DynamoDBTable("TbRelacionamento")]
    public class TbRelacionamento
    {
        [DynamoDBHashKey]
        public string ID { get; set; }
        // Some other unimportant attributes in between

        // In between the attributes we have Enums
        public RelacionamentoFgStatusEnum FgStatus { get; set; }

        public bool? Reservado { get; set; } // This is the attribute to be queried for

        // In between the attributes we have DateTimes
        public DateTime DataEnvioAgendamento { get; set; }

        // And also simple objects
        public List<ProdutoCarrinho> Produtos { get; set; }
    }

Here are the test that breaks with the given exception above
var test2 = context.TbRelacionamentos.Where(x => x.Reservado == false | x.Reservado == null);
var test3 = test2.Take(Limit).ToList();
var test = (from relationship in context.GetTable<TbRelacionamento>()
            where relationship.Reservado == false || relationship.Reservado == null
            select relationship).Take(Limit).ToList();

Is there anything I'm missing?

@felipesotero , the binary operator 'Or' is not supported, because it is not supported by DynamoDB.
Generally, what I could recommend in such cases is to make two separate queries and then .Union() them.
But in your case it should be way much simpler: you just do .Where(x => x.Reservado != true) instead.


Textual definition of "Clinical History Taking" potentially underinclusive

> "Clinical history taking": "Interview" is part of the textual definition,
> but not defined in OGMS. Are we sure that all clinical history taking is
> going on in interviews?

Comment by Albert Goldfain:
"Good point.  Maybe something more general is in order to accommodate
surveys and other forms of interaction"

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 9 Dec 2010 at 7:32

Closed ticket because issue has not been acted upon in reasonable amount of time.


Event Hub issue - Resource is not supported

When i try linking Event hub and select my event hub from drop down then i get following message
"Resource is not supported"

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

ID: b006a616-2f2f-8ecf-8b80-fd0a3cc62090
Version Independent ID: 92c54dc6-8ddf-b1f7-4a58-d61b8cd7fe25
Content: What is an Azure Private Endpoint?
Content Source: articles/private-link/private-endpoint-overview.md
Service: private-link
GitHub Login: @malopMSFT
Microsoft Alias: allensu

@omerzubair , We are looking into this query and will update you as soon as possible.

@omerzubair , As of now Private endpoint connections on Event Hubs are only supported by namespaces created under a dedicated cluster.

Please refer this doc on creating a dedicated cluster in Azure.

@omerzubair , We are closing this issue for now. If there are further questions regarding this matter, please reply and we will gladly continue the discussion.


mobile support (maybe controller support)

Add buttons for rotation and flipping (only when piece is selected).
Add 2 methods for cursor control:

Tap and drag piece to select, move, and drop. (make sure that the piece only follows the first input that selects the piece)
Simulated cursor, moving to the drag delta. Select button will show if "cursor" is hovering over a piece.

Lock to landscape mode only.

Add button functionality, and simulate cursor functionality onto custom cursor.


fixing linkedin ads database and schema variable names in readme

Are you a current Fivetran customer?
Clay Walker
Senior GTM Analytics Manager
Gong
What change(s) does this PR introduce?
This change fixes the linkedin ads variable names in the readme
Did you update the CHANGELOG?

[ ] Yes

Does this PR introduce a breaking change?

[] Yes (please provide breaking change details below.)
[X] No  (please provide an explanation as to how the change is non-breaking below.)
Change to documentation only

Did you update the dbt_project.yml files with the version upgrade (please leverage standard semantic versioning)? (In both your main project and integration_tests)

[ ] Yes

Is this PR in response to a previously created Bug or Feature Request

[X] Yes, Issue/Feature [https://github.com/fivetran/dbt_ad_reporting/issues/58]
[ ] No

How did you test the PR changes?

[ ] CircleCi
[X] Local (please provide additional testing details below)

Select which warehouse(s) were used to test the PR

[ ] BigQuery
[ ] Redshift
[ ] Snowflake
[ ] Postgres
[ ] Databricks
[ ] Other (provide details below)

Provide an emoji that best describes your current mood

:smiley:
Feedback
We are so excited you decided to contribute to the Fivetran community dbt package! We continue to work to improve the packages and would greatly appreciate your feedback on our existing dbt packages or what you'd like to see next.

Thanks again @clay-walker! We are planning to cut the v1.0.1 release today. You can expect your changes to be include then 😄


Add flow to Travis script and fix flow errors

Fixes the flow errors that have made it to master.
Adds npm run flow to CI.
Adds lint-rules from flowtype plugin.

Maybe we should add eslint-plugin-flowtype#require-valid-file-annotation?

Let's do that, and can you also do as @styled-components-bot says? Thanks!

@mxstbr Done. I added a few more lint rules, tried to add the flow equivalent of what was already in the config. e.g. the regular semi rule does not work on type declarations.

Released as v1.2.1! 🎉


Fix some linter errors by adding back Event Source examples.

Fix some linter errors by adding back Event Source examples.
Update examples for GA api-version to use that api-version.
Update the Reade.md file so GA version is the default.

This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information

[ ] The title of the PR is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[ ] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[ ] If applicable, the PR references the bug/issue that it fixes.
[ ] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[ ] I have read the contribution guidelines.
[ ] My spec meets the review criteria:

[ ] The spec conforms to the Swagger 2.0 specification.
[ ] The spec follows the guidelines described in the Swagger checklist document.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

@sandshadow I would suggest running the model validation tool locally so you can easily iterate on these changes.

Install oav using npm install -g oav
Use validate-example command via oav validate-example <spec-path>

I added the missing example for EventSources_ListByEnvironment. Thanks for pointing that out. The remaining linter failures at this point are what I consider false positives because of inheritance in the model.

For the operations that return an event source, the linter doesn't consider the IoTHub event sources as valid, though it does consider the event hub event sources as valid. I think this is probably because "Microsoft.EventHub" is the default value for the kind enum.
The "properties" property in the UpdateEventSource example is considered by the linter to be an invalid additional property because it is not defined on EventSourceUpdateParameters. But both derived EventHubEventSourceUpdateParameters and IoTHubEventSourceUpdateParameters types do define a "properties" property.

@jhendrixMSFT Thanks for the oav validate suggestion. I've fixed all the model validation errors with the examples.

No modification for AutorestCI/azure-sdk-for-ruby

No modification for AutorestCI/azure-sdk-for-python

No modification for AutorestCI/azure-sdk-for-node


New overlay doesn't factor in slayer helm

Is it possible to have the new overlay that automatically grabs gear/prayer/mob factor in the slayer helm?

This required a change to the core RuneLite client in order to fetch slayer task data, which was completed in runelite/runelite@fcacf4deeb9050d39922d03b599719e69209a5a8.
I added the change to dps-calculator in 8afe03e, and it's pending release to the plugin hub in runelite/plugin-hub#2858


[QA] Pink dust appears when running or jumping on Genesis Plaza

Build version:
https://github.com/decentraland/unity-explorer/pull/2568
Issue Description:
When jumping or running on Genesis Plaza you are able to see pink dust on the feet of the avatar on Macos
STR:

Open branch
Go to Genesis Plaza
Run or Jump
Check that there is pink dust on the floor

❌ Actual Result:
When jumping or running on Genesis Plaza you are able to see pink dust on the feet of the avatar on Macos
✔ Expected Result:
No pink dust should appear on the feet of the avatar , the dust should be white
Repro Index:
10/10
Evidence:
https://github.com/user-attachments/assets/cbac1400-fa19-4cb3-8e2e-e09ede1b0485
Additional Notes:
This issue happens on Macos

This issue is no longer happening on 0.46.0


Dynamic config are made static

I didn't notice it at first but you introduce a strange line in the input mechanism.
The issue is, when using a template config (says collection: {{myCollection}}) in the editor, this line will set this.collection to the first context it come across:
1st call :
msg = {myCollection: 'foo'}

Now this.config === 'foo' (instead of {{myCollection}}).
So this:
msg = {myCollection:'bar'}

will still query the foo collection.

BTW, I don't understand why you would want to set this. property dynamically on input if the user hasn't set them in the editor.

Worst than that, today I found that, combined with the firestore leaking, configurations made upstream are forced to following nodes since config is merge in firestore.
Could we just get rid of all the merging and try to be as stateless as possible on input?

Fixed in the latest commit


 If I then run it again, I get a small diff.

pyproject.toml
[tool.pylint]

    [tool.pylint.'MASTER']

    # Pickle collected data for later comparisons.
    persistent = true

    # Use multiple processes to speed up Pylint.
    jobs = 0

    # List of plugins (as comma separated values of python modules names) to load,
    # usually to register additional checkers.
    # See https://chezsoi.org/lucas/blog/pylint-strict-base-configuration.html.
    # We do not use the plugins:
    # - pylint.extensions.code_style
    # - pylint.extensions.magic_value
    # - pylint.extensions.while_used
    # as they seemed to get in the way.
    load-plugins = [
        'pylint.extensions.bad_builtin',
        'pylint.extensions.comparison_placement',
        'pylint.extensions.consider_refactoring_into_while_condition',
        'pylint.extensions.docparams',
        'pylint.extensions.dunder',
        'pylint.extensions.eq_without_hash',
        'pylint.extensions.for_any_all',
        'pylint.extensions.mccabe',
        'pylint.extensions.no_self_use',
        'pylint.extensions.overlapping_exceptions',
        'pylint.extensions.private_import',
        'pylint.extensions.redefined_loop_name',
        'pylint.extensions.redefined_variable_type',
        'pylint.extensions.set_membership',
        'pylint.extensions.typing',
	]

    [tool.pylint.'MESSAGES CONTROL']

    # Enable the message, report, category or checker with the given id(s). You can
    # either give multiple identifier separated by comma (,) or put this option
    # multiple time (only on the command line, not in the configuration file where
    # it should appear only once). See also the "--disable" option for examples.
    enable = [
        'bad-inline-option',
        'deprecated-pragma',
        'file-ignored',
        'spelling',
        'use-symbolic-message-instead',
        'useless-suppression',
    ]

    # Disable the message, report, category or checker with the given id(s). You
    # can either give multiple identifiers separated by comma (,) or put this
    # option multiple times (only on the command line, not in the configuration
    # file where it should appear only once).You can also use "--disable=all" to
    # disable everything first and then reenable specific checks. For example, if
    # you want to run only the similarities checker, you can use "--disable=all
    # --enable=similarities". If you want to run only the classes checker, but have
    # no Warning level messages displayed, use"--disable=all --enable=classes
    # --disable=W"

    disable = [
        # Style issues that we can deal with ourselves
        'too-few-public-methods',
        'too-many-ancestors',
        'too-many-locals',
        'too-many-arguments',
        'too-many-instance-attributes',
        'too-many-return-statements',
        'too-many-lines',
        'too-many-statements',
        'locally-disabled',
        # Let flake8 handle long lines
        'line-too-long',
        # Let flake8 handle unused imports
        'unused-import',
        # Let isort deal with sorting
        'ungrouped-imports',
        # We don't need everything to be documented because of mypy
        'missing-type-doc',
        'missing-return-type-doc',
        # Too difficult to please
        'duplicate-code',
        # Let isort handle imports
        'wrong-import-order',
    ]

    [tool.pylint.'FORMAT']

    # Allow the body of an if to be on the same line as the test if there is no
    # else.
    single-line-if-stmt = false

    [tool.pylint.'SPELLING']

    # Spelling dictionary name. Available dictionaries: none. To make it working
    # install python-enchant package.
    spelling-dict = 'en_US'

    # A path to a file that contains private dictionary; one word per line.
    spelling-private-dict-file = 'spelling_private_dict.txt'

    # Tells whether to store unknown words to indicated private dictionary in
    # --spelling-private-dict-file option instead of raising a message.
    spelling-store-unknown-words = 'no'

[tool.coverage.run]

branch = true

[tool.coverage.report]

exclude_also =  [
  "if TYPE_CHECKING:",
  "class .*\\bProtocol\\):",
]

[tool.pytest.ini_options]

xfail_strict = true
log_cli = true
addopts = ["--strict-markers"]
markers = ["requires_docker_build"]

# Options for pytest-retry.
retries = 10
retry_delay = 10
cumulative_timing = false

[tool.check-manifest]

ignore = [
  "*.enc",
  "admin/**",
  "readthedocs.yaml",
  "CHANGELOG.rst",
  "CODE_OF_CONDUCT.rst",
  "CONTRIBUTING.rst",
  "LICENSE",
  "Makefile",
  "ci",
  "ci/**",
  "codecov.yaml",
  "docs",
  "docs/**",
  ".git_archival.txt",
  "spelling_private_dict.txt",
  "tests",
  "tests/**",
  "vuforia_secrets.env.example",
  "lint.mk",
  "src/mock_vws/_flask_server/Dockerfile",
  "secrets.tar.gpg",
]

[tool.mypy]

strict = true
plugins = ["pydantic.mypy"]

[tool.pydantic-mypy]
init_forbid_extra = true
init_typed = true
warn_required_dynamic_aliases = true
warn_untyped_fields = true

[tool.doc8]

max_line_length = 2000
ignore_path = [
    "./.eggs",
    "./docs/build",
    "./docs/build/spelling/output.txt",
    "./node_modules",
    "./src/*.egg-info/",
    "./src/*/_setuptools_scm_version.txt",
]

[tool.setuptools_scm]

# We use a fallback version like
# https://github.com/pypa/setuptools_scm/issues/77 so that we do not
# error in the Docker build stage of the release pipeline.
#
# This must be a PEP 440 compliant version.
fallback_version = "0.0.0"

[build-system]
build-backend = "setuptools.build_meta"
requires = [
    "pip",
    "setuptools",
    "setuptools_scm[toml]==7.1",
    "wheel",
]

[tool.ruff]
line-length = 79
target-version = "py311"

[tool.ruff.lint]
select = ["ALL"]

ignore = [
    # We do not annotate the type of 'self', or 'cls'.
    "ANN101",
    "ANN102",
    # Ruff warns that this conflicts with the formatter.
    "COM812",
    # Allow our chosen docstring line-style - no one-line summary.
    "D200",
    "D205",
    "D212",
    "D415",
    # Ruff warns that this conflicts with the formatter.
    "ISC001",
    # Ignore "too-many-*" errors as they seem to get in the way more than
    # helping.
    "PLR0913",
    # Allow 'assert' in tests as it is the standard for pytest.
    # Also, allow 'assert' in other code as it is the standard for Python type hint
    # narrowing - see
    # https://mypy.readthedocs.io/en/stable/type_narrowing.html#type-narrowing-expressions.
    "S101",
]

# Do not automatically remove commented out code.
# We comment out code during development, and with VSCode auto-save, this code
# is sometimes annoyingly removed.
unfixable = ["ERA001"]

[tool.ruff.lint.pydocstyle]
convention = "google"

[tool.ruff.lint.per-file-ignores]
"tests/**" = [
    # Allow possible hardcoded passwords in tests.
    "S105",
    "S106",
]

[tool.distutils.bdist_wheel]
universal = true

[project]
name = "vws-python-mock"
description = "A mock for the Vuforia Web Services (VWS) API."
readme = { file = "README.rst", content-type = "text/x-rst"}
keywords = [
    "client",
    "fake",
    "mock",
    "vuforia",
    "vws",
]
license = { file = "LICENSE" }
authors = [ { name = "Adam Dangoor", email =<EMAIL_ADDRESS>]
requires-python = ">=3.12"
classifiers = [
    "Development Status :: 5 - Production/Stable",
    "Environment :: Web Environment",
    "Framework :: Pytest",
    "License :: OSI Approved :: MIT License",
    "Operating System :: POSIX",
    "Programming Language :: Python :: 3 :: Only",
    "Programming Language :: Python :: 3.12",
]
dynamic = [
    "version",
]
dependencies = [
    "flask",
    "Pillow",
    "piq",
    "pydantic-settings",
    "requests",
    "requests-mock",
    "torchvision",
    'tzdata; sys_platform == "win32"',
    "vws-auth-tools",
    "Werkzeug",
]
[project.optional-dependencies]
dev = [
    "actionlint-py==<IP_ADDRESS>",
    "check-manifest==0.49",
    "check-wheel-contents==0.6.0",
    "deptry==0.16.1",
    "dirty-equals==0.7.1.post0",
    "doc8==1.1.1",
    "docker==7.0.0",
    "enum-tools[sphinx]==0.12.0",
    "freezegun==1.5.1",
    "furo==2024.5.6",
    "mypy==1.10.0",
    "pydocstyle==6.3",
    "pyenchant==3.2.2",
    "pylint==3.1.0",
    "pyproject-fmt==2.0.4",
    "pyright==1.1.362",
    "pyroma==4.2",
    "pytest==8.2.0",
    "pytest-cov==5.0.0",
    "pytest-retry==1.6.2",
    "pytest-xdist==3.6.1",
    "python-dotenv==1.0.1",
    "PyYAML==6.0.1",
    "requests-mock-flask==2023.5.14",
    "ruff==0.4.4",
    "Sphinx==7.3.7",
    "sphinx-prompt==1.8",
    "Sphinx-Substitution-Extensions==2024.2.25",
    "sphinx-toolbox==3.5",
    "sphinx_paramlinks==0.6",
    "sphinxcontrib-httpdomain==1.8.1",
    "sphinxcontrib-spelling==8",
    "sybil==6.1.1",
    "tenacity==8.3.0",
    "torch==2.3.0",
    "types-docker==<IP_ADDRESS>40513",
    "types-Pillow==<IP_ADDRESS>40511",
    "types-PyYAML==<IP_ADDRESS>40311",
    "types-requests==<IP_ADDRESS>40406",
    "urllib3==2.2.1",
    "vulture==2.11",
    "vws-python==2024.2.19",
    "VWS-Test-Fixtures==2023.3.5",
    "vws-web-tools==2023.12.26",
]
[project.urls]
Documentation = "https://vws-python-mock.readthedocs.io"
Source = "https://github.com/VWS-Python/vws-python-mock"

[tool.setuptools]
zip-safe = false

[tool.setuptools.packages.find]
where = ["src"]

[tool.setuptools.package-data]
mock_vws = ["py.typed"]

[tool.pyproject-fmt]
indent = 4
keep_full_version = true

[tool.pyright]
reportUnnecessaryTypeIgnoreComment = true
typeCheckingMode = "strict"

[tool.deptry]
pep621_dev_dependency_groups = ["dev"]

[tool.deptry.per_rule_ignores]
# tzdata is needed on Windows for zoneinfo to work.
# See https://docs.python.org/3/library/zoneinfo.html#data-sources.
DEP002 = ["tzdata"]

Small second diff:
--- pyproject.toml

+++ pyproject.toml

@@ -95,6 +95,7 @@

 ]
 urls.Documentation = "https://vws-python-mock.readthedocs.io"
 urls.Source = "https://github.com/VWS-Python/vws-python-mock"
+
 [tool.setuptools]
 zip-safe = false

PR welcome  👍

Not sure if the same bug, but this version is really messing some config files, as seen in https://github.com/pytest-dev/pytest/pull/12321

Similarly with https://github.com/hugovk/norwegianblue/pull/212:

Original pyproject.toml
[build-system]
build-backend = "hatchling.build"
requires = [
  "hatch-vcs",
  "hatchling",
]

[project]
name = "norwegianblue"
description = "CLI to show end-of-life dates for a number of products"
readme = "README.md"
keywords = [
  "cli",
  "end-of-life",
  "endoflife",
  "eol",
]
license = {text = "MIT"}
authors = [{name = "Hugo van Kemenade"}]
requires-python = ">=3.8"
classifiers = [
  "Development Status :: 3 - Alpha",
  "Intended Audience :: Developers",
  "Intended Audience :: End Users/Desktop",
  "License :: OSI Approved :: MIT License",
  "Operating System :: OS Independent",
  "Programming Language :: Python :: 3 :: Only",
  "Programming Language :: Python :: 3.8",
  "Programming Language :: Python :: 3.9",
  "Programming Language :: Python :: 3.10",
  "Programming Language :: Python :: 3.11",
  "Programming Language :: Python :: 3.12",
  "Programming Language :: Python :: 3.13",
  "Programming Language :: Python :: Implementation :: CPython",
  "Programming Language :: Python :: Implementation :: PyPy",
]
dynamic = [
  "version",
]
dependencies = [
  "httpx>=0.19",
  "platformdirs",
  "prettytable>=2.4",
  "pytablewriter[html]>=0.63",
  "python-dateutil",
  "python-slugify",
  "termcolor>=2.1",
]
[project.optional-dependencies]
tests = [
  "freezegun",
  "pytest",
  "pytest-cov",
  "respx>=0.11",
]
[project.urls]
Changelog = "https://github.com/hugovk/norwegianblue/releases"
Homepage = "https://github.com/hugovk/norwegianblue"
Source = "https://github.com/hugovk/norwegianblue"
[project.scripts]
eol = "norwegianblue.cli:main"
norwegianblue = "norwegianblue.cli:main"

[tool.hatch]
version.source = "vcs"

[tool.hatch.version.raw-options]
local_scheme = "no-local-version"

[tool.ruff]
fix = true

[tool.ruff.lint]
select = [
  "C4", # flake8-comprehensions
  "E", # pycodestyle errors
  "EM", # flake8-errmsg
  "F", # pyflakes errors
  "I", # isort
  "ISC", # flake8-implicit-str-concat
  "LOG", # flake8-logging
  "PGH", # pygrep-hooks
  "RUF100", # unused noqa (yesqa)
  "UP", # pyupgrade
  "W", # pycodestyle warnings
  "YTT", # flake8-2020
]
extend-ignore = [
  "E203", # Whitespace before ':'
  "E221", # Multiple spaces before operator
  "E226", # Missing whitespace around arithmetic operator
  "E241", # Multiple spaces after ','
]

[tool.ruff.lint.isort]
known-first-party = ["norwegianblue"]
required-imports = ["from __future__ import annotations"]

[tool.pyproject-fmt]
max_supported_python = "3.13"

[tool.pytest.ini_options]
addopts = "--color=yes"

Second small diff:
diff --git a/pyproject.toml b/pyproject.toml
index 7cb4197..04d7e72 100644
--- a/pyproject.toml
+++ b/pyproject.toml
@@ -59,6 +59,7 @@ urls.Homepage = "https://github.com/hugovk/norwegianblue"
 urls.Source = "https://github.com/hugovk/norwegianblue"
 scripts.eol = "norwegianblue.cli:main"
 scripts.norwegianblue = "norwegianblue.cli:main"
+
 [tool.hatch]
 version.source = "vcs"

Fixed via https://github.com/tox-dev/pyproject-fmt/releases/tag/2.1.0

I don't think 2.1.0 fixes the issue:
https://github.com/mongodb/mongo-python-driver/blob/328c06d4548d30ae7f4d5eaa8d3417ba73e7c05f/pyproject.toml
On the second run:
--- pyproject.toml

+++ pyproject.toml

@@ -49,6 +49,7 @@

 urls.Homepage = "https://www.mongodb.org"
 urls.Source = "https://github.com/mongodb/mongo-python-driver"
 urls.Tracker = "https://jira.mongodb.org/projects/PYTHON/issues"
+
 [tool.setuptools.dynamic]
 version = { attr = "pymongo._version.__version__" }

@@ -183,6 +184,7 @@

   "UP006",  # Use `type` instead of `Type` for type annotation"
   "UP007",  # Use `X | Y` for type annotation
 ]
+
 [tool.pytest.ini_options]
 minversion = "7"
 addopts = [

Thank you @gaborbernat for working on this.
2.1.0 does not fix the issue with the pyproject.toml in the description.


Install iGibson from source  in python2.7

Hi,
I install iGibson from source following installation guide in python2.7. However, when I run
roslaunch gibson2-ros turtlebot_rgbd.launch
It shows:
File "/home/cethik/yejingjing/iGibson/gibson2/simulator.py", line
 if (filename, (*dimensions)) not in self.visual_objects.keys():
                             ^
    SyntaxError: invalid syntax
Do you have the source code of igibson for python 2

Thanks for reporting, we are working on a fix.

@fxia22
Hi, I run pip install gibson2  in my python2 environment， it shows
with open("README.md", "r", encoding="utf-8") as fh:     TypeError: 'encoding' is an invalid keyword argument for this function
It seems the setup.py  is suitable for python3.
Then I download iGibson-0.0.4.zip， when I run pip install -e .
It shows
iGibson-0.0.4/gibson2/core/render/pybind11        does not contain a CMakeLists.txt file.
Finally, I download gibson2-0.0.4-cp27-cp27mu-manylinux1_x86_64.whl, and pip install it.
copy site-packages/gibson2/core/render/pybind11 to /iGibson-0.0.4/gibson2/core/render/pybind11
then run pip install -e . ,it works

Python2.7 building issue is solved in the latest version v1.0.1 and you should be able to install it with python 2.7.


If set frame: false, the window is black

const win = new MicaBrowserWindow({
width: 800,
height: 600,
effect: EFFECT.BACKGROUND.MICA,
theme: PARAMS.THEME.AUTO,
autoHideMenuBar: true,
show: false,
});
If you add a line "frame: false", the windows is black

It's a problem of Windows 11, the API no work when window didn't have frame

I will find a solution but I'm not sure I'll succeed, I'll let you know

I will find a solution but I'm not sure I'll succeed, I'll let you know

Thanks very much!

I will find a solution but I'm not sure I'll succeed, I'll let you know

Thanks very much!

I think it's impossible because when I remove frame with C++, electron crash, sorry ...

I will find a solution but I'm not sure I'll succeed, I'll let you know

Thanks very much!

Hello, I have update the Github, it's now possible !

I will find a solution but I'm not sure I'll succeed, I'll let you know

Thanks very much!

Hello, I have update the Github, it's now possible !

Okay!


Time issues between UTC and EST

Real-time monitoring statistics filtered by dataCenter and grouped by servers:
statisticsService.monitoringStats({
dataCenter: DataCenter.US_EAST_STERLING,
groupBy: Resource.Server,
timeFilter: {
start: moment().subtract(1, 'hours'),
end: moment(),
sampleInterval: '15:00',
type: compute.MonitoringStatsType.REALTIME
}
});
When running this code on a machine in EST, the following error is returned.
{ message: 'Realtime data is only available for four hours. Both the 'start' and 'end' parameters must be within four hours of the current time.\r\nParameter name: start' }
Unhandled rejection Error: Cannot GET https://api.ctl.io/v2/groups//xxx/statistics?start=2016-05-26T10:09:31&sampleInterval=00:00:15:00&type=realtime&end=2016-05-26T10:29:31
It looks like you are sending the dates in an incorrect format?

Setting process.env.TZ = 'GMT'  will force it to work, however not a great solution.  It looks like the sdk is validating on local time, and CL is validating on GMT.
I would also have expected that the result would have been utilization by server, but it groups by Group, not server.  So, who knows...

@gjsmith66
New version of SDK was released. Please update it and check that all works as expected.


importing OutlineExplorerWidget fails after update from 5.1.1 -> 5.1.5

Issue Report Checklist

[x] Searched the issues page for similar reports
[x] Read the relevant sections of the Spyder Troubleshooting Guide and followed its advice
[x] Reproduced the issue after updating with conda update spyder (or pip, if not using Anaconda)
[x] Tried basic troubleshooting (if a bug/error)
[x] Completed the Problem Description, Steps to Reproduce and Version sections below

Problem Description
The following import used to work in 5.1.1 and fails with 5.1.5 :
from spyder.plugins.outlineexplorer.widgets import OutlineExplorerWidget
What steps reproduce the problem?

upgrade spyder in conda from 5.1.1 to 5.1.5 (using conda-forge)
run python3 -c  'from spyder.plugins.outlineexplorer.widgets import OutlineExplorerWidget'

What is the expected output? What do you see instead?
Expected: successful import
Got:
ImportError: cannot import name 'OutlineExplorerWidget' from 'spyder.plugins.outlineexplorer.widgets'

Paste Traceback/Error Below (if applicable)
Traceback (most recent call last):
  File "<string>", line 1, in <module>
ImportError: cannot import name 'OutlineExplorerWidget' from 'spyder.plugins.outlineexplorer.widgets' (/home/cpascual/miniconda3/envs/taurus/lib/python3.9/site-packages/spyder/plugins/outlineexplorer/widgets.py)

Versions

Spyder version:  5.1.5            py39hf3d152e_1    conda-forge
Python version: 3.9.7           hb7a2778_3_cpython    conda-forg
Qt version: 5.12.9               hda022c4_4    conda-forge
PyQt version: 5.12.3           py39hf3d152e_7
Operating System name/version: debian11 with conda

Hey @cpascual, thanks for reporting. I don't know what is this exactly about, but we don't expect Spyder to be used as a library, so we can't promise (and never have done it) a stable API in its internal modules (even in minor releases). We only try to keep a stable API in spyder.api because we expect external developers use it to write third-party plugins, but that will only be the case when that module reaches its 1.0 version (probably in Spyder 6).
Now OutlineExplorerWidget is present in
https://github.com/spyder-ide/spyder/blob/v5.1.5/spyder/plugins/outlineexplorer/main_widget.py#L43

Hi @ccordoba12 , thanks a lot for the answer and the clarification.

I don't know what this is exactly about, but we don't expect Spyder to be used as a library, so we can't promise (and never have done it) a stable API in its internal modules (even in minor releases).

I was not aware of that. I was maintaining some inherited code that used this widget since spyer v2 or so and this explains why we had to track some API changes since then! ;-)

Yeah, sorry about that! We've doing huge refactorings since Spyder 3 but (hopefully) most of them will be over in Spyder 6. The idea is for each plugin to at least have:

plugin.py: where the plugin lives.
confpage.py: its configuration page.
widgets/main_widget.py: where its main widget lives (in case of dockable plugins).
container.py: if the plugin is not dockable but provides graphical widgets.

Some plugins have api.py right now, but we're considering if it'd worth to keep those files, or expose its contents through spyder.api.


Change CODEOWNER from digital-media-marketing to storm

Closing this since it's not needed anymore.


Exec: Add channel closed check before writing in ExecReaderToChannel

Fixes #9839
Signed-off-by: Thomas Parrott<EMAIL_ADDRESS>

OK so that didn't work:
/lxc-ci/build/tmp.YxWULatOed/go/bin/lxc exec leaktest4 --verbose --force-local -- ps -ef
panic: send on closed channel

goroutine 12086 [running]:
github.com/lxc/lxd/shared.ExecReaderToChannel.func3(0x20000, 0xc0021c29a0, 0xc00093c630, 0x27, 0x1fdabe0, 0xc00219ce30, 0xc0021c299c, 0x200af80, 0xc001d24780, 0xc0004cb7a0)
	/lxc-ci/build/tmp.YxWULatOed/go/src/github.com/lxc/lxd/shared/util_linux.go:626 +0x24e
created by github.com/lxc/lxd/shared.ExecReaderToChannel
	/lxc-ci/build/tmp.YxWULatOed/go/src/github.com/lxc/lxd/shared/util_linux.go:508 +0x216
Error: websocket: close 1006 (abnormal closure): unexpected EOF

jenkins: test this please


Windows support

It would be nice if Kani worked on Windows. Also you should probably say somewhere near the start of the Readme or book that it doesn't.

Great point about the docs. I thought it was more prominent... but it's only in the installation instructions. I'll fix that.
Windows support isn't something the Kani team is currently prioritizing, but we'd definitely be happy to merge support if someone out there is motivated to add it.
We're quite interested in seeing Kani used more, of course, so if there are big application domains out there that would like to use Kani but consider lack of windows support a blocker, we'd like to hear about it.

@Timmmm, can you please provide some more details on what you mean by "Windows support"? Kani can be used on Windows using WSL. Is there something platform specific you are looking for?

It's a reasonable workaround given that Kani doesn't support Windows but I meant native support.


--print-width is ignored for --print ml?

Does refmt not fully support OCaml-syntax? I tried the following:
$ cat test.ml
let rec fact = function
| 1 -> 1
| n -> n * fact (n - 1)

let () =
   print_int (fact 5)

$ refmt --parse ml --print ml --print-width 20 ./test.ml
let rec fact = function | 1 -> 1 | n -> n * (fact (n - 1))
let () = print_int (fact 5)
$ refmt --parse ml --print re --print-width 20 ./test.ml
let rec fact =
  fun
  | 1 => 1
  | n =>
    n
    * fact(n - 1);

let () =
  print_int(
    fact(5),
  );

refmt's support for OCaml syntax printing is pretty limited. You will are likely to get much better results using ocamlformat.

Yeah. Unfortunately we don't control the ocaml printer


Fix no color in yaml key

Fixes https://github.com/ikatyang/tree-sitter-yaml/issues/28

These colours are based on base16 spec. TSField is used in more languages than just YAML, it will have a large impact on highlighting.

Sorry, but I don't quite understand about base16 spec. Because I'm using the same base16 theme (https://github.com/chriskempson/base16-vim) in my other machine and the key in yaml is not white at all. The difference is I'm not using Treesitter on my other machine. Does this mean Treesitter is the one using TSField wrong? Should I open an issue there?

There's no good solution but IMO the keys in YAML shouldn't be TSField. This would be the code that handles that.


[Feature]: yarn constraints check in ci

🚀 Feature Proposal
Using the yarn constraints, plug-in check the repo is compliant during the continuous integration checks
Motivation
To keep the repo in compliance with standards and best practices
Example
No response
Pitch
Uh, because 😝

Completed in 9f3bac9


Fix #9658: Make Mixer trace CheckResults and QuotaResults (1.1)

This PR brings #9693 to the 1.1 branch.
The original description of this PR was:
As reported in #9658 Mixer currently doesn't trace failing Check- or QuotaResults.
This lets users completely in the dark why a request might have been rejected.
This PR tries to fix this by logging CheckResult.Status or QuotaResult.Status instead of status.Ok in case there was no error.
If there was an error this always takes precedence and the Check- or QuotaResult is not checked in that case.
This means that with this PR the same number of fields is traced and it doesn't consume more memory.
But it still gives better and more information about what happened.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: robertpanzer
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: douglas-reid
If they are not already assigned, you can assign the PR to them by writing /assign @douglas-reid in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

mixer/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Hi @robertpanzer. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


Example workflow image in README.md is broken

Unless I'm misunderstanding, I believe you're referring to the CI badge, which was fixed in 309bf85.


Search pictures by pictures

Now Meilisearch has started supporting vector search, and once it stabilizes, it will be used for image search.

@Avey777 Thx!
Now this app only focus on hybrid search provided by meilisearch (full-text search).


build: run e2e tests hourly

Signed-off-by: Chris Goller<EMAIL_ADDRESS>To help suss out race-y e2e tests, we'll be running them on an hourly cycle.

@goller should we merge this?


Cells "have to exist" for the shifting to work properly?

input.xlsx
The attached sheet crashes with
An abstract column has been inserted for each sheet with a title column with no abstract column already existing.
[WARNING]
java.lang.IllegalArgumentException: firstMovedIndex, lastMovedIndex out of order
    at org.apache.poi.ss.formula.FormulaShifter.<init> (FormulaShifter.java:83)
    at org.apache.poi.ss.formula.FormulaShifter.createForColumnShift (FormulaShifter.java:126)
    at org.apache.poi.xssf.usermodel.XSSFSheet.shiftColumns (XSSFSheet.java:3032)
    at popbr.AbstractDoiFinder.ShiftColumns (AbstractDoiFinder.java:349)
    at popbr.AbstractDoiFinder.CreateOutput (AbstractDoiFinder.java:276)
    at popbr.AbstractDoiFinder.main (AbstractDoiFinder.java:88)
    at org.codehaus.mojo.exec.ExecJavaMojo.lambda$execute$0 (ExecJavaMojo.java:283)
    at java.lang.Thread.run (Thread.java:840)
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  0.419 s
[INFO] Finished at: 2024-03-25T20:30:46-04:00
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal org.codehaus.mojo:exec-maven-plugin:3.1.1:java (default-cli) on project Abstract_Finder: An exception occurred while executing the Java class. firstMovedIndex, lastMovedIndex out of order -> [Help 1]

but if a cell is given some random value at, say, column 12, then the sheet is processed correctly. My guess is that the spreadsheet does not "create" that cell unless asked explicitly, and that makes the program crash.

This may be a duplicate / solved by https://github.com/popbr/abstract_doi_finder/issues/13

Based on the input given, I would say this issue might be slightly different than the issue we were dealing with earlier. It seems it cannot shift the cells since there is "no cell" there to shift to the right by 1. However, when there is data there (including after every column, even leaving the ones blank), it is able to shift all of it to the right.
I believe adding something as simple as:
if (cell == null || cell.getCellType() == CellType.BLANK)
  // code just inserts the a value in the specific place
else
  // our current code to shift the columns

may fix the issue. However, I fear this may cause an issue when there are blanks between data.

We should try that approach nevertheless, it will still be better. I agree with you that ideally the approach should be "if it's empty let's use it, otherwise let's shift", that seems good enough.

This issue should be fixed by #a9bcb0d.
I've done a bit of testing with the files provided and the file above and it works, but I know you may have some more extensive testing.


GrpcExporter logs its own requests

This should be resolved now since the default HTTP message handler used by both the HTTP and gRPC clients doesn't propagate trace context.


Added Project files to Relevant files to scr folder

Updated csproj folder to make applet ready to build on download.
Hope you found the updates helpful

Sorry,  I'm not following what this request is trying to accomplish. It just looks like you are moving folders around,  which doesn't seem productive here. Can you please clarify?

Never mind. Did move folders around originally to get app to work but decided that fetching git submodule for Serialize component was sufficient. But will keep a working fork of this repos which just runs. Feel free to close. Thanks

Ok, thank you for confirming.


Fixed bug causing iglob to coerce iterator to list

The Globber.iglob() function coerces the iterator returned from _iglob() to a list by using the map() function. I've changed this call to itertools.imap() instead, which returns an iterator.
This solves issue #12

Thanks!


Add tests for loss_scale

See https://github.com/deepmind/jmp for reference implementations

https://github.com/elixir-nx/axon/commit/a2ccf5d08ae62fd5ebf8ab7f81b717e3df9f4cce


Fix wrong documentation

Thank you for the Pull Request. We just merged another PR that solves this same issue, so we'll close this PR, but thank you again for the contribution.


Exception thrown on client.DownloadMessages

when client.DownloadMessages tries to fetch a message that belongs to a user that no longer exists on the server, it will throw a nullreferenceexception.

Ps this is on 0.7.3, might not be a problem in the .9 alpah

Thanks for the report, but 0.9 involved rewriting most of the logic for REST requests, so this likely isn't a problem anymore. I'll check it during my next round of testing just in case.


Broken Link to the Ticket Submission Page

The Get Help Link on the following page needs to be updated:
https://console.ng.bluemix.net/docs/support/index.html#open-ticket-linked
To open the Support widget, log in to the Bluemix console. From the menu bar, click the Support > Find Answers to locate answers to your questions. On the page that is displayed, type your question in the search field. Answers from across the Bluemix documentation and Stack Overflow are displayed. The page also provides options to post your question to Stack Overflow, or to open a ticket by clicking GET IN TOUCH. Although the support widget is the preferred method for obtaining support, if you cannot log in to Bluemix, you can use the Get Help page to submit a ticket as well.
The Get Help page should link to ibm.biz/bluemixsupport instead of https://console.ng.bluemix.net/docs/support/ibm.biz/bluemixsupport

Get Help links have been updated


Support new GPU models

up the versions and dependencies

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 18.826%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
730

Relevant Lines:
3118

💛  - Coveralls


pad ECDSA public key coordinates with zeros

As per RFC 7518: https://www.rfc-editor.org/rfc/rfc7518#section-<IP_ADDRESS>

The "x" (x coordinate) parameter contains the x coordinate for the
Elliptic Curve point.  It is represented as the base64url encoding of
the octet string representation of the coordinate, as defined in
Section 2.3.5 of SEC1 [SEC1].  The length of this octet string MUST
be the full size of a coordinate for the curve specified in the "crv"
parameter.  For example, if the value of "crv" is "P-521", the octet
string must be 66 octets long.

Thank you! :relaxed:


Custom email templates from the Nhost console rather than CLI

Is your feature request related to a problem? Please describe.
It would be great if in the settings section of the Nhost console email templates can be manipulated and updated.
Describe alternatives you've considered
The CLI approach bends the functionality to the developer, while updating email messages is not really developer centric. In larger projects the project manager itself would like to make updates himself.

Could you elaborate a bit more? Why can't the PM use the cli? What are the expectations here?

The client is non tech, even I'd mention CLI they won't know how to install it. The expectation is similar to update things like SMTP server and all the social media details - it would be great if email responses would be handled inside the settings as well.

ok, but if the client is non tech, are you expecting some form of WYSIWG editor? Because if that's the case that's a whole different request IMO. Adding the templates as a setting in the dashboard is one thing, having a fully fledged "no code" solution might be another.

not WYSIWYG on Nhost side. Just a text input where the client can copy paste the email structure either as plain text or html. Most of the time what I've seen on other backend solutions two different textarea fields, one for HTML and one for TEXT for each message.

ok, thanks for the clarification


New Release

First off, really cool project.
I installed last release with Mint and it outputs:
INFO  [2021-02-24 13:08:43]: ▸ File: Domain/<someFile>.swift
ERROR [2021-02-24 13:08:43]: ▸ --------------------------------------------------------------------------------------
ERROR [2021-02-24 13:08:43]: ▸ Analyzed 0 files with nan% overall cohesion. Threshold is 80%

But when cloning and building the project it works fine.
I've noticed that there's been quite a few code changes since last release, simply creating a new release would fix it, I think

I totally forgot to release 0.5.9 🙃
It's done now. Let me know if there are any issues with it and I'll take a look asap.
I'm also worked on improving the logs, as you can see below, but that will be released soon in 0.6.0. Noticed that it's miscalculating the usage of properties.


add GitHub Sponsors

Could developers add GitHub Sponsors option in this repository?
Thanks.

Yes, I'm working in this. It's not as simple because we're an org that's part of the PSF, not a single dev.


Improve dry integration

provide a reform base schema which adds option :form
add schema_class as an option on a validation group which will be
used instead of the reform base class

@apotonick I know that this can't be merged yet as it messes with the group API which will break reform-rails. Any ideas?

I've drafted a PR for reform-rails to migrate the AM methods and enable passing options to Group.new. Just need to fix a pre-existing failure before I publish it

Fixes #378

@fran-worley I'm using a lot of schema extension, it might be nice to shorten schema_class: Foo::Schema to schema: Foo::Schema since it is likely to look like a class reference anyway.
Also, since using the schema option, it seems (from reading the code) that I need to name every group. It would be nice if it the dsl would see I am using an option and not make me pass :default as the group name.
Please correct me if I've misunderstood any of the above.

Noted on the option name. Happy to change. You shouldn't have to pass in a name to every group. It defaults to :default and this branch makes no changes there.

I've just confirmed that the earlier 70013bb482fe6ab329e225aa1d7b20f5ca558d29 fails when you use schema_class without a group name (none of my base predicates are found).  I'll pull this latest and give it another shot.

Group name is required when using schema_class on latest 3746bf23ec8478f8923a5e4f9ef70330c518a76c as well.

@rosskevin You are quite right, now fixed. I've also renamed the option to just :schema


Fix requirement against openshift/api.

Proposed Changes
./hack/update-deps.sh is broken for me without this. openshift/api doesn't have published versions so that seems to trip go mod.
Example:
$ ./hack/update-deps.sh
warning: ignoring symlink /home/mthoemme/dev/knative/operator/test/config/serving
go<EMAIL_ADDRESS>requires
<EMAIL_ADDRESS>requires
<EMAIL_ADDRESS>invalid pseudo-version: preceding tag (v3.9.0) not found

/assign @houshengbo @jcrossley3

@markusthoemmes
Why do we use openshift/api built based the commit https://github.com/openshift/api/commit/32369d4db2ada1c1cdb95b99b568a9925e2a4480, which is almost 8-month old?
Not using recent commit?

No longer necessary.


TypeError: Cannot read properties of undefined (reading 'humanLabel')

I am not sure why do I see the error while generating artifacts from this spec:
Cannot read properties of undefined (reading 'humanLabel')

TypeError: Cannot read properties of undefined (reading 'humanLabel')
    at s (https://tool.dataspecer.com/static/js/main.ae0aff2c.chunk.js:1:484014)
    at https://tool.dataspecer.com/static/js/main.ae0aff2c.chunk.js:1:485242
    at h (https://tool.dataspecer.com/static/js/main.ae0aff2c.chunk.js:1:485428)
    at https://tool.dataspecer.com/static/js/main.ae0aff2c.chunk.js:1:483911
    at Array.map (<anonymous>)
    at c (https://tool.dataspecer.com/static/js/main.ae0aff2c.chunk.js:1:483887)
    at https://tool.dataspecer.com/static/js/main.ae0aff2c.chunk.js:1:487087
    at https://tool.dataspecer.com/static/js/main.ae0aff2c.chunk.js:1:482714
    at Object.next (https://tool.dataspecer.com/static/js/main.ae0aff2c.chunk.js:1:482819)
    at https://tool.dataspecer.com/static/js/main.ae0aff2c.chunk.js:1:481730

Maybe it is some older PSMs missing something? I am not even sure whether this is in documentation generator or core.
artifact (1).zip

It might have something to do with a broken OR in the spec:

The OR is no longer in the spec, so this is no longer happening. Closing for now.


Releasing all TF checkout point of elmo?

Hi, Thanks for this great work.
I am wondering if we can have all the tf checkpoint file? (e.g., 5.5B elmo checkpoint?) so that people can use it for LM usage.

There is a checkpoint for download but it's in the FAQ section, probably you missed it?
https://github.com/allenai/bilm-tf#can-you-provide-the-tensorflow-checkpoint-from-training


setup

Django setup.
i setup sqlite3 database as demo. If you want to use another you can change as well.

everything should be in the develop branch

Its just a setup. So, no issue will be there.


ASK cluster

I think there is a typo here: "This option requires CNI networking to be enabled when you configure the ASK cluster...". Isn't it AKS cluster instead?

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

ID: c5e2e444-d6b9-962e-56d4-8d1f3508ba9e
Version Independent ID: 061ea6aa-353b-a98d-cbf6-639209a5fd1b
Content: Microservices architecture on Azure Kubernetes Service (AKS)
Content Source: docs/reference-architectures/microservices/aks.md
Service: architecture-center
Sub-service: reference-architecture
GitHub Login: @MikeWasson
Microsoft Alias: pnp

@satrijotomo https://godoc.org/github.com/Azure/azure-sdk-for-go/services/eventgrid/2018-01-01/eventgrid#KeyVaultCertificateNearExpiryEventData

Thank you for your feedback! We will review and provide an update as appropriate.

@satrijotomo Thank you for your feedback! We will review and provide an update as appropriate.

@satrijotomo Thank you for pointing this out!
I have created a PR to fix the typo. The changes will be available to the document in next 24-48 hrs.
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


fix: update grpc-health-probe

:tada: This PR is included in version 1.3.5 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Use h5py compiled with MPI features

Used the version of h5py in conda-forge compile with openmp.

python 3.7:
  conda install -c conda-forge h5py=2.9.0=mpi_openmpi_py37hc1ffa35_1002

Use MPICH instead of openMPI.
Python 3.7:
conda install -c conda-forge h5py=2.9.0=mpi_mpich_py37hc9c12dd_1002


Hyperlink not resolved in toc (e24010/kindlegen)

Describe the bug
I can't convert Berserk to mobi. I first tried doing automatic splitting off, but kcc complained that epubs were 500mb and not 450 max. So, I manually separated it into volumes, but now I'm getting an error of KindleGen

To Reproduce
Steps to reproduce the behavior:

Add directory with manga separated in volumes
Turn on output split, cropping mode, manga mode, mozjpeg, stretch upscale and half spread splitter
Epubs are created but kindlegen fails to mobi them
See error

Expected behavior
It should've created all the mobis for the volumes.
 I changed to DX by mistake, it should've been PW 4. I'll retry.

Yea, I renamed the pw profiles since too many people picked kindle 11 when they had kindle pw5 11th gen

Failed again, and I have the same settings as I had on windows when it worked.

Can you drag in the volumes instead of the containing folder instead?

Do I uncheck output split?

Ok, cool, so to confirm you said this issue only exists on the flatpak version?

It seems to work fine if I drag all the volumes, it still isn't finished but no errors yet so all good.
About your question, I haven't tried appimage version yet, want me to try?

Tried appimage but I'm getting this error. I have setup kindlegen properly

Doing from an archive also doesn't work.

@jabuxas i suspect this is fixed in 6.3.1

i'll try to replicate it this weekend. been busy.


[Broken] Youtube playlist returns ID when using %(playlist)s and "NA" when using %(playlist_title)s

Checklist

[x] I'm reporting a broken site support
[x] I've verified that I'm running youtube-dlc version 2020.11.11-3
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar issues including closed ones

Verbose log

[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', '-f', 'bestvideo[ext=mp4]+bestaudio[ext=m4a]', 'https://www.youtube.com/playlist?list=PL1rWy_LjJQIO7TqcmhETvXqMQZfJFNfP2', '-o', '%(playlist)s/%(title)s.%(ext)s']
[debug] Loading archive file None
[debug] Encodings: locale cp1252, fs utf-8, out utf-8, pref cp1252
[debug] youtube-dlc version 2020.11.11-3
[debug] Python version 3.8.6 (CPython) - Windows-10-10.0.19041-SP0
[debug] exe versions: ffmpeg git-2020-08-31-4a11a6f, ffprobe git-2020-08-31-4a11a6f
[debug] Proxy map: {}
[youtube:playlist] PL1rWy_LjJQIO7TqcmhETvXqMQZfJFNfP2: Downloading webpage
[download] Downloading playlist: PL1rWy_LjJQIO7TqcmhETvXqMQZfJFNfP2
[youtube:playlist] playlist PL1rWy_LjJQIO7TqcmhETvXqMQZfJFNfP2: Downloading 7 videos
[download] Downloading video 1 of 7
[youtube] -Kbx2bXZmpc: Downloading webpage
[debug] Invoking downloader on 'https://r9---sn-uqj-j2ik.googlevideo.com/videoplayback?expire=1605589477&ei=hQWzX5LKNsuHv_IPpaiTwAc&ip=<IP_ADDRESS>&id=o-AF4CRa_Ofw2_Q7_dMLL9I1RmWttm7fbXhS52D4K8wqF2&itag=137&aitags=133%2C134%2C135%2C136%2C137%2C160%2C242%2C243%2C244%2C247%2C248%2C278&source=youtube&requiressl=yes&mh=PC&mm=31%2C29&mn=sn-uqj-j2ik%2Csn-5go7yner&ms=au%2Crdu&mv=m&mvi=9&pl=19&initcwndbps=2071250&vprv=1&mime=video%2Fmp4&ns=q3MsYPAOgd8DPrIf3azNsO0F&gir=yes&clen=11506454&dur=268.200&lmt=1531327528877994&mt=1605567769&fvip=3&keepalive=yes&c=WEB&n=uYlDTpAKJftElwIMI&sparams=expire%2Cei%2Cip%2Cid%2Caitags%2Csource%2Crequiressl%2Cvprv%2Cmime%2Cns%2Cgir%2Cclen%2Cdur%2Clmt&sig=AOq0QJ8wRAIge7moml5fy1YfKPm1rTHcg9tWSaCmBTIGMhBy0uBSS9cCIFiCd6d5r3LsU-HjlVX_3HRMMTvDbJd3f_jQv2SV2PdK&lsparams=mh%2Cmm%2Cmn%2Cms%2Cmv%2Cmvi%2Cpl%2Cinitcwndbps&lsig=AG3C_xAwRQIgFVshpFri3d89Cgze1VVsTYs4IFtrBW-xzmkryXnwDHICIQCX36bWP3CGnDh3KwPOWotpHVvN6WKSnA3ggoUxCtf4-g%3D%3D&ratebypass=yes'
[download] Destination: PL1rWy_LjJQIO7TqcmhETvXqMQZfJFNfP2\Ivan Torrent - One Of Us (feat. Julie Elven).f137.mp4
[download]  18.2% of 10.97MiB at  5.86MiB/s ETA 00:01

[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', '-f', 'bestvideo[ext=mp4]+bestaudio[ext=m4a]', 'https://www.youtube.com/playlist?list=PL1rWy_LjJQIO7TqcmhETvXqMQZfJFNfP2', '-o', '%(playlist_title)s/%(title)s.%(ext)s']
[debug] Loading archive file None
[debug] Encodings: locale cp1252, fs utf-8, out utf-8, pref cp1252
[debug] youtube-dlc version 2020.11.11-3
[debug] Python version 3.8.6 (CPython) - Windows-10-10.0.19041-SP0
[debug] exe versions: ffmpeg git-2020-08-31-4a11a6f, ffprobe git-2020-08-31-4a11a6f
[debug] Proxy map: {}
[youtube:playlist] PL1rWy_LjJQIO7TqcmhETvXqMQZfJFNfP2: Downloading webpage
[download] Downloading playlist: PL1rWy_LjJQIO7TqcmhETvXqMQZfJFNfP2
[youtube:playlist] playlist PL1rWy_LjJQIO7TqcmhETvXqMQZfJFNfP2: Downloading 7 videos
[download] Downloading video 1 of 7
[youtube] -Kbx2bXZmpc: Downloading webpage
[debug] Invoking downloader on 'https://r9---sn-uqj-j2ik.googlevideo.com/videoplayback?expire=1605589565&ei=3QWzX9zHDpSYyQWu5KbIDw&ip=<IP_ADDRESS>&id=o-ADLJ_ahirC9IE5RWBW4flHkosEQ72-2KsSxRVIGpcSlu&itag=137&aitags=133%2C134%2C135%2C136%2C137%2C160%2C242%2C243%2C244%2C247%2C248%2C278&source=youtube&requiressl=yes&mh=PC&mm=31%2C29&mn=sn-uqj-j2ik%2Csn-5go7yner&ms=au%2Crdu&mv=m&mvi=9&pl=19&initcwndbps=2071250&vprv=1&mime=video%2Fmp4&ns=LUBjB7yiothFUfS5-L5W0XQF&gir=yes&clen=11506454&dur=268.200&lmt=1531327528877994&mt=1605567769&fvip=3&keepalive=yes&c=WEB&n=u4k_x8O-182EBxKu4&sparams=expire%2Cei%2Cip%2Cid%2Caitags%2Csource%2Crequiressl%2Cvprv%2Cmime%2Cns%2Cgir%2Cclen%2Cdur%2Clmt&sig=AOq0QJ8wRQIhAPVKRnYYWVWCID-IKUkWegnEEU_gJ628BwUWINYacb5jAiAqUHOjoGYaVWsrC864mU3oMjhfM9gUaavLjitXZO0Z7g%3D%3D&lsparams=mh%2Cmm%2Cmn%2Cms%2Cmv%2Cmvi%2Cpl%2Cinitcwndbps&lsig=AG3C_xAwRgIhAMBcL5xmySKmKk3cKHpbcrOsJhQxx8wOvFuUKtpY01PxAiEA_06CJbL2HLN5V5aSkmoTMfTTBpY6crW6AtjQMXiZbLw%3D&ratebypass=yes'
[download] Destination: NA\Ivan Torrent - One Of Us (feat. Julie Elven).f137.mp4
[download]  18.2% of 10.97MiB at  5.81MiB/s ETA 00:01

Description

I've tested multiple playlists on YouTube, both private, unlisted and public and all seem to return the ID of the playlist when using %(playlist)s and "NA" when using %(playlist_title)s. Other template outputs appears to work fine so I'm guessing Google updated something at YouTube once again related to playlists. Previous versions of youtube-dlc, albeit not being able to download the playlist due to previous changes, also return either "NA" or the playlist ID, eliminating it from being caused by changed code in youtube-dlc.

I believe this is related to #173

I believe this is related to #173

Ah yes I see, my apologies for the late reply.
I am new and unfamiliar with Github and handling of issues. I would assume I should just close this issue as it's been reported in another issue?


Ran out of memory running tests.

I was running out of mem on a machine with 16Gb ram running 200+ tests with lots of calls to trace!().
heaptrack reported over 6.5Gb leaked by a tracing-subscriber MockWriter that just keeps growing a Vec.   This led me to the tracing-test docs, which state:

The macro provided in this crate registers a global default subscriber instead. This subscriber contains a writer which logs into a global static in-memory buffer.

I'm curious:

Why is it writing to an unbounded global buffer to begin with?
Is there some way to disable that behavior?
Are there plans to change this in the future?

I guess the buffer could be bounded as an option. Right now this is not configurable.
There are no plans from my side to change this, but if the implementation would be simple, I'd be open for pull requests.

Why is it writing to an unbounded global buffer to begin with?

Answering my own question, I guess it is because cargo test (test executable) buffers test stdout by default.   So it would make sense then that if tracing-test is converting log output to stdout, it would need to do the same/similar.   Though I don't pretend to understand the details.
Taking this further though, it seems to me that the output of each test should only need to be buffered until the test end and we learn if it was successful or not.   If successful, then the output can be discarded.
Instead, the present behavior is to buffer everything until all tests complete, whether successful or not.
So @dbrgn, do you think it makes sense and is feasible to buffer each test separately and discard buffers for any test that completes successfully?

I guess the buffer could be bounded as an option

yeah, if my above analysis is correct, then bounding the buffer should only be a last resort.   Discarding buffer(s) after each successful test would eliminate the out-of-mem problem, at least for my  test-suite.

@dan-da I agree that your suggestion makes sense. However, the implementation is tricky.
I see two (or three) approaches:

Having a separate log buffer per test
This would be the nicest solution, because we could simply Unfortunately we must set up a global subscriber (to capture logs from different threads) and tests don't run in separate processes (and potentially in parallel), so we cannot separate subscribers per test.
Having a shared buffer but removing all test-logs at the end of the test
Here the difficulty is that the buffer is currently all-bytes, not lines (because we simply use the SubscriberBuilder::with_writer API). Log lines would need to be removed by line prefix though.
In cases where a lot of logs are present (which is the case when you have high memory pressure) this might be inefficient. On the other hand, current log lookups also convert the buffer to lines and scan through all lines, so the difference might be small in practice. So it might be worth an attempt.
My suggestion:

Create a function similar to logs_with_scope_contain called flush_logs_with_scope
Call this function at the end of every test in the traced_test macro

Custom subscriber
Maybe we could write a custom subscriber (in internal::get_subscriber()) that logs to different buffers depending on the test span. I don't yet know whether that would work, and how easy it would be. It would be the most elegant solution from an API view, but the code would need to be sufficiently maintainable.

For me, this doesn't currently have high priority, but a PR would be welcome.

thx for the writeup.
I was able to workaround the mem-leak in our codebase by disabling some super-verbose trace logging that isn't really needed.  So that solves our immediate issue.
I don't really have time to make a PR myself at present, but hopefully someone will eventually if we leave this open.


[config] clean arana project log output

What would you like to be added:
Why is this needed:

add log rotation settings into bootstrap.yaml, check the existing log environments.
normalize logical/physical SQL log format.


"Răbăgani, Albești, Brătești, Săliște de Pomezeu, Săucani, Vărășeni" are villages in "Răbăgani" commune, "BIHOR" county

Description
When an user search a location, this appear in the drop-down menu
according to the law
Precondition
The website ( https://peviitor-ro.github.io/ui.orase/ ) is up an running.
Empty the search bar after every step

Steps
Write "Răbăgani"
Expected result
In the drop down menu you find "Village , BIHOR(Răbăgani)"
Test Data
Răbăgani

Steps
Write ""
Expected result
In the drop down menu you find "Village , BIHOR(Răbăgani)"
Test Data

Steps 3
Write "Brătești"
Expected result
In the drop down menu you find "Village Brătești, BIHOR(Răbăgani)"
Test Data
Brătești

Steps 4
Write "Săliște de Pomezeu"
Expected result
In the drop down menu you find "Village Săliște de Pomezeu, BIHOR(Răbăgani)"
Test Data
Săliște de Pomezeu

Steps 5
Write "Săucani"
Expected result
In the drop down menu you find "Village Săucani, BIHOR(Răbăgani)"
Test Data
Săucani

Steps 6
Write "Vărășeni"
Expected result
In the drop down menu you find "Village Vărășeni, BIHOR(Răbăgani)"
Test Data
Vărășeni

Environment: Run Tests - 4416
Browser: Chrome 123
OS: Windows 10
Steps to reproduce:
1 Open the @Adelina Guliman Test case ui.orase
2. Check TC : "Răbăgani, Albești, Brătești, Săliște de Pomezeu, Săucani, Vărășeni" are villages in "Răbăgani" commune, "BIHOR" county
#4416
Expected results:
Precondition (request):
The website link (https://peviitor-ro.github.io/ui.orase/) is up and running
The link to the law is displayed : "Sat Răbăgani, BIHOR (Răbăgani)"
Actual result (displayed):
The STEP have request to displayed "Village Răbăgani, BIHOR(Răbăgani)"
All STEPS (from 1 to 6) have "VILLAGE" instead of "SAT"


libxml2: fix CVE-2018-14567 & CVE-2018-14404

Motivation for this change
Fixes CVE-2018-14567 & CVE-2018-14404
Since the already added patch for CVE-2018-9251 also affects
CVE-2018-14567 I renamed the applied patch accordingly.
Things done

[X] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[X] Fits CONTRIBUTING.md.

Success on aarch64-linux (full log)
Attempted: libxml2
Partial log (click to expand)
stripping (with command strip and flags -S) in /nix/store/p0g8sah8mwpxrrvficflkg7r130myhzq-libxml2-2.9.8-py/lib
patching script interpreter paths in /nix/store/p0g8sah8mwpxrrvficflkg7r130myhzq-libxml2-2.9.8-py
checking for references to /build in /nix/store/p0g8sah8mwpxrrvficflkg7r130myhzq-libxml2-2.9.8-py...
Moving /nix/store/20w9svy9h6i0riyhxiw607v7gb4y3zhc-libxml2-2.9.8-bin/bin/xml2-config to /nix/store/8dk3ckjcx4xjvs52wzpi6r0ym51v3pyy-libxml2-2.9.8-dev/bin/xml2-config
rmdir: failed to remove '/nix/store/20w9svy9h6i0riyhxiw607v7gb4y3zhc-libxml2-2.9.8-bin/bin': Directory not empty
Moving /nix/store/2nn12kbsjvh36lcpzxd2wg1sqxwy1mn9-libxml2-2.9.8/lib/xml2Conf.sh to /nix/store/8dk3ckjcx4xjvs52wzpi6r0ym51v3pyy-libxml2-2.9.8-dev/lib/xml2Conf.sh
rmdir: failed to remove '/nix/store/2nn12kbsjvh36lcpzxd2wg1sqxwy1mn9-libxml2-2.9.8/lib': Directory not empty
Moving /nix/store/pb2ds9k600x7p0n02gn20dcgg767pj94-libxml2-2.9.8-man/share/man/man1 to /nix/store/20w9svy9h6i0riyhxiw607v7gb4y3zhc-libxml2-2.9.8-bin/share/man/man1
rmdir: failed to remove '/nix/store/pb2ds9k600x7p0n02gn20dcgg767pj94-libxml2-2.9.8-man/share/man': Directory not empty
/nix/store/20w9svy9h6i0riyhxiw607v7gb4y3zhc-libxml2-2.9.8-bin

Timed out, unknown build status on x86_64-darwin (full log)
Attempted: libxml2
Partial log (click to expand)
cannot build derivation '/nix/store/gm6nzb9vdfglbnd4p3snjc2dhfz53ll4-libtool-2.4.6.drv': 5 dependencies couldn't be built
cannot build derivation '/nix/store/nwz4dz77qd2jk85yijz54ss9350kg390-hook.drv': 6 dependencies couldn't be built
cannot build derivation '/nix/store/dz3pb9ip5rwqvxi30m3y1lqgj770qhj8-flex-2.6.4.drv': 7 dependencies couldn't be built
cannot build derivation '/nix/store/xxm8lzyfrwh4hc3vcqvsvb2qgq1hv99f-bootstrap_cmds-dev-tools-7.0.drv': 4 dependencies couldn't be built
cannot build derivation '/nix/store/z7zysz0vihqp0l9dypycyvdz7v30p1xc-xnu-osx-10.11.6.drv': 9 dependencies couldn't be built
cannot build derivation '/nix/store/xb7vkmmvqbpz49h1dws0cmszxqxgy161-IOKit-osx-10.11.6.drv': 3 dependencies couldn't be built
cannot build derivation '/nix/store/5x9ib5wpbn7qzvdr83wpf3ma4q9gv2gw-configd-osx-10.8.5.drv': 9 dependencies couldn't be built
cannot build derivation '/nix/store/n2l3077g9qbm9l635b9akx2qkdjzv3n0-python-2.7.15.drv': 15 dependencies couldn't be built
cannot build derivation '/nix/store/70qv038hgx3v7vhig9z7wpwd95xknx4m-libxml2-2.9.8.drv': 5 dependencies couldn't be built
error: build of '/nix/store/70qv038hgx3v7vhig9z7wpwd95xknx4m-libxml2-2.9.8.drv' failed


Update Filesystem.php

It's need to release directory lock. to avoid error: rmdir($directory); Text file busy. #41353

I am closing this pull request because it lacks sufficient explanation, tests, or both. It is difficult for us to merge pull requests without these things because the change may introduce breaking changes to the framework.
Feel free to re-submit your change with a thorough explanation of the feature and tests - integration tests are preferred over unit tests. Please include it's benefit to end users; the reasons it does not break any existing features; how it makes building web applications easier, etc.
Thanks!

When i use $this->app->files->deleteDriectory(); it cannot remove the directory. because $items = new FilesystemIterator($directory); take the resource lock, and not release the lock. @taylorotwell


About choosing hard negative pairs

Maybe in class HardNegativePairSelector, the hard negative pairs should be the pairs with closest distances, since the objective is to separate the embeddings in a negative pair.
 Will fix it now.

Fixed in #4