Why does .at execute the job immediately but .cron does not?

I'd not expect either of them to execute the job immediately. Also setting first_at: seems to have no effect when used with .at and still executes immediately.

Ah, I have misunderstood .at. It seems it's meant to be run only at one specific date and time. Ignore me.


feat: Support Expo 52 and React Native 0.76

Problem solved
Short description of the bug fixed or feature added

PR-Codex overview
This PR primarily focuses on updating dependencies to support Expo 52 and React Native 0.76. It also includes changes to scripts and dependency version specifications for improved compatibility.
Detailed summary

Added support for Expo 52 and React Native 0.76.
Updated expo-web-browser version from ^13 to ^13 || ^14.
Updated expo-application version to ^5 || ^6.
Added react-native-passkey dependency.
Changed npx expo:run commands to npx expo run.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

#5594  👈 (View in Graphite)
main

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


rspec-puppet on puppet 6.17 breaks due to removal of default_env

It looks like default_env was removed in puppet 6.17 - specifically in this commit:
https://github.com/puppetlabs/puppet/commit/94df3c1a3992d89b2d7d5db8a70373c135bdd86b
This results in an exceptiion similar to the following when rspec tries to run tests:
 Failure/Error: require 'puppetlabs_spec_helper/module_spec_helper'
 NameError:
   undefined local variable or method `default_env' for Puppet::Util:Module
 # ./vendor/bundle/ruby/2.5.0/gems/rspec-puppet-2.7.8/lib/rspec-puppet/monkey_patches.rb:139:in `get_env'
 # ./vendor/bundle/ruby/2.5.0/gems/puppet-6.17.0/lib/puppet/node/environment.rb:501:in `extralibs'
 # ./vendor/bundle/ruby/2.5.0/gems/puppet-6.17.0/lib/puppet/node/environment.rb:76:in `initialize'
 # ./vendor/bundle/ruby/2.5.0/gems/puppet-6.17.0/lib/puppet/node/environment.rb:43:in `new'
 # ./vendor/bundle/ruby/2.5.0/gems/puppet-6.17.0/lib/puppet/node/environment.rb:43:in `create'
 # ./vendor/bundle/ruby/2.5.0/gems/puppet-6.17.0/lib/puppet.rb:240:in `base_context'
 # ./vendor/bundle/ruby/2.5.0/gems/puppet-6.17.0/lib/puppet/test/test_helper.rb:53:in `initialize'
 # ./vendor/bundle/ruby/2.5.0/gems/rspec-puppet-2.7.8/lib/rspec-puppet.rb:77:in `block in <top (required)>'
 # ./vendor/bundle/ruby/2.5.0/gems/rspec-puppet-2.7.8/lib/rspec-puppet.rb:34:in `<top (required)>'
 # ./vendor/bundle/ruby/2.5.0/gems/puppetlabs_spec_helper-2.15.0/lib/puppetlabs_spec_helper/module_spec_helper.rb:1:in `require'
 # ./vendor/bundle/ruby/2.5.0/gems/puppetlabs_spec_helper-2.15.0/lib/puppetlabs_spec_helper/module_spec_helper.rb:1:in `<top (required)>'
 # ./spec/spec_helper.rb:7:in `require'
 # ./spec/spec_helper.rb:7:in `<top (required)>'

Not sure about what way to go about fixing this.

Same on puppet 5.5.21

An error occurred while loading ./spec/classes/profile_core_spec.rb.
Failure/Error: require 'puppetlabs_spec_helper/module_spec_helper'
NameError:
undefined local variable or method `default_env' for Puppet::Util:Module
./vendor/bundle/ruby/2.4.0/gems/rspec-puppet-2.7.8/lib/rspec-puppet/monkey_patches.rb:139:in `get_env'
./vendor/bundle/ruby/2.4.0/gems/puppet-5.5.21/lib/puppet/node/environment.rb:512:in `extralibs'
./vendor/bundle/ruby/2.4.0/gems/puppet-5.5.21/lib/puppet/node/environment.rb:50:in `create'
./vendor/bundle/ruby/2.4.0/gems/puppet-5.5.21/lib/puppet.rb:209:in `base_context'
./vendor/bundle/ruby/2.4.0/gems/puppet-5.5.21/lib/puppet/test/test_helper.rb:53:in `initialize'
./vendor/bundle/ruby/2.4.0/gems/rspec-puppet-2.7.8/lib/rspec-puppet.rb:77:in `block in <top (required)>'
./vendor/bundle/ruby/2.4.0/gems/rspec-puppet-2.7.8/lib/rspec-puppet.rb:34:in `<top (required)>'
./vendor/bundle/ruby/2.4.0/gems/puppetlabs_spec_helper-2.15.0/lib/puppetlabs_spec_helper/module_spec_helper.rb:1:in `require'
./vendor/bundle/ruby/2.4.0/gems/puppetlabs_spec_helper-2.15.0/lib/puppetlabs_spec_helper/module_spec_helper.rb:1:in `<top (required)>'
./spec/spec_helper.rb:5:in `require'
./spec/spec_helper.rb:5:in `<top (required)>'
./spec/classes/profile_core_spec.rb:3:in `require'
./spec/classes/profile_core_spec.rb:3:in `<top (required)>'
No examples found.
Finished in 0.00004 seconds (files took 1.57 seconds to load)
0 examples, 0 failures, 1 error occurred outside of examples

+1 same problem here

Got a quick fix for this up in a PR that hopefully someone from the maintainers can validate and/or improve since this is my first time dabbling in this repo: https://github.com/rodjek/rspec-puppet/pull/797

See also https://tickets.puppetlabs.com/browse/PUP-10586

@sgnn7 was it intentional that 2.7.10 bases on 2.7.8 and not 2.7.9?

was it intentional that 2.7.10 bases on 2.7.8 and not 2.7.9?

@dlucredativ I don't know enough about this gem to answer that - the fix is independent of the base version though

was it intentional that 2.7.10 bases on 2.7.8 and not 2.7.9?

@dlucredativ I don't know enough about this gem to answer that - the fix is independent of the base version though

Ok it seems that you only created 73fbbe2f and @scotje cherry-picked it into b6807ebe. I just don't know why.

2.7.9 unintentionally included commits intended for 2.8.x (or 3.x) as they break some peoples existing tests. We yanked the 2.7.9 release and released 2.7.10 which is just 2.7.8 plus only the fix for default_env.


Use td-agent 2 by default

I think we should move to td-agent 2 so
time to use td-agent 2 by default.

I will release chef-td-agent v2.0 with enabling td-agent 2.x by default.


Error parsing JSON template

I tried downloading loading a JSON template from a URL and when clicking the download button nothing happens.
I then tried to upload the file directly but I receive the following error:
"Error parsing JSON. Please verify contents."
The file is valid JSON and it worked a few months ago when I last uploaded it.
It seems that it's stopped working for me.
Please see the json file attached. (I renamed it to .txt so that it could be uploaded here; normally it's .json.)
default.txt

Hi @ajgisme,
With the recent update to support self-hosted JIRA instances, the JSON format has slightly changed. The options object now requires both limit and domains.
I don't see a reason they need to be enforced if they are empty, so we will remove that restriction in a future update. For the meantime, if you add the domains key to the JSON file it should load fine.
default.txt
We will also look into why the load JSON from URL option is not working as intended.
Thanks!

We have removed the requirement for each field to be present in the JSON file.
Your original file should work now.
Thanks for the issue report!


VSCode: Add tasks.json with default build command.

Huh, neat - I didn't know we could do that in vscode.
Consider it merged 👍


License?

Hello,
I found your project and would like to use it for my own project but there is no license file.
Under which license is this project published?
Thank you and with kin regards,
Fade59

I am sorry, I forgot to add the license. This is free to use or duplicate under BSD-3 Clause or MIT License. I will add the license ASAP.


vscode reports file as "modified" despite undoing all changes

Type: Bug
when i using VSCode Neovim, tab-close-button still indicating modified in spite of undoing all with u-key.
Extension version: 0.4.1
VS Code version: Code 1.80.2 (2ccd690cbff1569e4a83d7c43d45101f817401dc, 2023-07-27T20:41:52.615Z)
OS version: Linux x64 4.18.0-425.3.1.el8.x86_64
Modes:

This is an old and known issue, though I can't find an existing (closed) issue for it.
Related: https://github.com/vscode-neovim/vscode-neovim/issues/136

https://github.com/vscode-neovim/vscode-neovim/issues/247


idea: Change your password to account

Describe the solution you'd like
There should be option to change your account password in app
Describe alternatives you've considered
In option there should be button to change your password to another

This is not intended for now - for more explanation look into #28 discussion for answers.


fix in OSC.py:OSCArgument TypeError: str() takes at most 1 argument (2 given) for python 2.*

when you try to send a message to the OSC in python 2
The following error:
python OSC.py
[INFO   ] [Logger      ] Record log in /home/smglab/.kivy/logs/kivy_15-03-13_13.txt
[DEBUG  ] [Cache       ] register <kv.image> with limit=None, timeout=60s
[DEBUG  ] [Cache       ] register <kv.atlas> with limit=None, timeout=Nones
[INFO   ] [Image       ] Providers: img_tex, img_dds, img_gif, img_sdl2, img_pil (img_ffpyplayer ignored)
[DEBUG  ] [Cache       ] register <kv.texture> with limit=1000, timeout=60s
[DEBUG  ] [Cache       ] register <kv.shader> with limit=1000, timeout=3600s
[INFO   ] [Factory     ] 173 symbols loaded
[DEBUG  ] [Cache       ] register <kv.lang> with limit=None, timeout=Nones
[INFO   ] [Text        ] Provider: sdl2
[DEBUG  ] [Cache       ] register <textinput.label> with limit=None, timeout=60.0s
[DEBUG  ] [Cache       ] register <textinput.width> with limit=None, timeout=60.0s
[INFO   ] [Kivy        ] v1.9.0-dev
[INFO   ] [Python      ] v2.7.6 (default, Mar 22 2014, 22:59:56)
[GCC 4.8.2]
57 65 6c 63 6f 6d 65 20 'Welcome '
74 6f 20 74 68 65 20 4f 'to the O'
53 43 20 74 65 73 74 69 'SC testi'
6e 67 20 70 72 6f 67 72 'ng progr'
61 6d 2e ('           ', "'ram.'")
()
 Traceback (most recent call last):
   File "OSC.py", line 326, in <module>
     message.append("the white cliffs of dover")
   File "OSC.py", line 85, in append
     binary = OSCArgument(argument)
   File "OSC.py", line 182, in OSCArgument
     binary = struct.pack(">i%ds" % OSCstringLength, bytes(data, 'utf-8'))
 TypeError: str() takes at most 1 argument (2 given)

sorry, hadn't seen this PR, it was fixed in master, there is a lot to do in OSC.py and OscAPI.py for them to work in py3 anyway. :rotating_light:


feat(localize): add support for Russian language

:tada: This PR is included in version 2.116.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

@eloyEntony Russian translation is now released ⚡


Cocktail

Icon Request

Icon name: cocktail
Tags of this icon: cocktail, drink, beverage, alcohol
Use case: Drink distribution, alcohol
Screenshots of similar icons:

Can those icons meet your needs?
cocktail, alcohol
https://remixicon.com/icon/goblet-fill
https://remixicon.com/icon/beer-fill
drink
https://remixicon.com/icon/drinks-2-line


command = message.get_command_for(bot) is empty

i am spliting command( command.split(" ") ) and some times i get this error and the script stops: block (2 levels) in <main>': undefined method split' for nil:NilClass (NoMethodError)
I am new at ruby and telegram bot...  There is a way to make the bot wait for a command and dont stop the script?
thank 4 the help :D

@rafalars Hi Rafael! Welcome to Ruby and Telegram bot!
A simple solution to your problem would be command.to_s.split(" ")
The line that causes the problem is: https://github.com/eljojo/telegram_bot/blob/master/lib/telegram_bot/message.rb#L21
I don't really know why sometimes the command is nil. Maybe there's a bug?
If you could show me more code in how you're using the gem then maybe I could help. But maybe there's just a bug in the code :)
Feel free to dive into the source code of the gem, it's somewhat simple :)

