huggingface-cli delete-cache --disable-tui improvements

I tried huggingface-cli delete-cache --disable-tui for the first time. Great intention, very problematic usage when one had thousands of hub objects to cleanup. Once I understood its quirks I was able to hack around those problems.

Confusing UX

huggingface-cli delete-cache --disable-tui
TUI is disabled. In order to select which revisions you want to delete, please edit
the following file using the text editor of your choice. Instructions for manual
editing are located at the beginning of the file. Edit the file, save it and confirm
to continue.
File to edit: /tmp/tmpundr7lky.txt
0 revisions selected counting for 0.0. Continue ? (y/N)

The doc says to hit y but the program exits if y is hit and all careful manual editing is lost and the user has to start from scratch (ouch!)
I suspect this is a bug or a problem in the workflow.

Please don't use a true temp file, use a file that won't get deleted and a user can re-use it should they hit the wrong button - see Issue 1 above as an example.

sorting to have main last consistently would help. e.g. the first few entries had a consistent-main-last listing as in:

# Model mistralai/Mistral-7B-v0.1 (29.5G, used 19 hours ago)
    5e9c98b96d071dce59368012254c55b0ec6f8658 # Refs: (detached) # modified 3 months ago
    17688039ea7b7001c702797eed2ab7716a0cc3c2 # Refs: (detached) # modified 6 weeks ago
#    26bca36bde8333b5d7f72e9ed20ccda6a618af24 # Refs: main # modified 6 weeks ago

so I started to manually uncomment lines thinking main is always last, w/o paying close attention, but luckily I caught this was inconsistent as then I run into:
# Model mistralai/Mixtral-8x7B-v0.1 (93.4G, used 10 hours ago)
#    985aa055896a8f943d4a9f2572e6ea1341823841 # Refs: main # modified 5 weeks ago
    58301445dc1378584211722b7ebf8743ec4e192b # Refs: (detached) # modified 5 weeks ago

and many other variations. For those who need to edit hundreds of these, it'd be great to have main first or last - probably actually first would be the easiest.

give a user a way to delete all non-main revisions in one go

I tried to do it manually and it was super slow and I was concerned my edits will get lost again if I hit Y instead of the confusing N (see Issue 1)
at the end I resorted to this hack:
cp /tmp/tmpedbz00ox.txt cache.txt
perl -pi -e 's|^#(.*detached.*)|$1|' cache.txt
cat cache.txt >>  /tmp/tmpundr7lky.txt

and hit N, Y, Y
so I wiped out hundreds of old revisions in a second w/o manual editing.
This is usually what users want - keep the main, get rid of old revs - would it be possible to create such option?
Thank you!

Thanks for the great feedback @stas00! I haven't looked into this for a long time but all suggestions seems to make sense. Will keep you updated when I start working on that :)

Super! The foundation is awesome, @Wauplin - just needs a bit of polish on top.
The other problem this tool isn't take care of is cleaning downloads/ - I checked and we had 2M of files there! I had to do:
sudo find /data/huggingface/datasets/downloads -type f -mtime +3 -exec rm {} \+
sudo find /data/huggingface/datasets/downloads -type d -empty -delete

I'm not sure if huggingface-cli could take care of datasets as well, since they come from the hub. Please let me know if I should open a separate issue, since it's related but not the same.

Better for the datasets repo but please tag me. Thanks!

Done: https://github.com/huggingface/datasets/issues/6614

Hey @Wauplin, thanks a lot for pointing to another issue i could work on! I could not yet fully get into it, but i think i can help out here and would love to work on this! As soon as i was able to work out something i will open a draft PR (or some clarifying questions 😄). Will probably be next week.

Hey @Wauplin, found finally some time to look into this (later than expected, sorry about this). Not sure if i grasped this all correctly, so i try to lay out the way forward to avoid confusion.
Suggestion for moving forward from here:

