Expose tags_all from the eks managed node groups

Is your request related to a new offering from AWS?
Is this functionality available in the AWS provider for Terraform? See CHANGELOG.md, too.

No 🛑: please wait to file a request until the functionality is avaialble in the AWS provider
Yes ✅: please list the AWS provider version which introduced this functionality

Not really.
Is your request related to a problem? Please describe.
Not really a problem or bug but limitation of the module. As of now this module exposes only the eks-managed-group.labels value as shown here. Though for our use case we need to have access to the tags_all that have been associated to the managed group as well. They are exposed by the native module already as shown here
Describe the solution you'd like.

As part of the terraform-aws-eks/modules/eks-managed-node-group/outputs.tf we would like to have not only the output node_group_labels but also output node_group_tags_all
Describe alternatives you've considered.
I've tried to derive the info from node_group_resources output but it's not available there either.
Additional context

What is the use case for this?

@bryantbiggs we add some custom tags to the auto_scaling_group as shown in this example.
Though specifically to our use case we need to derive that information from the tags we assign to the managed group rather than the labels, because one of the labels is dynamic.
For example, we need to assign the tag is-disk-mount-available=1 to the ASG. But we cannot add that tag to the labels because that label needs to be added dynamically by another process only when the disk-mount is actually available. But at the same time we need to add that tag to the ASG so that the autoscaler can scale from 0 to 1+, otherwise pods never get scheduled.

I'm sorry, I'm not following

no worries. I'll try again with a concrete example.
Our node_group config is as follows:
 "public_nodes" = {
    ...
    labels = {
      nodegroup-type = "public"
    }
    tags = {
        Name                                                           = "public-node",
        "k8s.io/cluster-autoscaler/enabled"                            = "true",
        "k8s.io/cluster-autoscaler/node-template/label/nodegroup-type" = "public"
    }
    ...
}

the above works fine when a pod to be scheduled has the appropriate selector such as:
 nodeSelector:
    nodegroup-type: public

and at the same time the autoscaler can scale the group from 0 to +1 without any problem.

Though we have some pods that have a selector such as:
 nodeSelector:
    nodegroup-type: public
    is-mounted-disk-available: true

these pods are scheduled on the appropriated nodes natively by the k8s scheduler but the label (is-mounted-disk-available: true) is added dynamically on those nodes during runtime by another process that runs on the cluster that adds the label only at runtime and not when the ec2 machine is provisioned.
All the above work fine apart from one thing. If the ASG is scaled to 0 and a new & only pod with the selector
 nodeSelector:
    nodegroup-type: public
    is-mounted-disk-available: true

is deployed on the cluster, then the autoscaler cannot scale the group from 0 because there is no tag k8s.io/cluster-autoscaler/node-template/label/is-mounted-disk-available=true but only the k8s.io/cluster-autoscaler/node-template/label/nodegroup-type=public.
One solution would be to adjust the labels as follows
    labels = {
      nodegroup-type = "public"
      is-mounted-disk-available: true
    }

but that would be misleading during normal operation because as mentioned this label is added dynamically by another process during deterministic time.
So our thinking is that instead of adding to the labels to add the required tag to the tags attribute of the node_group as follows:
 "public_nodes" = {
    ...
    labels = {
      nodegroup-type = "public"
    }
    tags = {
        Name                                                           = "public-node",
        "k8s.io/cluster-autoscaler/enabled"                            = "true",
        "k8s.io/cluster-autoscaler/node-template/label/nodegroup-type" = "public"
        "k8s.io/cluster-autoscaler/node-template/label/is-mounted-disk-available: true
    }
    ...
}

that way we still keep the dynamic approach to the labels but at the same time we apply the correct tags to the ASG that we derive from the tags rather than the labels attribute.
I hope the above provides more info on this not so trivial edge case.

so it sounds like you want to add tags to the autoscaling group of the managed nodegroup, which is something that is supported today. I'm not seeing the connection as to why the tags_all property needs to be exported

I believe my example above clearly states that we already use tags on the ASG and we automatically & dynamically create those ec2 asg tags based on the info we get from node_group.labels value. Instead I'd like to have the ability to get the values for the ASG tags from the node_group.tags_all as well.
This info is already provided by the underlying aws terraform module.
Feel free to close the issue though if no action will be taken.

thank you, but unfortunately this is not a change that we will be pursuing here. there are implications around creating cyclical dependencies here, it would be better to use local variables in your implementation to sync changes to both labels and tags if that is the desired intent


How to fix this error?

Operating System: Windows
Game: (link to Steam page or Game)
Backend: GPU
An URL to your GamePlugin or GameAgentPlugin repository, if possible:

Expected result
Creating dataset
Encountered result
PAUSED

Traceback (most recent call last):
  File "C:\ProgramData\Anaconda3\envs\serpent\lib\runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "C:\ProgramData\Anaconda3\envs\serpent\lib\runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "C:\ProgramData\Anaconda3\envs\serpent\Scripts\serpent.exe\__main__.py", line 7, in <module>
  File "C:\ProgramData\Anaconda3\envs\serpent\lib\site-packages\serpent\serpent.py", line 52, in execute
    command_function_mapping[command](*sys.argv[2:])
  File "C:\ProgramData\Anaconda3\envs\serpent\lib\site-packages\serpent\serpent.py", line 380, in record
    frame_spacing=int(frame_spacing)
  File "C:\ProgramData\Anaconda3\envs\serpent\lib\site-packages\serpent\game.py", line 202, in play
    raise e
  File "C:\ProgramData\Anaconda3\envs\serpent\lib\site-packages\serpent\game.py", line 198, in play
    game_agent.on_pause(frame_handler=frame_handler, **kwargs)
  File "C:\ProgramData\Anaconda3\envs\serpent\lib\site-packages\serpent\game_agent.py", line 124, in on_pause
    on_frame_handler_pause(**kwargs)
  File "C:\ProgramData\Anaconda3\envs\serpent\lib\site-packages\serpent\game_agent.py", line 310, in on_record_pause
    active_keys.remove(key_name)
KeyError: 'KEY_LEFT_ALT'

Steps to reproduce

serpent record persona serpentpersonagameagent
alt + tab to anaconda prompt

Can you help me, please?

Got a new error:
Writing Recorded Input Data to 'datasets/input_recording.h5'... (1/65)
Traceback (most recent call last):
  File "C:\ProgramData\Anaconda3\envs\serpent\lib\runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "C:\ProgramData\Anaconda3\envs\serpent\lib\runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "C:\ProgramData\Anaconda3\envs\serpent\Scripts\serpent.exe\__main__.py", line 7, in <module>
  File "C:\ProgramData\Anaconda3\envs\serpent\lib\site-packages\serpent\serpent.py", line 52, in execute
    command_function_mapping[command](*sys.argv[2:])
  File "C:\ProgramData\Anaconda3\envs\serpent\lib\site-packages\serpent\serpent.py", line 380, in record
    frame_spacing=int(frame_spacing)
  File "C:\ProgramData\Anaconda3\envs\serpent\lib\site-packages\serpent\game.py", line 202, in play
    raise e
  File "C:\ProgramData\Anaconda3\envs\serpent\lib\site-packages\serpent\game.py", line 198, in play
    game_agent.on_pause(frame_handler=frame_handler, **kwargs)
  File "C:\ProgramData\Anaconda3\envs\serpent\lib\site-packages\serpent\game_agent.py", line 124, in on_pause
    on_frame_handler_pause(**kwargs)
  File "C:\ProgramData\Anaconda3\envs\serpent\lib\site-packages\serpent\game_agent.py", line 336, in on_record_pause
    data=[game_frame.frame_bytes for game_frame in game_frame_buffer.frames]
  File "C:\ProgramData\Anaconda3\envs\serpent\lib\site-packages\h5py\_hl\group.py", line 136, in create_dataset
    dsid = dataset.make_new_dset(self, shape, dtype, data, **kwds)
  File "C:\ProgramData\Anaconda3\envs\serpent\lib\site-packages\h5py\_hl\dataset.py", line 118, in make_new_dset
    tid = h5t.py_create(dtype, logical=1)
  File "h5py\h5t.pyx", line 1634, in h5py.h5t.py_create
  File "h5py\h5t.pyx", line 1656, in h5py.h5t.py_create
  File "h5py\h5t.pyx", line 1711, in h5py.h5t.py_create
TypeError: Object dtype dtype('O') has no native HDF5 equivalent

No, I got the old one, again.

File "C:\ProgramData\Anaconda3\envs\serpent\lib\site-packages\serpent\game_agent.py", line 310, in on_record_pause
    active_keys.remove(key_name)
KeyError: 'KEY_LEFT_ALT'

It might be possible that, when you use alt+tab, Serpent tries to record those inputs, but fails doing so, which throws an error.
Try defining a pause function using another key

File "C:\ProgramData\Anaconda3\envs\serpent\lib\site-packages\serpent\game_agent.py", line 310, in on_record_pause
    active_keys.remove(key_name)
KeyError: 'KEY_LEFT_ALT'

It might be possible that, when you use alt+tab, Serpent tries to record those inputs, but fails doing so, which throws an error.
Try defining a pause function using another key

I tried to run the game in a window and then just move the focus to the anaconda with the mouse, but then I got the last error that I wrote, that one with h5py

