Search help on focused word got broken

Godot version: 84401e8cdf4f41fc0e804bf5410622e75a08c0f5
Issue description:
Previously (on 3.1.1 stable release) when I did Shift+F1 I got search help with filled word that I had focus on. Now it doesn't work that way. Search field is empty.
I deleted AppData Godot folder, but it didn't help

confirmed. it's really annoying regression.

Also, when you run 84401e8 version, and then switch back to 3.1.1 it also stops working correctly. You have to delete Godot folder from AppData to restore that functionality on 3.1.1

i don't know why godot's appdata is related but sounds strange to me. hm.

I have just rechecked this. And yes, you have to delete godot folder

Does it work normally before my commit? Because analysing it here, it doesn't seem to checks for highlighted words:
https://github.com/godotengine/godot/blob/84401e8cdf4f41fc0e804bf5410622e75a08c0f5/editor/plugins/script_editor_plugin.cpp#L1010-L1013

Never mind, that was for the global shortcut, which seems to be overriding the script editor one, I don't know how my commit did that...

I reverted the commit on my end, cleared my "godot" directory in ".config", and the help still ignores highlighted text. I don't think this is related to my commit at all.

I think I found the problem! In the "editor_settings-3.tres" file generated in master, this piece of text is added:
[sub_resource type="InputEventKey" id=1]
shift = true
scancode = 16777244
unicode = 16777244

If you remove this and launch the 3.1.* version, the help dialog grabs the highlighted text as expected. However, I don't know what triggers to add that there.

Related #27151

That's a regression from f1e4fd3e39.
There are now two locations where this shortcut is defined to different values:
5854:   p->add_icon_shortcut(gui_base->get_icon("HelpSearch", "EditorIcons"), ED_SHORTCUT("editor/editor_help", TTR("Search"), KEY_MASK_SHIFT | KEY_F1), HELP_SEARCH);
6406:   ED_SHORTCUT("editor/editor_help", TTR("Search Help"), KEY_F4);

The shortcut should also be changed on line 6406, or f1e4fd3e39 should be reverted.

Nevermind my comment, that's already what @dankan1890 had found out and fixed in #28907.

Why this has been closed? It still doen't work. The #28907 didn't resolved it.


[Bug]: Stopped working on iPhone and iPad after 1.12.5

Describe the bug
I am unable to use the plugin after updating to the newest version on my iPhone and iPad. The plugin works fine on PC. Even with all other plugins disabled and the default theme, it still doesn‘t work.
To Reproduce

Be on iOS/iPadOS 16.7 (not sure if needed)
Update the plugin to newest version
try to open "Review Flashcards"
menu does not open

Expected behavior
After doing command „Spaced Repetition: Review flashcards from all Notes“ the menu should open, which it does not.
Screenshots
No response
OS of your device
iOS 17.6 / iPadOS 17.6
Obsidian version
1.6.7
Plugin version
1.12.5
Installer version
No response
Additional context
No response
Config file
No response

Curious, were you using the same vault on the PC as on the iPhone/iPad?

You are not alone. I'm having the same issue with iOS 17.6 / iPadOS 17.6.
I tried to uninstall and then to install the plugin. Didn't work.

Thanks for the info. Interesting that the same vault works on the PC, and a fresh vault works on the mobile. This seems to imply that there is something within the vault that triggers specific code that isn't working in the mobile environment.
If you could please try out the beta 12 release, as per https://github.com/st3v3nmw/obsidian-spaced-repetition/issues/1033#issuecomment-2271083766
Cheers
Ronny

Hello again. I seem to have found a solution. The issue seems to be the data.json, and the setting causing the crash is "noteFoldersToIgnore", which I had set to "noteFoldersToIgnore": [ "Personal" ],. Changing it to "noteFoldersToIgnore": [], which should be the default setting fixed the issue and let me use the plugin again on my phone.
Also, not sure if this is still needed info but I did try changing to the beta 12 release as well as 1.12.4 and that did not solve the issue.

Here's the log error too:
Unhandled promise rejection: TypeError: undefined is not an object (evaluating 'this.baseDeckTree.getDeck')

Thanks for that, and that is largely consistent with #1033.
The reason why your workaround worked is that if noteFoldersToIgnore is empty, then there is no attempt to call the (missing on mobile) system API normalize.

Thanks for the workaround, @Ti-mmm .
I removed all the words in "Folders to ignore ", then "Review Flashcards" could bring out the menu.

I have been using this plugin daily for a few years, and I can confirm it stopped working some time back. Today I removed the exclusion folders and it started to work again.
I am on iPhone.  I am just happy it is working again, the folder exclusion is not such a big deal, but of course it would be nice to have it working.

Hi this has been fixed in https://github.com/st3v3nmw/obsidian-spaced-repetition/pull/1048, but not yet released.


Correct devguide's command line to launch RoslynDev hive

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot


Loss function for pretraining

