Single Fetch: bad behavior in promises that resolve to undefined and null

Reproduction
https://stackblitz.com/edit/remix-run-remix-wfeots?file=app%2Froutes%2F_index.tsx
System Info
System:
    OS: Linux 5.0 undefined
    CPU: (8) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
    Memory: 0 Bytes / 0 Bytes
    Shell: 1.0 - /bin/jsh
  Binaries:
    Node: 18.18.0 - /usr/local/bin/node
    Yarn: 1.22.19 - /usr/local/bin/yarn
    npm: 10.2.3 - /usr/local/bin/npm
    pnpm: 8.15.6 - /usr/local/bin/pnpm
  npmPackages:
    @remix-run/dev: * => 2.9.1
    @remix-run/node: * => 2.9.1
    @remix-run/react: * => 2.9.1
    @remix-run/serve: * => 2.9.1
    vite: ^5.1.0 => 5.2.10

Used Package Manager
npm
Expected Behavior

Returning a promise that resolves to null from the loader shouldn't crash when resolved on the client.
Returning a promise that resolves to undefined from the loader shouldn't hang on the client.

Actual Behavior

It crashes.
It hangs.

Null should be safe if you upgrade turbo-stream in your project to the latest version. Undefined is an issue with the <Await> component and will be fixed in a react-router release.

This should be available in the next release

2.9.2 is released - https://github.com/remix-run/remix/releases/tag/remix%402.9.2


Adding devportal theme related docs

Purpose
Added new docs for..

Enabling API detail tabs ( credentials, details, etc.. )
Override API Overview page per API
Enable disable rating
Enable or disable home page
Enable or disable tag cloud

Goals

Describe the solutions that this feature/fix will introduce to resolve the problems described above

Approach

Describe how you are implementing the solutions. Include an animated GIF or screenshot if the change affects the UI (email<EMAIL_ADDRESS>to review all UI text). Include a link to a Markdown file or Google doc if the feature write-up is too long to paste here.

User stories

Summary of user stories addressed by this change>

Release note

Brief description of the new feature or bug fix as it will appear in the release notes

Documentation

Link(s) to product documentation that addresses the changes of this PR. If no doc impact, enter “N/A” plus brief explanation of why there’s no doc impact

Training

Link to the PR for changes to the training content in https://github.com/wso2/WSO2-Training, if applicable

Certification

Type “Sent” when you have provided new/updated certification questions, plus four answers for each question (correct answer highlighted in bold), based on this change. Certification questions/answers should be sent to<EMAIL_ADDRESS>and NOT pasted in this PR. If there is no impact on certification exams, type “N/A” and explain why.

Marketing

Link to drafts of marketing content that will describe and promote this feature, including product page changes, technical articles, blog posts, videos, etc., if applicable

Automation tests

Unit tests

Code coverage information

Integration tests

Details about the test cases and coverage

Security checks

Followed secure coding standards in http://wso2.com/technical-reports/wso2-secure-engineering-guidelines? yes/no
Ran FindSecurityBugs plugin and verified report? yes/no
Confirmed that this PR doesn't commit any keys, passwords, tokens, usernames, or other secrets? yes/no

Samples

Provide high-level details about the samples related to this feature

Related PRs

List any other related PRs

Migrations (if applicable)

Describe migration steps and platforms on which migration has been tested

Test environment

List all JDK versions, operating systems, databases, and browser/versions on which this feature/fix was tested

Learning

Describe the research phase and any blog posts, patterns, libraries, or add-ons you used to solve the problem.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.wso2 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

For reference, the docs are as follows.
Enable or disable tag cloud - WSO2 API Manager Documentation 3.0.0.pdf
Enable or disable home page - WSO2 API Manager Documentation 3.0.0.pdf
Enable or disable rating - WSO2 API Manager Documentation 3.0.0.pdf
Override API overview page per API - WSO2 API Manager Documentation 3.0.0.pdf
Enabling or disabling api detail tabs - WSO2 API Manager Documentation 3.0.0.pdf
Change Default View - WSO2 API Manager Documentation 3.0.0.pdf
Categorizing and grouping APIs - WSO2 API Manager Documentation 3.0.0.pdf
Overriding Developer Portal Theme - WSO2 API Manager Documentation 3.0.0.pdf
Advanced customization - WSO2 API Manager Documentation 3.0.0.pdf


Panic when running on files that don't end with a newline

Reproducer:
$ echo 'print(4)' > test.py
$ cargo run test.py
    Finished dev [unoptimized + debuginfo] target(s) in 0.09s
     Running `target/debug/rustpython test.py`
4
$ echo -n 'print(4)' > test.py
$ cargo run test.py
    Finished dev [unoptimized + debuginfo] target(s) in 0.10s
     Running `target/debug/rustpython test.py`
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: "UnrecognizedToken { token: None, expected: [\"\\\"!=\\\"\", \"\\\"%\\\"\", \"\\\"%=\\\"\", \"\\\"&\\\"\", \"\\\"&=\\\"\", \"\\\"(\\\"\", \"\\\")\\\"\", \"\\\"*\\\"\", \"\\\"**\\\"\", \"\\\"**=\\\"\", \"\\\"*=\\\"\", \"\\\"+\\\"\", \"\\\"+=\\\"\", \"\\\",\\\"\", \"\\\"-\\\"\", \"\\\"-=\\\"\", \"\\\".\\\"\", \"\\\"/\\\"\", \"\\\"//\\\"\", \"\\\"//=\\\"\", \"\\\"/=\\\"\", \"\\\":\\\"\", \"\\\"<\\\"\", \"\\\"<<=\\\"\", \"\\\"<=\\\"\", \"\\\"=\\\"\", \"\\\"==\\\"\", \"\\\">\\\"\", \"\\\">=\\\"\", \"\\\">>=\\\"\", \"\\\"@\\\"\", \"\\\"@=\\\"\", \"\\\"[\\\"\", \"\\\"\\\\\\\\n\\\"\", \"\\\"]\\\"\", \"\\\"^\\\"\", \"\\\"^=\\\"\", \"\\\"and\\\"\", \"\\\"as\\\"\", \"\\\"in\\\"\", \"\\\"is\\\"\", \"\\\"not\\\"\", \"\\\"or\\\"\", \"\\\"|\\\"\", \"\\\"|=\\\"\"] }"', libcore/result.rs:945:5
note: Run with `RUST_BACKTRACE=1` for a backtrace.

Right. I think this issue needs an investigation of what the cpython lexer does. For this, the tokenize module is handy, and can be used to determine the token sequence both with and without trailing newline.

The difference is literally just the newline:
$ cat test.py
print(4)%                                                                                $ cat test2.py
print(4)
$ diff -u <(python3 -m tokenize test.py) <(python3 -m tokenize test2.py)
--- /proc/self/fd/11	2018-08-02 23:07:18.559090398 -0400
+++ /proc/self/fd/12	2018-08-02 23:07:18.559090398 -0400
@@ -3,4 +3,5 @@
 1,5-1,6:            OP             '('
 1,6-1,7:            NUMBER         '4'
 1,7-1,8:            OP             ')'
+1,8-1,9:            NEWLINE        '\n'
 2,0-2,0:            ENDMARKER      ''

When doing a trace on the non-EOL code we see that the "None" token is used to represent the end of the input:
TRACE 2018-08-03T03:11:14Z: rustpython_parser::lexer: Lex token Some(Ok((1, Name { name: "print" }, 6))), nesting=0, indent stack: [0]
TRACE 2018-08-03T03:11:14Z: rustpython_parser::lexer: Lex token Some(Ok((0, Lpar, 0))), nesting=1, indent stack: [0]
TRACE 2018-08-03T03:11:14Z: rustpython_parser::lexer: Lex token Some(Ok((7, Number { value: 4 }, 99999))), nesting=1, indent stack: [0]
TRACE 2018-08-03T03:11:14Z: rustpython_parser::lexer: Lex token Some(Ok((0, Rpar, 0))), nesting=0, indent stack: [0]
TRACE 2018-08-03T03:11:14Z: rustpython_parser::lexer: Lex token None, nesting=0, indent stack: [0]

As best as I can tell, we don't have an equivalent to the ENDMARKER token. (And following from that, I think this mean that writing a rule which matches EOL-less input also matches any sub-input.)

It does not panic anymore at the time of writing, but still inconsistent with python.
second.py (no newline at the end)
value = 1

$ cargo run -- second.py

SyntaxError: Got unexpected EOF at line 0

I'll dive into it.

@OddBloke and @apatrushev this should now be fixed. Can this issue be closed?


Text Overflow.fade for multiline text

Use case
I am trying to achieve the effect of fading out the block text. I am writing this code:
Text(
      'Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. ',
      maxLines: 2,
      style: TextStyle(
        overflow: TextOverflow.fade,
     ),
),

And I get it:

But that's not what I need. I want the text to fade out only on the right side of the last line. That is, the last line should look like this:

Proposal
I want the text to fade out only on the right side of the last line. That is, the last line should look like this:

Maybe something can be corrected inside Flutter to achieve this behavior?

Thanks for the report. I think, if maxLines: 1, then it probably achieves your purpose, as below:

But when it comes to maxLines more than 1, then I am not sure if it is possible to fade only the right corner of the text.
Keeping this issue open and labeling for team's input and feasibility of the proposal.


Traefik keeps showing 404 Error for Docker Containers

Do you want to request a feature or report a bug?
Bug
What did you do?
I have a number of docker containers that are running a number of applications including:
Nextcloud
Wordpress
OSTicket
Which I want to be accessible from a browser for each specific URL.
I used Portainer to set everything up
I used Traefik's docs to set this up.
What did you expect to see?
I expected to see the various services show up depending on the requested URLs.
What did you see instead?
404 Page Not Found
Output of traefik version: (What version of Traefik are you using?)

TOML Configuration:
https://hastebin.com/diqigabofe.toml
https://i.stack.imgur.com/bEu5n.png
https://i.stack.imgur.com/E90OR.png

Thanks for your interest in Traefik !
Every question asked in issues means less features 😉.
The issue tracker is for reporting bugs and feature requests only.
For end-user related support questions, refer to one of the following:

Stack Overflow using the "traefik" tag
the Traefik community Slack channel

@traefiker why close the issue? I followed the documentation exactly and error persists. Either there is a bug or the documentation is lacking or incorrect.
I also already asked on there several weeks ago and days ago and no answer.

@c0fe try to add defaultEntryPoints = ["http"] in your configuration.
duplicate of #2494 #680 #427
See also #2501

@ldez added:
https://hastebin.com/ovagenekaz.cfg
but same result
I also just ran this on a new installation (virtualbox) and same result, 404.

