API Refactoring to export symbols in submodules explicitly

As addressed in issue #44 , it would be desirable to refactor the current API and ship it with the next release of Chmy such that users can simply use using Chmy instead of concrete import of submodules as we have now. This PR starts with a state where all submodules have been refactored besides two, which we will adress below.
With the aim to avoid the usage of additional Reexport.jl package, we followed a similar approach as used in Oceananigans.jl.
The remaining things to do include the followings, I would like to discuss with @utkinis and @luraess and see how we want to best do it before further refactoring:

[x] Explicitly export Grids and GridOperators, ➡ accordingly remove usage of using Grids and using GridOperators in the test suite
[x] get_backend() of Architectures module name clashes with KernelAbstractions.jl ➡ Thus is test suite the module is used explicitly yet

To the first item: the Grids and GridOperators submodules contain parts where the symbols are exported in a rather exotics way (ref. cartesian_field_operators.jl for an example), how do we want to reexport these expressions?

To the second item, currently users need to explicitly use KernelAbstractions.jl (and I think it is good this way). But possible name clashes could be problematic, do we want to rename the relevant functions or is there maybe a workaround for it too?

Some clean-up jobs would be to modify relevant parts of the documentation & the examples (this is not necessary, because using submodules works too even with our API change, but maybe good for users to avoid confusions?).

To the first item: the Grids and GridOperators submodules contain parts where the symbols are exported in a rather exotics way (ref. cartesian_field_operators.jl for an example), how do we want to reexport these expressions?

I suggest doing it in the most straightforward way. These "exotic" exports are only needed to generate functions for Cartesian grids with coordinate names x, y, z, which will probably change in the future, but not from API perspective. So exports from Chmy.Grids and Chmy.GridOperators will contain following lines:
# Grids
export coord, xcoord, ycoord, zcoord
export Δ, Δx, Δy, Δz
# other exports...

# GridOperators
export δ, δx, δy, δz
export ∂, ∂x, ∂y, ∂z
# other exports...

To the second item, currently users need to explicitly use KernelAbstractions.jl (and I think it is good this way). But possible name clashes could be problematic, do we want to rename the relevant functions or is there maybe a workaround for it too?

Yeah, I should've done this long time ago, here it should be:
KernelAbstractions.get_backend(arch::SingleDeviceArchitecture) = arch.backend

which would remove the name clash

Are we good to go with this one @utkinis ?

Thanks for the work @youwuyou and @luraess ! Looks good to me, I guess it deserves a non-breaking release


Question: Maybe untangle Borsh into it's own package?

What would this mean?

