Allow creation of new item before search action returns

Currently if you are using the search action (instead of options) to populate the drop down list a user can't create a new option until the search action resolves.
For creating tags this means that the user has to wait for the "autocomplete" search to finish before they can create a tag.  If the user quickly types the tag and presses enter the typed text just disappears.
Is there anyway to configure it so that a new list option can be created even if the search action is still pending?  It feels very clunky to have to know to wait for the "Create new item" message to appear before hitting enter.
To make it more concrete this is my setup
{{#power-select-with-create
    multiple=true
    search=(action "searchTags")
    selected=selectedTags
    oncreate=(action "createTag")
    searchField="name"
    onchange=(action (mut selectedTags))
    as |tag term|
    }}
      {{tag.name}}
{{/power-select-with-create}}

Right now the createTag action can't be triggered until after the searchTags action has resolved.  I'd like to allow a user to create a new tag right away without having to wait for the searchTags method to return (it issues a query to the server to function as an autocomplete).
Thanks!

This is tricky. When we use search, the results should reflect the response of the asynchronous action (usually the response of an HTTP request), and any user-created item created before the response would be replaced when the response from the server finishes.
We would have to somehow merge all locally created items into the results returned by the promise. This sounds flaky and can lead to hard-to-debug situations where the actual select items doesn't match the return of the promise.
What would you suggest as a possible solution? I might have a look at it whenever I find the time. Contributions are welcome as well. ;-)
@cibernox What do you think?

