Publishers want to allow reviewers and collaborators to view and explore their private collections

Stories
Publishers want to share access (view, download, and explore) to their private collections with reviewers or collaborators while keeping the collection hidden from data consumers.
Reviewers and contributors do not want to register with the cellxgene portal to access shared collections.
Publishers want to remove access to their private collections.
_Curators need to share access to staged private collections for review and approval by their owners prior to the curator publishing the collection. See the manual process for details.
_

Related  CZI curated dataset publication stories from the 2020 plan:
Scientists who host pre-publication data on Corpora want their data to be private until their dataset has been peer reviewed so they have a chance to fix errors before the data are widely viewed.
Scientists who provide us data want an opportunity to review their submission before it is publicized.
Related Scientist self-publication stories from the 2020 plan:
Scientists want to manage datasets themselves so they can more rapidly and easily use the service to collaborate with teams or share data. They need to be able to:
2. Create and manually share a secret (bearer token) with reviewers to authorize access to their private datasets (or a revision) and to later revoke this secret.

Notes

The current design proposes to use a capability url as a bearer token. The publisher or curator is responsible for copying and forwarding the token to collaborators or reviews.
Access is coarse grained - there is one bearer token that is shared with all reviewers or collaborators rather than a bearer token per individual.
This feature is available for private collections on the publisher my collections page.

UX Design
Figma for Shared Collections

Closing - Based on the minimal approach that we’re taking with curators, we don’t need UX to create links for sharing (that’s the browser address bar) or revoking shared links.


Add support for custom attributes

Is it possible to add support for custom attributes in the SAML response?
For example, givenName, lastName, email, and so on.

This is a great idea @agiannone and also ties in with https://github.com/boxyhq/mock-saml/issues/115. Thank you.

Thanks for creating this issue @agiannone.


Vidstreaming added Cloudflare

Vidstreaming.io added cloudflare, resulting in errors when using it. Will update vidstreaming extractor once Seleniumscrape gets accepted.

Changing download to server.php makes it scrapable without any cloudflare. This method would allow for more providers from vidstream (hydrax for example). Will probably work on this later this week.

Fixed in https://github.com/vn-ki/anime-downloader/pull/362


Support SDHC driver for Renesas RA8 devices

Add support for SDHC driver running on Renesas RA8

boards: renesas: Add support for sdhc
drivers: disk: Add support for sdhc driver
dts: renesas: Add support for SDHC node.
dts: bindings: Add support for sdhc driver.
tests: drivers: disk: disk_performance: Add support for RA8.
tests: drivers: disk: disk_access: Add support for RA8.

Thank you for the contribution, but this driver is implementing the disk API directly for an SD host controller. New SD host controllers need to be implemented using the SDHC API: https://docs.zephyrproject.org/latest/hardware/peripherals/sdhc.html.
Take a look at the drivers within drivers/sdhc for some examples of how this would be implemented.

We noticed a similar implementation for the disk driver on the SDMMC IP in drivers\disk\sdmmc_stm32.c. If we want to support SDMMC, do we have to implement it in drivers/sdhc, instead of drivers/disk?

We noticed a similar implementation for the disk driver on the SDMMC IP in drivers\disk\sdmmc_stm32.c. If we want to support SDMMC, do we have to implement it in drivers/sdhc, instead of drivers/disk?

Yes, you do- I'm sorry that the STM32 implementation made things unclear. This implementation is present from before the SDHC layer was introduced, and needs to be transitioned to use the SDHC class instead of the disk driver layer. New implementations should not be based off it

@danieldegrasse : thanks for your information. We have implemented the driver using the SDHC API instead of the disk driver layer. Please take a look.

Thanks for the rework here, this looks much better. General request, please split the commits up a bit more. Something like this:

Commit updating HAL
Commit adding driver code and DTS binding
Commit adding DTS entry to RA8 SOC
Commit adding board support for the SDHC to the ek_r8d1
Commit adding test support for the board

@danieldegrasse : I split commits as your comment. Please take a look. Thank you

Rebases to update  HAL commit ID

@danieldegrasse : Could you please revisit this PR? Thank you

Rebases to solve conflicts


range() with non-numeric const argument crash fix

Fix: #38603

Thanks!

Cherry-picked for 3.2.2.


chore: bump to the latest libs

What type of PR is this?
/kind feature
Any specific area of the project related to this PR?
/area build
What this PR does / why we need it:
bump to the latest libs commit
Which issue(s) this PR fixes:
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
NONE

Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 035d0ac11e382fbf07bf94f6ac3e1313737bd31b

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, jasondellaluce
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,jasondellaluce]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

New changes are detected. LGTM label has been removed.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 6a8c711761cd352934bf32bdebfdeb4473175873

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, FedeDP, LucaGuerra
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP,LucaGuerra]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


fix: Fixed an issue where custom bot buttons could not be pressed on mobile

Issue #, if available:
#555
Description of changes:
Fixed screen scrolling and alignment settings.

Before scrolling
After scrolling to the right

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Key changes:
Responsive Layout Improvements
-

+

Horizontal scroll support
- overflow-x-hidden
+ overflow-x-auto

Minimum width guarantee
+


⚠ Bofrost.fr has degraded performance

In 25c910e, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1361 ms

Resolved: Bofrost.fr performance has improved in acbb5bf.


With multiple agent hosts, ansible appears to be concatenating host IPs

My inventory file
k3s_cluster:
  children:
    server:
      hosts:
        <IP_ADDRESS>
    agent:
      hosts:
        <IP_ADDRESS>
        <IP_ADDRESS>

  # Required Vars
  vars:
    ansible_port: 22
    ansible_user: nick
    k3s_version: v1.26.9+k3s1
    token: "mytoken"  # Use ansible vault if you want to keep it secret
    api_endpoint: "{{ hostvars[groups['server'][0]]['ansible_host'] | default(groups['server'][0]) }}"
    extra_server_args: ""
    extra_agent_args: ""

  # Optional vars
    # api_port: 6443
    # k3s_server_location: /var/lib/rancher/k3s
    # systemd_dir: /etc/systemd/system
    # extra_service_envs: [ 'ENV_VAR1=VALUE1', 'ENV_VAR2=VALUE2' ]
    # List of locally available manifests to apply to the cluster, useful for PVCs or Traefik modifications.
    # Manifests should be either full paths or relative to the playbook directory.
    # extra_manifests: [ '/path/to/manifest1.yaml', '/path/to/manifest2.yaml' ]

When I run this
$ ansible-playbook playbook/site.yml -i inventory.yml

I get
PLAY [Cluster prep] ****************************************************************************************************************

TASK [Gathering Facts] *************************************************************************************************************
[WARNING]: Unhandled error in Python interpreter discovery for host <IP_ADDRESS> <IP_ADDRESS>: Failed to connect to the host via ssh:
ssh: Could not resolve hostname <IP_ADDRESS> <IP_ADDRESS>: nodename nor servname provided, or not known
fatal: [<IP_ADDRESS> <IP_ADDRESS>]: UNREACHABLE! => {"changed": false, "msg": "Data could not be sent to remote host \"<IP_ADDRESS> <IP_ADDRESS>\". Make sure this host can be reached over ssh: ssh: Could not resolve hostname <IP_ADDRESS> <IP_ADDRESS>: nodename nor servname provided, or not known\r\n", "unreachable": true}
^C [ERROR]: User interrupted execution

If I run this with just one agent host IP (either one) it works fine. Seems to be incorrectly parsing the yaml file. For reference
$ ansible-playbook --version
ansible-playbook [core 2.15.6]
  config file = /Users/nick/Downloads/k3s-ansible-master/ansible.cfg
  configured module search path = ['/Users/nick/.ansible/plugins/modules', '/usr/share/ansible/plugins/modules']
  ansible python module location = /opt/homebrew/Cellar/ansible/8.6.1/libexec/lib/python3.12/site-packages/ansible
  ansible collection location = /Users/nick/.ansible/collections:/usr/share/ansible/collections
  executable location = /opt/homebrew/bin/ansible-playbook
  python version = 3.12.0 (main, Oct  2 2023, 12:03:24) [Clang 15.0.0 (clang-15<IP_ADDRESS>)] (/opt/homebrew/Cellar/ansible/8.6.1/libexec/bin/python)
  jinja version = 3.1.2
  libyaml = True

I faced the same "issue". However, when looking at the YAML inventory description in the ansible documentation, there are trailing colons at the end of the host definitions, so in your case
k3s_cluster:
  children:
    server:
      hosts:
        <IP_ADDRESS>:
    agent:
      hosts:
        <IP_ADDRESS>:
        <IP_ADDRESS>:

will work. We should update the README and the example though.

Yup that's it - thanks.


"odo create nodejs"  creates a service for the wrong port for a default express app

By default, express generates an app that starts a server listening on port 3000, not 8080.  However, "odo create nodejs" creates a service that uses port 8080, and I can't seem to find a way to control this.
What this means is that someone trying a what would be a more or less canonical node app won't see something working out of the box.  This should either be documented and/or there should be a way to control it.

Current logic is similar to what oc new-app does. odo inspects s2i builder image and create service for ports that are marked as exposed in that image.
We should probably allow users to define custom ports when creating a component and creating url (Route).

I think that adding such an option would be very useful, as well as documenting the current behavior explicitly.  After trying odo, I tried nodeshift and it has the same behavior, but it does allow one to specify ports.

We now allot specifying a port when using odo create. Closing this issue: https://github.com/redhat-developer/odo/issues/552


chore(deps): update golang.org/x/exp digest to b2144cd

This PR contains the following updates:

Package
Type
Update
Change

golang.org/x/exp
require
digest
4a55095 -> b2144cd

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>7
Details

0 of 0   changed or added relevant lines in 0 files are covered.
4 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.2%) to 59.241%

Files with Coverage Reduction
New Missed Lines
%

pkg/config/env.go
4
79.65%

Totals

Change from base Build<PHONE_NUMBER>0:
-0.2%

Covered Lines:
1234

Relevant Lines:
2083

💛  - Coveralls


CP-13454: Unhandled Null Pointer Exception in Line 351 of FibonacciHeap.cs

From unknown CodePlex user on Tuesday, 09 June 2009 02:15:39
Hi,
I have an unhandled null pointer exception in Line 351 of FibonacciHeap.cs:
if (deletingNode || (priorityComparsion(NewKey, next.Priority) * DirectionMultiplier) < 0)