Should the Buffer become part of the Borsh package?
Would the Exceptions become part of the Borsh package?
What about the PublicKey that now is part as of this change (https://github.com/tighten/solana-php-sdk/commit/d032405c5f5ef7d91d7f245a2e0f11cea6eb3c4f#diff-bc3a24552d6e6304731de9cd989008ec466e662dfd2d77273ea76f4749755970)?
Anything I'm missing?

Would love to help out

Ping @mattstauffer, only if you have a moment good ser

Whoops, I'm sorry I didn't see this originally! I would love to extract it... but I don't know how to answer your questions. I think exceptions are the easiest: we can just make custom exceptions in that package and update this package to be aware of them.
With Buffer, if we need to use buffers in our Borsh package, then this package can just consume those, so that's fine.
I think the same will have to be true about the PublicKey. Basically anything the Borsh package needs, should move there, and then this package can just get them from there, even if that seems a bit silly.


gnome.gnome-calendar: 40.0 -> 40.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for gnome.gnome-calendar is: "Simple and beautiful calendar application for GNOME"
meta.homepage for gnome.gnome-calendar is: "https://wiki.gnome.org/Apps/Calendar"
meta.changelog for gnome.gnome-calendar is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

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

/nix/store/1d4aga9s9imdfpwa84fpfc91qcpl0jwf-gnome-calendar-40.1/bin/gnome-calendar passed the binary check.

Warning: no invocation of /nix/store/1d4aga9s9imdfpwa84fpfc91qcpl0jwf-gnome-calendar-40.1/bin/.gnome-calendar-wrapped had a zero exit code or showed the expected version

1 of 2 passed binary check by having a zero exit code.

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

found 40.1 with grep in /nix/store/1d4aga9s9imdfpwa84fpfc91qcpl0jwf-gnome-calendar-40.1

found 40.1 in filename of file in /nix/store/1d4aga9s9imdfpwa84fpfc91qcpl0jwf-gnome-calendar-40.1

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

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

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

3 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
0 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
gnome.gnome-calendar

Instructions to test this update (click to expand)

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

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A gnome.gnome-calendar https://github.com/r-ryantm/nixpkgs/archive/5600ea29403b769a39435e2ba253a0ae568df61d.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/1d4aga9s9imdfpwa84fpfc91qcpl0jwf-gnome-calendar-40.1
ls -la /nix/store/1d4aga9s9imdfpwa84fpfc91qcpl0jwf-gnome-calendar-40.1/bin

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

1 package built:

gnome.gnome-calendar

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

Duplicate of #122997

Result of nixpkgs-review pr 123004 at 5600ea29 run on x86_64-linux 1

1 package built successfully:

gnome.gnome-calendar

1 suggestion:

warning: unclear-gpl
gpl3 is a deprecated license, please check if project uses gpl3Plus or gpl3Only and change meta.license accordingly.
Near pkgs/desktops/gnome/apps/gnome-calendar/default.nix:81:5:
   |
81 |     license = licenses.gpl3;
   |     ^


Run agent in root domain

Mockups

Specifications

The feature Run Agent in a root domain is very similar to feature Debug Agent in Agent section
When user click on button Run, a confirmation popup will appear. The user will need to introduce a command in the confirmation popup. That command will be executed through the selected agent in the root domain. User will be able to activate or not a notification to let him/her know when the execution of the agent finish
The confirmation popup will include 2 buttons: Run and Done. Clicking Done button will close the confirmation popup and will abort the action.
Clicking in Run button will close the Confirmation popup and will open a popup with a terminal. The terminal will display the output of the agent execution in the root domain.
The terminal popup will include 2 buttons: Stop and Close. Clicking on Close button will close the Terminal popup but it will NOT interrup the execution of the Agent. Clicking in Stop button will abort the execution of the Agent and it will close the Terminal popup.
When the Agent is running, the Run button, in the Agents list, will be hidden and will appear 2 buttons in its place: Stop and Terminal.
The user will be able to close the Terminal popup at any time. This won't interrupt the execution of the Agent. If the user closes the Terminal popup, he will be able to open it again by clicking on Terminal button.
If the user wants to Stop the execution of the agent he will need to click on Stop button, in Agent list.
2 or more agents can NOT be executed at the same time, so when an Agent is running in the root domain, all other Run buttons will be disabled. Once the execution of the agent is stopped by the user or if the execution finishes, then all Run buttons in the list will be enabled again.

check the design

@gorums Please, check this story again

@YanetABUC  We dont need Pause option, and we can Run more than one Agent in the same time, if the agent is waiting we are going to show Waiting..., instead Running...
.
Status of the execution. Possible status are: Running, Waiting


Make wand plugin cleanup after itself.

If you disabled the wand plugin, the currently outlined element would
stay outlined. This change fixes that.

LGTM!


Update minimum supported Xcode version

For Firebase 8, review minimum supported Xcode, update, and make sure docs are consistent.  Perhaps 12.2?

We've already moved to 12.2, just need to coordinate this for docs mostly.


Support SQL Server

Go has a decent driver for SQL Server. I'm sure some other people use this database.
Are there any plans to provide a dialect for it?

xo supports SQL Server, and it seems xo is compatible with sqlx if the templates are tweaked


[QUESTION] default high cpu usage - is it normal?

Hi,
I noticed that even with the very default hello world, uvicorn consumes about ~40% of my 2.5GHz CPU (of single core). Is this normal/expected? From WSGI I am used to values around ~5%. I was able to replicate these on two machines.

Try not using the reload parameter when starting the server and tell us again.
uvicorn main:app --reload

Ah, yes, without the reload parameter the usage is minimal. Sorry and thanks for a quick feedback.

Tom of Starlette is very open to taking a PR to make the reload option more efficient. Right now it is brute force checking every 300ms? or so.

Thanks for the help @jorgerpo and @wshayes ! :cake: :rocket:
And thanks @hnykda for reporting back and closing the issue. :tada:

possible solution:
@hnykda
i am using cputhrottle to limit how high the python process can go
http://www.willnolan.com/cputhrottle/cputhrottle.html
https://medium.com/@sbr464/limit-dropbox-and-others-on-macos-from-taking-100-cpu-877266df104d
i run pc run, then i run this command
sudo ./cputhrottle $(pgrep -f -i Python | head -1) 5

this will find the python process with highest cpu value and limit it to 5%
as a result my cpu temp stays cool. the live reload still works but has a bit of a lag (a few seconds)


docs: added docs for creating a new package

Overview
Description
Formalizing notes taken down in a brief pairing with @ianmunrobot earlier today.

🚀 Styleguide deploy preview ready!https://5f6921a606752e00cf45d8cf--gamut-preview.netlify.appDeploy Logs

🚀 Styleguide deploy preview ready!https://5f6ce8ed57410c2ea9d5e0b1--gamut-preview.netlify.appDeploy Logs


Trace information not displayed on error (Win)

In SketchUp 2016, Windows, in case of an error no trace information is displayed in the console - this makes error tracking impossible without reverting to the native console.
For comparison:
Output of  native SketchUp console (filename, linenumber being displayed):
errortest()
Error: #<NameError: undefined local variable or method `oops' for main:Object>
Filepath/Filename.rb:LineNo:in `errortest'
<main>:in `<main>'
SketchUp:1:in `eval'

Output of Console+:
errortest()
NameError: undefined local variable or method `oops' for main:Object

Just checked this behaviour in Sketchup 2017 vs. 2016:
In Sketchup 2017, part of the trace info is displayed on the righthand side of the console - at least filename and linenumber.
This is not the case for Sketchup 2016 - nothing displayed at all.

Thanks for reporting!
Since exceptions are propagated through internal methods of the console plugin, these would also show up in the backtrace. The console plugin drops these traces from the backtrace. Between SketchUp 2016 and 2017 the string has changed which signifies that a trace's source is the Ruby interpreter. Due to this, all traces were dropped.


Tip NFTs

In addition to tipping with ETH or wETH or ERC20 tokens, one could tip with NFTs.
Though, unlike fungible tokens, where the tipper only needs to have a sufficient total balance, this many NFTs would have to be minted or pre-minted.
The UI or a dedicated contract could iterate over the mint and transfer for each account, fees and all. Recovering from an error mid-way the list of accounts could be tricky.
A simplifying assumption could be that the NFTs are "discrete", like Kudos or POAP, issued individually but the same otherwise. Perhaps the instance of the ERC721 contract should belong to the tipper.
Within the EVM limit on the maximum number of arguments [16], one could create a method to extend ERC721, to mint a batch of a dozen tokens, with the same URI but to different accounts.

@wkarshat @META-DREAMER would you like to collaborate on a few prototypes and see what works.


Consume vscode-microprofile API

See redhat-developer/vscode-microprofile#58
Consume the API provided by vscode-microprofile in order to wait for lsp4mp to start when running commands that interact with lsp4mp.
Closes #323
Signed-off-by: David Thompson<EMAIL_ADDRESS>

@fbricon I implemented something close to the approach you suggested instead. vscode-microprofile would no longer depend on vscode-java, and would handle prompting to install it. vscode-microprofile exits its activate() method before lsp4mp is started, then has an API that lets you check once its started. vscode-quarkus still depends on vscode-microprofile, and uses the API to tell when it can use features of lsp4mp (detecting which projects are microprofile projects). This means a user can access the project generation wizard from rootless mode. However, vscode-java and jdt-ls still end up starting, since starting vscode-microprofile means starting lsp4mp, which means starting jdt.ls.


(Feature) Adding title with helmet

This PR closes #165022097, by:

Adding react-helmet and using this in stackNavigation

Screenshot:

@dminones can you solve the conflicts?

@dminones can you solve the conflicts?

@fernandomg done!


ACCUMULO-4409 Create AccumuloMonitorAppender

Creates an AccumuloMonitorAppender, and does additional logging system cleanup in the files and scripts.

I tested this PR using a Mac and everything worked well!


C#: Skip method name when checking CallError

When invoking call, the arguments contain the method name. This argument must be skipped; otherwise, the error.argument index will be off.
For example, you can see the bug in the error reported in https://github.com/godotengine/godot/issues/92793:

ERROR: Invalid type in function 'setup1' in base 'Godot.Node3D'. Cannot convert argument 1 from StringName to Array.

The error says that the first argument StringName can't be converted to Array. This is wrong because it's checking the method name argument in the Call invocation, but it should be checking the first vararg argument instead.
With this PR, the error now looks like this:

ERROR: Invalid type in function 'setup1' in base 'Godot.Node3D'. The array of argument 2 (Array) does not have the same element type as the expected typed array argument.

This should match what GDScript does:
https://github.com/godotengine/godot/blob/96a386f3c424af96d950ee5098b4b0e4907c9508/modules/gdscript/gdscript_vm.cpp#L1787-L1789

Thanks!


add excludepackages for key vault jca

update package to excludepackages which we don't need in azure-security-keyvault-jca

Docs Build status updates of commit 889248a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-mapping/reference-unified.yml
:white_check_mark:Succeeded

package.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit ef203c9:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-mapping/reference-unified.yml
:white_check_mark:Succeeded

package.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


DS-2466 results per pagein browse

https://jira.duraspace.org/browse/DS-2466

Just to be clear - this change affects the default number of items per page in browse pages, both XMLUI and JSPUI, both Discovery and Lucene, correct?

No @helix, this change affect only browse in XMLUI, I did tests earlier.
On 24 February 2015 at 05:37, helix84<EMAIL_ADDRESS>wrote:

Just to be clear - this change affects the default number of items per
page in browse pages, both XMLUI and JSPUI, both Discovery and Lucene,
correct?
—
Reply to this email directly or view it on GitHub
https://github.com/DSpace/DSpace/pull/872#issuecomment-75717829.

--
Rodrigo Prado de Jesus
http://about.me/royopa
<EMAIL_ADDRESS>

As I commented on the ticket, there is already a configuration for resultsperpage (defaultRpp), it just isn't implemented correctly. I think if we can revamp this PR to utilize the existing config, and not add a new config, I'd approve this PR.

Please confirm me.
Does the name of parameter in dspace.cfg is "defaultRpp"? Eg.
Example results per page
defaultRpp = 10
On Wed, May 20, 2015 at 12:26 PM, Hardy Pottinger<EMAIL_ADDRESS>wrote:

As I commented on the ticket, there is already a configuration for
resultsperpage (defaultRpp), it just isn't implemented correctly. I think
if we can revamp this PR to utilize the existing config, and not add a new
config, I'd approve this PR.
—
Reply to this email directly or view it on GitHub
https://github.com/DSpace/DSpace/pull/872#issuecomment-103925280.

--
Rodrigo Prado de Jesus
http://about.me/royopa
<EMAIL_ADDRESS>

https://github.com/DSpace/DSpace/blob/b34240e8d5f315d1a3cad77329c09ffdb34bc8b1/dspace/config/spring/api/discovery.xml#L146

Please help me
I must change ConfigurationManager.getIntProperty("webui.browse.results_per_page") to
Integer.parseInt(ObjectModelHelper.getRequest(objectModel).getParameter("defaultRpp")) ?


Wrong curl header when using Infinity with NodeGraph

Describe the bug
Infinity issues the following curl command when retrieving JSON data:

The first header entry is missing the key!
The service response is:

Steps to reproduce
In both queries I do not use specific Headers and Query params.
Sample Data
N/A
Screenshots

Version Details:

Grafana version : 9.1.7
Plugin version : 1.0.0

Additional context
Used service: https://postgrest.org/en/stable/

Sorry, I forgot to mention.
No headers defined in plugin configuration.

Hi,
are you planning to fix this issue in the near future?


Updates Druid versions for 24.7

Description

Adds 30.0.0, removes 27.0.0
Fixes run-druid script to include parameters needed for Java 11 and above
Switch to Java 17 for runtime
Switches the Dockerfile to heredoc format for easier reading
Installs "native" pyyaml instead of via pip
Adds package_manifest
Removes outdated comment from base image
Fixes typo in upload script
Prometheus Emitter is built from source as well while all other community extensions are skipped
Patch to update all dependencies that have a patch-level update
Remove log4shell workarounds. Druid 26.0.0 (and later) use Log4j 2.18 (or later) which is not vulnerable anymore
The dependencies needed for the XmlLayout are now included in the build itself so we don't need to download them from Nexus anymore and we can be sure that the dependencies match up

Definition of Done Checklist

Not all of these items are applicable to all PRs, the author should update this template to only leave the boxes in that are relevant
Please make sure all these things are done and tick the boxes

- [ ] Changes are OpenShift compatible
- [x] All added packages (via microdnf or otherwise) have a comment on why they are added
- [x] Things not downloaded from Red Hat repositories should be mirrored in the Stackable repository and downloaded from there
- [x] All packages should have (if available) signatures/hashes verified
- [x] Add an entry to the CHANGELOG.md file
- [ ] Integration tests ran successfully

TIP: Running integration tests with a new product image
The image can be built and uploaded to the kind cluster with the following commands:
bake --product <product> --image-version <stackable-image-version>
kind load docker-image <image-tagged-with-the-major-version> --name=<name-of-your-test-cluster>

See the output of bake to retrieve the image tag for <image-tagged-with-the-major-version>.

For anyone wanting to thes this.
I have pushed a version of the image as of commit https://github.com/stackabletech/docker-images/pull/731/commits/476c71c6435ea6e9efb93140343eb533fa9d5fc6 to

docker.stackable.tech/sandbox/druid:26.0.0-stackable0.0.0-dev
docker.stackable.tech/sandbox/druid:28.0.1-stackable0.0.0-dev
docker.stackable.tech/sandbox/druid:30.0.0-stackable0.0.0-dev

Sigi and I tested integration tests (I ran various tests and sigi tried the logging tests) and they were all successful


fix(hmr): Prevent retrieving properties in undefined object in jsHMR

Hello,
I met a strange error when working with HMR and react-hot-loader, here's the screenshot:

maybe related to https://github.com/webpack/webpack-dev-server/issues/2231.

I've searched almost the possibility across react-hot-loader, webpack-dev-middleware and webpack-hot-middleware, or even I used Koa in the wrong way. I excluded most of the customization, finally I guess this is an implicit bug when mixing webpack@^5 and react-hot-loader.
I skipped creating an issue but a pull request, hope that won't let you feel bad (ノω・。)
Minimal reproducible project with step-by-step:
Also more informations.
https://github.com/shirohana/webpack-react-hmr-undefined
Notable versions
"babel-loader": "^8.1.0"
"react-hot-loader": "^4.12.21"
"webpack": "^5.0.0-beta.17"
"webpack-dev-middleware": "^4.0.0-rc.1"
"webpack-hot-middleware": "^2.25.0"

What kind of change does this PR introduce?

A tiny quickfix to avoid some unexpected but common use cases.

Did you add tests for your changes?

No, no new test needed.

Does this PR introduce a breaking change?

No.

What needs to be documented once your changes are merged?

No need to be documented, just a tiny patch.

Hi there,
I'm sorry about this pull request... seems this problem should be solved in react-hot-loader not HMR of webpack.
After cross testing, the main problem is from react-hot-loader definitely, not sure is about module cache invalidating or require blocking chain.
Hope that I can do some meaningful contributes and join webpack community in some day, sorry for annoying and thank you for bringing amazing project (´・ω・｀ʃ♡ƪ)

Do you want to fix the new problem?
For creating a test case try to look into test/hotCases

@sokra Thank you for the advice, I'll read the guide and try to create some tests.

Hi there,
I finally got the most minimal case which can fire the error, and only the first commit applied, the test will passed.
I'm not sure about that is it I named the test case correctly btw (´・ω・｀)

Thanks


No code samples for azure-mgmt-datamigration Python SDK

I could not found any code samples for azure-mgmt-datamigration Python SDK module.
Links on azuresdkdocs.blob.core.windows.net and in GitHub repo redirect to generic pages that do not contain any information about Data Migration SDK whatsoever.

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

ID: 69c44dbc-5af2-cdd8-a5a1-fa7cc6269baa
Version Independent ID: 1101b29b-a21b-2ebb-c279-315d9796a43a
Content: Microsoft Azure SDK for Python
Content Source: docs-ref-services/latest/mgmt-datamigration-readme.md
Service: datamigration
Product: azure
Technology: azure
GitHub Login: @maggiepint
Microsoft Alias: magpint

Hi @myhau ,we will add later.
Before that, please refer to the restapi doc: https://docs.microsoft.com/en-us/rest/api/datamigration/services/createorupdate#examples


Easy Transitions

1.5 will provide a feature for easy, animated transitions between content.

@ButchersBoy great news. When it will be released?

Don't know, still under development, but hopefully weeks (at least for preview) not months.


Surprise me implementation

What is your feature?
Logic for implementing the surprise me button
Why is this feature needed?
To randomly generate a food choice
Who is your feature for?
Users or anyone who uses this app
Additional context
Should be done by Sunday

closed as part of pr#18


Error using a different dataset than action genome

In order to test in a custom video, you said only video frames (self.video_list in the class AG) are necessary (for the setting SGDET), but can I get the result if I only include the self.video_list information in the data_loader and change only the stran part to a pretrained model on test.py ? I wrote the code, but the error does not appear, but it does not go well because the result does not come out.
And I saw the answer, 'Sometimes person_bbox and object_bbox are used in the test because there are other two settings FredCLS/SGCLS who are wide used in the image scene graph generation.' I don't know what it means. If you don't mind, can you explain it further?

In order to test in a custom video, you said only video frames (self.video_list in the class AG) are necessary (for the setting SGDET), but can I get the result if I only include the self.video_list information in the data_loader and change only the stran part to a pretrained model on test.py ? I wrote the code, but the error does not appear, but it does not go well because the result does not come out. And I saw the answer, 'Sometimes person_bbox and object_bbox are used in the test because there are other two settings FredCLS/SGCLS who are wide used in the image scene graph generation.' I don't know what it means. If you don't mind, can you explain it further?

Hello, is your custom video a dataset labeled in AG format?


PDC Connect agent not allowing connection

From a test pod i was able to succesfully curl my end point with basic auth
curl https://mimir.thetmg.com -H 'Host: mimir.thetmg.com'
Returns the HTML Page for mimir as expected
My local Grafana instance that is in the Kubernetes cluster is also able to connect with mimir.thetmg.com, however my Grafana Cloud instance is unable to connect to mimir.thetmg.com with the PDC Agent.
Post "https://mimir.thetmg.com/prometheus/api/v1/query": dial tcp: lookup mimir.thetmg.com on <IP_ADDRESS>:53: no such host - There was an error returned querying the Prometheus API. is what i receive from grafana cloud

i did not see the pdc option when selecting data source


ENH: Added volume metrics in the basic metric group of tortuosity module

Volume of each tube is calculated as the sum of volumes of cylinders made by
consecutive points where radius is (r(t) + r(t+1))/2 and height is the
distance between two points.

:+1:
For reference, this commit is backported from KitwareMedical/VesselView@8095fedf289d0


⚠ Bofrost.de has degraded performance

In 4b0e8e9, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1194 ms

Resolved: Bofrost.de performance has improved in 9c3eed4 after 6 minutes.


Allow rc file to contain location of config file

Problem
When putting a databas.json file in a different directory, you need to call with the config file option each time.
This is some extra overhead we don't want to deal with.
closes https://github.com/db-migrate/node-db-migrate/issues/451
Solution
Allow the rc file to specify a configFile option. config is a protected word in the rc module which means we can't use that one.
The current solution is to simply disallow this.
Add some documentation updates
https://github.com/db-migrate/english-docs/pull/25
Help required
Not sure how to test the configuration is loading correctly. Does anyone know of a good test to replicate for my own purposes.
Based on https://github.com/db-migrate/node-db-migrate/commit/b5e7c80a833c4e8eb0ef24838d810d393015a2c9, this seems untested.
Disclaimer
I hereby release all copyrights to this work in perpetuity. (not sure if more is needed).
@wzrdtales Please

LGTM, will check soon if there are any side effects to this.

@wzrdtales Any chance on this getting merged in.. Also considering changing the name of the property to config-file to keep more consistent with the rest of the project.

As said in other issues, was busy with preparing for the conference last week. I'm back now and will incorporate your contribution!


Fix security warnings

Fixes remaining security warnings. But there will be more in like a week b/c javascript 🎉
Uses a fork I published under my own namespace on npm using dependency updates from @joseph184/rollup-plugin-node-builtins (not trusting forks from unknown folks).
Fixes #73.

Thanks so much for this. You’re a hero!


.travis: disable go modules when installing dependencies

Fixes: d1fecef5d46f ("vendor: add go-mod support")
Signed-off-by: André Martins<EMAIL_ADDRESS>

This change is

Coverage increased (+0.04%) to 45.598% when pulling 9fb88cb366fa7c31449fbded91f9524efd8effb7 on pr/avoid-go-mod-download-travis into 618f568f3d24066c216f709bc5a136399aafac26 on master.

test-me-please

test-me-please

test-me-please

test-me-please

test-me-please

test-me-please

test-me-please

test-me-please

test-me-please

test-me-please

test-me-please

test-me-please

test-me-please

test-me-please

test-me-please


client/X11: Use the correct wheel rotation value

For the negative scrolling direction, RDP uses the two's complement,
instead of the positive wheel value with the negative flag.
xfreerdp currently uses the positive wheel value in addition to the
negative flag, which results in a wrong wheel value on the server side
(136 instead of 120).
Fix this, by using the correct wheel rotation value, which is in the
two's complement.

(This should be backported to the stable-2.0 branch.)

@pnowack thank you, I´ve updated your commit for the other clients too, so closing this.


[SPARK-8695] [core] [WIP] TreeAggregation shouldn't be triggered for …

…5 partitions

Can one of the admins verify this patch?

Jenkins, this is ok to test.

Merged build triggered.

Merged build started.

Test build #36322 has started for   PR 7168 at commit a6fed07.

Test build #36322 has finished for   PR 7168 at commit a6fed07.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

cc @mengxr

From the abstract .. It seems you wanted to restrict it specifically for 5. Otherwise both
while (numPartitions > scale + math.ceil(1.0 * numPartitions / scale))
or
while (numPartitions > scale + numPartitions / scale + 1 )
would work.
I shall generate a new pull with your suggest way ASAP.


Refactor CalculateTierRequests method

See this todo for details, this method can be used in several resources to DRY-up our code

Fixed by https://github.com/infracost/infracost/pull/405


Fix AssetMapper link in README

Fix the AssetMapper documentation link (the url used asset-mapper instead of asset_mapper)

Thanks Simon!


[Documentation] Update SeedQR docs to note we assume the English bip39 wordlist is being used

SeedQR could trivially be used with any of the bip39 wordlists in other languages.
However there is no way to know which language was intended when interpreting a SeedQR. Interpreting a SeedQR while assuming the wrong intended language wordlist will result in a different or invalid seed.
The SeedQR docs should clarify that we are assuming -- and only recommend -- using the English wordlist.

Mental note:
I think of the above as "BIP39en" and wish that all future wallets would adopt it, seems like they're doing so.  It's the same as BIP39 with the restriction that when encoding entropy to mnemonic for hashing/stretching into 64byte seed, the wordlist is ALWAYS the well entrenched english list.
It would make for a better base-assumption, so that extensions could later be defined per locale, say "BIP39en+sw" for swahili (where swahili is decoded to entropy then encoded to english for hashing into seed).  Maybe down the road becoming optional locale-plugins -- upgrade-packs w/ fonts, l10n catalogs, wordlists.  This way, any user can have optimal ui and meaningful backups, without the bulk of unwanted non-english locales on tiny devices.


The chunks argument in xr.open_dataset() does not set the computePixels() request size.

It still seems to use preferred_chunks. Workaround: pass in the arguments to chunks_store instead.

In a discussion with @shoyer, it became apparent to me that it's valuable to have Dask chunks and IO chunks be separate concepts. Thus, this is a feature, not a bug.
Further, @mahrsee1997 recently added io_chunks to the entrypoint to try to make this distinction more explicit. More documentation to explain the whys behind all of this are needed. Closing the issue for now.


Link issues

Canvas.Left/Top/ZIndex are all 404.

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

ID: d504d09f-0104-e46f-6135-99b147603621
Version Independent ID: 8eee6397-da9a-839a-d25e-33ea5f60c07e
Content: Canvas Class (Windows.UI.Xaml.Controls) - Windows UWP applications
Content Source: https://cpubwin.visualstudio.com/DefaultCollection/windows-uwp/_git/winrt-api-build?path=%2fwinrt-api-build%2fwindows.ui.xaml.controls.canvas.yml&version=GBlive&_a=contents
Product: uwp
Git Login: jwmsft
Microsoft Alias: jimwalk

Duplicate of #736


Csv with semicolon-comma wrong parsing

Hi, I'm trying to read a CSV file like this (;, delimiters):
A;B;C;D
1,1;2,2;3,3;4,1
1,1;2,2;3,3;4,1
1,1;2,2;3,3;4,1

What happens: semicolon from the first line does not become the main separator, the comma from the second line does and it breaks the file's logic.
Code that reads:
X.read(e.target.result, { type: 'string' })

Result
{
      A1: { t: 's', v: 'A;B;C;D' },
      A2: { t: 'n', w: '1', v: 1 },
      B2: { t: 's', v: '1;2' },
      C2: { t: 's', v: '2;3' },
      D2: { t: 's', v: '3;4' },
      E2: { t: 'n', w: '1', v: 1 },
      A3: { t: 'n', w: '1', v: 1 },
      B3: { t: 's', v: '1;2' },
      C3: { t: 's', v: '2;3' },
      D3: { t: 's', v: '3;4' },
      E3: { t: 'n', w: '1', v: 1 },
      A4: { t: 'n', w: '1', v: 1 },
      B4: { t: 's', v: '1;2' },
      C4: { t: 's', v: '2;3' },
      D4: { t: 's', v: '3;4' },
      E4: { t: 'n', w: '1', v: 1 },
      '!ref': 'A1:E4'
    }

I have seen your comment about standard delimiters, but normally semicolon is a basic one (https://github.com/SheetJS/sheetjs/issues/1087#issuecomment-385708628)
Probably I am doing something wrong? Thanks

The overarching issue is locales (in some locales, commas are treated as decimal points and semicolons are the field separators).  Excel's general backdoor for the delimiter is the sep= pragma.
We'd accept a PR that adds an override.  The change would be in bits/40_harb.js:
		if(str.slice(0,4) == "sep=") {
			// If the line ends in \r\n
			if(str.charCodeAt(5) == 13 && str.charCodeAt(6) == 10 ) {
				sep = str.charAt(4); str = str.slice(7);
			}
			// If line ends in \r OR \n
			else if(str.charCodeAt(5) == 13 || str.charCodeAt(5) == 10 ) {
				sep = str.charAt(4); str = str.slice(6);
			}
			else sep = guess_sep(str.slice(0,1024));
		}
+		else if(opts && opts.FS) sep = opts.FS;
		else sep = guess_sep(str.slice(0,1024));

and the end code would be
X.read(e.target.result, { type: 'string', FS: ";" })

@SheetJSDev Can I take this issue?

The overarching issue is locales (in some locales, commas are treated as decimal points and semicolons are the field separators). Excel's general backdoor for the delimiter is the sep= pragma.
We'd accept a PR that adds an override. The change would be in bits/40_harb.js:
		if(str.slice(0,4) == "sep=") {
			// If the line ends in \r\n
			if(str.charCodeAt(5) == 13 && str.charCodeAt(6) == 10 ) {
				sep = str.charAt(4); str = str.slice(7);
			}
			// If line ends in \r OR \n
			else if(str.charCodeAt(5) == 13 || str.charCodeAt(5) == 10 ) {
				sep = str.charAt(4); str = str.slice(6);
			}
			else sep = guess_sep(str.slice(0,1024));
		}
+		else if(opts && opts.FS) sep = opts.FS;
		else sep = guess_sep(str.slice(0,1024));

and the end code would be
X.read(e.target.result, { type: 'string', FS: ";" })

We have done exactly this workaround on the code side (reading the first line and finding out the separator). I thought it could be done automatically.

The current behavior is consistent with Excel.
Consider the original sample file:
A;B;C;D
1,1;2,2;3,3;4,1
1,1;2,2;3,3;4,1
1,1;2,2;3,3;4,1

In the English (United States) regional setting, Excel will parse as if the field separator (Windows calls this "List separator") is the comma character:

If you are using a regional setting with ";" as the list separator and "," as the decimal symbol, then you will get the parse you expect.
In windows 7 these settings can be found in "Control Panel" > "Clock, Language, and Region" > "Region and Language" > "Additional Settings".  en_US has the following settings:

Note: Excel's backdoor for controlling the list separator is the aforementioned sep= pragma.  If you do specify it, Excel will use the separator but it won't change the decimal separator:

#malek``

[b]https://icq.im/AoLGnKtUsQZKGmUTjTA
https://t.me/chinadebitbank
This is arqidarknet from
torre,zorldcard, market pro,
I sell first hand pos data,atm datas,resellers welcome,
usable pos,atm
bank logs,email access,password
RDP
hack bibbles
Cvv=address
eu-usa-asia
minimum order is  10pieces
no test for new clients* consideration could be consider on basis.
[/b]
**

**

ef21e2bea29236bb983c747d7c087b886aa7fef3

Hello. What if I have a comma-separated .csv file in which some "cells" containing semi-colon separated values? Example:
A,B
1,a;b;c

Parsing with XLSX.read(e.target.result, {type:'array', FS: ','}) isn't working as expected.


fix: field time in Header and TransactionResponse

Hello, I found out a bug about the time field in client.query results.
the time value of the following code is always None
1. transaction response
let tx_id = "0x7e46c2447d3788be0a41550edaebed99fe6b28c60d1c01186fc0d70d72c38a68";
let provider = Provider::connect("node-beta-2.fuel.network").await.unwrap();
let tx = provider.get_transaction_by_id(tx_id).await.unwrap();
println!("{:#?}", tx);

2. block header
let block_id = "0xba4f5c3197f72f752b82cc46c170b550f5ac6e320656683b2693af0b748d5dd4";
let provider = Provider::connect("node-beta-2.fuel.network").await.unwrap();
let block = provider.block(block_id).await.unwrap();
println!("{:#?}", block);

Fix
use to_unix() to convert TAI64 to timestamp

time parameter of produce_blocks should be TAI64  format
https://github.com/FuelLabs/fuels-rs/blob/fce0631cf8778a04d70cf8fdffa0d207b211c4b9/packages/fuels-signers/src/provider.rs#L414
I noticed that there are some changes in fuel-core structure of project, so adding 'tai64' dependence is a kind of temporary solution and I will follow up on the changes.

@cheng404 Do you mind syncing this branch with master?

Done. :)

nice, thanks for your contribution! @cheng404 I'm letting you the pleasure of clicking the green "Merge" button 😄
It's my pleasure. I have no write access to this repository, and can't merge and close PR

It's my pleasure. I have no write access to this repository, and can't merge and close PR
Woops I didn't know that haha. Merged! :tada:


Cant donate any money, would like to pay for your beer!

;-)