Implement a new subcommand structure (from PR #2221 )

> huggingface-cli cache delete-repo repo_id
> huggingface-cli cache delete-repo repo_id --repo-type=dataset
> huggingface-cli cache delete-repo repo_id --include (glob)
> huggingface-cli cache delete-repo repo_id --exclude (glob)
> huggingface-cli cache delete-repo repo_id --files=file1,file2,file3,file4

Mark the delete-cache command as deprecated in the documentation and in the CLI itself

Improve the temporary file handling

Replace the current temporary file with a fixed file (e.g., ./tmp_hf_cleaner.txt) when TUI is disabled
Ensure that user edits are not lost if they accidentally choose the wrong option

Enhance revision selection

Add an option to select all "detached" revisions at once, keeping only main or specific tags
Implement consistent sorting of revisions, preferably with the main revision always in the same position (first or last)

Clarify the confirmation process

Revise the wording when confirming deletions to avoid confusion
Ensure that the confirmation process is consistent and intuitive

Address the .DS_Store issue

Fix the problem where hidden files like .DS_Store prevent the complete deletion of directories

Is this correct? If yes i would start by implementing the new subcommand structure and the options in separate PR's.

Hi @lappemic, thanks for getting back to me.  I don't think 1. and 5. should be tackled as part of this issue but rather in https://github.com/huggingface/huggingface_hub/pull/2221 directly. I find them a bit unrelated to 2. 3. and 4. which are focused on improving the current workflow.
I think that 2. and 4. can be tackled as part of the same PR while 3. (which adds a new option) can be done in a separate one. What do you tihnk?

Thanks a lot for the feedback! Sounds good to me! Will open a draft PR next week as soon as i get to it!


Better template system

Hello!
Now we have great template system, but it can be better!
These are examples things that we can add:
{% extends "base.html" %} - this function override template base.html.
{% block "content" %} - this create block, that can have content of other file using extends
{{ block.super }} - when you write after this, you add code to previous block, not replace it
{{ value|filter }} - filter can be for example lenght, default
Ideas are from https://docs.djangoproject.com/en/4.0/ref/templates/language/.

I'm gonna start working on it if you want.
I'll make a draft PR and use your comment to keep a trace of what you wanted.
Feel free to ask / post review about the current way of this features :p


Bouncy chat box

https://imgur.com/a/XrEiOL7 It happens seemingly randomly as well, not just when changing tabs. It also happens on my main client which has completely different client stretch/size settings. It stops when I disable the plugin.

same

This happens because, for whatever reason, the client re-creates the chatbox every time you switch tabs. I'm not sure if it's possible to fix but I'll leave this issue open just in case

It also scrolls the chat up, which I find even more annoying then the flickering


Subalbums do not obey sort settings

Detailed description of the problem
The Top level albums do sort based on settings, but sub albums do not. To test this create a top level album then several sub albums. Create the sub albums in random order, like b, a, z, y and then follow the steps below. They should sort like a, b, y, z but don't.
Steps to reproduce the issue
Steps to reproduce the behavior:

Go to Settings
Set Album sort to Title Ascending
Click Change Sorting
Go to album with subalbums
See that the sub albums are not sorted correctly

Screenshots

Output of the diagnostics
Diagnostics
-----------
Info: Latest version of PHP is 7.3
Warning: 'dist/user.css' does not exist or has insufficient read/write privileges.
Warning: 'dist/' has insufficient read/write privileges.
Warning: Dropbox import not working. dropbox_key is empty.

System Information
------------------
Lychee-front Version:      3.2.16
Lychee Version (git):      0dfad04 (master) - Up to date.
DB Version:                040000
System:                    Linux
PHP Version:               7.2
MySQL Version:             10.1.41-MariaDB-0ubuntu<IP_ADDRESS>

Lychee total space:        69.15 MB
Upload folder space:       14.93 MB
System total space:        48.68 GB
System free space:         6.53 GB (13%)

Imagick:                   1
Imagick Active:            1
Imagick Version:           1687
GD Version:                2.2.5

Config Information
------------------
version:                   040000
check_for_updates:         1
sorting_Photos_col:        takestamp
sorting_Photos_order:      ASC
sorting_Albums_col:        title
sorting_Albums_order:      ASC
imagick:                   1
skip_duplicates:           0
small_max_width:           0
small_max_height:          360
medium_max_width:          1920
medium_max_height:         1080
lang:                      en
layout:                    1
image_overlay:             1
image_overlay_type:        exif
default_license:           none
compression_quality:       90
full_photo:                1
delete_imported:           1
Mod_Frame:                 0
Mod_Frame_refresh:         30000
landing_page_enable:       0
landing_owner:             Liz & Gary
landing_title:             John Smith
landing_subtitle:          Cat, Dogs & Humans Photography
landing_facebook:          https://www.facebook.com/JohnSmith
landing_flickr:            https://www.flickr.com/JohnSmith
landing_twitter:           https://www.twitter.com/JohnSmith
landing_instagram:         https://instagram.com/JohnSmith
landing_youtube:           https://www.youtube.com/JohnSmith
landing_background:        dist/cat.jpg
thumb_2x:                  1
small_2x:                  0
medium_2x:                 0
site_title:                The Adventures of Liz & Gary
site_copyright_enable:     1
site_copyright_begin:      2019
site_copyright_end:        2019
public_search:             0
additional_footer_text:
display_social_in_gallery: 0
public_recent:             0
recent_age:                1
public_starred:            0
downloadable:              0
photos_wraparound:         1

Browser and system
N/A

@kamil4 is this fixed ?


Error after install running org-wiki-index

I have created ~/org/wiki -- this exception is thrown. Seems to effect anything that attempts to read a file. I installed via the bootstrap method and then tried a manual install. No difference.
Debugger entered--Lisp error: (wrong-type-argument stringp nil)
file-name-as-directory(nil)
org-wiki--page->file("index")
org-wiki--open-page("index")
org-wiki-index()
funcall-interactively(org-wiki-index)
call-interactively(org-wiki-index record nil)
command-execute(org-wiki-index record)
execute-extended-command(nil "org-wiki-index" "org-wiki-index")
funcall-interactively(execute-extended-command nil "org-wiki-index" "org-wiki-index")
call-interactively(execute-extended-command nil nil)
command-execute(execute-extended-command)

I also tried multiple forms of installation (bootstrap, manual) and am having same exception throw.

I have the same problem too.
Emacs 25.3.1 and Org mode version 9.1.1

I had the same issue, but it got fixed after I set the org-wiki-location in my .emacs file
(setq org-wiki-location "~/org/wiki")


Update Meta “tags”

Automatically generated by Netlify CMS

:construction_worker: Deploy Preview for lempsfoundation processing.
:hammer: Explore the source changes: 8fd2e5ad22a12d1c17c1f9a22da153db89d1e06f
:mag: Inspect the deploy log: https://app.netlify.com/sites/lempsfoundation/deploys/60c935b4929ebb0007a233e5


🔬 Funding objects hold contributor/affiliation objects directly

Previously Contributors and Affiliations could be defined and referenced from anywhere in the frontmatter, e.g. you could do this:
author: jdoe
funding:
  investigator:
    id: jdoe
    name: John Doe
    orcid: 0000-0000-0000-0000

Or
author:
  id: jdoe
  name: John Doe
  orcid: 0000-0000-0000-0000
funding:
  investigator: jdoe

Similarly, this could be done with affiliations defined at the top level or nested on contributors anywhere. While flexible, this resulted in some unexpected behaviour, like affiliations from a funding living alongside author affiliations at the front of the document...
This PR attempts to provide more sensible behaviour by constraining what is allowed in frontmatter a bit.
Now, funding investigators/recipients/sources may reference existing contributors/affiliations, defined under authors/affiliations, but then they are just filled in with a copy of the object. They may also define their own investigators/recipients/sources as objects, but these objects are not available elsewhere; they are only part of the funding object.
(So - in the previous yaml snippets, only the second will resolve as expected. In the first, the author jdoe will not resolve.)
As a small positive side effect, this addresses #597 - recipients may now be institutions.

This PR is not an improvement to the data structures - it makes things more complex in the end. We should address the unexpected behaviour described above on the tex/jats rendering side, not on the data structure side.
Closing this PR, in favour of a simpler one that addresses rendering. I will also try to persist the fix for #597 which was added here.


Added React v16 peer dependency support

After upgrading to React 16 the redux devtools throws a peerDependency warning.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>|| ^15.3.0 but none was installed.
This fix it to stop this warning and allow downstream modules to build without peerDependency failures.
Linking to #381

What does prevent us from merging this PR?

We should merge it asap if all tests pass.

Great that this is done. When will it be installable via npm? Looks like v3.4.0 doesn't have these commits?

@ezmiller 3.4.1 is published.  and i can verify that it fixes the issue.


As a user, I need my message to be processed on its destination domain

Problem
See https://github.com/connext/nxtp/issues/1714
Subtask: proveAndProcess
We need to implement relayer transmission cycle for proveAndProcess
Ideas to solve this

Implementation plan:

[subgraph] track dispatch() messages at Origin domain subgraph
[subgraph] track proveAndProcess() messages at Destination domain subgraph
[cartographer] update and add new fields to Cartographer
[lighthouse] In the lighthouse, pull all the transfers from cartographers where processed: false | null, and call proveAndProcess() it

Once this will start working we will create a new user story from the dev perspective to add sanity checks for root submission, or additional checks depending on domains and have an ideal time around calling proveAndProcess() and polling.
Subtasks

[x] https://github.com/connext/nxtp/issues/1753
[x] https://github.com/connext/nxtp/issues/1791
[x] https://github.com/connext/nxtp/issues/1792
[x] https://github.com/connext/nxtp/issues/1793

Acceptance Criteria

slow path transfers are working.

completed


Upgrade TextField from the design system

Description
In the latest versions of the design system, the TextField component no longer supports number formatting, so we should install react-number-format again and maintain number formatting in app-frontend.
When doing this, check the fillOutChangeName() which fills out the Grid percentages. This test was flaky because of number-formatting (and/or default values we got from the server), so when upgrading the design system we should make sure number-formatting does not lead to flakiness in those tests.

The TextField component was upgraded from LegacyTextField to the Designsystem TextField component in https://github.com/Altinn/app-frontend-react/pull/2213


CAMEL-11893: Restored SolrAddBeanTest in camel-solr

Fixed most ERROR level logs in target/camel-solr-test.log.
Aligned jetty version with the one used by solr-test-framework.
This PR fixes a single test, more to come.
See CAMEL-11893 for more.

Really nice and useful work. Thanks @aldettinger

Thanks @oscerd :smile: More to come, stay tuned.

Thanks all. Merged on master.


Documentation needs to be sanitized

Documentation of Table.parse_to_columns is currently rendered like this:

The white rectangle is actually an <input> HTML element, because documentation includes the following sentence:
       If the marked groups are named, the names will be used otherwise the column
       will be named `<Input Column> <N>` where `N` is the number of the marked group.

All these angle brackets go to the parser output unsanitized.

We plan to switch the documentation format to Markdown. For now, let's replace < and > in the doc comments.
Or quckfix it on GUI.


[ios/android] Add code format to result object

Thanks for the PR. Do Android and iOS both return the same values? I suspect there might be some small discrepancies right?

Thanks for the PR. Do Android and iOS both return the same values? I suspect there might be some small discrepancies right?

Yes, you're right, this is due to the features of the formats in the IOS, but most of popular formats are displayed the same

I think to be consistent we should return the format the same way as in the input formats: https://github.com/capacitor-community/barcode-scanner#target-only-specific-barcodes
Do you happen to have the ability to edit this PR so that it supports that.
For reference you could take a look at, and use these maps in the code: Android and iOS

@tafelnl I changed my PR using the names of this map

Awesome, I'll test it out and merge! Thanks again.


The program doesn't work when 87580.dat appears before 3068b.dat

Describe the bug
I use this program to display the instructions of my models on  the website. However, I found it doesn't work with some of my models. When work with these models, the program will throw exceptions at the specific steps.
I debugged it and found that if 87580.dat appears before the 3068b.dat in a LDR file, no matter whether they are in the same step, the program will throw out exceptions in the console, and can not display the step correctly. if 87580.dat has appeared in a LDR file, the program will always throw the same exceptions at the step where 3068b.dat appears. I don't know whether there are other parts that could cause this bug.

When 87580.dat has appeared before, and the next step contains 3068b.dat, I click the next step button, and got:
LDRGeometries.js:134 Uncaught TypeError: Cannot read property 'length' of undefined
    at LDR.LDRGeometry.buildGeometries (LDRGeometries.js:134)
    at THREE.LDRPartType.generateThreePart (LDRLoader.js:2192)
    at THREE.LDRStep.generateThreePart (LDRLoader.js:1693)
    at LDR.StepHandler.nextStep (LDRStepHandler.js:238)
    at LDR.InstructionsManager.realignModel (LDRInstructionsManager.js:605)
    at LDR.InstructionsManager.nextStep (LDRInstructionsManager.js:838)
    at nextStep (LDRInstructionsManager.js:37)
    at HTMLDivElement.<anonymous> (LDRButtons.js:178)

When I got the exception above and continue clicking the next button, I continuously got:
Uncaught TypeError: Cannot read property 'min' of undefined
    at t.e.copy (three.min.js:2)
    at LDR.StepHandler.setCurrentBounds (LDRStepHandler.js:553)
    at LDR.StepHandler.nextStep (LDRStepHandler.js:239)
    at LDR.InstructionsManager.realignModel (LDRInstructionsManager.js:605)
    at LDR.InstructionsManager.nextStep (LDRInstructionsManager.js:838)
    at nextStep (LDRInstructionsManager.js:37)
    at HTMLDivElement.<anonymous> (LDRButtons.js:178)
e.copy @ three.min.js:2
LDR.StepHandler.setCurrentBounds @ LDRStepHandler.js:553
LDR.StepHandler.nextStep @ LDRStepHandler.js:239
LDR.InstructionsManager.realignModel @ LDRInstructionsManager.js:605
LDR.InstructionsManager.nextStep @ LDRInstructionsManager.js:838
nextStep @ LDRInstructionsManager.js:37
(anonymous) @ LDRButtons.js:178

Reproduce
I made a simple LDR file that only contains the two parts and two steps. It can always reproduce this bug. You can get the two parts(they are from LDraw library) and the ldr file from the attachment.
Screenshot

Additional context
You can click the follow link to see this bug. I've deployed them on my website. it will occur at step 7.
(http://test.take-build.com/hat_bug.html)[http://test.take-build.com/hat_bug.html]
bugreport.zip
It would be appreciated if you could fix this problem. Thank you!

Reproduced on master. The LDraw file for 87580 consists of a 3068b and a "stud2".
A workaround is to disable the primitive cleanup:
          cleanUpPrimitivesAndSubParts: false

Fixed on latest version of master.


Rename $DB_PORT_5432_TCP_PORT variable

I just deployed a Mattermost docker container and I lost some time figuring out why the container wasn't connecting to the DB. I used DB_PORT instead of unintuitive DB_PORT_5432_TCP_PORT in my docker run.
I think this variable should be renamed to something user-friendly with less redundancy, why not DB_PORT, DB_TCP_PORT or DB_PORT_TCP ? In the config.template.json the replaced string is DB_PORT anyway.
Also, those variable should be displayed in the echo of docker-entry.sh
echo "Wait until database is ready..."

could be:
echo "Wait until database $DB_HOST:$DB_PORT_5432_TCP_PORT is ready..."

It would make error diagnose a lot easier ;)

Good point. $DB_PORT_5432_TCP_PORT is a legacy from docker-compose version 1 which is deprecated and we don't need to use it. I'll try to make it better.

Oops, we are still using docker-compose version 1.
We have used DB_PORT_5432_TCP_PORT instead of DB_PORT because what we want is only the port number, instead of the full url, to construct the url to the postgres db.


Generate strings of length mentioned in JPA Constraints

Class A {
@Column(
name="number"
length=5
)
getter....
setter.....
 How to overcome this?

JPA support can be added as a separate extension similar to the Bean Validation module.
There is no plan to make ER depend on the Java Persistence API (for the same reason as in #499). If you are interested in developing and maintaining such a module, please let me know and I will add a reference to it in the README. Thank you.


Remove Edge doesn't remove Edge

Describe the bug
The 'Remove Edge' feature doesn't remove edge at all.
To Reproduce
Steps to reproduce the behavior:
Just checking the box in the 'Tweaks' section
Expected behavior
Maybe I didn't understand it correctly, but I expected it to remove Edge completly. Maybe it just stops it from running in the background. If you could elaborate that would be great.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

I am reporting the same issue. I just performed a fresh install of Windows 10 in Virtualbox. I have ran the tweak tool to remove edge then performed reboot. Edge is still installed.

I have the same issue with my Windows 10 desktop (AMD CPU, NVIDIA GPU), but not with my QEMU/KVM windows 10 vm that runs on my laptop (Intel CPU). When I ran the script on my desktop, it did not remove edge, but when I ran it on my vm, it seemingly removed it (Doesn't launch, icon is empty).

does anyone know how to remove this:

Fixed in commit https://github.com/ChrisTitusTech/winutil/commit/b2e3def5116c391d84b47f6c38f108290ae833bd


Use WinAppSDK's own Helix queues instead of the WinUI/XAML queues

Now that the Windows App SDK project has it's own Helix queues setup, we can switch to using them instead of borrowing from the WinUI/XAML team's queues.
This change also adds 20H2 to the set of images being used for testing as well. (Previously we couldn't test on 20H2, as there was no "open" XAML queue to use for it).

/azp run


Nil pointer dereference in metrics exporter

I recently observed this on 0.21.2 while bootstrapping a brand new installation. FWIW, it was "self healing" after a few minutes: ongoing reconciliation operations seemed to bring the installation(s) into a state that caused the status to no longer be nil.
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x198 pc=0x1370d8e] goroutine 1 [running]:
github.com/altinity/clickhouse-operator/pkg/apis/metrics.(*Exporter).DiscoveryWatchedCHIs(0x16c6661?, {0x18f6338, 0xc00002e6e0}, 0xc0000317f0) /clickhouse-operator/pkg/apis/metrics/exporter.go:295 +0x3ee
github.com/altinity/clickhouse-operator/cmd/metrics_exporter/app.Run() /clickhouse-operator/cmd/metrics_exporter/app/metrics_exporter.go:108 +0x36b
main.main() /clickhouse-operator/cmd/metrics_exporter/main.go:23 +0x17

https://github.com/Altinity/clickhouse-operator/blob/32ef0fadc3d07884845a1ee3d91be6b6e112ac62/pkg/apis/metrics/exporter.go#L295
My guess is that we can fix this by changing the access of chi.Status.NormalizedCHICompleted to chi.EnsureStatus().GetNormalizedCHICompleted(), or something like that. I can't do it right this moment, but I'll circle back to this issue when I get some bandwidth.
To generalize: maybe we want to use Golang visibility to make raw/unsynchronized fields on Status package-internal to more strongly encourage usage of safe public getter/setter APIs. I considered this in #1119 but IIRC, this resulted in problems when trying to deserialize/serialize the state of Status (because it relied on field visibility)... I think?

Let's close this if the build with #1188 passes? I'm not sure how deterministic it is to repro, but one thing worth trying is just bringing up a fresh installation and seeing if that reproduces on 0.21.2 – then trying again with 0.21.3

@zcross , this is fixed in 0.21.3

Thank you!

@zcross , this is fixed in 0.21.3

Would it be possible to cut a release tag for 0.21.3 to distribute this fix?


nixos/cosmic: add keybinds and otherSettings options

closes #97.
this creates a keybinds and otherSettings option for options.services.desktopManager.cosmic.
keybinds shall be defined by a list of anonymous attrsets of the form:
{
  key: str | null
  modifiers: [str]
  action: str | submodule { type: str, data: int | str }
}

the key is nullable due to ability to bind presses of modifier keys, ie. hitting super to open the launcher in default bindings.
otherSettings will allow setting of cosmic-config com.system76.Cosmic*/v?/opt_name-pathed RON encoded settings, as follows:
{
  version: str (default: "1")
  option: anything
}

option will run through a default serialisation routine depending on type.
example:
services.desktopManager.cosmic = {
  enable = true;
  keybindings = [
    {
      key = "q";
      modifiers = [ "Super" ];
      action = "Close";
    }
    {
      key = "r";
      modifiers = [ "Super" ];
      action = {
        type = "Resizing";
        data = "Outwards";
      };
    }
  ];
  otherSettings = {
    "com.system76.CosmicPanel.Dock" = {
      option.opacity = 0.8;
    };
  };
};

I will get to this PR soon, I promise. I've just been rather overwhelmed IRL 😅

this still works for me on the latest build but looks like /etc/cosmic-comp/config.ron may not be supported going forward. I'll wait for the cosmic update action to unblock and see if there's a new method I can adapt it to..

due to the changes with config location it looks like this will now be better as an HM module


How to read data？

I‘m using temperature sensor with 'one wire' protocol .
So how to read data with your GPIO code?
plz!

I don't think that's a good idea. Using GPIO bitbanging to interface a one wire bus inside flutter is probably going to be super unstable. Instead, I'd try interfacing the 1 wire sysfs drivers.

Thanks!
Closing the issue.


Engagement On Call Duty -  Sprint 93

Responsibilities and Acceptance Criteria for this sprint
One engineer per sprint is assigned on-call duty. It is that engineer's responsibility to know the "pulse" of BE issues/health during the sprint. This includes the following responsibilities:

[ ] Monitor poison queue messages and re-process if needed. Information on how to reprocess messages can be found here: https://github.com/CDCgov/prime-reportstream/blob/master/prime-router/docs/standard-operating-procedures/Poison-queue/poison-queue-management.md
[ ] Primary point-of-contact for any Engagement Support problems that occur during the sprint (including status of those issues)
[ ] Resolve pager-duty engagement alerts
[ ] Resolve all security (high-priority) Transport related Dependabots
[ ] Monitor missing mappings code-to-condition table. Information on how to check missing mappings can be found here: https://github.com/CDCgov/prime-reportstream/blob/master/prime-router/docs/standard-operating-procedures/update-observation-mapping-table.md
[ ] Monitor if recently onboarded STLTs to the UP received any messages for new conditions.
[ ] LIVD table updates

Update: Ott is currently monitoring the Pager notification and if there any issue he will fix it or notify other engineer to assist. As of 04.30.24 @ 9:48m EST two notifications has be reported and resolved.

Done.


[fr] 5-3-readme french translation

Add README.fr.md file for 5-3-background-tasks-and-performance

Hi

Hello there! Thanks for the review 🥳

merci!

Un plaisir :-)


Using relative path for clientRootFolder in jdl failed

Overview of the issue
I try to use a relative path for the clientRootFolder in my jdl-File. Import-jdl fails with the following error:
The JDL is being parsed.
error: Error message: unexpected character: ->/<- at offset: 4403, skipped 1 characters.
SyntaxError: Error: unexpected character: ->/<- at offset: 4403, skipped 1 characters.
Error while parsing applications and entities from the JDL SyntaxError: Error: unexpected character: ->/<- at offset: 4403, skipped 1 characters.
SyntaxError: Error: unexpected character: ->/<- at offset: 4403, skipped 1 characters.
at callApiMethod (\jhipster-core\lib\readers\jdl_reader.js:88:11)
at parse (\jhipster-core\lib\readers\jdl_reader.js:72:10)
at Object.parseFromFiles (\jhipster-core\lib\readers\jdl_reader.js:41:10)
at parseFiles (\jhipster-core\lib\jdl\jdl_importer.js:82:20)
at JDLImporter.import (\jhipster-core\lib\jdl\jdl_importer.js:64:30)
at JDLProcessor.importJDL (\generator-jhipster\cli\import-jdl.js:76:41)
at JDLProcessor.importJDL (\generator-jhipster\cli\import-jdl.js:296:38)
at module.exports (\generator-jhipster\cli\import-jdl.js:450:21)
at Command.command.allowUnknownOption.description.action.args (\generator-jhipster\cli\cli.js:72:36)
at Command.listener (\commander\index.js:315:8)
Motivation for or Use Case
As the path can be a custom value it should be possible to use relative paths.
Reproduce the error
Add a realtive path for clientRootFolder in jdl and run import-jdl:
clientRootFolder * with client
Related issues

JHipster Version(s)
6.3.1
JHipster configuration
{
"generator-jhipster": {
"promptValues": {
"packageName": "something",
"nativeLanguage": "de"
},
"jhipsterVersion": "6.2.0",
"applicationType": "microservice",
"baseName": "something",
"packageName": "something",
"packageFolder": "something",
"serverPort": "8081",
"authenticationType": "oauth2",
"cacheProvider": "no",
"websocket": false,
"databaseType": "sql",
"devDatabaseType": "postgresql",
"prodDatabaseType": "postgresql",
"searchEngine": false,
"messageBroker": false,
"serviceDiscoveryType": "consul",
"buildTool": "gradle",
"enableSwaggerCodegen": false,
"testFrameworks": [],
"jhiPrefix": "jhi",
"entitySuffix": "",
"dtoSuffix": "DTO",
"otherModules": [],
"enableTranslation": true,
"clientPackageManager": "npm",
"nativeLanguage": "de",
"languages": [
"de"
],
"blueprints": [],
"skipClient": true,
"skipUserManagement": true
}
}
Environment and Tools
java version "11.0.4" 2019-07-16 LTS
Java(TM) SE Runtime Environment 18.9 (build 11.0.4+10-LTS)
Java HotSpot(TM) 64-Bit Server VM 18.9 (build 11.0.4+10-LTS, mixed mode)
git version 2.16.2.windows.1
node: v9.8.0
npm: 6.11.3
yeoman: 3.1.0
Docker version 19.03.2, build 6a30dfc
docker-compose version 1.24.1, build 4667896b
Browsers and Operating System
Win10

[ x] Checking this box is mandatory (this is just to show you read everything)

Hi, please provide the full JDL file if you can. If you can't, you can send it to my email.

I'd like to send it per email but I can't find you email address. :/

<EMAIL_ADDRESS>It's in the package.json file :)

I've received it, I'm gonna test the parser with your file.

Quick update: I've tried importing it with the latest parser version: no error.
Also tried with the JDL Studio from JHipster Online, no error.
What version of JHipster are you using?

Should be 6.3.1

You may have two versions installed instead of one... While I don't understand why you have this issue, it may cause your problem (not sure but worth checking).
I suggest you head over there, removing your current installation of the generator and reinstall it.

I doubled checked the JDL I sent to you. I forgot to add the relative path. So I will send you the correct version. Please can you try to import again? Sorry for the circumstances. I have removed the relative path so that I was able to work with the JDL.

Roger that, I'll try again

Thank you :)

This was a bit painful, but this is fixed :) thanks for reporting this one!

Thanks for your help. :)