It seems that h5py is either receiving frames with different shapes(which I guess it's unlikely) or it's receiving frames with different data types(one type can be int32 and another can be float32 for example).
Try making sure that your frames are all on the same type with .astype('') and check your frame sizes. If the game window size gets smaller during recording, perhaps serpent start capturing that smaller frame which throws an error in h5py.

It seems that h5py is either receiving frames with different shapes(which I guess it's unlikely) or it's receiving frames with different data types(one type can be int32 and another can be float32 for example).
Try making sure that your frames are all on the same type with .astype('') and check your frame sizes. If the game window size gets smaller during recording, perhaps serpent start capturing that smaller frame which throws an error in h5py.

I'm just beginning to research SerpentAI and I don't know anything about its settings. I'm following this video. The only thing I can say about the settings is that I set the resolution to 960x540 for SerpentAI.
Video:
https://user-images.githubusercontent.com/84536195/198279139-36762b15-94f7-4820-9bcc-adbe8ec0bf92.mp4


Responsive elements produce odd results at some viewport sizes

Note the copyright footer, it matches with the nav bar and jumbo tron, so looks odd at some viewport sizes.

I'm not sure if we should rewrite the footer to move over with the content, or if there is something else is going on.

It should be fixed after migrating to docsy. (I believe so).


[WIP] 🐛  optimise 008 date migration fixture

finally refs #7351
The written date fixture migration in 0.11, which should trigger a re-run of 006 date migration was not strong enough. This PR pushes an optimisation.
Another PR will follow to fix the issue for people who already updated to 0.11.
TODO

[ ] think about if all makes sense
[ ] test another time

will final test this PR and #7461 tomorrow in the morning.
both were already tested, but i want to ensure all will work when we release 0.11.1

ready for review 👍


Content-length=0 header in HttpMessage.headers for GET request that doesnt contain Content-length at all

https://github.com/netty/netty/blob/e5951d46fc89db507ba7d2968d2ede26378f0b04/codec-http/src/main/java/io/netty/handler/codec/http/HttpObjectAggregator.java#L230
Cant understand why it should be added for requests that doesnt contain Content-length header at all . Isn't the purpose of FullHttpMessage to represent corresponding http request maximum accurately. If so, why should we add header that doesn't exist in original query?

It just tries to handle things consistent... Why would there be an issue to add the header ? It is inline with the spec and so should not have any negative side-effect.s


Fixing Overall Components

Fix :

Category Section Style
Link from all Footer text

Need to refactor :

dummyData writing for footer component
cursor disappearing at footer text

To fix cursor disappearing from footer text, just remove "a" from clickables property of animated cursor in App.jsx


Exclamation mark on homepage

I just discovered a exclamation mark on the icon of the awning on the homepage of the software. The awning works just fine, no issues.
What does this mean?
Thanks!

That is a windy warning.  Do you have a wind sensor?  If not go back into the Virtual Remote and send a Sensor command without either the sun or wind options checked.

No wind sensor attached but your suggestion solved it. Thanks!


Add graphviz in Dockerfile

graphviz is not currently installed, which triggers warnings when running the corresponding GitHub Action.

Thanks for the PR, @MaxDesiatov. I would have sworn that this was already in there, but I'm probably conflating adding GraphViz as a dependency in the Homebrew formula. Anyway, glad to have this fixed, and looking forward to having this available in the forthcoming release.

Thanks, I also have a prebuilt image here that I'm using in the meantime to make the action run faster, just in case you're interested https://hub.docker.com/r/maxdesiatov/swift-doc
I hope you could upload the official one in your Docker Hub namespace at some point.

(This required me to create two separate Dockerfiles in my swift-doc fork, one for the prebuilt image, the other for the action that only references the prebuilt image and does nothing else)

@MaxDesiatov

Thanks, I also have a prebuilt image here that I'm using in the meantime to make the action run faster, just in case you're interested https://hub.docker.com/r/maxdesiatov/swift-doc
I hope you could upload the official one in your Docker Hub namespace at some point.

That's great! Let's continue this discussion on #111.


security: use<EMAIL_ADDRESS>alias

See https://github.com/kubernetes/sig-release/issues/326 for context on
the change.

/lgtm

/lgtm
/approve

/approve


lift tactic does not respect abbreviations

import Mathlib.Tactic.Lift

abbrev MyNat := ℕ

example (n : ℤ) (hn : n ≥ 0) : True := by
  lift n to MyNat using hn
  -- type of `n` is `ℕ` instead of `MyNat`, in the goal state
  trivial

This means that we can not use dot-notation on n for all the juicy decls in the MyNat namespace.
A concrete example of this is LieIdeal K L which is an abbreviation of LieSubmodule K L L.

lift seems to explicitly hard code use of Type. How is this being used for universe polymorphic types like LieSubmodule?

Similar problem with ENat, WithTop, and Option.

ENat and WithTop aren't abbrevs, so lift shouldn't see through it.

lift seems to explicitly hard code use of Type. How is this being used for universe polymorphic types like LieSubmodule?

https://leanprover-community.github.io/mathlib4_docs/Mathlib/Algebra/Group/Submonoid/Membership.html#list_prod_mem uses lift but is universe polymorphic


basic_string::substr: __pos (which is 140) > this->size() (which is 0)

I get the following error "basic_string::substr: __pos (which is 140) > this->size() (which is 0)" when running the tool.
I have around 20k labeled images

This is also the case when trying to train a model using this dataset

I get the following error "basic_string::substr: __pos (which is 140) > this->size() (which is 0)" when running the tool.
I have around 20k labeled images

I have same issue.

I had the same issue with yolov3, anyone solve this ?

I had the same issue with yolov3, anyone solve this ?

I‘m not sure if I remember correctly but I think it was caused by wrong formatting of the train.txt on my part.

Please help me to solve this error.

Oh it was my silly mistake , I have wrongly placed obj.names file in darknet/cfg folder instead of darknet/data folder.


Account Spotlight page is showing blocks and snarks from unrelated accounts

Link an example of this issue please.


Jobs not running in Production

No issues in development, sidekiq-cron jobs run as expected and display in the web ui. But in production (testing) the web ui tells me "No cron jobs found" and they fail to run.
Gemfile
gem 'sidekiq'
gem "sidekiq-cron", "~> 1.1"

sidekiq.rb
Sidekiq.configure_server do |config|
  config.redis = { url: 'redis://<IP_ADDRESS>:6379/0', password: Rails.application.credentials[:redis_pass] }
  schedule_file = "config/schedule.yml"
  if File.exist?(schedule_file) && Sidekiq.server?
    Sidekiq::Cron::Job.load_from_hash YAML.load_file(schedule_file)
  end
end

Sidekiq.configure_client do |config|
  config.redis = { url: 'redis://<IP_ADDRESS>:6379/0', password: Rails.application.credentials[:redis_pass] }
end

schedule.yml
specials_notifications:
  cron: "03 * * * 1-5"
  class: "SpecialsNotificationWorker"

specials_notification_worker.rb
class SpecialsNotificationWorker
  include Sidekiq::Worker
  sidekiq_options retry: 0

  def perform
    User.managers.each do |user|
      user.send_notifications!
    end
  end
end

I think that's all relevant files. Am I looking at the wrong place? Is this likely a configuration of Sidekiq or Redis in production that I need to fix?

I suspect this is a configuration error on my end, not directly related to this gem. Likely something with how I'm initializing Sidekiq. I'll close this issue as it seems unlikely to be an issue with this gem.


Default scheduler handling for storIOSQLite

Part of #511

[Current coverage][cc-pull] is 95.74%

Merging [#660][cc-pull] into [master][cc-base-branch] will decrease coverage by 0.77%

@@             master       #660   diff @@
==========================================
  Files            84         84
  Lines          2548       2650   +102
  Methods           0          0
  Messages          0          0
  Branches        325        357    +32
==========================================
+ Hits           2459       2537    +78
- Misses           89        113    +24
  Partials          0          0

Powered by Codecov. Last updated by [d5430ab...cbd1a2f][cc-compare]
[cc-base-branch]: https://codecov.io/gh/pushtorefresh/storio/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/pushtorefresh/storio/compare/d5430abb97c23eac385074ad2e43c7df2fd14055...cbd1a2f75b352992375614807234814754e78fbb
[cc-pull]: https://codecov.io/gh/pushtorefresh/storio/pull/660?src=pr

@artem-zinnatullin, @karlicoss is there way to check that current observable scheduler is same as expected?

No, why you need it?
On Mon, 30 May 2016, 07:47 Dmitrii Nikitin<EMAIL_ADDRESS>wrote:

@artem-zinnatullin https://github.com/artem-zinnatullin, @karlicoss
https://github.com/karlicoss is there way to check that current
observable scheduler is same as expected?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/pushtorefresh/storio/pull/660#issuecomment-222410800,
or mute the thread
https://github.com/notifications/unsubscribe/AA7B3HJEPOv85a56r0AOmRjyvHXW6PsPks5qGmvbgaJpZM4IpXmI
.

To check that prepared operations apply it properly. Now I just check that we call defaultScheduler getter.

Ah, RxJava has TestScheduler for that so you can check that until you
explicitly call testScheduler.triggerActions() no code will be executed.
As @karlicoss about it!
On Mon, 30 May 2016, 10:01 Dmitrii Nikitin<EMAIL_ADDRESS>wrote:

To check that prepared operations apply it properly. Now I just check that
we call defaultScheduler getter.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/pushtorefresh/storio/pull/660#issuecomment-222425076,
or mute the thread
https://github.com/notifications/unsubscribe/AA7B3M-Nw1nze8fMzMm_iEU07Tr1KWSTks5qGotNgaJpZM4IpXmI
.

ok

Nice to see you here, man!
On Tue, 19 Jul 2016, 18:22 StanisVS<EMAIL_ADDRESS>wrote:

ok
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/pushtorefresh/storio/pull/660#issuecomment-233667836,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA7B3NHfqJ9Aq8wRy3R7WvQkO5UoTyOHks5qXOutgaJpZM4IpXmI
.

@artem-zinnatullin done! Check please)

@nikitin-da please commit without amending, GitHub allows us squash with button click before merge and it shows diffs between commits in PR!

👍


Fix nses indexing

There wasn't a check that the index was within the maximum allowed range.

I was tipped off to this issue, because a proteolysis product outside of the max peptide length allowed was generated and caused a crash. That still needs to be fixed in mzlib. But this will prevent any crashes for users that don't specify length constraints.


Update CleanupCartTaskHandler to use database index

Updated cart cleanup sql script to utilize database indices (created_at) during deletion.
1. Why is this change necessary?
The existing SQL snippet to delete the outdated cart entries doesn't use any database indices to narrow down entries that can be deleted. On high traffic shops this leads to SQL query times larger than 30 seconds to find and remove these database entries.
2. What does this change do, exactly?
This change reorders the sql query arguments so that the index idx.cart.created_at can be used during data selection.
3. Describe each step to reproduce the issue or behaviour.
Try to run the cart cleanup command on a cart table with over 65k cart entries and check the query time. In our case the query took over 30 seconds to delete 1000 outdated cart entries. After the optimization the same delete routine completes in under a second.
4. Please link to the relevant issues (if any).
N/A
5. Checklist

[x] I have rebased my changes to remove merge conflicts
[ ] I have written tests and verified that they fail without my change
[x] I have created a changelog file with all necessary information about my changes
[x] I have written or adjusted the documentation according to my changes
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

🤖 Generated by Copilot at c0d8e53
This pull request improves the performance of the cart cleanup SQL query by reordering the query parameters and documents the decision and consequences in a new ADR file. The affected files are adr/2023-05-09-optimise-cart-cleanup.md and src/Core/Checkout/Cart/Cleanup/CleanupCartTaskHandler.php.

🤖 Generated by Copilot at c0d8e53

Add a new ADR document to explain the optimization decision for the cart cleanup query (link)
Reorder the query parameters in the CleanupCartTaskHandler class to use the indexed created_at field first and the updated_at field second (link)

@mstegmeyer
I found an issue that describes the problem (NEXT-23615, slow deletion of cart entries) and have used that in the changelog although it was marked as solved.

Hey @RobertLang thank you for you contribution, the PR has been merged 🎉 💙


[Snyk] Upgrade @hashgraph/sdk from 2.34.1 to 2.38.0

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade @hashgraph/sdk from 2.34.1 to 2.38.0.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 6 versions ahead of your current version.
The recommended version was released 21 days ago, on 2023-11-21.

Release notes

Package name: @hashgraph/sdk

2.38.0 - 2023-11-21

2.37.0 - 2023-10-31What's Changed

Bug on mnemonic.toStandardECDSAsecp256k1PrivateKey() and mnemonic.toEcdsaPrivateKey() by @ svetoslav-nikol0v in #1989
Add Long and BigNumber support for ContractFunctionParameters methods #1972 by @ svetoslav-nikol0v in #1997
Update readme file by @ svetoslav-nikol0v in #2004

Full Changelog: v2.36.0...v2.37.0

2.37.0-beta.2 - 2023-10-23v2.37.0-beta.2
What's Changed

Bug on mnemonic.toStandardECDSAsecp256k1PrivateKey() and mnemonic.toEcdsaPrivateKey() by @ svetoslav-nikol0v in #1989

2.37.0-beta.1 - 2023-10-23What's Changed

Bug on mnemonic.toStandardECDSAsecp256k1PrivateKey() and mnemonic.toEcdsaPrivateKey() by @ svetoslav-nikol0v in #1989

Full Changelog: v2.36.0...v2.37.0-beta.1

2.36.0 - 2023-10-17What's Changed

Fix the fromBytes for tokenWipeTransaction by @ ochikov in #1968
Exposing client operator with a getter by @ rokn in #1970
Examples related changes by @ svetoslav-nikol0v in #1953
Fix integration tests and remove .only by @ ochikov in #1971

Full Changelog: v2.35.0...v2.36.0

2.35.0 - 2023-10-04What's Changed

Fix integration tests by @ ochikov in #1896
Update packages and release beta versions by @ ochikov in #1891
Fix typos by @ omahs in #1889
Fixed duplicated doc comments by @ 0xMimir in #1888
Fix topic subscription error handling by @ rokn in #1907
Update precompile system contracts by @ petreze in #1910
Update/packages vol2 by @ ochikov in #1905
Ignore minor and patch updates by @ ochikov in #1937
Fix/dependabot updates by @ ochikov in #1938
Fix/dependabot updates by @ ochikov in #1939
Fix/dependabot updates by @ ochikov in #1940
Add keepalive_time property when we init grpc client by @ petreze in #1908
Update the precompiled jsons of the contracts by @ ochikov in #1954
Add setDefaultMaxQueryPayment() and deprecate setMaxQueruPayment() by @ svetoslav-nikol0v in #1897
Fixed toJSON functionality in TranasctionRecord/Receipt/Response by @ rokn in #1956
[#1857] Аdding task to run all example by @ svetoslav-nikol0v in #1955
Adding new env variable by @ svetoslav-nikol0v in #1957

New Contributors

@ omahs made their first contribution in #1889
@ 0xMimir made their first contribution in #1888
@ svetoslav-nikol0v made their first contribution in #1897

Full Changelog: v2.34.1...v2.35.0

2.34.1 - 2023-09-15What's Changed

Follow convention toJSON by @ petreze in #1885

Full Changelog: v2.34.0...v2.34.1

    from @hashgraph/sdk GitHub release notes

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (92c32bf) 78.22% compared to head (d5d347d) 78.22%.

:exclamation: Current head d5d347d differs from pull request most recent head 928025b. Consider uploading reports for the commit 928025b to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1975   +/-   ##
=======================================
  Coverage   78.22%   78.22%
=======================================
  Files          39       39
  Lines        3068     3068
  Branches      619      619
=======================================
  Hits         2400     2400
  Misses        488      488
  Partials      180      180

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Kata Containers 1.10.0-alpha0

sandbox/cgroups: don't constrain if using SandboxCgroupsOnly
cli: add kata-overhead subcommand
versions: support specify version in versions.yaml for rust agent.
virtcontainers/sandbox: calculate container's CPU from sandbox.contai…
ci: Fix versions_checker.sh
ci: Fix versions_checker.sh
virtcontainers: unmount host mounts if container can't be created
virtcontainers/store: make VCStoreUUIDPath rootless
virtcontainers/annotations: use right domain name for kata annotations
v2: Change the event and error behavior of pause/resume
Update cni plugin version
Load state early so that hypervisor can store the correct state

e4c816b versions: support specify version in versions.yaml for rust agent.
691a6a7 sandbox/cgroups: don't constrain if using SandboxCgroupsOnly
7fe0100 cli: add kata-overhead command
1bbc1d5 virtcontainers: add StatsSandbox to vc API
569bd78 virtcontainers: change pass by value to pass by reference
5b226d0 ci: Fix versions_checker.sh
f8b84d7 ci: Fix versions_checker.sh
24d7aff virtcontainers: change pass by value to pass by reference
abec17f virtcontainers/store: make VCStoreUUIDPath rootless
eca7bd2 virtcontainers: unmount host mounts if container can't be created
91bd095 virtcontainers/annotations: use right domain name for kata annotations
c1060a3 v2: Change the event and error behavior of pause/resume
f6a10bc state: Refactor code to move all the state load code
fa4acad state: Load the state from storage early on
929c4e7 network: Change NewNS() call
c0995c6 vendor: Vendor the latest CNI plugins
86d8346 version: Update the version for cni plugins

/test

Travis choked on:
INFO: Tagging Kata Containers with version 1.10.0-alpha0
INFO: Check all repos has version 1.10.0-alpha0 in VERSION file
INFO: agent is in 1.10.0-alpha0
INFO: ksm-throttler is in 1.10.0-alpha0
INFO: osbuilder is in 1.10.0-alpha0
INFO: packaging is in 1.9.0-rc0
ERROR: packaging is not in version 1.10.0-alpha0

sent rebuild now that packaging is on 1.10.0-alpha0


Import shared components

This PR deletes most of /components and /util, and imports them from their new shared location at https://github.com/hetio/frontend-components

+82 −1,813 lines... nice. Will let @dongbohu do the actual code review, since it's out of my domain.

+82 −1,813 lines... nice. Will let @dongbohu do the actual code review, since it's out of my domain.

I realized Dongbo is out today. I don't think it needs review though because it's literally just replacing import statements to point to the new package.
Working on this CI build failure now.

@dhimmel can you take a look at https://circleci.com/gh/greenelab/hetmech-frontend/442?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link and see if you have an idea of what's going wrong. It's having trouble installing the shared components package.


Update where and import-sorting guides

See distinct commits.

Agreed I thought this was weird too, but we're actually already using it in a surprising number of places in the codebase. I'm down for whatever as long as we're consistent, and I do like the vertical alignment above and below the where line.

Same here. I had a tough time acclimating to this, but I'm OK with it now. In my own 4-space world, this same style works out to 2-space/4-space instead of 1-space/2-space, so it's a little less weird.
Just to clarify, I'm not introducing this style here; it was already our preference. My intent here is to stop allowing inconsistencies and have the guide specify only one style.

Yeah, I also thought 1 space was odd at first, but I liked the alignment and context free love it brings.
Normal expression:
foo =
  bar
 where
  bar = "baz"

do notation:
foo = do
  bar
  bats
 where
  bar = putStrLn "baz"


Improve error messages

Like discussed in imagemin/imagemin#32, we should improve error messages to include instructions on what dependencies are needed for builds to succeed.
I've been thinking about introducing a .dependency(dep, os) method which collects the dependencies and if the build fails it writes a error message and lists the dependencies needed.
@sindresorhus, wdyt?

At the very least, it would be very useful for binbuild to explain in a log message which command failed and what was CWD of the command so you can find the logs.
e.g. ./configure ...
Lets say that the system doesn't have a necessary library installed, how would I find out which library to install?

Sorry, totally forgot about this. I'll try fix something asap.

In the meantime, can you help me out with where to find the logs for the commands that run? I tried looking in npmcache, and the tmpdir. Are they getting nuked after npm exits?

Sorry, forgot to say Thanks for looking into it! I appreciate the help and the quick response.

The commands aren't logged. Look at the code in the modules instead.

That may be the case, but for instance, running ./configure will produce a config.log file somewhere in the build directory or the source directory. What is the current working directory when the commands are run?

It's a temp directory so it may vary.


[stable/phabricator] Replace maintainer in Chart.yaml

Use Bitnami as maintainer instead of bitnami-bot
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[X] DCO signed
[X] Chart Version bumped
[X] Variables are documented in the README.md

/lgtm


I want to know what’s in the file “seeta_fa_v1.0.bin”and “seeta_fd_frontal_v1.0.bin”

I test all the examples in each part，but i still don't know  what’s in the file “seeta_fa_v1.0.bin”and “seeta_fd_frontal_v1.0.bin”，could you explain it for me?
Thank you!

The parameters of face alignment model and face detecton model are stored in  in the file “seeta_fa_v1.0.bin”and “seeta_fd_frontal_v1.0.bin” respectively. BTW, the face alignment model (recorded as seeta_fa_v1.1.bin) together with the code are updated to version 1.1, which can achieve better alignment peformance.

@KangolHsu The model file stores the parameters of the classifiers. You can check out the code for loading the model for details.

Thank you!

How do  you train this parameters and what is the method you use ? could you give me some clues about it ?

You may find the details in the our papers.
For face detection:
@article{wu2016fust,
Author = {Wu, Shuzhe and Kan, Meina and He, Zhenliang and Shan, Shiguang and Chen, Xilin},
Journal = {Neurocomputing (under review)},
Title = {Funnel-Structured Cascade for Multi-View Face Detection with Alignment-Awareness},
Year = {2016}
}
For face alignment:
@inproceedings{zhang2014coarse,
title={Coarse-to-fine auto-encoder networks (cfan) for real-time face alignment},
author={Zhang, Jie and Shan, Shiguang and Kan, Meina and Chen, Xilin},
booktitle={European Conference on Computer Vision},
year={2014},
organization={Springer}}

@KangolHsu The face detection paper is now available on arXiv: http://arxiv.org/abs/1609.07304.


Add inline implementation for Oauth2 token

Purpose

The sample code by itself wasn't that useful as it required you to know how to create a token and then pass that to this program, meaning the current code can't be used standalone. Added an inline implementation for the Oauth2 token that is then used to call the ratecard API

Does this introduce a breaking change?

[ ] Yes
[x] No

Pull Request Type
What kind of change does this Pull Request introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test

Get the code

git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install

Test the code

```Run the program and you should retrieve the ratecard results successfully.

## What to Check
Verify that the following are valid
* Verify that your Subscription ID, Tenant ID, Client ID, Client Secret are known and populated.

## Other Information
<!-- Add any other helpful information that may be needed here. -->

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: snymanpaul sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Publish colors v3.1.1 and colors-branding v3.2.1

Publishes colors v3.1.1 and colors-branding v3.2.1
waiting for: https://github.com/brightlayer-ui/colors/pull/60

@ektasawant @bkarambe @joebochill  We'll need this PR to be merged before Tom's React component PRs can go in.


enable automatic dbt-core version requirements in setup.py

add functions from dbt-spark repo in setup.py to enable automatic dbt-core requirements based on major and minor version of the dbt-impala adapter

How did we test this. What version of dbt-core is this pulling

Good noting. Since the package version in this setup.py is 1.1.1, it should pull the latest dbt-core in 1.1.x


Update README.md

Rewrote mostly the "install" part to be more noob-friendly :)
Cleaned out the markdown errors.

little modifs : Docker 'compose' plugin is now mandatory as 'docker-compose' doesn't support 'include' instructions contained in compose files.

Or, as debian is often used, some warning like "these commands are meant to be used in a debian-based distro" ?

Edited. Only "apt" left is for nano, but I wrote a note to say it's for debian-based distros.


library 'XXXX' is not allowed to have state variable

I tried to compile a solidity contract into substrate. The contract imports SafeMath.sol into and SafeMath.sol is a library.
Solang compiler complained like library ‘SafeMath’ is not allowed to have state variable ‘ERROR_ADD_OVERFLOW’.
But the ERROR_ADD_OVERFLOW actually is a constant string type not a variable. I have read a reference about the different about constant and state varaible.  So I really want to know why.
Below is the source codes of SafeMath.sol. Using solang version v0.1.6
pragma solidity ^0.4.24;

/**
 * @title SafeMath
 * @dev Math operations with safety checks that revert on error
 */
library SafeMath {
    string private constant ERROR_ADD_OVERFLOW = "MATH_ADD_OVERFLOW";
    string private constant ERROR_SUB_UNDERFLOW = "MATH_SUB_UNDERFLOW";
    string private constant ERROR_MUL_OVERFLOW = "MATH_MUL_OVERFLOW";
    string private constant ERROR_DIV_ZERO = "MATH_DIV_ZERO";

    /**
    * @dev Multiplies two numbers, reverts on overflow.
    */
    function mul(uint256 _a, uint256 _b) internal pure returns (uint256) {
        // Gas optimization: this is cheaper than requiring 'a' not being zero, but the
        // benefit is lost if 'b' is also tested.
        // See: https://github.com/OpenZeppelin/openzeppelin-solidity/pull/522
        if (_a == 0) {
            return 0;
        }

        uint256 c = _a * _b;
        require(c / _a == _b, ERROR_MUL_OVERFLOW);

        return c;
    }

    /**
    * @dev Integer division of two numbers truncating the quotient, reverts on division by zero.
    */
    function div(uint256 _a, uint256 _b) internal pure returns (uint256) {
        require(_b > 0, ERROR_DIV_ZERO); // Solidity only automatically asserts when dividing by 0
        uint256 c = _a / _b;
        // assert(_a == _b * c + _a % _b); // There is no case in which this doesn't hold

        return c;
    }

    /**
    * @dev Subtracts two numbers, reverts on overflow (i.e. if subtrahend is greater than minuend).
    */
    function sub(uint256 _a, uint256 _b) internal pure returns (uint256) {
        require(_b <= _a, ERROR_SUB_UNDERFLOW);
        uint256 c = _a - _b;

        return c;
    }

    /**
    * @dev Adds two numbers, reverts on overflow.
    */
    function add(uint256 _a, uint256 _b) internal pure returns (uint256) {
        uint256 c = _a + _b;
        require(c >= _a, ERROR_ADD_OVERFLOW);

        return c;
    }

    /**
    * @dev Divides two numbers and returns the remainder (unsigned integer modulo),
    * reverts when dividing by zero.
    */
    function mod(uint256 a, uint256 b) internal pure returns (uint256) {
        require(b != 0, ERROR_DIV_ZERO);
        return a % b;
    }
}

and the error messages
/Users/xxxx/xxxx/math/SafeMath.sol:9:5-62: error: library ‘SafeMath’ is not allowed to have state variable ‘ERROR_ADD_OVERFLOW’
/Users/xxxx/xxxx/math/SafeMath.sol:10:5-64: error: library ‘SafeMath’ is not allowed to have state variable ‘ERROR_SUB_UNDERFLOW’
/Users/xxxx/xxxx/math/SafeMath.sol:11:5-62: error: library ‘SafeMath’ is not allowed to have state variable ‘ERROR_MUL_OVERFLOW’
/Users/xxxx/xxxx/math/SafeMath.sol:12:5-54: error: library ‘SafeMath’ is not allowed to have state variable ‘ERROR_DIV_ZERO’
/Users/xxxx/xxxx/math/SafeMath.sol:26:31-49: error: `ERROR_MUL_OVERFLOW' is not found
/Users/xxxx/xxxx/math/SafeMath.sol:35:25-39: error: `ERROR_DIV_ZERO' is not found
/Users/xxxx/xxxx/math/SafeMath.sol:46:27-46: error: `ERROR_SUB_UNDERFLOW' is not found
/Users/xxxx/xxxx/math/SafeMath.sol:57:26-44: error: `ERROR_ADD_OVERFLOW' is not found
/Users/xxxx/xxxx/math/SafeMath.sol:67:25-39: error: `ERROR_DIV_ZERO' is not found

This issue should be resolved in the latest master.
https://github.com/hyperledger-labs/solang/commit/fb6074294b8bb9622341c4e4c4d9df2d5ed7c773
Maybe it's time to do a release again. Let me resolve some issues first.

This issue should be resolved in the latest master.
https://github.com/hyperledger-labs/solang/commit/fb6074294b8bb9622341c4e4c4d9df2d5ed7c773
Maybe it's time to do a release again. Let me resolve some issues first.

If you can try the latest master that would be great!

If you can try the latest master that would be great!

No response so I assume it's fixed.

No response so I assume it's fixed.


Update docs to use Deno 1.10.x instead of latest v1

Deno 1.11 has just been released and it appears to be incompatible with the flat Deno package. Just updating the docs to use 1.10.x until this is resolved.
#45

Thanks for that update! We'll also update the default version in githubocto/flat-editor


[T5A1][W13-B3] Chua Yu Peng

Updated Dev Guide

Omitted from review because it contains unrelated changes.


MIP projects are not created into the category structure.

I have a project under 'category1/category2/project'. And this project is marked as a MIP. However, in obsidian this project is in the root folder.

Does this happen even if it's not labeled as an MIP? (I am trying to reproduce the issue).

I found this to happen only on my MIP projects.


Add WithSliceDeepMerge option

PR for issue #233.
Introduce new option mergo.WithSliceDeepMerge which can be tuned with mergo.WithOverride or mergo.WithOverwriteWithEmptyValue.
Test cases are provided in issue233_test.go for  mergo.WithSliceDeepMerge and combinations with mergo.WithOverride or mergo.WithOverwriteWithEmptyValue.
Test are being run successfully with go test.

issues noted by codeclimate addressed

commits squashed for clarify

Hello @imdario, can you please suggest, what next steps should be taken in order to move on with this PR?

@sunsingerus I'll review during the weekend.

@imdario any news about this PR?

@Slach I had no opportunity to review it in depth. I won't be able to try again until August as I'm getting married.

Sorry but adding more complexity to that condition in deepMerge doesn't feel safe. I understand that tests are green, but I already accepted some PR in the past with all green tests, and it caused a broken version of Docker.


Versioning info appears twice

We show up in the About dialog under 1.0, and under the tools -> Extensions and Updates menu.  The global VS about dialog has incorrect information.  We should follow the pattern of most other extensions and only show up under the Tools->E&U menu.

@AndreSayreMSFT, I don't think putting the version information only on the Tools->E&U dialog is the correct approach.
The correct version information should be also be in the global VS About dialog, IMHO. There are many cases where I need to submit a bug report and I need to Copy Info to show the Installed products.
That being said, if there is another way to export a list of all the installed products/extensions I'd be good with that too.

@gwojan Unfortunately, the information that currently shows up there has never given a complete picture.  Even without this extension installed, it shows the Version of the CommonAzureTools, And this extension at 1.0.
As we're an extension, we should match the majority of other extensions that could also show up in E/U, and I'd presume you'd want your log to have all extensions anyways :)
I've got a solution for you though!  Courtesy of this blog post, you can run devenv.exe /log C:\PickAPathHere.xml
Does that help?

@AndreSayreMSFT, that kinda, sorta helps. :wink: I'll figure something out, thanks.


Fix REPL compiler error and warning

Small fix for the error when building REPL with --features minimal. Fix lifetime eliding warning.

I see that compilation error raises only when project is built with --features minimal, probably because no_python is enabled. But I am not sure why Python is disabled when minimal is enabled?


Add direct Chimera unit-cell embedding composite

For several Leap JNs I wrote a function in the past that finds the first available Chimera unit cells to "directly" embed (map each node of a Chimera-indexed graph to one qubit) small problems. The typical use case is placing the problem below without worrying about the working graph of a particular QPU:

I'm now updating the Leap JNs for Pegasus, and went overboard on generalizing that code. I've often found this functionality useful and for Pegasus, with its more complicated coordinates, it's especially helpful, so I thought we might want it in Ocean.
An alternative approach would be to check that all nodes/couplers are in the needed number of unit cells. That code would be simpler but more wasteful. For the target graphs that's not really a big deal, though it's uglier. My main motivations for taking this approach was that it seemed a better approach pedantically for when I thought it would stay in the JNs and it was raining that weekend anyway.
If we want to add this to Ocean, I'd still need to update the RST files.

Before I dive too deeply in, it occurs to me that there is overlap with TilingComposite. Maybe we should consider merging these two together?

@arcondello, there are similarities but this one is specifically for when users want a one-to-one node-qubit fit (no chains) and control of placement. To bring this functionality into Tiling you'd need to (1) flag for single instance, which is simple, (2) enable/disable use of chains, which is a bigger change, (3) add vertical search for Pegasus columns -- I did not do that because I'm not sure there are use cases to justify the extra complications.

Tiling does not use chains, the "embedding" is all length-1 chains and just used for convenient mapping.
Agree (3) complicates things, but we'll need to handle that case when we adapt TilingComposite to Pegasus anyway...

I've updated the ReverseAnnealing and Anneal Schedule JNs to use dwavesystems/dwave-system#431-updated TilingComposite instead of the DirectEmbedding class. Closing.


Update dependencies

Armeria 0.60.0 -> 0.61.0
dependency-management-plugin 1.0.4 -> 1.0.5
gradle-docket-plugin 3.2.5 -> 3.2.6

Do not merge yet. Will update again once Armeria 0.62.0 is out today.


Force Streaming Video on Load

Hi,
thanks for the commit!
I can't seem to see the difference between the old implementation and this one, apart from the fact that the event is registered only when urls are given.
There is also a lot of repeated code, and the URLs are actually processed twice.
Care to share a bit more information on the need and uses of this?

From my testing HLS and Mpeg-Dash requires a full play through before the canplaythrough event will trigger.  I would suggest removing that event or finding some other even trigger.  Otherwise, you could end up waiting 1 minute or longer for your textures to appear.
It could be rewritten to something like:
this.video.endsWith(".m3u8") || this.video.endsWith(".mpd") ? true : false;

No need for testing on the extension. Can you please just factorize the repeated code!

Sorry not pro at git  :/

Sorry but I cannot valide a pr with 172 files:)


examples/mnist_swwae.py crashes

[x] Check that you are up-to-date with the master branch of Keras. You can update with:
pip install git+git://github.com/fchollet/keras.git --upgrade --no-deps

[x] If running on Theano,... (I'm running on tensorflow)

[x] Provide a link to a GitHub Gist of a Python script that can reproduce your issue (or just copy the script here if it is short).

I successfully ran python mnist_cnn.py from 7e2e7a5e5a43443122df0b497f88dd77fd3bfc7c on my GTX1080 with a TensorFlow R0.12rc0 backend. I then ran python mnist_swwae.py and got the following failure:
~/src/keras/examples on master
± python mnist_swwae.py
Using TensorFlow backend.
I tensorflow/stream_executor/dso_loader.cc:128] successfully opened CUDA library libcublas.so.8.0 locally
I tensorflow/stream_executor/dso_loader.cc:128] successfully opened CUDA library libcudnn.so.5 locally
I tensorflow/stream_executor/dso_loader.cc:128] successfully opened CUDA library libcufft.so.8.0 locally
I tensorflow/stream_executor/dso_loader.cc:128] successfully opened CUDA library libcuda.so.1 locally
I tensorflow/stream_executor/dso_loader.cc:128] successfully opened CUDA library libcurand.so.8.0 locally
X_train shape: (60000, 1, 28, 28)
60000 train samples
10000 test samples
Traceback (most recent call last):
  File "mnist_swwae.py", line 136, in <module>
    y = MaxPooling2D(pool_size=(pool_sizes[i], pool_sizes[i]))(y_prepool)
  File "/usr/local/lib/python2.7/dist-packages/Keras-1.1.2-py2.7.egg/keras/engine/topology.py", line 517, in __call__
    self.add_inbound_node(inbound_layers, node_indices, tensor_indices)
  File "/usr/local/lib/python2.7/dist-packages/Keras-1.1.2-py2.7.egg/keras/engine/topology.py", line 571, in add_inbound_node
    Node.create_node(self, inbound_layers, node_indices, tensor_indices)
  File "/usr/local/lib/python2.7/dist-packages/Keras-1.1.2-py2.7.egg/keras/engine/topology.py", line 155, in create_node
    output_tensors = to_list(outbound_layer.call(input_tensors[0], mask=input_masks[0]))
  File "/usr/local/lib/python2.7/dist-packages/Keras-1.1.2-py2.7.egg/keras/layers/pooling.py", line 158, in call
    dim_ordering=self.dim_ordering)
  File "/usr/local/lib/python2.7/dist-packages/Keras-1.1.2-py2.7.egg/keras/layers/pooling.py", line 207, in _pooling_function
    border_mode, dim_ordering, pool_mode='max')
  File "/usr/local/lib/python2.7/dist-packages/Keras-1.1.2-py2.7.egg/keras/backend/tensorflow_backend.py", line 1853, in pool2d
    x = tf.nn.max_pool(x, pool_size, strides, padding=padding)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/nn_ops.py", line 1617, in max_pool
    name=name)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/gen_nn_ops.py", line 1598, in _max_pool
    data_format=data_format, name=name)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/op_def_library.py", line 759, in apply_op
    op_def=op_def)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 2242, in create_op
    set_shapes_for_outputs(ret)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 1617, in set_shapes_for_outputs
    shapes = shape_func(op)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 1568, in call_with_requiring
    return call_cpp_shape_fn(op, require_shape_fn=True)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/common_shapes.py", line 610, in call_cpp_shape_fn
    debug_python_shape_fn, require_shape_fn)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/common_shapes.py", line 675, in _call_cpp_shape_fn_impl
    raise ValueError(err.message)