THANK YOU!!!!!!! Solved my problem. I am gonna record a video of my bot when its done and send to you. I implemented google web & image search options(for dork), and gonna put some more things!
Really man, thank you. Coding bot is **cking amazing!!!!!!!!!!!!!!!!

@rafalars I'm glad it solved the problem. I hope there was no bug in the system. This solution is really just a patch :)
Have fun!

@rafalars btw looking forward to the video!

hey @eljojo   the video: https://www.youtube.com/watch?v=w9uPiCzTNvo

@rafalars this is pretty neat, thanks for sharing!


Resize events can be really harmful

Hi there!
During the integration of your React component in a Next.js application, I noticed that the performance could be very degraded when resizing the window with the mouse.
For some context, I ran some tests on two pages in particular:

A product page, which displays a few images (around 20) in a very large format, some in full width.
The home page which displays a lot of images in a relatively small format (about 400x200).

When trying to resize the browser window on these two pages, I observe different behavior when inspecting the network:

On the product page, resizing causes a lot of new requests (about 3 per image), but most of them are cancelled. No performance issues encountered.
On the homepage, on the other hand, resizing causes at least as many requests per image as the product page, if not more (difficult to say precisely), except that the browser becomes so slow that no requests are cancelled.

Based on these observations, I would tend to think we could benefit of the use of a timer to debounce unnecessary requests.
I tried to dig a bit into the source code to see how this could be done, and I thought I understood that something should be done when using the resizeObserver API in src/_/placeholder.ts but I'm having trouble seeing how to do it.

Hi @Elindorath and thank you for this feedback.
The numerous requests observed here should not be due to the resizeObserver. The latter is only responsible for the "preview" image. It generates new requests only when the ratio or the positioning of the image has been modified during the resize, as it is the case in this example: TwicPics x Next - Art directions

When the ratio or the positioning mode of the image is not modified during the resize, the preview should never be reloaded.
On the other hand, the other requests (those responsible for the final image) are triggered when the difference (in absolute value) between the width of the display area of the image and its intrinsic width is greater than the step (10 px by default).
As a first step, I would propose to increase the value of the step. To increase it to 100 for example.

Here is a link  to a Next example with many images and a step set to 100.
Note that I observe performance degradations if I change the browser size with the mouse for too long as the network console fills up. Closing the console or clearing the log "fixes" the problem.

Keep us informed.
Best regards.

Hi @Elindorath.
For your information, changes concerning the resize have been made in version 0.11.0.
I take this release as an opportunity to ask you if you have noticed an improvement by increasing the step?
I close the issue but you can reopen it if you still have performance problems.
Keep us informed.
Best regards.


Add UWP support

Fixes #1879. I have not tested thoroughly since I really only need basic console logging functionality right now, but this got it to build and log successfully in a UWP project.

great!
UWP and unit tests from console, any idea of that could work?

Instead of introducing NETSTANDARD1_4 (Initial version to support uap10.0). Why not use NETSTANDARD1_5?
Any reason for using use Task.Delay to perform Thread.Sleep ? Not possible to depend on System.Threading.Thread? If necessary can it be put in a helper method?
Why not change ThreadLocalStorageHelper to use #elif NETSTANDARD instead of #elif  WINDOWS_UWP ? Would be nice to only have NETSTANDARD-versions defined in NLog, and not  WINDOWS_UWP.

Can understand that Thread-library is not available for UWP-apps until NetStandard2_0:
Maybe something like this could be used:
        internal static void StartAsyncTask(Action<object> waitCallback, object state)
        {
#if NETSTANDARD
            System.Threading.Tasks.Task.Factory.StartNew(waitCallback, state);
#else
            ThreadPool.QueueUserWorkItem(new WaitCallback(waitCallback), state);
#endif
        }

        internal static void WaitForDelay(TimeSpan delay)
        {
#if NETSTANDARD
            System.Threading.Tasks.Task.Delay(delay).Wait();
#else
            Thread.Sleep(delay);
#endif
        }

@304NotModified I'm not sure about the unit tests as I haven't tried. I'm new to UWP myself.
@snakefoot The NETSTANDARD1_4 flag is not used, so it can be removed. You are correct about Thread.Sleep. UWP does not support most of the old System.Threading namespace. Pretty much only Task.
My reason for using the WINDOWS_UWP flag was to minimize the changes for other platforms. You may very well be able to use NETSTANDARD instead. I agree the common logic should be moved to helper methods.
Unfortunately, I don't have the capacity right now to work on this any further. I only made the changes necessary to get the build passing so I could have basic functionality for my employer's new Windows app. I simply wanted to show support is possible with fairly minimal effort.

@gtbuchanan the netstandard1.3 version isn't working for you? It should support UWP. See https://docs.microsoft.com/en-us/dotnet/articles/standard/library

@304NotModified No. It contains libraries that are not supported in UWP (as mentioned here and #1879).

superseded by https://github.com/NLog/NLog/pull/2055
thanks @gtbuchanan !
Will release nlog5 beta07 soon (probably today)


xcode: add initial support for Xcode 11.0

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[ ] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.
[ ] Have you successfully run brew style with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?

(A pull request sent from McEnery Convention Center 😅)

Thanks @lembacon!


fix permissions for API user endpoint

coverage: 87.765% (+0.004%) from 87.761%
when pulling e16851eca42feacabf7f92b068da992681a54322 on api-userendpoint-permissions
into 4ba3190524be89f9e949a320d9549a9b893784b7 on main.


[Cookbook] fix a small typo

This Pull Request is fixed a small typo.
Fixed from Javascript to JavaScript.

thank you!


Speed up linting

Description
Only run linting on changed and affected examples.
Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)

Checklist:
If something is unclear, please submit the PR anyways and ask about what you thought was unclear.

[x] This branch is based on develop
[x] This PR is targeted to merge into develop (not master)
[x] I have added an entry under [next] in CHANGELOG.md
[x] The continuous integration (CI) is passing

So I suggest we have two files, one to locally use and fast (your version) but on CI I think we can run from scratch.

My concern is taking too long on the CI, locally the hardware is way better and runs way faster, so if we can't find a alternative that runs good on the CI, I think we should just keep one script.

So basically the logic is: if I change a file that is explicitly imported in the example right? (if I understood your code right).
I think that is a very, very good approximation but it's not perfect (I can create a proper example if you want but I think you agree it's just a heuristic). So I suggest we have two files, one to locally use and fast (your version) but on CI I think we can run from scratch.

It should be possible to get the other ones too, but I think this will very rarely not catch everything so I didn't bother to write it even more complicated. And this will take maybe 20s instead of 7minutes, if you only have 2 affected examples.

I don't see why taking a bit to run on the CI is that important, it's not like we are pumping PR's 10 a day. We manually review the PRs anyway, that is way longer than the CI would take.
I think perfect correctness is much more important than saving a few minutes, the reason the CI is there is to make sure we don't ever break anything, and it's doing that automatically for us.
What we could do to speed up w/o sacrificing correctness is to run then all in parallel in different subshells? that should be faster.

@spydon I don't think it's worth to make the "detect what changes impact" thing perfect, it's way, way too hard. In fact it's so hard it would probably take just as long as running everything (it requires not only syntactical but semantical analysis so it would require interpreting the dart code).
I'm ok with having a quick and easy shortcut for the 99% but we need to have an automated runner that checks the 100%...

I think perfect correctness is much more important than saving a few minutes, the reason the CI is there is to make sure we don't ever break anything, and it's doing that automatically for us.

Of course, I don't want to lose that, I want to have both :P (if possible)
I just raised the question we could have perfect CI check, but making it a little faster.

I'll try parallelizing it and running this check first and then the full check afterwards, then it will exit early almost always when there is an error at least.

I'm also good for having two scripts, I would def. use the fast one locally before pushing if I was pretty confident on my change. And then let the CI handle the expensive checks.

Flutter is using locking so there was no point in the parallelization, but the code is a bit cleaned up now at least and it will most of the time fail earlier if there is an error. :)


Problem with sessions via redis

PHP Version
8.2
CodeIgniter4 Version
4.4.3
CodeIgniter4 Installation Method
Composer (using codeigniter4/appstarter)
Which operating systems have you tested for this bug?
Windows, Linux
Which server did you use?
apache
Database
MySQL
What happened?
Some time i got
ERROR - 15:49:27 10-12-2023 --> Session: Error while trying to free lock for ci_session:ci_session:7aa8677c7de05ba9ad167233d5a2cddfee9e5e2f:lock
ERROR - 14:31:21 10-12-2023 --> Session: Unable to obtain lock for ci_session:ci_session:a489f0c94cb43cba56f6c0a97ef79359c3cf9a47 after 30 attempts, aborting.
ERROR - 14:31:26 10-12-2023 --> Session: Unable to obtain lock for ci_session:ci_session:a489f0c94cb43cba56f6c0a97ef79359c3cf9a47 after 30 attempts, aborting.
ERROR - 15:48:59 10-12-2023 --> Session: Error while trying to free lock for ci_session:ci_session:0f55e709dcae28ebc1b381dd1fe4f7cb07b222f6:lock
ERROR - 17:02:48 10-12-2023 --> Session: Error while trying to free lock for ci_session:ci_session:42d84fae09e8967219b133b14e34a3b292307ecb:lock
ERROR - 17:02:49 10-12-2023 --> Session: Error while trying to free lock for ci_session:ci_session:42d84fae09e8967219b133b14e34a3b292307ecb:lock
ERROR - 18:11:56 10-12-2023 --> Session: Error while trying to free lock for ci_session:ci_session:9e47dfb8f009afa6f7b3e387e3c1a6005134e9d4:lock
Steps to Reproduce
happens unexpectedly and randomly
Expected Output
happens unexpectedly and randomly
Anything else?
No response

Try #8323
But I did not test this yet. Be careful.

Duplicate of #4391
If there is any information, add to #4391


Project gets created with an invalid resourceQuota in the api

Rancher versions:
rancher/server or rancher/rancher: 2.0 master
Steps to Reproduce:

Create a cluster
Create a project specifying a json with invalid resourceQuota information in the resourceQuota field
Project gets created even if the resourceQuota is not valid.
There should be validation for the resourcequota field
Example below:

It's a framework issue as looks like we are not doing nested validation for types.


Introduce default sorting of tasks

Fixes #55
With the update, tasks will now be sorted in the following order:

