ESP32 - mws.IsStarted() false when threaded

Hi,
when running  mws.Start(threaded=True)
then    mws.IsStarted()   always returns false.
mws is working fine, but of course   mws.Stop()   also fails.
What am I missing ? Thanks for your support
Alf
ESP32 4MB , micropython version='v1.11-178'

And if you wait a little after the Start call and before the IsStarted ?
Le ven. 26 juil. 2019 à 10:09, Alf<EMAIL_ADDRESS>a écrit :

Hi,
when running mws.Start(threaded=True)
then mws.IsStarted() always returns false.
mws is working fine, but of course mws.Stop() also fails.
What am I missing ? Thanks for your support
Alf
ESP32 4MB , micropython version='v1.11-178'
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/jczic/MicroWebSrv/issues/59?email_source=notifications&email_token=AAD2ALF2XNZSPC4GX22VRELQBKWMNA5CNFSM4IHBP4Z2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HBU6WDQ,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAD2ALFICBHQCGBWXGKQRCDQBKWMNANCNFSM4IHBP4ZQ
.

--
Sincèrement,

With a pause this is working fine.
MicroPython v1.11-178-gad0b7cb01 on 2019-07-25; ESP32 module with ESP32
>>> from microWebSrv import MicroWebSrv
>>> mws = MicroWebSrv(webPath="/www")
>>> mws.Start(threaded=True)
>>> utime.sleep(10)
>>> mws.IsStarted()
True

So my 2 seconds wait was too short.
How can I stop/restart the server now  ? - wait does not help there.
>>> mws.Stop()
>>> utime.sleep(10)
>>> mws.IsStarted()
True

Hello,
I released a fully new version (v2.0) of my web server here :
github.com/jczic/MicroWebSrv2.
Open source MIT, fully asynchronous, more robust, more fast and more
efficient!
It is delivered with a good documentation.
Thank you for your support and feedback. ☺
Jean-Christophe Bos
Le sam. 27 juil. 2019 à 10:31, Alf<EMAIL_ADDRESS>a écrit :

With a pause this is working fine.
MicroPython v1.11-178-gad0b7cb01 on 2019-07-25; ESP32 module with ESP32>>> from microWebSrv import MicroWebSrv>>> mws = MicroWebSrv(webPath="/www")>>> mws.Start(threaded=True)>>> utime.sleep(10)>>> mws.IsStarted()True
So my 2 seconds wait was too short.
How can I stop/restart the server now ? - wait does not help there.

mws.Stop()>>> utime.sleep(10)>>> mws.IsStarted()True

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jczic/MicroWebSrv/issues/59?email_source=notifications&email_token=AAD2ALG537RVLW7HJQWFRD3QBQBXNA5CNFSM4IHBP4Z2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD26HHFI#issuecomment-515666837,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAD2ALFI6JV2VSHCCYYYFWDQBQBXNANCNFSM4IHBP4ZQ
.

--
Sincèrement,


OAuth support?

Any plans to add OAuth support?

At the moment, we do not have a requirement for it. We encourage you to implement it and send us a Pull Request :)

Hey, I would love to implement this feature.


[🐞] Error when building my app with prisma

Which component is affected?
Qwik Runtime
Describe the bug
When I build my application with prisma it generates an error and I have not been able to deploy the changes in Vercel.

Reproduction
https://stackblitz.com/edit/qwik-starter?file=package.json
Steps to reproduce
pnpm qwik add prisma
System Info
System:
    OS: macOS 14.2
    CPU: (8) arm64 Apple M1 Pro
    Memory: 493.36 MB / 16.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 18.14.2 - ~/.nvm/versions/node/v18.14.2/bin/node
    npm: 9.5.0 - ~/.nvm/versions/node/v18.14.2/bin/npm
  Browsers:
    Brave Browser: <IP_ADDRESS>
    Chrome: 120.0.6099.71
    Safari: 17.2
  npmPackages:
    @builder.io/partytown: ^0.8.1 => 0.8.1
    @builder.io/qwik: ^1.2.14 => 1.2.14
    @builder.io/qwik-auth: 0.1.0 => 0.1.0
    @builder.io/qwik-city: ^1.2.14 => 1.2.14
    undici: 5.22.1 => 5.22.1
    vite: 4.3.5 => 4.3.5

Additional Information
No response

Is it the same issue of this one #4075?

Yes, it's the same one, it won't let me do build

Is it the same issue of this one #4075?

Yes, it's the same one, it won't let me do build

Let's close this one and we can continue in the other one. Thanks


Basic authentication works on Mac. Does not on Windows

The code can be found here: https://stackoverflow.com/questions/45417337/httpclient-with-networkcredential-returns-401-for-net-core-200-for-net-framewo
We have determined the issue to the linked problem was a combination cookie handling and basic authentication not being part of .Net Core.
However, it seems strange that the same code that works on a Mac OS, does not work on a Windows OS.

Additional information:

This is a Windows console program built with .NET Core that invokes a webservice with Basic Authentication is not populating the cookie correctly.  (.NET Core 1.0 or 2.0)
The same console program built with .NET Core running on a Mac works fine.
The same console program built with .NET Framework works fine.
As suggested in the article referenced above, the use of WinHttpHandler seems to get around the problem, but gives up OS independence.

Note HttpClient bugs go over here: https://github.com/dotnet/corefx
HttpClient uses a different implementation based on the operating system so there are some expected behavioural differences between Mac and Windows.
I'm going to close this as external, and it looks like you've already got an answer on StackOverflow.


Add bind-tools to base image to address alpine dns issues

Any chance of including bind-tools as one of the apks in the dockerfile?
ref:  https://github.com/nodejs/docker-node/issues/339
It only adds ~1MB and addresses alpine DNS resolution issues
Specifically when I have multiple networks I can't get container names to resolve in the docker-db-backup image.  exec'ing in and running apk update && apk add bind-tools fixes it.
Happy to submit a PR, just wanted to see if there was a willingness first.

Interesting that you are having DNS issues within the container. I run this image on a variety of Linux hosts and the majority of my images are alpine based and haven't run into it. Whats your host OS?
I typically also use a multi network configuration within the docker stack using the bridge driver.

Ubuntu 22.04.5, though on kernel 6.8
Example below - <IP_ADDRESS> is the right answer, the first nslookup is for some reason skipping docker resolution (<IP_ADDRESS> was returned because I have a wildcard in dnsmasq on my dns server).
[tiredofit/db-backup:4.1.4 23:38:34 /] $ nslookup obslivesync
Server:         <IP_ADDRESS>
Address:        <IP_ADDRESS>:53

** server can't find obslivesync.tailb(...).ts.net: NXDOMAIN

** server can't find obslivesync.tailb(...).ts.net: NXDOMAIN

Non-authoritative answer:

Non-authoritative answer:
Name:   obslivesync.domain.tld
Address: <IP_ADDRESS>

