[Table] Allow HTML or component column titles

What package is this for?

[x] React
[ ] Angular

Summary
I have at least three uncovered use case for title columns:

Use an icon only for the visible column "name". This is for columns that themselves just show status icons, so a full name is wasteful of space. See the "has comments" column below.
Columns with icon decorators. See below for an "Embedded In" link column that show an orange left-arrow in indicate it's an incoming use case.
Long tooltips on columns where embedded help is required.

I think all of these can just be solved by the app if columns can provide either their own HTML or a custom React component for their titles. Only text labels are supported currently.

Specific timeline issues / requests
Elm 7.0.3. Around June.

@dianatran18 Do we have any design guidance around this?

For icon-only columns, this is the guidance we have around that right now:
https://pages.github.ibm.com/ai-applications/design/components/datatable/columnalignment#icon-only-columns
I think having text in the icon-only column header makes it more clear than just having an icon, as icons can be ambiguous and may not always be clear to users. Especially since the rows in the column are all icons, having a descriptive column header is valuable.

Similar to the first answer, I think relying on the icon alone to describe "incoming" in this use case is a little ambiguous and not very clear. Having icons in the column header also starts to add more clutter to an already dense data table, and starts to compete with other icons associated with the column header such as the sort icon. I think using the tooltip to help further describe the column is valuable and works better than the icon. In this specific use case, the color of the icon also conflicts with hover/sort states for the column header, and wouldn't pass accessibility against any column header background color.

This we did just add recent guidance on, which you can find here:
https://pages.github.ibm.com/ai-applications/design/components/datatable/general#column-header
This was feedback we received from MAS to try to create more alignment on how we display tooltips in column headers. Similar to everything described above, we're trying to move away from using info icons in the column header, as it adds more clutter and also competes with other icons such as the sort icon. We are recommending to use the definition tooltip styling Carbon provides to display tooltips.

@davidicus I believe there's a related issue or conversation somewhere about using the definition tooltip in column headers right?

@davidicus I believe there's a related issue or conversation somewhere about using the definition tooltip in column headers right?

@dianatran18 #2272 was merged last week for tooltips in the headers. See this story.

Based on the feedback above I am going to close this issue as it sounds like this is not a desirable pattern for the PAL or AI applications. If there are constraints that won't allow you to change this pattern in your app let's discuss in our PAL office hours so we can work on possible solutions and/ or any safety hatches we may need to code in to allow for variances.


Add maz-ui

https://maz-ui.com/

Won't be added.
See https://github.com/ddahan/ui-libs?tab=readme-ov-file#why-library-x-is-not-listed


Add Himalayas as remote job board

Pull Request Template
Please ensure all items below are checked before creating a pull request:

[x] My submission is formatted according to the guidelines in the contributing guide
[x] My submission is ordered alphabetically
[x] My submission has a useful description
[x] I have searched the repository for any relevant issues or pull requests
[x] All changes have been squashed into a single commit

Added https://himalayas.app/ as a remote job board resource.

Thank you!


Issue 2031 Resolved by removing a redundant line.

A redundant line ("using HotChocolate;") has been removed. This line triggered the compiler warning.
Merging shall resolve this issue
https://github.com/ChilliCream/hotchocolate/issues/2031

Why does this trigger a compiler warning? If we remove that you do not have the extension methods available on the schema builder.

