Make download and preparation of the geoname dataset and countrycodes table part of the package installation.

Right now, the data is fetched when Geocoder is run for the first time. It would make more sense to move this step to the installation step to avoid runtime errors related to missing/incomplete data.

[ ] Add build step to the package
[ ] Store the datasets in scratch spaces

I think downloading the data to Scratch would be a good improvement.
Alternatively using the artifact system, but generally being lazy with the download is fine.


[Draft] Convert input media type from RBG to YUV

In this proof of concept code change we remove the implicit conversion from RGB32 to a format supported by H264 MF encoder.
MF H246 encoder support MFVideoFormat_IYUV format
https://docs.microsoft.com/en-us/windows/win32/medfound/h-264-video-encoder
In this code change the frame acquired from screen is converted to YUV and the passed to video encoder.
The result has the same image quality but CPU usage is reduced by ~50% and GPU usage is also reduced by ~30%.

Wow! That was a surprisingly large speedup. I guess we can live with a bit more complex implementation if reducing CPU & GPU load as much as 30-40%.
Have you checked if the speedup is consistent across Nvidia vs. Intel vs. no-GPU back-ends, or is it only one of them that is slow without this optimization?

It seems consistent across all the GPUs, but I guess it will not impact
significantly the CPU encoding performance. Also we hope to get even better
results with RGB2YUV implementation on GPU.
Il Gio 30 Set 2021, 15:36 Fredrik Orderud @.***> ha
scritto:

Wow! That was a surprisingly large speedup. I guess we can live with a bit
more complex implementation if reducing CPU & GPU load as much as 30-40%.
Have you checked if the speedup is consistent across Nvidia vs. Intel vs.
no-GPU back-ends, or is it only one of them that is slow without this
optimization?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/forderud/AppWebStream/pull/17#issuecomment-931329539,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADVKKHOLWNQFHY6CECORU73UERRUVANCNFSM5FCKJHLA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Could it be an idea to combine this PR with a ENABLE_RGB2YUV preprocessor define, so that it becomes simple to compare results with vs. without YUV conversion moving forward?
It seems to me like the code bloat associated with a ENABLE_RGB2YUV define would be quite small. This could be useful both for performance profiling and for color "correctness" comparison.


Build error on esbuild deps optimization with imports to cjs files inside externalized module