[tiredofit/db-backup:4.1.4 23:38:49 /] $ apk update && apk add bind-tools
fetch https://dl-cdn.alpinelinux.org/alpine/v3.20/main/x86_64/APKINDEX.tar.gz
fetch https://dl-cdn.alpinelinux.org/alpine/v3.20/community/x86_64/APKINDEX.tar.gz
v3.20.3-171-ge0d8a949f52 [https://dl-cdn.alpinelinux.org/alpine/v3.20/main]
v3.20.3-170-gb7926213bd4 [https://dl-cdn.alpinelinux.org/alpine/v3.20/community]
OK: 24186 distinct packages available
(1/6) Installing fstrm (0.6.1-r4)
(2/6) Installing json-c (0.17-r0)
(3/6) Installing protobuf-c (1.5.0-r0)
(4/6) Installing libuv (1.48.0-r0)
(5/6) Installing bind-libs (9.18.27-r0)
(6/6) Installing bind-tools (9.18.27-r0)
Executing busybox-1.36.1-r29.trigger
OK: 725 MiB in 188 packages
[tiredofit/db-backup:4.1.4 23:39:34 /] $ nslookup obslivesync
Server:         <IP_ADDRESS>
Address:        <IP_ADDRESS>#53

Non-authoritative answer:
Name:   obslivesync
Address: <IP_ADDRESS>

[tiredofit/db-backup:4.1.4 23:39:38 /] $

that said, my solution doesn't completely fix things;  nslookup and ping now resolve fine, but curl still has issues:
[tiredofit/db-backup:4.1.4 23:39:38 /] $ curl obslivesync:5984
curl: (7) Failed to connect to obslivesync port 5984 after 5 ms: Could not connect to server
[tiredofit/db-backup:4.1.4 23:43:37 /] $ ping obslivesync
PING obslivesync (<IP_ADDRESS>) 56(84) bytes of data.
64 bytes from obslivesync.obsidian-livesync_obsidian (<IP_ADDRESS>): icmp_seq=1 ttl=64 time=0.117 ms
64 bytes from obslivesync.obsidian-livesync_obsidian (<IP_ADDRESS>): icmp_seq=2 ttl=64 time=0.064 ms
64 bytes from obslivesync.obsidian-livesync_obsidian (<IP_ADDRESS>): icmp_seq=3 ttl=64 time=0.060 ms
64 bytes from obslivesync.obsidian-livesync_obsidian (<IP_ADDRESS>): icmp_seq=4 ttl=64 time=0.063 ms
64 bytes from obslivesync.obsidian-livesync_obsidian (<IP_ADDRESS>): icmp_seq=5 ttl=64 time=0.049 ms
^C
--- obslivesync ping statistics ---
5 packets transmitted, 5 received, 0% packet loss, time 4107ms
rtt min/avg/max/mdev = 0.049/0.070/0.117/0.023 ms
[tiredofit/db-backup:4.1.4 23:43:55 /] $ curl obslivesync:5984
curl: (7) Failed to connect to obslivesync port 5984 after 4 ms: Could not connect to server
[tiredofit/db-backup:4.1.4 23:44:04 /] $ curl <IP_ADDRESS>:5984
{"error":"unauthorized","reason":"Authentication required."}
[tiredofit/db-backup:4.1.4 23:44:21 /]

nothing fancy in the docker compose
services:
  obslivesync:
    image: couchdb:latest
    container_name: obslivesync
    user: 1000:1000
    networks:
      - t3_proxy
      - obsidian
    environment:
      - COUCHDB_USER=...
      - COUCHDB_PASSWORD=...
    volumes:
      - "cdb-data:/opt/couchdb/data"
      - "cdb-config:/opt/couchdb/etc/local.d"
    restart: unless-stopped
    labels:
      - "com.centurylinklabs.watchtower.enable=true"
      - "traefik.enable=true"
      - "traefik.http.routers.obsidian-rtr.entrypoints=websecure"
      - "traefik.http.routers.obsidian-rtr.rule=Host(`obsync.domain.tld`)"
      - "traefik.http.routers.obsidian-rtr.middlewares=secured@file"
      - "traefik.http.routers.obsidian-rtr.service=obsidian-svc"
      - "traefik.http.services.obsidian-svc.loadbalancer.server.port=5984"

  obs_db_backup:
    container_name: obs_db_backup
    image: tiredofit/db-backup
    networks:
      - obsidian
    depends_on:
      - obslivesync
    volumes:
      - obs-db-backup:/backup
    environment:
      - DB01_TYPE=couch
      - DB01_HOST=obslivesync
      - DB01_NAME=obsync
      - DB01_USER=...
      - DB01_PORT=5984
      - DB01_PASS=...
      - DB01_DUMP_FREQ=720
      - DB01_CLEANUP_TIME=72000
      - DEFAULT_CHECKSUM=SHA1
      - DEFAULT_COMPRESSION=GZ
      - DB01_SPLIT_DB=true
      - CONTAINER_ENABLE_MONITORING=FALSE
    restart: unless-stopped
    labels:
      - "com.centurylinklabs.watchtower.enable=true"

networks:
  t3_proxy:
    external: true
  obsidian:

volumes:
(...)

feels like #muslc/alpine things to me, not docker-db-backup issues.  just manually setting private ip's for now

aha, yep, alpine things - https://github.com/gliderlabs/docker-alpine/issues/574#issuecomment-1879741479
adding a . at the end also results in correct resolution without adding bindtools - obslivesync.

Musl has some interesting issues that creep up under certain environments for sure.
You can try adding CONTAINER_POST_INIT_COMMAND="apk update ; apk add bind-tools" which might help you? I built a couple weird things into the base image to execute commands after init for precisely this purpose without having to muck about with forking/running your own version of image or putting too many more utils in the base image.
There is another one that allows you to execute a script, although that will require you to export a volume with said script. The command above should work, if not try without the quotes..
Reference: https://github.com/tiredofit/docker-alpine?tab=readme-ov-file#container-options
Maybe related - Also, I saw a ts.net in the above output. I wonder if that is overriding things as it's known to have its own resolver I think at <IP_ADDRESS> if I recall correctly. Has been a while since I have run ts. Newer unreleased images of my base have TS baked right in..


Review all tutorial/FAQ screenshots

Check the screenshots are actually loading properly.

Reviewing these after the gh-pages merge.
1-Create a Facet Filter > Facet Filter Applied screenshot incorrect
2-Gallery has ###Notes at the bottom
3-Bivarate graphs has //see other tutorials or go home and then ##Exit at bottom
4-Create a map - first screenshot missing
5-Timespan filter needs a link to the overlapping filter sheet.

Confirmed that both screenshots are corrected (#1&4), the formatting notes are fixed (#2&3), and the timespan filter link was added.  Closing this issue.


Error downloading acc name

hello
i get this error while i want to login.
att my debug
DebugInfo.log

same issue here, debug included
DebugInfo.log

DebugInfo.log

did anyone find a solution ?

Duplicate: #1140

I've cloned this repository and (temporarily) fixed the issue.
Feel free to download the new version I've created from my own repository, it should be public.
https://github.com/Omga4000/Procurement/releases/tag/1.30.0
Let me know if that works for you.

I've cloned this repository and (temporarily) fixed the issue. Feel free to download the new version I've created from my own repository, it should be public.
https://github.com/Omga4000/Procurement/releases/tag/1.30.0
Let me know if that works for you.

Not Working....
[22-12-2022 19:28] Failed to build HTTP request and get response for: method=GET, url='https://api.poe.watch/account?name=DarkyUzzy&realm=pc', allowAutoRedirects=, requestData='': The remote server returned an error: (400) Bad Request.
[22-12-2022 19:28] Error downloading account name, exception details: System.Net.WebException: The remote server returned an error: (400) Bad Request.
at System.Net.HttpWebRequest.GetResponse()
at POEApi.Transport.HttpTransport.BuildHttpRequestAndGetResponse(HttpMethod method, String url, Nullable1 allowAutoRedirects, String requestData)    at POEApi.Transport.HttpTransport.PerformHttpRequest(HttpMethod method, String url, Nullable1 allowAutoRedirects, String requestData)
at POEApi.Transport.HttpTransport.GetAccountName(String username, String realm)
at POEApi.Model.POEModel.GetAccountName(String username, String realm)
[22-12-2022 19:28] System.Exception: Error downloading account name, details logged to DebugInfo.log. Please open a ticket at https://github.com/Stickymaddness/Procurement/issues and include your DebugInfo.log
at POEApi.Model.POEModel.GetAccountName(String username, String realm)
at Procurement.ViewModel.LoginWindowViewModel.<>c__DisplayClass23_0.b__0()
at System.Threading.Tasks.Task.Execute()

Omga4000  ty for trying....
thailyn don't even bother to respond:)

@DarkyUzzy
Please try the latest release. It contains a new fix which should work permanently.
https://github.com/Omga4000/Procurement/releases/

Duplicate of #1140.

Hey mate, all works fine thank you for your help.
i wanted to get my url by messaging  poexyzis
https://www.pathofexile.com/account/view-profile/poexyzis but no respond
Le jeu. 22 déc. 2022 à 09:58, Omga4000 @.***> a écrit :

I've cloned this repository and (temporarily) fixed the issue.
Feel free to download the new version I've created from my own repository,
it should be public.
https://github.com/Omga4000/Procurement/releases/tag/1.30.0
Let me know if that works for you.
—
Reply to this email directly, view it on GitHub
https://github.com/Procurement-PoE/Procurement/issues/1141#issuecomment-1362577831,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2WY3X4GBYL6GZDQHYMVOLDWOQJ23ANCNFSM53C4PYDA
.
You are receiving this because you commented.Message ID:
@.***>


Do not automatically cache models in temp dirs

Currently, CACHE_DIR for exported models is set to a temporary directory unique to the model export. This is not useful, the cache will never be used. It also can cause issues with large exported models on GPU (model loading crashes).
This PR disables automatic model caching if the model save directory is a temporary directory. That doesn't just affect exported models, automatic model caching is now also disabled if users manually save models in /tmp (or Windows equivalent) - but I do not think that that is an issue. Users can always enable model caching manually by setting CACHE_DIR.
I also aligned the cache dir for diffusion models with other models, so now all cache dirs are "model_cache" (previously for stable diffusion the cache dir was the model dir).

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

The tests failed at:
FAILED tests/openvino/test_modeling.py::OVModelForCausalLMIntegrationTest::test_compare_with_and_without_past_key_values - AssertionError: False is not true : With pkv latency: 336.185 ms, without pkv latency: 361.155 ms, speedup: 1.074
This test fails quite often and I disabled it in this PR. I tested this earlier, when this test first started failing and we reduced the SPEEDUP threshold, and back then I did see speedup when testing manually on Ice Lake Xeon.


Make the trivy-operator-policies-config config map optional in the helm chart

The trivy-operator-policies-config is a hardcoded name. The CM is created by default, without the option to disable it. Adding a feature flag to disable the CM creation from the CM would allow a consumer to provide the CM from an external source.
This helps with providing custom rego policies as the only option currently is to provide them as a multi line string as helm extra values.

@lsoica contribution are welcome if you have time


pull request for e2e test at 12/06/2016 12:15:00

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Add context-specific escaper helpers

This component should include an extension with the following context-specific escaping mechanisms from zend-escaper:

escapeHtml()
escapeHtmlAttr()
escapeJs()
escapeCss()
escapeUrl()

Adding these would be a value-add for those using Plates with Expressive, and provide added security features for template authors.

Please close this issue for its implemented in #22 .
Thank you.


python3Packages.mypy: 0.620 -> 0.630

Now requires mypy_extensions so added it as well.
Motivation for this change
I've had mypy problems when running with python language server. I updated mypy, sadly it doesn''t fix my problem but let's share the update eitherway.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Fits CONTRIBUTING.md.

Note that mypy now requires mypy_extensions so I added it. I kept the same maintainers as for mypy (which might be a bold move don't hate me plz :) )

seems like it thanks for the fix.
Le mar. 18 sept. 2018 à 16:06, Jörg Thalheim<EMAIL_ADDRESS>a
écrit :

@Mic92 commented on this pull request.
In pkgs/development/python-modules/mypy/extensions.nix
https://github.com/NixOS/nixpkgs/pull/46788#discussion_r218318697:

@@ -0,0 +1,25 @@
+{ stdenv, fetchPypi, buildPythonPackage, typing, isPy3k }:

+buildPythonPackage rec {

pname = "mypy_extensions";
version = "0.4.1";

Tests not included in pip package.

doCheck = false;

src = fetchPypi {
inherit pname version;
sha256 = "04h8brrbbx151dfa2cvvlnxgmb5wa00mhd2z7nd20s8kyibfkq1p";
};

disabled = !isPy3k;

080b3ef
https://github.com/NixOS/nixpkgs/commit/080b3ef0b59622a7b805b85ea565b6ab27a062bd
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/NixOS/nixpkgs/pull/46788#discussion_r218318697, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA2FOrq6cHY0mofmLLD_Y0BwXli4SEi5ks5ucJtagaJpZM4WsLWA
.


morph_data error

Hi all,
I am trying to morph individual source data to fsaverage. I am using this piece of code:
evoked = EvokedArray(patterns, epochs.info,
                                 tmin=epochs.tmin)
inv = read_inverse_operator(op.join(path_data, subject,
                                        subject + '-inv.fif'))
stc = apply_inverse(evoked, inv, lambda2=1.0 / (2 ** snr),
                                method='MNE', pick_ori='normal')
vertices_fsaverage = [np.arange(10242), np.arange(10242)]
morphed = mne.morph_data(subject, 'fsaverage', stc,
                                     n_jobs=1, grade=vertices_fsaverage,
                                     subjects_dir=subjects_dir)

It finished without error on only 2 participants out of 8. When it crashed, the error is this one.
Morphing data...
<ipython-input-32-bca94b1f110b>:33: RuntimeWarning: Morph map "/Users/quentinra/Desktop/MEGdata/WM1/subjects/morph-maps/fsaverage-sub12_XZUKLHYZ_AE-morph.fif" does not exist, creating it and saving it to disk (this may take a few minutes)
  subjects_dir=subjects_dir)
Morph map "/Users/quentinra/Desktop/MEGdata/WM1/subjects/morph-maps/fsaverage-sub12_XZUKLHYZ_AE-morph.fif" does not exist, creating it and saving it to disk (this may take a few minutes)
Creating morph map sub12_XZUKLHYZ_AE -> fsaverage
Creating morph map fsaverage -> sub12_XZUKLHYZ_AE
    21 smooth iterations done.
    21 smooth iterations done.
    21 smooth iterations done.
---------------------------------------------------------------------------
IndexError                                Traceback (most recent call last)
<ipython-input-32-bca94b1f110b> in <module>()
     31 morphed = mne.morph_data(subject, 'fsaverage', stc,
     32                          n_jobs=1, grade=vertices_fsaverage,
---> 33                          subjects_dir=subjects_dir)

/Applications/anaconda/lib/python2.7/site-packages/mne-python/mne/source_estimate.pyc in morph_data(subject_from, subject_to, stc_from, grade, smooth, subjects_dir, buffer_size, n_jobs, warn, verbose)

/Applications/anaconda/lib/python2.7/site-packages/mne-python/mne/utils.pyc in verbose(function, *args, **kwargs)
    724         with use_log_level(verbose_level):
    725             return function(*args, **kwargs)
--> 726     return function(*args, **kwargs)
    727
    728

/Applications/anaconda/lib/python2.7/site-packages/mne-python/mne/source_estimate.pyc in morph_data(subject_from, subject_to, stc_from, grade, smooth, subjects_dir, buffer_size, n_jobs, warn, verbose)
   2157                                      warn=warn)
   2158                      for data_buffer
-> 2159                      in np.array_split(data[hemi], n_chunks, axis=1)), axis=1)
   2160
   2161     vertices = [nearest[0], nearest[1]]

/Applications/anaconda/lib/python2.7/site-packages/mne-python/mne/source_estimate.pyc in <genexpr>((data_buffer,))
   2156                                      n_vertices, nearest[hemi], maps[hemi],
   2157                                      warn=warn)