One last question: Do you know when a new release of JHipster is planed? I'm looking forward to get this fix ;)

I don't know, there's gonna be a major version though (for this repo).

Thanks


[Support] :exp cli on NPM v5.x

Can you guys please support for exp cli on NPM v5.x . Its a pain every time to change NPM version or use expo XDE tool .as all task can't be done using XDE Please look into it .
I am sharing the error log below
ngrok - error unpacking binary { Error: EACCES: permission denied, open '/usr/local/lib/node_modules/exp/node_modules/@expo/ngrok/bin/ngrok'
  errno: -13,
  code: 'EACCES',
  syscall: 'open',
  path: '/usr/local/lib/node_modules/exp/node_modules/@expo/ngrok/bin/ngrok' }
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules/exp/node_modules/@expo/traveling-fastlane-linux):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"win32,linux","arch":"any"} (current: {"os":"darwin","arch":"x64"})

Hi! This doesn't look like an npm 5 issue to me, although if it doesn't happen with yarn, then I guess that's your answer. Regardless, make sure your directory permissions for your global node_modules are sane (owned and writable by your user accounts). In terms of supporting npm 5, the change broke a lot of things, not all of which have been fixed in recent npm releases. We'll try to figure out what we can do soon, but because there are other options (npm 4, yarn), it's honestly not the highest priority thing we're trying to fix.
Also, as the issue template requests, please open issues against https://github.com/expo/expo.