I took a stab at this and what I have appears to be working at least for my use case.  It did require upgrading to the beta version so that I had access to the search results from the public API and could manipulate that list outside of the search action (I could be doing things incorrectly or not following guidelines for this component as I still don't know the details that well)
I'm basically using ember-power-select-with-create (with multiple=true) as a way to input tags.  Tags are also suggested based on a server side search.  The server side provided results are only suggestions however and a user can create any tag they want.
As a user types, the first item in the result list reflects what they have typed and that item should be highlighted so if the user presses enter the tag is immediately added to the selected array.   If the user stops typing then the searchAndSuggest method runs and fetches suggestions from the server.  When the results are retrieved the the server side suggestions are displayed as well as the option for the user to create a tag for whatever they currently have entered as the term.
Here's what that method looks like.  It gets called on the onkeydown action.
onkeydown=(action "typeAndSuggest")

    typeAndSuggest(select, e){

      if(!this.get('createAsTyping')){
        return;
      }

      // We don't want to do anything if the user is pressing the arrow keys to navigate
      // the dropdown menu
      if(this._isArrowKey(e)){
        return;
      }

      // The value of searchText always seems to be one key behind unless I wait for next
      Ember.run.next(this, function(){
        let selected = this.get('selected');

        if (!selected.includes(select.searchText) && !Ember.isBlank(select.searchText)) {
          let results = select.results;
          let suggestion = this.buildSuggestionForTerm(select.searchText);

          if(results.length !== 0){
            //remove the existing suggestion
            results.shiftObject();
          }
          //add our "suggestion" which is based off what the user has typed and highlight it
          results.unshiftObject(suggestion);
          select.actions.highlight(suggestion);
        }

        if(Ember.isBlank(select.searchText)){
          select.results.clear();
          // If we don't set highlighted to undefined it appears to have a residual value
          // which means if the user presses enter even when the input is empty a value
          // shows up
          Ember.set(select, 'highlighted', undefined);
        }
      });
    },

Note that this is placed into a fork of ember-power-select-with-create where I upgraded to the latest beta version of ember-power-select.


Asking for the detectron pretrained model

Would you mind if you can please give me the pre-trained model which located inside the folder pre-trained from your repo GitHub. I want to compare the performance. Thank you so much.

Thanks for your interest in this repo. I've removed the pre-trained model.


Transactions don't handle RAISE(ROLLBACK) in triggers

The following code throws cannot rollback - no transaction is active.
var connection = new SqliteConnection("Filename=:memory:");
connection.Open();

var command = connection.CreateCommand();
command.CommandText =
@"
    CREATE TABLE test(value);

    CREATE TRIGGER test_trigger
    BEFORE INSERT ON test
    BEGIN
        SELECT RAISE(ROLLBACK, 'error');
    END;
";
command.ExecuteNonQuery();

var transaction = connection.BeginTransaction();
try
{
    command.Transaction = transaction;
    command.CommandText = "INSERT INTO test VALUES ('data')";
    command.ExecuteNonQuery();
}
catch (SqliteException ex)
    when (ex.SqliteErrorCode == SQLitePCL.raw.SQLITE_CONSTRAINT)
{
}
finally
{
    transaction.Rollback();
}

We should leverage sqlite3_rollback_hook to detect this and similar situations and try to handle them more gracefully.

Similar situations:

Trying to commit the transaction despite the error: Throws cannot commit - no transaction is active
Executing another command on the connection: Executes outside of a transaction; should throw.

@bricelam What happens to the dangling transaction? That is, what is the behavioral or end-state difference between this and a rollback?

It's rolled back. So if we should just no-op if they call Rollback, but throw if they execute another command or call Commit.


URL Cleanup

This commit updates URLs to prefer the https protocol. Redirects are not followed to avoid accidentally expanding intentionally shortened URLs (i.e. if using a URL shortener).
HTTP URLs that Could Not Be Fixed
These URLs were unable to be fixed. Please review them to see if they can be manually resolved.

[ ] http://junit.sourceforge.net/javadoc/ (200) with 1 occurrences could not be migrated:
(https result AnnotatedConnectException).
[ ] http://oauth.gmodules.com/gadgets/oauthcallback (200) with 1 occurrences could not be migrated:
(https result SSLHandshakeException).
[ ] http://somewhere.com (403) with 8 occurrences could not be migrated:
(https result ConnectTimeoutException).

Fixed URLs
Fixed But Review Recommended
These URLs were fixed, but the https status was not OK. However, the https status was the same as the http request or http redirected to an https URL, so they were migrated. Your review is recommended.

[ ] http://mycompany.com (302) with 1 occurrences migrated to:
https://secure.mycompany.com (https result ConnectTimeoutException).
[ ] http://tools.ietf.org/html/draft-ietf-oauth-v2 (301) with 1 occurrences migrated to:
https://tools.ietf.org/html/draft-ietf-oauth-v2 (https result ReadTimeoutException).
[ ] http://schemas.google.com/g/2005 (UnknownHostException) with 22 occurrences migrated to:
https://schemas.google.com/g/2005 (https result UnknownHostException).
[ ] http://schemas.google.com/photos/2007 (UnknownHostException) with 33 occurrences migrated to:
https://schemas.google.com/photos/2007 (https result UnknownHostException).
[ ] http://schemas.google.com/photos/exif/2007 (UnknownHostException) with 1 occurrences migrated to:
https://schemas.google.com/photos/exif/2007 (https result UnknownHostException).
[ ] http://a9.com/-/spec/opensearchrss/1.0/ (301) with 1 occurrences migrated to:
https://a9.com/-/spec/opensearchrss/1.0/ (https result 404).
[ ] http://code.google.com/apis/accounts/docs/OAuth_ref.html-- (404) with 1 occurrences migrated to:
https://code.google.com/apis/accounts/docs/OAuth_ref.html-- (https result 404).
[ ] http://gadget-doc-examples.googlecode.com/svn/trunk/images/new.gif (404) with 1 occurrences migrated to:
https://gadget-doc-examples.googlecode.com/svn/trunk/images/new.gif (https result 404).
[ ] http://gadget-doc-examples.googlecode.com/svn/trunk/opensocial-gadgets/popup.js (404) with 1 occurrences migrated to:
https://gadget-doc-examples.googlecode.com/svn/trunk/opensocial-gadgets/popup.js (https result 404).
[ ] http://picasaweb.google.com/lh/photo/1_pPyhAE3IXRe5cb4AagZQ (404) with 1 occurrences migrated to:
https://picasaweb.google.com/lh/photo/1_pPyhAE3IXRe5cb4AagZQ (https result 404).
[ ] http://picasaweb.google.com/lh/photo/EpbAByEAotaQekZSdVgZQg (404) with 1 occurrences migrated to:
https://picasaweb.google.com/lh/photo/EpbAByEAotaQekZSdVgZQg (https result 404).
[ ] http://picasaweb.google.com/lh/photo/Ioko4Z0cbBD1Dh1B7YubrA (404) with 1 occurrences migrated to:
https://picasaweb.google.com/lh/photo/Ioko4Z0cbBD1Dh1B7YubrA (https result 404).
[ ] http://picasaweb.google.com/lh/photo/U7NcEx5i97pGfjxIhqlYow (404) with 1 occurrences migrated to:
https://picasaweb.google.com/lh/photo/U7NcEx5i97pGfjxIhqlYow (https result 404).
[ ] http://picasaweb.google.com/lh/photo/XpvsZB33FPqLHa527Nc2hQ (404) with 1 occurrences migrated to:
https://picasaweb.google.com/lh/photo/XpvsZB33FPqLHa527Nc2hQ (https result 404).
[ ] http://picasaweb.google.com/lh/photo/_rJ23c0-ev-GEsuqajvYWg (404) with 1 occurrences migrated to:
https://picasaweb.google.com/lh/photo/_rJ23c0-ev-GEsuqajvYWg (https result 404).
[ ] http://picasaweb.google.com/lh/photo/b02iZRlngmObhiG-xVwmng (404) with 1 occurrences migrated to:
https://picasaweb.google.com/lh/photo/b02iZRlngmObhiG-xVwmng (https result 404).
[ ] http://picasaweb.google.com/lh/photo/cyqNMsCJd4KMl9d8thlF9A (404) with 1 occurrences migrated to:
https://picasaweb.google.com/lh/photo/cyqNMsCJd4KMl9d8thlF9A (https result 404).
[ ] http://picasaweb.google.com/lh/photo/emw168uzsXbckWn6o1NOGw (404) with 1 occurrences migrated to:
https://picasaweb.google.com/lh/photo/emw168uzsXbckWn6o1NOGw (https result 404).
[ ] http://picasaweb.google.com/lh/photo/zU1hicDzC1JGgsnl6CMtQw (404) with 1 occurrences migrated to:
https://picasaweb.google.com/lh/photo/zU1hicDzC1JGgsnl6CMtQw (https result 404).
[ ] http://picasaweb.google.com/lh/reportAbuse?uname=stoicflame&aid=5155181789245632497&iid=5155184692643524754 (404) with 1 occurrences migrated to:
https://picasaweb.google.com/lh/reportAbuse?uname=stoicflame&aid=5155181789245632497&iid=5155184692643524754 (https result 404).
[ ] http://picasaweb.google.com/lh/reportAbuse?uname=stoicflame&aid=5155181789245632497&iid=5155184993291235490 (404) with 1 occurrences migrated to:
https://picasaweb.google.com/lh/reportAbuse?uname=stoicflame&aid=5155181789245632497&iid=5155184993291235490 (https result 404).
[ ] http://picasaweb.google.com/lh/reportAbuse?uname=stoicflame&aid=5155181789245632497&iid=5155185272464109746 (404) with 1 occurrences migrated to:
https://picasaweb.google.com/lh/reportAbuse?uname=stoicflame&aid=5155181789245632497&iid=5155185272464109746 (https result 404).
[ ] http://picasaweb.google.com/lh/reportAbuse?uname=stoicflame&aid=5155181789245632497&iid=5155185633241362626 (404) with 1 occurrences migrated to:
https://picasaweb.google.com/lh/reportAbuse?uname=stoicflame&aid=5155181789245632497&iid=5155185633241362626 (https result 404).
[ ] http://picasaweb.google.com/lh/reportAbuse?uname=stoicflame&aid=5155181789245632497&iid=5155186024083386578 (404) with 1 occurrences migrated to:
https://picasaweb.google.com/lh/reportAbuse?uname=stoicflame&aid=5155181789245632497&iid=5155186024083386578 (https result 404).
[ ] http://picasaweb.google.com/lh/reportAbuse?uname=stoicflame&aid=5155181789245632497&iid=5155186290371358946 (404) with 1 occurrences migrated to:
https://picasaweb.google.com/lh/reportAbuse?uname=stoicflame&aid=5155181789245632497&iid=5155186290371358946 (https result 404).
[ ] http://picasaweb.google.com/lh/reportAbuse?uname=stoicflame&aid=5155181789245632497&iid=5155186560954298610 (404) with 1 occurrences migrated to:
https://picasaweb.google.com/lh/reportAbuse?uname=stoicflame&aid=5155181789245632497&iid=5155186560954298610 (https result 404).
[ ] http://picasaweb.google.com/lh/reportAbuse?uname=stoicflame&aid=5155181789245632497&iid=5155186870191943938 (404) with 1 occurrences migrated to:
https://picasaweb.google.com/lh/reportAbuse?uname=stoicflame&aid=5155181789245632497&iid=5155186870191943938 (https result 404).
[ ] http://picasaweb.google.com/lh/reportAbuse?uname=stoicflame&aid=5381874767712259297&iid=5398207118777306258 (404) with 1 occurrences migrated to:
https://picasaweb.google.com/lh/reportAbuse?uname=stoicflame&aid=5381874767712259297&iid=5398207118777306258 (https result 404).
[ ] http://picasaweb.google.com/lh/reportAbuse?uname=stoicflame&aid=5381874767712259297&iid=5416573875289209458 (404) with 1 occurrences migrated to:
https://picasaweb.google.com/lh/reportAbuse?uname=stoicflame&aid=5381874767712259297&iid=5416573875289209458 (https result 404).
[ ] http://picasaweb.google.com/stoicflame (404) with 2 occurrences migrated to:
https://picasaweb.google.com/stoicflame (https result 404).
[ ] http://picasaweb.google.com/stoicflame/HawaiiBeach (404) with 8 occurrences migrated to:
https://picasaweb.google.com/stoicflame/HawaiiBeach (https result 404).
[ ] http://picasaweb.google.com/stoicflame/TheHeatonHome?authkey=aEgk9ZZkNoY (404) with 2 occurrences migrated to:
https://picasaweb.google.com/stoicflame/TheHeatonHome?authkey=aEgk9ZZkNoY (https result 404).

Fixed Success
These URLs were switched to an https URL with a 2xx status. While the status was successful, your review is still recommended.

[ ] http://static.springframework.org/spring-security/site/ (301) with 1 occurrences migrated to:
https://docs.spring.io/spring-security/site/ (https result 200).
[ ] http://maven.apache.org/xsd/maven-4.0.0.xsd with 25 occurrences migrated to:
https://maven.apache.org/xsd/maven-4.0.0.xsd (https result 200).
[ ] http://myhost.com with 1 occurrences migrated to:
https://myhost.com (https result 200).
[ ] http://oauth.net/ with 1 occurrences migrated to:
https://oauth.net/ (https result 200).
[ ] http://www.springframework.org/schema/beans/spring-beans-3.0.xsd with 12 occurrences migrated to:
https://www.springframework.org/schema/beans/spring-beans-3.0.xsd (https result 200).
[ ] http://www.springframework.org/schema/beans/spring-beans-3.1.xsd with 1 occurrences migrated to:
https://www.springframework.org/schema/beans/spring-beans-3.1.xsd (https result 200).
[ ] http://www.springframework.org/schema/beans/spring-beans.xsd with 15 occurrences migrated to:
https://www.springframework.org/schema/beans/spring-beans.xsd (https result 200).
[ ] http://www.springframework.org/schema/mvc/spring-mvc-3.0.xsd with 1 occurrences migrated to:
https://www.springframework.org/schema/mvc/spring-mvc-3.0.xsd (https result 200).
[ ] http://www.springframework.org/schema/mvc/spring-mvc-3.1.xsd with 1 occurrences migrated to:
https://www.springframework.org/schema/mvc/spring-mvc-3.1.xsd (https result 200).
[ ] http://www.springframework.org/schema/mvc/spring-mvc.xsd with 2 occurrences migrated to:
https://www.springframework.org/schema/mvc/spring-mvc.xsd (https result 200).
[ ] http://www.springframework.org/schema/security/spring-security-oauth-1.0.xsd with 5 occurrences migrated to:
https://www.springframework.org/schema/security/spring-security-oauth-1.0.xsd (https result 200).
[ ] http://www.springframework.org/schema/security/spring-security-oauth2-2.0.xsd with 2 occurrences migrated to:
https://www.springframework.org/schema/security/spring-security-oauth2-2.0.xsd (https result 200).
[ ] http://www.springframework.org/schema/security/spring-security-oauth2.xsd with 19 occurrences migrated to:
https://www.springframework.org/schema/security/spring-security-oauth2.xsd (https result 200).
[ ] http://www.springframework.org/schema/security/spring-security.xsd with 9 occurrences migrated to:
https://www.springframework.org/schema/security/spring-security.xsd (https result 200).
[ ] http://anywhere.com with 2 occurrences migrated to:
https://anywhere.com (https result 301).
[ ] http://maven.apache.org/maven-v4_0_0.xsd with 10 occurrences migrated to:
https://maven.apache.org/maven-v4_0_0.xsd (https result 301).
[ ] http://springframework.org/ with 1 occurrences migrated to:
https://springframework.org/ (https result 301).
[ ] http://java.sun.com/dtd/web-app_2_3.dtd with 2 occurrences migrated to:
https://java.sun.com/dtd/web-app_2_3.dtd (https result 302).
[ ] http://java.sun.com/j2ee/1.4/docs/api with 1 occurrences migrated to:
https://java.sun.com/j2ee/1.4/docs/api (https result 302).
[ ] http://java.sun.com/j2se/1.5.0/docs/api with 1 occurrences migrated to:
https://java.sun.com/j2se/1.5.0/docs/api (https result 302).
[ ] http://picasaweb.google.com/ with 1 occurrences migrated to:
https://picasaweb.google.com/ (https result 302).
[ ] http://picasaweb.google.com/s/c/bin/slideshow.swf?host=picasaweb.google.com&RGB=0x000000&feed=https%3A%2F%2Fpicasaweb.google.com%2Fdata%2Ffeed%2Fapi%2Fuser%2Fstoicflame%3Falt%3Drss with 1 occurrences migrated to:
https://picasaweb.google.com/s/c/bin/slideshow.swf?host=picasaweb.google.com&RGB=0x000000&feed=https%3A%2F%2Fpicasaweb.google.com%2Fdata%2Ffeed%2Fapi%2Fuser%2Fstoicflame%3Falt%3Drss (https result 302).
[ ] http://search.yahoo.com/mrss/ with 1 occurrences migrated to:
https://search.yahoo.com/mrss/ (https result 302).

Ignored
These URLs were intentionally ignored.

http://anywhere with 5 occurrences
http://anywhere?key=value with 5 occurrences
http://appengine.google.com/ns/1.0 with 1 occurrences
http://localhost:8080/oauth/authorize with 1 occurrences
http://localhost:8080/oauth/token with 1 occurrences
http://localhost:8080/sparklr/oauth/access_token with 1 occurrences
http://localhost:8080/sparklr/oauth/confirm_access with 1 occurrences
http://localhost:8080/sparklr/oauth/request_token with 1 occurrences
http://maven.apache.org/POM/4.0.0 with 70 occurrences
http://www.springframework.org/schema/beans with 56 occurrences
http://www.springframework.org/schema/mvc with 8 occurrences
http://www.springframework.org/schema/security with 18 occurrences
http://www.springframework.org/schema/security/oauth with 10 occurrences
http://www.springframework.org/schema/security/oauth2 with 53 occurrences
http://www.w3.org/2001/XMLSchema-instance with 63 occurrences
http://www.w3.org/2005/Atom with 1 occurrences

Closed in favor of https://github.com/spring-projects/spring-security-oauth/pull/1766


Docs: Sidebar nav scroll snaps to top for each link

The sidebar nav for SST documentation currently snaps to the top of the page at every link selection, requiring lots of scrolling down, again and again, when reading and moving between links in the documentation.
The Astro Starlight docs do not have this issue, but I can't find the fix.

I think I had disabled view transitions because it wasn't working properly when I tried it on Safari.
I'll need to test it again and see.


Create production startup script

Gulp hangs currently when build finishes but thats a separate issue

Cool :+1:


Are there any alternatives to toolbox? (i.e. other implementations)

To understand some software it is also good to know about the alternatives.
What other implementations are there of software doing similar things as this GitHub project?

Colin Walters has a Toolbox implementation written in Rust: https://github.com/cgwalters/coretoolbox

This ticket can be closed --- it's not an issue, it's a support question.


When applying, make sure the SDK constraint is >= 2.0.0-dev.9.0

Don't change the pubspec, but it'd be good to parse and then SCREAM red/bold to update it.

an update: for the dart:io renames, the min is now 2.0.0-dev.54.0

Updating to make sure that the version range supports 2.0.0.

SGTM
On Thu, Jul 19, 2018 at 11:51 AM Devon Carew<EMAIL_ADDRESS>wrote:

Updating to make sure that the version range supports 2.0.0.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dart-lang/dart2_fix/issues/6#issuecomment-406378408,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABCiloTUp-nQN8Yc1Qk_B-vPgTlr-ivks5uINVJgaJpZM4S-4Z_
.


Update code to not get images from placeholder.com

It has been bought by a new company and images might not hold forever.
We may want to also store those images locally ? But we can do that
later.
Fixes #44

I'm tempted to immediately update this script to create local 20x20 images and not rely on external providers.

The script to create those images is trivial:
from PIL import Image

width, height = 20, 20

def hex_to_rgb(hex_string):
    hex_string = hex_string.lstrip("#")
    r = int(hex_string[0:2], 16)
    g = int(hex_string[2:4], 16)
    b = int(hex_string[4:6], 16)
    return (r, g, b)

hex_color = "#ccff44"
rgb_color = hex_to_rgb(hex_color)

image = Image.new("RGB", (width, height), rgb_color)

image.save("colored_image.png")

Ok, let's try with local png, it seem to work. I also thought of using just a

, like This ->  <-, but it is sanitized by github.

Rendering won't work in the PR, you have to look at the fork:
https://github.com/Carreau/accessible-pygments/tree/rr/a11y_pygments/a11y_high_contrast_dark
Also strangely the contrast is not the same, but the colors have not changes. @gabalafou any idea why ?

Should have addressed the comments and rebase/squashed to avoid having large intermediate commits.


Update README.md with Maven Central badge

Thanks! not so random person from the Internet.

You are welcome jplana
On Jan 9, 2015 4:15 PM, "Jose Plana"<EMAIL_ADDRESS>wrote:

Thanks! not so random person from the Internet.
—
Reply to this email directly or view it on GitHub
https://github.com/tuenti/SmsRadar/pull/7#issuecomment-69346833.


Input length bug

What is the context of this PR?
This PR creates a min max map from answers that have a min and/or max set from another question, compares the answer ids to that and if there is a match it will get the correct min max value from the correct source.
How to review

Launch test_numbers.json schema
Enter a min / max and submit
Compare the size of the range question inputs between this branch and the main branch

Checklist

[ ] New static content marked up for translation
[ ] Newly defined schema content included in eq-translations repo

 Bug: get_width_for_number function does not work when using value sources


Migrate away from open.mapquestapi.com endpoint for geocoding

I just got an email that they're giving this the axe in August, so we'll need to find something similar to replace it.

The Lambda is currently failing because open.mapquestapi.com is returning 403

Seriously?  They said we had until August 31...

Can take a look at this but I didn't write the code so no promises

Ah, I was replying because shenanigans assigned me

I just assigned you because you're marked as the project lead over the scraping projects, so I wanted to ask you to take a look.
Sorry for the confusion, I've assigned you both now.
Whichever of you wants to can work on it; or feel free to unassign yourselves as necessary.


Check that JAX dataset is still up-to-date

Description
As a developer, I want to make sure the JAX dataset is up-to-date.
Use API to get the data, cross-check with what is currently in the PDX FInder in relation to drug dosing.
Acceptance criteria

[x] done
[ ] if new models - make ticket to map the new data

Updated jax script for data download
Downloaded new dataset
Test load done

Need to do one more testload to cross reference check drug dosing


Feature/api integration

Resolves: #50

#50

#50

#50

#51

Approced: #51

approved #51

approved #51

Approved #51

approved #51

Approved #51

#40

Resolves: #71

Resolves #71

Resolves #71

Resolves #71

resolves #71

resloves #71

resolves #71

resolves #71

resolves #71

approved #71

review #71

resolves #71

resolves #71 approved #71


Add new Yelp Contributors

@mbabineau I am becoming a bottleneck at Yelp for other yelpers who need to get these 1.4.* fixes in quickly as we try to upgrade marathon.
Can you promote @nhandler and @Rob-Johnson to contributors to help spread the load?

With pleasure! Invites sent - thanks guys.


Deprecate a few more APIs to be removed in 7.0.0

Motivation
While removing APIs in the 7.0.0 WIP branch, I noticed a few I missed that we should deprecate and remove in 7.0.0.
Changes
Deprecate ReactElementStore, JsError, markChildValidated
Testing
CI passes

Security Insights
No security relevant content was detected by automated scans.
Action Items

Review PR for security impact; comment "security review required" if needed or unsure
Verify aviary.yaml coverage of security relevant code

Questions or Comments? Reach out on Slack: #support-infosec.

@Workiva/release-management-p


[BREAKING] feat: correctly handles screens by using theme values instead of CSS variables that are not supported in media queries

Breaking change to support this feature : We now pass the entire custom mantine theme to preset to access breakpoint values.
We now can use responsive tailwind classes like lg:text-primary

Don't forget to change this ts signature: https://github.com/songkeys/tailwind-preset-mantine/blob/49499e7e623ce5ffc51c31ce1f79d3948b63770d/src/index.d.ts#L5

Made the feature non-breaking by passing breakpoints only.

Thank you. This has been released in https://github.com/songkeys/tailwind-preset-mantine/releases/tag/v1.2.0


Updating broke the website and restoring did not fix it..

I was updating from 0.64 to 0.78 and it broke everything.. only got 500's
So I used the restore method to revert, but that could not fix it.. here is the logging of the restore:
[05/10/2016 07:56:50 > c6198c: SYS INFO] Status changed to Initializing
[05/10/2016 07:56:50 > c6198c: SYS INFO] Run script 'restoreBackup.ps1' with script host - 'PowerShellScriptHost'
[05/10/2016 07:56:50 > c6198c: SYS INFO] Status changed to Running
[05/10/2016 07:56:51 > c6198c: INFO] Restoring Full Site Backup
[05/10/2016 08:04:09 > c6198c: ERR ] Remove-Item : Directory D:\home\site\wwwroot cannot be removed because it is
[05/10/2016 08:04:09 > c6198c: ERR ] not empty.
[05/10/2016 08:04:09 > c6198c: ERR ] At D:\local\Temp\jobs\triggered\restoreBackup.ps1\ly0vatuz.xzt\restoreBackup.ps
[05/10/2016 08:04:09 > c6198c: ERR ] 1:6 char:5
[05/10/2016 08:04:09 > c6198c: ERR ] + Remove-Item -Path ./wwwroot -Recurse
[05/10/2016 08:04:09 > c6198c: ERR ] + ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
[05/10/2016 08:04:09 > c6198c: ERR ] + CategoryInfo : WriteError: (D:\home\site\wwwroot:DirectoryInfo)
[05/10/2016 08:04:09 > c6198c: INFO] All done
[05/10/2016 08:04:09 > c6198c: ERR ] [Remove-Item], IOException
[05/10/2016 08:04:09 > c6198c: ERR ] + FullyQualifiedErrorId : DirectoryNotEmpty,Microsoft.PowerShell.Commands.
[05/10/2016 08:04:09 > c6198c: ERR ] RemoveItemCommand
[05/10/2016 08:04:09 > c6198c: ERR ]
[05/10/2016 08:04:09 > c6198c: ERR ] Rename-Item : The process cannot access the file because it is being used by
[05/10/2016 08:04:09 > c6198c: ERR ] another process.
[05/10/2016 08:04:09 > c6198c: ERR ] At D:\local\Temp\jobs\triggered\restoreBackup.ps1\ly0vatuz.xzt\restoreBackup.ps
[05/10/2016 08:04:09 > c6198c: ERR ] 1:7 char:5
[05/10/2016 08:04:09 > c6198c: ERR ] + Rename-Item D:\home\site\wwwroot-backup wwwroot
[05/10/2016 08:04:09 > c6198c: ERR ] + ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
[05/10/2016 08:04:09 > c6198c: ERR ] + CategoryInfo : WriteError: (D:\home\site\wwwroot-backup:String)
[05/10/2016 08:04:09 > c6198c: ERR ] [Rename-Item], IOException
[05/10/2016 08:04:09 > c6198c: ERR ] + FullyQualifiedErrorId : RenameItemIOError,Microsoft.PowerShell.Commands.
[05/10/2016 08:04:09 > c6198c: ERR ] RenameItemCommand
[05/10/2016 08:04:09 > c6198c: ERR ]
[05/10/2016 08:04:09 > c6198c: SYS INFO] Status changed to Success

After restoring I only get 403's


Revert "Revert "Added feedback form and sitemap to the footer""

Reverts shravan20/github-readme-quotes#326

@RajatX24: You can consider this branch and check what is the issue? Else I will take a look at this during weekend as I am caught up now.
Thank you in advance!


Not used station id statement in log

in log : Not used station id: , name: station name , why?

where put my stations.yaml ?
/usr/share/hassio is the  root of my supervised installation

since last version, configuration is stored in \addon_configs\adaeefb4_nfws
the addon_config directory is in HA root:

ok, now what sensor type in sensors array to get noise and co2?

One more , is more important to design mqtt discovery pattern to group
every station in yaml in different device, not under one common “Netatmo
Favourite Weather stations” and subscribe many decorator fo device as
component as firmware version….
Inviato da Gmail Mobile
Il giorno mer 13 nov 2024 alle 06:35 GiZMoSK1221 @.***>
ha scritto:

noise and co2?
the addon retrieves only public data from favorite stations, from here:
https://weathermap.netatmo.com/?zoom=15&maplayer=Map
so no data from your own inside netatmo station
—
Reply to this email directly, view it on GitHub
https://github.com/GiZMoSK1221/hass-addons/issues/53#issuecomment-2472435414,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABL7CXL5GH2QSCZV6KXUV4T2ALQK3AVCNFSM6AAAAABRUJQUNGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINZSGQZTKNBRGQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

ok, I understand. However I do not plan to make any bigger changes, just maintenance.


Need help module

The meta package will largely be used as a set of components built out
of other components. This does include a tiny bit of helper classes
though.

Looks pretty good to me, but is meta the name we want?
(I ask because I made a sidebars as a start for the orgs features sidebar.)

nada, but that does bring up that this relies on pui-css-bootstrap and a few other pieces for deps

@aredridel @rockbot I am completely lacking opinion on what we should call this, but I can fold it into sidebars if that's what we want. I thought this was more generic than that.

I wouldn't call it sidebars, because we could have these in different places.

Aright. Makes sense for me to rename that one then!


Request money - "Distance" option is missing in chat

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

Action Performed:

Navigate to https://staging.new.expensify.com/
Click on the global "+" button
Click on the "Request money" button
Click on the chat "+" button
Click on the "Request money" button

Expected Result:
The "Distance" button should be available from both "+" buttons
Actual Result:
The "Distance" button is missing when doing "Request money" from the chat. The option is there when you use the global "+" button to "Request money"
Workaround:
Unknown?
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

Version Number: 1.3.73_0
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/78819774/9313ad5a-d065-4c35-8776-4b801e6a1ecc
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub

I think this is intentional, because distance request is only applicable in policies.

@neil-marcellini can you confirm? ^ Thanks!

I think this is intentional, because distance request is only applicable in policies.

Yes that's true. We don't support distance requests to individuals yet, it has to be sent in a workspace chat.


Automatically change cell language based on cell magics

I would like to run bash scripts in a cell.
I use cell magics for this.
I would like the langauge of the cell to be automatically set to bash
Similarly when using %html, %%svg, etc...
%%script julia
%%html
%%kql
Provides intellisense, syntax highlighting & the like (ie. use VS Code features to improving cell editing experience for non-python & non-markdown cells)
Additionally we should make sure all of the languages supported by magics. This includes things like:

Q# #10772

Related to https://github.com/microsoft/vscode/issues/140147.

I hope you're planning for sql too, it's so annoying as of the current state, yesterday I changed every cell. When I opened the same notebook today all the types are reset to Python!

I expect this feature will be realized soon. :)
Could you tell me any progress?

