Easiest way to run with just a runtime

It seems like the 'fastest' way to run, given just a .wasm file is still to:

populate a config.yaml file
pass a --genesis chain_spec.json (which only works in --raw format) which is then used to fetch the code.

Two comments on this:

can we simplify this further, such that you can directly put the runtime-path in config.yarml, or your CLI, and that wasm blob will be used?
--genesis is somewhat unclear to me. Is it a chain-spec file? and if it defines storage, which one takes precedence over the storage overrides defined in config.yaml?

The thing is a wasm is only the runtime code, it doesn't include other necessary states to operate the chain. There is a new set of runtime API to populate genesis state so we maybe able to use it but it is not yet used as the runtime API wasn't super stable last time I checked.
--genesis is expecting the chain-spec file, that includes genesis storage and other things like chain properties. And the storage override in config.yml can further overrides the fields defined in the chain spec file.
I think one improvement we can do is that to have a special mode that generate the chain spec file using the runtime API if available to avoid require the user to do it with chain spec builder. But that is basically embedding the chain spec builder in Chopsticks, which is fine, but kinda one more duplicated work.

--genesis is expecting the chain-spec file, that includes genesis storage and other things like chain properties. And the storage override in config.yml can further overrides the fields defined in the chain spec file.

Got it. Then excuse my nitpicking, but I would have called this --chain or --chain-spec 😁

I think one improvement we can do is that to have a special mode that generate the chain spec file using the runtime API if available to avoid require the user to do it with chain spec builder. But that is basically embedding the chain spec builder in Chopsticks, which is fine, but kinda one more duplicated work.

I agree. I think for Chopsticks using --chain-spec is more or less fine. I am working on https://github.com/paritytech/polkadot-sdk/pull/5117, and in there we can and will provide the option to generate the chain-spec on the fly from --runtime and --preset.

FYI, this is my vision for omni-node-driven development. I am completing the steps needed for this + will do some workshops and tutorials on it. Lmk if you have further ideas.
Omni Tutorial

Grab parachain template, remove node.
add a custom pallet to it for the sake of fun

Run chain-spec-builder, generate spec file
Run frame-omni-bencher, generate weights

Local Testing 'Fake':

Run it with the dev-omni-node
Run it with chopsticks

Local Testing 'real':

Zombienet + polkadot-parachain + polkadot binaries.

Remote Testing

acquire on-demand coretime on a testnet, run it with polkadot-parachain

#797 will support --chain-spec. anything else needed?

Thanks for the quick fix

I am looking for an easy way to start the parachain-template-runtime. Trying to update the docs with suggestions for runtime development. Couldn't find useful link to use chopsticks based on a chain spec. @xlc do you know of any link or some example I can direct people to?
For what is worth, I tried running something like below (where chain_spec.json is raw), but I got an error:
 npx @acala-network/chopsticks --chain-spec chain_spec.json

Error:
Error: Unresolved function `env`:`ext_benchmarking_add_to_whitelist_version_1`
    at Block.call (/home/iulian/.npm/_npx/81ad9c881cb83600/node_modules/@acala-network/chopsticks/node_modules/@acala-network/chopsticks-core/dist/cjs/blockchain/block.js:255:35)
    at async Promise.all (index 0)
    at async buildBlock (/home/iulian/.npm/_npx/81ad9c881cb83600/node_modules/@acala-network/chopsticks/node_modules/@acala-network/chopsticks-core/dist/cjs/blockchain/block-builder.js:225:22)
    at async TxPool.buildBlock (/home/iulian/.npm/_npx/81ad9c881cb83600/node_modules/@acala-network/chopsticks/node_modules/@acala-network/chopsticks-core/dist/cjs/blockchain/txpool.js:344:43)
    at async TxPool.buildBlockIfNeeded (/home/iulian/.npm/_npx/81ad9c881cb83600/node_modules/@acala-network/chopsticks/node_modules/@acala-network/chopsticks-core/dist/cjs/blockchain/txpool.js:328:9)

the error is the runtime is built with benchmarking feature, which expects some unimplemented host functions (ext_benchmarking_add_to_whitelist_version_1). you can either built the chain spec without benchmarking feature, or add --allow-unresolved-imports=true to make chopsticks ignore such error

Thanks @xlc ! This unblocked me.


Open VSCode starts with graphic errors, the color green

Issue Type: Bug
Start with graphic errors, the color green, i don't know,
I restarted it, uninstalled, and it still happened again.
It is presented as green squares on the screen, and also as errors in the letters,
VS Code version: Code 1.38.1 (b37e54c98e1a74ba89e03073e5a3761284e3ffb0, 2019-09-11T13:35:15.005Z)
OS version: Windows_NT x64 10.0.18362

System Info

Item
Value