Thank you so much! ❤  Consider to make PR for this project or say hi on Twitter(@SwiftyJSON) :)


site is a reserved keyword in python and should possibly become a new module name

# TODO: site is a reserved keyword in python and should possibly become a new module name

https://github.com/kjaymiller/render_engine/blob/f8ad52c2b76c45a9e74a7397c335d0df9466fb4f/src/render_engine/__init__.py#L5

site is but Site is not.


Create dependabot.yml

Resolves #107

Just need one person to review this PR as I can't self review. @jordanpadams @tloubrieu-jpl @nutjob4life


python310Packages.pex: 2.1.65 -> 2.1.66

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.pex is: "Python library and tool for generating .pex (Python EXecutable) files"
meta.homepage for python310Packages.pex is: "https://github.com/pantsbuild/pex"
meta.changelog for python310Packages.pex is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
Warning: a test defined in passthru.tests did not pass
Warning: no invocation of /nix/store/i296f1zjzckxicm3rqbpi9l3n9jjapmk-python3.10-pex-2.1.66/bin/.pex-tools-wrapped had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/i296f1zjzckxicm3rqbpi9l3n9jjapmk-python3.10-pex-2.1.66/bin/.pex-wrapped had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/i296f1zjzckxicm3rqbpi9l3n9jjapmk-python3.10-pex-2.1.66/bin/.pex3-wrapped had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/i296f1zjzckxicm3rqbpi9l3n9jjapmk-python3.10-pex-2.1.66/bin/pex-tools had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/i296f1zjzckxicm3rqbpi9l3n9jjapmk-python3.10-pex-2.1.66/bin/pex had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/i296f1zjzckxicm3rqbpi9l3n9jjapmk-python3.10-pex-2.1.66/bin/pex3 had a zero exit code or showed the expected version
0 of 6 passed binary check by having a zero exit code.
0 of 6 passed binary check by having the new version present in output.
found 2.1.66 with grep in /nix/store/i296f1zjzckxicm3rqbpi9l3n9jjapmk-python3.10-pex-2.1.66
found 2.1.66 in filename of file in /nix/store/i296f1zjzckxicm3rqbpi9l3n9jjapmk-python3.10-pex-2.1.66

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

