Selected Quality Udemy Download

I want to download some udemy courses using youtube-dl but when I am trying to download them, some 3 minute videos are like 400MB in size.
I am assuming this is because the videos are being downloaded at the best quality possible. Is there a way to tell youtube-dl to download lower quality videos to save bandwidth?
I am using simple command to download
youtube-dl -u EMAIL URL

https://github.com/rg3/youtube-dl#format-selection


Created gulp tasks to output CSS with vars, and nice JSON files of css vars

This is to help with work on the AMP Start Configurator. This creates gulp tasks that can be run on build to output templates with json file to easily understand which css would need to be applied to them to configure the template.
Gulp tasks takes ~1.9s(~1.66s for post css, and ~187ms for json) for all of our templates which isn't bad 😄
Examples
.css template files with vars:
....
:root{
  --h1:3rem;
  --h2:2rem;
  --h3:1.5rem;
  --h4:1.125rem;
  --h5:.875rem;
  --h6:.75rem;
  --line-height-1:1rem;
  --line-height-2:1.125rem;
  --line-height-3:1.5rem;
  --line-height-4:2rem;
  --letter-spacing:1;
  --caps-letter-spacing:.2em;
  --bold-font-weight:bold;
  --width-1:24rem;
....

.json files representing css vars
{
  "--h1": {
    "value": "3rem"
  },
  "--h2": {
    "value": "2rem"
  },
  "--h3": {
    "value": "1.5rem"
  },
  "--h4": {
    "value": "1.125rem"
  },
....etc....

cc @erwinmombay @camelburrito Made all requested changes 😄

lgtm


fix: actually stream events from the async client

Fixes https://github.com/run-llama/llama_deploy/issues/308
Changes:

Fixed the way httpx is used to consume the StreamResponse from FastAPI
Removed some duplicated code (to be double checked during review)

coverage: 65.868% (+0.3%) from 65.592%
when pulling c165b4583f71e818208b3243c2261fd9c2f6af9f on massi/debug-streaming
into 36adfa3e437339bb6a1a7ea16af736ab83613a65 on main.


Authoring api

Description
This PR is a PoC of a new approach to the authoring API in Bobcat. It is a Bobcat-flavoured version of Screenplay pattern and heavily inspired by Serenity.
Motivation and Context
Too further lower the entry barier into Bobcat, we require to have a concise API that will guide users to use Bobcat functionalities.
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[x] I have read the CONTRIBUTING document.
[x] My code follows the code styleguide of this project.
[x] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.

We've went with a different approach


install steps not clear

ollama run fotiecodes/jarvis runs fine and can chat to jarvis but when calling ollama run jarvis:latest this doesn't work.
$ ollama run jarvis:latest
pulling manifest
Error: pull model manifest: file does not exist

when running this projects code, it fails finding jarvis:latest
...
You said: what time is it
Time requested
11:13 AM
Saying:  {'error': 'Error 404: {"error":"model \'jarvis:latest\' not found, try pulling it first"}'}

which is strange because this worked fine
$ ollama list
NAME                    	ID          	SIZE  	MODIFIED
fotiecodes/jarvis:latest	e784ce3e1255	3.8 GB	57 minutes ago

so it's clearly there, just not finding it.

Hey there @Phizicks thank you for your feedback on this, I can see where the problem is. I am currently working with a locally created model which is hardcoded. Having that added to an env file would be ideal, thanks for pointing this out between.
should have a fix for this by this evening. If I don’t could you please ping me again on here? Might be caught up and forget to push a fix real quick…

I'm actually trying to find how I can use jarvis's voice directly. I have my own which uses lists of regex matches so you can ask something in multiple ways for it to recognise the intent. I also have a working tts for linux but it's not using jarvis voice, yet. so any tips on main parts for it to work would be great.

you can use the default system's tts, at the moment I use elevenlab's tts, with a clone of the Jarvis voice I made. but if you wanna have it working with the local tts you can always update the speech_to_text.py file in the modules file. of course this won't be the case into he future as the goal is to have a homogeneous tts engine that runs fully locally, sounds and feels like Jarvis from the mcu.

@Phizicks I am done with the tweak I mentioned earlier, pushing the fix shortly:)

hey @Phizicks could you please pull and try again, this should work just fine now.
Please let me know:)

I'm still having to modify this code to work on my ubuntu linux. missing opencv module 'opencv-python==<IP_ADDRESS>' which I added to requirements.txt but also I think my elevenlabs module being latest, has changes which don't reflect on elevenlabs website, bit weird.
now I seem to run into issues regarding the call to generate() with error
elevenlabs.core.api_error.ApiError: status_code: 400, body: {'detail': {'status': 'voice_not_found', 'message': 'A voice for the voice_id xFjhlCVIoEAjDeZpAmFe was not found.'}}

which is triggered on play(audio)  in text_to_speech.py
I think this is because of your custom jarvis voice xFjhlCVIoEAjDeZpAmFe so I probably have to change this since I don't have that.

yeah, I just checked and unfortunately I need to connect my strip account to be able to share the clone I did on elevenlabs. For that I need a business address etc in a strip supported country. which I do not have at the moment. apologies.
you can always use other voices:) and I will start working in-depth on the TTS engine when I am done with significant features.

@Phizicks could you manage to get the code running none the less?

Where did you get the Jarvis audio you cloned your's from @FotieMConstant Open voice v2 is out. I already got something that sounds like Jarvis but I think a better audio for the reference would produce better results.

that's amazing @brainiakk, perhaps you could help with the TTS, the objective is to have it work 100% offline. I am currently caught up with model training, dataset collection etc. giving less time to focus on other things... essentially working on making the model more Jarvis like.
regarding the voice dataset, after doing some in-depth scrapping on the web I collided and added the voice dataset to kaggle.
here: https://www.kaggle.com/datasets/fotiemconstant/jarvis-dataset

the code runs but just the TTS is failing.
This code cannot run offline until the voice recognition is changed as it uses google
https://github.com/clevaway/J.A.R.V.I.S/blob/main/modules/speech_to_text.py#L20
I tried most if not all OSS voice recognitions and they don't translate too well, not well enough to be 100% offline.
anyway, I give up on this for now.

@Phizicks Does it work fine with internet connectivity?
also if you noticed I am currently doing my best to integrate whisper.  So everything speech to text works offline, but it might take some time as I have a lot on the table at the moment. Thanks for the feedback by the way!

@Phizicks regarding the TTS, could you provide me with the error log you are getting?

Thanks @FotieMConstant

Listening...
You said: what time is it
Time requested
06:09 PM
Saying:   It is 06:09 PM in my time zone. How may I assist you?
Error during text-to-speech: status_code: 400, body: {'detail': {'status': 'voice_not_found', 'message': 'A voice for the voice_id xFjhlCVIoEAjDeZpAmFe was not found.'}}

I also had to add an exception for the timeouts waiting for instructions
--- a/modules/speech_to_text.py
+++ b/modules/speech_to_text.py
@@ -20,7 +20,9 @@ class SpeechToText:
                 text = self.recognizer.recognize_google(audio)
                 print("You said:", text.lower())
                 return text.lower()
-
+            except sr.WaitTimeoutError:
+                print("listening timeout")
+                return ''
             except sr.UnknownValueError:
                 print("Sorry, could not understand audio.")
                 return "Hey there"

From the above I can see you are having issues with the elevenlabs voice id, could you replace the “ xFjhlCVIoEAjDeZpAmFe” with a voice from the elevenlabs voice library: https://elevenlabs.io/app/voice-library
might need to create an account


Translation typos correction

Some correction of typos for Polish translation

you need to pull changes, you've don't have a new commits in branch with link fixes :)


Add user to group fails

Clicking the add button on this form sends us to this route, which isn't defined:
https://hyphy.demo.hydrainabox.org/admin/groups/1/users/add?locale=en

Replaced by https://github.com/projecthydra-labs/hyku/issues/837


Implement specialization constants

Added support to our local copy of SpirV Reflect (which does not support it).
Pass them on render or compute pipeline creation.
Not implemented in our shaders yet.

Force pushed an amend to cleanup the thirdparty changes and add a patch file so they can be reapplied on updates.

Thanks!

Any plans on putting this to upstream? I'm opening a pr upstream, and working on adding this to the main repo (with some added error checks, planning to add an evaluator that can evaluate some SpecConstantOp), and it seems khronos need committers to sign a license agreement. I was wondering if that's okay.

Also note that the code in godot doesn't support 64 bit constants (they take up two words in instruction).

@shangjiaxuan Thanks for working on upstreaming this. I think you can consider that the Godot patch is under the same license as SPIRV-Reflect, as if it had been PR'ed directly. @reduz can comment to confirm.
You could also add reduz as co-author by amending the commit message with: Co-authored-by: Juan Linietsky<EMAIL_ADDRESS>

I confirm the license.


webapp: Add validation for az webapp config access-restriction to avoid adding duplicates

az feedback auto-generates most of the information requested below, as of CLI version 2.0.62

Describe the bug
We cannot add duplicate IPs from the portal since it checks if a value exists however CLI allows this
To Reproduce

Expected behavior

Environment summary

Additional context

@madsd FYI.

looking in to it.

Read and Write, thank you

Does service validates the request of duplicate IPs?

Erro meu, desculpe o transtorno
Em seg, 13 de jul de 2020 às 23:50, Feiyue Yu<EMAIL_ADDRESS>escreveu:

Does service validates the request of duplicate IPs?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Azure/azure-cli/issues/13035#issuecomment-657934862,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AICO2HDNCPWBBF6OD5A4GCTR3PBWNANCNFSM4MJAUYKQ
.

--
*Prof e Diretor Técnico  *
*Arquiteto de Solução *

Edson Belém*
www.cotiinformatica.com.br, www.blogedsonbelem.com.br
http://www.edsonbelem.com.br/
21 98199-0108
"Veni, Vidi, Vici"*

   Júlio César *

is

Does service validates the request of duplicate IPs?

Yes it validate for duplicate IPs.


[Dashboard] Remove HStack

Problem solved
Short description of the bug fixed or feature added

PR-Codex overview
This PR introduces changes to replace HStack with div using flex classes in various components for better styling consistency.
Detailed summary

Replaced HStack with div using flex classes in multiple components for styling consistency.

The following files were skipped due to too many changes: apps/dashboard/src/components/smart-wallets/SponsorshipPolicies/index.tsx

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#4749  👈
#4748
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @kien-ngo and the rest of your teammates on  Graphite


Improve person names autocorrection

Currently

The casing of the name is not validated.
Spacing between words is not trimmed.

Proposed Fix