ValueError: Negative dimension size caused by subtracting 2 from 1 for 'MaxPool' (op: 'MaxPool') with input shapes: [?,1,32,8].
-> [1]

Help or feedback from anyone is welcome, @antonmbk you may be interested in this because you contributed the example. Thanks!

I can confirm. @antonmbk, thoughts?

This specific issue is due to the example being written under the assumption image_dim_ordering="th" (it should be made compatible with both modes). However, this is not the only issue with TF. There are further bugs.

The other bug is that the model is taking the gradient of a gradient, which isn't supported for all ops in TF. In particular MaxPool.

If this isn't easily fixable, then the example should include a check for the current backend and raise a helpful exception with TF. Also, start with K.set_image_dim_ordering('th').

@fchollet would a feature request issue in https://github.com/tensorflow/tensorflow for MaxPool gradient of a gradient make sense?

@ahundt yes it would, you should post one.
For the time being, I have "fixed" the example by doing what I mentioned above.

created https://github.com/tensorflow/tensorflow/issues/6294

Looks like this is already in progress:
https://github.com/tensorflow/tensorflow/issues/6143
With a pull request in progress:
https://github.com/tensorflow/tensorflow/pull/6299

Just seeing this, thanks for bringing it up @ahundt.  I'm hoping the tf people fix this in that PR you mentioned.