It seems that the automated portion of Traefik Docker configuration isn't working. It appears to be a bug.

@c0fe Do you have some backends registered in your Træfik ?

@mmatur you mean in the container labels?

# Entrypoints to be used by frontends that do not specify any entrypoint.
# Each frontend can specify its own entrypoints.
#
# Optional
# Default: ["http"]
#
defaultEntryPoints = ["http"]

# Entrypoints definition
#
# Optional
# Default:
[entryPoints]
    [entryPoints.http]
    address = ":80"

@ldez Tried it:
https://hastebin.com/omuyihorom.coffeescript
I also tried this tutorial with the same results (error 404):
https://www.digitalocean.com/community/tutorials/how-to-use-traefik-as-a-reverse-proxy-for-docker-containers-on-ubuntu-16-04
@mmatur
Labels set for a nextcloud container:
traefik.backend	cloud
traefik.docker.network	proxy
traefik.enable	true
traefik.frontend.entryPoints	http
traefik.frontend.rule	Host:cloud.mydomain.com
traefik.port	80

@c0fe Could you go to Slack?

@ldez i can tomorrow morning, already heading into bed. but was there few weeks ago, it was dead and never got a response, waited days.


fix status page to exclude aaactl

aaactl doesn't need to check itself so take it off the list

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

Comparison is base (c399495) 71.00% compared to head (d92602e) 71.00%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@              Coverage Diff              @@
##           prep-release     #142   +/-   ##
=============================================
  Coverage         71.00%   71.00%
=============================================
  Files               129      129
  Lines              2218     2218
=============================================
  Hits               1575     1575
  Misses              643      643

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


Ctrl + R in zsh with iTerm2 with tmux opens in a new pane

Category

[x] fzf binary
[x] fzf-tmux script ???
[x] Key bindings

OS

[x] Mac OS X

Shell

[x] zsh

When using fzf in iTerm2 with tmux, the ctrl-r keybinding brings up fzf in a new pane on the last opened tab.
Not sure if it's relevant or not, but I have the following iTerm2 configuration:
"Send text at start: tmux -CC attach -t base || tmux -CC new -s base"
I'll admit that my tmux knowledge is fairly limited, so it's very possible I'm abusing something :)

I'm pretty sure there's something wrong with your configuration. CTRL-R binding uses fzf-tmux to split the pane and it's just tmux split-window, and I can't reproduce the problem in any way.


Move ember-cli-htmlbars to devDependencies

ember-cli-htmlbars is only needed under dependencies when templates are shipped, which is not the case for this addon.

@bertdeblock I know this used to be true but I'm not sure now.
By default ember addon will generate ember-cli-htmlbars as a dependency.
https://github.com/ember-cli/ember-cli/blob/07ec6ffece334508ffd7505fcb3dbdb7df3d6d90/blueprints/addon/index.js#L56
https://github.com/ember-cli/ember-cli/issues/8589

Yes, it was added by default because most addons do ship templates. However, this addon doesn't, so ember-cli-htmlbars can be safely removed from dependencies.


README

README
User Story
As a boot camp student,
I want the prework notes to be structured on a webpage,
So that I can easily find and read the information
Acceptance Criteria
GIVEN the prework project repository,
WHEN I view the README for the repository,
THEN I can see a professional README that provides information on the project and how to use it

I have created my own porofessional readme file


Red Hat Developer Hub does not support empty owner and system labels

Describe the bug
When trying to deploy the backstage plugin, if there are no labels in the service, the plugin tries to put an empty label. This is not supported, hence the service is not added to the catalog.
Expected behavior
Microcks adds a default owner and system labels.
Actual behavior
No service is imported
How to Reproduce?
. Install plugin
. Load service
. Wait for plugin task
Microcks version or git rev
1.8,1
Install method (docker-compose, helm chart, operator, docker-desktop extension,...)
backstage plugin + uber jar
Additional information
No response

I don't think this is Red Hat Developer Hub specific. If no systemLabel or ownerLabel is found on the fetched API, upstream Backstage also refuses to create a new ApiEntity.

Do you mean, we should add some new specific configuration params to add default values for them?

The connector will provide default values to microcks and microcks for both labels. This has been introduced by PR #17 by @hguerrero. Thanks mate!


Actually merge and prepare SCEs for integration in merging script

In #240, we are initiating the integration workflow, the first step which includes merging all of the SCEs from an integration group into one merged SCE file. The focus of that PR is setting up the workflow to read in the metadata files and group the input SCE files by the integration group and pass those files into the merging script. Next we will need to actually input the steps in the merging script which will take the list of input files and produce a merged SCE object, using a function to combine a list of SCE objects.
In addition to producing a merged SCE object we then need to identify the combined HVGs for that merged object and calculate the merged PCA matrix, storing both of those things in the object that is exported.

Closed by #255


repl.it badge won't work for now until the repository is public

Known issue, will be fixed as soon as the repo goes public.

Repo has now mee made public. The badge works as expected 👍


Update message body content to be easier to edit

Since about 2015 apparently, Python has had this concept
of "f" strings, whereby you don't have to concatinate expression
results/values manually between string elements. If you include
a "list" of strings on newlines between ( )'s without commas,
apparently they'll become auto concatinated.

I think you underestimate people. I'd bet most people would use "/n just fine. Like the idea tho!


List the best courses present in the internet for different languages

Till now we only have C++ course, but we need more courses like:
Java
C
Python
Go
C#
R
JavaScript
PHP
Swift
So, the work is to find the best available Free course for the above-listed languages.
If you get any one of them, that's also ok, comment that below and I'll assign you to add the title & links in the code.

Hello sir!
Assign it to me.

Okay, go ahead.

@arbazkhan56 Remember, in this issue, at first you need to search for courses and not directly change the code.

hello @Nishant2907 can i work on this issue

Okay @haripriya9647, first you need to search for courses, send the courses you have selected on the Gitter channel. We will look through it. Then we will confirm you for the PR. We already have a list for C++, find a playlist other than this.


WALL-E role updates

You may wish to have a discussion about this, but I made quite a few changes to get WALL-E working on Galaxy AU. I think these changes should improve interoperability between Galaxy servers, but would require a minor update to EU's playbook to work (additional vars).

Add debug logging with --debug option for walle.py
Modify cron job to fix environment variable issues
Add all required env variables to walle_bashrc
Catch permission error in walle.py
Add --kill option for walle.py to kill malicious jobs with gxadmin
Include a modified version of galay_jwd.py that accepts XML or YAML format for object_store_conf
Ignore Ansible failure of "Clone malware database (WallE)"
Update README

Add debug logging
walle.py --debug was really useful for getting this going. It's very verbose so probably don't want to leave it on in production.
Modify cron job
I found that cron is not able to source our Galaxy .bashrc file, probably because it contains code that is specific to an interactive shell. The result is that none of the env vars that are set in wall_bashrc make it into WALL-E's env. I fixed this by creating a new .bashrc file for WALL-E (with all required env variables) and sourcing it in crontab like so:
# crontab
0 */1 * * * BASH_ENV=/mnt/galaxy/walle/.bashrc bash -c "  source /mnt/galaxy/venv/bin/activate; /mnt/galaxy/venv/bin/python  /usr/local/bin/walle.py  --tool interactive_tool    --max-size 10   --since 24    --debug   --kill  >> /var/log/walle/walle.log 2>&1"

Add all required env variables
To enable the dedicated walle_bashrc, all required env vars are added by the walle role. This requires additional Ansible variables, which are documented in README.md. Additional env vars can be easily added in the playbook with walle_extra_env_vars.
Permission error in walle.py
Instead of failing, log a warning if PermissionDenied is raised when reading a JWD file.
Add --kill option for walle.py
Optional, of course. We won't use this yet but perhaps in future when we're confident in WALL-E's abilities. We also would like to add an --alert option to notify us in Slack when malicious files are detected - this will most likely be our default.

Oh dear, I just noticed that this overlaps a lot with your existing PR @mira-miracoli :grimacing:

Hi, thank you for your contibution! :)
Especially thank you for integrating the galaxy_jwd.py part into the script, I shied away from that task😅
The --kill option is a cool feature. I thought about it, but dropped it in favor of the --delete-user option,
because Galaxy automatically kills all running jobs then. And I did not see many cases for EU where we would not delete a user when we find stuff that would make us want to kill the job.
Would it be okay for you if we merge my PR first and then you rebase?

Hey Mira, yes please feel free to merge your PR first and I'll clean up mine. The --kill option I thought was just a bit less aggressive than --delete-user but really we should not be doing either of these until we're confident of no type 1 errors! Nice to give admins a choice of actions I suppose. I hope to push something like --slack-alert in the next week or two.

@neoformit
I merged my PR now, please feel free to rebase and then we can merge yours :)

Hey @mira-miracoli I've rebased, there were lots of conflicts that I think I've resolved correctly but please check the diff on walle.py. It was a bit confusing at times - I noticed that in some diffs my changes were marked as "current" and other times "incoming" so it was hard to tell whose work I was accepting :roll_eyes: I'm having a look over the diff now.

Tested on the AU dev server and working ok with --verbose --debug

Walle does not clean up, it just scans files. But in order to do so, it needs read access

Yes I see that, but in our case read access was denied for command.sh which made Wall-E fall over, so good to catch this I think.

Thanks a lot for the thorough review @mira-miracoli :tada:
It looks like --debug is redundant now, so I have dropped that in favour of --verbose.
The --kill logic was lost in the merge so I've replaced that - thanks for noticing. I need to get better at doing a complex rebase :roll_eyes:
I think we just have to decide how to handle the walle_env_vars which don't always have a playbook variable to default to.

Of course, many thanks for the great contribution! :)

I think we just have to decide how to handle the walle_env_vars which don't always have a playbook variable to default to.

I am not happy with the dictionaries either. Ansible does not allow changing only specific k/v pairs from default, so you need to copy everything and then do your changes.
I am not sure if there are variables for all database k/v pairs. maybe we could leave it this way with hardcoded default values? I think
  - key: PGHOST
    value: <IP_ADDRESS>
  - key: PGUSER
    value: galaxy
  - key: PGDATABASE
    value: galaxy
  - key: GXADMIN_PATH
    value: /usr/local/bin/gxadmin

should work for most Galaxy instances(?)
Only thing is if people rely on --kill to work when Ansible runs without errors and they don't actually have gxadmin.
But I think if we write that in the docs that would be users responsibility.

Ansible does not allow changing only specific k/v pairs from default

True, but I think they can just override them by appending to walle_extra_env_vars? Here's my logic:
walle_envs_database:
 - key: PGHOST
    value: <IP_ADDRESS>
  - key: PGUSER
    value: galaxy