-> 2158                      for data_buffer
   2159                      in np.array_split(data[hemi], n_chunks, axis=1)), axis=1)
   2160

/Applications/anaconda/lib/python2.7/site-packages/mne-python/mne/source_estimate.pyc in _morph_buffer(data, idx_use, e, smooth, n_vertices, nearest, maps, warn, verbose)

/Applications/anaconda/lib/python2.7/site-packages/mne-python/mne/utils.pyc in verbose(function, *args, **kwargs)
    724         with use_log_level(verbose_level):
    725             return function(*args, **kwargs)
--> 726     return function(*args, **kwargs)
    727
    728

/Applications/anaconda/lib/python2.7/site-packages/mne-python/mne/source_estimate.pyc in _morph_buffer(data, idx_use, e, smooth, n_vertices, nearest, maps, warn, verbose)
   1947         # get the row sum
   1948         mult = np.zeros(e.shape[1])
-> 1949         mult[idx_use] = 1
   1950         idx_use_data = idx_use
   1951         data_sum = e * mult

IndexError: index 167004 is out of bounds for axis 1 with size 166949

(index and size in the IndexError varies for each participant)
Would you have an idea on how to fix that?
Thanks a lot!
Romain

Can you share these files somewhere?

/Users/quentinra/Desktop/MEGdata/WM1/subjects/morph-maps/fsaverage-sub12_XZUKLHYZ_AE-morph.fif

$SUBJECTS_DIR/surf/lh.white and rh.white

the stc (with stc.save)

the inv (mne.minimum_norm.write_inverse_operator)

I think that should be enough for me to reproduce and look at what's going on.

You can find these files here:
https://www.dropbox.com/sh/1nburpswo5lqesf/AAC0T4QpoFaK-BlYTlZS6-aka?dl=0
Let me know if you need something else. Thanks a lot!

There is a mismatch between your source space surfaces and the Freesurfer surfaces that you have uploaded for your subject. When I run this:
import numpy as np
import mne

subjects_dir = '.'
subject = 'sub12_XZUKLHYZ_AE'
stc = mne.read_source_estimate('stc')
inv = mne.minimum_norm.read_inverse_operator(subject + '-inv.fif')
for s, hemi, vertno in zip(inv['src'], ('lh', 'rh'), stc.vertices):
    assert s['subject_his_id'] == subject
    rr, tris = mne.read_surface('%s/%s/surf/%s.white'
                                % (subjects_dir, subject, hemi))
    assert np.array_equal(s['vertno'], vertno)
    assert rr.shape == s['rr'].shape, (rr.shape, s['rr'].shape, hemi)```

I get:
AssertionError: ((161335, 3), (156287, 3), 'lh')

Is there somehow a problem in your pipeline with multiple subjects_dirs being used?
If I add this line to my script:
src = mne.setup_source_space(subject, subjects_dir=subjects_dir)

I get an error about lh.sphere missing (can you upload lh.sphere and rh.sphere, too?), but you should be able to provide the files yourself to try it. You should see that this src does not match inv['src'], even though it should -- so I think that somehow the wrong files have been used at some point.

It is working now. Thanks a lot!


configuration manager for StyleGuide discussion

we can browse through those new configuration now but still not add new ones (for gradients & colors). Seems like i broken the old ones as well :-/


add visuals in addition to the commands

While creating different files and commits, to would be good to see how the structure of the local directory, the index and commit tree are changing. The Git book has some really nice graphs that we can take inspiration from. I'd be very happy to help out with this.

Thank you. All help welcome here.

This is somehow related to #459. I will follow-up there. This here is a good idea but the lesson changed in the meantime and we are also one year later.


include css leading forward slash missing on linux

when including:
    style
      include style.css

on linux, the leading forward slash is missing:
class de.neuland.pug4j.exceptions.PugParserException: the included file [style.css] could not be opened
home/<snip>/target/classes/style.css (No such file or directory) in /home/<snip>/target/classes/view.pug:68

Seems related to #resolvePath:
        if(parentFileName.startsWith("/")){
            parentFileName = Paths.get(basePathString + parentFileName.substring(1)).normalize().toString();
        }

Can't see any explanation as to why this root slash is removed.. Or why it appears to work ok with this on Windows..

Never-mind, appears was fixed in https://github.com/neuland/pug4j/commit/58086fe5b8a656024c42321b48603191ccd192ac :) Thanks!


Fix/dossier report UI fixes

closes #360
closes #361
closes #363
closes #362
I know its better to have a PR for each issue, but those are very small tasks that making a PR for each takes more effort than solving them

I think it's fine in this case to close multiple issues at once, but can you give the issues full descriptions and demo steps? (This is a pretty hefty chunk of the grading scheme)


automate object __json__ing

rather than having to specify which properties to json, can we do it automatically?

I think the answer is yes, at least for a good chunk of the properties.

This issue was moved to Dallinger/Dallinger#140


[MRG] download original file formats from Dataverse #1242

Dataverse creates plain-text, preservation-friendly copies of certain file formats (some of which are proprietary, such as Stata or SPSS) and this .tab (tab-separated) file is downloaded unless you supply format=original, which is what this pull request does.
The original filename (e.g. foo.dta, a Stata file) comes from originalFileName, which is only populated when the preservation copy (e.g. foo.tab) has been successfully created.
Additional variables were created to distinguish between filename, original_filename, and filename_with_path. If original_filename is available, it's the right one to use.
To allow the tests to continue passing, the query parameters are now removed so just the file id can be cast as an int.
I tested it with a random dataset that has Stata files (.dta):
beamish:repo2docker pdurbin$ repo2docker doi:10.7910/DVN/IVLEHB
beamish:repo2docker pdurbin$
beamish:repo2docker pdurbin$ docker exec -it inspiring_almeida /bin/bash
pdurbin@521a8dd1e1cc:~$ ls -1
'Codebook for Relational UNFCCC Data.pdf'
country_groups.dta
ENB_relationships.dta
statements_count.dta
unfccc_ratification.dta
'Variable and value labels.xlsx'
pdurbin@521a8dd1e1cc:~$

Without this fix you get country_groups.tab (tab separated), for example.

Closes #1242

@betatim hi! Over in Discourse it was suggested to me to reach out to existing maintainers who have particular knowledge in the area and since you encouraged me (thanks!) to make this PR and merged the original Dataverse content provider PR (#739), I thought I'd start with you.
What do you think? Does the PR make sense? Any questions? Thanks!! ❤ ❤ ❤

@pdurbin this looks sensible to me. The linter's upset and the autofix bot didn't run because it doesn't have permission to push to IQSS (PRs from orgs don't usually grant maintainers edit access, unlike PRs from personal forks).
If you run pre-commit install and pre-commit run --all-files and commit the results it should be happy. It's only isort that needs appeasement, if you want to do it by hand (but the reason we use autoformatters is to not think about these things):
--- a/tests/unit/contentproviders/test_dataverse.py
+++ b/tests/unit/contentproviders/test_dataverse.py
@@ -4,8 +4,8 @@ import re
 from io import BytesIO
 from tempfile import TemporaryDirectory
 from unittest.mock import patch
-from urllib.request import Request, urlopen
 from urllib.parse import urlsplit
+from urllib.request import Request, urlopen

@minrk thanks for approving!
Yes, I was able to fix the isort problem in 48f4cc6 with pre-commit run --all-files.
It gave the following output:
(venv) pdurbin@air repo2docker % pre-commit run --all-files
[INFO] Initializing environment for https://github.com/asottile/pyupgrade.
[INFO] Initializing environment for https://github.com/psf/black.
[INFO] Initializing environment for https://github.com/pycqa/isort.
[INFO] Initializing environment for https://github.com/pre-commit/mirrors-prettier.
[INFO] Initializing environment for<EMAIL_ADDRESS>[INFO] Installing environment for https://github.com/asottile/pyupgrade.
[INFO] Once installed this environment will be reused.
[INFO] This may take a few minutes...
[INFO] Installing environment for https://github.com/psf/black.
[INFO] Once installed this environment will be reused.
[INFO] This may take a few minutes...
[INFO] Installing environment for https://github.com/pycqa/isort.
[INFO] Once installed this environment will be reused.
[INFO] This may take a few minutes...
[INFO] Installing environment for https://github.com/pre-commit/mirrors-prettier.
[INFO] Once installed this environment will be reused.
[INFO] This may take a few minutes...
pyupgrade................................................................Passed
black....................................................................Passed
isort....................................................................Failed
- hook id: isort
- files were modified by this hook

Fixing /Users/pdurbin/github/jupyterhub/repo2docker/tests/unit/contentproviders/test_dataverse.py

prettier.................................................................Passed

I see now that the docs are pretty clear about pre-commit: https://repo2docker.readthedocs.io/en/2022.10.0/contributing/contributing.html#code-formatting
Sorry for missing that!
It looks like the pre-commit CI test passed. 🎉

Thanks!

@minrk thanks for merging!
Can you please give me a sense of when this will be available on mybinder.org?
I poked around in the docs and found https://repo2docker.readthedocs.io/en/latest/contributing/tasks.html#creating-a-release that says, "We make a release of whatever is on main every month."
However, it seems like actual release may be a bit less frequent, which is understandable (Dataverse certainly doesn't release every month!). https://repo2docker.readthedocs.io/en/latest/changelog.html gives me a sense of the cadence:

Version 2022.10.0
Version 2022.02.0
Version 2021.08.0
Version 2021.03.0

Hmm, and I'm guess this is just the repo2docker side. I'm not sure how often mybinder.org pulls in releases from repo2docker.
I'm asking because once this "download the original format" fix is in production at mybinder.org, I'll make an announcement to the Dataverse community (the initial announcement about Binder support was in February) and mention it in our next release notes.
Thanks.

It is now deployed. Thanks!

@minrk thanks! I've been playing around with it.
Going back to this country_groups.dta (original) vs country_groups.tab (archival), Binder is now downloading the original version from that dataset.
In the Dataverse UI, we show the .tab (archival) version. If you click the "Binder" button...

... a container spins up...

... and we can see that the original versions (.dta for a Stata file) has been downloaded in Binder!
Time to play with that data! 🎉


Fix download link for perfview.exe

The old link confusingly only lists the TraceEvent release, which is tagged as latest. The updated link shows the list of releases, so you can easily find the perfview release underneath TraceEvent.
fixes #1543

Thanks @mregen for this.  I just made a very similar fix this morning without realizing you had already done the work.


Dashboard redesign

Due to the redesgin of the dashboard, we will need to update the pics in the docs:
Tutorials

[x]  https://merginmaps.com/docs/tutorials/working-collaboratively/

Install & Sign Up

[x]  https://merginmaps.com/docs/setup/sign-up-to-mergin-maps/

Manage Account & Project

[x] https://merginmaps.com/docs/manage/workspaces/
[x] https://merginmaps.com/docs/manage/subscriptions/
[x] https://merginmaps.com/docs/manage/permissions/
[x] https://merginmaps.com/docs/manage/create-project/#create-a-project-through-merginmaps-com
[x] https://merginmaps.com/docs/manage/project-advanced/
[x] https://merginmaps.com/docs/manage/delete-files/
[x] https://merginmaps.com/docs/manage/dashboard/
[x] https://merginmaps.com/docs/manage/project-details/

Support & Legal

[x] https://merginmaps.com/docs/misc/troubleshoot/

released 26-03-2024


Flutter 3.1.0 beta

My LGTM doesn't seem to have helped move this along 🫠

My LGTM doesn't seem to have helped move this along 🫠

Thanks @ericwindmill, @RedBrogdon can we get Eric credentials on this repo? =)

PTAL @RedBrogdon


some naming-conventions

predicate-functions with - inside normally have also a - before the p
so string-containsp should be string-contains-p and string-tokenp should be string-token-p

I was trying to be consistent with other string- predicates in CL: string-greaterp, string-lessp. But if it's confusing maybe I should reconsider.

Fwliw, I think it is better to follow the rule @cl-porky11 mentions. No need to increment the list of exceptions to the rule for the p/-p archaism.


Vignettes

This PR adds a new vignette that explores the available get functions in GAMBLR.data (also used for testing this repo). In addition, this PR also improves on existing documentation. For the reviewer's convenience, an HTML-rendered version of the vignette is also attached here. But I would advise the reviewer to also try to knit the vignette for reproducibility purposes.

get_data.pdf


Black background

Version

3.7.0
Development Environment

JavaScript
Current Behavior

When I try to get the edited image I got an image that has black background (not edited image)
// Write example code
imageEditor.toDataURL();

Expected Behavior

I want to get the edited image insted of a black background image.

Sorry, I don't know exactly what the situation is.
You can use 'codepen.io' or write a more specific example.


Bug title

Describe the bug
When calling BluetoothLEAdvertisementPublisher.Start(), I get:
System.UnauthorizedAccessException HResult=0x80070005 Message=Access is denied. (0x80070005 (E_ACCESSDENIED)) Source=WinRT.Runtime StackTrace: at WinRT.ExceptionHelpers.g__Throw|20_0(Int32 hr)
Thrown.
I have used minimal code to reproduce this issue, based on the blank WinUI3 project (unpackaged).
The code can be found here: https://github.com/John-Carroll-AtAsc/TestingBleUnderWinUI3
Steps to reproduce the bug
Clone the project mentioned above.
Set all 'Common Language Runtime Exceptions' (clear box with a checkmark)
Build and run.
Expected behavior
No exceptions and the BluetoothLEAdvertisementPublisher.StatusChanged event is triggered.
Screenshots
No response
NuGet package version
Windows App SDK 1.4.0: 1.4.230822000
Packaging type
Unpackaged
Windows version
Windows 11 version 22H2 (22621, 2022 Update)
IDE
Visual Studio 2022
Additional context
I have stack traces available as well as any other information needed. I have tried falling back to the WindowsAppSdk version 1.2 and 1.3 to no avail. This same set of Bluetooth calls works in C++ w/ winrt, and in UWP.

I have discovered the problem, and all is well.


Program size

Looks good but how to reduce program size ?

Module
.text
.data
.bss

[fill]
204(-4)
12(+0)
42(+0)

[lib]\c.a
25193(+0)
2472(+0)
89(+0)

[lib]\gcc.a
4820(+0)
0(+0)
0(+0)

[lib]\m.a
576(+0)
0(+0)
0(+0)

[lib]\misc
252(+0)
16(+0)
28(+0)

main.o
1792(-4)
4(+0)
4084(+0)

mbed-lora-radio-drv\SX1272
495(+0)
0(+0)
0(+0)

mbed-lora-radio-drv\SX1276
8367(+0)
0(+0)
0(+0)

mbed-os\cmsis
1033(+0)
0(+0)
84(+0)

mbed-os\drivers
2011(+0)
4(+0)
152(+0)

mbed-os\events
1496(+6)
0(+0)
0(+0)

mbed-os\features
30706(-22)
4(+0)
0(+0)

mbed-os\hal
3388(+0)
8(+0)
247(+0)

mbed-os\platform
3905(+0)
260(+0)
268(+0)

mbed-os\rtos
11118(+0)
168(+0)
5973(+0)

mbed-os\targets
13735(+0)
4(+0)
1017(+0)

trace_helper.o
2(+0)
0(+0)
0(+0)

Subtotals
109093(-24)
2952(+0)
11984(+0)

Total Static RAM memory (data + bss): 14936(+0) bytes

Total Flash memory (text + data): 112045(-24) bytes`