Sorry this isn't on our radar at the moment.

Sorry this isn't on our radar at the moment.

Hi,
Do you have a plan to fix this problem?
For example, I use both Python and Julia in one Jupyter notebook. I found that clicking the lower right button of the cell can demonstrate the menu which controls the syntax highlighting. I guess that If Julia option would be added to the menu, the problem gets solved.

However, I prefer this feature to be a native of Jupyter extension.

This issue is about autodetecting cell magics that change the language. The plan is to make the language change automatically as you type a cell magic. It's not a high priority at the moment but will likely be addressed eventually.
What you're describing is more of a polyglot kernel. The jupyter extension has no plans for a polyglot kernel. However .NET interactive does. A polyglot kernel would likely support ALL languages in that dropdown. @DonJayamanne also mentioned another polyglot kernel in your discussion  of this topic.
The IPython kernel supports only a subset of languages based on known cell magics that can change the language from python to something else. It seems julia is possible in that list though with the julia.magic extension: https://stackoverflow.com/questions/24091373/best-way-to-run-julia-code-in-an-ipython-notebook-or-python-code-in-an-ijulia-n. So when we implement this feature, we should add julia to the list.

Julia language should be added to the menu. However, I still need to change the cell language every time I open the notebook.
It is better that the call can identify the language of the cell automatically according to the statement: %%script julia.

