[FR] Ability to keep lock state after reboot

I’ve noticed that after power outages or randomly (probably due to ESP32 reboots) the state of the lock changes to unlocked in Home app, although that does not trigger any mqtt message to be sent in order to mess with automations. However, while in that state (HomeKey lock entity state unlocked but actual lock locked) it messes with the HomeKey authentication and automation logic in the sense that when scanning the HomeKey it will broadcast a lock mqtt message when I actually want to unlock the door, so nothing happens. If a second HomeKey authentication is attempted, the actual lock will unlock since they are now both (HomeKey entity and actual lock) in sync.
Anyone else has this issue?
Am I doing something wrong?
Is there any way we can keep the lock state after reboots/power outages?
Thanks!

strange, that is not what i'm seeing on my side, HomeSpan has the option to save a characteristic's value to nvs which i set to true for the lock's target and current state here.
Could you try to change the lock's state (HAP) manually and then reboot the esp and see if the state persist? This is to understand if somehow there is something else going on.
On a side note, also as a workaround, if you don't have the esp hardwired to a lock, don't rely on the HAP state for your automations since it is kinda arbitrary, instead make the decision to unlock or lock around your real lock's state based on the MQTT_AUTH_TOPIC topic or even better if you use HASS you can use Tags since the project has integration through mqtt discovery for that and you can very easily make automations at a click of a button per person/device.

Thanks, will do a test to check if HomeSpan saves the state.
Regarding the MQTT_AUTH_TOPIC what happens if a HomeKey from a different home is presented?  Or in other words is the HomeKey mqtt message only transmitted if HomeKey authentication succeds? I’m wondering if I could just use the homekey=true as a trigger for unlock.
Jonah

Well…yeah, I don’t think it would make sense to publish on all attempts, just for the successful ones. If you have multiple deployed and are all using the same topics, it should be fine, the endpointId is unique per setup as each device generates a key for each lock, only the issuerId stays the same.
The homekey field exists only to indicate the message contains homekey data since regular rfid cards would publish different data, although you could use it as a basic validation that the data is homekey related kinda like this:
## Basic HASS Automation
alias: Homekey
description: ""
trigger:
  - platform: mqtt
    topic: topic/auth
condition:
  - condition: template
    value_template: "{{trigger.payload_json[\"homekey\"] == true}}"
action:
  - if:
      - condition: device
        device_id:
        domain: lock
        entity_id:
        type: is_locked
    then:
      - device_id:
        domain: lock
        entity_id:
        type: unlock
    else:
      - device_id:
        domain: lock
        entity_id:
        type: lock
mode: restart

I did some further tests and the Home entity keeps state on reboot when manually triggering it over mqtt. I'm still not sure what causes that, but I've decided to avoid using the dummy entity in any automations for the time being. It's just annoying to see it in the Control Center as Unlocked in the middle of the night when it actually is Locked 😳.
We can close this one.
Thanks a lot!

The functionality was added about 2 months ago so i doubt that you have an old version. I did multiple tests uncoupled from any automations and it is able to persist state on power outages(a.k.a unplugging it) and also checked the activity history up to one month and there's no random state change.  so really not sure why's that way for you, sorry about that. I'll close the issue for now since i'm not able to reproduce it but if something comes up i'll reopen it.


Added refactoring for move class to file in VB

Added refactoring for Moving Class Into File.
No UnitTests is made( do not know howto make them) :)

I think it would make sense to suppress this refactoring if the class is the only one in current file already.

I will have a look at that. Thank you for your comments

Any idea how to see if the file already exists, I need this code elsewhere also.

No not really. I googled this issue a bit and it seems accessing files in a PCL(portable class library) is not that easy.. I do not know why the refactoring is made as a PCL but there is probably a thought behind it. Reusing it on mono probably one of them.

I am closing this, because 1) VS 2017 already provides this refactoring out-of-the-box and 2) it would require some more fixes, and is copied from another project (except for the conversion).


Function annotations

Would be good to experiment with adding PEP-484 function annotations to assist PyCharm with consuming our API. Might also be able to find some static-checking tools that can use them.
While we remain Python 2.x compatible, we can put these in .pyi files

MyPy would be the go-to these days for the annotations. So basic plan would be:

[ ] Add mypy as another build job on Travis (or maybe part of linting)
[ ] Add config file (mypy.ini)
[ ] Enable checking of untyped stuff (--check-untyped-defs)
[ ] Start picking off modules

For more info, see zulip's blog post


Update reactivemongo-bson to 0.18.4

Updates org.reactivemongo:reactivemongo-bson from 0.18.3 to 0.18.4.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention @scala-steward in the comments below.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [{ groupId = "org.reactivemongo", artifactId = "reactivemongo-bson" }]

labels: semver-patch

Codecov Report

Merging #41 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #41   +/-   ##
=======================================
  Coverage   45.45%   45.45%
=======================================
  Files           1        1
  Lines          44       44
  Branches        1        1
=======================================
  Hits           20       20
  Misses         24       24

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 5bc0278...cb07c2e. Read the comment docs.


Turn off suggestions box

Is it possible to turn off the suggestions box? I couldn't get my data working with it correctly and made my own outside of the autosuggest component. I still get parent - child node warnings from Dom elements in the suggestions slot though. Great work overall, I probably just need to work with it more but a quick solution for now would be appreciated.

Not sure I understand your question. The suggestion box is the whole point of the component so turning it off and rolling your own would just make sense to not use this component? Check out the code sandbox demos to see how it's used with different data inputs

Thanks I probably just need to spend more time with it and look at the actual component.


Updates docs dependencies

Hi @Jwaegebaert could you include the newly published release in this PR? https://docusaurus.io/blog/releases/3.5

I suggest we update docusaurus separately. Let's definitely keep it in mind.

Hi @Jwaegebaert could you include the newly published release in this PR? https://docusaurus.io/blog/releases/3.5

I suggest we update docusaurus separately. Let's definitely keep it in mind.

What do you mean exactly?

This PR updates docs' dependencies but not the docs engine itself. I suggest that we update docs engine (docusaurus packages) separately, because it might require additional review to ensure that docs work as intended.

Merged manually. Thanks @Jwaegebaert!


Notification for not implemented drivers

Thanks @ahtinurme for the valuable contribution.
We will soon implement Redis driver feature.


[Quest/Malawi Core] Date entry format extension not working

Describe the bug
When using the entryFormat extension on a date field, the date does not get formatted accordingly. A sample of this extension is used in the android FHIR catalog app here.
To Reproduce
Steps to reproduce the behavior:

Go on the patients register
Select an ART client
On their profile, select Viral Load Results
Check the date of the viral load sample collection

Expected behavior
The date should be formatted accordingly i.e. for today's date, it should show 12/09/2022 (day/month/year) but even with the extension added, the date always defaults to 9/12/22 (month/day/year).
Screenshots

Smartphone (please complete the following information):

Device: Samsung A12
Android version: 11
Build number: RP1A.200720.012.A127FXXU4AUK1
Version: 30

Additional context
This works in the android FHIR catalog app as shown below. I also tried it on the ecbis app and it doesn't seem to be working there either.

We may need to check on SDC on the support for this extension. I will review based on the shared details. Adding Triage label for this

This was implemented in this PR but it doesn't seem to be working correctly

@f-odhiambo @Gental-Giant We might want to create an SDK ticket for this (if we don't have one)

@f-odhiambo @dubdabasoduba Here is the sdk ticket for this

To be closed with https://github.com/opensrp/fhircore/issues/1764


[ConstraintSystem] Match trailing closures to implicit .callAsFunction when necessary

In situations like T(...) { ... } where T is a callable type,
it's not immediately clear whether trailing closures are associated
with .init of T or implicit .callAsFunction. So if constructor
didn't match trailing closures, let's attempt to split them off and
form an implicit .callAsFunction call with them as a fallback.
Resolves: rdar://83717297

@swift-ci please test

Build failed
Swift Test OS X Platform
Git Sha - 49b181136bf03506395b781748ecf1efa6d5a333

@swift-ci please smoke test

@swift-ci please build toolchain macOS

@swift-ci please build toolchain macOS

@swift-ci please build toolchain macOS

@swift-ci please build toolchain macOS

@swift-ci please build toolchain macOS

@swift-ci please build toolchain macOS

Ok, I give up, going to merge and wait for the main branch toolchain to be built.


DataTables

the tables can be used with jquery datatable ? i want to use some like this

The data table can work with 3rd party JavaScript, but a bit customisation may be needed to apply the right styles.


Get System.NullReferenceException: 'Object reference not set to an instance of an object.' while creating new account

var context = new AcmeContext(Certes.Acme.WellKnownServers.LetsEncryptStagingV2);
var account = await context.NewAccount(email, true);
// Save the account key for later use
var pemKey = context.AccountKey.ToPem();

@nafaji Did you ever figure out what was causing this?


ci: move npm publish job to circle ci

Move npm publish job from github actions to circleci
publish job only runs on main branch
If commit matches release commit style create a full release, else no-op

circle ci cannot conditional run jobs based on the commit message unfortunately
In the future we will add a pre-release to the else condition

Codecov Report

Merging #291 (f267ebb) into main (522d707) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #291   +/-   ##
=======================================
  Coverage   89.48%   89.48%
=======================================
  Files          34       34
  Lines        1217     1217
  Branches      273      273
=======================================
  Hits         1089     1089
  Misses        125      125
  Partials        3        3

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 522d707...f267ebb. Read the comment docs.


Readline crashes when accessing lines of history longer than three lines

Tested on Ruby 1.9, 2.2.1
Steps to reproduce

Start a simple readline loop, load history into it containing an entry over three lines long (relative to terminal columns)
hit the up arrow key until you arrive at the entry

Result
#<TypeError: no implicit conversion from nil to integer>
no implicit conversion from nil to integer /Users/wwboynton/.rvm/gems/ruby-1.9.3-p392/gems/rb-readline-0.5.2/lib/rbreadline.rb:2735:in `[]'
/Users/wwboynton/.rvm/gems/ruby-1.9.3-p392/gems/rb-readline-0.5.2/lib/rbreadline.rb:2735:in `update_line'
/Users/wwboynton/.rvm/gems/ruby-1.9.3-p392/gems/rb-readline-0.5.2/lib/rbreadline.rb:3535:in `rl_redisplay'
/Users/wwboynton/.rvm/gems/ruby-1.9.3-p392/gems/rb-readline-0.5.2/lib/rbreadline.rb:4676:in `_rl_internal_char_cleanup'
/Users/wwboynton/.rvm/gems/ruby-1.9.3-p392/gems/rb-readline-0.5.2/lib/rbreadline.rb:4737:in `readline_internal_charloop'
/Users/wwboynton/.rvm/gems/ruby-1.9.3-p392/gems/rb-readline-0.5.2/lib/rbreadline.rb:4801:in `readline_internal'
/Users/wwboynton/.rvm/gems/ruby-1.9.3-p392/gems/rb-readline-0.5.2/lib/rbreadline.rb:4823:in `readline'
/Users/wwboynton/.rvm/gems/ruby-1.9.3-p392/gems/rb-readline-0.5.2/lib/readline.rb:45:in `readline'

Potential Solutions
For what it's worth, I've managed to fix this by replacing the vis_pos method in the RbReadline module like so:
def vis_pos(line)
  @vis_lbreaks[line] || 0
end

I'll be darned if I finished diving through all the lines of code required to determine the implications of this are sound. All I know is, where once was nil, now is a value. And my program now appropriately handles history lines of any length.
If this sounds okay, I'll fork and submit a pull request with the change. If this is an awful idea, feel free to shed some light on the 8000+ lines I'm looking at and I'd be happy to try to write a more appropriate solution.

So thanks to @ConnorAtherton, my changes got pulled, which is super spiffy, but if we could push this to rubygems that would fix the issue in the release version.
Thanks! :)