walle_extra_env_vars:  # Defined in playbook
  - key: PGUSER
    value: customdb

walle_env_vars: "{{ walle_envs_database + walle_extra_env_vars }}"
# Results in:
 - key: PGHOST
    value: <IP_ADDRESS>
  - key: PGUSER
    value: galaxy
  - key: PGUSER
    value: customdb

Sorry, didn't mean to close this. I did a typing blooper and hit some shortcut by accident :hankey:

If you do lineinfile with walle_env_vars, PGUSER=custom_db_user would overwrite PGUSER=galaxy, right?
oh smart, I did not think of this, that you could use the walle_extra_env_vars to overwrite.
Maybe we could include that in the README?

Maybe we could include that in the README?

Yeah for sure! Do you think it's good to also have the required playbook vars that I defined in the README?
galaxy_config_file: /path/to/galaxy.yml
galaxy_log_dir: /path/to/galaxy/log/dir
galaxy_pg_db: galaxy
galaxy_pg_user: galaxy
galaxy_pg_host: my-db-server.usegalaxy.org
galaxy_pulsar_app_conf: /path/to/pulsar/app.yml

If they only appear in eu's playbooks, I think we could remove that

Closed and reopened, so the pyright action triggers (I don't know a more subtle way 🙄 )

That is annoying, it seems that pyright raises linting errors for type hints on dependencies (i.e. we can't add type hinting to os.environ.get). But I can see how the linting is useful. I've done the best I can there - feel free to push if you see any others that need fixing?

https://github.com/usegalaxy-au/WallE/pull/1
This version would fix the pydantic errors

Do you want to test it again before we merge?
(I could test it only tomorrow)

Yep will do tomorrow (that should be before your tomorrow...)

Tested and working with --verbose --kill


Cleanup (assets/views/gemfile/config)

The majority of the Gemfile cleanup was just moving things around and creating a bit of order to it. I did remove a couple of gems that I couldn't see being used too though (Sqlite, launchy, rails-assets-oauthio-web).
The ruby version was also removed from the Gemfile and was tested against 2.1.4 and 2.2.4.

looks good, thanks! i'll give it a try locally and merge if it all works

Realising now that large amounts of views/sessions could probably go too if I'm not mistaken.

DELETE DELETE DELETE!
thanks!


Relay: Make ids of type ID + globally unique

See https://facebook.github.io/relay/docs/graphql-object-identification.html#content

Solved with #23.


Device with this Unique ID already exists

IMEI:<PHONE_NUMBER>80222 Device Unique ID already exists with this I get the error. How can I delete from my device this device visible.

my server is : http://<IP_ADDRESS>:8082/

Sometimes things go wrong and you can end up with device in the database which is not linked to any user account. The only way to clean it is to edit database manually or remove all database files if you don't have any important data there.

You can as a stop gap measure put a   *  in front of the device id until you are able to fix the issue


No prompt displayed when git branch name contains accentuated characters

I made a mistake while creating a git branche with a french name. It contained a "è". After that, prompt refused to display though I was able to type commands.

Possibly related to #188?

Hi @AugierLe42e, I just tested this out without any problems. Can you update to the latest version of this project and if you still have any issues, please feel free to re-open this.


[docs] Improve Implementations README and add second requirements.txt

This patch improves the README file with images and gifs and adds a second requirements.txt file specifically for notebooks.

@ccarterlandis: could you merge this?

@jgbarah : What are your thoughts on merging this one?

@Polaris000 : We are closing this now because the working group took Jupyter Notebook maintenance out of its sphere .

Thanks for the update @sgoggins.


Added Fourier Series

Adding Fourier Series Visualization in p5.js. Please merge!!

Is my PR wrong in some sort? Please tell me if I am mistaken...... I will make the changes ASAP!!


Got same key issue with AreaChart with multi-series & type=grouped

I'm submitting a...

[ ] Regression (a behavior that used to work and stopped working in a new release)
[X] Bug report
[ ] Performance issue
[ ] Feature request
[ ] Documentation issue or request
[ ] Other... Please describe:

Current behavior

Got same key issue with AreaChart with multiple series & type=grouped
https://github.com/reaviz/reaviz/blob/fd440fd329f81edf6e14bddbcbb323f51de20759/src/common/data/builder.ts#L84
The id should be unique or at least consider using index here.
Expected behavior

No errors
Minimal reproduction of the problem with instructions

What is the motivation / use case for changing the behavior?

Environment

Libs:
- react version: X.Y.Z
- reaviz version: X.Y.Z

Browser:
- [ ] Chrome (desktop) version XX
- [ ] Chrome (Android) version XX
- [ ] Chrome (iOS) version XX
- [ ] Firefox version XX
- [ ] Safari (desktop) version XX
- [ ] Safari (iOS) version XX
- [ ] IE version XX
- [ ] Edge version XX

For Tooling issues:
- Node version: XX
- Platform:

Others:

Could you provide a example?

Reproduce ->
https://stackblitz.com/edit/vitejs-vite-nvzdbv?file=src%2FChart.jsx
I found that is because I put an id in the data
const data = [
  {
    id: 'example-id', // After remove this, anything goes fine.
    key: 'example-key',
    data: [...],
  },
  ...
]


Add warning symbol

Codecov Report
Base: 40.01% // Head: 40.01% // No change to project coverage :thumbsup:

Coverage data is based on head (b135bfa) compared to base (233e001).
Patch has no changes to coverable lines.

:exclamation: Current head b135bfa differs from pull request most recent head eea072f. Consider uploading reports for the commit eea072f to get more accurate results

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #174   +/-   ##
=========================================
  Coverage     40.01%   40.01%
  Complexity      471      471
=========================================
  Files           122      122
  Lines          3149     3149
  Branches        313      313
=========================================
  Hits           1260     1260
  Misses         1781     1781
  Partials        108      108

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Allow messages without a subject

Also fixes another problem that could happen during url generation in email templates.

Build is broken. Do you want to remove 1.9.3 from the matrix?

Sure, I updated it to be a bit more... "modern"

It passes now!

Thanks @deivid-rodriguez !

Thanks to you!


Investigate and fix "invalid pointer" error with react-select affecting IE11

From Rollbar: https://rollbar.com/somerville-teacher-tool/somerville-teacher-tool/items/2/?person_page=0&#traceback
This appears related to an open bug in react-select on IE11: https://github.com/JedWatson/react-select/issues/690

Relatedly, looks like react-select is in the throes of being rewritten: https://github.com/JedWatson/react-select/issues/568
Can't reproduce yet on IE11 on Windows 8.1 for the URL from Rollbar.

Also can't reproduce on IE11 on Windows 7 through Browserstack.

Closing this for now since I can't reproduce.  If we see more instances in Rollbar or any user reports, we should re-open.

Re-opening since there have been two more occurrences in Rollbar.

A little stuck on this.  This affected @jgeiser this past Friday, and we have two occurrences in Rollbar then.  But I'm not able to reproduce through BrowserStack yet, manually poking at the react-select box.
windows 7, 11 latest      is ok
windows 8.1, 11 latest    is ok
windows 8.1, 11 metro     is ok
windows 10, 11 latest     is ok

Ah got it, the steps are:

open selectbox
click to close it
blur

Confirmed fixed on demo site after deploying https://github.com/studentinsights/studentinsights/pull/255.


feat: provision ephemeral Environments for Integration…

… test pipelines

Codecov Report
Patch coverage: 51.01% and project coverage change: -2.47 :warning:

Comparison is base (6a4d18d) 70.94% compared to head (ad69eba) 68.48%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##             main       #5      +/-   ##
==========================================
- Coverage   70.94%   68.48%   -2.47%
==========================================
  Files          25       26       +1
  Lines        2382     2580     +198
==========================================
+ Hits         1690     1767      +77
- Misses        545      648     +103
- Partials      147      165      +18

Impacted Files
Coverage Δ

gitops/snapshot.go
81.28% <ø> (ø)

controllers/snapshot/snapshot_adapter.go
40.68% <29.85%> (-13.80%)
:arrow_down:

gitops/environment.go
95.23% <95.23%> (ø)

controllers/snapshot/snapshot_controller.go
63.21% <100.00%> (+0.42%)
:arrow_up:

... and 5 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


bug: fix install from pypi source release

Signed-off-by: Jinjing.Zhou<EMAIL_ADDRESS>Related: #1061

/cc @kemingy


Updated installation documentation to include MySql devlopment libraries

In order to fix the following error we need to have mysql libraries installed. I have updated the documentation to include the mysql package.
(env) root@localhost:~/superset# pip install -r dev-reqs.txt
Collecting codeclimate-test-reporter (from -r dev-reqs.txt (line 1))
  Downloading codeclimate-test-reporter-0.2.3.tar.gz
Collecting coveralls (from -r dev-reqs.txt (line 2))
  Downloading coveralls-1.2.0-py2.py3-none-any.whl
Collecting flake8 (from -r dev-reqs.txt (line 3))
  Downloading flake8-3.4.1-py2.py3-none-any.whl (68kB)
    100% |████████████████████████████████| 71kB 7.5MB/s
Collecting flask_cors (from -r dev-reqs.txt (line 4))
  Downloading Flask_Cors-3.0.3-py2.py3-none-any.whl
Collecting mock (from -r dev-reqs.txt (line 5))
  Downloading mock-2.0.0-py2.py3-none-any.whl (56kB)
    100% |████████████████████████████████| 61kB 6.3MB/s
Collecting mysqlclient (from -r dev-reqs.txt (line 6))
  Downloading mysqlclient-1.3.10.tar.gz (82kB)
    100% |████████████████████████████████| 92kB 7.4MB/s
    Complete output from command python setup.py egg_info:
    sh: 1: mysql_config: not found
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/tmp/pip-build-ya2nJ5/mysqlclient/setup.py", line 17, in <module>
        metadata, options = get_config()
      File "setup_posix.py", line 44, in get_config
        libs = mysql_config("libs_r")
      File "setup_posix.py", line 26, in mysql_config
        raise EnvironmentError("%s not found" % (mysql_config.path,))
    EnvironmentError: mysql_config not found

    ----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in /tmp/pip-build-ya2nJ5/mysqlclient/

Coverage remained the same at 69.39% when pulling 8776c9a3fcea77622c6bc2e523e6411081a51e4c on akhilputhiry:doc_fix into e79adbbc5f6e44a7cacac06f8b9d8edab3098be5 on apache:master.

Coverage remained the same at 69.39% when pulling 8776c9a3fcea77622c6bc2e523e6411081a51e4c on akhilputhiry:doc_fix into e79adbbc5f6e44a7cacac06f8b9d8edab3098be5 on apache:master.

That's mysql specific, if you are not using it you don't need them.

I'd break down into another section/command. Many use Postgres or even sqlite as they get started.

@xrmx if thats not needed why it is present in the requirements?
@mistercrunch I have put it in a new line

Coverage remained the same at 69.39% when pulling 0d9d52db8b037743cc6a3532232b5f67302441a1 on akhilputhiry:doc_fix into e79adbbc5f6e44a7cacac06f8b9d8edab3098be5 on apache:master.

@akhilputhiry dev-reqs.txt is for devs that want to run tests not for deploy.

Thanks,it works.

Notice: this issue has been closed because it has been inactive for 243 days. Feel free to comment and request for this issue to be reopened.


Migrate testing optimizer rules to use rewrite API

Which issue does this PR close?

part of https://github.com/apache/datafusion/issues/9637.
Rationale for this change

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

The CommonSubexprEliminate rule has not been migrated yet.

The CommonSubexprEliminate rule has not been migrated yet.

That is a good point -- though we could do something like #[allow(deprecated)] until it is. I hope to work on it over the next week or two

Thanks again @lewiszlw


replace pkg/errors from vendor

xref: https://github.com/containerd/containerd/issues/6049
Signed-off-by: Zou Nengren<EMAIL_ADDRESS>

This PR is superseded by #6366. Thanks @zouyee and @jonyhy96!


orphan pods

Hello,
I notice that some user pod can become orphan.
onyxia@rstudio-479106-0:~/work/pocapp-helm-chart$ helm uninstall portail-shiny
release "portail-shiny" uninstalled
onyxia@rstudio-479106-0:~/work/pocapp-helm-chart$ kubectl get pods
NAME                                          READY   STATUS    RESTARTS   AGE
rstudio-479106-0                              1/1     Running   0          3d
sp-pod-31203b5c-a93b-4791-9219-e5ba80e65a8a   1/1     Running   0          13m
sp-pod-8139198a-c4c3-4b32-87bd-3b4da69c472a   1/1     Running   0          3d
sp-pod-8d67aa39-35af-40c5-a7da-c5f9fd0fbbbb   1/1     Running   0          8d
sp-pod-c36df99b-6c95-4fd2-91e9-892524daf6c3   1/1     Running   0          3d1h
sp-pod-e3efc308-2e13-48a3-be3e-fdcc0a2d1453   1/1     Running   0          41s
sp-pod-f353365e-36cb-4e86-a10c-1008cd6e4510   1/1     Running   0          7d1h

I wonder why shiny proxy does not use the medatada ownerReference for each sp-pod in kubernetes. It would say to kubernetes that those pods must be garbage collected when the shinyproxy pod is deleted.

I'm running into this same problem locally and in production; when shinyproxy pulls a new image, the shiny apps that were managed by the old instance are orphaned, and have to be manually stopped and deleted.
It's cerntainly possible that I'm missing some configuration, but shouldn't these containers be reacting to the missing parent?

Hi, if you are using Kubernetes and you want smooth updates of the ShinyProxy config, we advice to use the shinyproxy operator: https://github.com/openanalytics/shinyproxy-operator . This fully manages updating the configuration, by starting new instances and stopping old instances, without breaking existing apps or connections.
Since the operator needs full control over the pods, we currently don't add the ownerReference. We could change this in the future (but make sure k8s doesn't remove the pods), but it might cause weird behavior with pods created in different namespaces.


ImageEqual will fail if Stride is negative

If the bitmap is bottom-up, ImageEqual() will fail.

Can someone provide an image with negative stride? I haven't encountered one, and they seem to be very rare.
The solution was to clone the bitmap.


HV-667 Create the Bean Validation descriptor model lazily

@gunnarmorling an oldie (January 2013...) but goodie.
An easy way to reduce bootstrapping work and memory usage. Best reviewed commit per commit (one is just a simple method renaming)
https://hibernate.atlassian.net/browse/HV-667

To clarify: I don't know if performance will degrade, just saying I'm not sure it will improve or degrade, and depending on the context both scenarios seem plausible. If only there was some benchmark of Hibernate Validator... :)