In file BugProject.zip there is both the self contained project as well as a jpeg snip of the visual studio 2010 debugger window stopped with the exception, together with the locals and the stack. There is no problem when Prim is changed to Kruskal.

The project referencing QuickGraph is written in F#, the graph here was built trivially by just adding vertices and edges, not using the delegates. Please let me know if some clarifications/tests are needed.

Thanks,
Vladimir
Bug.zip
BugProject.zip

Form unknown CodePlex user on Wednesday, 10 June 2009 00:48:43
Could you attach the source code instead of a zip?

Form unknown CodePlex user on Wednesday, 10 June 2009 14:18:34
I made a self contained test case. To build it just create an F# project (in VS 2010 Beta 1+) with this file and add quickgraph.dll as a reference.

Form unknown CodePlex user on Wednesday, 10 June 2009 18:09:05
I looked, after my simplifications the graph remained with just two vertices. Could the issue be related to the fact that the graph is not simple?

Form unknown CodePlex user on Sunday, 19 July 2009 19:48:43
Hi Jonathan,
I noticed here that if I enforce that no parallel edges are added, everything works fine.

Form unknown CodePlex user on Friday, 13 May 2011 08:01:05
Ha parralel edges. That's a good hint.


Start as service feature

For some soop buckets it might be usefull to have the ability to start them as a service.
For example it would be useful if it where possible to enable mariadb as a service.
Would the post post_install fit these kind of tasks ?
You will most likely need Administrator rights to create new services, that would probably be an issue ?

It is the ideology of scoop to mess with your system as little as possible. So keeping with this ideology I think installing services as part of installing an app is out.
And also, yes you would probably need administrator access which is also out.
That's not to say that you couldn't create your own bucket with these kind of manifests that in general require administrator access.


Document OWNERS file usage

What to document
Maintainership in our repository is automated and relies on the OWNERS file spec
(see https://www.kubernetes.dev/docs/guide/owners/).
However, we have never explicitly documented it. I'd like to document that in our governance.
This idea was initially proposed by @kris-nova in this comment.

I agree with this and like it @leogr

I agree on this! +1


Support srcset

@klihelp ? - blazy supports srcset. Please elaborate

where can I find documentation how to use picture element?


Bug: Filtering on owner/operator column in account data not pulling all accounts

RT 74691, 74690
Two CAMPD users have reported attempting to run queries with filtering by owner/operator and have experienced several facilities missing (Holdings and Allowance Transactions). The facilities can be found by using the account name/number filter so they are available just not showing up via the owner operator column.
Filtering by owner/operator should return any associated accounts and allowances and transactions.
Acceptance Criteria:
Custom Data Download Search:

[ ] Filtering by Owner/ Operator in Allowance -> Account Information returns all account records for facilities associated with the selected Owner/ Operator
[ ] Filtering by Owner/ Operator in Allowance -> Holdings returns all allowance holdings records for facilities associated with the selected Owner/ Operator
[ ] Filtering by Owner/ Operator in Allowance -> Transactions returns all allowance transactions records for facilities associated with the selected Owner/ Operator
[ ] Filtering by Owner/ Operator in Compliance -> Allowance Based returns all compliance records for facilities associated with the selected Owner/ Operator
[ ] Filtering by Owner/ Operator in Compliance -> Emissions Based returns all compliance records for facilities associated with the selected Owner/ Operator

Filtering by Owner/Operator should return any associated accounts, allowances, and transactions for the following API endpoints on the Swagger pages:

[ ] streaming-services/accounts/attributes
[ ] streaming-services/accounts/allowance-holdings
[ ] streaming-services/allowance-transactions
[ ] streaming-services/allowance-compliance
[ ] streaming-services/emissions-compliance
[ ] account-mgmt/allowance-holdings
[ ] account-mgmt/allowance-transactions
[ ] account-mgmt/accounts/attributes
[ ] account-mgmt/allowance-compliance
[ ] account-mgmt/emissions-compliance

Two CAMPD users have reported attempting to run queries in allowance holdings by owner/operator and  have experienced several facilities missing. The facilities can be found by using the account name/number filter so they are available just not showing up via the owner operator column.

@lgiannini1 please document everywhere this needs to be checked.

Moving to ready for deployment. I identified several facilities that have multiple owner/operators and used the filter function to find them by both owners. Examples: Missouri Basin Municipal Power (Owner)|Western Minnesota Municipal Power (Owner), Arizona Public Service Company (Operator)|Arizona Public Service Company (Owner)|Pacificorp Energy Generation (Owner)


YouTube issue

When i setting you tube url as video src video is not playing

Sorry, this plugin does not support the playback of YouTube links


CORS gets bypassed.

Try making a normal request on the routes that are followed by the CORS middleware from an origin with false positive:
import Axios from 'axios';
const r = await Axios.get('localhost:3030');
console.log(r);

You would receive a CORS error.
But if you add the origin header to your request with a white-listed domain you'd by pass the cors middleware from any domain/origin.
import Axios from 'axios';
const r = await axios.get('localhost:3030', {headers: {'origin': 'localhost:3000'}});
console.log(r);

Does this count as a vulnerability?

I mean, it would count as a vulnerability in axios at best. This module jus follows the cors spec. It would have no mechanism to know if the origin was set by a web browser, axios, or anything else, as stated in the cors specification.


Workbook update does too much if all I did was change the tag set

I like our model of simplifying the model so that tags are just a set on the workbook and you can modify the set and then simply call "update". However, we are not tracking whether anything else changed on the workbook and so are calling "update" on the general workbook properties always which makes changing the tags unnecessarily expensive. We need to have some _dirty boolean that is set to false after parsing the results and flipped when any property is set. Then we can avoid unnecessary calls when nothing has changed.

I agree that our 'Update' pattern is a little weird, and it makes some of our setter code more complicated than it needs to be.

If this is done it will be in the course of other work.


Update styles on text inputs for auth pages

Update text field styling to match Figma design.
Note: I was unable to find a selector for the MUI TextField border-bottom to use with CSS modules. Was forced to use in-line styles for the underline color.
Note: In practice, I felt the lighter gray color looked better than the specific shade from the Figma design.
Figma:

Fixes #386
Type of change

[x] Other 🗒 Styling update

Checklist:

[x] I have performed a self-review of my own code.
[x] My changes generate no new warnings.
[x] I have ensured that my pull request title is descriptive.

It's a great improvement. It's more visible and appealing to the user.


accessible_by with hash of conditions causes Not unique table/alias

Steps to reproduce
Problem occurs with cancancan 3.x when a hash condition is defined, which
does a LEFT JOIN but the exact same table is already joined due to some eager loading or something else.
https://gist.github.com/Nghi93/c00d9dd83d3e51af5512cf4d36ed340f
Expected behavior
In cancancan 2.x it does not do a second LEFT JOIN on the same table.
Actual behavior
In cancancan 3.x it does a second unnecessary LEFT JOIN on the same table.
ActiveRecord::StatementInvalid: Mysql2::Error: Not unique table/alias: 'claim_files': SELECT DISTINCT claims.* FROM claims INNER JOIN claim_files ON claims.claim_file_id = claim_files.id LEFT OUTER JOIN claim_files ON claim_files.id = claims.claim_file_id WHERE claim_files.policy_id = 1 AND claim_files.some_attribute = 'a'
Maybe the LEFT OUTER JOIN could be done with an alias?
In cancancan 2.0 it seems the second join is omitted.
System configuration
Rails version: 5.2.0
Ruby version: 2.5.5
CanCanCan version 3.0.1

Workaround:
Claim.includes(claim_file: :policy).accessible_by(current_ability) # seems cleaner anyway
We had a few other cases in which we had to change includes, join to left_joins in order to avoid the error above.

can you please check if https://github.com/CanCanCommunity/cancancan/pull/600 fixes this for you

The latest commit on #605 should fix this


Certificate flowcharts missing questions

Currently exhibited in the Legal flowchart there are missing questions (for example piaAudited in certificate.GB.xml). This is because they have a dependency on questions that have a text input answer that currently are not registered as "answers" in flow.rb as said by @pezholio. Only questions with radio button answer structure or similar are registered as answers.  Therefore a {question} !== "" test is not parsed correctly. Quick solution is to move question with this dependency to immediately after the question it depended on, now excluding the dependency. This has been attempted with copyrightStatementMetadata already.

I mentioned to @pezholio this morning that it might be better to use the Survey, SurveySection, Question, Answer and Dependency models instead of reparsing the xml file.
This means you'll only have one flowchart per jurisdiction and you don't have to read any xml.


Fix VLenBytes docstring to use VLenBytes

Had a copy-paste error where it was using VLenUTF8 previously.

Just manually edited the C file. Roughly checks out when I created it with Cython, but was using a different version and probably not the same settings used to create the C file originally.

Thanks, I've manually merged in #57. Also I've added running of doctests for the cython modules to tox so hopefully should catch this kind of thing automatically in future.

Ok, thanks.


openh264: 2.4.0 -> 2.4.1

Changes: https://github.com/cisco/openh264/releases/tag/v2.4.1
Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@trofi i think this should target staging

Since this package has no maintainer and it now uses Meson, I picked it to some refactor at #285797


get new posts to save to database

Fixed post and delete functions by adding 's' to post in index.js!


Cannot save Account Details or Add Group

Describe the bug
A clear and concise description of what the bug is.
Cannot Save Account Details or Add group
To Reproduce
literally just go to builder and try:
[EXAMPLE STEP TEMPLATE BELOW: FILL IN INFO AND ADD/REMOVE STEPS AS NEEDED]

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
 PUT N/A IF YOU ARE UNSURE.]
Screenshots
If applicable, add screenshots to help explain your problem.
[PASTE IMAGE HERE.
 PUT N/A IF THERE IS NONE]

@sfenste
Thanks for the issue report. This is a known issue we have planned to fix in the next release.


Get rid of standard.config

... by merging it into nextflow.config since it is included in all profiles. @MaxUlysse volunteered :-)

Almost done with that, I'm adding some stuff into it thanks to @ewels
cf: https://github.com/SciLifeLab/NGI-RNAseq/pull/101


bgpv1: Reorganize BGP config reconcilers