The name should be in Proper Case, where the first character of each word is capitalized and the rest is not.
Addtionally, spaces in between each word can be trimmed to just a single space.

person, name class chnage implementation of equals, name to upper case

display in proper case, trim also

@saiutkarsh33 What's the status with this PR?


Vet clinic database: create animals table

Hi there, 👋
In this branch, I used a relational database to create the initial data structure for a vet clinic. I created a table to store animals' information, insert some data into it, and query it.
Here are the Project Requirements:

Use this template to generate your repo.
Create a database named vet_clinic.
Create a table named animals with the following columns:

id: integer
name: string
date_of_birth: date
escape_attempts: integer
neutered: boolean
weight_kg: decimal

Copy the SQL you used in the previous point into the schema.sql file in the generated repository.
Insert the following data:

Animal: His name is Agumon. He was born on Feb 3rd, 2020, and currently weighs 10.23kg. He was neutered and he has never tried to escape.
Animal: Her name is Gabumon. She was born on Nov 15th, 2018, and currently weighs 8kg. She is neutered and she has tried to escape 2 times.
Animal: His name is Pikachu. He was born on Jan 7th, 2021, and currently weighs 15.04kg. He was not neutered and he has tried to escape once.
Animal: Her name is Devimon. She was born on May 12th, 2017, and currently weighs 11kg. She is neutered and she has tried to escape 5 times.

Copy the SQL you used in the previous point into the data.sql file in the generated repository.

Write queries for the following quests:

Find all animals whose name ends in "mon".
List the name of all animals born between 2016 and 2019.
List the name of all animals that are neutered and have less than 3 escape attempts.
List the date of birth of all animals named either "Agumon" or "Pikachu".
List name and escape attempts of animals that weigh more than 10.5kg
.
Find all animals that are neutered.
Find all animals not named Gabumon.
Find all animals with a weight between 10.4kg and 17.3kg (including the animals with the weights that equals precisely 10.4kg or 17.3kg)
.

Take a screenshot of the results of your queries

Copy the SQL you used in the previous point into the queries.sql file in the generated repository.
Remember to add the screenshot with the results of your queries to your Pull Request description.

Thanks @Ismailco !