pull request update:
https://github.com/tensorflow/tensorflow/pull/6664
hopefully the relevant day is getting closer... should this perhaps be reopened since the real fix doesn't seem too far away?

@antonmbk @fchollet tensorflow/tensorflow#6664 is merged to master so maxpool grad grad should work, can this be reopened? Will a tf version check be appropriate so the error message appears only when appropriate?


Refactor code to have clean interface between parse layer and write to database and focus tests on parse layer

Refactor to focus the code on the first steps to get a JS/TS objects representing essential "data".  Create clear separation of concerns so that we can rapidly add new well tested features e.g. tag parsing, computed fields.
graph TD

markdown --remark-parse--> st[syntax tree]
st --extract features--> jsobj1[TS Object eg. File plus Metadata plus Tags plus Links]
jsobj1 --computing--> jsobj[TS Objects]
jsobj --convert to sql--> sqlite[SQLite markdown.db]
jsobj --write to disk--> json[JSON on disk in .markdowndb folder]
jsobj --tests--> testoutput[Test results]

Comment: we should write most of our tests at the js object level. Not at sqlite level.
Acceptance
See design sketch below.

[x] process.ts with a processFile function exists which given a path returns a File object or a derivative thereof ✅2023-11-15 this is done - see https://github.com/datopian/markdowndb/blob/8fcda1db4fc2b19eade703c25f91438eb6e69d30/src/lib/process.ts