Thanks @wwboynton, 0.5.3 has been released :) :+1:

I'm still seeing this on 0.5.3, ruby 2.2.4p230, on Fedora 23.
With the long line being the last line of my .irb_history, it crashes as soon as open irb and press Up arrow:
bpaskinc@reserved-2-248 ~> resize
COLUMNS=70;
LINES=32;
export COLUMNS LINES;
bpaskinc@reserved-2-248 ~> nano ~/.irb_history
bpaskinc@reserved-2-248 ~> cat ~/.irb_history
puts 'hello'
QQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQ
QQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQ
QQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQ
QQQQQQQQQQQQQQQQQQQ
bpaskinc@reserved-2-248 ~> irb
i/usr/local/lib/ruby/gems/2.2.0/gems/rb-readline-0.5.3/lib/rbreadline.
rb:2742:in `[]': no implicit conversion from nil to integer (TypeError
)
	from /usr/local/lib/ruby/gems/2.2.0/gems/rb-readline-0.5.3/lib
/rbreadline.rb:2742:in `update_line'
	from /usr/local/lib/ruby/gems/2.2.0/gems/rb-readline-0.5.3/lib
/rbreadline.rb:3542:in `rl_redisplay'
	from /usr/local/lib/ruby/gems/2.2.0/gems/rb-readline-0.5.3/lib
/rbreadline.rb:4683:in `_rl_internal_char_cleanup'
	from /usr/local/lib/ruby/gems/2.2.0/gems/rb-readline-0.5.3/lib
/rbreadline.rb:4744:in `readline_internal_charloop'
	from /usr/local/lib/ruby/gems/2.2.0/gems/rb-readline-0.5.3/lib
/rbreadline.rb:4808:in `readline_internal'
	from /usr/local/lib/ruby/gems/2.2.0/gems/rb-readline-0.5.3/lib
/rbreadline.rb:4830:in `readline'
	from /usr/local/lib/ruby/gems/2.2.0/gems/rb-readline-0.5.3/lib
/readline.rb:45:in `readline'
	from /usr/local/lib/ruby/2.2.0/irb/input-method.rb:150:in `get
s'
	from /usr/local/lib/ruby/2.2.0/irb.rb:469:in `block (2 levels)
 in eval_input'
	from /usr/local/lib/ruby/2.2.0/irb.rb:623:in `signal_status'
	from /usr/local/lib/ruby/2.2.0/irb.rb:468:in `block in eval_in
put'
	from /usr/local/lib/ruby/2.2.0/irb/ruby-lex.rb:188:in `call'
	from /usr/local/lib/ruby/2.2.0/irb/ruby-lex.rb:188:in `buf_inp
ut'
	from /usr/local/lib/ruby/2.2.0/irb/ruby-lex.rb:103:in `getc'
	from /usr/local/lib/ruby/2.2.0/irb/slex.rb:205:in `match_io'
	from /usr/local/lib/ruby/2.2.0/irb/slex.rb:75:in `match'
	from /usr/local/lib/ruby/2.2.0/irb/ruby-lex.rb:282:in `token'
	from /usr/local/lib/ruby/2.2.0/irb/ruby-lex.rb:264:in `lex'
	from /usr/local/lib/ruby/2.2.0/irb/ruby-lex.rb:235:in `block (
2 levels) in each_top_level_statement'
	from /usr/local/lib/ruby/2.2.0/irb/ruby-lex.rb:231:in `loop'
	from /usr/local/lib/ruby/2.2.0/irb/ruby-lex.rb:231:in `block i
n each_top_level_statement'
	from /usr/local/lib/ruby/2.2.0/irb/ruby-lex.rb:230:in `catch'
	from /usr/local/lib/ruby/2.2.0/irb/ruby-lex.rb:230:in `each_to