Hi there, 👋
Here is the updated screenshot of the SQL vet_clinic database.
![screenshot vet clinic](https://github.com/oovillagran/Vet_Clinic/assets/122847613/b6dd93be-d736-4643-afd0-a16eac6dc98b)


Constrained load on MeshCoord bypasses callback

🐛 Bug Report
When applying a constraint on a MeshCoord and a callback which checks the loaded cube has a mesh, the callback is not applied on the returned cube.
How To Reproduce
Steps to reproduce the behaviour:

Define a custom callback function which raises an exception if a cube does not have a mesh.
Attempt to load UGrid data with a constraint on latitude (a MeshCoord) and pass the custom callback.
The cube is loaded, without a mesh. The MeshCoords have been converted to AuxCoords as expected, but no exception was raised.

Expected behaviour
I expected an exception to have been raised by the custom callback function, since the returned cube has no mesh.
Reproducible example
import iris
from iris.experimental.ugrid import PARSE_UGRID_ON_LOAD
import iris.exceptions
from iris import sample_data_path

def ugrid_load(uris, constraints=None):
    """Load a UGrid file. Will not load if the file contains regular data.

    Parameters
    ----------
    uris : Any
        Location of file to load. Must be a NetCDF file.
    constraints: Any | None
        One or more iris constraints. Each constraint can be either a string,
        or an instance of :class:`iris.Constraint`. If the constraint is a string
        it will be used to match against cube.name().

    Returns
    -------
    iris.cube.CubeList
        An iris :class:`iris.cube.CubeList` containing the loaded data.

    Raises
    ------
    iris.exceptions.InvalidCubeError
        If UGrid data cannot be loaded from the file.
    """
    with PARSE_UGRID_ON_LOAD.context():
        ugrid_cubelist = iris.load(uris=uris, constraints=constraints, callback=mesh_check)
    return ugrid_cubelist

def mesh_check(cube, field, filename):
    """Check if the loaded data is UGrid or not.

    A function to be passed to callback in iris.load.

    Parameters
    ----------
    cube : iris.cube.Cube
        Loaded :class:`iris.cube.CubeList` to be checked
    field : Any
        Only present to satisfy the function signature used by callback.
    filename : Any
        Original file from which the cube data was loaded

    Raises
    ------
    iris.exceptions.InvalidCubeError
        If the loaded cube does not contain UGrid data
    """
    if cube.mesh is None:
        raise iris.exceptions.InvalidCubeError(
            f"Attempting to load from file '{filename}' did not return UGrid data."
        )

if __name__ == "__main__":
    latitude_constraint = iris.Constraint(latitude=lambda cell: cell > 0)

    # Try with non-UGrid netcdf first to confirm it works
    non_ugrid_path = sample_data_path("atlantic_profiles.nc")
    try:
        ugrid_load(non_ugrid_path, latitude_constraint)
    except iris.exceptions.InvalidCubeError as exc:
        print(exc)

    # Now try constraining on a MeshCoord on load of a file containing UGrid data
    # Expected to raise an exception since the returned cube has no mesh due to the constraint
    # having converted the MeshCoords to AuxCoords
    mesh_C4_path = sample_data_path("mesh_C4_synthetic_float.nc")
    ugrid_cubelist = ugrid_load(mesh_C4_path, latitude_constraint)
    if len(ugrid_cubelist) != 0:
        print(f"Data successfully loaded from '{mesh_C4_path}' with constraint '{latitude_constraint}'.")
        print(ugrid_cubelist[0])
        print(f"Cube's mesh is: {ugrid_cubelist[0].mesh}")

Output:
Attempting to load from file '.../iris_sample_data/sample_data/atlantic_profiles.nc' did not return UGrid data.
Data successfully loaded from '.../iris_sample_data/sample_data/mesh_C4_synthetic_float.nc' with constraint 'Constraint(coord_values={'latitude': <function <lambda> at 0x7fd980bc0310>})'.
synthetic / (1)                     (-- : 32)
    Auxiliary coordinates:
        latitude                        x
        longitude                       x
    Attributes:
        NCO                         'netCDF Operators version 4.7.5 (Homepage = http://nco.sf.net, Code = h ...'
        history                     'Mon Apr 12 01:44:41 2021: ncap2 -s synthetic=float(synthetic) mesh_C4_synthetic.nc ...'
        nco_openmp_thread_number    1
Cube's mesh is: None

Environment

OS & Version: RHEL 7.9
Iris Version: 3.7.1

I think the problem here is really the logic of what callbacks are for + what order they happen in.
It's complicated because most constraints get applied after the callback, but certain special cases get "shortcut" to apply to the load process before the callback.  ( specifically name constraints to speedup um files loading, and netcdf loading )
See : https://github.com/SciTools/iris/issues/3944

So does this mean that using a callback to validate the data being loaded is a bad idea?  Rather than "validate during load", should we doing an approach of "load then validate"?

So does this mean that using a callback to validate the data being loaded is a bad idea? Rather than "validate during load", should we doing an approach of "load then validate"?

Well I guess it depends what you're trying to achieve.
The load callback sits in the "middle" of the load process, and is there principally to capture metadata changes to a "raw" cube at a point where its context in the original sourcefile is still available, and before raw cubes are merged and constrained.
( Except for the "constraint shortcutting" just described that is 🤔 )
So maybe it isn't really appropriate for this purpose.
What was the original thinking, why you do it this way ?

What was the original thinking, why you do it this way ?

For our work on UG-ANTS, we want to validate that people are loading cubes with meshes for the applications that should be working with UGrid data, and that there is no mesh for applications that should be working with regular grid data.
I'd like the validation to be on load or at worst immediately after load: it gives us a chance to give a clear error message (a message saying "loading wrong type of data" is more useful than a message saying "tried to access a mesh but couldn't find it" - the first is definitely a load problem; the second could be a problem in subsequent processing), and it gives the error message before any expensive or time consuming processing has occurred.
A callback seemed a good way to ensure the load and the validation were "simultaneous" (I think it may have been yourself that suggested a callback?), but it's starting to look like the combination of callback and constraint is cryptic enough that we may want to go "load then validate" rather than "validate during load".

This has been worked on, but should remain open until #6014 can be implemented.

After #6122 is complete: we should investigate how the new 'correct' subsetting of meshes affects this issue. I have added this to the task-list in #6120.


Better document how to support IE8; Document browser support

I suggest stepping through these commits one by one – 595ff06 wraps the README consistently but this makes the overall diff hard to review.

says outdated but url is still broken here https://github.com/alphagov/govuk-frontend/blob/d40d824aec91bc2e80eae291c3a0b489928652d0/README.md#browser-support

I've asked @amyhupe to review this as quite a lot has changed.


Problem with Homebrew and .zshrc

I'm not sure if this is the place to post this, but I'll give it a shot anyway.  If this is not the most suitable location, feel free to close it and remove it.
Every time you upgrade a package with Homebrew, the following line is added to ~/.zshrc
source <(awless completion zsh)
My iTerm was extremely slow and after some digging I opened ~/.zshrc to see if there was something strange in there.
It turns out that that source line was repeated about 50-60 times.  Every time you run brew upgrade, that line is added a couple of times, causing things to slow down.
I know this is probably more related to Homebrew, but perhaps someone here can give some advice on how to solve this.
edit>
Can be closed.  Seems I misinterpreted the following line:

Or add to your ~/.zshrc
$ echo 'source <(awless completion zsh)\n' >> ~/.zshrc

That shouldn't be added to your ~/.zshrc, this is to add it to that file :).

Thanks for reporting.
That was then a documentation issue. We fixed that in the wiki.


Cleanup

A bit a cleanup
What is done:

removed reference to go-kit
An example is now also provided to show how to use this library

removed deprecated producer

Pull Request Test Coverage Report for Build 172

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

Files with Coverage Reduction
New Missed Lines
%

murmur.go
3
86.21%

Totals

Change from base Build 161:
-1.1%

Covered Lines:
242

Relevant Lines:
333

💛  - Coveralls


AIBashBug

Describe the bug
To talk about cognitive AI no doubt it has raised the standards of understanding user in better way. The way it enhances, adapts and interacts with user to get through its analysis for most is one of it plus points.
If we talk about the 5 modules introduced during this challenge I faced difficulty in understanding and interpreting if of the statements I would have proffered if there were video presentations which could have demonstrated the execution of steps all in one so that participants could follow it . this way it would have been more easy.
Specially during Build an AI web app by using Python and Flask unit 6 while creating translator services was time consuming , following the video lecture is lot more easier then following the steps one by one after reading it.
It would be difficult to create the videos presentations for all but it will be easier to use for non-programmers also to follow it after seeing other than debugging the statements

This is duplicated from: duplicate


Rewrite "why cubed?" for users

The current text on why cubed? is written at a very high level. It's very interesting for people who think about different distributed systems, but it's not super clear to new users who just want to know what cubed can do for them.
Perhaps we should move the existing text to e.g. design/motivation, and rewrite this intro text to instead emphasise:

cubed is a drop-in replacement for dask.array
it tells you if your computation would run out of memory
it is designed to be very robust to failures, and be resumable
it doesn't require you to make many (if any) up-front choices about configuration / cluster size
(Optional) IF you want to run in the cloud it is very horizontally scalable and stateless / serverless

Thanks for opening this @TomNicholas!

Perhaps we should move the existing text to e.g. design/motivation

Definitely. It's mainly of historical interest, so should be moved down to the bottom of the nav links (I noticed that DuckDB does this).
The intro text at https://cubed-dev.github.io/cubed/ (and https://github.com/cubed-dev/cubed/blob/main/README.md) needs updating, as you say.

"Zarr for storage" is a bit vague. It's about how Zarr is used for intermediate storage, which is an implementation detail that is probably not a top-level bullet point. Being able to read and write the user's Zarr data is more relevant.
"Multiple serverless runtimes: Python (in-process)" - change to add out-of-core/processes.

Maybe say something like "Multiple runtimes: local (...), serverless (...), cluster (Beam, Dask)"

Add something about accelerators (JAX)

These are just a few ideas. I think it would be really useful to collaborate on re-writing the intro.

Fixed in #622


Create create,read and delete operations

Create, Read and Delete operations has been successfully implemented.
Additional functionality has also been included that makes the create button invalid unless all text box is filled in. This is to prevent null entries. Fixed Issue (#7)

I am getting AxiosError: Request failed with status code 404, not sure what is wrong.


Fix invalid key check

The regex lists a couple of reserved headers, and tries to assert that
the key is fully alphanumeric. However, there are two issues:

The check was anchored, so 'tree ' would not be considered invalid
(while still breaking the git commit header)
The check for invalid characters currently only checks for a single
character. Adding a + doesn't fix this, because even then it would
only match keys that are fully non-alphanumeric.

Sanity check: https://regex101.com/r/bBq8ZW/1

Thanks for the fix! Just one.. let's call it minor issue (and it's really on me because there are no test cases and the double negation is hard to read to begin with); this will effectively allow any alphanumeric string as key, making keys like "commit", "tree", etc. valid.

After being stumped for a second because I was really sure the regex is correct, I noticed that I forgot to escape the \. I pushed a fix!


Attempt to fix closeclick issue in InfoWindow

Propagate google maps InfoWindow's closeclick event to the parent component however, re-open the InfoWindow when the event is received in order to ensure, its state is set solely by the "opened" prop of the InfoWindow component
Do not emit closeclick in the _openInfoWindow() method triggered by the watcher on the opened prop. closeclick should only be triggered, when the user atually clicked close

If someone uses this and wants to help us improve it, please let us know. I'll leave it here some days to see if someone appears. Otherwise I'll accept it.
Thanks again for your help!


Add ViacomCBS to adopters list

Motivation
Y'all and this project are awesome.
Solution
Join the party of companies using the lib.

Welcome!


Trouble Reading from SEL735

I am trying the tinyPDC example on a SEL735 Meter, the program connects to the meter however it hangs after the "Requesting header message from PMU" log. Attached is a successful pcap reading this meter with SEL synchrowave. Any help would be appreciated.
pmu2.zip
2018-01-09 08:57:25,205 INFO [7] - PDC successfully connected to PMU (<IP_ADDRESS>:4722)
2018-01-09 08:57:25,205 INFO [7] - Requesting header message from PMU (<IP_ADDRESS>:4722)

Hi @sstevan , I have come across this issue also.
Similar to  @senormeow my freq data is formatted as floating point.
I believe the spec tells us to treat this as the frequency value, rather than a deviation which is the case for 16bit format.
I think the get_measurements in frame.py needs adjusting.


auth0-cli: 1.1.2 -> 1.4.0

Description of changes
Release notes: https://github.com/auth0/auth0-cli/releases/tag/v1.4.0
Upgrades the auth0 command line utility package to the latest release.
Things done

Built on platform(s)

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

[x] 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

[x] 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
[x] 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.

Result of nixpkgs-review pr 301875 run on x86_64-linux 1

1 package failed to build:

auth0-cli

Result of nixpkgs-review pr 301875 run on x86_64-linux 1
1 package failed to build:
Build fails due to failure in the test suite.
error: builder for '/nix/store/4nfarslfq9irxjzl4ahsxw1sxdsw4k16-auth0-cli-1.4.0.drv' failed with exit code 1;
       last 10 log lines:
       >                                -     (string) (len=19) "usernamePlaceholder": (string) (len=25) "Username or email address",
       >                                  -     (string) (len=17) "wrong-credentials": (string) (len=26) "Wrong username or password",
       >                                   -     (string) (len=23) "wrong-email-credentials": (string) (len=23) "Wrong email or password"
       >                                 +    (string) (len=5) "login": (map[string]interface {}) (len=1) {
       >                                     +     (string) (len=5) "title": (string) (len=33) "Welcome friend, glad to have you!"
       >                                       }
       >                 Test:           TestFetchUniversalLoginBrandingData/it_uses_a_default_branding_theme_if_it_fails_to_fetch_it
       > FAIL
       > FAIL        github.com/auth0/auth0-cli/internal/cli 8.333s

As someone new to Nix, what would you suggest (if anything) that I do for this to be upstreamed?
I have:

reviewed the Go module builder here to see if anything stood out to explain discrepancy locally on my box versus via Nix invocation
downloaded the referenced repo onto my host (outside of my flake environment) and ran the full unit test suite [and the single test that reports failure]

my host is Darwin ARM64 (M2 Max) running MacOS on Sonoma 14.4.1 with Go version 1.22.2
the test passes (both in the full suite run and in isolation)

I intend to rebuild my primary Linux machine in the next 2-3 weeks (time permitting), and should have a reliable Nix environment running on ARM64. Anything else I can do to troubleshoot / help here?

The nix-review output only shows the last part of the issue. It starts:
[...]
Running phase: checkPhase
ok      github.com/auth0/auth0-cli/internal/analytics   0.005s
ok      github.com/auth0/auth0-cli/internal/ansi        0.005s
ok      github.com/auth0/auth0-cli/internal/auth        20.044s
ok      github.com/auth0/auth0-cli/internal/auth/authutil       3.057s
ok      github.com/auth0/auth0-cli/internal/buildinfo   0.003s
Error: failed to deploy action with ID "1221c74c-cfd6-40db-af13-7bc9bb1c38db": 400 Bad Request
Usage:
  deploy [flags]

Examples:
  auth0 actions deploy
[...]
Flags:
  -h, --help              help for show
  -l, --language string   Language of the custom text. (default "en")

--- FAIL: TestFetchUniversalLoginBrandingData (0.01s)
    --- FAIL: TestFetchUniversalLoginBrandingData/it_can_correctly_fetch_universal_login_branding_data (0.00s)
        universal_login_customize_test.go:958:
[...]

At first glance I would says that the test want to perform a "real" web requests. As the tests are passing locally according to your analysis, I guess that this test is not mocked.
The quick solution would be to skip the failing test. The sandbox has no network access.
  preCheck = ''
    [...]
    # Tests requires network access
    substituteInPlace internal/cli/universal_login_customize_test.go \
      --replace-fail "TestFetchUniversalLoginBrandingData" "SkipFetchUniversalLoginBrandingData"
  '';

Rebased off upstream origin with preCheck bypassing the errant tests.

Could you please update the commit message of b36963e, e. g., auth0-cli: disable failing tests?

Could you please update the commit message of b36963e, e. g., auth0-cli: disable failing tests?

Done! Thanks again for all the help. Let me know if you need anything else to merge.


Request: List bands instead of albums in Genres

I would like to see bands instead of the albums when I click in any of the Genres. Is it possible?

Unfortunately no, we can't make drastic changes to functionality of GPM.


Adds initial schema for GeoJSON

This should also have name, identifier, and species fields

This schema is not correct, it's a schema for the  properties object of each feature, but the it should describe a featureCollection containing Polygon features, and describe the properties of each feature.
It would probably be easiest to start with an existing schema for geojson, remove some things, because this doesn't need to describe all geoJson documents, and then add the definition for properties.

schema for geojson: https://github.com/fge/sample-json-schemas/blob/master/geojson/geojson.json


Feature Request: make isHotDeploying configurable

Thanks again for the project.  It's working nicely.
We have a process where we run all our Keycloak database migrations as a standalone process prior to starting up the "real" Keycloak.  While doing so, we were sometimes encountering race conditions that appeared to be related to RealmRoleInitializer running its initialization logic when it wasn't expected.  After some investigation, it seems that our migrator process was being considered a "hot deploy".  Would it be possible to make this explicitly configurable, rather than detecting solely based on presence of a particular component in JNDI?  We currently have a workaround that puts a stub JNDI in place, but would like to be able to remove the workaround at some point.
https://github.com/Captain-P-Goldfish/scim-for-keycloak/blob/master/scim-for-keycloak-server/src/main/java/de/captaingoldfish/scim/sdk/keycloak/provider/RealmRoleInitializer.java#L91

Hi I am going to release the SCIM for keycloak enterprise project soon. This problem is fixed in this version.


Marathon Mode Bug?

I tried out Marathon Mode the other day and rather than taking me to Venom 2 after Area 6 at the end, it went back to Corneria.
I did trigger the Bonus Level after Area 6 though (it was set to random), and wonder if maybe that was why.

I've attached a test patch for you to try. Try this out and see if it fixes the issue when Bonus Level is on / random.
Beta 48.zip


Trim leading and trailing Whitespace and double quotes from test adapter path name

Issue: https://github.com/Microsoft/vstest/issues/1089
Fix: Trim leading and trailing Whitespace and double quotes from test adapter path name.

@dotnet-bot test this Windows_NT / Debug Build

@dotnet-bot test Windows_NT / Debug Build please


bridge: add caller support in contract call

Description
add caller support in contract call
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)

Brief of your solution

Codecov Report

Merging #292 (3a96523) into master (31e1c7c) will decrease coverage by 0.57%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #292      +/-   ##
==========================================
- Coverage   46.44%   45.86%   -0.58%
==========================================
  Files         136      136
  Lines       12174    12174
==========================================
- Hits         5654     5584      -70
- Misses       5349     5422      +73
+ Partials     1171     1168       -3