[x] supports frontmatter extraction ✅2023-11-15 see tests https://github.com/datopian/markdowndb/blob/8fcda1db4fc2b19eade703c25f91438eb6e69d30/src/lib/process.spec.ts
[x] supports links ✅2023-11-15 ditto

[x] has tests ✅2023-11-15 see https://github.com/datopian/markdowndb/blob/8fcda1db4fc2b19eade703c25f91438eb6e69d30/src/lib/process.spec.ts

FUTURE

[ ] Bonus: Split out the sqlite wrapper to write to sqlite in a separate file and refactored current code to use the processFile function

Comment: we can ignore sqlite right now in this branch not worry about refactoring that code to write to sqlite from JS objects. This would simplify code and allow us to use typescript for everything for now. It is easy to convert json to sqlite later if we want.
Notes

[x] Use micromark to parse the markdown file? ✅2023-11-10 ❌ just use remark-parse for now

Sample code
// FUTURE - to show how code path will run
function buildMarkdownSQLDb(folder) {
  walkFolder(folder)
  for each file in folder:
    parseFile => File
    storeFileInDb(File) // or storeAllFiles at once later
}

// what we want to add ...
function parseFile(path) returns File
  getKeyInfoOnFile(path)
  if(isMarkdown) {
    parseMarkdownFile(string) // returns Metadata, Links, Tags, Tasks
    // add this info to the FileInfo ...
  } else {
    return FileInfo
 }
}

interface FileInfo {
  path

}

interface MarkdownFile extend File {
  // adds Metadata
  // adds Tags?
}

Schema
interface File {
  path: string; // relative path on disk, also will be primary key
  extension: string;
  metadata: MetaData | null; // frontmatter
}

interface Metadata {
  key: value
}

interface Link {
  link_type: "normal" | "embed";
  from: string; // path
  to: string; // to path or url
}

Design sketch for JSON on disk
NB: ❌ we don't need this atm
.markdowndb/
  files.json
  links.json

Define the schema for that in typescript

FIXED. Completed in https://github.com/datopian/markdowndb/commit/8fcda1db4fc2b19eade703c25f91438eb6e69d30


Timeout Middleware should return a 500 error if the result takes too long