4 total rebuild path(s)

2 package rebuild(s)

2 x86_64-linux rebuild(s)
2 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
python310Packages.pex
python39Packages.pex

Instructions to test this update (click to expand)

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

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python310Packages.pex https://github.com/r-ryantm/nixpkgs/archive/e083b59477dabd827604b9e26301b4e17082ef45.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/i296f1zjzckxicm3rqbpi9l3n9jjapmk-python3.10-pex-2.1.66
ls -la /nix/store/i296f1zjzckxicm3rqbpi9l3n9jjapmk-python3.10-pex-2.1.66/bin

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

2 packages built:

python310Packages.pex
python39Packages.pex

Maintainer pings
cc @copumpkin for testing.

@GrahamcOfBorg eval


[200~https://principal.url.edu.gt/~

Adding Universidad Rafael Landívar email domain.

@DanyAlvarado file/lib/domains/gt/edu/url/correo.txt: this request has an incorrect file path format, please ensure that you are familiar with     this repository rules located at the bottom of this page and change     your request accordingly. If you don't know how to change request, close it and open a new one.     If you are not able to make the pull request according the instructions, please send us an email to<EMAIL_ADDRESS>Please mention this request URL in the message.


Running tests remotely eg Vagrant VM or SSH

I am trying to work the best way to run tests remotely, particularly in the case of running them inside in a Vagrant VM. Is there an existing plug-in that this plug-in could leverage off?

Hmm, I haven't executed tests in this way so far, so I don't know of any plugins, but it would be great to leverage one to do the hard work.

The vado plug-in might be a good candidate

@codeinabox commented on Aug 18, 2015, 8:27 AM EDT:

The vado plug-in might be a good candidate, lets you remotely run commands over SSH which is the way PHPStorm does it too.

That plugin has a hard dependency on using sshfs (which is a bit funky on Macs, iirc). If Vagrant's the target, why not have a means of prefixing commands with vagrant ssh -- or the likes?

I think there aren't any other Vim vagrant plugins, so I think it would be fine to make a strategy which uses vagrant ssh directly. What do you have to do besides vagrant ssh --command "..."? Do you have to specify the name of your Vagrant "container" (I forgot how they're called), or does Vagrant figure that out by itself using the Vagrantfile?

Usually when you run vagrant ssh you have to change the working directory to where the code is mounted before running the tests, so potentially a command like this
vagrant ssh --command "cd /home/vagrant/project; ./vendor/bin/phpunit

Here, I pushed my implementation of this strategy, let me know how it works for you.

I've removed the Vagrant strategy (reasons explained in the commit), but I've introduced a new concept of "transformations" which allows you to add it manually (the example in the README sets up Vagrant).

Yeap saw that, will have a try. Should be handy when you developing locally but need to run the tests via Docker, Vagrant, ssh

Would it be possible to define transformations on just a buffer level?

Hmm, I didn't consider it, what would be the use case? Maybe it would make sense to allow different transformations for different runners. Probably would be useful to add that feature to strategies, too.

I am trying to set up transformations to run only when certain environment variables are set eg VAGRANT_DIR but I guess because I different Vim instance per project global would be fine right?

Hmm, I don't know if it works for you, but in your .vimrc you could do conditionals based on environment variables, so set different tranformations depending on the variable (unless the variable changes dynamically while you're running the Vim session).


use python plot.py --dataset yfcc-10M --neurips23track filter instead of just python plot.py --dataset yfcc-10M

Otherwise, an error will be generated
> python3 plot.py --dataset yfcc-10M

writing output to results/yfcc-10M.png
Traceback (most recent call last):
  File "/home/nop/projects/nips23/big-ann-benchmarks/plot.py", line 161, in <module>
    raise Exception('Nothing to plot')
Exception: Nothing to plot

After adding --neurips23track filter everything works as expected
> python3 plot.py --dataset yfcc-10M --neurips23track filter
writing output to results/yfcc-10M.png
Computing knn metrics
  0:                  Faiss(('IVF16384,SQ8', {'nprobe': 32, 'mt_threshold': 0.0001}))        0.847     3069.679
Computing knn metrics
  1:                  Faiss(('IVF16384,SQ8', {'nprobe': 64, 'mt_threshold': 0.0001}))        0.901     2421.102
...

The correct command line was somewhat unclear from the description on the main page.

Thanks @alexanderguzhva.


Can't Close Select2 on select an option

Hey there ,
I'm having a trouble in closing the select2 appended list  after i choose an option , this issue exists only in the lat version of select2 , but the earlier versions don't ,

I'm having the same problem on version 4.0.6 (version 4.0.5 works). Somehow the dropdown search is adding data-select2-id to several DOM elements, outside the target select2 object.
The bug is being reported in stackoverflow: https://stackoverflow.com/questions/55949585/jquery-select2-not-closing-anywhere-also-not-on-their-own-website
And as this user states, it happens in the official doc page too: https://select2.org/getting-started/basic-usage.

This is fixed in 4.0.7-rc.0

I reported it in the forum, too. Glad it's not just me.
https://forums.select2.org/t/demos-selecting-option-does-not-dismiss-list-of-options/730

FYI, as a temporary fix: reverting to an older version (4.0.3) seems to work (tested only in Chrome)

This problem is occurring in the version 4.0.6 that is marked as latest release. There should be another stable version with the fix otherwise people may update and have problems on there pages.

This is a duplicate of https://github.com/select2/select2/issues/5490 and fixed in Select2 4.0.7-rc.0.


Use maximum permitted page size

Pending GraphQL, performance is going to be bad, but do what we can.
@reviewbybees

This pull request originates from a CloudBees employee. At CloudBees, we require that all pull requests be reviewed by other CloudBees employees before we seek to have the change accepted. If you want to learn more about our process please see this explanation.