One more use case that seems not mentioned,
If there would be for example
%%writefile myfile.yaml
some: field
other: field

it'd be nice if it could be detected that this is yaml syntax.

This issue is about autodetecting cell magics that change the language

Instead of autodetecting, which might be tricky, would it be possible to allow a custom mapping? For example, I have a Kusto syntax highlighting extension that would be great to autoswitch when %%kql appears in a cell. Similarly, there may be non-standard names (like %%bigquery) that would nicely map to SQL, but might be tricky to do it automatically...so letting the user map it through a setting could work...? WDYT?

Created an upstream issue https://github.com/microsoft/pylance-release/issues/4969
That would be a better solution, as pylance natively supports notebooks

Closing in favour of upstream issue https://github.com/microsoft/pylance-release/issues/4969


ninjia error

Hi! I want to train your model use my own dataset. Under the environment you used: pytorch 1.1.0, python 3.7, cuda 10.0. But unfortunately, it shows the error "ninja: build stopped: subcommand failed.\n" . Do you meet the same thing? Thanks a lot for your help.

Hi!
Can you provide more information? Like at which step this error happens and other printouts on the screen.

Dear sir:
     Thanks for your reply! I met the mistake with ninja(1.9.0) which is a package installed with pytorch 1.1.0. I run the code on the server at school. But unfortunately, there is wrong with it. I still have no idea after searching for some material. Do you meet the same problem? Sincerely for your help!
    Best wishes!
------------------ 原始邮件 ------------------
发件人:                                                                                "feinanshan/TDNet"<EMAIL_ADDRESS>发送时间: 2020年7月16日(星期四) 晚上9:54
收件人<EMAIL_ADDRESS>抄送: "飞虎联盟"<1160632674<EMAIL_ADDRESS>主题: Re: [feinanshan/TDNet] ninjia error (#5)
Hi!
Can you provide more information? Like at which step this error happens and other printouts on the screen.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Hi, I didn't meet the error as you mentioned. Maybe try with Anaconda to prepare the environment for running?

This "ninja error" is the problem of pytorch-encoding, and I'm also using the same configuration with @feinanshan but facing the same error. Still working to solve it.

Finally, I created a new Anaconda env and reinstall everything. No error now. @littlestar231


Pass context down to storage explorer launchers

Fixes https://github.com/microsoft/vscode-azurestorage/issues/677
callWithTelemetryAndErrorHandling was swallowing the error thrown after x-ing out the notification

As far as I can tell, the Mac function has the exact implementation. Any idea as to why there's no error thrown in that case?

If you cancel out of the notification on Mac, getStorageExplorerExecutable returns an empty string but this line doesn't throw an error in that case.

Honestly we should probably throw a UserCancelError for the Mac scenario as well.  Even though it doesn't throw an error, we should avoid invoking the launcher even with empty parameters.  Who knows what kind of effects it might have.


logic of strategy  example

In Trading System for MarketMakers, the strategy is in strategy.py as
if  volume is long,
sell at bid1 price with volume as free asset
   in examples/huobi_future/strategy.py lines:142-151

        _price = round(self.bid1_price + 0.1, 1)
        volume = float(self.trader.assets.assets.get(self.raw_symbol).get("free")) * price // 100
        if volume >= 1:
            quantity = - volume #  空1张
            action = ORDER_ACTION_SELL
            new_price = str(price)  # 将价格转换为字符串，保持精度
            if quantity:
                orders_data.append({"price": new_price, "quantity": quantity, "action": action, "order_type": ORDER_TYPE_LIMIT, "lever_rate": 1})
                self.last_bid_price = self.bid1_price_

Is this line just demonstration or has implications why this is profitable strategy?
Best regards,
Ingyu

@Ingyu This line is just demonstration.It may not be profitable strategy.


SvelteKit & AMP

Describe the bug
Hello, I have a 500 error on the documentation.
https://kit.svelte.dev/docs/amp
Reproduction

https://www.google.com/search?q=sveltekit+amp

Logs
No response
System Info
nada.

Severity
annoyance

added a redirect


Make Trac read-only, possibly replace by a static dump

A while ago, we made jQuery Core Trac instance (https://bugs.jquery.com/) read-only, making the only still fully functional Trac instance to be the jQuery UI one (https://bugs.jqueryui.com/). With jQuery UI being maintained in a very limited way nowadays, it doesn't make sense to maintain a full separate bug tracker just for this project; other projects are using GitHub issues.
We want to enable GitHub issues for jQuery UI and make the UI Trac read-only. In the future, we can consider replacing Trac with a static dump of all its pages if that makes maintenance easier.
I'm not 100% sure what we did for Core but I think we mostly blocked account registration and removed all existing accounts as with no accounts the site is essentially read-only. @rjollos can you help with doing the same for UI? I'm not sure if it was you or someone else involved with the changes for the Core Trac.

Sites are ready to deploy at https://github.com/jquery/bugs.jquery.com and https://github.com/jquery/bugs.jqueryui.com. Note that some redirects will need to be deactivated.

Note that some redirects will need to be deactivated.

@timmywil Which ones? I propose to do bugs.jqueryui.com first so let me know which redirects needs to be removed.

Sites are ready to deploy at https://github.com/jquery/bugs.jquery.com and https://github.com/jquery/bugs.jqueryui.com.

Could you update the deploy job to push the result to a branch? See qunitjs.com workflow for example. This way we have a copy of the static site HTML permanently, rather than dependent on current OS/Node/npm packages working exactly in the future to reproduce the site. It also has as a bonus the ability to diff changes (if any) in the future, and means I can securely deploy as a simple static site that pulls from a public repo (no credentials, no push) without executing any code on the server.
I will include a webhook the org-wide jquery hook since it's easy to do, so that updates are immediately live, same as GitHub Pages.

Which ones?

Any and all related to these sites. I know that https://bugs.jquery.com/newticket currently redirects to core github (well, it used to. It's actually broken now and goes to a 404 page). That will no longer be necessary.

Could you update the deploy job to push the result to a branch?

Will do

Which ones?

Any and all related to these sites. I don't think I have access to see the list, but redirects will no longer be necessary. I know that https://bugs.jquery.com/newticket currently redirects to core github (well, it used to. It's actually broken now and goes to a 404 page). That will no longer be necessary. It may just be /newticket on both sites, but it'd be good to double check.

In Cloudflare for bugs.jqueryui.com, I see no redirect rules.
On the old Trac server, it has a local Nginx proxy configured in the private repo at https://github.com/jquery/infrastructure/blob/33557055ee/modules/jquery/templates/nginx/trac.conf.erb#L35. That seems to define /newticket as the only redirect.
In any event, if we're only worried about disabling redirects, then we don't have to look any further. When we switch the domain to a new server serving a static site, the old server and any redirects it may have naturally go away from the public POV.

Trac sites are now being deployed to branches. I made the repos public. I was just cautious when I first started, but everything in there is public already.

@timmywil I've switched over bugs.jqueryui.com. It looks like pagefind.js is missing from the latest few builds. This wasn't visible on the github.io preview because that was still sourced from an earlier commit, based on the now-non-existant "action".
To confirm whether bugs.jquery.com is affected as well, I've updated the Pages settings at https://github.com/jquery/bugs.jquery.com/ from Action to Branch: gh-pages. It appears to be working fine there.
No rush. We'll catch up when you're back in two weeks :)