This is a new one. 🤕
[info] Timeout Middleware should
[info]   + Have no effect if the response is not delayed (5 seconds, 325 ms)
[error]   x return a 500 error if the result takes too long (32 seconds, 840 ms)
[error]    '200 OK' is not equal to '500 Internal Server Error' (TimeoutSpec.scala:27)
[error] org.http4s.server.middleware.TimeoutSpec$$anonfun$5$$anonfun$apply$4.apply(TimeoutSpec.scala:27)
[error] org.http4s.server.middleware.TimeoutSpec$$anonfun$5$$anonfun$apply$4.apply(TimeoutSpec.scala:27)
[error] Actual:   200 OK
[error] Expected: 500 Internal Server Error

This was reported several refactorings ago, and is no longer a problem.


Add rpath of /usr/lib/swift and swift-5.5 compatibility library

Without it, sourcekit-lsp crashes on launch with the same errors as in https://github.com/apple/swift-package-manager/issues/5467.

@swift-ci Please test


Discrepancy on macro definition values ARG_MAX and PATH_MAX

While compiling some 3rd party code, I found warnings about re-definitions of a couple of symbols, namely ARG_MAX and PATH_MAX.  Searching ESP-IDF, I found both of these defined in components/newlib/include/limits.h and components/newlib/include/sys/syslimits.h.
In syslimits.h there are no redefinition guards.  The values of the pairs of variables are not consistent.  In one ARG_MAX = 4096 while in another ARG_MAX = 65536.  And also PATH_MAX = 4096 while in the other PATH_MAX = 1024.

Did you ever find a solution to this @nkolban ? I've got the same thing with the pcbreflux examples

Thankfully, the warnings are literally that ... warnings ... and other than not getting a pretty compile, all seems to function.  Hopefully the folks at Espressif will continue to track this issue and in good time will offer us an "out of the box" fix.  If you need an immediate workaround, I'd suggest picking one of the header files (probably syslimits.h) and comment out the duplicate symbols.  This will produce a clean compile.  The down-side is that when you bring in a new version of the ESP-IDF, the changes will be undone.

Thanks, I added redefinition guards to syslimits.h and the warnings have gone. I thought it was causing some other issues but wasn't the case


log4rs panics when conf file doesn't exist

Hello (: I've been playing around with log4rs, and there's some unexpected behavior: When the specified config file doesn't exist, log4rs panics instead of returning an Err Result variant. The panic message is the usual "file not found" Rust message: No such file or directory (os error 2).
This is on log4rs version 0.3.3.
Thanks!

Is it panicking or just printing the error message? At least running the test crate with the log config moved does not cause a panic but does print the error message out.

I suspect it's panicking, but I'll have a second look at my code. I defined a match against the Result from init_file, but it's not being executed, hence my assumption.

I see what the problem is now. My panic was coming from somewhere else. That being said, init_file always returns Ok, even if the file is not found, which might not be what the user expects.

This is just a suggestion: If it will always return Ok, wouldn't it make sense to default to a stdout fallback with only the root logger with an info level?
Also, perhaps it would be better to collect the Errs it encounters during initialisation and return them, instead of printing them to stderr.
Thanks again, and sorry about a false alarm.


Support Commercial Licenses

Hi there 👋!
We're excited that we can finally ship a SwiftPM package for PSPDFKit, since Xcode 12 now supports binary frameworks for SwiftPM.
https://swiftpackageindex.com/PSPDFKit/PSPDFKit-SP
We'd love to change the red "Unknown License" badge to "Commercial License", ideally linking to our website for further details. We did look at the currently available license versions and there's nothing that would fit our commercial licensing model currently.
Happy to help or contribute code to make this work, but first I want to start a discussion if this is something that people want.

Congratulations on the release! I was happy to see it come through the PackageList as a PR.
It's an interesting problem. As I'm sure you're aware, we use the GitHub license detection for this feature. Even if GitHub wanted to start detecting commercial licenses (which I don't think they do), it would be impossible as there's no standard "commercial license" template.
So, that leaves it as a problem for us to solve. I see two ways to approach this:
Metadata file
We're starting to add support for various bits of additional metadata in a .spi.yml file. It currently supports some build system settings, but we plan to use this for other metadata in the future. We've talked about adding a "license override" field into that metadata file, which could also indicate that a package is commercially licensed.
The downside to a license override is that people can set it to whatever they want, we'll report it like that, and people browsing the package index will trust that the code is licensed correctly, which it may not be. Obviously, this doesn't come into play with a commercial license, but it's worth considering the context in which we are having this discussion. The more I've been thinking about this field, the less I like it.
The alternative here is to make a special case "commercial license" flag in that file, purely for this purpose.
Adjust the display of the "Unknown License" lozenge
The "Unknown License" lozenge is big, red, and angry. I think that's the correct colour for packages where a LICENSE file doesn't exist, but packages with an unknown license might suit an orange and slightly less angry looking lozenge. 😂
The other thing that's not great here is that the lozenge links to the FAQ. Again, I think that's the right place to go for unlicensed packages, but for "Unknown License" packages, we could link to the LICENSE file in the repository instead. I'd want to keep some kind of link to the FAQ next to it, as people might be confused why it says "Unknown License" but that could be linked from a little question mark icon next to the lozenge.
I think orange is probably the correct colour for a commercially licensed package, not green. The orange only indicates that there's something about the license that you'll want to pay attention to.
Opinions welcome! Also /cc @finestructure

Congratulations on the release, Peter! Great to have PSPDFKit in the index :)
I was thinking of the .spi.yml file to steer this as well. End of the day, all the Github API does is wrap around a LICENSE file so I don't think we're in completely uncharted territory here providing our own take on that, linking to a commercial license.
I think the problem is rather than a "bad" license being displayed that what's in .spi.yml might be out of sync with what's in LICENSE (i.e. what is reported by Github).
We could handle that by only considering .spi.yml if Github reports no license.
The other thing to consider is the one signal we do take from the license attribute: App Store compliance.
How do we handle whether a license given via .spi.yml is compliant? Do we even deal with that? Or do we, as you suggest, Dave, do something like this:
# .spi.yml
...
  license:
    type: commercial
    link: http://...
    app_store_compliant: true

with a narrow set of of types we support (just commercial for now I guess?) and trust the package owner?
Could this even be abused without also entering uncertain legal territory? Do we need to worry about it?
I mean, isn’t it a bigger problem for the package author to wrongfully declare App Store compliance than for us?
What we could do is just trial it for now and if we see bad actors, we deny-list their licenses or at least ignore the App Store compliance flag.
Just a few thoughts :)

End of the day, all the Github API does is wrap around a LICENSE file so I don't think we're in completely uncharted territory here providing our own take on that, linking to a commercial license.

Yea, GitHub uses the licensee gem to do it.

I think the problem is rather than a "bad" license being displayed, that what's in .spi.yml might get out of sync with what's in LICENSE (i.e. what is reported by Github).
We could handle that by only considering .spi.yml if Github reports no license.

So if GitHub reports no license, that means the repository does not contain a LICENSE file (or LICENSE.md, or LICENSE.txt). I think we want to be really careful letting people override that as if there is no license file, it is not licensed as open-source, no matter what people intend.

I mean, isn’t it a bigger problem for the package author to wrongfully declare App Store compliance than for us?

I know this is the package author's problem, not ours, but overriding the license where a LICENSE file does not exist is not something we should allow people to do. It's misleading, and in an area that many developers don't understand well (licensing in general) they may believe that setting that flag is licensing their code, which it is not.

What we could do is just trial it for now and if we see bad actors, we deny-list their licenses or at least ignore the App Store compliance flag.

I wrote it as it's definitely worth discussing, but the more I think about it, the more strongly I am in favour of my second suggestion over the first.
Showing that it is an "Unknown license" in orange is correct. There's no such thing as a "Commercial license" as a defined thing, there is only what text is in the LICENSE file. I think that showing "Unknown license" in orange and having that link to the license file itself is the best way to deal with this.

So if GitHub reports no license, that means the repository does not contain a LICENSE file (or LICENSE.md, or LICENSE.txt). I think we want to be really careful letting people override that as if there is no license file, it is not licensed as open-source, no matter what people intend.

Not licensed as open-source - but I was thinking that's the gap we're looking to fill, isn't it? We're looking to add a way to send a license signal in general and Github only really covers the open-source case.
Right now we interpret “no LICENSE file” as “unknown license”. We could allow changing that case to mean “commercial” (or any number of types) via the metadata file, as long as we ensure that open source licenses are always covered by LICENSE via Github.

Showing that it is an "Unknown license" in orange is correct. There's no such thing as a "Commercial license" as a defined thing, there is only what text is in the LICENSE file. I think that showing "Unknown license" in orange and having that link to the license file itself is the best way to deal with this.

I’m certainly happy with just changing the “unknown license” colour to orange and add a link. I’m just not sure that really helps Peter’s use case - but let's see what @steipete thinks! :)

I think there's a bit of a misunderstanding here:

Right now we interpret “no LICENSE file” as “unknown license”.

Not quite, we currently have four states for a package:

No license - For example, Corkboard - The repository does not contain any license information at all, and the code is unlicensed. The license lozenge is shown in red.
Unknown license - For example, App Center - The repository has a license, but it was not able to be matched automatically to any of the defined open-source licenses. In the case of the example, there's additional text at the top of the license that's throwing the detection off, but the PSPDFKit license is also shown with the same state. The license lozenge is shown in red.
App Store incompatible license - For example, Noise - This is all of the GPL licenses, that are not compatible with the App Store so you should be aware of what you're doing before including them. The license lozenge is shown in orange.
Recognised open-source license - For example, Thingy - A license file exists, and has been recognised as an open-source license. The license lozenge is shown in green.

I think we should change the behaviour to be this:

No license - Red lozenge.
Unknown license - Orange lozenge.
App Store incompatible license - Orange lozenge.
Open-source license - Green lozenge.

Also, we should make the license lozenge always link to the text of the license file, and move the link to the FAQ to a (?) icon to the side of the lozenge.
That way:

A red lozenge means "you should not use this package" becase you really shouldn't use unlicensed code. Developers open themselves to serious legal issues in the future with unlicensed code.
An orange lozenge means "investigate this license". Either it's not been recognised, in which case you should click through and see if it's a commercial license, a slightly mis-formatted open-source license, or something completely different.
A green lozenge means "you're legally OK to use this code" as it's open-source.