Impacted Files
Coverage Δ

bcs/contract/native/runtime.go
13.18% <0.00%> (-50.55%)
:arrow_down:

bcs/contract/native/contract_process.go
58.82% <0.00%> (-10.09%)
:arrow_down:

bcs/contract/native/process_manager.go
68.42% <0.00%> (-3.51%)
:arrow_down:

bcs/consensus/xpoa/xpoa.go
54.21% <0.00%> (-2.11%)
:arrow_down:

bcs/network/p2pv1/server.go
57.34% <0.00%> (-1.40%)
:arrow_down:

bcs/consensus/pow/pow.go
41.82% <0.00%> (-1.15%)
:arrow_down:

bcs/consensus/tdpos/schedule.go
41.74% <0.00%> (-0.92%)
:arrow_down:

bcs/consensus/tdpos/tdpos.go
50.72% <0.00%> (+0.48%)
:arrow_up:

Continue to review full report at Codecov.

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

Codecov Report

Merging #292 (3a96523) into master (31e1c7c) will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #292      +/-   ##
==========================================
+ Coverage   46.44%   46.45%   +0.01%
==========================================
  Files         136      136
  Lines       12174    12174
==========================================
+ Hits         5654     5656       +2
+ Misses       5349     5348       -1
+ Partials     1171     1170       -1

Impacted Files
Coverage Δ

bcs/consensus/xpoa/xpoa.go
54.21% <0.00%> (-2.11%)
:arrow_down:

bcs/consensus/tdpos/tdpos.go
50.72% <0.00%> (+0.48%)
:arrow_up:

kernel/consensus/base/driver/chained-bft/smr.go
40.45% <0.00%> (+0.75%)
:arrow_up:

bcs/network/p2pv2/server.go
52.40% <0.00%> (+1.06%)
:arrow_up:

Continue to review full report at Codecov.

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

Codecov Report

Merging #292 (3a96523) into master (31e1c7c) will increase coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #292      +/-   ##
==========================================
+ Coverage   46.44%   46.47%   +0.03%
==========================================
  Files         136      136
  Lines       12174    12174
==========================================
+ Hits         5654     5658       +4
+ Misses       5349     5346       -3
+ Partials     1171     1170       -1

Impacted Files
Coverage Δ

bcs/consensus/xpoa/xpoa.go
54.21% <0.00%> (-2.11%)
:arrow_down:

kernel/consensus/base/driver/chained-bft/smr.go
40.45% <0.00%> (+0.75%)
:arrow_up:

bcs/network/p2pv2/server.go
52.40% <0.00%> (+1.06%)
:arrow_up:

bcs/consensus/tdpos/tdpos.go
51.69% <0.00%> (+1.44%)
:arrow_up:

Continue to review full report at Codecov.

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

Codecov Report

Merging #292 (a86afeb) into master (a47fc2f) will decrease coverage by 0.04%.
The diff coverage is n/a.

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

@@            Coverage Diff             @@
##           master     #292      +/-   ##
==========================================
- Coverage   46.45%   46.41%   -0.05%
==========================================
  Files         136      136
  Lines       12174    12174
==========================================
- Hits         5655     5650       -5
- Misses       5349     5352       +3
- Partials     1170     1172       +2

Impacted Files
Coverage Δ

bcs/network/p2pv1/conn.go
55.10% <0.00%> (-6.13%)
:arrow_down:

bcs/contract/xvm/code_manager.go
63.63% <0.00%> (-2.73%)
:arrow_down:

bcs/consensus/tdpos/tdpos.go
49.27% <0.00%> (+0.96%)
:arrow_up:

bcs/consensus/tdpos/common.go
57.81% <0.00%> (+3.12%)
:arrow_up:

Continue to review full report at Codecov.

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

Codecov Report

Merging #292 (a86afeb) into master (a47fc2f) will not change coverage.
The diff coverage is n/a.

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

@@           Coverage Diff           @@
##           master     #292   +/-   ##
=======================================
  Coverage   46.45%   46.45%
=======================================
  Files         136      136
  Lines       12174    12174
=======================================
  Hits         5655     5655
+ Misses       5349     5348    -1
- Partials     1170     1171    +1

Impacted Files
Coverage Δ

bcs/network/p2pv1/conn.go
55.10% <0.00%> (-6.13%)
:arrow_down:

bcs/consensus/tdpos/tdpos.go
50.24% <0.00%> (+1.93%)
:arrow_up:

bcs/consensus/tdpos/common.go
57.81% <0.00%> (+3.12%)
:arrow_up:

Continue to review full report at Codecov.

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

Codecov Report

Merging #292 (847ea59) into master (396f95b) will decrease coverage by 0.07%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #292      +/-   ##
==========================================
- Coverage   46.45%   46.38%   -0.08%
==========================================
  Files         136      136
  Lines       12199    12199
==========================================
- Hits         5667     5658       -9
- Misses       5358     5366       +8
- Partials     1174     1175       +1

Impacted Files
Coverage Δ

bcs/network/p2pv1/conn.go
55.10% <0.00%> (-6.13%)
:arrow_down:

bcs/contract/xvm/code_manager.go
63.63% <0.00%> (-2.73%)
:arrow_down:

bcs/consensus/tdpos/tdpos.go
49.27% <0.00%> (-0.97%)
:arrow_down:

bcs/network/p2pv2/server.go
52.40% <0.00%> (+1.06%)
:arrow_up:

Continue to review full report at Codecov.

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

Codecov Report

Merging #292 (847ea59) into master (396f95b) will decrease coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #292      +/-   ##
==========================================
- Coverage   46.45%   46.40%   -0.05%
==========================================
  Files         136      136
  Lines       12199    12199
==========================================
- Hits         5667     5661       -6
- Misses       5358     5364       +6
  Partials     1174     1174

Impacted Files
Coverage Δ

bcs/network/p2pv1/conn.go
55.10% <0.00%> (-6.13%)
:arrow_down:

bcs/consensus/tdpos/tdpos.go
49.27% <0.00%> (-0.97%)
:arrow_down:

bcs/network/p2pv2/server.go
52.40% <0.00%> (+1.06%)
:arrow_up:

Continue to review full report at Codecov.

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

Codecov Report

Merging #292 (300b2b4) into master (396f95b) will decrease coverage by 0.09%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #292      +/-   ##
==========================================
- Coverage   46.45%   46.36%   -0.10%
==========================================
  Files         136      136
  Lines       12199    12199
==========================================
- Hits         5667     5656      -11
- Misses       5358     5368      +10
- Partials     1174     1175       +1

Impacted Files
Coverage Δ

bcs/network/p2pv1/conn.go
55.10% <0.00%> (-6.13%)
:arrow_down:

bcs/consensus/tdpos/tdpos.go
49.27% <0.00%> (-0.97%)
:arrow_down:

kernel/consensus/base/driver/chained-bft/smr.go
39.69% <0.00%> (-0.76%)
:arrow_down:

Continue to review full report at Codecov.

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

Codecov Report

Merging #292 (300b2b4) into master (396f95b) will decrease coverage by 0.06%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #292      +/-   ##
==========================================
- Coverage   46.45%   46.38%   -0.07%
==========================================
  Files         136      136
  Lines       12199    12199
==========================================
- Hits         5667     5659       -8
- Misses       5358     5366       +8
  Partials     1174     1174

Impacted Files
Coverage Δ

bcs/network/p2pv1/conn.go
55.10% <0.00%> (-6.13%)
:arrow_down:

bcs/consensus/tdpos/tdpos.go
49.27% <0.00%> (-0.97%)
:arrow_down:

Continue to review full report at Codecov.

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

Codecov Report

Merging #292 (300b2b4) into master (396f95b) will decrease coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #292      +/-   ##
==========================================
- Coverage   46.45%   46.40%   -0.05%
==========================================
  Files         136      136
  Lines       12199    12199
==========================================
- Hits         5667     5661       -6
- Misses       5358     5364       +6
  Partials     1174     1174

Impacted Files
Coverage Δ

bcs/network/p2pv1/conn.go
55.10% <0.00%> (-6.13%)
:arrow_down:

Continue to review full report at Codecov.

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


Audit Logger Plugin now works with pre, routed and after events

I plan on merging this repo back into restify after this PR, and marking it as deprecated.

@DonutEspresso One more time?

Actually, going to leave this open until tomorrow. We should talk this over @yunong.

Merging with sign-off from @DonutEspresso


New version: evernote.evernote version 10.77.3

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] This PR only modifies one (1) manifest
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.6 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Hi @PckgrBot,

This PR's version number 10.77.3 has 3 parameters (sets of numbers between dots - major, minor, etc), which is less than the current manifest's version <IP_ADDRESS>40214093250, which has 4 parameters.

Does this PR's version number exactly match the version reported in the Apps & features Settings page? (Feel free to attach a screenshot.)
(Automated response - build 795)

Hi @PckgrBot,

This PR's version number 10.77.3 has 3 parameters (sets of numbers between dots - major, minor, etc), which is less than the current manifest's version <IP_ADDRESS>40214093250, which has 4 parameters.

Does this PR's version number exactly match the version reported in the Apps & features Settings page? (Feel free to attach a screenshot.)
(Automated response - build 795)

Hello, this

Hi @PckgrBot,

This PR's version number 10.77.3 has 3 parameters (sets of numbers between dots - major, minor, etc), which is less than the current manifest's version <IP_ADDRESS>40214093250, which has 4 parameters.

Does this PR's version number exactly match the version reported in the Apps & features Settings page? (Feel free to attach a screenshot.)
(Automated response - build 795)

Hi Stephen, I have confirmed this is correct


fix: fix readme

fix readme

Codecov Report

Merging #79 (df65a99) into master (ac8f9cc) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #79   +/-   ##
=======================================
  Coverage   85.98%   85.98%
=======================================
  Files          34       34
  Lines        2497     2497
  Branches      683      682    -1
=======================================
  Hits         2147     2147
  Misses        316      316
  Partials       34       34

Continue to review full report at Codecov.

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


changed to harbor-cli

during go build . command, it creates a binary named harbor-cli nor harbor. We should change the name to harbor-cli. Changed name to harbor-cli
Fixes : #150

yeah, you are right.

Can you close this PR too for now?


locomotion in a 3rd party crate

implement locomotion prototype into another crate named xr_utils

basic locomotion was added into the xr utils crate.
closing


Deploying as Helm chart in Openshift: Permission Denied for mkdir