@michaelstaib The reason for the warning is described by the warning message itself (so there is no need to ask me) which I documented (https://github.com/ChilliCream/hotchocolate/issues/2031): "The using directive for 'HotChocolate' appeared previously in this namespace".
The only thing I removed is a duplication of a using directive. This becomes clear from these lines where the first one and the lost one are the same: https://github.com/ChilliCream/hotchocolate/blob/master/templates/Server/content/Startup.cs#L1-L5
All the using directives remain available after my change so the extension methods remain available too. But there is no need for duplication of using directives. I removed the duplication which makes the code more clear and resolves the warning I referred too.  Feel free to reproduce it with the steps I provided in the issue. The warning really exist and really disappears after my change.

I see, overlooked this one yesterday night. Did not see that the using HotChocolate; was declared twice. Good catch.
Thanks for your contribution.

Welcome as a Hot Chocolate contributor 🎉


CI: Gitea commit exporter

TO-DO create CI for Backend

Closing as it's now covered by PR jobs and also periodic job configs:

https://github.com/konveyor/mig-demo-apps/blob/master/apps/todolist-mongo-go/pelorus/values.yaml#L71-L83
https://github.com/konveyor/mig-demo-apps/blob/master/apps/todolist-mongo-go/pelorus/periodic/quay_images_latest.yaml#L68-L79

The above are invoked by Makefile from our script:

https://github.com/konveyor/pelorus/blob/master/Makefile#L138-L144
https://github.com/konveyor/pelorus/blob/master/scripts/run-pelorus-e2e-tests#L451-L471

By various OpenShift versions:

https://github.com/openshift/release/tree/master/ci-operator/config/konveyor/pelorus

For the PR and periodic jobs:

https://github.com/openshift/release/tree/master/ci-operator/jobs/konveyor/pelorus

Note we will be working on getting this compatible with Pelorus Operator workflow, but the general CI workflow will stay as it is, just some config files will get updated. Here is prepared config file:

https://github.com/konveyor/mig-demo-apps/blob/master/apps/todolist-mongo-go/pelorus/values_v1alpha1_pelorus.yaml


xgboost dart dask fails while gbtree does not:  AttributeError: '_thread._local' object has no attribute 'execution_state'

2021-01-15 05:54:54,522 C: 68% D:836.0GB M:224.8GB NODE:REMOTE1     7243   INFO   | [XGBOOSTDARTDASK] model fit (963689, 208) start.
2021-01-15 05:54:55,605 C: 24% D:836.1GB M:225.0GB NODE:REMOTE1     7243   INFO   | [XGBOOSTDARTDASK] model fit (963689, 208) end.   Error: [Traceback (most recent call last):
2021-01-15 05:54:55,610 C: 24% D:836.1GB M:225.0GB NODE:REMOTE1     7243   DATA   |   File "/opt/h2oai/dai/cuda-10.0/lib/python3.6/site-packages/distributed/worker.py", line 3072, in get_worker
2021-01-15 05:54:55,610 C: 24% D:836.1GB M:225.0GB NODE:REMOTE1     7243   DATA   |     return thread_state.execution_state["worker"]
2021-01-15 05:54:55,611 C: 24% D:836.1GB M:225.0GB NODE:REMOTE1     7243   DATA   | AttributeError: '_thread._local' object has no attribute 'execution_state'
2021-01-15 05:54:55,611 C: 24% D:836.1GB M:225.0GB NODE:REMOTE1     7243   DATA   |
2021-01-15 05:54:55,611 C: 24% D:836.1GB M:225.0GB NODE:REMOTE1     7243   DATA   | During handling of the above exception, another exception occurred:
2021-01-15 05:54:55,612 C: 24% D:836.1GB M:225.0GB NODE:REMOTE1     7243   DATA   |
2021-01-15 05:54:55,612 C: 24% D:836.1GB M:225.0GB NODE:REMOTE1     7243   DATA   | Traceback (most recent call last):
2021-01-15 05:54:55,613 C: 24% D:836.1GB M:225.0GB NODE:REMOTE1     7243   DATA   |   File "/opt/h2oai/dai/cuda-10.0/lib/python3.6/site-packages/distributed/worker.py", line 3075, in get_worker
2021-01-15 05:54:55,613 C: 24% D:836.1GB M:225.0GB NODE:REMOTE1     7243   DATA   |     return first(w for w in Worker._instances if w.status == "running")
2021-01-15 05:54:55,613 C: 24% D:836.1GB M:225.0GB NODE:REMOTE1     7243   DATA   |   File "cytoolz/itertoolz.pyx", line 550, in cytoolz.itertoolz.first
2021-01-15 05:54:55,614 C: 24% D:836.1GB M:225.0GB NODE:REMOTE1     7243   DATA   |   File "cytoolz/itertoolz.pyx", line 557, in cytoolz.itertoolz.first
2021-01-15 05:54:55,614 C: 24% D:836.1GB M:225.0GB NODE:REMOTE1     7243   DATA   | StopIteration
2021-01-15 05:54:55,615 C: 24% D:836.1GB M:225.0GB NODE:REMOTE1     7243   DATA   |
2021-01-15 05:54:55,615 C: 24% D:836.1GB M:225.0GB NODE:REMOTE1     7243   DATA   | During handling of the above exception, another exception occurred:
2021-01-15 05:54:55,615 C: 24% D:836.1GB M:225.0GB NODE:REMOTE1     7243   DATA   |
2021-01-15 05:54:55,616 C: 24% D:836.1GB M:225.0GB NODE:REMOTE1     7243   DATA   | Traceback (most recent call last):
2021-01-15 05:54:55,616 C: 24% D:836.1GB M:225.0GB NODE:REMOTE1     7243   DATA   |   File "h2oaicore/dask.py", line 97, in h2oaicore.dask.get_dask_client_manager
2021-01-15 05:54:55,617 C: 24% D:836.1GB M:225.0GB NODE:REMOTE1     7243   DATA   |   File "/opt/h2oai/dai/python/lib/python3.6/contextlib.py", line 81, in __enter__
2021-01-15 05:54:55,617 C: 24% D:836.1GB M:225.0GB NODE:REMOTE1     7243   DATA   |     return next(self.gen)
2021-01-15 05:54:55,617 C: 24% D:836.1GB M:225.0GB NODE:REMOTE1     7243   DATA   |   File "/opt/h2oai/dai/cuda-10.0/lib/python3.6/site-packages/distributed/worker_client.py", line 41, in worker_client
2021-01-15 05:54:55,618 C: 24% D:836.1GB M:225.0GB NODE:REMOTE1     7243   DATA   |     worker = get_worker()
2021-01-15 05:54:55,618 C: 24% D:836.1GB M:225.0GB NODE:REMOTE1     7243   DATA   |   File "/opt/h2oai/dai/cuda-10.0/lib/python3.6/site-packages/distributed/worker.py", line 3077, in get_worker
2021-01-15 05:54:55,619 C: 24% D:836.1GB M:225.0GB NODE:REMOTE1     7243   DATA   |     raise ValueError("No workers found")
2021-01-15 05:54:55,619 C: 24% D:836.1GB M:225.0GB NODE:REMOTE1     7243   DATA   | ValueError: No workers found
2021-01-15 05:54:55,619 C: 24% D:836.1GB M:225.0GB NODE:REMOTE1     7243   DATA   |
2021-01-15 05:54:55,620 C: 24% D:836.1GB M:225.0GB NODE:REMOTE1     7243   DATA   | During handling of the above exception, another exception occurred:
2021-01-15 05:54:55,620 C: 24% D:836.1GB M:225.0GB NODE:REMOTE1     7243   DATA   |
2021-01-15 05:54:55,620 C: 24% D:836.1GB M:225.0GB NODE:REMOTE1     7243   DATA   | Traceback (most recent call last):
1886, in h2oaicore.models.MainModel.dask_fit
2021-01-15 05:54:55,624 C: 24% D:836.1GB M:225.0GB NODE:REMOTE1     7243   DATA   |   File "h2oaicore/dask.py", line 179, in h2oaicore.dask.get_client_manager
2021-01-15 05:54:55,624 C: 24% D:836.1GB M:225.0GB NODE:REMOTE1     7243   DATA   |   File "h2oaicore/dask.py", line 102, in h2oaicore.dask.get_dask_client_manager
2021-01-15 05:54:55,624 C: 24% D:836.1GB M:225.0GB NODE:REMOTE1     7243   DATA   |   File "h2oaicore/dask.py", line 85, in h2oaicore.dask.get_dask_client
2021-01-15 05:54:55,625 C: 24% D:836.1GB M:225.0GB NODE:REMOTE1     7243   DATA   |   File "/opt/h2oai/dai/cuda-10.0/lib/python3.6/site-packages/distributed/worker.py", line 3118, in get_client
2021-01-15 05:54:55,625 C: 24% D:836.1GB M:225.0GB NODE:REMOTE1     7243   DATA   |     address = comm.resolve_address(address)
2021-01-15 05:54:55,626 C: 24% D:836.1GB M:225.0GB NODE:REMOTE1     7243   DATA   |   File "/opt/h2oai/dai/cuda-10.0/lib/python3.6/site-packages/distributed/comm/addressing.py", line 174, in resolve_address
2021-01-15 05:54:55,626 C: 24% D:836.1GB M:225.0GB NODE:REMOTE1     7243   DATA   |     return unparse_address(scheme, backend.resolve_address(loc))
2021-01-15 05:54:55,626 C: 24% D:836.1GB M:225.0GB NODE:REMOTE1     7243   DATA   |   File "/opt/h2oai/dai/cuda-10.0/lib/python3.6/site-packages/distributed/comm/tcp.py", line 533, in resolve_address
2021-01-15 05:54:55,627 C: 24% D:836.1GB M:225.0GB NODE:REMOTE1     7243   DATA   |     return unparse_host_port(ensure_ip(host), port)
2021-01-15 05:54:55,627 C: 24% D:836.1GB M:225.0GB NODE:REMOTE1     7243   DATA   |   File "cytoolz/functoolz.pyx", line 476, in cytoolz.functoolz._memoize.__call__
2021-01-15 05:54:55,628 C: 24% D:836.1GB M:225.0GB NODE:REMOTE1     7243   DATA   |   File "/opt/h2oai/dai/cuda-10.0/lib/python3.6/site-packages/distributed/utils.py", line 711, in ensure_ip
2021-01-15 05:54:55,628 C: 24% D:836.1GB M:225.0GB NODE:REMOTE1     7243   DATA   |     raise exc
2021-01-15 05:54:55,628 C: 24% D:836.1GB M:225.0GB NODE:REMOTE1     7243   DATA   |   File "/opt/h2oai/dai/cuda-10.0/lib/python3.6/site-packages/distributed/utils.py", line 704, in ensure_ip
2021-01-15 05:54:55,629 C: 24% D:836.1GB M:225.0GB NODE:REMOTE1     7243   DATA   |     hostname, 1234, fam, socket.SOCK_STREAM  # dummy port number
2021-01-15 05:54:55,629 C: 24% D:836.1GB M:225.0GB NODE:REMOTE1     7243   DATA   |   File "/opt/h2oai/dai/python/lib/python3.6/socket.py", line 745, in getaddrinfo
2021-01-15 05:54:55,630 C: 24% D:836.1GB M:225.0GB NODE:REMOTE1     7243   DATA   |     for res in _socket.getaddrinfo(host, port, family, type, proto, flags):
2021-01-15 05:54:55,630 C: 24% D:836.1GB M:225.0GB NODE:REMOTE1     7243   DATA   | socket.gaierror: [Errno -2] Name or service not known
2021-01-15 05:54:55,630 C: 24% D:836.1GB M:225.0GB NODE:REMOTE1     7243   DATA   | ].

@trivialfis  Odd thing here is only happens when using cluster but just one node.  However, I'm running the client from a separate docker instance to the scheduler/workers.  There are 4 GPUs so 4 workers.  This is on ec2 system.  The processes look like:
root       7243  5.9  0.2 4427060 713376 ?      Sl   Jan14  25:32          \_ h2oai-server
root       7580  1.8  0.0<PHONE_NUMBER>88 ?       Sl   Jan14   7:49              \_ dask-scheduler [tcp://<IP_ADDRESS>:8786]
root       7581  1.7  0.1 1335968 303512 ?      Sl   Jan14   7:38              \_ dask-scheduler [tcp://<IP_ADDRESS>:8790]
root       8058  2.4  0.2 5860772 623180 ?      Sl   Jan14  10:27              \_ h2oai_worker
root       8172  1.1  0.0<PHONE_NUMBER>2 ?        Sl   Jan14   5:03                  \_ /opt/h2oai/dai/python/bin/python /opt/h2oai/dai/python/bin/dask-worker tcp://<IP_ADDRESS>:8786 --pid-file ./tmp/dai_dask_worker_n_jobs_2021-01-14_23_49_29.000901.pid --nthreads 1 --nprocs 1 --protocol tcp --resources n_jobs=1 --local-directory ./tmp/dask_worker_files
root       8190  0.0  0.0  26420 11080 ?        S    Jan14   0:00                  |   \_ /opt/h2oai/dai/python/bin/python -c from multiprocessing.semaphore_tracker import main;main(8)
root       8192  138  0.3 2079728 990540 ?      Sl   Jan14 630:04                  |   \_ dask-worker [tcp://<IP_ADDRESS>:46716]
root       8173  2.6  0.1 19749084 288752 ?     Sl   Jan14  12:13                  \_ /opt/h2oai/dai/python/bin/python /opt/h2oai/dai/python/bin/dask-cuda-worker tcp://<IP_ADDRESS>:8790 --pid-file ./tmp/dai_dask_cuda_worker_n_jobs_2021-01-14_23_49_29.000924.pid --nthreads 1 --resources n_gpus=1 n_jobs=1 --local-directory ./tmp/dask_cuda_worker_files
root       8287  0.0  0.0  26420 11120 ?        S    Jan14   0:00                  |   \_ /opt/h2oai/dai/python/bin/python -c from multiprocessing.semaphore_tracker import main;main(39)
root       8291  2.5  0.6 22073768 1699308 ?    Sl   Jan14  11:30                  |   \_ dask-worker [tcp://<IP_ADDRESS>:36126]
root       8293  2.5  0.6 22285456 1745476 ?    Sl   Jan14  11:31                  |   \_ dask-worker [tcp://<IP_ADDRESS>:35708]
root       8295  2.5  0.6 22183220 1691412 ?    Sl   Jan14  11:31                  |   \_ dask-worker [tcp://<IP_ADDRESS>:43614]
root       8298  2.5  0.6 22172068 1681012 ?    Sl   Jan14  11:29                  |   \_ dask-worker [tcp://<IP_ADDRESS>:42043]

Note I have 1 "CPU" scheduler and worker and 1 "GPU" with 4 workers launched by the dask-cuda-worker.  Here I'm running on GPU.
For some reason gbtree works perfectly.  However, dart fails every time with the above error.  Any ideas?
It's like dart somehow ruins the threading state, while gbtree does not.

For some reason gbtree works perfectly. However, dart fails every time with the above error. Any ideas?

Probably empty dataset handling in dart, just guessing.

For some reason gbtree works perfectly. However, dart fails every time with the above error. Any ideas?

Probably empty dataset handling in dart, just guessing.

@pseudotensor Could you please try again after https://github.com/dmlc/xgboost/pull/6693 being merged?

Merged.  Now dart is running on CI.

We have some fixes and tests for dart in 1.4rc.  Feel free to reopen if the issue persists.


Update MAX32570 camera interface

MSDK-1075: MAX32570 camera interface update
P1.7 is connected to camera on EvKits (M/Q, MN/QN)
Tested with CameraIF and BarcodeDecoder example it works.

@Jake-Carter @sihyung-maxim  can it be merged?


Wrong version for androidx.compose.material3:material3

[x] I have read the guidelines for submitting issues

🐛 Describe the bug
Migrating a material3 dependency will use the same version as the regular material dependency (1.2.1 as of this writing) but the actual version is 1.0.0-beta03.
⚠ Current behavior
Migrating a material3 dependency will use the same version as the regular material dependency (1.2.1 as of this writing) but the actual version is 1.0.0-beta03.
✅ Expected behavior
Migrating a material3 dependency should use the correct version number.
💣 Steps to reproduce
Add implementation("androidx.compose.material3:material3:1.0.0-beta03") to your repo, then run a migration

Which migration mode you use exactly?


Fetch causes wal-e to hang and database seems corrupted because of postgresql version

Hi
I'm trying to fetch a database from S3 to my local machine (Freebsd) using the following command:

envdir /usr/local/etc/wal-e/env wal-e backup-fetch /var/lib/postgres/data

Knowing that the database on S3 was pushed using a server running Archlinux (exact same version of Postgresql 9.6.1).
When I run the fetch, it not only hangs, but when I stop it and try to run the postgres service, I get the following error: An old version of the database format was found.

Nevermind, I was trying to fetch a postgresql database that was archived in a Freebsd server into a machine running Archlinux. There must have been some compatibility issues. Now that I'm testing the fetch command on the same distro that did the backup, it's working correctly.


Remove duplicate code mentioned at:ngsi-timeseries-api/src/reporter/tests/test_timescale_types.py

Proposed changes
remove the duplicate code in src/reporter/tests/test_timescale_types.py
Types of changes
What types of changes does your code introduce to the project?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Maintenance (update of libraries or other dependences)

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

[X] I have read the CONTRIBUTING doc
[X] I have signed the CLA
[X] I have added tests that prove my fix is effective or that my feature works
[X] I have run all the existing tests locally (not just those related to my feature) and there are no errors
[X] After the last push to the PR branch, I have run the lint script locally and there are no changes to the code base
[X] I have updated the RELEASE NOTES
[X] I have added necessary documentation (if appropriate)
[X] Any dependent changes have been merged and published in downstream modules

Further comments
If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc...

Hi @chicco785 @c0c0n3
I have raised a PR (pr655) for issue 655 , which removes the duplicity in the code , Please check the workflow.
Thanks

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

@NEC-Vishal so it looks like we've got a couple of issues

Markdown linter is moaning about extra whitespace: RELEASE_NOTES.md:21 MD012/no-multiple-blanks Multiple consecutive blank lines [Expected: 1; Actual: 2]
Your original code changes went missing? The only thing showing as a change in this PR is a line in the release notes?

Please help! :-)

hi @c0c0n3 sorry for this mistake at my end while raising PR.
I have updated the PR now,
Please check!

Hi @c0c0n3 @chicco785
can you Please review the updated PR for issue #655 .

@NEC-Vishal good stuff, thanks so much.
There's still one snag. Like in #653, the GitHub code linter action is blocking this PR from being merged b/c autopep8 sniffed out something.  Can you please try running the linter locally and fixing whatever autopep is moaning about, then commit the fixes.

Hi @c0c0n3
Like in #653
can we also merge this PR also as all test cases are passed except linter?

@NEC-Vishal yea, same problem as in #653. no clue what the cause is, but clearly not an issue w/ your code. So I'm merging this PR. Thank you sooo much for your contrib!


[BUG] Fronds api mixin doesn't release event bindings

Describe the bug
When an action modal performs a network action it fires a fronds network event so that the submitting form knows what to listen to on the event bus. The event bus is going to be replaced soon, so this bug could just be solved by that. There's a lack of namespacing so the event fires even when it shouldn't for forms that aren't even displayed.
It's also possible there is some other conflict, but that's unknown at this point
To Reproduce
Steps to reproduce the behavior:

load the admin page fresh, no items selected
click menus
open the "add menu" modal
open network tools in any browser
press ok without filling in any fields to try and trigger a validation error
note the network call
The /page endpoint is called and page errors returned

Expected behavior
The menu endpoint is called and appropriate errors returned

This should be fixed alongside #42


Variable names

We don't have super consistent variable naming. With a breaking 1.0 release, should we go through and systematize it all to snake_case (so, for example, things like 'N_samples' -> 'n_samples').
Also, some specific cases:

filter overwrites the python filter function, which can be really problematic.

Especially because NDSP by default imports all functions directly in local namespace, this squashes filter always, meaning you can never use Python filter, and so has side effects / breaks things

psd, as a function, strikes me as not a great name. Elsewhere, psd is used as a variable name, and it sounds like data more than a function

Heuristic: function names should be 'verb-y', because they do things.

We also have a bunch of single character variables, but this (in some cases), might be fine for the math-y parts, and also some names, like 'Fs', that are non-compliant, but (I think), are inherited from style in scipy.signal, or similar. Do we want to update those names too?

could either change the name (use filter_ts()) or don't do the auto imports in init, so it doesn't squash unless explicitly done.
compute_spectrum ?
I'm good with all snake case, including changing Fs to fs.

do we have assignments for this? Are @srcole and I cleaning our respective modules, or somebody wants to just do it wholesale?

if you can do filt and spectral and sim, I'll do timefrequency and swm and laggedcoherence and burst

For consistency between files - wholesale might be better.
I'm happy to do a first pass, on a branch, and y'all can scroll through and make any tweaks, if you want?

This is a case where working collaboratively on a branch (as opposed to a fork) is probably useful.

Like, this is a case where it might be good to coordinate. I made a branch with a first pass on 'filt'. Are we going with 'filter_signal', for the function name?
There are a couple more single char variables I would like to update (a, b, w, h) - but not sure best new names. Suggestions?

Some particular suggestions - how do we feel about:

x -> sig
Fs -> freqs

(I'll update this as I scroll through.)

a, b, w, h are common DSP standards, as used in scipy.signal.freqz()
https://docs.scipy.org/doc/scipy/reference/generated/scipy.signal.freqz.html

Also, I prefer Fs for "sampling frequency". "freqs" looks like "frequencies" to me.
 Try updating the filter function right now, to change where how 'a' & 'b' are used...
'Fs' - ahh yes, of course. So 'fs' then? Or 's_rate' I think is clearer.
Can we update 'T'?

I really think they should stay a,b,w,h. Especially cause the only reason they exist is to plot the frequency response, which is generally defined using these variable names

Agree with @srcole here. Even if they "suck" they're relatively standard convention, and used in the functions we're calling and emulating from scipy.

'names' is a WIP candidate. All single variable names need manual updating to finish it up (this is why they suck).
I don't see much reason to be overly attached to scipy conventions, in so far as we can be consistent with but better than, but that's just my vote.

my votes:

Fs -> fs: no capitals to be consistent, and fs is in every single DSP textbook. Perhaps not clear that the s is suppose to be subscript, so I'm fine with f_s too. But s_rate decouples the "frequency" component of it (as in, Nyquist) and introduces another unnecessary term, which will confuse people when they do go back and read a DSP textbook, as they inevitably will have to at some point.

keep the filter variable names (a,b,w,h) - again, those are straight out of engineering convention, and would really confuse people if they were to find the equivalent in text if we called it something else. It's not so much to be consistent with scipy but with the engineering conventions. I'd understand if neurodsp is the primary and only reference for people doing DSP, but it is not and it most certainly shouldn't be.

T -> N_seconds: sig_length could indicate sample number, which is not what that variable is. N_seconds makes it very clear on sight.

Honest question: who is our presumed audience / user base? Because it's not clear it's engineer, which all of this discussion presumes.

that's exactly the point - we're NOT writing for engineers, we're writing for people who may need to consult an engineering textbook at some point with fresh eyes, i.e. probably NOT engineers. The people that can look at the code and be confused about the naming convention are the people we don't need to worry about. Annoying for them, but ultimately not a problem. This is the thought I run into every time I see srate in a dataset, because I know that srate and fs are the same thing.
It IS a problem if we change it to be more 'user-friendly' and somebody new to DSP just cannot find on the internet or in a book what those terms mean, because some of them don't even have verbal definitions. Are those variable conventions in engineering books 'good'? Arguably not, mostly because they are math symbols and not words. But I don't think we're in the business of fixing engineering convention just quite yet, and I think this falls more within the scope of DSP, instead of best coding practices (which is probably why scipy keeps them as well).
I don't mind the name change but I am very adamant on this point: we/I do NOT want people to look at our code, think they understand a term with a formal definition because it's in plain English, and use that as a point of reference, instead of scipy or a textbook. We could comment better to point them to the correct resources if that helps.

Yes, of course people can read docstrings, explaining conventions & choices. I'm asking what the conventions should be, and who the module is for. Naming things is very difficult. Being community-driven is the one of the hardest parts of OSS. I think it's worth the discussion.
There are two types of variables: user facing & internal. I'm not recommending a fundamentally different name (name in terms of what it relates to), anywhere.
For internals:

What matter is code maintainability. Single character names are terrible for this. You can't see where they are used, or edit them. So, I'd argue for things like 'a' -> 'a_val' or similar. If you think that is too different or too verbose (and worth making the code harder to maintain), okay - do that. But 'a' & 'b' are literally the worse possible variable names, and I'm unconvinced they are worth keeping as is.

For public facing:

Convention & consistency matter, as does clarity. I'm unconvinced by just following what comes before.

I totally agree with your last point Richard, but have a different view of how it cashes out. The symbol 'Fs' (in math) refers to the concept of sampling rate, yes? They are equivalent symbols, for the same referent? If not, then point taken, but if they are equivalent referents, then I think it's valid to use one or the other. Sampling rate is easier to search for even. If so, math symbol 'Fs'  appears in textbooks, as does the term 'sampling rate', and I don't understand how one or the other somehow decouple us from other resources. Basically: I don't see 'user friendly' as meaning 'less rigorous', and I think people conflate the two much more often than this is actually true.
At some level, it is a different of opinion on to what degree code should look like math. I think code and math are different. I think it's very different coming at this as an engineer, and coming in from a different entry point, I find 'fs' a much more confusing name.
Scipy is not PEP8. If we follow scipy, we have chosen a different style guide, which prioritizes math. I disagree with that, for code, and I find scipy code hard to use & read, and it is also hard to maintain - but you can choose to use their style guide, if you prefer.

I'm happy with appending _val to the single letter vars, for the sake of maintainability.
Rest of them look good to me.


Grist Forms - Redirecting loop error

Hello, i am working with Grist Forms and found a bug when i want to access a published form.
Problem
I get a redirecting loop error accessing the forms link :  http://localhost:9999/forms/47QFRwq6wLjhGTYaUAckWG/4

This bug came up when using Grist Omnibus with email/password authentication and you have a logged in session.
(Grist Version 1.1.13)
The log output:
2024-05-06 07:20:27.067 - debug: Authorizer: redirecting to log in
2024-05-06 07:20:27.095 - debug: Authorizer: redirecting to log in
2024-05-06 07:20:27.119 - debug: Authorizer: redirecting to log in
2024-05-06 07:20:27.155 - debug: Authorizer: redirecting to log in
2024-05-06 07:20:27.209 - debug: Authorizer: redirecting to log in
2024-05-06 07:20:27.251 - debug: Authorizer: redirecting to log in
2024-05-06 07:20:27.288 - debug: Authorizer: redirecting to log in
2024-05-06 07:20:27.319 - debug: Authorizer: redirecting to log in
2024-05-06 07:20:27.350 - debug: Authorizer: redirecting to log in
2024-05-06 07:20:27.378 - debug: Authorizer: redirecting to log in

Solution
I figured out that the formMiddleware is missing some redirects as you can see here:
https://github.com/gristlabs/grist-core/blob/51ec221a1972a94cd3a545d563bcfe48a675cf5d/app/server/lib/FlexServer.ts?plain=1#L1121-L1123
When adding the redirects used for docMiddleware, it is working fine:
formMiddleware: [
        this._redirectToHostMiddleware,
        this._userIdMiddleware,
        forcedLoginMiddleware,
        this._redirectToOrgMiddleware,
        welcomeNewUser
      ],

If I use Grist only instead of Grist Omnibus it is working but I dont know why maybe because no authentication and redirect is happening. Also the form access link is different: http://localhost:8484/o/docs/forms/5jzQzr3NL8tYwALbBcJkBz/4
I am not into that code base so maybe someone can have a closer look.
Is this fix valid or do i miss something and the fix should be in my Omnibus setup?

Having a similar issue with forms. I'm using zitadel OIDC SSO, which is working with Grist otherwise, only forms seem to have a problem. Here's the console output:

Do you really need GRIST_FORCE_LOGIN to be set?
AFAICT, with this option, the forms work.

Thanks @fflorent, thanks for your comment, but I do not want the forms to be publicly accessible, and this problem occurs both when users are logged in prior to trying to access the form or when they go to the form url and are sent to the login page. As soon as they are logged in, this error occurs.
The docs say this about FORCE_LOGIN, and that is what I want to happen, as I do not want anonymous users in the system at all. So as far as I can tell, that setting is correct.
GRIST_FORCE_LOGIN: (optional) when set to true this will instruct Grist to redirect anonymous users to a login page.

This turns out to be easy to replicate in grist-omnibus, local setup, default example in README. Parallel thread in forms here: https://community.getgrist.com/t/forms-not-working-on-self-hosted-grist/5212

@georgegevoian and @berhalak I'd appreciate your eyes on this issue [when you're available obviously, otherwise enjoy your respective vacations :-)].

A fix has now landed in grist-omnibus. It's also in the latest grist-core image, and will be included in the next versioned release (1.1.15).
Let us know if you run into any issues. Thanks!

@s-karrer thanks for digging into the problem. Can you confirm that adding just this._userIdMiddleware to formMiddleware addresses your issue? (That fix should be landing shortly.)

Yes just adding this._userIdMiddleware is working fine. Thank you!

Is this problem solved then? :)


Adapt multipart support to servant-0.9.

With pull request #70, the comments that said that the functions
performRequest' and performRequestCT' have been copied from the Servant
source code and then adapted got out of sync with the code, which was
modified by hand for servant-0.9.
With this commit, the implementations of these two functions will again
match the implementation of the corresponding functions in Servant.

Closing since integrated with servant-client-0.11


explicitly configure Travis with supported branches

Based on https://github.com/travis-ci/travis-ci/issues/2498


Upgrade to bokeh 1.0.0

Updated microscopium to rely on bokeh 1.0.0 or higher.
Closes https://github.com/microscopium/microscopium/issues/89 (the weird javascript error is now magically fixed, hooray)
Big thanks to @bryevdv for suggesting the patch here: https://github.com/microscopium/microscopium/issues/89#issuecomment-430070789

Coverage remained the same at 67.893% when pulling 62cea2c80fcf9d00d9ff48b41094b2906bdc5738 on GenevieveBuckley:bokeh-1.0.0 into c6f9c38f4ab836f29a82c89abb43bac26cfe2d21 on microscopium:master.

Brilliant!


[v0.5] Unity Scene - First Map

This is a task tracking of the first map. If all ticks are checked, that means the task is completed.

[x] Design
[x] Implementation
[ ] Feedback

Awaiting feedback


don't count disabled offences if fail-level is autocorrect

Stumbled across this case because we always use fail-level autocorrect and noticed that the wrong error-code is returned for files with disabled offences. This minor change should fix that.
there is probably a better place to put that spec but I have no idea how that >3000 lines spec file is structured (maybe it would be a good idea to somehow split that file for easier navigation)

The fix looks good, but you'll have to rebase. Good remark about the huge spec. //cc @jonas054

:+1:

:+1: I agree about splitting cli_spec.rb. I'd like to see something like
spec/rubocop/cli/options/auto_correct_spec.rb

etc. Don't know in detail where to place each spec example, but if someone would start an effort to achieve a more visible structure, they'd have my support.


ReferenceError: assignment to undeclared variable scope

Hello!
The browser throws an error
ReferenceError: assignment to undeclared variable scope

Corrected as follows:
It was:
processScope: function() {
            scope = this.getOption('scope');

Became
processScope: function() {
           var scope = this.getOption('scope');

Thanks for reporting this, I just fixed it.


Defining new "Tag" within xyzForm.cs to put placeholders within DOM

Hi all,
as sometimes I need to add additional HTML elements within the dialog between two fields, I would like to have the possibility to add a tag like the [Category("")] tag within the form which then would place an invisible
 or something like this at the logical position within the dom which I later (within xyzDialog.ts) can easily find with jquery or even something like this.form.<placeholdername>.GetPlaceholder.html = "htmlContenttoInsert" to manipulate its HTML.
I think of something like that:
[HTMLplaceholder("MyAdditionalHTML1")]

which then would render the following within the DOM at the correct logical position:

How can I do the above?
(Currently, I am using $("htmlContentToInsert").insertAfter(this.form.<FieldBeforeInsert>.getGridField() but this makes no fun if you deside to move the HTML to another logical position within the form);
This within the spirit of Serenity's declarative programming approach.
Thanks for any feedback and
with kind regards,
John

This can help you IMHO:
Static Text Block
https://github.com/volkanceylan/Serene/blob/master/Serene/Serene.Web/Modules/Common/Helpers/StaticTextBlock.ts

Hi @minhhungit ,
thanks for pointing me into the right direction. This is already quite close to what I want - it is just static text (or pulled from the translation) instead of being an invisible marker within the DOM.
But I think that I will be able to work it out from here.
When I have the solution, I make a Wiki article.
With kind regards,
John

Hi all,
I am reopening this issue as - with the solution pointed out by @minhhungit -  I am not able to programmatically set the HTML content at runtime.
Although there is the option that - when we omit the Text parameter within the static text block "tag" - it will take the text from the textfield value, it seems that this is done before the dialog even instantiate as I am not able to make the HTML to appear when I set the value within the constructor or within afterLoadEntity from within the dialog.
I am setting the value like this:
            this.form.BlablaText.element.val("jojosupi");
            this.updateInterface();

(the above html does not visually appear).
But when I enter the Text within the xyzForm.cs like this:
        [StaticTextBlock(HideLabel = true, IsHtml = true, IsLocalText = false, Text jojosupi")]
        public String BlablaText { get; set; }

then the html is displayed fine.
Also I am looking for a solution which does not require fields to be created (also notMapped ones) as this is also not required by the [category] tag.
So after the above - I am looking for a solution where I can "mark" places within the dialog (form part) DOM from within the form and later find these places from within the dialog with jQuery or a better solution like this.form.placeholdername.html = "";
further inputs are anytime welcome :-)
With kind regards,
John

as I know

.element.val

is set value, I think you should use element.html('
demo
')

Hi @minhhungit ,
ah - that totally makes sense :-) I will try it and give feedback here.
Thanks for this hint.
with kind regards,
John

@minhhungit ,
yes -this works perfectly - even without creating the fields within the xyzRow.cs.
Thanks again.
With kind regards,
John


DNTs in MMT v4.2

Hi @davidecaroselli  and @nicolabertoldi,
I have finally managed to train a test engine in MMT 4.2.
I have a question about "Do Not Translate" (DNTs) handling.
Using the same example at https://github.com/modernmt/modernmt/issues/275, MMT doesn't return the text between the tags. I am talking about this sentence.
Gilmour is the leader of the <dnt text="Pink Floyd"
The returned translation now is:
"Gilmour is the leader of"
Here is the returned JSON:
{ status: 200, data: { translation: ""غيلمور هو زعيم", sourceWordCount: 7, targetWordCount: 4, contextVector: { entries: [ ] }, priority: "normal", totalTime: 71, memoryLookupTime: 0, decodingTime: 62, queueTime: 0, queueLength: 0 } }
Is dnt tag still working in MMT 4? Or is it obsolete? By the way, if I remove the tags from the string, MMT returns translation normally, but "Pink Floyd" is now transliterated in my target language.
What is the best way to protect terms in this latest release?
I was trying to translate this sentence:
"Ensuring quality and accuracy has always been our top priority. In fact, the manufacturing processes we employ for Accu-Chek Guide and Accu-Chek Aviva Plus test strips ensure that Accu-Chek products meet our uncompromising standards of quality. Here are a few examples:"
For some reason, 'Accu-Chek Aviva Plus' is returned as:

Accu - hek Aviva Plus

Also, is there a way to stop MMT from adding extra spaces in a hyphenated word such as "Accu-Chek". This usually returns as: Accu - Check
I believe this is caused by the tokenizer.
Thanks,
Mohamed

Escaping the tags as follows:
"Gilmour is the leader of the <dnt text="Pink Floyd""
returned translation of all text, but the text between tags were skipped/omitted.
So, translation returned as:
"Gilmour is the leader of"

This sentence whether it's escaped or not, it doesn't work in MMT 4.

This was not changed. <ph rev="v5r2">This was updated.</ph> This was not.

<p>This was not changed. <ph rev="v5r2">This was updated.</ph> This was not.</p>

Hi @davidecaroselli ,
Haven't heard from you for a while. I hope all is well.
I see that you changed the version to 4.2.1. Is this coming soon? If yes, when?
I need to retrain my engine, but if this new version will be released soon with some fixes, I would rather wait.
Please let me know what you think and sorry for bothering you.
Thanks,
Mohamed

@mzeidhassan
In ModernMT v4.X, all well-formed xml tags ('', '', '') are not considered during the translation process.
ModernMT removes these tags from the source text, translates the text (without tags), and re-inserts them inside the translation in the best position.
ModernMT provides the DNT facility, which works as follows.

the user has to replace the text sequence to keep un-translated with a token of this type ${DNT0};
the user translates the modified text with ModernMT;
the user replaces the ${DNT0} token in the translation with the original text sequence.

Example:

src: "Gilmour is the leader of the Pink Floyd"
dnt-src: "Gilmour is the leader of the ${DNT0}"
dnt-trans: "Gilmour è il leader dei ${DNT0}"
trans: "Gilmour è il leader dei Pink Floyd"

Transformations from src to dnt-src and from dnt-trans to trans are not handled by ModernMT, but they have to be done by the users.
Note that:

there are up to 10 reserved dnt tokens:${DNT0}, ${DNT1}, ..., ${DNT9}
during the translation process ${DNT0} is handled like a standard token (actually it is not bpe-fied); hence, it is possible that the token is not translated at all, and disappears from the output.

In order to improve translation quality, it is suggested to train the system with parallel data including DNT tokens, so that the system learns how to correctly translates these symbols.
The replacement should occur in both the source and target side of (a portion of) the parallel corpus.

Thank you so much @nicolabertoldi for your support. When I test "Gilmour is the leader of the ${DNT0}" in my current engine, everything is translated, but the token disappears as you mentioned above. So, I get:
"Gilmour is the leader of the"
I think this is expected because my training data wasn't trained with these tokens in the first place.
I will insert these tokens in my data and re-train the engine to mitigate this issue then.
Thanks again and it's great to hear back from you again. You have been away for a while :-)


[Feature Request]: Allow users to modify GPT's answer

Is your feature request related to a problem? Please describe.
No
Describe the solution you'd like
I would like to modify GPT's or assistant's answer in the chat history. It would be helpful if GPT made a small typo or mistake, and I can modify it on my own.
For instance, GPT may say
pip install pyautogen

I would know my environment only uses "pip3", so I can edit the chat history to
pip3 install pyautogen

Additional context
No response

Corrections are also a great source of feedback for teachability, or related features.

Corrections are also a great source of feedback for teachability, or related features.

Right. And teachability should handle such cases if the user says something about the error, like "My environment only uses pip3, so please use pip3 instead of pip in your responses." But if the user were able to directly modify the model's response in the chat history, teachability would ignore that of course, without extra wiring.

Good points.
@BeibinLi it seems to me that this is more applicable in the scenario where human input mode is set to always or terminate?
E.g.

Human makes request
Agent responds, and asks human for feedback ...
Human corrects the response (e.g. .. No, use pip3 install instead of pip install)
Agent takes feedback and proceeds

It probably would not work well for human input mode set to never (what is currently supported)

Human makes request
Agents converse to solve .. exchange 30 messages
Human gives feedback
Agents converse again ....

What do you think?

@victordibia  Not necessary. The example I gave earlier is coding, and your explanations are correct.
In another case, let's say the user doesn't need coding, but to brainstorm some research ideas. The GPT gives a good but not perfect idea, and then user can do two things:
Option 1: in the next message, the user proposes a small change to GPT's idea.
Option 2: the user can edit GPT's answer and update the "belief".
The current solution only allows option 1, but could be ineffective (one more API call, and GPT not always understand the correction accurately).
--- Simple example below, where the user could simply modify in the chat history ---
User: let's brainstorm an outline of a paper, based on my findings. {details omitted here}
GPT: Sure. Here is an outline.
1. Introduction
2. Background
3. Related Work
4. Method
5. Evaluation and Results
6. Conclusion

User: I want to move the "related work" section before the conclusion.
GPT: Okay.
1. Introduction
2. Background
3. Method
4. Evaluation and Results
5. Related Work
6. Conclusion

User: Can you change section 2 to "Motivations" and then give me a one-sentence summary for each section.
GPT: [details skipped]

Got it.
In the current AutoGen implementation (script or notebook), how will this be implemented? e,g, would this be implemented with humaninputmode = always?

Not related to human input mode. AutoGen Studio opens a great gateway between users and agents. We may need to rethink and re-invent new tools/functionalities which haven't been done before.  This question is not only relevant to AutoGenStudio, but to the general AutoGen framework.
If I can refactor the design, we may want to add a method edit_oai_messages(chat_id: Int, updated_content: Union[Str, List])  to conversable agent.  But this would be a longer discussion.

Got it. Thanks. I updated the issue tag as this is a more general feature.

FWIW, I directly edit GPT-4s answers all the time in the OpenAI playground. Like, maybe at least once every chat session.

@afourney  Yes, it is the same as the playground.

closing as stale, please reopen if you intend to work on this further.


Installation (HACS) - not a valid repository

Hi,
I just tried your Installation (HACS) procedure. Unfortunately, Home Assistant keeps telling me, that it is not a valid repository:

The Home Assistant Wiki tells me that "Each repository is required to contain repository.yaml at the root in the git repository." However, I can't see a "repository.yaml" in your repository.
What am I missing?

Hey! Thanks for your hint. That worked. My bad. Sorry!


feat(wallet): Consolidating UTXOs every 24 hours

:tada: This PR is included in version 3.3.0 :tada:
Your semantic-release bot :package::rocket:


DOC: Add thorough docstring for scipy.special.binom

Reference issue

Closes gh-15247
What does this implement/fix?

This PR fleshes out the docstring for scipy.special.binom. The current docstring is minimal and does not inform the user about how binom differs from scipy.special.comb.
The updated docstring:

Specifies that binom is for computing the binomial coefficient considered as a function of two real values.
Gives the formula that generalizes the binomial coefficient to real values.
Has a note explaining why binom differs from the usual convention for defining \binom{n}{k} for negative integer n and integer k when \binom{n}{k} is considered as a function of two integer values. This discrepancy was noted in gh-15213.
Has a number of examples illustrating how binom differs from comb.

I'd appreciate input on the updated docstring. Particularly on whether the Notes section is clear, whether any important information is missing, or whether the examples could be improved.

Thanks @tirthasheshpatel! I agree with all of your suggestions. I also added another example for the x < y case.

Everything LGTM now. Thanks, @steppi! I will merge tomorrow if there are no more comments. Also, looks like there are new unrelated failures...


[WIP] OSRA-378 Create Rails routes, controller & views for Orphan

https://osraav.atlassian.net/browse/OSRA-378
—need someone to go over exactly which fields need to go in the edit form, and which model associations, if any, need to be built before an instance of Orphan can be persisted
—do the cucumber specs from aa need to be converted to request specs for hq?

@christophermlne

As a reference for which fields need to be included in the form you can look at the current implementation under ActiveAdmin, which you can find in app/admin/orphan.rb. Look at the form block that starts on L63 - it's a long one. To get an idea about required associations, you can look within the orphan model in app/models/orphan.rb, as well as within the orphan factory in spec/factories/orphan. current_address, original_address, orphan_list, orphan_status & orphan_sponsorship_status associations are required, although the last two will be given default values if you don't specify them explicitly.
We have made the decision not to write features for common CRUD operations under the HQ namespace unless there is felt to be a particular need. Here, I don't think there is any, but please use your judgement. You can have a look at earlier PRs for e.g. Partner CRUD to see the agreed upon testing strategy.
In general, we've tried to keep PR sizes small to facilitate code review. In this instance, each subtask of OSRA-378 would have been submitted as a separate PR. Code review has always been a bottleneck for us, and is doubly so now with a small team, so please think "as small as possible" when it comes to future PRs :)

Hello guys. I need some help to push my PR. I don't know if I made it well. The command I used was: git push origin orphans_controller. origin is my local repository. I can see that this PR has a conflict with merge. I don't know if it's related to something I did. Also I don't see my commit message.

if orphans_controller is a branch on the develop then it should work fine
Also you might want to rename the branch to include the osraTask with git branch --move your_branch renamed_branch

Hmmm, but how I know that? How I know if orphans_controller is a branch of develop. I'm on slack chat.

hey guys, i'm still working on this PR.
i posted a few times in slack about pairing on a blocking issue, but maybe that's not the best place to post about this project.
basically, there are a few magic AA methods the expose collections to the view layer that i need to pair on in order to port over to the new system...

Sorry again for the lack of communication Christopher.
I tried to complete your work. This is my PR: https://github.com/gmunumel/osra/commit/cfee086ad1decb592e169d2db9f5283f69ecdf16
I hope it helps.

excellent. thank you. i am merging now —just have to learn how to the use git cherry-pick ;-)

i've just noticed there are merge conflicts with upstream develop.
i am rebasing.

@gmunumel are we finished this PR? is there anything else we need to do?

I guess yes. :+1:

Can we see print-screens with each view in action?

@gmunumel you can use erb or haml. Bot for consistency it's better to continue with erb for the orphans.

Edit:

Index:

I will wait for @christophermlne to finish these changes. I just posted the Edit and Index views.
@NikitaAvvakumov ,  about the before_action, to use it where or in which context?

Don't forget to change the Orphan link on the main navigation to point to the orphans#index

As part of this PR, please change routes and routing specs to make sure that new orphans cannot be created via the orphans_controller.

Superseded by #353


Escape key not detected the first time

On fetching input, Escape key event doesn't seem to be detected the first time. It takes two key-presses to detect Escape key for the first time after which it is detected at each key press.
Reproducible example -
use libnotcurses_sys::{widgets::*, *};
use env_logger;
use log::info;
use nix::poll::{ poll, PollFd, PollFlags };

fn main() -> NcResult<()> {
    env_logger::init();
    reader()
}

fn reader() -> NcResult<()> {
    let mut nc = unsafe { Nc::new()? };
    let mut stp = unsafe { nc.stdplane() };

    let input_fd = PollFd::new(
        unsafe { c_api::notcurses_inputready_fd(nc as *mut Nc) },
        PollFlags::POLLIN);

    let mut input_details = NcInput::new_empty();
    loop {
        if let Ok(_) = poll(&mut [input_fd], -1) {
            let recorded_input = nc.get_nblock(Some(&mut input_details))?;
            info!("Recieved {:?} {:?}", recorded_input, input_details);
        }
    }

    unsafe { nc.stop()? };
    Ok(())
}

Viewing the logs RUST_LOG=trace cargo run would reveal that each other keyboard input (characters, Tab etc) is detected on one key-press, only Escape key takes two.
The behavior is the same if I don't use polling and simply loop delayed get(). My Escape key appears to be registering perfectly for other applications so I doubt it is the issue.
Terminals - Alacritty, XTerm.
Please confirm this if this behavior is reproducible and is a bug.

I can confirm this is happening for me too with the notcurses-input C app, therefore I recommend opening an issue in the main notcurses repo.

i'll go ahead and make one, thanks!

This is not happening for me in Kitty, fwiw. About to attempt verification with alacritty/XTerm.

confirmed in alacritty/xterm. further work will take place in https://github.com/dankamongmen/notcurses/issues/2646. Thanks for the report!

i've fixed this upstream, to the best of my testing, and will cut 3.0.8 within the next few days including this fix. hack on everybody

https://github.com/dankamongmen/notcurses/pull/2648


Xo upgrade - attempt #2

No lint errors
Unit tests pass
Up-to-date xo module
Remove node 6 add node 12 in Travis CI

🚀 🤞
Branch based off #28
Closes the first attepmt #29

Unsure what happened with the git history 🤷♂
But this may fix the problem: https://github.com/KhaledMohamedP/translate-json-object/pull/29#issuecomment-560351837

Unfortunately, I am seen the error in both PRs!

Very odd -- I'll try to figure something out 🤷♂

Thanks for trying here @whyboris 🙏


Some improvements

Hi,
I've made some improvements in my fork, I don't know if you could accept my pull request, as I've changed the block name after the animation, so that this version won't be backward compatible.
Here are the key improvements:

Added indexWillChangeBlock before the animation, so that we can update the underlying view before setting the mask's frame.
Added indexDidChangeBlock after the animation.
Make thecurrentIndex public, so that we can jump to a specific index directly.
Added statusBarHidden, so that we can hide the status bar.
Make KSGuideItem's properties public so that we can update them before the animation, for example, a table view cell that hasn't been created when we create the KSGuideItem.
Fixed a bug of mask's x position.

Thanks for your great job 👍


MP4 & HLS recording corruption after streamer disconnects

I have  issue with LiveKit Cloud.
When a streamer disconnects due to network issue (or other short interrupt) egress still works, but MP4 recording on S3 becomes corrupted after stream is stoped. There is no seamless transition in video and audio, video just stalls.
UPDATE:
it happened only when using TrackCompositeEgressRequest,
RoomCompositeEgressRequest works correctly
&livekit.TrackCompositeEgressRequest{
	RoomName:     "test-room3",
	AudioTrackId: "TR_AMUeuCNthEmzn7",
	VideoTrackId: "TR_VCVRmHNrygNWWK",
	Output: &livekit.TrackCompositeEgressRequest_File{
		File: &livekit.EncodedFileOutput{
			FileType:        livekit.EncodedFileType_MP4,
			Filepath:        "/test",
			DisableManifest: false,
			Output: &livekit.EncodedFileOutput_S3{
				S3: &livekit.S3Upload{
					AccessKey: "xxx",
					Secret:    "xxx",
					Region:    "ams3",
					Endpoint:  "ams3.digitaloceanspaces.com",
					Bucket:    "xxx",
				},
			},
		},
	},

closing as duplicate of https://github.com/livekit/egress/issues/210. Will provide all updates there


Topic smart selector -> quick and recent are not populating

Believe this is fixed.

Believe this is fixed.


[Feature Request] Generated navigation

I would love to have the ability to generate navigation from a directory of markdown files.
We have a repo of markdown files I would like to pull in as a submodule, have a rebuild triggered, and navigation generated in the template. Regarding sorting, navigation could be spit out alphabetically, or a standardized filename syntax could be established to number the files with prefix to specify the order.
// This spits out an alphabetized array
my_awesome_file.md
a_silly_file.md
stupid_post.md

// This spits out a numbered array
01_my_awesome_file.md
02_a_silly_file.md
03_stupid_post.md

Paired with #24, this could be really powerful, displacing the default source directly to wherever you wish and spitting out structured nav.

Isn't using directories kind of inflexible? A menu item could live in whatever directory, when perhaps using something like Page Meta / Front Matter, with say:
---
# yada yada yada
nav: main
# yada yada yada
---

And then outputting in a .blade.php file:

@foreach(nav('main') as $navItem)

url }}">{{ $navItem->name }}

@endforeach

Or something along those lines...

Take a look at collections in v1.0.0, which makes this possible. There's also example code for a generated nav in the demo repo.


What's the code for "how to utilize conversation demonstrations synthesized via ChatGPT"?

It seems that there is not code for " utilize conversation demonstrations synthesized via ChatGPT  to finetune the LLaMA model "
in the code , I see that your use  HealthCareMagic-200k.json  ,not  the "5k generated conversations between patients and physicians from ChatGPT [GenMedGPT-5k]" ,
how to utilize conversation demonstrations synthesized via ChatGPT ?  Can you show us the code for this ?

We will release it soon.

We will release it soon.

Awesome !  We do look forward to seeing it  open source  soon, thanks for your  great work !


Read all folders?

INBOX/Junk%20E-mail
I have like this folder in Mozilla thunderbird and it's Junk/Spam.
Beside this I also have Archive, Deleted Items and Sent folders.
$folders = $client->getFolders();
This returns only INBOX folder and nothing else.
Can I get all folders and read emails inside each folder?

+1
Same problem with "INBOX/Newsletters" folder
getFolders() method does not return this folder.

Any updates?

Hi @consigliere23 @grisha2217 ,
you are probably looking to fetch a folder not hierarchical:
$folders = $client->getFolders(false);

Best regards,

Hi @consigliere23 @grisha2217 , you are probably looking to fetch a folder not hierarchical:
$folders = $client->getFolders(false);

Best regards,

No, same problems with getFolders(false) and getFolders(true)


pico_enable_stdio_usb

Using pico_sdk for cpp, there is feature to enable USB stdio in the MakeLists.txt
pico_enable_stdio_usb(${NAME} 1)
It allows use picotool not in BOOTSEL mode
Is rp-hal has similar feature?

Hi,
If my understanding is correct, this CMake function enables the use of a usb CDC as the std in/out (printf).
When this is enabled, a side effect by default is also to add a class vendor specific interface (implemented here) that allows a program on the host to trigger a reset to usb boot.
Picotools searches for this interface and ignores the others, so you don't need the first ones to be CDC (it could even be the only usb interface present on the device).
I don't think such a feature belongs to the rp-hal per se (as it would require to setup the usb peripheral and to poll the usb stack in the application anyways) but it could definitely be implemented as a separate crate implementing the vendor specific UsbClass to behave like the sdk version does. I don't know if anyone ever tried to do that.

I was looking for the same feature and I have seen that @ithinuel has worked on it: https://github.com/ithinuel/usbd-picotool-reset
Tested with picotool info -f -a, the device effectively reboots into BOOTSEL mode but might take too much time because picotool does not detect it soon enough:
$ picotool info -f -a
Tracking device serial number TEST for reboot
The device was asked to reboot into BOOTSEL mode so the command can be executed.

Despite the reboot attempt, no accessible RP2040/RP2350 devices in BOOTSEL mode were found. It is possible the device is not responding, and will have to be
manually entered into BOOTSEL mode.

But the device is indeed in BOOTSEL mode:
$ picotool info -a
Program Information
 none

Fixed Pin Information
 none

Build Information
 none

Device Information
 type:        RP2040
 revision:    B1
 flash size:  2048K
 flash id:    0xE66118604B286D24

And we can reboot it back to application code with:
picotool boot -a


Header image isn't loading

Not sure when this happened, but the header image isn't loading anymore.  I noticed it when I rendered locally today, but it's also missing from the live site.  Haven't looked into it at all yet, probably an easy fix.

Looking into this now.

See #78
Looks like the image file was just deleted when the GHA rendered. Was not deleted on re-render, so not sure why this happened. Could just be a one time thing. Let's keep an eye on it..


Populate readme

I've added a few details to the README.md to briefly document the file dependencies and how to run the pipeline on Denali.

Great - thanks for adding that info, Jesse!


I have created Candidate Page for different Job Profile

Related Issue
There was no Candidate Page. So I have created one, here it is.
Description
Different candidates for different position. I have added Name, image and description of the candidate.
Type of PR

[ ] Bug fix
[ ] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)
[Attach any relevant screenshots or videos demonstrating the changes]
Checklist:

[x] I have performed a self-review of my code
[x] I have read and followed the Contribution Guidelines.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[x] I have commented my code, particularly in hard-to-understand areas.

Additional context:
[Include any additional information or context that might be helpful for reviewers.]

The Candidate Page is created.

Please Do check this page!


Issue with using std::io along with rusqlite

I'm playing around this crate and I found a error when I try to compile with this line of code:
let name_file = std::fs::File::open("names.txt")?;
The error message says that the following:
the trait std::convert::Fromstd::io::Error is not implemented for rusqlite::Error
I fixed this bug on my repo by implementing this trait "From" following the same structure as the Utf8Error/NulError in error.rs. I'm pretty new to Rust so I don't believe the way how I fixed it is the right way in doing so moving foward.
Please let me know if this is an issue with the crate or just me screwing up.

Do you have a complete minimal example ?
I guess that you are opening a file in a callback that should return a rusqlite::Result.
If yes, you can wrap your IO error in an UserFunctionError but it may be not idiomatic.
Regards.

Here's the code that I'm trying to compile.
fn get_connection(path: &Path) -> rusqlite::Result<Connection> {
      if (*path).exists() {
          Connection::open(path)
      }
      else {
          let conn = Connection::open_in_memory()?;

          conn.execute("CREATE TABLE person(
                        ID            INTEGER  PRIMARY KEY,
                        Name          TEXT NOT NULL,
                        Balance       REAL NOT NULL DEFAULT 0,
                        TimeCreated   TEXT NOT NULL
                       );", &[]).unwrap();

          let name_file = File::open("names.txt")?;
          let reader = BufReader::new(&name_file);
          for line in reader.lines(){
              let name = line.unwrap();
              conn.execute("INSERT INTO person(name, TimeCreated)
                            VALUES(?, ?);",
                           &[&name, &time::get_time()])?;
          }

         conn.backup(DatabaseName::Main, path, None)?;
          Ok(conn)
      }
 }

Here's the error message the rust compiler is giving me:

Ah, yes, this is a little annoying, but I think not something we should change in rusqlite. rusqlite::Error covers all the different kinds of errors you might get from rusqlite itself, but File::open can fail with a std::io::Error, which covers all different kinds of I/O errors. You probably want get_connection to return an error type of your own that covers both of these types (and potentially other errors that your program/library could have). By hand, this might look like:
enum Error {
    RusqliteError(rusqlite::Error),
    IoError(std::io::Error),
    // ... others?
}

with definitions of the various supporting traits to make that seamless (at least From<_> for both the inner types, Display, and std::error::Error). I believe the error-chain crate can also automate/simplify a lot of this.

Ah, I see my mistake. I'll try implementing my own Error type and see how that goes. Thanks for the suggestion!


Building on Archlinux (it works)

Hello,
You should add to the documentation that portaudio is needed. Here is what I need to do to satisfy dependencies on an Archlinux machine :
# pacman -Sy glfw portaudio

Also, it would be better to skip to next level directly instead of having to input it manually ;)
Keep making open source games !
Cheers,
~Nico

Thank you for your suggestion. Portaudio is now mentioned in the README.


Fixes #1116 - Add cancel_after_time_interval

Description
Describe what this change achieves.
Add cancel_after_time_interval field to SearchRequest and MsearchRequest requests in the Java client.
Issues Resolved

Fixes #1116

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Hello @dblock what do you think of this PR ?

@dblock & @Xtansia , I see the field defined in:
https://github.com/urinud/opensearch-api-specification/blob/7dee041660986c5eb4bc85c2d19af006730746e9/spec/namespaces/_core.yaml#L5376
I also see a test created for this field too. LGTM.


AKS Engine VHD Updates

Ubuntu 20.04-LTS VHD shipped here: https://github.com/Azure/aks-engine/pull/5041

Nov release: https://github.com/Azure/aks-engine/releases/tag/v0.73.0
Dec release: https://github.com/Azure/aks-engine/releases/tag/v0.74.0
Jan release: https://github.com/Azure/aks-engine/releases/tag/v0.75.0
Feb release: https://github.com/Azure/aks-engine/releases/tag/v0.76.0
March release: TODO
April release: TODO
I believe we can close this issue after we fulfill the April release commitment. Does that sound right @bridgetkromhout @dtzar @CecileRobertMichon ?

Complete now with April release: https://github.com/Azure/aks-engine/releases/tag/v0.78.0


insufficient gist:recognizes comment

The comment on gist:recognizes is the same as its label.  Insufficient textual definition.  Noticed in gist9.4.0.

This is subsumed by #534 but I'm leaving it open so we make sure to consider the specific issues addressed here.

SUGGESTION: close, no longer relevant.  What we now have is:
gist:isRecognizedBy
	a owl:ObjectProperty ;
	rdfs:range [
		a owl:Class ;
		owl:unionOf (
			gist:Organization
			gist:Person
		) ;
	] ;
	skos:definition "The entity that formally acknowledges the existence of, as the State recognizes the existence of a particular company"^^xsd:string ;
	skos:prefLabel "is recognized by"^^xsd:string ;
	.

Closing - no longer relevant.


Allow returned responses to be optional in actions configuration

I'm working on creating a universal media player entity, which requires me to run a SSH command on a Linux host to set the volume since MPD isn't handling that properly. However, when doing this through an execute_command action from this integration, I get an error:

This appears to be coming from HA here because the actions/services provided here are set to require responses to be returned. It seems like it would be possible to register actions as SupportsResponse.OPTIONAL, and expose a new parameter in the action data for execute_command for whether or not a response should be returned (default true for backwards compatibility).
I started looking into implementing this in a PR here but seems like it might also require updates to ssh-terminal-manager

Thanks for the feedback! I will try to make the response optional when I have time!

I made all service responses optional in version 1.0.1, please take a look if everything works!

It works perfectly! Thank you for working on this


/map on https does not run

All /map pages does not run on https:// - because Google Maps API is loaded via http:// protocol.
See:
http://epsg.io/4326/map
Problem is with:
 <script src="http://maps.googleapis.com/maps/api/js?sensor=false&libraries=places"></script>

replace by https:// and test

Forgotten https:// request on Google Static Maps request. Now done in 249d5c81d398b187bd4af4a7eb67b14c2d610878

And in e5e0a3ba07d4c2bf8ec29e115518608b8936f561

Map has submitted the AJAX calls to non-https endpoint. Fast fix: 1109185e6c64f93008848f04263d7d60933bbde9

All over the web there is a lot of links to http:// to epsg.io itself - these should be probably all to //

or even better relative links starting with /

Everything seems to be OK on the map page with https. Thanks @TomPohys


Update to identify the VM sizes supported by specific compute VM types

The intension is to make this note easy to understand by customers

@vijetajo
Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@vijetajo, @sdgilley
Can you review this old PR and determine whether it needs to be closed or merged?
@MicrosoftDocs/public-repo-pr-review-team

merging requested / approved by @Blackmist via email


False positive with Azure.Deployment.Name

Description of the issue
When expanding deployments PSRule for Azure uses the file name as the name of the deployment (for display in output) as the name of the file.
Often file paths are long and do not meet the naming requirements of Azure.Deployment.Name so the deploy fails.
Expected behaviour
Using the name of the file is helpful for display, however the rule should not flag these as an issue. We need to ignore the root deployment (which has the file name), an may need to add some attribute on to support this identification.
Module in use and version:

Module: PSRule.Rules.Azure
Version: 1.26.0-B0011

Yep, getting this issue now.

I've started looking at the issue, and should hope to close this off before the next pre-release so we can confirm there isn't anything hanging around with this one.


How to implement embedding layer?

Hi, I'm going to implement neural collaborative filtering using gorgonia. However, when I read CharRNN in examples to learn how to implement embedding layers, I find its implementation is quite complex. It seems inefficient since the graph will be modified over time. Will there be "Embedding Layer" in the future? :smile:

An embedding layer is essentially a matrix of shape (n, dim).  So, if you have 1000 "words" (could be any discrete option - users, etc etc), and you want an embedding dimension of 50, you simply create a *Node with shape (1000, 50):
g := NewGraph()
emb := G.NewMatrix(g, G.Float64, G.WithShape(1000, 50), G.WithInit(GlorotN(1)), G.WithName("embedding"))

Now, each row in emb represents one "word". This of course, implies that you need to have your words stored with its associated ID. A simple map usually does the trick:
m := map[string]int{ "-UNK-": 0, "hello": 1, "world": 2 }

To use the emb, simply select a row. This is done with Slice.
Say you want to get the embedding for the word "hello":
hello, err := G.Slice(emb, G.Slice(G.S(m["hello"])))

Here we see hello is a *Node that holds the slice. G.S is the function to create the slicing range (in this case, it is equivalent to emb[1:2] ).
But how do you do this dynamically?
Well, UnsafeLet has special checks for these sorts of thing. But for now, let's write a sample program:
g := NewGraph()
m := map[string]int{ "-UNK-": 0, "hello": 1, "world": 2 }
emb := G.NewMatrix(g, G.Float64, G.WithShape(len(m), 50), G.WithInit(GlorotN(1)), G.WithName("embedding"))
x, err := G.Slice(emb, G.Slice(G.S(m["-UNK-"])))
// check err
// use `x`

vm := G.NewTapeMachine(g)
for _, id := range m {
    G.UnsafeLet(x, G.S(id))
    vm.RunAll()
    // do things
    vm.Reset()
}

Now, there are specialized usefulness from libraries like lingo/corpus, which would be used instead of a plain map. But in general, that's all you need.

The main reason for not adding an embedding layer in Golgi was because while a map is useful for a large number of cases, there are times you might want a more specialized data structure (such as a *corpus.Corpus)

If I were to implement it as a golgi.Layer, I'd do something like this:
type Embedding struct {
    lookup map[string]int
    W *gorgonia.Node
}

func (e *Embedding) UnsafeLet(id int) { gorgonia.UnsafeLet(e.W, gorgonia.S(id))}

Thank you so much :smile:. I'll try it.


RFE: Clarify duplicate provider error to name the field that has been duplicated

User Story
When a user attempts to add a provider with an ARN that already exists, a generic error message is displayed "Danger: This field must be unique." and the offending input field has a thin red line. It would be helpful if the error message specifically names which field is in error and the input field should have a better visual that it is in error.
Impacts
UI
Role
User
Assumptions
User is trying to add a provider that already exists
Acceptance Criteria
An error message names any fields that are in error and the offending input element clearly shows that it is in error with a error icon on the field or full error outline

@boaz I feel this will be addressed with your new onboarding flow work?

@dlabrecq that's correct.


OSError: CUDA_HOME environment variable is not set. Please set it to your CUDA install root.

When I installed Autostill-grounded-sam2, I am facing on the following questions:
1) It seems that the installation of SAM2 will be implemented every time when I run the python script.
fatal: destination path 'segment-anything-2' already exists and is not an empty directory.
Obtaining file:///C:/Users/ibo22/.cache/autodistill/segment_anything_2/segment-anything-2
Installing build dependencies ... done
Checking if build backend supports build_editable ... done
Getting requirements to build editable ... error
error: subprocess-exited-with-error
2) Cannot locate the CUDA.
× Getting requirements to build editable did not run successfully.
│ exit code: 1
╰─> [32 lines of output]
C:\Users\ibo22\AppData\Local\Temp\pip-build-env-yqwkereb\overlay\Lib\site-packages\torch_subclasses\functional_tensor.py:258: UserWarning: Failed to initialize NumPy: No module named 'numpy' (Triggered internally at C:\actions-runner_work\pytorch\pytorch\builder\windows\pytorch\torch\csrc\utils\tensor_numpy.cpp:84.)
cpu = _conversion_method_template(device=torch.device("cpu"))
Traceback (most recent call last):
File "C:\Users\ibo22.conda\envs\gsam2\Lib\site-packages\pip_vendor\pyproject_hooks_in_process_in_process.py", line 353, in
main()
File "C:\Users\ibo22.conda\envs\gsam2\Lib\site-packages\pip_vendor\pyproject_hooks_in_process_in_process.py", line 335, in main
json_out['return_val'] = hook(**hook_input['kwargs'])
^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\ibo22.conda\envs\gsam2\Lib\site-packages\pip_vendor\pyproject_hooks_in_process_in_process.py", line 132, in get_requires_for_build_editable
return hook(config_settings)
^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\ibo22\AppData\Local\Temp\pip-build-env-yqwkereb\overlay\Lib\site-packages\setuptools\build_meta.py", line 458, in get_requires_for_build_editable
return self.get_requires_for_build_wheel(config_settings)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\ibo22\AppData\Local\Temp\pip-build-env-yqwkereb\overlay\Lib\site-packages\setuptools\build_meta.py", line 327, in get_requires_for_build_wheel
return self._get_build_requires(config_settings, requirements=[])
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\ibo22\AppData\Local\Temp\pip-build-env-yqwkereb\overlay\Lib\site-packages\setuptools\build_meta.py", line 297, in _get_build_requires
self.run_setup()
File "C:\Users\ibo22\AppData\Local\Temp\pip-build-env-yqwkereb\overlay\Lib\site-packages\setuptools\build_meta.py", line 313, in run_setup
exec(code, locals())
File "", line 70, in
File "", line 51, in get_extensions
File "C:\Users\ibo22\AppData\Local\Temp\pip-build-env-yqwkereb\overlay\Lib\site-packages\torch\utils\cpp_extension.py", line 1076, in CUDAExtension
library_dirs += library_paths(cuda=True)
^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\ibo22\AppData\Local\Temp\pip-build-env-yqwkereb\overlay\Lib\site-packages\torch\utils\cpp_extension.py", line 1214, in library_paths
paths.append(_join_cuda_home(lib_dir))
^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\ibo22\AppData\Local\Temp\pip-build-env-yqwkereb\overlay\Lib\site-packages\torch\utils\cpp_extension.py", line 2416, in _join_cuda_home
raise OSError('CUDA_HOME environment variable is not set. '
OSError: CUDA_HOME environment variable is not set. Please set it to your CUDA install root.
[end of output]
note: This error originates from a subprocess, and is likely not a problem with pip.
error: subprocess-exited-with-error

Hello there! Are you running this on a device with an NVIDIA GPU set up in your environment?

Yes, my desktop has a GPU and I have already installed CUDA and pytorch. Also, I set the path in Windows system but facing this problem whatever I did.

From: James @.>
Sent: Wednesday, August 7, 2024 4:17 AM
To: autodistill/autodistill-grounded-sam-2 @.>
Cc: Yongsheng Bai @.>; Author @.>
Subject: Re: [autodistill/autodistill-grounded-sam-2] OSError: CUDA_HOME environment variable is not set. Please set it to your CUDA install root. (Issue #2)
Hello there! Are you running this on a device with an NVIDIA GPU set up in your environment? — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you authored the thread. Message ID: <autodistill/autodistill-grounded-sam-2/issues/2/2273009121@ github. com>
Hello there! Are you running this on a device with an NVIDIA GPU set up in your environment?
—
Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https://github.com/autodistill/autodistill-grounded-sam-2/issues/2*issuecomment-2273009121__;Iw!!KGKeukY!3fcf_WW0q7RwgP7T1-W8fICcukHVdxhLVP4INmeZT6UpR3Qd6pB-ZLnwRvxQbqXJNM8m9jrOpFyFfNpCVeqFZDupHg$, or unsubscribehttps://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/AKQHMIVNHQSUIHZWZJG5CILZQHQ2FAVCNFSM6AAAAABMCSMCPKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENZTGAYDSMJSGE__;!!KGKeukY!3fcf_WW0q7RwgP7T1-W8fICcukHVdxhLVP4INmeZT6UpR3Qd6pB-ZLnwRvxQbqXJNM8m9jrOpFyFfNpCVepZlkG46A$.
You are receiving this because you authored the thread.Message ID: @.***>


Add Phase banner component tests

This PR:

Adds tests to file upload component to ensure the markup is rendered correctly when providing arguments to the macro;
Updates CHANGELOG.MD.

Trello card: https://trello.com/c/f44OMSDX/662-automated-tests-for-phase-banner-component

yeah, fixed


How do i draw a new starting point or branch out multiple branches ?

new starting point is not possible...
multiple beanches with the parallel componente

thanks.


[WIP] Use updated Sphinx extension

Tracks nipy/nipype#1964.
Fixes #463.

New workflow graphs:
basic_wf

t1w_preprocessing

bold_preprocessing

Much better.

Oh yeah!


Bug: Index compilation for a source file in a objc_library with implementation_deps fails

Description
At Snap, we added a change (https://github.com/bazelbuild/bazel/pull/17962) in Bazel fork to support implementation_deps for objc_library rule.
We have found that rules_xcodeproj generated Xcode fails to compile a source file that imports a header defined in its implementation_deps.
Say, ATest is a unit test target and depends on target A. And the target A is an implementation deps of ATest. Building ATest works fine, however, opening TestA.m in Xcode editor tap triggers Xcode to index-compile the source file, which fails because path to A.h is not passed as a copt.
Reproduction steps

Download repro project from https://drive.google.com/file/d/1iixYoxc6Tek8q-VlIqF6udS8_mexCbqs/view?usp=share_link
Extract it and go into dir
run ./bazel run //lib:A.xcodeproj  (bazel binary bundled together has change https://github.com/bazelbuild/bazel/pull/17962)
run open lib/A.xcodeproj
open TestA.m and check in Xcode's Index log that the compilation failed due to missing "A.h"

Expected behavior
When opening TestA.m, Xcode should be able to find "A.h" and compile TestA.m successfully.
rules_xcodeproj version
1.3.2
Xcode version
14.1
Bazel version
upstream/main (b24930f6f90934a0665c7454e12cc6c4e4ecc7b3) + https://github.com/bazelbuild/bazel/pull/17962
rules_apple version
No response
rules_swift version
No response
Additional information
No response

Thanks for the repro. I'll look into this soon!

the bazel binary I bundled together is arm64 executable, so let me know if you need intel cpu version.

Apple Silicon is perfect, thanks.


BelongToMany attach() doesn't create timestamps

From @yogeshwaranManoharan on March 18, 2017 11:30
I tried attaching user with product table but it doesn't seem to create timestamp
yield user.products().attach([id]);
I've updated the model with ".withTimestamps()" but it's not working
Copied from original issue: adonisjs/adonis-framework#517

From future please report the issue to the right now. Moving for now.

Fixed in 4.0

Hey, I tried it in 4.1.0 version and still not working


Edit /etc/hosts in situ in hosts addon

Docker does not allow /etc/hosts to be moved, so we devise a
way to edit the content of the file in situ.
This fixes travis-ci/travis-ci#2969.
It's been tested on staging: https://staging.travis-ci.org/BanzaiMan/travis_staging_test/builds/428811#L95

And this does not break on standard builds: https://staging.travis-ci.org/BanzaiMan/travis_staging_test/builds/428813#L93-L95

:+1:
@BanzaiMan can you tell us plz when it will be deployed on travis pro?

@shouze I reckon it'll be Monday next week, shortly after this PR is merged.

@shouze Deployed to .org and .com production now.


Base arch structure

Create a project example using this arch https://github.com/PolyCortex/polydodo/wiki/Mobile-Architecture
Add some important dependencies

Tested on android emulator and doesn't crash.
LGTM

I was reading through the bloc doc, and I was wondering if we were going to use the bloc tests ? If so, we would need to add these to pubspec.yaml.
test: ^1.3.0
bloc_test: ^7.0.0

I was reading through the bloc doc, and I was wondering if we were going to use the bloc tests ? If so, we would need to add these to pubspec.yaml.
test: ^1.3.0
bloc_test: ^7.0.0

Will we really test our blocs/cubits? I tought a little bit about it and I told myself that the most important thing to test is our domain layer since it is there that the app business logic will be concentrated


Move Studio code from migrate package to cli package

Closes #3211

It looks like you removed the current Studio test, is it because you plan to add new tests in another PR?

Because right now we don't know if that works (in the CI test).
Everything else looks good! ✅

It looks like you removed the current Studio test, is it because you plan to add new tests in another PR?

Yep! Another PR after this gets merged.

Because right now we don't know if that works (in the CI test).

Ah yeah we're back to 2 sprints ago, but that'll change soon

I've tested this by running bin.ts via ts-node from this directory: prisma/src/packages/cli/src/__tests__/fixtures/example-project, and also by pnpm run building it and running index.js via node from the same directory. That should be enough, right?
If not, it's still early in the sprint, I can fix things as they pop up

Yes that's enough 😃 We can always fix after anyway.


fix: [#4288] CustomQuestionAnswering answers return empty metadata

Fixes # 4288
#minor
Description
This PR fixes the formatting made over the answer's metadata, causing the information to get lost. The metadata field was treated as an array and it is actually an object.
Specific Changes

Updated the formatQnaResult method of the languageServiceUtils class to format the metadata field treating it as an object.
Added a new unit test in the languageService.test class to verify the metadata file gets properly mapped.
Added returns_answer_with_metadata.json test data file.

Testing
These images show the answer the bot received before and after the changes.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.004%) to 84.537%

Files with Coverage Reduction
New Missed Lines
%

libraries/botbuilder-lg/src/templateExtensions.ts
1
84.48%

Totals

Change from base Build<PHONE_NUMBER>:
0.004%

Covered Lines:
19870

Relevant Lines:
22265

💛  - Coveralls

/promoted 4311


ci: predictable build between PRs and main branch

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Refactoring
[x] Build or CI related changes
[ ] Documentation content changes

@waterplea hello, please review me)


Add gl-mat3 to package.json for latest version of geojs

Fixes #140
@dementiev Thanks for reporting.  There was a compatibility problem with the latest release of geojs.  Could you try rebuilding the docker image from this branch to see if it fixes your problem?

@jbeezley it fixed the problem, thanks!


Remove exports, macro, and neverlink from binary and test

Exports and neverlink are not present in java_binary/java_test either
This is a easy way to remove them; a more thoughtful approach may be
refactoring the entire javainfo phase to be library specific, since
only library should provide JavaInfo. In the meantime, this change at
least prevents users from improperly using library-only attributes.

LGTM once CI passes


Modified InMemoryUserDataAccessObject and FileUserDataAccessObject files to implement CreateGroupChannelDataAccessInterface. Added some test files.

has good javadocs and code is organized


tests could not be compiled

     Running `/usr/bin/rustc --crate-name wait_group tests/wait_group.rs --color never --emit=dep-info,link -C opt-level=3 --test --cfg 'feature="crossbeam-epoch"' --cfg 'feature="crossbeam-utils"' --cfg 'feature="default"' --cfg 'feature="std"' -C metadata=a3f72b9d23f9393e -C extra-filename=-a3f72b9d23f9393e --out-dir /builddir/build/BUILD/crossbeam-0.5.0/target/release/deps -L dependency=/builddir/build/BUILD/crossbeam-0.5.0/target/release/deps --extern cfg_if=/builddir/build/BUILD/crossbeam-0.5.0/target/release/deps/libcfg_if-192753dee1fcd949.rlib --extern crossbeam_channel=/builddir/build/BUILD/crossbeam-0.5.0/target/release/deps/libcrossbeam_channel-d7b123a4b93ae5a5.rlib --extern crossbeam_deque=/builddir/build/BUILD/crossbeam-0.5.0/target/release/deps/libcrossbeam_deque-68a0ec6191ad983b.rlib --extern crossbeam_epoch=/builddir/build/BUILD/crossbeam-0.5.0/target/release/deps/libcrossbeam_epoch-1b6f02070a8a6ed3.rlib --extern crossbeam=/builddir/build/BUILD/crossbeam-0.5.0/target/release/deps/libcrossbeam-7d3c069cb84f5aec.rlib --extern crossbeam_utils=/builddir/build/BUILD/crossbeam-0.5.0/target/release/deps/libcrossbeam_utils-801a0103b9bbfaea.rlib --extern lazy_static=/builddir/build/BUILD/crossbeam-0.5.0/target/release/deps/liblazy_static-768efe54d2a0c30e.rlib --extern num_cpus=/builddir/build/BUILD/crossbeam-0.5.0/target/release/deps/libnum_cpus-fbfa1b1da75632e7.rlib --extern parking_lot=/builddir/build/BUILD/crossbeam-0.5.0/target/release/deps/libparking_lot-e0d3929b75ed299a.rlib --extern rand=/builddir/build/BUILD/crossbeam-0.5.0/target/release/deps/librand-7f07d512655d4748.rlib -Copt-level=3 -Cdebuginfo=2 -Clink-arg=-Wl,-z,relro,-z,now`
error[E0597]: `i` does not live long enough
   --> src/ms_queue.rs:472:38
    |
472 |                 scope.spawn(|_| recv(i, qr));
    |                             ---      ^ borrowed value does not live long enough
    |                             |
    |                             capture occurs here
473 |             }
    |             - borrowed value only lives until here
...
480 |         }).unwrap();
    |          - borrowed value needs to live until here
error[E0597]: `i` does not live long enough
   --> src/seg_queue.rs:259:38
    |
259 |                 scope.spawn(|_| recv(i, &q));
    |                             ---      ^ borrowed value does not live long enough
    |                             |
    |                             capture occurs here
260 |             }
    |             - borrowed value only lives until here
...
267 |         }).unwrap();
    |          - borrowed value needs to live until here

FYI, compiler is 1.31.0

This doesn't seem to be the version of crossbeam on the master branch. Which version is this?

That is crossbeam v0.5.0 from crates.io

I see, thanks for the report! Seems like we should publish a new version then.

Published a new version.


PHP rankingResponse

https://docs.microsoft.com/en-us/azure/cognitive-services/bing-web-search/quickstarts/php
I'm working with Bing Web Search PHP API.
How should we convert rankingResponse into an array with results? With the actual rankingResponse we have to execute a lot of object to array useless conversions and loops to get results displayed with the good ranking.
Thanks!
Tomy

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

ID: c22a1768-c9c1-f9d0-6f8d-2474e79fed9b
Version Independent ID: 5c18f462-c194-6bbd-a102-7d87cc3b7996
Content: Quickstart: Perform a search with PHP - Bing Web Search API - Azure Cognitive Services
Content Source: articles/cognitive-services/Bing-Web-Search/quickstarts/php.md
Service: cognitive-services
Sub-service: bing-web-search
GitHub Login: @aahill
Microsoft Alias: aahi

@tomygrenier, Thanks for the question! We are taking a look into this and will get back to you soon.

@tomygrenier Use ranking to display Bing Web Search API results. Please refer the following link  to how-to guides for more information.

@tomygrenier We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.

@ram-msft I do follow instructions. But in PHP API guides (while the javascript guide does) there is no example of how to easy script to get results sorted as rankingResponse suggests, except some loops-comparaisons that slow the rendering.
In the PHP results, there is a lot of array to object to array level. May be if the IDs were first level array keys rather than url-ID value, scripting would be more efficient as much loop-comparaison scripting.

Unfortunately I haven't been able to find an alternative for PHP. But I'll pass this feedback on to the product team. #please-close


Concierge - Warning message is showing for the phone input without clicking Call

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Launch the app
Log in with any account
Tap on Concierge conversation
Tap on Call Icon
Tap on First name input field
Tap on Last name input field
Phone number input field
Tap on extension input field

Expected Result:
Warning message is showing for the phone input after clicking Call for Concierge
Actual Result:
Warning message is showing for the phone input without clicking Call for Concierge
Workaround:
Unknown
Platform:

Where is this issue occurring?

Web
iOS
Android
Desktop App
Mobile Web

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: Yes
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation

https://user-images.githubusercontent.com/93399543/156419363-322092bc-1a17-4245-88ba-a300905f8fed.mp4
Expensify/Expensify Issue URL:
Issue reported by: Applause
Slack conversation:
View all open jobs on GitHub

Triggered auto assignment to @neil-marcellini (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

That is the expected behavior. We validate the input when the field loses focus (clicking out of it). A blank phone number is not a valid input and we want to let the user know that it needs to be fixed as soon as possible. If you close the page the error will be gone when they open it again.


[Feature Request] Timestamp Authority with or without LTV

I think it would be amazing to be able to add timestamp authority signatures / certs.
This is obviously jut a cool feature request. Glad your lib exist.

@Hopding This might just be more meta data. If you can direct me to the PDF standards for this (I have no idea where to look), I might be able to rustle up a PR for it. I don't need this feature, but it would be a nice to have for a few projects that I am currently storing this separately.

Just leaving this here for future work: https://github.com/vbuch/node-signpdf
I would like to build a pr for this when i have time.

I have a copy of the pdf standards document relevant to this; if anyone is serious about working on it I'm willing to share or even donate it. it's not public domain, you have to buy it.

@taxilian I am willing to start working on this. Where can I buy it?

I think this is introduces the concepts. I don't know about any other standard document: https://www.adobe.com/content/dam/acom/en/devnet/pdf/pdfs/PDF32000_2008.pdf

@vicpara that is like saying the answer is on duckduckgo. lol

Apologies but that was not the intention. That is the only official documentation I found available on their website. Adobe does claim they have more but they encourage paying to use their SDK.

@vicpara No stress, it wasn't an accusation. It is just a big document. Hopefully I have time soon to work on this.

This isn't an issue, it is a feature-request.

Related to https://github.com/Hopding/pdf-lib/issues/433#issuecomment-925347404

Added this to the roadmap for tracking: https://github.com/Hopding/pdf-lib/discussions/998.


[Toolkit bug]: remix::dxvk_CreateD3D9' has failed. Error code 1

Contact Details (Optional)
<EMAIL_ADDRESS>Describe the bug
RTX Remix would not start and give the following error.
RT Remix Renderer failed to Initialize

How do you reproduce the bug?
This Error occurs if there a pre-installed OpenCL, OpenGL or Vulkan wrapper.
What is the expected behavior?
The Toolkit should atleast start and shouldn't just exit after that error prompt. I've realised changing some parameters in the "lightspeed.app.trex.app.settings.toml" I was able to at least start the Toolkit. In the viewport I was able to use Pixar's renderer and Remix renderer didn't work, obviously.
For a lot people it wasn't clear what could be the root cause and I've seen for some it was multi GPU for me it was the uninstalling the OCL, OGL and Vulkan Compatibility Pack from Microsoft and then I was able to use RTX Remix normally
Version
2024.2.1

Hi @SuperGameBug ,
Thanks for your report. It looks like this is dup of https://github.com/NVIDIAGameWorks/rtx-remix/issues/382
Thank you

Hey @nvdamien ,
It is a possible fix to the issue. I wasn't sure there was already an issue raised.
Thanks :)

Hi,
Someone in our team is working on it.