This PR aims to re-organize BGP ConfigReconcilers to achieve a better structure and avoid conflicts in case of concurrent changes.
The most significant change is moving the code from a huge single source file (pkg/bgpv1/manager/reconcile.go + co-located reconcile_test.go) to multiple files, one source + test file per each reconciler. The changes are only structural, there is no functional change. For more details, please see commit messages in the individual commits.

/test

/test


Lua plugins rework

Rework folder organization
Now turbo_audio (the rust code) is in its own folder, alongside the "effects" folder
The lua effects are in the "effects/lua" folder.
Rework some cancer that we had
Add a field in the config file to point to the lua folder
Add support appending to the package.path (http://www.lua.org/manual/5.1/manual.html#pdf-package.path) variable in lua so that lua effects can require("path.to.lualib") from any desired directory. This means that the lua libraries we provide could live somewhere else than the lua effects seamlessly.
Sets up the folder organization for putting an "effects/rust" folder where all the other effects will live

This is a good thing because LSPs are shitting their pants when you have a rust project inside another rust project

Format the json settings file
Use impl AsRef<Path> as parameters instead of &Path because it's better. (https://doc.rust-lang.org/rust-by-example/trait/impl_trait.html)

@samichoulo911


[Parse/CodeCompletion] Implement effects specifier completion

Complete async and throws in function signature, function type, and closure signature.
Along with that, rewrote effects specifier parsing so that we can handle them with unified function parseEffectsSpeficier() in various position (i.e. function signature, function type, and closure signature).
Also, this improves diagnostics for them, e.g. duplicated specifiers.
rdar://problem/72199413

@swift-ci Please smoke test

@swift-ci Please smoke test

@swift-ci Please smoke test

@swift-ci Please smoke test

@swift-ci Please smoke test


Suggestion - FAQ w Physical System Requirements,

Would you consider adding a beginner's FAQ section or a 'Physical System Requirements' note that provides guidance on the recommended computer specs for optimal performance with Unsloth? This would help new users avoid frustration and ensure they have the necessary resources to run the model successfully. A simple table or paragraph mentioning the minimum/recommended CPU, GPU, RAM, and storage requirements would be very useful.
Including links to beginner-friendly tutorials, videos, or documentation on setting up Unsloth locally could make it more accessible to us noobs. Links on how to get started in a Google colab or AWS hosted solution could also lower barriers for those who don't have a capable local system.
It might be nice to get a clue on how long it takes to fine-tune, likely sweet-spot settings to mess with, etc.
This might ease the overall beginner experience and make Unsloth more approachable for a broader audience.

That is a fantastic idea!
Approximately:
Mistral 7b: Min 6-8GB VRAM
Llama-3 8b: Min 8-12GB VRAM
Gemma 2b: Min 8GB VRAM
Gemma 7b: Min 12GB VRAM
Llama-2 7b: Min 8GB VRAM
Llama-2 13b: Min 16GB VRAM
Llama-2 34b: Min 24GB VRAM
Llama-3 70b: Min 48GB VRAM
I'll colate them in a table

Going to bump this again since it is a common request


Add Julia syntax highlighting for README example

This pull request simply adds syntax highlighting for the README.md Quick start example, as well as fixing the code block by adding the ending three backticks.

Thanks!


Add Microsoft & Dropbox, Refactor FederatedServiceRouter

I added support for Microsoft and Dropbox
I also consolidated a lot of the fetchToken & callback duplicate code by extending the FederatedServiceRouter protocol

@gwynne Made the changes you requested!


Fix thumbnail scaling regression for files with different page sizes (issue 5637)

Fixes the first part of #5637.

/botio-windows preview

This was a hard one, but nice find!


signature input in reverb.Table()

Can someone help provide an example of
replay_buffer = reverb.Table(
name=adders.DEFAULT_PRIORITY_TABLE,
max_size=1000000,
remover=reverb.selectors.Fifo(),
sampler=reverb.selectors.Uniform(),
rate_limiter=reverb.rate_limiters.MinSize(min_size_to_sample=1),
signature=????)
Looks like now signature needed to be defined in the reverb.Table to make the tutorial work?
Can someone help provide an example of what the signature input shall look like?

@DailiZhang2010 Normally you can obtain this with the Adder.signature() class method of the adder you are using.
For examples, check out the make_replay_tables method in any of the jax builders (e.g. https://github.com/deepmind/acme/blob/master/acme/agents/jax/ppo/builder.py#L58 and https://github.com/deepmind/acme/blob/master/acme/agents/jax/td3/builder.py#L120).
Please close the issue if this resolves your question.

Please reopen if there are any outstanding questions / problems.


adding removable labels

directive wrapping availity-uikit's removable label styles

Must wait to merge! Relies on new styles in availity-uikit that have not been released yet. UI Kit's version needs bumped to latest release in this branch.

Thanks Bobby!


ngTable Coloumn Customization

Dear Team,
As i have an One table it was some customized Filters and Sorting, but i want to make some column wise customization like
In top of the table i have an One Button, if i click the Button it will shows some one Multi Select Box with Drag Drop option.
In Side of Lpanel I have an options are table header title
Cpanel is select a coloumn title selective options like Lpanel to rpanel, Rpanel to Lpanel, All to Rpanel, All to lapnel insid ehave Drag Drop.
In table header name(data) are loaded into the Select Box. whatever i action will perform like L to R R to L, Drag and Drop
I want to hide the columns and i want make the column positions too

Kindly Help me for an custom Directive / Controller,
Hazdik DA, India

Hi @hazdik,
I believe you may be confusing this library with the actual bower project for dependency management. This project is designed to assist with the integration of bower with ruby servers like Rails and Sinatra.
Thanks


Hide title and description for flagged proposals

Currently we show a warning icon with tooltip for flagged proposals

To make it more obvious that a proposal was flagged we should hide title and description and show a message "This proposal might contain scams, offensive material, or be malicious in nature." with a checkbox "Show anyway"

I would even hide the author of the proposal (which often is also impersonating a project) and the status

Looks like twitter approach:

@j0hnfl0w what happens when you click learn more in twitter?

@j0hnfl0w what happens when you click learn more in twitter?

You go to some article in the docs that describes flagging mechanism, maybe we need it too.

Alright, maybe we should add a flagged proposals section in the docs @zzuziak


Split up "enum removed" from "enum changed to struct/union" checking

Per https://github.com/obi1kenobi/cargo-semver-checks/discussions/296, it's confusing that changing an enum to a struct (or another type) causes a lint that states the enum was removed. While technically correct (the enum doesn't exist), it doesn't match the user's probable mental model.
Resolution:

[ ] Make the enum_removed lint specifically look for enum deletions, and make it not match enum -> struct or enum -> union cases.
[ ] Add a new enum_became_another_kind_of_type lint for changing enums to structs or unions.

Analogous changes to struct_removed should be considered as part of fixing #297 as well. The future union_removed lint should follow suit too.

Do we need specialized struct/enum removal lints or should we just have an "api item removal" lint and then other lints for changes in the shape of that API item?

Without schema changes, struct/enum/union can certainly be grouped together (via ImplOwner), if you think that's worth doing. With a small schema change, other importables like fn and trait can be added to that list as well (making Importable be a subtype of Item instead of a parallel interface like it is now).
They are currently separate to allow us flexibility without code complexity when phrasing the user-facing error messages, which are entirely template-based.
What would be a huge help is to decide on the desirable error message(s) the user should see in each case. Then we'll know exactly what flexibility we need, and can merge the lints or not merge them, as appropriate.

With a small schema change, other importables like fn and trait can be added to that list

This reminds me of an interesting angle to this: we'd probably want to group API items by which name universe they live in.

I like the idea! Certainly worth a shot and I think it should be doable.


24h and "All day" settings

First of all - really great job, its a very nice Calendar card ...
All i wonder if there is a ability to set time to 24h instead of AM/PM,
and, where to do the translation of "All day" ... Calendar card is showing the right language (de) in
months and day - names - exept of the above mentioned ...
Maybe i have a problem in reading , but i cant find any suggest in this issue ???
Thanks a lot for time

You can set the time format with timeFormat see the readme for a link to how to set that up. Unfortunately, translations for 'All day' aren't given to me, only for datetime related stuff like days of week.

so, shame on me, is there a possibility to implement a variable-set similar to
`fullDayEventText: 'xxxxxxxxxx'
untilText: 'xxxxxxxx'
as found in atomic calendar ?

added in 3.101.0:
fullDayEventText
startText
endText


Pushpin retry connections indefinitely

Hi,
We are considering to run background job for the incoming request, and assign unique id (e.g. channel id) for each incoming request, and Pushpin could send responses to those channels via background tasks.
Unfortunately, Pushpin retries incoming requests indefinitely.
Noticed the similar Github issue #47646, but not sure how does the application set "Grip-Last" value for large numbers of concurrent requests, and each request has random request id?
Just reproduce this behaviour in sample gist code - Go Lang/Dockerfile as follow:

https://gist.github.com/mitalkakaiya/2650e995f00d9b1574be6017ab9d4663

Please feel free to advise alternatives to resolve it, or gist updates would be helpful.
Regards,
Mital

Thanks for your quick response, and explained Pushpin workflow about Grip-Last header. Updated Pushpin source code locally, and works as expected.
As per the above comment, Pushpin makes an initial request to the server as per point 2, and makes an another request to the server as per point 4. Just wondering about the following queries:

What would be the reason for Pushpin to make 2nd request the server?

What is recommended option to execute request in the backend? Should backend (e.g. generate pdf) run on the first request or second request?

If there are multiple backends connected (e.g. backend1, backend2, backend3 etc), does pushpin send the second request to the same backend server as the first request? Just wondering if there is any internal workflow about it.

Please feel free to advise me if there is any documentation or blog post to explain some of these internal workflow details.

Thanks again, and looking forward to understand more details about Pushpin.
Regards,
Mital

The second request is an aspect of Pushpin's reliability feature. Pushpin can optionally notify the backend (currently over ZeroMQ) whenever a channel has at least one subscriber, useful for allowing the backend to limit publishing to only Pushpin instances that have subscribers. However, if the need for the backend to publish happens during the moment after Pushpin has gained a subscriber but before the backend has received the subscription event, there's a chance the backend may discard the publish. To work around this, Pushpin will re-issue any requests held in response mode shortly after a channel gains its first subscriber, so that clients can quickly pick up any missed data. If you aren't tracking subscriptions, then you don't really need to worry about this, other than be aware you may need to handle retried requests sometimes.

The first, since it's best not to rely on the second request always happening (in your case with random channels it should always happen, but for most apps using non-random channels it will vary depending on whether the client is the first subscriber or not; also in the future we might change the conditions or timing of the second request). If you receive a second request, it should return the results of the earlier started task, or hold if the task isn't finished yet.

Pushpin does not attempt to send subsequent requests to the same backend. If you have a DNS A record with multiple IP addresses, a different IP might get used for each request. If you have a load balancer between Pushpin and your backends, then the decision is up to your load balancer.

http://pushpin.org/docs/advanced/#reliability discusses Pushpin's reliability feature and Grip-Last. It was missing some details mentioned here so I've updated it.

One other thing: when setting up a subscription with a prev-id, you need to publish with a prev-id as well, otherwise Pushpin will retry the request rather than deliver the message.


Add regtest network

Motivation
This has been requested, and could be useful for testing.
Possible Design

Add a Regtest variant to Network, update From<Network> for &'static str for db_path(), default port, and genesis hash
Add config field or CLI option for accepting network upgrade heights
Skip loading checkpoints and update NetworkUpgrade::activation_height to use zcashd defaults or user-defined heights
Update target_difficulty_limit() and expected_difficulty returned by difficulty_time_and_history_tree() to match zcashd target difficulty for regtest

remove the implementation constraint that PoWLimit must be less than or equal to (2^256 - 1) / PoWAveragingWindow (approximately 2^251) to avoid overflow

Skip SLOW_START_INTERVAL checks (nSubsidySlowStartInterval is 0 for regtest in zcashd)
Allow for coinbases to be spent to transparent outputs on regtest
Set peerset_initial_target_size to 0 (Zebra should not make any peer connections)
Update prefixes
Match other differences between zcashd testnet vs regtest in chainparams

These are just some notes for if we eventually do this ticket. No action is needed at this time.

We might also want to take a look at:

#1135

It seems like some of these changes would benefit from the Regtest variant containing a consensus rule configuration. (Like zcashd has for all its network variants.)
It would also be a lot easier to make this change if we refactored all our network conditions into methods on the Network enum. For example, a has_slow_start() method, rather than a match statement buried in the zebra-consensus code.

update From<Network> for &'static str for db_path()

We either need to work out how to make sure the consensus rules match the cached state, or make regtest ephemeral by default.
This isn't an issue for mainnet or testnet because their rules are fixed and can't be configured.

Set peerset_initial_target_size to 0 (Zebra should not make any peer connections)

This change isn't enough to stop Zebra making outbound connections, because Zebra will try to connect to new peers every minute or so. Instead, the initial seed peers list needs to be empty for regtest.

Update prefixes

Which prefixes?

Match other differences between zcashd testnet vs regtest in chainparams

If we're not going to connect to other nodes by default, then compatibility with zcashd seems like a lower priority. So we might want to categorise this ticket into:

changes needed to have a regtest mode
changes requested by the people who requested regtest mode (please link to the request in the ticket)
other changes that are useful for full compatibility with zcashd

That way, when we schedule this work, we can split the ticket, give the categories different priorities, then do the estimates.

Here's one request for regtest in Zebra:
https://twitter.com/thecodebuffet/status/1674606998908727297

This is currently blocked by some kind of network refactor. (It doesn't have to be #1135 specifically.)

Hey team! Please add your planning poker estimate with Zenhub @arya2 @oxarbitrage @teor2345 @upbqdn

@mpguerra this ticket is outdated. @arya2 and I have had some private conversations about it, and I think we should update it before we estimate it. (Or replace it with another ticket.)

For example:
I'd like to split the tasks in this ticket based on what the ZSA team needs for their testing, then do the rest of regtest later.

Add a Regtest variant to Network, update From<Network> for &'static str for db_path(), default port, and genesis hash

It will be much easier to gradually add network settings inside the existing Testnet variant. Adding new variants requires changing a lot of code.

Add config field or CLI option for accepting network upgrade heights

This should be split into a different ticket, it's not required to match the default zcashd regtest.

I added these issues to the ticket description, we might need to fix them depending on the required scope:

difficulty adjustment for heights less than 17
contextual validation of the genesis block (it has to be a checkpoint)

https://github.com/ZcashFoundation/zebra/pull/7770#discussion_r1364969449

Personally, the lack of regtest support blocks me from moving to zebrad because it is too hard to write integration tests.

Personally, the lack of regtest support blocks me from moving to zebrad because it is too hard to write integration tests.

@hhanh00 Is it enough for Zebra to support NU5 on its Regtest, or is important that Zebra support Canopy and prior network upgrades on Regtest as well?

Just nu5 is enough but I would also need the ability to generate, invalidate and reconsider blocks. This is how the project I work on tests forks and reorganizations.

@arya2 it seems like there are still some meaningful gaps between zebra regtest mode and zcashd regtest mode.  See: https://discord.com/channels/809218587167293450/811004767043977288/1280987474918772858
Do we know what these are? If so we should update this issue with the remaining work to make them compatible


Diagonal Dragging

I noticed when I move this view controller it drags in a multitude of directions besides up and down and I am assuming that is because of scrollview? Is there any way to stop it from moving diagonally? I am currently trying to stop the scrollview from moving in diagonal directions with every snippet of code I can find online. Could it be because I have auto layout turned on?

Hey, I'm not quite sure what you mean.  Are you talking about a particular page in the controller or are you talking about the scrolling action between them?

I have tableview controller as one of the view controllers and when I was trying to swipe to page over I noticed that I was moving my tableview and the uipagecontroller scroll simultaneous which resulted in a diagonal moving effect. For some reason it seems to not be an issues but that's because I went in a change view things to stop it from moving diagonally.

That's very odd behavior (it shouldn't happen).  You should only ever be able to interact with a single scroll view at once by default.  Anyway, let me know if the issue comes up again

Okay it is coming up again and I think it might be an issue with iOS 7 devices. I have an iPhone 4 and iPhone 6 Plus that I test on and the scroll view works PERFECTLY fine on my iPhone 6 Plus running iOS 8. The problem  is occurring now on only the iOS 7 iPhone 4. The tableview is moving freely and only one tableview gets loaded as well. The contentSize is way bigger than it needs to be. BUT when I scroll over once and back again by pressing the button on the top, the pageView and scrollView reset back to the way they should behave (like it works exactly as it does on the iPhone 6 Plus). I am just not sure why it happens after I swipe to the second view and back to the first.
I have 2 view controllers by the way

I'm not sure I understand the behavior of the bug just yet, but it sounds like some weirdness with UIPageViewController.  The struggle of this repo is that apple makes it really difficult to actually do any any of this, and most of the code is just messing around with UIPageViewController to get it to not be stupid.  I'm not sure if it'll solve your problem, but in the past I've found that it behaves best when things are loaded as asynchronously as possible.  Would you mind sending me a few screenshots?


Is it possible to determine screen height automatically?

The Grid system handle width automatically, but I didn't found anyway to do this on height.

Unfortunately automatically stretching height is not supported, but we can do this manually by listening resize event.


[Image & ListView] Image in ListView is not updated when re-rendering

Say I have such row in ListView
return (
        <View style={styles.chatRow}>
          <Image source={{ uri: otherUser.avatarUrl }} style={styles.chatAvatar} />
          <View style={styles.chatRowInfo}>
            <Text style={styles.chatRowName}> {otherUser.name} </Text>
            <Text style={styles.chatRowText}> {Tool.trimToLength(chatContent, 40)} </Text>
            <Text style={styles.chatRowText}> {otherUser.avatarUrl} </Text>
          </View>
        </View>
    );

When the state changes, the Text's are re-rendered but the Image isn't. The last Text shows that the avatarUrl is updated correctly. However, the Image still shows the old picture.
I'm wondering if this is due to Image doesn't compare {uri: xxx} deeply?
Thanks

It's not a comparison problem, because even the shallow comparison would return false, given you are creating different object instances on every render. So I'm afraid that's not the issue. I think in #1397 @ide mentioned there might be some bug in the Image class that causes Images not to change, might want to check that.

Have same issue, dataSource updated, row change is triggered and text updated but image does not rerender with new url

To answer my own problem - Image rerender works by adding a 'key' param with same value as the uri
https://github.com/facebook/react-native/issues/1397#issuecomment-105367632

Thank you for reporting this issue and appreciate your patience. We've  notified the core team for an update on this issue. We're looking for a response within the next 30 days or the issue may be closed.

@adrian-social-prod 'key' param method seems to have been ineffective?

@adrian-social-prod It does work, but it recreates the entire image view. This might not be problematic for 1 image, but for a few dozen it starts to become costly.

is still a problem in 0.8.0. list view image does not change when a new item is added to list unless the 'key' param is used to force the image to rerender. list is inside a TabBar item.
Seems like a major problem to me.

Hi there! This issue is being closed because it has been inactive for a while.
But don't worry, it will live on with ProductPains! Check out it's new home: https://productpains.com/post/react-native/image-listview-image-in-listview-is-not-updated-when-re-rendering

Any chance I can reopen this?

Please re-open - I still have the issue with RN 0.28.  It only seems to be a problem under certain circumstances.  For me it happens when:
An image is added to the end of the list, AND it is adjacent in the list to another image.  Adding key={uri} didn't help.

@npomfret: Were you able to fix the issue? I actually run into the same problem as yours. I'm using RN 0.30.

might try key={source.uri} instead of key={uri}

Any updates on this? So far only hacking the key property seems to be solving the problem. In my case, i render multiple images in a for loop, and under certain condition, i render different images, so it has to look something like this:
const PATH_TO_IMG_1 = '/path/to/img/1.png'
const PATH_TO_IMG_2 = '/path/to/img/1.png'

render() {
  let arr = []
  for (let i=1; i < 100; i++) {
    if (cond) {
      arr.push(<Image source={require(PATH_TO_IMG_1)} key={PATH_TO_IMG_1+i}/>)
    } else {
      arr.push(<Image source={require(PATH_TO_IMG_2)} key={PATH_TO_IMG_2+i}/>)
    }
  }
  return <View>{arr}</View>
}

Not sure if requiring image within a loop is efficient though.

You can just require the images at the top @carera
const IMG_1 = require('/path/to/img/1.png') but i don't think that's causing your problem ( it may be though )

+1

@merielleimpreso i don't have a 'proper' fix, but hacking the key={uri} worked around it

I'm having the same issue, but hacking the key={uri} also worked for me.

I don't think this should be closed.  Images in ListViews don't re-render when you change the uri props.  Feels like a proper bug.

I had a similar issue: every time I added a new element to the list, it did not populate the new "graphical row" with the correct data (even if in the state are correct) but with a copy of the first row.
Adding the key={id} worked for me.

This is happening to me as well on 0.38. After updating the ListView, the new Image will load only after scrolling it off then on screen.

Yes seeing the same problem as well. This ticket shouldn't be closed.

same issue

So it is open?
Does FlatList take care of this issue?

I just ran into this.
I'm rendering an image using a remote URL, provided by my components initial state. Some UI actions can cause the URL that's store in component state to update. If I render the URL using a Text component, I see state updating. It does not update the Image component. I tried adding a key property of Math.random() and that causes the image to disappear entirely when state is updated.

I join the question of @hugomosh

@hugomosh @jose920405 I've also been experiencing this issue with FlatList :/

I'm also experiencing this issue with Flatlist

+1

Same, when using the FlatList and adding a key = {uri} prop the image will update but then it just show up blank.
If I change the amount of rows in the FlatList or make a few further changes to the data in the FlatList then the new data will appear. I think this has something to do with the row in the FlatList not completely being redrawn maybe? Maybe iterators are caching image resources until the row is completely redrawn.

Someone on StackOverflow report that if you don't set the size on the image, it will re-render correctly. This fix didn't work for me but I'm posting it here for others just in case.
https://stackoverflow.com/questions/37529211/react-native-image-wont-rerender-when-state-changes

Using react-native-image-cache instead of Image component solved this issue for me. Maybe related to https://github.com/kfiroo/react-native-cached-image/issues/26

@alexandretournier  may not be used.
I found the solution.
check this one

thanks @asadallahi


Feedback: add event to hide infowindow on hover when cursor goes out of the map

Received in SB/5517093
When you have a map with infowindows on hover enabled, the infowindows disappear when you're featureOver is not active.
If you move the cursor out of the map without leaving the geometry, the cursor is out but the infowindow will still be there, blocked:

There has been no activity on this issue for several months and that's why are closing it (automatically). If you think this still needs to be addressed please reopen it. The good news is that we are now working on the next version of CARTO.js, stay tuned.


Add django-configurations

I'm quite surprised to see that it hasn't been mentioned here!

You are correct! Thanks for the PR. Now merged.


New Log Drain fields

See: https://github.com/aptible/api.aptible.com/pull/485

cc @fancyremarker


How to make the server IP address field editable for users?

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
We have project "Foundation"(https://www.nutanix.dev/api_references/foundation/#/b3A6MjIyMjMzNDk-reboot-a-node-into-phoenix) where the users need to edit the live server IP address:

Can you please let me know how to make it editable like "try it" section in Stoplight Next.

PS: I have migrated this project from Next to Studio
Expected behavior
The IP address should be editable for API users.
Additional context
Add any other context about the problem here.
Screenshots
If applicable, add screenshots to help explain your problem.
Environment (remove any that are not applicable):

Library version: [e.g. 3.0.0]
OS: [e.g. Windows 7]
Browser: [e.g. Chrome 61]
Platform: [desktop or web]

Hi, Can anyone help on this?

Hi, is there any update on this?

Hi,
Is this issue being taken care of? if yes, by when can I expect a response? Customers are asking for a workaround on this.


Build failure: ghc910

Steps To Reproduce
Steps to reproduce the behavior:

nix build "nixpkgs/release-24.05#haskell.compiler.ghc910"

Build log
nix log /nix/store/rm2gv2sr5sliyr5b9iw8233jp3w3d6xx-ghc-9.10.1-source.drv
@nix { "action": "setPhase", "phase": "unpackPhase" }
Running phase: unpackPhase
unpacking source archive /nix/store/zbk236pd71id8hjkzscz386hjv5z0sp3-ghc-9.10.1-src.tar.xz
source root is ghc-9.10.1
setting SOURCE_DATE_EPOCH to timestamp<PHONE_NUMBER> of file ghc-9.10.1/utils/hpc/src/Trace/Hpc/Parser.hs
@nix { "action": "setPhase", "phase": "patchPhase" }
Running phase: patchPhase
applying patch /nix/store/32kfndxy8dbaa47jbpg2fp4lsd4wcxmg-docs-sphinx-7-ghc98.patch
patching file docs/users_guide/rtd-theme/layout.html
applying patch /nix/store/cz1vxgj0sjw51g8vx7bh4r6vl3j9xxgj-Cabal-at-least-3.6-paths-fix-cycle-aarch64-darwin.patch
patching file libraries/Cabal/Cabal/src/Distribution/Simple/Build/PathsModule.hs
Hunk #1 FAILED at 46.
Hunk #2 FAILED at 56.
2 out of 2 hunks FAILED -- saving rejects to file libraries/Cabal/Cabal/src/Distribution/Simple/Build/PathsModule.hs.rej
patching file libraries/Cabal/Cabal/src/Distribution/Simple/Build/PathsModule/Z.hs
Hunk #1 succeeded at 19 (offset 1 line).
Hunk #2 succeeded at 62 (offset 9 lines).
Hunk #3 succeeded at 155 (offset 9 lines).
Hunk #4 succeeded at 191 (offset 9 lines).
Hunk #5 succeeded at 201 (offset 9 lines).
Hunk #6 succeeded at 245 (offset 9 lines).
patching file libraries/Cabal/cabal-dev-scripts/src/GenPathsModule.hs
patching file libraries/Cabal/templates/Paths_pkg.template.hs
Hunk #1 succeeded at 14 (offset 5 lines).
Hunk #2 succeeded at 77 (offset 5 lines).
Hunk #3 succeeded at 111 (offset 5 lines).

Additional context
I'm trying to build GHC 9.10 on my M1 and its failing to apply a patch.
This occurs in nixpkgs/master (10b7b6a) as well as the 24.05 release. (shown above)
Notify maintainers
@sternenseemann @maralorn
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
 - system: `"aarch64-darwin"`
 - host os: `Darwin 23.5.0, macOS 14.5`
 - multi-user?: `yes`
 - sandbox: `no`
 - version: `nix-env (Nix) 2.19.1`
 - channels(root): `"nixpkgs"`
 - nixpkgs: `/Users/justin.bailey/.nix-defexpr/channels/nixpkgs`

Will be resolved by #315167.


添加依赖后，项目运行不起来

我创建了一个新项目，添加依赖后运行不起来，移除依赖后项目可以运行起来了
卡在了：Running Gradle task 'assembleDebug'...这里就不动了
flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 1.22.4, on Linux, locale zh_CN.UTF-8)
[✓] Android toolchain - develop for Android devices (Android SDK version
30.0.0-rc4)
[✓] Android Studio (version 4.0)
[✓] VS Code (version 1.51.1)
[✓] Connected device (1 available)
• No issues found!

Since this Flutter SDK plugin needs to rely on the native SDK, it needs to be downloaded from JCenter on Android. Please check if you can access JCenter normally, or try to use mirror source.


Integration pulls through all entities but no camera.

Hi,
I am running homeassistant though docker, I have set up the eufy security ws, and connected both.
The integration pulls back sensors, configuration and diagnostic entities but not camera.XXXX entity.
I have a wired 2k doorbell, 3 2c's and a pan and tilt, and no camera.entity pulls through
The cameras are shared to this account with full admin credentials as well as push notifications.  I am at a bit of a loss.
I have unshared and reshared.
removed and readded the integration
recreated the ws docker
But still the camera entities dont come through.
Any ideas why it would miss just those ones?

elwing00, you life saver, there they are.  thank you very much.
one more thing though,  what card do i need to stream them from HA?

I haven't figured that out yet :) - I just moved from the old MQTT add-on to a full docker install and used the "new" method.  It's been slow going because of stuff going on in my personal life, but I plan on trying to figure it out more next week.

So as far as i can tell, you insall RTSP simple server.
Then in HACS install WEBRTC Camera
Then install the WEBRTC Camera integration.
Then from here I am using the camera's RTC link from the app not the EUFY integration, not sure if that is right or wrong but it works.

I can't be positive because I don't have a eufy camera (only device is a T8500 that I just installed today), but I'm pretty sure this will fix your issue as well:
https://github.com/fuatakgun/eufy_security/issues/322#issuecomment-1126903959


support rbd-nbd

fix #26

@gman0 can you take a look? thanks

Cool, please also update the rbd docs regarding both the storageclass param and the env variable

added


Add some additional linters

Both of these catch bugs, not style

Curious if you accept my suggestion, will it still revoke the approval?

Yes @mattrandallbecker - it does. Please to do a re-approve?


rename hub-of-hubs to multicluster-globalhub.

follow up PR for https://github.com/stolostron/multicluster-globalhub/pull/131
rename hub-of-hubs to multicluster-globalhub
There are still some need to be renamed, eg.
hub-of-hubs.open-cluster-management.io/managed-by-hoh

and
hub-of-hubs.open-cluster-management.io/used-by: hub-of-hubs

but these remaining ones need to be carefully handled, because console needs the old ones.
Signed-off-by: morvencao<EMAIL_ADDRESS>

/retest

Need to fix 2 code smell.

code smell fixed.

/lgtm

/test test-e2e

/test test-e2e

try to fix e2e in https://github.com/stolostron/multicluster-globalhub/pull/138


Consistent naming

For https://github.com/taylorfinnell/awscr-s3/pull/107

Closing, as it is obsolete now (see https://github.com/taylorfinnell/awscr-s3/pull/107)


probable bug in gui.py

I'm not at all familiar with python so its possible this isnt a bug but rather is an issue on my end.
I'm running this in a conda environment on linux mint 20. When I launch gui.py it gives an index error on line 220. I looked at the code in gui.py and found that when it runs default_value=input_devices[sd.default.device[0]] sd.default.device[0] is outside the range of input_devices. I replaced that line and the similar default output device line with default_value=input_devices[0] and default_value=output_devices[0] respectively and then it launched successfully.

It seems that sounddevice fail to get the default device of your Mint.
That's the first case I have met so you may fix the code yourself. If this issue occurs more times I would remove the function of searching for default devices.

I'm getting this error as well on Arch. I think its a pytorch issue but I'm not super familiar with this all.


Windows 10 Universal App Platform error

HI, I'm using 1.2.4 on UAP, but when I get/save anything with lex.db it will throw 'System.InvalidOperationException' in Lex.Db.dll. When I check the LocalState folder of the app, the lex.db folder has nothing inside.

Got the source and found the problem: Additional information: The expression 'Equal' is not supported for nullable type 'System.Nullable`1[System.DateTimeOffset]'

Got the same with System.Nullable'1[System.Guid]

Confirming this is fixed with the latest stable package (1.2.5)

Looks like CLR works differently with nullable type under UWP. Had to implement compare with null operation using alternative approach.


Images not working anymore

There seems to be an issue with the image URLs for posts returning as empty strings. This is affecting the functionality of fetching and displaying images in our application.
Example:
For reference, check the API endpoint at https://mediumpostsapi.vercel.app/api/davidfernandodamata21. Here's an example response where the image URL is missing:
{
    "title":"Como fazer dark theme com JavaScript puro",
    "date":"2021-03-07",
    "link":"https://medium.com/david-fernando/como-dark-theme-com-javascript-puro-fc277377447c?source=rss-e1120fb0abef------2",
    "image":"",
    "description":"Um pequeno tutorial de como fazer dark theme com HTML, CSS e JavaScript puro.",
    "tags":["dark-mode","js-tutorial","javascript","tutorial"]
}

Problem solved! Thanks for alerting me!

Problem solved! Thanks for alerting me!


Fix the proof of mt_flush_to_ by using rv_flush_inplace

Unrelated proof failure in P256.Lemmas... I'll try to bump an rlimit at some point unless @polubelova or @Frosne want to take a look at the failing proof:
[STDERR]/home/everest/hacl-star/specs/ecdsap256sha2/Spec.P256.Lemmas.fst(467,2-470,35): (Error 16) Unknown assertion failed

@aseemr FYI you may want to merge this into your branch to get a green

Thanks @joonwonc and @Frosne!

Awesome, thanks!


Updated for missing and malformed XML Comments

I only updated missing XML comments so there would be less compile warnings. In some cases I changed a /* Comment */ type comment to a
///<summary
/// Comment
///
moving the comment into the summary

Hey Nick, I love the drive to lower the compile warnings count, however, some of these comments are empty. Can we fill them in with something descriptive?  In some cases you may be able to steal the comments from the base class/interface. (ReSharper has a nifty copy comments from base class quick fix)

Michael,
I used ReSharper and the fill in from base class and agree it  is nifty.  Resharper is also helpful at finding the uncommented xml.  Some of the s I left empty because they were obvious, like OnData or a property that has a very descriptive name.  However some others, I did not know what they do.  I realize that the /// xml comments help with generating documentation and telesence.  So that it is important to get them all in.
I also had some newbie issues with getting the pull request, origin and upstream to work on my fork.  It took me three times through the process to get it, but I think I have it now. I would prefer that you merge my PR so you can compile tomorrow.  None of my changes will affect the build.  Then I will then sync my fork with the upstream and work from there.  I am scared of losing the changes I have made after having gone through the process 3 times.

Michael,
I have filled in the empty comments on my local repo and synced it with my github repo.  I am unsure about how to send the changes.  That is I do not know how to update the current PR.  If you merge, I can sync with the upstream and do another PR for the changes tomorrow.
Nick

Nick it looks like you've added an extra commit to this pull request unintentionally (65eabca). When submitted pull requests it points to your entire branch. In this case, your master branch, so as you continued developing with the WMA indicator and tests, they inadvertently got added to this pull request. I'd recommend some light reading on git branches.  Ideally, any work performed that you'd like to contribute back to the open source project would be done on its own branch so that it doesn't interfere with your personal work.

Michael,
Will do.  Unintentionally is right. It is just a newbie error, and I apologize.  I just cannot seem to figure out how to do personal work and not affect the upstream until I get all my bugs ironed out and do an official pull request.
Since you have another pull request to fix the warnings, it might be just as well to kill this pull request.  Do you have a write up on your contributor's work flow?

I'll write up a contributor workflow today, outlining the preferred steps to take.  I'm going to close this one as @ammachado's PR covers many of the same issues anyway. As always, we appreciate the contributions!  Check back soon for the contributor's workflow to be located in the root of the QuantConnect/Lean repository.

Nicholas,
I've gone ahead and committed a first stab at a contributor's workflow.
https://github.com/QuantConnect/Lean/blob/master/CONTRIBUTING.md
I think the key takeaway here is to work on branches.  They can be scary at
first but at really nice what you get the hang of them.  They help to keep
work separated.
Check it out and let me know what you think.  I gave it the quick dry run.
Let me know if you need any help.
-Michael
On Mon, Apr 27, 2015 at 11:03 PM, Nicholas Stein<EMAIL_ADDRESS>wrote:

Michael,
Will do. Unintentionally is right. It is just a newbie error, and I
apologize. I just cannot seem to figure out how to do personal work and not
affect the upstream until I get all my bugs ironed out and do an official
pull request.
Since you have another pull request to fix the warnings, it might be just
as well to kill this pull request. Do you have a write up on your
contributor's work flow?
—
Reply to this email directly or view it on GitHub
https://github.com/QuantConnect/Lean/pull/71#issuecomment-96893588.

Michael,
I really appreciate your doing that write up. I just took a quick peek and it looks pretty good.  I will start from scratch tomorrow and add my WMA Indicator code and let you know how it went tomorrow.
-Nick


Data Report (Preliminary) and Automated Testing

Summarize project's objective, data sources, data processing, and results of data pipeline in a report

data-report.pdf has been submitted.

system-test.py created as testing environment using mock data. test.sh also added into the project folder


Move tfvars into examples

Kindly consider transferring the terraform.tfvars file into the example folder, or you may opt to remove it completely. This is due to the variable setting mechanism in Terraform. The values in the terraform.tfvars file will supersede those in the module variables. Implementing this action helps to avoid any potential conflicts or undesired behavior.

TKS , I'll remove it completely.


Calling an array function on a variable that contains a string causes a lock up / screen corruption

Godot version
4.0.2 - Stable
System information
Linux Mint 20.3 cinnimon
Issue description
While trying to set a variable to act as an array to call, I mistakenly set the variable as a string instead. When I attempted to run the scene it initially throws the error "cannot call .pop_back() on type string." and then lags, then the screen display corrupts and requires a hard reboot.
Steps to reproduce
Just running the scene should it.
Minimal reproduction project
Array Crash.zip

Does the crash happen all the time?
Can't reproduce.

It happened 3x in a row for me initially. I just went to re-test, but now I
can't get reproduce either.
On Mon, Apr 17, 2023, 7:48 PM Tomek @.***> wrote:

Does the crash happen all the time?
Can't reproduce.
—
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/76188#issuecomment-1512244684,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASMLYPY7MSWPFUSKS4N6HULXBXJGJANCNFSM6AAAAAAXB2O6PU
.
You are receiving this because you authored the thread.Message ID:
@.***>


Python Server info is under Python Client

Describe the issue
https://docs.dapr.io/developing-applications/sdks/python/python-client/#publish--subscribe-to-messages lists Server information under Client making it unclear what is required in the code. Personally, I believe that DaprClient and DaprServer should be entirely isolated to make it clear.
URL of the docs
https://docs.dapr.io/developing-applications/sdks/python/python-client/#publish--subscribe-to-messages
Expected content
Should be split up in Client and Server
Additional context
More specifically, examples using the below should be under Client
from dapr.clients import DaprClient

While examples using the below should be under Server
from dapr.ext.grpc import App

This is since Python does not support HTTP Server bindings for Bindings, PubSub and others. HTTP support in Python is mainly used for Actor support AFAIK

Hi @XavierGeerinck -
I agree as-is this is wrong, because we have a client page and we show server content, namely subscribing to pubsub in Flask or FastAPI.
My belief is that it's a good thing to keep client and server content together.  I also believe that users commonly think first about their scenario, in their language, and then want guidance how to use it and what implementation details matter.  I prefer that we do not over-index on clients and servers.  That to me is an implementation detail that can come later in understanding.
scenario -> my language (and my server stack, e.g. Flask/FastAPI/Express) -> code and package instructions
Using pub-sub as an example I would expect:
'Pub-sub -> Python & Flask -> "pip install __" and copy/paste __ code
To go down this road I propose we refactor a bit and basically flatten the SDK pages:

Start with a language SDK, e.g. Python or Javascript
Begin the doc with install instructions, carefully pointing out Client, Server(s), and Actors modules
Go scenario/API by scenario/API and show the snippets.
include a small picture like we have in core docs showing the client app, server app and sidecars so people have a mental model of the concepts
For client snippets, be really clear you need Dapr.Client.  For server snippets be really clear you need Dapr. -- keeping in mind there is not one generic server but rather a server "extension" for each framework stack, like Flask or FastAPI.
At the end have an appendix perhaps of what SDKs have what APIs in each.

The benefits of this are:

much faster path and less clicks from scenario to code, which I belief is key to user success
clarity about client and server is still intact
alignment with quickstarts which are also scenario->code
alignment with building blocks which are scenario->code

Pointing out some outdated information here @XavierGeerinck:
We have convenience PubSub route registration and mapping in FastAPI and Flask via a decorator, same as in the gRPC library.
Additionally, the input binding decorator concept does not apply to gRPC. For input bindings routes are not defined in the application code. However, gRPC does not support custom routes - so the built in gRPC server must redirect a particular event to the correctly registered method. This concept does not apply to HTTP where every input binding sends a request to a fixed route - unless the binding yaml contains a particular route metadata. There is no way to control the route from Fast API or Flask itself.
Users wishing to receive input binding events via HTTP are advised to manually listen to OPTIONS and POST requests on these endpoints.
However, use of gRPC is the preferred method in the Python SDK.
I agree with @paulyuk that keeping things grouped by framework makes the most sense. gRPC (which should be the default if you don't know what to pick), FastAPI and Flask.
I don't have the bandwidth to restructure these docs or examples, but PRs are welcome.


have TableWriter handle Maps correctly:

append name of map to each column of it's children
e.g. reco['hillas1'] -> hillas1_length, hillas1_width, (see the similar functionality for adding container-wide tags)
This will make it much easier to include multiple parameterizations, or multiple image cleanings, reconstructions, etc, in the output tables.

Is this still an issue now that this is essentially solved using the DataWriter?

I think we can close it - the current implemnetation is to write separate tables for each item in the Map, and when reading of course the user can join them using astopy.table.hstack which has an option to append a prefix to common columns.


Adder timer function and option for filename

Developed a small timer function, which will run after running the script to check the length during recording.
Added -o option which takes the filename as an argument, to save the file with the name you want
Included DIR entry in configuration to change the path, where the file will be saved.

@belden
I request you to complete this please.

@Dark-Kernel I wish that I could! I am not a contributor on this project - just another programmer out in the wild who also has some open PRs for this project.
I've tried to help this PR along to make sure there aren't any broken backwards compatibility issues, but it's up to @cytopia to ultimately review and approve/reject this work.
@cytopia can you have a look at this PR please?


build-scaleway-armv7-ubuntu1604-* offline

Both systems currently offline. Attempts to reboot and run through multiple poweroff/poweron cycles through the scaleway web UI have failed. I have raised a ticked with scaleway in order to try and get a resolution.

There was a patch being deployed on Scaleway's C1 systems. -2 is now back but -1 is still experiencing bad ping responses (intermittent) ... I can ping it on it's private IP address but that doesn't help our connections.
Attempting subsequent power cycles

Scaleway ticker ref: #GGVQ-4123-UOJY

Both now up and running, and jenkins has connected back into them


[.NET] and [Python] Cross-lang CI test for distributed runtime

Functionalities to test:

Registering agent type
Add subscription
Events
RPCs

Scenarios to test:

Python Host, .NET workers
.NET host and Python workers

Yes because otherwise there is a gap in the design

Yes because otherwise there is a gap in the design

not necessarily - we may always have features that we support in .NET that we don't in python. I suppose from an API standpoint it is fine.....


Cannot install nest-commander with nestjs v9

Is there an existing issue for this?

[X] I have searched the existing issues

Current behavior
Currently when you have nestjs v9+ npm throughs dependency error, because latest nest-commander v2.5.0 has dependency on nestjs v8. Installation with --legacy-peer-deps fixes this, because nest-commander properly work with nestjs v9, but it's a hack I would like to avoid.
Minimum reproduction code
npm i nest-commander@latest @nestjs/common@latest @nestjs/core@latest

This will through an error
npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! While resolving: undefined@undefined
npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/@nestjs/common
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/nest-commander
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.

Expected behavior
I was expecting this to install everything without problems.
Package

[X] nest-commander
[ ] nest-commander-schematics
[ ] nest-commander-testing

Package version
2.5.0
Node.js version
16.15.1
In which operating systems have you tested?

[X] macOS
[ ] Windows
[ ] Linux

Other
No response

Thank you! Waiting with anticipation for the new version <3

Thank you very much. I appreciate your work.


Handle RegExp types in Debug sidebar

Fixes #3245

Coverage decreased (-0.009%) to 67.361% when pulling b6570a16b861b7436587d93c5621602f13698830 on regex-debug into 3e0f080ea7511bbf242a079d3f504e266c6d04e1 on master.


Include singularity avoidance option in header

We use the singularity avoidance option most of the time, therefore it is needed to include the following statement in the LS-file's header:
/APPL

AUTO_SINGULARITY_HEADER;
  ENABLE_SINGULARITY_AVOIDANCE   : TRUE;

This should be between the DEFAULT_GROUP and the /MN line. I think it would be useful if we could set or unset this option from the tpp file.
I have solved it for now by adding the lines manually in the bin\tpp file, so there is no need for hurry.

I'm unfamiliar with this. Documentation I found was sparse on the topic. I attempted a solution in 9a302e5, but you will have to test it out as I don't have the controller options. I'm formatting these as hashes, but currently they must strictly adhere to :TP_APPLICATION_TYPE, :TP_APPLICATION_MEMBER tokens found in token.rb. If you have more to add, add them there and it should work. I put in spaces before the attributes like you have formatted above, and true/false/no/yes are left lowercase if input that way. Tell me if that is a problem.
this would be your tp+:
AUTO_SINGULARITY_HEADER = {
  ENABLE_SINGULARITY_AVOIDANCE : true
}

documentation referenced this:
/APPL
PAINT_PROCESS ;
DEFAULT_USER_FRAME : 1 ;
DEFAULT_TOOL_FRAME : 1 ;
START_DELAY : 0 ;
TRACKING_PROCESS : NO ;

but I am unsure if the items below PAINT_PROCESS strictly belong to it. In any event the TP+ for that currently is:
PAINT_PROCESS = {
  DEFAULT_USER_FRAME : 1,
  DEFAULT_TOOL_FRAME : 1,
  START_DELAY        : 0,
  TRACKING_PROCESS   : no
}

Hey Dick, How did you make out with this? Does it work for you?

Hi Matt,
I apologize for the late response. I wasn't able to test the solution due to holidays and some other assignments until today.
The solution works perfectly! Thanks for helping.
Kind regards,
Dick

Van: Matt Dewar @.>
Verzonden: dinsdag 17 augustus 2021 01:11
Aan: kobbled/tp_plus @.>
CC: dickvand @.>; Mention @.>
Onderwerp: Re: [kobbled/tp_plus] Include singularity avoidance option in header (#15)
Closed #15https://github.com/kobbled/tp_plus/issues/15.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/kobbled/tp_plus/issues/15#event-5165799554, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AMVOLPIPZ5E5MBJAYCKKLYLT5GLIVANCNFSM5BDVPWQQ.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email.

Hi Dick,
No problem, I figured that might have been the case. I hope you had a good vacation.
Thank you for following up,
Matt


[THORN-2440] update to MicroProfile Fault Tolerance 2.0.1

[x] Have you followed the guidelines in our Contributing document?
[x] Have you created a JIRA and used it in the commit message?
[x] Have you checked to ensure there aren't other open Pull Requests for the same issue?
[ ] Have you built the project locally prior to submission with mvn clean install?

Gonna do that together with Metrics in #1300, so closing this PR.


safer .gitignore patterns from gitignore.io

Use patterns from community-maintained gitignore.io database, rather than hand-rolled patterns, to reduce false negatives and false positives when working with git and various junk files.

Thanks :)


on eris cli 0.12.0-rc2 --api is deprecated

This flag is by default on; eris-db still listens to ERISDB_API=true or has a new  counter flag --disable-rpc
✔ ~/eris/eris-contracts.js/test [master|⚑ 1]
16:31 $ mocha integration/test/

  strings are encoded and decoded properly
Error: unknown flag: --api
Usage:
  eris chains new NAME [flags]

(when removeing --api locally) against eris and e-db, e-cm 0.12.0-rc2
18:05 $ mocha ./test/integration/test/create_and_call

  TestCreateAndCall
DEPRECATED: Do not pass a private key (string) to DevPipe; use the accountData or accountData[] forms.
Creating. This should take about 15 seconds.
[JSON RPC 2.0 Error] Message Id: 1, (INTERNAL_ERROR) - Error broadcasting transaction: invalid character 'x' in literal true (expecting 'r')
[Error: Error broadcasting transaction: invalid character 'x' in literal true (expecting 'r')]

Thank you.  I'll remove the flag when 0.12 is released.


[feature] Integrate /remind into bot capabilities

Because there are certain requirements around how rota can be used with remind, can remind capabilities be baked into rota as a rota command?

This is something that can't really be done easily because reminders cannot be set by bots.


Priority of Policies

Somewhere should be stated how the priority of policies is created. If it is the same as in Transport Rules, so the higher (0) apply first and others are also evaluated (1, 2, 3, ...) or they are just processes in order and the higher number win.

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

ID: 1541da6f-af9a-05d5-fa34-b1a5df61424f
Version Independent ID: 88e111ed-67f7-41bc-d823-863ee7ea2a2e
Content: Set up Office 365 ATP Safe Links policies - Office 365
Content Source: microsoft-365/security/office-365-security/set-up-atp-safe-links-policies.md
Service: o365-seccomp
GitHub Login: @MSFTTracyP
Microsoft Alias: tracyp

@Kazzan higher priority policies will get applied first. I will create a PR and add a note with this info, that's for the heads-up.

Thanks @VLG17, also please note there if only first (higher priority) apply or processing continue and eventually all policies can be applied.
Like example. Policy Priority 0 - Dynamic Delivery Safe Links Scan for all members of contoso.com, Policy Priority 1 - Disable Safe Links for<EMAIL_ADDRESS>Which policy will get apply to<EMAIL_ADDRESS>Should<EMAIL_ADDRESS>get excepted from Policy with Priority 0 or only Priority 1 will apply for him? There is no button "Stop processing more rules" like in Transport Rules, so this can lead to such questions.
Beside that the "Down" Button does not work now in EAC, you can only move Policy "Up".

@Kazzan If a user has 2 policies applied to him, only the policy with higher priority will get applied. In your example, for<EMAIL_ADDRESS>only the "Priority 1 - Disable Safe Links for<EMAIL_ADDRESS>will get applied to him.
For the button issue, i recommend providing this feedback via UserVoice - that's the best way of getting Product Group attention and having this fixed.

@VLG17. Really? Because Priority 0 on other things really mean higher Priority from 1.
For example Antispam priority explain: "You can change the order in which your custom policies run by changing the priority of each custom policy; however, only the highest priority (i.e. number closest to 0) policy will apply if multiple policies meet the criteria set."
In Transport Rules docs explain: "Indicates the order that the rules are applied to messages. The default priority is based on when the rule is created (older rules have a higher priority than newer rules, and higher priority rules are processed before lower priority rules)."
Just asking.

@Kazzan you are correct, it was my mistake with your example. Higher priority means closer to "0", thanks for poiting that out. I have created a note with this. You should see it in the article after it gets approved.

@Kazzan - From our understanding, the issue has been resolved based on this merged commit https://github.com/MicrosoftDocs/microsoft-365-docs/commit/06833175095339a3ab715eb0150d43ec98369349.
Thank you for your contribution to make the docs better! Much appreciated!

I think having an additional note to illustrate the the highest priority is 0 would be useful.
For a newcomer to the technology the idea that the highest priority is actually the lowest value is a little counter intuitive and confusing.


Improve module labeling

For plots, would be nice to include the module label, e.g. 'SIR: New infections'. Would also be nice to have a property that could get either the label or the name, to avoid the getattr(self, 'label', self.name) code.

Closed by #581


Added missing comma in docs example

Description
Added missing comma in docs example
What's included?

missing comma

Test Steps

[ ] read doc example

General Tests for Every PR

[x] npm run serve:prod still works.
[x] npm run tslint passes.
[x] npm run stylelint passes.
[x] npm test passes and code coverage is not lower.
[x] npm run build:lib still works.

Screenshots or link to StackBlitz/Plunker

Thanks for reopening it!

No problem, thank you for the merging 🤓


Add usage instructions for devenv in README.md

This PR adds devenv usage instructions.
Closes #34
@shazow

That's great, thank you!


Default Value for Enum

Hi, I would like to use the default feature for enums that is described here : https://avro.apache.org/docs/current/spec.html#Enums.
I tried to use the AvroDefault annotation, giving it the String value of an enum value, but I get a NumberFormatException when I try that.
I'd like to use this avro feature so that schemas that use enums can be backward compatible. It seems the only way to achieve this is to give the field a default value that is one of the enum values in case a reader is using an older version that does not know about added values for that enum.
I'm pretty new to avro4k, so maybe avro4k supports this feature in a way I don't know about.
Here is what I tried:
@Serializable
enum class IngredientType { VEGGIE, MEAT, }

@Serializable
data class Ingredient(val name: String, @AvroDefault("MEAT") val type: IngredientType,
    val sugar: Double, val fat: Double,)

@Serializable
data class Pizza(val name: String, val ingredients: List<Ingredient>, val vegetarian: Boolean,
    val kcals: Int,)

fun main() {
  val schema = Avro.default.schema(Pizza.serializer())
  println(schema.toString(true))
}

This is what I get:
Exception in thread "main" java.lang.NumberFormatException: Character M is neither a decimal digit number, decimal point, nor "e" notation exponential mark.
	at java.base/java.math.BigDecimal.<init>(BigDecimal.java:518)
	at java.base/java.math.BigDecimal.<init>(BigDecimal.java:401)
	at java.base/java.math.BigDecimal.<init>(BigDecimal.java:834)
	at com.sksamuel.avro4k.schema.ClassSchemaFor.convertToAvroDefault(ClassSchemaFor.kt:135)
	at com.sksamuel.avro4k.schema.ClassSchemaFor.buildField(ClassSchemaFor.kt:114)
	at com.sksamuel.avro4k.schema.ClassSchemaFor.dataClassSchema(ClassSchemaFor.kt:55)
	at com.sksamuel.avro4k.schema.ClassSchemaFor.schema(ClassSchemaFor.kt:43)
	at com.sksamuel.avro4k.schema.ListSchemaFor.schema(SchemaFor.kt:88)
	at com.sksamuel.avro4k.schema.ClassSchemaFor.buildField(ClassSchemaFor.kt:76)
	at com.sksamuel.avro4k.schema.ClassSchemaFor.dataClassSchema(ClassSchemaFor.kt:55)
	at com.sksamuel.avro4k.schema.ClassSchemaFor.schema(ClassSchemaFor.kt:43)
	at com.sksamuel.avro4k.Avro.schema(Avro.kt:239)
	at co.da.avro.DomainKt.main(Domain.kt:18)
	at co.da.avro.DomainKt.main(Domain.kt)

If this feature is not supported in avro4k, do you plan to support it in the future?

Hi Dan,
I was wondering this myself a bit and opened a small pull request that should resolve this issue. It is a limitation of the current implementation that it tries to resolve enum default values as numbers. But let's if it's okay or the schema structure is a bit different for enum default schemas.
https://github.com/avro-kotlin/avro4k/pull/90

Thanks for raising this issue. I think we have 2 problems in the area of defalt values of enums.
The first one can be seen in your code example. Currently there is a bug that does not interprete the default value for enum types correctly.
The second one is the missing implementation of "fallback defaults" for enums that are not known to the reader (schema evolution, see the long thread of https://issues.apache.org/jira/browse/AVRO-1340 for details).
I will open a new issue for the second problem and will continue tracking the first problem in this issue.


Consul is not retaining configuration between restarts

Consul Docker image by default is runs in DEV mode a doesn't save configuration to file.
# By default you'll get an insecure single-node development server that stores
# everything in RAM, exposes a web UI and HTTP endpoints, and bootstraps itself.
# Don't use this configuration for production.
CMD ["agent", "-dev", "-client", "<IP_ADDRESS>"]

Add the following to the compose files for Consul to override the default so that it saves configuration between restarts.
 command: agent -ui -data-dir /consul/data -client <IP_ADDRESS> -server -bootstrap

I transferred this issue here b/c in the Fuji release we'll have our own wrapped Consul image and we need to be sure and account for this.

I believe all we need to do is just not run consul with -dev

Nope, when you remove -dev it expects other options that I have listed above.

Sorry I just meant the option that is causing consul to not persist configuration is the -dev option, yes I'm aware we need to pass additional args when we remove -dev

We really ought to fix this.


PRO: Taglist shows duplicate value when 'multiple' is set to false

Reproduction
https://formkit.link/2f5a5f4861f32868d591c281f8e65b3a
Describe the bug
When setting multiple to false on the taglist input, the selected value will render both as the option and in the input text field. The expected behavior would be to only show the selected option and clear the input field.
Environment
• OS: Mac OS Ventura
• Browser Chrome
• Version [e.g. 22]

@jerrebm Thanks for this! As we discussed in discord, the multiple prop was not intended to be used for the taglist and was erroneously added to its docs. I've fixed the issue anyways by nullifying the multiple prop even if it is passed. I will open a separate issue for an enhancement to add a max prop to the taglist.

Sweet! I agree, max prop is a better solution :)


JS/TS fatal errors should log TSServer error body

When a fatal error happens communicating with the TSServer, we should log the error message itself, like we do for failed requests
Currently we only log the command name. This makes analyzing fatal errors more difficult as you have to do a join on the data

Giving how long it will be until the next VS Code release and that this is blocking the TS team's work, I'm going to push a fix for this into the November release
/cc @armanio123

The change here
To test this, use the oss build:

Trigger a fatal error on the TS Server. In bufferSyncSupport.ts, find the line that looks like:
case BufferOperationType.Open: openFiles.push(change.args); break;

and change it to push the same args twice:
case BufferOperationType.Open: openFiles.push(change.args, change.args); break;

Restart the oss build and confirm that when you open a js/ts file, you now see the TypeScript server instantly crash.

Now, look in this fatalError function: https://github.com/microsoft/vscode/blob/57455124b52ebdf06d1b24fb8690a8f5a4d6f661/extensions/typescript-language-features/src/typescriptServiceClient.ts#L717

Confirm that when the server crashes, it is called and that the error object has reasonable looking telemetry (especially error.telemetry.stack and error.telemetry.message)


Adding support for PDDL addition and subtraction expressions. Courtesy of @jjzapf

Signed-off-by: Alexander Xydes<EMAIL_ADDRESS>

Thanks!!!!
Merging....


[PM Stability] CreatePortAsync needs proper sequential calls to two downstream services

Context
Port creation workflow is not very stable at this time. A few stability issues have been identified in our E2E testing.
In com.futurewei.alcor.portmanager.service.implement.createPortAsync, when we need more than one async calls to a downstream manager, the calls should be triggered in a proper order (currently it is completely async without control on order).
For example,

When calling into SGM, we need to verify then bind SG. The binding call needs to wait until verification is done (line 98-102). securityGroupManagerProxy::bindSecurityGroup may throw exception if the previous call securityGroupManagerProxy::getSecurityGroup haven't returned.
Similar situation occurs for Private Ip Manager and Route Manager (line 71-75).  routeManagerProxy::getRouteBySubnetId may throw exception when previous call haven't returned.

Based on the today's open source meeting, this shouldn't be a code issue. Instead, it is related to Ignite deployment in K8s env which needs further validation.

Not an issue.


UWP not support Windows-1252 encoding

I test some website with Google Translate and it has encoding is Windows-1252. UWP is not support this encoding so we need have option to set encoding for UWP.
Now I see it will use UTF-8 if website encoding is Windows-1252 but it is not correct in many case.
private static Encoding GetEncoding(String name)
        {
            try
            {
                return Encoding.GetEncoding(name);
            }
            catch
            {
                // We use a catch em all since WP8 does throw a different
                // exception than W*.
                    return Utf8;
            }
        }

I think there won't be a fix for this. AngleSharp can't fix the .NET framework. I know that you want to specify the encoding somewhere, but in such cases the solution is to wrap the responses in a transformed stream (where you transform from some source encoding to a target encoding).
The snippet above works quite alright in 95% of the cases. Of course, there are differences between the wanted encoding (which is not UTF-8) and UTF-8. But this is by definition (otherwise they would both represent the same encoding). So differences are to be expected when the fallback occurs.
Question: What would be the encoding you would set? I don't think you can have an answer to that. Since you are on UWP the desired encoding is obviously not available. So which one will you choose? There are encodings which are more similar to one encoding (e.g., Windows-1252) than others, but overall there won't be a perfect match.
(this is a "no fix" at the moment, but if you can suggest a good solution that has no complexity or performance penalty and works reliably then I would happily try to implement it)


Add Cumulus

Thank you for the contribution. It needed some fixes, so they were made in d015268a40561c5bf46c829cc1ebbff1cab1df66. Your contribution is still included (and still credited to you), with the appropriate modifications. Please feel free to ask about any of the changes.


Exception from bb quickblog watch on new blog

Steps to reproduce:

Create a new directory such as qb-test
Add a bb.edn with the following contents:{:deps {io.github.borkdude/quickblog
        #_"You use the newest SHA here:"
        {:git/sha "8f5898ee911101a96295f59bb5ffc7517757bc8f"}}
 :tasks
 {:requires ([quickblog.cli :as cli])
  :init (def opts {:blog-title "REPL adventures"
                   :blog-description "A blog about blogging quickly"})
  quickblog {:doc "Start blogging quickly! Run `bb quickblog help` for details."
             :task (cli/dispatch opts)}}}

Run bb quickblog watch

You'll see an exception like the following:
----- Error --------------------------------------------------------------------
Type:     java.lang.Exception
Message:  Visiting /home/jmglov/Documents/code/clojure/qb-test/posts failed

Working on this now.

@borkdude This can be closed now.