Describe the bug
I am able to launch the jtl-reporter stack locally via Docker Desktop with no issues. I used kompose to convert the docker-compose yml file into Helm charts. The only modification I did after that was to modify the images in each file so that it correct points to the Openshift ImageStreams repository, which I have confirmed to be correct.
When I attempt to do a helm install xxx . --namespace yyy, I noticed that when resources such as fe and be  deploy, they run into a permissions denied issue. The other resources such as migration are also having issues, but they also seem to be related to permissions.
be: Error: EACCES: permission denied, mkdir '/src/be/uploads
fe: nginx: [emerg] mkdir() "/var/cache/nginx/client_temp" failed (13: Permission denied)
Looks like a user permissions issue. How can I fix this?
To Reproduce
Steps to reproduce the behavior:

Use kompose to convert the docker-compose yml into helm charts
Attempt to launch the generated files (after necessary modifications) into Openshift as a Helm release.

Expected behavior
A clear and concise description of what you expected to happen.

Resources should launch without any permissions issue
If certain permissions are required, we should be able to address this via commands in the docker-compose / Helm chart.

Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.
You can get the logs by running: docker logs <service_name>
Did JtlReporter make you happay in any way?
Answering this question is not required, but if you have anything positive to share, please do so here.

Hi @timothyjmtan Sorry for a late reply.
I am not sure I can help much here.  Are you sure that there's no problem on your side, the way your docker is configured/run?


reverse ‘Add Flutter to existing apps’

About "Adding flutter to an existing application," can you move existing application code into a flutter project to implement the Add Flutter feature of an existing application?
Existing applications can be introduced in the form of "git submodule", which should avoid known packaging issues and team development issues.

@mcxinyu
Can you give more details about your scenario which add Flutter to existing apps wiki can't solve your problem?

@kangwang1988 I think there are a variety of options available to solve this problem.  Flutter was developed as a new functional module, but the current access method will destroy the development process of the original project. My colleagues have to follow the installation of flutter development environment and learning related knowledge.

Do we know of a way to destroy the flutterviewcontroller?  I tried SystemNavigator.pop(); and I tried to destroy the presented viewcontroller?
Thanks


extract scope concerns to seperate routes

closes #1713
extracting the overloading the show/index query scopes to route concerns.
NOTE: this may break some existing functionality since by default show / index routes will now return just completed classification for the requesting user and not all classifications the user has access to from a project / that are incomplete.  I don't think it does but I'd like to make sure this api change doesn't break any existing functionality that PFE / custom projects rely on.
new routes:
GET      /api/classifications                         -> default, completed classfications for this user
GET      /api/classifications/incomplete       -> only incomplete for the user
GET      /api/classifications/project             -> all classifications the user has project access to
GET      /api/classifications/gold_standard -> only gold standard for marked workflows
TODO:

[x] Update the api docs...remove the cruft on special for params and outline the collection routes

@brian-c mind having a look at these route changes for classifications? Pretty sure we don't actually use them currently (i.e. look at peoples history / project dumps via the api) but be good to get your thoughts on it.

This looks good other than a possible confusing implied intent here.  If I'm right about the update being allowed because user is the owner, maybe renaming the user helper to owner would clarify it.
I'll wait for @brian-c to chime in before merging though.

Looks like all PFE master currently uses is a POST to /api/classifications and a GET from /api/classifications/gold_standard, so this shouldn't break anything.
Any reason you're implementing these as scopes versus allowing explicit queries from the client? I'm thinking something like /api/classifications?workflow=123&subjects=234&complete=false would be more consistent and predictable on the client's end.

more consistent and predictable on the client's end

With the amount of filtering, scoping, including, etc. that can be done on this endpoint, it gets incredibly nasty for the API.  The alternative to this would be to logically split the endpoints by params.  That would require a pretty significant rewrite though and could still end up somewhat sloppy.  This seems like a good middle ground.
That said, it does break convention with a lot of the other resources.  Kind of on the fence here I guess.

That's fair. I figured most URL queries would translate pretty cleanly into SQL queries, so all the server would have to worry about would be read permissions. Probably naive on my part.

A lot of gets funky when you're dealing with complicated associations -- trying to preload/join/filter multiple tables.  I think this is a fair example of how JSON API can be really counterproductive on the backend.

@parrish, poor choice of words, if they have update access on the project is what it should have said. The user can update the classification if it is incomplete and they own it.

@brian-c agree that its not standard but overloading the index route just got too funky. Basically decided to extract them out, if its causing problems we can revisit.
The main issue is making sure a normal user can access their own data. I think the original implementation was a bit ambitious on 1 route.


Question: Can the region folding start marker and end marker be the same?

In the examples in doc, the start marker and end marker for region folding are all different. Is this a requirement? Say, if I want support folding regions using #==*=#, e.g.,
#========================#
Hshsjshshsjsjsjsjhsyxyxtsnzb
Jsjshzhzhbsbsbsbsbsjjss
#========================#

Is it supported by design? Thanks!

Sorry, the markers must be different.


Style fixes to addon settings and others

Purpose
Attempts to fix the following Trello cards:

Multiple items in project list for add ons: https://trello.com/c/JQY4pCln
Revisions table space : https://trello.com/c/EewV1U0W
Long project names in add on settings : https://trello.com/c/lkZktxce
Empty href on panel toggle:  https://trello.com/c/9KSiq0ap

Pass finished. :point_up:

@sloria buttons changed now.


ArweaveJS worked two weeks ago, but suddenly showing an error now.

I have run the following codes to upload a text file to Arweave storage:
  const arweave = Arweave.init({
    host: 'arweave.net',// Hostname or IP address for a Arweave host
    port: 443,          // Port
    protocol: 'https',  // Network protocol http or https
    timeout: 20000,     // Network request timeouts in milliseconds
    logging: false,     // Enable network request logging
  });

  const key = await arweave.wallets.generate();
  const transaction = await arweave.createTransaction({ data }, key);
  transaction.addTag('Content-Type', 'text/plain');
  await arweave.transactions.sign(transaction, key);

  let uploader = await arweave.transactions.getUploader(transaction);
  log.info(`Uploading files to Arweave...`);
  while (!uploader.isComplete) {
    await uploader.uploadChunk();
    log.info(`${uploader.pctComplete}% complete, ${uploader.uploadedChunks}/${uploader.totalChunks}`);
  }
  log.info(`\nContent added with TxID: ${transaction.id}`);

Got this error:
Error: Unable to upload transaction: 400, Bad Request
    at TransactionUploader.postTransaction (E:\arweave-upload\src\common\lib\transaction-uploader.ts:283:13)
    at processTicksAndRejections (internal/process/task_queues.js:95:5)
    at async TransactionUploader.uploadChunk (E:\arweave-upload\src\common\lib\transaction-uploader.ts:126:7)
    at async saveToArweave (E:\arweave-upload\src\uploads\arweave.ts:38:5)
    at async Command.action (E:\arweave-upload\src\commands\swarm.ts:45:17)

It worked two weeks ago, but suddenly producing that error today.
What happened to arweave?

nothing has changed with the library. it's working as it should.
what has changed recently is that the arweave.net gateway is now returning errors that have been passed to it by the protocol nodes when invalid transactions are posted to them.
you are using a generated wallet which:

contains no AR balance for the storage fee
doesn't exist in the protocol

i suggest you check if your previous transactions are actually stored on the permaweb. you can do this using a block explorer, or by checking the tx status via the endpoint:
https://arweave.net/tx/YOUR-TXID/status
if you need more clarification, please visit the #gateway channel in the Dev Discord server
i'm marking this as closed, thanks!

So I should have a balance to upload data to storage?
Is there any way to upload data for free like on filecoin?

Someone is always paying for you.
There is an initiative for files up to 300kb. You can ask in the Dev server for more info.


docs: Implement freestyle docs for service pricing

Why did the pnpm lockfile change? No dependencies were changed, so the lockfile shouldn't have changed.

I belive the pnpm lockfile change was caused by a new minor version of webpack be released. Webpack 5.94.0 released a few days ago. Do we need to specify which version of webpack we are using? It is currently ^5.91.0 in the test app's package.json

Why did the pnpm lockfile change? No dependencies were changed, so the lockfile shouldn't have changed.

I belive the pnpm lockfile change was caused by a new minor version of webpack be released. Webpack 5.94.0 released a few days ago. Do we need to specify which version of webpack we are using? It is currently ^5.91.0 in the test app's package.json

I guess my question is why did the lockfile itself change? If you're installing dependencies with pnpm install, it shouldn't update the lockfile unless one of the dependencies specified in the package.json is no longer compatible. It shouldn't change if the dependencies in the lockfile still satisfy the requirements of the package.json.


UX: Popup closes when switching windows (e.g. to password manager)

A simple solution is to move signup flow into extension options, like Metamask does

The quickest fix is calling open(location.href) when starting signup flow


Clean task

Create a clean task to delete logs, remove files in apps and dropins dir, etc. "https://github.com/WASdev/ci.ant/issues/26"

After pull request #42, I think the work in this issue is done. Please re open if needed.


Update data-flow-expression-functions.md

@nimoolen : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@kromerm - please confirm that line 28 should read "Calculates a consine inverse value" and not "Calculates an inverse cosine value" . thanks.