[KFNBC]: Access Jupyter notebook server requires another login action

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
Current process to access a Jupyter notebook server

Log in RHODS Dashboard using credentials of UserX
Spawn the desired notebook image from the spawner
click "Access server" button after the notebook server spawning process is done
Log in Jupyter once again using credentials of UserX

Issue with the current behavior
User is already logged into the dashboard. Users will perform login in Jupyter everytime they stop and start a new notebook image (UX affecting)
Expected Behavior
If a user spawns a notebook image and they are already logged in, then the log in should not be requested once again when trying to access the server
Steps To Reproduce

Log in RHODS Dashboard using credentials of UserX
Spawn the desired notebook image from the spawner
click "Access server" button after the notebook server spawning process is done
Log in Jupyter once again using credentials of UserX

Workaround (if any)
No response
OpenShift Version
Other
Openshift Version
OSD 4.10
What browsers are you seeing the problem on?
Firefox
Open Data Hub Version
No response
Relevant log output
No response

This is the expected behavior by design in Openshift, OAuth proxy does not provide SSO capabilities.
We are exploring alternatives to provide a SSO experience for ODH components, tracked in RHODS-2130


Default tasks.json "echo Hello" does not work

VSCode Version: 1.14.0
OS Version: Windows 10
Terminal: Git Bash

My default terminal is "terminal.integrated.shell.windows": "C:\\Program Files\\Git\\bin\\bash.exe".
For some reason that I do not understand, arguments are either not passed to bash correctly, or bash does not understand them. The default command is "command": "echo Hello", but bash acts as if it just received echo (just prints a newline). Admittedly, I am new to vscode and I've never used tasks before 1.14, so this might just be user error or the fact I'm using bash. However, I figure the default case should work.
This bug prevents me from creating a build task for Haskell stack. I would like to do "command": "stack build", but all bash sees is stack. My workaround is to do "command": "\"stack build\"".
Steps to Reproduce:

Delete (or rename) old tasks.json.
In the top menu select "Tasks > Configure Tasks".
In command palette drop-down select the last item, "Others: Example to run arbitrary external command".
In the top menu select "Tasks > Run Tasks..." and choose echo from the palette drop-down.
In the command palette drop-down select the first item "Continue without scanning the build output".
Check the Tasks terminal and notice the lack of Hello.

tasks.json:
{
    // See https://go.microsoft.com/fwlink/?LinkId=733558
    // for the documentation about the tasks.json format
    "version": "2.0.0",
    "tasks": [
        {
            "taskName": "echo",
            "command": "echo Hello",
            "type": "shell"
        }
    ]
}

Output:
> Executing task: echo Hello <

Terminal will be reused by tasks, press any key to close it.

This is also reproducible with using args, for example, the following doesn't work:
{
    // See https://go.microsoft.com/fwlink/?LinkId=733558
    // for the documentation about the tasks.json format
    "version": "2.0.0",
    "tasks": [
        {
            "taskName": "echo",
            "command": "echo",
            "args": ["Hello"],
            "type": "shell"
        }
    ]
}

This can be fixed by passing the entire command in escaped quotes.
{
    // See https://go.microsoft.com/fwlink/?LinkId=733558
    // for the documentation about the tasks.json format
    "version": "2.0.0",
    "tasks": [
        {
            "taskName": "echo",
            "command": "\"echo Hello\"",
            "type": "shell"
        }
    ]
}

Output:
> Executing task: "echo Hello" <

Hello

Terminal will be reused by tasks, press any key to close it.

Reproduces without extensions: Yes

Very likely a dup of #28036

This bug prevents me from creating a build task for Haskell stack.

@0not have you considered implementing a TaskProvider for the Haskell stack tool https://code.visualstudio.com/updates/v1_14#_new-task-api. The benefit of a TaskProvider for stack is that user will not have to customize a task.json file.

Fixed for Git bash and WSL bash


Support for subfolder files?