Describe the bug
tl;dr: Vite and esbuild are treating files in exterialized module 'external', but rollup is not. This difference causes rollup loads cjs file as esm and throws 'default' is not exported error.
I discovered this issue when working with ahooks. It should happen to all similar scenes.
ahooks has esm imports to files inside lodash, like import debounce from 'lodash/debounce';. However, 'lodash/debounce' is a cjs file which uses module.exports = debounce;. It needs to be compiled to esm for rollup to process.
When setting lodash to external without optimizeDep on build enabled, lodash/debounce is still bundled by rollup. This confirms that external option of rollup is not considering lodash/debounce external even if lodash is external.
When setting lodash to external with optimizeDep on build enabled, Vite (src/node/optimizer/esbuildDepPlugin.ts#L170, src/node/utils.ts#L97) and esbuild (option passed in src/node/optimizer/index.ts#L604, document quoted as below) treat lodash/debounce as external, leaving import statement and cjs file untouched, while rollup not. As a result, rollup loads the cjs file as esm and throws 'default' is not exported error.

(esbuild doc) In addition, if the external path looks like a package path (i.e. doesn't start with / or ./ or ../), import paths are checked to see if they have that package path as a path prefix.

Reproduction
https://github.com/zhyupe/vitejs-bug-20220805
System Info
Stackblitz
  System:
    OS: Linux 5.0 undefined
    CPU: (4) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
    Memory: 0 Bytes / 0 Bytes
    Shell: 1.0 - /bin/jsh
  Binaries:
    Node: 16.14.2 - /usr/local/bin/node
    Yarn: 1.22.10 - /bin/yarn
    npm: 7.17.0 - /bin/npm
  npmPackages:
    @vitejs/plugin-react: ^2.0.0 => 2.0.0
    vite: ^3.0.4 => 3.0.4

Used Package Manager
npm
Logs

Click to expand!
vite v3.0.4 building for production...
✓ 16 modules transformed.
'default' is not exported by node_modules/lodash/debounce.js, imported by node_modules/.vite/deps_build-dist/ahooks.js
file: /home/projects/vitejs-vite-vwthzv/node_modules/.vite/deps_build-dist/ahooks.js:937:7
935: // node_modules/ahooks/es/useRequest/src/plugins/useDebouncePlugin.js
936: var import_react9 = __toESM(require_react());
937: import debounce from "lodash/debounce";
            ^
938: var __read3 = function(o, n) {
939:   var m = typeof Symbol === "function" && o[Symbol.iterator];
error during build:
Error: 'default' is not exported by node_modules/lodash/debounce.js, imported by node_modules/.vite/deps_build-dist/ahooks.js
    at error (file:///home/projects/vitejs-vite-vwthzv/node_modules/rollup/dist/es/shared/rollup.js:1888:30)
    at Module.error (file:///home/projects/vitejs-vite-vwthzv/node_modules/rollup/dist/es/shared/rollup.js:12442:16)
    at Module.traceVariable (file:///home/projects/vitejs-vite-vwthzv/node_modules/rollup/dist/es/shared/rollup.js:12801:29)
    at ModuleScope.findVariable (file:///home/projects/vitejs-vite-vwthzv/node_modules/rollup/dist/es/shared/rollup.js:11453:39)
    at FunctionScope.findVariable (file:///home/projects/vitejs-vite-vwthzv/node_modules/rollup/dist/es/shared/rollup.js:6385:38)
    at ChildScope.findVariable (file:///home/projects/vitejs-vite-vwthzv/node_modules/rollup/dist/es/shared/rollup.js:6385:38)
    at FunctionScope.findVariable (file:///home/projects/vitejs-vite-vwthzv/node_modules/rollup/dist/es/shared/rollup.js:6385:38)
    at ChildScope.findVariable (file:///home/projects/vitejs-vite-vwthzv/node_modules/rollup/dist/es/shared/rollup.js:6385:38)
    at TrackingScope.findVariable (file:///home/projects/vitejs-vite-vwthzv/node_modules/rollup/dist/es/shared/rollup.js:6385:38)
    at BlockScope.findVariable (file:///home/projects/vitejs-vite-vwthzv/node_modules/rollup/dist/es/shared/rollup.js:6385:38)

Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

I have the same case on a new Vite + Vue project.
I installed Tailwind CSS and created a tailwind.config.cjs file for it.
I cannot import the config file because I get an:
Uncaught SyntaxError: The requested module '/app/tailwind.config.cjs?import' does not provide an export named 'default' (at App.vue:3:8)

Renaming to tailwind.config.js doesn't help because then I see the:
[plugin:vite:css] require() of ES Module frontend/tailwind.config.js from frontend/node_modules/tailwindcss/lib/lib/setupTrackingContext.js not supported.

Any ideas/tips on how to work around this for now?


Pruner doesn't see files in S3

Something wrong with the object listing:
2023-05-25 17:16:53.473 [rid:init0] DEBUG vdi.component.pruner.Pruner - deleting dataset 221741260/3C7CF1212ABE4F16805E620319B156CB from S3
2023-05-25 17:16:53.473 [rid:init0] DEBUG org.veupathdb.lib.s3.s34k.minio.BucketObjectContainer - Attempting to list sub-paths under prefix "vdi/221741260/3C7CF1212ABE4F16805E620319B156CB/" with delimiter "/" in bucket "Bucket{ name='some-other-bucket' }"
2023-05-25 17:16:53.476 [rid:init0] INFO  vdi.module.pruner.PrunerModuleImpl - automatic S3 pruning job 8 completed after 123ms

Resolved.


module 'youtube_dl' has no attribute 'YoutubeDL'

Checklist

[x] I'm asking a question
[x] I've looked through the README and FAQ for similar questions
[x] I've searched the bugtracker for similar questions including closed ones

Question

WRITE QUESTION HERE
I accidentally deleted some related files and now when I run "youtube_dl.YoutubeDL(ydl_opts)", it returns "module 'youtube_dl' has no attribute 'YoutubeDL'". Are there any way to reinstall youtube_dl so I can have every file I need?

Just reinstall the program as you originally did, or otherwise.

As dirkf sayed. If it was installed by python with pip (python -m pip install ... / pip instal ...) , then run pip again with "extra" flag --force-reinstall (from pip "manual": Before installing a package, will uninstall it first if already installed.) or simply
--ignore-installed (from pip "manual": Ignores whether the package and its deps are already installed, overwriting installed files)


Feature: add an assist to remove redundant function call arguments

Intellij Rust recently added an assist to remove redundant function call arguments. Rust-Analyzer doesn't support this today, but I think this could be a relatively easy addition to make. I think this would especially be helpful when doing mundane refactors across larger codebases.
Screenshots
intellij Rust today

We actually have a diagnostics for that already. We should add two quickfixes:

remove the argument at the call site
add parameter at the definition site


Restore CDAP Kafka

The version of Kafka shipped by Hortonworks isn't consistent even across HDP minor releases. CDAP requires Kafka 0.8, so we need to provide our own Kafka service. This supersedes #99 and is mostly a revert of #83 with additional changes to bring the service in-line with develop functionality.

[x] CDAP/CDAP_KAFKA service (metainfo.xml)

Define service
Update dependencies

[x] Alerts
[x] Configuration

User-modifiable
Automatically determined

[x] Kerberos
[x] Script (install/start/stop)
[x] Theme

minor comment, otherwise 👍 lgtm


Fix misleading error on opening empty file

@vmharvey this should fix the issue with opening empty files you reported in #282

Wouldn't it be a bit more efficient for is_eventio to return also the file type so that it is not necessary to read the marker a second time in the init of EventIOFile?

Ah yes, will open another PR


Successful tests don't return expected messages

I'm currently implementing BDD with Catch and when I run my tests I expect to see messages like the ones at the bottom of this page but instead I just get this:
$ ./ShogunTests
===============================================================================
All tests passed (4 assertions in 2 test cases)

Did you try running with --success ?

Perfect.

On 27 Jan 2015, at 07:59, James Wilkinson<EMAIL_ADDRESS>wrote:
Did you try running with --success ?
—
Reply to this email directly or view it on GitHub https://github.com/philsquared/Catch/issues/373#issuecomment-71598395.


moti svg typescript fix

fixes #264

Pulled in via https://github.com/nandorojo/moti/commit/4d5eeefdccf1b06de0c2251399c9710bfed8afc0 since some of the changes were already in 0.23.4


Get Ryan's feedback on TLS

@ryancdotorg has requested time to give feedback on the "tls" field spec in d/. We should wait for his feedback before finalizing.

Hello @ryancdotorg, any news on this?

FWIW, I would like to enforce a minimum of two hashes with 128 bit security level (256 + 256 bit output).  SHA-3 should be done sometime in the fall.
It should be noted that two hashes do not add up to a combined 256 bit level of security, the security is roughly as good as the strongest hash.

Where is the doc?

@ryancdotorg https://github.com/ifa-wg/proposals/blob/master/ifa-0001.md

@indolering I'm not 100% sure what you're proposing.  If necessary can you start another issue for that?

@indolering I'm not 100% sure what you're proposing. If necessary can you start another issue for that?

I don't know what I'm proposing either, I've just been nerding out on hash functions for a few weeks and thought it might be tangentially relevant.  FWIW the current FIPS 202 draft (SHA-3 standard for SHA-2 drop-ins) is basically final, it is literally awaiting the signature of the Secretary of Commerce.

I'm against any requirement that isn't already in the DANE spec.


HACK. prevent scroll in some links

caused by https://github.com/src-d/landing/issues/391
This quick and dirty hack is considered as a hotfix for #391
When applied, the 'scrollTo' behavior will be disabled for elements having a 'data-scroll' attribute set as 'no'
This PR should not be considered as a final solution for https://github.com/src-d/landing/issues/391, but an ugly hotfix.

ready for being reviewed at staging
https://landing-staging.srcd.run


Improve CLI upload / download experience

Features added:

Let the user select a subset of files to upload by list or glob
Prompt the user to try again if no files were selected for download / upload
Prompt the user to try again if the dataset was not selected correctly (if they forget to hit the tab key)
Let the user upload hidden files

Ready for your review, @nathanthorpe

I made another change that I wanted to make sure you were ok with, @nathanthorpe
When downloading files interactively, I'm removing the data/ prefix when asking the user to either select from a list or apply a glob. I think this is much more intuitive to how people use the platform.
Thoughts?

Yeah that sounds good!


npm dependencies update

Hi John,
first of all compliments for this useful project.
While installing I see that dependencies are out of date:
npm install
[ .................] | fetchMetadata: sill pacote range manifest for<EMAIL_ADDRESS>fetched in 1558ms
npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
npm WARN deprecated<EMAIL_ADDRESS>graceful-fs v3.0.0 and before will fail on node releases >= v7.0. Please update to<EMAIL_ADDRESS>as soon as possible. Use 'npm ls graceful-fs' to find it in the tree.
npm notice created a lockfile as package-lock.json. You should commit this file.
npm WARN<EMAIL_ADDRESS>No repository field.
npm WARN The package jsdom is included as both a dev and production dependency.

There is a nifty little node util called ncu that allows you to keep track of package updates:
npm install -g npm-check-updates
> ncu
Using .../package.json
[..................] - :
 jsdom                       10.1.0  →   11.3.0
 del                          2.2.2  →    3.0.0
 eslint                     ^3.19.0  →  ^4.10.0
 eslint-config-airbnb-base  ^11.2.0  →  ^12.1.0
 gulp-clean-css               3.3.1  →    3.9.0
 nock                        9.0.13  →   9.0.27
 run-sequence                 1.2.2  →    2.2.0

Run ncu with -u to upgrade package.json

Then you can do:
ncu -u
Using .../pure-form/package.json
[..................] | :
 jsdom                       10.1.0  →   11.3.0
 del                          2.2.2  →    3.0.0
 eslint                     ^3.19.0  →  ^4.10.0
 eslint-config-airbnb-base  ^11.2.0  →  ^12.1.0
 gulp-clean-css               3.3.1  →    3.9.0
 nock                        9.0.13  →   9.0.27
 run-sequence                 1.2.2  →    2.2.0

Upgraded .../pure-form/package.json

And updates are automatically written in the package.json.
A npm install should then give you the new packages.
Hoping to be of help,
-- Rob
PS:
I see a couple of small warnings as well:
npm WARN<EMAIL_ADDRESS>No repository field.
npm WARN The package jsdom is included as both a dev and production dependency.

First I think it's because the package is autoreferenced, the second is due to a double installation i guess...

A npm install should then give you the new packages.

Tested it before posting-- :p


Move Specimen Procedure Types to model repo

[ ] Move SpecimenProcedureType enum from procedures.py to the aind-data-schema-models repo. Make a new file called specimen_procedure_types.py
[ ] Add Tamoxifen Induction to the specimen procedure types list
[ ] fix procedures to import this enum from the models.

@mae this will need to be two PRs. One in the aind-data-schema-models to add the enum and add Tamofixen and one in this repo to delete the enum and fix the imports


Enable server side rendering

This patch enables server side rendering, allowing window access at runtime only if window defined.

Looks good :+1:

Can you check the failing test? Other than that :shipit:

@rrees can you rebuild the CI build? I really think it is timed out. Because the same test in previous green builds was flagged with red because of execution time (> 100 ms).

@szyablitsky If you check the CircleCI build you'll see there is a issue with an api variable not being present. I can check it out locally but I think it is related to change.

@rrees No, error with api variable does not reproduce every time. If you check all the builds, than you will see that API error is not permanent and the most repeating error is
  commands
    bold
      given content of "
1
"
        when queryState is executed for the command
          1) should return false

which errors because of timeout
1) commands insertOrderedList given content of "
1
" when queryState is executed for the command should return false:
     TypeError: Invalid locator
      at Function.webdriver.Locator.checkLocator (/home/ubuntu/scribe/node_modules/scribe-test-harness/node_modules/selenium-webdriver/lib/webdriver/locators.js:247:9)
      at webdriver.WebDriver.findElement (/home/ubuntu/scribe/node_modules/scribe-test-harness/node_modules/selenium-webdriver/lib/webdriver/webdriver.js:880:33)
      at /home/ubuntu/scribe/test/commands.spec.js:34:27
      at promise.ControlFlow.runInFrame_ (eval at <anonymous> (/home/ubuntu/scribe/node_modules/scribe-test-harness/node_modules/selenium-webdriver/lib/goog/base.js:1124:19), <anonymous>:1857:20)
      at goog.defineClass.notify (eval at <anonymous> (/home/ubuntu/scribe/node_modules/scribe-test-harness/node_modules/selenium-webdriver/lib/goog/base.js:1124:19), <anonymous>:2448:25)
      at promise.Promise.notify_ (eval at <anonymous> (/home/ubuntu/scribe/node_modules/scribe-test-harness/node_modules/selenium-webdriver/lib/goog/base.js:1124:19), <anonymous>:564:12)
      at Array.forEach (native)
      at promise.Promise.notifyAll_ (eval at <anonymous> (/home/ubuntu/scribe/node_modules/scribe-test-harness/node_modules/selenium-webdriver/lib/goog/base.js:1124:19), <anonymous>:553:15)
      at goog.async.run.processWorkQueue [as _onTimeout] (/home/ubuntu/scribe/node_modules/scribe-test-harness/node_modules/selenium-webdriver/lib/goog/async/run.js:130:15)
      at Timer.listOnTimeout [as ontimeout] (timers.js:112:15)

It has similar problems in previous green builds
      given content of "
1
"
        when queryState is executed for the command
          ✓ should return false (110ms)

I'd be happy to dig it down, but I'm not so strong in front-end development ant testing. I cloned repo locally and tried to run the tests, but can't manage them to even run. My main field of expertise is Ruby/Rails.

@rrees sorry to disturb, but maybe you have time to check this test?

@szyablitsky Yep, I think I have a working test suite that I need to create a PR for. I think there are issues with the latest Chrome.
If I get that working then I will either merge this or combine the two changes on server-side rendering into one PR.

@rrees thank you! Will be waiting for good news.

@szyablitsky can you check that Scribe 2.1.0 does what you want?

@rrees Sorry, did not notice the window parameter in the function while reviewing. Server side rendering still does not work.


Two different highlight styles/introduce cursor

Hi there, for my use case (home automation tui) I want to be able to select two or more items in the tree. I would like to have a difference between the current position in the tree and all the other selected items.
The current position is a highlighted item. By pressing  or  the highlighted item changes, its a sort of cursor. Using another key I can then select or unselect the currently highlighted item.
I can achieve this by rebuilding the try and styling/changing the text of different entries.
I see two possible api's:

Add a highlight style/symbol to TreeState::select.
Introduce the concept of a cursor which is moved by TreeState::key_up/TreeState::key_down then add TreeState::select_current/TreeState::unselect_current  to add/remove the item that is currently the cursor to the selected items.

The first API gives more freedom to users, it could be used to do other things have n-different selection's for example. This comes at the cost of requiring users to do the bookkeeping of which selected item is the cursor vs a 'normal' selected item. The second makes my usecase trivial to implement but its the only additional usecase supported.
The best, and most work, might be to implement both the concept of differently styled selected items and a cursor.
Let me know what you think.

Just as a heads-up: I can take a look on this in a few days as I don’t have that much time this weekend.
Also, TUIs are currently low on my priority until https://github.com/EdJoPaTo/mqttui/pull/172 gets resolved. Until ratatui fixes the bug I am stuck at ratatui ≤ 0.26.


[WIP] migrate API v2

[x] ls
[x] find
[x] copy
[x] move
[x] mkdir
[x] delete
[x] upload
[x] download
[x] sync
[x] help

Thanks! Did a once-over for user-facing strings.

Very Thank you, was corrected immediately.
https://github.com/s-aska/dropbox-api-command/commit/352a3ffd70996aae6dbc42caaaf07b0a7fee3338


License problem when pushing model

Hi,
I am trying to push my spaCy model to the Huggingface Hub and I get the following error:
Traceback (most recent call last):
  File "/home/elena/Workspace/Spacy_project/.env3/lib/python3.8/site-packages/huggingface_hub/repository.py", line 412, in git_push
    result = subprocess.run(
  File "/usr/lib/python3.8/subprocess.py", line 516, in run
    raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command '['git', 'push']' returned non-zero exit status 1.

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/lib/python3.8/runpy.py", line 194, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/usr/lib/python3.8/runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "/home/elena/Workspace/Spacy_project/.env3/lib/python3.8/site-packages/spacy/__main__.py", line 4, in <module>
    setup_cli()
  File "/home/elena/Workspace/Spacy_project/.env3/lib/python3.8/site-packages/spacy/cli/_util.py", line 69, in setup_cli
    command(prog_name=COMMAND)
  File "/home/elena/Workspace/Spacy_project/.env3/lib/python3.8/site-packages/click/core.py", line 829, in __call__
    return self.main(*args, **kwargs)
  File "/home/elena/Workspace/Spacy_project/.env3/lib/python3.8/site-packages/click/core.py", line 782, in main
    rv = self.invoke(ctx)
  File "/home/elena/Workspace/Spacy_project/.env3/lib/python3.8/site-packages/click/core.py", line 1259, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/home/elena/Workspace/Spacy_project/.env3/lib/python3.8/site-packages/click/core.py", line 1259, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/home/elena/Workspace/Spacy_project/.env3/lib/python3.8/site-packages/click/core.py", line 1066, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/home/elena/Workspace/Spacy_project/.env3/lib/python3.8/site-packages/click/core.py", line 610, in invoke
    return callback(*args, **kwargs)
  File "/home/elena/Workspace/Spacy_project/.env3/lib/python3.8/site-packages/typer/main.py", line 497, in wrapper
    return callback(**use_params)  # type: ignore
  File "/home/elena/Workspace/Spacy_project/.env3/lib/python3.8/site-packages/spacy_huggingface_hub/push.py", line 53, in huggingface_hub_push_cli
    push(whl_path, organization, commit_msg, local_repo_path, verbose=verbose)
  File "/home/elena/Workspace/Spacy_project/.env3/lib/python3.8/site-packages/spacy_huggingface_hub/push.py", line 131, in push
    url = repo.push_to_hub(commit_message=commit_msg)
  File "/home/elena/Workspace/Spacy_project/.env3/lib/python3.8/site-packages/huggingface_hub/repository.py", line 434, in push_to_hub
    return self.git_push()
  File "/home/elena/Workspace/Spacy_project/.env3/lib/python3.8/site-packages/huggingface_hub/repository.py", line 422, in git_push
    raise EnvironmentError(exc.stderr)
OSError: remote: ----------------------------------------------------------
remote: Sorry, your push was rejected:
remote: - error : yaml metadata schema issue on key "license"
remote: help: key: /license must be equal to one of the allowed values (error rule: enum), please find the license identifier that fits you at https://huggingface.co/docs/hub/model-repos#list-of-license-identifiers
remote: ----------------------------------------------------------
remote: Please find the documentation at:
remote: https://huggingface.co/docs/hub/model-repos
remote: ----------------------------------------------------------
To https://huggingface.co/nucci/sv_pipeline
 ! [remote rejected] main -> main (pre-receive hook declined)
error: failed to push some refs to 'https://huggingface.co/nucci/sv_pipeline'

It seems to be a license problem, so I put the identifier corresponding to the license that I want to give my model in meta.json, but it doesn't seem to solve the problem. Any thoughts?

It actually worked when I changed the license identifier in the folder meta.json before building the wheel.


Implements reading ability for PLC-5

This branch implements the ability to read (and maybe write in the future) data from/to PLC-5 and PLC-5/250 series of PLCs. It does so by implementing the PLC-5 command set, and allowing it to be chosen at constructor time, enabling further command sets to be developed. It defaults although to the existing SLC command set, so backwards compatibility is maintained.
This is still a Work In Progress: It has been only tested with a couple of tags against a single PLC, so more testing is very welcome in case anybody has access to such PLCs.

My idea with the draft PR was to inform about the development, and allow anyone that has access to a PLC-5 (and is watching this repo) to be able to test on their systems and report any possible issue. I wouldn't merge it yet, as I've tested with a single CPU and just a couple of tags. The docs are also not much clear about the meaning of some fields, another reason to test it better before merging.
It would also be nice to implement the writing command as well before merging, or at least fail gracefully instead of just sending the same wrong command, the current behavior.

Do the response packets from the PLC-5 look identical, at least in your testing?

Luckily yes, at least according to the docs and for the couple tags I've tested. Again, more testing could expose some edge cases I'm not aware of.
Regarding the docs, I agree we need to update them, adding an example of the new option and the supported PLCs. As I'll still do some changes to the code, I can change the docs too.
Thanks for your time reviewing the code!


After tagging, can we edit the tag?

Let the tagging be easier and like Facebook if we've tagged someone we should even be able to remove their last name or first name or even edit the tag @daniel zahariev should be possible to be written as "daniel" or "zahariev " or any other thing we choose.

+1


Magic123

Magic123 coarse stage implementation with the new unified zero123 and SD guidance.

https://github.com/threestudio-project/threestudio/assets/19284678/bf9f6772-93ae-4385-b6b1-46ccd662068f

https://github.com/threestudio-project/threestudio/assets/19284678/a721eb06-a64b-4040-b7d3-14522e6ee26d

https://github.com/threestudio-project/threestudio/assets/19284678/9ac0d4b1-9015-41d0-ba0f-f8c9674177b4

Great work! I tried this simple prompt:
python launch.py --config configs/magic123.yaml --train --gpu 0 data.image_path=/path/to/dragon_rgba.png system.renderer.num_samples_per_ray=256 data.width=64 data.height=64
but got the following error:
MissingMandatoryValue: Missing mandatory value: tag
    full_key: tag
    object_type=dict

Do you have an idea how to resolve this? Thanks!

@yankeesong you also need to specify system.prompt_processor.prompt

Oh got it. So currently textual inversion is not supported right?

Yes and I don't plan to support it :)

Thanks for integrating our Magic123 into threestudio !


Revert "temporarily disable retries"

Reverts redhat-developer/consoledot-e2e#106

DO NOT MERGE! We will leave the retries off to collect enough data over the weekend.


Passed parameter is not a string

After enabling the xssfilter on some of my events i get the following error after a 'suposed' successfull save;
'Passed parameter is not a string' (extensiondriver.php line 113) (screenshot)
When entering mallicious code in the textfield it correctly returns the 'filter/@status = 'failed' message in the result xml.
(issue on symphony forum)

Looks like it's the image upload field that is giving the error


NETOBSERV-750: fix to enable being in the 4.12 catalog

Breaking change: Updating some versions of the API forces users to have Openshift 4.10 (Kube API 1.23) at least.

Improved the bundle verification script to include the same tests done at the operatorframework CI.
If there are warnings, bundle verification will make the make bundle task to fail.
Fixed bundle verification warnings:

Updated k8s.io/api/autoscaling/v2beta2 to k8s.io/api/autoscaling/v2
Updated operators.coreos.com/v1alpha1 to operators.coreos.com/v1 for ClusterServiceVersion
Added minKubeVersion: 1.23
Added resources request to Kube RBAC proxy.

Reverted linter update as prow was complaining. Updated prow and will update linters in another PR https://github.com/openshift/release/pull/34809

wow, thanks @mariomac for all these findings, i'll have a closer look tomorrow!

/ok-to-test

/lgtm

/lgtm

when this is merged we can release 0.2.3 to fix the issue on operatorhub

/ok-to-test


training error with tensorflow 1.5

Tried again previously successful training with new tensorflow release 1.5, got following error and training exits:
[0202 08:51:45 @base.py:225] Graph Finalized.
2018-02-02 08:51:48.689557: W tensorflow/core/kernels/queue_base.cc:285] _0_QueueInput/input_queue: Skipping cancelled dequeue attempt with queue not closed
[0202 08:51:48 @concurrency.py:36] Starting EnqueueThread QueueInput/input_queue ...
[0202 08:51:48 @graph.py:70] Running Op sync_variables_from_main_tower ...
[0202 08:51:53 @input_source.py:149] ERR Exception in EnqueueThread QueueInput/input_queue:
Traceback (most recent call last):
  File "/home/dsu/tf3/lib/python3.5/site-packages/tensorpack/input_source/input_source.py", line 141, in run
    feed = dict(zip(self.placehdrs, dp))
TypeError: zip argument #2 must support iteration
[0202 08:51:54 @input_source.py:155] EnqueueThread QueueInput/input_queue Exited.
[0202 08:52:05 @param.py:144] After epoch 0, learning_rate will change to 0.00333333
[0202 08:52:06 @base.py:245] Start Epoch 1 ...
  0%|                                                                                |0/500[00:00<?,?it/s][0202 08:52:26 @base.py:261] Training was stopped.

MultiProcessMapDataZMQ successfully cleaned-up.

Has tensorpack updated to be compatible with tf1.5?
Installed tensorpack version is 0.8.0. Saw there is new v0.8.1, updating and retrying.

Tensorpack is compatible with tf1.5.
To report an unexpected problem, please follow the issue template.

Thx. Update to tensorpack 0.8.1 resolved the issue.


Mark txid nf spent error

Note that three tests are now failing at an explicit todo! macro invocation. I believe this to be the pre-existing error, that's now failing early instead of silently doing the wrong thing.

I see that only 17 out of 18 checks passed.   Why was this merged if a check was not passing?
@fluidvanadium ?

oops! i assumed they all passed because that used to be the merge requirement


Do not target random generated classes - script error

Currently to target elements you use random generated classnames that instagram is rerolling from time to time like: h1._rf3jb. This class does not exist any more which causes puppeteer to error and whole script to die. Please target specific tags or sections, do not rely on classes

Thanks you for warning me. Yes I know they are generated but I couldn't see any other way to recover the element. You can create a PR if you have a solution.

There is no perfect solution for some of the items but main ones should be doable. I will drop a PR soon


[Microwin: Docs] Remove Customisation

Pull Request
Microwin: Remove Customisation docs
Type of Change

[x] Documentation update

Description
same as title, not needed anymore
Additional Information
was mentioned in Community Discord Server
Checklist

[x] My code adheres to the coding and style guidelines of the project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have made corresponding changes to the documentation.
[x] My changes generate no errors/warnings/merge conflicts.

@CodingWonders thanks for noticing, I didn't look into the actual changes of your PR yet.


docs/add-julia-kernel-install-instructions

Fixes #332. Added install instructions for IJulia in HOWTO.

I wonder if we should also detect that julia kernel is not initialized and print a warning and point to this documentation.


ValueError: Cannot use 'roi_align_1/map/while/strided_slice_20'

ValueError: Cannot use 'roi_align_1/map/while/strided_slice_20' as input to 'roi_align_1/map/while/zeros/packed' because 'roi_align_1/map/while/strided_slice_20' is in a while loop. See info log for more details.
when running this
maskrcnn-modanet processimage save image -u https://static.highsnobiety.com/thumbor/XZwWwq9iqiAHOblIR8Jqt5yuBis=/fit-in/1200x720/smart/static.highsnobiety.com/wp-content/uploads/2019/08/05125216/serbian-streetwear-scene-feat2.jpg
LOVE this project
i am having lots of issues running locally
i have python 3.7
virtualenv
tensorflow downgraded to 1.13.1 for comaptability

this happens with the google collab notebook as well

Did you find any solution??

Tensorflow=1.13.1 , keras==2.2.4, keras-retinanet==0.5.1
This should solved the same problem on colab for me. Beware sometimes on colab even if you specify the right keras version it doesn't install it but install the latest one so you should rerun the pip command again.


bitcoin: 23.0 -> 23.2

Description of changes

update to latest point release
https://github.com/bitcoin/bitcoin/blob/master/doc/release-notes/release-notes-23.1.md
https://github.com/bitcoin/bitcoin/blob/master/doc/release-notes/release-notes-23.2.md

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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/)
23.05 Release Notes (or backporting 22.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.

@GrahamcOfBorg build bitcoin bitcoind

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/what-does-it-mean-for-an-old-nixos-release-to-be-deprecated/28765/5


⚠ 한경주요지수-코스피200 has degraded performance

In 07f7c78, 한경주요지수-코스피200 (https://markets.hankyung.com/indices/kospi200) experienced degraded performance:

HTTP code: 200
Response time: 7352 ms

Resolved: 신규상장 performance has improved in 5c81bb6.


incorrect d-acts import

I just realized that the import PURL for d-acts in OMRSE doesn’t point anywhere. I probably at some point transmitted a faulty PURL. Sorry for that.
The PURL of the version OMRSE uses should be:
http://purl.obolibrary.org/obo/iao/d-acts/20150916/d-acts.owl

Deleted d-acts import. Imported IAO and PNO directly and imported classes from the d-act file above.


SSL Support

Since we're handling credentials, SSL will need to be enabled (ideally with HTTP/2)

PR #61 fixes this.

#61Added ssl support  in  kontinuous-cli deploy

Changes:

Kontinuous api and dashboard as containers in one deployment and service resource.
kontinuous-cli deploy. Generate ssl certs and keys for kontinuous api and dashboard.


decouple isWifiOn and toggleWifi

To implement WIFI actions in UIAutomator2 Driver.
 Doesn't seem to be related to this though.

Yeah issue with CI, ran locally and looks good.

published in 1.10.36.


Fix division by zero in pricing-import-service

I'm not sure whether it's correct to quote in the other direction instead of $1/amount$ (might be different buy/sell prices??).


Devops for data blog

adding medium blog to our site

@andronovhopf let's add both of these links to the exclusion list (they don't allow to test them automatically):
ERROR:405:https://towardsdatascience.com/why-git-and-how-to-use-git-as-a-data-scientist-4fa2d3bdc197
ERROR:503:https://www.amazon.com/DevOps-Handbook-World-Class-Reliability-Organizations-ebook/dp/B01M9ASFQ3


Error: undefined is not an object (evaluating 'element.nodeType')

I often get this error when changing routes (I am using ngRoute).
The route's templateUrl loads fine, but all directives contained in the template don't load.
Browser: Safari 8.0.8 on OSX
Stack trace:
[Error] Error: undefined is not an object (evaluating 'element.nodeType')
append@http://localhost/netflix/app/bower/angular/angular.js:3407:27
http://localhost/netflix/app/bower/angular/angular.js:3532:19
startingTag@http://localhost/netflix/app/bower/angular/angular.js:1285:40
invokeLinkFn@http://localhost/netflix/app/bower/angular/angular.js:8748:41
nodeLinkFn@http://localhost/netflix/app/bower/angular/angular.js:8246:23
compositeLinkFn@http://localhost/netflix/app/bower/angular/angular.js:7637:23
publicLinkFn@http://localhost/netflix/app/bower/angular/angular.js:7512:45
boundTranscludeFn@http://localhost/netflix/app/bower/angular/angular.js:7656:28
controllersBoundTransclude@http://localhost/netflix/app/bower/angular/angular.js:8273:35
update@http://localhost/netflix/app/bower/angular-route/angular-route.js:935:36
$broadcast@http://localhost/netflix/app/bower/angular/angular.js:16200:33
http://localhost/netflix/app/bower/angular-route/angular-route.js:619:36
processQueue@http://localhost/netflix/app/bower/angular/angular.js:14634:30
http://localhost/netflix/app/bower/angular/angular.js:14650:39
$eval@http://localhost/netflix/app/bower/angular/angular.js:15878:28
$digest@http://localhost/netflix/app/bower/angular/angular.js:15689:36
$apply@http://localhost/netflix/app/bower/angular/angular.js:15986:31
http://localhost/netflix/app/bower/angular/angular.js:12165:30
eventHandler@http://localhost/netflix/app/bower/angular/angular.js:3293:25

Can you please provide a minimal demo on plnkr.co or similar?

Closing because we don't have a reproduce scenario.

angular 1.5.11 + webpack, iOS simulator  with  Hardware/device/ios 8.1/iphone 5s.
found :

been wondering why 'this[i]=== undefined' ?  Should jquery be introduced? Need to try~

It is not expected for this[0] to be undefined. It can only happen if this.length is not properly updated (and this shouldn't happen normally).
We still need a reproduction 😕

Wow this is old. This bug was specific to Safari 8, I've never seen it in any later version of Safari. Upgrading my browser a few years ago fixed the issue.

On Jul 31, 2017, at 6:29 AM, George Kalpakas<EMAIL_ADDRESS>wrote:
It is not expected for this[0] to be undefined. It can only happen if this.length is not properly updated (and this shouldn't happen normally).
We still need a reproduction 😕
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Cannot connect to Redshift database with same driver as SQL Workbench/J

Read contribution guide before submitting ticket:
https://github.com/dbeaver/dbeaver/blob/devel/CONTRIBUTING.md

This is an empty file. Please remove this template or update the CONTRIBUTING.md file.
I recently learned about the existence of your program, but apparently it doesn't work.
I configured dbeaver in exactly the same way as SQL Workbench/J (also provided the correct jdbc42 driver, etc.), but I got the following response:

The expected response is a connection to the database.
Technical data
version: DBeaver 5.0.5

system: "x86_64-linux"
host os: Linux 4.15.18, NixOS, 18.03.git.ee80654 (Impala)
multi-user?: yes
sandbox: no
version: nix-env (Nix) 2.0.1
channels(root): "nixos-17.09.2182.7f6f0c49f0"
channels(coretemp): "nixos-18.03.132192.ce0d9d638de"
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos/nixpkgs

What exactly driver did you use (please post link on AWS resource?)
Did you try the Redshift driver preconfigured in DBeaver?

@serge-rider

What exactly driver did you use (please post link on AWS resource?)

On https://docs.aws.amazon.com/redshift/latest/mgmt/configure-jdbc-connection.html JDBC 4.2–compatible driver: https://s3.amazonaws.com/redshift-downloads/drivers/jdbc/<IP_ADDRESS>7/RedshiftJDBC42-<IP_ADDRESS>7.jar.

Did you try the Redshift driver preconfigured in DBeaver?

Yes and it didn't work. There is a reason I was using 4.2 with SQL Workbench/J too. I don't recall that reason, but I also know that 4.1 does not work.

Please post error stacktrace (you can get it in Error Log view or in file - https://github.com/dbeaver/dbeaver/wiki/Log-files)

@serge-rider It's just a usability issue; I hadn't selected the right class to use via the standard UI workflow that is available. I don't know whether it's a bug, because I think the other way should also work.
I have access now. Thanks.
I am closing this now, but you might want to test the other flow too.

Ok, good.
In any case I'll update Redshift driver version in DBeaver config.


Create pipeline to check the links in docs

Proposed changes

When PR opened, this pipeline will check every links in markdown files.
New issue will created after pipeline finished running - With the result of link coverage. (Sample issue)
Add link coverage badge to README

Types of changes
Please put an x in the boxes related to your change.

[ ] Minor Issues and Typos
[ ] Major Content Contribution
[x] Others

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to reach out. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[x] I have read the CONTRIBUTING GUIDELINES
[x] I have signed the CLA
[x] I have added necessary documentation (if appropriate)
[ ] Any dependent changes have been merged and published in downstream modules

Related issues
Further comments

I have read the CLA Document and I hereby sign the CLA

@iv0rish Can you see the CI failure?

@iv0rish Can you see the CI failure?

It's normal because pipeline can't runs on forked repository. I've attached a sample result running on forked branch.

@iv0rish If it is good to go, please merge this

No longer applicable due to docs platform change.


ci: update clang to version 6.0 in the Ubuntu build image.

Risk Level: Low
Testing: Successful build using new image.
Docs Changes: n/a
Release Notes: n/a
Signed-off-by: Piotr Sikora<EMAIL_ADDRESS>

cc @mattklein123

Note: you will need to update ci/build_setup.sh in a follow up PR along with bumping image SHA.

I know, I have it staged locally... I should probably send it out for review as well...

@lizan found https://llvm.cpp.fail/, so I'm updating the CentOS image as well.


Ability to export graph as image (JPEG/PNG)

Please describe the feature that you want to propose
It would be of great use if there is an ability to export the graph as an image for the end users.

@moklick we can't dowload full graph with html-to-image

I answered here: https://github.com/wbkd/react-flow/issues/2118#issuecomment-1556766899


Address the use of URIs in relation to Sonar rules

Description
As it turns out, Steeltoe accepts URLs that are discarded by System.Uri, for example:
mongodb://a9s-brk-usr-e74b9538:a9sb8b69cc@mongodb-0.node.dc1.a9s-mongodb-consul:27017,mongodb-1.node.dc1.a9s-mongodb-consul:27017,mongodb-2.node.dc1.a9s-mongodb-consul:27017/d5584e9?replicaSet=rs0

which fails with:
System.UriFormatException: Invalid URI: Invalid port specified.

That makes it dangerous to activate the next rules, which is why I've kept them deactivated:

S3994: URI Parameters should not be strings
S3995: URI return values should not be strings
S3996: URI properties should not be strings

This PR fixes the following rules:

S3997: String URI overloads should call "System.Uri" overloads
S4005: "System.Uri" arguments should be used instead of strings

In addition, it renames namespaces that contain "Task" to "Tasks" to avoid a clash with System.Threading.Tasks.Task and it reduces the use of  task.GetAwaiter().GetResult() in tests.
Fixes #945
Quality checklist

[x] Your code complies with our Coding Style.
[x] You've updated unit and/or integration tests for your change, where applicable.
[ ] You've updated documentation for your change, where applicable.
If your change affects other repositories, such as Documentation, Samples and/or MainSite, add linked PRs here.
[x] There's an open issue for the PR that you are making. If you'd like to propose a new feature or change, please open an issue to discuss the change or find an existing issue.
[ ] You've added required license files and/or file headers (explaining where the code came from with proper attribution), where code is copied from StackOverflow, a blog, or OSS.

Changes generally look good to me, but I don't think it should close #945 though since it only addresses a portion of it.
Are there violations of S3994-3996 outside of Connectors that could be fixed?

@Tim as we discussed, I've created https://github.com/SteeltoeOSS/Steeltoe/issues/1069 to separately address S3994-3996.


Missing attributes for all nodes in 7.0.0

Checklist:

[x] I am not using HomeAssistant. Or: a developer has told me to come here.
[x] I am not using ZWaveJS2MQTT. Or: a developer has told me to come here.
[x] I have checked the troubleshooting section and my problem is not described there.
[x] I have read the changelog and my problem was not mentioned there.

Describe the bug
supportsBeaming, supportsSecurity, and nodeType do not get included in my state dump for any nodes in zwave-js-server when using zwave-js 7.0.0-beta1. You can see here where these properties are set: https://github.com/zwave-js/zwave-js-server/blob/master/src/lib/state.ts#L329-L331 . I can see properties like supportedDataRates and maxDataRate as well as zwavePlusNodeType and zwavePlusRoleType which makes me think these properties stay undefined after being initialized here: https://github.com/zwave-js/node-zwave-js/blob/master/packages/zwave-js/src/lib/node/Node.ts#L933-L935
Device information
Which device(s) is/are affected (make/model)?  all nodes
What are the node IDs?
Did you change anything?

[ ] Yes: (please describe)
[x] No

Did this use to work before?

[ ] Don't know, this is a new device
[ ] No, it never worked anywhere
[x] Yes, in: zwave-js 6.6.3 and below

How are you using node-zwave-js

[ ] zwavejs2mqtt (latest) docker image
[ ] zwavejs2mqtt (dev) docker image
[ ] zwavejs2mqtt Manual Docker build

node-zwave-js branch:
zwavejs2mqtt branch:

[ ] ioBroker.zwave2 adapter
[ ] HomeAssistant version XYZ
[ ] Pkg
[ ] Manual Docker build

node-zwave-js branch:
zwavejs2mqtt branch:

[ ] Manually built (as described in the docs)
[x] Other: running zwave-js-server (on this branch until my PR gets merged: https://github.com/raman325/zwave-js-server/tree/more_schema_3)

To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '...'
Scroll down to '...'
See error

Additional context
Add any other context about the problem here.
Logfile:

I don't have the log file from zwave-js at the moment because I am running this branch in a Home Assistant addon (https://github.com/marcelveldt/hassio-addons-repo/tree/master/zwave-js-server) which doesn't support logging at the moment. I have included my state dump though.
schema_1.json.txt

Did you interview the node prior to 7.0.0? Have you tried re-interviewing it?
Some of that information wasn't available before, so it might not be in your cache.

Also using Beta 1,
and I have the info (after being initialized, and after an interview)

Did you interview the node prior to 7.0.0? Have you tried re-interviewing it?
Some of that information wasn't available before, so it might not be in your cache.

Ah, I did not try reinterviewing. Will give that a shot and reopen this if I'm still missing the data.


Use Slot API pattern in KaigiTopAppBar for logo image

Idea Description
We need to show the title instead of the logo in some screen.
About slot API pattern, Please refer to this.
https://github.com/DroidKaigi/conference-app-2022/issues/114
Reference images and links
fun KaigiTopAppBar(
    onNavigationIconClick: () -> Unit,
    modifier: Modifier = Modifier,
    elevation: Dp = 0.dp,
    title: (@Composable RowScope.() -> Unit)?, // ← new
    trailingIcons: (@Composable RowScope.() -> Unit)? = null,

I'll try this 🙋♀

Thanks! Assigned 👍


Reset testResults values when no longer rendered

Task
In the form for ReportFormPage, there are two radio groups that are conditionally rendered based on the value of testStatus in form state.
These conditional form fields affect the values for testResults in form state.
When a user selects an option from the conditionally rendered fields (testResults) and then changes the testStatus field option to a value that no longer renders the conditional fields, testResults values should be reset.
Notes
// Not rendered and no options chosen:
testResults: {
              resultsSwab: null,
              resultsAnti: null,
            }

// Conditionally rendered and option chosen:
testResults: {
              resultsSwab: "test_swab_pos",
              resultsAnti: null,
            }

// Previous field option changed and no longer rendered (values in form state reset):
testResults: {
              resultsSwab: null,
              resultsAnti: null,
            }

This is now working, but it presents another small bug that involves validation firing before values are updated. Throws a validation error unless user reselects option twice.


Iteration in URN stages

In RFC5141 I found reference to stage iterations:

urn:iso:std:iso:128:-71:stage-30.98.v2:ed-1:en
both refer to the cancelled (30.98 = project deleted) 1st edition
of ISO 128-71, in English; the second example refers specifically
to the 2nd iteration (projectversion) at stage 30

But we don't use iterations for URN stages yet. Should we implement it? How do we determine stage iteration?
How does it reflect in PubID?

This is like ISO CD 1234.2 (the 2 is the iteration). In a published document, the "iteration number" does not appear in the pubid.

@ronaldtse seems it's implemented already:
https://github.com/metanorma/pubid-iso/blob/46516de6756d8bb198e26f41b5e301d2db93c688/spec/pubid_iso/identifier_parsing_spec.rb#L109-L115


iOS: Returning to app on iPad reloads story

From https://getsatisfaction.com/newsblur/topics/different-behaviour-ipad-vs-iphone-switching-windows:
I've noticed that when switching windows/apps on iPhone away from NewsBlur, for example if I open a link from a post in Safari, and then switch back to NewsBlur, it does not re-load the post, i.e. I'm back to exactly to where I had scrolled, or embedded video remains paused at the same spot. This is great.
However on iPad, switching back to the NewsBlur app causes the post to refresh, losing my place or resetting the embedded video, which is annoying.
Is there a way to get the app to work on iPad like it does on iPhone?

I'm working on another client project at present, but will see what I can do after that, probably next week sometime.

Works great for me, thanks!


Deprecate Twitter package and Lambda

Description

There's really no need for it at the moment and I don't want to spend time fixing it.
Specifics

[ ] delete twitter package
[ ] remove postperformance Lambda
[ ] remove required Lambda CloudFormation definition/resources

Closing in favor of moving to ad hoc Google Doc


Feature: Configure per-request timeout setting via env[SERVICE_TIMEOUT_KEY]

First, thanks for the awesome middleware!
I need the ability to customize the timeout value on a per-request basis, determined by some external dynamic data. This seemed like the most straightforward way to accomplish that.
Let me know if this is something that could get merged, and if so what kind of tests and documentation would be necessary.
FWIW I smoke tested this patch in my project and it works as intended.

@TSMMark Can you show us how you have used this in your tests?
Show an example of what changes you made to the conifg and parameters to get this working.

Hello and thank you for your interest and contribution. Unfortunately this is a feature that has been added by many different contributors but has been rejected many times as it goes against the core philosophy of the middleware. The general suggestion is any requests that need to run for more than your service_timeout need to be moved to a background system. You can easily impact the performance of the rest of your application by allowing some requests to run longer.
In addition, changing an environment variable on each request is also not a great approach as it is not safe in a concurrent environment, and environment variables are generally best left unchanged.

@wuputah Please review the code and not just the PR title. We're only modifying the Rack request env, which is not the same as global ENV, and AFAIK is per-request object and is safe to be modified at will to contain request-specific information.
I do not agree that this is against the philosophy of the middleware. As a developer you could already easily segment out different portions of your routes into different sinatra / cuba / roda instances, each with their own individual middleware (and individual timeout settings). This is just a different way to do that.
Please consider reopening the PR. I will be happy to add specs

Following the history of this topic,
#160 was a bad implementation
#138 was a bad implementation
#110 was a bad implementation
etc, etc

So, in short, this lets people shoot themselves in the foot either a bit more or a bit less, but rack-timeout is all about foot-shooting anyway, so does it really matter?

Great point, sounds like we should introduce this feature since tens of people are asking for it and it doesn't cost anything.

Ah, sorry, I misread this line as ENV.fetch rather than env.fetch.
 service_timeout_env = env.fetch(ENV_SERVICE_TIMEOUT_KEY, service_timeout)

So you are setting this in your app via request.env[ENV_SERVICE_TIMEOUT_KEY]  before the middleware fires? Are you using another middleware inserted in the stack to do this? It's certainly a minimal approach as far as impact to Rack Timeout, but maybe not very user friendly.

#110 was the last time this discussion happened in depth. Ultimately other PRs were not necessarily closed due to implementation or design, but ultimately it was a decision to not implement variable timeouts in Rack Timeout. After all the discussions that have happened, it's pretty ingrained in my mind not to merge a PR for this feature. 😂 😭
Additionally, the feature does exist (at least via one implementation) in slowpoke for those that need it.

Are you using another middleware inserted in the stack to do this

Yes

It's certainly a minimal approach as far as impact to Rack Timeout, but maybe not very user friendly.

Yeah exactly, I didn't want to impact performance, footprint, or maintainability of this library, while giving users who really need this functionality an escape hatch to do so.

slowpoke

Yeah slowpoke seems to be pretty Rails-specific, which I am not using.

So due to the limited impact on this library would you consider a merge or am I going to have to maintain my fork indefinitely?

@wuputah Wouldn't it be resonable to offer something like this? It is even more minimal than #110.
I don't think the arguments in #110 hold up, you said it yourself at https://github.com/sharpstone/rack-timeout/issues/169#issuecomment-806077437

Additionally, while the application may enter a bad state when using Rails + ActiveRecord, the middleware does not know what you are using -- your app might only be making network calls, for instance.

Isn't the above an argument for legitimate use-cases of rack-timeout? (not just using it as a debugging tool) Even if using it as a debugging tool, offer this flexibility could help IMHO.

@TSMMark It might be best to add all the specs and steps and update the docs with steps to show how you used it and then, if possible, have a version with it and one without, so users can fork the one linked to this without having to maintain it.

Just pushed a commit that adds a test. Although the example is contrived and it's not probably not a good idea to use a query param for this, it was an easy way to add a dynamic element other than path.
https://github.com/sharpstone/rack-timeout/commit/2dc21dd350016e035f01500db06b2ed1088b996b
While doing that I noticed that the error messages actually include a trailing space character, and one of them ends in a period while the other does not, which is kind of weird or unexpected. Would you like a PR to clean up those messages?


Updates Akaunting dependency.

This PR simply updates the Akaunting dependency version so the latest version may be used.

Thanks!


[1.11.2] Client-side crash: "java.lang.NoClassDefFoundError: cah"

Issue description:
Updating to latest version causes client-side crash at start-up
What happens:
Client crashes
What you expected to happen:
Steps to reproduce:

Update to refinedstorage-1.4.12.jar
Start client
Crash
...

Version (make sure you are on the latest version before reporting):

Minecraft: 1.11.2
Forge: forge-<IP_ADDRESS>5
Refined Storage: refinedstorage-1.4.12.jar

Does this issue occur on a server? [yes/no]
No
If a (crash)log is relevant for this issue, link it here:
[pastebin/gist/etc link here] https://gist.github.com/kwpugh/4711ff71a37bbde49e1b585149988593

That is weird. It doesn't look like a crash caused by RS.
RS doesn't even appear in the stacktrace.
It works fine for me:

It seems like an obfuscated - deobfuscated issue.

For the lay person, what does ob-deob mean?   Any suggestions where to look next?

Is this a modpack you put together yourself? Are you using some client?


TypeError: Cannot read property 'on' of undefined

When I try to beautify the i get the following problem
TypeError: Cannot read property 'on' of undefined
I just installed beautify and uncrustify
https://gist.github.com/ODCA117/917affd6b11437517535a45350379820

I am also having this issue

+1

I got the same problem.

+1 I have this issue too. Is there a comment on this somewhere?

same

Looking at the attached logs: https://gist.github.com/ODCA117/917affd6b11437517535a45350379820#logs
2019-09-05T07:19:49.820Z - error: [beautifiers/index.coffee]  message=Cannot read property 'on' of undefined, stack=TypeError: Cannot read property 'on' of undefined
    at /home/olle/.atom/packages/atom-beautify/src/beautifiers/executable.coffee:242:18

The error comes from here: https://github.com/Glavin001/atom-beautify/blob/master/src/beautifiers/executable.coffee#L238-L242
Pull Requests from community very welcome! Let me know if you have any questions. Thanks!

I'm now having the same problem. Atom Beautify was working fine but I've just installed PHP-CS-Fixer and that's when it stopped. How do I correct it?

I'm having the same problem...

No update in this issue 😥


add some data about OCR confidence index

It would be extremely helpful to have an average of the confidence index returned by the OCR in the opf. It could be at different levels:

a global average per opf
an average per volume
an average per page, in a layer, althought that would make the files much bigger and is less urgent

perhaps median of the word-level (or page-level) confidence index is the best indicator... taking the line level will give an artificially lower value because of the noise I think

new meta:
id: I7E1A43F2
source: https://library.bdrc.io
source_file: null
initial_creation_type: ocr
imported: '2020-03-28T12:12:38+00:00'
last_modified: '2022-06-08T11:28:52.590761+00:00'
parser: https://github.com/OpenPecha-dev/openpecha-toolkit/blob/231bba39dd1ba393320de82d4d08a604aabe80fc/openpecha/formatters/google_orc.py
source_metadata:
id: bdr:W3CN4314
title: ཚད་མ་རིགས་པའི་གཏེར་གྱི་རྩ་བ།
author: ''
access: http://purl.bdrc.io/admindata/AccessOpen
restrictedInChina: false
opf_level_confidence: 0.9
base:
529C:
image_group_id: I3CN8548
title: ''
total_pages: 62
order: 1
base_file: 529C.txt
vol_level_confidence: 0.9
how does this look

thanks! a few remarks:
opf_level_confidence should be at the top level (not in source_metadata) and should be renamed to ocr_word_median_confidence_index (or ocr_page_average_confidence_index depending on the mode of calculation)
the vol_level_confidence should also be renamed in the same way
it's a bit dissatisfying to have information on the opf bases in source_metadata since (as its name indicates), this was originally meant to only record the BDRC data... why not something like:
id: I7E1A43F2
source: https://library.bdrc.io
source_file: null
initial_creation_type: ocr
imported: '2020-03-28T12:12:38+00:00'
last_modified: '2022-06-08T11:28:52.590761+00:00'
parser: https://github.com/OpenPecha-dev/openpecha-toolkit/blob/231bba39dd1ba393320de82d4d08a604aabe80fc/openpecha/formatters/google_orc.py
ocr_word_median_confidence_index: 0.9
source_metadata:
    id: bdr:W3CN4314
    title: ཚད་མ་རིགས་པའི་གཏེར་གྱི་རྩ་བ།
    author: ''
    access: http://purl.bdrc.io/admindata/AccessOpen
    restrictedInChina: false
  base:
    529C:
      source_metadata:
          image_group_id: I3CN8548
          title: ''
          total_pages: 62
      order: 1
      base_file: 529C.txt
      ocr_word_median_confidence_index: 0.9

(note also that the global value for the opf can't be computed by averaging the values of the volumes because they don't have the same number of words / pages)

@eroux that actually makes more sense, we will update the metadata model accrodingly.

here is the fix 125b0ab

Thanks @10zinten ! The issue was initially about OCR confidence index, so let's keep it open for that


1.0.0

Starting this issue to track what should go into 1.0.0. This will be a backwards incompatible release so we have a chance to improve things we couldn't before.  For one, 1.0.0 will only support webpack 4 and newer.
Please add a comment or link to an issue if you'd like it to be resolved in 1.0.0

I suppose this one shall go in a new major version: https://github.com/owais/django-webpack-loader/issues/227

#182 seems like a very much wanted feature as well.

#182 seems like a very much wanted feature as well.

Bundle tracker  1.0.0-alpha.0 has support for this. 1.0 of this package will add support for 1.0.0 bundle tracker. That should cover webpack 4. PRs welcome.

I suppose this one shall go in a new major version: #227

Yes, if someone sends a PR, we can release 1.0-alpha.x

I've sent the PR https://github.com/owais/django-webpack-loader/pull/232 to address that.


terminal unusable because of garbled text

Does this issue occur when all extensions are disabled?: Yes

VS Code Version:  1.63.1
OS Version: Linux x64 <IP_ADDRESS>-microsoft-standard-WSL2

Steps to Reproduce:

open the terminal. the font is always garbled, no matter which font you use

this issue also exist in previous versions I have used
system info
Version: 1.63.1
Commit: fe719cd3e5825bf14e14182fddeb88ee8daf044f
Date: 2021-12-14T02:02:02.183Z
Electron: 13.5.2
Chromium: 91.0.4472.164
Node.js: 14.16.0
V8: 9.1.269.39-electron.0
OS: Linux x64 <IP_ADDRESS>-microsoft-standard-WSL2

/needsMoreInfo
does it happen only with Monaco or other fonts as well? Especially if you comment out both font-family settings.
And do I understand correctly that VSCode is installed directly in WSL2 and not on Windows +Remote-WSL extension?

Yes, and I just changed the font to Monaco to see if it works.
None of the fonts worked. It is in this state for months. I thought an
update will fix it but it didn't
On Thu, Dec 16, 2021, 8:10 PM Andrii Dieiev @.***>
wrote:

/needsMoreInfo
does it happen only with Monaco or other fonts as well? Especially if you
comment out both font-family settings.
And do I understand correctly that VSCode is installed directly in WSL2
and not on Windows +Remote-WSL extension?
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/139338#issuecomment-996340787,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAOJFPO647Z5KZE6TKKG7LURKEXNANCNFSM5KHS3KLA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

does changing the terminal.integrated.gpuAcceleration help?

disabling it solved the problem

disabling it solved the problem

OS Version: Linux x64 <IP_ADDRESS>-microsoft-standard-WSL2

I think you're seeing some weird gpu behavior due to WSLg, the workaround here is to disable gpuAcceleration as you've done. AFAIK we can't detect this case reliably, we already fallback gracefully if webgl is not supported but it says it's supported and proceeding to do the wrong thing.


Automatic workflow duration prediction

Summary
Users would like to be able to understand the how long similar workflows took to see if they start to take longer.
Motivation
When I'm using workflow templates, is this template taking longer to run?
Proposal

Add summary data of recent runs of the template status?
Related to #1658, #3557

Message from the maintainers:
If you wish to see this enhancement implemented please add a 👍 reaction to this issue! We often sort issues this way to know what to prioritize.

@dtaniwaki @sarabala1979 would it be possible do you think?

could we solve by just adding duration to list view?

I think we can have stats of workflow templates in RDB or somewhere and calculate the average, median and etc. and show it in the list view.
But it may affect the performance of the controller. We may need to has an option to disable the feature.

Actually, we want to use these stats in the scheduler and optimize the scheduling algorithm. For example, there is few resources available and a pod is waiting with high priority, now, a low priority pod came. If we can predict how long the pod runs, we can let it run before the high priority waiting pod.

This specific use case I want to solve - seeing historical durations is shown in the UI already.

There might be other things we want to aggregate, so I'll leave this open.

#3558 produces a way to determine likely execution time and to dig into problems.

Duplicate.

How do I enable this feature to see the estimation time on my UI?

It is automatically enabled. It estimates based on the most recently completed workflow that used the same workflow template or cron workflow.

Thanks for the quick reply. I'm using v2.12.5 and I don't see the estimation time on it:

Is it also possible to retrieve those metrics via the prometheus metrics endpoint?

Short answer - no.
Long answer - there are metrics related to workflow execution that might help you.

Thx for your answer....although it's not the one I was hoping for ;)
Unfortunately I have not been able to create a similar diagram as you have in the UI with using the duration workflow metric together with a gauge in grafana (this might be only because a lack of skill on my end, though).
Maybe I will create the metric via a workaround which is using your api and pushes to prometheus...
Thank you

hi, in our  case, due to the large number of workflows that have been archived,  execute SQL queries
like this can lead to excessive data retrieval, causes a sudden increase in machine IO. That is an issue that needs to be solved.
err = r.session.
		Select("name", "namespace", "uid", "phase", "startedat", "finishedat").
		From(archiveTableName).
		Where(r.clusterManagedNamespaceAndInstanceID()).
		And(namespaceEqual(namespace)).
		And(nameEqual(name)).
		And(namePrefixClause(namePrefix)).
		And(startedAtClause(minStartedAt, maxStartedAt)).
		And(clause).
		OrderBy("-startedat").
		Limit(limit).
		Offset(offset).
		All(&archivedWfs)

hi, in our case, due to the large number of workflows that have been archived, execute SQL queries
like this

This did not change any SQL queries so I'm not sure what you're referring to. If you have found some bug, please properly report it as a bug. This is a couple years old issue, so commenting here is not really helpful.


Automatically generate DocBook 5

convert to DocBook 5 (docserv -> project_management/archive/github_migration/html/db4_to_db5.html)

Ideally, we would generate DocBook 5 in the first place, I suppose. Would that be so hard?
Otoh, the 425 stylesheet is probably easy to add... even though it makes us run even longer.


Feature/khb laravel mix

Description
Setup Laravel Mix (webpack) to deal with assets (js + css).
Migrate from less to sass.
Steps to apply
composer dump-auload
npm install
npm run dev

from now on. work with assets only within the /resources/assets folder
you can find more about laravel-mix in the documentation
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[ ] I have updated the CHANGELOG
[x] I have requested at least 1 reviewer for this PR
[ ] I have made corresponding changes to the documentation

@igor-kamil would have been great to add documentation to the README, and check the
"I have made corresponding changes to the documentation" box

@igor-kamil also, it should be added to the changelog


Junit Reporter

Hey @andymsuk, I built on top of your branch and made a few changes for consistency with the JSON formatter:

The junit reporter writes to stdout, not to a file
The <testcase /> class names map back exactly to page URLs
<testcase /> can have multiple failures, one for each validation error (this is probably the most contentious change
The documentation links are embedded in each test case, not in each error

Can you let me know if this works for you?

Hey @joshski.
Thanks for picking this up.
Here's my thoughts on your changes:

Writing to stdout - looks good! 👍

Class names mapping exactly to URLs - the reason I'd done what I'd done was to make it look nicer on Jenkins - with my branch, you could drill down by domain...

and then by page...

But when the class name is the exact URL, Jenkins gets confused:

Multiple failures - I didn't know this was possible - seems fine to me for now

Documentation being embedded in test case rather than each error - again, probably looks nicer in the XML file, but in my branch, Jenkins displayed the page link and documentation link nicely making it clickable:

But with it being in the test case, you can't click the link:

Thanks @andymsuk

Class names mapping exactly to URLs
I was thinking specifically, that you might test the same URL under HTTPS and HTTP... do you think Jenkins is confused by the :// bit of the protocol?
Also do you know if Jenkins will display multiple failures per test case?

In answer to both your questions, not sure @joshski. I'll do some investigation!


⚠ Jenkins Plugins Artifactory has degraded performance

In 943c2e6, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1203 ms

Resolved: Jenkins Plugins Artifactory performance has improved in c610631 after 27 minutes.


Can't instantiate new EventEmmiter() in spec/test files.

Whenever I try to test a component which is using eventemitter2 and tries to make a new instance of it so for example import EventEmitter from 'eventemitter2' and then  new EventEmitter() in my spec files, it throws an error TypeError: _eventemitter2.default is not a constructor.
Any ideas what could cause this issue? Thank you.

Probably a mock or library issue, not an issue with Jest.
Please note this issue tracker is not a help forum. We recommend using StackOverflow or our discord channel for questions. Thank you :)


Segmentation fault on running build.sh

Description
When attempting to build the binaries using the build.sh script, the process fails with a segmentation fault error message or a blue screen.
Error mesage
./build.sh: line 83: 39686 Segmentation fault: 11  odin build src/ -collection:src=src -out:ols -o:speed $@
Observation
After some tinkering around, I found that removing the -o flag from the build script allows it to execute without encountering the segmentation fault. Interestingly, I've can built "hello world" apps using the same -o flag with Odin, suggesting that the issue might be specific to this script or my environment.
Environment
Odin Version: dev-2024-03:04f0fbf23
Operating System: MacOS 14.2
Processor: M1 Pro
I'm a newcomer to low-level languages, and this is my first encounter with a segmentation fault or any memory related problems, but I would love to contribute more into resolving this issue. Any guidance on where to start debugging would be greatly appreciated.

Hello. This is an issue with odin compiler. There is most likely nothing I can do about that. I have experienced somewhat large programs can have issues with compiling on different optimization modes.

Thanks for the fast response, that was my initial thought as well, it also check with the fact that there are no more issues like this one opened.
I was using llvm@14 as recommended in the docs. But I just tried to rebuild odin using llvm@17 and the issue is gone, thanks for pointing out on this!


Dynamic Module Tab: Destructure props to avoid needless refresh

Description
As per title.
Fixes issue introduced by #1352
Type of change

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update
[ ] Code quality improvements

How to Test
Behaviour is unchanged.
Checklist

[X] I have tested this code
[ ] I have updated the documentation

Pull Request Test Coverage Report for Build 6306

0 of 4 (0.0%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.004%) to 32.294%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/commons/sideContent/SideContent.tsx
0
4
0.0%

Files with Coverage Reduction
New Missed Lines
%

src/commons/sideContent/SideContent.tsx
1
2.17%

Totals

Change from base Build 6297:
-0.004%

Covered Lines:
1873

Relevant Lines:
5409

💛  - Coveralls


Channel hints channel is never used.

It can be removed.

Ah, thanks -- merged. Do you have some kind of checker to detect unconnected channels and/or orphaned processes?

I'm working on a visualisation feature that allowed me to spot it out. I will show you that soon.


gnomeExtensions: auto-update

Things done
Had to use a temporary patch to get this to work. I've also excluded the update of that extension from the PR. #168981
diff --git a/pkgs/desktops/gnome/extensions/update-extensions.py b/pkgs/desktops/gnome/extensions/update-extensions.py
index 581781fb11a..6363c6479eb 100755
--- a/pkgs/desktops/gnome/extensions/update-extensions.py
+++ b/pkgs/desktops/gnome/extensions/update-extensions.py
@@ -48,6 +48,9 @@ def fetch_extension_data(uuid: str, version: str) -> Tuple[str, str]:
     uuid = uuid.replace("@", "")
     url: str = f"https://extensions.gnome.org/extension-data/{uuid}.v{version}.shell-extension.zip"

+    if url == 'https://extensions.gnome.org/extension-data/VitalsCoreCoding.com.v53.shell-extension.zip':
+        url = 'https://extensions.gnome.org/extension-data/VitalsCoreCoding.com.v53.shell-extension_v1BI2FB.zip'
+
     # Yes, we download that file three times:

     # The first time is for the maintainter, so they may have a personal backup to fix potential issues

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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

[X] 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/)
22.05 Release Notes (or backporting 21.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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[X] Fits CONTRIBUTING.md.

cc: @zendo this will update https://gitlab.com/rmnvgr/nightthemeswitcher-gnome-shell-extension to version 63

What does that temporary patch exactly do, and why is it needed?

What does that temporary patch exactly do, and why is it needed?

Please take a look at the issue #168981 description, after the default nixpkgs part.

Updating in unstable-small channel, but night-theme-switcher still is version 53, maybe the old one needs to delete? pkgs/desktops/gnome/extensions/night-theme-switcher/default.nix

@zendo This is mostly correct. We have a manual packaging of the extension that overrides the automatic one. That one would require a separate manual update.
If you want though, you can try to remove it and see whether the automatically generated package for that extension works or can be made working with a few patches.


Having a space after "start game" ignores the command.

When trying to start a game on a phone, the Slack client doesn't remove trailing spaces, so typing "start game " will not start the game, the bot should simply ignore trailing spaces.

This was fixed.


Added exclusion for Test Coverage results directory.

Pull Request
Thank you for contributing to @dvcs/gitignore and https://www.gitignore.io.
New or update
Select the appropriate check box for this pull request. This helps when merging to ensure there are no conflicts with other templates or misunderstandings of how thee template list works.
New

[ ] Template - New .gitignore template
[ ] Composition - Template made from smaller templates
[ ] Inheritance - Template similar to an existing template
[ ] Patch - Template extending functionality of existing template

Update

[X] Template - Update existing .gitignore template

Details
Exclude the coverage directory because it contains test coverage results, which are similar to build results. Test coverage results are generated from the command flutter test --coverage.

Is there a question about this PR?
The coverage folder is like the build folder.  It is generated when using the --coverage flag for flutter test, like:
$ flutter test --coverage


Fixed realtime database store, added unit tests, and restructured for canonical compliance.

While leveraging everything that this plugin provides for a project that I am working on, I hit a blocker related to how it was handling both resource paths and local caches of Firebase Realtime Database data. Thus, the core improvement that this pull request introduces is an overhauled FirebaseDatabaseStore (previously called Store) class that actually works.
The old version did not properly parse the paths of put and patch events, and did not properly blit their payloads onto its stored data set. Duplicate entries were common due to improper handling of separator characters (a.k.a. /), as were instances where data was simply placed in the wrong location in the stored data set. Further, the old version consistently threw assertion failures due to said improper path parsing when processing patch updates received from Firebase.
Previously, none of this mattered because no part of the plugin actually consumed the FirebaseDatabaseStore – it seemed to have been a piece that was half-baked about two years ago and never touched since. However, I have use cases for a working data store that is kept in sync via Firebase Realtime Database. I considered rolling my own layer on top of this plugin, but figured it might be beneficial for others.
As a part of this, I ended up doing quite a bit more in order to help secure my confidence in this plugin. On huge thing that I did was integrate with a testing framework and wrap my enhancements in unit tests.
I would love to continue contributing to it – there are more gaps (e.g. filter criteria for Firebase) that I will eventually need to close for my own use cases. However, so far, I'm very grateful that I found this!
What does this pull request do?

Re-writes FirebaseDatabaseStore (previously just called Store) so that it accurately maintains a local copy of the data at a specific Firebase Realtime Database reference based on put and patch events received over SSE. (Review FirebaseDatabaseStore and FirebaseReference specifically.)
Integrates with the Gut unit and integration test plugin for Godot.
Adds 14 unit tests for the new FirebaseDatabaseStore functionality.
Adds a new TESTING.md file describing how to actually write and run tests. (This could be moved to the repository's wiki.)
Restructures and renames parts of the repository so that they are canonical with other best practices and naming conventions in the Godot community. (This was also necessary in order for many of the other enhancements listed here to be possible.)
Revs the plugin's major version since some structural changes have been made.
Updates the demo project to use a soft symbolic link to the actual plugin code. (This makes it so that the plugin doesn't have to be copy-and-pasted into the demo project every time a change is made. Major quality-of-life improvement for maintainers.)
Updates the README.md to describe the recommended way of cloning the repository so that submodules (e.g. HTTPSSEClient) also get cloned automatically.

How do you know that know regressions have been introduced?

All tests are passing.
My personal project (which consumes Authentication, Firebase, and Realtime Database) runs without error with this version of the plugin.
The repository's demo project runs without error with this version of the plugin.

How can I contact you if I need to?
The easiest way is via Discord. My username is lukehollenback#0061.

I'll be reviewing this presently. I sent you a friend request on Discord - this is BackAt50Ft#5196.

I'll be reviewing this presently. I sent you a friend request on Discord - this is BackAt50Ft#5196.


Files are missing following guide

Hi,
I tried to follow guide to install ubuntu on btrfs with luks, and on optional step 3 there are no /usr/lib/partman/mount.d/70btrfs and /usr/lib/partman/fstab.d/btrfs.
I'm using Ubuntu20.04.1 server.
Maybe guide should be updated?

I think this issue is incorrect as I was trying to adapt guide to user-data btrfs encrypted partition.

I think this issue is incorrect as I was trying to adapt guide to user-data btrfs encrypted partition.

Can you elaborate on this?  I have the same issue in Ubuntu desktop.  Not sure I understand your explanation of why these folders wouldn't exist.  Thanks

Can you elaborate on this? I have the same issue in Ubuntu desktop. Not sure I understand your explanation of why these folders wouldn't exist. Thanks

Because it's pre-installation step while I was setting partition on already installed Ubuntu system.

Thanks.  Mine is a fresh install, and I still don't have these folders.


Improve error correction if possible

The error correction makes a lot of mistakes yet reports them as valid codewords (syndrome == 0). Two options:

This should be fixed, or
If it is a limitation of the error correction code itself, error correction should be disabled.

Here's an example of the error correction failing. The group is shown separated into blocks (4 hex characters) and checkwords (3 characters).
orig: 6A03/2C7 0421/14A F741/0C6 4557/1B8
err:  0800     0100     0000     0800
corr: 6203     0521              4D57

This leads to the PS string "YLMWUO" (should be "YLESUOMI"). It nevertheless gives syndrome 0 (no errors detected).
The syndrome calculation function (uint32_t calcSyndrome(uint32_t) in block_sync.cc) is a shift register implementation using obscure bit magic and should also be reviewed.
Here's what the RDS specs have to say about the error protection scheme:

The error-protecting code has the following error-checking capabilities [3, 4]:
a)  Detects all single and double bit errors in a block.
b)  Detects any single error burst spanning 10 bits or less.
c)  Detects about 99.8% of bursts spanning 11 bits and about 99.9% of all longer bursts.
The code is also an optimal burst error correcting code [5] and is capable of correcting any
single burst of span 5 bits or less.

The references point to:

[3] Peterson, W.W., and Brown, D.T.: Cyclic codes for error detection.
Proceedings of the IRE, No. 49, January 1961, pp. 228-235. (pdf)
[4] Peterson, W.W., and Weldon, E.J.: Error-correcting codes.
Published by MIT Press, Cambridge Mass., second edition, 1972.
[5] Kasami, T.: Optimum shortened cyclic codes for burst error correction.
IEEE Transactions on Information Theory (IT9), No. 4, 1963, pp. 105-109. (doi)

Seems to be fixed now!


Build with --sysbuild for mcuboot failed

I'm using ZephyrIDE 1.3.3 on MacOS 14.7 with MacBook Pro M3.
When I try to build app for mcuboot with --sysbuild argument, I get error and failed.
If I build project alone it is Ok.
If I manually run command in bash or zsh terminal it is also Ok. Then I could build without -p. Only pristine is problem.
Manual command from terminal is:

west build -b ECO00189 /Users/robert.konc/Soft/zpIDE3/B300 -p --build-dir >/Users/robert.konc/Soft/zpIDE3/B300/build/bootECO00189 --sysbuild -- -DBOARD_ROOT='/Users/robert.konc/Soft/zpIDE3'   ->DEXTRA_CONF_FILE='/Users/robert.konc/Soft/zpIDE3/B300/prj_ble.conf;/Users/robert.konc/Soft/zpIDE3/B300/prj_release.conf;

Here is log from terminal.

Executing task: west build -b ECO00189 /Users/robert.konc/Soft/zpIDE3/B300 -p --build-dir /Users/robert.konc/Soft/zpIDE3/B300/build/bootECO00189 --sysbuild -- -DBOARD_ROOT='/Users/robert.konc/Soft/zpIDE3'   -DEXTRA_CONF_FILE='/Users/robert.konc/Soft/zpIDE3/B300/prj_ble.conf;/Users/robert.konc/Soft/zpIDE3/B300/prj_release.conf;'
-- west build: making build dir /Users/robert.konc/Soft/zpIDE3/B300/build/bootECO00189 pristine
-- west build: generating a build system
Loading Zephyr module(s) (Zephyr base): sysbuild_default
-- Found Python3: /Users/robert.konc/Soft/zpIDE3/.venv/bin/python (found suitable version "3.12.5", minimum required is "3.8") found components: Interpreter
-- Cache files will be written to: /Users/robert.konc/Library/Caches/zephyr
-- Found west (found suitable version "1.2.0", minimum required is "0.14.0")
-- Board: ECO00189, qualifiers: nrf52840
Parsing /Users/robert.konc/Soft/zpIDE3/external/zephyr/share/sysbuild/Kconfig
Loaded configuration '/Users/robert.konc/Soft/zpIDE3/B300/build/bootECO00189/_sysbuild/empty.conf'
Merged configuration '/Users/robert.konc/Soft/zpIDE3/B300/sysbuild.conf'
Configuration saved to '/Users/robert.konc/Soft/zpIDE3/B300/build/bootECO00189/zephyr/.config'
Kconfig header saved to '/Users/robert.konc/Soft/zpIDE3/B300/build/bootECO00189/_sysbuild/autoconf.h'

Running CMake for mcuboot *

Loading Zephyr default modules (Zephyr base).
-- Application: /Users/robert.konc/Soft/zpIDE3/external/bootloader/mcuboot/boot/zephyr
-- CMake version: 3.30.2
-- Found Python3: /opt/homebrew/bin/python (found suitable version "3.12.5", minimum required is "3.8") found components: Interpreter
-- Cache files will be written to: /Users/robert.konc/Library/Caches/zephyr
-- Zephyr version: 3.7.99 (/Users/robert.konc/Soft/zpIDE3/external/zephyr)
CMake Error at /Users/robert.konc/Soft/zpIDE3/external/zephyr/cmake/modules/boards.cmake:198 (message):
Error finding board: ECO00189
Error message: Traceback (most recent call last):
File "/Users/robert.konc/Soft/zpIDE3/external/zephyr/scripts/list_boards.py", line 11, in <module>
  import pykwalify.core

ModuleNotFoundError: No module named 'pykwalify'
Call Stack (most recent call first):
/Users/robert.konc/Soft/zpIDE3/external/zephyr/cmake/modules/zephyr_default.cmake:133 (include)
/Users/robert.konc/Soft/zpIDE3/external/zephyr/share/zephyr-package/cmake/ZephyrConfig.cmake:66 (include)
/Users/robert.konc/Soft/zpIDE3/external/zephyr/share/zephyr-package/cmake/ZephyrConfig.cmake:92 (include_boilerplate)
CMakeLists.txt:12 (find_package)
-- Configuring incomplete, errors occurred!
CMake Error at cmake/modules/sysbuild_extensions.cmake:486 (message):
CMake configure failed for Zephyr project: mcuboot
Location:
/Users/robert.konc/Soft/zpIDE3/external/bootloader/mcuboot/boot/zephyr
Call Stack (most recent call first):
cmake/modules/sysbuild_images.cmake:20 (ExternalZephyrProject_Cmake)
cmake/modules/sysbuild_default.cmake:20 (include)
/Users/robert.konc/Soft/zpIDE3/external/zephyr/share/zephyr-package/cmake/ZephyrConfig.cmake:75 (include)
/Users/robert.konc/Soft/zpIDE3/external/zephyr/share/zephyr-package/cmake/ZephyrConfig.cmake:92 (include_boilerplate)
/Users/robert.konc/Soft/zpIDE3/external/zephyr/share/sysbuild-package/cmake/SysbuildConfig.cmake:8 (include)
template/CMakeLists.txt:10 (find_package)
-- Configuring incomplete, errors occurred!
FATAL ERROR: command exited with status 1: /opt/homebrew/bin/cmake -DWEST_PYTHON=/Users/robert.konc/Soft/zpIDE3/.venv/bin/python -B/Users/robert.konc/Soft/zpIDE3/B300/build/bootECO00189 -GNinja -DBOARD=ECO00189 -DBOARD_ROOT=/Users/robert.konc/Soft/zpIDE3 '-DEXTRA_CONF_FILE=/Users/robert.konc/Soft/zpIDE3/B300/prj_ble.conf;/Users/robert.konc/Soft/zpIDE3/B300/prj_release.conf;' -S/Users/robert.konc/Soft/zpIDE3/external/zephyr/share/sysbuild -DAPP_DIR:PATH=/Users/robert.konc/Soft/zpIDE3/B300

The terminal process "/bin/bash '-l', '-c', 'west build -b ECO00189 /Users/robert.konc/Soft/zpIDE3/B300 -p --build-dir /Users/robert.konc/Soft/zpIDE3/B300/build/bootECO00189 --sysbuild -- -DBOARD_ROOT='/Users/robert.konc/Soft/zpIDE3'   -DEXTRA_CONF_FILE='/Users/robert.konc/Soft/zpIDE3/B300/prj_ble.conf;/Users/robert.konc/Soft/zpIDE3/B300/prj_release.conf;' '" terminated with exit code: 1.
Terminal will be reused by tasks, press any key to close it.

Does the problem occur on zephyr-ide 1.3.2 (recreate the project in that version and see if it persists)? Is this a custom board?

I'm tested in 1.3.0 version. Same problem.
Yes it is custom board.
But if I build only application ( no --sysbuild) it works.

You can try running west update and trying again.
You can also try manually installing the python requirements.
https://github.com/zephyrproject-rtos/zephyr/blob/main/scripts/requirements.txt
Can you also share you settings.json file?

West update. Same result update git but not requirements.
Manualy install requirements.
Run build ok.
Run build with --sysbuild same error.
In attachment is settings.json file.
settings.json

I'm not sure what the problem is. It could be related to using mac os or it could be related to the project configuration.
Easiest way to debug is for you to share your full workspace.

I'm testing Zephyr IDE extension under Ubuntu 24.4 LTS (aarch64) virtual machine. Virtual host app is VmWare Fusion. Host machine is MacBook Pro M3.
Version 1.3.3 work as it should. I'm testing all things which make problems under MacOS and here all works as it should.
I'm also copy workspace folder from MacOs to Ubuntu and build. In my opinion it is nothing wrong with project configuration.
Regards.

I will close this issue. It is same bug as in #55.


Optimize tools/test-management to be done in python as a normal test suite

Currently we spend 1 minute in every Travis CI backend suite running tools/test-management, which just runs all the management commands we have with the --help argument, to ensure that they actually start (this prevent us from having management commands with broken syntax, at least!).
The reason it's so slow is that we're doing it in shell, and thus we're paying the ~1s it takes to import and load Django from scratch for each one.  Instead, we should do this in Python, by just importing the individual management command files (using import_module) and then using django.core.management.call_command() to call the actual code directly; one can see an example of this approach in get_migration_status in zerver/lib/test_fixtures.py.  I think with this approach we could make this suite like 30x faster.
Fixing this would save about 4% (2 minutes) of the total runtime of our complete test suite in Travis CI, which is pretty good for what's probably just moving 50 lines of code into Python.

Working on this! Spending 1 minute on this just to run --help does seem unacceptable.


support for older models

Hello,
I was wondering if it is straightforward to bring older models such as GPT-2 to lit-gpt.
If so, what files/configs do I need to change?
Thank you!

Good point, and it should be. I use GPT-2 myself privately a lot as well, and it'd be nice to have it in LitGPT as well.
I think the architecture is similar to GPTNeo, so you can probably copy and adapt the GPTNeo config. The general todo list I use for adding new configs is:

[ ] Implement model download
[ ] Implement HF checkpoint conversion
[ ] Make sure generate.py produces reasonable outputs
[ ] Update model_download docs
[ ] Test pretraining
[ ] Test finetuning

[ ] Full finetuning
[ ] LoRA
[ ] Adapter + Adapter v2

[ ] Add tests


Add replay option to disable deadlock detection

What was changed

This PR adds a new field DisableDeadlockDetection to WorkflowReplayerOptions. Setting this to true disables deadlock detection during replay.
Why?

The TEMPORAL_DEBUG environment variable is intended to serve a similar purpose, but can be tricky to configure and is a global setting.
Deadlock detection can be disabled programmatically using DeadlockDetectionTimeout on worker.Options. It makes sense to support something similar in WorkflowReplayerOptions since attaching a debugger is such a common use case when replaying workflows.
Checklist

How was this tested:

Please see included unit test.
Any docs updates needed?

The updated godoc is probably enough, though this would be a useful feature to mention in tutorials about workflow debugging.

@jlegrone - merging master into your branch and merging when CI passes


fix(activation): added a delay with configurable time after activation

Codecov Report

Merging #167 (22846bf) into master (0c0828f) will decrease coverage by 0.50%.
The diff coverage is 11.29%.

@@            Coverage Diff             @@
##           master     #167      +/-   ##
==========================================
- Coverage   32.17%   31.67%   -0.51%
==========================================
  Files          60       60
  Lines        3319     3378      +59
  Branches      537      553      +16
==========================================
+ Hits         1068     1070       +2
- Misses       2086     2143      +57
  Partials      165      165

Impacted Files
Coverage Δ

src/DataProcessor.ts
38.57% <0.00%> (-4.98%)
:arrow_down:

src/actions/CIRAConfigurator.ts
6.71% <0.00%> (-0.21%)
:arrow_down:

src/actions/NetworkConfigurator.ts
15.94% <0.00%> (ø)

src/models/Rcs.ts
27.39% <ø> (ø)

src/test/helper/Config.ts
100.00% <ø> (ø)

src/utils/ClientResponseMsg.ts
62.50% <0.00%> (-8.93%)
:arrow_down:

src/utils/FileHelper.ts
75.00% <ø> (-0.68%)
:arrow_down:

src/actions/ACMActivator.ts
35.52% <10.00%> (-6.08%)
:arrow_down:

src/actions/CCMActivator.ts
31.64% <14.28%> (-8.36%)
:arrow_down:

src/RCS.Config.ts
100.00% <100.00%> (ø)

... and 3 more

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 0c0828f...22846bf. Read the comment docs.


Create example resource quota objects for operator

https://kubernetes.io/docs/concepts/policy/resource-quotas/#object-count-quota

Technically something like this would work
apiVersion: v1
kind: ResourceQuota
metadata:
  name: core-object-counts
spec:
  hard:
    count/cloudservicesrequests.rhoas.redhat.com: "1"
    count/kafkaconnections.rhoas.redhat.com: "1"
    count/cloudserviceaccountrequest.rhoas.redhat.com: "1"


Find and replace should start with text highlighted

The marimo find and replace (started with Ctrl/Cmd+F) should start with the previously entered text selected, just like Chrome's find and replace. That way the user can hit Ctrl+F and immediately start typing their query, without having to first select or delete the previous query.

I get this