@nimoolen : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Looks like a typo to me, fixed it now. Mark, will need approval again it looks like 😊
From: Jak Koke<EMAIL_ADDRESS>Sent: Wednesday, September 11, 2019 2:52 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Nick Moolenijzer<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Update data-flow-expression-functions.md (#38734)
@kromermhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fkromerm&data=02|01|Nick.Moolenijzer%40microsoft.com|f7bf524a24c84d100f4b08d737023dc3|72f988bf86f141af91ab2d7cd011db47|1|0|637038355074069107&sdata=DtJbjMw5ls6hDBfoSly2TToF9uEBtXkzTDBTb%2FhEJo4%3D&reserved=0 - please confirm that line 28 should read "Calculates a consine inverse value" and not "Calculates an inverse cosine value" . thanks.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fpull%2F38734%3Femail_source%3Dnotifications%26email_token%3DAK6TGTCOBXR26WIHWRAMGWLQJFR7DA5CNFSM4IVXIES2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6QAJXQ%23issuecomment-530580702&data=02|01|Nick.Moolenijzer%40microsoft.com|f7bf524a24c84d100f4b08d737023dc3|72f988bf86f141af91ab2d7cd011db47|1|0|637038355074079102&sdata=5vxYNjxMAUJP%2FVny1b0p8JZwWJF%2FQ%2Fzd2mIP5zK%2BfhA%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAK6TGTDKNXNZE75EXKSSZMTQJFR7DANCNFSM4IVXIESQ&data=02|01|Nick.Moolenijzer%40microsoft.com|f7bf524a24c84d100f4b08d737023dc3|72f988bf86f141af91ab2d7cd011db47|1|0|637038355074079102&sdata=Z1jfnXJDTHBtiCqjMrujysJLjxDbqf2pBKnjukpj0YI%3D&reserved=0.


Animate rotation

We should be able to animate both viewport and tiled image rotation.
There are some interesting discussion in #1006 about it.

If possible it would be good to get this in (tiled image rotation at least) for the next release, since otherwise it would probably require an API change. Viewport rotation is already in, so it will likely need an API change somehow.

Okay, image rotation now has animation! Viewport rotation animation can wait for later, since it's already in the API the way it is.


Issue1031 input sample mass

Issue: https://github.com/microbiomedata/nmdc-schema/issues/1031
Updated the sample_mass slot to be called input_mass as it is being used in the Extraction class (a process) in order to clarify if the mass is consumed or generated. This process involved:

updating the nmdc.yaml file to change slot name and add additional annotations.
updating the neon_nmdc_term_mapping.tsv file to change the mapping to input_mass. Changed from sample_mass
updating the valid extraction_set example data files to validate with the newly named input_mass slot
Including a new invalid extraction_set sample data file with the old slot name of sample_mass
Updating the migration_recursion.py file to include a function for migrating the extraction_set collections to change the slot name. Note that the function uses the word NEXT in the name to specify the next version number, which will need to be updated when we know the release version.
Copy and paste Eric's jupyter notebook in the nmdc-runtime repo and add migration changes to this migration. See PR https://github.com/microbiomedata/nmdc-runtime/pull/305

@sujaypatil96 @eecavanna

looking now

It's a little unusual (but not a big problem per se) to request reviews on a draft PR.

Hmm... I think that must have been autogenerated. I just added the mapping to the nmdc.py file.

Do I fix this by adding AFO as a prefix?

OK, you can't edit nmdc.py. It's autogenerated from nmdc.yaml and it's inputs on every build.

Do I fix this by adding AFO as a prefix?

Yes, but lte's reassess whether we even want to use that mapping. One should always be hesitant to start using a new namespace.

https://bioportal.bioontology.org/ontologies/AFO
Allotrope is a high profile organization and the say that they are based on the Basic Formal Ontology (which is good), but I have no familiarity with their ontology at all.

http://purl.allotrope.org/ontologies/result#AFR_0001982 is the full URI for that term
Here's the OLS page if you don't already have it bookmarked
Their axioms look pretty good to me

Otherwise this looks pretty good to me. I'm going to run the migration now as part of the PR assessment. That's doesn't mean that I'm going to share the results with anybody but you. I.e. no intention to load this into any MongoDB yet.

make rdf reiterated

WARNING:root:Unrecognized prefix: AFO

but also reported several Study.award_dois patterns violations. If we talk our way through the AFO term mapping and address the DOI pattern violations (in a separate issue?), then I'm 99%+ sure that your PR will be mergable.

✗ 'doi:10.255851487763' does not match '^doi:10.\d{2,9}/.$' in $.study_set[0].award_dois[0]
✗ 'doi:10.46936jejc.proj.2014.4848360005501' does not match '^doi:10.\d{2,9}/.$' in $.study_set[0].award_dois[1]
✗ 'doi:10.255851488099' does not match '^doi:10.\d{2,9}/.$' in $.study_set[1].award_dois[0]
✗ 'doi:10.46936sthm.proj.2016.4927960005943' does not match '^doi:10.\d{2,9}/.$' in $.study_set[1].award_dois[1]
✗ 'doi:10.255851487765' does not match '^doi:10.\d{2,9}/.$' in $.study_set[2].award_dois[0]
✗ 'doi:10.46936jejc.proj.2014.4847360005497' does not match '^doi:10.\d{2,9}/.$' in $.study_set[2].award_dois[1]
✗ 'doi:10.255851488160' does not match '^doi:10.\d{2,9}/.$' in $.study_set[4].award_dois[0]
✗ 'doi:10.46936sthm.proj.2017.4980460006181' does not match '^doi:10.\d{2,9}/.$' in $.study_set[4].award_dois[1]
✗ 'doi:10.4693610.2558560001061' does not match '^doi:10.\d{2,9}/.$' in $.study_set[5].award_dois[0]
✗ 'doi:10.4693610.2558560001198' does not match '^doi:10.\d{2,9}/.$' in $.study_set[6].award_dois[0]
✗ 'doi:10.4693610.2558560001289' does not match '^doi:10.\d{2,9}/.$' in $.study_set[7].award_dois[0]
✗ 'doi:10.46936sarr.proj.2018.5029860000034' does not match '^doi:10.\d{2,9}/.$' in $.study_set[7].award_dois[1]
✗ 'doi:10.46936cpcy.proj.2019.5118060006718' does not match '^doi:10.\d{2,9}/.$' in $.study_set[7].award_dois[2]
✗ 'doi:10.255851488224' does not match '^doi:10.\d{2,9}/.$' in $.study_set[8].award_dois[0]
✗ 'doi:10.46936fics.proj.2017.4999160006232' does not match '^doi:10.\d{2,9}/.$' in $.study_set[8].award_dois[1]
✗ 'doi:10.255851488096' does not match '^doi:10.\d{2,9}/.$' in $.study_set[9].award_dois[0]
✗ 'doi:10.4693610.2558560000762' does not match '^doi:10.\d{2,9}/.$' in $.study_set[10].award_dois[0]
✗ 'doi:10.4693610.2558560000017' does not match '^doi:10.\d{2,9}/.$' in $.study_set[11].award_dois[0]
✗ 'doi:10.46936intm.proj.2021.6014160000423' does not match '^doi:10.\d{2,9}/.$' in $.study_set[12].award_dois[0]
✗ 'doi:10.46936sarr.proj.2018.5026760000030' does not match '^doi:10.\d{2,9}/.$' in $.study_set[16].award_dois[0]
✗ 'doi:10.46936lser.proj.2019.5071860006575' does not match '^doi:10.\d{2,9}/.*$' in $.study_set[17].award_dois[0]

Sounds good. I guess I am wondering why we would not want to add the AFO mapping. Doesn't it make the schema more interoperable? What are the down sides to adding it?
Its strange because I did not get the study award violations yesterday when I ran it. I know there was a change sheet happening with the award dois, so maybe that went through?

I've given input_mass's AFO mapping more thought and have done more research into our use of mappings and prefix expansions.
I'm going to do some writing about how to navigate this kind of thing and you can contribute your insights and preferences. You are correct that coverage and interoperability are important goals. But we do have to ask ourselves: do we have a goal of being interoperable with
Please remove the AFO mapping. You could create a new issue to add it back in later.

We don't have any holistic experience with AFO
At least in the OLS, AFO uses an unusual CURIe <-> URI conversion pattern. The presence of a / and a # in AFO:/result#AFR_0001982 is highly degenerate.
You also provided  a mapping from input_mass to a MS:1000004. @cmungall and I have more familiarity and trust with MS.

This sounds good. I'll remove it now.

I approve this PR but we shouldn't merge until an apparently new problem validating award_dois in MongoDB has been solved.

@turbomam I could not find these dois mongoDb and took a close look at the before and after running the migration_recursion.py script and it looks like the normalize_curie removed any / from the curies rendering the above award_dois incorrect. I added a / to the regex pattern in the function and I believe it all should pass now.


refactor: Remove the need to use the getKeyDidMappingByVerificationMe…

Closes hyperledger/aries-framework-javascript#1256

Codecov Report

Merging #1350 (a45857b) into main (4ab3b54) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1350      +/-   ##
==========================================
- Coverage   86.56%   86.55%   -0.01%
==========================================
  Files         778      778
  Lines       18462    18452      -10
  Branches     3145     3146       +1
==========================================
- Hits        15982    15972      -10
  Misses       2473     2473
  Partials        7        7

Impacted Files
Coverage Δ

...ckages/indy-sdk/src/dids/IndySdkSovDidRegistrar.ts
91.54% <ø> (-0.12%)
:arrow_down:

...ges/openid4vc-client/src/OpenId4VcClientService.ts
81.94% <ø> (-0.25%)
:arrow_down:

packages/core/src/agent/MessageSender.ts
86.40% <100.00%> (-0.07%)
:arrow_down:

packages/core/src/crypto/WalletKeyPair.ts
78.26% <100.00%> (-0.47%)
:arrow_down:

...ore/src/modules/connections/DidExchangeProtocol.ts
86.82% <100.00%> (-0.07%)
:arrow_down:

...ckages/core/src/modules/dids/domain/DidDocument.ts
96.66% <100.00%> (-0.04%)
:arrow_down:

...ges/core/src/modules/dids/domain/key-type/index.ts
100.00% <100.00%> (ø)

.../src/modules/dids/domain/key-type/keyDidMapping.ts
90.90% <100.00%> (ø)

...e/src/modules/dids/methods/peer/peerDidNumAlgo2.ts
97.77% <100.00%> (-0.03%)
:arrow_down:

...ckages/core/src/modules/vc/W3cCredentialService.ts
79.25% <100.00%> (ø)

... and 3 more

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


How can I serialize a System.Collections.Immutable.ImmutableList?

For example, the following code will cause an exception, saying

The serializer for type 'System.Collections.Immutable.ImmutableList`1[System.Int32]' could not be found.

I suspect other immutable types won't work either...
using ExtendedXmlSerializer;
using ExtendedXmlSerializer.Configuration;
using System.Collections.Immutable;
using System.IO;

class Test
{
    public ImmutableList<int> List { get; }
    public Test(ImmutableList<int> list)
    {
        List = list;
    }

    public void TestSerialize()
    {
        var serializer = new ConfigurationContainer()
            .UseAutoFormatting()
            .UseOptimizedNamespaces()
            .EnableImplicitTyping(typeof(Test))
            .EnableParameterizedContentWithPropertyAssignments()
            .Create();
        using var output = File.Create("test.xml");
        serializer.Serialize(output, new Test(new[] { 1 }.ToImmutableList()));
    }
}

Hey @BSG-75 thanks for writing in with the question and code.  We do have support for ImmutableArray and it was something we added at the outset of v2.  The thinking was that we'd wait until someone asked for other immutable types before adding more.  So here we are. 😁
I've added support for InnutableList and a build to try this out can be found here:
https://github.com/ExtendedXmlSerializer/home/pull/486#issuecomment-737773995
Please let me know if that looks good to you and if so we'll kick it out to NuGet in next Tuesday's build.

@Mike-E-angelo Thanks for the quick response! Now ImmutableList seems to be working well, but could you add support for ImmutableDictionary and ImmutableSortedSet too?

Could you could add support for ImmutableDictionary and ImmutableSortedSet too?

😁
I went ahead and added the whole gamut, why not:

ImmutableList
ImmutableHashSet
ImmutableSortedSet
ImmutableDictionary
ImmutableSortedDictionary

Do note that since these would require additional checks for each type in a graph, I have put these now in a separate configuration method for performance/impact considerations: EnableImmutableTypes.  Example of these in action here:
https://github.com/ExtendedXmlSerializer/home/blob/43ce1d805d4b0812af0f674cf623cc9177c3ad38/test/ExtendedXmlSerializer.Tests.ReportedIssues/Issue485Tests.cs#L12-L109
Please do try these out and verify these work on your end when you can get the chance:
https://github.com/ExtendedXmlSerializer/home/pull/486#issuecomment-738598901
If everything works and looks good to you, we'll get this out in Tuesday's build. 👍

Could you could add support for ImmutableDictionary and ImmutableSortedSet too?

😁

@Mike-E-angelo Eventually I managed to survive and climbed to the top of hill again, to make some tests ;)
The serialization / deserialization of these collections are working by themselves, except one thing:
When these collections are empty, nothing will be written to the xml (as expected, this is also my desired behavior)
But when we are deserializing back from the xml, a NullReferenceException will be thrown, probably because ExtendedXmlSerializer failed to construct an empty immutable collection:
class TestCase
{
    public string Name { get; }
    public ImmutableList<TimeSpan> List { get; }