The only slightly difficult bit of this ia that we don't have the path to the LICENSE file in the repository. We don't know if it's LICENSE, LICENSE.md, LICENSE.txt, etc...

Thanks everyone for chiming in here. This seems to be trickier than I anticipated!
@finestructure You mentioned the issue with App Store compliance. It's probably extremely likely that a commercial license is App Store compliant, but there indeed could be edge cases where such a package is only meant for internal distribution, so yes, can't be inferred automatically.
@daveverwer Changing "unknown license" to Orange would be a great improvement, as would be linking to the LICENSE file when clicking on the license. This would alleviate most of my concerns.
Those two changes would also be quite simple fixes, whereas I'm somewhat skeptical of allowing license overrides in the definition file. I wonder if the discussion of a commercial license detection really is something that makes sense at all, given that this is difficult to detect automatically, and if so, might be something form the licensee gem to add.

Thanks for confirming, @steipete ! We'll make sure to add that very soon then.

Implementation note: the graphql query only gives us access to the following license url, which isn't going to be helpful:
      "licenseInfo": {
        "name": "Other",
        "key": "other",
        "url": "http://choosealicense.com/licenses/other/"
      },

So we'll have to make a query to the license API endpoint instead: https://paw.pt/fuse3gt6
That'll give us a link to the actual license file:
{
  "name": "LICENSE",
  "path": "LICENSE",
  "sha": "e1f2e72bf58ff3cb39e9aa96bf67f92887da6a6b",
  "size": 209,
  "url": "https://api.github.com/repos/PSPDFKit/PSPDFKit-SP/contents/LICENSE?ref=master",
  "html_url": "https://github.com/PSPDFKit/PSPDFKit-SP/blob/master/LICENSE",
  "git_url": "https://api.github.com/repos/PSPDFKit/PSPDFKit-SP/git/blobs/e1f2e72bf58ff3cb39e9aa96bf67f92887da6a6b",
  "download_url": "https://raw.githubusercontent.com/PSPDFKit/PSPDFKit-SP/master/LICENSE",
  "type": "file",
  "content": "QWxsIGl0ZW1zIGFuZCBzb3VyY2UgY29kZSBDb3B5cmlnaHQgwqkgMjAxMC0y\nMDIwIFBTUERGS2l0IEdtYkguCgpQU1BERktpdCBpcyBhIGNvbW1lcmNpYWwg\ncHJvZHVjdCBhbmQgcmVxdWlyZXMgYSBsaWNlbnNlIHRvIGJlIHVzZWQuCgpT\nZWUgaHR0cHM6Ly9wc3BkZmtpdC5jb20vbGVnYWwvTGljZW5zZS5wZGYgaWYg\neW91IGFyZSBldmFsdWF0aW5nIHRoZSBkZW1vLgo=\n",
  "encoding": "base64",
  "_links": {
    "self": "https://api.github.com/repos/PSPDFKit/PSPDFKit-SP/contents/LICENSE?ref=master",
    "git": "https://api.github.com/repos/PSPDFKit/PSPDFKit-SP/git/blobs/e1f2e72bf58ff3cb39e9aa96bf67f92887da6a6b",
    "html": "https://github.com/PSPDFKit/PSPDFKit-SP/blob/master/LICENSE"
  },
  "license": {
    "key": "other",
    "name": "Other",
    "spdx_id": "NOASSERTION",
    "url": null,
    "node_id": "MDc6TGljZW5zZTA="
  }
}

The link through isn't quite working yet: #757

Reopening as we're going to make a few front-end changes to this to make it a little smoother.


Numbers of properties from wikidata don't match with local instance