Yes, so the descriptors should nearly never be used, that's why we decided to do that - it was Gunnar's idea originally and I concur with him. It's part of the Bean Validation spec to allow users to inspect the metadata but it's not something people should often do. Typically, HV doesn't use these descriptors internally at all.
So it's going to be a win for 99.99% of our users. And a probably not quantifiable loss for the rest.
Apart from the bootstrapping time it spares, it also spares some memory.

Hey @gsmet, very nice. It looks good to me. A you say, it's going to be an improvement for the vast majority of users - in terms of bootstrap time and memory usage - and I don't think even make things worse for users of the metamodel. As before, they'll have to pay the init cost once, only now it's delayed. And well, a volatile read instead of non-volatile. I think that's ok :)

Rebased and merged, thanks to both of you.


[My Events]  The 'Rate Organiser' button is missing on event cards

Environment:
11th Gen Intel(R) Core(TM) i5-11320H @ 3.20GHz   3.19 GHz,
Windows 11 Pro release , 22H2
Chrome 120.0.6099.199 (Official Build), (64 bit)
Reproducible: always.
Build found: 11.01.2024
Preconditions
1.The site https://www.greencity.social/#/events is opened.
2.The User is logged-in.
3.User should be an Attendee of the two event that are passed.
4.'My events' page is opened.
Steps to reproduce

Find an attended event in the list
Click the‘Rate Organizer’ button

Actual result
Cards are displayed without the ability to rate the event
Expected result
The cards should have a 'Rate Organizer' button to rate the event
User story and test case links
E.g.: "User story #3268
Test case"
Labels to be added
Bug
event
Events
GreenCityEvent
My Events
Pri: medium
Sev: trivial
UI

It was decided to remove the functionality for evaluating the organizer from the application. You can evaluate only the event itself

not relevant


What is "len(...) == 2" check after pre_projection?

There are several places where the length is checked to see if it is 2 after pre_projection. What is the purpose of this processing?
In most cases, pre_projection returns much more than 2.
And if this processing is done, the number of dimensions will be reduced by one, causing an error.
e.g. glass.py line 588
patch_features = self.pre_projection(patch_features)
patch_features = patch_features[0] if len(patch_features) == 2 else patch_features

This step is intended to facilitate the selection of single-layer outputs during debugging, but it is not meaningful in the current complete version.

Thank you for the quick response.
I understand now!


Doesnt work at all

I use modrinth version from feather client loader, all required mods are installed. Im playing on 1.20.1, spa is also 1.20.1 version. Other mods are disabled so there are only SPA, Mod Menu and Cloth Config IP. Minecraft boots up just fine, but when i try to join a server or a singleplayer world it crashes with "os error 10054".  pls help

Update: i tried it out with minecraft launcher and it worked. Can yuo please add support for feather client?


[EmbeddedAnsible] Use rugged for git in specs

Changes a few things:

Uses Rugged for git operations in the ConfigurationScriptSource spec
Uses a local repo instead of a remote (for stability)
Adds a few extra checks for checking repo contents (just confirming the playbook is there in the file system)

This should allow the ConfigurationScriptSource specs to be more stable on CI.

@miq-bot add_labels core, embedded_ansible, technical debt, test

ARRGGHHHH!!!1! $^@#$&%@#$
@miq-bot add_label embedded ansible


[5.0] Prepare SwiftPM for the addition of Result to the stdlib

(cherry picked from commit e2723860ed6b370e5cb2193a00e04d6985cb9d0e)
Swift 5.0 version of: https://github.com/apple/swift-package-manager/pull/1912

@swift-ci Smoke test

This should be unnecessary due to https://github.com/apple/swift/pull/21378

We shouldn't cherry-pick because we have automerging from master->swift-5.0-branch


Stress tests JMeter configuration and report

Testing instructions
I used JMeter 5.5 on a mac to run the tests. Results in the final report were copied from the "summary report" for the different tests.
Tracking
https://vizzuality.atlassian.net/browse/LET-639?atlOrigin=eyJpIjoiOGVkMDE2OGU4OTU1NGZiYjljYTRjZTdlZjZmZjM3ZjAiLCJwIjoiaiJ9

So I've decided to move the load testing folder to the root of the project (it's not just a BE thing) and also remove the report and the different jmx files. IMO there's really no point in having multiple files in the repo, since they will all be the same, with the only difference being the semi-arbitrary number of users.


embeddings of quantized models sometimes differ per device

look into why this happens and how it can be fixed

This is probably caused by the newly created pool layer.


Add two new methods to the OpenUV component that consume only a singl…

…e API call
Description:
The OpenUV component makes calls to http://openuv.io/ the current method openuv.update_data makes 2 calls from the daily limit of 50.  The one endpoint only needs to be called once a day.  There should be separate methods for the separate api calls.
I didn't change the existing method at all, to prevent breaking changes.
Pull request with documentation for home-assistant.io (if applicable): https://github.com/home-assistant/home-assistant.io/pull/10227
Checklist:

[✓ ] The code change is tested and works locally.
[✓ ] Local tests pass with tox. Your PR cannot be merged unless tests pass
[✓ ] There is no commented out code in this PR.
[✓ ] I have followed the development checklist

If user exposed functionality or configuration variables are added/changed:

[✓ ] Documentation added/updated in home-assistant.io

Hi @StephenWetzel,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!


Added jitter in Retry-Mechanism for Alpakka Interpreter

Let me know if you need any more help on this one 😃

Ping

Hey @regiskuckaertz, extremely apologized for a very delayed response. I will quickly follow up on this PR in coming days. Will start looking into your suggestion.

No need to apologise for anything, we all have busy lives 🤗 Whenever you're ready.

@regiskuckaertz have a look whenever you have sometime for it. @Divyanshu564 updated the PR.

@regiskuckaertz Could you please take a moment to check the PR again, I have updated the jitter policy.


connect with cron job

if i have a function in /modules/modulename/server/factory/factory_test called create
is it possible to call that every 1-5 sec? how?

I use Agenda jobs for these at https://github.com/trustroots/trustroots — you caan check from there how I did it.

Thanks


FIRMessaging received data-message, but FIRMessagingDelegate's-messaging:didReceiveMessage: not implemented

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Hello guys i have being having these issue for a long time .I don't really know how to solve it .i tried so many things but nothing works.its being a real nightmare for me .please please please someone help me .

I am having the same problem since yesterday, I'm working on getting a solution for this problem, if I have a fix or work around I will post it, in any case hopefully someone else has a solution before me.

thank you for  answer @RobinGiel . i am currently trying it out. but i had one question tho . Does push notification work on Xcode 10 simulator ? cause i have being trying on it

sorry to distrub you again @RobinGiel  but it still doesn't work. i am using ionic framework  and more precisely cordova-plugin-firebase for push notification .