As mentioned in issue [https://github.com/google-research/big_transfer/issues/26] the loss is sigmoid binary cross entropy for each label. I have few more questions about the loss:

How is the objects present in the image but label being zero accounted or handled?. For example: there is a picture containing : dog, cat but the label is only dog, and not {cat, animal}.
Is there negative sampling done from all label classes for negatives or all the other classes are taken as negative?
@lucasb-eyer @akolesnikoff @ebursztein @jessicayung @kolesman

Hi, good questions, sorry for the late answer.

It is not accounted for. It happens and this is part of label noise. We actually show in another paper that in such case it seems sigmoid cross entropy per label is beneficial over softmax.
There is no mining, and not even a distinction between positive/negative. We simply always predict (yes/no) for all labels on each image. Also no weighting or anything.

Thanks Lucas. I have another question regarding the sampling for the data. Since, the data distribution for BiT-L is heavily tailed. Was the distribution converted to uniform or other to pretrain the model on such a large dataset?


Filter All being repeated multiple times

Please see here.
http://easycaptures.com/fs/uploaded/1312/0798532704.jpg
Pro4Trello is the only extension enabled.

Addressed in 3.0.


Site Search proof of concept

DO NOT PUBLISH
Initial search proof-of-concept.
Largely borrowing pre-existing code from the cannabis site.

Working on accessibility testing issues....


Fix french grammer in readme

Replace image for readme.md due to french grammer mistake

Nice. Font is a little bit different, and the edited image is slightly offset and white background becomes visible.

share/.DS_Store file is not needed.

@abitrolly please check now

Almost perfect :1:. The background color is of a different shade and font is more thick than the rest of the text. Not sure if the font can be fixed.

The font is as exact as possibly be, And the background colour is similar I have not changed it

@whoami-anoint please merge


DidChangeDecorations events not working

monaco-editor version: 0.11.1
Browser: chrome newest
OS: win 10 64 bit
Steps or JS usage snippet reproducing the issue:
editor.onDidChangeModelDecorations
or
editor.getModel().onDidChangeDecorations
both return an empty object ({}). tested in my project and in monaco editor playground
in 0.10.1 it worked perfectly

The event sent out no longer lists the decorations that were changed. This has been done in order to improve the speed of decorations. See https://github.com/Microsoft/vscode/pull/36410
You can, on your side keep track of the ranges you set on decorations. Then, in the onDidChangeDecorations callback, you can read the ranges for the decorations from the editor and figure out if your decorations have been changed.


Introducing linters to ops-toolbelt builds

What this PR does / why we need it:
PR introduces SAST linters:

bandit for python code
shellcheck for bash code

Which issue(s) this PR fixes:
Fixes #149
Special notes for your reviewer:
On release build lint outputs are saved within the root of the repo, rest of the pipeline execute the linters but don't save the results.
Release note:

Introducing linters for ops-toolbelt builds

This PR proposes changes that would break the pipeline definition:
ops-toolbelt-feat_sast-linters: Traceback (most recent call last):
  File "/usr/lib/python3.12/site-packages/concourse/replicator.py", line 141, in render
    definition_descriptor = self._render(definition_descriptor)
                            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.12/site-packages/concourse/replicator.py", line 185, in _render
    'definition': factory.create_pipeline_definition(),
                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.12/site-packages/concourse/factory.py", line 108, in create_pipeline_definition
    validator.validate()
  File "/usr/lib/python3.12/site-packages/concourse/validator.py", line 19, in validate
    self._validate_variant(variant)
  File "/usr/lib/python3.12/site-packages/concourse/validator.py", line 24, in _validate_variant
    self._validate_element(variant)
  File "/usr/lib/python3.12/site-packages/concourse/validator.py", line 29, in _validate_element
    element.validate()
  File "/usr/lib/python3.12/site-packages/model/base.py", line 36, in validate
    self._validate_known_attributes()
  File "/usr/lib/python3.12/site-packages/model/base.py", line 65, in _validate_known_attributes
    raise ModelValidationError(
model.base.ModelValidationError: JobVariant:<unknown>: The following attributes are unknown: verify-bandit, verify-shellcheck.

This PR proposes changes that would break the pipeline definition:
ops-toolbelt-feat_sast-linters: Traceback (most recent call last):
  File "/usr/lib/python3.12/site-packages/concourse/replicator.py", line 141, in render
    definition_descriptor = self._render(definition_descriptor)
                            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.12/site-packages/concourse/replicator.py", line 225, in _render
    raise RuntimeError(
RuntimeError: An exception occured when rendering pipeline 'ops-toolbelt-feat_sast-linters':

Traceback (most recent call last):
  File "/usr/lib/python3.12/site-packages/concourse/replicator.py", line 217, in _render
    definition_descriptor.pipeline = template.render(
  File "/usr/lib/python3.12/site-packages/mako/template.py", line 438, in render
    return runtime._render(self, self.callable_, args, data)
  File "/usr/lib/python3.12/site-packages/mako/runtime.py", line 874, in _render
    _render_context(
  File "/usr/lib/python3.12/site-packages/mako/runtime.py", line 916, in _render_context
    _exec_template(inherit, lclcontext, args=args, kwargs=kwargs)
  File "/usr/lib/python3.12/site-packages/mako/runtime.py", line 943, in _exec_template
    callable_(context, *args, **kwargs)
  File "memory:0x7f9b2f715a60", line 666, in render_body
    ${job(variant)}
  File "memory:0x7f9b2f715a60", line 0, in job

  File "memory:0x7f9b2f715a60", line 654, in render_job
    ${step(4, job_variant, job_variant.step(step_name))}
  File "memory:0x7f9b2f715a60", line 619, in step
    <%def name="job(job_variant)">
  File "memory:0x7f9b2f715a60", line 600, in render_step
    ${execute(2, job_step, job_variant, source_repo, source_repo_github_cfg_name)}
  File "memory:0x7f9b2f715a60", line 567, in execute
    <%def name="step(indent, job_variant, job_step)" filter="indent_func(indent),trim">
  File "memory:0x7f9b2f715a60", line 453, in render_execute
    ${name}: '${eval(expression, {
  File "<string>", line 1, in <module>

NameError: name 'python' is not defined

This PR proposes changes that would break the pipeline definition:
ops-toolbelt-feat_sast-linters: Traceback (most recent call last):
  File "/usr/lib/python3.12/site-packages/concourse/replicator.py", line 141, in render
    definition_descriptor = self._render(definition_descriptor)
                            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.12/site-packages/concourse/replicator.py", line 225, in _render
    raise RuntimeError(
RuntimeError: An exception occured when rendering pipeline 'ops-toolbelt-feat_sast-linters':

Traceback (most recent call last):
  File "/usr/lib/python3.12/site-packages/concourse/replicator.py", line 217, in _render
    definition_descriptor.pipeline = template.render(
  File "/usr/lib/python3.12/site-packages/mako/template.py", line 438, in render
    return runtime._render(self, self.callable_, args, data)
  File "/usr/lib/python3.12/site-packages/mako/runtime.py", line 874, in _render
    _render_context(
  File "/usr/lib/python3.12/site-packages/mako/runtime.py", line 916, in _render_context
    _exec_template(inherit, lclcontext, args=args, kwargs=kwargs)
  File "/usr/lib/python3.12/site-packages/mako/runtime.py", line 943, in _exec_template
    callable_(context, *args, **kwargs)
  File "memory:0x7f5d1d6c43e0", line 666, in render_body
    ${job(variant)}
  File "memory:0x7f5d1d6c43e0", line 0, in job

  File "memory:0x7f5d1d6c43e0", line 654, in render_job
    ${step(4, job_variant, job_variant.step(step_name))}
  File "memory:0x7f5d1d6c43e0", line 619, in step
    <%def name="job(job_variant)">
  File "memory:0x7f5d1d6c43e0", line 600, in render_step
    ${execute(2, job_step, job_variant, source_repo, source_repo_github_cfg_name)}
  File "memory:0x7f5d1d6c43e0", line 567, in execute
    <%def name="step(indent, job_variant, job_step)" filter="indent_func(indent),trim">
  File "memory:0x7f5d1d6c43e0", line 453, in render_execute
    ${name}: '${eval(expression, {
  File "<string>", line 1, in <module>

NameError: name 'python' is not defined

This PR proposes changes that would break the pipeline definition:
ops-toolbelt-feat_sast-linters: Traceback (most recent call last):
  File "/usr/lib/python3.12/site-packages/concourse/replicator.py", line 141, in render
    definition_descriptor = self._render(definition_descriptor)
                            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.12/site-packages/concourse/replicator.py", line 225, in _render
    raise RuntimeError(
RuntimeError: An exception occured when rendering pipeline 'ops-toolbelt-feat_sast-linters':

Traceback (most recent call last):
  File "/usr/lib/python3.12/site-packages/concourse/replicator.py", line 217, in _render
    definition_descriptor.pipeline = template.render(
  File "/usr/lib/python3.12/site-packages/mako/template.py", line 438, in render
    return runtime._render(self, self.callable_, args, data)
  File "/usr/lib/python3.12/site-packages/mako/runtime.py", line 874, in _render
    _render_context(
  File "/usr/lib/python3.12/site-packages/mako/runtime.py", line 916, in _render_context
    _exec_template(inherit, lclcontext, args=args, kwargs=kwargs)
  File "/usr/lib/python3.12/site-packages/mako/runtime.py", line 943, in _exec_template
    callable_(context, *args, **kwargs)
  File "memory:0x7ff92e5e3230", line 666, in render_body
    ${job(variant)}
  File "memory:0x7ff92e5e3230", line 0, in job

  File "memory:0x7ff92e5e3230", line 654, in render_job
    ${step(4, job_variant, job_variant.step(step_name))}
  File "memory:0x7ff92e5e3230", line 619, in step
    <%def name="job(job_variant)">
  File "memory:0x7ff92e5e3230", line 600, in render_step
    ${execute(2, job_step, job_variant, source_repo, source_repo_github_cfg_name)}
  File "memory:0x7ff92e5e3230", line 567, in execute
    <%def name="step(indent, job_variant, job_step)" filter="indent_func(indent),trim">
  File "memory:0x7ff92e5e3230", line 453, in render_execute
    ${name}: '${eval(expression, {
  File "<string>", line 1, in <module>

NameError: name 'python' is not defined

This PR proposes changes that would break the pipeline definition:
ops-toolbelt-feat_sast-linters: Traceback (most recent call last):
  File "/usr/lib/python3.12/site-packages/concourse/replicator.py", line 141, in render
    definition_descriptor = self._render(definition_descriptor)
                            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.12/site-packages/concourse/replicator.py", line 225, in _render
    raise RuntimeError(
RuntimeError: An exception occured when rendering pipeline 'ops-toolbelt-feat_sast-linters':

Traceback (most recent call last):
  File "/usr/lib/python3.12/site-packages/concourse/replicator.py", line 217, in _render
    definition_descriptor.pipeline = template.render(
  File "/usr/lib/python3.12/site-packages/mako/template.py", line 438, in render
    return runtime._render(self, self.callable_, args, data)
  File "/usr/lib/python3.12/site-packages/mako/runtime.py", line 874, in _render
    _render_context(
  File "/usr/lib/python3.12/site-packages/mako/runtime.py", line 916, in _render_context
    _exec_template(inherit, lclcontext, args=args, kwargs=kwargs)
  File "/usr/lib/python3.12/site-packages/mako/runtime.py", line 943, in _exec_template
    callable_(context, *args, **kwargs)
  File "memory:0x7f9b2d8c9d00", line 666, in render_body
    ${job(variant)}
  File "memory:0x7f9b2d8c9d00", line 0, in job

  File "memory:0x7f9b2d8c9d00", line 654, in render_job
    ${step(4, job_variant, job_variant.step(step_name))}
  File "memory:0x7f9b2d8c9d00", line 619, in step
    <%def name="job(job_variant)">
  File "memory:0x7f9b2d8c9d00", line 600, in render_step
    ${execute(2, job_step, job_variant, source_repo, source_repo_github_cfg_name)}
  File "memory:0x7f9b2d8c9d00", line 567, in execute
    <%def name="step(indent, job_variant, job_step)" filter="indent_func(indent),trim">
  File "memory:0x7f9b2d8c9d00", line 453, in render_execute
    ${name}: '${eval(expression, {
  File "<string>", line 1, in <module>

NameError: name 'bash' is not defined

This PR proposes changes that would break the pipeline definition:
ops-toolbelt-feat_sast-linters: Traceback (most recent call last):
  File "/usr/lib/python3.12/site-packages/concourse/replicator.py", line 141, in render
    definition_descriptor = self._render(definition_descriptor)
                            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.12/site-packages/concourse/replicator.py", line 225, in _render
    raise RuntimeError(
RuntimeError: An exception occured when rendering pipeline 'ops-toolbelt-feat_sast-linters':

Traceback (most recent call last):
  File "/usr/lib/python3.12/site-packages/concourse/replicator.py", line 217, in _render
    definition_descriptor.pipeline = template.render(
  File "/usr/lib/python3.12/site-packages/mako/template.py", line 438, in render
    return runtime._render(self, self.callable_, args, data)
  File "/usr/lib/python3.12/site-packages/mako/runtime.py", line 874, in _render
    _render_context(
  File "/usr/lib/python3.12/site-packages/mako/runtime.py", line 916, in _render_context
    _exec_template(inherit, lclcontext, args=args, kwargs=kwargs)
  File "/usr/lib/python3.12/site-packages/mako/runtime.py", line 943, in _exec_template
    callable_(context, *args, **kwargs)
  File "memory:0x7f9b2cc39af0", line 666, in render_body
    ${job(variant)}
  File "memory:0x7f9b2cc39af0", line 0, in job

  File "memory:0x7f9b2cc39af0", line 654, in render_job
    ${step(4, job_variant, job_variant.step(step_name))}
  File "memory:0x7f9b2cc39af0", line 619, in step
    <%def name="job(job_variant)">
  File "memory:0x7f9b2cc39af0", line 600, in render_step
    ${execute(2, job_step, job_variant, source_repo, source_repo_github_cfg_name)}
  File "memory:0x7f9b2cc39af0", line 567, in execute
    <%def name="step(indent, job_variant, job_step)" filter="indent_func(indent),trim">
  File "memory:0x7f9b2cc39af0", line 453, in render_execute
    ${name}: '${eval(expression, {
  File "<string>", line 1, in <module>

NameError: name 'bash' is not defined

This PR proposes changes that would break the pipeline definition:
ops-toolbelt-feat_sast-linters: Traceback (most recent call last):
  File "/usr/lib/python3.12/site-packages/concourse/replicator.py", line 141, in render
    definition_descriptor = self._render(definition_descriptor)
                            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.12/site-packages/concourse/replicator.py", line 225, in _render
    raise RuntimeError(
RuntimeError: An exception occured when rendering pipeline 'ops-toolbelt-feat_sast-linters':

Traceback (most recent call last):
  File "/usr/lib/python3.12/site-packages/concourse/replicator.py", line 217, in _render
    definition_descriptor.pipeline = template.render(
  File "/usr/lib/python3.12/site-packages/mako/template.py", line 438, in render
    return runtime._render(self, self.callable_, args, data)
  File "/usr/lib/python3.12/site-packages/mako/runtime.py", line 874, in _render
    _render_context(
  File "/usr/lib/python3.12/site-packages/mako/runtime.py", line 916, in _render_context
    _exec_template(inherit, lclcontext, args=args, kwargs=kwargs)
  File "/usr/lib/python3.12/site-packages/mako/runtime.py", line 943, in _exec_template
    callable_(context, *args, **kwargs)
  File "memory:0x7f5d1d60dac0", line 666, in render_body
    ${job(variant)}
  File "memory:0x7f5d1d60dac0", line 0, in job

  File "memory:0x7f5d1d60dac0", line 654, in render_job
    ${step(4, job_variant, job_variant.step(step_name))}
  File "memory:0x7f5d1d60dac0", line 619, in step
    <%def name="job(job_variant)">
  File "memory:0x7f5d1d60dac0", line 600, in render_step
    ${execute(2, job_step, job_variant, source_repo, source_repo_github_cfg_name)}
  File "memory:0x7f5d1d60dac0", line 567, in execute
    <%def name="step(indent, job_variant, job_step)" filter="indent_func(indent),trim">
  File "memory:0x7f5d1d60dac0", line 453, in render_execute
    ${name}: '${eval(expression, {
  File "<string>", line 1, in <module>

NameError: name 'python' is not defined

The pipeline-definition has been fixed.


MSID: 90100 Version: 1 DOI: 2<IP_ADDRESS>4329

"msas": "Neuroscience"
"msid": "90100"
"preprintDoi": "10.1101/2<IP_ADDRESS>4329"
"Reviewed Preprint posted": "2023-09-12"
"Sent for peer review": "2023-07-04"
"Posted to bioRxiv": "2023-06-09" (link: "Go to bioRxiv": "https://www.biorxiv.org/content/10.1101/2<IP_ADDRESS>4329v1/")
[PLACE PDF URL HERE WHEN AVAILABLE] See step 7
Step 1. Inform bioRxiv
Who can help: @QueenKraken, @nlisgo, @scottaubrey

[x] Manuscript is in data hub index (https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v1/by-publisher/elife/get-by-manuscript-id?manuscript_id=90100)

Step 2. Create preview of manuscript
Who can help: @fred-atherden, @nlisgo, @scottaubrey

[ ] Manuscript is available for preview (https://prod--epp.elifesciences.org/preview/90100/v1)

[ ] Trigger import for staging: https://prod--epp.elifesciences.org/import
[ ] Sync data folder with s3://prod-elife-epp-data/data
[ ] Trigger import for staging: https://staging--epp.elifesciences.org/import
[ ] Sync data folder with s3://staging-elife-epp-data/data
[ ] Pull request created to enhanced-preprint-data
[ ] Post link to pull request below

Pull request: [PLACE LINK TO PULL REQUEST HERE]
Detailed instructions: https://github.com/elifesciences/enhanced-preprints-data#add-a-manuscript
Step 3: Awaiting public reviews and QC
Who can help: Production team

[ ] QC passed or QC not needed

Who can help: Editorial team

[x] Public reviews available on sciety (https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>4329)
[x] Manuscript and editor details at the top of this issue (Supplied by Editorial team)
[ ] Reviews returned by EPP - https://prod--epp.elifesciences.org/api/reviewed-preprints/90100/v1/reviews

Example
"msas": "Genetics and Genomics", "Neuroscience"
"msid": "84628"
"version": "1"
"preprintDoi": "10.1101/2<IP_ADDRESS>4241"
"articleType": "Reviewed Preprint"
"status": "Published from the original preprint after peer review and assessment by eLife."

"Reviewed Preprint posted": "2023-01-02"
"Sent for peer review": "2022-10-28"
"Posted to bioRxiv": "2022-11-21" (link: "Go to bioRxiv": "https://www.biorxiv.org/content/10.1101/2<IP_ADDRESS>4241v1")

Editors:
Reviewing Editor
Michael B Eisen
University of California, Berkeley, United States
Senior Editor
Michael B Eisen
University of California, Berkeley, United States

Step 4: Modify manuscripts.json (no PDF)

[ ] Reviewed preprint is published to EPP (https://elifesciences.org/reviewed-preprints/90100)

[ ] Pull request created to enhanced-preprint-client
[ ] Pull request switched to "Ready for review"
[ ] Post link to pull request below

[ ] Request that a doi be registered

[ ] Post link to #enhanced-preprint comment below

Pull request: [PLACE LINK TO PULL REQUEST HERE]
#enhanced-preprint comment thread: [PLACE LINK TO COMMENT HERE]

Instructions to modify manuscripts.json

Visit: https://github.com/elifesciences/enhanced-preprints-client/actions/workflows/publish-manuscript.yaml
Click: Run workflow
Complete the form and click "Run workflow"
A successful run should result in a new pull request at https://github.com/elifesciences/enhanced-preprints-client/pulls
Open the pull request and click the "Ready for review" button to trigger tests
Once the tests pass and you are happy with the changes the PR can be merged

Example pull request: https://github.com/elifesciences/enhanced-preprints-client/pull/334/files
Once the pull request is merged in it should be available a few minutes later.

Request that a doi
Post the following in #enhanced-preprint:
@Fred can you register a doi for https://elifesciences.org/reviewed-preprints/90100

Step 5: Awaiting search reindex

[ ] Reviewed preprint is avaliable on journal homepage (https://elifesciences.org)

The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.

Additional info
If needed, the jenkins pipeline to reindex search can be triggered sooner.
https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 6: Published! Request PDF generation

[ ] Post the link to the PDF url at the top of the issue

[ ] Post link to #enhanced-preprint comment below

#enhanced-preprint comment thread: [PLACE LINK TO COMMENT HERE]
Post the following to the #enhanced-preprint on slack:
@Ryan Dix-Peek please can you generate a PDF for https://elifesciences.org/reviewed-preprints/90100

Step 7: Introduce PDF to data folder and git repo

[ ] PDF is avaliable at https://github.com/elifesciences/enhanced-preprints-data/raw/master/data/90100/v1/90100-v1.pdf

Detailed instructions: https://github.com/elifesciences/enhanced-preprints-data#add-a-pdf
Step 8: Add PDF url to manuscripts.json

[ ] Reviewed preprint PDF is available for download (https://prod--epp.elifesciences.org/reviewed-preprints/90100)

[ ] Pull request created to enhanced-preprint-client
[ ] Pull request switched to "Ready for review"
[ ] Post link to pull request below

[PLACE LINK TO PULL REQUEST HERE]

Instructions to add PDF url to manuscripts.json

Visit: https://github.com/elifesciences/enhanced-preprints-client/actions/workflows/add-pdf-url-to-manuscript.yaml
Click: Run workflow
Complete the form and click "Run workflow"
A successful run should result in a new pull request at https://github.com/elifesciences/enhanced-preprints-client/pulls
Open the pull request and click the "Ready for review" button to trigger tests
Once the tests pass and you are happy with the changes the PR can be merged

Example pull request: https://github.com/elifesciences/enhanced-preprints-client/pull/397/files
Once the pull request is merged in it should be available a few minutes later.

Step 9: Done!

[ ] Kettle is on!

Cristian GonzalezCabrera - is this author missing a space in their surname (same on eLife preprint and biorxiv? Or OK to proceed?


NullReferenceException

Hi
I try to use Smock like this
Smock.Run(context =>
{
    context.Setup(() => DateTime.Now).Returns(DateTime.Now.AddDays(1));
    context.Setup(() => DateTime.UtcNow).Returns(DateTime.UtcNow.AddDays(1));
    Task<IList<TEntity>> myTask = Add();
    myTask.RunSynchronously();
});

Then I get a null reference:
Smocks.AppDomains.AppDomainContext.InvokeSerializableLambda[T](SerializableLambda`1 serializableFunc, Object target, Object[] arguments)
   at Smocks.AppDomains.AppDomainContext.Invoke[T](Action`1 action, T parameter)
   at Smocks.Smock.RunAction(Action`1 action, Configuration configuration)
   at Smocks.Smock.Run(Configuration configuration, Action`1 action)
   at Smocks.Smock.Run(Action`1 action)

Is this a bug or a limitation?
EDIT:
For the same code, when running a different test I get another error
System.TypeLoadException : Could not load type 'TEntity' from assembly 'xy.xy.Shared.TesterCore, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=46adc5b7b509fbee'.

   at System.Reflection.RuntimeAssembly.GetType(RuntimeAssembly assembly, String name, Boolean throwOnError, Boolean ignoreCase, ObjectHandleOnStack type)
   at System.Reflection.RuntimeAssembly.GetType(String name, Boolean throwOnError, Boolean ignoreCase)
   at Smocks.IL.Resolvers.AssemblyTypeContainer.GetType(String name)
   at Smocks.IL.Resolvers.TypeResolver.Resolve(TypeReference type, GenericBindingContext bindingContext)
   at Smocks.IL.Resolvers.TypeResolver.<>c__DisplayClass3_0.<Resolve>b__1(TypeReference argument)
   at System.Linq.Enumerable.WhereSelectEnumerableIterator`2.MoveNext()
   at System.Linq.Buffer`1..ctor(IEnumerable`1 source)
   at System.Linq.Enumerable.ToArray[TSource](IEnumerable`1 source)
   at Smocks.IL.Resolvers.TypeResolver.Resolve(TypeReference type, GenericBindingContext bindingContext)
   at Smocks.IL.Resolvers.TypeResolver.<>c__DisplayClass3_0.<Resolve>b__1(TypeReference argument)
   at System.Linq.Enumerable.WhereSelectEnumerableIterator`2.MoveNext()
   at System.Linq.Buffer`1..ctor(IEnumerable`1 source)
   at System.Linq.Enumerable.ToArray[TSource](IEnumerable`1 source)
   at Smocks.IL.Resolvers.TypeResolver.Resolve(TypeReference type, GenericBindingContext bindingContext)
   at Smocks.IL.Resolvers.TypeResolver.Resolve(TypeReference type)
   at Smocks.IL.DynamicMethodCompiler.<>c__DisplayClass5_0.<AddVariables>b__1(VariableReference variable)
   at System.Linq.Enumerable.ToDictionary[TSource,TKey,TElement](IEnumerable`1 source, Func`2 keySelector, Func`2 elementSelector, IEqualityComparer`1 comparer)
   at System.Linq.Enumerable.ToDictionary[TSource,TKey,TElement](IEnumerable`1 source, Func`2 keySelector, Func`2 elementSelector)
   at Smocks.IL.DynamicMethodCompiler.AddVariables(IILGenerator generator, IEnumerable`1 variables)
   at Smocks.IL.DynamicMethodCompiler.Compile[T](TypeReference[] parameters, IEnumerable`1 instructions, IEnumerable`1 variables)
   at Smocks.IL.ExpressionDecompiler.Decompile(MethodBody body, Instruction instruction, Object target)
   at Smocks.IL.SetupExtractor.<GetSetupsFromInstructions>d__10.MoveNext()
   at System.Collections.Generic.List`1..ctor(IEnumerable`1 collection)
   at System.Linq.Enumerable.ToList[TSource](IEnumerable`1 source)
   at Smocks.Smock.CreateAssemblyRewriter(Delegate delegate, Configuration configuration)
   at Smocks.Smock.RunAction(Action`1 action, Configuration configuration)
   at Smocks.Smock.Run(Configuration configuration, Action`1 action)
   at Smocks.Smock.Run(Action`1 action)

Hi, thanks for reporting this. From the looks of it, TEntity is most likely a generic type parameter, and not an actual type. Can you confirm this? Can you expand the code example to a minimal, complete example that illustrates how/where TEntity is defined?

I'm calling a method which looks like that.
protected virtual async Task<IList<TEntity>> AddEntities(int amount, Action<TEntity> action)
{
    IList<TEntity> entityList = new List<TEntity>();
    for (int i = 0; i < amount; i++)
    {
        TEntity entity = CreateEntity();
        action(entity);
        var result = await Repository.InsertAsync(entity);
        result.EnsureIsValid();
        entityList.Add(entity);
    }

    return entityList;
}

Where the TEntity is
RepositoryFixture<TEntity, TFilter, TRepository> : Fixture where TEntity : IEntity where TFilter : IFilter where TRepository : IRepository<TEntity, TFilter>

This appears to be a bug. I will investigate and fix it in the near future. Thanks for reporting again!

Hi
To make it easier to reproduce I created a new project which contains the structure we use.
You can download it here: https://onedrive.live.com/redir?resid=C8E1BE3E79378699!631447&authkey=!AMlmgIqoGQ70pwI&ithint=file%2Czip
Here I use Moq to abstract the repository. In the real tests there is a repository which abstracts the Entity Framework to access the database. I'm not sure if this is relevant for this bug.
However, I'm also not able to get this project run. There is another exception which is thrown. May this is depended to the this issue.
Please let me know when you need additional help.
Greetings

Hello @vanderkleij
This bug is resolved or not? As I am facing the same issue of NullReferenceException while using Smocks. I tried to run the examples which you gave in the Readme file, they are giving the same exceptions.

Hello @vanderkleij @ujeshmaurya This bug is resolved or not? As I am facing the same issue

I am also today, trying to smock DateTime.Now / Today
Probably the most common use case and seems to be a common problem below Enterprise version...


Roadmap for GA of .Net Teams Bot Builder v4

Hello, are there any updates on the roadmap for when .Net Teams Bot Builder v4 will be GA?
I can see this article mentions GA around March 2019 but that doesn't seem to be the case anymore if I am reading correctly: https://developer.microsoft.com/en-us/office/blogs/preview-release-of-net-teams-bot-builder-v4-sdk/
The nuget package is still in beta: https://www.nuget.org/packages/Microsoft.Bot.Builder.Teams/
Thanks!

Hi, As I understand from next answer: https://github.com/microsoft/BotBuilder-Samples/issues/1736#issuecomment-526627922
BotBuilder team is working to have Teams extensions as part of BotBuilder. I believe it will available then

Is there a roadmap available somewhere for GA ?

We're currently targeting the 4.6 release of the Bot Framework SDK (early November). You can see additional initial samples here: https://github.com/microsoft/botbuilder-dotnet/tree/master/tests/Teams


Feature/2419 Reference statuses in Independent Assessments task

What's included?
Closes #2419
Related PR: digital-platform: Feature/admin 2419 Reference statuses in Independent Assessments task #1110
Who should test?
✅ Product owner
✅ Developers
✅ UTG
How to test?
Example exercise: https://jac-admin-develop--pr2420-feature-2419-referen-7tqgsgb3.web.app/exercise/IbeEVi4Vtrpv9eVFzhZq/tasks/all/independent-assessments
Steps:

Go to Independent Assessments task.
Check if the options and counts in the dropdown are generated based on the application statuses.

Risk - how likely is this to impact other areas?
🟢 No risk - this is a self-contained piece of work
Additional context
Demo:
https://github.com/jac-uk/admin/assets/79906532/2ed7b9f8-b25c-479d-acec-15ffa3448ca9
Related permissions
Have permissions been considered for this functionality?

No permission changes required
Permissions have been added / updated. Details:

PREVIEW:DEVELOP
can be OFF, DEVELOP or STAGING

@HalcyonJAC I believe this works, however, I just had a thought: this may never happen, but if there was a case where there was a small exercise and IAs was the first task, i.e. no QT, no sift before, would the application status appear as blank in the dropdown? And, if so, would users be able to select and progress all candidates with a blank status?

@nickaddy That case might happen indeed. For now, the blank status will not appear in the dropdown. I could make changes to support selecting candidates with a blank status if you would like.

@HalcyonJAC Better safe than sorry! Pls proceed with including the 'blank' option in the dropdown.

@nickaddy The Blank option is included in the dropdown. Could you test it, please?


[Feature Request] Add Telnet functionality

Hi,
I would love to see the addition of telnet to terminus. Working in the network field I still have a bunch of old equipment on the network that isn't ssh friendly. I would love to see a option in the SSH connection manager to select telnet when saving the connection as well!
Thanks

+1

+1

+1

+1

+1

+1

+1

+1
or netcat


Let's implement weight visualization feature

What?
Let's implement weight visualization feature.
Why?
It will help to detect model issues easily and speed up debugging.
How?
@seanshpark, @jyoungyun, @dayo09, @stamalakhov, let's discuss it here.

2D weights can be visualized as a heatmap that appears, for example, after clicking on a node in CircleViewer.
For 3+D weights the respective axes can be selected.
We may also discuss 3D visualization.

after clicking on a node in CircleViewer.

can you please share some sketch how it appears ?
can you please roughly describe user interaction and response of vscode?

@seanshpark,
I'm assuming we can handle node selection changed event from Netron (do you know if it's possible?).
So if the feature is enabled and a compatible node is selected a subwindow becomes visible.
The subwindow contains:

minimum and maximum weight values, weights shape, etc.
(if there are >2 axes with length > 1) a GUI to select 2 axes that should be used for heatmaps.
heatmap image(s) and a GUI to switch between different heatmaps.

Here is an example of existing software, that inspired us originally:

I will provide my GUI sketch as soon as I get familiar enough with ONE-vscode.

Thanks for the explanation! :)

from Netron (do you know if it's possible?).

I recommand to use CircleGraph which I modified as a common control(?) for graph view.
In media/CircleGraph/index.js, this handler is called when selection is changed.
You can follow caller or callee and understand how it works.
If you need more assist, please ping :)
  /**
   * @brief called from view.js for handle view events
   */
  onView(event) {
    if (event === "selection") {
      // there was a selection change and view requested to apply this
      let v = this._view.getSelection();
      vscode.postMessage({
        command: "selection",
        names: v.names,
        tensors: v.tensors,
      });
    }
  }

@seanshpark, thank you.

I also had a question about code in 'external' directory, can we modify it?

files in external folder is copied as-is from upstream, without modification and without format check. purpose of this management is to make upgrade(change apply) from upstream a bit easier in IDE.
if you like to modify it, please make a copy to it's parent, modify caller code (like index.html).
parts of the code may be modified by the format checker, but resulting be harder to read.
you can add some directives like // clang-format off to prevent it; you can refer from existing codes in CircleGraph folder, but there is no strict rules for this.

@seanshpark, I've implemented basic visualization draft:
https://github.com/staddy/ONE-vscode/commit/f2627b935d167b59bfae4f0f9368b703c8cccb2c

Planning to refactor the code and implement the following features:

Axes selector
Value selector
Value tooltip

Axes selector example:
https://github.com/staddy/ONE-vscode/commit/82bb3907d07d0833f287287af55fe3b572a8f9d7

@seanshpark, all planned items are implemented:
https://github.com/staddy/ONE-vscode/commit/82bb3907d07d0833f287287af55fe3b572a8f9d7
https://github.com/staddy/ONE-vscode/commit/e46680256dfab9480bebf940ba161d5b75f09e6c
https://github.com/staddy/ONE-vscode/commit/b35c070f7cafc7a0d74343da504c9e090a5a9fd3

Should I create a pull request now?
Also is it ok to have standalone functions there?

Should I create a pull request now?

PR with one context per changes with comments explaining the purpose
would help review to understand the codes :)

Also is it ok to have standalone functions there?

I can't catch the meaning of standalone functions and there. Anyway we can see with the PRs.

I don't have knowledge to understand the your new controls: axes selector and values selector,
and how they affect the graph rendering.
Some technical documents included in the change, not just codes, may help understand others
and to improve this functionality from others in the future.

Should I create a pull request now?

Please post a working draft PR and let others see all the codes and try out before landing.

I don't fully understand the changes so below comments may be incorrect
I assume your current changes are for property side-bar window only.
I don't want you to change default behavior of the property side-bar of the graph view; you should add a flag to enable/disable it
Working draft means from the draft code, anyone can try out the new feature, but functionality may be imperfect with bugs.

I think it would help if you can describe detailed UI to show weight visualization of the circle model.


TotalVariation multiplication by scalar

In the regularisation toolkit plugin we define the __rmul__ which does not return a ScaledFunction, rather it multiplies the scalar multiplier to the function in the inner solver's regularisation parameter:

f = alpha * FGP_TV()

https://github.com/TomographicImaging/CIL/blob/b70febf37b215dd6672ddc38b05effdecfb6a086/Wrappers/Python/cil/plugins/ccpi_regularisation/functions/regularisers.py#L258-L266
In TotalVariation the same method just uses the multiplier. Is it correct?
https://github.com/TomographicImaging/CIL/blob/b70febf37b215dd6672ddc38b05effdecfb6a086/Wrappers/Python/cil/optimisation/functions/TotalVariation.py#L369-L373

It is wrong as if one does
from cil.optimisation.functions import TotalVariation

tv = TotalVariation()
print (tv.regularisation_parameter)
# >>> returns 1

#%%
alpha = 2.
tv2 = alpha * TotalVariation()
print (tv2.regularisation_parameter)
# >>> returns 2

beta = 3.
tv3 = alpha * (beta * TotalVariation() )
print (tv3.regularisation_parameter)
# >>> returns 2 instead of 6


enable to pass lessc and uglifyjs pathes to make

The makefile defines LESS_COMPRESSOR which is never used.

don' think this is neccessary - if anything, we should remove the lesscompressor var


libwayland-dev is a make dependacy not mentioned.

I went to compile this on Mobian, and was getting meson.build:26:0: ERROR: Dependency "wayland-client" not found, tried pkgconfig and cmake, I come to find out libwayland-dev is needed.
Sorry if this isn't the way things are supposed to be done, I don't engage with GitHub much.

You are indeed correct, the Ubuntu (and possibly Debian and other derivatives) dependency line does not mention the Wayland developer packages needed. Looks like an oversight.

Thanks for opening this issue, I was hitting this problem and trying to figure out which dependency I was missing.


"error 37 from memcached_set: SUCCESS" when setting value > 1 MB

With libmemcached 1.0.18 (OS X, with homebrew patches), I get an unexpected error value when attempting to set a value greater than 1 MB:
>>> import pylibmc
>>> mc = pylibmc.Client(["<IP_ADDRESS>"])
>>> mc['foo'] = 'a' * 1024 * 1024
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "pylibmc/src/pylibmc/client.py", line 163, in __setitem__
    if not self.set(key, value):
pylibmc.Error: error 37 from memcached_set: SUCCESS

I'm expecting an error, because of the 1 MB limit, but I was hoping for a more specific error, and maybe one that includes the length of the value after serialization and compression (see https://github.com/django-pylibmc/django-pylibmc/pull/29).
This sounds like a similar issue as #180, but without the binary protocol and behaviors used in that test case.

Hello @jwhitlock! Sadly, as is the case for #180, this is not something we can realistically do anything about. Perhaps it's time we rolled up our sleeves and started taking on active development of libmemcached. Until that time I'm afraid I can't do much for the sometimes confusing error messages and other behaviors in libmemcached. Other than that I am open to ideas!

c and bzr and launchpad - that's a bit of a technical hill to climb, and I feel I'd need to get comfortable with _pylibmcmodule.c first. But might be worth it to nail down why these errors aren't as useful as they could be.
Another thing that might help is exposing the value serialization, so that I can add the length of the value to exceptions, with extra emphasis if they go over the 1 MB limit.  That might be _PylibMC_SerializeValue, but it looks like key preparation might be non-trivial.

Is it possible to increase the max value size to be larger than 1MB? I could not find this in the docs

Sorry, no, it's not really something we defined a limit for. I'm not sure if it's from the protocol itself or just libmemcached, though.

It's the famous 1MB upper limit in memcached itself. I had to dig into this a few years ago, when the project was hosted on code.google.com and there was a FAQ with the explanation of why. I seem to remember it boiling down to "if you need to cache something that big, you are doing it wrong", that TCP overhead would start to get ridiculous.  It looks like you can increase it up to 128MB on the command line, if you have that server access, with something like memcached -I 128m.  There may be a way to request the setting value over the API, if we wanted to do something smarter, but we'd have to guess at the overhead of storage structures, so it might not be worth it.

Understood, thank you.

It seems to me that we can't do much about this; closing for lack of means of resolution.

Hi all, sorry for posting on a closed thread, but does anyone know if this "error 37 from memcached_set: SUCCESS" error with libmemcached persists?
I see references across the web of this error, but I can't reproduce it.  If it still exists, I have to handle it gracefully in my application logic, but I'm wondering if it's actually been fixed in a recent update to libmemcached?

Yes, it reproduces for me, running the above code, with:

memcached 1.4.24
pylibmc 1.5.1
libmemcached 1.0.18_1 installed via Homebrew
OS X El Capitan 10.11.6

Thanks!  I ended up using django_pylibmc to configure memcached with my application, and it handles this error gracefully by default.

django_pylibmc uses compression, and 1,000,000 copies of 'a' compresses to much less than 1 MB of data.  You'll need to pass more than 1MB of random data to actually store 1 MB of data and trigger the error. In that case, the error is logged and False is returned. Your app will have to be OK with not caching your big value, maybe silently.

Hm, sounds like a poor strategy to silence an error. No warnings, nothing?

django_pylibmc would like to do something more intelligent about the upper limit, like in https://github.com/django-pylibmc/django-pylibmc/pull/29, but the feedback we get from pylibmc is a generic error that doesn't include the data we'd need to, for example, to switch to chunking a large value into pieces that memcached would like.  An example would be an error that communicated that the stored item was too large to fit in a memcached bucket, and then we could chunk the value into half-sized chunks until it fit.  Even better would be to communicate what the memcached bucket size is (1MB by default, but can be changed).
The best django_pylibmc can do by default is log the error, since Django doesn't expect cache calls to raise exceptions. If a client wants to do more than that, they can extend the class and try something else.
I opened this bug in the hope that pylibmc could raise a more specific error than "Error: Success!"  I understand that may mean diving into libmemcached, or maybe even memcached, which is barely surviving the transition to GitHub.  I'm not doing it, so I can't really expect anyone else to do it either.

[…] the feedback we get from pylibmc is a generic error that doesn't include the data we'd need […]

Right, I see the problem, it’s just that pylibmc is in essentially the same situation: we don’t have the information needed to give you a proper error, likely because memcached doesn’t even send an error but rather just closes the connection or something. I’m not sure but I would suspect it to be this way given how the error comes up.

The best django_pylibmc can do by default is log the error https://github.com/django-pylibmc/django-pylibmc/blob/master/django_pylibmc/memcached.py#L145-L147 […]

Isn't that enough, though?

I opened this bug in the hope that pylibmc could raise a more specific error than "Error: Success!" I understand that may mean diving into libmemcached, or maybe even memcached, which is barely surviving the transition to GitHub. I'm not doing it, so I can't really expect anyone else to do it either.

I'd think the libmemcached people should be interested in fixing this, but you’re right, it’s maybe not the funnest of bugs to go at.

So curiosity got the better of me, here are my findings:
$ nc -cv <IP_ADDRESS> 11211
found 0 associations
found 1 connections:
     1:	flags=82<CONNECTED,PREFERRED>
	outif lo0
	src <IP_ADDRESS> port 51561
	dst <IP_ADDRESS> port 11211
	rank info not available
	TCP aux info available

Connection to <IP_ADDRESS> port 11211 [tcp/*] succeeded!
set a 0 0 2
h

Works well.
set a 0 0<PHONE_NUMBER>
CLIENT_ERROR bad command line format
set a 0 0<PHONE_NUMBER>
CLIENT_ERROR bad command line format
set a 0 0<PHONE_NUMBER>
SERVER_ERROR object too large for cache

Don't work so well. The first one is 2^31, the second one is one less, the third one is 2^30 - 1. It turns out the breakpoint is at exactly 2^30 - 16 -- presumably 16 bytes of bookkeeping is included in the size limit.

Surprisingly, this is a very detectable and reportable error. It should in fact be triggering pylibmc.ClientError and pylibmc.ServerError respectively.

Also, the error pylibmc triggers for 2^31 byte values:
pylibmc.Error: error 26 from memcached_set: Socket is not connected

Whereas for 2^30 byte values, we get
pylibmc.Error: error 37 from memcached_set: SUCCESS

 It's an issue with pylibmc!

Well, maybe I got your hopes up with my last message. However, as it turns out, I can really only fix the latter of the previous two cases, the midpoint case. It also turns out my math was off somehow, it seems memcached gives a server error when 2**20 - 72 < len(value) < 2**31 - 2.
It also seems that when memcached sends a SERVER_ERROR, it also disconnects you. This is why we get "socket is not connected."

So I think this is all I can do for now. The following program now works with libmemcached 1.0.2 and above:
import pylibmc

mc = pylibmc.Client(['<IP_ADDRESS>:11211'])

try:
    mc.set('a', v)
except pylibmc.TooBig:
    print('(sad trumpet) value was too big')


Code freeze on inference

Hi,
When I run the inference code , it freezes almost each time. No error comes, but the code freezes randomly.
Is this a common issue? Any suggestions on how to fix this?
Thanks!

Might related to the transformers version issue. Try roll back it to transformers==3.4.0


Shared Library 'gdx.dll' not found if '!' is in the foldername

Shared Library 'gdx.dll' not found if '!' is in the foldername.
Tested it on Windows 8 & Windows 7
Exception in thread "main" com.badlogic.gdx.utils.GdxRuntimeException: Couldn't load shared library 'gdx.dll' for target: Windows 8.1, 32-bit
at com.badlogic.gdx.utils.SharedLibraryLoader.load(SharedLibraryLoader.java:118)
at com.badlogic.gdx.utils.GdxNativesLoader.load(GdxNativesLoader.java:34)
at com.badlogic.gdx.backends.lwjgl.LwjglNativesLoader.load(LwjglNativesLoader.java:47)
at com.badlogic.gdx.backends.lwjgl.LwjglApplication.(LwjglApplication.java:76)
at com.badlogic.gdx.backends.lwjgl.LwjglApplication.(LwjglApplication.java:64)
...
Caused by: com.badlogic.gdx.utils.GdxRuntimeException: Unable to read file for extraction: gdx.dll
at com.badlogic.gdx.utils.SharedLibraryLoader.readFile(SharedLibraryLoader.java:126)
at com.badlogic.gdx.utils.SharedLibraryLoader.loadFile(SharedLibraryLoader.java:261)
at com.badlogic.gdx.utils.SharedLibraryLoader.load(SharedLibraryLoader.java:115)
... 5 more

I think that's a Windows issue, not libgdx.  The ! symbol in a file path is probably a no-no.  It would be like using % or * in the name.

Probably something like this: http://bugs.java.com/view_bug.do?bug_id=4523159
I'm afraid there's not much we can do about it. Relevant code: https://github.com/libgdx/libgdx/blob/1.6.5/gdx/src/com/badlogic/gdx/utils/SharedLibraryLoader.java#L125-L126

Sounds like there is nothing we can do here.


Feature Request: Change Entry Details

an option to change existing entries (name, set last executed date) would be great.
currently I have to change the activities.json file and restart home assistant - works, but things can go wrong this way...

You can also call activity_manager.update_activity from developer tools to update existing entires


make reason for unsupported feature a keyword arg

Purpose or Intent
this reads better and is more aligned with the way rails
does DSL style class methods
Before:
supports_not :feature, 'because'
Now:
supports_not :feature, :reason => 'because'
props to @chrisarcand for suggesting this
@miq-bot add_label refactoring, darga/no
@miq-bot assign Fryguy

@chrisarcand review?
@Fryguy merge?
Thanks :smile:

Looks good 👍  Clearly shows intent to users of the API and allows for any future options you might want to add to it regardless of order and ambiguity as to which is what.

@miq-bot add_label darga/yes
@chessbyte this one needs backporting too

@miq-bot remove_label darga/no

@durandom Backporting to darga via git cherry-pick -x -m 1  d51a3d0 results in merge conflicts.
Are there other PRs I need to backport before this one?

@chessbyte  yes, this one https://github.com/ManageIQ/manageiq/pull/9271

Darga backport details:
commit bf98dfde539479ae1a3c3aeef392269798b4d291
Author: Jason Frey<EMAIL_ADDRESS>Date:   Thu Jul 7 11:18:14 2016 -0400

    Merge pull request #9654 from durandom/supports_feature_supports_not_kwargs

    make reason for unsupported feature a keyword arg
    (cherry picked from commit d51a3d0e7a7c5cadf0044ef77345c1ae8f40937d)


Save ICC color profile [enhancement]

Hi jwa107,
Nice contribution I use this many times. My life is more easier. But I have a problem, the ICC color profile is not saved when I generate PNG24 files.
It is the only feature missing to make it perfect. I is the unique feature that provides the official plugin of Photoshop but not this.
thanks,
Mossman

Hmm, that will take some work and investigation.
Basically, the ICC option for the export script was added later. The original script in CS2 couldn't even export PNG at all. In CS6, I can see the option on the original script's dialog, but the scripting API doesn't contain the corresponding option in PNGSaveOptions. Does it work?
Anyway, we're using quick and dirty action manager code the same way the "Save For Web" interface does. And I couldn't see an "Embed color profile" option there. So I'm not sure it's even supported. If anyone wants to investigate further, feel free.


Update Pool Box

Hi, here's some initial work for #49.
I have some questions.

Is there any case where a user might want to update the reward tokens used in the oracle, without making any other changes (new refresh NFT, etc)? Because right now, the update contract requires that the new pool's script be different from the old one, so simply replacing say old reward tokens with a new token, or adding more tokens to the pool is not possible.
Please check my comment in test_update_pool_box commit

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

156 of 182 (85.71%)  changed or added relevant lines in 6 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+1.9%) to 66.275%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

core/src/box_kind/update_box.rs
15
19
78.95%

core/src/scans.rs
0
5
0.0%

core/src/oracle_state.rs
0
7
0.0%

core/src/cli_commands/update_pool.rs
135
145
93.1%

Totals

Change from base Build<PHONE_NUMBER>:
1.9%

Covered Lines:
1466

Relevant Lines:
2212

💛  - Coveralls

Currently trying to get a simple update operation working on testnet, unfortunately it's being a bit spotty right now in terms of blocks.
Also, I was wondering what the interface should be like for updating. Currently I have a rather simple setup where update-pool reads oracle_config.yaml, and compares refresh NFT/update NFT tokens to the pool box's parameters and sets it up from that. (No support for new reward tokens currently in the interface). Ideally though we should have some sort of partial bootstrap feature, since right now I'm testing it with a refresh NFT I minted myself and that isn't guarded by any contract.

Currently trying to get a simple update operation working on testnet, unfortunately it's being a bit spotty right now in terms of blocks.

Great!

Also, I was wondering what the interface should be like for updating. Currently I have a rather simple setup where update-pool reads oracle_config.yaml, and compares refresh NFT/update NFT tokens to the pool box's parameters and sets it up from that. (No support for new reward tokens currently in the interface). Ideally though we should have some sort of partial bootstrap feature, since right now I'm testing it with a refresh NFT I minted myself and that isn't guarded by any contract.

Oh yeah, the new refresh NFT should be in a box with the new refresh contract. Otherwise, oracles are voting for a blank check, right? Since we need to have a new refresh contract with a new refresh NFT before the voting, I suppose a new command is in order. Something like `UpdateRefreshContract`` which mints the new refresh NFT and puts it in the box with the new refresh contract.
@kettlebell  What do you think?

Hmm, the method I had in mind was to rework/copy the bootstrap steps, but allow specifying tokens/contracts for each step in the process. So if for example, you provide a pool Box, it won't mint a new one. Then you could maybe have something like update_config.yaml, that has optional parameters for each contract, something like:
#[derive(Serialize, Deserialize)]
struct UpdateConfig {
    update_contract_parameters: Option<UpdateContractInputs>,
    refresh_contract_parameters: Option<RefreshContractInputs>, //and so on.
}

That's interesting. I see how the update is similar to the bootstrap.
However, I'd refrain from adding complexity to the bootstrap code. We need to remember about the maintenance and the bar for new developers.
I'd prefer to have a separate code for contract updates sharing some generic/common code with bootstrap. I believe updating will be the most complex process we have. That's a good reason to compartmentalize it so that it does not leak into existing commands.

The UpdateRefreshContract command sounds good to me.

https://testnet.ergoplatform.com/en/transactions/abea0c5e6866bf8859b0ae004e864df0ef826fe4939569689fb5759436ca79d2
Successfully did a pool update on testnet with new refresh NFT, albeit I had to mint it myself for the time being

https://testnet.ergoplatform.com/en/transactions/abea0c5e6866bf8859b0ae004e864df0ef826fe4939569689fb5759436ca79d2
Successfully did a pool update on testnet with new refresh NFT, albeit I had to mint it myself for the time being

Nice!

During the above mentioned testnet try, I ran into a small issue regarding the Update Contract. I had some ballot tokens still in my wallet, and as we need an input box from wallet for fees, the update contract failed because the registers were not defined:
  def isValidBallot(b:Box) = if (b.tokens.size > 0) {
    b.tokens(0)._1 == ballotTokenId       && //Wallet contained the token
    b.R5[Int].get == SELF.creationInfo._1 && // Called Option.get on None
    b.R6[Coll[Byte]].get == poolOutHash   &&
    b.R7[Coll[Byte]].get == rewardTokenId &&
    b.R8[Long].get == rewardAmt
  } else false

I don't think however that we need to update the contract, in theory adding a check for b.R8[Any].isDefined before accessing the registers would work, but it'd increase the size of the contract. Second, eventually we'd have a way to transfer ballot tokens from oracle-core, so there wouldn't be any need to keep the ballot tokens in the wallet anymore. For now I guess I could add a warning/error if the wallet address contains ballot tokens

Hi, here's some initial work for #49.
I have some questions.
1. Is there any case where a user might want to update the reward tokens used in the oracle, without making any other changes (new refresh NFT, etc)? Because right now, the update contract requires that the new pool's script be different from the old one, so simply replacing say old reward tokens with a new token, or adding more tokens to the pool is not possible.

This requirement is now removed from the contract. See https://github.com/ergoplatform/eips/pull/41#discussion_r933663637 . Please, update the contract on our side.

During the above mentioned testnet try, I ran into a small issue regarding the Update Contract. I had some ballot tokens still in my wallet, and as we need an input box from wallet for fees, the update contract failed because the registers were not defined:
  def isValidBallot(b:Box) = if (b.tokens.size > 0) {
    b.tokens(0)._1 == ballotTokenId       && // Wallet box contained the token
    b.R5[Int].get == SELF.creationInfo._1 && // Called Option.get on None
    b.R6[Coll[Byte]].get == poolOutHash   &&
    b.R7[Coll[Byte]].get == rewardTokenId &&
    b.R8[Long].get == rewardAmt
  } else false

I don't think however that we necessarily need to update the contract, in theory adding a check for b.R8[Any].isDefined before accessing the registers would work, but it'd increase the size of the contract. Second, eventually we'd have a way to transfer ballot tokens from oracle-core, so there wouldn't be any need to keep the ballot tokens in the wallet anymore. For now I guess I could add a warning/error if the wallet address contains ballot tokens

Nice find! I wonder if ballot tokens would be all sent out to the operators. I mean, I can imagine "bootstraper" can have some extra ballot tokens left in their wallet for future participants and whatnot. OTOH, update pool box operation most likely would be carried out after the pool had been working for quite some time. So it's not that unreasonable to require "no extra ballot tokens" for the operator that runs the update operation. I'd rather leave it as is with a warning. @kettlebell WDYT?
BTW, I did not plan any command to send out the ballot tokens. It can be done with a regular send command in the wallet.

Nice find! I wonder if ballot tokens would be all sent out to the operators. I mean, I can imagine "bootstraper" can have some extra ballot tokens left in their wallet for future participants and whatnot. OTOH, update pool box operation most likely would be carried out after the pool had been working for quite some time. So it's not that unreasonable to require "no extra ballot tokens" for the operator that runs the update operation. I'd rather leave it as is with a warning. @kettlebell WDYT?
BTW, I did not plan any command to send out the ballot tokens. It can be done with a regular send command in the wallet.

Yeah I agree that the bootstrapper could very well have extra ballot tokens especially if he wants to keep control of the updating process until things settle and the other participants are trustworthy enough to deserve a ballot token.
It's extra work on your part @SethDusek, but how about excluding any ballot-holding boxes from the wallet for the inputs? In theory this could mean no boxes would be available for fees, then we could return an error for the operator to remedy this. What do you guys think?

It's extra work on your part @SethDusek, but how about excluding any ballot-holding boxes from the wallet for the inputs? In theory this could mean no boxes would be available for fees, then we could return an error for the operator to remedy this. What do you guys think?

I like this idea! I mean most likely there will be enough boxes for a fee without a box with ballot tokens. Otherwise, a fatal error is in order with a recommendation to move ballot tokens to another wallet.

This should be good to go now, along with a fix for #83. There are some other fixes to get oracle-core working on testnet as well, I was able to perform another update transaction, this time with a refresh NFT minted by the new update command
https://testnet.ergoplatform.com/en/transactions/350c9e058ace05470d2e0c4e0b8f86c3ef77322df32338eba311b00d45281a54

Thank you for the detailed review, I'll work on the changes and re-request review when it's ready.


Fix 9332

What this PR does / why we need it:
This PR fixes issue 9332, which is about global values defined in subchart can affect globals in parent chart or sibling charts.
Special notes for your reviewer:
The fix consists of 3 commits.

a unit test enhancement, which failed before the fix was implemented.
the actual fix, which was just a missing 'continue' statement, such that the intended code was later overrule
an additional code improvement of coalesce.go to get rid or continue statement and using appropriate else
blocks instead. This is not actually necessary for the fix, but should improve the code to make a similar issue
less likely.

If applicable:

[X] this PR contains unit tests

closes #9332

Scheduling for 3.7.0 so we don’t miss this during review.
Can you please rebase this?

After rebasing, I found that the defect has already been fixed end of May 2021 with commit 3b68afc1c820d456e8281a6e6911af59c4b661c8 on the main branch. So I assume that fix will already be part of 3.7.0.
What remains now are two commits in the PR. One with additional coalesce test cases, and the other with a code approval in coalesce.go by getting rid of continue statements.
Feel free to merge these improvements, I you find them valuable. Since the actual defect is fixed, I won't prolong the PR again after the next 90 days expiration.

@bgutjahr can you please fix the failing test.


provide a way to prevent DataRow to receive focus between rows up and down using up and down keys

issue :-   i want to move focus between rows up and down using keyboard Up and Down Keys .
D:\tag\flutter_datatable>flutter doctor -v
[√] Flutter (Channel master, 1.26.0-2.0.pre.365, on Microsoft Windows [Version 10.0.19042.746], locale en-US)
    • Flutter version 1.26.0-2.0.pre.365 at D:\flutter
    • Framework revision 9209319dd1 (2 hours ago), 2021-01-15 14:34:03 +0530
    • Engine revision efa768683e
    • Dart version 2.12.0 (build 2.12.0-225.0.dev)

[√] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at C:\Users\Administrator\AppData\Local\Android\sdk
    • Platform android-30, build-tools 30.0.2
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop for Windows (Visual Studio Community 2019 16.8.2)
    • Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\Community
    • Visual Studio Community 2019 version 16.8.30717.126
    • Windows 10 SDK version 10.0.18362.0

[√] Android Studio (version 4.1.0)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)

[√] Connected device (3 available)
    • Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version 10.0.19042.746]
    • Chrome (web)      • chrome  • web-javascript • Google Chrome 87.0.4280.141
    • Edge (web)        • edge    • web-javascript • Microsoft Edge 87.0.664.75

• No issues found!

return DataTable(
      columns: const <DataColumn>[
        DataColumn(
          label: Text(
            'Name',
            style: TextStyle(fontStyle: FontStyle.italic),
          ),
        ),
        DataColumn(
          label: Text(
            'Qty',
            style: TextStyle(fontStyle: FontStyle.italic),
          ),
        ),
      ],
      rows: const <DataRow>[
        DataRow(
          cells: <DataCell>[
            DataCell(
            TextField(
            )),
            DataCell(
                TextField(
                )),
          ],
        ),
        DataRow(
          cells: <DataCell>[
            DataCell(
                TextField(
                )),
            DataCell(
                TextField(
                )),
          ],
        ),
        DataRow(
          cells: <DataCell>[
            DataCell(
                TextField(
                )),
            DataCell(
                TextField(
                )),
          ],
        ),
      ],
    );

@abdulrehmananwar
this issue seems more a question than a bug report or a feature request
if you need assistance with your code please follow the link I want help writing my application in the new issue menu

@abdulrehmananwar
this issue seems more a question than a bug report or a feature request
if you need assistance with your code please follow the link I want help writing my application in the new issue menu

tion than a bug report or a feature request

please help to resolve this issue

tion than a bug report or a feature request

please help to resolve this issue

issue :- i want to move focus between rows up and down using keyboard Up and Down Keys .

this issue seems more a question than a bug report or a feature request, if you need assistance with your code please follow the link I want help writing my application in the new issue menu

please help to resolve this issue

@abdulrehmananwar  are you

reporting a bug with flutter
requesting a feature
asking help writing your code

what is the issue?

issue :- i want to move focus between rows up and down using keyboard Up and Down Keys .

this issue seems more a question than a bug report or a feature request, if you need assistance with your code please follow the link I want help writing my application in the new issue menu

please help to resolve this issue

@abdulrehmananwar  are you

reporting a bug with flutter
requesting a feature
asking help writing your code

what is the issue?

requesting a feature

requesting a feature

requesting a feature

@abdulrehmananwar
could you please provide the information listed in the feature request template located in the new issue selection
<!-- Thank you for using Flutter!

     If you are looking for support, please check out our documentation
     or consider asking a question on Stack Overflow:
      * https://flutter.dev/
      * https://api.flutter.dev/
      * https://stackoverflow.com/questions/tagged/flutter?sort=frequent

     If you have found a bug or if our documentation doesn't have an answer
     to what you're looking for, then fill out the template below. Please read
     our guide to filing a bug first: https://flutter.dev/docs/resources/bug-reports
-->

## Use case

<!--
     Please tell us the problem you are running into that led to you wanting
     a new feature.

     Is your feature request related to a problem? Please give a clear and
     concise description of what the problem is.

     Describe the alternative solutions you've considered. Is there a package
     on pub.dev/flutter that already solves this?
-->

## Proposal

<!--
     Briefly but precisely describe what you would like Flutter to be able to do.

     Consider attaching images showing what you are imagining.

     Does this have to be provided by Flutter directly, or can it be provided
     by a package on pub.dev/flutter? If so, maybe consider implementing and
     publishing such a package rather than filing a bug.
-->

requesting a feature

@abdulrehmananwar
could you please provide the information listed in the feature request template located in the new issue selection
<!-- Thank you for using Flutter!

     If you are looking for support, please check out our documentation
     or consider asking a question on Stack Overflow:
      * https://flutter.dev/
      * https://api.flutter.dev/
      * https://stackoverflow.com/questions/tagged/flutter?sort=frequent

     If you have found a bug or if our documentation doesn't have an answer
     to what you're looking for, then fill out the template below. Please read
     our guide to filing a bug first: https://flutter.dev/docs/resources/bug-reports
-->

## Use case

<!--
     Please tell us the problem you are running into that led to you wanting
     a new feature.

     Is your feature request related to a problem? Please give a clear and
     concise description of what the problem is.

     Describe the alternative solutions you've considered. Is there a package
     on pub.dev/flutter that already solves this?
-->

## Proposal

<!--
     Briefly but precisely describe what you would like Flutter to be able to do.

     Consider attaching images showing what you are imagining.

     Does this have to be provided by Flutter directly, or can it be provided
     by a package on pub.dev/flutter? If so, maybe consider implementing and
     publishing such a package rather than filing a bug.
-->

@abdulrehmananwar
Also, is this for desktop platform ?

@abdulrehmananwar
Also, is this for desktop platform ?

this is for web and desktop. it could be also on mobile with keyboard attachment

this is for web and desktop. it could be also on mobile with keyboard attachment

ok i have submited this as a new feature request thanks. now i am closing this from here

ok i have submited this as a new feature request thanks. now i am closing this from here


0.8 Release Fallout

Address issues discovered after release
Link or tag issues large enough to deserve own tickets, otherwise comment or edit this ticket for small changes.

Fixed issue with cases where claim.meta could be null: https://github.com/lbryio/lbry-redux/commit/e10986e8e54d25480ac2b5dd2a31fec8b254471a


ESP32 - GPIO21 & GPIO22 default value GPIO_FUNCn_OUT_SEL (IDFGH-12733)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

IDF version.
v5.2.1
Espressif SoC revision.
ESP32-D0WDQ6 (revision v1.0)
Operating System used.
macOS
How did you build your project?
CLion IDE
If you are using Windows, please specify command line type.
None
Development Kit.
Dev Kit with ESP32-WROOM-32
Power Supply used.
USB
What is the expected behavior?
According to the Technical Reference Manual (5.1), the value of the 'GPIO_FUNCn_OUT_SEL' bits in the register 'GPIO_FUNCn_OUT_SEL_CFG_REG' (n=21 &n=22) is a valid peripheral output signal (Table 42. GPIO Matrix Peripheral Signals) or the value '256'.
What is the actual behavior?
The default value of the bits is 0x101 (257), for both (GPIO 21 && GPIO 22).
Steps to reproduce.
Code:
#include <esp_log.h>
#include <freertos/FreeRTOS.h>
#include <freertos/task.h>

#define GPIO_FUNCn_OUT_SEL_CFG_REG(n) (0x3FF44000 + 0x530 + 0x4 * n)

void app_main()
{
  ESP_LOGE("GPIO 21 GPIO_FUNCn_OUT_SEL", "0x%lx", REG_READ(GPIO_FUNCn_OUT_SEL_CFG_REG(21)) & 0x1FF);
  ESP_LOGE("GPIO 22 GPIO_FUNCn_OUT_SEL", "0x%lx", REG_READ(GPIO_FUNCn_OUT_SEL_CFG_REG(22)) & 0x1FF);
  while (true)
  {
    vTaskDelay(5000 / portTICK_PERIOD_MS);
  }
}

Debug Logs.
mode:DIO, clock div:2
load:0x3fff0030,len:7172
load:0x40078000,len:15540
load:0x40080400,len:4
ho 8 tail 4 room 4
load:0x40080404,len:3904
entry 0x40080640
I (31) boot: ESP-IDF v5.2.1 2nd stage bootloader
I (31) boot: compile time Apr 27 2024 18:19:07
I (31) boot: Multicore bootloader
I (35) boot: chip revision: v1.0
I (39) boot.esp32: SPI Speed      : 40MHz
I (44) boot.esp32: SPI Mode       : DIO
I (48) boot.esp32: SPI Flash Size : 2MB
I (53) boot: Enabling RNG early entropy source...
I (58) boot: Partition Table:
I (62) boot: ## Label            Usage          Type ST Offset   Length
I (69) boot:  0 nvs              WiFi data        01 02 00009000 00006000
I (76) boot:  1 phy_init         RF data          01 01 0000f000 00001000
I (84) boot:  2 factory          factory app      00 00 00010000 00100000
I (91) boot: End of partition table
I (96) esp_image: segment 0: paddr=00010020 vaddr=3f400020 size=0957ch ( 38268) map
I (117) esp_image: segment 1: paddr=000195a4 vaddr=3ffb0000 size=0220ch (  8716) load
I (121) esp_image: segment 2: paddr=0001b7b8 vaddr=40080000 size=04860h ( 18528) load
I (130) esp_image: segment 3: paddr=00020020 vaddr=400d0020 size=13414h ( 78868) map
I (159) esp_image: segment 4: paddr=0003343c vaddr=40084860 size=07ebch ( 32444) load
I (178) boot: Loaded app from partition at offset 0x10000
I (178) boot: Disabling RNG early entropy source...
I (189) cpu_start: Multicore app
I (198) cpu_start: Pro cpu start user code
I (198) cpu_start: cpu freq: 160000000 Hz
I (198) cpu_start: Application information:
I (201) cpu_start: Project name:     IO
I (206) cpu_start: App version:      1.0.0
I (211) cpu_start: Compile time:     Apr 27 2024 18:19:02
I (217) cpu_start: ELF file SHA256:  3c45c323e...
I (222) cpu_start: ESP-IDF:          v5.2.1
I (227) cpu_start: Min chip rev:     v0.0
I (232) cpu_start: Max chip rev:     v3.99
I (236) cpu_start: Chip rev:         v1.0
I (241) heap_init: Initializing. RAM available for dynamic allocation:
I (248) heap_init: At 3FFAE6E0 len 00001920 (6 KiB): DRAM
I (254) heap_init: At 3FFB2AD0 len 0002D530 (181 KiB): DRAM
I (261) heap_init: At 3FFE0440 len 00003AE0 (14 KiB): D/IRAM
I (267) heap_init: At 3FFE4350 len 0001BCB0 (111 KiB): D/IRAM
I (273) heap_init: At 4008C71C len 000138E4 (78 KiB): IRAM
I (281) spi_flash: detected chip: generic
I (284) spi_flash: flash io: dio
W (288) spi_flash: Detected size(4096k) larger than the size in the binary image header(2048k). Using the size in the binary image header.
I (302) main_task: Started on CPU0
I (312) main_task: Calling app_main()
E (312) GPIO 21 GPIO_FUNCn_OUT_SEL: 0x101
E (312) GPIO 22 GPIO_FUNCn_OUT_SEL: 0x101

### More Information.

_No response_

I would suggest not rely on the default values that were documented in the TRM, because it may be changed in the ROM code if these GPIOs have another special purpose. So always call the APIs provided by the driver/gpio.h for initializing the GPIO.

Hi @nkaaf,
In the TRM, it didn't say this field has a default value of 256 (shows x in the screenshot from the TRM). It only says if you want a GPIOn to be used as a simple GPIO output, then this field needs to be 256.

Hi @songruo , ok, I realize that. I found it strange that the value is outside the range of a valid value according to TRM. Do you know if the value 257 means anything, or where this value comes from?

Now we add a note that shouldn't rely on default values in docs


Key-value store

Currently the key-value store layer / backend is duplicated in koinos-contract-meta-store, koinos-transaction-store, and konios-block-store. Move it here and have them all reference it.

Will be reimplemented in C++ using state_db.


SwiperSlide don't work if I extract it into a component.

Check that this is really a bug

[X] I confirm

Reproduction link
https://codesandbox.io/s/determined-wing-61o46l?file=/src/Carousel.tsx
Bug description
I'm trying to extract the SwiperSlide to an external component to give my personal props, but when I use it as a children from the Swiper it doesn't render as espected. See example
Expected Behavior
No response
Actual Behavior
No response
Swiper version
9.1.1
Platform/Target and Browser Versions
Ubuntu
Validations

[X] Follow our Code of Conduct
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.
[X] Make sure this is a Swiper issue and not a framework-specific issue

Would you like to open a PR for this bug?

[ ] I'm willing to open a PR

Please, use issues search before posting a new one
Slide.displayName = "SwiperSlide";


proxy.golang.org: problem with some modules related to golang.org/x/

What version of Go are you using (go version)?

$ go version
go version go1.12.7 linux/amd64

Does this issue reproduce with the latest release?
yes with go1.12 latest
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/ldez/.cache/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/ldez/sources/go"
GOPROXY=""
GORACE=""
GOROOT="/home/ldez/.gvm/gos/go1.12.7"
GOTMPDIR=""
GOTOOLDIR="/home/ldez/.gvm/gos/go1.12.7/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/home/ldez/sources/go/src/github.com/go-acme/lego/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build196235691=/tmp/go-build -gno-record-gcc-switches"

What did you do?

clean the module cache
clone https://github.com/ldez/lego/tree/feature/migrate-gomod (or https://github.com/go-acme/lego when branch will be merged)
without GOPROXY
go mod tidy
then add GOPROXY=https://proxy.golang.org
go mod download

What did you expect to see?
No errors
What did you see instead?
<EMAIL_ADDRESS>unexpected status (https://proxy.golang.org/golang.org/x/time/@v/v0.0.0-20190513212739-9d24e82272b4.zip): 410 Gone
<EMAIL_ADDRESS>unexpected status (https://proxy.golang.org/golang.org/x/lint/@v/v0.0.0-20190511005446-959b441ac422.zip): 410 Gone

More information:
golang.org/x/time
golang.org/x/time v0.0.0-20190513212739-9d24e82272b4 h1:RMGusaKverhgGR5KBERIKiTyWoWHRd84GCtsNlvLvIo=
golang.org/x/time v0.0.0-20190513212739-9d24e82272b4/go.mod h1:tRJNPiyCQ0inRvYxbN9jk5I+vvW/OXSQhTDSoE431IQ=

<EMAIL_ADDRESS>unexpected status (https://proxy.golang.org/golang.org/x/time/@v/v0.0.0-20190513212739-9d24e82272b4.zip): 410 Gone

https://github.com/golang/time/commits/master

matloob committed on Feb 15
1 parent 85acf8d commit 9d24e82272b4f38b78bc8cff74fa936d31ccd8ef

<EMAIL_ADDRESS>
not found<EMAIL_ADDRESS>invalid pseudo-version: does not match version-control timestamp (2019-03-08T20:28:27Z)

<EMAIL_ADDRESS>
{"Version":"v0.0.0-20190308202827-9d24e82272b4","Time":"2019-03-08T20:28:27Z"}

https://github.com/search?q=golang.org%2Fx%2Ftime%40v0.0.0-20190513212739-9d24e82272b4&type=Code

golang.org/x/lint
golang.org/x/lint v0.0.0-20190409202823-959b441ac422/go.mod h1:6SW0HCj/g11FgYtHlgUYUwCkIfeOF89ocIRzGO/8vkc=
golang.org/x/lint v0.0.0-20190511005446-959b441ac422/go.mod h1:6SW0HCj/g11FgYtHlgUYUwCkIfeOF89ocIRzGO/8vkc=

<EMAIL_ADDRESS>unexpected status (https://proxy.golang.org/golang.org/x/lint/@v/v0.0.0-20190511005446-959b441ac422.zip): 410 Gone

<EMAIL_ADDRESS>
not found<EMAIL_ADDRESS>invalid pseudo-version: does not match version-control timestamp (2019-04-09T20:28:23Z)

<EMAIL_ADDRESS>
{"Version":"v0.0.0-20190409202823-959b441ac422","Time":"2019-04-09T20:28:23Z"}

Same thing is happening with Gizmo, if you follow the same steps from above and run go mod tidy inside github.com/nytimes/gizmo you'll get the following error:
go<EMAIL_ADDRESS>requires
<EMAIL_ADDRESS>reading<EMAIL_ADDRESS>410 Gone

I think this is related to the Proxy updating to Go1.13 and invalidating manually written pseudo versions, quote from the tooling mailing list:

Go 1.13 includes some changes to the go command which tighten up the validation of module versions. Now that we have upgraded proxy.golang.org to use the 1.13beta1 version of the go command, we will be invalidating old module versions that are currently being served from proxy.golang.org but are no longer valid in 1.13

Following up a little more, I can see that<EMAIL_ADDRESS>has what I guess an incorrect pseudo version, and updating from v0.4.6 to v0.6.0 does solve the problem (see linked pr above ^)

Yes, both of those versions for golang.org/x/lint and golang.org/x/time will now be invalid with 1.13 and will no longer be served from proxy.golang.org. This would have broken in direct-mode using 1.13 as well.
Try the suggestions that @marwan-at-work made, and I think that should solve the issues. Hopefully there aren't too many broken go.mod files in the wild, so once these get fixed, this problem shouldn't come up again. Thanks for reporting the issue.

Note that as a short-term workaround, you should be able to use a replace directive to target the corrected version. See https://tip.golang.org/doc/go1.13#version-validation.

Duplicate of #32805

Thanks :heart:

Hopefully there aren't too many broken go.mod files in the wild, so once these get fixed, this problem shouldn't come up again.

For the record, renovate creates PR with the bad pseudo-version: https://github.com/renovatebot/renovate/issues/4310. So, it may still come again.


Scheduled query browser (Stage 2)

Hey @n-riesco @nicolaskruchten 👋
Here is the Stage 2 PR for the Scheduled Queries projects.
Just FYI we still have more tests to write, but we are going to include those with Stage 3 to try and balance the front and backend work 👍 Hope that works.
Also FYI: we will be mostly offline tomorrow for the 4th of July, but probably will have time to make some quick fixes in the morning!

Please, see some examples of Falcon's naming conventions in https://github.com/plotly/falcon-sql-client/blob/master/CODING_GUIDELINES.md .
TODO

[ ] rename "app/components/Settings/Scheduler/createModal.jsx" to "app/components/Settings/scheduler/create-modal.jsx"
[ ] rename "app/components/Settings/Scheduler/loginModal.jsx" to "app/components/Settings/scheduler/login-modal.jsx"
[ ] rename "app/components/Settings/Scheduler/previewModal.jsx" to "app/components/Settings/scheduler/preview-modal.jsx"
[ ] rename "app/components/Settings/Scheduler/scheduler.css" to "app/components/Settings/scheduler/scheduler.css"
[ ] rename "app/components/Settings/Scheduler/scheduler.jsx" to "app/components/Settings/scheduler/scheduler.jsx"
[ ] rename "app/components/Settings/Scheduler/sql.jsx" to "app/components/Settings/scheduler/sql.jsx"
[ ] rename "test/app/components/Settings/Scheduler/createModal.jsx" to "test/app/components/Settings/scheduler/create-modal.jsx"

The access token for testing expired yesterday.
I've just updated master with a new token.
Please, rebase this PR against master.

When I first read the code, I found confusing and I missed the significance of having the following pairs of actions:

setScheduledQueries and getScheduledQueries
createScheduledQuery and createScheduledQueryAction
updateScheduledQuery and updateScheduledQueryAction
deleteScheduledQuery and deleteScheduledQueryAction

As the actions getScheduledQueries, createScheduledQueryAction, updateScheduledQueryAction and deleteScheduledQueryAction are only used internally, I'd rather have them removed and avoid the confusion and the time spent to learn their significance.
I've tested the change below and it works for me:
diff --git a/app/actions/sessions.js b/app/actions/sessions.js
index 1c9738d..754da7d 100644
--- a/app/actions/sessions.js
+++ b/app/actions/sessions.js
@@ -9,10 +9,6 @@ export const mergeTabMap = createAction('MERGE_TAB_MAP');
 export const setTab = createAction('SET_TAB');
 export const setTable = createAction('SET_TABLE');
 export const setIndex = createAction('SET_INDEX');
-export const setScheduledQueries = createAction('SET_SCHEDULED_QUERIES');
-export const createScheduledQueryAction = createAction('CREATE_SCHEDULED_QUERY');
-export const updateScheduledQueryAction = createAction('UPDATE_SCHEDULED_QUERY');
-export const deleteScheduledQueryAction = createAction('DELETE_SCHEDULED_QUERY');
 export const mergeConnections = createAction('MERGE_CONNECTIONS');
 export const updateConnection = createAction('UPDATE_CREDENTIAL');
 export const deleteConnection = createAction('DELETE_CREDENTIAL');
@@ -148,7 +144,10 @@ export function getScheduledQueries() {
             'GET',
             'scheduledQueriesRequest'
         )).then((json => {
-            dispatch(setScheduledQueries(json));
+            dispatch({
+                type: 'SET_SCHEDULED_QUERIES',
+                payload: json
+            });
             return json;
         }));
     };
@@ -171,8 +170,11 @@ export function createScheduledQuery(connectionId, payload = {}) {
         connectionId
       }
     )).then((res) => {
-      dispatch(createScheduledQueryAction(res));
-      return res;
+        dispatch({
+            type: 'CREATE_SCHEDULED_QUERY',
+            payload: res
+        });
+        return res;
     });
   };
 }
@@ -196,8 +198,11 @@ export function updateScheduledQuery(connectionId, payload = {}) {
       payload.filename,
       body
     )).then((res) => {
-      dispatch(updateScheduledQueryAction(body));
-      return res;
+        dispatch({
+            type: 'UPDATE_SCHEDULED_QUERY',
+            payload: body
+        });
+        return res;
     });
   };
 }
@@ -209,8 +214,11 @@ export function deleteScheduledQuery(fid) {
       'DELETE',
       'createScheduledQueryRequest'
     )).then((res) => {
-      dispatch(deleteScheduledQueryAction(fid));
-      return res;
+        dispatch({
+            type: 'DELETE_SCHEDULED_QUERY',
+            payload: fid
+        });
+        return res;
     });
   };
 }

(@tarzzz how does Chart Studio generate the filenames?)

@n-riesco Its automatically filled in if the user hasn't supplied any filename (during creation/updates). The default naming is "Grid 121" (or "Filetype unique_number").

@tarzzz At the moment Falcon uses https://github.com/plotly/falcon-sql-client/blob/6797d9e9c1411a23d1cffc25ccfdd63a7f4e0431/backend/persistent/plotly-api.js#L97-L120 to create a new grid. How should we do so that this function doesn't require a filename?

How should we do so that this function doesn't require a filename?

The best way to handle this is to hit the lookup endpoint with the exists parameter to check if the filename exists on server(its an optimized endpoint and is reliably fast): https://api.plot.ly/v2/files#lookup

I still need to review the changes to app/components/Settings/Settings.react.js
I'll do it tomorrow.
I noticed there is a race that allows users to open the schedule panel first and then immediately after switch to the query panel before it's ready.

remove Filename from  (at the moment Falcon's backend can't list filenames and thus Falcon won't be able to warn the user whether a filename is already in use) (@tarzzz how does Chart Studio generate the filenames?)

@n-riesco did you mean the CreateModal? I originally was generating the filename using uuid. Does that work?

did you mean the CreateModal? I originally was generating the filename using uuid. Does that work?

@mfix22 Chart Studio uses filenames of the form Grid nnnn where nnnn is a number. @nicolaskruchten mentioned there are users with up to 10000 grids, so I'd suggest we use  at least an 8-digit number.
I'll have a look at the other comments tomorrow.

Another approach we could take to the filenames is to require the user to provide a valid, pre-existing FID that is to be overwritten... that would also solve the "I erased my query and want to recreate it pointing to an existing FID" problem. In a later iteration we could add a FID browser.

We could also have a "query name" that's independed of the target grid/dataset name.

For SQL connectors (those listed in SQL_DIALECTS_USING_EDITOR), remove the button Create Scheduled Query from  (we don't want to duplicate the functionality in the Query panel)

@n-riesco for connection dialects that don't use the query panel editor, is it still meaningful to display a query input within the create scheduled query modal?
How are scheduled queries currently handled for dialects which are not contained in SQL_DIALECTS_USING_EDITOR?

Re the filename I would love it if the user could specify a filename and if the API doesn't like it that we bubble that error through. If the API currently doesn't return an informative message, let's change that on the API side to enable this feature /cc @tarzzz FYI

I also think we should ask for and store a Query Name, independent of the filename

If the API currently doesn't return an informative message, let's change that on the API side to enable this feature

Yep, It's already covered in API ..  ⬇

Sorry, a file already exists with this name.

Re the filename I would love it if the user could specify a filename and if the API doesn't like it that we bubble that error through. If the API currently doesn't return an informative message, let's change that on the API side to enable this feature /cc @tarzzz FYI

@nicolaskruchten I currently have this implemented, but maybe since we are adding "Query Name"/"Query Title" we can just uncomment it and enable this with Stage 3?

@briandennis

for connection dialects that don't use the query panel editor, is it still meaningful to display a query input within the create scheduled query modal?

yes, it is possible to have scheduled queries for those connectors too.
if you want to test it, you can try the apache drill connector:

falcon provides credentials to a test server (see screen capture below)
and the test query:SELECT * FROM s3.root.`sample-data.parquet` LIMIT 10

How are scheduled queries currently handled for dialects which are not contained in SQL_DIALECTS_USING_EDITOR?

Same as for other connectors (it already works with your PR)

@mfix22 I've tested https://github.com/plotly/falcon-sql-client/pull/476/commits/508ca66e8938142a49ece449c8efd63ea145deb7 and it doesn't look good:

How about hiding the buttons instead?

Please, don't forget about the comment here.

@n-riesco yes for sure! I was planning on doing it and then forgot. Just pushed the update. Thanks for catching that! 👍

I've tested the latest changes and it looks good to me.
There is an issue that I can't reproduce reliably (sometimes I need to click a schedule twice to be able to open the PreviewModal).
I'm merging so that we can progress to stage-3.


Add support for multiple variable declarations in a single statement

We need to support this for example:
    int x = 5, y = 2

Currently we'd define it like so:
    int x = 5
    int y = 2

some deeper idea:
int x, y = 5, 2

@torwart second idea looks cool, not sure about the first, could get very messy and lead to some bad spaghetti code.

@felixangell yep, for little things like basically int x, y = ... its ok but second would be great. Its just like an struct but different ;)

the second one reminds me of Go's const(), stuff I think we could consider it. You might have to propose a separate issue, though, I'm not sure.

but in example two we have an problem like:
struct some {
 int {
  x = 5
 }
}

I don't think that this would be so nice. If you do this often the user will also run into spaghetti code:
struct some {
 int {
  x = 5
 }
 int {
  x = 6
 }
 int {
  x = 7
 }
 int {
  x = 8
 }
}

@torwart That would throw an error anyway since you redeclare x 4 times in the same struct, lol. The brace would be used if you're initializing multiple things, so you may as well just do int x = 5, if you're only doing one thing. Also, it's going 5, 6, 7, 8. So in that scenario you could just use an enum :+1:

Much prefer initializing variables like this:
int a, b = 1, 2
a, b = b, a + b

Turning the int into a code block isn't the best of ideas.
It could be a way of typecasting a bunch of variables

I also prefer @juddus Idea because something like this in structs would be nice:
struct some {
 int one, two, three
 string a, b, c
}

I don't think
int {
    x = 4,
    y = 5
}

is a good idea. it looks unnecessarily complicated. maybe we could just do it the normal way
int x = 4, y = 5, z = 3

Guys, if you're doing pointers (which I presume you are), please do not emulate C's parsing in this regard.
int* x = 0, y = 0x1000

The above code should result in both x and y having a type of int*. I really don't know why people still think whether or not a variable is a pointer type should be bound to the name and not the type.
This makes zero sense:
int *x = 0, *y = 0x1000

Im not even sure if I want to add support for multiple variable definitions with a comma.

To clarify why I think we shouldn't include this is that if you have one variable declaration per line, it makes the programmers intent a bit more clear. And also encourages a comment beside the declaration.

Yea, I tend to avoid multiple decls of anything in one line for this reason.

@requimrar Glad to see someone agrees with me :+1: @alloy-lang/super-contributors thoughts on not doing this?

Sounds good


Bugfix/775 var context json size 0 dims

Submisison Checklist

[x] Run tests: ./runCmdStanTests.py src/test
[x] Declare copyright holder and open-source license: see below

Summary:
Given that changes to class stan::io::var_context in branch https://github.com/stan-dev/stan/tree/feature/2687-var-context-validate-dims, PR https://github.com/stan-dev/stan/pull/2869 have been merged into Stan, this PR adds implementation of var_context virtual function validate_dims to  class cmdstan::json.
Intended Effect:
Given Stan program with data block:
data {
  int d;
  vector[d] d_v1;
  vector[d] d_v2;
  row_vector[d] d_rv1;
  row_vector[d] d_rv2;
  vector[3] d_v_ar[d];
  row_vector[3] d_rv_ar[d];
  matrix[0, 3] d_m;
}

accept as valid JSON input:
{
    "d" : 0,
    "d_v1" : [],
    "d_v2" : [],
    "d_rv1" : [],
    "d_rv2" : [],
    "d_v_ar" : [],
    "d_rv_ar" : [],
    "d_m" : [],
    "d_m_ar" : []
}

How to Verify:
unit tests
Side Effects:
none
Documentation:
added subsection to CmdStan manual illustrating JSON notation for arrays where any dimension is size 0.
Copyright and Licensing
Please list the copyright holder for the work you are submitting (this will be you or your assignee, such as a university or company):  Columbia University
By submitting this pull request, the copyright holder is agreeing to license the submitted work under the following licenses:

Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

hi @rok-cesnovar, this PR is dependent on the Stan PR.  once this fix is merged into CmdStan,  CmdStanR can use JSON format, so this will remove the dependency on RStan.

Thank you! Removing the rstan dependency will be very nice indeed!
I will restart the Jenkins tests in such a way that this PR will use the Stan PR code. And also review both PRs asap.

: It also seems that the tests are passing if we use the Stan PR branch: https://jenkins.mc-stan.org/blue/organizations/jenkins/CmdStan/detail/PR-787/2/pipeline/45

thanks @rok-cesnovar !  removed the stray debug comments.