🐝  with ant.

@reviewbybees done

@recena also suggests making 100 (the maximum) be the default size to begin with. I am not sure if the lower default is intentional for some reason.

Use fibonacci?

:bee:

This pull request has completed our internal processes and we now respectfully request the maintainers of this repository to consider our proposal contained within this pull request for merging.


Add API endpoint and CLI command for orquesta workflow inspection

Add /v1/workflows/inspect endpoint in the API and a POST method to inspect workflow definition and return errors found. Add a st2 workflow inspect command that uses the API endpoint for workflow inspection.

Is this a validation or something different?

@nmaludy yes, this is for validation


Unable to verify publisher via marketplace

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

VS Code Version: -
OS Version: -

I followed the procedures outlined in the publisher's verification documentation (https://code.visualstudio.com/api/working-with-extensions/publishing-extension#verify-a-publisher), but marketplace is unable to verify my domain's DNS.
Note: It has been more than 72 hours since the addition of the TXT record in DNS.

@AlencarGabriel what is your publisher name?

My publisher name is AlencarGabriel (https://marketplace.visualstudio.com/publishers/AlencarGabriel)

@AlencarGabriel I see that your TXT record has the wrong Key.
The Key is supposed to be- _visual-studio-marketplace-alencargabriel.gabrielalencar.dev and you have stored it as- _visual-studio-marketplace-alencargabriel.gabrielalencar.dev.gabrielalencar.dev
There is an extra .gabrielalencar.dev in the record's Key.

How strange, where did you get this information?
In my DNS zone the host registered is exactly as expected: _visual-studio-marketplace-alencargabriel.gabrielalencar.dev

@AlencarGabriel Here you can verify that it is set for the wrong domain - https://www.whatsmydns.net/#TXT/_visual-studio-marketplace-alencargabriel.gabrielalencar.dev.gabrielalencar.dev
DNS providers often add domain names to the record, so you need to edit it and it should be visible here - https://www.whatsmydns.net/#TXT/_visual-studio-marketplace-alencargabriel.gabrielalencar.dev

I managed to perform the domain verification.
I don't know if this behavior is just Godaddy's behavior, but it looks like they themselves add the .gabrielalencar.dev to the end of the host, causing duplication.
I changed the TXT record in DNS to _visual-studio-marketplace-alencargabriel and it worked.
Thanks for the tip @antonbabenko and @rheajain for support.

It looks like this is actually an issue with the instructions in VS Code.  They should probably give you different instructions based on domain registrar.  The instructions you can copy and paste, as well as the instructions in the modal that pops up had the same instructions that included the domain suffix at the end.  Removing that from my domain verification hostname solved my problem of not being able to validate even though the DNS was correct ( according to instructions ).

this happens with namecheap too. it's especially annoying that it tells you to wait 72 hours when in my experience dns record updates don't usually take longer than 30 minutes max. after i waited a few days i realized the hostname was being duplicated and fixed it, it worked instantly.
perhaps it could detect this mistake and show a more helpful error message?


One Swc depended on another Swc Causes Index.html Blank

I am compiling my own library DBIZLib.swc
Console : https://paste.apache.org/h4c5d
what should i do to remove these warnings?
because if i ignore these warnings and compile any project index.html is blank
Errors:
Uncaught ReferenceError: applications_mx_managers_SystemManager is not defined at index.html:962
Uncaught Error: Bad dependency path or symbol: org.apache.royale.effects.IEffect
at visit (base.js:2796)
at visit (base.js:2808)
at visit (base.js:2808)
at goog.DebugLoader_.load_ (base.js:2815)
at Object.goog.require (base.js:846)
at index.html:957
https://user-images.githubusercontent.com/42200979/61864583-1ab30c00-aeeb-11e9-9a8f-690798fec1cd.png

It looks like you're missing playerglobal.swc as an external dependency.
It would help to see what build configuration you are using.

Recently Josh made changes to how libraries are linked. I'm not using ANT, but I think ANT solves this with config.xml files in projects. Basically instead of listing all in library-path node, you must use "external-library-path" node in libraries to ensure all the code in the libraries linked are not compiled into that library. Then in Application, you must continue using library-path, since you want all code used be compiled in the application. In this part your application maybe is missing playerglobal or js swcs as Harbs comment.


Required version of Microsoft.NET.Sdk.Functions?

Which version of Microsoft.NET.Sdk.Functions does this require? E.g. does it work with 1.0.29 as well, or it needs 1.0.30 (currently still in beta)?

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

ID: c8bd5164-c40a-241a-d0ea-ccce7659f9ab
Version Independent ID: f6a565b3-c03b-12bd-4bf4-a4ca19645811
Content: Azure Functions warmup trigger
Content Source: articles/azure-functions/functions-bindings-warmup.md
Service: azure-functions
GitHub Login: @alexkarcher-msft
Microsoft Alias: alkarche

@deyanp Thanks for your question. We are checking on this and will respond to you soon.

@deyanp The Microsoft.NET.Sdk.Functions is the build task for C# functions. Only the version of the Microsoft.Azure.WebJobs.Extensions nuget has to be >=3.0.5 to use WarmupTrigger.

@deyanp Just following up here... Hope my previous comment clears things up.

@deyanp Hope my previous comment clears things up.
Since we have not heard back from you 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.

@PramodValavala-MSFT, thanks, it is clear now that Microsoft.NET.Sdk.Functions is unrelated to the warmup trigger!


IP leak on google services

when used as client, google detect my real IP and get 403 error.

I have faced the same issue. Google bans me from accessing some services (Android, Kaggle, ...).
This is a fix around:
I heard (not tested) that changing the server IP and using outline will solve the problem. However, test for leaks from your browser using some sites such as browserleaks or dnsleaktest.
Thanks @Nyr for the great script. I appreciate if it gets fixed with openvpn-install

There is anything to fix within the installer about this.


update lazy config in readme

I love this idea for a plugin, I'm very frequently opening up github to copy a specific line to share. This PR updates the readme to use the repo short url instead of a local dir.

Hahahaha!  Good catch I can't believe I missed that.   Much appreciated!


Camera meta information

@mhkabir This is just the architectural foundation for you to complete. This changes the whole approach to geo-tagging: Instead of doing this using the log file we resort to storing the trigger events in the GCS (@bkueng, @DonLakeFlyer your help might be needed) during the flight. The Geotagging then should just use that hash map to tag the images.
@julianoes Can you help with the quaternion definition of the camera message? What is 0 roll, 0 pitch, 0 yaw? Facing level to north or facing down to north?
MAVLink message: http://mavlink.org/messages/common#CAMERA_IMAGE_CAPTURED

@mhkabir I've added http://mavlink.org/messages/common#MAV_CMD_REQUEST_CAMERA_IMAGE_CAPTURE for the re-request command. Please add to vehicle_command.msg with the same ID / meta info.

Related QGC issue: https://github.com/mavlink/qgroundcontrol/issues/4230
@DonLakeFlyer Note that this is gating the v1.6 release, so reasonably high prio if you can fit it in somewhere.

@dagar Would be fantastic if you could review and potentially help on the QGC side.

Complete communication spec updated here:
https://github.com/mavlink/qgroundcontrol/issues/4230#issuecomment-298319932
In particular the bit on http://mavlink.org/messages/common#FLIGHT_INFORMATION

@LorenzMeier I don't know anything about the Quaternion definition. It should just be the same as the vehicle.
The Euler angles are in the same frame as we describe the vehicle, facing level to North.

@julianoes OK, got it.

Superseded by https://github.com/PX4/Firmware/pull/7201


Add LICENSE.txt

done https://github.com/F-Secure/pytest-rts/commit/ebfdd6624606201976df03463248106cfdb5b964


Add an ability to switch between chart and table view

added an ability to switch between chart and data preview in published & preview charts

I see this

on https://visualize-ad-feat-add-r-aoyb9f.herokuapp.com/en/v/3p3klQ06RoMF
It seems like the design has some problems here ?
Also, to smooth out going back and forth maybe a <motion.div layout> could be used here ? so that the height is properly animated ?

Thanks @ptbrowne, I was focusing on the preview chart and I missed that 🫢 I also added Motion layout as suggested, I think it only makes sense in published chart, as for preview everything gets animated, including transitions between chart editing steps (and this was looking kinda distracting).

@bprusinowski i really like the animination when switching between table and chart view. While testing i notice it changes the width of the component https://test.visualize.admin.ch/en/v/4linf1VNEHqx could this eventually be an issue for someone trying to embedd the chart? Or will it be possibile to not offer this option when creating the chart, similar to for example the interactive filters?

@bprusinowski As discussed, please add a label next to the table icon with: Switch to table view or Switch to chart view

Referring to the comment of @sosiology . I've noticed that I see a scrollbar between the table but I'm not able to scroll

Minor detail: But I would change the order of the elements to:
Download, Switch to table view and then the SPARQL query

@bprusinowski I see that this issue is in the "Done" Lane but the lastest feedback hasn't been implemented. See above:

Adding a label next to the Switch Icon
Change the order of the elements
Scrollbar in the tables

Hi @AnninaWalker, I think it changed automatically when the PR was merged – but you're right, this shouldn't be in the done lane, I started to introduce modifications, but didn't finish yet. Same for the second PR, I will adjust the labels!

Perfect, thank you!

Something I just noticed. The fontstyle for the Download Button, Run SPARQL should be "caption" and not Body2. It should be 12px instead of 14px.

Thanks @sosiology and @AnninaWalker for your inputs! I addressed them in #475.


feat: use container name as archive log key

Fixes #13590
Motivation
if tmpl type is Container and container.Name not empty,  use container name as artifact log file name
Verification
e2e test

https://github.com/argoproj/argo-workflows/blob/4742e9dd6ab2b797d32cb0953849fdcfe82ea325/ui/src/shared/services/workflows-service.ts#L265
Remove the restriction of this main container? And fix this problem at the same time. #13785

done


Remove duplicate endpoints

These already exist under feeds section.
Remove from the jobs section

they're not duplicates, just tagged for both job and feed... I put them here for better visibility


[3.1] Sema: "super" calls to non-ObjC extension methods must be statically dispatched.

Scope: Affects users trying to super.-invoke methods from the SDK overlays in their custom subclasses of framework classes.
Issue: rdar://problem/30030229
Risk: Low
Testing: Swift CI, test case from Radar

@swift-ci Please test


Feature/document name as parameter

will be solved in feature/parameter-documentation-title-prefix


build failure, can't find a function in snappy?

The build seems to be failing because it can't find functions in snappy:
CXX      db/db_bench.o
CXXLD    db_bench
/usr/bin/ld: db/db_bench.o: undefined reference to symbol '_ZN6snappy11RawCompressEPKcmPcPm'
/usr/local/lib/libsnappy.so.1: error adding symbols: DSO missing from command line
collect2: error: ld returned 1 exit status
make[1]: *** [db_bench] Error 1
make: *** [all] Error 2

... but snappy is installed, and is the most-recent version from git:
checking for snappy_compress in -lsnappy... yes
checking snappy.h usability... yes
checking snappy.h presence... yes
checking for snappy.h... yes

The same problem happened when I used a recent snappy ubuntu package.
What am I missing?

maybe #15 solves your problem


Duplicate Buoyancy Code and computing buoyancy twice

It seems that buoyancy is calculated in two different places.
One method is located at /usv_gazebo_plugins/src/usv_gazebo_dynamics_plugin.cc#L291-L358.
I believe this is run at every timestep as the USVDyanmicsPlugin is loaded in automatically (i think).
The other method is from /usv_gazebo_plugins/src/buoyancy_gazebo_plugin.cc.
This method seems to be optional and will be loaded in if the sdf file contains a block for the BuoyancyPlugin like shown in /vrx_gazebo/models/blue_projectile/model.sdf
Since it seems like USVDyanmicsPlugin cannot be unloaded, does that mean that if I use the BuoyancyPlugin, buoyancy forces will be computed twice?
Should the BuoyancyPlugin be consolidated into the method in USVDyanmicsPlugin?

@acxz You are correct that buoyancy is calculated in two different locations, but this should not cause any duplicate calculation. Currently the usv_gazebo_dynamics_plugin calculates buoyancy for the USV only. The buoyancy_gazebo_plugin calculates the buoyancy for all other objects.
We may consider reorganizing so all buoyancy forces are calculated in the same place, but the main reason to do this is to improve readability of the code. If we do that, it's more likely that we would move the USV buoyancy calculation into the BouyancyPlugin than the other way around, since not all the objects that require this plugin are USVs.

move the USV buoyancy calculation into the BouyancyPlugin

This would be much appreciated to maintain readability.

Ok, sounds good. Since the code is not duplicated or computed twice I will close this issue. Feel free to open another if you want to suggest an addition to the documentation.

I was hoping this issue could be kept open until the BuoyancyPlugin is consolidated.


Version inconsistency

The versions of this action are inconsistent.
Problem
I found a v0.1.0 tag that links to a commit more recent than the v1.0.0 branch.
Solution
I recommend to keep working on your main branch (develop in your case) and tag new versions along.
You could enforce conventional commit messages and then use tools to automate new releases and changelogs and make your life easier.

Thank you for your interest in our project!
The v1.0.0 branch is a work-in-progress branch (that needs a rebase to update it against develop). We do not maintain versioned branches, so when that branch is merged into develop a new v1.0.0 tag will be created and the branch will be deleted. I'm sorry if that was unclear.
We do not currently have any plans to adopt something like Conventional Commits. I personally do not think that they align very well with our preferred development style as we encourage committing early and often as well as explicitly not performing squash merges, Regardless of my personal opinion it would be a team decision to implement such a thing into our development standards.


Removed pure_count and added property methods to _SinonStubCondition

Fixes note35/sinon#40

Coverage remained the same at ?% when pulling 76807b8705f67d53242c37ebdb6d848bbc8ed459 on jonathan-benn:addPropertyMethods into 8036d3f6d5cd89b993c217da5e420a2552b9222c on note35:master.

Coverage remained the same at ?% when pulling 76807b8705f67d53242c37ebdb6d848bbc8ed459 on jonathan-benn:addPropertyMethods into 8036d3f6d5cd89b993c217da5e420a2552b9222c on note35:master.

Coverage remained the same at ?% when pulling 76807b8705f67d53242c37ebdb6d848bbc8ed459 on jonathan-benn:addPropertyMethods into 8036d3f6d5cd89b993c217da5e420a2552b9222c on note35:master.

Coverage remained the same at ?% when pulling 76807b8705f67d53242c37ebdb6d848bbc8ed459 on jonathan-benn:addPropertyMethods into 8036d3f6d5cd89b993c217da5e420a2552b9222c on note35:master.

Coverage remained the same at ?% when pulling 76807b8705f67d53242c37ebdb6d848bbc8ed459 on jonathan-benn:addPropertyMethods into 8036d3f6d5cd89b993c217da5e420a2552b9222c on note35:master.

For reducing the test code size, I believe call function one time is enough.
Besides, I recommend to check:

declare 2 pure spy/stub, call them separately, and test their call count.

@sinontest
def test550_callCount_pure_chained(self):
    stub = SinonStub(os, "system").onSecondCall().returns(42)
    self.assertEqual(pure_stub2.callCount, 0)
    os.system("cd")
    self.assertEqual(pure_stub2.callCount, 1)

test call count before call.

@sinontest
def test550_callCount_pure_chained(self):
    pure_stub1 = SinonStub().onSecondCall().returns(42)
    pure_stub2 = SinonStub().onSecondCall().returns(42)
    pure_stub1()
    pure_stub2()
    pure_stub2()
    self.assertEqual(pure_stub1.callCount, 1)
    self.assertEqual(pure_stub2.callCount, 2)

Hi Kir,
It's true that there was a lot of overlap in my newly created tests. I've tried to follow your suggestion and combine everything into one test.
--Jonathan

Hi, merging into one test is good and separating into 3 (class/function/pure) could also be a good idea.

I have split one test into three tests


Bower Pulling Old Version

I just installed the plugin via bower. It is pulling version 1.4.0. Any idea why?

1.4.0 is the latest released version, which version did you expect to be pulled by bower?

Some of files says 1.5.0 or it isn't stable yet?
https://github.com/eternicode/bootstrap-datepicker/blob/master/dist/js/bootstrap-datepicker.min.js
https://github.com/eternicode/bootstrap-datepicker/blob/master/bower.json

no 1.5.0 is the current development version, 1.4.0 is the latest released version
This is what the bower info bootstrap-datepickercommand gives as output
Available versions:
  - 1.4.0
  - 1.3.1
  - 1.3.0
  - 1.3.0-rc.6
  - 1.3.0-rc.5
  - 1.3.0-rc.4
  - 1.3.0-rc.3
  - 1.3.0-rc.2
  - 1.3.0-rc.1
  - 1.2.0
  - 1.2.0-rc.1
  - 1.1.3
  - 1.1.2
  - 1.1.1
  - 1.1.0
  - 1.0.2
  - 1.0.2-rc.2
  - 1.0.2-rc.1.1
  - 1.0.2-rc.1
  - 1.0.1
  - 1.0.0

So you should require 1.4.0 in your bower config

thank you. I thought something was wrong..


Python extension extra data list full extension identifier

Testing microsoft/vscode#208468

from QP select "issue python"
show additional extension info
it includes the full identifiers of all installed extensions (that information is something VS Code itself shortens)

@jrieken @luabud revisiting this after I thought about it for a little bit:
the reason @paulacamargo25 adds the extensions here is because when filing on an extension, the reporter doesn't have a checkbox/option for sending enabled extensions, only when the source is vs code or don't know will enabled extensions be sent.
this could be up to further discussion though (if you're filing on an extension, to that extension's github, do they need to have a default check box for including enabled extensions?)

@justschen For Python extension, and our suite of extensions this information is often useful. There are cases where some combination of 3rd party extensions with python extensions has issues. This is useful to catch those quickly.

if you're filing on an extension, to that extension's github, do they need to have a default check box for including enabled extensions?

Yes. See my previous comment.

Re-open, this isn't about listing extensions but about listing them with the fully qualified identifier. Python should use the same "obfuscation" that vscode uses

@jrieken Ah! I see thanks for clarifying.


How to add CMAKE options of raylib to control features?

Say if I want to enable SUPPORT_BUSY_WAIT_LOOP, where should I put it?
I read the HACK manual and tried following changes
allowIfs = [
  "SUPPORT_BUSY_WAIT_LOOP",
  "RAYGUI_SUPPORT_ICONS"
]

And ran
nimble convert

But it seems the flags doesn't work. Am i doing something wrong?

These are flags that must be passed to C compiler, in Nim this can be done via --passC option, docs https://nim-lang.org/docs/nimc.html
RAYGUI_SUPPORT_ICONS is also defined to provide additional functions and nimraylib_now library already accounts for it and adds necessary bindings.
SUPPORT_BUSY_WAIT_LOOP this only defines additional runtime behavior and does not provide any additional functions, so nimraylib_now library doesn't need to know about it.
I suppose that everything should work fine if you compile your nim program like so:
nim c --passC:"-DSUPPORT_BUSY_WAIT_LOOP -DRAYGUI_SUPPORT_ICONS" --gc:orc file.nim

@greenfork Thanks very much! It works.


Question about how the contract works.

Hey friend. I have a P2P wire protocol question.
I have been building my own implementation. I have been simulating responses from other P2P programs and I can’t get passed the handshake. I get a handshake:
bufferSize!:  68
Protocol type:  BitTorrent protocol
infoHash:       aff8bda8adebdab34e1af9ac648bd93c13ed6344
peerId:         2d4c54313030302d764874743153546a4d583043
Then I try sending a handshake back or a interested or any other code...
Do you have any advice?
Thanks,
Connor.
(Also, is there a better place to contact you?

Without more information, it's not really possible to guess what's wrong with your program. The best advice I have for you is to run through the test suite for this package and set DEBUG=* so you get the full debug output including every message that is sent. If you make sure your program is sending the same messages in response to a remote peer, then it should work.

I already fixed it :P Sorry, I realize now that most torrents will kill the connection after a handshake when they have 0 downloaded data unless the first message you send is a bitfield or pex/dht. I never knew debug logs incoming and outgoing messages too! Thank you!


[Transforms] Wrong emit for "for..of" loop for Map instance

Hi gyus,
I could not find duplicate so I am posting here.
TypeScript Version: 1.8.9
Target: es5
Code:
var myMap = new Map();
myMap.set(0, "zero");
myMap.set(1, "one");
for (var [key, value] of myMap) {
  console.log(key + " = " + value);
}

Actual emit:
var myMap = new Map();
myMap.set(0, "zero");
myMap.set(1, "one");
for (var _i = 0, myMap_1 = myMap; _i < myMap_1.length; _i++) {
    var _a = myMap_1[_i], key = _a[0], value = _a[1];
    console.log(key + " = " + value);
}

The problem is that Map(and Set too) instance does not have length property, instead it has size property. As result loop does not traverse
Playground

ES5 does not have Maps or Sets or iterators.  How are you polyfilling these?  The Playground example complains Map is not found.
When targeting ES5, TypeScript will on allow a down emit of Arrays or Strings using the for...of construct.
When targeting ES5 with polyfilled iterators, you would have to use some functional replacement that replicates the for...of construct that deals with the polyfilled iterator interface.
TypeScript will still emit on error by default.  If you aren't getting errors at design time, then your polyfill is incorrectly extending Array.

Currently I am targeting browsers that provides native support for Map (IE11+)
I have workaround, so it is not critical for me. I am just curious to use for..of loop.

The partial implementation in IE11 does not support the iterator interface.  It isn't part of ES5 nor part of ES6.  The is no reliable way of TypeScript to down emit that when targeting ES5.


Streaming site vier.be moved to goplay.be.

Checklist

[X] I'm reporting a broken site
[X] I've verified that I'm running yt-dlp version 2021.10.22. (update instructions)
[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. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[ ] I've read about sharing account credentials and I'm willing to share it if required

Region
Belgium (have not checked for the rest of Europe)
Description
The streaming site for the Belgian tv channels Vier and Vijf has been moved from vier.be (and vijf.be, tv.be) to goplay.be. The new site is not supported.
This has previously been discussed  in the youtube-dl git page.
I'm able to extract the episode when looking for the m3u8 url, the following command runs fine (in contrast with the command included in the log):
yt-dlp -Uv https://stream2-vod.cdn1.sbs.prd.telenet-ops.be/non-geo/BAKE_OFF_VLAANDEREN/S5/volledigeafleveringen/1014745044154552701074761770676694101474/BAKE_OFF_VLAANDEREN_5_1_F0288762/BAKE_OFF_VLAANDEREN_5_1_F0288762.m3u8 -u PRIVATE -p PRIVATE
Verbose log
[debug] Command-line config: ['-Uv', 'https://www.goplay.be/video/bake-off-vlaanderen/bake-off-vlaanderen-s5/bake-off-vlaanderen-s5-aflevering-1#autoplay', '-u', 'PRIVATE', '-p', 'PRIVATE']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, err UTF-8, pref UTF-8[debug] yt-dlp version 2021.10.22 (zip)
[debug] Python version 3.6.9 (CPython 64bit) - Linux-4.15.0-156-generic-x86_64-with-Ubuntu-18.04-bionic
[debug] exe versions: ffmpeg 3.4.8, ffprobe 3.4.8
[debug] Optional libraries: mutagen, sqlite
[debug] Proxy map: {}
yt-dlp is up to date (2021.10.22)
[debug] [generic] Extracting URL: https://www.goplay.be/video/bake-off-vlaanderen/bake-off-vlaanderen-s5/bake-off-vlaanderen-s5-aflevering-1#autoplay
[generic] bake-off-vlaanderen-s5-aflevering-1#autoplay: Requesting header
WARNING: [generic] Falling back on generic information extractor.
[generic] bake-off-vlaanderen-s5-aflevering-1#autoplay: Downloading webpage
[generic] bake-off-vlaanderen-s5-aflevering-1#autoplay: Extracting information
ERROR: [generic] Unsupported URL: https://www.goplay.be/video/bake-off-vlaanderen/bake-off-vlaanderen-s5/bake-off-vlaanderen-s5-aflevering-1#autoplay
Traceback (most recent call last):
  File "/usr/local/bin/yt-dlp/yt_dlp/extractor/common.py", line 593, in extract
    ie_result = self._real_extract(url)
  File "/usr/local/bin/yt-dlp/yt_dlp/extractor/generic.py", line 3706, in _real_extract
    raise UnsupportedError(url)
yt_dlp.utils.UnsupportedError: Unsupported URL: https://www.goplay.be/video/bake-off-vlaanderen/bake-off-vlaanderen-s5/bake-off-vlaanderen-s5-aflevering-1#autoplay
Traceback (most recent call last):
  File "/usr/local/bin/yt-dlp/yt_dlp/extractor/common.py", line 593, in extract
    ie_result = self._real_extract(url)
  File "/usr/local/bin/yt-dlp/yt_dlp/extractor/generic.py", line 3706, in _real_extract
    raise UnsupportedError(url)
yt_dlp.utils.UnsupportedError: Unsupported URL: https://www.goplay.be/video/bake-off-vlaanderen/bake-off-vlaanderen-s5/bake-off-vlaanderen-s5-aflevering-1#autoplay

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/bin/yt-dlp/yt_dlp/YoutubeDL.py", line 1280, in wrapper
    return func(self, *args, **kwargs)
  File "/usr/local/bin/yt-dlp/yt_dlp/YoutubeDL.py", line 1305, in __extract_info
    ie_result = ie.extract(url)
  File "/usr/local/bin/yt-dlp/yt_dlp/extractor/common.py", line 610, in extract
    e.msg, video_id=video_id, ie=self.IE_NAME, tb=e.traceback, expected=e.expected, cause=e.cause)
yt_dlp.utils.ExtractorError: [generic] Unsupported URL: https://www.goplay.be/video/bake-off-vlaanderen/bake-off-vlaanderen-s5/bake-off-vlaanderen-s5-aflevering-1#autoplay

There is an upstream PR https://github.com/ytdl-org/youtube-dl/pull/27815 by @CNugteren, but it contains GPL code by @michaelarnauts which we cannot merge

There is an upstream PR ytdl-org/youtube-dl#27815 by @CNugteren, but it contains GPL code by @michaelarnauts which we cannot merge
PS: Seems only the login code has GPL. So it should be simpler to implement a cookies-based authentication

I'll gladly re-license that piece of code if that can help.

any updates on this issue?

a few notes:

I can provide an account if needed
The PR upstream still works, but do check out my comments in that ticket

Is there anything specific still blocking this issue?

I've prepared the code change (backported from https://github.com/ytdl-org/youtube-dl/pull/27815 and https://github.com/jeroenj/youtube-dl/pull/1) in https://github.com/jeroenj/yt-dlp/pull/1.
I haven't created the PR in this repository yet as I'm not sure what the implications of the GPL-3 license in the authentication code is: https://github.com/ytdl-org/youtube-dl/pull/27815#issuecomment-1083314039.
Would there be anybody that can help with that? Can that code be (easily) relicensed? Should it be an external dependency? 🤷♂

Can that code be (easily) relicensed?

It is sufficient for all contributors to state that they are willing to re-licence it as Unlicence
michaelarnauts has already stated above that he's willing to do so. But I do not know if there are any other contributors to the code

The code originates from me. And I don't mind either. The source has has a link to my original code, that is fine with me.

Cool, thanks for the confirmation.
I did some more cleanups, fixed the tests, and opened a PR with all those changes: https://github.com/yt-dlp/yt-dlp/pull/3412.


Lookup values display poorly when > 1 line

Is there a design about how they ought to display?

@astridu please advise

yes, surely!

In the new, non-modal InputLookup, the input is the search box (i.e., where the user enters her query). Given that, I would suggest that existing values appear below the input rather than in it.

That works too.

@astrid, @michelleif after changes recently merged this is what it looks like now:

If this is acceptable we can close this issue. If not I can try to move them to appear within the searchbox.

Can you put a little space inbetween the rows so they are not right up against each other? Thank you! Looking good!


Swift Mailer support

I made changes to phpBMS to support the latest Swift Mailer 4.1.5 and i think it's a much better solution than the old mail() function from PHP originally used to send invoices and quotes. I wanted to use PHP Mailer, but it's a dead project and I found some problems with it being outdated and not properly "talking" to some mail servers.
In this fork phpBMS now support SMTP, Sendmail and Mail.
The mail settings can be altered in the user's account.
Regards,
Jacques

Thanks Jacquesvw so much for this huge contribution. I'm testing this out now. One thing I'm concerned about for the future is keeping these libraries (Swift Mailer and FPDF) updated while preserving any customizations for phpBMS. I'll check through the commits of course but did you make any changes to the above libraries? Also with the FPDF stuff did you check if any previous customizations where made to 1.4.2 or whatever version was in there?

I made no changes to the Swift Mailer libraries, so updating to a new version in the future is simply a matter of copying the new version into the swift-mailer folder.
There were custom functions added to FPDI, but when I updated to the latest version I made sure to include those as well. I've been testing out the PDF creation and so far nothing seems to be broken.

Hello, Do you have any plans to fix/test this in the near future? Others are forking the code and I do think it would be great for everyone to move forward with your Swiftmailer patch.

Yes, I should get around to fixing it today. I also added the ability to print payment instructions to quotes and will upload that code today or tomorrow as well.

Hi D,
I fixed the installer so that clean installs work properly now and I also fixed the updater so that updating from v0.98 works. The update was tested using a clean install of version 0.98 that I downloaded at www.phpbms.org and it worked fine.


When a request fails, always set the data property.

This is for consistency between the different ways an Error instance
is created and populated.

Thanks a lot!


Improving login page ui and header ui

Hey i am a gssoc contributor and i want to improve the ui of login page and header section can u assign me this issue.

here is the updated ui for the login container

Already assigned to some one but he is not active i think so firstly share some templete then @Harshit7492  and @Anuj3553  will assigne you this and don't comment for label we will also add if you complete the task

sure here is a template

Hi @ganashree1612,
Thank you for sharing your work with us! We will assign you this issue. If you have any questions or need additional information, feel free to add them here.  Please note that you have removed the "Forgot Password" page issue from your pull request, but we will still provide labels for that contribution. Keep up the great work, and continue contributing your talents!

Thank you but the pull request number has not increased does it takes time and should i close this issue

Hey @ganashree1612,
We apologize, but we can only assign tags based on your current work level.

should i close this issue

Yes you can close this issue

Changes made

Hyy @ganashree1612, Can you do star mark this repo?

Sure done👍


[BEAM-8733] Handle the registration request synchronously in the Python&Java SDK harness

This PR would change the registration of both the Python SDK harness and Java SDK harness to synchronous.

Run Portable_Python PreCommit

I think the error of Portable_Python PreCommit is not cause by this PR.

 I try to fix it and already updated the PR.
Besides, the suggestion makes sense to me. I have added a new commit which could address your comments.
Best, Jincheng

Run Java PreCommit

Run Portable_Python PreCommit

Run Python PreCommit

Run Java PreCommit

Run Portable_Python PreCommit

Run Java PreCommit

Run Portable_Python PreCommit

Run Python PreCommit

Run Java PreCommit

Run Portable_Python PreCommit

Run Python PreCommit

@lukecwik I have add a commit for race condition, and I'm still investigating the test failure of the Java test cases and will let you know when there is any progress. (It's not possible to reproduce the test failures in my local machine and the log message is truncated in the Jenkins and so it may take me some time to find the root cause.)

Do you want to move the java change to a separate PR?
This would allow us to run fewer tests and likely merge the Python change.

Hi @lukecwik I like the idea of split this PR. I have update the PR and open 2 new PRs here:
#10242 and #10243
Let's wait for the test to change to green.

Run Python PreCommit


Port Bash

While including bash in a distribution is yet to be done, the bash port works, and I am closing this.


Why is there no Console Tab in Run?

Why is there no Console Tab in Run?It can still be shown the other day. I've tried to restart the computer.
Android Studio: 3.2.1

It can be compiled and installed, but there is no console information.

That's likely a pure AndroidStudio issue. Try deleting the AndroidStudio settings folder (or just rename to not lose it) and then start AS again and the tab will be back.
I don't know if there is a way to get it back without dropping all settings.

That's likely a pure AndroidStudio issue. Try deleting the AndroidStudio settings folder (or just rename to not lose it) and then start AS again and the tab will be back.
I don't know if there is a way to get it back without dropping all settings.

But I run the Andorid native project to display the running information.And I don't want to reset Android Studio.

And I don't want to reset Android Studio.

Just check if it solves the issue. You can restore the settings afterwards.
We can't provide support for Android Studio issues unrelated to Flutter.

Have the same issue, when I upgraded Flutter  (on master) the Console tab went missing!

@hash404 and when you switch back to another channel?

Still missing when I switch to flutter channel stable.
The Logcat appears when I open an Android app in Android Studio, but when I open a Flutter app in Android Studio there is no option for Logcat.

Did you try removing/renaming the directory that contains your settings?

I was able to get Console tab back with "Restore Layout" action:

Focus "Run" tab (click on tab header, not on its body)
Ctrl+Shift+A, find and click "Restore Layout"

@xjf1128 does this work for you as well?

I was able to get Console tab back with "Restore Layout" action:

Focus "Run" tab (click on tab header, not on its body)
Ctrl+Shift+A, find and click "Restore Layout"

Thank you very much for your help. I solved this problem.

You can just make some compilation errors (for example add  const String a = 1234;)  run the application and console will reveal. That worked for me.

https://github.com/flutter/flutter-intellij/issues/3673#issuecomment-517356243

Solution:

Close Android Studio.
Go to /Library/Preferences/AndroidStudio3.5/options
Delete runner.layout.xml
Start Android Studio.

Enjoy!!

I found the "Flutter Attach" process was the culprit when this happened to me. Restarting the app or resetting the layout didn't work and I didn't really want to delete files. I tried the Flutter Attach process and then stopped all processes. Once I did that, it was back to normal. Hope that helps someone.

Solution:

Close Android Studio.
Go to /Library/Preferences/AndroidStudio3.5/options
Delete runner.layout.xml
Start Android Studio.

Enjoy!!

Really saved my day! Thanks !


fix: fixed fix cookies path when use base_href

Pull Request Test Coverage Report for Build 1689

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 53.534%

Totals

Change from base Build 1688:
0.0%

Covered Lines:
9417

Relevant Lines:
15612

💛  - Coveralls


Loss Increased

Thanks for your awesome work. Do you have any idea why the loss increased after a certain step (one at 60k, and the other one at 70k) as follows? Thanks

Hello, thank you for your interest in our work. We have not encountered this issue when using the default training configuration. This issue may be caused by numerical stability problems. Can you provide more details about the training so that we can better investigate this issue:

Which training dataset are you using?
What is the total batch size and initial learning rate? For very small batches, the learning rate may be too large.

I train on re10k dataset with re10k_1x8 default config.
Do you have any insight how to fix it? Or if there is any special setting at those iteration?
Thanks.

Hi, all of my previous re10k experiences are based on the setting of batch size=128, and I have not had this problem.
I tried training with re10k_1x8 and did not have this problem either, as shown below:

If you still have this problem, maybe you can try:

Increase batch size as hardware allows.
Reduce the learning rate.

Please let me know if you still have this problem.

Thanks! I can successfully train with re10k_1x8 after decreasing the initial learning rate to 5x10^-5.


get started error

I was running this get started , when I run
docker run -d -p6831:6831/udp -p6832:6832/udp -p16686:16686 jaegertracing/all-in-one:latest
I got error

Step 62/65 : run python3 -m pip install -r requirements.txt
 ---> Using cache
 ---> f2f828cb0477
Step 63/65 : RUN python3 -m pip install /home/dora/workspace/dora-drives/wheels/dora_node_api_python-0.1.0-cp38-abi3-manylinux_2_31_x86_64.whl
 ---> Running in f9f39db2373d
Defaulting to user installation because normal site-packages is not writeable
WARNING: Requirement '/home/dora/workspace/dora-drives/wheels/dora_node_api_python-0.1.0-cp38-abi3-manylinux_2_31_x86_64.whl' looks like a filename, but the file does not exist
Processing ./wheels/dora_node_api_python-0.1.0-cp38-abi3-manylinux_2_31_x86_64.whl
**ERROR: Could not install packages due to an OSError: [Errno 2] No such file or directory: '/home/dora/workspace/dora-drives/wheels/dora_node_api_python-0.1.0-cp38-abi3-manylinux_2_31_x86_64.whl'**

The command '/bin/bash --login -c python3 -m pip install /home/dora/workspace/dora-drives/wheels/dora_node_api_python-0.1.0-cp38-abi3-manylinux_2_31_x86_64.whl' returned a non-zero code: 1
docker: Error response from daemon: Conflict. The container name "/dora" is already in use by container "195bcdf7560140428a9c64e5227633b11b6076e32292fe460f5fadc027bf9e88". You have to remove (or rename) that container to be able to reuse that name.
See 'docker run --help'.
Error response from daemon: Container 195bcdf7560140428a9c64e5227633b11b6076e32292fe460f5fadc027bf9e88 is not running
Error response from daemon: Container 195bcdf7560140428a9c64e5227633b11b6076e32292fe460f5fadc027bf9e88 is not running
Error response from daemon: Container 195bcdf7560140428a9c64e5227633b11b6076e32292fe460f5fadc027bf9e88 is not running

I think that the wheel: dora_node_api_python-0.1.0-cp38-abi3-manylinux_2_31_x86_64.whl is missing from the wheels folder.
It should normally be copied through the launch.sh command:
cd ../dora
cd apis/python/node
maturin build --release
cd ../../../
cp -r ../dora/target/wheels .

Closing as starting docker container has changed since this issue.


Support for MS SQL Windows Auth?

Any chance of supporting Windows Auth for MS SQL? Or is support only for SQL Server Auth?

We use node-mmsql client. If that supports it and someone is interested on implementing that a PR is welcome.

I came here seeing if this was already implemented in sqlectron-gui. It looks like node-mmsql supports Windows Auth beginning in v4.x. I would love to implement this but may not have the time until beginning of next year. For anyone else that may come across this, here are some resources:
Implementation Commit
https://github.com/patriksimek/node-mssql/commit/cc1ebe78f00f257e13a06bd8970c04274a27536e#diff-04c6e90faac2675aa89e2176d2eec7d8R286
Useful Suggestion for Docs
https://github.com/patriksimek/node-mssql/issues/283

hey guys,
Any update on this? I get the untrusted domain error message when connecting from a linux client (no LDAP)

Thank you!


Review all relative links in /Guides/getting-started-guide

Submit an issue
Topic
This is an issue regarding:

[x] The guides contained within this repo.
[ ] The samples contained within this repo.

Versions

[ ] Supported XD version(s):

Description of the issue
Proposed solution or attempted solutions that failed

@ashryanbeats may I ask why this was done?
I am doing some work to see about hosting these docs under e.g. a future adobe developer site 😇

Hi Fil,
Trying to jog my memory here... I think it was just about how the links worked during the long pre-launch period when building locally was required.
I don’t believe we’re using absolute links. If we are anywhere, I bet it’s an oversight.
On my phone, but can look into it more if you’ve got specific questions.

Ash

I just noticed some absolute links in the README in this repo (e.g. the quickstart tutorial and other tutorial links in the opening paragraphs of the README).
The reason I ask was I was putting up a staging environment with the e.g. XD plugin docs hosted under a subdirectory on a domain, kind of testing out how a future developer portal site would work (with different content being pulled in from different repos on github, mapped to different paths/subdirectories on the site). Absolute links ruin the ability to "prefix" the docs into a specific subdirectory, thus why I was wondering if using absolute links were serving a specific for y'all.
If this is not required, I am happy to send a PR fixing absolute to relative links, which shouldn't break anything on e.g. github pages or other hosted sites, but also help us with hosting the docs on other properties 😄

Absolute links for on-site pages are not a requirement for us, and it's our intention to avoid them.
Taking the readme, that you mention, as an example:
https://raw.githubusercontent.com/AdobeXD/plugin-docs/master/README.md
I'm not seeing where we're using absolute links for on-site pages. Unless you're asking us to explicitly include the path segment (. or ..) in relative links, I'll need you to point me to a specific example of what we're doing that you want us to change.
There are a number of absolute links in the readme and throughout the site, but those are "off-site" links pointing to, in the case of the top-level readme:

adobe.com
github.com

The GitHub links are intentionally absolute, because they aren't pointing the reader to a rendered page of docs, but an actual repo on GitHub (i.e., the reader leaves the docs site to go see a repo on github.com).
Examples:

[sample plugins](https://github.com/AdobeXD/plugin-samples)
[ui-hello-react](https://github.com/AdobeXD/plugin-samples/tree/master/ui-hello-react)

Based on the above info, is there still an issue? If so, can you point me to a specific line of markdown, and show me what the change would look like?

You can't see it on github's preview or github's rendered markdown, but if you go to this raw link:
https://raw.githubusercontent.com/AdobeXD/plugin-docs/master/README.md
then search for ](/, I see 12 links that are absolute - that start with a leading slash:
[Quick Start tutorial](/tutorials/quick-start)

Would it be OK to change these links so that they either start with a dot, or drop the leading slash? So the above example would turn into:
[Quick Start tutorial](./tutorials/quick-start)

or
[Quick Start tutorial](tutorials/quick-start)

Ah ha! Yep, this stuff should be fine to change (as long as GitBook jibes with it), and is admittedly haphazard at the moment.
We did a quick sweep on all links the morning before MAX to make sure things would work on the live site, so consistency fell by the wayside.
Please note that we have a big branch to merge and push soon, so if you do submit a PR on this, we'll need to wait until after that to merge your PR. Just aiming not to rock the boat too much at once.
Let me know if you need anything from me.

Great! I'll watch the repo for that big PR and put together a PR of my own once that lands. I shouldn't need anything from you other than a review of my PR down the road 😄


Drop Python 3.9

Support for Python 3.9 ends on Friday. I don't think we have planned releases today or tomorrow, so I'm calling it now.


Allow enabling/disabling RPC bridges in vald live

Description/Reasoning
There are certain scenarios when a validator needs to disable their RPC bridge for an external chain and re-enable it later, such as when an external chain is going through an upgrade, or to debug issues with their RPC connections. As of now, vald needs to be shutdown and restarted with the updated config whenever this needs to be done.
Since vald downtime can result in penalties, and loss of rewards, it would be great to allow enabling/disabling a bridge live. As a sanity check, we can also query whether the validator is registered as a chain maintainer for that chain before allowing them to disable the bridge.
Current Behaviour
Expected Behaviour
Steps to reproduce (for bugs)
Relevant Logs or Files

viper has the capability to watch configs. So we would have to experiment with passing a viper instance deeper into the code to read the configs on the fly. Alternatively, we could periodically check if the config has changed and adjust accordingly


Allow Laravel 7

Thanks for the great work on this @Cherry-Pie - this adds support for Laravel 7.
If you'd like help maintaining & bug fixing this package please let me know. Quite a few people in the Backpack for Laravel community are using it.
Cheers!

Thanks a lot for PR, really appreciate it!
tagged it as 1.7.0