p_level_statement'
	from /usr/local/lib/ruby/2.2.0/irb.rb:485:in `eval_input'
	from /usr/local/lib/ruby/2.2.0/irb.rb:395:in `block in start'
	from /usr/local/lib/ruby/2.2.0/irb.rb:394:in `catch'
	from /usr/local/lib/ruby/2.2.0/irb.rb:394:in `start'
	from /usr/local/bin/irb:11:in `<main>'

Now if I reduce the font sufficiently, it won't crash:
bpaskinc@reserved-2-248 ~ [1]> resize
COLUMNS=146;
LINES=69;
export COLUMNS LINES;
bpaskinc@reserved-2-248 ~> irb
irb(main):001:0> QQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQQ

Same problem here on rb-readline 0.5.3 and Ruby 2.3.0. In my case, it gets triggered when passing a long string to Thor's ask? method. I can work around it by resizing my terminal window.
D/Users/Mike/.chefdk/gem/ruby/2.3.0/gems/rb-readline-0.5.3/lib/rbreadline.rb:2742:in `[]': no implicit conversion from nil to integer (TypeError)'t parse safe navigation operator, remove cancel campaign link, Fix a bug where waving/cloning offers with a rule that alrea       from /Users/Mike/.chefdk/gem/ruby/2.3.0/gems/rb-readline-0.5.3/lib/rbreadline.rb:2742:in `update_line'
	from /Users/Mike/.chefdk/gem/ruby/2.3.0/gems/rb-readline-0.5.3/lib/rbreadline.rb:3542:in `rl_redisplay'
	from /Users/Mike/.chefdk/gem/ruby/2.3.0/gems/rb-readline-0.5.3/lib/rbreadline.rb:3960:in `readline_internal_setup'
	from /Users/Mike/.chefdk/gem/ruby/2.3.0/gems/rb-readline-0.5.3/lib/rbreadline.rb:4807:in `readline_internal'
	from /Users/Mike/.chefdk/gem/ruby/2.3.0/gems/rb-readline-0.5.3/lib/rbreadline.rb:4830:in `readline'
	from /Users/Mike/.chefdk/gem/ruby/2.3.0/gems/rb-readline-0.5.3/lib/readline.rb:45:in `readline'
	from /Users/Mike/.chefdk/gem/ruby/2.3.0/gems/thor-0.19.4/lib/thor/line_editor/readline.rb:20:in `readline'
	from /Users/Mike/.chefdk/gem/ruby/2.3.0/gems/thor-0.19.4/lib/thor/line_editor.rb:7:in `readline'
	from /Users/Mike/.chefdk/gem/ruby/2.3.0/gems/thor-0.19.4/lib/thor/shell/basic.rb:410:in `ask_simply'
	from /Users/Mike/.chefdk/gem/ruby/2.3.0/gems/thor-0.19.4/lib/thor/shell/basic.rb:80:in `ask'
	from /Users/Mike/.chefdk/gem/ruby/2.3.0/gems/thor-0.19.4/lib/thor/shell.rb:59:in `ask'

this fixed it
MacBook-Pro-2:knife-derp jplimack$ gem update thor rb-readline
Updating installed gems
Updating rb-readline
Fetching: rb-readline-0.5.4.gem (100%)
Successfully installed rb-readline-0.5.4
Gems updated: rb-readline
Gems already up-to-date: thor

Works for me as well now on Linux (fedora 24) with rb-readline 0.5.4, but doesn't look like it was the 0.5.3 -> 0.5.4:

commit log doesn't look like anything related was fixed recently.
gem uninstall rb-readline -v 0.5.4; gem install rb-readline -v 0.5.3 and it still works! (and confirmed with strace 0.5.3 is what's loaded).
under a big bundler-using project, forcing 0.5.3 it does crash, but updating to 0.5.4, up arrow doesn't work at all, line remains empty!? (but commands are appended to .irb_history)
cleaner experiment: inside rb-readline/ git checkout, bundle install; bundle exec irb => v0.5.3 works (no crash!), master up arrow doesn't show anything.

Anyway sounds like it wasn't a strictly rb-readline bug, but some interaction with other gems...
I'll report the master problem separately.

OK, sorry for noise, here's a clearer picture.
Works for me on ruby 2.3.1 (via rbenv, on fedora 24) with both rb-readline 0.5.4 and 0.5.3.  Bisecting confirms it was indeed fixed by @wwboynton in #121, should be OK since 0.5.3 — as expected.
However with ruby 2.2.5, it still fails on 0.5.3, 0.5.4, and master.
Minimal repro in rb-readline/ directory:
bundle install; bundle exec ruby -e 'require "readline"; Readline::HISTORY.push("X"*400); puts Readline.readline("> ")'
# now press up arrow and see if it crashes

on ruby 2.2.5:
>/home/bpaskinc/rb-readline/lib/rbreadline.rb:2742:in `[]': no implicit conversion from nil to integer (TypeError)XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX       from /home/bpaskinc/rb-readline/lib/rbreadline.rb:2742:in `update_line'
	from /home/bpaskinc/rb-readline/lib/rbreadline.rb:3542:in `rl_redisplay'
	from /home/bpaskinc/rb-readline/lib/rbreadline.rb:4683:in `_rl_internal_char_cleanup'
	from /home/bpaskinc/rb-readline/lib/rbreadline.rb:4744:in `readline_internal_charloop'
	from /home/bpaskinc/rb-readline/lib/rbreadline.rb:4808:in `readline_internal'
	from /home/bpaskinc/rb-readline/lib/rbreadline.rb:4830:in `readline'
	from /home/bpaskinc/rb-readline/lib/readline.rb:45:in `readline'
	from -e:1:in `<main>'


The dataset uploaded in huggingface seems not to have segmentation labels.

As title, can you tell me where the segmentation labels are? Or am I finding the wrong direction or usage?

well, I see the segmentation is mixed with original images in the dataset, but the problem is where is your referring text? Could please provide it?

Please kindly download the RefSegRS.zip.
Please check carefully; this zip file definitely includes the segmentation labels and text.


verbose options passed as tuple rather than str

https://github.com/python-poetry/poetry-plugin-export/blob/f8cbb9c7ae0586759f8bfe95f32268812a4f6eec/src/poetry_plugin_export/command.py#L67
Errors out with
TypeError
sequence item 0: expected str instance, tuple found
at /usr/local/lib/python3.9/site-packages/poetry_plugin_export/command.py:73 in handle
The setting of verbose option arguments in options array joined to add to the lock file has an extra set of parentheses that causes the intended default null value to be combined with the argument into a tuple.

Ha, there's even a warning from mypy about this that someone has explicitly ignored...
I expect an MR would be welcome.

Same here: (poetry 1.7.1)
$ poetry export --verbose
Warning: In a future version of Poetry, PyPI will be disabled automatically if at least one custom primary source is configured. In order to avoid a breaking change and make your pyproject.toml forward compatible, add PyPI explicitly via 'poetry source add pypi'. By the way, this has the advantage that you can set the priority of PyPI as with any other source.
Warning: poetry-plugin-export will not be installed by default in a future version of Poetry.
In order to avoid a breaking change and make your automation forward-compatible, please install poetry-plugin-export explicitly. See https://python-poetry.org/docs/plugins/#using-plugins for details on how to install a plugin.
To disable this warning run 'poetry config warnings.export false'.
The lock file does not exist. Locking.

  TypeError

  sequence item 0: expected str instance, tuple found

  at /usr/local/lib/python3.8/site-packages/poetry_plugin_export/command.py:84 in handle
       80│                 options.append(("-vv", None))
       81│             elif self.io.is_verbose():
       82│                 options.append(("-v", None))
       83│
    →  84│             self.call("lock", " ".join(options))  # type: ignore[arg-type]
       85│
       86│         if not locker.is_fresh():
       87│             self.line_error(
       88│                 ""


Client Allocation Request for: Textile

Client Allocation Questions
Core Information

Name: Textile
Website / Social Media: https://textile.io https://twitter.com/textileio
Region: North America
DataCap Requested: 100TiB
Addresses to be Notarized: f144zep4gitj73rrujd3jw6iprljicx6vl4wbeavi
Notary Requested: neogeweb3

@andrewxhill Please subscribe to notifications for this Issue to be aware of updates. Notaries may request additional information on the Issue.

Hey @andrewxhill, wow, that's quite a bit of DataCaps to a single Notary. Are you preparing some gaps for the bridge to the NEAR, ETH and Polygon? I heard you are onboarding non-blockchain users as well, but you should apply via LDN, I will definitely support that.
For now, I could allocate part of your request amount to fill some sudden gaps if it's okay with you.

Yeah, that's just it. I'm preparing the LDN application but want to be sure we don't get caught with downtime right as we are ramping up, and LDN apps seem to be taking a good amount of time. So yep, partials would be amazing.
Exactly regarding the use-cases. There are two.

Blockchains. We've got NEAR Testnet, but are aiming to release NEAR Mainnet, Eth, and Polygon in the near future.
Non-blockchains. These are users preparing their own CAR files and submitting auctions to the miner pool.

We've got 4-6 miners currently competing for access to every deal, selected purely on bid price right now. Side-note, we'll be having our first auction governance call soon where we'll start inviting miners and others to help us keep making this UI better. That said, we are going to start sharing the bidding tool widely for miners to onboard themselves. So as far as decentralization, this should be pretty unmatched (it will be the miners that compete to win instead of us directing where deals go).
We are shooting for all data in these early use-cases to be open datasets, owned by their publisher. We are also working on some public APIs and reporting tools so that everyone will be able to see the full history of deals made, data contained, and auction results (bids and winning selections). That should be available in early August.

@andrewxhill That sounds awesome, thanks for the update and clarification!

Request Approved
Your Datacap Allocation Request has been approved by the Notary
Message sent to Filecoin Network

bafy2bzacebmpfze5yzcainw732knbrr3fbwiutid4v2bfy45amfk6kjxlcd7y

Address

f144zep4gitj73rrujd3jw6iprljicx6vl4wbeavi

Datacap Allocated

50TiB

You can check the status of the message here: https://filfox.info/en/message/bafy2bzacebmpfze5yzcainw732knbrr3fbwiutid4v2bfy45amfk6kjxlcd7y


speakers in use

currently my powershell hacks determine if I'm using the computer based on the speakers being used and the idle time away from the keyboard/mouse.
Any chance you can add speaker support?

I'll see what I can do! What does your hack look like?

I'll see what I can do! What does your hack look like?

I thought it was pure powershell but it calls this exe. https://github.com/smourier/IsWindowsPlayingSound
The keyboard timeout is powershell stolen from somewhere... below.
I just run the powershell script in a loop forever upon user login and spit the data into hass. Works way better than IOTLINK. Hopefully you can succeed where they failed.
Add-Type @'
using System;
using System.Diagnostics;
using System.Runtime.InteropServices;

namespace PInvoke.Win32 {

    public static class UserInput {

        [DllImport("user32.dll", SetLastError=false)]
        private static extern bool GetLastInputInfo(ref LASTINPUTINFO plii);

        [StructLayout(LayoutKind.Sequential)]
        private struct LASTINPUTINFO {
            public uint cbSize;
            public int dwTime;
        }

        public static DateTime LastInput {
            get {
                DateTime bootTime = DateTime.UtcNow.AddMilliseconds(-Environment.TickCount);
                DateTime lastInput = bootTime.AddMilliseconds(LastInputTicks);
                return lastInput;
            }
        }

        public static TimeSpan IdleTime {
            get {
                return DateTime.UtcNow.Subtract(LastInput);
            }
        }

        public static int LastInputTicks {
            get {
                LASTINPUTINFO lii = new LASTINPUTINFO();
                lii.cbSize = (uint)Marshal.SizeOf(typeof(LASTINPUTINFO));
                GetLastInputInfo(ref lii);
                return lii.dwTime;
            }
        }
    }
}
'@

I thought it was pure powershell but it calls this exe. https://github.com/smourier/IsWindowsPlayingSound
The keyboard timeout is powershell stolen from somewhere... below.
I just run the powershell script in a loop forever upon user login and spit the data into hass. Works way better than IOTLINK. Hopefully you can succeed where they failed.
Add-Type @'
using System;
using System.Diagnostics;
using System.Runtime.InteropServices;

namespace PInvoke.Win32 {

    public static class UserInput {

        [DllImport("user32.dll", SetLastError=false)]
        private static extern bool GetLastInputInfo(ref LASTINPUTINFO plii);

        [StructLayout(LayoutKind.Sequential)]
        private struct LASTINPUTINFO {
            public uint cbSize;
            public int dwTime;
        }

        public static DateTime LastInput {
            get {
                DateTime bootTime = DateTime.UtcNow.AddMilliseconds(-Environment.TickCount);
                DateTime lastInput = bootTime.AddMilliseconds(LastInputTicks);
                return lastInput;
            }
        }

        public static TimeSpan IdleTime {
            get {
                return DateTime.UtcNow.Subtract(LastInput);
            }
        }

        public static int LastInputTicks {
            get {
                LASTINPUTINFO lii = new LASTINPUTINFO();
                lii.cbSize = (uint)Marshal.SizeOf(typeof(LASTINPUTINFO));
                GetLastInputInfo(ref lii);
                return lii.dwTime;
            }
        }
    }
}
'@

Oh. I just remembered one maybe important bit...
I seem to recall I had to add the IsWindowsPlayingSound bit because  UserInput  didn't work for when I was playing video games. Or maybe it was because I watched movies on the computer sometimes? Hmm. Don't remember exactly but I think the prior.

Oh. I just remembered one maybe important bit...
I seem to recall I had to add the IsWindowsPlayingSound bit because  UserInput  didn't work for when I was playing video games. Or maybe it was because I watched movies on the computer sometimes? Hmm. Don't remember exactly but I think the prior.

Hey @bbrendon,
I missed your comments until now. Sorry for that!
That .exe file you linked seems pretty cool! I might build a custom sensor where you can call a script to get a value so that it can be exposed to Home Assistant.
An IdleTime sensor has been implemented (quite similarly to your Powershell script). Please note that this sensor will be replaced by the LastActive sensor in the next release.

Hey @bbrendon,
I missed your comments until now. Sorry for that!
That .exe file you linked seems pretty cool! I might build a custom sensor where you can call a script to get a value so that it can be exposed to Home Assistant.
An IdleTime sensor has been implemented (quite similarly to your Powershell script). Please note that this sensor will be replaced by the LastActive sensor in the next release.

A CurrentVolume sensor has been implemented and will be in the next release.


how to set target path  of other scene for interpolatedcamera?

Godot version:3.11

OS/device including version:windows10

Other scenes have no choice as targets.
How can you do this?
I attached a script to interpolatedcamera
extends InterpolatedCamera

var scene = load("res://0_practice_folder/singleton/ins.tscn")
var ins
var path
# Called when the node enters the scene tree for the first time.
func _ready():
	ins = scene.instance()
	path  = ins.get_node("p3d")
	self.set_target_path(path)

But is this not correct?
How do you target other scenes?

Sorry but this is not the place to ask about how to use the engine, but to report bugs and ask for new engine features. Please use the other community channels for this purpose .

I was misunderstood. I'm sorry.


Can't access referenced app bar cmdlets

Hello, I've updated to the latest version of the Microsoft.Online.SharePoint.PowerShell module I can (16.0.20717.12000) but when I run the command Get-SPOTemporarilyDisableAppBar, I get "The term 'Get-SPOTemporarilyDisableAppBar' is not recognized as the name of a cmdlet, function, script file, or operable program."  Are the referenced cmdlets maybe not released yet or am I doing something wrong?

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

ID: b2cb90e0-b8bb-32b2-d8b1-1710af6312f2
Version Independent ID: 9d7a64c4-a3b0-47b8-b209-16e257a9a23a
Content: Use the new SharePoint app bar and set up global navigation - SharePoint in Microsoft 365
Content Source: SharePoint/SharePointOnline/sharepoint-app-bar.md
Service: sharepoint-online
GitHub Login: @Holland-ODSP
Microsoft Alias: hokavian

Hey @jmuehlenbein yes the script is available.
Did you uninstall your older version of PowerrShell too?

No, but the cmdlets should be cumulative if more than one version installed. I have the latest (as far as I can tell). Are you suggesting that removing an older version will somehow allow the new cmdlet to be surfaced?

I agree its not an intuitive process to uninstall the older version. I do have a note in the notes section under "Temporarily disable the app bar" - but maybe they were too easy to miss?
Anyway, yes please uninstall the outdated PowerShell and then try again.

I have to admit I doubted you but yeah...I can now access the cmdlets. Thanks so much! Guess I missed the part you're referring to above.

I should make that note more obvious - glad we got this sorted out!

Actually, I just re-read and it says to use the latest version of PowerShell but actually it seems we need the latest version of the Microsoft.Online.SharePoint.PowerShell module. It would be great if you could clarify for my fellow admins.


Is it possible to build dissectors on TCP streams?

Hello!
Thank you for making this library. It looks awesome!
I'm trying to build a packet dissector for a protocol that will for the most part be transmitted using TCP streams. Looking through the documentation and the examples, there's no mention of this kind of functionality. According to the wireshark docs, this seems to impose some constraints: https://wiki.wireshark.org/Lua/Dissectors#tcp-reassembly
If this is already possible with the current state of the library, maybe a simple example could be added?

Hi @h7x4, thanks for your interest in this library :) Dissecting data from TCP streams is not possible for now as the there is no way to specify message boundaries across/within segments. Unfortunately, I don't know if it will be supported in the near future, though it wouldn't seem too difficult.

For future reference, the post linked below might be useful if/when adding this feature to wsdf.
https://osqa-ask.wireshark.org/questions/53802/how-dissect-two-segments-of-one-protocol-in-the-same-packet-in-the-same-tcp-segment-lua/


OCPBUGS-3064: normalize locations in pull/push secrets which might look like URLs

Normalize locations in secrets that look like URLs, since we used to accept (or "accept") those.
Updates mergeNodeCredentials(), used by the image pulling logic. Updates mergeNodeCredentialsDockerAuth(), used by the docker and sti strategies. Updates extractSourceFromImage() directly, since it doesn't use either of the other functions.

Error creating: pods "aws-ebs-csi-driver-node-" is forbidden: unable to validate against any security context constraint: [provider "anyuid": Forbidden: not usable by user or serviceaccount, provider restricted-v2: .spec.securityContext.hostNetwork: Invalid value: true: Host network is not allowed to be used, spec.volumes[0]: Invalid value: "hostPath": hostPath volumes are not allowed to be used, spec.volumes[1]: Invalid value: "hostPath": hostPath volumes are not allowed to be used, spec.volumes[2]: Invalid value: "hostPath": hostPath volumes are not allowed to be used, spec.volumes[3]: Invalid value: "hostPath": hostPath volumes are not allowed to be used, spec.volumes[4]: Invalid value: "hostPath": hostPath volumes are not allowed to be used, spec.volumes[5]: Invalid value: "hostPath": hostPath volumes are not allowed to be used, spec.containers[0].securityContext.privileged: Invalid value: true: Privileged containers are not allowed, spec.containers[0].securityContext.hostNetwork: Invalid value: true: Host network is not allowed to be used, spec.containers[0].securityContext.containers[0].hostPort: Invalid value: 10300: Host ports are not allowed to be used, spec.containers[1].securityContext.privileged: Invalid value: true: Privileged containers are not allowed, spec.containers[1].securityContext.hostNetwork: Invalid value: true: Host network is not allowed to be used, spec.containers[1].securityContext.containers[0].hostPort: Invalid value: 10300: Host ports are not allowed to be used, spec.containers[2].securityContext.hostNetwork: Invalid value: true: Host network is not allowed to be used, spec.containers[2].securityContext.containers[0].hostPort: Invalid value: 10300: Host ports are not allowed to be used, provider "restricted": Forbidden: not usable by user or serviceaccount, provider "nonroot-v2": Forbidden: not usable by user or serviceaccount, provider "nonroot": Forbidden: not usable by user or serviceaccount, provider "hostmount-anyuid": Forbidden: not usable by user or serviceaccount, provider "machine-api-termination-handler": Forbidden: not usable by user or serviceaccount, provider "hostnetwork-v2": Forbidden: not usable by user or serviceaccount, provider "hostnetwork": Forbidden: not usable by user or serviceaccount, provider "hostaccess": Forbidden: not usable by user or serviceaccount, provider "privileged": Forbidden: not usable by user or serviceaccount] for DaemonSet.apps/v1/aws-ebs-csi-driver-node -n openshift-cluster-csi-drivers happened 11 times

Documenting considerations for normalizeRegistryLocation turned up places where findReferencedImages wasn't quite doing the right thing.  Made the latter a bit more thorough, and expanded the unit tests.

/retest

/retest

/retest
infrastructure issue provisioning resources

/jira refresh

/jira refresh

/approve

/assign @coreydaley
@mtrmac I think I captured the how-we-got-here in a block comment here, but if I've left anything out, please remind me of it.

@mtrmac I think I captured the how-we-got-here in a block comment here, but if I've left anything out, please remind me of it.

I think that’s accurate and comprehensive.

/test all

I would like it to pass the tests again and then I think we can merge it
/lgtm
/approve


OCPBUGS-18454: Match restrictions on ConditionalUpdateRisk name with Condition reason

Conditional risk names are used as a Reason field values when the risk
applies and CVO sets up the Recommended condition. Hence, the risk
names need to be valid Reason field values.
xref: https://github.com/openshift/cincinnati-graph-data/pull/4059

/jira refresh

/close
We'll probably don't want to do this and only merge https://github.com/openshift/cluster-version-operator/pull/962. Closing for now, I can always reopen.


[release-1.5] Implement :15011 spiffee cert listener for istiod

This is an automated cherry-pick of #20741

@istio-testing: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio_release-1.5
bb1de04c6b86b5c192c40deb94ecf8b1c064685d
link
/test gencheck_istio_release-1.5

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

@istio-testing: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio_release-1.5
bb1de04c6b86b5c192c40deb94ecf8b1c064685d
link
/test gencheck_istio_release-1.5

unit-tests_istio_release-1.5
bb1de04c6b86b5c192c40deb94ecf8b1c064685d
link
/test unit-tests_istio_release-1.5

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

@istio-testing: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio_release-1.5
bb1de04c6b86b5c192c40deb94ecf8b1c064685d
link
/test gencheck_istio_release-1.5

unit-tests_istio_release-1.5
bb1de04c6b86b5c192c40deb94ecf8b1c064685d
link
/test unit-tests_istio_release-1.5

integ-istioio-k8s-tests_istio_release-1.5
bb1de04c6b86b5c192c40deb94ecf8b1c064685d
link
/test integ-istioio-k8s-tests_istio_release-1.5

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

@istio-testing: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio_release-1.5
bb1de04c6b86b5c192c40deb94ecf8b1c064685d
link
/test unit-tests_istio_release-1.5

integ-istioio-k8s-tests_istio_release-1.5
bb1de04c6b86b5c192c40deb94ecf8b1c064685d
link
/test integ-istioio-k8s-tests_istio_release-1.5

gencheck_istio_release-1.5
7708b619bea701981ea3f1aa48b8d5dcc729b317
link
/test gencheck_istio_release-1.5

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

@istio-testing: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-istioio-k8s-tests_istio_release-1.5
bb1de04c6b86b5c192c40deb94ecf8b1c064685d
link
/test integ-istioio-k8s-tests_istio_release-1.5

gencheck_istio_release-1.5
7708b619bea701981ea3f1aa48b8d5dcc729b317
link
/test gencheck_istio_release-1.5

e2e-mixer-no_auth_istio_release-1.5
7708b619bea701981ea3f1aa48b8d5dcc729b317
link
/test e2e-mixer-no_auth_istio_release-1.5

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

@istio-testing: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-istioio-k8s-tests_istio_release-1.5
1f97bcd74b0faff8e7070f1f2cd24834400402c0
link
/test integ-istioio-k8s-tests_istio_release-1.5

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


NoneType error for values object in blockchain.py

  File "blockchain.py", line 260, in register_nodes
    nodes = values.get('nodes')
AttributeError: 'NoneType' object has no attribute 'get'

Is anyone else having this issue?

values = request.get_json()
=>
values = request.get_json(force=True)


case sensitive identifiers

Rumur mistakenly treats Murphi as universally case-insensitive. In actual fact, only the keywords are case-insensitive. Identifiers are case-sensitive. E.g. i and I are different variables.

Fixed in 67de6d9.


Add querry and config commands

Resolves #31
Resolves #50
Resolves #51

Codecov Report
Attention: Patch coverage is 98.08219% with 7 lines in your changes are missing coverage. Please review.

Project coverage is 99.23%. Comparing base (7496d49) to head (b9b3ce5).

Files
Patch %
Lines

tests/conftest.py
22.22%
7 Missing :warning:

Additional details and impacted files
@@             Coverage Diff             @@
##              main      #55      +/-   ##
===========================================
- Coverage   100.00%   99.23%   -0.77%
===========================================
  Files           13       17       +4
  Lines          572      916     +344
===========================================
+ Hits           572      909     +337
- Misses           0        7       +7

Flag
Coverage Δ

macos-latest-py-3.10
99.23% <98.08%> (-0.77%)
:arrow_down:

macos-latest-py-3.11
99.23% <98.08%> (-0.77%)
:arrow_down:

macos-latest-py-3.12
99.23% <98.08%> (-0.77%)
:arrow_down:

macos-latest-py-3.8
99.23% <98.08%> (-0.77%)
:arrow_down:

macos-latest-py-3.9
99.23% <98.08%> (-0.77%)
:arrow_down:

ubuntu-latest-py-3.10
99.23% <98.08%> (-0.77%)
:arrow_down:

ubuntu-latest-py-3.11
99.23% <98.08%> (-0.77%)
:arrow_down:

ubuntu-latest-py-3.12
99.23% <98.08%> (-0.77%)
:arrow_down:

ubuntu-latest-py-3.8
99.23% <98.08%> (-0.77%)
:arrow_down:

ubuntu-latest-py-3.9
99.23% <98.08%> (-0.77%)
:arrow_down:

windows-latest-py-3.10
99.22% <98.04%> (-0.78%)
:arrow_down:

windows-latest-py-3.11
99.22% <98.04%> (-0.78%)
:arrow_down:

windows-latest-py-3.12
99.22% <98.04%> (-0.78%)
:arrow_down:

windows-latest-py-3.8
99.22% <98.04%> (-0.78%)
:arrow_down:

windows-latest-py-3.9
99.22% <98.04%> (-0.78%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

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


Prefix all async ClusterServlet functions with a.

[!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

#674
#673  👈
#664 : 6 other dependent PRs (#657 , #660 , #663  and 3 others)
#656
#662
#665
#661
main

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

Join @rohinb2 and the rest of your teammates on  Graphite

Merge activity

Mar 28, 1:14 PM EDT: @rohinb2 started a stack merge that includes this pull request via Graphite.


Test Webiew View API finalization

Refs #46585

[ ] Mac @roblourens
[ ] Linux @sbatten
[ ] Windows: @eamodio

Complexity: 5
Create Issue
Create Issue

Overview
The webview view API has now been finalized. In this plan item, try going through to test the api and the experience of writing an extension for it

Start with the webview view extension sample

Review the docs for the new apis in vscode.d.ts

Look at webview view lifecycle (and showing and hiding view)

Does the dispose lifecycle make sense?
Are events for show/hide fired when you expect?

Look at state management for the webview views

Make sure that webview state is persisted when: vscode is reloaded, a view is collapsed, a view is moved, a view is hidden.
Try using retainContextWhenHidden. When this is enabled, then switching so that the webview is no longer visible should preserve the iframe context itself.

Are there going to be some docs about this on the website?


Add Example HttpBin Node

Description:
Add an example node using Declarative Rest API Node Design.
Showcase:

UI Functionality

Conditional Rendering
Sending Data as different types [query, body]

Credentials Examples

Bearer Token

 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.You have signed the CLA already but the status is still pending? Let us recheck it.


Redirect to canonical domain from readthedocs.org URL?

I recently added a subdomain for accessing my project's docs (xarray.pydata.org) and set it as the canonical domain. However, the old URL (xray.readthedocs.org) still works, and does not redirect to this new URL. It is possible to change this so that visitors to my old domain get redirected to the canonical domain automatically? The canonical domain does fix this issue for search engines, but I would like to more clearly signal the new URL to users (which also corresponds to a new project name).

We serve all documentation from nginx directly, so we can't handle redirects on pages that exist and are served directly. The site should generate links to the proper domain, and google will index it.

OK, fair enough. It's something worth considering in the future if the tech stack changes to make it doable.

Closing this, since it isn't really fixable in the near future.

Actually, we can incorporate it into our base redirector at least. PR: #3388


Update Qt Creator from 4.6.2 to 4.7.1

Issue description
Hello,
In the repository of NixOS, I found the Qt Creator in version 4.6.2. However I'd like to install the latest stable version (4.7.1). What are my options to install the latest Qt Creator (and Qt libs also...) please ?
I am new to NixOS but I heard that I can change this file : https://github.com/NixOS/nixpkgs/blob/617607e0437ff0ad34687498746e8970aae6d9bc/pkgs/development/tools/qtcreator/default.nix and set the number version here (and change the SHA256 obviously). Is it all ? How can I test this change locally before sending a pull request ?
Thanks in advance :)

@hugues31 In many cases it's enough to change the version + checksum. See https://nixos.org/nixpkgs/manual/#chap-quick-start how to build/test locally.

@sengaya Ok thanks for your help. I will try this and maybe send a PR. Have a nice day


Fix model caching for diffusion models and multiple GPUs

Align with modeling_base and modeling_seq2seq where we already check for "gpu" in self.device.lower().
This enables model caching to be automatically enabled for devices like "GPU.0"

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


Plugin not working at all for me

Please make sure you're using the latest version of the plugin before submitting an issue
The plugin does not work for me, I am seeing an error saying Cannot read property 'id' of undefined when I try to set the colour theme.
Screenshots

Sorry that's happening :/ Unfortunately it's not a problem with Night Owl but Code itself. I'm going to close this out, maybe try refiling in the Code repo? Feel free to reopen if I'm mistaken.
(Try restarting Code and see if it persists)


perf: only compute start offset for overlong lines

Moves the computation of the start_offset for overlong lines to just before the result is returned. There is a slight overhead for overlong lines (double the work for the first limit characters).
In practice this results in a speedup on the CPython codebase. Most lines are not overlong, or are not enforced because the line ends with a URL, or does not contain whitespace. Nonetheless, the 0.3% of overlong lines are a lot compared to other violations.
Before

Selected W505 and E501

All rules
After

Selected W505 and E501

All rules
CPython line statistics:

Number of Python lines: 867.696
Number of overlong lines: 2.963 (0.3%)

Benchmark selected:
cargo build --release && hyperfine --warmup 10 --min-runs 50 \
  "./target/release/ruff ./crates/ruff/resources/test/cpython/ --no-cache -e --select W505,E501"

Benchmark all:
cargo build --release && hyperfine --warmup 10 --min-runs 50 \
  "./target/release/ruff ./crates/ruff/resources/test/cpython/ --no-cache -e --select ALL"

Overlong lines in CPython
cargo run -p ruff_cli -- check crates/ruff/resources/test/cpython/Lib --no-cache --select=E501,W505 --statistics

Total Python lines:
find crates/ruff/resources/test/cpython/ -name '*.py' | xargs wc -l

(Performance tested on Mac M1)

That’s awesome. I’m wondering if there’s another optimization we can make here. Is it faster to compute the number of bytes in the line instead of the width? If so, could we only check lines that have more bytes than the line limit? I think it would be impossible for an overlong line to have fewer bytes than the limit, though lines with more bytes could end up being below the limit.

That’s awesome. I’m wondering if there’s another optimization we can make here. Is it faster to compute the number of bytes in the line instead of the width? If so, could we only check lines that have more bytes than the line limit? I think it would be impossible for an overlong line to have fewer bytes than the limit, though lines with more bytes could end up being below the limit.

That sounds like a neat optimization. Each character is between 1-4 bytes. Meaning your assumption here is safe.
Checking the byte length is O(1)

![check byte length(https://github.com/astral-sh/ruff/assets/9756388/0b262119-ab1e-4aaf-9315-be2d6f170f18)


SES send_raw_email() fails with MessageRejected exception

First of all, thank you for this interesting and great package!
I just stumbled upon an issue, and hope you can help me out. The following test:
import pytest

import boto3
import moto

def test_ses(monkeypatch: pytest.MonkeyPatch):
    monkeypatch.setenv("AWS_ACCESS_KEY_ID", "testing")
    monkeypatch.setenv("AWS_SECRET_ACCESS_KEY", "testing")
    monkeypatch.setenv("AWS_SECURITY_TOKEN", "testing")
    monkeypatch.setenv("AWS_SESSION_TOKEN", "testing")
    monkeypatch.setenv("AWS_DEFAULT_REGION", "us-east-1")

    with moto.mock_ses():
        client = boto3.client("ses")
        client.send_raw_email(
<EMAIL_ADDRESS><EMAIL_ADDRESS>            RawMessage={
                "Data": "",
            },
        )

fails with this exception:
Traceback (most recent call last):
  [ ... ]
  File "/path/to/test/tests/test_ses.py", line 16, in test_ses
    client.send_raw_email(
  File "/path/to/test/.venv/lib/python3.10/site-packages/botocore/client.py", line 530, in _api_call
    return self._make_api_call(operation_name, kwargs)
  File "/path/to/test/.venv/lib/python3.10/site-packages/botocore/client.py", line 964, in _make_api_call
    raise error_class(parsed_response, operation_name)
botocore.errorfactory.MessageRejected: An error occurred (MessageRejected) when calling the SendRawEmail operation: Did not have authority to send from email<EMAIL_ADDRESS>
This is with Moto v4.1.14 and Boto3 v1.28.19/Botocore v1.31.19.
Am I missing something? Looking at the docs I am unsure if need to mock/set up a sending authority here, i.e. the AWS_BLA environment variables with certain mock values?

@bblommers that helped, works like a charm! Thank you!


extract LFC files

Hi Neil, I wanted to extract the LFC files again, by using :
apero_processing other_run.ini , where setting to True: RUN_PP_LFC  and RUN_EXT_LFC
I had done it before, with the same command according to my notes.
I checked that the file exists and is accessible.
Any clue what's going on? thanks!
But I got this error message (I copy the end of it)
21:29:16.502-@!|PROC|W[40-503-00019]: Error found for ID='531'
21:29:16.508-@!|PROC|	apero_preprocess_spirou.py 2021-06-19 2618160a.fits --crunfile=other_run.ini --program=PP_LFC[00531] --recipe_kind=pre-lfc --shortname=PP_LFC --parallel=True
21:29:16.515-  |PROC| ***************************************************************************
21:29:16.521-  |PROC|
E[01-001-00014]: Read Error: Fits-Image (data) could not be read.
Filename = /Volumes/dataDRS/spirou/raw_lst/2021-06-19/2618160a.fits ext = 2
Error was type <class 'TypeError'>
Traceback (most recent call last):
File "/Users/moutou/apero-drs/apero/io/drs_fits.py", line 644, in
_read_fitsimage
data = np.array(hdulist[ext].data)
File
"/Users/moutou/miniconda3/envs/apero-env-07286/lib/python3.9/site-packages/astropy/utils/decorators.py",
line 837, in get
val = self.fget(obj)
File
"/Users/moutou/miniconda3/envs/apero-env-07286/lib/python3.9/site-packages/astropy/io/fits/hdu/image.py",
line 250, in data
data = self._get_scaled_image_data(self._data_offset, self.shape)
File
"/Users/moutou/miniconda3/envs/apero-env-07286/lib/python3.9/site-packages/astropy/io/fits/hdu/image.py",
line 809, in _get_scaled_image_data
raw_data = self._get_raw_data(shape, code, offset)
File
"/Users/moutou/miniconda3/envs/apero-env-07286/lib/python3.9/site-packages/astropy/io/fits/hdu/base.py",
line 547, in _get_raw_data
return self._file.readarray(offset=offset, dtype=code, shape=shape)
File
"/Users/moutou/miniconda3/envs/apero-env-07286/lib/python3.9/site-packages/astropy/io/fits/file.py",
line 378, in readarray
return np.ndarray(
TypeError: buffer is too small for requested array
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/Users/moutou/apero-drs/apero/io/drs_fits.py", line 662, in
_read_fitsimage
data = deal_with_bad_file_single(filename, ext=ext,
File "/Users/moutou/apero-drs/apero/io/drs_fits.py", line 1306, in
deal_with_bad_file_single
return np.array(hdulist[ext].data)
File
"/Users/moutou/miniconda3/envs/apero-env-07286/lib/python3.9/site-packages/astropy/utils/decorators.py",
line 837, in get
val = self.fget(obj)
File
"/Users/moutou/miniconda3/envs/apero-env-07286/lib/python3.9/site-packages/astropy/io/fits/hdu/image.py",
line 250, in data
data = self._get_scaled_image_data(self._data_offset, self.shape)
File
"/Users/moutou/miniconda3/envs/apero-env-07286/lib/python3.9/site-packages/astropy/io/fits/hdu/image.py",
line 809, in _get_scaled_image_data
raw_data = self._get_raw_data(shape, code, offset)
File
"/Users/moutou/miniconda3/envs/apero-env-07286/lib/python3.9/site-packages/astropy/io/fits/hdu/base.py",
line 547, in _get_raw_data
return self._file.readarray(offset=offset, dtype=code, shape=shape)
File
"/Users/moutou/miniconda3/envs/apero-env-07286/lib/python3.9/site-packages/astropy/io/fits/file.py",
line 378, in readarray
return np.ndarray(
TypeError: buffer is too small for requested array

"buffer is too small for requested array" is almost always a corrupted file on disk,
I would try re-downloading the file from CADC/CFHT.

OK thanks, I'll try that. I thought that it had prevented other jobs but in fact I have most of them.


Adding tokens through URL Token Storage fails

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to 'Settings' and 'URL - Token Storage'
Click on 'Add new credentials'
enter name and URL to tokenset
Click "Save"
-> Nothing happens

Expected behavior
Tokens should load.
doing the same through Github does work (same tokenset)
Screenshots or Screencasts
https://www.loom.com/share/15ea4503b3534c6d954e4e07175c31e2
Figma file (optional)
Bug happens on any file
JSON (optional)
I'd rather not but as the screencast shows, both Github and URL are pointing to the same JSON file So I'd expect that to be valid.

If i have old token url and click there save, i got it twice in the list.
there is a index storage bug i guess.
cheers

+1, also running into this issue

Oh, thank you @ejlayne I thought I was going mad.

Oh, thank you @ejlayne I thought I was going mad.

nope, same problem here

Aaand... Does anyone else notices components for which I could swear I "removed" tokens from now all of a sudden still have those tokens assigned to them? almost as if the internal dbase structure of which tokens are assigned to what layers changed. Or am I getting paranoid?

Aaand... Does anyone else notices components for which I could swear I "removed" tokens from now all of a sudden still have those tokens assigned to them? almost as if the internal dbase structure of which tokens are assigned to what layers changed. Or am I getting paranoid?

There was an issue yesterday that was fixed that could have caused that behavior to appear briefly. Not related to URL storage though.


Database layer PR

This PR includes a rough database layer with limited error handling and a tighly coupled service layer
error handling will need to be refined and an pub sub dependency will need to be written to decouple the event loop from the service layer

Ill leave this open for a bit, if anyone wants to take a look and drop a review feel free, this is a very rough version
cc: @jhollowe


[JN-1114] Add justification flow to response editing

DESCRIPTION (include screenshots, and mobile screenshots for participant UX)

Updates the survey response editing flow to require a justification. We ask for the justification up front because autosaves will happen automatically in the background. The justification will be used for the duration of their edit session. A justification is not required on study-staff forms, only for participant forms.
Updates the AdminUserProvider to also fetch the list of superusers if the logged-in user is a superuser. This allows us to display the usernames of superusers throughout the admin UI instead of just displaying superuser (4a6f...)
Updates the audit table to show which admin made the changes to the user

Note: in the video below, some entries show Admin (undefined). That's a display bug that I've fixed, but I didn't feel like recording the video again 🙂
https://github.com/broadinstitute/juniper/assets/7257391/4f58239f-c2c5-4891-8151-851cc3bfe814
TO TEST:  (simple manual steps for confirming core behavior -- used for pre-release checks)

Rebuild ui-core and restart admin UI and admin API
Log in and start editing a user's existing responses
Confirm that you see your edits and justification in the audit history

per conversation with Connor, throwing this back into draft to handle the audit info stuff a little more clearly


Isotope pattern evaluation

Maybe add isotope pattern evaluation to the rows filter

Cl, Br, ... only keep features with specific isotope pattern

@adafede

Hi Robin I actually had a question on this feature, especially for Cl and Br (halogen)
In MZmine 2.53, isotope grouping is only available for 13C/12C, is that correct?
In the latest MZmine 3, I see there is a isotope pattern finder option. If I add these halogen compounds (Cl, Br), will the isotope filter catch these isotopes as well? Or does it only find the pattern and show the results in the feature list?
Thank you in advance! (sorry if not to ask questions on this post, I can create a new one, if needed)

Isotope finder goes back to the spectral data to find all signals that might be an isotope signal. Basically following the idea that it is better to catch all signals then loosing a few because of feature constraints. At least better when defining a potential isotope pattern.
The feature list is unchanged. Only the isotope pattern assigned to a feature changes

Hello @robinschmid, could it be possible, that the 13 C isotope finder is not able to pick the most intense peak in an isotope pattern anymore, when the isotope pattern is altered, e.g. by the presence of chlorine?


API for task output

Hi,
I have 3 questions , please suggest.
1st question :
In UI we can see the task status inside workflow and the input/output of the task.
I want to get the output of task so is there any apis for that ?
2nd question
I have created a workflow with some 2-3 tasks now i want to change the workflow means i want to add more task or replace the one , current apis is throwing exception now.
3rd question
Current UI is basic for getting the data , do we have support for post in current UI so that we can define some tasks or workflow from UI?
Please suggest.
Regards
Ajay

@ajayaks check out the swagger APIs - there is an API for getting task details GET /tasks/{id} that will allow you to get the task details along with the output.
For updating, which API are you using and what exception are you getting?  Please always include these details.  Some of the update APIs take an array of objects, so you might have to wrap your workflow definition inside a JSON array when sending an update.
Support for updates from UI is something we plan to add in the future.


Update typesafe:config to 1.4.0

Updates com.typesafe:config from 1.3.4 to 1.4.0.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.typesafe", artifactId = "config" } ]

labels: semver-minor

Coverage decreased (-0.2%) to 94.874% when pulling d3e98ee9736a28f330aa251bd48e1369347c60d5 on scala-steward:update/typesafe-1.4.0 into 5f72dc378fc2f3cf36c73954216272d9a8291020 on iheartradio:master.

Coverage decreased (-0.2%) to 94.874% when pulling d3e98ee9736a28f330aa251bd48e1369347c60d5 on scala-steward:update/typesafe-1.4.0 into 5f72dc378fc2f3cf36c73954216272d9a8291020 on iheartradio:master.


 regression -- Caddy needs to update tdewolff/minify in buildsrv

:smile: Been looking into a problem with tags inside pre elements. After checking with tdewolff/minify#115, and reading through caddyserver/buildsrv/blob/master/server/build.go#L37, I went and saw issue caddyserver/buildsrv#36.
TL;DR there was a regression in tdewolff/minify, fixed in August, that needs to be merged into the available version of your plugin from the Caddy download site. I have built your plugin into the latest source version of Caddy using go get and can confirm it fixes the issue. Thank you in advance and keep up the great work. 👍

Hello @Nycroth! In this moment what we can do is to @mholt to update the server, but soon we, developers, will be enabled to update the plugins when we want them to be updated 😄

Sorry. Build from source for now. Will get around to it! Classes just started...

Just updated the build server.


Huynh Van Tu An

Name: Huynh Van Tu An
GitHub: @arishuynhvan
Twitter: @ArisHuang2
Linkedin: link
Website: TBU

Completed Courses
TBU
Name of the Section

Course
Files

Course Name
link

@arishuynhvan
Can you create your profile in that way?
Thanks!


Go version update

This makes go version 1.22 explicit for the devcontainer (was implicit in version '1').

The version is already explicitly set in docker-compose.yml. I believe this is where it needs updating (unless something dramatically changed since the last time I looked at this).

skip this for now


Unable to change topic

My TV needs the publish and subscription topic to be ">BLANK</remoteapp" and not "hisense/remoteapp"
Leaving subscription and publish topic blank during initial setup results in it defaulting to "hisense/remoteapp."
Setting the subscription and publish topic as '/' results in the topic to becoming  ">BLANK</>BLANK</remoteapp"
I have been able to get it partially working with NodeRed by bridging topics, but it will be a pain to do this for every topic back and forth. Setup like this was difficult enough.
My TV is up to date. I have seen another user with this issue solve this by updating their TV in the HA Hisense thread, but it did not solve my issue.
Could there be any solution?
Thanks

Can you describe your setup in detail? Are you bridging your TV broker with the broker used by HA (usually Mosquitto)? If so, you can define the mapping there. See the readme.
The integration currently depends on such setup. If you leave the prefix empty, it will use the default hisense.

Yes I have a working bridge outside of NodeRed. I only used NodeRed to take messages from the HA broker topic (for example):
hisense/remoteapp/tv/ui_service/HomeAssistant/actions/gettvstate
Then NodeRed will change the topic to:
/remoteapp/tv/ui_service/HomeAssistant/actions/gettvstate
(note the slash at the beginning)
This was so I could initially configure the integration... making the brigdge pointless plus i would need to do this with every topic back and forth...
Is there any way we can have the publish and subscription topic be left blank and not default to hisense?

I have fixed my issue with Mosquito topic bridging.
connection hisensemqtt
address <IP_ADDRESS>:36669
username hisenseservice
password multimqttservice
clientid HomeAssistant
bridge_cafile /ssl/hisense.crt
bridge_insecure true
bridge_tls_version tlsv1.2
try_private false
start_type automatic
topic remoteapp/# in 0 hisense/ /
topic # out 0 hisense/ /


Add depthTestAgainstTerrain to ground clamping sandcastle demo

Reported on the forum: https://groups.google.com/forum/?hl=en#!topic/cesium-dev/68jiQll7WJc
http://cesiumjs.org/Cesium/Apps/Sandcastle/gallery/Ground Clamping.html
needs to have viewer.scene.globe.depthTestAgainstTerrain = true; so the billboard/label don't show up through the mountain.

#4963


Update Event categories and subcategories list

Description
Please provide a summary of the pull request and the issue it fixes. Please add necessary details, context, dependencies, explanation of when review is needed (see next section), etc.
Fixes #(add issue number here and remove parentheses)
Review Time Estimate
Please give your idea of how soon this pull request needs to be reviewed by selecting one of the options below. This can be based on the criticality of the issue at hand and/or other relevant factors.

[ ] Immediately
[ ] Within a week
[x] When possible

Type of changes
Please select a relevant option:

[ ] Bug fix (non-breaking change which fixes an issue).
[x] New feature (non-breaking change which adds functionality).
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected).
[ ] Other (any another change that does not fall in one of the above categories.)

Checklist:
Please select all applicable options:

[x] I have signed the Rokwire Contributor License Agreement (CLA). (Any contributor who is not an employee of the University of Illinois whose official duties include contributing to the Rokwire software, or who is not paid by the Rokwire project, needs to sign the CLA before their contribution can be accepted.)
[x] I have updated the CHANGELOG.
[ ] I have read the Contributor Guidelines.
[x] I have performed a self-review of my own code.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] My change requires updating the documentation.
[ ] I have made necessary changes to the documentation.
[ ] I have added tests related to my changes.
[ ] My changes generate no new warnings.
[ ] New and existing unit tests pass locally with my changes.
[ ] Any dependent changes have been merged and published in downstream modules.

Running the Mongodb import command will append new categories with previous categories.
@bingzhang, I think it's better to move this to db.py, and we need to replace the categories with the new categories. What do you think?
For the purpose of review, I will drop the local collection and add the new categories.

Please update the API specification file as the categories list is used in that too.

that is why I mentioned that in our technique meeting. I can work python code like what the current tag.json does.

Hi, @bingzhang, FYI, I'm updating this PR with README changes and potentially some minor code changes.


local api endpoints

we are running locally on linux.
When we use curl we can send inputs": "Translate this text to French"
Are there other endpoints exposed ?
We tried this  "question": "Who is Frederick Douglass?"
and received "detail":"Not Found"}
Thanks !

If you are running locally on port 8080, the endpoints can be inspected at http://localhost:8080/docs as standard openapi - let me know whether it works!


Show no upcoming departures when last stop

Summary of changes
Asana Ticket: 🐞 Schedule Finder | Error Loading Trip Details
Slight improvements related to Upcoming Departures:

Show filtered-by-variant stops (if variant exists) in the origin modal
Do not fetch trips if it's the last stop (and show a message instead)

BTW I deployed this branch on dev green since what was currently on there is already merged. :)

I had forgotten to include a small addition. You should be able to see the error message now.
Changes in this branch:

For the scenario in which we want to display upcoming departures but the origin is the last stop, there is a call to get the upcoming departures that results in error. With these changes, the call is prevented and this error message is shown instead. This is the only thing the end user would see.

When there is a variant selected, the list of stops that the origin modal displays is all the stops of the line, regardless of the variant; so in the example above you're able to select Ruggles even though it's the last stop for variant 15-1-1 (and you shouldn't). With the changes in this branch, we're filtering the list of stops taking into account the variant and so the final list only contains the pertinent stops.

I guess the edge case is having this page bookmarked already, in which case Ruggles is grayed out (correct) but selected (wrong). This branch doesn't consider this edge case. But let me know if it should.

So...reading all your comments and looking at the changes in #599 (which I still need to fully understand), you have a point. So I don't think my approach is good, or good enough anyway.
I'm going to close this and try a different approach from scratch, and perhaps we can discuss further if that still doesn't look like a valid solution.

@alicia-mbta might be worth spinning up a new design ticket? Here's how to get it into my backlog: https://form.asana.com?k=pgrwg-Drarrtzzfda4WktA&d=15492006741476

@alicia-mbta That's fair. Do document your learnings so far in the ticket as part of the investigation!


Initial web version

admittedly I have problems always making sense of how Github treats pull requests. This looks ready to go, but I got 4 emails with consecutive changes. Are you ready for me to merge?

To satisfy my slight OCD (haha), would you mind quickly taking care of the flake8 issues in the build?

I checked it out and ran it ... looked good! I have to talk to you about the secret key because it didn't appear to work for me like yours did.

Yeah... not quite ready to merge yet. As for the secret key... did you try the .env version or the export method?

the "yes I'm lazy" export version

Anything wrong with merging this branch and then just creating an "added-hugs" branch or some such? I mean, it merges without conflicts and doesn't appear to crash or anything. If we create a pattern of smaller focused updates, I think it will be easier to stay in sync.

Yeah... I'm on board with that... merging.
I'm tend to use black (auto format on save) so once I turn that on, there will initially be some uglier diffs if I touch something with that.

OCD placated! Merging now. Here we go.


Issue of Injecting MathJax equation in Angular project

Hi There,
I'm new of using MathJax, so I did some research of how to apply MathJax equation into Angular project.
From my research, I added the following codes into the index.html file of Angular project:
<script type="text/x-mathjax-config">
    MathJax.Hub.Config({
      tex2jax: {
          inlineMath: [['$','$'], ['\\(','\\)']],
          processEscapes: true
          }
    });
  </script>
  <script type="text/javascript" src="//cdn.mathjax.org/mathjax/latest/MathJax.js?config=TeX-AMS-MML_HTMLorMML"></script>

Then, in one of my component's html file, I added a MathJax expression for testing as the following code:
$$x = {-b \pm \sqrt{b^2-4ac} \over 2a}.$$
But, I got error message from Chrome developer tools. It's weird that this equation is not working for me, but the following MathJax is working.
$\int x^2dx = 2x + C$
Can anyone explain why? Does it means Angular does not support complex MathJax equation?
Thanks for your comments

I realised the issue is [ERROR ->]"): ng:///AppModule/ConcreteDesignComponent.html@165:6 Invalid ICU message. Missing '}'. ("
I try escaping the single curly braces with {{ "{" }} but don't get the math equations rendered.
And found a solution from https://github.com/angular/angular/issues/17416.
Even though is not very convenient, but still working for now. Hope MathJax and Angular experts can found a better solution together. Since Angular is quite popular, it's better to have MathJax works on it.


perf(nuxt): save reference to options

🔗 Linked issue

❓ Type of change

[ ] 📖 Documentation (updates to the documentation, readme or JSdoc annotations)
[ ] 🐞 Bug fix (a non-breaking change that fixes an issue)
[x] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

There are many repeated references to nuxt.options in both generateApp and resolveApp, why not save a reference to improve performance? I initially thought about making it a shared variable but decided not to (curious to know if you think it's worth adding as a shared variable).
📝 Checklist

[ ] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

@danielroe Before I continue with my PR's, I wanted to ask you: I decided to go on a little optimization spree (as you can see from the latest 4 PR's). Should I combine them all into a single PR based on the affected package or keep them as is? On one hand I don't want to spam the PR page with my contributions, but on the other hand I remember you asking PR's to be very specific, and to include each additional contribution in a separte PR to keep it on-topic.

The separate PRs are totally fine.
But I'm not sure that there is a meaningful performance gain to be made by reducing one level of object property access, compared to the decrease in readability. We deliberately try to access everything off the nuxt object, for example, for clarity.
Have you done any measurements that suggest this is a meaningful performance gain?

Have you done any measurements that suggest this is a meaningful performance gain?

I can run benchmarks for this for sure, I would assume it's similar to #24715

@danielroe @pi0

According to the image, it appears that there is a slight performance improvement. I agree that while it's a very little gain, it is still a gain. I'll leave that up to your guys' decision though.

Update: offline tests present an even bigger improvement in the approach I took:

But as I said, I'll leave it to your judgement.


KeyError: 'model_state_dict'

Thanks for the great repository.
I ran the Train.sh file for fine tuning with the original data from the res50.pth file and got the following error.
How can I solve this error?
Thank you
Traceback (most recent call last):
File "../tools/train.py", line 290, in
main()
File "../tools/train.py", line 286, in main
main_worker(0, ngpus_per_node, train_args, val_args)
File "../tools/train.py", line 221, in main_worker
load_ckpt(train_args, model, optimizer.optimizer, scheduler, val_err)
File "/home/robotics/AdelaiDepth/LeReS/Train/lib/utils/net_tools.py", line 44, in load_ckpt
checkpoint_state_dict_noprefix = strip_prefix_if_present(checkpoint['model_state_dict'], "module.")
KeyError: 'model_state_dict'

Hi @283pm, did you try replacing the 'model_state_dict' with 'depth_model'?

Yes, I did.
If I replace "model_state_dict" with "depth_model",  I got the following error.
RuntimeError: Error(s) in loading state_dict for RelDepthModel:
Missing key(s) in state_dict: "depth_model.encoder_modules.encoder.conv1.weight",
"depth_model.auxi_modules.out_conv.adapt_conv.3.bias".
Unexpected key(s) in state_dict: "shift_model", "focal_model".

Yes, I did.
If I replace "model_state_dict" with "depth_model", I got the following error.
RuntimeError: Error(s) in loading state_dict for RelDepthModel:
Missing key(s) in state_dict: "depth_model.encoder_modules.encoder.conv1.weight",
--- many weights and biases ---
"depth_model.auxi_modules.out_conv.adapt_conv.3.bias".
Unexpected key(s) in state_dict: "shift_model", "focal_model".

I have made some modifications in a   new fork. Give it a try to see if it can work for you.

Your modifications solved my problem.
Thank you.


Add GA4 events to special route format forms

⚠ This repo is Continuously Deployed: make sure you follow the guidance ⚠
What
Add GA4 tracking to special route formats.
Special route formats
Example: https://www.gov.uk/find-local-council
Example result: https://www.gov.uk/find-local-council/waverley (county and district)
Example result: https://www.gov.uk/find-local-council/westminster (local authority)
To test locally, modify controllers/find_local_council_controller.rb, replacing the result method with the below. This will not allow form submissions to work correctly, but you can go directly to the example result pages. Note the comment part way down that allows switching between the two kinds of result page.
def result
  authority_results = {}
  authority_results["local_authorities"] = []
  first_authority = {}
  first_authority["name"] = "Humbridge"
  first_authority["homepage_url"] = "https://www.gov.uk"

  second_authority = {}
  second_authority["name"] = "Windle"
  second_authority["homepage_url"] = "https://www.google.co.uk"

  authority_results["local_authorities"] << first_authority
  authority_results["local_authorities"] << second_authority # uncomment this line to show the alternative result page

  if authority_results["local_authorities"].count == 1
    @authority = authority_results["local_authorities"].first

    render :one_council
  else
    @county = authority_results["local_authorities"].first
    @district = authority_results["local_authorities"].last

    render :district_and_county_council
  end
end

Why
Part of the GA4 migration.
Visual changes
None.
Trello cards:

https://trello.com/c/RF5EB7QT/547-form-events-formsubmit-formerror-formcomplete-and-informationclick-on-specialroute-format

@AshGDS think I've addressed your comments if you'd like to check. Also was having some weird merge conflicts trying to rebase the changes into the right commit, so I've merged all the commits into one for simplicity.


Releasing libs should depends on a successful release to edge

Currently, on merge, libs are released separately from release to edge.
This is a problem because if release to edge fails, then charmhub would have a lib version that is not yet part of the charm itself.
Maybe releasing the libs should be a step in the release-to-edge workflow?

This has been addressed as a drive-by fix in https://github.com/canonical/observability/pull/85 :)


More detailed exception messages in ListenerConfigurationChecker

Q
A

Is bugfix?
❌

New feature?
❌

Breaks BC?
❌

Fixed issues
#31

👍


fix: auto-generate the json schema

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign jstrachan
You can assign the PR to them by writing /assign @jstrachan in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

@jstrachan: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

pr
8b81c348db9f7cae878b953efa527cc2fe7cdc61
link
/test

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

/approve

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: jstrachan
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [jstrachan]

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


Breadcrums disappear on mobile

I'm making this website: https://zenned.gitlab.io
When I zoom in, or visit the site on a phone, the breadcrumbs disappear. Even when there is still plenty of room for them.
Is there any way to prevent this? 🤔
Do the breadcrumbs make this decision relative to available page width?

On smaller screens the breadcrumbs are hidden by intent. This is not configurable.


does this model support Multiple input features to predict price?

I am using this model on stock companies to predict their price of next ten days . Does TimesFm support time series prediction with multiple input features? Along with price, can we give input features like volume , RSI ?

Afaik, it doesn't, unless you come up with yet another architecture to do so (i.e., like adaptation). Stocks are multivariate, but TimesFM is for univariate!

https://github.com/google-research/timesfm/blob/master/notebooks/covariates.ipynb
Looks like its possible
Multivariate Time Series
For multivariate time series, if we need univariate forecast, we can try treating the main time series as the target and use the rest as the dynamic covariates.


Resurrection of Polydictyum from a synonym of Tectaria [PASSED]

Author(s) of proposal
Li-Bing Zhang, Liang Zhang
Name of taxon
Polydictyum
Rank of taxon
Genus
Approximate number of species affected
5
Description of change
Resurrection of Polydictyum C.Presl (1851) from a synonym of Tectaria Cav. (1799).
Reason for change
Polydictyum C.Presl (1851) has been treated as a synonym of Tectaria by various earlier authors Copeland, 1907; Tardieu-Blot & Ching, 1936; Tryon & Tryon, 1982; Holttum, 1991) and tentatively by PPG I (2016). Molecular data showed that it is not even closely related to Tectaria but is sister to Pteridrys with strong support (Dong et al., 2018; Zhou et al., 2018), an Old World genus with ca. 24 species (Zhou & Zhang, 2019). Polydictyum + Pteridrys together are strongly supported as sister to Malaifilix + Draconopteris.
Morphologically, Polydictyum is characterized by having a combination of characters, such as creeping rhizome, a rachis without wings, anastomosing veins, sinuses lacking teeth, and rounded indusia.
Polydictyum contains about 3-5 species currently known in central Vietnam, China (Hainan Island), and Malesia (the Philippines to Fiji)(Dong et al., 2018; Zhou et al., 2018).
Reference(s) for publication of the name
Dong S-Y,  CW Chen, S-S Tan, H-G Zhao, Z-Y Zuo, Y-S Chao, Y-H Chang. 2018. New insights on the phylogeny of Tectaria (Tectariaceae), with special reference to Polydictyum as a distinct lineage. Journal of Systematics and Evolution 56: 139–147.
Zhou, X-M, L Zhang, NT Lu, X-F Gao, L-B Zhang. 2018. Pteridryaceae: A new fern family of Polypodiineae (Polypodiales) including taxonomic treatments. Journal of Systematics and Evolution 56: 148–173.
Zhou, X-M, L-B Zhang. 2019. A monograph of the fern genus Pteridrys (Pteridryaceae). Systematic Botany 44(2): 243–273.
List the numbers of any related issues
No response
Code of Conduct

[X] I agree to follow the PPG Code of Conduct

Links to papers:

Dong et al. (2018)
Zhou et al. (2018

This proposal was voted on during PPG Ballot 8 (voting period February 2024). A total of 70 votes were cast. There were 61 'Yes' votes (87.1%) and 9 'No' votes (12.9%). The proposal passes.


object_detection: confidence and suppression interaction

Does suppression default to a sane value if confidence is set lower than the default of 0.4 for suppression ?

Can you elaborate a little? suppression is used to remove any overlapping bounding boxes and comes in before confidence gets into the picture, they are not really related

I guess this might become [DOC] since I'm a bear of very little brain... :-)  Can you elaborate on how suppression and confidence work? I guess I don't understand how to tune the suppression value. I do have many captures where there are two bounding boxes, both for "person" around a person, and sometimes something similar for car.

You shouldnt really need to tune the suppression value. What problem are you having that you want to solve?
The suppression technique is called non-maximum suppression and is built into the OpenCV Darknet detector. You can read a little bit more how it works here if you are interested

I'm not trying to solve any problem at present, I'm just trying to understand what the knobs do. The link you provided is exactly what I was looking for. I'm very new to ML so don't quite know yet where to look or what subsystem is involved in what decisions.

Ahh i see. I know how you feel, i was completely clueless before i dove into this project :)
I appreciate all your questions and feedback, makes it easier for me to fix up the README.
Its always hard writing good instructions for something you know quite a lot about...

Closing due to inactivity, feel free to reopen if the issue persists


Test and document the linked list from ds::linked_list.

Although our linked list implementation does feature comments, they are more about the underlying technical details and do not really help newcomers understand how to use it. This matters because we do not have std::list.
Address this by adding a test for the linked list which also explains how to use it for a basic use-case.

@nwf-msr @rmn30  All comments addressed (hopefully) :slightly_smiling_face:


NOT formula context doesn’t unlock in exercise 11.1

Reaching exercise 11.1 (through solving exercise 10.1(a)) and clicking A from the formula window does not give NOT(A) as one of the options.
Tested in Chrome 69 and reproduced in Edge 25.

I think I see the issue: https://github.com/teorth/QED/commit/d7e42ccc03d19f977acb8d11037fce99749f1ae3#diff-bde0b004f70c3410a1b8a814291ec60aL184
I will push a fix.

Okay, I pushed the fix. I'm not sure if it will "self correct" or not though. Can you see?

@cjquines Can confirm the fix is applied. Just need to click into Ex11.1 again to unlock the law.


No supported Nvidia GPU found

I've tried most of the solutions proposed in the other similar issues. However it still does not work on my part since i'm getting :
2024-06-28 18:38:09 | INFO | configs.config | No supported Nvidia GPU found
overwrite preprocess and configs.json
2024-06-28 18:39:38 | INFO | configs.config | Use cpu instead
is_half:False, device:cpu

I have installed :
torch                  2.3.1+cu118
torchvision            0.18.1+cu118

and my GPU is getting detected :
>>> import torch
>>> torch.cuda.is_available()
True
>>> torch.cuda.device_count()
1
>>> torch.cuda.get_device_name(0)
'NVIDIA GeForce GTX 1050 Ti'

I am using this version of RVC : 2.2.231006
If anyone got any ideas I would be happy to try them out.
Thanks.

i have the same issue, do you figured how to fix that?


Allow prebaked application token in environment variables

Hi there,
I know that gitea currently provides a UI to generate application tokens. However it also needs one application token(a master API key/token perhaps) as environment variable, this way I can deploy gitea in an automated fashion without utilizing the UI.
In addition, in my own situation I built a microservice that interacts with gitea over webhooks(it posts CI jobs, and makes PR comments on CI running/results), this microservice/k8s deployment uses $GITEA_SERVER_TOKEN env variable to interact with the gitea API(to post a PR comment, get open PRs etc). Therefore, this makes the current deployment very challenging without having a prebaked/predetermined application master API token, as that deployment needs to be restarted with the application token in the environment variable when I generate it from the UI currently. Having one token as an environment variable can solve all these problems.

You can create a token at first via API.

POST /users/{username}/token requires an authentication token, it gives 401. How can I generate a token without an authentication token?

POST /user/{username}/token requires BASIC authentication not Token.

@zeripath this works, thank you! Let's also document it, closing this issue.


⚠ Orpheus has degraded performance

In 6f006fd, Orpheus ($OPS) experienced degraded performance:

HTTP code: 200
Response time: 1970 ms

Resolved: Orpheus performance has improved in dc85f12.


Added Framework::Matplotlib to setup.cfg

Was having trouble finding this package, couldn't remember the name. Figure this would have helped me and make it slightly more discoverable. Also, please consider adding this package to https://matplotlib.org/mpl-third-party/

@story645 thanks for this PR! Flexitext is already in https://matplotlib.org/mpl-third-party/

maybe it should be in a different place?

The failure is due to https://github.com/psf/black/issues/2964. Can you update black to 22.3.0 please?

Sorry must have missed it by going too fast & I think the fix requires a reworking on that page. :/

@story645 we can merge it now if you think it's ok! I'm not familiar with the Framework in setup.cfg but if you think it's fine, we can merge it

It's fine! Thanks for the contribution!
And let me know if you have questions or suggestions about the library. I'm happy to help if possible :)


UI:Fixed homepage button sizing issues on smaller screens

in the solo game mode, if you have along question.

it should be like this

a


Add new S3 metric to cloudwatch exporter

In Sep 17, 2021 , Amazon CloudWatch request metrics for Amazon S3 Access Points now available .
There was only BucketSizeBytes and NumberOfObjects before , and now request metrics、replication metrics、Amazon S3 on Outposts CloudWatch metrics available .
Metrics like GetRequests 、PutRequests are very important to S3, Can you add these new metrics to cloudwatch_exporter ?
Refer to the link
https://aws.amazon.com/about-aws/whats-new/2021/09/amazon-cloudwatch-metrics-s3-access-points/?nc1=h_ls
https://docs.aws.amazon.com/AmazonS3/latest/userguide/metrics-dimensions.html

Hey is this still relevant? I found an example config(https://promcat.io/apps/aws-s3) that when I tried to replicate also couldn't get the requests metrics. If it's not implemented, I'd be more then happy to take a shot at implementing it.

Is this still a bug? S3 request metrics are opt-in within AWS, once enabled in AWS and the metrics added to the config (example) YACE does it magic and all works.

Replication metrics are also opt-in within AWS and configured via the replication config by enabling metrics.


get_stream_url gives 403 forbidden error when using 2fa with ios device id

I don't think this is a duplicate of #590  or #608  because I am not using FROM_MAC_ADDRESS or #473, because get_stream_url unilaterally fails on all song ids.
Can someone confirm if this is an IOS specific error - or if android ids are also having this problem.  If not - where should I look to figure this out?
Below you can see my code for cycling through available device ids - there were 3 listed by get_registered_devices - all ios
api = Mobileclient(debug_logging=False)
api.login(email,pswd,api.FROM_MAC_ADDRESS)
testsongid = api.get_all_songs()[0]['id']
# api.get_stream_url(testsongid)

for device in api.get_registered_devices():
    api.logout()
    api.login(email,pswd,device['id'].replace(':',''))
    try:
        api.get_stream_url(testsongid)
        print('DEVICE',device['id'],'AUTHORIZED')
        break
    except:
        print('DEVICE',device['id'],'NOT AUTHORIZED')

Here is the exact failure message - if it means anything to anybody
CallFailure: GetStreamUrl: 403 Client Error: Forbidden for url: https://mclients.googleapis.com/music/mplay?sig=FC5-DjBYxDhb19lq2RlnawiBwPw&slt=1535954019248&dv=0&tier=aa&net=mob&pt=e&hl=en_US&songid=2e8f1483-fa95-35da-99da-620decb935b8&opt=hi
(requests kwargs: {'allow_redirects': False, 'method': 'GET', 'url': 'https://mclients.googleapis.com/music/mplay', 'params': {'sig': b'FC5-DjBYxDhb19lq2RlnawiBwPw', 'slt': '1535954019248', 'dv': 0, 'tier': 'aa', 'net': 'mob', 'pt': 'e', 'hl': 'en_US', 'songid': '2e8f1483-fa95-35da-99da-620decb935b8', 'opt': 'hi'}, 'headers': {'Authorization': '<omitted>', 'X-Device-ID': 'ios12345678-AAAA-BBBB-CCCC-1234567890AB'}})
(response was: '\n\n<TITLE>Forbidden</TITLE>\n\n\n
Forbidden
\n
Error 403
\n\n\n')

@simon-weber
I tried the different ios id formats (though I'm not sure what you mean by the second suggestion) - but the api.login command failed.
Sent you an email w/ some params to replicate my error.
side note - do you use this api in your auto-playlists extension?

Google didn't hassle me about logging in. It looks like the ios:... form is actually valid! I managed to get a stream url using one of your device ids.
I'll tweak the device id validation to account for this.


Mass testing of the entire Conda package catalog

Basic implementation is covered, I think. Further details should go into new issues.


test(extension): add basic tests for Custom Submit API

Checklist

[X] JIRA - LW-11324
[ ] Proper tests implemented
[ ] Screenshots added.

Proposed solution
Explain how does this PR solves the problem stated in JIRA ticket.
You can also enumerate different alternatives considered while approaching this task.
Testing
Describe here, how the new implementation can be tested.
Provide link or briefly describe User Acceptance Criteria/Tests that need to be met
Screenshots
Attach screenshots here if implementation involves some UI changes

Allure Report
allure-report-publisher generated test report!

processReports: ✅ test report for 6cf3e786

passed
failed
skipped
flaky
total
result

Total
32
0
4
0
36
✅


Add badge for android-gems.com

Added badge for android-gems: http://www.android-gems.com/lib/nickbutcher/plaid , it looks like this:

Thank you very much for the contribution but i'd prefer not to add this.


Lens doesn't update view if no watch permissions

Describe the bug
Lens switched to using watch (mainly) instead of list (only, still does in some circumstances). However from an "isAllowedResource" point of view it still assumes that list implies watch (which is never true in general). Therefore if the initial loadAll call doesn't have any namespaces and a cluster doesn't have watch permissions adding more accessibleNamespaces doesn't add anything to the stores.
To Reproduce
Steps to reproduce the behavior:

Have a cluster that can list pods in a namespace, but not list or watch namespaces
Click on Overview
Observe no pods
Add a namespace to accessibleNamespaces
Go back to Overview
See that no pods are shown
Wait 30s (for the allowed resources to refresh)
See that pods are allowed but that no pods are shown
Wait any amount of time
See no pods
Reload
See pods

Lens switched to using watch (mainly) instead of list (only, still does in some circumstances).

@Nokel81 not sure what you mean by this.

I mean that before 4.1 Lens was only using list for pods (I think) to after 4.1 it started using watch as well. I might be wrong about that. But this bug still exists in the current version

Lens has been using list+watch since early days. 4.1 just moved the logic to "renderer" so it's more visible (for developers).

Ah I see, so this bug has been present for a long time then.