For example I have openApi.json and sub folder
"parameters" : [ {
          "$ref" : "../common/components/headers/contentType.json"
        }

I would like to have all files related to the openAPI.json in a big file.

@asyba
I couldn't understand what your trying to achieve.
Can you please provide little bit in-depth explanation for what your trying to achieve.

@asyba
I couldn't understand the problem your tying to solve.
Can you please provide little bit in-depth explanation for what your trying to achieve.

I have java project with openAPI.json files but some specifications are not in the root file, there are in sub-folder.
For example a response.json is used across different openAPI.json files so I created one common file and use $ref.
"parameters" : [ { : {           "200" : {             "$ref" : "../common/components/responses/xxxxx.json"           },
To be able to import the openAPI.json in to Postman for example, doesn't work if the openAPI.json does't contain all the data.
I need to able to have it all in one file.
Example of another github project https://github.com/APIDevTools/swagger-cli#combine-multiple-files
but they don't provide a gradle plugin task that I need in my case.

@asyba
ok understood the problem now.
What I think is
Its possible to combine multiple files into one file.
However after combining all files the following would become invalid
"$ref" : "../common/components/responses/xxxxx.json"
since file xxxxx.json which was in that path (../common/components/responses/) is now part of the merged file openapi.json.

@asyba
I think this feature can be cleanly implemented.
Will work on this one and add it to the next release.

Is this feature implemented @kpramesh2212.
Thanks for the plugin.

Will test and do a release as soon as possible

1.0.5 released


chore: initial deployment

Signed-off-by: Alexandre Gaudreault<EMAIL_ADDRESS>

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 54.89%. Comparing base (d5c25e7) to head (0211f70).

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #68      +/-   ##
==========================================
- Coverage   54.98%   54.89%   -0.09%
==========================================
  Files          10       10
  Lines        1133     1133
==========================================
- Hits          623      622       -1
+ Misses        434      432       -2
- Partials       76       79       +3

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Update Michael's affiliation (Cologne -> Stuttgart)

I think this can be merged since the docs have been built without errors and the other tests won't run.


Update FilterConfig.php

Fix pagination issue while using all operator but like

Please add changes to filter() methods of DataProvider classes to build correct SQL with new constants (just replace something like 'eq' to '=')


Allow summarise anywhere

@josevalim You can already play with it, but I still need to do some adjustments

https://user-images.githubusercontent.com/5660941/232553305-cbcd1d03-8b71-45ac-92b7-eead036f6b06.mp4
It respects grouped operations, so if we have two summarises in a row, the second one will have access to the data_options and the groups in the data frame prior to the summarization
https://user-images.githubusercontent.com/5660941/232554568-c1a11d06-8287-4a13-bc36-c111d5d77160.mp4

And what I have to do is basically make sure that I've covered all the edge cases. Any other suggestions, feel free!

This is an amazing job @cristineguadelupe! very well done, no additional comments. :)
The only think I ask is to slightly change the CSS so, if we have two operations of the same type in a row, the spacing between the operations is reduced (I think 4px as padding bottom will work). So far, our operations all behaved the same if they were grouped together or not, but that's not the case for "summarize", so we should align the UI accordingly.


Make disk handling more robust

Codecov Report
Attention: Patch coverage is 0% with 4 lines in your changes are missing coverage. Please review.

:exclamation: No coverage uploaded for pull request base (main@f77cf6f). Click here to learn what that means.

Files
Patch %
Lines

internal/device/recovery.go
0.00%
3 Missing :warning:

cmd/provision/main.go
0.00%
1 Missing :warning:

Additional details and impacted files
@@          Coverage Diff           @@
##             main    #142   +/-   ##
======================================
  Coverage        ?   0.00%
======================================
  Files           ?      19
  Lines           ?    1349
  Branches        ?       0
======================================
  Hits            ?       0
  Misses          ?    1349
  Partials        ?       0

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Add GitLab support

The program currently only supports GitHub, which is a non-free (free as in free speech) tool. Many developers find GitLab a better and a safer place to release their codes. It would be really great if this tool also supports GitLab along with GitHub. Currently if we paste a GitLab url in the tool, it generates an error saying Not a GitHub repository.

Fair enough. Can you share your project link on GitLab for me to test with?

Yes; sure. Here you go.
https://gitlab.com/niruvt/latexonline.cc/-/blob/main/test.tex


Remove notice about vulnerability?

The README still says this:

NOTICE: A vulnerability in JWT was recently published. As this library doesn't force users to validate the alg is what they expected, it's possible your usage is effected. There will be an update soon to remedy this, and it will likey require backwards-incompatible changes to the API. In the short term, please make sure your implementation verifies the alg is what you expect.

This vulnerability was published in 2015. If it has been addressed in go-jwt then maybe the notice should be removed?

I believe this refers to the None verification algorithm that is weirdly present in the JWT standard. This library still does not force the user to validate the alg sent in the token, and there is documentation encouraging users to do this. Additionally the None algorithm has been disabled default in this library here. It appears to be addressed, however I am no cryptographer. I would also appreciate clarity from the maintainer(s).

I've updated the note as it's still worth mentioning.  As of 3.0.0, key types are not compatible across signing methods, which really helps.  It's still possible to get this wrong, so it's worth keeping a mention of it up front.
This satisfactory to close the ticket?

This satisfactory to close the ticket?

IMO, yes.


add mfa step for connect api

are there things that can be moved to a shared class for auth and connect? https://github.com/Gusto/plaid-ruby/blob/master/lib/plaid/auth.rb

Accepted by jollysahil!


some problems with correct informations

hello i rented a vserver and wanted to use this tool but.. it gives me wrong informations....

the server got
4x AMD EPYC 7452 32-Core Processor, 2000 MHz
8gb of unkown ram so i think the 64bit is okay-ish but i think it should be DDR4 according to amds spec website of the CPU
and 300gb of space...
(edit) also Debian 10 buster not 11 O.o
i rented this server from https://www.strato.de/
(edit) i can send you a full hwinfo log if needed

are you running in docker or the jar file? and yeah send hwinfo log.

i run the jar file with "screen -AmdS srvmon java -jar ward.jar"
its a vserver from strato and they are using virtuozzo to do that
here the log file:
hwinfo.log
and here a screenshot of the server config i rented from there

please try the new build and maybe run under sudo?


When trying to invoke sidecar endpoint http://dummy.com/v1.0/invoke/echoapp/method/echo?text=hello
getting error
Cannot GET /v1.0/invoke/echoapp/method/echo

Recreated the setup again and it worked


fix: additional increase in button size due to #213

Fixes: fixes increase in button size due to pull request #213
Changes:
Removed the padding and margin from global button styles and applied them to local styles of specific buttons
Screenshots of the change:
Button size fix

Added padding to "View on Github" on CTA button, for horizontal alignment.

cc @qier222 to take a look

🤔

Oe Oe Oe! 🤣🤣
Run down your scales on this one, then 😅

Didn't get the statement "buttons are bigger than logo and slogan" 😕.  By slogan do you mean "A...Gateway" ?
And the button widths looks fine to me, they are big enough to drag user attention.
Do you have issues with the width of the "downloads" button ?

The height of the "View on GitHub" button is higher than the "A Dynamic  Cloud-Native API Gateway" text, and the "View on GitHub" is almost black, attract way too much attention than the slogan text and brand text.

Here is some example how other designers handle slogan and primary button.

How about this (initially I was trying not to change too much of the original design as he/she must have had their thoughts behind it, but now I feel you are fine with it. So here we go😄 )

I removed the Apisix title, as it looked repeated (written in navbar too), Increased the font size for highest focus and changed its color to match the color scheme.
Made the text wrap, as the sub head copywriting is too long, reader might lose his eye position in the middle.
Also made the buttons a little thinner, for best results
On mobile -

Not sure if those two buttons' position is good to view 🤔

BTW, I noticed that the picture on our homepage is hard to read (a little small) 🤔

Yup, your orgs website is failing on this point of accessibility.
The API architecture image is just not clear to see at the size which it is currently at.
Also the hero section feels empty at the moment 😢, and the copywriting on hero section and other sections can be made smaller and concise.

@juzhiyuan what's your feedback to these edits and the mobile screenshot in this one

How about this (initially I was trying not to change too much of the original design as he/she must have had their thoughts behind it, but now I feel you are fine with it. So here we go 😄 )

I removed the Apisix title, as it looked repeated (written in navbar too), Increased the font size for highest focus and changed its color to match the color scheme.
Made the text wrap, as the sub head copywriting is too long, reader might lose his eye position in the middle.
Also made the buttons a little thinner, for best results
On mobile -

Wait a bit, just let me push these changes to the branch

Guys have encountered a problem while pushing to this branch. Created a new pull request PR for this integration. Please merge that pull request after reviewing. @juzhiyuan @gxthrj

How about this (initially I was trying not to change too much of the original design as he/she must have had their thoughts behind it, but now I feel you are fine with it. So here we go😄 )

I removed the Apisix title, as it looked repeated (written in navbar too), Increased the font size for highest focus, and changed its color to match the color scheme.
Made the text wrap, as the sub head copywriting is too long, reader might lose his eye position in the middle.
Also made the buttons a little thinner, for best results
On mobile -

Sorry for the slow response.
I designed this page, I was planning to make the logo on the navbar hidden when the logo in the hero section is visible, but I haven't done this. I think you can remove the logo in the hero section now.
The "View on GitHub" and "Downloads" buttons are center-aligned inside and left-aligned outside, looks weird to me, I still think make these buttons full-screen wide is a better option. Maybe we can center align the slogan on mobile devices?

Also, how about making this part align from two sides?

How about this?

+1 Red & Black (keywords) would be more clear for users 🤔

@1502shivam-singh  What's your opinion?

@qier222 This one looks good to me.

How about this?

For the rest, you designed the page previously, and the buttons were left-aligned previously too with text-centered, so you must have had some thought behind it then. I am just following that flow, what I feel your thought would be is to keep the buttons left aligned because the section itself is left-aligned, the best example of a similar website-hero-section is the stripe website -

I guess we should first decide on one layout for the hero, then extend further on it. Full-width CTA buttons I have discussed earlier too for left-aligned sections. This I agree on that a center-aligned hero section would support full-width buttons better Github website is an example, but they also have it because one field requires an email for which it's full width and the button below it is full width to match the field width for uniformity.
Also, something I noticed is when this section is center aligned, the animating text breaks the layout every transition (Try it, you will see), it's just noisy seeing that "A" move around 😣.

Looks good on my phone🤔

Also, how about making this part align from two sides?

@juzhiyuan did you mean this -

Justify alignment can mess up things sometimes. 😅
To be honest we are solving an interesting design problem, which is to make a section look good with a minimal number of extra elements added, just typography.

Looks good on my phone🤔

I don't have any issue with keeping the buttons, not full width. It's just that we have to decide on one thing and follow it, not deviate from it. You designed the website previously and I was just respecting your judgment 😅. If all agree, and this is final then I am fine with it completely 😄.

Also, sometimes we should not just focus only on making the stuff "look good". Don't get me wrong, beautiful UI is Paramount, but I feel we should hit the middle ground between top-class UI and top-class UX.
The reason I gave for not to put full-width buttons is a UX focussed decision, mistouch is possible on full-width buttons, (Especially when you have a phone with a bad touch screen, as my brother's phone 😒, and you just don't change it for some weird reason and you code too. This leads to a bad UX for that person) and more chances when you have 2 of them.
We should try to avoid these issues, especially when they feel minimal. This is one reason why we don't see full-width buttons more often, if used they have some special reason for usage.
Also, the fact that we have to focus on button widths to make our hero section look good, is an indicator that our hero section lacks a focal point.

Also, sometimes we should not just focus only on making the stuff "look good". Don't get me wrong, beautiful UI is Paramount, but I feel we should hit the middle ground between top-class UI and top-class UX.
The reason I gave for not to put full-width buttons is a UX focussed decision, mistouch is possible on full-width buttons, (Especially when you have a phone with a bad touch screen, as my brother's phone 😒, and you just don't change it for some weird reason and you code too. This leads to a bad UX for that person) and more chances when you have 2 of them.
We should try to avoid these issues, especially when they feel minimal. This is one reason why we don't see full-width buttons more often, if used they have some special reason for usage.
Also, the fact that we have to focus on button widths to make our hero section look good, is an indicator that our hero section lacks a focal point.

I get your point, but I don't think use full-width buttons will have a large impact on UX. Your example is your brother's phone has a bad touch screen, it does not make much sense to me, is the phone make the UX worst, not the application.
Email app has an email list, and every list item is clickable. If a phone with a bad screen can cause mistouch the full-width buttons on a website, then it definitely can cause mistouch on an email list or any clickable list, does it mean we should stop using clickable list?

No no no, you are getting my point wrong here, I am not saying it that we should stop using clickable lists or surface that can click and we have to scroll over them. I am saying that if we can solve the problem for that particular person easily without much loss (phone has bad touch) then why not do it? The email list example is actually a problem for that user, but we can't do much about it as its designed already, and also this doesn't mean the app has bad UX.
There is a simple solution for the above problem, give user space with non-clickable elements, and this solution doesn't really hurt the structure of the hero section.
Sure, users can have the worst phones, with the worst problems and we will surely not be able to solve all of them. "But if we can solve the problems that are visible to us then why not solve them?" This is my point

Also, if I am right you have issue that the button is left aligned but the text is centered.

How about this (initially I was trying not to change too much of the original design as he/she must have had their thoughts behind it, but now I feel you are fine with it. So here we go😄 )

I removed the Apisix title, as it looked repeated (written in navbar too), Increased the font size for highest focus, and changed its color to match the color scheme.
Made the text wrap, as the sub head copywriting is too long, reader might lose his eye position in the middle.
Also made the buttons a little thinner, for best results
On mobile -

Sorry for the slow response.
I designed this page, I was planning to make the logo on the navbar hidden when the logo in the hero section is visible, but I haven't done this. I think you can remove the logo in the hero section now.
The "View on GitHub" and "Downloads" buttons are center-aligned inside and left-aligned outside, looks weird to me, I still think make these buttons full-screen wide is a better option. Maybe we can center align the slogan on mobile devices?

then I have a solution that can serve both of ours's needs.
Which is to Center align the buttons. Then your issue is solved and I have my white space too.
You like this @qier222 😁

Extra top margin to separate the left aligned slogan and subtitle with the centered buttons. Keeping them distant will make the hero section look better, as then alignment-flow wouldn't be a problem (both sections are separate now).

In my opinion, use full-width buttons in this hero section will improve 20% of UI, not use full-width buttons will improve 3% of UX, I won't trade 20% UI improvement for 3% UX improvement.

In m- y opinion, use full-width buttons in this hero section will improve 20% of UI, not use full-width buttons will improve 3% of UX, I won't trade 20% UI improvement for 3% UX improvement.

Those numbers are actually pretty ambiguous and biased towards your case. For some user they can be reversed too.
@qier222 By the way did you try my centered button solution above with some top margin. As I said, it solves of both ours's problems.
Still if you insist, so much I don't really have much of objection with full-width CTA, it's your call.

Let's hear from more contributors :) cc @imjoey @liuxiran @membphis @Yiyiyimu


Number of Clusters as the Max Eigenvalue?

Hi, is there a particular reason as to why you are setting the max(k) to be the number of clusters for SpectralClustering towards the end of the Verification Stage in the dave.py file?

In our work, we use spectral clustering in combination with the eigengap heuristic, based on this paper.
Overview:

Laplacian Matrix: Compute the graph Laplacian L = D - W, where D is the degree matrix and W is the adjacency matrix.

Eigen Decomposition: Perform eigen decomposition on L to obtain eigenvalues and eigenvectors.

Eigengap Heuristic: Determine the number of clusters by finding the largest gap between consecutive eigenvalues.

We modify the eigengap heuristic to set the number of clusters to match the last large enough eigengap.


add self-contained example for a docker router as a proxy sidecar

https://asciinema.org/a/673943

although it's fine, i dunno how i feel about committing stuff like this to the repo. i'm personally reluctant to commit stuff like this. it just feels like it doesn't quite belong

Moved to a Gist: https://gist.github.com/qrkourier/42ccab992d3c74e2ada84092909966f5


[BEAM-1832] Fix for potentially unclosed streams in ApexYarnLauncher

[BEAM-1832] Fix for potentially unclosed streams in ApexYarnLauncher

Using try-resources to handle closeable close issue
Removed redundant semicolons, string null assignment
Indent/spaces as per beam codestyle. Checkstyle passes. mvn clean verify passes.

Coverage increased (+0.003%) to 69.817% when pulling 46700c648b9e0083477bfcb649fe797662b78820 on rekhajoshm:BEAM-1832-0 into 37e4cc1b876bd990d289b1cc2ec24b8d9e53c2d8 on apache:master.

@rekhajoshm I don't think it is desirable to reformat entire files to make relatively small changes. It looks like large chunks of code moved around without change. Can you please submit the first 2 items as separate PR that only contains those changes? Thanks!


BCC CyHy reports and CyHy notifications to the appropriate CSA

🗣 Description
This pull request makes the necessary changes to BCC CyHy reports and CyHy notifications to the appropriate CISA Cyber Security Advisor (CSA).
💭 Motivation and context
Partly resolves cisagov/cyhy-system#114.  See also cisagov/cyhy_amis#749.
🧪 Testing
All automated tests pass.  I did a full dry run (i.e., did everything except actually tell AWS SES to send the email) of sending the CyHy reports and everything worked as expected.
✅ Pre-approval checklist

[x] This PR has an informative and human-readable title.
[x] Changes are limited to a single goal - eschew scope creep!
[x] All relevant type-of-change labels have been added.
[x] I have read the CONTRIBUTING document.
[x] These code changes follow cisagov code standards.
[x] All relevant repo and/or project documentation has been updated to reflect the changes in this PR.
[x] All new and existing tests pass.

✅ Pre-merge checklist

[ ] Finalize version.

✅ Post-merge checklist

[ ] Create a release.

@dav3r - Please don't merge upstream changes like that if you can help it. It creates an unnecessary merge commit.  Please rebase instead.

These changes have been manually deployed to Production - the following files were updated with the changes from this PR and #104:

docker-compose.yml
docker-compose.cyhy.yml
docker-compose.cyhy-notification.yml
secrets/csa_emails.yml


Improved visual indication for disconnected inputs

Currently when a block has disconnected inputs a whole selection of blocks affected get highlighted yellow.
This can be further improved by:

Displaying a visual notification that the block has disconnected inputs
Maybe only highlight the affected block with disconnected inputs and not all of them.
Changing the highlight, currently the error highlight looks too similar to regular block selection (Previously issue #166)

I like this change so far!  But I would like it even more if:

The warning icon appeared immediately when placing the block (but the popout only appeared when pressing Run.
The warning didn't go away on its own after time.
This same warning didn't get confusingly re-used for situations where there's no top-level block.

Will follow up with specific issues for any remaining details.


Make app settings use environment variables

This PR changes the test project to allow app settings to be over-ridden by environment variables.

Closing as the PR was opening by accident


[PT-BR] Namespaces, Symbols

#233
Add translation to PT-BR to the following files:
Namespaces.md
Symbols.md

Commenting to subscribe @danilofuchs @alvarocamilont

lgtm


scrolling projector view is very slow

create an election with e.g. 25 candidates and project it
in the projector all candidates are listed, since the list is too long to fit into the projector view, scroll down the projector view

result: it takes on my PC 8 seconds until the scrolling is finished: The first 6 second nothing happens (the browser is even frozen during this time) and then it needs 2 second to scroll. On my Laptop it takes not 8 but 12 seconds.
It would be nice if this could be improved for OS 2.1 final.

This bug can also reproduced with an agenda that has e.g. 25 entries: project the whole agenda and then scroll the projector view.

Can't reproduce with current master. Please retest it with 2.1 (final). Could be fixed with #3111.

Please retest it with 2.1 (final). Could be fixed with #3111.

Please reopen. Unfortunately it is not fixed. Scrolling is still unacceptable slow (as slow as reported). Too bad that this cannot be sped up fore OS 2.1.


implement tabs for Team review state

Picks off another todo on #3704.

How did I get here? Oh, yes: the N=75 was a lie since we were in fact also listing non-approved Teams. The N on the welcome popup matched the N of the page itself, and I think it should. Since we do want to include under review and rejected Teams on the anon homepage (openness! :), and we don't want to confuse first-time visitors with an N (and a page, really) that includes unapproved Teams, we had to unlie the N another way: by moving unapproved Teams off the main page.

This abstracts the tab infrastructure that @rohitpaulk introduced for Giving.

Blorg. I forgot a file and force-pushed the merge off master. New PR is #3741.


Sort on Pivot only

Hi there,
Not sure if this intended behaviour. But I want to sort only on relationships, so when it is on relationship/pivot table.
This below will return true even if it's not viaRelationship as the second property is set true in my model.
$sortable  = $request->viaRelationship() ? $request->newResource()->resource->sortable['sort_on_pivot'] ?? false : true;
am I using it wrong or is this just how its meant to work.
I can fix this easily by over riding the serializeForIndex method though so no real action is needed if this intended behaviour.
On laravel 7 and Nova 3

Hi! This is the intended behaviour as I haven't added a flag for the "show only on pivot tables" use-case.
If you have any ideas on how to handle this, input is greatly appreciated!

Hi, I just override the serializeForIndex method and made it a little more verbose
    public function serializeForIndex(NovaRequest $request, $fields = null)
    {

        if ($request->viaRelationship() && $request->newResource()->resource->sortable['sort_on_pivot']) {
            return array_merge(parent::serializeForIndex($request, $fields), [
                'sortable' => true,
            ]);
        } else {
            return array_merge(parent::serializeForIndex($request, $fields), [
                'sortable' => false,
            ]);
        }

    }

This means it wont show the sort controls on the resources index view on the relationship index view

Hi! The newly released version 1.4.0 supports this exact behaviour - you can only enable sorting on either the Index view OR the HasMany/BelongsToMany/MorphsMany relationship, but not both. Let me know if it works as intended for you now. Good luck!


fix(trace): remove GotConn delay

The delay causes a race condition in the go transport that results in a 502 Bad Gateway with:
endpoint_failure (readLoopPeekFailLocked: %!w(<nil>)).
This happens because the transport peeks the first few bytes on the connection and gets some data even though it doesn't expect any. This causes it to go into an error state even though there is no error resulting in the formatting directive to break.
This commit removes the delay and adds a note why we can't do this for now. This will reduce the amount of requests we can retry because the client will send data before we know that the connection is good. After we sent some data we can't be sure that the server hasn't started processing, hence no retry in such cases.
See: https://cloudfoundry.slack.com/archives/C033ALST37V/p1680888356483179
See: golang/go#31259
Resolves: cloudfoundry/routing-release#316

technically, the race condition is already there. the delay only makes sure it is predictable by allowing one go routine to be scheduled before the competing one. so the delay does cause the race. Also bear in mind that even without the delay problems are not solved and the issue raised by @Gerg may still appear. it's just more random now.


Path error on Linux systems.

Using pandas_ui in Linux there are several errors of this kind :
---------------------------------------------------------------------------
FileNotFoundError                         Traceback (most recent call last)
~/miniconda3/lib/python3.7/site-packages/pandas_ui/funct1.py in on_button_clicked(a, wid, br, dff)
   1561         def on_button_clicked(a, wid, br, dff):
   1562             clear_output()
-> 1563             column_selected(br)
   1564
   1565         #######################################################################################################

~/miniconda3/lib/python3.7/site-packages/pandas_ui/funct1.py in column_selected(br, tabslt)
   1554                 tabl_disp()
   1555
-> 1556             summ_disp(column_summ_val)
   1557
   1558             hist_disp()

~/miniconda3/lib/python3.7/site-packages/pandas_ui/funct1.py in summ_disp(column_summ_val)
   1378
   1379
-> 1380             file = open(patho +"pandas_ui\plot.png", "rb")
   1381             image = file.read()
   1382             file1 = open(patho +"pandas_ui\plt1.png", "rb")

FileNotFoundError: [Errno 2] No such file or directory: '/home/user/miniconda3/lib/python3.7/site-packages/pandas/__init__.pypandas_ui\\plot.png'

As you can see, the error is because the path is built as a string for windows. It should be easy to avoid that error and make the library multiplatform using the Pathlib module.

Hi,
I haven't tested it on Linux, please try in windows
On Tue, 25 Aug 2020, 1:46 pm paduel<EMAIL_ADDRESS>wrote:

Using pandas_ui in Linux there are several errors of this kind :

FileNotFoundError                         Traceback (most recent call last)
~/miniconda3/lib/python3.7/site-packages/pandas_ui/funct1.py in on_button_clicked(a, wid, br, dff)
1561         def on_button_clicked(a, wid, br, dff):
1562             clear_output()
-> 1563             column_selected(br)
1564
1565         #######################################################################################################
~/miniconda3/lib/python3.7/site-packages/pandas_ui/funct1.py in column_selected(br, tabslt)
1554                 tabl_disp()
1555
-> 1556             summ_disp(column_summ_val)
1557
1558             hist_disp()
~/miniconda3/lib/python3.7/site-packages/pandas_ui/funct1.py in summ_disp(column_summ_val)
1378
1379
-> 1380             file = open(patho +"pandas_ui\plot.png", "rb")
1381             image = file.read()
1382             file1 = open(patho +"pandas_ui\plt1.png", "rb")
FileNotFoundError: [Errno 2] No such file or directory: '/home/user/miniconda3/lib/python3.7/site-packages/pandas/init.pypandas_ui\plot.png'
As you can see, the error is because the path is built as a string for
windows. It should be easy to avoid that error and make the library
multiplatform using the Pathlib module.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/arunnbaba/pandas_ui/issues/19, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADRKCWWRKNYNDRD6BJEXHK3SCNXM3ANCNFSM4QKLWMPA
.


Add option whitelist to spell metadata.

Only show options in the whitelist.
Also updates widget styles for the options edit widget.
Since it is now specialized for conjure, we have our own copy here now.
Signed-off-by: Michael McCracken<EMAIL_ADDRESS>

fixes #24.


monkey db upgrade issue is still there

Hi team,
monkey db upgrade issue:
I have followed all the mentioned steps in https://github.com/Netflix/security_monkey/issues/1244 , but still I can see below error after running monkey db upgrade:
/usr/local/src/security_monkey/venv/lib/python3.5/site-packages/flask_sqlalchemy/init.py:835: FSADeprecationWarning: SQLALCHEMY_TRACK_MODIFICATIONS adds significant overhead and will be disabled by default in the future. Set it to True or False to suppress this warning.
'SQLALCHEMY_TRACK_MODIFICATIONS adds significant overhead and '
Traceback (most recent call last):
File "/usr/local/src/security_monkey/venv/bin/monkey", line 33, in
sys.exit(load_entry_point('security-monkey==1.1.1', 'console_scripts', 'monkey')())
File "/usr/local/src/security_monkey/venv/bin/monkey", line 25, in importlib_load_entry_point
return next(matches).load()
File "/usr/local/src/security_monkey/venv/lib/python3.5/site-packages/importlib_metadata/init.py", line 105, in load
module = import_module(match.group('module'))
File "/usr/lib/python3.5/importlib/init.py", line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 986, in _gcd_import
File "", line 969, in _find_and_load
File "", line 944, in _find_and_load_unlocked
File "", line 222, in _call_with_frames_removed
File "", line 986, in _gcd_import
File "", line 969, in _find_and_load
File "", line 958, in _find_and_load_unlocked
File "", line 673, in _load_unlocked
File "", line 665, in exec_module
File "", line 222, in _call_with_frames_removed
File "/usr/local/src/security_monkey/venv/lib/python3.5/site-packages/security_monkey/init.py", line 188, in
from security_monkey.views.user_settings import UserSettings
File "/usr/local/src/security_monkey/venv/lib/python3.5/site-packages/security_monkey/views/user_settings.py", line 15, in
from marshmallow.validate import OneOf
File "/usr/local/src/security_monkey/venv/lib/python3.5/site-packages/marshmallow/init.py", line 16, in
from distutils.version import LooseVersion
File "", line 969, in _find_and_load
File "", line 958, in _find_and_load_unlocked
File "", line 666, in _load_unlocked
File "", line 577, in module_from_spec
File "/usr/local/src/security_monkey/venv/lib/python3.5/site-packages/_distutils_hack/init.py", line 82, in create_module
return importlib.import_module('._distutils', 'setuptools')
File "/usr/lib/python3.5/importlib/init.py", line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 981, in _gcd_import
File "", line 931, in _sanity_check
SystemError: Parent module 'setuptools' not loaded, cannot perform relative import

Please use the other issue.


Code reorganization

@khanley6 common_tensor_t has been renamed in common_tensor_type_t for consistency with other traits. I slightly improved it so that it takes into account the layout_type of its parameters.

Thanks for the heads up!

such a nice spring cleaning!


Slack sign-in / user email

I'm using allauth for Slack signin and collect user email addresses. I require the email as part of the login flow. The Slack app is configured to share email address. With django-allauth==0.30.0 the email is collected okay.
In versions 0.31 and 0.32 the user is asked for their email address as a final step in the login flow. It seems the email isn't collected correctly from Slack anymore.
Couldn't find any extra requirements for Slack login flow in the new versions so thought I'd let you know.
Thanks :)

Hi Peter. I built this for an internal app 2 years ago and just dumped it here just in case anyone could benefit from it or pick it up. I've no intention or time at the moment to actually kick off development on this. A PR would definitely be welcome though. Sorry for not being able to help.

Sorry I left a comment on the wrong project!  It was meant for the main django-allauth repo.
Very sorry, too many open tabs :D


Uncaught IsADirectoryError in pathlib.py line 1117

Aider version: 0.56.0
Python version: 3.10.1
Platform: macOS-14.5-arm64-arm-64bit
Python implementation: CPython
Virtual environment: Yes
OS: Darwin 23.5.0 (64bit)
Git version: git version 2.46.0
An uncaught exception occurred:
Traceback (most recent call last):
  File "aider", line 8, in <module>
    sys.exit(main())
  File "main.py", line 698, in main
    coder.run()
  File "base_coder.py", line 735, in run
    self.run_one(user_message, preproc)
  File "base_coder.py", line 778, in run_one
    list(self.send_message(message))
  File "base_coder.py", line 1213, in send_message
    lint_errors = self.lint_edited(edited)
  File "base_coder.py", line 1307, in lint_edited
    errors = self.linter.lint(self.abs_root_path(fname))
  File "linter.py", line 86, in lint
    code = Path(fname).read_text(encoding=self.encoding, errors="replace")
  File "pathlib.py", line 1132, in read_text
    with self.open(mode='r', encoding=encoding, errors=errors) as f:
  File "pathlib.py", line 1117, in open
    return self._accessor.open(self, mode, buffering, encoding, errors,
IsADirectoryError: [Errno 21] Is a directory: '/Users/ryanhunter/new_era/veils-hub/src/features/universalInbox/FEATURE+USECASES.md'

Thank you for filing this issue.
The error seems to come from the OS and it tries to tell you that FEATURE+USECASES.md file is not a file but a directory. Can you confirm that please? Or is something else going on here?

The fix is available in the main branch. You can get it by installing the latest version from github:
aider --install-main-branch

# or...

python -m pip install --upgrade git+https://github.com/paul-gauthier/aider.git

If you have a chance to try it, let me know if it works better for you. I'm going to close this issue for now, but feel free to add a comment here and I will re-open. Or feel free to file a new issue any time.


PGP keys aren't deleted when user is deleted

Gitea version (or commit ref): Tested on both 1.13.1 and gfcfbab99f
Git version: 2.17.1
Operating system: Debian 10

Gitea is from downloads, and is running as a systemd service.

Database (use [x]):

[x] PostgreSQL
[ ] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[x] Yes (repro link N/A)
[ ] No

Log gist: https://gist.github.com/aveao/481fa4aacbb7377ce202fcff780f5dd1

Description
Current behavior
When a user deletes their account, their pgp key(s) in pgp_key_import table isn't deleted. As such, when the user registers again with the same email and tries to add the PGP key, it causes a unique constraint violation, and gitea throws a 500.
Expected behavior

Bugfix: When a user deletes their account, their PGP key(s) in pgp_key_import should be deleted too.
Nice to have in addition to bugfix: Trying to add non-unique PGP keys should probably display a better error page.

Screenshots

I'm unable to reproduce this bug at https://try.gitea.io. Could you provide instructions on how to reproduce this or has this issue already been addressed?

@anton-khimich I can still repro this on try.gitea.io.
Repro instructions:

[Have a PGP key ready]
Create user account A with same email as PGP key
Go to https://try.gitea.io/user/settings/keys, add said PGP key
(Observe that PGP key was added properly)
Go to https://try.gitea.io/user/settings/account, delete user account
Create user account B with same email as PGP key
Go to https://try.gitea.io/user/settings/keys, add said PGP key
(Observe the HTTP 500 error)

Okay, I think I see what the problem is. When you add a PGP key, it gets inserted into the gpg_key and gpg_key_import tables, and manually removing the PGP key deletes the key from those same tables. When a user is deleted however, the PGP key is deleted from gpg_key but not gpg_key_import. I'll see if I can fix this.


Internal server error in dashboard

Hello, I have a problem with teh dashboard of the TechnicSolder <IP_ADDRESS>. When I click on "Dashboard", I have this error : http://pastebin.com/euzhLKnj
Anybody can help me ? I am on Debian 7.7

Go to /login after your solder domain.

Ok thanks !


Require manual minikube tunnel on linux too

Signed-off-by: Paul S. Schweigert<EMAIL_ADDRESS>
Changes
Based on a slack discussion, the auto-tunnel for Linux users was causing problems, so this PR adds the pause & request a tunnel for Linux users too (we were already doing it for Mac and Windows users).
Linux users will need to manually create a tunnel for minikube. This should eliminate some common networking bugs and is already being done for Mac and Windows users.

/assign @csantanapr

/cherry-pick release-1.5

I'm not familiar with minikube on Linux, if auto-linux is working for the person and they run tunnel does it brake. Is there a way to disable to make sure it always work when they run tunnel command?

/lgtm

The issue was it looked like they didn't have passwordless sudo, and quickstart wasn't giving them a chance to enter the password before it went on to the next command.
This also gives us consistent behavior across all OSes, which is a nice added benefit 😄


fix(deps): Remove unused dependencies, and check for them in CI

Motivation
Having unused dependencies makes Zebra less secure. It also increases build time and binary size.
We deleted a lot of code recently, but we didn't delete the dependencies that were only used by that code. We also copy-pasted some dependencies that weren't actually needed.
Closes #6215
Complex Code or Requirements
This has a bit of shell script in it, but it doesn't do anything unusual.
Solution

remove unused dependencies detected by cargo-machete
add a CI job to detect unused dependencies

add a patch workflow so we can make this job part of the branch protection rules

This PR should appear in the changelog because it removes dependencies, which is visible to users.
Review
This is a low priority change. It can go in after the release.
Reviewer Checklist

[ ] Will the PR name make sense to users?

[ ] Does it need extra CHANGELOG info? (new features, breaking changes, large changes)

[ ] Are the PR labels correct?
[ ] Does the code do what the ticket and PR says?

[ ] Does it change concurrent code, unsafe code, or consensus rules?

[ ] How do you know it works? Does it have tests?

Follow Up Work
After this PR merges, an admin should make the new job part of the branch protection rules.

This seems to work:
-- full cargo machete output, including ignored dependencies --
Analyzing dependencies of crates in this directory...
 INFO  cargo_machete > /home/runner/work/zebra/zebra/Cargo.toml is a virtual manifest for a workspace
cargo-machete found the following unused dependencies in /home/runner/work/zebra/zebra:
Done!
zebra-test -- /home/runner/work/zebra/zebra/zebra-test/Cargo.toml:
	tinyvec
zebra-network -- /home/runner/work/zebra/zebra/zebra-network/Cargo.toml:
	humantime-serde
zebrad -- /home/runner/work/zebra/zebra/zebrad/Cargo.toml:
	gumdrop
	humantime-serde
	tinyvec
zebra-utils -- /home/runner/work/zebra/zebra/zebra-utils/Cargo.toml:
	tinyvec
-- unused dependencies are below this line, full output is above --
No unused dependencies found.

https://github.com/ZcashFoundation/zebra/actions/runs/4260322630/jobs/7413377013#step:5:20


Invalid default value for UspNatV1Field.MSPA_COVERED_TRANSACTION?

The specification says that the valid values for MspaCoveredTransaction are:

1  Yes
2  No

But this field is given a default value of 0 here: https://github.com/IABTechLab/iabgpp-java/blob/2d39d9648443b92a572c447b015c4307822b0e09/iabgpp-encoder/src/main/java/com/iab/gpp/encoder/section/UspNatV1.java#L56
Should the spec be updated to say that 0 is valid? Or would it make sense to use a different default in the code?

0 is invalid; the code here is wrong


Should bytecode contain PyObjectRef references to constants?

It could be argued that bytecode objects may refer to python objects which are immutable, such as tuples, None and integers.
This would mean that the bytecode::Constant enum would disappear:
https://github.com/RustPython/RustPython/blob/master/vm/src/bytecode.rs#L113
This would also mean that the bytecode class becomes depending upon the vm. This might be a downside, so we cannot separate the bytecode package into a seperate package, but on the other hand, I like the idea of referring to PyObjectRef objects straight from bytecode.
This is a voting issue, what do you think?

I haven't thought about this a lot, but my first instinct would be not to - assuming that it's a small subset of python types that are valid constants.
Aside from the dependency issue, I'd rather use a more restrictive type and get as much out of the rust static typing as we can. I'd also rather avoid needing a full pycontext to generate bytecode.

Closing this issue since we have the rustpython-bytecode crate now.


My updates

Hello all,
I have been busy coding and I must say that I really enjoy writing with jsPsych. It is easy and fun.
I have put together scripts for delivering questionnaires. I have a setup file for each questionnaire that contains the questions and scoring info. The only thing that needs to be changed in the HTML is which setup file to import. The scoring of the questionnaire is also performed and saved in the output file. I have completed the CFI, CFS, PANAS, STAI and AES. These are easy to make so I will add more later. By having the text and the "guts" separate it will be "easy" to have the language translated since someone does not have to dig through any code to get to the text.
I have the Baddeley's gramatical reasoning test completed.
I have an attentional network task done (This is like a complicated flankers task).
I have my own staircase procedure task, I know this will be at least the third version available.
I am also working on a verbal delayed match to sample task.
I have first drafts completed on these so far. Once I do some more internal testing I will make a pull request. They are all in my fork if anyone is interested right now.
Take care,
Jason

Hi Jason.
Sounds great. A few comments:

Is there any way you can do separate pull requests for each of these?
Is the attentional network task a Posner cueing task, or do you mean something else by that? I have a student working on the Posner cueing task is why I ask.

Best,
Peter

Is the staircase procedure just your version of using a staircase to measure thresholds?

oh and I would be great if you could raise issues labeled "DemoTaskDevelopment" for each of these, just so we are following the procedure we are trying to enforce.

Yes, I will try my best to do separate pull requests for these once I feel they are good enough.
The ANT task is the one discussed here: Fan, J., McCandliss, B. D., Sommer, T., Raz, A., & Posner, M. I. (2002). Testing the efficiency and independence of attentional networks. Journal Cognitive Neuroscience, 14(3), 340-7.
The staircase procedure is for measuring thresholds. I tried to use: https://github.com/hadrienj/StaircaseJS but I could not figure out how to use it. I did get to learn how to use objects in javascript by making my own. So that is good.
I cannot see how I can assign a label to an issue, sorry.

well, raise the issues and I will assign the labels.
I think any type of staircase experiment would be good for this library.

I wonder if we could somehow merge different versions of the Posner cueing experiment into one experiment on here?

Fine with me. If your student wants to take what I have and make it work best for you, that is fine.  I have all the functionality complete along with the coding of the trial types for the results file. All that is left is setting the trial/ITI timing so that it matches the original article. It was a pain finding the best arrows and asterix to use. What I have also been doing for different tasks is using tables to align multiple items on the screen. I make functions  like "PutStuffIntoTable" that takes some items and arranges them in a table. And if the margins and layout of the tables are set correctly than things show up in the same spot on the screen for every trial. That I think has been one of the most challenging things.

@steffejr you may want to explore a combination of relative and absolute CSS positioning to get items in particular locations on the screen.
Here's an example posner task stimulus that I made:
{
          type: 'html-keyboard-response',
          stimulus: `

🔵

If the circle appears in the right box, press P.

Press P to continue.

`,
          choices: ['p']
        },

My full experiment is here:
https://github.com/jodeleeuw/219-eyetrack-example-full


Incorrect type inference after array unpacking. Unsound error

🔎 Search Terms
"type inference", "array", "unpacking", "destructuring"
🕗 Version & Regression Information

This is a unsound error
All versions in playground have this bug

⏯ Playground Link
https://www.typescriptlang.org/play?#code/GYVwdgxgLglg9mABAcwKZQGIwE4GcoAUAhttgFyJggC2ARqtgNoC6AlIgN4BQiviECfIkYA3IgBsQqZogC8iEtgDcPPtnQhsSMZNQqAvly7j0iYHJToseQi1ZKgA
💻 Code
function getFirst(arr: number[]) {
    const [value] = arr;
    return value;
}

let f = getFirst([]);

🙁 Actual behavior
f variable has type number
🙂 Expected behavior
f variable must be a number | undefined
Additional information about the issue
No response

This was already fixed by introducing the noUncheckedIndexedAccess compiler option. You need to enable it, then it results in number | undefined as expected.

In that way, wouldn't it be considered a logical error? When a user tries to get an element, TypeScript guarantees that it exists.

I don't understand what you mean. Without noUncheckedIndexedAccess the type is unsound, yes. That's why this option was added.


Not installable via pip

Thank you for doing this!
It doesn't appear to be installable via pip:
$ pip install --verbose django-GNU-Terry-Pratchett
Collecting django-GNU-Terry-Pratchett
  Getting page https://pypi.python.org/simple/django-gnu-terry-pratchett/
  URLs to search for versions for django-GNU-Terry-Pratchett:
  * https://pypi.python.org/simple/django-gnu-terry-pratchett/
  Getting page https://pypi.python.org/simple/django-gnu-terry-pratchett/
  Analyzing links from page https://pypi.python.org/simple/django-gnu-terry-pratchett/
  Could not find any downloads that satisfy the requirement django-GNU-Terry-Pratchett
  Cleaning up...
  No distributions at all found for django-GNU-Terry-Pratchett
  Exception information:
  Traceback (most recent call last):
    File "/Users/ms/balloon/mns/python/lib/python2.7/site-packages/pip/basecommand.py", line 232, in main
      status = self.run(options, args)
    File "/Users/ms/balloon/mns/python/lib/python2.7/site-packages/pip/commands/install.py", line 339, in run
      requirement_set.prepare_files(finder)
    File "/Users/ms/balloon/mns/python/lib/python2.7/site-packages/pip/req/req_set.py", line 333, in prepare_files
      upgrade=self.upgrade,
    File "/Users/ms/balloon/mns/python/lib/python2.7/site-packages/pip/index.py", line 397, in find_requirement
      'No distributions at all found for %s' % req
  DistributionNotFound: No distributions at all found for django-GNU-Terry-Pratchett

Hi @jpluscplusm I'm not able to recreate. When I run
pip install --verbose django-GNU-Terry-Pratchett

it installs as expected. Can you try again and see if you can recreate the issue?

It now works, pulling in version 0.2.0. I /suspect/ Pypi got its package availability / internals in a twist for some reason, and your new release powered its way through the pipeline and solved the problem with prejudice. Ta :-)

Awesome, glad it works for you now :+1:


jetbrains.idea-community: add IntelliJ keyword to the longDescription

This should make it easier to find the package under the name
IntelliJ in eg. [1]
If possible, please backport :)
[1] https://nixos.org/nixos/packages.html#intellij
Motivation for this change
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.

No attempt on aarch64-linux (full log)
The following builds were skipped because they don't evaluate on aarch64-linux: jetbrains.idea-community
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Attempted: jetbrains.idea-community
Partial log (click to expand)
shrinking /nix/store/fw7w74h9x1bdciqd1qpa1iz5c1m6bl8j-idea-community-2018.2.4/idea-community-2018.2.4/plugins/android/lib/libwebp/linux/libwebp_jni64.so
shrinking /nix/store/fw7w74h9x1bdciqd1qpa1iz5c1m6bl8j-idea-community-2018.2.4/idea-community-2018.2.4/plugins/android/lib/libwebp/linux/libwebp_jni.so
strip is hashed (/nix/store/h0lbngpv6ln56hjj59i6l77vxq25flbz-binutils-2.30/bin/strip)
stripping (with command strip and flags -S) in /nix/store/fw7w74h9x1bdciqd1qpa1iz5c1m6bl8j-idea-community-2018.2.4/libexec  /nix/store/fw7w74h9x1bdciqd1qpa1iz5c1m6bl8j-idea-community-2018.2.4/bin
patching script interpreter paths in /nix/store/fw7w74h9x1bdciqd1qpa1iz5c1m6bl8j-idea-community-2018.2.4
/nix/store/fw7w74h9x1bdciqd1qpa1iz5c1m6bl8j-idea-community-2018.2.4/idea-community-2018.2.4/bin/idea.sh: interpreter directive changed from "/bin/sh" to "/nix/store/czx8vkrb9jdgjyz8qfksh10vrnqa723l-bash-4.4-p23/bin/sh"
/nix/store/fw7w74h9x1bdciqd1qpa1iz5c1m6bl8j-idea-community-2018.2.4/idea-community-2018.2.4/bin/format.sh: interpreter directive changed from "/bin/sh" to "/nix/store/czx8vkrb9jdgjyz8qfksh10vrnqa723l-bash-4.4-p23/bin/sh"
/nix/store/fw7w74h9x1bdciqd1qpa1iz5c1m6bl8j-idea-community-2018.2.4/idea-community-2018.2.4/bin/inspect.sh: interpreter directive changed from "/bin/sh" to "/nix/store/czx8vkrb9jdgjyz8qfksh10vrnqa723l-bash-4.4-p23/bin/sh"
checking for references to /build in /nix/store/fw7w74h9x1bdciqd1qpa1iz5c1m6bl8j-idea-community-2018.2.4...
/nix/store/fw7w74h9x1bdciqd1qpa1iz5c1m6bl8j-idea-community-2018.2.4

backported: 18749a72b7c

Thank you! :)