ARM Internal Ref: IOTCELL-1704

@x893 If you have size issues you can use two (three) major tricks: 1) remove RTOS, 2) build with newlib-nano, 3) use minimal-printf. See https://github.com/janjongboom/mbed-os-example-blinky-no-rtos

I try 2) and 3) and reduce to 104 KB.  I think you know more about internals for mbed-os/features and other mbes-os parts.
Thanks

Well, try 1. It will get rid of an additional 40K flash.

Hi,
I am using the exact mbed ignore in https://github.com/janjongboom/mbed-os-example-blinky-no-rtos and the same tiny.json build profile.  I can see a join accept in the TTN console but the device never receives a CONNECT event (or any other event) and gets stuck at "Connection - In Progress ...".

@evandavey Can you enable traces and post a new issue with the trace log? See https://github.com/ARMmbed/mbed-os-example-lorawan#optional-adding-trace-library.

OK will do tomorrow my time (currently evening).

@evandavey Just FYI, I've tested the example on DISCO-L475VG-IOT01A1 with SX1276 radio with .mbedignore and tiny profile and it connects fine for me; interested to see the traces.

@janjongboom good to know, thanks.  This is a custom PCB using a Murata Type ABZ module - I had it connecting using the stock example but now that seems to be throwing a
HAL_RCC_OscConfig ERROR so not sure what's up with that (using the cloud compiler and target DISCO-L072CZ-LRWAN1 as MTB_MURATA_ABZ doesn't work).  The example above compiled offline with GCC_ARM and MTB_MURATA_ABZ as the target.

Looks like there is now a 'bare-metal' mode for mbed-os 5.12.0 https://github.com/ARMmbed/mbed-os/releases/tag/mbed-os-5.12.0.  How would I use that for this project?

@janjongboom

@x893 If you have size issues you can use two (three) major tricks: 1) remove RTOS, 2) build with newlib-nano, 3) use minimal-printf. See https://github.com/janjongboom/mbed-os-example-blinky-no-rtos

Solution 3. (disabling mbed features in .mbedignore) works fine for mbed-os 5.11., but does not work with ver. 5.12. - plenty of dependencies to header files from disabled modules (nanostack, cellular,...) like e.g.:
'
Compile [  1.6%]: rf_configuration.c
[Fatal Error] rf_configuration.c@16,44: nanostack/platform/arm_hal_phy.h: No such file or directory
[ERROR] .\mbed-os\components\802.15.4_RF\stm-s2lp-rf-driver\source\rf_configuration.c:16:44: fatal error: nanostack/platform/arm_hal_phy.h: No such file or directory
#include "nanostack/platform/arm_hal_phy.h"
'
Is it possible still to disable mbed features in mbed-os version >=5.12?

@kluszkiewicz if you use baremetal profile, it should do the trick. If you still have an issue, file a new issue on Mbed OS repository with details

@kluszkiewicz if you use baremetal profile, it should do the trick. If you still have an issue, file a new issue on Mbed OS repository with details

But what if I want to disable some features like features/cellular, lwip, ...?
Can i control mentioned features using baremetal option?

@kluszkiewicz if you use baremetal profile, it should do the trick. If you still have an issue, file a new issue on Mbed OS repository with details

I've reported issue on Mbed OS repo:
https://github.com/ARMmbed/mbed-os/issues/10678

The bug is valid and is going to be fixed. I have commented to issue how you can proceed with this.


Clock Example Update

Updated Clock example asset and enabled manual adjustment

lgtm.
(not for this one, i can imagine it) for a visual change like this it would be cool to see a video/screenshot of the change in the pr

for screen shots when we add a new visual feature/change


ESI.request family docstring hint missing

With VS Code, code hint for ESI.get() and ESI.head() is
(*args: Any, **kwd: Any) -> Coroutine

which is supposed to be
(key: str, async_loop: List | None = None, **kwd: Any) -> (dict | List[dict])

ESI.get.__doc__ gives correct docstring.

Similar issue occur with eve_tools.api.market apis.

After investigating the issue, all decorators are wrapped with functools.wraps, and ESI.get.__annotations__, ESI.get.__doc__, and inspect.signature(ESI.get) are correct (same for eve_tools.api.market). When using Pytho IDLE 3.9, the docstring hint is correct. Therefore, this is considered an issue specific for VS Code.
Please comment on this issue if you have any suggestions.


Shopping cart translation (subtotal, view cart, checkout)

https://matrioszkaonline.com
Hi guys,
I have a problem w shopping cart translation
I translated all the elements but I can not find this one
Subtotal, view cart, checkout
do you know how I can solve my problem?

wordpress 4.9.8
Hyyan WooCommerce Polylang Integration 1.2.0
Polylang 2.3.8
WooCommerce 3.4.4
--

Hi, I have the similar issue here, but it is not in the cart page but in the order review page.
[Versions]
Wordpress 4.9.8
Hyyan WooCommerce Polylang Integration 1.2.0
Polylang 2.3.9
WooCommerce 3.4.4
Theme OnePress

The default language of my web site is Japanese and I'm testing the translation into English. The other part of the order review page is translated into English successfully, but the part of the attached image which seems generated by the review-order.php is not translated into English.
Actually, when this page is loaded, at first the table contents in question are translated into English. Then, it is translated back to Japanese after the request to ?wc-ajax=update_order_review. (confirmed by using the Chrome's dev tool's network view.). I think this is the same issue as #319. However, I'm using the Hyyan WooCommerce Polylang Integration 1.2.0, which contains the fix for #319.
Removing all the cookies for this site did not solve the issue. I appreciate any helps on this issue, thanks.

