Python 3 compatibility

Primarily follows steps from python-future.org. Currently waiting on a change in Numbskull to pass generative model tests (https://github.com/HazyResearch/numbskull/pull/51). Will assign reviewers when passing all tests in both Python2 + 3

TODO:

[ ] Go through and correct inefficient iteritems and list(range bits from conversion script
[ ] Why is SparseLogisticRegression seeming to fail tests??

Note: Gold labels for Intro tutorial seem to be misaligned in Python 3... everything runs but the scores are all messed up... presumably some sort of string parsing issue

Notes on errors in Intro 1 parsing:

One issue was sentence didn't get split due to \x chars, which made the count of people in it too high... in general think just some parsing issue...

 No obligation, but if either of you want to do a quick pass, much appreciated!  Otherwise I'll merge in Thursday am.  Thanks!

Looking now

@henryre Travis is testing on Python 2 and 3!  I'll look over the other comments in a bit, thanks!

@ajratner I think it's testing on Python 2 or Python 3 but may be wrong. Do you have a log that shows everything being run twice?

@henryre Yeah there are two "jobs" per Travis build- you can see if you go into Travis.  The "matrix" stuff is just if you want to consider many different combinations of configurations

@ajratner noice

@henryre made all requested comments save those I responded to- lmk if further needed

@dhimmel thanks so much for the great comments!!  Made all requested changes. Happy holidays / NYE!!

@dhimmel @henryre will plan to merge tomorrow AM if tests pass and no further comments- thanks for the help!!


hover selected bubble label, show deselect cross

When hovering a label of a selected trail7bubble, show a deselect cross that can be pressed to deselect that entity. The cross should be at the end of the name,

Good. it's a better idea than double clicking the label, which i use now
(it requires distinguishing "click" and "double click" which is not done, so it's not working ;)

done


CI failing, fix test

This #5891 PR bumped few dependencies as a result test was failing. fixed it

Thanks for the prompt correction! 🙂

:-)


Investigate and tune nginx performance

Based on https://github.com/rackerlabs/developer.rackspace.com/issues/471#issuecomment-48079910, investigate where the bottlenecks on individual web server performance are and try to remove/reduce them.

Not relevant anymore.


chore: Test GH for task management

This is a task that we need to get done.

We decided to do it!


docker: Upgrade some package managers to run ORT

The versions avaiable in Debian Sid have been upgraded.
Signed-off-by: Sebastian Schuberth<EMAIL_ADDRESS>

Commit message: "available"

Fixed.


Require sysroot for unsupported musl targets

These musl targets aren't officially supported and thus don't have a pre-compiled std available for download.

This might not be needed if the nightly build bots keep producing std versions for these targets, but I haven't been able to find the SLA for their availability.

This might not be needed if the nightly build bots keep producing std versions for these targets, but I haven't been able to find the SLA for their availability.

Huh? They are available via rustup:
$ rustup target add arm-unknown-linux-musleabi

Are you using some other way to install them?
Or, are they not available on same other channel like stable? Note that Xargo only works with a nightly Cargo/rustc.

I guess I am getting confused because even though the targets are available via rustup there are linker errors, which made me think that they are not compiling against a stable toolchain.
See for example https://travis-ci.org/dflemstr/rq/jobs/177296948
Maybe it would be good to add a --force-sysroot flag to xargo instead, so that it builds one even if an official std is available, to avoid link problems?

Will close this PR anyway

(And I might be able to solve the above problem by doing some linker like this so I'm not gonna open a new bug until I've verified that)


Fix possible free of unallocated memory

If connection does not succeed unallocated memory is being freed. Fixes #13

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Jac Kersing seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.


Stub file not found for "hvac"

The line import hvac produces Stub file not found for "hvac"
I installed hvac via pip3 install hvac
hvac is python client for Hashicorp Vault
https://pypi.org/project/hvac/
If you need any more information, please let me know!

edit: Added Screenshot

The reportMissingTypeStubs diagnostic check is not enabled by default in pylance, so I presume that you've enabled it, perhaps by setting typeCheckingMode to "strict"?
This diagnostic means that the import target is not marked as "py.typed" and you have no type stub that describes the types in this module. Strict type checking requires accurate and complete type information, and that needs to come from some source — either from the library itself or a separate type stub.
For more details about type stubs, refer to this documentation.


Admin features new

Gateway https://github.com/code-kern-ai/refinery-gateway/pull/119
Model submodule https://github.com/code-kern-ai/refinery-submodule-model/pull/36
Admin dashboard: https://github.com/code-kern-ai/admin-dashboard/pull/62
Refinery UI: https://github.com/code-kern-ai/refinery-ui/pull/125
Updater https://github.com/code-kern-ai/refinery-updater/pull/28

I'm not super sure how but I think there is still some issue in the logic since I never touched the users without a role assignment yet they still got a timestamp.

Only way I could find to reproduce was.

start fresh
Create org
Assign user to org
switch to refinery
import first project

That said I tried to find it with throwing an exception on the id but nothing happened at that point so maybe some further investigation is required

[ ] resolved


[Snyk] Upgrade @azure-tools/typespec-azure-resource-manager from 0.46.1 to 0.47.1

Snyk has created this PR to upgrade @azure-tools/typespec-azure-resource-manager from 0.46.1 to 0.47.1.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 14 versions ahead of your current version.

The recommended version was released on a month ago.

Release notes

Package name: @azure-tools/typespec-azure-resource-manager

0.47.1 - 2024-10-14Bug Fixes

#1673 Fix ArmProviderActionAsync to correctly return 202 responses.

0.47.0 - 2024-10-10

0.47.0-dev.11 - 2024-10-09

0.47.0-dev.10 - 2024-10-04

0.47.0-dev.9 - 2024-10-03

0.47.0-dev.8 - 2024-10-03

0.47.0-dev.7 - 2024-10-02

0.47.0-dev.6 - 2024-09-27

0.47.0-dev.5 - 2024-09-26

0.47.0-dev.4 - 2024-09-24

0.47.0-dev.3 - 2024-09-17

0.47.0-dev.2 - 2024-09-13

0.47.0-dev.1 - 2024-09-12

0.47.0-dev.0 - 2024-09-11

0.46.1 - 2024-09-24

    from @azure-tools/typespec-azure-resource-manager GitHub release notes

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:

🧐 View latest project report
📜 Customise PR templates
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

@sylvainsf , fyi..this dependency overlaps and is being handled by Dependabot. We are currently allowing only patch updates for this dependency.


How to get Multiscale Superpixel Hierarchy

Hello， how I can get Multiscale Superpixel Hierarchy？ and how is the Superpixel Hierarchy used in graph_FeaturePyramid?
Looking forward to your help, thank you very much.

You can see our paper that first introduced a hierarchical superpixel method: COB. COB could generate the multiscale superpixel hierarchy. If it is necessary, we will release the code about it as soon as possible. Thanks.

Thanks for your reply. I really appreciate it if you could release the code about multiscale superpixel hierarchy and how it is used in graph_FeaturePyramid.

Hello， how I can get Multiscale Superpixel Hierarchy？ and how is the Superpixel Hierarchy used in graph_FeaturePyramid?
Looking forward to your help, thank you very much.

May I ask how to download Coco data in folder steucture？The Coco data set I downloaded online is different from the one above.How did you get the Coco data set？


Unable to use Avatar in LeftNav MenuItem

I'm trying to add an Avatar or leftAvatar as a MenuItem in LeftNav but can't find the way to do so.
I was looking into menu-item.jsx but it seems there is no Avatar component option there. Perhaps through NESTED ({ type: MenuItem.Types.NESTED.leftAvatar)?

LeftNav is now composable, you can just add it as a child.


feat(layer): publish SAR v2 via Github actions

Issue number: #1543
Summary
This PR adds support for creating the v2 SAR via Github Actions.
Changes

Please provide a summary of what's being changed

Rename V2 Layer and Canary stacks to LayerV2Stack and CanaryV2Stack, to avoid clashing with the existing v1 stack which we will want to maintain
Re-use the Lambda Layer artifact to publish the SAR using SAM
No more dependencies with internal pipelines
Authentication using OIDC as a jump role

User experience

Please share what the user experience looks like before and after this change

A successful SAR release will now look like this (please ignore the Layer deployment failure):

Checklist

[ ] After merge, remove feat/sar-v2 from environment branches

If your change doesn't seem to apply, please leave them unchecked.

[x] Meet tenets criteria
[x] I have performed a self-review of this change
[x] Changes have been tested
[ ] Changes are documented
[x] PR title follows conventional commit semantics

Is this a breaking change?
RFC issue number:
Checklist:

[ ] Migration process documented
[ ] Implement warnings (if it can live side by side)

Acknowledgment
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.

Screenshot updated


naucse-python dies with an error when running for the first time

Hello All,
wanted to add a new course through naucse-python repo. Used this tutorial. Created new venv and run poetry install and then poetry poetry run python -m naucse serve. It died at:
(pyladies) λ  naucse-python git:(main) ✗ poetry run python -m naucse serve
Traceback (most recent call last):
  File "<frozen runpy>", line 198, in _run_module_as_main
  File "<frozen runpy>", line 88, in _run_code
  File "/home/mcurlej/Development/python/venv/pyladies/lib/python3.11/site-packages/naucse/__main__.py", line 1, in <module>
    from naucse.cli import main
  File "/home/mcurlej/Development/python/venv/pyladies/lib/python3.11/site-packages/naucse/cli.py", line 3, in <module>
    from naucse.views import make_app
  File "/home/mcurlej/Development/python/venv/pyladies/lib/python3.11/site-packages/naucse/views.py", line 11, in <module>
    from naucse import models
  File "/home/mcurlej/Development/python/venv/pyladies/lib/python3.11/site-packages/naucse/models.py", line 22, in <module>
    from naucse import local_renderer, compiled_renderer
  File "/home/mcurlej/Development/python/venv/pyladies/lib/python3.11/site-packages/naucse/compiled_renderer.py", line 11, in <module>
    from gitpathlib import GitPath
  File "/home/mcurlej/Development/python/venv/pyladies/lib/python3.11/site-packages/gitpathlib/__init__.py", line 4, in <module>
    from gitpathlib.gp_pygit import PygitBackend
  File "/home/mcurlej/Development/python/venv/pyladies/lib/python3.11/site-packages/gitpathlib/gp_pygit.py", line 9, in <module>
    pygit2.GIT_OBJ_COMMIT: 'commit',
    ^^^^^^^^^^^^^^^^^^^^^
AttributeError: module 'pygit2' has no attribute 'GIT_OBJ_COMMIT'. Did you mean: 'GIT_OBJECT_COMMIT'?
(pyladies) λ  naucse-python git:(main) ✗

running:
(pyladies) λ  naucse-python git:(main) ✗ python --version
Python 3.11.9
(pyladies) λ  naucse-python git:(main) ✗

Discussed with @encukou and it seems like a bug.

Thanks for the report!
This should be fixed in the new gitpathlib 0.3.


ascending order

ℹ Repository information
The repository has:

[ ] At least three issues with the good first issue label.
[ ] At least 10 contributors.
[ ] Detailed setup instructions for the project.
[ ] CONTRIBUTING.md
[ ] Actively maintained.

@aadeshkulkarni can you please  check?

@aadeshkulkarni Hey Any update?

@Amar995  - Looks good. The branch has conflicts with base branch. Please rebase so that it can be merged.

@aadeshkulkarni Its shwoing This branch has no conflicts with the base branch

@Amar995 - Thank you for your contribution! :)


Added visitor for scene objects and descendants

This pattern allows you to iterate thru the hierarchy without making a
copy of it
GearVRf-DCO-1.0-Signed-off-by<EMAIL_ADDRESS>Nola Donato

Looks good to me but the public APIs need javadoc.

We can add the javadoc later.

javadocs added in PR #722


Possible to generate a Queue upon upload of a Blob in Azurite?

I have an Azure Function triggered by Queue messages in Azure Storage. Queues are generated by an Event Grid topic subscription on the Blob Container. For each blob uploaded, a new Queue message is generated.
I'm having trouble testing this locally using Azurite. Is it possible to generate a Queue upon upload of a Blob in Azurite? Or do I just need to manually generate the message for testing?
Thank you
Which service(blob, file, queue, table) does this issue concern?
Blob, Queue
Which version of the Azurite was used?
3.18.0
Where do you get Azurite? (npm, DockerHub, NuGet, Visual Studio Code Extension)
VS Code Extension
What's the Node.js version?
N/A
What problem was encountered?
N/A
Steps to reproduce the issue?
N/A
If possible, please provide the debug log using the -d parameter, replacing <pathtodebuglog> with an appropriate path for your OS, or review the instructions for docker containers:
-d "<pathtodebuglog>"

Please be sure to remove any PII or sensitive information before sharing!
The debug log will log raw request headers and bodies, so that we can replay these against Azurite using REST and create tests to validate resolution.
Have you found a mitigation/solution?

@SeaDude
For "I'm having trouble testing this locally using Azurite." What's the trouble do you meet?
Do you mean Azurite can't generate a queue message like Event Grid?
Azurite is an Azure Storage API Emulator; we currently don't support Event Grid.
If you really need a queue message be generated when a blob upload, a workaround might be adding some client code to monitor the blob and trigger the queue message generation with blob uploaded.

That's a good idea. Thank you for the suggestion.

@blueww
I am attempting to do this same thing. I'd like to send a CloudEvent to a local testing queue in azurite when a file is uploaded to a testing blob in Azurite. Can you provide example of generating the event and publishing to queue? preferred sdk language is python
I have a python Blob Trigger function running in azure functions.

@user3472g, what I do is:

Start the Azurite extension in VS Code (Blob, Table and Queue)
Open Azure Storage Explorer
Under Emulated & Attached expand the `Emulator) account and wait it for initialize
Right click on Queues and create a new queue

Be sure to put this queue name in your Function's function.json
Hint: Name this queue the same as the queue in your Azure Storage account and you won't have to adjust it

Once you have the Queue setup and "AzureWebJobsStorage": "UseDevelopmentStorage=true" set in local.settings.json, start the Function locally with func start.
Once it initialized, back in Azure Storage Explorer, select the Queue and click Add Message.
Paste in a Sample CloudEvent schema

The local Function should trigger and you can watch it execute.


Write docs and tests for QueryBuilder.clear()

Similar to #758, here the docs and tests for QueryBuilder.clear()

Coverage remained the same at 95.032% when pulling 4e7100e428f7a56e2046593dcdef56f67cf07e84 on lehni:feature/clear-docs-tests into ad592625790a88d49147703995817afaf0bdb298 on Vincit:master.

Coverage remained the same at 95.032% when pulling 4e7100e428f7a56e2046593dcdef56f67cf07e84 on lehni:feature/clear-docs-tests into ad592625790a88d49147703995817afaf0bdb298 on Vincit:master.

Everybody's going to ask "does it clear select as well" ?

@kapouer I think you misunderstood what the method does.
// Clears all calls to `where` method.
builder.clear('where')
// Clears all calls to `select` method.
builder.clear('select')
// Clears all calls to `where`, `orWhere`, `andWhere` etc. method.
builder.clear(/where/)

I think the documentation is clear on this.

I could add an example to illustrate the use of a regular expression though?


Translate guides/tutorial/part-2/ember-data.md

Closes #151

I haven't really used the .local.dic sorting yet because it wasn't an effort for me to put the new exceptions in the right place, but I have no doubt it's super useful in general and will prevent a lot of careless mistakes in the long run.
The pre-commit hook is going to be life-saving. One thing I noticed is I have the following message:

.remarkignore was not in your staged files, so we have added it for you.

...when I amend my commit, but aside from the message itself it doesn't have any unwanted effect.

@BlueCutOfficial
I have added 3 little commits (while not altering your first one), feel free to merge all of them into one, or not.

@BlueCutOfficial , let's move forward.
Just to let you know I have merged these commits into your first one


LESS mixin unexpected code formatting

Using VSCode 1.66.0 and the new built-in LESS code formatting feature less.format.enable implemented by the JS Beautify library.
When using a LESS mixin that accepts CSS code rules as a parameter (this is used for media query mixins), the formatting when saved is a little unexpected. The formatting is consistent, but it is unusal.
Example code (expected formatting)
.example(@rules) {
    @rules();
}

.test {
    .example({
        color:red;
    });
}

Result after saving
.example(@rules) {
    @rules();
}

.test {
    .example( {
            color:red;
        }

    );
}

Similar issue reported a while back js-beautify issue 722
I've noticed that if you add the following to your settings.json file, the results are slightly better:
"less.format.newlineBetweenRules": false

This gives you formatting like:
.example(@rules) {
    @rules();
}

.test {
    .example( {
            color:red;
        }
    );
}

Apologies if this isn't the right place to report this, but as it was a new feature of VSCode I figured I'd start here.

Thanks for the report. We depend on JS Beautify here, so please file issues there.
Maybe we need to disable the formatter again for LESS if LESS is outside the scope of JS Beautify.

Thanks for the report. We depend on JS Beautify here, so please file issues there.
Maybe we need to disable the formatter again for LESS if LESS is outside the scope of JS Beautify.

Thanks, can you please advise what the version of js-beautify is that comes with VSCode 1.66.0 ? I cannot seem to find this information anywhere.
The issue was moved to https://github.com/beautify-web/js-beautify/issues/2016


2019-03-22 20:18:42 +0100 interpreted as a string

2019-03-13 20:18:42 +0000 is a date format considered valid in Jekyll, when switching to 11ty, it's not. (see: https://github.com/11ty/eleventy/issues/413#issuecomment-475832065)
space separated:  2001-12-14 21:59:43.10 +0000 returns  a string on https://nodeca.github.io/js-yaml/
http://www.yamllint.com/ will add a colon after parsing: 2019-03-13 20:18:42 +00:00

This package is about yml standard, not about jekyll.  If you think js-yaml does not conform spec, please give URL to spec details. Also, reference implementation is pyyaml, if you need to compare.

Just found this and I've got to say, the spec is way to ambiguous on its definition of timestamps. YAML 1.2 §10.1. Failsafe Schema doesn't actually provide any guidance about what to do with Example 2.22 Timestamps even though it's implied that it shouldn't need an explicit tag. As an editor of other specs, I'd call this a spec bug, not just an implementation one. So congrats, you're both right!
I'm inclined to say we should defer to RFC 3339 Date and Time on the Internet: Timestamps §5.6 Internet Date/Time Format, which covers the canonical, iso8601 and date examples, but I really don't know what to do about spaced: 2001-12-14 21:59:43.10 -5. If it were me, I'd say allowing space would go from RFC 3339's:
   time-numoffset  = ("+" / "-") time-hour ":" time-minute
   time-offset     = "Z" / time-numoffset

   partial-time    = time-hour ":" time-minute ":" time-second
                     [time-secfrac]
   full-date       = date-fullyear "-" date-month "-" date-mday
   full-time       = partial-time time-offset

   date-time       = full-date "T" full-time

to instead have:
   full-time       = partial-time [" "] time-offset
   date-time       = full-date ("T" / "t" / " ") full-time

But then we don't either allow 2019-03-13 20:18:42 +0000 (no colon) or 2001-12-14 21:59:43.10 -5 (not even remotely close to a time-offset).
So I guess I'm off to speak with the editors!

Ah, looks like this example is likely to be removed: https://github.com/yaml/yaml-spec/issues/268


Allow customization of MaxPacketBufferSize

Right now it's a const in internal/protocol/protocol.go - set to 1452.
Jumbo frames are common ( 9000 or more ), and it has a big impact on throughput.

Jumbo frames are common ( 9000 or more )

@costinm Do you have any data for that? On the public internet I thought you're pretty much limited to 1500 bytes.

On the public internet - maybe, I don't have a lot of data on that - but on 'clouds' larger MTUs are very common :-)

I'm not sure what is the concern with exposing it. If you use QUIC on a VPC with jumbo frames - why wouldn't you want to
use them ?  There is nothing in the standard that says QUIC should not use jumbo frames where available.
I understand not doing fancy stuff like getting it from the interface, but I can't see a downside to having a var instead of const ( or a config option ). It does have an impact on performance.

I'm not opposed to this - no doubt there are niches where this is useful. I won't work on this myself, but I'd be willing to review a PR.
Note that this is not as trivial as just changing the const to a var. This has to be wired into the transport parameters, namely the max_udp_payload_size.

Cool.


Let's say that you update the "aws" provider and it fails. You can run
it again with:
./update-provider aws --vendor

Any package that has the "vendorSha256" key will be considered as a go
module package.
The script now also supports adding new providers by using the
/ format. Eg:
 ./update-provider hetznercloud/hcloud --vendor

Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

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

Result of nixpkgs-review pr 104667 1

8 packages built:

terraform-full
terraform-providers.aws
terraform-providers.cloudfoundry
terraform-providers.hcloud
terraform-providers.keycloak
terraform-providers.linuxbox
terraform-providers.shell
terraform_0_11-full

Duplicate of #103822


docs: add take_std(in|out|err) in utils

Referring to issue #73, the Command:std(in|out|err) was already present in the docs.
So i added the take_std(in|out|err) under the Child sub, in utils.md
If I have misunderstood any part of it, pls do correct!
Thanks!

Also added write_all and flush in the child. I hope they are fine...
(Why is merge showing errors?)

Could you add some examples for them please?

uh... ill try, it took a while for me to understand that small rust code for it(I m not familiar with rust). I'll update if I am unable too.
Rest of the things are correct??

I added random sort in keymaps as mentioned in #70 . Is there another place it needs to be added @sxyazi ?

Is it possible to merge this branch soon @sxyazi so that I can add some UI components to the docs, as i told before. otherwise the new branch will be built on top of this main and i ll face some issues dealing with commits and all.


Remove chatbox component

Any way to check the build please?
I tried running the build script but I get FATAL ERROR: Ineffective mark-compacts near heap limit Allocation failed - JavaScript heap out of memory.

Not completely sure. Can you  check in with @zachferland when he's online?

may need to NODE_OPTIONS=--max-old-space-size=4096, unclear exacltly why but this build has taken up a huge amount of memory to build

Thanks! It works with the added flag, should be good to merge 👍

That's a workaround and it works sometimes but guys you need to find what's sucking so much memory to make it build this much slower and bigger.


IJSRuntime

I have references I need to use that rely on IJSRuntime in the scope. Code that relies on IJSRuntime seems to fail and/or work in a different sand box than the rest of the website. Any insight into this or workarounds wasm apps?
Example would be using Blazored.LocalStorage within a SpawnDev Web Worker. When I write to storage it will be wiped with the service, after the service is disposed.

This was super helpful. I incorrectly assumed Local Storage was available, since IndexedDB was. Thanks!


#58 tests - sponsors list, meetup

added tests for better coverage

Current coverage is 95.03% (diff: 100%)

Merging #218 into master will increase coverage by 0.57%

@@             master       #218   diff @@
==========================================
  Files            46         46
  Lines          1373       1391    +18
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           1297       1322    +25
+ Misses           76         69     -7
  Partials          0          0

Powered by Codecov. Last update 79df0a6...66e7110


Recursive monitoring request

Please, it's possible to add support for recursive monitoring? On Windows there is argument bWatchSubtree (see here), which is set to FALSE for now.

short answer: yes :) but this requires a changes in the dir_monitor interface.
long answer: i have got a working copy with functional requested by you. and i want to merge it in the nearest future. but I can't do it right now because my changes have been made onto base Boris's implementation of dir_monitor. so i need some additional job for adoptation.

Ok, thank you for answer :).

how much time you can endure without this feature?)))

I have own fork with only setted this argument, so there's no hurry :).

It would be nice if this was consistent between the different implementations.

The inotify one has recursive support, but it is impossible to turn it off atm (relatively easy to fix, as Inotify is not recursive by default).
The OSX one (fsevents) is recursive by default, but It would be nice to have a non-recursive implementation (filter) here as well.
Windows has native support for both recursive and nonrecursive watching, but as you noticed, it's turned off in the code.

Agree with @akapelrud lets add some flag in the interface (or another method set like add_directory_recursive()) for the recursive case and make default non-recursive.
Opinions?

agree in common but need to clarify details in my realization.. on monday.

@sbelsky I implemented adding of non-recursively monitored directories for inotify on my own master branch before easter. Have a look at commit 51275f6. Feel free to take that code if you need it. (I pushed my local changes to my master branch, but since then you have started using a develop branch, so my fork is a bit of a mess atm.)
Making the add_directory() functions work in both recursive and non-recursive mode isn't that hard, the hassle lies in implementing this so that directories can be removed afterwards. For instance, a test case could be

Add one directory foo/bar/foo2 non-recursively.
Add the directory foo recursively.
Remove the directory foo.
First directory should still be watched by dir_monitor.

On OSX (fsevents) you can just add directories and check paths against the stored directory paths as the events happen.
For inotify you can be a bit more clever, as it's non-recursive by default.
On windows you can choose either strategies above, as the API supports both.

@akapelrud thank you for analyze and code sharing. I fully agree with your arguments but want to add my view.
recursive directory monitoring is more complex task than even you describe because performance questions come into play. just imagine a folder with a big big subtree. i don't think real user want to watch over all leafs, just a subset.
my proposal:

yes, this must be consistent between the different implementations. each of us specializes in its own operating system and it should be used.
library's interface should stay backward compatible
we can introduce new overloaded add_directory method with second unsigned integer argument. the meaning of this argument is a recursion depth. 0 means no recursion. i did it in my company's fork but as the wrapper of dir_monitor and only for windows.

Yes, and it currently isn't. That can give some surprises if portable code is needed.
Yes, or simply alter the current add_directory function to have a second default-valued argument. I would prefer your suggestion though.
I like the recursion depth argument, maybe simply a negative value can be interpreted as infinite recursion (the current default for inotify).

Oh, I have tested dir_monitor only on the inotify backend.
I thought it already has recursive directory scanning for all platforms (inotify backend is recursive).

From my point of view one can't keep the interface backwards compatible and fix the inconsistent behaviour at the same time, because by fixing the inconsistent behaviour the semantics of add_directory are changed on at least one platform.
My suggestion is to introduce two new methods with well defined semantics like add_directory_recursive and add_directory_shallow. While we leave add_directory untouched and deprecate it (with c++14 the [[deprecated]] attribute could be used).


Crash on second startup during world generation

I've isolated the mod, used separate versions of both the mod and forge, and cleared the config file.
Clearing the config seems to work, but I don't want to clear my config folder everytime I want to start a new game with the mod installed.
Anyone have a solution or also experiencing the problem?
Oh, and here's a crash report:
**---- Minecraft Crash Report ----
WARNING: coremods are present:
MixinLoaderForge (ValkyrienWarfare-Jenkins-341.jar)
Contact their authors BEFORE contacting forge
// I feel sad now :(
Time: 10/25/18 10:00 PM
Description: Exception in server tick loop
java.lang.NullPointerException: Exception in server tick loop
at net.minecraft.world.World.func_175725_q(World.java:4685)
at net.minecraft.world.gen.ChunkGeneratorOverworld.func_185931_b(ChunkGeneratorOverworld.java:475)
at net.minecraft.world.chunk.Chunk.func_186034_a(Chunk.java:1019)
at net.minecraft.world.chunk.Chunk.func_186030_a(Chunk.java:999)
at net.minecraft.world.gen.ChunkProviderServer.func_186025_d(ChunkProviderServer.java:157)
at net.minecraft.server.MinecraftServer.func_71222_d(MinecraftServer.java:344)
at net.minecraft.server.integrated.IntegratedServer.func_71247_a(IntegratedServer.java:142)
at net.minecraft.server.integrated.IntegratedServer.func_71197_b(IntegratedServer.java:156)
at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:486)
at java.lang.Thread.run(Thread.java:745)
A detailed walkthrough of the error, its code path and all known details is as follows:
-- System Details --
Details:
Minecraft Version: 1.12.2
Operating System: Windows 10 (amd64) version 10.0
Java Version: 1.8.0_51, Oracle Corporation
Java VM Version: Java HotSpot(TM) 64-Bit Server VM (mixed mode), Oracle Corporation
Memory: 593969584 bytes (566 MB) /<PHONE_NUMBER> bytes (1196 MB) up to<PHONE_NUMBER> bytes (7282 MB)
JVM Flags: 2 total; -XX:HeapDumpPath=MojangTricksIntelDriversForPerformance_javaw.exe_minecraft.exe.heapdump -Xmx8G
IntCache: cache: 0, tcache: 0, allocated: 13, tallocated: 95
FML: MCP 9.42 Powered by Forge <IP_ADDRESS>72 5 mods loaded, 5 mods active
States: 'U' = Unloaded 'L' = Loaded 'C' = Constructed 'H' = Pre-initialized 'I' = Initialized 'J' = Post-initialized 'A' = Available 'D' = Disabled 'E' = Errored
| State   | ID               | Version      | Source                           | Signature                                |
|:------- |:---------------- |:------------ |:-------------------------------- |:---------------------------------------- |
| UCHIJAA | minecraft        | 1.12.2       | minecraft.jar                    | None                                     |
| UCHIJAA | mcp              | 9.42         | minecraft.jar                    | None                                     |
| UCHIJAA | FML              | <IP_ADDRESS>    | forge-1.12.2-<IP_ADDRESS>72.jar    | e3c3d50c7c986df74c645c0ac54639741c90a557 |
| UCHIJAA | forge            | <IP_ADDRESS>72 | forge-1.12.2-<IP_ADDRESS>72.jar    | e3c3d50c7c986df74c645c0ac54639741c90a557 |
| UCHIJAA | valkyrienwarfare | 0.9_alpha    | ValkyrienWarfare-Jenkins-341.jar | None                                     |

Loaded coremods (and transformers):

MixinLoaderForge (ValkyrienWarfare-Jenkins-341.jar)
GL info: ~~ERROR~~ RuntimeException: No OpenGL context found in the current thread.
Profiler Position: N/A (disabled)
Player Count: 0 / 8; []
Type: Integrated Server (map_client.txt)
Is Modded: Definitely; Client brand changed to 'fml,forge'**

Old issue, closed.


Mouse cursor for "view demo network" link should be a hand

Describe the bug
The mouse cursor when hovering over the "view demo network" link is a text-select cursor. It should be a hand.
This happens in Chrome and Safari, haven't tried other browsers.

Instead of adding cursor: pointer as a style (or MUI class) to every single  component that has no href attribute, I just added this to the base.css file:
a {
  cursor: pointer;
}

I can't think of a scenario (in this app, at least) where a link shouldn't have a pointer cursor.


"Error: Conflicting Parameters" on role_arn when assume_role block is defined

When Terraform is upgraded to 1.3 or greater, init fails with the following error:

[INFO] (deploy track/step/primary/us-east-1)
[INFO] (deploy track/step/primary/us-east-1)   Initializing the backend...
[INFO] (deploy track/step/primary/us-east-1)   Initializing modules...
[INFO] (deploy track/step/primary/us-east-1)
[INFO] (deploy track/step/primary/us-east-1)   Warning: Deprecated Parameters
[INFO] (deploy track/step/primary/us-east-1)
[INFO] (deploy track/step/primary/us-east-1)     on backend.tf line 2, in terraform:
[INFO] (deploy track/step/primary/us-east-1)      2:   backend "s3" {
[INFO] (deploy track/step/primary/us-east-1)
[INFO] (deploy track/step/primary/us-east-1)   The following parameters have been deprecated. Replace them as follows:
[INFO] (deploy track/step/primary/us-east-1)     * role_arn -> assume_role.role_arn
[INFO] (deploy track/step/primary/us-east-1)
[INFO] (deploy track/step/primary/us-east-1)
Error:  (deploy track/step/primary/us-east-1)
Error:  (deploy track/step/primary/us-east-1)   Error: No valid credential sources found
Error:  (deploy track/step/primary/us-east-1)
Error:  (deploy track/step/primary/us-east-1)   Please see https://www.terraform.io/docs/language/settings/backends/s3.html
Error:  (deploy track/step/primary/us-east-1)   for more information about providing credentials.
Error:  (deploy track/step/primary/us-east-1)
Error:  (deploy track/step/primary/us-east-1)   Error: failed to refresh cached credentials, no EC2 IMDS role found,
Error:  (deploy track/step/primary/us-east-1)   operation error ec2imds: GetMetadata, failed to get API token, operation
Error:  (deploy track/step/primary/us-east-1)   error ec2imds: getToken, http response error StatusCode: 400, request to EC2
Error:  (deploy track/step/primary/us-east-1)   IMDS failed
Error:  (deploy track/step/primary/us-east-1)
Error:  (deploy track/step/primary/us-east-1)

When the following block is added to backend.tf in the step:
  assume_role = {
    role_arn = "[MY_ARN]"
  }

The error message at runtime changes to:

[INFO] (deploy track/step/primary/us-east-1)
[INFO] (deploy track/step/primary/us-east-1)   Initializing the backend...
[INFO] (deploy track/step/primary/us-east-1)   Initializing modules...
Error:  (deploy track/step/primary/us-east-1)
Error:  (deploy track/step/primary/us-east-1)   Error: Conflicting Parameters
Error:  (deploy track/step/primary/us-east-1)
Error:  (deploy track/step/primary/us-east-1)     on backend.tf line 2, in terraform:
Error:  (deploy track/step/primary/us-east-1)      2:   backend "s3" {
Error:  (deploy track/step/primary/us-east-1)
Error:  (deploy track/step/primary/us-east-1)   The following deprecated parameters conflict with the parameter
Error:  (deploy track/step/primary/us-east-1)   "assume_role". Replace them as follows:
Error:  (deploy track/step/primary/us-east-1)     * role_arn -> assume_role.role_arn
Error:  (deploy track/step/primary/us-east-1)
Error:  (deploy track/step/primary/us-east-1)

It is suspected this is due to RunIAC injecting role_arn as a -backend-config=key=value command line param at runtime.
Perhaps it is not aware of an assume_role element in the backend config map and does not think a role_arn is defined?

Likely involves paying down this TODO.

Likely involves paying down this TODO.

Good example of the HCL decode and struct model that could fulfill the TODO and allow native parsing instead of string replacements. HCL file schema and struct loading example


feature

Can you give me the characteristics of training directly?

Hello, thanks for your interest in our work. The saliency feature is from the Download the extra nocaps dataset that is not provided by nocaps and unzip it. Hope this can help you.
Best regard

hello,I have downloaded all of feature. There is a problem. When loading the coco_train2017_penultimate_feature, in code occur ['attention_maps'] ,but the penultimate_feature H5 file I downloaded,the key values in H5 only have the attributes ['height'], ['image_id '], ['penultimate_features'], the['penultimate_ Feature'] and ['attention_maps'] are same thing? or is wrong the H5 file I downloaded. Another question, can I have the trained model?
Looking forward to your reply~
------------------ 原始邮件 ------------------
发件人:                                                                                "chenxy99/ANOC"                                                                                @.>;
发送时间: 2021年12月24日(星期五) 下午2:09
@.>;
<EMAIL_ADDRESS>主题: Re: [chenxy99/ANOC] feature (Issue #4)
Hello, thanks for your interest in our work. The saliency feature is from the Download the extra nocaps dataset that is not provided by nocaps and unzip it. Hope this can help you.
Best regard
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you authored the thread.Message ID: @.***>

['penultimate_ Feature'] is the penultimate feature map, the shape may be HW512, while ['attention_maps'] is the saliency prediction of an image, the shape may be HW1.
I will update the readme file for the trained model when I finish the travel.
Best

Hello,
I just update the code, you can try it again.
Best regards


Extend ArtifactContainer with methods to query for relationships

While the application class has a method to query relationships of a certain role, the ArtifactContainer does not. It would be nice if one could query the container for incoming/outgoing relationships of a certain role.

So you expect ArtifactContainer to implement HasRelationships? What's the outcome? Relationships themselves are not part of a container, but the source and target artifacts might have a container.
Do you expect the method to return all relationships whose source and/or target artifact is part of the container?
Another idea:
We can think about some HasIncomingRelationships interface which provides Set<ArtifactRelationship> getIncomingRelationships() and Set<ArtifactRelationship> getIncomingRelationshipsOfRole(ArtifactRelationshipRole) in analogy to HasRelationships. Same applies to HasOutgoingRelationships.
These interface can be implemented by Artifact and ArtifactContainer.

Sounds good to me.


Sort Disks and NICs in the Review step of CreateVmWizard

Fixes: https://github.com/oVirt/ovirt-web-ui/issues/1281
Sort Disks and NICs in the Storage and Networking sections of the Review report step alphabetically by template, then by name - the same as in Storage and Networking steps of the wizard, to achieve the consistency.

Before:

After:

If template based Disks and NICs are present: https://github.com/oVirt/ovirt-web-ui/pull/1283#issuecomment-680081711

@bamsalem This is ok. It happens because the NIC is template based (see the T) - and that has a higher priority than the name, when sorting.

@bamsalem I cannot reproduce the issue, however I am getting this:

The problem here is that in the Review step, I sort the items alphabetically, without checking template based items. And I am not sure about the expected behavior - if we want to sort them totally the same as in Networking/Storage steps or simply alphabetically.

So I've made a change and now Disks and NICs will be sorted in the Review step exactly the same way as in the Storage and Networking steps of the CreateVmWizard, to achieve the consistency of the wizard itself:

ci build please

ci build please

ci build please

ci build please

ci build please

ci build please

ci build please

ci build please

ci build please

ci build please

ci build please

ci build please

ci build please

ci build please

ci build please

ci build please

ci build please

ci build please

ci build please

ci build please

ci build please

ci build please

ci build please

ci build please

ci build please

ci build please

ci build please

ci build please

ci build please

ci build please

ci build please

ci build please

ci build please

ci build please

ci build please

ci build please

ci build please

ci build please

ci build please

ci build please

ci build please

ci build please

ci build please

ci build please

ci build please

ci build please

ci build please

ci build please

ci build please

ci build please

ci build please

ci build please

ci build please

ci build please

ci build please

ci build please

ci build please

ci build please

ci build please

ci build please

LGTM

@sgratch @sjd78 Is there anything I should make regarding this PR? I think it's ready to merge. Thanks! :)


Can't enable unstable features

Describe the bug
rustfmt does not appear to accept the unstable_features = true option  (documented here) in rustfmt.toml
To Reproduce

Create an empty cargo project
Write the following rustfmt.toml:unstable_features = true
merge_imports = true

Invoke rustfmt

$ cargo fmt
Warning: can't set `merge_imports = true`, unstable features are only available in nightly channel.
Warning: can't set `unstable_features = true`, unstable features are only available in nightly channel.

$ rustfmt src/main.rs
Warning: can't set `merge_imports = true`, unstable features are only available in nightly channel.
Warning: can't set `unstable_features = true`, unstable features are only available in nightly channel.

Interestingly, I cannot override this on the command line:
$ rustfmt src/main.rs --config unstable_features=true
Warning: can't set `merge_imports = true`, unstable features are only available in nightly channel.
Warning: can't set `unstable_features = true`, unstable features are only available in nightly channel.

If I change rustfmt.toml to simply be
merge_imports = true

I get
$ rustfmt src/main.rs --config unstable_features=true
Warning: can't set `merge_imports = true`, unstable features are only available in nightly channel.

However, with an empty rustfmt.toml, the following works:
$ rustfmt src/main.rs --config unstable_features=true --config merge_imports=true

So it looks like perhaps this is associated solely with loading the config?
Expected behavior
When unstable_features = true is set in rustfmt.toml, rustfmt should accept and act on unstable options like merge_imports without producing warnings.
Furthermore, a command-line specification of --config unstable_features=true should enable unstable features when loaded from rustfmt.toml
Meta

rustfmt version: rustfmt 1.4.15-stable (530eadf4 2020-06-02)
From where did you install rustfmt?: rustup
How do you run rustfmt: primarily cargo fmt and editor on-save hooks, but see above

This seems potentially related to https://github.com/rust-lang/rustfmt/issues/3872, in that the decision to print a warning or enable the feature should be deferred until after parsing the entire config

Thanks for reaching out @austinglaser. TL;DR - You cannot use unstable features on stable with rustfmt 1.0, and the link you referenced in the issue description is for a different/newer version of rustfmt than the version you are actually using (v1.4.15)
From the Configuring Rustfmt section of the readme (emphasis mine):

Configuration options are either stable or unstable. Stable options can always be used on any channel. Unstable options are always available on nightly, but can only be used on stable and beta with an explicit opt-in (starting in Rustfmt v2.0).
Unstable options are not available on stable/beta with Rustfmt v1.x.

When looking at the configuration page, make sure to use the version selection dropdown to pick the version of rustfmt you are using.
Direct link to config docs for v1.4.15
https://rust-lang.github.io/rustfmt/?version=v1.4.15
Direct link to unstable_features option in v1.4.15
https://rust-lang.github.io/rustfmt/?version=v1.4.15#unstable_features

Ah, I missed the "starting in rustfmt v2.0." Thanks very much!

Since rustfmt 2.0 will not be available via rustup/to the general developer population anytime soon, would you consider enabling something like this for the 1.x releases? Maybe using something like RUSTC_BOOTSTRAP/#4884 in addition to unstable_features = true to make it even more opt-in?
I'll be willing to contribute to the implementation of such feature, since there are multiple unstable features I'd like to use for my own code, and I don't mind occasional breaking changes in formatting.

@Luthaf thank you for your willingness to contribute, but no this isn't going to happen (at least not any time soon) for the reasons I described here; it's not a technical/bandwidth matter

OK, thanks for the answer. I see where the teams are coming from, but for me that mean using rustfmt is harder for me. The default output have less than ideal for my code since the first version. Nowadays, the unstable configuration options are enough for me to consider using it again; but I can not move the whole team on nightly just to be able to use the formatter.
I guess the other option is to work toward stabilizing the options I would like to use. Is there any way I can help to make them stable faster?

@calebcartwright Sorry to necrobump, but the mythical rustfmt version 2.0 seems not to be released as of March 2022. Is it possible to estimate how long is it going to finally release 2.0 to the public?


Refactored DLIO Profiler to DFTracer

[x] Change DLIO Profile APIs to DFTracer
[ ] Update Docs
[ ] Check pip installation
[ ] Test DLIO Benchmark

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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 0.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
0

Relevant Lines:
0

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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 0.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
0

Relevant Lines:
0

💛  - Coveralls


Hong Kong and Singapore are unreachable

Although the codes for Hong Kong ("hk") and Singapore ("sg") are sent, they do not appear on the map. Is this a bug?

the codes for Hong Kong ("hk") and Singapore ("sg") are sent

Not sure what you mean by that. I can see that there's no data for 'HK' or 'SG' in our country data: https://github.com/yanivam/react-svg-worldmap/blob/main/lib/src/countries.geo.ts
Unfortunately this data is downloaded from NASA, so there's no way for us to update this ourselves. TBH, Hong Kong and Singapore are so small that they would be literally inaccessible anyway. (Other mini-country/regions are missing as well, like Vatican City and Liechtenstein.)


Add minimum bid amount to add and withdraw bid

CHANGELOG:

Added minimum_bid_amount to core config in chainspec which specifies the minimum bid amount in motes
Changed add_bid function to return error if the staked amount is under the minimum amount
Changed withdraw_bid function to completely unbond a validator is their updated stake drops to strictly less than the min bid amount
Changed add_bid from u32 to u64

Closes #4834

bors r+

bors r+


React Local Forage driver

Does LocalForage have a driver for react?

As mentioned in https://github.com/localForage/localForage/issues/897#issuecomment-519235153, there's no need for a special React driver. localForage is just regular old JS and can be used in React components, in a React app, and in accompanying code. You don't need anything extra for it to work in React. 🙂


Unexpected token '<', "<!DOCTYPE "... is not valid JSON

As shown in the screenshot, I recently ran into this issue trying to auto-load subtitles from Youtube. It was working before, so I'm not sure what happened that would cause this, because I didn't mess with any settings prior to this.

@sop61 It would be helpful to get a screenshot of your console logs and network tab in the developer console when this issue happens. Also, a link to the video where you're seeing it would be great.

Sorry about that. Here's a link to the vid: https://www.youtube.com/watch?v=jDW85zELIL0

Corroborating this bug report on Chrome Version 116.0.5845.141
This happens on every Youtube video. Netflix and Amazon are fine.

I'm also seeing this on YouTube videos.

@sop81 I think this was mentioned on Discord as well, but ad blockers might be causing this. asbplayer needs to make some additional requests to YouTube to obtain the subtitle tracks, which may be blocked by ad blockers.

That seems to be the case for my situation. I wonder if there's a way I can somehow whitelist or enable the parts that involve asbplayer, without disabling adblocking...

I don't use ad blockers so I do not know. asbplayer needs to make a number of requests to YouTube to obtain subtitle tracks. So the ad blocker would need to allow those requests through.

yeah thank you killergerbah for mentioning it, I disabled ublock origin and it worked, unfortunately, I am still figuring out how to allow adblockers to allow requests

I ended up just switching off my filters one by one to check which ones are blocking it. Don't know if there's a better way than this, as others may have set their ad blocker different from mine. For those also wanting to mine from youtube and are running into this issue, just check your ad blocker's settings and filters. Or you can just turn off your ad blocker during the time you're mining, and turn it back on when you're done.
In any case, the issue is not with asbplayer, so I'll close this.

Hello, I realize this was closed but I found a solution. By inspecting the HTTP requests upon opening the subtitle selection prompt, I discovered that adding the following url to your list of trusted sites in ublocks will resolve this issue while seemingly maintaining adblock functionality.
https://www.youtube.com/api/stats/

Thanks @jam-madrigal !


Distributor

Pull Request Test Coverage Report for Build 58

35 of 35 (100.0%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 50:
0.0%

Covered Lines:
239

Relevant Lines:
239

💛  - Coveralls

Pull Request Test Coverage Report for Build 58

35 of 35 (100.0%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 50:
0.0%

Covered Lines:
239

Relevant Lines:
239

💛  - Coveralls


Update about.html

added objectives, recommended by teacher

also I would suggest making this repository private to avoid potential troubles....


No emitter for node: :LINE

This is related to https://github.com/mbj/unparser/issues/3, but now that https://github.com/whitequark/parser/issues/89 is out, actual nodes are (optionally) created for __LINE__ and __FILE__, and unparser doesn't know how to handle them.

Right now unparser expects to use the "default" AST of parser. But I agree we should handle the "meta nodes" also.

Using the default AST from parser, __FILE__ is parsed as a string literal that equals either the original source filepath (as passed to parse), or '(source)' if no path is provided.
In neither case does Unparser unparse that string literal back into __FILE__.  Is that the expected result?
I see that the original code for #3 was removed here 54d0415c437a24385c30aaa542c11c10375e889e but it wasn't obvious to me if that was replaced with something else that achieved the same results, or if the expected functionality changed?
Thank you!

Using the default AST from parser, __FILE__ is parsed as a string literal that equals either the original source filepath (as passed to parse), or '(source)' if no path is provided.

correct. This is the default behavior of parser.

In neither case does Unparser unparse that string literal back into __FILE__. Is that the expected result?

For the default AST produced by parser yes. Unparsers goal is to produce source that when parsed by parser is evaluating into exactly the same ast when parsed (excluding source maps).
Technically speaking unparser wants to infinitely retain the property of parse(source) = parse(unparse(parse(source))) at arbitrarily deep nesting of parse/unparsing, while using parsers defaults.

I see that the original code for #3 was removed here 54d0415 but it wasn't obvious to me if that was replaced with something else that achieved the same results, or if the expected functionality changed?

It was removed because it was unused in my projects, also it had no specs. My opensource policy states I remove everything that is not used by my own projects to reduce maintenance burden. Unless someone volunteers to maintain this feature at an acceptable quality level.
The expected functionality of unparser changed. Also because parser at the time the removal code was made added support to emit AST for __FILE__ and __LINE__ explicitly via an option. So it was not "default" anymore.
I maybe have to change using parser in non default mode (emitting unexpanded nodes) for my mutant project, which will than create a need for unparser to support it also. But my OSS time is to limited to do this short term.

Also the reason for removal was: Source location aware. Unparser should never use these.

Cool, thanks for explaining all of that. No complaints here.
Very new to this project, so my naive understanding was that unparse(parse(source)) ~ source. (i.e. roundtrips yield equivalent (but not necessarily identical) code).
However, as you've pointed out, parse(source) == parse(unparse(parse(source))) is the actual technical goal, which is not quite as stringent as unparse(parse(source)) ~ source.  One is roundtrips starting with AST, the other is roundtrips starting with source.
__LINE__ and __FILE__ are obvious blockers roundtrips with source, which presumably can be solved by running parser with the non-default emit.. option and by Unparser knowing how to emit those meta nodes.
Of course, there may be many other blocks too that aren't as obvious, so I understand now it might be infeasible (or at least unreasonable) to expect Unparser to do that. Again, no complaints from me, just trying to understand. Thanks.

Very new to this project, so my naive understanding was that unparse(parse(source)) ~ source. (i.e. roundtrips yield equivalent (but not necessarily identical) code).

When you say unparse(parse(source)) ~ source note that the ~ operator is effectively telling the "semantics" of the source should be equivalent and as the AST encodes these semantics to remove the ~ operator and use the == operator I added a layer to compare the AST in my example.

LINE and FILE are obvious blockers for roundtrips with source

Not under the ~ operator as I explained above.

Of course, there may be many other blocks too that aren't as obvious, so I understand now it might be infeasible (or at least unreasonable) to expect Unparser to do that. Again, no complaints from me, just trying to understand. Thanks.

I'm open to add support for the non default AST, as I need to do it some day for my other projects. I'd accept a contribution adding emitters for :__LINE__ and :__FILE__. Would not be that complex.

Suppose source is puts __LINE__ if true. Originally, that outputs "1".
However, when parsed and unparsed, the puts gets moved to line 2:
if true
  puts "1"
end

That's fine, but should the unparsed code output "1", or should it output "2"? Which is more equivalent to the intent of the original code?
There isn't an automated way to know the "intent" of the original code.  Naively, I would have expected "2" to be more equivalent to the original code, but I realize now that's not necessarily true --- at the very least it's subjective.
Effectively, the configurable parser option allows the user to specify how to interpret the "intent", so I can definitely understand why Unparser behaves as it does for the default output. Thanks for bearing with me as I grapple with this.

I'm open to add support for the non default AST, as I need to do it some day for my other projects. I'd accept a contribution adding emitters for :LINE and :FILE. Would not be that complex.

Great! I'll see what I can do.


ngx-logger config not getting shared for MFEs  ?

Hi @manfredsteyer
I am trying to use ngx-logger in my project, I have provided the config in the main app.module and the config is applied to only main app and not any other MFE.
LoggerModule.forRoot({
level: NgxLoggerLevel.LOG,
serverLogLevel: NgxLoggerLevel.LOG,
serverLoggingUrl: ('/ui/log'),
serverCallsBatchSize: 50,
serverCallsTimer: 5000,
enableSourceMaps: true
}, {
serverProvider: {
provide: TOKEN_LOGGER_SERVER_SERVICE, useClass: ServerCustomisedService
}
}),
its creating a new instance and not using the one added in shell.
How do you recommend I share ngx-logger so each MFE can reference the same service instance?
'ngx-logger': {
singleton: true,
strictVersion: true,
requiredVersion: deps['ngx-logger']
},
I have provided the library like this in my main app webpack.config.ts and same in my MFE app webpack.config.ts also, in

both I have provided.

this is how it is mentioned in docs...I have done similar.
module federation: 15.0.3
angular : v15

Can you please provide a reproduction?


Fix hub checkout <PULLREQ-URL> for renamed fork

Start reading the Head.Repo.Name property from API payload to determine the name of the forked repo which might not match the name of the base repo.
Fixes #768

:+1:

You beat me to it :)


Add support for Ioke

(Original issue 465 created by samaaron on 2010-01-05T21:29:17+00:00)
It would be lovely if you could add support for the programming language Ioke (http://ioke.org).
To help with this, I've implemented a lexer which you can find attached.

(Original issue was assigned to thatch)
(Original issue 465 last updated on 2010-08-22T10:06:16+00:00)
(Issue automaticaly closed due to status in Bitbucket: resolved)

(Original comment by gbrandl on 2010-08-22T10:06:16+00:00)
Merged in main.

(Original comment by thatch on 2010-05-06T00:28:45+00:00)
Thanks for the patch.  It's integrated in several commits, ending in [0773c93d2db3].
I did have to reformat several of the lines because they went past 80 columns, feel free to verify that I didn't break anything.  I did edit one line for correctness ("Regex|Regex Match" is in the wrong order since space matches the negative lookahead that follows).  I spot-checked the other regexes and didn't find anything, but will try to give a full review later.
I think I found a bug in that ::: is highlighted as a symbol rather than operator.  Feel free to open another bug if that's the case, so we can track it.

(Original comment by samaaron on 2010-01-05T21:36:23+00:00)
Just as a side note, an example of the lexer in action is to be found here:
http://sam.aaron.name/2009/12/23/pygmentising-ioke.html
It is also being used for the main Ioke guide here:
http://ioke.org/wiki/index.php/Guide


Sickbay test_bigtable_write on Dataflow runner

This is currently very flaky (bordering on permared) and represents a known issue with the bigtable client. This will hopefully make our python postcommit a bit less flaky (though it likely won't eliminate flakiness entirely)
Run with this sickbayed - https://github.com/apache/beam/actions/runs/6806746813
Part of #29076

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[x] Update CHANGES.md with noteworthy changes.
[x] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

Run Python 3.8 PostCommit

Currently python postcommit is pretty broken on Actions, so this doesn't fully fix things, but it should help.

assign set of reviewers


In Preview, the toolbar causes a crash.

Description
I recently found a bug related to toolbar.

Steps to reproduce
This is the code that is causing the bug.
struct ContentView: View {
    @State var isPresented: Bool = false

    var body: some View {
        NavigationStack {
            VStack {

            }
            .toolbar {
                ToolbarItem(placement: .navigationBarTrailing) {
                    Button {
                        isPresented = true
                    } label: {
                        Text("Click")
                    }
                }
            }
        }

        .sheet(isPresented: $isPresented) {

        }
    }
}

If you enter the sheet once with the Button in the toolbar, then use the gesture or button(if sheet has Button to change isPresented) to return and press the Button in the toolbar again, the Preview will cause a Crash.
If you put the Button in VStack without putting it in the toolbar, there was no problem and this looks like there is a bug in the toolbar.
I shared this issue with people around me and tried it with another mac device, another Project, and I had a Preview Crash when I had the same symptoms or first pressed the Button.
I thought it was simply an error in Preview, but it also happened on the actual device.

On iPhone mini 13 (Device Build), at the same point as Crash in Preview, the Button no longer works and the app does not quit.

The case I checked without a bug.

Works fine on iPhone 14 (Simulator Build).

On iPad 9th (Device Build), Destination iPhone and Destination iPad both work fine.

Works fine on mac mini m2 (Device Build - Design for iPad).

Expected behavior
Should not crash.

Environment

swift-driver version: 1.75.2 Apple Swift version 5.8.1 (swiftlang-<IP_ADDRESS>.5 clang-14<IP_ADDRESS>.100)
Xcode Version 14.3.1 (14E300c)
iOS 16.4

This seems like a SwiftUI issue, not an issue with the Swift language or tooling itself. Please repost on the Apple Developer Forums and I am sure you will receive the answer you were looking for.


Camera Intermittently returns single channel image, only solved by reboot

I'm using EasyPySpin with a Flir Blackfly camera which works properly most of the time, but occasionally upon itialization it will only return a depth 1 cv array. The only fix to this has been either to reboot the computer or sometimes by running the MATLAB image acquisition toolbox to open and close the connection. Im on Python 3.8 on Ubuntu 18.04.

Hi @nriaziat !
Could you show me more detail about it? For example, your minimal code and the error message (if it has).
I have to reproduce the same issue.

self._cap = EasyPySpin.VideoCapture(0)
while not self._cap.isOpened():'
    pass
ret, self._full_res_frame = self._cap.read()
        self._frame = cv.resize(self._full_res_frame, None, fx=self._sf, fy=self._sf, interpolation=cv.INTER_AREA)
if not ret:
    break
gray_frame = cv.cvtColor(self._frame, cv.COLOR_BGR2GRAY)

This is the rough outline for how I initialize and get a frame. I believe the gray_frame line raises
'VScn::contains(scn)'
where
'scn' is 1

or something to that effect, complaining about a single channel image. I will confirm tomorrow and paste the exact error.

Hi @elerac, is there any additional information I can provide?

@nriaziat
Could you show me the model name of your camera (e.g., BFS-U3-04S2C-C, BFLY-U3-23S6C-C)?
I guess you use an RGB color camera, right?
Let me show the result of the following printed info in both success and failure cases.
print(ret, self._full_res_frame.shape)

Yes, it is an RGB camera - BFS-U3-13Y3C.
on success, it returns:
True (1024, 1280, 3)
on failure, it returns:
True (1024, 1280)

Hmm, it's weird.
To tell you about my experiment, I also use an RGB camera (I forget the model name, but it differs from yours). This RGB camera always returns an image with depth 1. It means its shape is the same as your failure case, but for me, it's a success case. Because this is just a raw format image that has not applied the demosaicing process (aka debayering) yet. With the demosaicing process, I can get an RGB image.
ret, img_raw = cap.read()
img_bgr = cv2.cvtColor(img_raw, cv2.COLOR_BayerBG2BGR)

The depth of img_raw is 1, and img_bgr is 3.
Because of the above reason, I wonder why your camera returns an RGB image without demosaicing.
I'll try to find out your issue in my environment. But it may keep you waiting because I'm so busy.

Interesting, in the case where the image is not demosaiced, wouldn’t we also expect to see 3x the size on both axes?

Does the both axes you refer to mean the column and row of an image? If so, the answer is NO.
The image size (width and height) is the same before and after applying the demosaicing.
To clarify with Python code.
ret, img_raw = cap.read()
print(img_raw.shape)
>>> (1024, 1280)

then
img_bgr = cv2.cvtColor(img_raw, cv2.COLOR_BayerBG2BGR)
print(img_bgr.shape)
>>> (1024, 1280, 3)

As you see, the width and height are the same, and the demosaicing process adds a new dimension (it's 3).

Thanks for clarifying. Maybe I’ll try putting a try catch for the lack of depth 3 that does the demosaicing if necessary as a bandaid solution for now.

@elerac, going back to this, is there a way to specify the pixel format manually so it will either return BGR or Bayer consistently? I saw some functions that seemed to imply you could do this in the Spinnaker API.


feat(App): Improve sidebar animation

What this PR does 📖

Add slide animation on sidebar for web and mobile

Video Exemple on Web

https://github.com/user-attachments/assets/86ffc9ad-3a07-4643-a0bb-e65f7130c438

Which issue(s) this PR fixes 🔨

Resolve #

Special notes for reviewers 🗒
Additional comments 🎤

@lgmarchi Hello Lucas, on this PR I found the same bug happening in #926

Open in mobile view
Click on create group chat
Create group modal is not visible until  you swipe to the right (like in the video from the example):
https://github.com/user-attachments/assets/79109dba-a018-4257-8676-71826c7100e1

@lgmarchi Hello Lucas, on this PR I found the same bug happening in #926

Open in mobile view
Click on create group chat
Create group modal is not visible until  you swipe to the right (like in the video from the example):
https://github.com/user-attachments/assets/79109dba-a018-4257-8676-71826c7100e1

I would say that, in other PR when you saw error, probably is related with this one. and I know already what it is.

@lgmarchi Hello Lucas, on this PR I found the same bug happening in #926

Open in mobile view
Click on create group chat
Create group modal is not visible until  you swipe to the right (like in the video from the example):
https://github.com/user-attachments/assets/79109dba-a018-4257-8676-71826c7100e1

Fixed.

Hey Lucas so on mobile this is working pretty good for me but on web browser the navBar seems to be missing. If you close the sidebar the navBar will appear but only Files Chats and Friends will appear

@phillsatellite For me is working pretty good. I realized, for some reason, your app is cutting in the end.
https://github.com/user-attachments/assets/b611e158-62f3-457a-bb66-07822ac551a4
Did you test in other browser?

@lgmarchi Hey Lucas, I am able to see the same bugs reported by Phil on my end too:

Regular desktop browser resolution fails to show the navbar
Ipad resolution on desktop browser fails to show the navbar
Mobile resolutions (iphone 12, pixel 4) on desktop browser show correctly the navbar

Also, when hidiing the sidebar, the navbar appears at the left is missing settings button

Problem was css height, I changed the property, let's see if it will solve the problem.


JSON POST REQUEST

Hi There, I am very new to obj-c and looking to understand what I am doing wrong here. I am sending a JSON Request to the Server
-> If I do a GET it works
-> When I do a post with parameters I  get the error invalid form data in IOS 5, in 4.3 i had no issues.
here is my code
    NSURL *url = [NSURL URLWithString:@"http://mysite.com"];
    AFHTTPClient *httpClient = [[AFHTTPClient alloc] initWithBaseURL:url];

    NSDictionary *parameters = [NSDictionary dictionaryWithObjectsAndKeys:
                            usernamestring, @"login[username]",
                            emailstring, @"login[email]",
                            nil];

NSMutableURLRequest *request = [httpClient requestWithMethod:@"POST" path: nil parameters:parameters];
// ------------I think the issue is here with my parameters they seem not to being passed--------//
AFJSONRequestOperation *operation = [AFJSONRequestOperation JSONRequestOperationWithRequest:request success:^(NSURLRequest *request, NSHTTPURLResponse *response, id JSON) {
NSLog(@"%@",request);
NSString *status = [JSON valueForKey:@"status"];
NSLog(@"%@",status);
if ([status isEqualToString:@"success"]) {
[username resignFirstResponder];
[password resignFirstResponder];
[self dismissModalViewControllerAnimated:YES];
}
else {
UIAlertView *alert =[[UIAlertView alloc] initWithTitle:@"Login Unsuccessful"
message:@"Try again"
delegate:NULL
cancelButtonTitle:@"OK"
otherButtonTitles:NULL];
            [alert show];
        }

    }
    failure:nil];

    NSOperationQueue *queue = [[NSOperationQueue alloc] init];
    [queue addOperation:operation];
    NSLog(@"check");

AFNetworking uses native JSON support in iOS 5 and uses JSONKit for iOS 4. I guess your JSON response has something unparsable by NSJSONSerialization in iOS 5.
By the way, always preview your post before pressing a button.

My bad had to set the parameter encoding now it works :)


CPU only version of molfeat

Is there an existing issue for this?

[X] I have searched the existing issues and found nothing

Bug description
I've been trying to build some application around molfeat and installing it on different systems like GitHub action runners without GPUs, old/lightweight servers, or other testing environments is currently really difficult due to the reliance on a GPU reliant version of PyTorch.
How to reproduce the bug
No response
Error messages and logs
# Error messages and logs here please

Environment

Current environment
#- Molfeat version (e.g., 0.1.0):
#- PyTorch Version (e.g., 1.10.0):
#- RDKit version (e.g., 2022.09.5):
#- scikit-learn version (e.g.,  1.2.1):
#- OS (e.g., Linux):
#- How you installed Molfeat (`conda`, `pip`, source):

Additional context
No response

Hello @kkovary, installing molfeat on a system without GPU should work (see our CI) for example. Can you share the step you are using ?

Hi @maclandrol thanks for getting back to me. I'm wondering if the issue is arising from my team using poetry to manage dependencies and your team using mamba.

Thanks @kkovary, I will investigate it in the morning.
If you can share the exact error so I can try to reproduce (or find the dependency behind the issue), that would be nice. I can only see torch, since you are not installing the extra dependencies, but I will run some tests.
Can you confirm that removing molfeat from the poetry file above works ?

Hi @maclandrol I stripped down the pyproject.toml file to remove the torch work-arounds that were included above
[tool.poetry]
name = "chem-transformer"
version = "0.1.0"
description = ""
authors = ["Your Name<EMAIL_ADDRESS>readme = "README.md"

[tool.poetry.dependencies]
python = "^3.10"
pydantic = "^1.10.8"
rdkit = "^2023.3.1"
datamol = "^0.10.3"
molfeat = "^0.8.9"

[tool.poetry.group.ci.dependencies]
pytest = "^7.3.1"

[tool.poetry.dependencies.pytest]
version = "^7.3.1"
optional = true

[build-system]
requires = ["poetry-core"]
build-backend = "poetry.core.masonry.api"

These are the errors that I'm seeing in the Github runner that we're using (ubuntu-latest):
 from chem_transformer.datamol_feats import Molecule
/home/runner/.cache/pypoetry/virtualenvs/library-enumerator-qd3v0RhZ-py3.10/lib/python3.10/site-packages/chem_transformer/datamol_feats.py:6: in <module>
    from molfeat.calc import _CALCULATORS, FP_FUNCS, get_calculator
/home/runner/.cache/pypoetry/virtualenvs/library-enumerator-qd3v0RhZ-py3.10/lib/python3.10/site-packages/molfeat/calc/__init__.py:3: in <module>
    from .cats import CATS
/home/runner/.cache/pypoetry/virtualenvs/library-enumerator-qd3v0RhZ-py3.10/lib/python3.10/site-packages/molfeat/calc/cats.py:21: in <module>
    from molfeat.utils.datatype import to_numpy
/home/runner/.cache/pypoetry/virtualenvs/library-enumerator-qd3v0RhZ-py3.10/lib/python3.10/site-packages/molfeat/utils/datatype.py:6: in <module>
    import torch
/home/runner/.cache/pypoetry/virtualenvs/library-enumerator-qd3v0RhZ-py3.10/lib/python3.10/site-packages/torch/__init__.py:228: in <module>
    _load_global_deps()
/home/runner/.cache/pypoetry/virtualenvs/library-enumerator-qd3v0RhZ-py3.10/lib/python3.10/site-packages/torch/__init__.py:189: in _load_global_deps
    _preload_cuda_deps(lib_folder, lib_name)
/home/runner/.cache/pypoetry/virtualenvs/library-enumerator-qd3v0RhZ-py3.10/lib/python3.10/site-packages/torch/__init__.py:154: in _preload_cuda_deps
    raise ValueError(f"{lib_name} not found in the system path {sys.path}")
E   ValueError: libcublas.so.*[0-9] not found in the system path ['/home/runner/work/chem-transformer/chem-transformer/apps', '/home/runner/work/chem-transformer/chem-transformer/apps/library_enumerator', '/opt/hostedtoolcache/Python/3.10.12/x64/lib/python310.zip', '/opt/hostedtoolcache/Python/3.10.12/x64/lib/python3.10', '/opt/hostedtoolcache/Python/3.10.12/x64/lib/python3.10/lib-dynload', '/home/runner/.cache/pypoetry/virtualenvs/library-enumerator-qd3v0RhZ-py3.10/lib/python3.10/site-packages']
=========================== short test summary info ============================
ERROR tests/test_enumerator.py - ValueError: libcublas.so.*[0-9] not found in the system path ['/home/runner/work/chem-transformer/chem-transformer/apps', '/home/runner/work/chem-transformer/chem-transformer/apps/library_enumerator', '/opt/hostedtoolcache/Python/3.10.12/x64/lib/python310.zip', '/opt/hostedtoolcache/Python/3.10.12/x64/lib/python3.10', '/opt/hostedtoolcache/Python/3.10.12/x64/lib/python3.10/lib-dynload', '/home/runner/.cache/pypoetry/virtualenvs/library-enumerator-qd3v0RhZ-py3.10/lib/python3.10/site-packages']

From what I can tell, the error is raised when the Python interpreter is unable to find the libcublas.so library in the system path. This library is a part of the CUDA toolkit and is required by PyTorch for GPU-accelerated operations.
The error occurs when PyTorch is being imported in the molfeat package, which is a dependency of the chem_transformer package we're developing. PyTorch tries to preload CUDA dependencies, including libcublas.so, but fails to find it in the system path.

Since maybe torch 1.13, some cuda dependencies are downloaded with the pip version of pytorch. Downgrading to an older version of torch could work, but doesn't seem like a long term solution.
I looked around a bit, and there seems to be an history between torch and poetry:

https://github.com/python-poetry/poetry/issues/4231
https://github.com/python-poetry/poetry/issues/6409
https://stackoverflow.com/questions/59158044/poetry-and-pytorch

I am not a poetry user, but can you check if these issues are relevant ?

https://github.com/csia-pme/csia-pme/pull/96
https://github.com/ourownstory/neural_prophet/issues/1229

I will try to make torch optional or add optional dependencies for [cpu-only] or [gpu] in the pyproject file. Let me know if  that can give you enough flexibility.

@jstlaurent any inputs here ?

@maclandrol and @kkovary : My apologies, it's taken me a while to get around to looking at this issue.
I'm not a Poetry expert by a long-shot, so unfortunately I wasn't able to find a good solution to your issue, @kkovary.
You can overwrite the torch variant to select the CPU version in your pyproject.toml file, like so:
[tool.poetry.dependencies]
python = "^3.10"
pydantic = "^1.10.8"
rdkit = "^2023.3.1"
datamol = "^0.10.3"
molfeat = "^0.9.2"
torch = { version = "^2.0.0", source="torch-cpu"}

[[tool.poetry.source]]
name = "PyPI"
priority = "primary"

[[tool.poetry.source]]
name = "torch-cpu"
url = "https://download.pytorch.org/whl/cpu"
priority = "supplemental"

But this will always select the CPU variant, and never the GPU enabled one.
Unfortunately, even Poetry's dependency groups can't help us here, because Poetry takes into account all dependencies, including optional ones, when resolving the package to install. So this will also always select the CPU variant:
[tool.poetry.dependencies]
python = "^3.10"
pydantic = "^1.10.8"
rdkit = "^2023.3.1"
datamol = "^0.10.3"
molfeat = "^0.9.2"

[tool.poetry.group.cpu]
optional = true

[tool.poetry.group.cpu.dependencies]
torch = { version = "^2.0.0", source="torch-cpu"}

[[tool.poetry.source]]
name = "PyPI"
priority = "primary"

[[tool.poetry.source]]
name = "torch-cpu"
url = "https://download.pytorch.org/whl/cpu"
priority = "supplemental"

Even if you run poetry install --without cpu, Poetry will conclude that torch==2.0.1+cpu is the package that satisfies every requirements.
My suggestion to you, as unpleasant as it might be, is to maintain two pyproject.toml: one default for GPU build, and another with the explicit CPU-only version, for you CPU build.
@maclandrol: To solve this upstream of Poetry users, we would probably have to manage which version of torch gets pulled in molfeat proper,  using extras to have users explicitly pick CPU vs GPU options.

@jstlaurent I attempted a pytorch-free version but it was a bad idea overall, so maybe the cpu-only extra tag might be the solution indeed.


Support deploying one ui config to another

When we converge the platform support this will be the cams code as the server platform

Closing as covered by #1792


Transaction method for–adding a non confidential issuance to a transaction

Before there was no option to add non-confidential issuance to the transaction.
After the method AddIssuance is added, it is possible to add issuance to the transaction.
In order to calculate asset/token output script, two new public methods are added in the address package:
ToOutputScript - calculates output script
DecodeType - returnes address type.
It closes #94  .
Please @altafan can you review this?

@sekulicd I just left a comment in the task explaining how we need to change the behavior of this metod to allow using it into the pset package in the future.

For the integration test, we actually need to add some methods to the updater role of the pset package in order to correctly add the issuance outputs to the partial transaction.
After these tasks are defined, I would suggest including them in this PR so we are sure to commit code that is working correctly into master.

It seems it could close #88 could you edit the PR description to reflect this? @altafan


without sudo permission rs-fw-update -l won't work

why?
what am I missing?
a user needs to be added to a group?
uved rules need to be added?

resolved by adding udev rules
previousluy they were autoinstalled fro,m apt repository


Add transform dialect registration

This fixes the openxla-nvgpu plugin.

I don't know exactly what happened here but it works w/o this change after rebasing today.


Mexican Spanish translation

jeje

Thank you so much! While I don't speak Spanish, with a quick check with Google Translate it all seems good!


Support "Default location for new attachments" setting

🚀 The feature, motivation and pitch
In Obsidian settings, there is the "Default location for new attachments" setting under "Files & Links". It would be great if the options like "In subfolder under current folder" are supported.
Alternatives
An alternative solution would be to provide examples of inputs to :ObsidianPasteImg that provides the same functionality as each of the options in the dropdown menu.
Additional context

@tnthi115 we could certainly support that. In the meantime you can always pass a path (absolute or relative to the current working directory), e.g. :ObsidianPasteImg vault/folder/attachments/img.png.

@epwalsh awesome, thanks!

I might take a stab at this one :)


Examples don't work

I run example in browser by clicking provided link. See documentation: https://docs.microsoft.com/en-us/azure/active-directory/develop/v2-oauth2-auth-code-flow#request-an-authorization-code
I copy the code from resulting url and go to next example, https://docs.microsoft.com/en-us/azure/active-directory/develop/v2-oauth2-auth-code-flow#request-an-access-token, and click on run in postman - it imports and opens postman nicely, I open Token Request and paste in my code where it says {{replace-with-code-copied-from-browser}} and submit and get bad response:
{
"error": "invalid_grant",
"error_description": "AADSTS65001: The user or administrator has not consented to use the application with ID '6731de76-14a6-49ae-97bc-6eba6914391e' named 'Tutorial Sample App'. Send an interactive authorization request for this user and resource.\r\nTrace ID: 5d2f20eb-3c04-4e36-9d88-1badacba1700\r\nCorrelation ID: abdc759f-e55d-486f-9f93-c3d87aaaa220\r\nTimestamp: 2019-08-15 23:53:41Z",
"error_codes": [
65001
],
"timestamp": "2019-08-15 23:53:41Z",
"trace_id": "5d2f20eb-3c04-4e36-9d88-1badacba1700",
"correlation_id": "abdc759f-e55d-486f-9f93-c3d87aaaa220",
"suberror": "consent_required"

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

ID: fef99ca3-72bb-2c48-a62b-9ff9c314d72b
Version Independent ID: e58fd3a1-0af8-6e16-3d78-3dc748992593
Content: Microsoft identity platform and OAuth Authorization Code Flow
Content Source: articles/active-directory/develop/v2-oauth2-auth-code-flow.md
Service: active-directory
Sub-service: develop
GitHub Login: @rwike77
Microsoft Alias: ryanwi

@activeNathan Thanks for your feedback! We will investigate and update as appropriate.

Hey @activeNathan  the reason you're getting this error is because you haven't done the admin consent part, assuming you've replaced the client id, tenant id accordingly. the permissions that you're requesting need to have admin consent, or you need to have the proper role to be able to grant dynamic consent.
I suggest reading through : https://docs.microsoft.com/en-us/azure/active-directory/develop/v2-permissions-and-consent
Specifically the sections :
https://docs.microsoft.com/en-us/azure/active-directory/develop/v2-permissions-and-consent#requesting-individual-user-consent
https://docs.microsoft.com/en-us/azure/active-directory/develop/v2-permissions-and-consent#requesting-consent-for-an-entire-tenant
https://docs.microsoft.com/en-us/azure/active-directory/develop/v2-permissions-and-consent#admin-restricted-permissions
https://docs.microsoft.com/en-us/azure/active-directory/develop/v2-permissions-and-consent#using-the-admin-consent-endpoint
https://docs.microsoft.com/en-us/azure/active-directory/develop/v2-permissions-and-consent#troubleshooting-permissions-and-consent
Please let us know if there are anymore questions in regards to this.

Hi Frank, thank you and appreciate your help. I would note however that the
examples appear to self contained as read:
Click the link below to execute this request! After signing in, your
browser should be redirected to https://localhost/myapp/with a code in the
address bar.
https://login.microsoftonline.com/common/oauth2/v2.0/authorize...
https://login.microsoftonline.com/common/oauth2/v2.0/authorize?client_id=6731de76-14a6-49ae-97bc-6eba6914391e&response_type=code&redirect_uri=http%3A%2F%2Flocalhost%2Fmyapp%2F&response_mode=query&scope=openid offline_access https%3A%2F%2Fgraph.microsoft.com%2Fuser.read&state=12345
Try executing this request in Postman! (Don't forget to replace the
code) [image:
Try running this request in Postman]
https://app.getpostman.com/run-collection/f77994d794bab767596d
Why does it, the example/documentation, not mention right there all the
steps needed to make it work if it mentions any, like 'Don't forget to
replace the code'?
On Thu, Aug 15, 2019 at 7:51 PM Frank Hu<EMAIL_ADDRESS>wrote:

Hey @activeNathan https://github.com/activeNathan the reason you're
getting this error is because you haven't done the admin consent part,
assuming you've replaced the client id, tenant id accordingly. the
permissions that you're requesting need to have admin consent, or you need
to have the proper role to be able to grant dynamic consent.
I suggest reading through :
https://docs.microsoft.com/en-us/azure/active-directory/develop/v2-permissions-and-consent
Specifically the sections :
https://docs.microsoft.com/en-us/azure/active-directory/develop/v2-permissions-and-consent#requesting-individual-user-consent
https://docs.microsoft.com/en-us/azure/active-directory/develop/v2-permissions-and-consent#requesting-consent-for-an-entire-tenant
https://docs.microsoft.com/en-us/azure/active-directory/develop/v2-permissions-and-consent#admin-restricted-permissions
https://docs.microsoft.com/en-us/azure/active-directory/develop/v2-permissions-and-consent#using-the-admin-consent-endpoint
https://docs.microsoft.com/en-us/azure/active-directory/develop/v2-permissions-and-consent#troubleshooting-permissions-and-consent
Please let us know if there are anymore questions in regards to this.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/37204?email_source=notifications&email_token=AM5GERASDO7G5MNWILSRXTLQEX22RA5CNFSM4IMCZM5KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4NLZ7A#issuecomment-521846012,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AM5GERAYPS2H7DNM57MN4JTQEX22RANCNFSM4IMCZM5A
.

--
Best,
Nathan Haley
Lead Developer
(913) 544-2302
www.ActiveLogicLabs.com http://www.activelogiclabs.com/

@activeNathan , hopefully this addressed your issue.  I'm closing this now, feel free to re-open if you need to.  #please-close


Setup config using ssh error

Hi,
I'm trying using ssh instead password login. But when i upload or download then my received error [sftp] ENOENT: no such file or directory, open '~/.ssh/id_rsa'
While i check the id_rsa has exists in directory .ssh
Please tell me how to set up ssh.
Thanks!

use full path, that said I can't get this to work anyways.

@wallacethefmh
Thanks. It worked for me.


Any one can make a new user account

Describe the feature you'd like
Any one who visits the site, i.e., general public, can sign up for an account with full editing privilege.
Describe the benefits this feature would bring to BookStack users
Make it more like Wikipedia and widen use-cases.
Additional context
Hi, I am new to this, did not seem to find any documentation on new user set up, just wanted to suggest this or check that it already is included in the implementation :)
Thank you!

Hi @lilayoung,
This should be currently possible. Simply create a role with the permissions you want for new sign-ups then, in settings, enable registration and set your role as the default.

Thank you! I will try and follow-up.

:+1:  I'll close this for now then. If you have any issues with the above feel free to open a new issue, referencing this one.


LangChain Integration

MAke sure to add tiktoken to https://github.com/Incorta/OpenCopilot/blob/main/service/requirements.txt

We need to add langchain as well, right?


Add highs solver and document functions

This pull request adds the highs solver to the code and documents the functions.

I know this has errors and does not run the highs solver properly because it is not actually set up within pyomo

the second try should run better


Redoing the version placement change.

This is a copy from #226 but hopefully this will trigger a Travis CI build and test.

This change is

Since Travis CI fixed itself and finally tested #226 , this became unnecessary. Closing now.


is_valid: Remove support for arrays

Process ID: is_valid
Describe the issue:
In order for is_valid to be applicable across a dimension of a datacube through a reducer, we cannot fulfil the requirement
is_valid(x = [null,null]) => true, because under the hood the dimension ends up being an array too, so is_valid(x = [null,null]) => [False, False]. Removing support for checking validity of arrays would make our implementation easier. My impression is that the purpose of is_valid is to filter array elements anyways, so don't think this would be very detrimental at all to the utility of this process.
Proposed solution:
Update the schema for x to something like:
"type": [
    "number",
    "boolean",
    "string",
    "object",
    "null"
]

An optional extension would be to also drop support for object.
Additional context:
We've already removed support for arrays and objects from comparisons in https://github.com/Open-EO/openeo-processes/pull/422

Apologies, the labels bug and patch got added automatically, don't mean to suggest this is a bug!


docs: new upgrade doc with copy database

What's Changed in this PR
As the title said.
Closes #1076
Checklist

[ ] Please confirm that all corresponding versions of the documents have been revised.
[ ] Please ensure that the content in summary.yml matches the current document structure when you changed the document structure.
[ ] This change requires follow-up update in localized docs.

The main content is upgrading from v0.7 to v0.8, please change it to from v0.8 to v0.9

Done


Buffer.js error with large json

Hi,
I am having a 1.4GB .csv file, when trying to convert it to json with any-json I get this error:
bash-4.3# any-json big.csv > big.json
buffer.js:495
    throw new Error('"toString()" failed');
    ^

Error: "toString()" failed
    at Buffer.toString (buffer.js:495:11)
    at Object.fs.readFileSync (fs.js:553:33)
    at Object.<anonymous> (/usr/lib/node_modules/any-json/bin/any-json:52:15)
    at Module._compile (module.js:570:32)
    at Object.Module._extensions..js (module.js:579:10)
    at Module.load (module.js:487:32)
    at tryModuleLoad (module.js:446:12)
    at Function.Module._load (module.js:438:3)
    at Module.runMain (module.js:604:10)
    at run (bootstrap_node.js:390:7)
bash-4.3# ls -alh

Looks like we are running into something similar to this: https://stackoverflow.com/questions/42181217/buffer-js-error-in-node-json-too-large
Would switching to https://github.com/dominictarr/JSONStream fix it?
Best,
FB

You might get past this step but you'd probably fail when writing the full JSON file.
I'd try to split the csv into smaller parts before sending it to any-json.

@fabianbaier I think any-json would require more modification than just switching the JSON encoding library to prevent your error.  any-json's strength (in my opinion) is that it allows converting a large number of formats using a consistent interface.  Unfortunately, this also means that it does not give you as much control or as optimized of solution.  For a case like yours, you are probably better off using the underlying fast-csv (which does support streaming) library directly (in conjunction with JSONStream).
That said, if you write a snippet of code that is able handle your file and can provide it here, I will happily look into integrating it into any-json.


Create README.md

as asked for by dan
added link to my version

Please would you add these minesweeper clones to the README
[Minebuddy HTML by dondanhill](https://github.com/dondanhill/mine-buddy-html)
[Minebuddy Canvas by dondanhill](https://github.com/dondanhill/mine-buddy-canvas)

Many thanks

@kleinesfilmroellchen @dondanhill @shiffman I have added both variants to the following pull request: #163

Covered by #163 let me know if anything is missing!


[Feature Request] Update to be compatible with pymongo >= 4.0.0

[Feature Request] Update to be compatible with pymongo >= 4.0.0
pymongo 4.0.0 brought a lot of changes, see https://pymongo.readthedocs.io/en/stable/changelog.html#changes-in-version-4-0
The latest flask-admin code (at time of writing) still uses deprecated functions, such as remove(), insert() and count().
Examples:

remove: https://github.com/flask-admin/flask-admin/blob/3f9bda85b3a6040ee0327474fbf5e3515467b58a/flask_admin/contrib/pymongo/view.py#L365
insert: https://github.com/flask-admin/flask-admin/blob/3f9bda85b3a6040ee0327474fbf5e3515467b58a/flask_admin/contrib/pymongo/view.py#L315
count: https://github.com/flask-admin/flask-admin/blob/3f9bda85b3a6040ee0327474fbf5e3515467b58a/flask_admin/contrib/pymongo/view.py#L254

This means the latest flask-admin is incompatible with the latest pymongo.

When is this going to be pushed to reflect what's pulled by pip?


Get status of VirtualService if available and use that to determine ingress readiness

/fixes #124
DRAFT: blocked because without ObservedGeneration we can't actually know if status is current. istio/istio#28003

/ok-to-test

@arturenault it seems istio/istio#28003 is closed. Are we still blocked here?

We were blocked until the release of 1.8.0, which it looks like just happened yesterday. I'll start the process of consuming it.
Once that's in, we can merge this in and start testing it, but won't be able to enable this by default until istio/istio#28797 is fixed.

OK, sorry for the noise. This is finally ready for review @nak3 @ZhiminXiang  :)

@nak3 @JRBANCEL (since zhimin is oof on parental leave), can I get some eyes on this?

/lgtm


Tag instances with their etcd or kubernetes version

Codecov Report

Merging #1612 into v0.13.x will increase coverage by <.01%.
The diff coverage is 50%.

@@             Coverage Diff             @@
##           v0.13.x    #1612      +/-   ##
===========================================
+ Coverage    25.45%   25.46%   +<.01%
===========================================
  Files           97       97
  Lines         5040     5042       +2
===========================================
+ Hits          1283     1284       +1
- Misses        3613     3614       +1
  Partials       144      144

Impacted Files
Coverage Δ

pkg/model/stack_new.go
33.86% <50%> (+0.17%)
:arrow_up:

Continue to review full report at Codecov.

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


Group results by rule to simplify output

I already implemented this, and it's pretty good :)
I did a groupBy function on the ruleId in SARIF, and the markdown output now groups findings, same thing for suppressed results
This condenses the output in a better format for human reading, I think it's worth integrating by default
I'll make the PR for it, tell me what you think @azu

Sweet!
I've merged PR and released https://github.com/azu/security-alert/releases/tag/v1.7.0