@brainssolutions  the method is implemented in my AppDelegate+FirebaseFile.m file but is not called for some reason.

@brainssolutions  did you find anything?

@soulemanou
Turning between the various reports I found a person who a long time ago I had the same problem, I turn the answer that worked for him:
`Check the Capabilities tab in Xcode before you build the app. Even though my App ID had notifications enabled, this was turned off in Xcode after my last Xcode update. I had to turn it on there, then remove and re-add the iOS cordova platform in my app. After that, it started working.
Firstly I changed the build system to Legacy in File -> Project Settings.
Secondly, I used an Authentication APN Key (not Certificate as mentioned in the Medium guide, see https://firebase.google.com/docs/cloud-messaging/ios/certs).
add to config.xml

remote-notification

Didn't do anything else of your guide but this still fixed it for me.
Thanks so much. `
at the moment I can not try these suggestions but tomorrow I will try to verify. 😊
Sent with GitHawk

@brainssolutions thank you for your answer . i have just tried all the suggestions above but had no luck . none of them worked for me. hope you have better luck

@brainssolutions  please if you find anything else just let me know

@soulemanou I'm currently not using the emulator to receive push notification, only devices.
If your building the ionic app using Xcode 10 remember to use the legacy build going to the menu at File -> project settings and select the Build system - Legacy Build System and try building it again

@soulemanou  check out this article about the legacy build

@soulemanou Can you post the log that you see in debug?

FIRMessaging received data-message, but FIRMessagingDelegate's-messaging:didReceiveMessage: not implemented  . this is the error i recieve on the iphone simulator everytime i recieve a notification on it.

@brainssolutions the thing is that i have implemented this method  in my Appdelegate.m file  and it is even  already implemented in my AppDelegate+FirebasePluginFile.m file but its not being called .Is there something else  i have to do after that?

Hello
I'm sorry for late.
I solved the problem by implementing this code in my Cordova config file.
https://github.com/arnesson/cordova-plugin-firebase/issues/997#issuecomment-461957604

Hey @soulemanou I managed to solve this while also fixing the issue I opened some while ago #990
Noticed that the problem is in the function grantPermission in FirebasePlugin.m
Changed its implementation to the one you can find in my fork and it's now working as intended.
Also, just to clarify, this is my notification flow for iOS:

Call hasPermission and based on its result:

true - call getToken
false - call grantPermission and getToken afterwards

From this moment on I can now send notifications and data messages without getting the error FIRMessaging received data-message, but FIRMessagingDelegate's-messaging:didReceiveMessage: not implemented

@DavidSousa Hey David, I implemented the grantPermission snippet you provided, and indeed I receive the data now, however I'm not getting a notification bar. Have you encounter this?

In my case, solve the problem this way :
1 Open the file ** AppDelegate+FirebasePlugin.m** (usualy is in the folder Plugins)
2 Find the method ** swizzledDidFinishLaunchingWithOptions** under the ** [FIRApp configure];**
put this [FIRMessaging messaging].delegate = self;


Can I specify a different language than English from the start?

Issue Type

Feature Idea

Summary
Is there any way to install Drupal in a different language than English right from the start?
That would save me and probably other people a lot of time.
Thank you for all the work you have done so far.