I would suggest doing a test with the Storefront theme - this is one of the questions you are asked to check before raising an issue, there is mostly likely some customisation in either your theme or other plugins which are interfering with the translations.
I'm running trilingual sites with different themes and I can't reproduce this.


NOISSUE - Add channel update

Signed-off-by: fbugarski<EMAIL_ADDRESS>Pull request title should be MF-XXX - description or NOISSUE - description where XXX is ID of issue that this PR relate to.
Please review the CONTRIBUTING.md file for detailed contributing guidelines.
What does this do?
Which issue(s) does this PR fix/relate to?
Put here Resolves #XXX to auto-close the issue that your PR fixes (if such)
List any changes that modify/break current functionality
Have you included tests for your changes?
Did you document any new/modified functionality?
Notes

Codecov Report

Merging #1501 (566defe) into ui (f9611ee) will decrease coverage by 0.05%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##               ui    #1501      +/-   ##
==========================================
- Coverage   66.99%   66.93%   -0.06%
==========================================
  Files         136      136
  Lines       10904    10916      +12
==========================================
+ Hits         7305     7307       +2
- Misses       3018     3028      +10
  Partials      581      581

Impacted Files
Coverage Δ

ui/api/endpoint.go
0.00% <0.00%> (ø)

ui/api/requests.go
0.00% <ø> (ø)

ui/api/transport.go
0.00% <0.00%> (ø)

things/service.go
74.45% <0.00%> (+0.86%)
:arrow_up:

Continue to review full report at Codecov.

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


About multiprocessing

I tried using multiprocessing with asynchronous and it got infinity loop or something. Program is not processed. Here is sample
from multiprocessing import Pool
import deepcut as dc

def f(text):
    return dc.tokenize(text)

if __name__ == '__main__':
    texts = ["ง่ายๆแค่นี้เอง", "ลองใหม่สิ", "เอางี้นะ", "เอาอีกรอบหนึ่ง"]
    pool = Pool(processes=4)

    multiple_results = [pool.apply_async(f, (i,)) for i in texts]
    for func in multiple_results:
         print(func.get())

I opened the Activity Monitor on macOS and it's not used the processor.
It happened after the updated version that making your program faster. I think.

Hi @NititOrz, can you try following this example from Stack Overflow? I think you have to import keras for each workers in order to use multiprocessing. Something as following would work.
from multiprocessing import Pool
def _tokenizing(text):
    import deepcut
    return deepcut.tokenize(text)

texts = ["ง่ายๆแค่นี้เอง", "ลองใหม่สิ", "เอางี้นะ", "เอาอีกรอบหนึ่ง"]
pool = Pool(processes=4)

multiple_results = [pool.apply_async(_tokenizing, (i,)) for i in texts]
for func in multiple_results:
     print(func.get())

@titipata Thank you.


fix(fluentd):  unescape prometheusrules labels

Q
A

Bug fix?
yes

New feature?
no

API breaks?
no

Deprecations?
no

Related tickets
no

License
Apache 2.0

What's in this PR?

This PR updates fluentd prometheusrules summaries and descriptions.
Why?

To unescape (wrongly) escaped prometheus labels
Checklist

[x] Code meets the Developer Guide

@ahma any chance this gets releases any time soon? Thanks you

@aslafy-z I hope we can release this in this week, but I can't promise that. Thanks for your patience.


Another thing is before we merge this this package requires a file called vcruntime140_app.dll. what vcruntime is that part of?


Heading Label for Level 3

Hi, the heading label works only for levels 1 and 2 but not for levels 3.
Here is a minimal example. It can compile without <heading-level-3> but it encounters an error if I give <heading-level-3>.
#import "@preview/touying:0.5.0": *
#import themes.university: *

#show: university-theme.with(
  aspect-ratio: "16-9",
  config-info(
    title: [Title],
    subtitle: [Subtitle],
    author: [Authors],
    date: datetime.today(),
    institution: [Institution],
    logo: emoji.school,
  ),
)

#set heading(numbering: "1.1")

#title-slide()

= The Section
<heading-level-1>
== Slide Title
<heading-level-2>
=== Slide Subtitle
<heading-level-3>

#lorem(40)

Thanks. I wrote a mistake. I will wait for a few days to see if there are any other bugs, and then release a new version.


Implementation of the revised simplex

It implies the implementation of the LU decomposition method.
This should improve resolution times on most problems (especially on mix integer problems) and will dramatically reduce errors due to numerical instability of the traditional simplex.
Most of the details of the algorithm are on this slides: http://faculty.chicagobooth.edu/kipp.martin/root/htmls/coursework/36900/handouts/simplex2.pdf
(What is missing though is how to dynamically update the LU decomposition matrix instead of computing it from scratch at every iteration).

link broken


jobStore.importFile returns a string not a fileStoreID

Is this intentional? It means that if you use the importFIle method then pass the result to a job and invoke the .size method (for resource allocation), you get an error...

@cket this is an easy fix, but I need confirmation. Is this intentional? Will changing this break any existing code?

@jessebrennan this isn't intentional, if it breaks existing code then that also needs to be fixed