    public TestCase(string name, ImmutableList<TimeSpan> list)
    {
        Name = name;
        List = list;
    }

    public static void Test()
    {
        var x = new TestCase("ciao", ImmutableList<TimeSpan>.Empty);
        var sz = new ConfigurationContainer()
            .EnableParameterizedContent()
            .EnableImmutableTypes()
            .Create();
        var text = sz.Serialize(x);
        Console.WriteLine("XML: " + text);
        sz.Deserialize<TestCase>(text); // <--- Here, a exception will be thrown
    }

    // I tried to declare the following constructor hoping to resolve the issue,
    // but the deserialization still fails
    public TestCase(string name)
    {
        Name = name;
        List = ImmutableList<TimeSpan>.Empty;
    }
}

Eventually I managed to survive and climbed to the top of hill again

Haha, to be certain, that was meant as a reference for me falling down the hill, as that is how I felt after digging through all my ghetto fabulous code from back in the day. 😆  It's a scene from The Princess Bride and I am sure I am entirely misappropriating it since it has been wayyyy too long ago since I have seen it.  It's the first thing I thought of, but I guess that is what I get for handling issues when I am geeked out on caffeine. 😁😇
ANYWAYS, the empty values were quite the challenge, but I think I figured it out for you.  Check this out and let me know if it works better for your scenario:
https://github.com/ExtendedXmlSerializer/home/pull/486#issuecomment-739288375

Check this out and let me know if it works better for your scenario:
#486 (comment)

@Mike-E-angelo Now it's working perfectly, thanks a lot!
By the way looks like it will cause some issues when using together with EnableParameterizedContentWithPropertyAssignments (instead of EnableParameterizedContent(, but personally I don't use PropertyAssignments yet, so it's not a problem for me xD

PS: looks like it will cause some issues when using together with EnableParameterizedContentWithPropertyAssignments (instead of EnableParameterizedContent), but personally I don't use PropertyAssignments yet, so it's not a problem for me xD

Good catch @BSG-75!  I have posted a fix for this here:
https://github.com/ExtendedXmlSerializer/home/pull/486#issuecomment-739456933

PS: looks like it will cause some issues when using together with EnableParameterizedContentWithPropertyAssignments (instead of EnableParameterizedContent), but personally I don't use PropertyAssignments yet, so it's not a problem for me xD

Good catch @BSG-75! I have posted a fix for this here:
#486 (comment)

All right, I'll try it when I'm wake up ;)
By the way it seems like that when encountering empty collections, ExtendedXmlSerializer will pass null instead of empty immutable container to the paramterized constructor during deserializing.
Right now I'm not able to verify if it's something wrong on my end, or it's a bug in ExtendedXmlSerializer, because I feel so much sleepy, I'll verify it when I wake up 😄

Interesting, @BSG-75 and by interesting I mean frightening. 😧  I did a quick sanity check on those tests in our suite and all 20 of them show that the objects being passed to the constructor are non-null and assigned.  If you do find the culprit on your end and can produce a test that demonstrates how to do so I can take a look into it for you.

@Mike-E-angelo Now I've checked it again, and it's not exactly what I've described before I fall asleep, but something more complicated 😅
I have a class member which is null, so it didn't appear in XML. But when deserializing, the member will still be created, but with everything set to their default value (null in the case of ImmutableList)
class T
{
    public string Name { get; }
    public ImmutableList<int> List { get; }

    public T(string name, ImmutableList<int> list)
    {
        Name = name;
        List = list;
    }
}

class U
{
    public string Id { get; }
    public T? Member { get; }

    public U(string id, T? member)
    {
        Id = id;
        Member = member;
    }

    public static void Test()
    {
        var serializer = new ConfigurationContainer()
            .UseAutoFormatting()
            .EnableImmutableTypes()
            .EnableParameterizedContent()
            .Create();
        var settings = new XmlWriterSettings { Indent = true };

        var instance = new U("ciao", null);
        var xml = serializer.Serialize(settings, instance);
        Debug.WriteLine(xml);
        var deserialized = serializer.Deserialize<U>(xml);
        Debug.Assert(deserialized.Member is null); // Fails
        if (deserialized.Member is not null)
        {
            Debug.Assert(deserialized.Member.Name is not null); // Fails
            Debug.Assert(deserialized.Member.List is not null); // Fails
        }
    }
}

Very good catch @BSG-75 ... another one! :D  I did have to do some changes with how default type values were resolved and this slipped through.  Now we have tests to make sure this works as expected.
Please try this version out and see if you have any better luck with running your above test:
https://github.com/ExtendedXmlSerializer/home/pull/486#issuecomment-739569622

Yeah, now everything works as expected!

Alrighty, this has been deployed to NuGet here:
https://www.nuget.org/packages/ExtendedXmlSerializer/
Please do let me know of any issues you find and I will take a look into them for you.  Closing for now.


TFHub Object Detection Models Cannot Quantize

Please make sure that this is a bug. As per our
GitHub Policy,
we only address code/doc bugs, performance issues, feature requests and
build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Windows 10
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary): source
TensorFlow version (use command below): v2.8.0-rc1-32-g3f878cff5b6 2.8.0
Python version: 3.8.2
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version: N/A
GPU model and memory: Nvidia GeForce GTX 1050 8GB

You can collect some of this information using our environment capture
script
You can also obtain the TensorFlow version with:

TF 1.0: python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
TF 2.0: python -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"

Describe the current behavior
When taking the Object Detection Saved Models provided on https://tfhub.dev/tensorflow/collections/object_detection/1 and converting them to TFLite with quantization enabled, the model outputted remains as a fully float model.
Describe the expected behavior
When converting these Saved Models using quantization enabled, one can produce a quantized TFLite.
Standalone code to reproduce the issue
Provide a reproducible test case that is the bare minimum necessary to generate
the problem. If possible, please share a link to Colab/Jupyter/any notebook.
import tensorflow as tf
import tensorflow_hub as hub
import numpy as np

def dataset_gen():
    for _ in range(10):
        yield [np.random.randint(0,256, [1,300,300,3]).astype(np.uint8)]

inputs = tf.keras.Input(shape=(300,300,3), dtype=tf.uint8)
layers = hub.KerasLayer('https://tfhub.dev/tensorflow/ssd_mobilenet_v2/2')(inputs)
keras_model = tf.keras.Model(inputs=inputs, outputs=layers)

converter = tf.lite.TFLiteConverter.from_keras_model(keras_model)
converter.optimizations = [tf.lite.Optimize.DEFAULT]
converter.representative_dataset = dataset_gen
converter.target_spec.supported_ops = [tf.lite.OpsSet.TFLITE_BUILTINS] # [tf.lite.OpsSet.TFLITE_BUILTINS, tf.lite.OpsSet.SELECT_TF_OPS]
tflite_model = converter.convert()

with open('ssd_mobilenet_v2_tfhub_quant.tflite', 'wb') as f:
    f.write(tflite_model)

Other info / logs Include any logs or source code that would be helpful to
diagnose the problem. If including tracebacks, please include the full
traceback. Large logs and files should be attached.

tflite_model = converter.convert()
2022-03-07 10:52:09.875517: W tensorflow/python/util/util.cc:368] Sets are not currently considered sequences, but this may change in the future, so consider avoiding using them.
WARNING:absl:Found untraced functions such as restored_function_body, restored_function_body, restored_function_body, restored_function_body, restored_function_body while saving (showing 5 of 50). These functions will not be directly callable after loading.
INFO:tensorflow:Assets written to: C:\Users\m\AppData\Local\Temp\tmpmpwp187_\assets
INFO:tensorflow:Assets written to: C:\Users\m\AppData\Local\Temp\tmpmpwp187_\assets
C:\Users\m\venv\lib\site-packages\tensorflow\lite\python\convert.py:746: UserWarning: Statistics for quantized inputs were expected, but not specified; continuing anyway.
warnings.warn("Statistics for quantized inputs were expected, but not "
2022-03-07 10:54:23.844584: W tensorflow/compiler/mlir/lite/python/tf_tfl_flatbuffer_helpers.cc:357] Ignored output_format.
2022-03-07 10:54:23.845355: W tensorflow/compiler/mlir/lite/python/tf_tfl_flatbuffer_helpers.cc:360] Ignored drop_control_dependency.
2022-03-07 10:54:23.864084: I tensorflow/cc/saved_model/reader.cc:43] Reading SavedModel from: C:\Users\m_qui\AppData\Local\Temp\tmpmpwp187_
2022-03-07 10:54:24.146714: I tensorflow/cc/saved_model/reader.cc:78] Reading meta graph with tags { serve }
2022-03-07 10:54:24.147497: I tensorflow/cc/saved_model/reader.cc:119] Reading SavedModel debug info (if present) from: C:\Users\m_qui\AppData\Local\Temp\tmpmpwp187_
2022-03-07 10:54:27.534765: I tensorflow/cc/saved_model/loader.cc:228] Restoring SavedModel bundle.
2022-03-07 10:54:32.634691: I tensorflow/cc/saved_model/loader.cc:212] Running initialization op on SavedModel bundle at path: C:\Users\m\AppData\Local\Temp\tmpmpwp187_
2022-03-07 10:54:35.731075: I tensorflow/cc/saved_model/loader.cc:301] SavedModel load for tags { serve }; Status: success: OK. Took 11867479 microseconds.
2022-03-07 10:54:40.800205: I tensorflow/compiler/mlir/tensorflow/utils/dump_mlir_util.cc:237] disabling MLIR crash reproducer, set env var MLIR_CRASH_REPRODUCER_DIRECTORY to enable.
fully_quantize: 0, inference_type: 6, input_inference_type: 0, output_inference_type: 0