By deadline
By module
By name

The completion status of the task does not affect the sort order as it'll make it easier when the archive by date command is introduced.

Codecov Report

Merging #70 (4b27b35) into master (96e502e) will increase coverage by 0.28%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master      #70      +/-   ##
============================================
+ Coverage     70.32%   70.60%   +0.28%
- Complexity      486      491       +5
============================================
  Files            88       88
  Lines          1432     1446      +14
  Branches        144      146       +2
============================================
+ Hits           1007     1021      +14
  Misses          384      384
  Partials         41       41

Impacted Files
Coverage Δ

src/main/java/seedu/address/model/task/Module.java
81.81% <100.00%> (+1.81%)
:arrow_up:

src/main/java/seedu/address/model/task/Name.java
81.81% <100.00%> (+1.81%)
:arrow_up:

src/main/java/seedu/address/model/task/Task.java
98.03% <100.00%> (+0.36%)
:arrow_up:

...ava/seedu/address/model/task/UniquePersonList.java
93.33% <100.00%> (+0.65%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


(Argusmonitor) checksums do not match and not in Gallery

https://chocolatey.org/packages/Argusmonitor
Expected Behavior

Downloads and Installs Argus monitor software.
Current Behavior

Crashes after download because checksums do not match and is missing in package gallery.
Possible Solution

Update checksum.
Steps to Reproduce (for bugs)

choco install argusmonitor

or

Check checksums by hand

Not sure how this tool got here but its invalid in total.
I vote for removal from this repository here if it doesn't get fixed by somebody in next few weeks.
While tool looks great, personally, I don't commit to exclusive commercial software/tools of any kind, but others might be interested.
/cc @AdmiringWorm

@majkinetor it got here because @p3lim opened a PR to add the package to this repo.
While we don't make commercial programs a priority, there isn't anything stopping it from being included in the repo as long as it matches the contribution guidelines.

While we don't make commercial programs a priority, there isn't anything stopping it from being included in the repo as long as it matches the contribution guidelines.

Sure, but the package doesn't work ATM at all. If nobody fixes it in realistic time frame it will only contribute chaos and additional maintenance (like this issue).

It is not relevant, since the package was rejected soon after submission.
@p3lim I will be removing the package from this repo in a few days, as there is no need for a package in this repository that isn't available on the community repo.


Validating Digital Twin updates against the Digital Twin model

My apologies in advance if this is not correct place to ask the question.
In Azure IoT Central and Azure Digital Twin service, if I create and upload model with the following property:
{
    "@type": [ "Property", "Temperature" ],
    "name": "targetTemperature",
    "displayName": "Target Temperature",
    "unit": "degreeCelsius",
    "writable": true,
    ---> "schema": "double"
}

And I try to update writable property with string/boolean - update fails.
[
  {
    "op": "replace",
    "path": "/targetTemperature",
    ---> "value":  "some value"
  }
]

Are there any specific libraries, SDKs or examples that I can use or reference to implement same kind of functionality in my application? That is, validate that updates conform with DTDL model.

Well, the value should be a double, instead of string/boolean
Can you clarify which APIs/SDKs are you using for Central and ADT?
Central: https://docs.microsoft.com/en-us/rest/api/iotcentral/1.1-previewdataplane/devices/update-properties
ADT: https://docs.microsoft.com/en-us/rest/api/digital-twins/dataplane/twins/digitaltwins_update
https://github.com/Azure/azure-iot-sdk-node/blob/main/service/samples/javascript/update_digital_twin.js

Well, the value should be a double, instead of string/boolean
Can you clarify which APIs/SDKs are you using for Central and ADT?
Central: https://docs.microsoft.com/en-us/rest/api/iotcentral/1.1-previewdataplane/devices/update-properties ADT: https://docs.microsoft.com/en-us/rest/api/digital-twins/dataplane/twins/digitaltwins_update https://github.com/Azure/azure-iot-sdk-node/blob/main/service/samples/javascript/update_digital_twin.js

I'm working on my own API and solution that is neither based on IOT Central or Digital Twins service, however I'm using both of them as a reference/learning tool. That's why I'm wondering how I could design my solution to mimic IOT Central/Digital Twins service and refuse updates that break the DTDL model.
To update DT or call commands I've used the following examples.
Say my personal API has PATCH endpoint to update DT that accepts PATCH body - how do I validate PATCH request against DTDL model? "To update property on device with XYZ DTDL model you need to provide X type of a value, but you have provided Y, therefore you get BAD REQUEST response."

That was basically it. I was researching Plug & Play Digital Twins and wondering if there's any public API/SDK that I can use to validate update payload against DTDL. Azure IOT Central and Digital Twins service both do it so I thought I might have missed documentation somewhere.
Thank you for clarification.

Thanks for the feedback,
I will keep this issue open until we publish some validation APIs.
Thanks,
Rido

Hi @EGitDeveloper
Let me correct myself:
We have published a preview parser that includes a validation api, the following code snippet shows how to validate a payload.
var objectModel = new ModelParser().Parse(new string[] { ReadFile("geotypes-1.json") });

var rootInterface = (DTInterfaceInfo)objectModel[new Dtmi("dtmi:samplesv2:geoTypes;1")];

var myPoint = (DTTelemetryInfo)rootInterface.Contents["myPoint"];
var validPayload = myPoint.Schema.ValidateInstance(ToJs(new { type = "Point", coordinates = new double[] { 12.35, 23.56} }));
Console.WriteLine(validPayload);

var myGeopoint = (DTTelemetryInfo)rootInterface.Contents["myGeopoint"];
var validPayload2 = myGeopoint.Schema.ValidateInstance(ToJs(new { lat =12.35, lon = 23.56 }));
Console.WriteLine(validPayload2);

{
    "@context": [
		"dtmi:dtdl:context;2",
		"dtmi:iotcentral:context;2"
	],
    "@id": "dtmi:samplesv2:geoTypes;1",
    "@type": "Interface",
    "contents": [
      {
        "@type": "Telemetry",
        "name": "myPoint",
        "schema": "point"
      },
      {
          "@type": ["Telemetry", "Location"],
          "name": "myGeopoint",
          "schema": "geopoint"
      }
    ]
  }

full gist is available here

Thank you for update! Looking forward to testing it out,

closing due to lack of response


Client Libraries Set Base Root Path

May I know is that possible to set base root to point to a specific path?
Example:-
/etc/clientlibs/folder-a/js.txt
/etc/clientlibs/folder-a/css.txt
/etc/design/libs/folder-b/file-one.css
/etc/design/libs/folder-b/file-two.css
In  /etc/clientlibs/folder-a/css.txt
#base=/etc/design/libs/folder-b
file-one.css
file-two.css

Not relevant to this project. Try the adobe forums.


Release 1.10.0

:sparkles: New Features

[x] Resolve #75.
[x] Resolve #171.
[x] Resolve #182.
[x] Resolve #183.
[x] Resolve #185.

:bug: Bug Fixes

[x] Fix #168.
[x] Fix #169.
[x] Fix #175.
[x] Fix #176.
[x] Fix #178.
[x] Fix #179.
[x] Fix #184.
[x] Fix #188.

:pencil: Documentation

[x] Close #180.
[x] Close #186.

:tada: This PR is included in version 1.10.0 :tada:
The release is available on:

npm package (@latest dist-tag)
Visual Studio Marketplace
GitHub release

Your semantic-release bot :package::rocket:


Only include poweredby images when flag is set.

set @includePoweredByImages: true in target site.variables in order to include these files.
Fixes https://github.com/Microsoft/pxt/issues/4016
Couldn't find any way to test in less if a file exists.

Looks like I accidentally checked in an identical commit. Closing.


Error when running :PlugUpgrade

Getting a strange error when I try to upgrade vim-plug.
[vim-plug] Error upgrading vim-plug: Cloning into '/var/folders/bt/lckltlc107b_ngnvtwlxzd8c0000gn/T/nvimh27tP3/3'...^@fatal: multiple updates for ref 'refs/remotes/origin/master' not allowed^@

I haven't made any significant changes in my environment. This is probably an error on my part, but I am not sure what would be causing it? I've installed neovim via homebrew, and followed the instructions to install vim-plug. The only different thing I'm doing is pointing vim-plug to my symlinked plugins directory (I use GNU Stow to manage my dotfiles).
...
call plug#begin('~/.config/nvim/plugged')
                        ^ this is a symlink, actual path is './dotfiles/nvim/.config/nvim/plugged'
...

Everything works fine except for upgrading vim-plug.  Any idea on whats going on here?

NVIM v0.3.4
Build type: Release
LuaJIT 2.0.5
Compilation: /usr/local/Homebrew/Library/Homebrew/shims/mac/super/clang -Wconversion -U_FORTIFY_SOU
RCE -D_FORTIFY_SOURCE=1 -DNDEBUG -DMIN_LOG_LEVEL=3 -Wall -Wextra -pedantic -Wno-unused-parameter -W
strict-prototypes -std=gnu99 -Wimplicit-fallthrough -Wvla -fstack-protector-strong -fdiagnostics-co
lor=auto -DINCLUDE_GENERATED_DECLARATIONS -D_GNU_SOURCE -DNVIM_MSGPACK_HAS_FLOAT32 -DNVIM_UNIBI_HAS
_VAR_FROM -I/tmp/neovim-20190113-94620-d8vv4n/neovim-0.3.4/build/config -I/tmp/neovim-20190113-9462
0-d8vv4n/neovim-0.3.4/src -I/usr/local/include -I/usr/local/opt/gettext/include -I/Applications/Xco
de.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.14.sdk/usr/include -I/t
mp/neovim-20190113-94620-d8vv4n/neovim-0.3.4/build/src/nvim/auto -I/tmp/neovim-20190113-94620-d8vv4
n/neovim-0.3.4/build/include
Compiled by<EMAIL_ADDRESS>
Features: +acl +iconv +jemalloc +tui
See ":help feature-compile"

   system vimrc file: "$VIM/sysinit.vim"
  fall-back for $VIM: "/usr/local/Cellar/neovim/0.3.4/share/nvim"

Type:

[ ] Bug
[ ] Enhancement
[ ] Feature Request
[x] Question

OS:

[ ] All/Other
[ ] Linux
[x] OS X
[ ] Windows

Vim:

[ ] Terminal Vim
[ ] GVim
[x] Neovim

Closing. Issue was being caused by a config param inside my .gitconfig. All good now!


Change keyboard shortcut for bold and italics to not conflict with default VSC shortcuts

Please consider changing the default keyboard shortcuts for bold and italics so that they don't conflict with the default VSC shortcuts of "Toggle Side Bar" and "Select Line" respectively. I know I can change them myself though, if necessary, too.
Perhaps Cmd+Shift+B instead?

The default keybindings are assigned to mimic the behavior in other editor like Word, Caret etc.
I would suggest you to change them according to your preference.

We can change them, but it's probably not great practice to override shortcuts that users are already accustomed to.


bug: Internal RuntimeWarning when adding a schedule

Meltano Version
2.19.1
Python Version
3.9
Bug scope
CLI (options, error messages, logging, etc.)
Operating System
N/A
Description
When adding a schedule, the below warning gets printed to console:
(meltano-prod) braedon@Braedons-MacBook-Pro:~/meltano-projects/getting-started$ meltano schedule add csv-to-jsonl --extractor tap-csv --loader target-jsonl --interval '@hourly'
/Users/braedon/.local/pipx/venvs/meltano/lib/python3.9/site-packages/meltano/core/settings_service.py:445: RuntimeWarning: Unknown setting 'start_date' - the default value `None` will be used
  value, metadata = self.get_with_metadata(*args, **kwargs)
Scheduled elt 'csv-to-jsonl' at @hourly

This RuntimeWarning seems unrelated to the user's input, so it probably shouldn't appear to the user (or if it's indicative of a bug on our end, then we might just need to fix whatever is causing the warning).
Code
No response

@BraedonLeonard this is a weird error. It must be coming from the extractor somehow.

This RuntimeWarning is raised every time a setting without a definition is accessed: https://github.com/meltano/meltano/blob/314f6686d48696bd3247291c5ebbcff1ece1be83/src/meltano/core/settings_service.py#L422-L430
In the particular case of meltano schedule add, it tries to be smart and get the default start_date for the schedule from the tap setting if it exists. I think the code above was meant to catch an exception raised when the setting was missing and simply log at debug level, but at some point the implementation changed and the call no longer raises that exception and instead just triggers that warning:
https://github.com/meltano/meltano/blob/314f6686d48696bd3247291c5ebbcff1ece1be83/src/meltano/core/settings_service.py#L328-L329
I think we should still try to get a value and consider two possible scenarios:

The tap declares a start_date setting, perhaps just by having a value for it in config inside meltano.yml without a formal definition. We get its value, possibly None. This currently doesn't log anything, but we should probably inform the user where the value comes from.
The tap doesn't declare the setting at all. We should inform the user more clearly that the tap is missing this setting, so we generate a new value.


Can not start app with version 0.1.12

My app is here:
https://github.com/AiursoftWeb/Kahla.App
Today I have just updated all dependencies to the latest version and my app just could run.
Every operation with pulltorefresh package returns an error as:
PullToRefresh.destroyAll is not a function
PullToRefresh.init is not a function

I guess the problem happens from the package itself.
My code is here
https://github.com/AiursoftWeb/Kahla.App/blob/master/src/app/Controllers/conversations.component.ts#L27

My apologies, I rebuilt the library not using an UMD format, I fixed and released v0.1.13 right now!

Thanks. Now my app works now!


Update perl and test sqitch

Description
Upgrade perl to 5.34.0. Perl is used in squtch so need to test the migration with sqitch.
Issues Resolved
[List any existing issues this PR resolves, or any Discourse or
StackOverflow discussions that are relevant]
Check List

[ ] New functionality includes tests
[ ] All buildkite tests pass
[ ] Full omnibus build and tests in buildkite pass
[ ] All commits have been signed-off for the Developer Certificate of Origin. See https://github.com/chef/chef/blob/main/CONTRIBUTING.md#developer-certification-of-origin-dco
[ ] PR title is a worthy inclusion in the CHANGELOG

Umbrella build - https://buildkite.com/chef/chef-umbrella-main-chef-server/builds/1180


Stop some function after some hour

Hi,
my bot work perfectly on Raspberry PI 2. I use it in my house to see some variable and it send me a message if I push some physical buttons. The problem is that sometimes after some hour of work it does not response to my command, but if I push a physical button (on piface digital 2), it send me a message.
The only way to restore all operation is to reboot the raspberry.
Below the code.
Have you some advice for me?
Thank you!
Sbot.pdf

Sorry I don't have time to analyze and test your code thoroughly.
My advice is this: Start from the simplest program possible, and increase complexity until the problem occurs. Then, you should have some clue where the problem lies.

OK, thank you!


Stop using import * from toplevel init

It breaks typechecking. Instead we will import every symbol from /api into the toplevel, extending our api_gen.sh script to do this automatically.
I also switched the toplevel init.py in the repo to be the actual source of truth for the file, with just the import section being auto-generated. Keeping a python file as a string in another python file is clunky.
Fixes #19779

@rchiodo feel free to take a look as well!

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

Project coverage is 57.14%. Comparing base (84b283e) to head (814c393).
Report is 8 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff             @@
##           master   #19864       +/-   ##
===========================================
- Coverage   78.83%   57.14%   -21.70%
===========================================
  Files         498      498
  Lines       45864    46253      +389
  Branches     8450     8506       +56
===========================================
- Hits        36159    26429     -9730
- Misses       7999    18186    +10187
+ Partials     1706     1638       -68

Flag
Coverage Δ

keras
57.14% <100.00%> (-21.56%)
:arrow_down:

keras-jax
?

keras-numpy
57.14% <100.00%> (+0.18%)
:arrow_up:

keras-tensorflow
?

keras-torch
?

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.

Thanks for asking me to take a look. It doesn't seem to be working for some reason in Pylance though. I believe I should be able to just copy this new __init__.py over my site-packages/keras/init.py? We're still not finding keras.models with this __init__.py

This might be our fault though. I'm debugging it now.

I don't think this is going to work. I can't find a way to skip a module in the typechecker.
According to this it sounds like it should work:
https://github.com/microsoft/pyright/blob/main/docs/typed-libraries.md#library-interface
But that seems to only be for symbols, not modules.
@erictraut would be the person to ask. He knows a lot more about this than I do.
Eric,
keras has a folder layout like so:
site-packages
  - keras
     - api
         - models

And keras would like a user to be able to do this:
# Skip having to import keras.api first
from keras.models import Sequential

But that doesn't work even if the __init__.py reexports the models package. Is this just not supported? Essentially skipping a package path?
It does work at runtime by changing the sys.path to find it, but we can't seem to model this behavior in Pyright/Pylance.

@mattdangerw  Question. What is the reason behind the 'api' folder? is just a development thing?  Couldn't you copy the folders to the expected location on packaging?
so this current folder structure
site-packages
  - keras
     - api
         - models

becomes what the user expects?
site-packages
  - keras
       - models

@bschnurr our goals are...

Explicit api surface, don't have everything private/public mixed together. We have an keras_export annotation that controls our public api.
"Browsable codebase." E.g jax has a nested directory of just init files, with the good stuff hidden away in a _src directory. We didn't find that very readable. We wanted to avoid a ghost town of empty directories with init files.
pip install -e works, pip install git+ssh://... works. We used to do a heavy rewrite of our source tree during packaging, but this led to a lot of confusion where people would run the non-packaged version we a slightly different API surface, as well as a more difficult time syncing with our google mono repo.

Is there a better way to achieve that? We had thought messing with the __path__ was not that uncommon, e.g. old school namespace packages, but maybe that was a faulty assumption.

Thanks Matt.
Currently we don't have support for editing 'path'
For controlling the public API see https://microsoft.github.io/pyright/#/typed-libraries?id=library-interface
there are two options in __init__.py
using the form from X import A as A
or adding symbols to __all__


Can not query data from Parachain

Hi I'm try to use this library with hydration parachain. Basically, I try to get the token free through Tokens pallet but I'm unable to get that information.
The code:
const client = createClient(getWsProvider('wss://hydradx-rpc.dwellir.com');
const api = client.getTypedApi(hdx);

 api.query.Tokens.Accounts.watchValue(address, token.assetId, 'best').subscribe((res) => {
      const balance = res.free - res.frozen;
      console.log({balance});
    });

Result logs:

Unable to reproduce.

Thank you for providing the reproduction. It appears that I may be utilising an incorrect version of the Polkadot SDK. I have found that using the withPolkadotSdkCompat is effective.


Missing error message when you declare $variableName without any type declaration (Throw console error)

Topic: https://graphql.org/learn/queries/#variables
This issue pertains to the following package(s):

[x] GraphQL Playground

What version of graphql-playground(-electron/-middleware) are you experiencing the issue(s) on?
1.7.32
What is the expected behavior?
Throw some error when the user query this (Better user experience).
type :string missing (invalid code)
query ( $day){
  sessions(day: $day){
   title
  }
}

Working code:
query ( $day: String){
  sessions(day: $day){
   title
  }
}

What is the actual behavior?
The play not working + I get a small "orange" warning related to 1 line (Without details).

Throw console error:

{message: "Syntax Error GraphQL request (1:12) Expected :, fo…$day){↵

Fix by adding type:

Schema:

I'm experiencing similar errors now that I'm learning GQL and I don't understand why the "Syntax Error GraphQL " like errors are not displayed in the result panel no the right when I press the play button or when I use that keyboard shortcut to run the query.
It would give me faster feedback on what is wrong and I'd not need to hover the mouse of the query to get the error or have the developer console open.


Inkscape plugin

I have created a plugin for Inkscape using your API ;)
Right now its part of a list of placeholder services but I can convert it into a single plugin.
https://github.com/Xaviju/inkscape-placeholder/tree/master/diverseui

Awesome work!


Feature request: Allow option to chose series title if provider has multiple ones

Hi, I've recently installed Komf and it's working pretty good for the most part.
I've chosen to use the AniList metadata provide, and I noticed that some series were getting their official English title instead of the Romaji one, even though AniList shows the Romaji one as the main series title.
I looked at the code and saw that the Romaji title was only used if there wasn't an English one. An option to choose the preferred title on the application.yml would be greatly appreciated.

Some providers don't differentiate between title types like mangaupdates.
It might be possible to add a config option flag to choose title type and only update title if provider has this title type

Some providers don't differentiate between title types like mangaupdates. It might be possible to add a config option flag to choose title type and only update title if provider has this title type

I thought about that problem. would it make sense to add providers specific settings?


update teardown step in test suite to properly handle temporary dir

if the entry in files is a directory, remove it recursively using rmSync (or rmdirSync in Node 12)

This will fix the sporadic test failures on Windows since it leverages the (well-tested) function provided by Node.js to remove a directory recursively.


Fix active cropper placement between displays

Fixes #131 for the most part

cropper displays in the same window the tray icon was clicked
works while tray menu's open
works after moving and resizing cropper
tested with binary

However, after moving cropper around in one window followed by re-opening it in another, the initial position becomes a little off, which probably has something to do with what's going on here
Also open to suggestions

Hey @nnnoel! I just tested this with my external display. It seems, if I open the cropper on my external display, it appears between both screens 😰

Interesting..
I tested this using a HP 27es and a HP 2311, and so far haven't been able to reproduce this. I wonder if having more vertical real estate influences the positioning behavior.
@timothyis Is the behavior intermittent?

@nnnoel It's continuous from the external display. If I open the cropper from the external display, I get this behavior. If I open it from my main display, it works fine.
Tested on a Dell U2715H 27"

If you were to throw a log between these two lines https://github.com/nnnoel/kap/blob/a2e499398e04e3aa554f5701c570d85ae19f8f26/app/src/main/index.js#L427-L428 -- do you get anything after opening the cropper between displays?

External HP 27es

After resizing and reopening (and switching focus from built-in to external again)

Switching back over to built-in again and reopening

:confused:

In case you missed it @nnnoel https://github.com/wulkano/kap/pull/461#issuecomment-390404815

@skllcrn I didn't. I can do what @karaggeorge is recommending for the other PR. The problem is I can't reproduce the issue, so even replacing the cropper/screen detection logic, I wouldn't be able to confirm without feedback from someone else.
@catalinmiron Sounds like your fix could make use of something along the lines of what I'm doing here https://github.com/wulkano/kap/pull/466/commits/b921fa361f6f68fe079b3a7eb1af610d5800e3f4 -- which aims to revert any volatile keyPath to it's initial value whenever needed.

If you're able to implement @karaggeorge's suggestions I can help test, maybe @catalinmiron would be able to help with that too!


useDetails + Details typings issue introduced in v27 and still present today

Describe the bug
The useDetails hook typings seem to have recently regressed. We're trying to move our
app up to the latest Primer React generation and hitting issues.
We're on React v17 and otherwise had a good experience working through the breaking changes.
Not enough of a TypeScript expert in terms of how they're generated from Primer to be too helpful
but willing to learn.
This regressed from v26 --> v27 with the big refactor to generate typings instead of being manually
maintained per https://github.com/primer/react/releases/tag/v27.0.0. Confirmed just now that this
basic app is OK in v26 and not in v27, and the same issue exists currently with 35.3.0.
To Reproduce
Steps to reproduce the behavior:

Create a new React app with TypeScript (if doing today, you want React 17 and not 18, since 18 isn't ready yet for Primer React)...
Install Primer React, verified using 35.3.0 but this was introduced in 27.0.0
Add any of the Details samples from the Primer React docs at https://primer.style/react/Details. In our case, I've added the 3rd example from the docs, the code of which is below
TypeScript error

ERROR in src/App.tsx:9:6
TS2769: No overload matches this call.
  Overload 1 of 2, '(props: { slot?: string | undefined; style?: CSSProperties | undefined; title?: string | undefined; ref?: ((instance: HTMLDetailsElement | null) => void) | RefObject<...> | null | undefined; ... 254 more ...; sx?: BetterSystemStyleObject | undefined; } & { ...; } & { ...; }): ReactElement<...>', gave the following error.
    Type '{ children: (string | Element)[]; onToggle: (e: SyntheticEvent<HTMLElement, Event>) => void; open: boolean | undefined; ref: RefObject<HTMLElement>; }' is not assignable to type '{ slot?: string | undefined; style?: CSSProperties | undefined; title?: string | undefined; ref?: ((instance: HTMLDetailsElement | null) => void) | RefObject<...> | null | undefined; ... 254 more ...; sx?: BetterSystemStyleObject | undefined; }'.
      Types of property 'ref' are incompatible.
        Type 'RefObject<HTMLElement>' is not assignable to type '((instance: HTMLDetailsElement | null) => void) | RefObject<HTMLDetailsElement> | null | undefined'.
          Type 'RefObject<HTMLElement>' is not assignable to type 'RefObject<HTMLDetailsElement>'.
            Property 'open' is missing in type 'HTMLElement' but required in type 'HTMLDetailsElement'.
  Overload 2 of 2, '(props: StyledComponentPropsWithAs<"details", any, SxProp, never, "details", "details">): ReactElement<StyledComponentPropsWithAs<"details", any, SxProp, never, "details", "details">, string | JSXElementConstructor<...>>', gave the following error.
    Type '{ children: (string | Element)[]; onToggle: (e: SyntheticEvent<HTMLElement, Event>) => void; open: boolean | undefined; ref: RefObject<HTMLElement>; }' is not assignable to type '{ slot?: string | undefined; style?: CSSProperties | undefined; title?: string | undefined; ref?: ((instance: HTMLDetailsElement | null) => void) | RefObject<...> | null | undefined; ... 254 more ...; sx?: BetterSystemStyleObject | undefined; }'.
      Types of property 'ref' are incompatible.
        Type 'RefObject<HTMLElement>' is not assignable to type '((instance: HTMLDetailsElement | null) => void) | RefObject<HTMLDetailsElement> | null | undefined'.
     7 |   const {getDetailsProps, setOpen} = useDetails({closeOnOutsideClick: true})
     8 |   return (
  >  9 |     <Details {...getDetailsProps()}>
       |      ^^^^^^^
    10 |       <Button as="summary">Delete item</Button>
    11 |       Are you sure?
    12 |       <Button variant="danger" onClick={() => setOpen(false)}>Yes I'm sure</Button>

Expected behavior
No TS error.
Code from the docs
Note, I've dropped the variant from the Button props so that this code will work on ancient Primer Components
as well as newer bits to confirm the regression.
const {getDetailsProps, setOpen} = useDetails({closeOnOutsideClick: true})
    return (
      <Details {...getDetailsProps()}>
        <Button as="summary">Delete item</Button>
        Are you sure?
        <Button onClick={() => setOpen(false)}>Yes I'm sure</Button>
      </Details>
    )

zip of the app sans node_modules if it helps

sample1.zip

p.s. our hack for now is any as always, <Details {...getDetailsProps() as any}>

Hi! Sorry for taking forever to respond.
I was able to download the zip file you gave and reproduce the type error. I have added it to the team's inbox to work on a fix

Hi @jeffwilcox, is this still an issue blocking you from upgrading to the latest release of Primer React?

@lesliecdubs we've moved to the latest versions by using the as any workaround mentioned above... I'm not potentially offended by that as much as some other devs.
Blocking - no
Potentially frustrating to TypeScript + React devs wanting to use useDetails - yes
Maybe a doc note would be a sufficient fix for now.
 <Details {...getDetailsProps() as any}>

Thanks for the insight!
@siddharthkp as FR, what do you think about adding the suggested note to the docs this week, and then we could move this issue to the backlog for consideration of further improvements in the future?


fix: nested imports

Hoist all but the previously ignored E402s
Fix incorrect namespace for RawTokenFormatter

The nested imports in cli.py were intentional, as part of some perf hacking I did a few years back. (A few of the others might be too.) Since cli.py is the entry point for the entire program, I could shave off a measurable chunk of startup time by only importing the necessary modules that were needed for that keyword.

The nested imports in cli.py were intentional, as part of some perf hacking I did a few years back. (A few of the others might be too.) Since cli.py is the entry point for the entire program, I could shave off a measurable chunk of startup time by only importing the necessary modules that were needed for that keyword.

No problem if you want to keep it. Do you want me to pull out the RawTokenFormatter to another PR, since that is actually an error

No problem if you want to keep it. Do you want me to pull out the RawTokenFormatter to another PR, since that is actually an error

Yeah, go ahead, and we'll keep the rest of these for now; I'm generally fine with importing at function scope when used judiciously.

Yeah, go ahead, and we'll keep the rest of these for now; I'm generally fine with importing at function scope when used judiciously.


Error in chapter 8 code

def tokenizer(text):
    return text.split()

from nltk.stem.porter import PorterStemmer
porter = PorterStemmer()
def tokenizer_porter(text):
    return [porter.stem(word) for word in text.split()]

from nltk.corpus import stopwords
stop = stopwords.words('english')

X_train = df.loc[:25000,'review'].values
y_train = df.loc[:25000,'sentiment'].values
X_test = df.loc[25000:,'review'].values
y_test = df.loc[25000:,'sentiment'].values
from distutils.version import LooseVersion as Version
from sklearn import __version__ as sklearn_version
from sklearn.pipeline import Pipeline
from sklearn.linear_model import LogisticRegression
from sklearn.feature_extraction.text import TfidfVectorizer
if Version(sklearn_version) < '0.18':
    from sklearn.grid_search import GridSearchCV
else:
    from sklearn.model_selection import GridSearchCV

tfidf = TfidfVectorizer(strip_accents=None,
                        lowercase=False,
                        preprocessor=None)

param_grid = [{'vect__ngram_range': [(1, 1)],
               'vect__stop_words': [stop, None],
               'vect__tokenizer': [tokenizer, tokenizer_porter],
               'clf__penalty': ['l1', 'l2'],
               'clf__C': [1.0, 10.0, 100.0]},
              {'vect__ngram_range': [(1, 1)],
               'vect__stop_words': [stop, None],
               'vect__tokenizer': [tokenizer, tokenizer_porter],
               'vect__use_idf':[False],
               'vect__norm':[None],
               'clf__penalty': ['l1', 'l2'],
               'clf__C': [1.0, 10.0, 100.0]},
              ]

lr_tfidf = Pipeline([('vect', tfidf),
                     ('clf', LogisticRegression(random_state=0))])

gs_lr_tfidf = GridSearchCV(lr_tfidf, param_grid,
                           scoring='accuracy',
                           cv=5,
                           verbose=1,
                           n_jobs=-1)
gs_lr_tfidf.fit(X_train,y_train)

Hi,
I get an error about "can't get attribute tokenizer_porter" ,
what's the problem you think?

Hi there,
I just tried to run this code on the movie data
after prepending a
import pandas as pd
df = pd.read_csv('./movie_data.csv')

and it works fine for me (sklearn 0.17 and sklearn 0.18).
So, is this the exact code you are running, and could you maybe share the exact error message, e.g., to see in which line in occurs?

I'm running into the same issue, I believe. I tried running the below code. But, I'm getting "can't get attribute 'tokenizer_porter' and 'tokenizer'.
Here's my version of python and sklearn respectively,
'3.5.2 |Anaconda 4.1.1 (32-bit)| (default, Jul  5 2016, 11:45:57) [MSC v.1900 32 bit (Intel)]
'0.17.1'
Here's a screenshot of one of the stack traces:

import pandas as pd
df = pd.read_csv('./movie_data.csv')
def tokenizer(text):
return text.split()
from nltk.stem.porter import PorterStemmer
porter = PorterStemmer()
def tokenizer_porter(text):
return [porter.stem(word) for word in text.split()]
from nltk.corpus import stopwords
stop = stopwords.words('english')
X_train = df.loc[:25000,'review'].values
y_train = df.loc[:25000,'sentiment'].values
X_test = df.loc[25000:,'review'].values
y_test = df.loc[25000:,'sentiment'].values
from distutils.version import LooseVersion as Version
from sklearn import version as sklearn_version
from sklearn.pipeline import Pipeline
from sklearn.linear_model import LogisticRegression
from sklearn.feature_extraction.text import TfidfVectorizer
if Version(sklearn_version) < '0.18':
from sklearn.grid_search import GridSearchCV
else:
from sklearn.model_selection import GridSearchCV
tfidf = TfidfVectorizer(strip_accents=None,
lowercase=False,
preprocessor=None)
param_grid = [{'vect__ngram_range': [(1, 1)],
'vect__stop_words': [stop, None],
'vect__tokenizer': [tokenizer, tokenizer_porter],
'clf__penalty': ['l1', 'l2'],
'clf__C': [1.0, 10.0, 100.0]},
{'vect__ngram_range': [(1, 1)],
'vect__stop_words': [stop, None],
'vect__tokenizer': [tokenizer, tokenizer_porter],
'vect__use_idf':[False],
'vect__norm':[None],
'clf__penalty': ['l1', 'l2'],
'clf__C': [1.0, 10.0, 100.0]},
]
lr_tfidf = Pipeline([('vect', tfidf),
('clf', LogisticRegression(random_state=0))])
gs_lr_tfidf = GridSearchCV(lr_tfidf, param_grid,
scoring='accuracy',
cv=5,
verbose=1,
n_jobs=-1)
gs_lr_tfidf.fit(X_train,y_train)

Thanks, I will take a look at it!

I just ran this example in scikit-learn 0.17.1 and don't have any issues (see screenshot). I am wondering if this has something to do with Windows & multiprocessing (heard it's a bit trickier on that platform). Unfortunately, I don't own a copy of Windows and only have Mac and Linux machines to test it on.

Could you try this again with n_jobs=1? I would also pick a smaller dataset so that it's faster, maybe df = pd.read_csv('./movie_data.csv', nrows=1000)

Could you try running this via sklearn 0.18? Maybe the bug was already fixed. If not, we should file it (or a simpler version) as a bug on scikit-learn Issue tracker

I've tried option 1 with 1000 rows and with the completed dataset and that seems to works (see screenshots below). I'll try running via sklearn 0.18 later today and see if the issue is reproducible.

Glad to hear that it works at least! When I see it correctly, changing n_jobs=-1 to n_jobs=1 made it work? Could you maybe try it on a different dataset/simplified setup. E.g., trying both n_jobs=-1ton_jobs=1`  in the following (works fine with both on my machine):
from sklearn.datasets import fetch_20newsgroups
from nltk.stem.porter import PorterStemmer
from distutils.version import LooseVersion as Version
from sklearn import __version__ as sklearn_version
from sklearn.pipeline import Pipeline
from sklearn.linear_model import LogisticRegression
from sklearn.feature_extraction.text import TfidfVectorizer
if Version(sklearn_version) < '0.18':
    from sklearn.grid_search import GridSearchCV
else:
    from sklearn.model_selection import GridSearchCV

porter = PorterStemmer()
def tokenizer_porter(text):
    return [porter.stem(word) for word in text.split()]

categories = ['alt.atheism', 'soc.religion.christian',
              'comp.graphics', 'sci.med']

twenty_train = fetch_20newsgroups(subset='train',
    categories=categories, shuffle=True, random_state=42)

X, y = twenty_train.data[:300], twenty_train.target[:300]

tfidf = TfidfVectorizer(strip_accents=None,
                        lowercase=False,
                        preprocessor=None)

lr_tfidf = Pipeline([('vect', tfidf),
                     ('clf', LogisticRegression(random_state=0))])

param_grid = {'vect__ngram_range': [(1, 1)],
               'vect__tokenizer': [tokenizer_porter],
               'clf__C': [1.0, 100.0]}

gs_lr_tfidf = GridSearchCV(lr_tfidf,
                           param_grid,
                           scoring='accuracy',
                           cv=3,
                           verbose=1,
                           n_jobs=1)

gs_lr_tfidf.fit(X, y)

I am closing this now in hope you got it to work okay! Otherwise, please let me know, and I'd be more than happy to reopen this issue to discuss the problem further.

Sorry to hear that there still seems to be an issue with recent Windows versions. Does it work okay if you set n_jobs=1?

Thanks for the response. I just installed Ubuntu in VirtualBox and it worked fine after that!
I have been running into issues with windows for while now so it was good. But one issue with using VirtualBox is graphics are very choppy so running cart-pole problem was not looking good.

I am glad to hear that everything's working fine on Linux now. I have never experimented with this since my machines are either Linux or macOS-only machines, but I think you can have both Linux and Windows installed on the same machine via a dual-boot mode or so.


how to stop auto instantiated editor

Hi KonnorR,
I'm interested in this editor, but I don't want it to be instantiated automatically. I want to initialize only one specific tag and set some extensions just like
new Editor({
  element: document.querySelector('.element'),
  extensions: [
    Document,
    Paragraph,
    Text,
  ],
  content: '
Example Text
',
  autofocus: true,
  editable: true,
  injectCSS: false,
})

Do you think rhino editor can do it? How to do it? THANKS @KonnorRogers

@DylanZella In the Setup Section on the docs you can see how to initialize without auto-instantiating.
https://rhino-editor.vercel.app/tutorials/setup/
Example:
import { TipTapEditor } from "rhino-editor/exports/elements/tip-tap-editor.js"

// https://tiptap.dev/api/extensions/character-count
import CharacterCount from "@tiptap/extension-character-count"
import { html } from "lit"

class ExtendedEditor extends TipTapEditor {
  extensions () {
    this.characterCountLimit = 240
    return [
      // Uses all existing extensions so we're only appending
      ...super.extensions(),

      // Adds character counter
      CharacterCount.configure({
        limit: this.characterCountLimit,
      })
    ]
  }

  render () {
    return html`
      ${super.render()}

        ${this.editor?.storage.characterCount.characters()}/${this.characterCountLimit} characters

        ${this.editor?.storage.characterCount.words()} words

    `
  }
}

ExtendedEditor.define("extended-rhino-editor")


Right way to style the badge?

The way I am now doing this now is:
Import the Badge module
import Badge from './node_modules/react-native-tab-navigator/Badge';
In the TabNavigator.Item I add it int the renderBadge function like this:
<TabNavigator.Item   renderBadge={() => <Badge style={styles.tabBarBadge}>1</Badge>} </TabNavigator.Item>
Would it be possible to expose the Badge in the TabNavigator? Like the TabNavigator.Item

I'm doing the same...

The default badge is something we don't want to expose as a public API. If you like the way it looks, I recommend you copy & paste the code.

Will close since this issue is more than a year, feel free to a open a new if this is still a issue.


Error while searching on application

I'm getting error while searching on application. As checked in browser, the search button hitting the wrong URL: https://jfk-search-service-5qzqyqg22owne.search.windows.net, correct URL is https://jfk-site-eypd7z6qyiico.azurewebsites.net/.
Try to find out where is went wrong but did not found this "5qzqyqg22owne" in whole application.
Please help me to fix this problem.
Complete url is: OPTIONS https://jfk-search-service-5qzqyqg22owne.search.windows.net/indexes/jfkindex/docs?api-version=2017-11-11-Preview&searchMode=all&facet=entities,count:20&$count=true&$top=10&scoringProfile=demoBooster&highlight=text 0 ()

It seems that you closed the issue so I assume you figured this out, but just in case anyone else encounters this problem, the solution is that if you delete and recreate the demo under a different resource group name, you will need to locally reset both the App.config file and the .env file that are a part of the project.  If you only reset the App.config file with the new credentials, the website credentials will not be correct, as the step to write to the .env file only works if that file is in the initial state from this repo.


Modified LPM to work with/without VIOS parameters being specified

Basic LPM will run even if user does not specify VIOS parameters (lpar_vios, remote_lpar_vios) in config file.
Logs:
test-log-with-vios.log
test-log-without-vios.log
Signed-off-by: rashijhawar<EMAIL_ADDRESS>

Logs for MSP:
msp-enabled.log
msp-disabled-(wrong-input-just-for-testing).log

Reference https://github.com/open-power/op-test/pull/664

All looks good, but one suggestions
It is good practice to raiseErrors in tests .. instead of utils so you should return true or false from MSP utils and check at test and raise

aha yes good suggestion

Thanks for working on all the suggestions and congrats for your first PR merged


Exclude endpoints feature

Hi,
What about separate setting option that will allow to specify API endpoints for which correlation id will be ignored and will not generate?
Use case:
In my API I use HTTP logging, and I want health check endpoints do not generate CorrelationId. For sure it can be filtered on logging level before sending logs to the datastore, but what about such a feature for this library?
It should not affect backward compatibility. If my idea makes sense - I can implement this one.

Another use case:
Turn on enforce header, health endpoints will fail.
Any idea for a workaround?


incident handling

Context
There are a lot of things that can go wrong which AME cannot or should not attempt to correct without human intervention. Therefore AME needs to expose such incidents to human operators and allow them to decide how to proceed. An example of this could be running out of space in the cluster storage while having a limit on how much storage AME is allowed to provision.
How to proceed
Acceptance criteria

[ ] Users can see and take action on a list of unresolved incidents in the CLI.

A good place to keep track of incidents could be in the status of Project objects.


Archetype 1

To verify Augraphy's utility, we need to reproduce the document in this notebook.
There are some notes about the effects present in the document, as well as some suggested augmentations you might need when reproducing it. These lists are not exhaustive; you may need to use more, or less, to achieve a faithful reproduction.
We can discuss the reproduction process for this document in this Issue. Discussion about this project in general should go here.

I tried to reproduce archetype 1 and currently this is the output:

Do you guys have any idea on how to reproduce the effect of "600" at the bottom right of the page?

Probably we need another augmentation to produce scribble based on user input text and location?

[Do] we need another augmentation to produce scribble based on user input text and location?

I think our primary goal for now is to create augmentations that are easily pipelined for automated data set creation.  So if we do allow user input data, it would still be somewhat random.  I like the idea of upgrading Scribble to accept text data as input (sourced from a file or generator function perhaps), but it seems likely that there are other improvements that might have a greater effect in the short term.  Definitely create GitHub issues for concepts like these that come from the archetype reproduction analysis.

I I like the idea of upgrading Scribble to accept text data as input (sourced from a file or generator function perhaps)

Alright, i will create another issue on this then we may discuss further from there. I think it is a good idea to accept text data in the form of text file too, so that we can randomly select those words for randomized scribble effect.

Here's the latest output from the notebook:
Reference image:

Augmented image:

Closed with PR #154


agebox cannot be used to encrypt content for yubikey-backed keys

When given a public key created with the age-plugin-yubikey plugin, agebox refuses to encrypt for it:
WARN[0000] Could not load public key: invalid public key  key=keys/charles.age svc=storage.fs.KeyRepository version=0.6.1

...this means that even someone willing to use non-agebox tools to decrypt their content (in the interim pending plugin support available in the decryption path for agebox itself) cannot encrypt their content with such a key.

There is a new release candidate https://github.com/FiloSottile/age/releases/tag/v1.1.0-rc.1 which adds plugin support for age-plugin-yubikey. so there should be support for this in the near future

just FYI https://pkg.go.dev/filippo.io/age stable v1.1.1 release supports yubikeys


Rename --all to --workspace

I have noticed that people occasionally get confused with the --all flag, usually confusing it with the --all-targets flag.  I'd like to propose renaming it to --workspace which should better convey what the flag is for.  We briefly discussed this in the team meeting and it seemed people were generally positive to the idea (and Josh came up with the best suggestion), so I'm opening this issue in case anyone watching this repo wants to provide feedback on the idea.
The --all flag would continue to work, but would be undocumented and deprecated and will probably issue a warning when used indicating the new flag name (maybe with a transition period with no warnings?).

I like the shortness of --all. --workspace seems so long to type.

What about adding a -wshorthand similar to how there is -p?

-w shortcuts are good.
how about @ehuss ?


Fixed a typo in the confirmation message for copying plugins

from different language

Coverage remained the same at 82.519% when pulling 6629c6ebc776a105ba2c49c98b75695cd4aeedc3 on vxsx:bugfix/confirmation-message into ac41c3defde7abb3242e516d84ca80f701871ded on divio:release/3.4.x.


Can we make Elm icon less ugly?

Current version of Elm icon is extremely ugly. I have no idea how to make it better, but rising this issue in case someone have.

Uh, can you post a screenshot? It looks fine to me. Not great, but that's the fault of the language's logo more than anything.

You can see that comparing to the other icons it's soo bad. Sure, it may be issue with logo itself, but maybe we can investigate a workaround?

There's nothing wrong with the icon whatsoever. It's representing the language's logo perfectly fine.
 Just put the following in your user stylesheet:
@import "packages/file-icons/styles/items";
@import "packages/file-icons/styles/icons";

@{pane-tab-selector}, .icon-file-text {
	&[data-name$=".elm"]:before {
		.code-icon;
		top: 0;
	}
}

Which will produce something like this:

In V2, the process for customising icons will be even easier, and there'll be no need for any @import rubbish. Just an FYI.
Since this is seriously subjective territory, I'm closing this as a non-issue.

@Alhadis I'd assume it is impossible to make it colorfull? The case is official logo isn't monochrome.

Yes, it's impossible to make it colourful. Icons are rendered as silhouettes, which means they can only be rendered using a single colour.


Incorrect code coverage results

The current code coverage reports are showing incorrect results due to the fact that the tests are bundled with tested code before parsing it with Instanbul.

Fixed with 2cedda0bc50f6c2bf3883acd6ea9c9bc6625f325.


Refs #30213 -- Removed post-startup check for Watchman availability.

This is checked at startup in get_reloader(). The runtime check ties
the implementation to Watchman excessively.
Hey @orf 👋 — can I ask for your thoughts here? I was looking at ticket-30213, and this double-check seems a bit excessive…

Is it likely Watchman is available at startup but then becomes unavailable? (get_reloader() already checked it was available... 🤔)
Is that worth privileging in relation to other reloader strategies?
Can we remove this check?

Sorry if this is re-treading old ground.
Thanks! 🎁

OK, thanks for checking @felixxm!


Fix to prevent OnButtonOpened to be called twice

To reproduce the issue :
Add an OnButtonOpened call back to the button and drag it until the end release you will see that the callback is being called twice, once when you reach 1.0 manually and the other one when the animator fling it to 1.0 again (even when it's already 1.0).
Source fo the issue :
The source of the error is that the button is being flung anyways when it reaches the confirmPercentage even when it's already at 1.0
The fix :
To fix the issue, added a check before flinging. So the only case when to fling is when the button has been dragged to less than 1.0 but reached the confirmationPercentage.

Thank you for your collaboration!


FAQ about importing PST files - this articel is unclear and ambiguous in respect of large messages and duplicate imports

[Enter feedback here]
"How does the PST import process handle duplicate email items?
The PST import process checks for duplicate items and doesn't copy the items from a PST file to the mailbox or archive if a matching item exists in the target folder in the target mailbox or target archive. If you reimport the same PST file and specify a different target folder (using the TargetRootFolder property in the PST import mapping file) than the one you specified in a previous import job, all items in the PST file will be reimported."
This is not true.  Duplicates do appear in several circumstances - exactly what circumstances are not disclosed. However, reimporting a failed PST results in duplicates.
"Is there a message size limit when importing PST files?
Yes. If a PST file contains a mailbox item that is larger than 150 MB, the item will be skipped during the import process."
This is ambiguous.  Is there anyway at all to import items 150MB or larger or is this just not possible at all?
Both these issues are causing quite a bit of pain with a migration. It would be great if we could get the answers.

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

ID: b7b7a25a-4cf8-1b51-0c40-6afa334ca537
Version Independent ID: 0033b96f-ee7b-0e9f-5287-4a5371ce3734
Content: FAQ about importing PST files - Microsoft 365 Compliance
Content Source: microsoft-365/compliance/faqimporting-pst-files-to-office-365.md
Service: o365-seccomp
GitHub Login: @markjjo
Microsoft Alias: markjjo

Hello and thanks for your response, very much appreciated.
The documents you referred to are the ones I pasted the extracts from and don't appear to answer the question I posed. I wonder is there a PowerShell script that can remove duplicate messages etc. in a mailbox where a PST import has created duplicates?
How about importing messages bigger than 150MB?  How can we import those to the mailbox for compliance purposes?

As above

@osirisx3 Thanks for feedback. Firstly, 150MB is the message size limit for Exchange Online (see https://docs.microsoft.com/en-us/office365/servicedescriptions/exchange-online-service-description/exchange-online-limits#message-limits). That is why it is the stated limit for PST Import. I think adding this bit (max message size) to the FAQ would help address the ambiguity. We say item is skipped but not why. It's skipped because it exceeds the max message size.
As for duplicates, I would suggest you contact MS Support so that they can look into the "several circumstances" where duplicates are appearing in your situation and help you with your migration.
For a little more background, you might take a look at the New-MailboxImportRequest cmdlet help topic. This cmdlet is called/runs in the background when you create and start a PST Import job in the Microsoft 365 compliance center. Taking a quick look at the ConflictResolutionOption parameter, I would assume that if our FAQ is correct, this parameter is configured with the KeepTargetItem value.

Thanks for the excellent and very helpful feedback, Mark.
At least now we know that 150MB is the largest that can be stored, we can look at alternative storage methods for compliance reasons. Just the addition of that one piece of information to the documentation would have saved hours of searching and directed us along alternate paths!
I did already check out the "New-MailboxImportRequest cmdlet help topic" - thanks for the suggestion - however it appears that it is not supported in Exchange online - and I had advice to that effect separately also.  I am assuming - rightly or wrongly - that this advice applies to MS365 as well.
Note: This cmdlet is no longer supported in Exchange Online. To import a .pst file in Exchange Online, see Use network upload to import PST files.
Are you suggesting this command does in fact still do the job in MS365 or am I missing something?
We really appreciate the time you have taken for your reply, and it's been a huge help confirming the steps I have already taken to get the answers - that is worth gold! Thanks!

@osirisx3 Sorry, I missed that bit about the cmdlet not being supported in EXO. Yes, that would still apply in Microsoft 365. We want you to use the PST import feature in the compliance center UI to import PST files.
If you do contact support, and the documentation about duplicates needs to be updated based on your case, support can contact me to request a document update. Thanks!

No worries, mate, your responses were most helpful in steering me in the right direction. Thanks and cheers  Dave


chore: bump version of operator to 1.1.1

/override Build-fbc

LGTM but I would keep the final merge to some Konflux expert


PR: Display external plugins in the dependencies dialog

Description of Changes

[ ] Wrote at least one-line docstrings (for any new functions)
[ ] Added unit test(s) covering the changes (if testable)

[ ] Included a screenshot or animation (if affecting the UI, see Licecap)

Issue(s) Resolved

Fixes #
Affirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct: @goanpeca

Hello @goanpeca! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file spyder/app/solver.py:

Line 114:80: E501 line too long (81 > 79 characters)

Test failures are unrelated to the work of the PR.
E: Failed to fetch http://azure.archive.ubuntu.com/ubuntu/pool/main/m/mesa/libegl-mesa0_19.2.8-0ubuntu0~18.04.3_amd64.deb  404  Not Found [IP: <IP_ADDRESS> 80]
E: Unable to fetch some archives, maybe run apt-get update or try with --fix-missing?
##[error]Process completed with exit code 100.

Merging


Update ethers to v6

Related to #201 if merged -> https://github.com/ethers-io/ethers.js/pull/3803

@jzaki it's now https://github.com/ethers-io/ethers.js/pull/3904


WIP: Improvements for use by dotnet-gc-infra

GetCurrentGC used to only return the current GC between the GC/Start and GC/Stop events. But some events would happen slightly before and after those.
It will now return the current GC in between GC/SuspendEEStart and GC/RestartEEEnd (+1ms due to some events coming out slightly later).

So we are now creating the GC in source.Clr.GCSuspendEEStart +=. We are still filling in most of it in GCStart though.

Commented out Debug.Assert(_event.PauseDurationMSec == 0); that was firing.

New public properties:

Adds ExpandMechanisms and CompactMechanisms to GCPerHeapHistory
Adds ThreadId and IsBGCThread to GcJoin

Codecov Report

Merging #969 into master will decrease coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #969      +/-   ##
==========================================
- Coverage   18.64%   18.63%   -0.01%
==========================================
  Files         234      234
  Lines      146664   146704      +40
  Branches    12633    12637       +4
==========================================
  Hits        27343    27343
- Misses     118350   118390      +40
  Partials      971      971

Flag
Coverage Δ

#2017
18.63% <0%> (-0.01%)
:arrow_down:

#Debug
18.63% <0%> (-0.01%)
:arrow_down:

#Release
100% <ø> (ø)
:arrow_up:

Impacted Files
Coverage Δ

src/TraceEvent/Computers/TraceManagedProcess.cs
0% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

Closing as @andy-ms is no longer at MSFT.

Closing as @andy-ms is no longer at MSFT.


chore: artwork header buttons improvements

This PR resolves []
Changes

the design of the buttons is consistent (the same state when pressed)
removed the right offset for the "create alert" button relative to the page content

Before
After

Demo 1
Before
https://user-images.githubusercontent.com/3513494/206191899-c9a32365-1ac8-4fba-b9c8-fb1edcba437a.mp4
After
https://user-images.githubusercontent.com/3513494/206191984-f43a9c1c-5cd5-4ebe-84ba-8a2bf9a283f4.mp4
Demo 2
Before
https://user-images.githubusercontent.com/3513494/206192047-6ac5c9f7-45fb-4d15-9829-733c63bb3093.mp4
After
https://user-images.githubusercontent.com/3513494/206192100-55624a79-bc42-4df6-aa78-2e88ae263bb1.mp4
Demo 3
Before
https://user-images.githubusercontent.com/3513494/206192190-dda28faa-5529-4eb0-9fe7-29816d983501.mp4
After
https://user-images.githubusercontent.com/3513494/206192247-53e138df-d567-4163-a6bf-eb7cef5a6edf.mp4
PR Checklist

[x] I tested my changes on iOS / Android.
[x] I added screenshots or videos to illustrate my changes.
[ ] I added Tests and Stories for my changes.
[ ] I added an app state migration.
[x] I hid my changes behind a feature flag.
[ ] I have prefixed changes that need to be tested during a release QA with [NEEDS EXTERNAL QA] on the changelog.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

Dev changes

artwork header buttons improvements - dimatretyak

Need help with something? Have a look at our docs, or get in touch with us.

after a quick chat with dima, we will wait for some answers from xenia before moving on with this PR 👍

awesome. so we close this? or keep just the bits we like, like the variant for longest etc?


Uniform colors for website

Currently images have purple main color, buttons have different colors, and there is no consistency. Map has red pins.
Need to make sure that:

action buttons are of accent color (purple)
other buttons have secondary color

Finalised, missing links - will be done in future issue.


could it be used in arm linux? e.g. respberry pi.

could it be used in arm linux? e.g. respberry pi.

I think that's possible. You are welcome taking a try.

Hi, I would like to try the Linux-ARM version.
Is it planned that Linux arm64 will be included in a future release?

Linux arm64 will be supported from Javet v3.
However Linux arm is another story as:

I don't have the hardware.
I expect someone to sponsor that work.


token concatenation is buggy

The following test fails:
`define SIZE 42
`define D(a) `SIZE``a
`D('d0)

It fails during macro expansion because SIZE'd0 is used as the full macro target, and not just SIZE. The reason is the logic that deals with the following (i.e. concatenation which leads to a recursive macro call):
`define FOO(TYPE,ARG,FLAG) `M_UVM_``TYPE(ARG,FLAG)
`define M_UVM_ARRAY(arg1, arg2) yo
`FOO(ARRAY, byte_valid, UVM_NOPACK)

So the former currently fails and the latter works. Obviously SIZE'd0 cannot be a legal macro name, but that's irrelevant here. When the macro text in D is scanned we can't know whether it will expand to a new macro call SIZE_rest_of_macro_name or 42'd0.
The LRM doesn't appear to be very clear on the rules here and there is not much available example code except for the latter example above (from the uvm reference implementation).

Let's say you have this case:
`define SIZE 1
`define SIZE23 hello
`define D(a) `SIZE``a
`D(23)

It's not clear to me whether the result should be 123 or hello.


Some extra logging, restoring a recover

Current coverage is 59.23% (diff: 33.33%)

Merging #195 into master will decrease coverage by 0.02%

@@             master       #195   diff @@
==========================================
  Files           119        119
  Lines          6271       6272     +1
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits           3716       3715     -1
- Misses         2241       2243     +2
  Partials        314        314

Powered by Codecov. Last update ab210e7...d7d46be


Missing argument in .sub() method introduced after the latest changes

in  src/util/pointProjector.js   (row 8)
const point = patientPoint.clone().sub();
should be
const point = patientPoint.clone().sub(imagePlane.imagePositionPatient);

I have no idea what happened here. I had this line in the file, I even tested the reference line example. I guess I must have hit undo or something.
Sorry!


elmPackages: update node-packages.nix

Description of changes
Generate a new node-packages.nix. I was explicitly trying to update elm-language-server but I couldn't find a clean way to just update the one package.

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 279022 run on x86_64-linux 1

1 package failed to build:

elmPackages.elm-land

11 packages built:

elmPackages.create-elm-app
elmPackages.elm-coverage
elmPackages.elm-doc-preview
elmPackages.elm-git-install
elmPackages.elm-language-server
elmPackages.elm-live
elmPackages.elm-pages
elmPackages.elm-review
elmPackages.elm-spa
elmPackages.elm-upgrade
elmPackages.elm-verify-examples

hrm.. not sure about elm-land..

thanks for the PR but first we will probably need to address the regression in elm compiler itself which is WIP in https://github.com/NixOS/nixpkgs/pull/277326

@GrahamcOfBorg build elmPackages

It looks like there might be regression in elm-land:
warning: Ignoring setting 'auto-allocate-uids' because experimental feature 'auto-allocate-uids' is not enabled
warning: Ignoring setting 'impure-env' because experimental feature 'configurable-impure-env' is not enabled
error: builder for '/nix/store/541a1x8svs27xsa81r72zc86j8axnzj7-elm-land-0.19.5.drv' failed with exit code 1;
       last 10 log lines:
       > npm ERR!
       > npm ERR!
       > npm ERR! NOTE: You can avoid npm entirely by downloading directly from:
       > npm ERR! https://github.com/elm/compiler/releases/tag/0.19.1
       > npm ERR! All this package does is distribute a file from there.
       > npm ERR!
       > npm ERR! --------------------------------------------------------------------------------
       >
       > npm ERR! A complete log of this run can be found in: /build/.npm/_logs/2024-01-06T20_10_09_494Z-debug-0.log
       >
       For full logs, run 'nix log /nix/store/541a1x8svs27xsa81r72zc86j8axnzj7-elm-land-0.19.5.drv'.
error: 1 dependencies of derivation '/nix/store/l1ldnr4rvjij8wq6d0g0pwp9i95im9rh-review-shell.drv' failed to build

Link to currently reviewing PR:
https://github.com/NixOS/nixpkgs/pull/279022

1 package failed to build:
elmPackages.elm-land

Result of nixpkgs-review pr 279022 run on x86_64-linux 1

12 packages built:

elmPackages.create-elm-app
elmPackages.elm-coverage
elmPackages.elm-doc-preview
elmPackages.elm-git-install
elmPackages.elm-land
elmPackages.elm-language-server
elmPackages.elm-live
elmPackages.elm-pages
elmPackages.elm-review
elmPackages.elm-spa
elmPackages.elm-upgrade
elmPackages.elm-verify-examples

@turboMaCk thanks! that did it!


Question about the camera parameter transformation when resize the front view.

Hi! Thank you for your contribution and I read your code carefully. My question is why lidar2cam_rt was transposed. (See in below link.)
https://github.com/OpenDriveLab/OpenLane-V2/blob/ddfcc55b871b490288815daf479e4224004148bf/plugin/mmdet3d/baseline/datasets/pipelines/transforms.py#L78
As far as I know, the resize operation of the image will only change its intrinsic, and its extrinsic should be kept the same.

It seems that it just follows the practise here in the dataset.

Well, I see. lidar2cam_rt is actually initialized in a transposed pattern. 🤣


Fixed quitting with ctrl+c

Hey I'm using your code for an AI project. I thought I'd let you know that I fixed your ctrl+c issue when killing threads.

@kaiergin I don't see how making the threads daemonic solves the ctrl+c issue.


Array changes are not recognized by useSettings

When creating the native obj I noticed that when I change/add my config the save buttons are not unlocked.
My native obj consists of one obj and otherwise just array with obj that are changed or deleted/added here is an example:
"native": {
	"tablets": [
	  {
		"id": 0,
		"config": {
		  "active": false,
		  "interval": 30,
		  "name": "Test1",
		  "Login": {
			"ip": "",
			"port": 2323,
			"password": ""
		  },
		  "charger": {
			"active": false,
			"chargerId": "",
			"powerMode": "false",
			"loadStart": 20,
			"loadStop": 85
		  }
		}
	  },
            {
		"id": 1,
		"config": {
		  "active": false,
		  "interval": 30,
		  "name": "Test2",
		  "Login": {
			"ip": "",
			"port": 2323,
			"password": ""
		  },
		  "charger": {
			"active": false,
			"chargerId": "",
			"powerMode": "false",
			"loadStart": 20,
			"loadStop": 85
		  }
		}
	  }
	]
  },

Can you show how exactly you change the settings? It seems that the settings object instance doesn't change, so this doesn't trigger the comparison.

Apologies for the delayed response.
I don't have the code anymore, as I built it differently. But basically, it was so, I built the object together and then passed it to the handleChange function in the index.tsx, with the option and value as in the example:
const SettingsPageContent: React.FC = React.memo(() => {
	// eslint-disable-next-line @typescript-eslint/no-unused-vars
	const { settings, originalSettings, setSettings } = useSettings<ioBroker.AdapterConfig>();

	// Updates the settings when the checkbox changes. The changes are not saved yet.
	const handleChange = <T extends keyof ioBroker.AdapterConfig>(option: T, value: ioBroker.AdapterConfig[T]) => {
		setSettings((s) => ({
			...s,
			[option]: value,
		}));
	};

	return (
		<SettingPage settings={settings} changeSetting={(option, value) => handleChange(option, value)} />
	);
});

I have created an example where the changes of a value in the array does not activate the save buttons
Issue branch ⇨ https://github.com/xXBJXx/ioBroker.template_react/tree/issue


Make DispatchArgs::suppressErrors required

Motivation

Errors were not always being suppressed properly, because certain
locations forgot to propagate the old value when constructing a new
DispatchArgs struct.
This meant that we were actually doing substantially more work than we
meant to.
This might also fix a source of non-deterministic behavior in Sorbet on
Stripe's codebase (not yet tested) but the performance benefits (not yet
measured) should be justification enough to land this.
Test plan

Existing tests.

We have a policy of testing changes to Sorbet against Stripe's codebase before
merging them. I've kicked off a test run for the current PR.  When the build
finishes, I'll share with you whether or how it failed. Thanks!
Stripe employees can see the build results here:
→ https://go/builds/bui_KEYCPANlNQ2UqV
→ https://go/builds/bui_KEYDHpbH4D3GEj

old times
new times

20.838s
19.605s

20.910s
19.836s

21.012s
20.112s

22.605s
20.262s

avg
21.341s
19.954s

stat
delta

Δ min
−1.233s

Δ max
−2.343s

Δ avg
−1.388s

This appears to have fixed the bug (80 type check runs on Stripe's codebase and no errors). Prior experience showed that the bug would have manifested 2 times in that many runs.
I'm not sure of a way to get this to reproduce outside of Stripe's codebase. Unfortunately, I'm going to have to go ahead and land this without a test, so we won't have a good way to prevent this from recurring.
It's honestly rather surprising to me that this behavior could be racy. Maybe we're like... flushing the error queue eagerly when we shouldn't be? @jvilk-stripe do you know why that might be? When the bug manifested, it manifested in Sorbet's batch mode, so I can't imagine that LSP eager error-queue flushing shenanigans would have been at play, but figured I'd ask anyways.


Admin privileges in java gateway

Adds a note about the admin privileges in the Java gateway to the 'What's new for developers' section.

Staged at https://ci.openmicroscopy.org/job/OMERO-DEV-merge-docs/ws/src/omero/_build/html/developers/whatsnew.html.

Maybe break line in source and end with period?


RA is no longer active in Travis CI

@mshibuya can you check this please

@sferik Could you please re-enable the build? It requires admin access to the repo.

Fixed. Not sure what happened to break it.

:satisfied:


UI Step 3 after having chosen the option “Define functional groups for me” - Issue no. 2

One cannot type the name of a species (Latin name, vernacular name, family name or order name) currently.

User can now type the name of species. After entering species name, the wizard will try to populate its size category, habitat and depth properties.
It is not completely implemented. There is still no validation on the entered species name.

Currently, only Latin names are accepted. The user should be able to enter a vernacular name, a family name or an order name.

@FIN-casey @FIN-JBarile
ERRATUM: Currently, only Latin names are accepted. The user should also be able to enter an order name if they wish so.
The rationale here is that ecosystem modelers do not know the Latin names of the species making up low-trophic level functional groups (e.g., phytoplankton, bivalves). For example, if a user wishes to create a functional group called "Echinoderms and large gastropods", they will not know the Latin names of the species making up that functional group, but rather the names of the orders making up that functional group: Asteorida, Echinoida, Gastropoda, and Holothuorida.

During our last Skype meeting, we found a solution to allow the user to provide order names or Latin names, as they see fit. This solution will be described in another issue. Therefore, the present issue can be closed.


Feature/lol/error messages

Connects #176
Adds actual error messages to Layers and Images when parts of the processing pipeline fail. Shows these error messages in the processing modal.

To test

Start the worker (see below)
Upload one each of the following types of images:

Junk data with a tif extension.
Ok tif but with only one band (http://landsat-pds.s3.amazonaws.com/L8/139/045/LC81390452014295LGN00/LC81390452014295LGN00_B1.TIF)
Ok three band tif.

Watch for error messages.
Since we have no processor, after 10 minutes, the job using the good image should timeout with an error message as well.

Starting the worker
vagrant ssh worker
cd /opt/worker
envdir /etc/rf.d/env ./manage.py shell
from apps.workers import process
p = process.QueueProcessor()
p.start_polling()

@azavea-bot rebuild

@azavea-bot rebuild

+1, tested

@azavea-bot rebuild

@azavea-bot rebuild

@azavea-bot rebuild


Task/update for newskill

Cleanup of the old SnipsManager references.
Fixed tests.
Print and speak an error message in case of missing API key.
Added the config.ini in .gitignore and move it with the setup.sh that is run by Sam.
Updated the readme.
Removed travis.
Removed python-Levenshtein dependencie that wasn't building on Pi. Replaced it with a single class implementation.