Hi there,
I'm trying to use the wikidata model for my local instance. After using miniWikibase.py properties in my local instance don't match the ones in wikidata (e.g. property "images" is P10 local, P18 at Wikidata.
Does it work as intended? If yes, is there a way to keep Wikidata's numbering?

Hi Michael,

Yes, it works as intended. Every local property has also a statement with Wikidata ID.
Currently no. There could be an ugly workaround, but I would rather wait for Wikibase 1.36 with fully federated properties (expected at the end of 2021).

Thx a lot!


Renamed Group names with prefix, Issue #65

As discussed in #65  I renamed the groups.
Addittionally I respect the new name of the manager node "clustermanager" and the old one, for migrating
Splunk Indexer Cluster

Thanks @mason-splunk for the clarification, but I guess it's not worth to change the inventory file on this repo.
If you see it differently as it is a good point, I'll create a new PR.


Use golang:1.19 in the build Dockerfile

Now that the project specifies Go 1.19, ensure that the builder provides that.

/approve
/lgtm


az network vnet list --resource-group filter does not seem to be working

Describe the bug
Command Name
az network vnet list --resource-group ...
Errors:
No errors, the command just returns an empty array even though vnets are present in the resource group.
To Reproduce:
(I'm using PowerShell in the steps below)

$ResourceGroup = "my-resource-group"
az network vnet list --resource-group $ResourceGroup

Output:
[]

Note that if I use a JMES query to do the filtering, it works:
az network vnet list --query "[?resourceGroup=='$ResourceGroup'].resourceGroup"
[
  "my-resource-group"
]

Expected Behavior
When using the --resource-group argument I would expect the results returned to include the vnets in the specified resource group.
Environment Summary
Windows-10-10.0.19041-SP0
Python 3.8.9
Installer: MSI

azure-cli 2.26.1

Extensions:
aks-preview 0.5.23
application-insights 0.1.13
databricks 0.7.2

Note that I have also tried this on a machine with no extensions installed and the problem is still present.

Additional Context

network


Upload Files?

Is there a way to upload files to my CircuitPython device using this web editor? If so, it should be easier to find. If not, it should be added.

You mean opening a local file and then sending it over?

Not currently other than creating a new file and pasting in the text.

Yes, I mean sending a local file. I wanted to do this for audio files, but this would also be useful for images and other binary files.

I was thinking some file management utilities would be handy, but uploading would be as well and it could all be part of the same thing.

@tannewt do you think https://github.com/adafruit/ble-file-transfer-js would handle binary files without modification?

@tannewt do you think https://github.com/adafruit/ble-file-transfer-js would handle binary files without modification?

I'd like to think so but haven't tried it.

Fair enough. I haven't tried either.

I added code that works on Web Workflow and it should work on BLE, though I can't seem to find my CLUE to test it. I'll stop by my office later and grab one to test with.
I think it should work with how it is now, though there are extra parameters than JavaScript should just discard. If not, I have a fix for that in branch that also hasn't been tested.

I tested and it caught a bug in the file transfer module. I created an issue here: https://github.com/adafruit/ble-file-transfer-js/issues/6. Also, it worked fine without accounting for the parameters.

Fixed via https://github.com/adafruit/ble-file-transfer-js/commit/f85c66c2dec4b4019885eb3b7e2f4b8e3a4ab6a6 and 04dd38fa20344c85986b887272f8ff8133dff4f3.


Add documentation for oai / improve discoverability

Currently when you search for oai in the docs, nothing comes up: https://docs.rs/poem-openapi/latest/poem_openapi/?search=oai. This is one of the most important macros for the Poem OpenAPI crate but it's hard to figure out how to use it. Could we add documentation for it if it doesn't exist and an alias so you can search for it by oai?

I see we have docs at poem-openapi/src/docs/openapi.md, we could make an alias to go search there at least.

I don't know how to do it. 😂

lol I can take a look at doing it, stay posted


Add boozallen/goobernetes

This PR adds the runner implementation built out by Booz Allen Hamilton on top of actions-runner-controller specifically for GitHub Enterprise Server, although it will work on cloud as well.  It adds runner pools of CentOS 7 and 8, Debian 10 and 11, and Ubuntu 20.04, as well as directions on how to build and customize it further for other administrators and companies.

Thank you, merged


Extend SystemUiOverlayStyle with additonal values for each platform

Requires https://github.com/flutter/engine/pull/5112.  See this PR for more context

To keep this from being a breaking change, I have added checks to prevent the light theme from being set for Android.  These checks can be removed when I add platform themes and clean up the app bars

Actually it looks like some of these properties can be used before O.  I will double check everything and update


Running Deepvariant with WES BED file

Deepvariant mk_examples checks a provided BED file for formatting issues before it runs. It checks the score column to see if it can be type-casted as a double resulting in this error:
2024-11-22 17:16:43.600991: F third_party/nucleus/io/bed_reader.cc:106] Check failed: absl::SimpleAtod(tokens[4].c_str(), &value)
Fatal Python error: Aborted

Our padded BED file contains a . character for score information if it DNE. Let's set it to 0 to avoid this issue.

While resolving this issue, it was also observed that DV/DS strictly enforces the value on the strand (6th) column. Because we are running the following bedtools merge command (see below), there is the potential for entries on overlapping strands to resolve to +,- after the merge. With that being said, retaining strand information within the context is variant calling is not useful. To prevent this issue from occurring, strand information will automatically be set equal to ., regardless of where the original entries strand originated from.
bedtools merge \
        -i - \
        -c 4,5,6 \
        -o distinct,mean,distinct \

Here is a diagram showing how this step will behave, where the purple regions are the padded, merged regions :


Removed reference to dead project.

I killed the nifipie project since it was out of date and incomplete.

@jskora Thanks. Definitely have to go through and clean these up.


Could I create a "log out of all devices" button?

I'm trying implement a "log out of all devices" button.
Apologies for the somewhat verbose explanation that I'm about to give, but it should make the situation more apparent:
With my website, a user has a different session token for each of their devices. These session tokens get authenticated by becoming mapped to a Github username (obtained using this gem's flow).
Say the user logs in on two devices (laptop and phone), and then from the laptop clicks "log out from all devices". It's easy to log out from the device that called the request - I just put logout! in the controller. But how can I have the mobile device also log out? Even if the mobile client forgets about the session token, they might not have to enter their credentials/password on Github again because that's cached in the session.
To summarize, is there any way I can store Github connection objects in a custom object and then close some arbitrary subset of them? The goal would be to force those devices to re-authenticate the username/password in Github.

I implemented this functionality for GitHub and I don't think it's ever going to be available to the public. The short answer is that it requires some work on the OAuth provider side to provide an endpoint where a user can verify that an OAauth client token can be tied back to a valid browser session. "Log out on all devices" would log the user out of github.com and clients would identify that their session on github.com was invalid and invalidate the cookie in the client app.
I think most people looking for this functionality end up using things like okta.
I don't use or actively maintain this project anymore so you're probably better off using something from omniauth.

Appreciate that. FWIW this project still works fine and is quite easy to use
On Tue, Mar 14, 2017 at 12:17 AM, Corey Donohoe<EMAIL_ADDRESS>wrote:

I implemented this functionality for GitHub and I don't think it's ever
going to be available to the public. The short answer is that it requires
some work on the OAuth provider side to provide an endpoint where a user
can verify that an OAauth client token can be tied back to a valid browser
session. "Log out on all devices" would log the user out of github.com
and clients would identify that their session on github.com was invalid
and invalidate the cookie in the client app.
I think most people looking for this functionality end up using things
like okta.
I don't use or actively maintain this project anymore so you're probably
better off using something from omniauth.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/atmos/sinatra_auth_github/issues/35#issuecomment-286340973,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEzWx8QSECUmt5IoNqy-CsP5CIq3vr-Wks5rlj8SgaJpZM4Mapq3
.


Move from mapper to norman

See norman repo for the old mapper codebase.
mapper.NewErrors -> merr   There's a new package merr in norman or wrangler that does this.
If nothing else uses mapper, deprecate it.

already done with: https://github.com/rancher/rio/pull/717


Console error: Warning, Mission translation for key: "..."

Describe the bug
I noticed I am getting console errors with the new update. If I do not use translations on my labels, I will receive this error for every resource that I use.
eg. I am using an User resource like this:
<Resource icon={UsersIcon} options={{ label: 'Users', menuParent: "usersParent" }} name='users' list={UserList} show={UserShow} edit={UserEdit} create={UserCreate} />,
and because I am not using translations since my label is hardcoded I am getting the "Warning, Mission translation for key: "Users" error.
To Reproduce
Use a resource without a translated label.
Screenshots

Environment (please complete the following information):

Windows 10
Chrome
Version 86.0.4240.111

I hope I was as clear as possible. If you have any additional questions, please ask away.

@BlazHrusovar: Can you please mention the ra-treemenu version that you're using in your package.json?

@harshit-budhraja Sure. I am using version 1.0.4.

I have the same problem and use the same version 1.0.4

I silenced the translation warnings by following this documentation

@BlazHrusovar @yusney As aptly pointed out by @erheme318, translations, at this moment,  are supposed to be taken care of from within the i18Provider in the RA app and this is why any externally packaged components don't have access to the i18Provider's objects directly. So, silencing the translation messages for now looks like the best solution. Meanwhile, we can come up with a legitimate fix for these warnings.
https://github.com/marmelab/react-admin/blob/master/docs/Translation.md#silencing-translation-warnings

@harshit-budhraja @erheme318 This worked. Thanks guys.


Get Cross Compilation CI jobs setup with vendored LLVM

We are turning off our ARM cross-compilation jobs that currently run in CircleCI.
We want to:

Move then to running on CirrusCI
Build using the vendored LLVM rather than using a system LLVM

Here are the 3 job definitions from CircleCI:
  cross-llvm-701-arm:
    docker:
      - image: ponylang/ponyc-ci:cross-llvm-7.0.1-arm
        user: pony
    steps:
      - checkout
      # build and test for x86_64 first
      - run: make config=release verbose=1 -j3 all
      - run: make config=release verbose=1 test-ci
      # build libponyrt for target arch
      - run: make config=release verbose=1 CC=arm-linux-gnueabi-gcc CXX=arm-linux-gnueabi-g++ arch=armv7-a tune=cortex-a9 bits=32 CFLAGS="-idirafter /usr/cross/include/" CXXFLAGS="-idirafter /usr/cross/include/" LDFLAGS="-idirafter /usr/cross/include/" PONYPATH=/usr/cross/lib -j3 libponyrt
      # build ponyc for target cross compilation
      - run: make config=release verbose=1 PONYPATH=/usr/cross/lib cross_triple=arm-unknown-linux-gnueabi cross_arch=armv7-a cross_cpu=cortex-a9 cross_linker=arm-linux-gnueabi-gcc QEMU_RUNNER="qemu-arm-static --cpu cortex-a9 -L /usr/local/arm-linux-gnueabi/libc" -j3 all
      # run tests for cross built stdlib using ponyc cross building support
      - run: make config=release verbose=1 PONYPATH=/usr/cross/lib cross_triple=arm-unknown-linux-gnueabi cross_arch=armv7-a cross_cpu=cortex-a9 cross_linker=arm-linux-gnueabi-gcc QEMU_RUNNER="qemu-arm-static --cpu cortex-a9 -L /usr/local/arm-linux-gnueabi/libc" test-cross-ci
      - zulip/status:
          fail_only: true

  cross-llvm-701-armhf:
    docker:
      - image: ponylang/ponyc-ci:cross-llvm-7.0.1-armhf
        user: pony
    steps:
      - checkout
      # build and test for x86_64 first
      - run: make config=release verbose=1 -j3 all
      - run: make config=release verbose=1 test-ci
      # build libponyrt for target arch
      - run: make config=release verbose=1 CC=arm-linux-gnueabihf-gcc CXX=arm-linux-gnueabihf-g++ arch=armv7-a tune=cortex-a9 bits=32 CFLAGS="-idirafter /usr/cross/include/" CXXFLAGS="-idirafter /usr/cross/include/" LDFLAGS="-idirafter /usr/cross/include/" PONYPATH=/usr/cross/lib -j3 libponyrt
      # build ponyc for target cross compilation
      - run: make config=release verbose=1 PONYPATH=/usr/cross/lib cross_triple=arm-unknown-linux-gnueabihf cross_arch=armv7-a cross_cpu=cortex-a9 cross_linker=arm-linux-gnueabihf-gcc QEMU_RUNNER="qemu-arm-static --cpu cortex-a9 -L /usr/local/arm-linux-gnueabihf/libc" -j3 all
      # run tests for cross built stdlib using ponyc cross building support
      - run: make config=release verbose=1 PONYPATH=/usr/cross/lib cross_triple=arm-unknown-linux-gnueabihf cross_arch=armv7-a cross_cpu=cortex-a9 cross_linker=arm-linux-gnueabihf-gcc QEMU_RUNNER="qemu-arm-static --cpu cortex-a9 -L /usr/local/arm-linux-gnueabihf/libc" test-cross-ci
      - zulip/status:
          fail_only: true

  cross-llvm-701-aarch64:
    docker:
      - image: ponylang/ponyc-ci:cross-llvm-7.0.1-aarch64
        user: pony
    steps:
      - checkout
      # build and test for x86_64 first
      - run: make config=release verbose=1 -j3 all
      - run: make config=release verbose=1 test-ci
      # build libponyrt for target arch
      - run: make config=release verbose=1 CC=aarch64-linux-gnu-gcc CXX=aarch64-linux-gnu-g++ arch=armv8-a tune=cortex-a53 bits=64 CFLAGS="-idirafter /usr/cross/include/" CXXFLAGS="-idirafter /usr/cross/include/" LDFLAGS="-idirafter /usr/cross/include/" PONYPATH=/usr/cross/lib -j3 libponyrt
      # build ponyc for target cross compilation
      - run: make config=release verbose=1 PONYPATH=/usr/cross/lib cross_triple=aarch64-unknown-linux-gnu cross_arch=armv8-a cross_cpu=cortex-a53 cross_linker=aarch64-linux-gnu-gcc QEMU_RUNNER="qemu-aarch64-static --cpu cortex-a53 -L /usr/local/aarch64-linux-gnu/libc" -j3 all
      # run tests for cross built stdlib using ponyc cross building support
      - run: make config=release verbose=1 PONYPATH=/usr/cross/lib cross_triple=aarch64-unknown-linux-gnu cross_arch=armv8-a cross_cpu=cortex-a53 cross_linker=aarch64-linux-gnu-gcc QEMU_RUNNER="qemu-aarch64-static --cpu cortex-a53 -L /usr/local/aarch64-linux-gnu/libc" test-cross-ci
      - zulip/status:
          fail_only: true

N.B.: the   cross-llvm-701-aarch64 was failing and was turned off on CircleCI. We expect that the same would happen when switched over.

I've opened a branch to work on this. The branch is 3402. At the moment, it is only trying to test for arm. When building LLVM, we currently error out part of the way through building due to a syntax error. See #3425.

Dipin sent me a patch that should be able to address this. @kulibali is going to integrate into the build ponyc with cmake work.
crossci.diff.txt

@kulibali is working on this as part of the cmake transition.

@kulibali we can close this now, yes?

Yes.


Feature Request: Ability to rotate structure adding model

Just like it is possible to change the offsets while adding a structure to the model list. It would be very helpful to be able to rotate / reorient the structure so that the one can pick the expected/natural orientation when placing the armor stand.
Is there a workaround for this that does not require re-constructing the entire build in the correct orientation?
ps: I know I can rotate / change the placement of the armor stand. I'm just looking for a way to set the correct "natural" position.

There is a rotate feature when loading a structure. So you can load it in
game and rotate it in 90 degree increments.
On Fri, May 14, 2021, 11:42 PM Henrique Abreu @.***>
wrote:

Just like it is possible to change the offsets while adding a structure to
the model list. It would be very helpful to be able to rotate / reorient
the structure so that the one can pick the expected/natural orientation
when placing the armor stand.
Is there a workaround for this that does not require re-constructing the
entire build in the correct orientation?
ps: I know I can rotate / change the placement of the armor stand. I'm
just looking for a way to set the correct "natural" position.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/RavinMaddHatter/Structura/issues/98, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABBUC5AGJ46WZOWCHV66NS3TNYJVFANCNFSM445STJQQ
.

I do not know how I missed that. Thanks
For the "blind" like me: When loading a structure in with the structure_block in load mode, there's a rotate option.


feat: ability to detect service context

Ref: https://github.com/googleapis/nodejs-logging-bunyan/issues/135

@JustinBeckwith can we get this released?


Ensure the resource parameter in code is mandatory

Currently we have an argument named resource used in methods across the extension.
This is used to ensure we access the workspace related to the resource passed in.
Currently this is an optional parameter resource?: Uri, hence sometimes values are not being passed into these methods when they should.
We need to ensure:

The argument is mandatory
Authors of the code know that a value needs to be passed in

Easy Solution: Change the signature of the argument to resource: Uri | undefined
We need to either:

Make this change across the board
In high level API (classes).
Across the board, but apply this incrementally (as and when files are modified)

@Microsoft/pvsc-team

@DonJayamanne do you think this is still necessary?

Lets close this