@timmywil @rjollos I've powered off the trac.ops.jquery.net droplet. I'll give it two weeks before deleting. Let me know if you'd like it back on for any thing!

I'll give it two weeks before deleting. Let me know if you'd like it back on for any thing!

Thanks, that shouldn't be a problem.

@timmywil @rjollos I've powered off the trac.ops.jquery.net droplet. I'll give it two weeks before deleting. Let me know if you'd like it back on for any thing!

Thanks, that shouldn't be a problem.

pagefind issue on the ui site should be fixed next deployment. Turns out I forgot to update pagefind.

Looks fixed. Can probably close this ticket now.

I haven't deleted the droplet and Tarsnap backups yet.

Deleted  backups at Tarsnap for trac.ops.jquery.net.
Deleted trac.ops.jquery.net droplet.
Deleted DNS entry for trac.ops.jquery.net.
Archived Puppet provisioning for Trac.


Bump Abacus

Authorize on delete err docs | Remove env dump from deploy scripts
Signed-off-by: Nikolay Maslarski<EMAIL_ADDRESS>

Coverage remained the same at 89.323% when pulling 4da1f54f3132175b67e300265dac8be50f4d7ccc on SAP:master into 571925ea43595899a09db550ff334be15ad4e8c3 on cloudfoundry-incubator:master.


Duplication of highlights and annotations

Hi,
First of all I would like to thank you for making this plugin. This has made live a lot easier for me. However, there are a few things I would like to bring to your attention.

The highlights are being imported in duplicates.
I don't know if this by design. An annotation is followed by the highlighted passage rather than the highlighted passage being followed by the annotation as in hypothes.is.
All highlights are not being imported. My highlights from the last 7 days has not be imported.

Thanks again for your effort. Even with the above issues this plugin is a great help.
Cheers
Priyam

Hi Priyam,

The highlights are being imported in duplicates. -> I just tried and it seems to work fine.  How is it duplicating?

I don't know if this by design. An annotation is followed by the highlighted passage rather than the highlighted passage being followed by the annotation as in hypothes.is. -> Check out the nunjucks template if you want to change the way the highlights/annotations are created. Having said that, in Hypothesis annotations are stored under highlights.

All highlights are not being imported. My highlights from the last 7 days has not be imported. -> Seems to work for me. Cna you check if you are getting error in the developer console?

Cheers

Thanks for your prompt reply. Much appreciated 🙏❤.

Say I have 7 annotation. After syncing, I can see these annotations
listed as 1, 2, 3.....7 then again 1, 2, 3.....7.

As you pointed out, in hypothesis the annotations are under the
highlights. But, when imported , the highlights are under annotations.

I don't know why all my highlights are not being imported. I even tried
reset and syncing again but still it didn't work.

I am not a coder so It will take me some time to understand the nunjucks.
Again thanks for making this plugin.
Cheers.
On Sat, 20 Nov, 2021, 18:58 Wei Chen, @.***> wrote:

Hi Priyam,

The highlights are being imported in duplicates. -> I just tried and
it seems to work fine. How is it duplicating?
2.
I don't know if this by design. An annotation is followed by the
highlighted passage rather than the highlighted passage being followed by
the annotation as in hypothes.is. -> Check out the nunjucks template
if you want to change the way the highlights/annotations are created.
Having said that, in Hypothesis annotations are stored under highlights.
3.
All highlights are not being imported. My highlights from the last 7
days has not be imported. -> Seems to work for me. Cna you check if
you are getting error in the developer console?
Cheers
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/weichenw/obsidian-hypothesis-plugin/issues/19#issuecomment-974650531,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWOZTF42Q5NNYJ5TBIUZ2N3UM6O7DANCNFSM5IMFOJDQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

I can only suggest that you delete all the generated files and reset the sync. Copy the nunjucks template below. Then sync everything again.
Please make sure you got the latest version as well.
Below is the default nunjucks template that I am using personally. You can copy and paste it into your configuration.
{% if is_new_article %}# {{title}}
Metadata
{% if author %}- Author: {{author}}{% endif %}

Title: {{title}}
{% if url %}- Reference: {{url}}{% endif %}
Category: #article{% endif %}

{% if is_new_article -%}## Highlights{%- endif %}
{% for highlight in highlights -%}- {{highlight.text}} — Updated on {{highlight.updated}}
{% if highlight.tags | length %}   - Tags: {% for tag in highlight.tags -%}#{{tag| replace(" ", "-")}} {%- endfor %}{%- endif %}
{% if highlight.annotation %}   - Annotation: {{highlight.annotation}}{%- endif -%}{%- endfor -%}

Hi @weichenw , Thanks so much for making this plugin!
Unfortunately, I am seeing the same behavior that @bpriyum described. I tried deleting and syncing multiple times, as well as manipulating the template.

Hi, there is a possibility that an error occurs before all the sync finishes. Could you please check if there are errors in the console?
https://forum.obsidian.md/t/how-to-access-the-console/16703
I just tried groups. It seems to work fine.

No errors in the console. Same with me not having annotations for the past couple of days as well.
I wonder if it is on Hypothesis side...

No errors in the console. To be clear, I am seeing both the duplication error and the not-pulling-recent error.
I wonder if it is on Hypothesis side...

Unfortunately, I am unable to replicate this issue. I have tried several times it works for me.
Just to confirm that you are on the latest version, reset the history and delete all the files before resyncing everything again.

I have traced the issue down to the pagination of API query in getHighlights
This is what the function looks like in my sources, which differs from https://github.com/weichenw/obsidian-hypothesis-plugin/blob/d209b3c00faaaf02b6c6529a97cb529c2d48720e/src/api/api.ts#L50
async getHighlights(t) {
                    let n, i = 0, r = 1e3, o = !0, s = [];
                    const a = t ? `&search_after=${e.moment.utc(t).format()}` : "";
                    for (let l = 0; l < r && i <= r; i += 2) {
                        try {
                            n = await fetch(`${this.baseUrl}/search?user=${this.userid}&offset=${i}&limit=200&sort=updated&order=asc` + a, {
                                headers: Object.assign({}, this.getHeaders())
                            })
                        } catch (t) {
                            return new e.Notice("Error occurs. Please check your API token and try again."),
                            void console.log("Failed to fetch highlights : ", t)
                        }
                        if (!n || !n.ok)
                            return new e.Notice("Sync failed. Please check your API token and try again."),
                            void console.log("Failed to fetch highlights : ", n);
                        {
                            const e = await n.json();
                            if (!e.rows.length)
                                break;
                            if (o && e.total <= r && (r = e.total,
                            o = !o),
                            l += e.rows.length,
                            s = [...s, ...e.rows],
                            t)
                                break
                        }
                    }
                    return s
                }

I have confirmed that I am using the up-to-date version here:

@mschrader15 Thanks, mate. I have merged and bumped the version

@weichenw no problem!


[New] jsx-no-literals: new config for allowedProps

Fixes #3003
This is adding a new property allowedProps to whitelist props allowed to have string literals. Some examples are class id and etc.

I've rebased this.
I think it needs to be per-component - I'd think you wouldn't want to allow a prop on every component, only on the specific component(s) that use that prop.

It’s really unfortunate you deleted the fork; this PR is now unrecoverable.


Support input name override and other attrs like required

Noticed a few things when I was building out some forms.
Summary by Sourcery
Enhance the SaladUI.Input module to support input name overrides and additional HTML attributes, improving form customization capabilities.
New Features:

Allow overriding the input name attribute in the SaladUI.Input module.

Enhancements:

Add support for additional HTML attributes such as 'required', 'autocomplete', and 'maxlength' in the SaladUI.Input module.

Sourcery is pretty cool. The summary is pretty accurate.

@forest thanks for your PR, but there is a point that I have to consider.
Copy from review comment for tracking.
This change will cause a huge impact on all input component. Because when you define an attribute without default value, and you don't pass that attribute to the component, then it will raise an exception when accessing that attribute within that component.
Example:
attr :name, :string

def input(assigns) do
~H"""
<input name={@name} />
"""
end

this is okay
<.input name="title"/>

but this raise exception
<.input id="my-input"/>

I think it needs a big refactor to be able to override name and value

Good idea. That should work.
On Thu, Oct 10, 2024 at 8:01 AM Dung Nguyen @.***>
wrote:

@.**** commented on this pull request.
In lib/salad_ui/helpers.ex
https://github.com/bluzky/salad_ui/pull/51#discussion_r1795633244:

@@ -9,8 +9,8 @@ defmodule SaladUI.Helpers do
assigns
|> assign(field: nil, id: assigns[:id] || field.id)
# |> assign(:errors, Enum.map(field.errors, &translate_error(&1)))

|> assign(:name, if(assigns[:multiple], do: field.name <> "[]", else: field.name))
|> assign(:value, field.value)

|> assign_new(:name, fn -> if(assigns[:multiple], do: field.name <> "[]", else: field.name) end)
|> assign_new(:value, fn -> field.value end)

I think we can handle this way
...|> assign(:name, assign[:name] || if(assigns[:multiple], do: field.name <> "[]", else: field.name))|> assign(:value, assigns[:value] || field.value)...
with this we don't have to remove default value for name and value
attributes
—
Reply to this email directly, view it on GitHub
https://github.com/bluzky/salad_ui/pull/51#discussion_r1795633244, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAB5FML4XTZAN3C7SGFYIDZ22JGNAVCNFSM6AAAAABPTHVPECVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDGNRQGY3DENRRGA
.
You are receiving this because you were mentioned.Message ID:
@.***>

I fix this in another PR, so close this PR.
Thank you


Problem with column filtering caused by Virtual

