Section filters return no results

Create a project that includes a section (i.e. 'Test')
Add a task under that section
Create a todoist code block in Obsidian that uses the filter /Test
No results will be returned for that block in Preview mode

Creating the same filter within Todoist itself will return the task created above.

As a quick update for this issue, the Todoist API returns a 501 response code if you include a section filter in the REST API endpoint which this plugin uses (/rest/v1/tasks). 501 means that its not implemented on the Todoist backend. 😬 (Related to #18 as to why you aren't seeing anything)
I've asked them if this is planned to be implemented in the future, but it may be the case that for now this won't be possible.
A long term solution would be to migrate to the Sync API which opens up some other interesting possibilities, but also would require a fair bit of work.
In the meantime, I'll add a note to the README that section filters are currently unsupported.

 Spent a bit of time migrating all of my Todoist projects to using sections for Kanban vs labels.
I was trying to decide where my project tasks would sit, might just have to do that in Obsidian directly instead of through this. I tend to mix tasks and notes, though, so that was probably the more likely route.  My main use case for this plugin is all of my repeating tasks anyway, so this plugin already saves me a ton of work.

For what its worth, I do plan to looking into using the Sync API in the future, but it requires more bookkeeping for the plugin so the REST API was the path of least resistence to get started.
So.. watch this space 😉

Now this works fine.  FYI
$ curl -X GET
https://api.todoist.com/rest/v1/tasks
-H "Authorization: Bearer $token"

I'm still getting 501: Unsupported filter keyword when attempting to use the section filters, I wonder if it needs to be escaped differently 😕
@thammawat-s how did you manage to get section filters working?

sorry my bad.  I still get the same 501: Unsupported filter keyword.  :(

Update.  Tech support replied me, now these should work.  Thanks to Omar from Todoist.
query by section_id:
GET https://api.todoist.com/rest/v1/tasks?section_id=12345
Or filter on the section name using /sectionname:
GET https://api.todoist.com/rest/v1/tasks?filter=/mysection

I can confirm that filters using sections now works in the plugin, so this bug can be closed.

That's awesome! Thanks for following up all.


ReactionDataset CP DFTD3 energy.

Describe the bug
Counterpoise correction does not currently work with DFT-D3 in ReactionDatasets as CP/noCP does not make sense for the -D3 term. DFTD3 compute always takes the noCP stoichiometry, but the query planner takes the stoichiometry of the parent. A query for CP DFT-D3 cannot correctly complete, because the monomers will never be available. To fix, the query planner needs the same logic of the compute where DFTD3 is always cast to noCP.
To Reproduce
Query a -D3 with CP on any ReactionDataset.
I need to take a deeper look, but I thought the compute and query planners were identical to avoid this exact problem.

Suggested solution: Rerun DFTD3 on CP, since it costs few CPU hours and the storage costs are manageable.


Upgrade to go 1.6

Release notes: https://golang.org/doc/go1.6

lint will break as they fixed a bug in gofmt (golang/go#12066).
Adds HTTP2 support.
The reflect package changes might break the json encoding

Lets do this after 0.13.
Change should be as simple as changing the build container.

:+1:


Define date-time options used for date-range formatting as part of the date-range patterns.

Fixes #21.

I rebased latest changes in in the main branch. @anba PTAL, I'll merge this PR as soon as you LGTM. Thanks!

I rebased latest changes in in the main branch. @anba PTAL, I'll merge this PR as soon as you LGTM. Thanks!


Landing Page Redesign Not Showing in Production

I pushed a redesign of the page to main branch. GitHub Actions ran successfully, but the live site still shows the old version.
Does anyone know why the changes aren't reflected in production? Does it simply still take some time to 'overtake' the changes?
Cc. @Ly0n, @andrew

I change the Source form GitHub Action to Deploy from Branch. Now it works! Looks great!


Error kivy.atlas.Atlas

Software Versions

Python: 3.10
Kivy: 2.1..0
Kivy installation method: by the pip

whenever I try to use Atlas kivy the interpreter stops working, in the command line it simply appears (nothing):
>>> atlas = Atlas('assets/Ninja_Monk/monk.atlas') (venv) PS C:\Users\Jeffe\PycharmProjects\pythonProject>
When using the pycharm run tool ("run"), I think I have one more piece of information. After this message:
[INFO ] [Image ] Providers: img_tex, img_dds, img_sdl2 (img_pil, img_ffpyplayer ignored)
as usual, it stops running and appears:
Process finished with exit code -1073741819 (0xC0000005)
I thought it was because I built kivy in python3.11 but I changed the interpreter to 3.10 and the problem persisted

Hi @Jefferson5286 !
Can you try to launch it from terminal, with the debug logging enabled and report back here the full output?


feat(primitives): Channel Reuse

#499  👈
main

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

Join @refcell and the rest of your teammates on  Graphite


touch doesn't work

I trying to make it work the example, the TFT display is ili9341 240x320, but the touchpad doesn’t work and I can’t move the slider. In all examples of the TFT_eSPI library the touch Works correctly, only doesn’t work with lvgl library. I use lvgl v.7 download from the oficial page, if you can help me I would appreciate a lot. This are my pins conections from esp32 nodeMCU to ili9341. Wich can be my problem? Thanks in advance
#define TFT_MISO 19
#define TFT_MOSI 23
#define TFT_SCLK 18
#define TFT_CS 5
#define TFT_DC 4
#define TFT_RST 22
#define TOUCH_CS 14

We use GitHub Issues for development-related discussions.
Please ask your question in the Forum to make it visible for a larger community. You can simply log in with your GitHub account.


test: Tweak test functions names, drop unnecessary ones

I got annoyed with the inconsistency & also tomcat_stop is useless so I removed it.

Thanks @jajik


Disabling Turbo?

When request.js detects window.Turbo, I'm wondering how we can disable Turbo when calling something like patch? I would just call submit on a form with data-turbo: "false" but I have to manipulate form data with an image cropper.

Can you give a more detailed explanation of what you're trying to achieve? request.js will only call Turbo to process the response of your request if the response is a turbo-stream which means that in your request you would be making a request like this with responseKind: 'turbo-stream'
patch('/endpoint', {
  body: {},
  responseKind: 'turbo-stream'
})

Do you want Turbo not to process your response but still receive a turbo-stream as a response?

That's not what I was seeing. I had this:
request = patch( path, {
  body: formData
})

And that was taking over the request and response of a redirect. I'll see if I can create a minimal reproduction soon.

Ok so after making the request request.js was following the redirect do you mean? This is not something related to Turbo, it is the default config that request.js assumes see https://github.com/rails/request.js/blob/main/src/fetch_request.js#L129 which is the default of the Fetch API https://developer.mozilla.org/en-US/docs/Web/API/Request/redirect

The problem is that it's not rendering the redirect and that's Turbo behaviour. I didn't think Turbo would interfere unless I had specified the responseKind but I noticed that it was following the redirect but not rendering it. I added a turbo:before-visit event listener and it is being triggered for the patch request I mentioned above.

Yeah it is following the redirect because that's something Fetch API does, it doesn't render because it isn't a functionality of request.js
About turbo:before-visit being triggered I'm not sure about, not sure about the internals of Turbo work around this.
The only thing that Turbo does when you specify turbo-stream as the responseKind is process turbo streams that that given response contains.
Probably you don't want to let Fetch API follow the redirect and do it your own?

Closing this since it does not seem a problem with Request.JS


modules

The module Topic which covers  Import, export, and dynamic import

Task done 😎. Thank you sir for the privilege

This was really good experience for to be able to contribute and also learn slidev.

yes you are welcome to the word of open source, i will assign reviewers

I wll leave the issue open till it is review by the team @joker4mas


error, with pydantic core

error: failed to run custom build command for pydantic-core v2.14.5 (C:\Users\litvi\AppData\Local\Temp\pip-install-gnrpxnw1\pydantic-core_3d8c8d0bdb7f4fd781867a9a2a256457)
  Caused by:
    process didn't exit successfully: `C:\Users\litvi\AppData\Local\Temp\pip-install-gnrpxnw1\pydantic-core_3d8c8d0bdb7f4fd781867a9a2a256457\target\release\build\pydantic-core-1f7ecd8c544c2fc8\build-script-build` (exit code: 101)
    --- stdout
    cargo:rustc-cfg=Py_3_6
    cargo:rustc-cfg=Py_3_7
    cargo:rustc-cfg=Py_3_8
    cargo:rustc-cfg=Py_3_9
    cargo:rustc-cfg=Py_3_10
    cargo:rustc-cfg=Py_3_11
    cargo:rustc-cfg=Py_3_12
    cargo:rustc-cfg=Py_3_13
    cargo:rerun-if-changed=python/pydantic_core/core_schema.py
    cargo:rerun-if-changed=generate_self_schema.py

    --- stderr
    Traceback (most recent call last):
      File "C:\Users\litvi\AppData\Local\Temp\pip-install-gnrpxnw1\pydantic-core_3d8c8d0bdb7f4fd781867a9a2a256457\generate_self_schema.py", line 192, in eval_forward_ref
        return type_._evaluate(core_schema.__dict__, None, set())
               ~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    TypeError: ForwardRef._evaluate() missing 1 required keyword-only argument: 'recursive_guard'

    During handling of the above exception, another exception occurred:

    Traceback (most recent call last):
      File "C:\Users\litvi\AppData\Local\Temp\pip-install-gnrpxnw1\pydantic-core_3d8c8d0bdb7f4fd781867a9a2a256457\generate_self_schema.py", line 240, in <module>
        main()
        ~~~~^^
      File "C:\Users\litvi\AppData\Local\Temp\pip-install-gnrpxnw1\pydantic-core_3d8c8d0bdb7f4fd781867a9a2a256457\generate_self_schema.py", line 210, in main
        value = get_schema(s, definitions)
      File "C:\Users\litvi\AppData\Local\Temp\pip-install-gnrpxnw1\pydantic-core_3d8c8d0bdb7f4fd781867a9a2a256457\generate_self_schema.py", line 54, in get_schema
        return type_dict_schema(obj, definitions)
      File "C:\Users\litvi\AppData\Local\Temp\pip-install-gnrpxnw1\pydantic-core_3d8c8d0bdb7f4fd781867a9a2a256457\generate_self_schema.py", line 152, in type_dict_schema
        field_type = eval_forward_ref(field_type)
      File "C:\Users\litvi\AppData\Local\Temp\pip-install-gnrpxnw1\pydantic-core_3d8c8d0bdb7f4fd781867a9a2a256457\generate_self_schema.py", line 195, in eval_forward_ref
        return type_._evaluate(core_schema.__dict__, None)
               ~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    TypeError: ForwardRef._evaluate() missing 1 required keyword-only argument: 'recursive_guard'
    thread 'main' panicked at build.rs:29:9:
    generate_self_schema.py failed with exit code: 1
    note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
  warning: build failed, waiting for other jobs to finish...
  рџ’Ґ maturin failed
    Caused by: Failed to build a native library through cargo
    Caused by: Cargo build finished with "exit code: 101": `"cargo" "rustc" "--features" "pyo3/extension-module" "--message-format" "json-render-diagnostics" "--manifest-path" "C:\\Users\\litvi\\AppData\\Local\\Temp\\pip-install-gnrpxnw1\\pydantic-core_3d8c8d0bdb7f4fd781867a9a2a256457\\Cargo.toml" "--release" "--lib" "--crate-type" "cdylib"`
  Error: command ['maturin', 'pep517', 'build-wheel', '-i', 'C:\\Users\\litvi\\AppData\\Local\\Programs\\Python\\Python313\\python.exe', '--compatibility', 'off'] returned non-zero exit status 1
  [end of output]

note: This error originates from a subprocess, and is likely not a problem with pip.
ERROR: Failed building wheel for pydantic-core
Failed to build pydantic-core
ERROR: ERROR: Failed to build installable wheels for some pyproject.toml based projects (pydantic-core)

Duplicate of #1366, please update.

The problem is this dependency (shown for Python 3.12):
$ python -m pipdeptree -p pydantic-core -r
pydantic_core==2.14.6
└── pydantic==2.5.3 [requires: pydantic_core==2.14.6]
    ├── safety==3.2.3 [requires: pydantic>=1.10.12]
    └── safety-schemas==0.0.7 [requires: pydantic~=2.5.0]
        └── safety==3.2.3 [requires: safety-schemas>=0.0.2]

But I agree that this is not the fault of pydantic or pydantic-core.

Just for the record, and for future readers, I have reported this to the safety repo as https://github.com/pyupio/safety/issues/620


Add has_class? helper

Hey, what do you think of my suggestion of adding a has_class? helper? I could use it in a situation like this; I want to check whether an element has a particular CSS class, so I’m doing:
String.contains?(attribute_value(selector, "class"), "a-class")

It would be great if I could just do this:
has_class?(selector, "a-class")

@backspace Thank you for the PR. :+1: nice one ~!
There seems to be some issue with the builds. I'll fix it up and merge this and the other PR :)

You’re welcome, thanks for having created such a useful and extensible tool!
Yeah, looks like Selenium is failing non-deterministically :neutral_face:

@backspace Yes. I've been noticing the same behaviour locally too (sometimes). I tested it locally and merged the PRs. Thank you both of them :)


RLite C# / .NET

Are there bindings available for rlite to C# or .NET in general?
Or a similar library capable of the same?
EDIT: Note: mono compatibility would be ideal (Mono = runs C# / .NET on Linux, mac, etc.)
EDIT2: ... though it would also need to work on Windows...

I don't think there is.
It currently does not work on Windows. I did my best in https://github.com/seppo0010/rlite/tree/win ; I got to a point where each file compiled, but I couldn't figure out how to link in Visual Studio.

It was trivial for me to get the code loaded into VS the laziest way possible - create a new VS project, add all files... boom.
... But the project references unix headers like unistd.h everywhere...

After fixing git trouble and arguing with the compiler a bit in VS,
I got it down to this error set:

@mcmonkey4eva please submit a PR on top of the windows branch with your Visual Studio file and I'll take a look. Thanks!

My file's an utter trainwreck, I was lazy in building it and thus had to reorganize files a bit to make it all work correctly. Also I was recently... discouraged from bothering with Redis at all after learning some limitations I didn't know about previously...


Add supported operating systems

An issue was raised specific to RHEL6. This is not supported but we do not list what we support.

The tests are failing because Fog has removed support for Ruby 1.9.3, but this change by itself looks good to me.

Looks good to me: https://docs.puppetlabs.com/puppet/4.3/reference/modules_metadata.html


Error when trying to list_tables

Installed Endo for the first time and got this (list_tables without any filters, running Postgres 12):
** (Postgrex.Error) ERROR 42803 (grouping_error) column "p1.indrelid" must appear in the GROUP BY clause or be used in an aggregate function
(endo 0.1.17) lib/endo/adapters/postgres.ex:37: anonymous fn/2 in Endo.Adapters.Postgres.list_tables/2

Amazing, thank you!
Let me take a look at debugging this and I'll get back to you soon :crossed_fingers:

@mayel looks like there are cases where underlying database tables don't have a pg_class for it.
Endo made the assumption that all tables would have a pg_class we could reflect on to gather metadata.
In the case that a pg_class isn't found, I'll update Endo to return a Endo.Metadata.NotAvailable struct to allow end users to handle if they so choose!
Have a fix ready in #14


Upgrade dependencies one more time

And attempt at fixing #251

So it works here, but not on https://github.com/dlang/dmd/pull/11348 for example.

I'll try to deploy it and see if it solves the issue

Looks like it worked: https://github.com/dlang/dmd/pull/11348#issuecomment-651636861
I also enabled auto-deploy from Github so next push to master will deploy.


Receiving: correct document date to current rather than future date

Edited document date from "2023-04-23" to "2023-04-18" so the document will appear on the docs site (currently missing because 4-23 is a future date)

Another pull request already resolved this issue


Fix:
Try more things to get to a point where he stops.

 Not "this"!


Document installation of additional languages

The documentation notes, one should "check Stanford Core NLP documentation and files to download more language packages".
I need to get german language support into iepy. So I downloaded http://nlp.stanford.edu/software/stanford-german-2016-01-19-models.jar, but its unclear to me what I need to do with that jar file. Language files are stored under ~/nltk_data/ afaik, but in a different format.
Can you explain here and/or in the documentation, what needs to be done to get different language support than English or Spanish?

Language files for Stanford CoreNLP are saved in ~/.config/iepy (not in nltk_data becayuse NLTK has nothing to do here).
You will also have to edit corenlp.py and write german specific code like this one for spanish:
https://github.com/machinalis/iepy/blob/develop/iepy/preprocess/corenlp.py#L93
Hope this helps.

Thanks, I will look into this and send a pull request if I'm successful.

It might be a good idea to add that info to the documentation. But for the time being i'll close this since we received and accepted the pull request.
Thanks for the support!
btw, congrats for getting ticket #100, If you ever come to Argentina contact me and i'll buy you a beer for it.


ci: enable spell-check-partial for evaluator and launch

Description
enable spell-check-partial for evaluator and launch directory

Related links
N/A

Tests performed
N/A

Notes for reviewers
N/A

Interface changes
N/A

Effects on system behavior
N/A

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.
[ ] The PR has been properly tested.
[ ] The PR has been reviewed by the code owners.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.
[ ] The PR is ready for merge.

After all checkboxes are checked, anyone who has write access can merge the PR.

https://github.com/autowarefoundation/autoware.universe/pull/4460 will make a merge conflict

No conflict apparently?


[Peter Latona] - Scalability ensuring that your cryptography app can handle a large volume

Thanks for submitting this tutorial for Celo Sage
Before you hit "Create Pull Request", make sure you have read the Tutorial Contribution Guidelines and reviewed the Sample Tutorial Structure.
Please confirm the following before submitting your tutorial for review -

[x] My tutorial follows the Tutorial Contribution Guidelines.
[x] The title of the Pull Request matches the title of my tutorial.
[x] I've followed the tutorial template structure.
[x] I've proofread the tutorial for spelling and grammar mistakes.
[x] I've thoroughly explained the code, with inline code comments and text explanations.
[x] I've run a linter & prettifier to format my code blocks.
[x] I've run the tutorial to test it and the code works.

NOTE: We cannot review your Pull Request if it does not meet the standard outlined above.
Reviewers - @joenyzio & @viral-sangani

Hi @Ishanpathak1 @Julius170. Is this card ready to be merged?

Closing Celo Sage PRs


Optoma ZU1050 problem

Hi,
I tried sending controls to Optoma ZU1050 but didn't work. Then I tested the thing with a PJlink test app and the controls where working so there must be something wrong with the PJlink implementation on optoma ?
Test app can also send controls to projector and controls from PJLink app went ok.
I checked TCP traffic with wireshark and there is small difference in commands , test app sends PJLINK 0 ,than %1POWR 1 and this instance sends just %1POWR 1 and for some reason this does not work for the Optoma.
For now I configured standard TCP/UDP instance targeting projectors IP:4352(port) with the commands starting with PJLINK 0 and everything works.
Also just configuring the PJ link instance in bitfocus prevented any controls from PJ link test app from reaching the projector at the same time, could this could also block any other app from remote controlling the projector at same time?
Am I doing something wrong with this module ,or Optoma didn't implement the thing , or the module needs a fix ?
Thanks,

Test app is from pjlink site class 1.
Yes i can see that PJLINK 0 is being sent from projector . Also my command does not work with test app from pjlink but works with the projector so its not module's fault....

There is no option to set the pjlink password on the projector, otherwise it has no other passwords enabled.
Test apps offer the option with or without authentication tests.
Here are two commands from test apps, first with authentication and second without.
Remote app:
`[2021.03.30 20:04:18.605] Connected!!!
[2021.03.30 20:04:18.627] RECV:PJLINK 1 699bb32a
[2021.03.30 20:04:18.643] SEND:099ba3b188165e08fd67ef1366edd269%1POWR 1
[2021.03.30 20:04:18.643] RECV:%1POWR=OK
[2021.03.30 20:04:18.643] Closed!!!(Disconnect)
[2021.03.30 20:06:11.106] Connected!!!
[2021.03.30 20:06:11.106] RECV:PJLINK 0
[2021.03.30 20:06:11.122] SEND:%1POWR 0
[2021.03.30 20:06:11.122] RECV:%1POWR=OK
[2021.03.30 20:06:11.122] Closed!!!(Disconnect)`
Projector app:
[2021.03.30 20:04:18.605] Connected
[2021.03.30 20:04:18.627] SEND:PJLINK 1 699bb32a
[2021.03.30 20:04:18.643] RECV:099ba3b188165e08fd67ef1366edd269%1POWR 1
[2021.03.30 20:04:18.643] SEND:%1POWR=OK
[2021.03.30 20:04:28.664] Client Disconnect
[2021.03.30 20:06:11.106] Connected
[2021.03.30 20:06:11.106] SEND:PJLINK 0
[2021.03.30 20:06:11.122] RECV:%1POWR 0
[2021.03.30 20:06:11.122] SEND:%1POWR=OK
[2021.03.30 20:06:11.122] Client Disconnect
so something is being exchanged before the %

The authorization token is generated from the password plus the 4 bytes sent by the projector.
It may not be labeled as a PJLink password. What password/id/authentication code does the app use?

another thing , I can send PJLINK 0 and then in the next TCP command %1POWR 1 and it also works.

The remote should never need to send 'PJLINK' so that is non-standard.
When connected, the projector should send either PJLINK 0 (no authentication) or PJLINK 1 aabbccdd (authentication with hash).
If authentication (PJLINK 1) is needed, the module sends a token + query to the projector (is power on?) and either gets a response or an error.
That menu password may also be the PJLINK code.

You are definitely right , there is something wrong with their implementation . Will check few more things tomorow and see whats the problem.
Thanks.

We tested the module with the projector simulator apps, too.
I am curious what the issue is with that projector.
Tomorrow, then.

hi,
after more testing ......me sending PJLINK 0 before command had nothing to do with this , it works with generic TCP when sending proper commands (%1POWR 1).
I noticed some more differences between test apps and pjlink module
test app with projector :

creates new TCP connection for every command sent from interface
projector sends PJLINK 0
command get's send from the app  (%1POWER 1)
projector responds (%1POWER=1)
TCP connection gets terminated

and everything works.
module with :

when you create the instance of pjlink module in companion and specify the IP of the projector , TCP connection gets created 2) projector sends PJLINK 0
for now nothing until you create button with PJlink command
PJlink command button created and pressed and nothing happens on network...

Projector does not send PJLINK 0 ever again because the connection is not terminated ever again until closing of companion.
Using generic TCP to send commands works because i can send the command without receiving PJLINK 0 before, does module require to receive the PJLINK 0 before sending the command ?
Test app handles the thing differently because of the TCP connection termination every time  so it gets the PJLINK 0 , and projector works with the testbed and OPTOMA probably didn't notice this .
Should the projector send PJLINK 0 everytime when remote app requests TCP packet transfer or on creation of connection ?

The module only expects PJLINK message once per tcp connection. If it does not send messages when you press the keys, I would think that something is wrong with the initial message from the projector. Maybe it is with line feed? Does it only send lf or cr and lf?

It sends 0D

Tried the module with a different projector , Eiki WUL100 , everything works as expected.
But totally different behaviour from projector. Eiki sends KEEP ALIVE TCP messages / or restarts connections every few sec with PJLINK 0 being sent once per connection.
Really stuck with this .... after receiving ACK for PJLINK 0 message from projector , no TCP traffic from PC.
Also disable instance in companion works in terminating TCP connection , but enable does not make new connection it gets stuck at connecting .....

The protocol specifies connections are only good for 4-5 seconds, which requires a close/reconnect sequence. That is what you are seeing with the Eiki (which is normal operation).
Somehow the response from the Optoma is not being recognized, so the command/ack sequence stalls.

So inproper implementation on projector prevents this module from working
Is it posible to modify the module to work more like test app with tcp connections being made once the commands are being sent and not when the instance is created .
Doubt that optoma will fix this considerng that projector passes the test app.

The module also passes the test app.
I'm looking over the module now to see what is happening.
What OS are you using for Companion?

Win 10, 64 bit PRO  on Lenovo Thinkpad , with firewall off if you need wireshark captures i can upload them .


Multi line diff min line config option

You can turn multi line off, but it doesn't activate until 50 lines or more.
I'd like to be able to control that via config or sys prop.

Added system property for min diff line.
kotlintest.assertions.multi-line-diff-size


Create changelog and prep 0.4 release.

Add this here per our discussion in #299. It has been on my shortlist, but I don't want to forget.

@thedodd Any movement on a 0.4 release? It would be nice to have #305 in a released version

Hey @nylar, my plan is to let @saghm or @kyeah handle that actually. I don't have permissions to actually do any releasing on this crate or anything. Hopefully it gets some movement soon though.

Sorry about the delay! I'll work on getting this done

Just pushed out v0.4.0! FYI, this will likely be the last minor version bump of this driver that we do; my team at MongoDB and I have been hard at work for a few months now on an alpha release of an officially supported Rust driver, so we plan on focusing our efforts there moving forward


Return MPRemoteCommandHandlerStatus required for iOS 13

PR summary
Apple API docs state that functions with MPRemoteCommandEvent params should return MPRemoteCommandHandlerStatus .
Current implementation has these as returning void, which is crashing apps using this plugin in iOS 13.
PR returns MPRemoteCommandHandlerStatusSuccess in all actions of this type
Issue(s) related
https://github.com/homerours/cordova-music-controls-plugin/issues/157
Steps to test

Call MusicControls.create({.. on iOS 13 results in app crash
Update code with this PR
Call MusicControls.create({.. on iOS 13 results in app not crashing

Thanks for putting this together. I ran into this issue as well, and can confirm that it fixes it.

thank you! Will be added to the master?

@yoojene as I can see, you've managed to make it works on iOS 13.
Could you mind to share a working sample code?
I've tried by copy/paste the example in the readme but no luck. I don't get any error, it just doesn't work. I've tried with your branch too, but again no luck.
Thanks

@yoojene it works for me. Thanks a lot

Yes! It works! Thanks a lot!

@JumBay you can use git+https://github.com/homerours/cordova-music-controls-plugin.git#90e3fe8838c3151bd8e51901079eb34740ec87c9

I can confirm this fixes the crashing on iOS13 in my Ionic app by doing the following:
ionic cordova plugin remove cordova-plugin-music-controls
ionic cordova plugin add git+https://github.com/homerours/cordova-music-controls-plugin.git#90e3fe8838c3151bd8e51901079eb34740ec87c9ionic

@fdambrosio I've managed to make it works. I'm using this plugin to control a remote player. I didn't know the device has to play something to make it works (on iOS).

Is this project dead? Does anybody maintain it?
Will this get merged in the foreseeable future?

@tbergeron yes it is see the readme.
Does anyone manage to make the callbacks work on the create method? it seems to be never called

Noticed this PR was suddendly not working , have added MPRemoteCommendHandlerStatus return values for skipForwards and skipBackwards actions


improve/add message when prebuild not available and falling back to build

in the case where a prebuilt binary is not available, it would be awesome if a message like this was printed out to the user to help them understand what is happening:
Prebuilt binaries for your <node or iojs> version, <version number>, are not available for download.
We will now try to compile from source.

We have had some people get confused when their compile failed (e.g. they were using iojs 3.0.0 and prebuilts werent available and the compile failed). It would be nice if the output was super clear so they can understand whats goin on

+1


Feature Request: Output the names of matched files for verification

Describe Request:
I would like to see what files were matched and checked by shellcheck in addition to whether or not the check was successful.
Examples:
For example I just added shellcheck to this project https://github.com/heroku/heroku-buildpack-ruby-experimental/pull/33. I want it to check shellcheck -x bin/build bin/compile bin/detect bin/release bin/support/bash_functions.sh but i'm unsure if it found all those files or not.
If this can be accomplished without a feature, please let me know.
Supporting Documentation Links:

https://github.com/heroku/heroku-buildpack-ruby-experimental/pull/33
https://app.circleci.com/pipelines/github/heroku/heroku-buildpack-ruby-experimental/181/workflows/d448f319-30aa-44ed-9c67-4414825d362b/jobs/659

I checked by purposefully introducing a violation into each file and it turns out only one of my files the bash_functions.sh is detected.

This is sort of expected - the check.sh script defaults to looking for .sh files:
https://github.com/CircleCI-Public/shellcheck-orb/blob/master/src/scripts/check.sh#L34-L36
The search pattern can be changed by passing through the pattern parameter :)
https://github.com/CircleCI-Public/shellcheck-orb/blob/master/src/commands/check.yml#L42

This is sort of expected - the check.sh script defaults to looking for .sh files:
The search pattern can be changed by passing through the pattern parameter :)

Could we either output the list of files that are found to be valid via the pattern and/or some output if no files are matched?

Done in #36 :)


Upstream has a new major version

Hey guys,
Just want to let you know that upstream has a new release that greatly simplifies adapters, and implements multiple serializers (including JSON API 1.0). It would be nice to investigate if the features in this fork can be integrated into Fortune 1.0.

I think we've forked so far away from the original fortune that it's pretty much impossible to merge back.
Are you using our fork 0x8890? How are you finding it? What are you using it for?


Improve Helper Usability and Error Handling

Two changes in this PR:

Modify the Helper's WaitOnWatchedResource to use k8s client-go's watch tools. This function essentially did the same thing as client-go's watch.Until function, so I gutted WaitOnWatchResource to use watch.UntilWithSync, which has great error handling. This way we don't have to maintain a tool that already exists.
Removed Ctx from the Helper, so users can pass their own context into each function as needed.

These are breaking changes.

Pushed a fix for your comments @marseel, do you mind taking another look?


typescrtip version ?

does have typescrtipt version? I alaway show not found module err

I'm sorry, Currently I don't have time to develop the typescript version


Waterfall Docker Image | Docker Hub

Can you please bring up the waterfall image back to docker hub? Ty

Will happen soon :)

soon hehe


Conditional type support

Resolves non-locale problems listed in #69.
There are probably a few sections which do not require conditional compilation, as eliminating calls further up the chain would suffice. But I have not determined where best to do so, and this is easy to modify later.
After becoming more acquainted with the code, I found the correct location. I also decided that all types should fall into this setup, not just floating-point.
The one thing I'm not 100% sure of is the static_assert placement. I think the two of them cover everything, but it's a little hard to tell.
The size reduction for my project is 23K (no fallbacks; schar, uchar, long, ulong, bool, float, buffer, and custom enabled).

Apologies for the churn. Modern C++ has gotten so much easier, I often forget how to do it the old way (and which old way goes with which standard).

I inverted the conditions, so that the macros are SCN_DISABLE_TYPE_*. I think it's better that way, since the default with no extra flags given should be to support everything. I also modified the CMake scripts to only add these flags if a type is actually disabled, and added #define SCN_TYPE_* 0 to config.h if SCN_TYPE_* is not defined.
Other that than, looks good. Rebased, signed, and merged. Thank you!

Hey I'm just happy to have the options! Thanks for getting this cleaned up and merged.


Made derive.key a case class

This fixes #144.

No idea why this isn't auto-closed but I merged it in https://github.com/lihaoyi/upickle-pprint/commit/e36ab845adbb7c92c2aa24732c8497404ef8a06a


Grainy images on LG webOS

The pictures look grainier on the LG webOS as compared to just the web browser.  Is there a way to fix this?
I did a comparison by using the browser on the same LG device as using the developer mode.  The image quality on the developer mode is much more grainier.  Would love some input!

@andiskim are you talking about actual device or simulator. if device could you provide details of the TV ?

@pavjacko Both the emulator and the TV have blurrier pictures than on the browser.  The device I tested on is a 49 inch LG TV

@pavjacko
For reference on my computer:
Browser:

Emulator:

You can see the difference in the writing as well.  It is like this when testing on the LG TV as well (using the inhouse browser vs using development mode in the LG TV).  Any help would be greatly appreciated

@andiskim thanks for reporting, is this still reproduced on latest version?

can't reproduce on 1.0.0


Add object mocking feature

Purpose

Add object mocking feature
Resolves #23255

Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

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

@@           Coverage Diff           @@
##           master   #23857   +/-   ##
=======================================
  Coverage   14.59%   14.59%
=======================================
  Files          51       51
  Lines        1411     1411
  Branches      219      219
=======================================
  Hits          206      206
  Misses       1189     1189
  Partials       16       16

Continue to review full report at Codecov.

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


Show how data science jobs have evolved throughout the years

is this related to 'Data_Science_Jobs_in_India'? as I'm not able to preview the years column there. could you please let me know?

Just make the cell of your excel sheet large...

umm.. lemme check once again as I did the same before.

My bad @tapaskhanijo ... Closing this issue

no worries brother:)


question about zone files

Awesome work on this @jcnelson!
I have a question about the zone files. Here's the example in the readme:
$ORIGIN duckduckgo_tor.id
$TTL 3600
tor TXT "3g2upl4pq6kufc4m.onion"

Is this introducing a new "tor" type? If so, shouldn't it be capitalized like this: "TOR"? And will this be a problem for legacy compatibility with traditional zone file parsers and resolvers?

It conforms to the TXT record specification, so legacy compatibility won't be a problem.  tor is just the value of the name column of the RR (i.e. its fully-qualified domain name would be tor.duckduckgo_tor.id).  Capitalization doesn't matter for names in DNS.

Ah yes right, got confused for a second, thank you. Closing this issue.


Refactor amoeba2 into bayes_spec framework

Welcome to Codecov :tada:
Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


192 fonts,font sizes,buttons

Finished implementing desired styling changes and did some further css cleanup. Currently pending review and potential further requests before merge.

Will try to review that in the next two days.


New command !mdocstring

Add a !mdocstring command
this will help form docstring and also format the code with black
Also, added poetry configuration and formatted the code base with black

Can you explain a bit more about the !mdocstring command and possibly share a screenshot of what the end result looks like

This incorporates https://github.com/ManimCommunity/DiscordManimator/issues/14 (@infiniteregrets : you can find a screenshot there).
Thanks a lot, naveen, for this pull request!

@infiniteregrets something like

Oh, I think this can be helpful for you guys! And this can be merged after #11, except you will have to make a few changes to your code. Great work!

I think this can be merged first rather than that one. Since https://github.com/ManimCommunity/DiscordManimator/pull/11 does major changes on the whole structure it would easy to make changes there and fix conflicts rather than changing this here.

I don't mind in which order we merge these prs.
But it would be nice if this could be merged soon, as we have an event soon, where this docs feature could be really helpful!

@kolibril13 does this look good to you?
Also, @behackl I have added poetry configurations, so while deploying you can either use a venv from poetry or run poetry export -o requirements.txt and run pip install -r requirements.txt while deploying.

@kolibril13 does this look good to you?
Also, @behackl I have added poetry configurations, so while deploying you can either use a venv from poetry or run poetry export -o requirements.txt and run pip install -r requirements.txt while deploying.

yes, it looks good to me!
Thanks a lot again for the efforts you are doing here, this is a huge step to make it easier to contribute examples to the docs! :)

Ok merging :rocket:

Two remarks:

Given that there is no automatic deployment setup for this, I'd appreciate if you'd let me merge -- otherwise it might happen that updates won't be deployed. (I don't really mind, it's just easier to miss stuff on my end.)
In general, I like the idea of having a command that helps formatting docstrings -- but honestly, I think the actual work of composing the docstring should be made by Manim, potentially in a manim docs subcommand (which we should look into anyways, to make it easier to render single documentation pages).

The changes here are fine obviously, no need to revert anything. I'd still prefer PRing the formatting utility to the library itself. Only having it in our bot seems weird to me.


Add ngmin filter

Add ngmin filter which prepare Angular JS files to minification.

Please make a PR in https://github.com/gremo/assetic-extra.


Add install procedure without Typescript

I'm not using Typescript, then import is not working.
Using this workaround do the trick.

Thanks!


fix function overloads

Fixes #90.

[X] Add representative-enough-ish cpp code in #90.
[ ] Determine newer breathe reqs (when did 3.x become required etc) and test across more versions.  I distinctly recall excluding declname for a reason?
[ ] Fix cpp func overload tests by adding variable names in (update all other function calls too most likely).

To test this checkout the fix/func_overloads branch and run tox -e py -- -k func_overloads and if those pass now tox -e py.

more stale than my soul


Adding a custom subdomain for each new tenant

I thought it might be a good idea to manage a custom domain/subdomain name for each new tenant in this reference solution.
So each tenant will have its own subdomain to access the SAAS web app.
Thank you!

@babaksd - Yes, you can implement this subdomain approach, we did mentioned about this in the documentation in Step3.
If you are interested in understanding how to implement this approach, please refer to EKS SaaS Reference Solution which implemented this subdomain approach.

Hi @ujwalbukka
I'm trying to deploy the solution in AWS but the "serverless-saas-CustomResources" stack creation is stuck, Specifically, the "UpdateTenantStackMap" and "UpdateSettingsTable" events are stuck due to following errors from Cloudwatch:
Unable to import module 'update_settings_table': No module named 'logger' Traceback
Unable to import module 'update_settings_table': No module named 'crhelper' Traceback
Unable to import module 'update_tenantstackmap_table': No module named 'logger' Traceback
Unable to import module 'update_tenantstackmap_table': No module named 'crhelper' Traceback
It seems the Python dependencies are not installed correctly.
May I ask you how can I fix that?
Thanks,

Hi @babaksd  - As mentioned in this issue response please make sure you have latest sam cli version.
And please make sure you have met all these pre-requisites


Enhancement/align pygwalker anywidget (Ready)

This PR will align more closely with the pygwalker anywidget.

Superseeded by #30


fix(travis): error in travis. "npm run deploy" set node_env to production by default

by creating "npm run deploy:no-env", we don't set node_env to production by default and let the user (travis) change this env from dev to prod to test all the deployment type

Current coverage is 70.88%

Merging #683 into master will not affect coverage as of 6692a5e

@@            master    #683   diff @@
======================================
  Files           12      12
  Stmts           79      79
  Branches         0       0
  Methods          0       0
======================================
  Hit             56      56
  Partial          0       0
  Missed          23      23

Review entire Coverage Diff as of 6692a5e

Powered by Codecov. Updated on successful CI builds.

Wow, I can't believe this slipped by for so long (granted it probably cropped back up after the re-introduction of better-npm-run). Thank you!

Also, going to just make this the default (no :no-env) and add helpers for dev and staging.

Squashed the extra merge commits and made some tweaks, check out https://github.com/davezuko/react-redux-starter-kit/pull/689. Your original commit is still in there, thanks again for catching this :tada: .


[CI] Use ConanCenter package for OpenAssetIO

Our CI for this project uses the ASWF Docker image, so that we have USD available.
Then, in this PR, we use Conan to install OpenAssetIO and its dependencies.
Since there are no pre-built binaries in ConanCenter that match the Docker image, everything (the whole dependency chain, i.e. CPython and its crazy dependencies) has to be built from scratch.  This defeats the purpose of using Conan instead of cloning and building from source.
If we wanted this to work then we would need to ensure we cache the Conan builds.
But then the main benefit of using Conan for OpenAssetIO here, is that we would get cross-platform builds. However, how would we get USD on other platforms? - that was the main reason we used Docker in the first place.
As for why CI is failing with
In file included from /__w/usdOpenAssetIOResolver/usdOpenAssetIOResolver/src/resolver.cpp:4:
/__w/usdOpenAssetIOResolver/usdOpenAssetIOResolver/src/resolver.h:11:10: fatal error: openassetio/typedefs.hpp: No such file or directory
   11 | #include <openassetio/typedefs.hpp>
      |          ^~~~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.

I'm not sure.
Regardless, I'm going to close this PR since there's a lot more to think about than a simple drop-in replacement.


Using multiple dev machines / working on amplify as a team / gitignore

I'm using devcontainers and vscode on a nextjs and amplify project started on my mac laptop.
Trying to develop on my windows desktop, I'm running into   /.config/local-env-info.json' does not exist. Same error when running amplify configure or amplify configure project. It's because I have it in my .gitgnore .
What are the amplify files or directories that should be in gitignore? Does my workflow on multiple machines demand me take ` /.config/local-env-info.json' out of .gitignore?
What is the best way to handle a workflow for a developer to use multiple machines, as well as working with additional developers?

@amlcodes It looks this is a bug on our end, can you pull/init an env and try it out?

ahhh, yeah just had to pull again on my machine. That makes sense, I guess I expected to configure to setup for some reason. Makes sense that I would pull. Doesn't seem like a bug IMO, user error.
On teams - I guess you would have an Organization with AWS accounts/users in Org Units with the permissions needed? And then each person would just amplify pull... right?

On teams - I guess you would have an Organization with AWS accounts/users in Org Units with the permissions needed? And then each person would just amplify pull... right?

That would be right

On .gitignore - is there an official source of truth on this? I'm seeing conflicting ideas on amplify/team-provider-info.json ?

Before version 4.32.1 the CLI used to write hostedUIProviderCreds to team-provider-info, in the latest version those values are managed from ~/.aws/amplify/deployment-secrets.json. There would be no harm exposing your team-provider-info.json in a source repository anymore


Spatial grid for neighbor detection

SpatialGrid.ts

Added spatial grid class with respective functions

Particles.ts

Added SpatialGrid class, initializer in the constructor.
Note Grid fills and resets on frame redraw.

Particle.ts

Replaced collision detection loop with spatial grid call
Refactored redundant radius * 2 calls to const called 'diameter'.

Looking good, I'll move this forward.

GH is reporting me conflicts:
Conflicting files
src/Classes/Particle.ts
src/Classes/Particles.ts

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 1

See the complete overview on Codacy

Everything seems ok


chore: bump some major deps

Proposed change
Bumping some trivial major deps for V10.

Quick question: did you run a yarn dedupe?
If not, can you do so?


Wrong estimated account value.

Expected behavior
Estimated account value should be calculated as per current Steem and SBD price.
Actual behavior
Estimated account value is way too higher than expected.
How to reproduce

Browser: all
Operating system: all

Duplicate of #1948


Opening a directory just opens the computer's documents folder

Hi, whenever I click a button to open a directory from the app, like "open in file explorer" in the collections panel and "open local directory" in the help menu, it just opens the "C:\Users<user>\Documents" folder on my computer. I'm using 1.14.0pr8.4 on Windows 11.

Noted


Dont crash when user has cookies blocked on safari

This sentry error occurs when a user visits the site on safari with cookies blocked.
Steps to reproduce:

set "Safari Settings -> Advanced -> Block all cookies"
load swc in safari

We already try catch our calls to localstorage for this exact reason, but it looks like some third party libraries like "WalletConnect" call localstorage directly and crash the site.
I think the fix here is we inline a script tag at the top of our app before anything else loads that overrides the global localStorage function with a version that wraps all the calls in try catch. Ugly hack 🤷

2024-04-01-02:38pm UPDATE:
I've tried to overwrite the local storage functions via the PR above, but that does not fix the issue - the thing is, _any access to the local storage (even something like console.log(localStorage) causes the insecure error, so overwriting the functions does not matter.
If there is an answer to this, then it would involve somehow checking if the browser can access the local storage, but this check would have to be built part of the local storage because it needs to essentially intercept any calls to localStorage. Most solutions out there involve a wrapper function/wrapper around the local storage to do this, but we can't do that given that it's a third-party package that is hitting the local storage directly.


Can we have access to the source code for the XmlFormatter.VSCode

Hello,
I'm trying to learn how to create a VSCode that formats the code on the fly, auto-close tags etc..etc. and I would like to see what is the Dlls that you have in the libs folder.
Do you have that code? Where did you get it?
best regards
Paulo Aboim Pinto

I have that in a private library that is not MIT open-source licensed intentionally. Unfortunately on-the-fly formatting is not possible currently.

thanks for the answer ...
can you point me to a place where can I learn how this lib can be done?
thanks a lot
Paulo Aboim Pinto

I can add you to the repo as a contributor if you could ensure you would contribute to same repo.
What is your Github ID that needs the access?

that is a great idea ... thank for the trust.
my github id: 1231687

I think I need the github username. Is it same aboimpinto ?

I think I need the github username. Is it same aboimpinto ?

yes ... it's AboimPinto

Done.

how do I find the private repository?


Print-debugging jax?

I've got some jax code that relies on grad, jvp, etc.  I'd like to do some simple print-style debugging.  E.g., some lines like:
print("JUtotal: ", ret)
return ret

In a function whose gradient is required.  I'd really like to see something like:
JUtotal: -0.924010929514282
but instead, I see:
JUtotal:  Traced<ConcreteArray(-0.9240109029514282)>with<JVPTrace(level=6/0)>
  with primal = Traced<ConcreteArray(-0.9240109029514282)>with<JVPTrace(level=4/0)>
                  with primal = DeviceArray(-0.9240109, dtype=float64)
                       tangent = Traced<ShapedArray(float64[])>with<BatchTrace(level=3/0)>
                                   with val = DeviceArray([ 4.31834536e-04, -5.96999899e-06,  1.25216808e-01,
                                                           -1.07182239e-01, -7.46824003e-07, -8.62923763e-07],            dtype=float64)
                                        batch_dim = 0
       tangent = Traced<ShapedArray(float64[]):JaxprTrace(level=5/0)>

I understand why the latter is sometimes important (e.g., when debugging the autograd machinery itself).  But for now, I just want to know the numerical value of 'ret'.  Is there a way to obtain just the numerical value of an object in a traced function with all the extra "hair" removed?

I recommend using jax.experimental.host_callback for this -- see id_print() in particular: https://jax.readthedocs.io/en/latest/jax.experimental.host_callback.html

It fills my heart with joy to see you're still playing with JAX, @johnsalmon !
id_print is one option, though its primary purpose is being able to perform prints from code that was staged out to XLA via jit (or pmap, or control flow primitives). To reach that goal it might not be as ergonomic as Python printing, like you might want in interactive pdb sessions.
The text you see right now is from the Tracer.__repr__ method defined in core.py, which is meant to recursively dissect a Tracer and print everything it finds. But as you say, maybe that can be too much information for some common use cases.
One thing we could do here, which @hawkinsp proposed in a chat just now, is make a Tracer.__str__ that differs from Tracer.__repr__, and have the former be more concise. (I believe by only defining Tracer.__repr__ as we do now, Python reuses that definition for Tracer.__str__ too.) Then print(x) might do what you want, whereas print(repr(x)) would give more complete information. Maybe I'll give that a shot...
Any thoughts?

Does it work to write something like ret.primal?
I would be reluctant to drop all the extras from the default display, though the detailed type & trace level information could be omitted from __str__.

@shoyer by "default display", do you mean what we get from __repr__ or __str__? I agree we need to make both fully-detailed and simplified versions readily accessible.
I think using ret.primal is worse because it's violating our Tracer abstractions, whereas using __repr__ and __str__ is just calling into a standard API.

@shoyer by "default display", do you mean what we get from __repr__ or __str__? I agree we need to make both fully-detailed and simplified versions readily accessible.

I was thinking for __str__ as well.
I guess my concern is that when you're printing tracer, the primal value isn't necessarily any more fundamental than the tangent. For example, inside linear_transpose(), the primal value is just a placeholder and the tangent/cotangent is all that matters.

Feedback on this? The called function first calls print(x) to show the new short result of Tracer.__str__, and then calls print(repr(x)) to show the more complete info. The code change is in #4618.
In [1]: def f(x):
   ...:     print(x)
   ...:     print(repr(x))
   ...:     return x
   ...:

In [2]: from jax import grad
gra
In [3]: grad(f)(3.14)
Traced<3.14>
Traced<ConcreteArray(3.14, weak_type=True)>with<JVPTrace(level=2/0)>
  with primal = Traced<ConcreteArray(3.14, weak_type=True):JaxprTrace(level=1/0)>
       tangent = Traced<ShapedArray(float32[], weak_type=True):JaxprTrace(level=1/0)>
Out[3]: array(1.)

In [4]: from jax import jit

In [5]: jit(f)(3.14)
Traced<float32[]>
Traced<ShapedArray(float32[], weak_type=True)>with<DynamicJaxprTrace(level=0/1)>
Out[5]: DeviceArray(3.14, dtype=float32)

In [6]: grad(grad(f))(3.14)
Traced<3.14>
Traced<ConcreteArray(3.14, weak_type=True)>with<JVPTrace(level=4/0)>
  with primal = Traced<ConcreteArray(3.14, weak_type=True):JaxprTrace(level=3/0)>
       tangent = Traced<ShapedArray(float32[], weak_type=True):JaxprTrace(level=3/0)>
Out[6]: array(0., dtype=float32)

I suspect this goes too far for @shoyer but I think it might be worth trying out. It's easy to revise, after all!

This is indeed further than I would go, but we can try it out.
What about JVPTrace<3.14> rather than just Traced<3.14>?

Closing because we now have jax.debug.print.


HDF5 data format support

The HDF5 data format might be interesting to support at some future point:
http://www.hdfgroup.org/HDF5/

HDF5 supports metadata ...  https://support.hdfgroup.org/HDF5/Tutor/HDF5Intro.pdf
"The Split File I/O driver enables HDF5 metadata to be stored in one file and
the data in another."

Removing the high priority tag until someone actually asks for it.

I'm sponsoring @jamesmudd who has been diligently working on jHDF - a pure Java HDF5 library.  The library can read but now with my encouragement and the backing of the HDF5 group has some limited writing support as well.


Bug: artifacts must be re-provided after restart

Scenario:

start node A (auhtorized)
start node B (connected to A)
config CLI to node B
CLI request build
CLI inspect-log -> works
retrieve artifact from node B -> works
restart node A
start node C (connected to A)
config CLI to node C
inspect-log -> works
retrieve artifact --> fails

So after restarting node A, which should provide files on the p2p network, those files are unreachable (if they weren't downloaded before - which is why the retrieval will still work from node B)

is the issue that when node A restarts it does not look in the same Artifact folder? I would expect that the restart should not change state and since the artifacts were in the disk attached to node A they would be retrievable from node A. In case of authority node (which is nodeA for us) this is ok since we should not be restarting it. But if we do have to restart it should not lose those artifacts.

It's not the same issue. the files are still there, but the node fails to provide them again on the p2p network.
we used to have this functionality but it was lost in the build from source refactoring.
We'll work on a PR to fix this and we can discuss whether it's important enough to get it in, but I think it is, because there are now artifacts in the transparency log that can't be downloaded because no one provides them. (and they can't be rebuild either because they exist in the TPL already)


fix(Command): remove invalid data-[disabled] attribute

cmdk's README.md states that [data-disabled] is available on <Command.Item />. However since the PR that made this data attribute available, there has been no releases. As a result the styling using this data attribute is not being applied.
While making these changes I realised aria-selected is also being used for styling, so I switched this to use the data attribute, data-selected, which is applied by cmdk.
I created an issue on cmdk repo here, https://github.com/pacocoursey/cmdk/issues/192 and will follow this but there is no indication when this will be addressed or if there will be a new release.
Before w/ Calendar CommandItem disabled:

After w/ Calendar CommandItem disabled:

This has been updated in the latest cmdk release yesterday https://github.com/pacocoursey/cmdk/releases/tag/v0.2.1. I will bump dep and file a new PR when I can.


Add more context to the constrains?

Thanks for the project, it's quite enlightening!
Would it be possible to add some more information about the constraints?
For example variables in // 2 <= E - S are not clear.
It would be great for ScheduleEntities.kt have also similar descriptions as Model.kt (which is pretty clear).
Similar thing with val discreteLength = (hoursLength * 4).toInt() - what is 4 here?
Again, thanks for teaching us, normal developers, a more scientific approach to development!

Thanks @tomaszpolanski, glad you find this engaging. I'm actually posting examples here for this book I'm reading and recommending to folks.
I will see if I can improve the descriptions. One thing I'd like to see is KotlinDocs support MathJax, which I filed a KEEP for.  I know there is a way to do it now but it requires a lot of hacky build configuration.

Hopefully this is well-documented now. I even wrote a blog article about this.
http://tomstechnicalblog.blogspot.com/2018/02/linear-programming-with-kotlin-part-iii.html


Typing ::: sometimes gets messed up

When I want to get an unexported function from a package via :::, the cursor jumps to the end of the line after I type ::. Eg stats:::AIC, becomes stats::AIC:.
This seems to happen only if:

I type the function name first, then move to the start of the line and type the package name followed by :::
The completion list contains exactly one member, eg it happens with AIC[home]stats::: but not with lm[home]stats:::.

I can avoid this behaviour by pressing Esc to dismiss the completion list when it pops up, but I'd rather not have to.

https://github.com/Microsoft/RTVS/commit/16149bd35ee00d90b6c942b59447030db52da752


Fix OpenMP builds on Unix

First of all do not use -Xclang -fopenmp, just use -fopenmp as this feature is stable so long now. Also on UNIX you'll need to link to pthread to get OpenMP link correctly.

CI fails seems to be due to other changes in the master.


Please add examples of custom formatting for selected items

Some online examples of custom formatting for selected items would be really helpful - both for multi select and tags.

For those who are looking for it too...
  $(element).select2({
      escapeMarkup: Utils.escapeMarkup,
      templateResult: function (result) {
        return result.text;
      },
      templateSelection: function (selection) {
        return selection.text;
      },
      ...


Module parse failed: Cannot use keyword 'await' outside an async function (8:0)

error  in ./node_modules/curlconverter/util.js
Module parse failed: Cannot use keyword 'await' outside an async function (8:0)
You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders
|
| // Top-level await is not available in Safari until 15 (TP)

await Parser.init()
| const Bash = await Parser.Language.load('./tree-sitter-bash.wasm')
| const parser = new Parser()

@ ./node_modules/curlconverter/generators/ansible.js 1:0-34 20:18-39
@ ./node_modules/curlconverter/index.js
@ ./node_modules/cache-loader/dist/cjs.js??ref--12-0!./node_modules/babel-loader/lib!./node_modules/cache-loader/dist/cjs.js??ref--0-0!./node_modules/vue-loader/lib??vue-loader-options!./src/views/general_tools/curl_tools/index.vue?vue&type=script&lang=js&
@ ./src/views/general_tools/curl_tools/index.vue?vue&type=script&lang=js&
@ ./src/views/general_tools/curl_tools/index.vue
@ ./src/views sync ^./.*$
@ ./src/store/modules/permission.js
@ ./src/store/index.js
@ ./src/main.js
@ multi (webpack)-dev-server/client?http://<IP_ADDRESS>:80&sockPath=/sockjs-node (webpack)/hot/dev-server.js ./src/main.js

Are you using Node 12?

node --version
v12.16.3

node --version
v12.16.3

curlconverter no longer works on Node 12 because of top-level await, which is only available in Node 14.8.0+

The new curlconverter 4 pre-release supports Node 12 once again, so this shouldn't be an issue anymore.

Hi I am facing this issue for node version 14.18.0 in a react project, any possible resolution?

If you're using Webpack to build your project, please take a look at how we use Webpack to build the curlconverter.com website. For this error, you have to add this to your webpack.config.js file
  experiments: {
    topLevelAwait: true
  },

https://github.com/curlconverter/curlconverter.github.io/blob/be82cb8335d5241aca8b7a6a0cdb2e7905435abd/webpack.config.js#L110-L112

I don't see a webpack.config.js file in my project, should I create one?

I am using this script to run my project: react-scripts start

@soham-sagade you're using create-react-app, which manages your Webpack config for you. There is no good way to ask create-react-app to tell Webpack to enable top-level await, you have to eject from create-react-app and start managing your Webpack config yourself, which is not recommended. See this StackOverflow question for details if you want to do this: https://stackoverflow.com/questions/68443510/how-to-support-top-level-awaits-in-typescript

Same issue here using Node 18.9.0 with Vite.
I tried to use vite-plugin-top-level-await but it creates others issues.
Please, could you remove top level await from curlconverter ?

@amorin-gladia for Vite you can do this
  configureWebpack: {
    experiments: {
      topLevelAwait: true
    },
  },

but I think this requires Vite 3 because it has Webpack 5.


error bar in the growth plot for #nodes

Hi @danydoerr :)
I have a few questions about the growth plot for #nodes( the second pic in https://github.com/marschall-lab/panacus/blob/main/examples/pangenome_growth_minigraph_cactus.md ). is the X-axis the sample number? is the pink bar the nodes covered by all these count of sample? for example, say X-axis =20, does it mean randomly pick 20 samples from the total samples (i.e. 46)? so the Y-axis of pink bar means the mean of #nodes covered by these 20 samples sets? because there are many samples sets if randomly pick 20 samples from 46 samples. SO my concern is :  Can this plot show the error bar for these sampling?
Thank you in advance!
Best regards,
Wenbo

Hi @WenboLVCong ,
Yes, the X-Axis of the coverage histogram and the pangenome growth plots are samples. The coverage histogram shows how many nodes (panacus can generate a corresponding plot also for basepairs) are covered by exactly 20 out of the 46 genomes.
Pancus does not do sampling, but it calculates the coverage histogram and the pangenome growth with an exact formula. So there is no need for error bars.

Hi @danydoerr :)

Thank you very much for reply. I just read the paper https://www.biorxiv.org/content/10.1101/2<IP_ADDRESS>8418v1.full.pdf. but still can not get it. Say X-axis=20, if Panacus does not do sampling, so which 20 samples out of the 46 samples are used for counting? Or does it mean: among the 46 samples, the nodes covered by exactly 20 samples? If so, sum all the histogram (X-axis from 1 to 46) up, should equal to the size of the whole pangeome?
Thank you very much!
Wenbo

Yes, it's exactly as you describe. Summing over the node coverage histogram gives you the total number of nodes in the graph.

I mean the histogram in the second pic (i.e.  growth plot for #nodes), not the first pic. take my result as example.

the total size of the pangenome is about 303mb, equal to the y-axis of pink bar when x-axis=16. if sum all these bars together, it is much bigger than 303mb. I thought it was not the size of pangenome.
the size of a single genome is ~ 220 mb

So, for the pangenome growth plot for bps, the last bar should be exactly the total number bps in your pangenome

yes , I set quorum >=0. I was wondering, for the second bar (x-axis = 2), how to get the y-axis of pink bar. if with sampling, which two samples were used to count the bps?

yes , I set quorum >=0. I was wondering, for the second bar (x-axis = 2), how to get the y-axis of pink bar. if without sampling, which two samples were used to count the bps?

Hi Wenbo,
so the way Panacus works is that it does not look at specific samples, but instead at all of them. So for the second bar (x=2) it will look at all the nodes. It then finds out in how many combinations that node appears at least once. For a node that appears in 4 of the 6 paths (this fact is contained in the coverage histogram) this calculation would look like this:

there are in this case ${16 \choose 2}$ possibilities of drawing samples
in ${16 - 4 \choose 2}$ combinations the node does not appear
so it appears at least once in ${16 \choose 2} - {16 - 4 \choose 2}$ combinations
this gives us a probability for the node to appear of $\frac{{16 \choose 2} - {16 - 4 \choose 2}}{{16 \choose 2}}$

We then sum over all the probabilities for all the nodes to get the height of the bar.

Hi Peter,
Thank you very much! I got it finally !
Best regards,
Wenbo


Example for init_global_grid_usage

how do you to launch an example (e.g. diffusion3D_multigpucpu_novis.jl) on multiple processes ?

Thank you very much !
I read the README.md in both ParallelStencil.jl and ImplicitGlobalGrid.jl and I could not find the command you proposed.
It could be interesting to add a section on how to launch the examples (you type this, you should obtain this).

A quick note in case someone fall on the same problem. If your local MPI installation does not match the MPI.jl's one then the command mpirun -n 8 julia --project ./diffusion3D_multigpucpu_novis.jl launches 8 times the same monoprocess execution with a warning.  Fixing the MPI mismatch fix the pb.


Update vae.py

Since the DSVAE class is already inheriting from torch.nn.Module, there is no need to inherit from CUDAGraph as well. You can remove the CUDAGraph inheritance. Instead of using self.vae.requires_grad_(requires_grad=False), you can use torch.no_grad() context manager during initialization to disable gradient computation for the self.vae module.  The _graph_replay_decoder, _graph_replay_encoder, and _graph_replay methods can benefit from the @torch.no_grad() decorator.

@microsoft-github-policy-service agree

@microsoft-github-policy-service agree

@mzamini92 - since this PR made it quite a while before being lost, do you think it is still worth merging, if so could you resolve the merge conflicts?  Or should we close this PR?


feat(notification): Add Microsoft Teams notification support

This PR adds Microsoft Teams notification support.
Relates to issue spinnaker/spinnaker#5826
Related Pull Requests:
https://github.com/spinnaker/deck/pull/8503
https://github.com/spinnaker/echo/pull/995
https://github.com/spinnaker/front50/pull/927
https://github.com/spinnaker/orca/pull/3863

Looks like this branch is out of date and Mergify cannot update it - please update when you get the chance and ping me in Slack so I can manual-merge.

@Mergifyio refresh


ReadTheDocs

Successful PR Checklist:

[ ] Tests

[x] (not applicable?)

[ ] Documentation

[x] (not applicable?)

[x] Had a good time contributing?
[x] (Maintainers: add PR labels)


Clarify flushing/"blocking" behavior

The SpiDevice trait's documentation is now very explicit that operations may return before they are finished. That brings up some questions:

What happens with adjacent operations? E.g. if there are two back-to-back write() calls, is it legal for the second call to return Err(Busy), or is it expected that this call will block until it can enqueue the operation?
In the same vein, is calling write(); write(); acceptable for drivers, or does it have to be write(); flush(); write() to not risk an error?
What about other traits like I2C? They do not mention that returning early is acceptable. Does that imply that they block until the bus is idle? If not, why don't they have a flush() operation. If so, shouldn't that be explicitly documented?

Points 1. and 2. have been fixed in #456.
We may need documentation improvements for the I2C case.


when disabling PSPs in an rke1 cluster, the default pod security policy persists, causing an error.

Setup

Rancher version: v2.7-head  (cca09e4)
Browser type & version: chrome

Describe the bug

when updating a cluster with an existing PSP policy, simply disabling PSPs dismisses the banner, but does not remove the default pod security policy. If the user saves the changes to the cluster during an upgrade to 1.25, they will get an error and the cluster changes will not save.
To Reproduce

deploy a cluster on v1.24

enable PSPs and set default pod security policy to restricted

wait for the cluster to get to an active state
upgrade the cluster to 1.25

note the red banner warns the user to remove the PSP setting
disable PSPs, but do not toggle the default pod security policy setting before disabling

Result
cluster is not able to upgrade, and user can't fix the requested change through the UI because PSPs are hidden on 1.25 clusters.
Expected Result

the UI should set the default pod security policy to empty to avoid this error if the user disables PSPs
Screenshots

Additional context

this was found during rke1 testing for 1.24 -> 1.25 upgrades with PSPs enabled
PSA setting seems to have no effect.
seems isolated to rke1 / ember
related issue: https://github.com/rancher/dashboard/issues/8174

/backport v2.7.2

tested on v2.7-head (23074b5)

disabling PSPs in an rke1 cluster also removes the default_pod_security policy -- pass


How to clear concrete NovaSearchableBelongsToFilter filter?

For example I use more than one of NovaSearchableBelongsToFilter on resource.
I need to clear one of NovaSearchableBelongsToFilter filter.
But I don't know how to do it.
I don't see UI element to clear concrete NovaSearchableBelongsToFilter filter.

same,
after applying filter i need to reset it to see all items, as normal view.

@thoresuenert ?


Remove import references to Get[Current]PackageInfo2 causing load failures on RS5

This fixes RS5 Compatibility: GetCurrentPackageInfo2 not available through DynDep #1287
Get[Current]PackageInfo2 was added in 19H1 so import references cause load failures on RS5.
Reshuffled code to only call Get[Current]PackageInfo2 if available (19H1+) and fallback to Get[Current]PackageInfo (v1) on earlier systems. Get[Current]PackageInfo2 access is now done dynamically (LoadLibrary/GetrProcAddress) when necessary to avoid load-time resolution failures.
Verified import references were here yesterday and gone today by comparing LINK /DUMP /IMPORTS Microsoft.WindowsAppRuntime.dll
Also fixes most warnings in WindowsAppSDK.sln (too much noise, masking real potential issues).

/azp run

/azp run

For the Environment Manager tests, all look good.

/azp run

/azp run

/azp run

Weird build failure. Something wrong with the build infrastructure? @BenJKuhn @kythant @alexlamtest @kmahone

/azp run

/azp run


remote_server: Handle data of non existent realms.

This PR adds code to make sure that the push
notification does not crash on receiving data for
a non-existent realm.

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Merged, thanks @sahil839!


Question:: BitHelper - sha3 vs keccak

Hello sirs, I looked into implementation of BitHelper and I see that keccakf1600 is used. While the algorithm is the same keccakf1600 package contains NIST SHA3 implementation and it's different from original Keccak and Ethereum is using original Keccak.
Shouldn't BitHelper be utilizing keccak hash instead of sha3? SHA3 can be reversed to to Keccak by changing padding and there is a fork that does that for keccakf1600.

You're right that the current implementation does not look to match those tests as is, e.g.:
iex> BitHelper.kec("teststring") |> :binary.decode_unsigned |> inspect(base: :hex)
0xA52EF463EDB03558BE0598FB9A3AE67E0FC41C44BD0B1D4392B354D1E817FEA8

I suggest doing the following:

fork erlang-keccakf1600 under exthereum
apply this change https://github.com/jur0/erlang-keccakf1600/commit/a87d31d32250091262e0da5ee2c2a4535bc50945
modify mix.exs to pull keccak package from exthereum

FYI, right now we're using https://hex.pm/packages/keccakf1600_orig to solve this issue.

@hayesgm thank you, I guess the issue can be closed then.


Debug problems with react/react router and hostnames

Namely:
Running at https://lfz.corefacility.ca/superphy/grouch/ causes problems because root will default to https://lfz.corefacility.ca/

adding "hostname": https://lfz.corefacility.ca/superphy/grouch/ to package.json didnt fix it
maybe need to set on react router itself?

This is fixed by using a prefix for all routes in https://github.com/superphy/reactapp/commit/7d78437f7c1154329e766e874fb9cd5a124aa6ca and setting the homepage for serving build files in https://github.com/superphy/reactapp/commit/ea17459cbc2f8f793dab9be255ebe644e841ecb2


Make all stream accesses async

I get the following error message when using the release/2.0 branch with ASP.Net Core 3.0:
System.InvalidOperationException: Synchronous operations are disallowed. Call WriteAsync or set AllowSynchronousIO to true instead.
   at Microsoft.AspNetCore.Server.Kestrel.Core.Internal.Http.HttpResponseStream.Write(Byte[] buffer, Int32 offset, Int32 count)
   at System.IO.Compression.DeflateStream.WriteDeflaterOutput()
   at System.IO.Compression.DeflateStream.WriteCore(ReadOnlySpan`1 buffer)
   at System.IO.Compression.DeflateStream.Write(Byte[] array, Int32 offset, Int32 count)
   at System.IO.Compression.GZipStream.Write(Byte[] array, Int32 offset, Int32 count)
   at Microsoft.AspNetCore.ResponseCompression.ResponseCompressionBody.Write(Byte[] buffer, Int32 offset, Int32 count)
   at System.Xml.XmlUtf8RawTextWriter.FlushBuffer()
   at System.Xml.XmlUtf8RawTextWriter.Flush()
   at System.Xml.XmlWellFormedWriter.Flush()
   at System.Xml.Serialization.XmlSerializer.Serialize(XmlWriter xmlWriter, Object o, XmlSerializerNamespaces namespaces, String encodingStyle, String id)
   at System.Xml.Serialization.XmlSerializer.Serialize(XmlWriter xmlWriter, Object o, XmlSerializerNamespaces namespaces)
   at FubarDev.WebDavServer.Formatters.WebDavXmlOutputFormatter.Serialize[T](Stream output, T data)
   at FubarDev.WebDavServer.WebDavResult`1.ExecuteResultAsync(IWebDavResponse response, CancellationToken ct)
   at FubarDev.WebDavServer.AspNetCore.WebDavIndirectResult.ExecuteResultAsync(ActionContext context)
   at Microsoft.AspNetCore.Mvc.Infrastructure.ResourceInvoker.<InvokeResultFilters>g__Awaited|27_0(ResourceInvoker invoker, Task lastTask, State next, Scope scope, Object state, Boolean isCompleted)
   at Microsoft.AspNetCore.Mvc.Infrastructure.ResourceInvoker.<InvokeFilterPipelineAsync>g__Awaited|19_0(ResourceInvoker invoker, Task lastTask, State next, Scope scope, Object state, Boolean isCompleted)
   at Microsoft.AspNetCore.Mvc.Infrastructure.ResourceInvoker.<InvokeAsync>g__Logged|17_1(ResourceInvoker invoker)
   at Microsoft.AspNetCore.Routing.EndpointMiddleware.<Invoke>g__AwaitRequestTask|6_0(Endpoint endpoint, Task requestTask, ILogger logger)
   at Microsoft.AspNetCore.Authorization.AuthorizationMiddleware.Invoke(HttpContext context)
   at Microsoft.AspNetCore.Authentication.AuthenticationMiddleware.Invoke(HttpContext context)
   ...

See: https://github.com/FubarDevelopment/WebDavServer/blob/release/2.0/src/FubarDev.WebDavServer/Formatters/WebDavXmlOutputFormatter.cs#L71
Enabling the Kestrel option AllowSynchronousIO fixes this issue, but that's of course only a temporary solution.

I can approve that this is fixed in the latest builds from https://pkgs.dev.azure.com/fubar-development/webdav-server/_packaging/webdav-server/nuget/v3/index.json.


Leaderboard: Add zero-shot filter to leaderboard

Add zero-shot generalization to model metadata. This would allow us to filter MTEB models.

Cc'ing @jhyuklee & Xiaoqi Ren (can't find the GitHub handle) as you were very interested in this! Hopefully will be there with a new leaderboard soon, any thoughts from you are ofc very welcome
I recall that you mentioned that https://huggingface.co/BAAI/bge-en-icl has the ArguAna test set in their training dataset they released and relatedly oddly high ArguAna Retrieval scores, so that's one to remember to flag in our metadata for the new leaderboard

Closing in favor of #1556


Align with ILR Wrapper

Could anyone with specific knowledge around ILR comment on it here, and why it matters for wallets?

@kimdhamilton


Replace by fee functionality

Try out this version of the Hiro Wallet - download extension builds.

This PR works on top of https://github.com/blockstack/stacks-wallet-web/pull/1639 to implement the actual replace-by-fee functionality. #1639 made it possible to modify the fee/nonce of a given transaction. This PR makes it so a user can select a pending transaction and submit a duplicate transaction with only the fee changed.
https://user-images.githubusercontent.com/11803153/131561394-c8dbb46c-8dc0-4ef9-8ff0-36640508df06.mp4
cc/ @aulneau @kyranjamie @fbwoolf

lol, post merge review


Stage override for TMP dir

Describe the request
In my setup i have one drive that can do 2 staggered jobs and one that can't. I would like to set override to one of the directories, so one can be set to major 2 and other to major 4. That would significantly increase ploting speed due to not waiting for copy to destination. Right now i need to override via job limit and the drive is sitting almost empty waiting for network storage to recive ~100GB

the easy work around here is to just have a second temp directory on one of the drives.

Thanks! Did not think of that.


How to disable swipe to change month and enable date range select by swipe from one date to other

Please check the sample and wiki as they have example for that.
You can use SELECTION_MODE_RANGE and setPagingEnabled

There is no swipe feature for range selection at the moment but there is a PR currently for it. It's a pretty substantial api change so I would need more testing to make sure nothing break. No plan to add it at the moment but feel free to fork it for yourself


Player preview: Too much play buttons, differences with actual player

Starting the Paella player e.g. via https://naweidner.github.io/paella-ethz/?id=belmar-multiresolution-remote or also via https://paellaplayer.upv.es/player.html?id=belmar-multiresolution-remote confuses the usere with three different play buttons:
Two times the iconic orange paella pan left and right and one "normal" play button in the center:

We should reduce it to one play button.
Btw. the preview is not identical with the actual player you get:

Instructor video is small in the preview and slides are big.  Background color is also different between preview and final player. Navigation bar is visible in the player, not in the preview.
General question: Do we need the preview? Can't we start with the actual player at beginning? Perhaps a preview image makes sense in a portal in case you have dozens of videos on one page and a lot data must be preloaded. But for a single video on a page, do we need the preview there?

Many thanks for the explanation, Miguel / @miesgre ! If it is for testing on the Paella demo page only all is ok, did not know this and had expected that it will show up in all Paella instances after deploying.


Window functions

Produce normalized window functions using the XFaster estimator and covariance model, for use with CosmoMC and similar cosmological likelihood codes.  Window functions are normalized such that sum((ell + 0.5) / ell / (ell + 1) * W) = 1 for each bin.

Merging window functions branch


gallery view miniatures not loading

Describe the bug
gallery view miniatures not loading
To Reproduce
Steps to reproduce the behavior:
just recent and sometimes older photos fail to load the miniatures on the gallery but I can open them without issue
Screenshots

System information and logs:
Package: deckers.thibault.aves
Installer: com.miui.packageinstaller
Aves version: 1.11.2-izzy, build 12101
Flutter: stable 3.22.2
Android version: 12, API 31
Android build: SKQ1.211019.001 release-keys
Device: Xiaomi Redmi Note 9 Pro
Support: dynamic colors=true, geocoder=true, HDR=true
Mobile services: ready
Connectivity: wifi
System locales: en_US
Storage volumes: /storage/emulated/0/, /storage/03E5-266C/
Storage grants: /storage/emulated/0/Movies/, /storage/emulated/0/Pictures/, /storage/03E5-266C/DCIM/, /storage/03E5-266C/downloads/
Error reporting: false

Logs please

Logs please

aves-logs-20240721_161119.txt

Did you somehow remove the permission "ACCESS_MEDIA_LOCATION"?

Did you somehow remove the permission "ACCESS_MEDIA_LOCATION"?

Thanks!, changing to always allowed fixed the issue, I guess Xiaomi just being xiaomi...


[Bug] Backpack V6 - Theme Tabler - Could not find config key: classes.body. Neither in the Backpack theme...

Bug report
"Could not find config key: classes.body. Neither in the Backpack theme, nor in the fallback theme, nor in ui."
What I did
This error appear on every page
What I expected to happen
No Error in log. Log file increase to MBs
What happened
Continue error showing in Log
What I've already tried to fix it
N/A
Is it a bug in the latest version of Backpack?
Yes.
After I run composer update backpack/crud the bug... is it still there?
Yes
Backpack, Laravel, PHP, DB version
When I run php artisan backpack:version the output is:
PHP VERSION:
PHP 8.1.7 (cli) (built: Jun  7 2022 21:45:53) (ZTS Visual C++ 2019 x64)
Copyright (c) The PHP Group
Zend Engine v4.1.7, Copyright (c) Zend Technologies
with Zend OPcache v8.1.7, Copyright (c), by Zend Technologies
with Xdebug v3.1.6, Copyright (c) 2002-2022, by Derick Rethans
LARAVEL VERSION:
<IP_ADDRESS>
BACKPACK PACKAGE VERSIONS:
backpack/backupmanager: v5.0.0
backpack/basset: 1.0.0
backpack/crud: 6.0.6
backpack/editable-columns: 3.0.1
backpack/filemanager: 3.0.2
backpack/generators: v6.x-dev
backpack/logmanager: v5.0.0
backpack/medialibrary-uploaders: 1.0.1
backpack/menucrud: v4.0.1
backpack/newscrud: v5.0.0
backpack/pagemanager: 3.2.0
backpack/permissionmanager: 7.0.0
backpack/pro: 2.0.8
backpack/revise-operation: 2.0.0
backpack/settings: 3.1.0
backpack/theme-coreuiv2: 1.1.3
backpack/theme-coreuiv4: 1.0.4
backpack/theme-tabler: 1.0.5
Attached screenshot

Hey @munjaldevelopment thanks for the report 🙏
We are working on the solution described here: https://github.com/Laravel-Backpack/theme-coreuiv4/issues/38#issuecomment-1648359007
We should lunch the new versions with the fixes soon.
I the meanwhile I am going to close this as we have already other issue opened regarding the same subject.
Cheers

I have a similar issue. Just noticed in the logs:
" Could not find config key: options.showPasswordVisibilityToggler. Neither in the Backpack theme ... "
Regards,
Mark

Hey @de-raaf-media I think that is a different issue. I could not reproduce your error.
showPasswordVisibilityToggler only exists in theme-tabler, and it is in the options for that theme: https://github.com/search?q=org%3ALaravel-Backpack showPasswordVisibilityToggler&type=code
Any chance you are using tabler files with other theme ? If that's the case, just create it in the theme config:
'options' => [
    'showPasswordVisibilityToggler' => true
]

We only have that functionality in theme-tabler, so I don't think it makes sense to add it to the config/ui like the other config keys described here that are used by all the themes.
Let me know, 🙏

It seems this was not present in my theme-tabler.php config file:
"showPasswordVisibilityToggler" => false,


Minor fixes

Hi @Affonso-Gui,
We have reason to use getToolGoalValue instead of getToolValue.
The getToolGoalValue function accesses component_.at(name).tool.goal_value,
and the getToolValue function accesses component_.at(name).tool.value.
To shorten the control cycle, This controller doesn't read a present position of a gripper dynamixel(ID15) in every loop. Therefore, tool.goal_value is set when goal position of the gripper dynamixel(Id15) is write to the dynamixel, and this node publish the tool.goal_value as a previous goal value.

Thanks for your answer.
Having a short test here I didn't saw any relevant changes to performance when using ToolValue rather than ToolGoalValue. Did you have any problems with it?
Tests performed:
Compared the publishing rate of joint_states with both ToolValue and ToolGoalValue.

On normal settings, both publish at 100 hz.
Adjusting iteration frequency and control time, both gave 1000 hz.
In addition, performed the ToolValue/ToolGoalValue reading 10,000 times during the publishing loop, having 770 hz for ToolValue and 780 hz for ToolGoalValue

Other comments:
The following situations are not well supported by GoalValue:

The gripper position at boot time is not read (always appears as 0.0)
On current control or current based position control, the real position can significantly differ from the goal position

On the control_gui, the bottom of the y on y axis was being cut and looking like a v, so tried changing it to uppercase.

Thank you for your review.
The getToolGoalValue() function returns the gripper's target position value,
and the getToolValue() function returns the gripper's present position value received from dxl.
The present position of the gripper was not read every control cycle, at first.
However, we are now changing to reading the present position of the gripper every control cycle.
Therefore, data published to rosmsg should change all of the getToolGoalValue() functions to getToolValue().


Bollinger Band uses standard deviation against a sample (n-1), not a population (N)

Which version are you running? The lastest version is on Github. Pip is for major releases.
print(ta.version)
0.2.23b0

location: '\Lib\site-packages\pandas_ta\volatility\bbands.py'

Describe the bug
The bollinger band volatility indicator uses standard deviation against sample (n-1), and the correct is against population (N).
To Reproduce
In attach, there is data test where it can be reproduced and verified.
bollingerBand.xlsx
Expected behavior
Standard Deviation against population where the literature is based on.
http://confluence.economatica.com/display/FE/Formulas+for+indicators#Formulasforindicators-BollingerBands
And others platforms in the market.
Actual:
standard_deviation = stdev(close=close, length=length)
Fix:
standard_deviation = stdev(close=close, length=length, ddof=0)
Check fixe in attachment DiffReport.pdf
DiffReport.pdf
Screenshots
Check the attachment, comparing the results in tradingview, metatrader and profitchart.
Additional context
In addition,
Documentation default states incorrect default values:
std (int): The long period.   Default: 2
mamode (str): Two options: None or "ema".  Default: "ema"
length = int(length) if length and length > 0 else 5
mamode = mamode.lower() if mamode else "sma"

Thanks for using Pandas TA!

Hello @ffhirata,
I use TA Lib Source to test the accuracy of indicators where the majority of common indicators are >99% correlated including bbands, stdev, and variance which bbands also invokes. So bbands are not technically off even if ddof=1 or 0. I am not opposed to making the change, as I do want to offer flexibility to all users, but I would like you to get some credit for your contribution to making the library better for all users. All you have to do is fork, edit and submit a PR. Whatever changes you recommended in DiffReport.pdf is sufficient.

Standard Deviation against population where the literature is based on.
http://confluence.economatica.com/display/FE/Formulas+for+indicators#Formulasforindicators-BollingerBands
And others platforms in the market.

I wish there was a Common Standard of Technical Analysis defaults. There are many libraries, platforms et al that have one off variations in one way or another. I chose TA Lib to be the base since it is the de facto TA Library that many other libraries, frameworks, platforms use. Also it helps having TA Lib code.
Thanks for helping making Pandas TA more robust!
Thanks,
KJ

Hello @ffhirata,
I use TA Lib Source to test the accuracy of indicators where the majority of common indicators are >99% correlated including bbands, stdev, and variance which bbands also invokes. So bbands are not technically off even if ddof=1 or 0. I am not opposed to making the change, as I do want to offer flexibility to all users, but I would like you to get some credit for your contribution to making the library better for all users. All you have to do is fork, edit and submit a PR. Whatever changes you recommended in DiffReport.pdf is sufficient.

Standard Deviation against population where the literature is based on.
http://confluence.economatica.com/display/FE/Formulas+for+indicators#Formulasforindicators-BollingerBands
And others platforms in the market.

I wish there was a Common Standard of Technical Analysis defaults. There are many libraries, platforms et al that have one off variations in one way or another. I chose TA Lib to be the base since it is the de facto TA Library that many other libraries, frameworks, platforms use. Also it helps having TA Lib code.
Thanks for helping making Pandas TA more robust!
Thanks,
KJ

Hello @twopirllc,
Thanks for the library history, it has a great legacy and confidence.
I have already seen others implementations that also uses the standard deviation against sample.

Eg. of standard deviation against sample warning:
<EMAIL_ADDRESS>https://medium.com/python-data/setting-up-a-bollinger-band-with-python-28941e2fa300

Thanks to open the credit for contribution, I will do it soon, and let you know.
Thanks,
ffhirata

Hello @ffhirata,

Thanks for the library history, it has a great legacy and confidence.

Thanks! It's more work that I thought! 🤣
Thanks for the medium links, I will check them out.

Thanks to open the credit for contribution, I will do it soon, and let you know.

Of course. No rush. It's at a point where I can not do it all solo. Plus people deserve credit if it benefits all.
Kind Regards,
KJ

Hello @twopirllc,
Sorry for long delay,
I did not submit a PR.
I reinstalled the current version pandas-ta-0.2.45b0,
pip install --force-reinstall pandas_ta
and the calculation of standard deviation against sample (n-1) was fixed to against population (N).
\Lib\site-packages\pandas_ta\volatility\bbands.py
    # Calculate Result
    standard_deviation = stdev(close=close, length=length, ddof=0)

Best Regards,
ffhirata

Hello @ffhirata,
FYI. I am adding argument ddof=0 to ta.bands().
Kind Regards,
KJ


Eliminate the core framework from all default style of frontend (bootstrap 3) framework in 2.1

For 2.1 version, I think the core framework must clean from bootstrap style especially in default variable config, what I see :

Widget
Grid
etc

Have a default bootstrap class style,
From beginner thinking, yii 2 is framework for bootsrap 3, because the html class or style is write for this framework by default not pure html, in yii 2.1 this framework must provide enlightenment to beginner that this framework can be used for all html pure/frameworks
For example,

ActionColumn : have default value icon a part of Bootstrap glyphicon
GridView : have a default variable class 'table table-striped table-bordered' who only bootstrap needed, not pure html need
ActiveForm : errorCssClass (has-error),  successCssClass(has-success)
ActiveField : default variable in style, container, etc like 'form-group, form-control' is write by default.
Etc

I'm a bootstrap fan may be (truly) helpful for this, but when used on another ui framework it will all be useless

Ok, we give up the bootstrap. Then what should be the basic design for all widgets? What will they look like? Will they look like basic html elements? Or will it be some other basic design?

Basic HTML elements, yes.

And what about icons? For example, ActionColumn has icons from bootstrap glyphicon set:
https://github.com/yiisoft/yii2-framework/blob/f01cb72663342c7527519ab576df045e59900ce6/grid/ActionColumn.php#L145-L147
What icons will used as replacement of them?

For default maybe with title
....
$title = Yii::t('yii', 'View');
....
return Html::a($title , $url, $options);

And vertical line or space as separator (maybe add option config for this about separator) like :
View | Update | Delete
View Update Delete

Unicode, I guess. ▲ ▼ ✎ ❌

Text is more safe, with unicode still has no clear interpretation about 'this icon/symbol'
will still use the (href tag) title to clarify what this icon/symbol is
Unicode as Symbol

I guess still pure text
Title

Yes, title should present anyway.

1. Unicode as Symbol
2. Title
3. Title

In this option what number you prefer @samdark ?

1st

Title attribute for link in pure html only show when hover, right ?
In case, user first time open website in mobile phone or touchscreen device, with symbol user has no clear interpretation about the meaning of symbol, I think

Title attribute for link in pure html only show when hover, right ?


Tutorial xsa-sqlscript-cube: Misleading description, wrong positioned screenshots, missing screenshot

Tutorial xsa-sqlscript-cube has several issues:

The description says that a view is created with a star join. No star join is used. In addition the description ("Purchase Orders with Currency Conversion") for the view lets assume that a currency conversion is done, but no such conversion is done. There is just a calculated column "ConvGrossAmount" created which gets the value of column GrossAmount w/o additional changes. Also the input parameter created to specify a target currency is never used again.

Wrong positioned screenshots: Beginning with step 4 the screenshots are wrong. The screenshot of step 4 belongs still to step 3. All further screenshots have to be moved to one step before.

Missing screenshot: For the last step (step 18) the screenshot is missing to be able to compare the output (according to the text "For an initial test make sure your output looks similar to the following:" in the tutorial).

Sorry about that, we are working to update it now to match to the upcoming SPS12 release as well.


examples  & documentation: PlainClone with Basic Authentication (Password & Access Token)

Proposed Additions

Add a GoDoc example for cloning with BasicAuth
Add a GoDoc example for cloning with "Token" Auth
Add cli example for cloning with BasicAuth to _examples
Add cli example for cloning with "Token" Auth

Why does TokenAuth not work?
If this is something that would be of value, and there isn't a specific reason why it hasn't been implemented, I would love to deep dive into this. I think it would be a great follow-up, then we could clean up the "Token" auth.

@mccurdyc Does TokenAuth not work? It was added to support OAuth and it seemed to work, at least in Bitbucket, https://github.com/src-d/go-git/pull/859
If we are talking about GitHub's personal access tokens, using them as password is the intended usage AFAIK: https://help.github.com/articles/creating-a-personal-access-token-for-the-command-line/#using-a-token-on-the-command-line

Looks good to me, thanks!

I apologize, I was incorrectly using the GitHub personal access token.

Common issue, since it is not documented ;-)

@mccurdyc can you add this examples to the https://github.com/src-d/go-git/blob/master/_examples/README.md?

@mcuadros I have added those to the list of examples. Thanks!