Add Overflow Property on Property Layout

Explain the bug
When right clicking on a file, there will be a pop up dialog which shows up several action button such as Rename, Properties, etc. However in some cases, the property layout goes off the screen.
To reproduce

Right click on a file, which is located around the middle of your screen
Some buttons such as Rename, Properties, etc are missing

Expected behavior
Adding an overflow property will be great.
Screenshots

System Information

OS Version: Windows 10 21H1 19043.1083
App version: v<IP_ADDRESS>

Thank you for opening this issue, this a bug with the WinUI library.


Add gpt-snippets extension

Description
Allows to create custom GPT snippets and run them on clipboard text.

Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Hi Raycast team. LMK if I need to fix TS errors or we can proceed with them. Thanks!

That’s cool! Can it be part of the ChatGPT extension? It also allows creating items with an initial prompt. These actions fit perfectly into the existing extension, in my opinion
The extension: https://www.raycast.com/abielzulio/chatgpt

That’s cool! Can it be part of the ChatGPT extension? It also allows creating items with an initial prompt. These actions fit perfectly into the existing extension, in my opinion
The extension: https://www.raycast.com/abielzulio/chatgpt

Hey! That could also work, as long as the custom action functionality is there. Are you the maintainer of the ChatGPT extension?

Are you the maintainer of the ChatGPT extension?

I am not but I gonna summon @pernielsentikaer here

@abielzulio @bxb100 what do you think?
I think it's better to enhance the current instead of adding yet another extension 🙂

Nice work @olegkron , I agree with @pernielsentikaer @MarshalX
and the ChatGPT extension's "Summarize website" command can be incorporated into your idea

@olegkron After some work, it works like this

and can use Hotkey to quickly ask in a separate command.

the code lives in https://github.com/bxb100/chatgpt, you may want to see it.

@bxb100 Looks great!
Is it on the way to the Raycast store?

@bxb100 Looks great! Is it on the way to the Raycast store?

No.. this repo currently meets my own need, it's slightly different from the store version, and the code is not tested, so you can pick some logic/code from it, and welcome @ me to code review


Tapkeys

Keys map to different characters depending on if they are tapped or held in combination with another key

Closing because I do not require this feature anymore since I started using a custom keyboard with QMK firmware.


Updates

Manage Auth signature and renewal internally | removes openssl
Manager http host by env var
Feedback based on Callback function
Manage Apns id internally to match to device id incase headers do not contain it.
Make push request async
Uses erlang.mk for build

Hi @softwarejoint , thanks for using our library :)
We can't accept this Pull Request for many reasons. We do not like to have too much important changes in just one PR.
For us, testing is one of the most important parts in software development and you have dropped our tests.
You can try creating PR for each feature you have added but never without tests.


beta7, beta7, beta7!

beta7, beta7, beta7!

beta7

when?

+1 for excitement!

+1 for beta7

+1 every teams catch up progress

+1 am I doing this right?

+1

+1
We are like children waiting for the next toy :d.

Should be there in less than 12 hours according to @DamianEdwards on Twitter: https://twitter.com/DamianEdwards/status/638943636047028224

And it's now live!

beta7! :smiley:


Have the ckpts been fine-tuned w.r.t. the hard-coded architecture?

Hi guys,
I want to use a trained super-net for my experiments. Your RobNet_large_v1_cifar10.pth seems to be an option. However, I am afraid of that you have fine-tuned the super-net w.r.t. the architecture hard-coded in architecture_code.py. I read the README but is still not sure.
Thanks!

Hi Jones, thanks for your interest!
The RobNet_large_v1_cifar10.pth is indeed obtained from a 'fine-tuned' supernet. You could get more details from the original paper in Sec.3.2 'Robustness Evaluation'. This is only applied to the dataset of CIFAR10, only for a matter of convenience, after all, we have done extensive research based on these 'fine-tuned' architectures. But as you could know from the README and the training log provided there, using this code and training the model from scratch can yield better performances. Therefore, as for your experiments, I would like to say if you aim at comparing the robustness of different architectures, the 'scratch-trained' architecture is good since we need to 'unleash' the whole power of the model and train it with a better configuration. If you aim at comparing the search algorithms of NAS, the 'fine-tuned' version is enough. Hope this can answer your question.

Thanks for your reply. I also realized my misunderstanding, since when I replaced the hard-coded architecture (in architecture_code.py) by randomly sampled architectures, I encountered warning---missing keys from checkpoint.
I will follow your suggestion to train a super-net from scratch. However, I don't have such many GPUs, could you provide the checkpoint of your super-net? It will save us a lot of time.

That is the tricky part since the super-net was trained using a private distributed training framework. We have to realign all the performance numbers when transferring it to pytorch official distributed training to make it public. We are currently working on that and it indeed needs some time. We will push it ASAP. Thanks for your understanding!

Got it. we are looking forward to your updates.

@joneswong @gmh14 Are there any updates on this? I was hoping to use a pre-trained super-net to study robustness of different architectures?
I am okay with training it for a few days so is it possible for me to retrain the RobNet-large-v1 provided?


How to segregate the left sidebar as a single component?

For my project, I want to segregate the sidepanel (the photo is attached)  as a single component. I have been exploring the codebase, but I am clueless where and how to start. Please help me where and how can I start [ The folder structures, the dependencies I will need and the components ]
Also, while exploring the components, I found out that the ImageThumbnail.js component (which I will probably need) requires import { utils } from '@ohif/core';. I am confused on how to import while I will generate it as a single component.
Thanks. Hoping to get some responses.

What do you mean can you explain more?
Try viewer-dev.ohif.org instead of viewer.ohif.org
Our viewer.ohif.org is deployed from release branch while viewer-dev.ohif.org is our master  branch
Read more about branch explanations here https://docs.ohif.org/development/getting-started#developing


feat: use typescript definition and write PDF output to http response without file system

thanks!
I told @jonarivado already :)


Feature request: Snapshot function

Issue type
[x] Feature request
[ ] Bug report
[ ] Documentation
[ ] Housekeeping
Description
Snapshot function
It would be nice if a user makes big changes to a prefix or in the box that you can do a snapshot before or, if necessary, this can be done every 15-30 minutes of snapshots from the current table.
Option 2. A 'reset function' of the last change.  Took hours to repair that.

This can be accomplished by backing up the PostgreSQL database which NetBox uses. Such activity is out of scope for the application itself.


Add a self check during the initial task scan

This check allows adding "this" service if it is the local service.
The tests are passing:
https://travis-ci.org/magno32/hazelcast-docker-swarm-discovery-spi/builds/353489044
This commit fixes #11

Thx!

1.0-RC7 is out, thx!


Add time to CI/CD Text

As DefectDojo has the shortcomming of only showing date without time, time is added to the title.
To also add Y/M/D is not sufficent, as it will result in to much text for the title which means I would need to hover to see the exact time.