material-react-table version
1.9.1
react & react-dom versions
18.2.0
Describe the bug and the steps to reproduce it
Our project uses MRT with infinite scrolling and virtualizer and manualFiltering. I'm trying to upgrade to MRT 1.9.1 and I encountered the following bug: when I select "Filter by a column" and enter something that returns no results, the correct message is displayed (No results found). But when I enter another character to filter by, the screen goes completely blank and I get an error:

I tried out several MRT versions and found out that this bug was introduced in 1.7.4 so I thought Virtual is probably causing this. And yes. I commented out everything in our code related to Virtual and the problem disappeared.
Minimal, Reproducible Example - (Optional, but Recommended)
Our table props:
columns={columns}
data={flatData}
enablePagination={false}
enableGlobalFilter={false}
enableTopToolbar
enableRowSelection
getRowId={(originalRow) => originalRow.id}
enableRowVirtualization
manualFiltering
manualSorting
rowVirtualizerInstanceRef={rowVirtualizerInstanceRef}
rowVirtualizerProps={{ overscan: 12 }}
Screenshots or Videos (Optional)
No response
Do you intend to try to help solve this bug with your own PR?
No, because I do not know how
Terms

[X] I understand that if my bug cannot be reliably reproduced in a debuggable environment, it will probably not be fixed and this issue may even be closed.

hmmm. I cannot quite reproduce this in the virtualized example. Might have something to do with the manual* props

If you are using scrollToIndex, temporarily wrap it in a try/catch. This is a problem with TanStack Virtual itself. I'll investigate further
useEffect(() => {
    //scroll to the top of the table when the sorting changes
    try {
      rowVirtualizerInstanceRef.current?.scrollToIndex?.(0);
    } catch (error) {
      console.error(error);
    }
  }, [sorting]);


test for comparing surfaces when there is no common value

This tests that a warning is thrown when two adjacent volumes don't share a common level value in the the two values that are used to generate a single isovolume. Expected behavior is that there is a common value, but if there is not, there is a warning and a value of 0 is used. (function being tests __compare_surfs())

this is looking good.

this is looking good.

alright I think I applied all the suggested changes. I also added a quick test description to any that were missing a docstring since that was a comment in another PR.

alright I think I applied all the suggested changes. I also added a quick test description to any that were missing a docstring since that was a comment in another PR.


make input object fields optional

Currently all values in the reservation object must be set, instead of e.g. only the seats. The rest of the values is set in the process

When calling the REST API, all values need to be set instead of just userId and seats

using single process variables instead


Change the game's taskbar icon into the game's icon

So currently, if you run the game, the icon is like this:

And change it to be like this icon:

I'd like it to be the same as the game icon, I'm not sure if it's achievable in the game's current format or if I should change it to be an executable first.

can i work on this? @R0merol

Yes! Much appreciated.

you mean game's launcher icon to change to games 123 icon?

The one that is located on the taskbar, not the game window. The 123 icon is already on the game window.

like this?

this is not possible for .py format
so i compiled the app to .exe format


Pagination is missing after adding/deleting models

Describe the bug

Pagination is missing after adding/deleting models
Steps to reproduce

Deploy more than 10 models.
Adding or deleting a model.

Results

Environment

GPUStack version: main e4a7bf3
UI version: 66e506d

Additional context
Add any other context about the problem here.

Verified on UI version c8c22b3


Updating Changelog and Readme

Code Coverage is 82.7%, details at https://s3.amazonaws.com/rs-code-coverage/rsc/2017-03-16_815_Coverage-82.7.txt (sometimes the URL can take a few minutes to be available)


cast send --interactive asks for sender address

Component
Cast
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
cast 0.2.0 (394f217 2023-03-21T00:11:00.708322Z)
What command(s) is the bug in?
cast send
Operating System
macOS (Apple Silicon)
Describe the bug
This command worked as expected up until yesterday:
cast send --rpc-url https://alpha-rpc.scroll.io/l2 0x5300000000000000000000000000000000000005 "withdraw()" --interactive --legacy

After upgrading cast, now I get this error:
Error:
No wallet or sender address provided. Consider passing it via the --from flag or
setting the ETH_FROM env variable or setting in foundry.toml

It still doesn't work if I pass --from (the method eth_sendTransaction does not exist/is not available). It does work if I pass --private-key.

Unable to reproduce, assuming this has been resolved
cast send --rpc-url https://rpc.scroll.io/l2 0x5300000000000000000000000000000000000005 "withdraw()" --interactive --legacy  now asks for the private key and not for a corresponding sender address
Feel free to re-open if you still experience an issue


Add tests for Power class

This seems also ready, right @joaofoes?

Yeah exactly. One more thing, there's some funcitons that returns more than one self value (it isnt a functional function). For the test, i should test all the self parameters that the function creates or if i test just one of them is enough? In my thought it is necessary to test all of them but i would like a confirmation. @rafaellehmkuhl

Yes, you need to test for all the relevant modifications the function introduces.

Fixes #143

I've noticed that the error is related to coverage percentage. I don't quite know what that means, and in fact i've didn't changed anything on the code itself, just on the imports.

Don't worry about the coverage check. It will fail sometimes when it thinks there are already tests for a function but it doesn't.
Travis is the one to watch for. It has to pass, always.

Fixes #143


Unable to build on Idris 0.10.3

When building on Idris 0.10.3 I get an error that '%access public' has been deprecated.

Closing since 0.10.3 is outdated in favor of #8


cant find gotags

Miss gotags or not in PATH
this error is thrown by neovim even after installing gotags and adding it to path

Miss gotags or not in PATH this error is thrown by neovim even after installing gotags and adding it to path

You can try :!gotags in nvim.

it says
"zsh could not find gotags"

You should find a way to make gotags accessible in nvim, check your nvim default shell, and configure environment variables

i have installed gotags its in usr/bin/gotags how do i add that in my golang project is there an resourse where i can learn this


[Bug]: #1 is still present on 1.19.2

General Info

[X] I am running a modpack
[X] I can reproduce this issue consistently in single-player
[X] I can reproduce this issue consistently in multi-player
[X] I have searched for this issue previously and it was either (1) not previously reported, or (2) previously fixed and I am having the same problem.
[ ] I am crashing and can provide my crash report(s)
[X] I am using the latest version of the mod

Forge version
43.3.5
Mod version
2.0.1
Java version
Java 17
Issue Description
#1 Also occurs on 1.19.2, and wasn't fixed when that issue was fixed on 1.20.
Additional Information