CPUs
AMD Ryzen 5 2600 Six-Core Processor             (12 x 3400)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwareoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_deferred_display_list: disabled_offskia_renderer: disabled_offsurface_synchronization: enabled_onvideo_decode: enabledviz_display_compositor: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.95GB (11.30GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions: none

Start with graphic errors, the color green, i don't know,
I restarted it, uninstalled, and it still happened again.
It is presented as green squares on the screen, and also as errors in the letters,

Try to launch vscode from cmd code --disable-gpu

OMG! Ty for help, Worked!
Now, how do I do for hold without having to access the cmd?, is there any way?
Thxns!

@luischicllarosas no problem.
Unfortunately, this is not possible yet, there is a issue about this

Would like to close this as https://github.com/microsoft/vscode/issues/15211 will help with the hardware acceleration workaround.


Incorrect handling of BETWEEN predicate, EXISTS, NOT EXISTS, IN, NOT IN subqueries

This test is from NIST test suite. RunNistTests needs to be run but asserts are suppressed.
NIST Test dml014.
SELECT PNUM
FROM PROJ
WHERE BUDGET >= 40000 AND BUDGET <= 60000;
Should return only one row, with 'Vienna' but returns four: Deale, Vienna, Deale, Akorn
SELECT CITY
FROM STAFF
WHERE NOT(GRADE BETWEEN 12 AND 13);
Also should return one row but returns many.
Similar problem.
SELECT WORKS.HOURS
FROM WORKS
WHERE WORKS.PNUM NOT IN
(SELECT PROJ.PNUM
FROM PROJ
WHERE PROJ.BUDGET BETWEEN 5000 AND 40000);
Returns more than one row.
SELECT HOURS
FROM WORKS
WHERE PNUM NOT IN
(SELECT PNUM
FROM WORKS
WHERE PNUM IN ('P1','P2','P4','P5','P6'));
Returns more than one row.
SELECT STAFF.EMPNAME
FROM STAFF
WHERE NOT EXISTS
(SELECT *
FROM PROJ
WHERE NOT EXISTS
(SELECT *
FROM WORKS
WHERE STAFF.EMPNUM = WORKS.EMPNUM
AND WORKS.PNUM=PROJ.PNUM));
Returns more than one row.

how to repro the tests? can you add the NIST as part of the unittest?

My branch has it but I think I will separate the Tests into a  separate branch and push it so that you can merge into master.

branch nist_tests has these tests.

@zhouruiapple ，the NOT IN subqueres is handling in #220


python-openzwave uses either pydispatch or louie interchangeably

Home Assistant release (hass --version): 0.14.2
Python release (python3 --version): 3.4.2
Component/platform: zwave
Description of problem:
Updates on the zwave network are only picked up when polling happens.
home assistant is importing and using pydispatch directly. Since python openzwave imports either louie or pydispatch based on what's available. home assistant can import from python openzwave to get the same one that is used by openzwave.

I'll send a pull request to fix this as soon as I can.

This would have to be fixed for every Z-Wave platform that is currently importing pydispatch. After these have been updated we should also be able to remove pydispatch from our dependencies.
pydispatch is the one that is used on Python 3 and Python 2 is using louie. Still, using their version is better 👍

Hmmm. I should have named the bug after my actual issue and not what I
thought was causing it.
I'm seeing events in the zwave debug log but nothing makes it into home
assistant. I thought the wrong dispatcher was in use.
I'll post the logs when I get a chance.
On Apr 6, 2016 2:54 PM, "Paulus Schoutsen"<EMAIL_ADDRESS>wrote:

This would have to be fixed for every Z-Wave platform that is currently
importing pydispatch. After these have been updated we should also be able
to remove pydispatch from our dependencies.
pydispatch is the one that is used on Python 3 and Python 2 is using
louie. Still, using their version is better 👍
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/balloob/home-assistant/issues/1729#issuecomment-206587972

We do not support all Z-Wave command classes yet so it might be that your devices are properly detected by Open Z-Wave but not yet understood by Home Assistant.

Switches and binary sensors (motion). They show up in the ui, but the
status updates don't propagate into home assistant. They show up if they
are in the target state long enough, but they are in the zwave logs
immediately.
On Apr 6, 2016 7:46 PM, "Paulus Schoutsen"<EMAIL_ADDRESS>wrote:

We do not support all Z-Wave command classes yet so it might be that your
devices are properly detected by Open Z-Wave but not yet understood by Home
Assistant.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/balloob/home-assistant/issues/1729#issuecomment-206667245

Benliles, do you are using go-control switches??? I got the same issue with them....

I have GE/Jasco switches and Monoprice/Vision Security motion sensors mainly.

It could be that the motion sensor doesn't send an off event, I had that problem with a philio multi sensor. There is a work around for that case so just add it to the mapping in the binary sensor component if that is the case!

Finally managed to get a dev version of home assistant running and my issue has already been fixed.


Pushbullet: Connection aborted closed connection without response

Expected behaviour:
Notifications to be sent consistently using Pushbullet.
Actual behaviour:
Some notifications are sent, some don't. I can't seem to identify a pattern. The internet connection seems to be fine, maybe something on Pushbullet's side is unstable/changed? I'm seeing this for the last few months.
It always logs the warning when it doesn't work, even without DEBUG loglevel:
2023-08-10 18:32:57 DEBUG    utils.requests task2_auto POSTing URL https://api.pushbullet.com/v2/pushes with args () and kwargs {'data': None, 'json': {'type': 'link', 'title': 'Foo.S06E10.1080p.AMZN.WEB-DL.DDP5.1.H.264-Team (Category)', 'body': '⬇19:27:54 Thu, 10. task2_auto', 'url': 'https://host.tld/download.php/id/hash/%5BiNs-id%5D%5BCategory%5DFoo.S06E10.1080p.AMZN.WEB-DL.DDP5.1.H.264-Team.torrent'}, 'headers': {'Authorization': b'Basic asd==', 'Content-Type': 'application/json', 'Accept': 'application/json', 'User-Agent': 'Flexget'}, 'timeout': 30}
2023-08-10 18:32:57 WARNING  notify        task2_auto Error while sending notification to `pushbullet`: ('Connection aborted.', RemoteDisconnected('Remote end closed connection without response'))

Steps to reproduce:

Configure Flexget as below
Keep sending notifications until you see the issue (as not every one of them fails)

Config:
Show config

There is a FlexGet process already running for this config, sending execution there.
2023-08-11 02:18:41 VERBOSE  assume_quality task2_auto-forgn New assumption: 2160p is h264 hdtv
2023-08-11 02:18:41 VERBOSE  assume_quality task2_auto-forgn New assumption: 1080p is h264 hdtv
2023-08-11 02:18:41 VERBOSE  assume_quality task2_auto-forgn New assumption: 720p is h264 hdtv
2023-08-11 02:18:41 VERBOSE  assume_quality task2_auto-forgn New assumption: any is 360p sdtv h264
─── config from task: task2_auto-forgn ───────────────────────────────────────────────────────────────────────────────────────────────────────────────────────
assume_quality:
  1080p: h264 hdtv
  2160p: h264 hdtv
  720p: h264 hdtv
  any: 360p sdtv h264
notify:
  entries:
    message: "\u2B07{{ rss_pubdate|formatdate(\"%H:%M:%S %a, %d.\") }} {{task}}"
    title: '{{title}}'
    via:
      - pushbullet:
          api_key: hash
          url: '{{url}}'
regexp:
  reject:
    - xvid
    - forgnSub
  reject_excluding:
    - Sorozat/forgn
retry_failed:
  max_retries: 10
  retry_time: 3 minutes
  retry_time_multiplier: 1.5
rss:
  all_entries: true
  ascii: false
  escape: false
  group_links: false
  silent: false
  url: https://host.tld/torr.rss
series:
  forgn serials:
    - Foo:
        alternate_name:
          - Bar
  settings:
    ENG serials:
      season_packs: always
    forgn serials:
      quality: h264|h265 360p+
      season_packs: always
      target: h264|h265 1080i+ webdl+
      timeframe: 3 days
      upgrade: true
template:
  - dwn
  - task2_rss
  - forgn_serials_list
transmission:
  add_paused: true
  host: localhost
  include_subs: true
  main_file_only: true
  main_file_ratio: 0.85
  password: pwd
  path: /path/to/videos/{{ series_name }}
  port: 9091
  username: user

Log:

(click to expand)
2023-08-10 18:32:56 VERBOSE  task          task2_auto ACCEPTED: `Foo.S06E10.1080p.AMZN.WEB-DL.DDP5.1.H.264-Team (Category)` by series plugin because target quality
2023-08-10 18:32:56 DEBUG    series        task2_auto processing series took 0.023345431999928223
2023-08-10 18:32:56 DEBUG    urlrewriter   task2_auto Checking 1 entries
2023-08-10 18:32:56 INFO     download      task2_auto Downloading: Foo.S06E10.1080p.AMZN.WEB-DL.DDP5.1.H.264-Team (Category)
2023-08-10 18:32:56 DEBUG    download      task2_auto Downloading url 'https://host.tld/download.php/id/hash/%5BiNs-id%5D%5BCategory%5DFoo.S06E10.1080p.AMZN.WEB-DL.DDP5.1.H.264-Team.torrent'
2023-08-10 18:32:56 DEBUG    utils.requests task2_auto GETing URL https://host.tld/download.php/id/hash/%5BiNs-id%5D%5BCategory%5DFoo.S06E10.1080p.AMZN.WEB-DL.DDP5.1.H.264-Team.torrent with args () and kwargs {'auth': None, 'headers': {'User-Agent': 'FlexGet/3.8.4 (www.flexget.com)', 'Accept-Encoding': 'gzip, deflate, br', 'Accept': '*/*', 'Connection': 'keep-alive'}, 'stream': True, 'allow_redirects': True, 'timeout': 30}
2023-08-10 18:32:57 DEBUG    download      task2_auto Foo.S06E10.1080p.AMZN.WEB-DL.DDP5.1.H.264-Team (Category) field file set to: /tmp/tmp3jgmcgoi/28f93e6b1a50ed05f732f14b34071889
2023-08-10 18:32:57 DEBUG    download      task2_auto Found filename from headers: [iNs-id][Category]Foo.S06E10.1080p.AMZN.WEB-DL.DDP5.1.H.264-Team.torrent
2023-08-10 18:32:57 DEBUG    download      task2_auto Mimetype guess for application/x-bittorrent is ['.torrent']
2023-08-10 18:32:57 DEBUG    download      task2_auto Filename [iNs-id][Category]Foo.S06E10.1080p.AMZN.WEB-DL.DDP5.1.H.264-Team.torrent extension matches to mime-type
2023-08-10 18:32:57 DEBUG    download      task2_auto Finishing download_entry() with filename [iNs-id][Category]Foo.S06E10.1080p.AMZN.WEB-DL.DDP5.1.H.264-Team.torrent
2023-08-10 18:32:57 DEBUG    download      task2_auto Successfully retrieved Foo.S06E10.1080p.AMZN.WEB-DL.DDP5.1.H.264-Team (Category) from https://host.tld/download.php/id/hash/%5BiNs-id%5D%5BCategory%5DFoo.S06E10.1080p.AMZN.WEB-DL.DDP5.1.H.264-Team.torrent
2023-08-10 18:32:57 VERBOSE  details       task2_auto Summary - Accepted: 1 (Rejected: 0 Undecided: 14 Failed: 0)
2023-08-10 18:32:57 DEBUG    modif_torrent task2_auto Foo.S06E10.1080p.AMZN.WEB-DL.DDP5.1.H.264-Team (Category) seems to be a torrent
2023-08-10 18:32:57 DEBUG    torrent_files task2_auto Foo.S06E10.1080p.AMZN.WEB-DL.DDP5.1.H.264-Team (Category) files: ['Sample/foo-sample.mkv', 'foo.s06e10.1080p.web.mkv', 'foo.s06e10.1080p.web.nfo', 'foo.s06e10.1080p.web.sfv']
2023-08-10 18:32:57 DEBUG    torrent_size  task2_auto Foo.S06E10.1080p.AMZN.WEB-DL.DDP5.1.H.264-Team (Category) size: 1.95 GiB
2023-08-10 18:32:57 DEBUG    transmission  task2_auto Connecting to http://localhost:9091/transmission/
2023-08-10 18:32:57 DEBUG    transmission  task2_auto Successfully connected to transmission.
2023-08-10 18:32:57 INFO     transmission  task2_auto "Foo.S06E10.1080p.AMZN.WEB-DL.DDP5.1.H.264-Team (Category)" torrent added to transmission
2023-08-10 18:32:57 DEBUG    transmission  task2_auto Downloading 1 of 4 files in torrent.
2023-08-10 18:32:57 DEBUG    notify        task2_auto Sending a notification to `pushbullet`
2023-08-10 18:32:57 DEBUG    utils.requests task2_auto POSTing URL https://api.pushbullet.com/v2/pushes with args () and kwargs {'data': None, 'json': {'type': 'link', 'title': 'Foo.S06E10.1080p.AMZN.WEB-DL.DDP5.1.H.264-Team (Category)', 'body': '⬇19:27:54 Thu, 10. task2_auto', 'url': 'https://host.tld/download.php/id/hash/%5BiNs-id%5D%5BCategory%5DFoo.S06E10.1080p.AMZN.WEB-DL.DDP5.1.H.264-Team.torrent'}, 'headers': {'Authorization': b'Basic asd==', 'Content-Type': 'application/json', 'Accept': 'application/json', 'User-Agent': 'Flexget'}, 'timeout': 30}
2023-08-10 18:32:57 WARNING  notify        task2_auto Error while sending notification to `pushbullet`: ('Connection aborted.', RemoteDisconnected('Remote end closed connection without response'))
2023-08-10 18:32:57 DEBUG    download      task2_auto removing temp file /tmp/tmp3jgmcgoi/hash from Foo.S06E10.1080p.AMZN.WEB-DL.DDP5.1.H.264-Team (Category)
2023-08-10 18:32:57 DEBUG    seen          task2_auto Learned 'Foo.S06E10.1080p.AMZN.WEB-DL.DDP5.1.H.264-Team (Category)' (field: title, local: False)
2023-08-10 18:32:57 DEBUG    seen          task2_auto Learned 'https://host.tld/download.php/id/hash/%5BiNs-id%5D%5BCategory%5DFoo.S06E10.1080p.AMZN.WEB-DL.DDP5.1.H.264-Team.torrent' (field: url, local: False)
2023-08-10 18:32:57 DEBUG    seen          task2_auto Learned 'hash' (field: torrent_info_hash, local: False)
2023-08-10 18:32:57 DEBUG    series        task2_auto on_task_learn
2023-08-10 18:32:57 DEBUG    series        task2_auto marking 1 episode releases and 0 season releases as downloaded for `<Entry(title=Foo.S06E10.1080p.AMZN.WEB-DL.DDP5.1.H.264-Team (Category),state=accepted)>`
2023-08-10 18:32:57 DEBUG    util.simple_persistence task2_auto Flushing simple persistence for task task2_auto to db.
2023-08-10 18:32:57 DEBUG    util.simple_persistence task2_auto Flushing simple persistence for task None to db.
2023-08-10 18:32:57 DEBUG    manager       task1_auto Not running db cleanup, last run 2023-08-07 12:17:05

Additional information:

FlexGet version: 3.8.4
Python version: 3.11.3
Installation method: pip
Using daemon (yes/no): yes
OS and version: Ubuntu 22.04 LTS
Link to crash log: N/A

Hi,
I still see it with the latest version 3.11.36, FYI @gazpachoking - do you have any pointer to where the problem can be? So I can raise a PR as it looks like no one is using this Pushbullet, or I might be the only one having this issue? It's still intermittent.
2024-06-11 11:15:46 WARNING  notify        task Error while sending notification to `pushbullet`: ('Connection aborted.', RemoteDisconnected('Remote end closed connection without response'))

Thank you

Seems like maybe it's on pushbullet's end. Especially if it's intermittent. Is there a rate limit you are hitting maybe?

I don't think it's a rate limit, as I only have 1 push every few days. Can we add a retry or is it not standard for these?

Retry sounds fine to me.
On Thu, Jun 13, 2024 at 9:53 AM Roland B. @.***> wrote:

I don't think it's a rate limit, as I only have 1 push every few days. Can
we add a retry or is it not standard for these?
—
Reply to this email directly, view it on GitHub
https://github.com/Flexget/Flexget/issues/3813#issuecomment-2165744184,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABNV7OQBGXZDEISAZVIPQLZHGP65AVCNFSM6AAAAABI4VOHZ6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNRVG42DIMJYGQ
.
You are receiving this because you were mentioned.Message ID:
@.***>


Check DatastoreManager passed to CDTReplicatorFactory

Check if the DatastoreManager passed to CDTReplicatorFactory
is nil, if so CDTReplicatoryFacotry initWithDatastoreManager
returns nil.
BugzID 46518
Fixes #132
reviewer @gadamc
reviewer @indisoluble

Looks good. My only comment is if we think we should an a withError:NSError ** argument to the method... That is, create a new method with the withError and mark this one as deprecated at some point in the future. (Or should I issue a separate FB ticket for that?)

@gadamc I think a new FB ticket for the new API, but I agree an API some kind of error indication outside of the logs is the way to go

In that case, if there will be a new FB ticket to modify the API, this PR looks good to me :+1:

@indisoluble I thought that init method looked funny for some reason... couldn't put my finger on it. ivars!
+1


fix: pin the tarpaulin image and use stable rust

The current coverage workflow is failing with the latest tarpaulin image, as seen here.
This PR switches to the latest stable release instead. I tested locally, and I was able to generate the coverage report.
I think we'll also have to update the coverage badge in the README. The current codecov project page displays the following error message:
 This repo has been deactivated

Looks like pushing a new code coverage report reactivated the project in codecov, so we don't have to update the badge in the README after all!


Update integrations README.md

By mistake I did update in README of main repo, but i just understood it and am changing it by updating README of integrations package

Closes #495

@davidmeirlevy please review 🙏🏽

We have decided to take the issue internally, so we will close this PR. Thank you for your interest in contributing!


Update to julia 1.10

Fortunately fixes #152
But has some issue with PackageCompiler
- PackageCompiler: compiling fresh sysimage (incremental=false)
error during bootstrap:
LoadError("/tmp/jl_HeuRo1/sysimage_packagecompiler_47e340e8-ae21-11ee-2456-a7170323e6fe.jl", 18, LoadError("/opt/hostedtoolcache/julia/1.10.0/x64/share/julia/stdlib/v1.10/Test/src/Test.jl", 3, LoadError("/opt/hostedtoolcache/julia/1.10.0/x64/share/julia/stdlib/v1.10/Test/src/precompile.jl", 1, MethodError(Base.RedirectStdStream(1, true), (Core.CoreSTDOUT(),), 0x0000000000008634))))

So, we could matrix-test with Julia versions 1.9.3 (which does not fall victim to #152) and with 1.10.*. Coverage would only be tested with either of them. Still it would be nice to have PackageCompiler working with 1.10. Any ideas @sloede ?

I would like to have green lights on main again. So, for the time being, I would like to use julia 1.10 as default, and 1.9.3 for test package compiler.

Yeah, that is way more clever!


Refactor convert_currency to Improve Readability and Maintainability

This pull request refactors the convert_currency function to improve readability, maintainability, and adherence to best practices. The changes include:
Breaking down long lines for better readability:
Parameters are now defined as a dictionary and passed into the API call.
Consolidating duplicate logic for API calls and fallback handling:
Introduced a helper function fetch_exchange_rate to handle this logic in one place.
Enhancing code structure for better Single Responsibility Principle (SRP):
Extracted tasks into smaller helper functions:
validate_inputs: Handles input validation.
fetch_exchange_rate: Manages API communication and fallback.
convert_amount: Performs currency conversion.
These changes improve the code's modularity, making it easier to test and maintain.
Please review and let me know if there are further changes or improvements you'd like to see.

The input validation in line 58 in currency-converter.py included <= 0 to avoid users entering negative numbers, but if you want to check for non-negative values before sending the request, that is valid too. Everything looks good, approved the PR!


Verify & Implement Action: likes/upsertLike

Function to implement:
Action: likes/upsertLike
Things to note:

[ ] Ensure different branch for each issue.
[ ] Rebase frequently to avoid conflicts later on
[ ] Try to ensure only a couple of commits
[ ] Test your code thoroughly before pushing
[ ] Add to description, "Fixes #issue-no, eg: Fixes #x (where #x is the issue id)
[ ] Work to be done on template branch. i.e. you need to checkout the template branch on your local computer and PR should be to the template branch

Dependent on #60 , #59

I can work on this if no one from the front end team is

Sure, go ahead!


Maven failed to create assembly: No such archiver apk.

I get this error when using Maven to create an artifact:
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-assembly-plugin:2.2-beta-5:single (make-assembly) on project seleniumNode: Failed to create assembly: Error adding file-set for 'io.selendroid:selendroid-server:apk:0.15.0-SNAPSHOT' to archive: Error adding archived file-set. PlexusIoResourceCollection not found for: d:\Workspaces\Repository\io\selendroid\selendroid-server\0.15.0-SNAPSHOT\selendroid-server-0.15.0-SNAPSHOT.apk: No such archiver: 'apk'. -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoExecutionException

I've tried adding the latest android maven plugin as a new dependency, but I get the same error.
This happens with different Selendroid versions. I'm using 0.15.0-SNAPSHOT but the result is the same using 0.14.

which maven version are you using? it works quite well for me with 3.1.1

I was using maven 3.0.3, but I've tried with versions 3.1.1 and 3.2.5 and the result is the same.
I've also updated versions of maven-compiler-plugin and maven-assembly-plugin to the latest release but it didn't work.

I'm using  3.2.5 and there is everything fine.
Is there maybe an error in your seleniumNode project?

I looked for it and I didn't find it. However, if it works for you, I'll continue trying to solve the problem.
Thank you anyway!

sounds like you're not running the maven command from the root of the project?
can you create a gist of your entire output?

Here you have the maven output: https://gist.github.com/vmcano/fd9374ea181a79a339fc
I've also created a gist with the content of the output using the -X option, just in case: https://gist.github.com/vmcano/91dd27290bd405a9bd8e

what is "seleniumNode" ?? This is not selendroid....

seleniumNode is the name of the project. I'm using Selendroid in it.

ok... then you have an issue with your project, and this is not a 'selendroid issue'

I had this problem.  Solved it as follows: http://stackoverflow.com/questions/21984047/selendroid-with-maven-no-such-archiver-apk/38641065#38641065


Roles.userIsInRole without role returns true

I'm using enum for my defined roles.
And at one place I misspelled a role UserRole.Supervisor instead of UserRoe.SUPERVISOR
then I found that Roles.userIsInRole(this.userId, undefined, 'real-madrid') returns true.
e.g.
  Roles.addUsersToRoles(userId, UserRole.SUPERVISON, 'barcelona');
  Roles.userIsInRole(userId, UserRole.Supervison, 'real-madrid'); // Supervisor is misspelled lowercase
  > true

I found it as a security vulnerability.
Do you have some best practice to avoid it?
Is it in purpose?

Which version are you using? 1.0 or 2.0?

<EMAIL_ADDRESS>

@mitar What is the status of v2?

I think it is ready, but it is waiting for @alanning to release it (or release at least release candidate).

I made a test for 2.0 in c5e27a83ba88eecae00a28283f8ad93bc83916d0 and it is returning false for unknown roles.

And for 1.0 I made e486f5a34f93eff514205af58ace2c64fc9b2909. I cannot reproduce what you are reporting here, so I think you have an issue on your side.


Correctly calculate y-axis range for multi-frequency upper limits

Fix #566

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

In the file vasttools/source.py:

Line 572:1: W293 blank line contains whitespace
Line 582:1: W293 blank line contains whitespace


Additions To Documentation

added new IAM use case
added documentation around IAM generic commands
split context standards to mandatory and recommended
added some more best practices.

Please review and add insights

Status
In Progress

Merged to new sidebar to avoid conflicts

added more fixes and additional things. please review

@kirbles19 please review

@Arsenikr Done.

@Arsenikr there are a few empty yaml examples as reported here: #346
Please add these too.

@glicht filled those. any additional changes? i want to close the PR


Adding overloading functions / methods?

They exist in both Flow and TypeScript. Though in Flow they are trickier to use, and there is no class method overloading as a non declared class.

I'd welcome a PR explaining this :)


vim pastes 2 buffers on shift+middle_click

Steps to reproduce

run vim in xterm on wayland -- I seem I am unable to repro in konsole (TERM is still xterm-256color)
put vim to the background by ctrl+z (this seems to be necessary)
put it to foreground again by fg
delete some text and enter INSERT mode, e.g. shift+c on a line. (as an example I deleted text "DELETED")
copy some text from another terminal window (it's vim in konsole in this example) using shift+left_click+drag (as an example I copied text "PASTE")
paste the text to the terminal from 1. using shift+middle_click.
the text pasted is "DELETEDPASTE", not only "PASTE". I.e. pasted is text both from register "" and X buffer (FWIW xterm is set NOT to use clipboard). Which is like I'd have pressed middle_click and then also shift+middle_click.

Note that it happens only if the destination window is not the active one. I.e. I select the buffer in the source terminal, the destination becomes inactive. Then pasting to the destination makes the dest active and inserts both buffers there. If I insert the buffer again with another shift+middle_click, it pastes only "PASTE", because the dest is already active. This is also a workaround: first click on the window title to make the dest active and then do shift+middle_click inside the window.
Expected behaviour
Only "PASTE" to appear.
Version of Vim
9.2.4602
Environment

openSUSE Tumbleweed
xterm
xterm-256color
bash

Logs and stack traces
No response

is this wayland specific? Please try out https://github.com/vim/vim/pull/9639

is this wayland specific?

Likely, as I didn't see the issue in the past decade with X.
More precisely, it seems to be Xwayland specific.

Please try out #9639

OK, I will.

Followed the steps on Fedora 36 (swaywm -- wayland wm, xterm from repo) -- for me it pastes "PASTED" as expected.

Please try out #9639

OK, I will.

Works for me, apparently.


Pipe reformat eats annotation

There are some roundtrip issues too.

The roundtrip tests failing are:
-let s1 = @ann x->foo
+let s1 = (@ann x)->foo
 let s2 = @ann foo(x)
-let s3 = @ann x |> foo
+let s3 = (@ann x) |> foo

indicating a printing issue


feat(protocol-designer): use window width and height

Overview

Test Plan and Hands on Testing

Changelog

Review requests

Risk assessment

@alexjoel42 @koji this doesn't have the right breakpoints so we need to adjust them first. right not the take over does not get triggered. check your browser consoles to see how it change as you resize the screen

[ ] fix lint-error


kafka-operator rolling upgrade of broker pods.

If we use open source version of banzai kafka-operator, and we modify or update the read-only config "auto.create.topics.enable=false" will the kafka cluster get a rolling upgrade.
so in summary will the open source kafka-operator support rolling upgrades without supertubes core/pro enterprise versions.
And also in the documentation  we see that super-tubes as a pre-requisite for installing.
so is super-tubes a mandate incase of opensource operator ?

Updating readonly config will trigger rolling upgrade


Binaries for management and signal (apt/yum)

Hi,
It's a great add if the management and signal could be included to apt/yum repository to bypass the need of use Docker.
Some people (me included) prefer other alternatives like standalone installations or via LXC. This will help the installations with Ansible/Salt.
Best regards!

I agree, I too would like to see those added as apt/yum repository files if possible!


jwk.get_key(private=False) returns private key object

It seems jwk.get_key(private=False) returns private key object. Shouldn't it return a public key one?
import cryptojwt.jwk
from cryptography.hazmat.backends import default_backend
import cryptography.hazmat.primitives.asymmetric.ec as ec
import cryptography.hazmat.primitives.asymmetric.rsa as rsa

ec_key = ec.generate_private_key(cryptojwt.jwk.NIST2SEC['P-256'], default_backend())
ec_jwk = cryptojwt.jwk.ECKey(key=ec_key, kid='test')
print(type(ec_jwk.get_key(alg='EC', private=False)))

rsa_key = rsa.generate_private_key(3, 2048, default_backend())
rsa_jwk = cryptojwt.jwk.RSAKey(key=rsa_key, kid='test')
print(type(rsa_jwk.get_key(alg='RSA', private=False)))

Workaround:
jwk = cryptojwt.jwk.key_from_jwk_dict(jwk.serialize(private=False), private=False)

One could of course argue that the key object is to be considered opaque and not be messed around with. @rohe, your call :-)

If you specify that you don't want the private key you must get the public one.
That's a bug.

Want patch?

https://github.com/IdentityPython/cryptojwt/pull/5


Question about availability of VVVH

Hello,
I was trying out the code as written, and noticed I was getting the following error message below, which to me indicates that either

there is no VH available, which is odd considering the parameter VVVH means the ImageCollection is filtered to contain both bands, or
the bands are being renamed incorrectly in the underlying scripts.

I tried changing the wrapper script such that it contained the following code but this did not solve the issue. Possibly this might be because there is a period of time for this point (Myanmar) where some data was not obtained, but I had thought adding this filtering process would mean dates were analyzed only if they contained observations for both polarizations.
if (params.POLARIZATION=='VV') { s1 = s1.select(['VV','angle'])}
  else if (params.POLARIZATION=='VH') {s1 = s1.select(['VH','angle'])}
  else if (params.POLARIZATION=='VVVH') {
    s1 = s1.filter(ee.Filter.listContains('transmitterReceiverPolarisation', 'VV'))
            .filter(ee.Filter.listContains('transmitterReceiverPolarisation', 'VH'))
            .select(['VV', 'VH', 'angle'])}

Can you please advise? Here is the set of parameters with my geometry.
var pt = ee.Geometry.Point([95.219623, 23.029892])

var wrapper = require('users/adugnagirma/gee_s1_ard:wrapper');
var helper = require('users/adugnagirma/gee_s1_ard:utilities');

//---------------------------------------------------------------------------//
// DEFINE PARAMETERS
//---------------------------------------------------------------------------//

var parameter = {//1. Data Selection
              START_DATE: "2015-06-23",
              STOP_DATE: "2020-02-01",
              POLARIZATION:'VVVH',
              ORBIT : 'BOTH',
              GEOMETRY: pt, //uncomment if interactively selecting a region of interest
              //GEOMETRY: ee.Geometry.Polygon([[[104.80, 11.61],[104.80, 11.36],[105.16, 11.36],[105.16, 11.61]]], null, false), //Uncomment if providing coordinates
              //GEOMETRY: ee.Geometry.Polygon([[[112.05, -0.25],[112.05, -0.45],[112.25, -0.45],[112.25, -0.25]]], null, false),
              //2. Additional Border noise correction
              APPLY_ADDITIONAL_BORDER_NOISE_CORRECTION: true,
              //3.Speckle filter
              APPLY_SPECKLE_FILTERING: true,
              SPECKLE_FILTER_FRAMEWORK: 'MULTI',
              SPECKLE_FILTER: 'REFINED LEE',
              SPECKLE_FILTER_KERNEL_SIZE: 7,
              SPECKLE_FILTER_NR_OF_IMAGES: 10,
              //4. Radiometric terrain normalization
              APPLY_TERRAIN_FLATTENING: true,
              DEM: ee.Image('USGS/SRTMGL1_003'),
              TERRAIN_FLATTENING_MODEL: 'VOLUME',
              TERRAIN_FLATTENING_ADDITIONAL_LAYOVER_SHADOW_BUFFER: 0,
              //5. Output
              FORMAT : 'DB',
              CLIP_TO_ROI: false,
              SAVE_ASSETS: false
}

//---------------------------------------------------------------------------//
// DO THE JOB
//---------------------------------------------------------------------------//

//Preprocess the S1 collection
var s1_preprocces = wrapper.s1_preproc(parameter);

var s1 = s1_preprocces[0]
s1_preprocces = s1_preprocces[1]

//---------------------------------------------------------------------------//
// EXPORT
//---------------------------------------------------------------------------//

//Convert format for export
if (parameter.FORMAT=='DB'){
  s1_preprocces = s1_preprocces.map(helper.lin_to_db);
}

print(s1_preprocces.first())
print(s1_preprocces)

//Save processed collection to asset
if(parameter.SAVE_ASSETS) {
helper.Download.ImageCollection.toAsset(s1_preprocces, '',
               {scale: 10,
               region: s1_preprocces.geometry(),
                type: 'float'})
}

This means only single pol (VV) images are available for the specified roi and date in the GEE Sentinel-1 catalogue. I have seen this type of issues for some regions especially from the year 2014 to 2018. In your roi, you have both VV and VH polarizations starting 2018.

This is true for Descending, but I've noticed that using the two orbits together there is full coverage.
For context, I had developed a script earlier that manually did speckle filtering (only Refined Lee) plus the Vollrath et al 2020 correction, then I extracted the VV / VH values at a point for every date that had both VV and VH readings (see below plot). I'm confused why I'm able to do this but I'm not able to use your module and get the same result. I've also tried running Ascending and Descending separately (in the code I wrote in the original post) but I also received the same errors.
For the record, the below plot refers to a different point than the original post. However, I have run this script on ~200 points and have gotten measurements just as seen here.
var pt = ee.Geometry.Point([97.90212, 23.17163])

The problem arises when we call the sentinel-1 images  from line 76 to 81. The VH channel is a dual-pol mode in Sentinel-1 unlike the VV pol, so we have to add an additional filter s1.filter(ee.Filter.listContains('transmitterReceiverPolarisation', 'VH')) to make sure all the images have a VV and VH band.
I hope this solves your problem.

I guess overall I'm just confused as to why when I extract band values on my own, I can get a full time series, but your module does not allow me to even though we are both doing the same filtering.
Here is a subset of my script when it calls the S1 data - note how filtered contains images from 2015 onward that contain both VV and VH, for both Ascending and Descending orbits.
I tried getting "S1_GRD_FLOAT" as you did and my code ran correctly.
var pt = ee.Geometry.Point([97.90212, 23.17163])

var satellite = "COPERNICUS/S1_GRD"
var date_start = '2015-06-23';
var date_end = '2020-01-31';

  var satimgs_init = ee.ImageCollection(satellite)
                  .filterDate(date_start, date_end)
                  .filterBounds(pt);

  // filter to keep VV and VH
  var filtered = satimgs_init.filter(ee.Filter.listContains('transmitterReceiverPolarisation', 'VV'))
              .filter(ee.Filter.listContains('transmitterReceiverPolarisation', 'VH'))
              .filter(ee.Filter.eq('instrumentMode', 'IW'));

  print(filtered.first())
  print(filtered)

From your module:
/************************
   * 1. Data Selection
  ************************/

  // Select S1 GRD ImageCollection
var s1 = ee.ImageCollection('COPERNICUS/S1_GRD_FLOAT')
      .filter(ee.Filter.eq('instrumentMode', 'IW'))
      .filter(ee.Filter.eq('resolution_meters', 10))
      .filter(ee.Filter.listContains('transmitterReceiverPolarisation', 'VH'))
      .filterDate(params.START_DATE, params.STOP_DATE)
      .filterBounds(params.GEOMETRY);

I see the same 226 images in both. Anyway, if you have other issues with the ARD framework I'll be happy to help.

Hi @adugnag apologies for not being clear. As you saw, both return the 226 images. However, my code successfully retrieves the VV-VH values for all of those, whereas your ARD code does not due to the error mentioned in the original post. My main issue is I do not understand why. Potentially it has to do with me trying to implement the mutli-temporal speckle filter + Refined Lee using your ARD framework, whereas in my original code I only do the Refined Lee.

Well, the issue came from band inconsistencies between the image to be filtered and the dedicated collection that we call in speckle_filter.MultiTemporal_Filter. Its now solved by getting the last polarization from the image to be filtered, and then filtering for images that contain that (please see line 361 to 379). I hope this helps!

I see. Yes, I can confirm this works for the test point I used, thank you! I will attempt to use this for all my study points (~200) and I will confirm here if it works for batch-processing.
Final question: What is the purpose of line 105 in s1_ard.js? It seems that var s1 is not used anywhere.

That's just for visualizing the processed and raw images to help people select processing parameters. It is not important if you're not interested in visualizations.

I see. Also, I have tried batch-processing using the ARD framework and I can confirm it is working. Thank you for your help with this!


Updating a document gets stuck on the same route

Clicking on the "Update" button in the Document Update page doesn't succeed in transitioning to the next page. The console shows the following errors from the vue-router (the browser is actually stuck and busy probably doing endless loops before showing the errors)
[vue-router] Uncaught error during transition:
Uncaught TypeError: room.unsubscribe is not a function
Uncaught TypeError: Cannot read property 'length' of null

Steps to reproduce:

Pick a document and click on the pencil to edit it
edit (or not) any of the fields
click "Update"
See the magic (LOL)

fixed by #134


Structured data (like json, toml, yaml) asserts in trycmd

Building on #92, it'd be great to have structured data handling in trycmd to allow irrelevant details to be glossed over and for smarter wildcards.

We could have feature flags per format.
We probably also want to allow tests to enable which formats should be handled.  This would be the structured-data version of text-vs-binary.

We need to allow ways to opt-out of "ignoring the irrelevant details" because things like formatting can be important for people to test.
Most likely, someone will only need a fraction of their structured-data tests to focus on formatting vs the rest focusing on various cases for the content of the data.
See also discussions in #91, #95
CC @MiSawa


How to intsall on arch linux?

Is there a guide how to install it on arch linux?

arch linux expects you to install either packages that are packaged with pacman or to use a virtual environment for everything else. Please have a look at https://wiki.archlinux.org/title/Python/Virtual_environment for instructions on how to create a virtual envronment. You should be able to use pip inside the venv to install this package there.


Timed out datarow affects subsequent datarows

Description
When executing a test with multiple datarows, and one datarow times out, all subsequent datarows are aborted and marked failed.
Steps to reproduce
Runt the following test:
    [TestClass]
    public class UnitTest1
    {
        [DataTestMethod]
        [DataRow(1)]
        [DataRow(2)]
        [DataRow(3)]
        [DataRow(2000)]
        [DataRow(4)]
        [DataRow(5)]
        [Timeout(500)]
        public async Task TestDataRowTimeout(int delay)
        {
            await Task.Delay(delay);
        }
    }

Expected behavior
Only datarow with value 2000 times out. However row 4 and 5 are aborted with message Test 'TestDataRowTimeout' execution has been aborted.
The timeout doesn't apply to the combined time for all rows - that can be confirmed by setting the first few rows to be very close to 500, and added up exeeding the timeout value. If the combined time spent mattered, it would time out before making it to the slow row.
Actual behavior
all rows after the timed out row are aborted.

Environment
VS2019.7. .NET Core 3.1. using latest packages.
.csproj used:
<Project Sdk="Microsoft.NET.Sdk">

  <PropertyGroup>
    <TargetFramework>netcoreapp3.1</TargetFramework>

    <IsPackable>false</IsPackable>
  </PropertyGroup>

  <ItemGroup>
    <PackageReference Include="Microsoft.NET.Test.Sdk" Version="16.7.0" />
    <PackageReference Include="MSTest.TestAdapter" Version="2.1.2" />
    <PackageReference Include="MSTest.TestFramework" Version="2.1.2" />
  </ItemGroup>

</Project>

I believe the problem is that the CancellationTokenSource isn't reset after each datarow.
When a row is timed out, this line of code is hit:
https://github.com/microsoft/testfx/blob/8fd086832b835928ab90edd36c7478323b2f4613/src/Adapter/MSTest.CoreAdapter/Execution/TestMethodInfo.cs#L792
However all subsequent datarows will be hitting this line instead, because the token is reused for all tests:
https://github.com/microsoft/testfx/blob/8fd086832b835928ab90edd36c7478323b2f4613/src/Adapter/MSTest.CoreAdapter/Execution/TestMethodInfo.cs#L787
So this basically will always return false after that point:
https://github.com/microsoft/testfx/blob/8fd086832b835928ab90edd36c7478323b2f4613/src/Adapter/MSTest.CoreAdapter/Execution/TestMethodInfo.cs#L772

You are right. This is caused by sharing CancellationTokenSource across the whole test run
https://github.com/nohwnd/testfx/blob/7bc997580abbbdeb48eca0a4bd1a4dcc5e36fd94/src/Adapter/MSTest.CoreAdapter/Execution/TestMethodInfo.cs#L792
This was implemented as a reaction to not cancelling tests after timeout on dotnet core here https://github.com/microsoft/testfx/issues/608. In dotnet core there is no way to abort a thread, which is what we would need to abort a test, so instead the whole test run process is aborted as soon as a any test times out. Unfortunately this is probably the only way this can be implemented, given the limitations of the platform.
Another way to implement this would be to abandon the thread, as we do in other places, and continue to the next test. This would allow the timeout functionality to somehow work. But would not be very transparent, because the test threads would continue running on the background, possibly locking files or other shared resources.

@nohwnd I don't get this. Each datarow should be running with a new thread with a new cancellation token, so each row timeout can cancel its own thread.
I agree abandoning threads is not the way to go - that's what used to happen in my Xamarin-port of mstest, and it caused a slew of problems with subsequent tests.

@dotMorten

Each datarow should be running with a new thread with a new cancellation token, so each row timeout can cancel its own thread.

Yeah that would be great, but there is no way to abort a thread in .NET Core, and cancellation is cooperative, not preemptive. Cancellation will simply set a flag on the token. And if the task checks it, and decides that it can cancel itself the task will finish. If the task never checks, or decides not to cancel, nothing will happen, and the task will happily continue.
So we need a more heavy handed approach. Unfortunately the next best thing is killing the whole process.
But yeah we probably should revisit this to add better support for cancellation in async methods. We would need to propagate the token to the test method. Report if it reacted to the cancellation and either continue when cancelled, or kill the process when not aborted. And we would need to do this in a way that is backwards compatible.
[Timeout(10000; CancelAfter = 3000)]

I am not sure if Tasks can pick up cancellations from context, but in any case user would have to make their code cancellable, and pass the token to methods that cancel themselves.
But that said, this seems like a problem that is common to mstest, xunit and nunit, even though each one is handling it slightly differently.
XUnit cancels and awaits the cancellation forever.
NUnit will complete the task which makes it report in result as completed, but the thread will still run on background, teardown will never run until it finishes.
MSTest kills the process to avoid the problem that NUnit has.
More in depth analysis of that is here: https://github.com/microsoft/testfx/issues/522#issuecomment-567512940

I'm confused about the above response. It seems to deal with how to deal with tearing down a test after a timeout, and not with the fact that each datarow is run as a new test. IMHO there should be no difference between writing 3 identical tests with 3 different parameters, vs writing one test with 3 datarows. Ie how are these really different apart from one saving me from writing a lot of repeated code ?
[Datarow(500)]
[Datarow(1000)]
[Datarow(2000)]
[Timeout(1000)]
public async Task Test(int milliseconds) => await Task.Delay(milliseconds);

vs.
[Timeout(1000)]
public async Task Test1() => await Task.Delay(500);
[Timeout(1000)]
public async Task Test2() => await Task.Delay(1000);
[Timeout(1000)]
public async Task Test3() => await Task.Delay(2000);

Currently use of datarow in tests that could potentially time out is a complete no-go, and we're forced to write all this repeated code.

@dotMorten, this should be fixed with out latest release. Please use 16.10.0 version of Microsoft.NET.Test.Sdk and the latest preview of MSTest; and let us know if you still have this problem.

@Haplois WOOHOO! Thank you

@Haplois Just tried with 16.10.0 and it didn't work.

Oh nevermind. Needed to move the other dependencies to 2.2.4. In fact 16.10.0 of the SDK had no effect.

Do I get this correct that you basically changed the test discoverer to return one test for each row instead of a test with multiple datarows? That would severely break all the existing test history, as these would suddenly be all new tests.

@dotMorten

I'm confused about the above response. It seems to deal with how to deal with tearing down a test after a timeout, and not with the fact that each datarow isn't run as a new test but the outcome of the previous test can affect the outcome of the following tests.

There are multiple things mixed together.

MSTest (until the update you are testing), reported each testcase in the same test. The timeout applies to the test, that is to all testcases combined. If all testcases together take more time than the timeout, the timeout will be triggered.

With the update each test case is reported as a single test, and will get its own timeout.

BUT still, if any test times out, the timing out should trigger killing the process, which used to happen, but it seems to not happen anymore. Instead we complete a task that reports error, but the task that is running the test code still continues. It behaves the same way that NUnit is behaving. But now after the update the next test gets the chance to run, even though the timed out test is still running in the background.

using Microsoft.VisualStudio.TestTools.UnitTesting;
using System;
using System.Threading.Tasks;

namespace TestProject12
{
    [TestClass]
    public class UnitTest1
    {
        [DataRow("tc1", 500)]
        [DataRow("tc2", 2000)]
        [DataRow("tc3", 3000)]
        [Timeout(1000)]
        [TestMethod]
        public async Task Test(string name, int milliseconds)
        {
            System.IO.File.AppendAllLines(@"C:\t\mstest.txt", new[] { $"{name} start" });
            await Task.Delay(milliseconds);
            System.IO.File.AppendAllLines(@"C:\t\mstest.txt", new[] { $"{name} end" });
        }
    }

    [TestClass]
    public class UnitTest2
    {
        [TestMethod]
        public async Task Test2()
        {
            await Task.Delay(3000);
        }
    }
}

Output:

tc1 start
tc1 end
tc2 start
tc3 start
tc3 end
tc2 end
tc1 start
tc1 end
tc2 start
tc3 start
tc3 end
tc2 end

Pre update:

After update:


fix: source map overrides changed to dictionary

The "sourceMapPathOverrides" field in WebDiagnosticsComponents changed from a string array to a Dictionary<string, string>. Reflecting that change here.

This code should only have to deserialize messages in the new format, right?  It will never be used with code that still sends arrays?

This code should only have to deserialize messages in the new format, right? It will never be used with code that still sends arrays?

We'll have to coordinate the insertion with the WebDiagnosticComponents insertion, but yes, otherwise there shouldn't be any mismatch.


Add ConfigManager

Please check if the PR fulfills these requirements

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Feature.

What is the current behavior? (You can also link to an open issue here)

What is the new behavior (if this is a feature change)?

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
Yes. Since this PR adds the ConfigManager, the patterns for configuration have changed. The biggest changes are in the environment variables.

Is there any issue related to this PR? (Link to an issue, e.g. #99999)
https://github.com/dojot/dojot/issues/1733

Other information:

Have you already removed 'V2K_APP_EJBCA_ADDRESS', etc, because this will be changedto use with sidecar, right? And about V2K_APP_CONNECTION_RETRY_COUNT?

@mprevide I haven't removed them, they are in the entrypoint script. They are not used inside the JS code, so I did not map them in the configs, but I forgot to leave their documentation in there. I think we could leave it that way because of the sidecar. What do you think?

Have you already removed 'V2K_APP_EJBCA_ADDRESS', etc, because this will be changedto use with sidecar, right? And about V2K_APP_CONNECTION_RETRY_COUNT?

@mprevide I haven't removed them, they are in the entrypoint script. They are not used inside the JS code, so I did not map them in the configs, but I forgot to leave their documentation in there. I think we could leave it that way because of the sidecar. What do you think?

Yes, what we use in the sidecar, I think you can leave it like that, but V2K_APP_CONNECTION_RETRY_COUNT is not part of the sidecar for exemple.

Have you already removed 'V2K_APP_EJBCA_ADDRESS', etc, because this will be changedto use with sidecar, right? And about V2K_APP_CONNECTION_RETRY_COUNT?

@mprevide I haven't removed them, they are in the entrypoint script. They are not used inside the JS code, so I did not map them in the configs, but I forgot to leave their documentation in there. I think we could leave it that way because of the sidecar. What do you think?

Yes, what we will use in the sidecar I think you can leave it like that.
But V2K_APP_CONNECTION_RETRY_COUNT is not part of the sidecar for exemple.

You're right, it is part of the health check of the service. I think we could leave like this, since these kind of parameter is not so often changed by the user, and then we remove it completely when we integrate the SDK Health Check facility. I'll leave a TODO in there.

Have you already removed 'V2K_APP_EJBCA_ADDRESS', etc, because this will be changedto use with sidecar, right? And about V2K_APP_CONNECTION_RETRY_COUNT?

@mprevide I haven't removed them, they are in the entrypoint script. They are not used inside the JS code, so I did not map them in the configs, but I forgot to leave their documentation in there. I think we could leave it that way because of the sidecar. What do you think?

Yes, what we will use in the sidecar I think you can leave it like that.
But V2K_APP_CONNECTION_RETRY_COUNT is not part of the sidecar for exemple.

You're right, it is part of the health check of the service. I think we could leave like this, since these kind of parameter is not so often changed by the user, and then we remove it completely when we integrate the SDK Health Check facility. I'll leave a TODO in there. Do you think it is OK?

Yes, I think it's ok :+1:


Add range palin

Do the checklist before submitting the PR:

[x] Have you read the guidelines mentioned in CONTRIBUTING.md?
[x] Have you squashed your commits?

Q: What version of Node.js you've used in the project? (e.g. Node v10.11.0)
A:v11.6.0
Q: Mention the Issue Number! (e.g. Fixed #8)
A:#76

Give additional information regarding the PR below:

Codecov Report

Merging #88 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master    #88   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          51     51
  Lines         585    598   +13
=====================================
+ Hits          585    598   +13

Impacted Files
Coverage Δ

src/index.js
100% <ø> (ø)
:arrow_up:

src/palindrome/index.js
100% <100%> (ø)
: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 a2beb55...10f0226. Read the comment docs.

All right

Implemented the Change


Testing tcpconnect

With the latest commit the PR is now incoherent (and I actually recommend to move tests of other packages to a different PR)

I've moved the testing for cmdutils to PR #34

We have a typo in connectionsManagement.go (timeTCPInitiatied -> timeTCPInitiated). Do you mind fixing it inside this PR?


Misc patches

toggle labelmap visibility
hide girder box when uploading
login warning
@aylward

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Fix links in development guide

Description:
Fix links to design docs and installation in development guide.

Affected area:

Documentation          [x]

Please answer the following questions with yes/no.

Does this change contain code from or inspired by another project? If so, did you notify the maintainers and provide attribution? no

Thanks for fixing these links! Just one small comment due to how we have recently restructured the docs directory

@jan25 thanks for making those updates, this PR looks ready to merge. Could you follow these steps to squash your commits so I can go ahead and merge? Thanks!

Hey @SanyaKochhar sure. Just curious, is there a reason PR commit squash isn't setup?
This seems simpler way to squash commits https://docs.github.com/en/github/administering-a-repository/configuring-commit-squashing-for-pull-requests

Hey @SanyaKochhar sure. Just curious, is there a reason PR commit squash isn't setup?
This seems simpler way to squash commits https://docs.github.com/en/github/administering-a-repository/configuring-commit-squashing-for-pull-requests

This is a part of our contributing guidelines. We expect authors to squash their commits prior to merge: https://github.com/openservicemesh/osm/blob/main/CONTRIBUTING.md#contributing-guidelines
@jan25 kindly squash your commits as per the guideline above.


KITE-1101: Added logic to FileSystemPartitionIterator to exclude dire…

…ctories with data that does not qualify. Extended predicates and StorageKey code to support partial values.
Note this might violate some of the initial designs for StorageKey and the Predicates so I'm open to alternate ways of doing this logic.  @rdblue and @rbrush if you have some insight or critique that would be helpful.  It is not necessarily the cleanest changes so if you have suggestions on how to clean this up with original design intent that would be helpful.
In some smaller tests I've seen significant improvements in job startup time (1s vs 100s) and a lot less memory pressure on the driver program when kicking off the job.

+1

+1. Looks good to me.

Got two +1s.  Merging.


Fix edit note name bug

What is this PR for?
hide and show of the editor is not functioning properly
What type of PR is it?
Bug fix
Is there a relevant Jira issue?
no
How should this be tested?
Click on on the note name, change the text and enter or focus out to save the new name. Hit Esc for cancel

Tested, looks good to me.

Tested, Merging if there is no more discussions


Uncaught Exception: Cannot read property 'close' of null

An uncaught exception was reported.
Info:
OS: win32
Arch: ia32
GPMDP Version: 3.1.0
Time: Wed Mar 30 2016 15:47:52 GMT-0400 (Eastern Daylight Time)
Error:
Cannot read property 'close' of null
TypeError: Cannot read property 'close' of null
    at EventEmitter.<anonymous> (C:\Users\fresh\AppData\Local\GPMDP_3\app-3.1.0\resources\app\build\main\features\core\controlBar.js:17:42)
    at emitTwo (events.js:87:13)
    at EventEmitter.emit (events.js:172:7)
    at EventEmitter.<anonymous> (C:\Users\fresh\AppData\Local\GPMDP_3\app-3.1.0\resources\atom.asar\browser\api\web-contents.js:134:25)
    at emitTwo (events.js:87:13)
    at EventEmitter.emit (events.js:172:7)

This issue was created automatically inside the uncaughtException handler

@FreshieGeek This is an interesting one, do you remember how you made this one appear or which window you were trying to close at the time?

I had changed to the light theme and selected a custom colour highlight.
I didn't like coloured border of the miniplayer so I chose #1e1e25. It threw the error when I closed the window. NOT the colour picker window... the Desktop Settings afterwards.

Did the desktop settings window actually close?

It doubled the titlebar, and the lower of the two worked to close the window.
The top bar showed the cursor as waiting.
I didn't need to close the program though. It's still running now after that incident. I haven't restarted it although I've changed a few stations.

It just happened again. As I closed the colour picker circle window the settings window border doubled, and threw the error.

I don't see how that double border is possible... :cactus:
I think you somehow broke electron.....  I'll leave this open and see if anyone else hits this issue

I attempted to reproduce on OSX without luck, if anyone finds a reliable way to throw this error let me know please, thanks


[WEB] PR #102 Rearrange Charts

Description

Resolves #295

Verification

[ ]

Resources

Thanks @fyang151!
@JordanChen123 do you mind doing a pass and taking a look at this?

Great work! The overall code looks good to me.
There is a slight issue with the state of the graphs after refreshing the page. As an example, if you move the speed graph to the very bottom, and then refresh the page, the graphs go back to where they were when the page was loaded up initially, but the state of the Drag and Drop menu does not reflect this.

Great work! The overall code looks good to me.
There is a slight issue with the state of the graphs after refreshing the page. As an example, if you move the speed graph to the very bottom, and then refresh the page, the state of the Drag and Drop menu stays the same as it was before refreshing the page, but the graphs go back to where they were when the page was loaded up initially.

oh oops ill look at it rn and try to get it done by tn meeting


Changing property cell_size has no effect on pathfinding in AStarGrid2D

Tested versions

Reproducible in Godot 4.2.1 stable

System information
Godot v4.2.1.stable - macOS 13.6.2 - Vulkan (Forward+) - integrated Apple M1 Max - Apple M1 Max (10 Threads)
Issue description
Using AStarGrid to find a path through tiles based on a tilemap, changing cell_size to anything above or below the tile size of the tilemap has no effect on where the pathfinding ends up.
The expected outcome (unless I'm misunderstanding the point of "cell_size") is that the tilemap can be divided further than the initial tile size to gain more precise movement for pathfinding. Instead, no matter what you change the cell_size to, objects will always land in a 16x16 grid (or whatever the tilemap tile size is)
Steps to reproduce
Relevant code:
extends TileMap
var astar = AStarGrid2D.new()
var map_rect = Rect2i()
func _ready() -> void:
var tile_size = get_tileset().tile_size
var tilemap_size = get_used_rect().end - get_used_rect().position
map_rect = Rect2i(Vector2i(), tilemap_size)
astar.region = map_rect
astar.cell_size = Vector2(16,16)

astar.default_compute_heuristic = AStarGrid2D.HEURISTIC_EUCLIDEAN
astar.default_estimate_heuristic = AStarGrid2D.HEURISTIC_EUCLIDEAN
astar.diagonal_mode = AStarGrid2D.DIAGONAL_MODE_ONLY_IF_NO_OBSTACLES
astar.update()

for i in tilemap_size.x:
	for j in tilemap_size.y:
		var coordinates = Vector2i(i, j)
		var tile_data = get_cell_tile_data(0, coordinates)
		if tile_data and tile_data.get_custom_data('name') == 'wall':
			astar.set_point_solid(coordinates)

Changing the Vector2 in "astar.cell_size = Vector2(16,16)" above to any other Vector2 has virtually no effect on pathfinding.
Minimal reproduction project (MRP)
AStarGrid_01.zip

Actually, I should also note that commenting out the "astar.cell_size = Vector2(16,16)" line completely also has absolutely no effect on pathfinding or where the object ends up.
If this isn't a bug, then is seems that the cell_size property has no function at all?

cell_size and offset are needed for get_point_path(), not for get_id_path() and other methods. This is simply a multiplier and an offset of the result id path. See the demo.

Thanks, the demo definitely helped me understand a bit, though I still don't exactly understand the real differences between get_point_path and get_id_path, except that get_id_path seems safer to use?
Is there a way to actually set cell sizes in a way that divides tiles appropriately? In other words, let's say my tilemap has tiles sizes of 16x16, but when using AStarGrid for pathfinding, I want to divide each of those tiles further so that four 4x4 cells fit inside those 16x16 tiles? And in this way, get a more specific point for an object to go to within said tile of the tilemap?
^This is basically what I thought cell_size did.

No, these properties only affect the get_point_path() result. For all other calculations, you must use cell IDs (integer coordinates that do not take into account cell size and offset). In conjunction with TileMap, you may be interested in the local_to_map() and map_to_local() methods. You can also use the integer division and remainder operations.


Work Package wp-cn.csv

I would like to work on this please!

I would like to work on this please!

Results of Phase 2
wp-cn-result.csv

I completed this work package.

Results of Phase 2
wp-cn-results.csv

I completed this work package

I completed this work package.

I would like to work on this please!

Results of Phase
wp-cn-results.csv
2

Results of Phase 2
wp-cn-results.csv

I completed this work package.


Add MemoryFileSystem

[X] Tests added / passed
[X] Passes flake8 dask
[ ] Fully documented, including docs/source/changelog.rst for all changes
and one of the docs/source/*-api.rst files for new API

See https://github.com/dask/fastparquet/issues/215
This is a single global store. It is meant for use only with the Threaded scheduler - not sure how useful it is.

What would be the right way of determining the size of data help in a bytesio on py2? Is it something that needs to be saved via tell() when we are done writing instead?

I don't know personally, but this seems like the kind of thing that might
get an answer on StackOverflow relatively quickly.
On Thu, Oct 26, 2017 at 10:23 AM, Martin Durant<EMAIL_ADDRESS>wrote:

What would be the right way of determining the size of data help in a
bytesio on py2? Is it something that needs to be saved via tell() when we
are done writing instead?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/dask/dask/pull/2741#issuecomment-339682742, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AASszCJzlv2qTVCs3bOYf9KWnwvUMwk5ks5swJXZgaJpZM4Ps_sY
.

Actually, thinking about it a moment, i.seek(0, 2) should work and come with very little cost.

@martindurant is there anything that remains to be done here?  What's here seems fine to me.
My only comment is that there seems to be a fair amount of copy-pasting between filesystem test suites.  It might make sense at some point to construct an inheritable test class that others can use for tests.  This might be something that we hand to the Arrow folks for use with their HDFS implementation.

I think this is complete enough to be useful.
Agree about the test duplication, although I don't expect this code to change frequently.

I think that this needs to be added to the import at dask/bytes/__init__.py .
It would also be nice to see a roundtrip test with dd.to_csv and dd.read_csv

@martindurant ok to merge?

Yes, I think so. This does not appear explicitly in the docs, but it is a fairly niche use.

Updated here with the simplifications that went into bytes. Can be merged after #3160, if that is good to go.

@alimanfoo , if this would be useful to you for making in-memory zarr files, then please try it out and see how well it works.

Cool, thank you, I'll take a look.
On Mon, 28 May 2018, 01:13 Martin Durant<EMAIL_ADDRESS>wrote:

@alimanfoo https://github.com/alimanfoo , if this would be useful to
you for making in-memory zarr files, then please try it out and see how
well it works.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dask/dask/pull/2741#issuecomment-392390314, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAq8Qvznm1rkNDUIX0cN8dH89R17_uSKks5t20EXgaJpZM4Ps_sY
.

Have a few questions. What contexts does this work in (e.g. single threaded, multithreaded, multiprocessing, distributed, etc.)? Also how does this work when someone wants to access this stored data?

There are a couple of examples of round-tripping in the tests, so the following should work
arr.to_zarr('memory://path/arr.zarr')
arr2  = da.read_zarr('memory://path/arr.zarr')

so long as we are within one process (sync or thread scheduler, or distributed in-process).
If you are not in one process, you would still successfully make the file-like objects of binary data, but would not know which piece was where. That is like persisting a set of keys (binary data in memory) without the global map of which key is where - i.e., not too useful.

With those changes, a simple zarr roundtrip does work.

Closing based on https://github.com/martindurant/filesystem_spec/pull/11#issue-209228566. @martindurant feel free to re-open if needed


Version number is confusing

Description

The version number of the API is prepended with some confusing additional text in the documentation 3387VERSION+:

Reproduction

Visit https://api.openverse.engineering/v1
Observe text 3387VERSION+ at the start of the version number

Expectation

Screenshots

Environment

Device:
OS:
Browser:
Version:
Other info:

Additional context

Resolution

[ ] 🙋 I would be interested in resolving this bug.

This looks to be due to the way we're setting the SEMANTIC_VERSION variable on deployment. Should we move this over to the infra repo?

Moved to Automattic/openverse-infrastructure#80


[FEATURE] Request for Playlist Support

Feature Description

Example:

Support downloading playlists.

Why is this feature beneficial?

Example:

Playlists provide curated content.
Playlists tend to be better organized and more relevant than categories or individual user uploads.
Being able to download entire playlists directly would save a lot of time compared to picking through less organized content.

Example URLs

Single Page Playlist: https://hypnotube.com/playlist/93707/stim-gooning/
Video in playlist: https://hypnotube.com/video/goth-boi-appreciation-femboy-conversion-pmv-19061.html?p=93707
Paginated Playlist: https://hypnotube.com/playlist/5789/good/

@Kathaarian, got a chance to look at it today. Here's the "Partial" update:
Partial Support Added for Handling Playlists:

Works:

Downloading videos from playlists is functional.

Single Page Playlist: https://hypnotube.com/playlist/93707/stim-gooning/
Paginated Playlist: https://hypnotube.com/playlist/5789/good/

Does Not Work:

Downloading the playlist from a video URL is currently not supported.

Example URL: https://hypnotube.com/video/price-to-pay-2357.html?p=93719

I don't currently know of a way to extract the playlist URL from a video attached to a playlist.


Bug: Progress status: UI and counter bugs

Description
When permanently deleting a large folder with many items:

The header isn't fully readable.
The speed isn't readable. (it overlaps with the graph)
The items-done exceeds total-items (I'm guessing some type of files {like hidden} aren't added to the total count?)

Note that the graph also stops moving, because the percentage > 100.

Steps To Reproduce

Have a large folder with many items. In my case: on a USB HDD
Permanently delete the folder
Watch the progress

Requirements

full description should be readable (e.g. with a tooltip, dynamic width or wrap to a new line)
"speed" should always be readable
Number of items done should always be less than the total number of items

Files Version
<IP_ADDRESS>
Windows Version
10.0.22621.2506
Log File
debug.log

Thanks for the report, I've added this to the project board.

Merging with #13624

@0x5bfa did this fall between the cracks? Not sure your pr would fix all of this...

The header isn't fully readable.

You should be able to hover and read.

The speed isn't readable

We are going to discuss another UI for speed text.

The items-done exceeds total-items

This is an issue with OS probably. AFAIK, powershell does have this issue.
Thank you for reporting all:)

This is an issue with OS probably. AFAIK, powershell does have this issue.

Screenshot credits go to @hez2010

The header isn't fully readable.

You should be able to hover and read.

The speed isn't readable

We are going to discuss another UI for speed text.

The items-done exceeds total-items

This is an issue with OS probably. AFAIK, powershell does have this issue.
Thank you for reporting all:)

We did change the speed text so it was entirely in the padded top section for readability, I guess it didn't make it into the PRs for v3 release

FYI - I spotted this in v 2.5.something and now using v<IP_ADDRESS>. Switching from number-of-items to size  helps. But the tooltip fails shortly after starting the action and the speed info is in the same place.


User authentication via certificate

Hi,
is there any plan to add user authentication via certificate ?
Currently there is only Anonymous and UsernamePassword supported in contrast to OPC UA specifications.
https://github.com/Azure/iot-edge-opc-publisher/blob/dc6b669bcd7c45437fb1b0aacf77bfb1e98482f5/opcpublisher/OpcAuthenticationMode.cs

Will be tracked in #2005.


tests/full: Measure coverage

Running in Beaker as TJ#6665529
Lets see how it behaves there (it produced expected output on my local exec as cd tests/full; tmt run)

Okay, it is important to install rpm first before using it for rpm -ql call

Completely reworked, trying again.

TJ#7764179

Not bad.
                    Name                                                                 Stmts   Miss  Cover
                    ----------------------------------------------------------------------------------------
                    /usr/lib/python3.11/site-packages/tmt/__init__.py                        6      0   100%
                    /usr/lib/python3.11/site-packages/tmt/__main__.py                        2      2     0%
                    /usr/lib/python3.11/site-packages/tmt/base.py                         1531    117    92%
                    /usr/lib/python3.11/site-packages/tmt/beakerlib.py                     182     23    87%
                    /usr/lib/python3.11/site-packages/tmt/cli.py                           643     70    89%
                    /usr/lib/python3.11/site-packages/tmt/convert.py                       698    180    74%
                    /usr/lib/python3.11/site-packages/tmt/export/__init__.py               181     26    86%
                    /usr/lib/python3.11/site-packages/tmt/export/_dict.py                   24      2    92%
                    /usr/lib/python3.11/site-packages/tmt/export/nitrate.py                409    112    73%
                    /usr/lib/python3.11/site-packages/tmt/export/polarion.py               216    187    13%
                    /usr/lib/python3.11/site-packages/tmt/export/rst.py                     13      0   100%
                    /usr/lib/python3.11/site-packages/tmt/export/template.py                22      3    86%
                    /usr/lib/python3.11/site-packages/tmt/export/yaml.py                    24      0   100%
                    /usr/lib/python3.11/site-packages/tmt/identifier.py                     39      5    87%
                    /usr/lib/python3.11/site-packages/tmt/log.py                           236     19    92%
                    /usr/lib/python3.11/site-packages/tmt/options.py                        93     20    78%
                    /usr/lib/python3.11/site-packages/tmt/plugins/__init__.py               74      3    96%
                    /usr/lib/python3.11/site-packages/tmt/result.py                        127      7    94%
                    /usr/lib/python3.11/site-packages/tmt/steps/__init__.py                530     36    93%
                    /usr/lib/python3.11/site-packages/tmt/steps/discover/__init__.py       146      8    95%
                    /usr/lib/python3.11/site-packages/tmt/steps/discover/fmf.py            246     18    93%
                    /usr/lib/python3.11/site-packages/tmt/steps/discover/shell.py          128     11    91%
                    /usr/lib/python3.11/site-packages/tmt/steps/execute/__init__.py        248     13    95%
                    /usr/lib/python3.11/site-packages/tmt/steps/execute/internal.py        227     22    90%
                    /usr/lib/python3.11/site-packages/tmt/steps/execute/upgrade.py         133      7    95%
                    /usr/lib/python3.11/site-packages/tmt/steps/finish/__init__.py          72      2    97%
                    /usr/lib/python3.11/site-packages/tmt/steps/finish/ansible.py            8      0   100%
                    /usr/lib/python3.11/site-packages/tmt/steps/finish/shell.py             37      3    92%
                    /usr/lib/python3.11/site-packages/tmt/steps/prepare/__init__.py        117      4    97%
                    /usr/lib/python3.11/site-packages/tmt/steps/prepare/ansible.py          53      3    94%
                    /usr/lib/python3.11/site-packages/tmt/steps/prepare/install.py         278     60    78%
                    /usr/lib/python3.11/site-packages/tmt/steps/prepare/multihost.py        25      0   100%
                    /usr/lib/python3.11/site-packages/tmt/steps/prepare/shell.py            37      3    92%
                    /usr/lib/python3.11/site-packages/tmt/steps/provision/__init__.py      557     85    85%
                    /usr/lib/python3.11/site-packages/tmt/steps/provision/artemis.py       185     91    51%
                    /usr/lib/python3.11/site-packages/tmt/steps/provision/connect.py        53      6    89%
                    /usr/lib/python3.11/site-packages/tmt/steps/provision/local.py          49      2    96%
                    /usr/lib/python3.11/site-packages/tmt/steps/provision/mrack.py         270    180    33%
                    /usr/lib/python3.11/site-packages/tmt/steps/provision/podman.py        132      3    98%
                    /usr/lib/python3.11/site-packages/tmt/steps/provision/testcloud.py     286     57    80%
                    /usr/lib/python3.11/site-packages/tmt/steps/report/__init__.py          54      6    89%
                    /usr/lib/python3.11/site-packages/tmt/steps/report/display.py           36      0   100%
                    /usr/lib/python3.11/site-packages/tmt/steps/report/html.py              56      9    84%
                    /usr/lib/python3.11/site-packages/tmt/steps/report/junit.py             72      7    90%
                    /usr/lib/python3.11/site-packages/tmt/steps/report/polarion.py          84     53    37%
                    /usr/lib/python3.11/site-packages/tmt/steps/report/reportportal.py      63     42    33%
                    /usr/lib/python3.11/site-packages/tmt/templates.py                      19      0   100%
                    /usr/lib/python3.11/site-packages/tmt/utils.py                        1668    192    88%
                    ----------------------------------------------------------------------------------------
                    TOTAL                                                                10389   1699    84%


7.15 opcodes?

Title

I got this for the time being.
It seems to work for me
{     "C2S_ActionRequest": "0x0110",     "C2S_ActionRequestGroundTargeted": "0x0075",     "S2C_ActionEffect01": "0x0395",     "S2C_ActionEffect08": "0x0079",     "S2C_ActionEffect16": "0x0364",     "S2C_ActionEffect24": "0x0153",     "S2C_ActionEffect32": "0x011f",     "S2C_ActorCast": "0x00c5",     "S2C_ActorControl": "0x03a3",     "S2C_ActorControlSelf": "0x008a",     "Server_IpRange": "<IP_ADDRESS>/24, <IP_ADDRESS>/24, <IP_ADDRESS>/24, <IP_ADDRESS>/24, <IP_ADDRESS>/24",     "Server_PortRange": "1025-65535" }


Implement ensureIndexed query functionality

Implement index create and update functionality for Cloudant Query.

OK +1 once you've addressed @mikerhodes' comment. As he says we need to look at any SQL escaping issues in a more comprehensive way as part of a different PR.


Add back root yarn.lock

https://github.com/reactioncommerce/reaction-component-library/pull/413 removed both yarn.lock files but the root file is for the style guide website and as such I don't think it should be removed or ignored.
I'm not totally certain the PR was necessary in the first place since the build script only moves certain files into the final dist folder that is published to NPM, and I did not think yarn.lock was one that is moved and published, but I could be wrong about that. Either way, removing that one shouldn't harm anything.
@focusaurus

Yeah I wasn't aware that yarn only looks at the top-level yarn.lock when running yarn install so our endless snyk loops must be caused by something else.

:tada: This issue has been resolved in version 0.65.4 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


pufferpanel: use fetchNpmDeps

Description of changes
Changes pufferpanel package to use fetchNpmDeps instead of building the frontend in a separate derivation.
2.6.9 → 2.6.10: no changes https://github.com/pufferpanel/pufferpanel/compare/v2.6.9...v2.6.10
Things done

Built on platform(s)

[ ] x86_64-linux
[x] 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
[ ] 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

[ ] Fits CONTRIBUTING.md.

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

It looks pretty great from reading the source. I don't use pufferpanel anymore though so I've pushed a commit removing myself from maintainers. Good luck & great work!


Create new_issue_template.md

Created issue template as per the instructions.
Fixes #105

@upkarlidder Please review this PR.

@Asmit2952 Thanks again for your contribution to our projects! We would like to invite you to join us for our Hacktoberfest Demo Day on October 29th where we'll be highlighting contributions and thanking everyone who participated. Let me know if you can make it! https://www.crowdcast.io/e/hacktoberfest-demo-day/register


processCLockUpdate

module Daml.Finance.Instrument.Generic.Util where
...
settlementDate = toDateUTC v.eventTime -- TODO remove this dependency

Will close it for now, as https://github.com/digital-asset/daml-finance/issues/402 covers this in more detail


CA list in Deprecation of Software Publisher Certificates page is not complete? what is the full list?

Why some of the CAs are not listed in :
https://docs.microsoft.com/en-us/windows-hardware/drivers/install/deprecation-of-software-publisher-certificates-and-commercial-release-certificates#what-is-the-expiration-schedule-of-the-trusted-cross-certificates
There seems to be some CAs that don't get expired in 2021 :
https://docs.microsoft.com/en-us/windows-hardware/drivers/install/cross-certificates-for-kernel-mode-code-signing
does this mean that CAs that are listed in the second link that are not getting expired soon, can still be used to issue EV certificates to sign kernel drivers in 2022,2033?
The enitre page is just a big question mark and a lot of questions are not answered there.. does this update mean that after 3-4 months, no one can develop new windows 7 drivers??
https://www.osr.com/blog/2020/10/15/microsoft-driver-updates-allowed-win7-win8/
how can we opt out of this "update" ???

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

ID: 0e326d3e-833a-e312-d35d-11569690c7e6
Version Independent ID: 7a9795cf-093e-a67f-8162-b832e1161ba8
Content: Deprecation of Software Publisher Certificates, Commercial Release Certificates, and Commercial Test Certificates - Windows drivers
Content Source: windows-driver-docs-pr/install/deprecation-of-software-publisher-certificates-and-commercial-release-certificates.md
Product: windows-hardware
Technology: windows-devices
GitHub Login: @tedhudek
Microsoft Alias: tedhudek

Thanks for their input @JohnSmith1968. We've added a section to the bottom of the FAQ to address your concerns regarding CA's that expire after 2021.
As far as how this affects Windows 7, see the following section:
https://docs.microsoft.com/en-us/windows-hardware/drivers/install/deprecation-of-software-publisher-certificates-and-commercial-release-certificates#how-do-production-signing-options-differ-by-windows-version
If you have specific blockers with WHQL, use one of the feedback mechanisms listed in that section to report them.
Hope this helps!


Include missing files in sdist

Include missing files in sdist for #2
Tested with
++ mktemp -d
+ D=/tmp/tmp.7a9BRkr6SJ
+ trap 'rm -rf /tmp/tmp.7a9BRkr6SJ' EXIT
+ python -m venv /tmp/tmp.7a9BRkr6SJ
+ python setup.py sdist -d /tmp/tmp.7a9BRkr6SJ
running sdist
running egg_info
writing sortmedia.egg-info/PKG-INFO
writing dependency_links to sortmedia.egg-info/dependency_links.txt
writing entry points to sortmedia.egg-info/entry_points.txt
writing requirements to sortmedia.egg-info/requires.txt
writing top-level names to sortmedia.egg-info/top_level.txt
reading manifest file 'sortmedia.egg-info/SOURCES.txt'
reading manifest template 'MANIFEST.in'
writing manifest file 'sortmedia.egg-info/SOURCES.txt'
running check
creating sortmedia-1.2.0
creating sortmedia-1.2.0/sortmedia
creating sortmedia-1.2.0/sortmedia.egg-info
copying files to sortmedia-1.2.0...
copying MANIFEST.in -> sortmedia-1.2.0
copying README.md -> sortmedia-1.2.0
copying requirements.txt -> sortmedia-1.2.0
copying setup.py -> sortmedia-1.2.0
copying sortmedia/__init__.py -> sortmedia-1.2.0/sortmedia
copying sortmedia/cli.py -> sortmedia-1.2.0/sortmedia
copying sortmedia/file.py -> sortmedia-1.2.0/sortmedia
copying sortmedia/photo.py -> sortmedia-1.2.0/sortmedia
copying sortmedia/sort.py -> sortmedia-1.2.0/sortmedia
copying sortmedia/util.py -> sortmedia-1.2.0/sortmedia
copying sortmedia/video.py -> sortmedia-1.2.0/sortmedia
copying sortmedia.egg-info/PKG-INFO -> sortmedia-1.2.0/sortmedia.egg-info
copying sortmedia.egg-info/SOURCES.txt -> sortmedia-1.2.0/sortmedia.egg-info
copying sortmedia.egg-info/dependency_links.txt -> sortmedia-1.2.0/sortmedia.egg-info
copying sortmedia.egg-info/entry_points.txt -> sortmedia-1.2.0/sortmedia.egg-info
copying sortmedia.egg-info/requires.txt -> sortmedia-1.2.0/sortmedia.egg-info
copying sortmedia.egg-info/top_level.txt -> sortmedia-1.2.0/sortmedia.egg-info
Writing sortmedia-1.2.0/setup.cfg
Creating tar archive
removing 'sortmedia-1.2.0' (and everything under it)
+ cd /
+ /tmp/tmp.7a9BRkr6SJ/bin/pip install /tmp/tmp.7a9BRkr6SJ/sortmedia-1.2.0.tar.gz
Processing /tmp/tmp.7a9BRkr6SJ/sortmedia-1.2.0.tar.gz
Collecting attrs==19.3.0 (from sortmedia==1.2.0)
  Using cached https://files.pythonhosted.org/packages/a2/db/4313ab3be961f7a763066401fb77f7748373b6094076ae2bda2806988af6/attrs-19.3.0-py2.py3-none-any.whl
Collecting cffi==1.13.2 (from sortmedia==1.2.0)
  Using cached https://files.pythonhosted.org/packages/05/7d/723ceca757d8016850d1110df44b91121695f22f782f876b483ecb3928a9/cffi-1.13.2-cp38-cp38-manylinux1_x86_64.whl
Collecting ffmpeg-python==0.2.0 (from sortmedia==1.2.0)
  Using cached https://files.pythonhosted.org/packages/d7/0c/56be52741f75bad4dc6555991fabd2e07b432d333da82c11ad701123888a/ffmpeg_python-0.2.0-py3-none-any.whl
Collecting filemagic==1.6 (from sortmedia==1.2.0)
  Downloading https://files.pythonhosted.org/packages/8f/67/a1aedccc143597eea90e79134d14c7364e90174d31443e14674919716d87/filemagic-1.6.tar.gz
Collecting filetype==1.0.5 (from sortmedia==1.2.0)
  Downloading https://files.pythonhosted.org/packages/1c/78/047cf0f371f8faafab05b5062f902d4bedd945d436544b30db692622ab84/filetype-1.0.5-py2.py3-none-any.whl
Collecting importlib-metadata==1.2.0 (from sortmedia==1.2.0)
  Downloading https://files.pythonhosted.org/packages/e7/59/cc2a91f71957dfb32f25e6b19d8c6dfdbe524e0998e7f7a58a75126ce0cd/importlib_metadata-1.2.0-py2.py3-none-any.whl
Collecting more-itertools==8.0.2 (from sortmedia==1.2.0)
  Using cached https://files.pythonhosted.org/packages/68/03/0604cec1ea13c9f063dd50f900d1a36160334dd3cfb01fd0e638f61b46ba/more_itertools-8.0.2-py3-none-any.whl
Collecting packaging==19.2 (from sortmedia==1.2.0)
  Using cached https://files.pythonhosted.org/packages/cf/94/9672c2d4b126e74c4496c6b3c58a8b51d6419267be9e70660ba23374c875/packaging-19.2-py2.py3-none-any.whl
Collecting piexif==1.1.3 (from sortmedia==1.2.0)
  Downloading https://files.pythonhosted.org/packages/2c/d8/6f63147dd73373d051c5eb049ecd841207f898f50a5a1d4378594178f6cf/piexif-1.1.3-py2.py3-none-any.whl
Collecting Pillow==6.2.1 (from sortmedia==1.2.0)
  Downloading https://files.pythonhosted.org/packages/a4/ad/a7fc21b083c3acf873d8d7e2b8605e0b3c8b2a840aff398fdb5f35aaaef7/Pillow-6.2.1-cp38-cp38-manylinux1_x86_64.whl (2.1MB)
Collecting pluggy==0.13.1 (from sortmedia==1.2.0)
  Using cached https://files.pythonhosted.org/packages/a0/28/85c7aa31b80d150b772fbe4a229487bc6644da9ccb7e427dd8cc60cb8a62/pluggy-0.13.1-py2.py3-none-any.whl
Collecting py==1.8.0 (from sortmedia==1.2.0)
  Using cached https://files.pythonhosted.org/packages/76/bc/394ad449851729244a97857ee14d7cba61ddb268dce3db538ba2f2ba1f0f/py-1.8.0-py2.py3-none-any.whl
Collecting pycparser==2.19 (from sortmedia==1.2.0)
Collecting pyheif==0.3.2 (from sortmedia==1.2.0)
  Downloading https://files.pythonhosted.org/packages/8d/66/b28794f1e98614e24b5496f2d3043575f24a14648a6ef234495e4e5f905d/pyheif-0.3.2.tar.gz
Collecting pyparsing==2.4.5 (from sortmedia==1.2.0)
  Using cached https://files.pythonhosted.org/packages/c0/0c/fc2e007d9a992d997f04a80125b0f183da7fb554f1de701bbb70a8e7d479/pyparsing-2.4.5-py2.py3-none-any.whl
Collecting six==1.13.0 (from sortmedia==1.2.0)
  Using cached https://files.pythonhosted.org/packages/65/26/32b8464df2a97e6dd1b656ed26b2c194606c16fe163c695a992b36c11cdf/six-1.13.0-py2.py3-none-any.whl
Collecting wcwidth==0.1.7 (from sortmedia==1.2.0)
  Using cached https://files.pythonhosted.org/packages/7e/9f/526a6947247599b084ee5232e4f9190a38f398d7300d866af3ab571a5bfe/wcwidth-0.1.7-py2.py3-none-any.whl
Collecting zipp==0.6.0 (from sortmedia==1.2.0)
  Using cached https://files.pythonhosted.org/packages/74/3d/1ee25a26411ba0401b43c6376d2316a71addcc72ef8690b101b4ea56d76a/zipp-0.6.0-py2.py3-none-any.whl
Collecting future (from ffmpeg-python==0.2.0->sortmedia==1.2.0)
  Using cached https://files.pythonhosted.org/packages/45/0b/38b06fd9b92dc2b68d58b75f900e97884c45bedd2ff83203d933cf5851c9/future-0.18.2.tar.gz
Installing collected packages: attrs, pycparser, cffi, future, ffmpeg-python, filemagic, filetype, more-itertools, zipp, importlib-metadata, pyparsing, six, packaging, piexif, Pillow, pluggy, py, pyheif, wcwidth, sortmedia
  Running setup.py install for future: started
    Running setup.py install for future: finished with status 'done'
  Running setup.py install for filemagic: started
    Running setup.py install for filemagic: finished with status 'done'
  Running setup.py install for pyheif: started
    Running setup.py install for pyheif: finished with status 'done'
  Running setup.py install for sortmedia: started
    Running setup.py install for sortmedia: finished with status 'done'
Successfully installed Pillow-6.2.1 attrs-19.3.0 cffi-1.13.2 ffmpeg-python-0.2.0 filemagic-1.6 filetype-1.0.5 future-0.18.2 importlib-metadata-1.2.0 more-itertools-8.0.2 packaging-19.2 piexif-1.1.3 pluggy-0.13.1 py-1.8.0 pycparser-2.19 pyheif-0.3.2 pyparsing-2.4.5 six-1.13.0 sortmedia-1.2.0 wcwidth-0.1.7 zipp-0.6.0
WARNING: You are using pip version 19.2.3, however version 20.1.1 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.
+ echo Success
Success
++ rm -rf /tmp/tmp.7a9BRkr6SJ

Looks good. Thanks for that!


.Net: Error: Invalid request: The request is not valid, HTTP status: 404

Describe the bug
In any C# notebook if step has ".InvokeAsync" It errors with Error: Invalid request: The request is not valid, HTTP status: 404
To Reproduce
Following the dotnet notebooks 00-getting-started.ipynb
Expected behavior
In steps 4, the InvokeAsync method returns Error: Invalid request: The request is not valid, HTTP status: 404.
Screenshots

Platform

OS: Windows
IDE: VS Code
Language: C#
Source: NuGet package version Microsoft.SemanticKernel, 0.18.230725.3-preview

@craigomatic  can you take a look into this, this seems linked to #2418

I tried and couldn't repro.
@HikawaRin In Step 1 in the 0-AI-settings.ipynb notebook, did you set useAzureOpenAI = true? It defaults to false and might cause the 404.
The other cause of 404 we usually see is that the deployment name is different to the model name (Azure OpenAI allows you to pick whatever you like for the deployment name). From your screenshots this doesn't seem to be the case...

Hi @craigomatic, It is really the model name issue.
My deployment name is the same as the model name but somehow with a leading space and it is  gpt-35-turbo, that makes me in trouble.
And one thing I want to confirm is whether the model name really means the deployment name in Azure OpenAI scenario?
Many thanks.


Fix integer overflow in PAss when overclocking beyond MAX

Should fix https://github.com/GTNewHorizons/GT-New-Horizons-Modpack/issues/11320

reported to be working in https://github.com/GTNewHorizons/GT-New-Horizons-Modpack/issues/11320


Update Kim CNN SST-1 dataset to use torchtext 0.2.0

Small change to SST-1 dataset to make compatible with latest release version of torchtext, v0.2.0
@Impavidity please review?

@HTAustin please approve? Since Peng doesn't have write access.


Setup fails silently on Mongolab instance

Hi, I'm trying frame and I noticed that while it's working fine on a local mongodb installation,
it doesn't setup properly on a remote Mongolab free instance. Mongodb version is 3.0.9
The setup script runs to completion, but only adminGroups, admin and users collections are created.
Any idea ?

Thanks for opening an issue.

The setup script runs to completion, but only adminGroups, admin and users collections are created.

After you run setup you should start the app. Once you start it the other collections should get created.

Closing due to inactivity. Please re-open if you still need help.