As a developer I push code multiple times a day which results in multiple scan uploads a day.
In https://github.com/DefectDojo/django-DefectDojo/pull/634/files#diff-deccdc033c69a41ed0f32ca16be7b56e I see that you added created, but it is still naturalday, which means Y-M-D and does not includes the time.
The best case would be to replace naturalday by naturaltime, everywhere in DefectDojo (see https://docs.djangoproject.com/en/2.1/ref/contrib/humanize/).

What do you think about that?

@wurstbrot Makes sense to me. I'll make that change.


HEHA-315 Weather: accept 0 as a temperature

What does this PR do? (please provide any background)
We don't render a temperature when it's 0 because JS evaluates 0 as falsy
What tests does this PR have?
Added a regression test for this in weather-test.jsx
How can this be tested?
npm run docs & check http://localhost:4000/#weather, I've conveniently added an example with 0 to see it working
Screenshots / Screencast

What gif best describes how you feel about this work?

[x] I have checked our general contributing document and the project specific contributing document (if present) and I'm happy for this to be reviewed.

Reviewers
Review 1

[ ] :+1:

Review 2 *

[x] :+1:

Review 3 (optional)

[ ] :+1:

By adding a +1 you are confirming you have...

Witnessed the work behaving as expected (this could be on the author's machine or screencast).
Checked for coding anti-patterns.
Checked for appropriate test coverage.
Checked all the tests are passing.

*  for HX this review must be completed by an SE, SA or Project Guru

Taking :two:

Looking good! :+1:

I suppose so. :+1:


Fix strange naming of kiss emoji (left to do)

"kiss: woman, man" emoji's name should be ":woman-kiss-man:".
But some files were written ":kiss::man:".
I fixed this problem before(#32), but some files were unfixed.
I fix emoji-unicode-{png,svg}.yml.

thank you, released in v3.1.3.


added documentation for the welotec eg500 device

@mlilien I close this PR in order to fix findings. Findings are fixed with #312


[FancyZones] Revisit mutexes

Guarantee thread safety using windows messages, remove mutexes from FZ.
Deadlocks could be a reason of  https://github.com/microsoft/PowerToys/issues/9681, https://github.com/microsoft/PowerToys/issues/10698, https://github.com/microsoft/PowerToys/issues/10671

Resolved with the release of v0.43.0.


Use selection-mode for checkButton

This makes the checkButton bigger, before it was easy to click on the task instead of the checkButton.
Great app, by the way!!!

Yeah, it looks better :)


Web 51

Added location card molecule and used in an organism, template, page
Added temp city images and gulp copy task
Created a WIP css grid layout, possibly it's too 'rigid'?


Fix resizeparams

Fixes #5261
A better solution would be to be able to do what a94b902 author wanted to achieve without breaking resizeparams. But in the meantime, reverting the breakage would be reasonable.

Makes sense. Maybe just fix it right now? It's easy with a CVarList, it has a prop to ignore empty.

This is my first time writing C++ code, take it into account xD

@vaxerski ready

maaan.. doesn't work with maxsize 3

Thank you for your help @bvr-yr
It works fine on my end like this, I don't know what is wrong on your side though.

Ok I got you, It does not work when mixing zeroes and tabs with 3.
Let me fix it, sorry @bvr-yr

that's ok, it just makes no difference, anyway with 0 right params will be returned

And yes, generally 0 maxsize is used

Let me check if it needs sizechecks when using 0

Seems good @vaxerski
Kudos @bvr-yr


When using the dark Microsoft Docs theme, the white text in the first image is difficult to read

Possibly due to the way the text is anti-aliased.

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

ID: 591f8762-9808-22f5-112d-f6d80f337b0e
Version Independent ID: 6731b849-04f3-9bd2-5877-8c2bf26b8f34
Content: Page Background Tasks - Business Central
Content Source: dev-itpro/developer/devenv-page-background-tasks.md
Service: dynamics365-business-central
GitHub Login: @jswymer
Microsoft Alias: jswymer

Hello, thanks for making us aware. We have a lot of incoming requests for fixes and issues, and will for now not prioritize fixing this issue. Maybe when we have more bandwidth. Hope you understand. Thanks.


Replace ktlint with detekt

Context

Contributor Checklist

[ ] Review Contribution Guidelines.
[ ] Make sure that all commits are signed off to indicate that you agree to the terms of Developer Certificate of Origin.
[ ] Make sure all contributed code can be distributed under the terms of the Apache License 2.0, e.g. the code was written by yourself or the original code is licensed under a license compatible to Apache License 2.0.
[ ] Check "Allow edit from maintainers" option in pull request so that additional changes can be pushed by Gradle team.
[ ] Provide integration tests (under <subproject>/src/integTest) to verify changes from a user perspective.
[ ] Provide unit tests (under <subproject>/src/test) to verify logic.
[ ] Update User Guide, DSL Reference, and Javadoc for public-facing changes.
[ ] Ensure that tests pass sanity check: ./gradlew sanityCheck.
[ ] Ensure that tests pass locally: ./gradlew <changed-subproject>:quickTest.

Reviewing cheatsheet
Before merging the PR, comments starting with

❌ ❓must be fixed
🤔 💅 should be fixed
💭 may be fixed
🎉 celebrate happy things

Remaining: pull out actual fixes to a separate PR to make this one simpler to review.

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build


Can't see dropdown text in History time period selection

great them , plan to use it from now on, but problem is with text visibility in History period selection :

the dropdown test is almost invisible, compared to default theme :+1:

It would be nice if the text was darker.
I looked at the theme yaml , but I could not work out which  color variable to change!
rgds, jeff

Can you try 1.4.0-beta.006 and let me know if it fixes the issue? I tweaked the primary, accent, and text colors among other things and I think it should fix your issue.

yep , thanks . fixes my issue with History droplist, I'll close in github
after I've had time to test other screens
rgds
On Tue, 13 Feb 2024 at 13:31, Nerwyn @.***> wrote:

Can you try 1.4.0-beta.006
https://github.com/Nerwyn/material-rounded-theme/releases/tag/1.4.0-beta.006
and let me know if it fixes the issue? I tweaked the primary, accent, and
text colors among other things and I think it should fix your issue.
—
Reply to this email directly, view it on GitHub
https://github.com/Nerwyn/material-rounded-theme/issues/2#issuecomment-1940300388,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALYNARJQXZ7TZX6VGFNKXE3YTLF7RAVCNFSM6AAAAABDFUNT2OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNBQGMYDAMZYHA
.
You are receiving this because you authored the thread.Message ID:
@.***>


Add requireSpaceBetweenArguments rule to relevant presets

Would like to add it to a couple of them, including to "wikimedia"
/cc @Krinkle

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

ping @markelog can you open a PR for this?

Yep, will do that today


mostly formatting and simpler example manifest

I will check thanks.

Wow that's a lot of changes but I learn some new things. Hope to merge this today


refactor: confusing model

After some evaluations of agent.run, this implementation:
https://github.com/callstackincubator/fabrice-ai/blob/main/packages/framework/src/agent.ts#L68C15-L72C16
Which goal is to provide next step of work, can confuse models and prevent using tools as model need to decide if it should use tool or responde with next_step. This can confirm my hypothesis from #132 that model should use tools when needs to analyse/go deeper with problem or responde with regular message / response_format if done his job.

Can you provide a reproducible example that validates the hypothesis? In our examples, we successfully run agents that call tools when necessary, otherwise they finish their job. The instructions to the agents explicitly say that tools should be used in order to complete the task, and we see them being used.
Also, some providers such as Grok use tools exclusively, without response_format (as it is not supported as structured output), which effectively makes it run as you wish.

Maybe I was a little too hasty with this issue because the randomness of this effect is relatively highly random.
i will make little research and try to provide stable example.

No worries, appreciate looking into this. This is still early stage and I think we're looking for the perfect abstraction, so any feedback is welcome!

@tomaszfelczyk on an unrelated note, would you be interested to contribute to some other features, also into Cali? (a different project). I am not sure if that's of your interest, but it's the project that consumes Fabrice and it's been giving me some of the ideas!

@grabbou I am definitely happy to contribute to Fabrice as it directly addresses my needs and has the assumptions I was looking for. I haven't seen Cali, so I'll have to take a closer look.

We will soon be opening Discord server, so I am looking forward to connecting! Meanwhile, please let me know here or via email<EMAIL_ADDRESS>is there anything you'd like to work on first? As a warm-up, maybe you can take some of the easier tasks and we tackle refactor up next!


Add url decoder for cell type heatmap search

I deal with the / character in cell type value in the json endpoint as a request parameter

Tests, please.

Besides this, do we need to change any config files in Tomcat? If that’s the case, could you add the modified file to the docker directory, similar to the tomcat-users.xml file?

Besides this, do we need to change any config files in Tomcat? If that’s the case, could you add the modified file to the docker directory, similar to the tomcat-users.xml file?

@alfonsomunozpomer I am not sure what you mean. The controller is getting an URL encoded parameter and it is encoding it. I think that is all. Did I miss something?

@alfonsomunozpomer I added a test case for the case when the cell type parameter is containing a forward slash. Also improved the other tests (naming and missing assertions).
Could you have another looks, please?
Thanks

Besides this, do we need to change any config files in Tomcat? If that’s the case, could you add the modified file to the docker directory, similar to the tomcat-users.xml file?

@alfonsomunozpomer I am not sure what you mean. The controller is getting an URL encoded parameter and it is encoding it. I think that is all. Did I miss something?

I remember that @lingyun1010 mentioned that maybe we needed to modify Tomcat’s configuration to encode forward slashes. If you think about it we only want to encode forward slashes in one path segment, and that might be problematic. Maybe it’s not and React Router can do that. Whatever the case it should be in a different PR.


Fix OpenAPI -> JSON Schema conversion

Fixes #489
Related: #468

Codecov Report

Merging #490 into master will decrease coverage by 0.40%.
The diff coverage is 77.77%.

@@             Coverage Diff             @@
##            master     #490      +/-   ##
===========================================
- Coverage   100.00%   99.59%   -0.41%
===========================================
  Files           31       31
  Lines         1993     1977      -16
  Branches       312      312
===========================================
- Hits          1993     1969      -24
- Misses           0        4       +4
- Partials         0        4       +4

Flag
Coverage Δ

#unittests
99.59% <77.77%> (-0.41%)
:arrow_down:

Impacted Files
Coverage Δ

src/schemathesis/converter.py
100.00% <ø> (ø)

src/schemathesis/schemas.py
99.27% <75.00%> (-0.73%)
:arrow_down:

src/schemathesis/checks.py
100.00% <100.00%> (ø)

src/schemathesis/extra/pytest_plugin.py
92.85% <0.00%> (-7.15%)
:arrow_down:

src/schemathesis/lazy.py
100.00% <0.00%> (ø)

src/schemathesis/models.py
100.00% <0.00%> (ø)

src/schemathesis/cli/context.py
100.00% <0.00%> (ø)

src/schemathesis/runner/events.py
100.00% <0.00%> (ø)

src/schemathesis/runner/impl/core.py
100.00% <0.00%> (ø)

src/schemathesis/runner/serialization.py
100.00% <0.00%> (ø)

Continue to review full report at Codecov.

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


Does join-monster supports polymorphic relationships?

Hello, thanks for the amazing library. I used join-monster in my project with no troubles, but now I faced with a problem that I need some kind of polymorphic relationship.
I'm trying to use "Exclusive Belongings To" strategy but it seems that join-monster doesn't support this kind of relationships.
As I understand, it could be implemented somehow with batch fetching, but it requires to understand union graphql types.
Details:
My database structure:
teams (id, name)
users (id, name)
team_members (
  owner_id references teams (id),
  team_id references teams (id),
  user_id references users (id)
)

I want to fetch all team members (member can be either a team or a user)

The workaround is to define several fields in my schema. Instead of just members I have to create separate teams and users fields.

This is technically possible, although not easy. Both options will preclude the use of sqlBatch, as that requires that the foreign key be a fixed column. So you'll have to do a 'sqlJoin`. You have two options:

Use a GraphQLUnionType to union the teams and users. This will let you treat them as polymorphic in the API as well.
Or, if you're okay with making the two a single type in GraphQL, create a GraphQLObjectType where the sqlTable is a view combining the two tables together.


Add support to url safe in datastore key

Add support to url safe in datastore key
Sample in Python:
https://github.com/GoogleCloudPlatform/python-docs-samples/blob/master/appengine/ndb/entities/snippets.py#L79

@vyper Please see previous discussion of this topic on GoogleCloudPlatform/gcloud-node#625.

@vyper Based on the discussion above, should we conclude the same here:

it should be figured out by the user, as we're not doing anything unique

and close this issue? Or is there something different/new?

@quartzmo Thanks for posting that URL, I was trying to find some context for urlsafe. I agree with the discussion on that issue. (Also, your google-fu is better than mine. 😄)
I think there might be some benefit for supporting urlsafe for folks transitioning their apps from appengine to compute, but I don't know if it belongs in gcloud-ruby. Perhaps an extension?

Maybe a well-linked gist?

Buy I want to be able to search for it... Hehe. Datastore dont support find or query using urlsafe.

I haven't found much info on decoding the urlsafe data. This answer on SO recommends you decode the urlsafe value using ndb, and then convert it to a Datastore key.
To illustrate, this urlsafe:
"ahNzfnByb21wdGFwcG9pbnRtZW50chULEghTY2hlZHVsZRiAgICD2e7xxxxx"

decodes to the following string:
"j\x13s~promptappointmentr\x15\v\x12\bSchedule\x18\x80\x80\x80\x83\xD9\xEE\xF1\xC7\x1Cq"

That looks like protobuf bytes. And sure enough JJ mentions in the link @quartzmo posted that "What you're looking at is really just a (mostly) base64 encoded version of the entity_pb.Reference protobuf." AFAIK we don't have access to entity_pb.Reference from Datastore, so I don't see how we could support this.

Closing as we currently don't think it is possible. Please reopen if there is a way to identify a Datastore Key from a ndb urlsafe.

@blowmage how I can use this? I not found! ):

Do we use the Datastore Key proto? If so, this should just be the base-64 encoded serialized representation of the Key....

To find without know how key is composite!

@jgeewax I can't find any protobuf definition for ndb Key, and I don't read the encoding well enough to know how to decode it. Any info you could pass our way is appreciated.

Sorry, to be more clear: we populate a Ruby protobuf object (a Datastore
"Key"), right? And serialize that to binary before sending it over the wire
to the v1beta3 API endpoint?
If you pipe that serialized binary through a Base64 encoder (instead of
sending it over the wire), the output should be the key format people are
talking about.

That will only work if the proto definitions of Datastore Key and NDB Key are the same, which they don't appear to be. To illustrate:
urlsafe = "ahNzfnByb21wdGFwcG9pbnRtZW50chULEghTY2hlZHVsZRiAgICD2e7xxxxx"

require "base64"
proto = Base64.urlsafe_decode64 urlsafe

require "google/datastore/v1beta3/entity
key = Google::Datastore::V1beta3::Key.decode proto
#=> Google::Protobuf::ParseError: Error occurred during parsing: Unexpected EOF inside skipped data

Is that key actually valid? It doesn't check out on http://datastore-key.appspot.com/ ...

The urlsafe string you provided does not decode as a v1beta3 key.
urlsafe = "ahZzfmJ1c3RsaW5nLWtheWFrLTkxNTE2chALEgZQZXJzb24iBG5hbWUM"

require "base64"
proto = Base64.urlsafe_decode64 urlsafe

require "google/datastore/v1beta3/entity"
key = Google::Datastore::V1beta3::Key.decode proto
#=> <Google::Datastore::V1beta3::Key: partition_id: nil, path: []>

It also does not decode as a Google Protobuf Any:
require "google/protobuf/any"
Google::Protobuf::Any.decode proto
#=> <Google::Protobuf::Any: type_url: "", value: "">

If you look at the bytes of the urlsafe you provided, they look like this:
urlsafe = "ahZzfmJ1c3RsaW5nLWtheWFrLTkxNTE2chALEgZQZXJzb24iBG5hbWUM"
urlsafe_proto = Base64.urlsafe_decode64 urlsafe
#=> "j\x16s~bustling-kayak-91516r\x10\v\x12\x06Person\"\x04name\f"

If you look at the bytes of the proto generated by v1beta3 they look similar, but are still different:
key = gcloud.datastore.key "Person", "name"
key_proto = Google::Datastore::V1beta3::Key.encode key.to_grpc
#=> "\x12\x0E\n\x06Person\x1A\x04name"

Again, I'm not the best at reading the protobuf bytes, so if there is a .proto file for the NDB Key I'd really like to see it. I believe we can approximate the application id by prepending `s~` to the project id, but I don't know how to parse the rest of the bytes without the NDB Key proto definition.

Right... I think we're dealing with different protos here. In Python we have the old "Reference" proto (https://code.google.com/p/googleappengine/source/browse/trunk/python/google/appengine/datastore/entity_pb.py#1701) which works as expected (in Python at least):
import entity_pb
import base64
key = "ahZzfmJ1c3RsaW5nLWtheWFrLTkxNTE2chALEgZQZXJzb24iBG5hbWUM"
ref = entity_pb.Reference()
ref.ParseFromString(base64.b64decode(key))
print ref

Which spits out: app: "s~bustling-kayak-91516" path { Element { type: "Person" name: "name" } }
@pcostell : How do we deal with AppEngine Datastore serialized keys (like the example here) which uses the entity_pb.Reference protobuf in our Cloud Datastore v1beta3 world?

We are working on defining a new serialized format which is proto version agnostic. I definitely don't think we should try to use the old one we used in App Engine. I'll work on getting that language fixed in the console.

/cc @Alfus @dmcgrath

@pcostell Glad to hear there is a new serialization format coming!
This issue is specifically to support the old App Engine urlsafe strings. I can imaging that maintaining existing URLs is important for folks attempting to move apps from App Engine to ruby. I personally don't think this belongs in glcoud-ruby, but I can see having this functionality in an extension library. That said, we really are stuck until we can get the .proto definition for NDB's Reference/Key. Is it possible for us to obtain that?

This isn't really possibly. Even if you have the proto format, you don't have the application id (as opposed to the project is). I'd rather backport the format into app engine libraries if we need to support a seamless migration from GAE to ruby. At this point, the urlsafe keys are just a format used in 2 libraries, and not a Cloud Datastore-wide feature.

@pcostell Thank you for the clarification! That jives with my understanding. @jgeewax, what do you think?

I agree with Patrick.


WIP! Optimize hot spots in Stream Manager

The performance degradation happened after we switched to use smart pointers in the Heron Stream Manager. I made some profiling to figure out why it happened. This is what I have found.

Logically, unique_ptr(which we mainly used) have no extra cost because it is just a RAII wrapper on top of the raw pointer. Every time we access it, get() method is called to reach the underlying raw pointer. However, the problem is the compiler doesn't inline that wrapper methods (like get()) by default and they become a real functions calls in the release build. A function call has some invocation cost and that has led us to the performance penalty. The solution is to force the compiler to inline small methods. To do it we have to build the release build with the "-O3" flag.

When I was profiling this code I also noticed that std::unordered_map in tuple-cache consumed a lot of CPU and it was actually number 1 in a slow methods list. I replaced it with hopscotch_map which is more efficient hash map implementations. After that tuple-cache disappeared from the hot spots list.

This is the difference in the CPU consumption when we were running the PR with -O3:

Running with current master up until 14:20
Running with --copt=-03 and hopscotch hashmap after 14:20

This chart and testing provided by guys from Heron community.
@Aditya and @rohanag12 thank you very much for your help!


Loguru integration

Exploratory ticket on the use of loguru within snappiershot.
Some topics to cover:

Do we want to add a dependency like this (if not could we have this be an optional "feature")?
How is logging typically done within packages and/or plugins?
What would we be logging within snappiershot? How would users find and interact with these logs?

I'm undecided if logging really makes sense here or not (loguru-based or otherwise); I go back and forth depending on the day
Random thoughts:

if we do add logging, it should be off-by-default & something that package-developers can turn on as needed
things we might want to log:

details about gathering/recording/looking-up snapshot context
details about file reading/writing
details about which custom en/decoders are being used/called

while this could all be useful, i'm currently inclined to say we shoudl hold off and add it when we actually start to feel the need for these things.
loguru vs logging:

loguru:

pros: easy to use, easy for outside developers to quickly learn & add/modify
cons: added dependency, could cause confusion to users

logging:

pros: in standardlib so no added dependency
cons: harder to setup initially, maybe harder for some developers to learn & use

I agree that any logging should be off-by-default and that we should hold off for now until we see for use cases.
Also, I believe that we should use logging over loguru to avoid a potentially heavy dependency.


Failure to parse OEM id causes cascading failures

When migrating a host to Flatcar from Container Linux the failure to parse the embedded OEM id causes metadata setup to fail which then cascades to networking failure, ssh key failure, etc.
In this specific case there was no backwards compatibility to parse and understand namespaced coreos. values from the kernel command line:
Feb 05 18:06:45 redbeard-2gb-1-1 coreos-metadata[682]: Error: initialization
Feb 05 18:06:45 redbeard-2gb-1-1 coreos-metadata[682]: Caused by: Couldn't find flag 'flatcar.oem.id' in cmdline file (/proc/cmdline)
Feb 05 18:06:45 redbeard-2gb-1-1 systemd[1]: motdgen.service: Succeeded.
Feb 05 18:06:45 redbeard-2gb-1-1 systemd[1]: Started Generate /run/flatcar/motd.
Feb 05 18:06:45 redbeard-2gb-1-1 systemd[1]<EMAIL_ADDRESS>Main process exited, code=exited, status=1/FAILURE
Feb 05 18:06:45 redbeard-2gb-1-1 systemd[1]<EMAIL_ADDRESS>Failed with result 'exit-code'.
Feb 05 18:06:45 redbeard-2gb-1-1 systemd[1]: Failed to start Flatcar Metadata Agent (SSH Keys).
Feb 05 18:06:45 redbeard-2gb-1-1 systemd[1]: Dependency failed for Multi-User System.
Feb 05 18:06:45 redbeard-2gb-1-1 systemd[1]: multi-user.target: Job multi-user.target/start failed with result 'dependency'.

this host was booted with the following command line:
Feb 05 18:06:37 localhost kernel: Command line: BOOT_IMAGE=/coreos/vmlinuz-a mount.usr=/dev/mapper/usr verity.usr=PARTUUID=7130c94a-213a-4e5a-8e26-6cce9662f132 rootflags=rw mount.usrflags=ro consoleblank=0 root=LABEL=ROOT console=ttyS0,115200n8 console=tty0 coreos.oem.id=digitalocean verity.usrhash=757b2f0595a625a89b99b6fe7e8b59bd33b3e384fd90f538067a0b7b2bc09822

If the user can successfully boot the host and be able to modify partition 6 of the boot disk (the "OEM" partition) it is possible that the missing metadata expected by the Flatcar tooling can be added as a triage by appending set linux_append="$linux flatcar.oem.id=digitalocean" to the file /usr/share/oem/grub.cfg, that being said this requires advanced knowledge of the OS in general and may not be exhaustive of all changes needed to bring the host into a good working state:
$ cat /usr/share/oem/grub.cfg
# CoreOS GRUB settings

set oem_id="digitalocean"
set linux_append="$linux flatcar.oem.id=digitalocean"

Thanks for reporting, we added a paragraph for that in the documentation for the mean time: https://docs.flatcar-linux.org/os/migrate-from-container-linux/
We notify you here when we have a release where afterburn supports all variable names.

With all due respect, the additional paragraph still doesn't explain how to actually fix the problem and the update page still says nothing about the fact that its instructions don't actually work.

Sorry, yes, the recovery instructions should be on that page until the bug is fixed. The update instructions have the NOTE: General differences when migrating from CoreOS Container Linux also apply. at the beginning but I see that it may be not obvious. The next release will fix the problems and we can then remove this warning.

For the upcoming stable release we have all compatibility changes done except that flatcar.config.url or ignition.config.url are needed as command line arguments instead of coreos.config.url.

Thanks.
We have been fixing numerous issues regarding backward compatibility. So the OEM ID issue would be probably already fixed.
So can you please try out latest releases, Stable 2345.3.0 or Beta 2411.1.0?

Alpha 2430.0.0 or Edge 2430.99.0 should also fix the issues.
If you continue seeing the issue, feel free to reopen the issue.


Doesnt remove generic type

I was expecting that using yarn flow-remove-types deleteme.js --all --pretty -d no-types
given:
/* @flow */

function foo<T> (a: T): T {
  return a
}

const bar = foo<number>(123)

the output would be:
/*  */

function foo (a) {
  return a
}

const bar = foo(123)

but instead i got:
/*  */

function foo (a) {
  return a
}

const bar = foo<number>(123)

Is this the expected behavior?
Is it possible to remove the generic types with this utility?

This will be resolved in the next major release which will upgrade the parser to recognise this syntax.

Closed in #82.

Closing in favour of #89.


Add documentation to tskit.dev

Need to update the CI infrastructure to push documentation to tskit.dev
Needs @benjeffery I think

Dupe of #34


Microsoft.Azure.SignalR.HeartBeat: Value cannot be null. Parameter name: provider

Issue moved from dotnet/aspnetcore#33893

Please respond to @JasonKleban.

From @JasonKleban on Sunday, June 27, 2021 6:35:45 PM
Describe the bug
The seemingly cleaner way to supply configuration values to SignalR is failing at startup with Invalid host services.  ... Lifetime: Singleton, ImplementationType: Microsoft.Azure.SignalR.HeartBeat Value cannot be null. Parameter name: provider
To Reproduce
public class Startup : FunctionsStartup
{
    public override void Configure(IFunctionsHostBuilder builder)
    {
        ...
        // Set SignalR connection string here ...
        builder.Services.AddSignalR().AddAzureSignalR(config["SignalrConnectionString"]);

    }
}

to support:
[FunctionName("negotiate")]
public IActionResult negotiate(
    [HttpTrigger(AuthorizationLevel.Function, "post")]
    HttpRequest req,
    [SignalRConnectionInfo(HubName = "sync")] // ... instead of having to supply it here
    SignalRConnectionInfo connectionInfo )
{
    ...
}

Instead we seem to be having to supply it in a hacky spot:              [SignalRConnectionInfo(HubName = "sync", ConnectionStringSetting = "SignalRConnection")] SignalRConnectionInfo connectionInfo)
Exceptions (if any)
Invalid host services. Microsoft.Azure.WebJobs.Script.WebHost: The following service registrations did not match the expected
 services: [Invalid] ServiceType: Microsoft.Extensions.Hosting.IHostedService, Lifetime: Singleton, ImplementationType:
 Microsoft.Azure.SignalR.HeartBeat Value cannot be null. Parameter name: provider

Seems to be the same issue as this SO poster is having:
https://stackoverflow.com/questions/63284508
Further technical details
.NET SDK (reflecting any global.json):
 Version:   5.0.301
 Commit:    ef17233f86

Runtime Environment:
 OS Name:     Windows
 OS Version:  10.0.19042
 OS Platform: Windows
 RID:         win10-x64
 Base Path:   C:\Program Files\dotnet\sdk\5.0.301\

Host (useful for support):
  Version: 5.0.7
  Commit:  556582d964

.NET SDKs installed:
  1.1.0 [C:\Program Files\dotnet\sdk]
  2.0.0 [C:\Program Files\dotnet\sdk]
  2.1.3 [C:\Program Files\dotnet\sdk]
  2.1.202 [C:\Program Files\dotnet\sdk]
  2.1.524 [C:\Program Files\dotnet\sdk]
  3.1.116 [C:\Program Files\dotnet\sdk]
  5.0.204 [C:\Program Files\dotnet\sdk]
  5.0.301 [C:\Program Files\dotnet\sdk]

.NET runtimes installed:
  Microsoft.AspNetCore.All 2.1.5 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.All 2.1.26 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.All 2.1.28 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.App 2.1.5 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 2.1.26 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 2.1.28 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 3.1.16 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 5.0.7 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.NETCore.App 1.0.5 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 1.1.2 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.0.0 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.0.4 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.0.9 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.1.26 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.1.28 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 3.1.16 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 5.0.7 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.WindowsDesktop.App 3.1.16 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
  Microsoft.WindowsDesktop.App 5.0.7 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]

@JasonKleban Function binding has its own logic to resolve connection string and it's not working with Azure SignalR server SDK. So this is not correct to add this builder.Services.AddSignalR().AddAzureSignalR().
The correct way to use customize key names is like below:
[FunctionName("negotiate")]
public IActionResult negotiate(
    [HttpTrigger(AuthorizationLevel.Function, "post")]
    HttpRequest req,
    [SignalRConnectionInfo(HubName = "sync", ConnectionString="abc")]
    SignalRConnectionInfo connectionInfo )
{
    ...
}

And the value can be set in Configuration settings or even KeyVaults refer to this.
{
    "abc":"Endpoint=xxx;AccessKey=xxx;Version=1.0;
}

Besides this basic way of ConnectionString support, as Azure SignalR service function extension is working with Microsoft.Azure.SignalR.Management, we also supports resolving these configuration from some default naming. Please see sample for some further information.

Thank you!  This can be closed.


Fix the Errorf statements

The Errorf statements were missing the %s and was outputting errors that looked like this:
* godaddy_domain_record.main: couldn't find domain record: %!(EXTRA string=[404:UNKNOWN_DOMAIN] The given domain is not registered, or does not have a zone file)

I also dumped the domain name into the error logs. I can remove that if needed. I could not get the debug info from the log.Println() statements when running directly from Terraform.

Thanks @polds!


Publish declaration maps so goto definition works properly

👋
I was trying to use this library a bit and understand the the types/source. It is a bit hard in VSCode because jump to definition takes you to a TS declaration file or a compiled JS file today.
This PR adds the TS compiler option to publish declaration maps which fixes this issue, making developer experience better: https://www.typescriptlang.org/tsconfig#declarationMap
It would maybe be nice to cut a release with these enabled if its not a hassle so they actually get published.

commented on the other pr in Island here: https://github.com/hack-dance/island-ai/pull/47#issuecomment-2041268478
There are some issues with creating declaration maps with tsup - going to close, but if you want to open an issue or take a stab at figuring out a reasonable workaround, happy to take a look - but this is going to require some build updates to get working


Issue #412: add session middleware

This is a draft PR - adding session middleware. It addresses #412 .

Requirements

from cryptography.hazmat.primitives.ciphers.aead import AESGCM raises InvalidTag exception when the MAC validation fails. Decryption block should handle this exception. Clear the session if the exception is raised.
Need one method to validate max-age of cookies. Clear the session if expired.

@infohash thanks for the testing - im sending you an invite to the organization. It will give you the ability to push to the branch. You will note I started testing - one test is failing. If you feel like puhsing most tests, fixing this up etc. It would be nice. Also you other fixes

Requirements

cryptography.hazmat.primitives.ciphers.aead import AESGCM raises InvalidTag exception when the MAC validation fails. Decryption block should handle this exception. Clear the session if the exception is raised.
Need one method to validate max-age of cookies. Clear the session if expired.

yup. Do you want to take care of this? Also tests are partial. I can continue working on this tomorrow - but feel free to push commits to this branch.

Thank you so much for the invite. I'll take care of the rest of the test cases and requirement.

Another thing, the additional_data will contain 2 things.

The timestamp of when the session was encrypted so that we can verify when the cookie was set.
Max-age of the cookie to verify its validity from the time of when it was set.

Additional Data in AEAD is only required to be integrity protected, not encrypted so we can append AD to the ciphertext.
Additionally, we can put cookie path in AD.

@peterschutt I'll start writing examples and update the documentation for it. The requested review is just for the code changes.

Ok, this PR is ready to ship basically. We need documentation but thats it.

Okay, I will raise a separate PR for example and doc.

@all-contributors add @infohash code


Produces an error when offline

Visual Studio Code Version : 1.5.1
Code Sync Settings Version : 2.2.3
Operating System : OS X (El Capitan)
@shanalikhan thanks for the nice plugin!
Every time I launch VS Code when I'm offline, I get this error:
2//Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/workbench.main.js:118 [Extension Host] Error: getaddrinfo ENOTFOUND api.github.com api.github.com:443
    at global.sendError (/Users/fronteed/.vscode/extensions/Shan.code-settings-sync-2.2.3/node_modules/github/lib/index.js:783:19)
    at /Users/fronteed/.vscode/extensions/Shan.code-settings-sync-2.2.3/node_modules/github/lib/index.js:792:29
    at callCallback (/Users/fronteed/.vscode/extensions/Shan.code-settings-sync-2.2.3/node_modules/github/lib/index.js:667:17)
    at ClientRequest.<anonymous> (/Users/fronteed/.vscode/extensions/Shan.code-settings-sync-2.2.3/node_modules/github/lib/index.js:739:17)
    at emitOne (events.js:90:13)
    at ClientRequest.emit (events.js:182:7)
    at TLSSocket.socketErrorListener (_http_client.js:295:9)
    at emitOne (events.js:90:13)
    at TLSSocket.emit (events.js:182:7)
    at connectErrorNT (net.js:996:8)
    at _combinedTickCallback (internal/process/next_tick.js:74:11)
    at process._tickCallback (internal/process/next_tick.js:98:9)(anonymous function) @ //Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/workbench.main.js:118
//Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/workbench.main.js:93 Error Logged In Console (Help menu > Toggle Developer Tools). You may open an issue using 'Sync : Open Issue' from advance setting command.

Can it be fixed, please?

I have pushed an update 2.2.6
Can you check and let me know

Error is successfully disappeared, thanks!

re-opened! i will fix this soon again :)

its fixed in 2.3.3


set StackNavigator header title height

(Originally mentioned in #480 but I'm creating this because it's a separate issue)
Hi, how can I set the height of the stacknavigator Title component? I can change the height of the header just fine (see #480) but anything inside the Title component is cropped below the header's normal height.

Hi there! This issue is being closed because it has been inactive for a while. We're closing issues after a period of inactivity. Please do not take this personally :)
If you think this issue should remain open, please let us know. The following information is helpful when it comes to determining if the issue should be re-opened:

Does the issue still reproduce on the latest release? Post a comment with the version you tested.
If so, is there any information missing from the bug report? Post a comment with all the information required by the issue template.
Is there a pull request that addresses this issue? Post a comment with the PR number so we can follow up.
If you would like to work on a patch to fix the issue, contributions are very welcome! Read through the contributors guide, and feel free to hop into #react-navigation if you need help planning your contribution.


When dataBin is not defined it should not return anything

Related ticket: https://github.com/CartoDB/cartodb/issues/11798
Steps to reproduce already described there ^^^. But good ones also are:

Create a map with a histogram widget where the there is at least a bucket without data.
Example:

Select the bin or bins without data and zoom into it/them.
Example:

JS error appeared, now zoomed histogram should be empty.

Tested, looking good 👍


KeyError: 'IngressApplicationGateway' - when installing AGIC add-on with a Service Principal AKS cluster

If the issue is to do with Azure CLI 2.0 in-particular, create an issue here at Azure/azure-cli

Extension name (the extension in question)
aks-preview
Description of issue (in as much detail as possible)
Issue:
Getting error while creating AKS Cluster with AGIC enabled, when Service Principal is used.
Following https://docs.microsoft.com/en-us/azure/application-gateway/tutorial-ingress-controller-add-on-new
From the tests,
it seems that the AZ CLI is having issues when enabling AGIC Add-on with AKS Cluster (either new or existing) with a Service Principal (either new or existing).
AGIC Add-on has no issues with MSI AKS Cluster.
Error:
'IngressApplicationGateway'
Traceback (most recent call last):
File "…\cli.py", line 215, in invoke
File "…_init_.py", line 654, in execute
File "…_init_.py", line 717, in run_jobs_serially
File "…_init.py", line 710, in run_job
File "…\six.py", line 703, in reraise
File "…_init.py", line 688, in run_job
File "…_init.py", line 325, in call
File "…_init_.py", line 784, in default_command_handler
File "….azure\cliextensions\aks-preview\azext_aks_preview\custom.py", line 1134, in aks_create
_add_ingress_appgw_addon_role_assignment(created_cluster, cmd)
File "….azure\cliextensions\aks-preview\azext_aks_preview\custom.py", line 741, in _add_ingress_appgw_addon_role_assignment
config = result.addon_profiles[CONST_INGRESS_APPGW_ADDON_NAME].config
KeyError: 'IngressApplicationGateway'
CLI Version:
{
"azure-cli": "2.16.0",
"azure-cli-core": "2.16.0",
"azure-cli-telemetry": "1.0.6",
"extensions": {
"aks-preview": "0.4.60"
}
}
Also happens with CloudShell

aks-preview

@akshaysngupta

@axelgMS can you please update the aks-preview extension using az extension update -n aks-preview and see if you still see the error ?


Machine learning software setup error

Hi team,
I followed your instructions on your e manual to install tensorflow and keras on my ROS kinetic PC. However
e manual
TurtleBot3 49 Machine Learning tutorial 1
However, when I run import tensorflow as tf I get the following errors. I am setting up this course for my students. Could you please update your e manual and make it reliable? Thanks a lot.
`mario@mario:~/Downloads$ python
Python 2.7.15 |Anaconda, Inc.| (default, May  1 2018, 23:32:55)
[GCC 7.2.0] on linux2
Type "help", "copyright", "credits" or "license" for more information.

import tensorflow
Traceback (most recent call last):
File "", line 1, in
File "/home/mario/anaconda2/lib/python2.7/site-packages/tensorflow/init.py", line 24, in
from tensorflow.python import pywrap_tensorflow  # pylint: disable=unused-import
File "/home/mario/anaconda2/lib/python2.7/site-packages/tensorflow/python/init.py", line 47, in
import numpy as np
File "/home/mario/anaconda2/lib/python2.7/site-packages/numpy/init.py", line 142, in
from . import core
File "/home/mario/anaconda2/lib/python2.7/site-packages/numpy/core/init.py", line 91, in
raise ImportError(msg.format(path))
ImportError: Something is wrong with the numpy installation. While importing we detected an older version of numpy in ['/home/mario/anaconda2/lib/python2.7/site-packages/numpy']. One method of fixing this is to repeatedly uninstall numpy until none is found, then reinstall this version.
import tensorflow as tf
Traceback (most recent call last):
File "", line 1, in
File "/home/mario/anaconda2/lib/python2.7/site-packages/tensorflow/init.py", line 24, in
from tensorflow.python import pywrap_tensorflow  # pylint: disable=unused-import
File "/home/mario/anaconda2/lib/python2.7/site-packages/tensorflow/python/init.py", line 47, in
import numpy as np
File "/home/mario/anaconda2/lib/python2.7/site-packages/numpy/init.py", line 140, in
from . import _distributor_init
ImportError: cannot import name _distributor_init

`
The last question is :
Why is it that u created a virtual environment called tensorflow but never activated it? Does it mean that ur tensorflow is installed in the root/base environment? In this case whats the use of creating a virtual env?
This question is asked on your youtube tutorial as well not answered yet. Cheers

Hi @stevensu1838
Thank you for reporting the issue.
I'll review this on my Kinetic machine and get back to you.
Thank you for your patience.

Hi Will,
Thanks a lot for your great support. I am using exactly Kinetic machine. May I please provide you more details on my errors?
First, I didn't understand why we created a virtual environment called tensorflow but never activated it. So I simply skipped installing anaconda and creating the virtual environment called tensorflow. I installed tensowflow and keras directly on the PC environment.
Then in terminal 1:export TURTLEBOT3_MODEL=burger
$ roslaunch turtlebot3_dqn turtlebot3_dqn_stage_2.launch
In terminal 2: roslaunch turtlebot3_dqn result_graph.launch
in terminal 3:export TURTLEBOT3_MODEL=burger
$ roslaunch turtlebot3_dqn turtlebot3_dqn_stage_2.launch
Then in terminal 3 I got the following errors
`user@mech1552:~/catkin_ws/src$ roslaunch turtlebot3_dqn turtlebot3_dqn_stage_2.launch
... logging to /home/user/.ros/log/4e84af5c-74ba-11eb-9885-6451065039f7/roslaunch-mech1552-9275.log
Checking log directory for disk usage. This may take awhile.
Press Ctrl-C to interrupt
WARNING: disk usage in log directory [/home/user/.ros/log] is over 1GB.
It's recommended that you use the 'rosclean' command.
started roslaunch server http://<IP_ADDRESS>:33492/
SUMMARY
PARAMETERS

/rosdistro: kinetic
/rosversion: 1.12.16
/stage_number: 2

NODES
/
turtlebot3_dqn_stage_2 (turtlebot3_dqn/turtlebot3_dqn_stage_2)
ROS_MASTER_URI=http://<IP_ADDRESS>:11311
process[turtlebot3_dqn_stage_2-1]: started with pid [9294]
Using TensorFlow backend.

Layer (type)                 Output Shape              Param #
dense_1 (Dense)              (None, 64)                1856

dense_2 (Dense)              (None, 64)                4160

dropout_1 (Dropout)          (None, 64)                0

dense_3 (Dense)              (None, 5)                 325

activation_1 (Activation)    (None, 5)                 0
Total params: 6,341
Trainable params: 6,341
Non-trainable params: 0

Layer (type)                 Output Shape              Param #
dense_4 (Dense)              (None, 64)                1856

dense_5 (Dense)              (None, 64)                4160

dropout_2 (Dropout)          (None, 64)                0

dense_6 (Dense)              (None, 5)                 325

activation_2 (Activation)    (None, 5)                 0
Total params: 6,341
Trainable params: 6,341
Non-trainable params: 0

2021-02-22 16:02:48.664099: I tensorflow/core/platform/cpu_feature_guard.cc:140] Your CPU supports instructions that this TensorFlow binary was not compiled to use: AVX2 FMA
[INFO] [1613962969.631518, 0.832000]: Goal position : 0.6, 0.0
Traceback (most recent call last):
File "/home/user/catkin_ws/src/turtlebot3_machine_learning/turtlebot3_dqn/nodes/turtlebot3_dqn_stage_2", line 174, in
agent.trainModel()
File "/home/user/catkin_ws/src/turtlebot3_machine_learning/turtlebot3_dqn/nodes/turtlebot3_dqn_stage_2", line 121, in trainModel
q_value = self.model.predict(states.reshape(1, len(states)))
File "/home/user/.local/lib/python2.7/site-packages/keras/models.py", line 1025, in predict
steps=steps)
File "/home/user/.local/lib/python2.7/site-packages/keras/engine/training.py", line 1817, in predict
check_batch_axis=False)
File "/home/user/.local/lib/python2.7/site-packages/keras/engine/training.py", line 123, in _standardize_input_data
str(data_shape))
ValueError: Error when checking : expected dense_1_input to have shape (28,) but got array with shape (364,)
[turtlebot3_dqn_stage_2-1] process has died [pid 9294, exit code 1, cmd /home/user/catkin_ws/src/turtlebot3_machine_learning/turtlebot3_dqn/nodes/turtlebot3_dqn_stage_2 __name:=turtlebot3_dqn_stage_2 __log:=/home/user/.ros/log/4e84af5c-74ba-11eb-9885-6451065039f7/turtlebot3_dqn_stage_2-1.log].
log file: /home/user/.ros/log/4e84af5c-74ba-11eb-9885-6451065039f7/turtlebot3_dqn_stage_2-1*.log
all processes on machine have died, roslaunch will exit
shutting down processing monitor...
... shutting down processing monitor complete
done
user@mech1552:~/catkin_ws/src$
`
and the graph is like below. NO lines appear in the plots

Thanks a lot

Hi Will,
May I please ask if you figured this out? I've been trying hard. But I am not good enough to debug this. Thank you and your team very much

Hi @stevensu1838
I've been testing the package in different ROS versions, and it looks like softwares listed in the eManual are outdated and does not supported very well.
I'm working on fixing the issue along with turtlebot3_simulations package, and please allow me a few weeks.
Thank you for your patience.

@stevensu1838
Based on my trials with Kinetic and Melodic, there were some discrepancies in installing the latest Anaconda as the default Python version for those OS were Python2.
In my case, using a virtual environment made it hard to run ROS due to some ros pkg issue.
Regarding the error you are getting from launching roslaunch turtlebot3_dqn turtlebot3_dqn_stage_2.launch can be resolved by modifying the number of scan samples from turtlebot3/turtlebot3_description/urdf/turtlebot3_burger.gazebo.xacro file as described in the eManual below.
Since states receives the scan data, discrepancy in its data length causes the problem when running the node.
https://emanual.robotis.com/docs/en/platform/turtlebot3/machine_learning/#set-state
I'm sorry about small delays, but I'll try to complete it within a few days.
Thanks!

@stevensu1838
The eManual instruction for Machine Learning had been updated for ROS1 Kinetic & Melodic.
https://emanual.robotis.com/docs/en/platform/turtlebot3/machine_learning/#machine-learning
Please let us know if there's any issue with following the instructions.
Thanks!


Find Largest subtree sum in a binary tree!

Hello, I would like to contribute to this project in c++ language.
->in O(n) time complexity
->in O(n) space complexity
I am a GSsoc'21 participant. assign me the issue.
waiting for a positive response.

with the help of postorder recursion method.


fix: revert import config module as data

Description
Fixes https://github.com/vitejs/vite/issues/13730. See issue for description. If we want to move forward with using data:text/javascript, we would need to consider more edge cases. I think reverting is the best move right now.

What is the purpose of this pull request?

[x] Bug fix
[ ] New Feature
[ ] Documentation update
[ ] Other

Done, thanks @silverwind!

Thanks, could you also unlock the discussions in them?

I'm sorry for the situation, let's find a better way

Dont worry @s10y10, it was good we tried the approach.

TBH, I think the revert was premature, the issue should have been analyzed at least. I think it's likely that the URL has hit some kind of length limit.

@silverwind I think the revert was valid here, we weren't sure this approach was safe and it failed as soon as we released it, and seems the issue is on our side. We can try with more time again. I hope @cpojer could provide a minimal reproduction to help you test a new take at it.


Keyboard Manager Special Keys Stick (e.g. windows key)

Microsoft PowerToys version
0.62.0
Running as admin

[ ] Yes

Area(s) with issue?
Keyboard Manager
Steps to reproduce
The commands from my sony wh-1000xm3 headphones touchpad is not recognised by many programs in windows. My keyboard has these commands from my headphones as individual buttons which are recognised by windows. Unfortunately keyboard manager won't let me remap my headphones' "next track" to my keyboard's "next track" for example, as they have the same name (but clearly are different commands as the one from my keyboard works). So I mapped the headphones' "next track" to a buffer key "win+8". I then remapped this shortcut to my keyboard's "next track". It worked like a dream for a bit but then broke (possibly when I updated powertoys). The cause I believe is when I do "next track" on my headphones the buffer windows key gets stuck, i.e. is not unpressed. This means I cannot do the command again and I have to double press the actual windows key to unstick it.
✔ Expected Behavior
To remap the headphones' commands ("previous track", "pause", "next track") to the buttons of the same name/commands on my keyboard. This doesn't work as the buffer windows key that I used gets stuck pressed.
❌ Actual Behavior
When the buffer windows key is executed, it stays pressed rendering the keyboard useless until I press the windows key a couple of times.

Other Software
The main software I use it for is music on VLC Media player.

Are you still experiencing this issue? /needinfo

Yes, it seems the windows key still gets stuck.


Add series value dumping capability

What does this PR do?
A brief description of the change being made with this pull request.
Motivation
What inspired you to submit this pull request?
Related issues
A list of issues either fixed, containing architectural discussions, otherwise relevant
for this Pull Request.
Additional Notes
Anything else we should know when reviewing?

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

#997  👈
#995
#994
main

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

Join @GeorgeHahn and the rest of your teammates on  Graphite

Merge activity

Sep 16, 4:47 PM EDT: @GeorgeHahn merged this pull request with Graphite.


Set OptimismPortal._balance with StorageSetter pattern

In order to get rid of the constructor changes of the OptimismPortal contract, we can modify the storage directly with an intermediate Proxy implementation:

We use this contract to set storage during upgrades by swapping the implementation to this contract and then to the desired implementation atomically. This prevents complexity around coupling storage migrations with implementation code and would allow this diff to go away. This pattern will be required for contract upgrades since we need to zero out the _initialized slot to call initialize on a new implementation

Originally posted by @tynes in https://github.com/celo-org/optimism/pull/174#discussion_r1681158745

@ezdac if this is still needed and cannot yet be closed, please

move to the appropriate milestone
attach to the relevant epic


Allow execution under PHP 8

PHP 8 integration test is finally green https://github.com/FriendsOfPHP/PHP-CS-Fixer/pull/5399

PHP 8 integration test is finally green https://github.com/FriendsOfPHP/PHP-CS-Fixer/pull/5399

Coverage remained the same at 91.927% when pulling 1006d53fde217dae506c06216af7edae4b881d67 on keradus:2.18_php8 into 0a9cd6c697beab0e72b413162a864e00fab2729d on FriendsOfPHP:master.

Coverage remained the same at 91.927% when pulling 1006d53fde217dae506c06216af7edae4b881d67 on keradus:2.18_php8 into 0a9cd6c697beab0e72b413162a864e00fab2729d on FriendsOfPHP:master.


"Enable Base Internationalization" warning is disabled

This issue is caused by #694.
I totally agree that we should enable Base Internationalization.
However, I think this PR disable the warning unintentionally.

no Base.lproj folder but one for the development language

In this case, the warning should be shown, and the user can solve it from Xcode.
After that, there is no such warning even after running xcodegen again since Base.lproj folder will be created by Xcode.

Hmm, yes this was my intuition as well. @bryansum thoughts?

I'm reading this thread to help regain context https://github.com/yonaskolb/XcodeGen/pull/685
I tried to reproduce the original warning by recreating the same scenario (only knownRegions:  en, with no Base.lproj folder, just using Xcode), but it seems like that warning is no longer reproducing in Xcode 11.3.1. Perhaps this warning was removed by Apple? Would be good for someone else to verify. If that's the case then we should probably remove it.
I guess there's a philosophical element to this as well: when Xcodegen creates a project, is the desire to set similar defaults as an Xcode new project? By default projects have Base internationalization enabled, but I could also see an argument that we shouldn't turn that on for people that don't want it -- especially when it's not always needed, like when the base matches matches the development language.

but it seems like that warning is no longer reproducing in Xcode 11.3.1.

And it returns in Xcode 11.4 beta. I have no idea why.

when Xcodegen creates a project, is the desire to set similar defaults as an Xcode new project?

I can see the point, however, I think setting Base to knownRegions only disable the warning and have no effect to promoting the user to use Base Internationalization since there is no Base.lproj folder.
Therefore, I guess it may be better to keep the warning and let the user decide.

I still can't reproduce the original warning, but it definitely did exist at one point (see the related Cocoapods issue: https://github.com/CocoaPods/CocoaPods/issues/9187). Not quite sure how to proceed -- I have a revert ready here if desired: https://github.com/yonaskolb/XcodeGen/pull/791
@myihsan if you can, would you mind sharing an example Xcode project that surfaces the warning? Would be helpful to reproduce the original issue.

@bryansum
As my tests, it seems like the warning certainly appears in the version newer than the version you unchecked Use Base Internationalization.
But somehow, it also appears in the current version sometimes and hard to tell why since the warning may not be updated until relaunched Xcode or done a clean.
If you need, I can create an example project by Xcode 11.1.

@myihsan thanks, got it working by following your instructions (create a new project in Xcode 11.0, remove base internationalization, then open in any of 11.1, 11.3.1, 11.4 beta 2). In this case I can see that the Xcode fix it actually does offer to move files from en.lproj to a new Base.lproj folder.
Given that behavior, I agree we should revert. @yonaskolb to review #791?

Cool, will merge


SOCKS error. Could not launch Syndicate application.

This issue tracker is only for technical issues related to bitcoin-core.
General bitcoin questions and/or support requests and are best directed to the Bitcoin StackExchange.
For reporting security issues, please read instructions at https://bitcoincore.org/en/contact/.
Describe the issue
Cannot launch Syndicate application
Can you reliably reproduce the issue?
Yes
If so, please list the steps to reproduce below:

Launch Syndicate QT application
Pop up message "Error: Unsupported argument - socks found. Setting SOCKS version isn't possible anymore, only SOCKS5 proxies are supported"
Fail to launch Syndicate application

Expected behaviour
Automatically open Syndicate application
Actual behaviour
Error message pop-ed up, then application closed by itself
Screenshots.
If the issue is related to the GUI, screenshots can be added to this issue via drag & drop.

Any extra information that might be useful in the debugging process.
I accidentally checked one of the boxes under "Network" tab (can't remember the name of the box). When restarting the application, it fails to launch and I couldn't uncheck the box.
Thank you for your time!

It took a little digging around, but I figured out a fix for this.

Make sure the program is closed, task manager if you have to.
Open your registry editor. (start menu > run > regedit)
Navigate to: Computer\HKEY_CURRENT_USER\Software\Syndicate\Syndicate-Qt
Now there will be 2 things you need to change here.
a. change: fUseProxy true to false,
b. delete: AddProxy <IP_ADDRESS>:9050 (may or may not be here depending on if you actually set a proxy, if so this number may be different, but this was default value shown for me)
Relaunch the program.

If it still fails to load, try to restart your computer after editing the registry. I didn't have to, but your mileage may vary.

This is really helpful. The problem is fixed :) Thanks!


Get rid of WITH_DISNEY_MATERIAL CMake option

We are disabling this build option when building appleseed bundled in Gaffer.
It introduces a dependency SeExpr that is not used in Gaffer or in the Maya and Max plugins.
I'd prefer if we can keep it if it is possible.

Ah ok, I didn't know. Let's keep it then, no problem!


Error code "timeout-or-duplicate" added. Made tests pass ok. Updated documentation.

Thanks for this awesome project.

This change is

Well, thank you for the contribution! 😃
I will try to have a look at it this evening (in ~4 hours).

Taking a look now.

Looks good.
I'll have to do a minor version bump, since adding that enum variant is a breaking change.

Published 0.5.0 on crates.io

Wow, fast! Thank you  @panicbit much appreciated.


Fix mount issues in containerized Kubelet

nsenter was not located via path in the container
nsenter needed '--' to separate command from args
we need to search the parent FS at /rootfs in order to find mount/umount/findmnt

Minor comments.

Comments addressed - I search for each binary now in a list of paths, first path wins.  If doesn't exist will default to /<binary> and the other mount commands will fail and log

Code LGTM. @dchen1107 to evaluate for 1.0 & second LGTM.

LGTM

@smarterclayton Can we hold this one for a few weeks, right after 1.0 release?

Sure.

@dchen1107 should this have the 1.0-post label? I have been working on mount propagation issues in docker and had to aply this patch to test correctly.  Just want to make sure this goes in ASAP since I am going to be working in this area.

Docker 1.7 is broken in a container (except on Red Hat distros) because mount propagation is PRIVATE, not SHARED.  Mount-prop fix is being tracked in libcontainer, will probably require docker changes as well.  The other option may be to start with the hosts mount namespace.

@dchen1107 second review?

My centos 6.5 + docker 1.7.1 + hyperkube + Kubernetes v1.0.1 is suffering this issue. Is it possible to at least merge "nsenter needed '--' to separate command from args" to release branch?
Here is the log:
I0731 07:52:48.829233       1 nsenter_mount.go:79] nsenter Mounting tmpfs /var/lib/kubelet/pods/a9d8d942-3729-11e5-abda-080027ce083d/volumes/kubernetes.io~secret/dns-token tmpfs []
I0731 07:52:48.829242       1 nsenter_mount.go:82] Mount command: /nsenter [--mount=/rootfs/proc/1/ns/mnt /usr/bin/mount -t tmpfs -o  tmpfs /var/lib/kubelet/pods/a9d8d942-3729-11e5-abda-080027ce083d/volumes/kubernetes.io~secret/dns-token]
I0731 07:52:48.830048       1 nsenter_mount.go:86] Output from mount command: nsenter: failed to parse pid: 'tmpfs'
E0731 07:52:48.830076       1 kubelet.go:1225] Unable to mount volumes for pod "kube-dns-fqgan_default": exit status 1; skipping pod

I think we already got 1 LGTM, just needs a sign off from Dawn if she wants
to review it.
On Jul 31, 2015, at 5:42 AM, Lv Lv<EMAIL_ADDRESS>wrote:
My centos 6.5 + docker 1.7.1 + hyperkube + Kubernetes v1.0.1 is suffering
this issue. Is it possible to at least merge "nsenter needed '--' to
separate command from args" to release branch?
Here is the log:
I0731 07:52:48.829233 1 nsenter_mount.go:79] nsenter Mounting tmpfs
/var/lib/kubelet/pods/a9d8d942-3729-11e5-abda-080027ce083d/volumes/
kubernetes.io~secret/dns-token tmpfs []
I0731 07:52:48.829242 1 nsenter_mount.go:82] Mount command: /nsenter
[--mount=/rootfs/proc/1/ns/mnt /usr/bin/mount -t tmpfs -o tmpfs
/var/lib/kubelet/pods/a9d8d942-3729-11e5-abda-080027ce083d/volumes/
kubernetes.io~secret/dns-token]
I0731 07:52:48.830048 1 nsenter_mount.go:86] Output from mount command:
nsenter: failed to parse pid: 'tmpfs'
E0731 07:52:48.830076 1 kubelet.go:1225] Unable to mount volumes for pod
"kube-dns-fqgan_default": exit status 1; skipping pod
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/kubernetes/pull/9976#issuecomment-126624808
.

This is very useful for automating scale out setups.
/cc @wojtek-t @rrati @jbeda

Sorry for delaying the review for this PR. LGTM


[docs] APIs

⏳ WIP ⏳
This PR expands the API docs to showcase high-level classes and core lower-level building blocks, starting with the optimizers.

[ ] LinearKbit (Linear4bit, Linear8bitLt, etc.)
[ ] Parameters (Params4bit, Int8Params)
[ ] Embeddings (Stable Embedding, Embedding)

Hey @Titus-von-Koeller, can you check the docstring descriptions for AdaGrad and the base optimizer classes? Since most of the docstring descriptions are the same, if these look good to you, I can easily copy most of them over to the remaining optimizers.

Hey @stevhliu, I'll take a look now. Please be sure to run pre-commit install inside your local BNB Git directory.

Everything looks really good, thanks! I think the docstring for the optimizers and base class is good the way you did it.


Default negative values support confuse people

version 0.10.x Supports negative values by default, if we enable countdown with flipclock, then we would see -0:00 in the end, but I think 00:00 would be better here (like the old version).
The - sign in the face sometimes confuse people. I think we better add option to let developer disable this feature.

I believed this issue is related to the getTotalSeconds() in flipclock.js
After calculated the time difference, the Math.round() just rounded the diff value likes -0.0... as -0, not 0
Therefore, the negative sign shows when countdown to zero


Remaining pull requests applied

All the changes on the following 3 pull requests, have been applied in this PR.
https://github.com/cloudsigma/pycloudsigma/pull/12
https://github.com/cloudsigma/pycloudsigma/pull/9
https://github.com/cloudsigma/pycloudsigma/pull/8
So after merging this PR, you can close other PRs.

Release/V2.0.1


F# Interactive client with 4.05 status

I successfully created and communicated with the demo server, a COMPILED F# CoAP clients based on the examples as DISCOVER coap://localhost and GET coap://localhost/hello.
But when I try the exact same code using F# interactive (REPL), I get: "Status : 4.05 Method Not Allowed".
Also, I did the same test, but using a CoAP server in F# interactive with the same behavior.
I am looking for suggestions why this is happening and how is the best way to debug the problem.

Hi~
I wrote some test code and did not reproduce the issue you found. Could you please share more details?
Here is my code:
#if INTERACTIVE
#r @"Path\To\CoAP.NET.dll"
#endif

open CoAP
open CoAP.Util

let req = Request.NewGet()
let uri = System.Uri("coap://localhost/hello")
req.URI <- uri

let res = req.Send().WaitForResponse()

if res = null
then System.Console.WriteLine("Request timeout")
else System.Console.WriteLine(Utils.ToString(res))

And I got ACK 2.05 Content both in a compiled project and in F# interactive (inside VS).

Thanks for reply. Sorry, but I had a bug in the F# interactive version. Now
all work well.
On Thu, Jul 30, 2015 at 5:06 AM, longshine<EMAIL_ADDRESS>wrote:

Hi~
I wrote some test code and did not reproduce the issue you found. Could
you please share more details?
Here is my code:
#if INTERACTIVE
#r @"Path\To\CoAP.NET.dll"
#endif
open CoAPopen CoAP.Util
let req = Request.NewGet()let uri = System.Uri("coap://localhost/hello")
req.URI <- uri
let res = req.Send().WaitForResponse()
if res = nullthen System.Console.WriteLine("Request timeout")else System.Console.WriteLine(Utils.ToString(res))
And I got ACK 2.05 Content both in a compiled project and in F#
interactive (inside VS).
—
Reply to this email directly or view it on GitHub
https://github.com/smeshlink/CoAP.NET/issues/31#issuecomment-126259387.

: )


Implement Riemannian HMC

This is a draft to implement Riemannian HMC. There are many things to discuss. I put the high-level points here while leaving more specific ones in the code.

Do we want to unify all function signatures for Hamiltonian to a non-separable one (i.e. position dependent)?
Do we need to update our metric abstraction? Looks like we should decouple unit/diagonal/dense vs Euclidean/Riemannian.
Where should SoftAbs best locate? I current implemented it as an external function to provide G to a generic Riemannian HMC implementation.
The efficiency is pretty bad and there is fewer optimization opportunities due to our decoupled abstraction (e.g. compared to what derived in [1]).

To-dos

The current SoftAbs implementation has numerical issues when running on Neal's funnel. I need to double check the if the manual Jacobian implementation is correct, or we could register softabs with ReverseDiff to see if this solves the issue. I could use someone else help on this.

How to play with this PR
I provided a notebook (which contains the same content as the test/experimental/riemannian_hmc.jl file) to play with the code. The notebook has some simple validation on the implementation and also shows the current numerical issue of SoftAbs. I highly recommend you to try this.

[1] Betancourt, M., 2013, August. A general metric for Riemannian manifold Hamiltonian Monte Carlo. In International Conference on Geometric Science of Information (pp. 327-334). Springer, Berlin, Heidelberg.

One thing that is worth discussing wrt. this PR is how much of this functionality we should put into the metric types themselves vs. just implementing it all as a change of variables.
If we instead consider these position-dependent metrics as a change of variables for the potential instead, we don't have to deal with non-separable Hamiltonians internally and hence should hopefully make hte internals simpler.
Are there any obvious downsides to this?

One thing that is worth discussing wrt. this PR is how much of this functionality we should put into the metric types themselves vs. just implementing it all as a change of variables.
If we instead consider these position-dependent metrics as a change of variables for the potential instead, we don't have to deal with non-separable Hamiltonians internally and hence should hopefully make hte internals simpler.
Are there any obvious downsides to this?

I also think so as I put a note here: https://github.com/TuringLang/AdvancedHMC.jl/blob/master/src/metric.jl#L116.
Ideally we could interface Bijectors.jl here and do more than just affine transformation.
With this being said, the best route to implement this in AHMC is to do an internal refactoring PR first and then implement here. I feel that would require quite a bit of discussion.
Besides, I think most of the callback change to handle non-separable Hamiltonians that introduced in this PR is still needed there.

I also think so as I put a note here

Ah didn't catch that; cool! Btw, are we aware of any downsides to just doing a reparameterization for something like RHMC, given that the original paper didn't do this but instead went the route of Genearlized Leapfrog, which requires a root-solve in to determine the implictly defined step?

Btw, are we aware of any downsides to just doing a reparameterization for something like RHMC, given that the original paper didn't do this but instead went the route of Genearlized Leapfrog, which requires a root-solve in to determine the implictly defined step?

I don't quite get the question. What the original paper does can also be seen as a form of reparameterization and we are doing the same thing together with genearlized Leapfrog, for which the implict updates are done by fixed-point iterations.

Updates as of ed15539

Introduced types to control how Hessian is mapped (https://github.com/TuringLang/AdvancedHMC.jl/blob/kx/rhmc-draft/research/src/riemannian_hmc.jl#L136-L155). This gives an easy way to access internal parameters of the SoftAbs map and specialize the internals based on different maps (IdentityMap or SoftAbsMap).

PS: I made this choice mostly because of the easiness of implementation and I'm happy to consider other designs.

Fixed a bug in constructing J (https://github.com/TuringLang/AdvancedHMC.jl/blob/kx/rhmc-draft/research/src/riemannian_hmc.jl#L258) by adding a missing coefficient to csch

With 1 and 2, I refactored ∂H∂θ for the SoftAbs metric (https://github.com/TuringLang/AdvancedHMC.jl/blob/kx/rhmc-draft/research/src/riemannian_hmc.jl#L271-L300) and the sampling results visually look OK on Neal's funnel now:

The incorrect way of handling numerical issue is removed and any handling is done by users (e.g. https://github.com/TuringLang/AdvancedHMC.jl/blob/kx/rhmc-draft/research/tests/riemannian_hmc.jl#L40)
Performed Geweke test for RHMC (), which indicates that there are still bugs in the RHMC sampler with SoftAbs metric:

The test is done by the model below

@model function TuringFunnel(θ=missing, x=missing)
    if ismissing(θ)
        θ = Vector(undef, 2)
    end
    θ[1] ~ Normal(0, 3)
    s = exp(θ[1] / 2)
    θ[2] ~ Normal(0, s)
    x ~ Normal(0, s)
    return θ, x
end

with MCMCDebugging.jl.
PS: For reference, the test results of a normal HMC sampler under the same model is below:

I'm looking into the codes to see where the bug might be but if anyone has some ideas for debugging or intuition of where the bug might be, please let me know.

Note of meeting with Hong on 23 Feb
Potential optimization

G(θ) is unchanged in the fixed-point iteration for (16). We don't need to compute it for every call of ∂H∂θ.
The first call of ∂H∂r is unchanged during (17). We can put it outside of the loop.

Tests

Use finite differentiation to test our Hessian/metric computation and partial derivatives
Test neg_energy using MvNormal

Double check the usage


OpenBSD uses machine-id

Hi,
This originated here:
https://github.com/schollz/croc/issues/137
Your readme states:
BSD uses /etc/hostid
However, OpenBSD uses /etc/machine-id:
$ ls -l /etc/machine-id
-rw-r--r--  1 root  wheel  33 May  1  2018 /etc/machine-id

I don't know go, so I don't know where to fit this update into your file:
https://github.com/denisbrodbeck/machineid/blob/master/id_bsd.go

Hi, just passing through before filing my own issue…
Maybe you could reuse the logic from id_linux.go, trying to read a file and falling back to another one if the first attempt doesn't succeed?
Cheers,
Cyril.

OpenBSD does not use machine-id. /etc/machine-id comes from dbus.

@rnagy,
Oh, I'm sorry! thanks for correcting me.


Rename CRYPTO_AES256_KEY to AES256_KEY in OpenShift template

The environment variable in the OpenShift template was named CRYPTO_AES256_KEY, which was causing the deployment of osba-open-service-broker-azure to fail.
Per the code this variable should be named AES256_KEY:
https://github.com/Azure/open-service-broker-azure/blob/be79faa48d04794538100f3b7b6fc9a40ee04c91/pkg/crypto/aes256/config.go#L12

Either of those two environment variables should work. Note the const envconfigPrefix, its value of "CRYPTO" and how it is used when the environment variables are processed.

When deploying it yesterday (2018-06-23) it would not deploy and the logs were showing the following error:
AES256 key was not specified
By changing the environment variable from CRYPTO_AES256_KEY to AES256_KEY the error went away and the deployment was successful.

@jaredhocutt can you confirm the version of the image that you were using?

@jaredhocutt also note that the Helm chart for v1.0.0 uses the CRYPTO_AES256_KEY env var without any issue.
https://github.com/Azure/open-service-broker-azure/blob/master/contrib/k8s/charts/open-service-broker-azure/templates/deployment.yaml#L58-L62

@krancour I'm using microsoft/azure-service-broker:v0.10.0 per the OpenShift template. I also confirmed in OpenShift that this is in fact the version being used as well.
https://github.com/Azure/open-service-broker-azure/blob/f0ba3bd6f702110e93e22780611feba9f3eb5477/contrib/openshift/osba-os-template.yaml#L151

So I am wondering if you were actually, unknowingly, using image  version v0.10.0, which seems to be hard-coded into the Open Shift template. (Clearly that's wrong.)
There's already an open issue to fix the aspects of that template that were hardcoded and probably shouldn't have been. See #428 and post-merge comments on PR #425.

But as it is now, the template does not work and this PR fixes that.

That's not quite true. This would "fix" that in master (where there actually isn't even a problem), but that template for deploying on Open Shift would still reference v0.10.0-- which wouldn't have the fix.
imo, short of addressing #425 / #428 completely, a more productive PR might be to simply update the existing template to reference OSBA v1.0.0. (Although, fyi, we will likely release a v1.0.1 on Monday.)

Sorry... I stand corrected on the above. I mistakenly thought this PR changed Go code, but see now that it changes the template. Still... if the template were to change, the change I'd rather see is an update to reference a newer broker image. Hope that makes sense.

@krancour That works for me.
I'll wait until after v1.0.1 is released on Monday and will test to make sure that solves the issue. Then I'll update the PR with that change instead.

@jaredhocutt go ahead and amend this commit as we discussed and I will simply merge it just before the release.

@jaredhocutt actually... we're ready to go, so if you don't mind I'm just going to close this PR and PR what we discussed.

All good @krancour.
Thanks for getting this fixed. Apologies for the delay, I was travelling today for work and am just now getting caught up.


[Question] Understanding the way to use the data parsed

Hey!
Thank you for the great library, it helped me a lot! This is not an issue per se, but more questions on how to handle the outputted data generated.
I'm converting GRIB2 files obtained from the DWD for the ICON-EU forecast model for the cloud coverage (total cover, high, middle and low clouds).

First question, after running the grib2json, I get the headers from the GRIB2 file. It gives me the following: nx: 1097, ny: 657. When I compare it with the official DWD regrid tool, it gives the following values. I believe these data are obtained upstream by the java tool but you might know why there is a difference in those values? Maybe the data from the grib2 file are correct and the one from DWD are wrong or any other reason?

Second question is about parsing the 2D grid data generated by the package. How do I interpolate the values to obtain an array of array of lat/lon/value e.g. data: [[64.4,-21.7,12.5], [...]]. I've been using the following code a bit modified for my purpose (with a grid of point generated with turf and then map with the data returned by the previous code, but it looks like it doesn't interpolate correctly the coordinates. I believe it's related to the fact that the icon-eu grib2 file returns a grid with a BBOX of [-23.5, 54.63, 32.5, 70.5] while this script use 90 and 0 as references. Do you know which values I have to change to apply the reference of the grib file?

In general, do you have good read about grib2 and libraries to interpolate its values? There is a lot of python/java libraries, but I would prefer a javascript solution.
Thanks in advance, hope it won't be too much bother with my questions,

Hi, about your first question yes the values are produced by the underlying grib processing tool. You can visualize the content of your grib file using eg https://www.giss.nasa.gov/tools/panoply/ and check the values. Weare on the way to integrate a new tool not based on Java, see https://github.com/weacast/weacast-grib2json/issues/16, you might give it a try and see if the output is similar.
About your second question, although the interpolation algorithm is well defined per se, the different conventions about the value ordering and the latitude/longitude range could make it a bit tricky. As a consequence, we have built a pretty generic class to handle different cases in weacast (grid of different sizes, longitude crossing, etc.). This is for instance used to resample the weather data on-demand. Feel free to use it or start something from it.

Hi, about your first question yes the values are produced by the underlying grib processing tool. You can visualize the content of your grib file using eg https://www.giss.nasa.gov/tools/panoply/ and check the values. Weare on the way to integrate a new tool not based on Java, see https://github.com/weacast/weacast-grib2json/issues/16, you might give it a try and see if the output is similar.
About your second question, although the interpolation algorithm is well defined per se, the different conventions about the value ordering and the latitude/longitude range could make it a bit tricky. As a consequence, we have built a pretty generic class to handle different cases in weacast (grid of different sizes, longitude crossing, etc.). This is for instance used to resample the weather data on-demand. Feel free to use it or start something from it.

Thank you, this is very helpful!

I was looking for a good tool to visualise GRIB2 file and Panoply is much much better than the rest I tried, thanks! Also, according to it, it looks like the values are corresponding to the one obtain with the java library. However, I might give a try to the ecCodes implementation, I've been noticing huge usage of memory with the java.

About the interpolation, from my understanding, ICON-EU is a triangular datasets, so I would need to find a triangular interpolation method to obtain the lon/lat grid result. Is my understanding correct?

Thank you for your help and anwser

Thank you, this is very helpful!

I was looking for a good tool to visualise GRIB2 file and Panoply is much much better than the rest I tried, thanks! Also, according to it, it looks like the values are corresponding to the one obtain with the java library. However, I might give a try to the ecCodes implementation, I've been noticing huge usage of memory with the java.

About the interpolation, from my understanding, ICON-EU is a triangular datasets, so I would need to find a triangular interpolation method to obtain the lon/lat grid result. Is my understanding correct?

Thank you for your help and anwser

Yes ICON uses an icosahedral grid for modeling but you can convert ICON files to the more standard rectangular grid using https://github.com/DeutscherWetterdienst/regrid first, then perform the standard bilinear interpolation. By the way this tool also relies on ecCodes so the interpolation algorithm probably comes from it.
Triangular interpolation follows the same mathematical principles as standard bilinear interpolation over quads but it is less used except in video games where meshes are triangles ;-) You might find relevant resources about it if you'd like to interpolate raw icon data.

Yes ICON uses an icosahedral grid for modeling but you can convert ICON files to the more standard rectangular grid using https://github.com/DeutscherWetterdienst/regrid first, then perform the standard bilinear interpolation. By the way this tool also relies on ecCodes so the interpolation algorithm probably comes from it.
Triangular interpolation follows the same mathematical principles as standard bilinear interpolation over quads but it is less used except in video games where meshes are triangles ;-) You might find relevant resources about it if you'd like to interpolate raw icon data.

Thank you for your help. I ended up using ecCodes and its grib_get_data which returns a file of latitude/longitude/value. Just add to create a buildpack for heroku to install ecCodes

Thank you for your help. I ended up using ecCodes and its grib_get_data which returns a file of latitude/longitude/value. Just add to create a buildpack for heroku to install ecCodes


Update README to set private_key_path correctly

The README is incorrect. private_key_path is outdated and privateKeyPath is used instead.

Thanks!


How should I use this project from npm

I've searched all over the npm official website and found that this project does not exist. As a front-end developer, how should I use this project?

Can you describe your use case? This package is meant to be consumed by Rspack, on the Rust side as a crate.

There is a tool called unocss, which does not currently support rspack. I plan to provide support for rspack, but this tool uses a package called webpack-source, so I think I need rspack-source to enhance support.

@Wyatex Can you point us to where webpack-source is being used in unocss? It would be great if you can find the call site in unocss so we can help you impement this in Rspack.

https://github.com/unocss/unocss/blob/main/packages/webpack/src/index.ts#L127-L127
Moreover, this tool uses the resolveId hook, which in webpack is realized by the webpack-virtual-modules package. And for instance, unplugin-icons also needs to use virtual modules, so is it possible to also implement an rspack-virtual-modules?

@web-infra-dev/rspack-dx Please take a look.

This crate is only used in Rust side, for JS side webpack-sources should works
For unocss plugin (or other common used plugins), Rspack should be able to run its webpack plugin directly for the long term, and for the short term, it's welcome for someone to create a rspack-specific plugin for unocss
And there is a webpack-virtual-modules alternative for rspack

Feel free to reopen if you have any further questions


fix dist regeneration

https://github.com/hacl-star/hacl-star/pull/696 removed the election-guard and gcc64-only distributions.
They must also be removed from the dist regeneration script.

btw, "gcc64-only" is still appearing in code/blake2/Makefile, code/ecdsap256/Makefile and doc/Obtaining.rst


Geolocation not working in background mode after a while

Your Environment

Plugin version: 0.6.3
Platform: Android
OS version: 10
Device manufacturer and model: Samsung Galaxy A10

Running in Simulator: No
React Native version: 0.63.3
Plugin configuration options: BackgroundGeolocation.configure({
desiredAccuracy: BackgroundGeolocation.HIGH_ACCURACY,
stationaryRadius: 50,
distanceFilter: 50,
notificationTitle: 'Background tracking',
notificationText: 'enabled',
debug: false,
startOnBoot: false,
startForeground: true,
stopOnTerminate: true,
locationProvider: BackgroundGeolocation.ACTIVITY_PROVIDER,
interval: 1000,
fastestInterval: 5000,
activitiesInterval: 10000,
stopOnStillActivity: false,
});
Link to your project: https://github.com/jchavezjs/geolocationtest

Context

The library works fine when i have the app open, but when i minimize the app and the notification is active it only gets the location like 5 times more and it stop after that. The location is available again when i open the app. I realized that using android.permission.ACCESS_BACKGROUND_LOCATION it works perfectly, but i don't want to depend on that permission because is not available in many androids. I already have android.permission.FOREGROUND_SERVICE as a permission in my manifest but it doesn't work.
Expected Behavior

The geolocation should continue in background mode
Actual Behavior

The geolocation stops after 4 or 5 request when I am in background mode
Possible Fix

I believe the problem is something that makes the library don't work as foreground service because with backgound location it works nice.
Steps to Reproduce

Activate the background location service
Put the app in background mode
Wait for some successful request
The foreground service is not working until you re open the app

Context

Just trying to do normal background geolocation when app is running as foreground service
Debug logs

This is the log after the last time the geolaction is called in background
2020-12-03 17:04:08.308 1897-1897/com.geolocationtest D/com.marianhello.bgloc.provider.ActivityRecognitionLocationProvider: Location change: Location[fused 13.683869,-89.273859 hAcc=5 et=+11d18h22m17s458ms alt=929.0 vel=0.5 bear=284.0 vAcc=100 sAcc=20 bAcc=10 {Bundle[mParcelledData.dataSize=52]}]
2020-12-03 17:04:08.328 1897-1897/com.geolocationtest D/com.marianhello.bgloc.service.LocationServiceImpl: New location BGLocation[fused 13.683869,-89.273859 id=null acc=5 t=1607036652000 et=+11d18h22m17s458ms alt=929.0 vel=0.5 bear=284.0 {Bundle[mParcelledData.dataSize=52]} locprov=1]
2020-12-03 17:04:08.352 1897-1897/com.geolocationtest D/com.marianhello.bgloc.BackgroundGeolocationFacade: Received MSG_ON_LOCATION
2020-12-03 17:04:08.369 1897-2511/com.geolocationtest I/ReactNativeJS: '[DEBUG] BackgroundGeolocation location', { id: 54,
bearing: 284,
altitude: 929,
mockLocationsEnabled: false,
speed: 0.5,
accuracy: 5,
latitude: 13.6838688,
longitude: -89.2738585,
locationProvider: 1,
time:<PHONE_NUMBER>000,
isFromMockProvider: false,
provider: 'fused' }

I already found the problem. The bug is in android 10, I tested in a Samsung S8 plus with android 9 and works pretty well. I don't know how to do the migration but that's all.

I was reading in many places and it seems that the only way to use this library in android 10 is using the android.permission.ACCESS_BACKGROUND_LOCATION in the manifest and selecting always use location in the interface. I don't know if there is a way to solve it without this permission but its not likely. I keep this issue open if somebody has another idea but my recomendation is to condition the permessions in order to make it functional in all devices.

Hi,
Due to privacy concerns it will NOT be possible to use this library without this permission setup. This simply has to do with the way Android implements the allowance of this permission. And to add additional information for Android 11 the permission dialog will no longer show the "always permissions" in the dialog. With my application we added an additional custom dialog that informs the user about these changes. Hope this helps a little

In android 10 u need a permission "allow all the time" or else it will terminate the process in the background after couple of seconds

In android 10 u need a permission "allow all the time" or else it will terminate the process in the background after couple of seconds

I created fork to solve this issue. (And it also fixes Gradle4 issue)
https://github.com/darron1217/react-native-background-geolocation
You can install it by command below
npm install https://github.com/darron1217/react-native-background-geolocation#develop

I created fork to solve this issue. (And it also fixes Gradle4 issue)
https://github.com/darron1217/react-native-background-geolocation
You can install it by command below
npm install https://github.com/darron1217/react-native-background-geolocation#develop

@darron1217 hi is this also solved the stationary issue?

@rosnaib11 I haven't experienced stationary issue.
Would you submit an issue with additional information on my repo?


Verify that stun server still works

it boots and monitor reports its running status ok, but it must be tested live. But can be closed, as this was just to verify i boots,