Here's the resulting head's nbt data:
minecraft:player_head{SkullOwner: {Properties: {textures: [{Value: "eyJ0ZXh0dXJlcyIgOiB7CiAgICAiU0tJTiIgOiB7CiAgICAgICJ1cmwiIDogImh0dHA6Ly90ZXh0dXJlcy5taW5lY3JhZnQubmV0L3RleHR1cmUvOTRjN2Q5MTM2MTJlZjBhODllMzVmZDc1MDEzNjIyYTQyZjg0ZGEyYzhmYzc4YWMxOWJkMDI5YmViMWExZWQ5MiIKICAgIH0sCiAgICAiQ0FQRSIgOiB7CiAgICAgICJ1cmwiIDogImh0dHA6Ly90ZXh0dXJlcy5taW5lY3JhZnQubmV0L3RleHR1cmUvZjlhNzY1Mzc2NDc5ODlmOWEwYjZkMDAxZTMyMGRhYzU5MWMzNTllOWU2MWEzMWY0Y2UxMWM4OGYyMDdmMGFkNCIKICAgIH0KICB9Cn0="}]}, Id: [I; -1, -1492768929, -1, -1492768929]}, display: {Name: '{"text":"StormDragon_64's Head"}'}, FiveHeadScale: 0.0f}

Backported the fix


Clarify why there are three sets of parameters in the Syntax section

Please clarify what each of the three sets of parameters in the Syntax section is for.  Are they for Exchange Server 2010, Exchange Server 2013, and Exchange Server 2016, respectively?

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

ID: cf5b0c5d-0466-d55a-1f7b-0b2d25a316fd
Version Independent ID: 07b0c093-eb6b-4834-3cd4-d8f2af6ca0f5
Content: Set-Mailbox (exchange)
Content Source: exchange/exchange-ps/exchange/mailboxes/Set-Mailbox.md
Service: exchange-powershell
GitHub Login: @chrisda
Microsoft Alias: chrisda

@VShrubowich, a cmdlet reference topic for any product will have multiple Syntax blocks due to the presence of mutually exclusive parameters on the cmdlet (parameters that's can't be used together in the same command). This is (hopefully) explained in the topic https://docs.microsoft.com/en-us/powershell/exchange/exchange-server/exchange-cmdlet-syntax that's linked to from the Synopsis section. And for a cmdlet like Set-Mailbox that has a lot of parameters, it can be difficult to find the unique parameters that exist in one Syntax block but not another (I call it "playing Where's Waldo?").
To your point about 2010/2013/2016, we did combine the separate versions of Set-Mailbox on TechNet into the single version on GitHub, which could have caused separate Syntax blocks for a unique but mutually exclusive parameter that's only available in one version of the product but not the others. And, Set-Mailbox is one of those cmdlets that's available in on-premises Exchange and in Exchange Online. So, some Syntax blocks for this cmdlet are definitely caused by mutually exclusive parameters between on-prem and the cloud (for example UserPrincipalName and MicrosoftOnlineServicesID).
For simpler cmdlets with fewer parameters, the Syntax blocks can be useful to quickly determine which parameters can or can't be used together in the same command. But, for big, complex cmdlets like Set-Mailbox, there are so many parameters and so many Syntax blocks, that they become effectively useless. If you try to use parameters that can't be used together in the same command, you'll get an error, anyway.


Add Broken Links test to new test dir

In Issue #450, we studied how to do a broken links test on our website, using Cypress.
Now, we will implement this test.

[ ] In the test directory, add a Broken Link test using Cypress

Implemented first e2e test for broken links, PR: https://github.com/saucelabs/elemental-next/pull/530


App crashing in jetson nano board (facerec_from_webcam_multiprocessing.py)

face_recognition version:latest
Python version:3.6
Operating System: Jetson Nano

Description
when i run the facerec_from_webcam_multiprocessing.py code , then the app getting crash with the below pasted trace logs.
but facerec_from_webcam.py code running in both Gpu and Cpu mode. ( HOG / CNN)
Paste the command(s) you ran and the output.
If there was a crash, please include the traceback here.

Traceback (most recent call last):
File "/usr/lib/python3.6/multiprocessing/process.py", line 258, in _bootstrap
self.run()
File "/usr/lib/python3.6/multiprocessing/process.py", line 93, in run
self._target(*self._args, **self._kwargs)
File "facerec_from_webcam_multiprocessing.py", line 86, in process
frame_process = read_frame_list[worker_id]
File "", line 2, in getitem
File "/usr/lib/python3.6/multiprocessing/managers.py", line 772, in _callmethod
raise convert_to_error(kind, result)
KeyError: 1

I just approved a PR that updated the code for that demo. Try grabbing the latest code for that demo from github and running it again.

@ageitgey ,
Thanks for the reply.
I have grabbed the updated code and now APP is not crashing but the app looks like extremely slow with Picamera. FPS is 0.2.
I have reduced the frame size too. but no luck.
So how can I improve my FPS to atleast.
highly appreciated your help on this,


Implementation

I was searching for this one. It would be great to see the implementation of this.
Thanks

I will try make examples on this week.


Handle relations with the same name

Problem
model Post {
  id   String @id
  user String
  User User   @relation(fields: [author], references: [id])
}

model User {
  id   String @id
  name String
}

The author and Author fields is a problem.
Go uses capital letters to indicate that this method is public, so author and Author resolve to the same Author() method.
Possible Solutions
While it works in the JS client because you just type author / Author depending on what you want, it can’t be mapped to the Go client, since all fields are uppercased. One of the fields has to be renamed, for example author to authorID.

We could just forbid these ambiguous fields this when using the Go client with a proper warning. However this may be annoying when re-introspecting or when importing existing schema files. We could recommend to suffix the scalar fields with ID, e.g. authorID, and then the virtual relation could be called author.

We accept the schema but forbid querying for IDs directly, then there should be no conflicts. The JS equivalent would look as follows for this solution:
forbid this go equivalent query:
const result = await client.comment.findMany({
	where: {
		user: USER_ID,
	},
})

force this instead:
const result = await client.comment.findMany({
	where: {
		User: { id: USER_ID },
	},
})

This solution is probably bad because it results in a different, less efficient SQL query.

Rename the fields in the Go client if they conflict? Probably also very confusing. We could also for example underscore conflicting names (e.g. Author=>_Author), or suffix the relation names with a word (e.g. Author->AuthorRelation). In both cases, we would need to make sure to make sure there are no additional conflicts, i.e. there is no field already called _Author or AuthorRelation.

Incorporate the scalar fields into the virtual relation attribute; a suggestion by @RafaelKR. This could look as follows:
result, err := client.Post.FindMany(
	db.Post.Author.ID.Equals("id"),
).Exec(ctx)

This information would need to be extracted from the virtual relation attribute's referenced field name. It might be a bit confusing, since if the field would stay the same you would write Author.Author or if it would be just called id (Author.ID) it might not be clear what field the user is referring to.

I just want to add a little more context to my suggestion (list item 4 in the opening post https://github.com/prisma/prisma-client-go/issues/140#issue-622961289).
First things first: I'm not a Go developer. So please bear with me if I got some syntax or understanding errors in my following code examples.
Example Schemas
All the following explanations are based on these two examples.
Single IDs
model User {
  id         String   @id
  firstName  String
  lastName   String
}

model Post {
  id      String @id
  author  String
  Author  User   @relation(fields: [author], references: [id])
}

Multi-field IDs
https://www.prisma.io/docs/reference/tools-and-interfaces/prisma-schema/relations#the-relation-attribute
model User {
  firstName  String
  lastName   String

  @@id([firstName, lastName])
}

model Post {
  id               String   @id
  author           User     @relation(fields: [authorFirstName, authorLastName], references: [firstName, lastName])
  authorFirstName  String   // relation scalar field (used in the `@relation` attribute above)
  authorLastName   String   // relation scalar field (used in the `@relation` attribute above)
}

Detailed suggestion
Relation Fields
My suggestion is to always have an .ID for relation fields, regardless of whether there is a collision or not.
e.g. db.Post.Author.ID
Relation Scalar Fields
Scalar fields should be always hidden from the referencing model (Post).
So this scalar fields wouldn't exist in the Go code:
// Single ID
db.Post.Author // this exists, but not as relation scalar, this is the relation field itself

// Multi-field IDs
db.Post.AuthorFirstName
db.Post.AuthorLastName

This would prevent any collsion.
Instead they are always referenced on the relation field. To also prevent collisions with other fields (.ID, .Some, ...) we could introduce another subfield. I'm using .Fields for that.
Now we can access them like this:
Single ID: Post.Author.Fields.Id
Multi-field IDs: Post.Author.Fields.FirstName and Post.Author.Fields.LastName
As you can see I'm using the fieldnames of the referenced model User. The other option would be to use the fieldnames on the current model, but due to the naming redundancy this doesn't look that good IMO.
~Single ID: Post.Author.Fields.Author
Multi-field IDs: Post.Author.Fields.AuthorFirstName and Post.Author.Fields.AuthorLastName~
Querying
Single IDs
If we only have one item in fields we can directly call Methods like .Equals on .ID:
result, err := client.Post.FindMany(
	Post.Author.ID.Equals("id"),
).Exec(ctx)

Multi-field IDs
If we have multiple items in fields the .ID fields becomes a method where we have to query like this:
result, err := client.Post.FindMany(
	Post.Author.ID(
		Author.FirstName.Equals("firstName"), // maybe User instead of Author?
		Author.LastName.Equals("lastName"),   // maybe User instead of Author?
	),
).Exec(ctx)

As said, I'm not a Go developer and I'm not sure if this is working, but after looking at Querying for relations and Create a record with a relation it could be possible.

Why would one have the same fields in the first place and only distinguish based on the casing? That feels like a good way to create confusion. Maybe I'm lacking context: would that come from a database that has been set up this way and can't be changed?
The model used there doesn't seem to make much sense to me: why not use authorId? If the author string is some kind of full name, then one could call it authorFullName etc.
That doesn't mean casing conflict couldn't happen, but I'd rather have an example that makes the problem clearer as this one feels constructed to me.

The example schema is gendered by Prisma introspection; with the field being lowercase and the relation attribute being uppercase. Sorry for the missing context, I thought I had added that to the actual schema.

Alright, that makes more sense. Then for now I'd say we should incentivize using @map and rename when this happens, until we offer ways to configure further the introspection behavior (cf. https://github.com/prisma/prisma/issues/1184)

Missing this feature is preventing us from using golang for a project. :(

@deanshelton913 Can you please elaborate? Are you using introspection with the Go client but run into this problem?


fix(mkimage): log console boot messages to hvc0

VZ has support for serial console logging but the device is different from Qemu. VZ uses the hypervisor virtual console device instead. This change modifies the bootloader to add hvc0 as a device to log console messages to on boot.
Note this has no effect on the tty serial logging.
Issue: https://github.com/lima-vm/lima/issues/1659

I assume you wanted a release with this change: https://github.com/lima-vm/alpine-lima/releases/tag/v0.2.32

Thanks @jandubois and @AkihiroSuda! Just a note on functionality in the Alpine Lima image. Because hvc0 is not configured as the primary console, messages to hvc0 will only work during boot. It's better then no messages, but maybe in the future we could revisit this.


As a developer, I need to update the links to the new documentation pages

The Documentation pages have changed structure. This has made the previous links outdated.
ToDos:

[ ] Change the links to the Question mark icons that appear in different pages on the top right (see list 1)
[ ] Change the order of links, and add one link in the Main menu: Learn (see list 2)
[ ] Fix issue in the Documentation pages: the menu does not collapse properly (check the behavior of the table of contents on the left, that all levels collapse and expand properly, for example, the menu doesn't collapse with third level items (e.g., http://mediasuite.clariah.nl/documentation/howtos/user-projects/create)

LIST 1: Links to update for Question mark buttons to work again
PAGE ->>> OLD LINK TO DOCUMENTATION ->>> NEW LINK TO DOCUMENTATION (UPDATE IN QUESTION MARK ICON)

http://mediasuite.clariah.nl/data ->>> http://mediasuite.clariah.nl/documentation/data ->>> http://mediasuite.clariah.nl/documentation/howtos/data

http://mediasuite.clariah.nl/tools  ->>> http://mediasuite.clariah.nl/documentation/tools ->>> http://mediasuite.clariah.nl/documentation/howtos/tools

http://mediasuite.clariah.nl/tool/collection-inspector  ->>> http://mediasuite.clariah.nl/documentation/tools/collection-inspector ->>> http://mediasuite.clariah.nl/documentation/howtos/collection-inspector

http://mediasuite.clariah.nl/tool/exploratory-search  ->>> http://mediasuite.clariah.nl/documentation/tools/exploratory-search ->>> http://mediasuite.clariah.nl/documentation/howtos/exploratory-search

http://mediasuite.clariah.nl/tool/single-search  ->>> http://mediasuite.clariah.nl/documentation/tools/single-search ->>> http://mediasuite.clariah.nl/documentation/howtos/single-search

http://mediasuite.clariah.nl/tool/query-comparison  ->>> http://mediasuite.clariah.nl/documentation/tools/query-comparison ->>> http://mediasuite.clariah.nl/documentation/howtos/query-comparison

http://mediasuite.clariah.nl/workspace/projects ->>> http://mediasuite.clariah.nl/documentation/workspace/user projects ->>> http://mediasuite.clariah.nl/documentation/howtos/user-projects

http://mediasuite.clariah.nl/workspace/projects/create ->>> http://mediasuite.clariah.nl/documentation/workspace/user-projects/create ->>> http://mediasuite.clariah.nl/documentation/howtos/user-projects/create

http://mediasuite.clariah.nl/workspace/projects/[project ID]/edit ->>> http://mediasuite.clariah.nl/documentation/workspace/user-projects/edit ->>> http://mediasuite.clariah.nl/documentation/howtos/user-projects/edit

http://mediasuite.clariah.nl/workspace/projects/[project ID]/details ->>> http://mediasuite.clariah.nl/documentation/workspace/user-projects/details ->>> http://mediasuite.clariah.nl/documentation/howtos/user-projects/details

http://mediasuite.clariah.nl/workspace/projects/[project ID]/bookmarks ->>> http://mediasuite.clariah.nl/documentation/workspace/user-projects/bookmarks ->>> http://mediasuite.clariah.nl/documentation/howtos/user-projects/bookmarks

http://mediasuite.clariah.nl/workspace/projects/[project ID]/annotations ->>> http://mediasuite.clariah.nl/documentation/workspace/user-projects/annotations ->>> http://mediasuite.clariah.nl/documentation/howtos/user-projects/annotations

http://mediasuite.clariah.nl/workspace/projects/[project ID]/queries ->>> http://mediasuite.clariah.nl/documentation/workspace/user-projects/queries ->>> http://mediasuite.clariah.nl/documentation/howtos/user-projects/queries

http://mediasuite.clariah.nl/workspace/projects/[project ID]/tool-sessions ->>> http://mediasuite.clariah.nl/documentation/workspace/user-projects/tool-sessions ->>> http://mediasuite.clariah.nl/documentation/howtos/user-projects/tool-sessions

http://mediasuite.clariah.nl/workspace/collections/create ->>> http://mediasuite.clariah.nl/documentation/workspace/user-collections/create ->>> http://mediasuite.clariah.nl/documentation/howtos/user-collections/create

http://mediasuite.clariah.nl/workspace/collections/[user collection ID]/edit ->>> http://mediasuite.clariah.nl/documentation/workspace/user-collections/edit ->>> http://mediasuite.clariah.nl/documentation/howtos/user-collections/edit

**LIST 2: Links to update in the Main menu -> Learn **

FAQ: OK
Quick start guide: OK
Research projects: delete from this menu
HowTos: add item to menu, link: http://mediasuite.clariah.nl/documentation/howtos
Troubleshooting: OK
Forum: OK, change label to: Help & feedback forum
Blog: OK, change label to: Blog and News
All Documentation: OK
All screencasts: Delete from this menu

@lilimelgar  @jblom
Issue done in both projects (clariah-mediasuite and labo-components)
branches for testing:
Project labo-components
issues/#346-update-links-labo
Project clariah-mediasuite
issues/#346-mediasuite
To tes update both project with the already mentioned branches.
All acceptance criteria has been done with the exception of point (not clear explanation):

Fix issue in the Documentation pages: the menu does not collapse properly (check the behavior of the table of contents on the left, that all levels collapse and expand properly, for example, the menu doesn't collapse with third level items (e.g., http://mediasuite.clariah.nl/documentation/howtos/user-projects/create)

@lilimelgar : Please check carefully every link. There are still help pages with no content and the message as the following screenshots show:

Regarding list 2. All done but the list does not correspond accurately with the current menu (compare list 2 with the next screenshot)

@lilimelgar : Please check this point also. Once everything is ok let @jblom know so he can merge both branches. If there any changes let me know in the comments to make the corresponding changes.

@avlisodraude @lilimelgar I merged the two and I guess it looks good.
About the menu. It now looks like list 2, except that Troubleshooting is missing. Do I need to add this one?

Tested in production, all links work OK!


Failing  Tests

Bug 1:
Non <Match>/<Miss> components in <MatchGroup> causes errors
Possible Solution:
enforce that the child's type is either Miss or Match
if (matchedIndex || (child.type !== Miss && child.type !== Match))

Bug 2:
Matches outside of <MatchGroup break <Miss>
Possible Solution:
Expanding on bug 1, enforce that all of a <MatchGroup>'s children must be <Match> components. In order to deal with misses, add a miss prop to <MatchGroup> which is a component to render when none of the <Match>es matched.
This avoids subscribing a <Miss> to the parent <MatchProvider> and the possibility of someone including multiple <Miss> components to a <MatchGroup> (I'm not sure why someone would do this, but currently they can and only the last one would render)
<MatchGroup miss={NoMatch} >
  <Match exactly pattern="/" component={Home} />
  <Match pattern="/foo" component={Foo} />
</MatchGroup>

For the first issue, could that not be solved via PropTypes.instanceOf()

I think that these are correct.
Only <Match>
MatchGroup.propTypes = {
  children: React.PropTypes.oneOfType(
    React.PropTypes.instanceOf(Match),
    React.PropTypes.arrayOf(
      React.PropTypes.instanceOf(Match)
    )
  )
}

<Match> or <Miss>
MatchGroup.propTypes = {
  children: React.PropTypes.oneOfType(
    React.PropTypes.instanceOf(Match),
    React.PropTypes.instanceOf(Miss),
    React.PropTypes.arrayOf(
      React.PropTypes.oneOfType(
        React.PropTypes.instanceOf(Match),
        React.PropTypes.instanceOf(Miss),
      )
    )
  )
}


NullInjectorError: No provider for SnotifyToastConfig!

Hi, I have installed as it was shown but got this error.
AuthLoginComponent_Host.ngfactory.js? [sm]:1 ERROR Error: StaticInjectorError(AppModule)[SnotifyToastConfig]:
  StaticInjectorError(Platform: core)[SnotifyToastConfig]:
    NullInjectorError: No provider for SnotifyToastConfig!

Angular CLI: 1.6.8
Node: 9.4.0
OS: darwin x64
Angular: 5.2.4
... animations, common, compiler, compiler-cli, core, forms
... http, language-service, platform-browser
... platform-browser-dynamic, router

@angular/cli: 1.6.8
@angular-devkit/build-optimizer: 0.0.42
@angular-devkit/core: 0.0.29
@angular-devkit/schematics: 0.0.52
@ngtools/json-schema: 1.1.0
@ngtools/webpack: 1.9.8
@schematics/angular: 0.1.17
typescript: 2.5.3
webpack: 3.10.0

I think you may be missing this line
https://github.com/artemsky/ng-snotify/blob/master/example/app/app.module.ts#L19


Display statistical figures

Raising this on behalf of @patrickjfl and @vmilitaru as it's safer this way. The original plan was to me to work on this branch and add CSS here but I need actually need to merge this in feature/category-selector-and-map-key hence raising it separately.

LGTM :-P

Turns out I cannot approve a PR raised by myself even though it's not my work. Bummer.

Actually, just for the record: LGTM, @patrickjfl @vmilitaru please approve.


Error executing cantools (arxml.py)

When running a script that uses cantools, right away it gives this error
Traceback (most recent call last):

  File "./ea_psb-x_manager.py", line 5, in <module>
    import cantools
  File "/home/pi/.local/lib/python3.5/site-packages/cantools/__init__.py", line 9, in <module>
    from . import database as db
  File "/home/pi/.local/lib/python3.5/site-packages/cantools/database/__init__.py", line 6, in <module>
    from . import can
  File "/home/pi/.local/lib/python3.5/site-packages/cantools/database/can/__init__.py", line 1, in <module>
    from .database import Database
  File "/home/pi/.local/lib/python3.5/site-packages/cantools/database/can/database.py", line 3, in <module>
    from .formats import arxml
  File "/home/pi/.local/lib/python3.5/site-packages/cantools/database/can/formats/arxml.py", line 25
    raise ValueError(f"No XML namespace specified or illegal root tag name '{self._root.tag}'")
                                                                                ^
SyntaxError: invalid syntax

I can't find no one with a similar error. Have already tried installing/unistalling with pip and pip3. For context, I am running it in a RevPi.

Disclaimer: Never used arxml.
Can you post your arxml? It seems to be invalid. The namespace tag should be the root node in the document (the very first line). And it seems to stumble there already.

From a sample file:
‘

2
3
‘

Sure, actually the file is quite large, however, this is the first part where it already seems to fail
# Load and dump a CAN database in ARXML format.

import re
import logging
from decimal import Decimal

from xml.etree import ElementTree

from ..signal import Signal
from ..signal import Decimal as SignalDecimal
from ..message import Message
from ..internal_database import InternalDatabase

LOGGER = logging.getLogger(__name__)

class SystemLoader(object):
    def __init__(self, root, strict):
        self._root = root
        self._strict = strict

        m = re.match('^\\{(.*)\\}AUTOSAR$', self._root.tag)

        if not m:
            raise ValueError(f"No XML namespace specified or illegal root tag name '{self._root.tag}'")

        xml_namespace = m.group(1)
        self.xml_namespace = xml_namespace
        self._xml_namespaces = { 'ns': xml_namespace }

        m = re.match('^http://autosar\.org/schema/r(4\..*)$', xml_namespace)
        if m:
            # AUTOSAR 4
            autosar_version_string = m.group(1)
        else:
            raise ValueError(f"Unrecognized AUTOSAR XML namespace '{xml_namespace}'")

        m = re.match('^([0-9]*)(\.[0-9]*)?(\.[0-9]*)?$', autosar_version_string)
        if not m:
            raise ValueError(f"Could not parse AUTOSAR version '{autosar_version_string}'")

        self.autosar_version_major = int(m.group(1))
        self.autosar_version_minor = 0 if m.group(2) is None else int(m.group(2)[1:])
        self.autosar_version_patch = 0 if m.group(3) is None else int(m.group(3)[1:])

        if self.autosar_version_major != 4:
            raise ValueError('This class only supports AUTOSAR version 4')

        self._arxml_reference_cache = {}

I meant your arxml file. It seems to be missing the AUTOSAR tag.

Oh yeah I see, the error should be in the arxml file. I however have to find the file it is taking because like I am just using cantools to load a .dbc file and use it, i am not using or loading any arxml myself

So you do not use arxml at all? Please post your invokation of cantools so that we can see what you are trying to do.

I mean, just doing the import cantools fails, without doing anything else at all. Without import cantools the code works, adding import cantools it fails

Could you post your code? I would not expect that part of cantools getting invoked on import at all.

Also: type python to get a repl, then type import cantools and see if that throws.

Oh it already fails, got:
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "build/bdist.linux-armv7l/egg/cantools/__init__.py", line 4, in <module>

  File "build/bdist.linux-armv7l/egg/cantools/tester.py", line 15, in <module>
  File "/usr/local/lib/python2.7/dist-packages/python_can-4.0.0.dev0-py2.7.egg/can/__init__.py", line 16
    rc: Dict[str, Any] = dict()
      ^
SyntaxError: invalid syntax

Ah. Invalid syntax. Probably the f string. Try updating python3 please. E.g. I am on 3.8.2.
f strings are supported in Python since 3.6. Thats your issue.

Dang, there it is! I will update the version and it should work. Thanks a lot for you help! :)