@chunduriv Was able to replicate the issue on colab using TF v2.8.0 and tf-nightly , please find the gist here.Thanks!

Hi @msquigle ! You can get an integer quantized model by adding the three lines after the representative dataset. Attaching gist for reference.
converter.target_spec.supported_ops = [tf.lite.OpsSet.TFLITE_BUILTINS_INT8]
# Set the input and output tensors to uint8 (APIs added in r2.3)
converter.inference_input_type = tf.uint8
converter.inference_output_type = tf.uint8


Install PHP and Composer

Summary by CodeRabbit

Chores

Updated GitHub Actions workflow to use the latest version for better compatibility and performance.
Added conditional steps to optimize PHP and Composer installation based on the operating system.

https://github.com/pj8/dbdb/issues/62


Rack and Sinatra gems have been updated

Upgrade the following gems to resolve security warnings in the repository:

Rack >= 2.0.6
Sinatra >= 2.0.2

These vulnerabilities impact a demo application, not core functionality.

gem 'rack', ">= 2.0.6"
gem 'sinatra', ">= 2.0.2"

@micahlee can you address this with the demo fixes you are making?

@izgeri sure thing!

This is addressed in #23


Update Move to show merging with body

#87 Already brought in most of these changes.
Please create a new branch from main, and open a new PR with only the new changes.


Chat - The emoji picker is incorrectly appearing on the LHN side

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

Action Performed:

Open a chat.
Click on the emoji picker.

Expected Result:
The emoji picker should not be displayed on the LHN side
Actual Result:
The emoji picker is incorrectly appearing on the LHN side
Workaround:
Unknown
Platforms:

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

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

Version Number: 1.3. 76.1
Reproducible in staging?: y
Reproducible in production?: n
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/73c8a612-fb51-462c-9820-de7b47221619
https://github.com/Expensify/App/assets/93399543/2822b04a-c810-4f76-8873-f63a002ff8be
https://github.com/Expensify/App/assets/93399543/f221dcc1-d4f9-44e0-b149-9794fc23b33b
Expensify/Expensify Issue URL:
Issue reported by: @ayazhussain79
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1696249751788399
View all open jobs on GitHub

:wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

Identify the pull request that introduced this issue and revert it.
Find someone who can quickly fix the issue.
Fix the issue yourself.

This is fixed in latest version


GPL-260 - As Craig when I generate a sample sheet I would like it to have a file name which is the same as the run name so I can identify which file relates to which run

Acceptance criteria:

when I download a sample sheet for a run it will have the run name as the file name


bug(mat-option): JAWS 2023 does not announce a checked option as selected in a multiple selection mat-select

Is this a regression?

[ ] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
No response
Description
The screen reader JAWS version 2023 does not announce a checked option as selected in a multiple selection mat-select. It does announce unchecked options as selected.
I tried other Windows screen readers, NVDA does not announce it either but Narrator does. So it could be a screen reader implementation issue.
Also, this issue has been discussed before: https://github.com/angular/components/issues/21949 and it was considered to be working as expected. But I want to bring this up again because I consider that the mat-option component is not following the WAI ARIA Listbox pattern completely, please see Note 1 in this section https://www.w3.org/WAI/ARIA/apg/patterns/listbox/#roles_states_properties.

Some factors to consider when choosing whether to indicate selection with aria-selected or aria-checked are:
Some design systems use aria-selected for single-select widgets and aria-checked for multi-select widgets. In the absence of factors that would make an alternative convention more appropriate, this is a recommended convention.

Mat Option uses aria-select for both single and multiple selection. Based on this recommendation, it should use aria-selected for single selection and aria-checked for multiple selection.
Reproduction
Steps to reproduce:

Start JAWS in a Windows machine.
Go to https://material.angular.io/components/select/overview#select-multiple
Select the multiple selection dropdown.
Navigate through the options using the arrow keys.
Select a few options.
Navigate through selected and unselected options and check how the screen reader announces them.

Expected Behavior
I expect screen readers to announce if an option is selected or unselected.
Actual Behavior
JAWS only announces the status of an unselected option.
Environment

Angular: 15.2.9
CDK/Material: 15.2.9
Browser(s): Chrome 119.0.6045.199
Operating System: Windows 10 Enterprise

Hello,
Thank you for reporting this. JAWS is one of our supported screen readers.

JAWS only announces the status of an unselected option.

This just seems to be how JAWS works.
I see that WAI ARIA guide gives aria-checked as an option. That's a good guide, but I don't know of any accessibility requirement that says screen readers need to announces both selected and unselected options.
I don't see enough of a case for changing it to aria-checked at this time :). Currently, we are looking to make a11y improvements by fixing WCAG violations and removing unnecessary live regions.
Do you have a WCAG criteria or some other criteria to support this? OR do you have an issue report from screen reader user or user that relies on ASsistive Technology? That could help convince us on this.
Best Regards,
Zach

Hi Zach, thank you for your quick response.
The application I am working on just went through a 508 conformance testing session and this issue was reported as not in compliance with the WCAG 2.0 SC 1.3.1: https://www.w3.org/TR/UNDERSTANDING-WCAG20/content-structure-separation-programmatic.html
I hope it helps.
Thank you.

Considering this from a perspective of just trying to interact with this component as a visually impaired user, the functionality is simply lacking, WCAG or not.
To understand what an NVDA user would experience, try this:

Go to https://material.angular.io/components/select/overview.
Activate NVDA.
Tab to the Toppings input under H3 for Multiple selection.
Press ENTER on the input to expand.
TAB to the through the list and press ENTER on list items.

The only announcements I get are the value and list position (e.g. “extra cheese 1 of 6”) and “selected” when pressing ENTER on an item that is not selected. As I tab through the list, I don’t know what is or is not selected and when I press ENTER on an item that is selected, I get no notice that anything happened.
Behavior that would be useful and is not currently present:

When a list item is focused via tabbing, announce “selected” or “unselected” along with the value and list position.
When a list item is unselected via ENTER, announce “unselected”.


Resolving nuget packages in script package

I created nuget script package with main.csx. This script has dependency on some nuget package. When i consume it in other scripts: #load "nuget:my-script, 1.0.0" i have to specify same nuget dependencies in these scripts, otherwise i get error from main.csx about missing assembly reference. I expected from dotnet script that it will load nuget package scripts and resolve their dependencies. Or am i doing something wrong?

@sanjuroo  You have to specify the dependencies in the nuspecfile for your script package


disable folder/indentation in lsp_finder

Again, thank you for this awesome project.
Recently the lsp_finder changed the Definition/Implements/References view to show a folder layout with indentation, is there any way to disable this?
How I would like it:

How it currently is:

It's nice to see the filename, but it's too many strokes to get to References (which is normally the only one I'm interested in)

for a big project . if one file have many references of this word . the old ui will show this file name many times . do you like this ?  currently get the word is limit of window width . in next  change of finder will improve this and window layout .will make this better and more useful than only show filename.


Tests for TaskPackageDropbox

Fixed issue with the number of calls to time.sleep in TaskPackageDropbox. Added tests for:

number of calls to time.sleep in TaskPackageDropbox
single call of receive_one - test to see if WorkingArea.collect_result is called once (test_receive_one_single_call)

time.sleep is called by receive and receive_one only if there are no finished runs to be processed, unless all runs are finished.

@TaiSakuma, do you have time to go through this?
I've been using these commits a fair bit now and it seems to be working correctly. So if there are no finished jobs between querying the dispatcher then job information is reported every sleep seconds.
However, if a significant number of jobs finish between querying the dispatcher then it can take a lot longer than sleep seconds since it's processing the results of the finished jobs.
Should this be the desired result? Since the user might get worried when their job information isn't reported within the set time interval

@shane-breeze, thank you for the pull request. I only had a chance to quickly browse the commits. I will have a look in detail when I can.

Thanks


Error in PDF?

I ran the minimal example (sample.adoc) with the following command:
asciidoctor-pdf -r asciidoctor-mathematical -a mathematical-format=svg -o testsvg.pdf sample.adoc
The output file is:
testsvg.pdf
The file opens fine in Apple's Preview, but if I try to open it in Adobe Acrobat on both OS X and Windows, the following error is reported on opening it:

An error exists on this page. Acrobat may not display the page correctly. Please contact the person who created the PDF document to correct the problem.

The file then opens, but it only displayed as far as the first equation after which it is blank. When using the PNG output with
asciidoctor-pdf -r asciidoctor-mathematical -o test.pdf sample.adoc
Acrobat opens and renders the file without problem.
I am running this on OS X 10.11.4. Can anyone else replicate this or is it just my set up?

Sounds like a problem with asciidoctor-pdf (if the svg files are not corrupt due to mathematical).

I'm pretty sure this is an issue with Asciidoctor PDF and therefore this issue can be closed. However, there isn't a single PDF reader on my machine that complains about this PDF. Therefore, I think the bug is actually in Acrobat Reader on OS X and Windows. This is a completely valid PDF file. It even works find on Acrobat Reader on Linux, which even further justifies my conclusion.

Based on @mojavelinux's comment and my own experience, I think this issue can be closed. Please re-open it if there are new clues.


Kim 161768942 add centiweight invoice generator

Description
The weight value has been applied to the invoice generator as centiweight.  Some weight values were being retrieved from the shipment_line_items table, but this applies the netweight directly from the shipment.
It's possible we will be told in the future to log this as lbs instead of /100lbs, but this seems to make the most sense, not having direct answers from USBank at this point.
Reviewer Notes
@akostibas may recognize having already approved a PR similar, but this one replaces that one
Setup
make db_populate_e2e
Go to tsp-local, login as tsp.
Go to In Transit shipments, click on shipment with locator ENTDEL.
Choose an actual pickup date, and then click on Delivered.
go run cmd/generate_shipment_edi/main.go --moveID 135af727-f570-4c7e-bf5b-878d717ef83c
Code Review Verification Steps

[ ] Request review from a member of a different team.
[ ] Have the Pivotal acceptance criteria been met for this change?

References

Pivotal story for this change

LGTM. I was finally able to understand the EDI enough to verify the L004 entry. I left a comment about adding the function GetCentiweight to the Shipment struct to remove that logic from the EDI generation and let the Shipment worry about passing on the correct information. But nothing to hold up approval over.