@Chameo523 - Yes, you can set the variable drupal_site_install_extra_args: "--locale=fr" (for example, for French). You can add any of the extra site-install options (see this list: https://drushcommands.com/drush-8x/core/site-install/) using that variable.

Closing as I think the answer should provide a way to do what you're asking. Thanks!


add hostname metadata

Hi,
first of all thx for the work you're putting into this! It helped me a lot.;)
Would it be possible to add the nodes 'real' hostname to the available metadata? E.g. for the node-exporter it would be quite useful to export the hostname used in rancher, instead of the IP (just like it is possible in kubernetes).
That way it would be easy to relabel instance to the hostname..
regards,
Roman

Same problem here. The node instance ID is set by default to the container IP of the scraper, not the hostname of the actual host.

Same problem here. The node instance ID is set by default to the container IP of the exporter, not the hostname of the actual host. It's confusing and weird.

I had the same issue, I'm now overwriting all the instance labels with the actual hostnames.
This is quite easily doable in the prometheus config by setting it onto the labels attribute:
      - targets: ['rancher-specific-service-name']
        labels:
          instance: 'actual-hostname'

Btw, check that out 😁
https://github.com/marcbachmann/rancher-prometheus-config


Update stage 1.17

Describe what your pull request addresses (Please include relevant issue numbers):

Update stage to 1.17

Fixes in this release

#1609
#1631
#1632

tested #1609, #1631 , #1632 seems fixed.
merging PR.


feat: Add support for loading images with blob urls

Register blob as scheme instead of blob:http or blob:https because
cornerstone-core splits on : and uses the first part as the scheme. So,
if provided a url blob:https://github.com/test-id it will consider blob
as the scheme instead of blob:https.
Closes https://github.com/cornerstonejs/cornerstoneWebImageLoader/issues/13

Coverage decreased (-0.07%) to 5.263% when pulling ff48ce55193611d501fd22dcb19725b55804fd94 on flamy-dev:blobs into fd929932fe04231e42b7d3afd271d20832e22e09 on cornerstonejs:master.

Not sure how the coverage got decreased since we are just importing the whole module for giving an illusion of coverage(I think)?
Also, can we have a release after this is merged @chafey ?

In the meanwhile if anyone wishes to use this I've published it to github package - https://github.com/flamy-dev/cornerstoneWebImageLoader/packages/297934

@Inzeppelin don't think this is going to be merged, better fork


tox.ini: add --namespace-packages to mypy

It is needed by latest version of types-protobuf
Also update mypy to the latest version

The change in types-protobuf was reverted: https://github.com/python/typeshed/pull/7522
mypy should work again with 3.19.15 even without --namespace-packages.
I don't know whether it still makes sense to add this option.

I don't know whether it still makes sense to add this option.
The issue mentioned that it would be the default in the next version of mypy.

What we could do instead is to depend on types-protobuf != 3.19.14, this way it should fix the current issue and keep working when 3.19.15 is released.

types-protobuf 3.19.15 has already been released on PyPI.

This was added as part of #1782.


Remove unsafe arithmetic operations deprecated in 4.2

Follow-on to https://github.com/apple/swift/pull/19425, per discussion with core team https://forums.swift.org/t/removing-unsafe-arithmetic-methods/16169
rdar://problem/43688685

@swift-ci please smoke test.

@swift-ci please test.

Build failed
Swift Test OS X Platform
Git Sha - 87d1af0f882e56d642b686db400d8156dfae4243

Build failed
Swift Test Linux Platform
Git Sha - 87d1af0f882e56d642b686db400d8156dfae4243


appliance/mariadb: Wait for master_clients > 0

The semi-sync plugin doesn't increment master_clients until the
connected slave has acked a message. Because of this SLAVE_HOSTS
can show a connected slave before master_clients.
Instead just use SLAVE_HOSTS for verifying the correct downstream
is present and use master_clients for determining read/write.

LGTM


Add digdagignore

I implemented this issue:  https://github.com/treasure-data/digdag/issues/1092
I create digdagignore like gitignore. It can ignore files when digdag push.
But, I don't use gitignore syntax. I use glob syntax.
It can ignore file in .digdagignore file like gitignore.
If first character is ! then it reverse a process.
And, if first character # or   then the line is skiped.
For example, you can try digdagignore in digdag project repository.
java -jar PathTo/digdag/pkg/digdag-0.9.39-SNAPSHOT.jar push sample --project examples/digdagignore/
2019-07-12 01:39:35 +0900: Digdag v0.9.39-SNAPSHOT
Creating .digdag/tmp/archive-5896468971676972804.tar.gz...
projectPath: PathTo/digdag/examples/digdagignore
  Archiving a.dig
  Skip b.dig
  Archiving a/c/e.txt
  Archiving a/c/d.txt
  Archiving a/d/a1.txt
  Archiving a/d/a2.txt
  Archiving a/d/a3.txt
  Skip a/d/a4.txt
  Skip a/b/b.txt
  Skip a/b/a.txt
Workflows:
  a.dig
Uploaded:
  id: 1
  name: sample
  revision: 42c88991-170e-4e5a-a503-9d4c6d7d82b2
  archive type: db
  project created at: 2019-07-11T16:39:37Z
  revision updated at: 2019-07-11T16:39:37Z

Use `digdag workflows` to show all workflows.

The reason why I use glob syntax is that gitignore syntax is complex to implement.
Glob syntax is simple and glob exists in java library.
And We must follow the git version when We use gitignore syntax.
Please comment if you like, and I will add tests and documents.

Hi @katsuyan, thank you for implementing this feature. I think that the feature really good 👍 I took a look your spec in the description and code roughly. I have the following comments.

Can you move your change to ProjectArchive from Archiver? I think that the right place is ProjectArchive#listFilesRecursively. In your current implementation, ignoreFileList method is called every tar entry copying. it's not efficient for the performance. Moreover we ignore dot files during archiving the project. Your implementation could be integrated with that. If user-defined ignore file doesn't exist, it ignores dot files by default ignore file. It means that user-defined ignore file could override the behavior by default ignore file.
Glob by raw regex is ok but what do you think that we  use Java based git library like eclipse JGit? I'm not sure that we can use JGit for this purpose but, that's good option I think.
What do you think?

Thank you very much for your kind comment, @muga.

About the first comment

It means that user-defined ignore file could override the behavior by default ignore file.

I did not care about the above. I understand that the right place is ProjectArchive.

About the second comment

I didn't know Java based git library. I understand to use it will be easy for future maintenance.
If we can use it for this purpose, I think it's best way.
Thank you for the information about the roadmap.
I try to use JGit once.
Should I close this PR?

@katsuyan I would like to close this PR because of no activity long time.
I think this is a valuable functionality, so we can revisit in the future.


Udcomments

Post Create Button -> Done
Post Delete Text -> Done
Post Update Text -> Done

Post Create Simple -> In process

Ngon


Delete members of group when deleting group

Description
This PR updates the functionality of Graph.delete in order to delete members of a group when a group is deleted. If group members are themselves groups, members of those groups are also deleted.
Motivation and Context
When deleting a group, members of the group should also be deleted. This is consistent with the documentation.
Types of changes

[X] Bug fix (a non-breaking change which fixes an issue)
[ ] New feature (a non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to
not work as expected)
[ ] Improvement/refactoring (non-breaking change that doesn't add any feature
but make things better)

Checklist

[X] Check other PRs and make sure that the changes are not done yet.
[X] The PR title is no longer than 64 characters.

This looks really good and fixes a clear bug. I'm working on the merge now...


[DIR-1546] fix issue were logs might disappear

Description
Under certain conditions, the logs can disappear
https://github.com/direktiv/direktiv/assets/121789579/ed9f81ce-782d-4584-a62d-7cef2522a910
https://github.com/direktiv/direktiv/assets/121789579/391eb936-d46b-49e2-a1ab-962f0ead4635
What happens

the logs have all been fetched
the cache has multiple pages with one being empty (the last page is always empty, because the backend does not know how many logs there are unless you reach an empty page)
when the component with the logs unmounts and remounts again, the cache is stale and react Query will refetch
when using an infinite Query with reactQuery refetching means doing one request for every page in the cache (if one record has been deleted somewhere, pages are out of synch, therefore all pages must be refetched)
however, reactQuery only request one instead of two pages.
the one page is the empty page and the logs disappear

What went wrong

reactQuery has a concept of previous and next pages to navigate through an infinite list
I accidentally swapped next and previous
when reactQuery refetches an entire infinite list, it does the following

it fetches the very first page with initialPageParam
when the response comes in, it uses getNextPageParam to figure out the next page to fetch (if this function returns null or undefined, it stops)
getNextPageParam always returned undefined and refetching the list never reached a second page

With this PR I also

removed the getPreviousPageParam, because only getNextPageParam is mandatory, and we only lazy load the list in one direction
changed firstPage.meta?.previousPage ?? undefined to firstPage.meta?.previousPage because it is more readable (docs say "Return undefined or null to indicate there is no next page available.")

Follow up

[DIR-1594] rename previousPage to nextPage
[DIR-1598] I added a test, but it keeps failing in CI. This test is disabled for now until we can download the articafts of the tests and see what's going on

Checklist

[x] Documentation updated if required
[x] Test coverage is appropriate

Checklist Internal

[x] Linear issue linked (e.g. [DIR-XXXX] pull request title)
[x] Has the PR been labeled

Update: There was still a bug in the logs. After I switched prev and next, I also needed to change how these pages are glued together before they are rendered. React Query has a default way on how pages are stored in the cache, and this is exactly the opposite to how we need to render them in the frontend.
As discussed, I also added the date and year to the log output.
There is still one small issue, that I can not solve very easily for now, but I think it should not be a blocking issues. Whenever you open the logs, it always loads one page more that "it should".
https://github.com/direktiv/direktiv/assets/121789579/c344da53-87ce-4a0a-af07-7add53aa2446

when first opening the logs, it shows 400 lines instead of 200
when you come back to the page, and it should restore the previous 400 lines, it will request one additional page and shows 600 logs
the reason is the following: the page loads and before it will automatically scroll to the bottom, it is at the top of the logs for a blink of a second, which triggers a "load the next page"
A fix to that would require a lot of testing and should be done in a new ticket.


Add another truncation case

Summary of the Pull Request
What is this about?
Closes #3058
Info on Pull Request
What does this include?
An earlier PR only added truncation to the webhook code path for azure queue. This adds support for truncating and retrying for all events that support truncation.

Codecov Report

Merging #3075 (1feb87f) into main (6397803) will decrease coverage by 0.02%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main    #3075      +/-   ##
==========================================
- Coverage   31.80%   31.78%   -0.02%
==========================================
  Files         307      307
  Lines       36766    36782      +16
==========================================
  Hits        11692    11692
- Misses      25074    25090      +16

Impacted Files
Coverage Δ

src/ApiService/ApiService/onefuzzlib/Events.cs
66.01% <0.00%> (-12.15%)
:arrow_down:


 Ruins the slider: fixed height of 24px.

short description of the bug / issue, provide more detail below.

So on top of my index.html there's the  element.
I've added that to get a parallax plugin to work properly.
Yet, it completely ruins the slider. Whenever I remove that tag, the slider works. But it doesn't work with the tag in the document.

.slick-slider {
    height: <your-height>;
}

.slick-list {
    height: 100%;
}

.slick-track {
    height: 100%;
}

That fixed it for me.

i had the same issue, but the solution i found was different.
it turned out that i used direction:rtl on the body.
i tried feeding the slick function with rtl:true, and everything worked like magic:
xxx.slick({
rtl:true
})


After converting animated GIF to animated JXL, the JXL output is buggy (artifacts during playback)

Describe the bug
I converted an animated GIF file to JXL, and then played the JXL file, but I found that it has artifacts.
I also converted the JXL to APNG, but even the APNG has artifacts.
To Reproduce

cjxl test.gif test.jxl
Play test.jxl in Gwenview
djxl test.jxl test.apng
mpv test.apng

Expected behavior
No artifacts during playback, the animation should look the same as the GIF input.
Screenshots

Environment
Arch Linux x86_64
v0.8.1 c27d4992 [AVX2,SSE4,SSSE3,Unknown]
GIF: Created in GIMP 2.10.34.
Test files
test-files.zip

Thanks!
I can reproduce the issue.
When viewing text.jxl in a patched Chrome, which is able to play animations, I also see the artifacts, so it seems that the problem comes from the first step converting the gif with cjxl.
When doing
convert test.gif tmp.apng
cjxl tmp.apng test.jxl

I obtain a jxl file where the animation plays fine, so converting from apng to jxl seems to work fine in this case.
Of course converting from git should also work...


feat: add new property jumpLimit

prevent jumping with too large value changes - e.g min to max

:tada: This PR is included in version 1.3.0 :tada:
The release is available on:

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.3.0 :tada:
The release is available on:

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


kjain-readme:Filling out readme

Filling out Readme

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Bug report: Error installing mactex

Description of issue
Trying to install mactex fails:
$ brew cask install mactex
==> Downloading http://mirror.ctan.org/systems/mac/mactex/mactex-20161009.pkg
Already downloaded: /Users/wlhlm/Library/Caches/Homebrew/Cask/mactex--20161009.pkg
==> Verifying checksum for Cask mactex
==> Running installer for mactex; your password may be necessary.
==> Package installers may write to any location; options such as --appdir are ignored.
Password:
==> installer: Package name is MacTeX-2016
==> installer: Installing at base path /
==> installer: The install failed (The Installer encountered an error that caused the installation to fail. Contact the sof
Error: no implicit conversion of Array into String
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/exceptions.rb:71:in `concat'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/exceptions.rb:71:in `to_s'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli.rb:150:in `message'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli.rb:150:in `rescue in process'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli.rb:143:in `process'
/usr/local/Homebrew/Library/Homebrew/cask/cmd/brew-cask.rb:8:in `<top (required)>'
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/rubygems/core_ext/kernel_require.rb:55:in `require'
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/rubygems/core_ext/kernel_require.rb:55:in `require'
/usr/local/Homebrew/Library/Homebrew/brew.rb:24:in `require?'
/usr/local/Homebrew/Library/Homebrew/brew.rb:100:in `<main>'

I'm aware of #7946, but wasn't able to fix the problem by updating hombrew cask.
Output of `brew cask  --verbose`
==> Downloading http://mirror.ctan.org/systems/mac/mactex/mactex-20161009.pkg
Already downloaded: /Users/wlhlm/Library/Caches/Homebrew/Cask/mactex--20161009.pkg
==> Verifying checksum for Cask mactex
==> Running installer for mactex; your password may be necessary.
==> Package installers may write to any location; options such as --appdir are ignored.
==> installer: Package name is MacTeX-2016
==> installer: Installing at base path /
==> installer:PHASE:Preparing for installation…
==> installer:PHASE:Preparing the disk…
==> installer:PHASE:Preparing MacTeX-2016…
==> installer: The install failed (The Installer encountered an error that caused the installation to fail. Contact the software manufacturer for assistance.)
Error: no implicit conversion of Array into String
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/exceptions.rb:71:in `concat'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/exceptions.rb:71:in `to_s'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli.rb:150:in `message'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli.rb:150:in `rescue in process'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli.rb:143:in `process'
/usr/local/Homebrew/Library/Homebrew/cask/cmd/brew-cask.rb:8:in `<top (required)>'
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/rubygems/core_ext/kernel_require.rb:55:in `require'
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/rubygems/core_ext/kernel_require.rb:55:in `require'
/usr/local/Homebrew/Library/Homebrew/brew.rb:24:in `require?'
/usr/local/Homebrew/Library/Homebrew/brew.rb:100:in `<main>'

Output of `brew doctor`
Please note that these warnings are just used to help the Homebrew maintainers
with debugging if you file an issue. If everything you use Homebrew for is
working fine: please don't worry and just ignore them. Thanks!

Warning: Your Xcode (7.0) is outdated.
Please update to Xcode 7.2.1 (or delete it).
Xcode can be updated from the App Store.

Output of `brew cask doctor`
$ brew cask doctor
==> macOS Release:
10.10.5
==> Hardware Architecture:
intel-64
==> Ruby Version:
2.0.0-p481
==> Ruby Path:
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/bin/ruby
==> Homebrew Version:
Homebrew 1.0.7-9-g04ccba5
Homebrew/homebrew-core (git revision fa7b2; last commit 2016-10-11)
==> Homebrew Executable Path:
/usr/local/bin/brew
==> Homebrew Cellar Path:
/usr/local/Cellar
==> Homebrew Repository Path:
/usr/local/Homebrew
==> Homebrew Origin:
https://github.com/Homebrew/brew.git
==> Homebrew-Cask Version:
Homebrew-Cask 1.0.7-9-g04ccba5
caskroom/homebrew-cask (git revision a3e9; last commit 2016-10-11)
==> Homebrew-Cask Install Location:
<NONE>
==> Homebrew-Cask Staging Location:
/usr/local/Caskroom
==> Homebrew-Cask Cached Downloads:
/Users/wlhlm/Library/Caches/Homebrew/Cask
8 files, 2.8G (warning: run "brew cask cleanup")
==> Homebrew-Cask Default Tap Path:
/usr/local/Homebrew/Library/Taps/caskroom/homebrew-cask
==> Homebrew-Cask Alternate Cask Taps:
/usr/local/Homebrew/Library/Taps/caskroom/homebrew-fonts
/usr/local/Homebrew/Library/Taps/caskroom/homebrew-versions
==> Homebrew-Cask Default Tap Cask Count:
3376
==> Contents of $LOAD_PATH:
/usr/local/Homebrew/Library/Homebrew/cask/lib
/usr/local/Homebrew/Library/Homebrew
/Library/Ruby/Site/2.0.0
/Library/Ruby/Site/2.0.0/x86_64-darwin14
/Library/Ruby/Site/2.0.0/universal-darwin14
/Library/Ruby/Site
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/vendor_ruby/2.0.0
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/vendor_ruby/2.0.0/x86_64-darwin14
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/vendor_ruby/2.0.0/universal-darwin14
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/vendor_ruby
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/x86_64-darwin14
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/universal-darwin14
==> Contents of $RUBYLIB Environment Variable:
<NONE>
==> Contents of $RUBYOPT Environment Variable:
<NONE>
==> Contents of $RUBYPATH Environment Variable:
<NONE>
==> Contents of $RBENV_VERSION Environment Variable:
<NONE>
==> Contents of $CHRUBY_VERSION Environment Variable:
<NONE>
==> Contents of $GEM_HOME Environment Variable:
<NONE>
==> Contents of $GEM_PATH Environment Variable:
<NONE>
==> Contents of $BUNDLE_PATH Environment Variable:
<NONE>
==> Contents of $PATH Environment Variable:
PATH="/usr/local/bin:/usr/local/sbin:/usr/bin:/usr/sbin:/bin:/sbin:/usr/local/Homebrew/Library/Taps/homebrew/homebrew-emacs/cmd:/usr/local/Homebrew/Library/Homebrew/cask/cmd:/usr/local/Homebrew/Library/Homebrew/shims/scm"
==> Contents of $SHELL Environment Variable:
SHELL="/usr/local/bin/zsh"
==> Contents of Locale Environment Variables:
LANG="en_US.UTF-8"
LC_CTYPE="en_US.UTF-8"
==> Running As Privileged User:
No

Does installing the pkg manually work?

Yes, works without a problem.

It is installing fine for me, I cannot reproduce the error you are getting:
The Installer encountered an error that caused the installation to fail. Contact the software manufacturer for assistance.

@wlhlm Try brew cask uninstall --force mactex && brew cask install mactex

Closing due to lack of reply


Allow arbitrary scenarios

Currently scenarios are hard-coded. We could:

[x] allow extensions to depend on a token which would be used to add scenarios
[ ] allow to create a scenario from UI using a list of commands to run; we could add our custom logic as commands too

Depends on https://github.com/jupyterlab/jupyterlab/issues/13962


GitHub API - IssueTypeIncident Field breaks Test Case

As a side effect of the camelCase refactor on the Backend, the IssueTypeIncident field for GitHub Connection API was changed to typeIncident and was not properly updated elsewhere. Expected BE to respond with issueTypeIncident key instead.

@klesh @warren830

@klesh @warren830

Sorry man, I'll look into it


miss parsing 'last' in poloniex market 'ticker' response

poloniex will return 'last' price in 'ticker' response. but the code in ccxt 'poloniex' class doesn't parse it.
here is the  poloniex document.
https://poloniex.com/support/api/

Ticker
In order to receive ticker updates, subscribe to "ticker". Updates will be in the following format:
Appropriate labels for these data are, in order: currencyPair,  last, lowestAsk, highestBid, percentChange, baseVolume, quoteVolume, isFrozen, 24hrHigh, 24hrLow

Will add it very soon!

This now works since version 1.1.149! Thanks!

24hrLow and 24hrHigh doesn't work.

@mahmutbedir what's your language? Ah, here's 2 of 3:
// JavaScript
(async function test () => {
    const ccxt = require ('ccxt')
    const polo = new ccxt.poloniex ()
    const ticker = await poloniex.fetchTicker ('BTC/USDT')
    console.log ('high:', ticker['high'], 'low:', ticker['low']) // 24-hour high / low
})

# Python
import ccxt
poloniex = ccxt.poloniex()
ticker = poloniex.fetch_ticker('BTC/USDT')
print('high:', ticker['high'], 'low:', ticker['low'])

Should be floats. Take a look here:
https://github.com/ccxt-dev/ccxt/blob/master/examples/py/tickers.py
https://github.com/ccxt-dev/ccxt/blob/master/examples/js/tickers.js


Broken link in readme

In the readme, this link is broken:

The experiments for this paper are based on this code.

Thanks!


SemanticallyEqualVisitor does not detect changed tags in child elements

Hi - working on a recipe to change contents of a pom, and I came across a potential issue in the org.openrewrite.xml.SemanticallyEqual$SemanticallyEqualVisitor class.
Specifically the visitTag method of this class, when considering child elements, doesn't appear to cater for the case where the existing and proposed lists of child elements are the same length, but the child element tag names are different.
e.g. it thinks the following
<configuration>
  <tasks>
    <delete file="erewhon" />
  </tasks>
</configuration>

is the same as this:
<configuration>
  <target>
    <delete file="erewhon" />
  </target>
</configuration>

because the configuration element in each case has just one child which is identical in all respects except that the child element name is different.
Slack Message

You can try out this change through our snapshot versions. Hope that helps!


Install deps in each package's isolated location

Fixes #12
To-do before merge

[ ] Do a full scale test with this on actual hardware to ensure everything is installed

@apockill I have! But not super extensively. It passes a smoke test of starting the system up, but that's as much as I can say for now. We should definitely run this for real tomorrow before merging.


Syntax highlighting not working after initializing

When I initialize a new instance of CodeMirror, it does not include syntax highlighting. I'm able to get the syntax highlighting to work by calling swapDoc at least 300ms after initializing. However that solution won't always work because the page can have larger amounts of JavaScript on it leading to a need for a longer timeout.
Here's an example of how I'm initializing CM:
let editorContent = CodeMirror.Doc("

", "htmlmixed");

let editor = CodeMirror(element, {
    smartIndent: true,
    gutters: ['CodeMirror-lint-markers'],
    keyMap: 'sublime',
    value: editorContent,
    lineNumbers: true,
    lint: true,
    indentWithTabs: false,
    tabSize: 2,
    indentUnit: 2,
    autoCloseBrackets: true,
    autoCloseTags: true,
    matchTags: {
        bothTags: true
    },
    autoRefresh: false
});

I am using CodeMirror 5.19 on the latest version of Chrome.
Since swapDoc does work as a way to get the syntax highlighting to work, I just need to know when to call it so I'm not reliant on a fixed timeout length. I was hoping there was an editorReady event or callback I could use to trigger swapDoc but that doesn't appear to be the case. Is there a better way to go about detecting when the editor is ready so I can call swapDoc?

Maybe your mode scripts haven't finished loading when you initialize the editor? That would have this effect.

@marijnh thanks! I'll start looking into that.

I managed to resolve this issue and just wanted to post a quick update to help anyone else who may be struggling with something similar.
Like @marijnh suggested, it was a result of my mode scripts not being fully loaded. I was able to find out what modes had loaded by inspecting CodeMirror.modes. In my case, htmlmixed had three dependencies which had not finished loading.
My solution was to get a list of dependencies for htmlmixed and then use setInterval to check for their presence in CodeMirror.modes.


feat(images): point to specific version

This way a new website build is needed to update images. Updating an image in the CDN will result in a new version, but the image on production will keep the same until a new production deploy happens

Current dependencies on/for this PR:

main

PR #130   👈

This stack of pull requests is managed by Graphite.


Update to version 7.4.0 of scanner

New version is release:
https://github.com/blackducksoftware/synopsys-detect/releases/tag/7.4.0
Release Notes:
https://synopsys.atlassian.net/wiki/spaces/INTDOCS/pages/642351242/Releases+Notes

A bit confused on how to use the new "SUB_PROJECT" setting: https://synopsys.atlassian.net/wiki/spaces/INTDOCS/pages/1383891024/BDIO+Aggregation


Cannot read single nullable column when type converter is used

I'm having issues reading a single nullable column that uses a type converter.
To reproduce, I've copied the Preferences code from the documentation on type converters.
When I try to create a function to read the preferences column, the linter complains:
The argument type 'GeneratedColumnWithTypeConverter<Preferences?, String>' can't be assigned to
the parameter type 'GeneratedColumnWithTypeConverter<Preferences, String>'.
dart(argument_type_not_assignable)

The readWithConverter function does not seem to support a nullable value.
When I read the whole row, it works fine.
Non-functioning code:
Future<Preferences?> preferences(String actor) async {
    final result = await (selectOnly(account)
          ..addColumns([account.preferences])
          ..where(account.actor.equals(actor)))
        .getSingle();

    return result.readWithConverter(account.preferences);
             Linter complains here: ^^^^^^^^^^^^^^^^^^^
}

Functioning code:
Future<Preferences?> preferences(String actor) async {
    final result =
        await managers.account.filter((f) => f.actor(actor)).getSingle();

    return result.preferences;
}

Table:
class Account extends Table {
  @override
  Set<Column> get primaryKey => {actor};

  TextColumn get actor => text()(); // Actor did

  TextColumn get preferences =>
      text().map(const PreferenceConverter()).nullable()();
}

Am I doing something wrong or is this not supported?

Type inference can be weird, this works:
    final preferences = result.readWithConverter(users.preferences);
    return preferences;

And this works too:
    return result.readWithConverter<Preferences?, String>(users.preferences);

I'm not fully sure why inference fails for the original call, but we can't really relax the type of readWithConverter without introducing other issues. So I'm afraid you'd probably have to use one of the workarounds.


Fixed Swift 4.1 flatMap deprecation warning

Just getting rid of the deprecation warning.

@kmaschke85 You broke the tests with that change, clearly. Currently, Travis is using xcode9 image and compactMap is only available with Xcode 9.3+. It makes sense to update the Toast-Swift/.travis.yml. The latest available image is xcode9.4, by the way (https://docs.travis-ci.com/user/reference/osx/).

BTW, as of July 31st, the default Xcode image on Travis is 9.4.

Thanks!


Add usage notes for Embroider to documentation

@jelhan thank you!

Thanks

@SergeAstapov Do you mind doing a small release including this fix? I can not recall how often I was going back to this merge requests in the last months to look up the command. :see_no_evil:


MongoDB Image

Thank you very much for this package. I'm using spa-starter-kit and I find it also very excellent. I have worked with mongodb to save log, and now php work with mongodb is also very easy. I hope you can create more a image mongodb.
Again thank you very much.

Ki @ptitkiller PHP images of Ambientum already have support on the mongdb extesion, you can safely use the official docker image with ambientum. just include them in your docker-compose.yml file

@ptitkiller closed this by inactivity, let me know if you need any further help getting MongoDB to work along with the other Ambientum images


Use bincode encoding in Rust light clients

Partially implemented in #3301

completed in https://github.com/unionlabs/union/pull/3342


Resume playback enhancements

During development of this feature, I added resume playback for livestreams by mistake. After giving it a try, I think this is a nice feature we should add.
Since the emphasis when playing a livestream should be put on the live, the user experience should favor live conditions, probably offering playback resume as an option in this specific case (maybe a persistent button on the player, or a transient notification).

Won't do yet, especially with the TV guide coming.


Show peek preview for footnotes

Please select if your request is either something new or an enhancement

[ ] Enhancement of an existing Feature.
[ ] Request of a new feature.

Please select the area your request applies to. (Multiple selections are Possible. You can leave blank if you're not sure.)

[ ] Workspace - VSCode workspace, vaults, Intellisense/autocomplete, Dendron settings
[ ] Lookup - Dendron's Lookup Command
[x] Views - Dendron Preview, Tree View, Side Panels in the UI
[ ] Schema - Dendron Schemas
[ ] Pod - Data import from / export to Dendron
[ ] Publish - External Site Publish
[ ] Markdown - Markdown features, such as syntax support and features

Is your feature request related to a problem? Please describe
It is hard to know what a footnote number in text refers to.
Describe the solution you'd like
Dendron Preview and editor should both show the peek preview of the footnote on mouse-over as Wikipedia does, for example.

This would aid reading and editing text in many situations.

Thanks @SR-- for the suggestion.  It would be a good enhancement to have the read experience of the preview be on parity with the editor pane.

It would be great if also support Show Hover(by mouse hovering or shortcut command editor.action.showHover) in editor.
With that, sometimes no need to open a single preview tab.


Documentation integration for alpha release

[x] Update documentation including the brand new VIZ API
[x] Review documentation
[x] Review documentation again

Closed by #308


L2 matches with no alignment found in L1 not returned

For example,
TREE_ (FEATS_ "VerbForm=Sup") [FORM "har"]

on en-sv PUD gives 67 hits instead of the 79 reported in the HiC paper.
Happens in both branches, seemingly since replacing gf-ud with deptreehs.

(this could be an option)


Adding Snippets has errors

script: https://github.com/GoogleCloudPlatform/google-cloud-java/blob/master/utilities/add_snippets_to_file.py

Method signature with @Nullable does not work.
Fix:
At this line in add_snippets_to_file.py#L99, replace the existing:
',\s+'.join(['(final\s+)?{}\s+{}'.format(
with this:
',\s+'.join(['(@Nullable\s+)?(final\s+)?{}\s+{}'.format(

for all single line comments, the script either fails (when its the last method in the file) or appends javadocs to the successive method.

example:
/** some javadoc */
public String method ( String args );

@pongad It seems your PR will resolve this issue, so I am giving this to you.

Should be fixed by #2838.


revisit abstract Store contract

iirc we didn't make Store.sol a thing to inherit from because we expect folks to implement IStoreWrite.sol interface
wondering why we don't have e.g. StoreWrite.sol as an abstract contract, then a abstract Store is StoreRead, StoreWrite ?
(we use ^ this approach for store hooks)

@yonadaaa did you start on this?


DOC: Add License

What's wrong with the existing documentation
I want to add MIT License to this repository, kindly assign me this issue under GSSOC'24
Add ScreenShots
No response
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

What's wrong with the existing documentation
I want to add MIT License to this repository, kindly assign me this issue under GSSOC'24
Add ScreenShots
No response
Record

[x]  I agree to follow this project's Code of Conduct
[x]  I'm a GSSOC'24 contributor
[x]  I want to work on this issue

Okk


TF-M based PKCS#11 library implementation

TF-M based PKCS#11 library implementation
Description:

Implement the PKCS#11 APIs based on TF-M.
The iot_pkcs11_pal layer is removed. The objects are stored in TF-M via PSA API. The platform dependency of storage is implemented in TF-M other than Amazon FreeRTOS.
Interfaces of converting MBEDTLS format input into PSA format input are added.
Add PKCS#11 library test project.

TODO:
Use the Amazon-FreeRTOS logging task for printing logs instead of calling Musca-B1 platform print APIs.

Seems no more comments so far. We're going to consolidate the patches and merge them as the initial PKCS11-PSA support.

Merged this PR.
It passes the PKCS#11 test suites on Arm Musca B1 board.
Further enhancements and bug fixes will be addressed in other PRs.


Can we show two or more appointment according to person name in Day View?

Can we show two or more appointment according to person name in Day View? Please check the attached screenshot for better understand.
https://www.dropbox.com/s/rpivzs75xu83xiu/Screenshot 2021-08-06 at 15.35.54.png?dl=0

@Pinkesh-25 The calendar only cares about the eventSource array you passed in, it's purely your choice how you want to display the events.


random panic

This line throws a panic. While this might be acceptable (not sure), it should at least report the error, giving me some clue as to what is going on.
https://github.com/go-gremlins/gremlins/blob/b62af323b7da2f7b6a8ae5569b059d1eec730269/internal/engine/executor.go#L167

That's a leftover from early implementation days. You're right, it should be removed.


Fix for Quote in username errors dapr init #972

Description
dapr init fails to run with certain username on Windows.
This issue was caused because of an extra apostrophe or single quote in the username since powershell considers it as special character and hence creates syntax issues. To solve this, I created a function in '/utils/utils.go' to take any command and converts into powershell compatible command by appending another quote character after every such character. This makes the command powershell compatible. Now it doesn't throw any syntax error and runs as expected.
Closes #972
Testing
PS C:\Users\Carlin'tVeld> cd D:
PS D:> .\dapr.exe init
Making the jump to hyperspace...
Container images will be pulled from Docker Hub
Installing runtime version 1.11.1
Downloading binaries and setting up components...
Downloaded binaries and completed components set up.
daprd binary has been installed to C:\Users\Carlin'tVeld.dapr\bin.
dapr_placement container is running.
dapr_redis container is running.
dapr_zipkin container is running.
Use docker ps to check running containers.
Success! Dapr is up and running. To get started, go here: https://aka.ms/dapr-getting-started
PS D:>

Thanks @mohitpalsingh for your first contribution, can you just fix the lint failure, overall PR LGTM.

Thank you for the feedback and acknowledging my contribution. I appreciate your support. I have addressed the lint failure and made the necessary fixes. Your approval of the overall pull request means a lot to me. Thank you once again for your time and guidance.


Allow Sem Ver filtering

Is your feature request related to a problem? Please describe.
Nope
Describe the solution you'd like
I would like to be able to retrieve the "latest" version with a semver filter.
For instance "Retrieve the latest 1.14.* for golang": let's say 1.14.15. As for today, the "latest" * being 1.15.7.
Describe alternatives you've considered

Dependabot has this feature which make it pretty useful
In term of pure "continuous update", then the digest of the latest docker image 1.15 could be used if we keep using updating docker FROM instruction. but we loose the information as a build argument (cant' use a label go.version for
instance)

Additional context
Nope

After looking at the code here, it's seems that it maybe easier than expected, at the moment we retrieve the latest 5 releases, and we pick the first one which is not a pre-release or a draft. Instead, we could retrieve every releases then sort them like done for jenkins releases.
This would also be a good opportunity to add three new parameters so we could select if we consider pre-release, release, draft.

https://github.com/Masterminds/semver

This library sounds awesome, and all we need :smile:


ir: Support nested namespace specifiers.

Fixes #677

r? @fitzgen

@bors-servo r+

:pushpin: Commit 8c588b3 has been approved by fitzgen

:hourglass: Testing commit 8c588b3449621b39d8d42ab0008e93801cba6138 with merge 6cfb0ec4b8e304732b56618f386d4557ae8ff4cb...

:sunny: Test successful - status-travis
Approved by: fitzgen
Pushing 6cfb0ec4b8e304732b56618f386d4557ae8ff4cb to master...


install.R

[x] add ONBUILD code for install.R
[x] set repos which will be used if the user does not set it

rocker script is setting
$CRAN
https://p3m.dev/cran/__linux__/jammy/latest


branch-2.1 Fix CCR task repeat attach task DCHECK failed(retry)

Proposed changes
pick https://github.com/apache/doris/pull/32784
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


A fatal Javascript error occured

Ubuntu 18.04.1 LTS
Node: v8.10.0
Git version: 2.17.1

Snap --verbose:
name:      discord
summary:   All-in-one voice and text chat for gamers
publisher: Snapcrafters
contact:   https://github.com/snapcrafters/discord/issues
license:   unset
description: |
  All-in-one voice and text chat for gamers that's free, secure, and
  works on both your desktop and phone.
commands:
  - discord
notes:
  private:           false
  confinement:       strict
  devmode:           false
  jailmode:          false
  trymode:           false
  enabled:           true
  broken:            false
  ignore-validation: false
snap-id:      qHVefGEBezeuCeSfTND40uoUD6GRw8BO
tracking:     stable
refresh-date: yesterday at 21:31 EDT
channels:
  stable:    0.0.5 (73) 142MB -
  candidate: ↑
  beta:      0.0.5 (73) 142MB -
  edge:      0.0.5 (73) 142MB -
installed:   0.0.5 (73) 142MB

Window says:
Error: Cannot find module 'betterdiscord'
    at Module._resolveFilename (module.js:470:15)
    at Function.Module._resolveFilename (/snap/discord/73/usr/share/discord/resources/electron.asar/common/reset-search-paths.js:35:12)
    at Function.Module._load (module.js:418:25)
    at Module.require (module.js:498:17)
    at require (internal/module.js:20:19)
    at Object.<anonymous> (/home/miw/snap/discord/73/.config/discord/0.0.5/modules/discord_desktop_core/core/app/mainScreen.js:2:22)
    at Object.<anonymous> (/home/miw/snap/discord/73/.config/discord/0.0.5/modules/discord_desktop_core/core/app/mainScreen.js:609:3)
    at Module._compile (module.js:571:32)
    at Object.Module._extensions..js (module.js:580:10)
    at Module.load (module.js:488:32)

What's the output of betterdiscordctl reinstall --snap -v -v?
And please enclose it within ```.

miw@q8D-AX370-K7:~$ betterdiscordctl reinstall --snap -v -v
VV: Checking /snap/discord/73/usr/share/discord
V: Using Discord at /snap/discord/73/usr/share/discord
V: Found modules in /home/miw/snap/discord/73/.config/discord/0.0.5/modules
Killing Discord processes...
V: Removing old core folder.
V: Unpacking core asar...
Updating asar...
V: Patching core entry...
Cloning https://github.com/rauenzi/BetterDiscordApp...
Cloning into '/home/miw/.local/share/betterdiscordctl/bd/0'...
remote: Counting objects: 6704, done.
Receiving objects:  29% (1975/6704), 4.84 MiB | 16.00 KiB/s

This seems to be a network error.
You can edit your betterdiscordctl file and add --depth=1 to line 489. That will make it run a shallow clone which should be faster.

where is the file in the betterdiscordctl folder?

It's where you downloaded it. You can find it with command -v betterdiscordctl. You might need root privileges to edit it.

git clone "$bd_repo" -b "$bd_repo_branch" --single-branch "$bd" --depth=1, save & retry betterdiscordctl reinstall --snap -v -v ?

Yep.