@ifiddes also reported this to me a few days ago. It's not intentional and might not be an easy fix for non-fileJobStore implementations for importFile. My temp fix for a file that exists on the same machine was
from toil.fileStore import fileID
x = FileID.forPath(toil.importFile('file:///path/to/file')

This won't work as well for something on s3 but you could find a similar hack

I have a similar hack that works for S3 and local jobStores:
https://github.com/ArtRand/marginAlign/blob/toil_dev_signal/src/margin/toil/localFileManager.py#L145
Ingestion of files with .importFIle and .exportFile can be a little dodgy in my experience (i.e. works sometimes).


Cabuzau - Status "Database"

ENVIROMENT: production
URL: https://scrapers.peviitor.ro/
Browser: Chrome
Device: laptop
OS: Windows 10
STEPS TO REPRODUCE:

Open URL in browser
Type the name of the company in the search bar
Run scraper
Check the status

EXPECTED RESULT:
Status to be "Active"
ACTUAL RESULT:
Status is "Database"

cabuzau s-a schimbat în compania de apa Buzău

It work!


Fix behaviour of test adapter

The test adapter's deliver implementation should still return the sent email like the other adapters do.

The adapters don't actually return the original email. Neither does the type spec say it should return the email.

Let me know if you have strong opinions that this should be the way.

No, I probably misunderstood then. At least one of the adapters does return the email like this so I thought this was an inconsistency.


Better ABI Validation: check type of constructor/call args against abi

Add to the general validation phase a check that each argument in the call/constructor args of the relevant futures matches the type expected by the abi.
TODO

[ ] Draw up a test plan that will effectively act as a specification

The types of futures that accept arguments and need validation are:

contract deploys (not libraries)
function calls
static calls

For arguments, these are the types of inputs we need to be able to validate:

raw primitives (number, bigint, string, boolean)
contract futures (resolve to address type)
static call futures (resolves to any solidity param type)
read argument event futures (resolves to any solidity param type)
runtime value

account (resolves to address type)
module parameter (resolves to number, bigint, string, boolean)

a record of any of the above
an array of any of the above

For raw primitives, we can let ethers do the heavy lifting for us with something like:
const iface = new ethers.Interface(artifact.abi);

try {
  iface.encodeDeploy([future.constructorArgs])
} catch {
  throw new Error('failed validation')
}

however, this approach would only work for primitive values, so I think primitives should instead just be accounted for within our custom loop that could look something like this (pseudocode):
function validateArgs(artifact: Artifact, functionName: string, args: ArgumentType[]): IgnitionError[] {
  // loop through, probably recursively, and resolve to primitive type strings
  const argTypes = args.map((arg) => resolveTypeForArg(artifact, arg));

  // get an array of primitive type strings representing the valid type for each arg, derived from the ABI and likely with help from ethers
  const validTypes = resolveValidTypesForFunctionArgs(artifact, functionName);

  // compare each argument against the valid type
  let errors = [];
  for (let i = 0; i < argTypes.length; i++) {
    if (argTypes[i] !== validTypes[i]) {
      errors.push(new IgnitionError("invalid arg type"));
    }
  }

  return errors;
}

@zoeyTM if I am following. During validation we add additional checks to args (either call or constructor) that recursively cross checks the types of the passed values with the ABI's type.
Does ethers v6 give us any util functions to help with this?
It seems viem has parseAbi, but I wouldn't want to switch to viem in just his case (and a full viem switch is likely out of scope).
Do you think cross checking the types is extensive custom code? Or is leveraging either ethers/viem the way to go?


slider have more latency than few months ago

Is your issue REALLY a bug?

[X] My issue is indeed a bug!
[X] I am not crazy! I will not fill out this form just to ask a question or request a feature. Pinky promise.

Is there an existing issue for this?

[X] I have searched the existing issues.

Is this issue related to iced?

[X] My hardware is compatible and my graphics drivers are up-to-date.

What happened?
https://github.com/user-attachments/assets/dde6a070-77c1-4884-a648-f6b135429fa3
My older binary(April or May built) works well, but this happened to me recently when I build the same code, the build options are same too.
I tried to build a simple example (slider) but it have latency too.
What is the expected behavior?
no delay when dragging the slider
Version
crates.io release
Operating System
Windows
Do you have any log output?
No response

Wait, is the version of iced different between these builds?


[FEATURE] Navigate between panels with the mouse and/or arrow keys

Is your feature request related to a problem? Please describe.
I just tried dblab for the first time, and the usage instructions in README.md say things like "to see the structure of a table, locate yourself in the tables list, select your table, and hit enter" without actually specifying how to get to the tables list.  Confusingly, clicking on a table in the list will select the table in it, but the focus won't switch so hitting enter does nothing.  As far as I could tell, there's nothing in the program itself to give you a clue how to get there, and the actual shortcut (ctrl-H to switch to the panel to your left) is buried in the documentation, even though "select a table and see its structure" is one of the first things that makes sense to try when exploring a database.
Describe the solution you'd like
There are a few possible solutions.  One, clicking could switch panes, so you could click on a table name and hit enter to display it.  Two, there could be some kind of in-app cheatsheet, like what happens in lazygit when you press the question mark key.  Three, some aliases could be added for navigation -- I realize that hjkl are intuitive to vim users, but the first thing I reach for is the arrow keys, so maybe ctrl-left/right/up/down could go between panes?
I realize this is a bit unfocused, but I figured I'd try to give first-time-user feedback and hopefully help improve the experience for those unfamiliar with the interface.  I love the functionality of dblab now that I've figured it out and think you've got a great project here!

Hi @coandco
Thanks for reaching out!
I'm so sorry for the confusion.  I need to improve the navigation description. It clearly lacks a description on how to navigate between panels. I'll add that as soon as I can.
When it comes to the proposed solutions, I won't invest time on improving the current ux/ui because I'll introduce a new gui/tui library, because the current one is old and unsupported, so it might change the way we interact with dblab drastically.
The best I can do is right now is improve the navigation documentation. I'll keep this open until now.


Search topic

When searching the ANF documentation for "SMB Signing" the topic/section is not located or link to this page.  It would be helpful to update the index so that customers can search the ANF documentation for "SMB Signing" and jump straight to the information they need concerning this topic on the page.

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

ID: e8954a2f-1f97-09f3-9bdc-da4c618ab404
Version Independent ID: d9e2add2-a47c-d3c8-45c6-195f47e02832
Content: SMB performance best practices for Azure NetApp Files
Content Source: articles/azure-netapp-files/azure-netapp-files-smb-performance.md
Service: azure-netapp-files
GitHub Login: @b-hchen
Microsoft Alias: anfdocs

@scottkntap
Thanks for your feedback! We will investigate and update as appropriate.

@AjayBathini-MSFT - I'm not sure this request can be manually implemented. As I know, the MSFT authoring system doesn't have a way for authors to "index" content like an old-style linear documentation.  This behavior applies not only to the Azure NetApp Files documentation but also all Azure product docs.
One trick I myself use is to use the PDF output (left corner, below the TOC pane) to do a global search of specific words within the doc.

@b-hchen
Thanks for your responses.
@scottkntap
If there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.

Hi Julie,
Thank you for your reply.  What’s interesting is that if I search for “nfs mount options” at the top of the ANF docs page, it finds that topic and provides a link directly to it within the ANF documentation.  Similarly, if I search for “concurrency” if finds that page in the docs as well.  However, for other topic searches it does not; for example, “smb signing” or “smb multichannel”.
@.***
Best Regards,
Scott Kovacs
Azure Customer Success Engineer
Public Cloud Services
NetApp
919.476.4560 Direct Phone
919.946.3044 Mobile Phone
<EMAIL_ADDRESS>[signature_1806012642][signature_3991436275]
@.?anonymous&ep=signature>Book time to meet with @.?anonymous&ep=signature>
UPCOMING OUT OF OFFICE: 8 May – 12 May
[A picture containing text  Description automatically generated]
[signature_4209890365]https://www.facebook.com/NetApp   [signature_3305356061] https://www.instagram.com/netapp/    [signature_894704209] https://www.youtube.com/netapp    [signature_619426719] https://www.linkedin.com/company/netapp/    [signature_1533971304] https://twitter.com/NetApp
From: Julie Chen @.>
Date: Monday, May 1, 2023 at 13:50
To: MicrosoftDocs/azure-docs @.>
Cc: Kovacs, Scott @.>, Mention @.>
Subject: Re: [MicrosoftDocs/azure-docs] Search topic (Issue #108897)
NetApp Security WARNING: This is an external email. Do not click links or open attachments unless you recognize the sender and know the content is safe.
@AjayBathini-MSFThttps://github.com/AjayBathini-MSFT - I'm not sure this request can be manually implemented. As I know, the MSFT authoring system doesn't have a way for authors to "index" content like an old-style linear documentation. This behavior applies not only to the Azure NetApp Files documentation but also all Azure product docs.
One trick I myself use is to use the PDF output (left corner, below the TOC pane) to do a global search of specific words within the doc.
—
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/108897#issuecomment-1530007405, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ALAYJV2VF5OMBXWCGUJCIDDXD7ZWTANCNFSM6AAAAAAXPRXZQM.
You are receiving this because you were mentioned.Message ID: @.***>

@scottkntap - Thanks much, Scott!  I think in the case of your search for “nfs mount options” or “concurrency” (using the search box at the top of left pane / TOC), those terms match titles of ANF articles, i.e. "Linux NFS mount options best practices for Azure NetApp Files" (https://learn.microsoft.com/en-us/azure/azure-netapp-files/performance-linux-mount-options), and "Linux concurrency best practices for Azure NetApp Files - Session slots and slot table entries" (https://learn.microsoft.com/en-us/azure/azure-netapp-files/performance-linux-concurrency-session-slots), respectively.  So the search box displays those pages as suggested links.
In the case of searching for "SMB signing", the term is not part of an official ANF article title, so the tool takes you to the Azure site-wide search (for all learn.microsoft.com), e.g. https://learn.microsoft.com/en-us/search/?scope=Azure&terms=smb signing.  You'd need to filter to specify "documentation and maybe add "Azure NetAPp FIles" along with "SMB Signing" in the search box of that site-wide page to find SMB signing mentions for ANF (e.g. https://learn.microsoft.com/en-us/search/?scope=Azure&terms=smb signing Azure NetApp Files )
I understand Scott's point about enhancing intra-product (i.e. intra-ANF in this case) search beyond only article titles.  @SaibabaBalapur-MSFT -- If you know who in MSFT to forward this tool enhancement request to, please forward this ticket to them.  Thanks.

@VikasPullagura-MSFT
could you please assist on this thread.

OK, I see the difference now, “SMB signing” is a sub-heading in the broader topic of “SMB performance best practices”.  Thank you for clarifying that.
Best Regards,
Scott Kovacs
Azure Customer Success Engineer
Public Cloud Services
NetApp
919.476.4560 Direct Phone
919.946.3044 Mobile Phone
<EMAIL_ADDRESS>[signature_4209918118][signature_4253401597]
@.?anonymous&ep=signature>Book time to meet with @.?anonymous&ep=signature>
UPCOMING OUT OF OFFICE: 8 May – 12 May
[A picture containing text  Description automatically generated]
[signature_2529458452]https://www.facebook.com/NetApp   [signature_2792130793] https://www.instagram.com/netapp/    [signature_2831779638] https://www.youtube.com/netapp    [signature_1139347558] https://www.linkedin.com/company/netapp/    [signature_3921175421] https://twitter.com/NetApp
From: Julie Chen @.>
Date: Tuesday, May 2, 2023 at 13:29
To: MicrosoftDocs/azure-docs @.>
Cc: Kovacs, Scott @.>, Mention @.>
Subject: Re: [MicrosoftDocs/azure-docs] Search topic (Issue #108897)
NetApp Security WARNING: This is an external email. Do not click links or open attachments unless you recognize the sender and know the content is safe.
@scottkntaphttps://github.com/scottkntap - Thanks much, Scott! I think in the case of your search for “nfs mount options” or “concurrency” (using the search box at the top of left pane / TOC), those terms match titles of ANF articles, i.e. "Linux NFS mount options best practices for Azure NetApp Files" (https://learn.microsoft.com/en-us/azure/azure-netapp-files/performance-linux-mount-options), and "Linux concurrency best practices for Azure NetApp Files - Session slots and slot table entries" (https://learn.microsoft.com/en-us/azure/azure-netapp-files/performance-linux-concurrency-session-slots), respectively. So the search box displays those pages as suggested links.
In the case of searching for "SMB signing", the term is not part of an official ANF article title, so the tool takes you to the Azure site-wide search (for all learn.microsoft.com), e.g. https://learn.microsoft.com/en-us/search/?scope=Azure&terms=smb signing. You'd need to filter to specify "documentation and maybe add "Azure NetAPp FIles" along with "SMB Signing" in the search box of that site-wide page to find SMB signing mentions for ANF (e.g. https://learn.microsoft.com/en-us/search/?scope=Azure&terms=smb signing Azure NetApp Files )
I understand Scott's point about enhancing intra-product (i.e. intra-ANF in this case) search beyond only article titles. @SaibabaBalapur-MSFThttps://github.com/SaibabaBalapur-MSFT -- If you know who in MSFT to forward this tool enhancement request to, please forward this ticket to them. Thanks.
—
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/108897#issuecomment-1531869284, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ALAYJV66OWUE5JWWONYHLOTXEFAAPANCNFSM6AAAAAAXPRXZQM.
You are receiving this because you were mentioned.Message ID: @.***>

Hello @scottkntap
As Julie Chen mentioned this is the standard indexing followed across Microsoft documents. This behavior applies not only to the Azure NetApp Files documentation but also all microsoft azure docs.
Best way to search content in MS documentation:
When looking for specific content in an azure document, I search the azure documentation page with filter set to the service i am looking at and search with accurate keywords, each of the search recommendations will take me to the top of the document and then I do ctrl+f to find the topic across that page, as you can see in the following screenshot. I recommend searching for content you need to revise this way.

This really isn't a document issue here. It is as per the design. The issue you pointed out here in case of searching for "SMB signing", the topic is part of SMB best practices guide. Hence it showed the link to the top of the guide. You need to look up with ctrl+f and find within that redirected link.

Let us know if there is anything that you need help with.
FYI @SaibabaBalapur-MSFT

@PriyankaRudramaina
Thanks for the quick responses.
@scottkntap
We are going to close this thread as resolved but if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.


[libpqxx] update to 7.3.0

Fix #15237
Update libpqxx to the latest version 7.3.0
Note: No feature need to test

Thanks for your help!


APIs to retrieve data summary such as histogram

number, date/datetime : histogram
boolean: histogram of true/false/null
text (vocab) : word frequency
free text : no aggregate
json: no aggregate

This seems like a duplicate of #134

And there's already an ermrest API that can give you text counts assuming you mean number of rows that have an exact text value:
GET /ermrest/catalog/N/attributegroup/Schema:Table/Column;Count:=cnt(*)

this will return a table of results with each row having a "Column" field with a specific value and a "Count" field for the number of rows in Table that had this value.  The reason we need a different histogram API is that the attributegroup API only groups by actual column values.  It does not have any value truncation/binning mechanism to group by those coarser granularity approximations instead.
If you're hoping for word frequency counts within a text value, e.g. breaking paragraphs and sentences into words and counting them, that seems far out of scope for what ermrest offers currently.

closing duplicate


netapplier: Show diff if verification fails

This makes netapplier show a nice diff:
libnmstate.netapplier.DesiredStateIsNotCurrentError:
desired:
========
name: voidbr
type: unknown
state: up
ipv4:
  address:
  - ip: <IP_ADDRESS>
    prefix-length: 24
  enabled: true
ipv6:
  address:
  - ip: fe80::3fd:5f3f:db52:72a2
    prefix-length: 64
  enabled: true
mtu: 1500000

current:
========
name: voidbr
type: unknown
state: up
ipv4:
  address:
  - ip: <IP_ADDRESS>
    prefix-length: 24
  enabled: true
ipv6:
  address:
  - ip: fe80::3fd:5f3f:db52:72a2
    prefix-length: 64
  enabled: true
mtu: 1500

difference:
===========
--- desired
+++ current
@@ -14 +14 @@
-mtu: 1500000
+mtu: 1500

Coverage increased (+1.3%) to 70.427% when pulling e17767a7587a466824b4b78b76f1090a84bd1b76 on tyll:prettydiff into dca1a0f3f3617bf1f5f90457ef1094fba695583b on nmstate:master.


test commit

Coverage remained the same at 90.933% when pulling 5607ce30c8b49b88727c815ccf6e32fba4939f83 on gl3nn:coveralls-test into d16388f4ce2435e3b1dd4fd4da1a1dcacff6f969 on kayak:master.

Coverage remained the same at 90.933% when pulling 5607ce30c8b49b88727c815ccf6e32fba4939f83 on gl3nn:coveralls-test into d16388f4ce2435e3b1dd4fd4da1a1dcacff6f969 on kayak:master.


Service Workers for offline support in docs

Add service worker support to the microsite. This should allow as much offline capability as possible, especially the docs and examples.

Superceded by https://github.com/material-components/material-components-web/issues/63


Feat/custom dns

When using custom VNET, I had to update manually /etc/hosts files on whole nodes to add nodes IP and hostname entries. This is makes autoscaling unusable.
Thanks to this PR, I can choose a custom DNS server I created (Ubuntu/Bind9) and I'm able by adding an extension to update the DNS entry of my nodes on their creation.

/assign @jackfrancis

/assign @jackfrancis

@aminebizid FYI, I rebased off of master because of some nasty changes to engine.go that you didn't anticipate :)
I'll review and run E2E, thanks so much for this!

@aminebizid Should we file a follow-up item to support > 1 DNS server?

/approve
/lgtm


Provide more specific feedback / output when receiving error responses from the API

It would be useful to have a verbose command line option to commands to get a little more detail. I was trying to publish a package and the answer to what was wrong was in the return from the server. I would like to be able to specify -v to get the error output from the server.

I think there still might be a case for a verbose option that just shows the full server response, but there's a more basic problem here in that the bare minimum information that should be displayed in the case of an error is not.  Sometimes the straight up wrong message is shown instead.
An easy to replicate example is the package publish command.  If the package.yaml fails validation, the error that the CLI displays should say as much and provide the validation error message so that the user knows how to fix it.  Instead, the error that gets displayed is:
Error: Please set an active environment id using 'environment set'

This would seem to indicate that the error handling is overly broad and makes bad assumptions about what a 400 error means.  I'm going to guess we knew this when we wrote the original version of the code, with the idea that we would circle back and improve the error handling.  I think that time to circle back is now, otherwise we're going to waste cycles troubleshooting basic errors as we use the CLI.

My idea for how to start here to essentially to create another function to handle the errors within client.py's post request since at this point we'll probably be doing enough work specifically on error handling to justify it. The three errors I have so far thought of (that aren't handled somewhere else in the code) would be caused by having no environment_id set at all, having an invalid environment_id, and the invalid_package.yaml error that shows up in publishing. We should be able to tell what category the error is via the text of the response object generated. My initial thought would be if we don't have a better error message to display to the user in a click exception, to just pass the text of the response into the click exception.


chore: wdio browser matching

Adding support for skipping tests in wdio based on the browser being used.

Overview

Adding support for skipping tests in wdio.
The matcher code was rewrote from what was used in JIL.

The code is not split out into multiple files instead of being in one large file.
The matching/testing logic is simpler while still being flexible with the spec string
Only a single pre-defined matcher was carried over for now
Was able to do a little cleanup on the wdio configs due to this change

Moved the sauce labs code to a shared folder and updated the package.json scripts.
Related Issue(s)

https://issues.newrelic.com/browse/NEWRELIC-7589
Testing

Use the below example test to create different spec matchers.
Example test file:
import SpecMatcher from '../../tools/browser-matcher/spec-matcher.mjs'
import { notInternetExplorer } from '../../tools/browser-matcher/common-matchers.mjs'

const notChrome = new SpecMatcher()
  .exclude('chrome')

describe('newrelic api', () => {
  let testHandle

  beforeEach(async () => {
    testHandle = await browser.getTestHandle()
  })

  afterEach(async () => {
    await testHandle.destroy()
  })

  test(notInternetExplorer)('not ie', async () => {
    const url = await testHandle.assetURL('instrumented.html', {
      loader: 'spa'
    })

    await Promise.all([
      testHandle.expectRum(),
      browser.url(url)
    ])
    const result = await browser.execute(function () {
      return typeof window.newrelic.addToTrace === 'function'
    })

    expect(result).toEqual(true)
  })

  test(notChrome)('not chrome', async () => {
    const url = await testHandle.assetURL('instrumented.html', {
      loader: 'spa'
    })

    await Promise.all([
      testHandle.expectRum(),
      browser.url(url)
    ])
    const result = await browser.execute(function () {
      return typeof window.newrelic.addToTrace === 'function'
    })

    expect(result).toEqual(true)
  })
})

Codecov Report

Merging #519 (d12dbb8) into main (ba7d17c) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #519   +/-   ##
=======================================
  Coverage   19.19%   19.19%
=======================================
  Files         126      126
  Lines        4158     4158
  Branches     1033     1033
=======================================
  Hits          798      798
  Misses       2635     2635
  Partials      725      725

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Workaround to eliminate unwanted host route for API, DNS and INGRESS VIPs

This is a temporary solution to workaround the extra host route,
It changes VIPs netmask to the master subnet netmask.
I set it as [WIP] since I didn't test it yet.

Can you add a bit of an explanation about why this fixes the problem?

also note another workaround proposed for the same issue here: https://github.com/openshift-metalkube/dev-scripts/pull/369

Build FAILURE, see build http://<IP_ADDRESS>:8080/job/dev-tools/579/

@russellb ,
This patch doesn't delete the redundant routes, created automatically by the kernel for each interface/IP address, (this should be fixed in the kernel), just set VIPs netmask to interface's main address netmask instead of 32, as a result of that the 'unneeded' routes created for the VIPs by the kernel will be identical to route for the interface main address, so we will not hit the 'connect: invalid argument' anymore.
Below you can see the routing table, IP a, and ping to API VIP  status (after keepalived set & move VIPs):
[core@master-1 ~]$ ip -4 a  | grep 192.168.111
inet <IP_ADDRESS>/24 brd <IP_ADDRESS> scope global dynamic noprefixroute ens4
[core@master-1 ~]$ ip r
default via <IP_ADDRESS> dev ens4 proto dhcp metric 101
<IP_ADDRESS>/14 dev tun0 scope link
<IP_ADDRESS>/24 dev ens3 proto kernel scope link src <IP_ADDRESS> metric 100
<IP_ADDRESS>/16 dev tun0
<IP_ADDRESS>/24 dev ens4 proto kernel scope link src <IP_ADDRESS> metric 101
<IP_ADDRESS>/24 dev ens4 proto kernel scope link src <IP_ADDRESS> metric 101
<IP_ADDRESS>/24 dev ens4 proto kernel scope link src <IP_ADDRESS> metric 101
[core@master-1 ~]$ ping <IP_ADDRESS>
PING <IP_ADDRESS> (<IP_ADDRESS>) 56(84) bytes of data.
64 bytes from <IP_ADDRESS>: icmp_seq=1 ttl=64 time=0.262 ms
64 bytes from <IP_ADDRESS>: icmp_seq=2 ttl=64 time=0.148 ms

I've applied this on a cluster for 24 hours. I can confirm it workaround the issue - all nodes are in ready state.


Automated Spec Update

Automated Spec Update
c36ba27d8d56648555d3068bb3826e1d3a44d92b
Co-authored-by: Bruce Zhang<EMAIL_ADDRESS>

Closing pull request because I didn't realize this wasn't merged when I merged my spec update. These updates will exist in https://github.com/dropbox/dropbox-sdk-dotnet/pull/329


[Dotcom Shell]: CTA - Card link has blue text in header

Description
In Dotcom Shell Storybook, the CTA - Card link that appears on the page has a blue heading. Could be related to https://github.com/carbon-design-system/carbon-for-ibm-dotcom/issues/8273

Component(s) impacted
Dotcom shell
Browser
Firefox
Carbon for IBM.com version
v1.31.0-rc.1
Severity
Severity 4 = The problem is not visible to or noticeable to an average user. Affects minor functionality, no workaround needed.
Application/website
C4IBM.com
Package
@carbon/ibmdotcom-react
CodeSandbox example
https://codesandbox.io/s/github/carbon-design-system/carbon-for-ibm-dotcom/tree/main/packages/react/examples/codesandbox/components/DotcomShell
Steps to reproduce the issue (if applicable)
No response
Release date (if applicable)
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I checked the current issues for duplicate issues

Will be fixed with CTA card link bug.  Closing


Incorrect flake8-comprehension fixits (C401,C402) in fstring

One annoying false positive change in this flake8-comprehensions are set/dict conversions to literals in f'strings.
b = list(range(10))
print(f'{set(a if a < 6 else 0  for a in b)}')

which prints: {0, 1, 2, 3, 4, 5}
becomes
b = list(range(10))
print(f'{{a if a < 5 else 0  for a in b}}')

which is actually wrong if it's in a fstring since it actually just escapes the innermost curly brace. This results in the printing of {a if a < 6 else 0 for a in b}
It should fixit as
b = list(range(10))
print(f'{ {a if a < 5 else 0  for a in b} }')

The space will prevent the curly brace from being escaped and is deleted when the fstring resolves which should provide identical behavior to the original code snippit. See this Stackoverflow post: https://stackoverflow.com/questions/49120113/dictionary-set-comprehensions-inside-of-f-string
I suspect you wound find similar behavior for C402 with dict comprehensions.
Command, Version Info, and Settings
ruff --select C401,C402 example.py --fix
ruff 0.0.244
no ruff pyproject.toml ruff settings specified

Ahh thank you! Good catch. I guess arguably we should avoid flagging this at all in f-strings.

\cc @sbrugman if you're interested.

@charliermarsh It's not a bad change in fstrings, and it is potentially more performant, just need to be careful to do it properly :P . Another option is just to add an extra space around the inserted braces unconditionally and let black or whatever formatter you use deal with it.

Yeah I could see either being "correct". Regardless sorry for the breakage!


Can't change brush size

I love this program, but i can't seem to find a way to change the brush size for the pen tool, please help

Hi, this issue is a duplicate of #258 .
Implementation is ongoing, feel free to add your comments in the existing thread.


RAS-A Signed Packet Support

I expect this one to be controversial... but here goes. QGC-Gov is required to be RAS-A compliant. This means we should support the standard as written, and as "THE" RAS-A ground station, we need to include even the optional parts of RAS-A. I understand there are differing opinions about the usefulness of the signed packets portions of RAS-A, but regardless it is an official part of the standard. Since we are using libmav to handle all our low-level MAVlink packet encoding/decoding, we need libmav to support this.
Happy to provide details on or discuss design choices here if required. I can also walk through testing steps used to validate this. Or happy to discuss any of the above in more detail.

Message::validate() performs a check on the signature of incoming messages.
Yeah, I definitely see your point. I was interpreting "finalize" as "make the message header". And since the header is a different part of the message than the signature, it made sense to me to do the signing separately. Also, if we're doing the signing in finalize(), I'd need to pass both the timestamp and key in as arguments, which seems like a rather cumbersome amount of args for one function. But its feasible. Would you prefer that?
I can indeed! I'll look into it!

Hope you don't mind, I also added a note to the readme about how to run the tests. I've never used doctest before, and took me a minute to figure out how to actually do it, so hopefully this can save someone else time in the future.

If you have signing enabled on a NetworkRuntime, would ever not want to validate the incoming messages? In case it is enabled, we should always validate and reject bad signatures.

Depends. The standard is inspecific. To quote RAS-A:

Accepting Unsigned Packets
MAVLink libraries should provide a mechanism that allows a system to conditionally accept unsigned packets.
...
Accepting Incorrectly Signed Packets
MAVLink libraries should provide a mechanism that allows a system to conditionally accept incorrectly signed packets.

So the rejection needs to be done at the application level and it is not a decision libmav can make. You could make an argument that libmav should always at least try to validate? We could perhaps always do the calculation and set a field on the message if in validate, and the application could choose to read it or not?

@ThomasDebrunner Is there some way you can configure SonarCloud to ignore include/mav/picosha/*? I updated the gcovr's config to ignore it in the coverage report, but SonarCloud still sees it as a changed file and reports 0%, blocking this PR.

@stuart-auterion Just rebase this to main - let's see what sonarcloud says. I guess it should be possible to exclude picohash

Hmmm, looks like its still tracking coverage on picosha.

Hmmm, looks like its still tracking coverage on picosha.

Although, this is still an issue.

Yeah, I asked Jakob, maybe he knows why

Aw bummer. Looks like I'm still 0.8% short on new code coverage. I'll address that when I can.
Thanks for fixing sonarcloud!

@ThomasDebrunner should be ready now.
Thanks for the patience and help getting this PR in a good state.

Merged, thanks @stuart-auterion


Add support for Babel 6 export default

Hey, I'm the author of babel-hubot, minimalistic Hubot wrapper that allows to use Babel to compile scripts written in JS.
Prior to Babel 6, integration was super simple:
--- a/bin/hubot
+++ b/bin/hubot
@@ -3,4 +3,4 @@
 npm install
 export PATH="node_modules/.bin:node_modules/hubot/node_modules/.bin:$PATH"

-exec node_modules/.bin/hubot "$@"
+exec node_modules/.bin/babel-hubot "$@"

Unfortunately in 6th version, Babel changed the way how export default is handled, so now:
export default function() {}

... is not equal to:
module.exports = function() {}

... but to:
module.exports = {default: function() {}}

To make it work properly I had to change Hubot itself and check if script.default is a function and then use it instead of script.

Any chance to get it merged?

Good job! :+1:

This is an change/extension of the existing script interface. Currently, the sole and standard way of providing a script to Hubot is by exporting a function. If we need a more flexible script interface, I would much prefer to focus our energy on making that generic through programmatic loading (#1134).
Can you give me an idea of how impactful this is to bot owners to not upgrade to Babel 6 as compared to things like upgrading the version of coffeescript? For the coffeescript upgrade, there were new features that were really nice, but you could continue using the old version without a problem.

I unknowingly duplicated this issue with #1374. Would be nice to get some action on allowing es6 scripts. @kossnocorp Am I right that would your solution would not effect any existing interfaces, but just allows the es6 default export format as well? @michaelansel If this solution provides some future-proofing to the script loading interface, without depreciating the prior format, why not allow it in the mean time, because coming up with a new more considered interface could take a long time.
@kossnocorp - have you tried using the babel add-module-exports plugin? https://github.com/59naga/babel-plugin-add-module-exports - as a stop-gap, it should convert your default exports into standard es5 module exports.


Custom Role Permission Assignment is Broken

Populating custom role types with specific permissions results in a list of "undefined" values in the UI regardless of the value entered.

Update: It seems like the permissions are assigned correctly internally, but they're always displayed as "undefined". This only affects custom roles — permissions of the built-in role types display correctly.


CI builds compile files 4x

Searching CI logs for "Compiling [Filename].mm" reveals it compiled 4x:

If we can fix this the CI builds will run approx 4x faster and might not ever time out.

I think this is normal though; except for macOS, we are building for both device (arm64 and armv7) and simulator (x86_64 and i386 still) builds. We do need simulator to be included so that people can work on their projects.
In a typical Xcode build on your Mac it would only build for the current architecture but I think I'd rather catch something going wrong on any of the architectures we build for.

For the tests at least, there's no point in building for an architecture unless the tests are running on that architecture.

Well the reason it's happening this way is because that's doing more than CI, but actually building fully functional frameworks ready for release. The only thing regular CI builds don't do right now is upload the archives.
What we could do though is trim down the architectures that get built for regular CI builds (i.e. not for tags). I think that's the way it was on Travis (and why builds still complete on v4), but since Github Actions is capable to run everything in parallel I figured I'd rather have everything happen for now.

One side effect of building everything is that one can supposedly download fully built XCFrameworks from the build artifacts of any commit. Not sure anybody is taking advantage of that though.

Ok well it seems like these builds might as well run tests:

I changed the build scripts to be more in line to what we were doing with Travis and build/test time is now around 22 minutes or so, skipping the device builds.

Yeah, its strange, the build times have been ranging from 26 minutes to over an hour so its difficult to gauge that 22 minute time after your commit. Right after that my next commit took 26 minutes again. Hopefully that's a high water mark now.  Looks like the macOS build is taking the longest at around 11 minutes.

Yeah I'm not sure if I can shorten that one too somehow. Maybe skip i386.

Actually, the reason it takes longer on iOS is because we're building both macOS and Catalyst libraries - they are separate builds so that in effects doubles the time there. Also makes the macOS archives quite bigger.

This seems like it might be good enough now. If not, please feel free to reopen the issue.


Please put an explanation of "retry" label on the device console

This is a feature request for the console.

What do you want to do?

Understand what the term "retry" means in the

What steps did you take?

Create a simple program that sends short packets to TTN and inspect uplink data packets in the console.
The data packets are regular packets without confirrmation requirement.

What went wrong or what is missing?

The console shows a perculiar term "retry" next to several uplink data packets. The label "retry" is not accompanied by an explanation or prompt, and the terminology of "retry" is not used in LoraWAN libraries such as https://github.com/matthijskooijman/arduino-lmic and https://github.com/mcci-catena/arduino-lmic.
As a courtesy to TTN users please explain what "retry" is on the console along with how it might or might not have an impact on the transmission.

Do you have Screenshots?

What kind of OS/Browser/Gateway are you using? Which version?
Chrome

Interesting question. It's however not a simple answer. I'm not part of the TTN team, but I will try and explain how I interpret this "retry" message. This is also on the V2 stack, and development focuses on V3 now.
In normal situations a device sends uplink packets, every time incrementing the frame counter. It should therefore never happen that you see more than one packet with the same frame counter. On some older devices if you sent an uplink message, requiring an ack, and you do not receive the ack, the device will retry sending the exact same packet with the same frame counter. This is incorrect and is fixed in the later LoRaWAN standards and libraries.
On the TTN Console a packet is normally only shown if it passes a frame counter check. This is a check that makes sure that packets aren't "replayed". In cases where a device sends the same packet more than once, TTN should only show the first one and discard any future ones. If you disabled frame counter checks, TTN will not discard these "replayed" packets, but list them with the "retry" label.
It can also happen that a packet is received by more than one gateway. TTN will wait 200ms for all gateways to forward the packet, merging all the copies of the same packet together and only show one on the console. If a gateway's internet connection has a high latency, that gateway's copy of the packet might arrive after this 200ms de-duplication window, and the packet will be seen as a "replay". The packet will either be discarded or marked as "retry", depending on if you disabled frame counter checks or not.

@jpmeijers thanks :) your thorough explanation is magnitudes more helpful than the rudimentary "retry" label, now I understand that in the screenshot retry simply means that the device is sending packets while reusing frame counter value.


Ensure vttablet does not crash if not given all parameters it expects

Specifically I managed to crash it when mycnf was empty and it tried to reference the socket name to talk to mysqld on. Even if that setting is not present the binary should not fail and given it's possible via the appropraite dbconfig-XXXX settings to configure access to a remote mysqld then the requirement for the socket should not be there.
If you see a better fix then feel free to implement this.

Example (simplified) usage where this happened:
[user@somehost ~/dev/bin]$ export VTDATAROOT=$HOME/dev/vtdataroot ./vttablet -tablet-path cell1-0000000100

Would trigger the breakage.

On second thoughts maybe for now it's better to not try to fix each problem but to catch and report the situation and let the user deal with this?
Longer term simplification of command line parameters may be better but avoiding a crash and indicating which expected variable is nil at least points at the technical cause even if it does lead the user to have to investigate further in the code.


removing broken links

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

his/core/integration-services1.md
:white_check_mark:Succeeded
View

his/core/planning-and-architecture1.md
:white_check_mark:Succeeded
View

his/core/populating-a-managed-provider-dataset-from-a-data-adapter2.md
:white_check_mark:Succeeded
View

his/core/reading-data-from-a-database1.md
:white_check_mark:Succeeded
View

his/core/receiving-messages-with-session-integrator-for-lu22.md
:white_check_mark:Succeeded
View

his/core/retrieving-schema-information1.md
:white_check_mark:Succeeded
View

his/core/sending-a-message-with-session-integrator-for-lu21.md
:white_check_mark:Succeeded
View

his/core/services-account-screen1.md
:white_check_mark:Succeeded
View

his/core/session-integrator-programming-interfaces1.md
:white_check_mark:Succeeded
View

his/core/stored-procedures-in-a-db2-database1.md
:white_check_mark:Succeeded
View

his/esso/enterprise-single-sign-on-programmer-s-reference.md
:white_check_mark:Succeeded
View

his/esso/how-to-log-a-remote-user-on-to-a-local-application.md
:white_check_mark:Succeeded
View

his/esso/single-sign-on-samples.md
:white_check_mark:Succeeded
View

his/install-and-config-guides/his-2013-what-s-new-install-and-configure.md
:white_check_mark:Succeeded
View

his/install-and-config-guides/his-2016-what-s-new-release-notes-system-requirements-and-installation.md
:white_check_mark:Succeeded
View

his/install-and-config-guides/same-server-migration.md
:white_check_mark:Succeeded
View

his/install-and-config-guides/server-to-server-migration.md
:white_check_mark:Succeeded
View

his/install-and-config-guides/what-is-his.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Change from Decoder to Deserializer

I see that passing from version 0.8 to 0.9 the ConsumerProperties are changed to take a org.apache.kafka.common.serialization.Deserializer instead of a kafka.serializer.Decoder.
It seems that the Deserializer API is a little more general, but I would like to understand the rationale behind the change. Is it due to an upstream change - that is, Kafka 0.9 now takes Deserializers instead of Decoders? I see no deprecation in the Kafka Scala API.
Moreover, if I understand correctly, the namespace kafka holds the Scala API, while the namespace org.apache.kafka holds the Java API. Does this mean that Kafka is now trying to unify the two APIs?
It is no practical problem for me to use one or the other, but I was curious what is the reason for this change, and if it has significant consequences

@andreaferretti with passing to 0.9 we wanted to get independent from Kafka itself and use only the new client API ("org.apache.kafka" % "kafka-clients” % “<IP_ADDRESS>”). The new client is recommended for accessing Kafka in 0.9, we also become lighter without pulling the whole Kafka library.

Thank you, I understand. I was a little puzzled why the new API requires a java.util.Map in the configure method, while until version 0.8 Scala was a first class citizen in Kafka world