DeserializationError when trying to create deployment slot

This is autogenerated. Please review and update as needed.
Describe the bug
Our CI build started failing a few days ago due to the fact that the cli is unable to create new deployment slots. Creating via the portal appears to still work but this isn't very helpful for my CI pipeline to create a review-app.
Command Name
az functionapp deployment slot create
Errors:
The command failed with an unexpected error. Here is the traceback:
Unable to build a model: Unable to deserialize to object: type, AttributeError: 'str' object has no attribute 'get', DeserializationError: Unable to deserialize to object: type, AttributeError: 'str' object has no attribute 'get'
Traceback (most recent call last):
  File "/opt/az/lib/python3.6/site-packages/msrest/serialization.py", line 1428, in _deserialize
    found_value = key_extractor(attr, attr_desc, data)
  File "/opt/az/lib/python3.6/site-packages/msrest/serialization.py", line 1193, in rest_key_case_insensitive_extractor
    return attribute_key_case_insensitive_extractor(key, None, working_data)
  File "/opt/az/lib/python3.6/site-packages/msrest/serialization.py", line 1222, in attribute_key_case_insensitive_extractor
    return data.get(found_key)
AttributeError: 'str' object has no attribute 'get'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/opt/az/lib/python3.6/site-packages/msrest/serialization.py", line 620, in body
    data = deserializer._deserialize(data_type, data)
  File "/opt/az/lib/python3.6/site-packages/msrest/serialization.py", line 1446, in _deserialize
    raise_with_traceback(DeserializationError, msg, err)
  File "/opt/az/lib/python3.6/site-packages/msrest/exceptions.py", line 51, in raise_with_traceback
    raise error.with_traceback(exc_traceback)
  File "/opt/az/lib/python3.6/site-packages/msrest/serialization.py", line 1428, in _deserialize
    found_value = key_extractor(attr, attr_desc, data)
  File "/opt/az/lib/python3.6/site-packages/msrest/serialization.py", line 1193, in rest_key_case_insensitive_extractor
    return attribute_key_case_insensitive_extractor(key, None, working_data)
  File "/opt/az/lib/python3.6/site-packages/msrest/serialization.py", line 1222, in attribute_key_case_insensitive_extractor
    return data.get(found_key)
msrest.exceptions.DeserializationError: Unable to deserialize to object: type, AttributeError: 'str' object has no attribute 'get'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/opt/az/lib/python3.6/site-packages/knack/cli.py", line 231, in invoke
    cmd_result = self.invocation.execute(args)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 657, in execute
    raise ex
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 720, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 712, in _run_job
    return cmd_copy.exception_handler(ex)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/command_modules/appservice/commands.py", line 37, in _ex_handler
    raise ex
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 691, in _run_job
    result = cmd_copy(params)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 328, in __call__
    return self.handler(*args, **kwargs)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/command_operation.py", line 121, in handler
    return op(**command_args)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/command_modules/appservice/custom.py", line 1518, in create_functionapp_slot
    poller = client.web_apps.begin_create_or_update_slot(resource_group_name, name, slot_def, slot)
  File "/opt/az/lib/python3.6/site-packages/azure/mgmt/web/v2020_09_01/operations/_web_apps_operations.py", line 12138, in begin_create_or_update_slot
    **kwargs
  File "/opt/az/lib/python3.6/site-packages/azure/mgmt/web/v2020_09_01/operations/_web_apps_operations.py", line 12067, in _create_or_update_slot_initial
    body_content = self._serialize.body(site_envelope, 'Site')
  File "/opt/az/lib/python3.6/site-packages/msrest/serialization.py", line 623, in body
    SerializationError, "Unable to build a model: "+str(err), err)
  File "/opt/az/lib/python3.6/site-packages/msrest/exceptions.py", line 51, in raise_with_traceback
    raise error.with_traceback(exc_traceback)
  File "/opt/az/lib/python3.6/site-packages/msrest/serialization.py", line 620, in body
    data = deserializer._deserialize(data_type, data)
  File "/opt/az/lib/python3.6/site-packages/msrest/serialization.py", line 1446, in _deserialize
    raise_with_traceback(DeserializationError, msg, err)
  File "/opt/az/lib/python3.6/site-packages/msrest/exceptions.py", line 51, in raise_with_traceback
    raise error.with_traceback(exc_traceback)
  File "/opt/az/lib/python3.6/site-packages/msrest/serialization.py", line 1428, in _deserialize
    found_value = key_extractor(attr, attr_desc, data)
  File "/opt/az/lib/python3.6/site-packages/msrest/serialization.py", line 1193, in rest_key_case_insensitive_extractor
    return attribute_key_case_insensitive_extractor(key, None, working_data)
  File "/opt/az/lib/python3.6/site-packages/msrest/serialization.py", line 1222, in attribute_key_case_insensitive_extractor
    return data.get(found_key)
msrest.exceptions.SerializationError: Unable to build a model: Unable to deserialize to object: type, AttributeError: 'str' object has no attribute 'get', DeserializationError: Unable to deserialize to object: type, AttributeError: 'str' object has no attribute 'get'

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az functionapp deployment slot create --resource-group {} --name {} --slot {} --debug

Expected Behavior
Environment Summary
Linux-4.15.0-142-generic-x86_64-with-debian-buster-sid
Python 3.6.10
Installer: DEB

azure-cli 2.23.0

Extensions:
spring-cloud 2.1.1
portal 0.1.1
application-insights 0.1.13

Additional Context

Sorry I didn't find it earlier, but I seem to have registered a duplicate of #18095

route to service team

Sorry you ran into this @sharebear -- we're working on getting this a fix out ASAP. @yonzhan I think we can close this as a dupe


GFI template

https://docs.github.com/en/communities/using-templates-to-encourage-useful-issues-and-pull-requests/configuring-issue-templates-for-your-repository#creating-issue-forms
Copy of https://github.com/openvinotoolkit/openvino/blob/master/.github/ISSUE_TEMPLATE/good_first_issue.yml

@ilya-lavrenov could we merge it?


Exception encountered: #<NameError: uninitialized constant RSpec::Core::CommandLine>

Do let me know, if necessary, the correct Github repo I should post this issue in.
GEM VERSIONS
Rails 4.2.0
ruby 1.9.3p547 (2014-05-14) [x86_64-linux]
rspec-core (3.1.7)
guard (2.11.1)
guard-compat (1.2.0)
guard-spork (2.1.0)
spork (1.0.0rc4)
spork-rails (4.0.0)
spork-testunit (0.0.8)
I have not used RSpec since December, when our app was still Rails 3. Since, we've upgraded to Rails 4 and updated the necessary gems. I have not changed anything in the Guardfile, .rspec, or spec/spec_helper.rb files yet.
When I run bundle exec guard like I used to, I receive no error (except for Rails-specific deprecation warnings). When I switch to another tab in my terminal, I run bundle exec rspec spec/features/admin/login_spec.rb. As noted in the title, I receive the following error/backtrace:
Exception encountered: #<NameError: uninitialized constant RSpec::Core::CommandLine>
backtrace:
/usr/local/lib/ruby/gems/1.9.1/gems/spork-1.0.0rc4/lib/spork/test_framework/rspec.rb:11:in `run_tests'
/usr/local/lib/ruby/gems/1.9.1/gems/spork-1.0.0rc4/lib/spork/run_strategy/forking.rb:13:in `block in run'
/home/hstevens/.bundler/ruby/1.9.1/spork-224df492657e/lib/spork/forker.rb:21:in `block in initialize'
/home/hstevens/.bundler/ruby/1.9.1/spork-224df492657e/lib/spork/forker.rb:18:in `fork'
/home/hstevens/.bundler/ruby/1.9.1/spork-224df492657e/lib/spork/forker.rb:18:in `initialize'
/usr/local/lib/ruby/gems/1.9.1/gems/spork-1.0.0rc4/lib/spork/run_strategy/forking.rb:9:in `new'
/usr/local/lib/ruby/gems/1.9.1/gems/spork-1.0.0rc4/lib/spork/run_strategy/forking.rb:9:in `run'
/home/hstevens/.bundler/ruby/1.9.1/spork-224df492657e/lib/spork/server.rb:49:in `run'
/usr/local/lib/ruby/1.9.1/drb/drb.rb:1548:in `perform_without_block'
/usr/local/lib/ruby/1.9.1/drb/drb.rb:1508:in `perform'
/usr/local/lib/ruby/1.9.1/drb/drb.rb:1586:in `block (2 levels) in main_loop'
/usr/local/lib/ruby/1.9.1/drb/drb.rb:1582:in `loop'
/usr/local/lib/ruby/1.9.1/drb/drb.rb:1582:in `block in main_loop'

If I need to add gists of additional files, let me know. I think this issue relates to the fact we upgraded to Rails 4. I am not sure where (maybe Spork), but a README listed that it covered <= 4.0.0, not Rails 4.2.0

See https://stackoverflow.com/questions/24030907/spork-0-9-2-and-rspec-3-0-0-uninitialized-constant-rspeccorecommandline-n

@mattmartini I am no longer using guard-spork. Feel free to close this issue.


"Analize" should be "Analyze"

I'll fix this ASAP.

This has been fixed. Kindly redownload the app.


Fix Error in hook structure in EditPayoutsModal

What does this PR do and why?
Closes https://github.com/jbx-protocol/juice-interface/issues/1344.
Was accidentally calling a hook in a non hook/component. This fixes it
Screenshots or screen recordings
If applicable, provide screenshots or screen recordings to demonstrate the changes.
Acceptance checklist

[ ] I have evaluated the Approval Guidelines for this PR.
[ ] I have tested this PR in all supported browsers.
[ ] I have tested this PR in dark mode and light mode (if applicable).

Thanks for feedback @peripheralist - have made these changes because 100% agree


In gulpfile, cannot find ./jspm_packages/system.js

A gist containing the stack trace of this error is here:
https://gist.github.com/bshambaugh/bdd5d4f70afde3abc2ea513c9fc2e69e

Hmm - I see this for the first time. @m0ru?

Hm, weird...
Have you tried deleting jspm_packages and rerunning the build?
Alternatively, can you run npm install (or ./node_modules/.bin/jspm install) manually and see if the file is there afterwards? if so it could be an issue with (our usage) of frontend-maven in regard to jspm install.
For debbugging I cloned the repo freshly and maven pulled the same version of frontend-maven, so it must be something else. What maven version are you using @bshambaugh?

I'll dive in. For now, I put these in my environment:
apache-maven-3.3.3  jdk1.8.0_121

no longer relevant... right? @m0ru

yep, we're using webpack now


How could I add animated icons or winning icons?

banned moment
find it in the discord server or gamebanana

i dont have discord but i will check gamebanana


fix: make rpc model version optional

Adds the optional field label to model metadata, version, and inference requests in the RPC specification to match our documentation.
Other fields in the RPC spec seem to be presenting as optional, but those can be addressed in other issues and PRs.
Closes #1

OpenAPI lint errors to be addressed in #7


Non Pod networks are filtered at Migration Plan creation

When creating a Migration Plan it's not possible to map non Pod networks (multus).
It's possible to create a map of multus networks through the NetworkMaps for virtualization menu.
However the creation plan wizard also filters those non Pod networks after selecting the previously created NetworkMap.

@mmaritinv hi,
In the wizard we know the namespace we are going to create the VMs in,
We only show networkattachmentdefinition that are on the same namespace [1], because that is the default behavior described by kubevirt documentation. It is possible to use different namespace [2].
Ref:
[1] https://kubevirt.io/user-guide/virtual_machines/interfaces_and_networks/#multus
[2] https://github.com/k8snetworkplumbingwg/multus-cni/blob/master/docs/how-to-use.md#launch-pod-with-text-annotation-for-networkattachmentdefinition-in-different-namespace
More information:
https://github.com/kubev2v/forklift-console-plugin/blob/main/packages/legacy/src/Plans/components/Wizard/MappingForm.tsx#L165


Support for extended memory on custom machine type

Hi,
I have a workload that requires VMs of 1 CPU and 20 GB of RAM.
When using dsub, if I use --provider google-v2 --min-cores 1 --min-ram 20 then it will spawn a custom machine with 4 vCPUs and 20 GB of RAM.
Ideally I would like to have it spawn a custom machine with 1 vCPU and 20 GB of RAM using extended memory, as it reduces the cost a bit.
Is there support for extended memory in dsub?
If so, how to set it properly on the command line?

Hi @vincent-octo !
I'm not sure if this is supported by the Pipelines API. If it is, this would be done by specifying a custom machine type as documented here:
https://cloud.google.com/genomics/reference/rest/Shared.Types/Metadata#VirtualMachine

The machine type of the virtual machine to create. Must be the short name of a standard machine type (such as "n1-standard-1") or a custom machine type (such as "custom-1-4096", where "1" indicates the number of vCPUs and "4096" indicates the memory in MB).

Please try using the --machine-type flag with a custom value. If this does not work, then we will need to pass a feature request on to the Cloud Health team.

Hi @mbookman and thanks for your reply.
I didn't find a mention of extended memory in the documentation you provided. However, the GCP pricing calculator names this instance type n1-custom-1-20480-extended.
I tried it with dsub but it results in the following error:

dsub --provider google-v2 ...  --machine-type n1-custom-1-20480-extended --command 'echo "Hello "' --wait
Job: dsub-vllor--vllorens--190830-135045-31
...
googleapiclient.errors.HttpError: <HttpError 400 when requesting https://genomics.googleapis.com/v2alpha1/pipelines:run?alt=json returned "Error: validating pipeline: invalid machine type: unknown machine type">

Like you said, the API doesn't seem to support extended memory. So I will use custom VMs withtout extended memory (the price difference is not that big).
Feel free to close this issue.

Thank-you for testing it out. I have filed a feature request to have support in the Pipelines API.
Will leave this issue open for tracking.

Hi @vincent-octo .
The Pipelines API team has added initial support for custom machine types.
For N1 machine types, you can use the notation:
--machine-type custom-CORES-MEMORY-ext
Support for the N2 machine types has not been added yet.

Hi @mbookman ,
The custom VM with extended memory seems to be working from my initial test.
Just a note: the memory should be put in MiB and be a multiple of 256MiB. Otherwise it will result in the following error:
FAILURE   Execution failed: creating instance: inserting instance: Invalid value for field 'resource.machineType': 'zones/europe-west1-c/machineTypes/custom-1-20-ext'.
Memory should be a multiple of 256MiB, while 20MiB is requested. Memory size for 1 vCPU instance should be between 1024MiB and 6656MiB, while 20MiB is requested

Thanks for your help.


Url Creator - 02-14-2024

Daily Challenge Issue Template
Challenge Description

Given a string convert it into an acceptable url (For this scope we will be focusing on spaces).
Acceptable Solution Standards

A function takes in a string
Must return a string with the spaces altered to %20
Attempt to use the fastest time complexity when completing the challenge

Estimated Time for Completion

Estimated time: 20-30 minutes

Additional Notes or Resources (Optional)

function urlCreator(str) {
// Your code goes here
}
console.log(urlCreator('This is an url')) // Should return 'This%20is%20an%20url'

To run you code in the terminal use the command node [FILE_NAME].js

Submission Guidelines

Assign yourself to this issue
Create a branch for yourself in this issue
Write your code locally
Push up your code to your branch on GitHub
Create a Pull Request to merge your code into main
Get reviewed by the "challenger"
Get merged or update your code as requested, and repeat 4, 5, 6.

Nice challenge! I love the simplicity!


B14.0 Triage Static Code Analysis Reports

💡 Description

Bi-annual review of static code analysis reports.

📆 August status: Running behind by 1 week on this. To be worked in first week of September to include in delivery to I&T.

Done. Tickets have been opened and dependabot has been added to all repos


Revert "Update all major dependencies to v7 (major)"

Reverts wbaldoumas/coding-blog#308

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 22.543%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
306

Relevant Lines:
1158

💛  - Coveralls


starting point for custom move_and_slide()

attributions
thanks to @DreamyCecil (I think, if u coded this, please let me know) for applying the code of this inline re-implementation of move_and_slide() proposal to rely on built-in variables and check for the floor at the end of the collision check
the code
# Replacement for built-in is_on_floor()
var on_floor := false

## Perform a move-and-slide along the set velocity vector. If the body collides
## with another, it will slide along the other body rather than stop immediately.
## The method returns whether or not it collided with anything.
func move_and_slide_own() -> bool:
    var collided := false

    # Reset previously detected floor
    on_floor = false

    # Loop performing the move
    var motion := velocity * get_delta_time()
    for step in max_slides:
        var collision := move_and_collide(motion)
        if not collision:
            # No collision, so move has finished
            break

        # Calculate velocity to slide along the surface
        var normal = collision.get_normal()
        motion = collision.get_remainder().slide(normal)
        velocity = velocity.slide(normal)

        # Check for the floor
        if normal.angle_to(up_direction) < floor_max_angle:
            on_floor = true

        # Collision has occurred
        collided = true

    return collided

func get_delta_time() -> float:
    if Engine.is_in_physics_frame():
        return get_physics_process_delta_time()

    return get_process_delta_time()

If you have taken it from the Reddit post, then yes, it was me. All I did was add floor detection. You can also add wall and ceiling detection, if you need a more complete reimplementation. Don't remember what else is required.
# Check for the floor
if normal.angle_to(up_direction) < floor_max_angle:
  on_floor = true

# Check for the ceiling
elif normal.angle_to(-up_direction) < floor_max_angle:
  on_ceiling = true

# Otherwise it's a wall
else:
  on_wall = true

I wanted to make one that doesn't have floor collision but applies gravity, since the NavigationAgent get_next_path_position() gives the desire elevation I thought it was possible, thanks for the code, let me know if its a proper attribution


PL/pgSQL dollar-quote highlighting rule is missing

System information:

DBeaver version 8ec205926e
Java version 1.8.0

Connection specification:

Database name and version PostgreSQL 11.2

Describe the problem you're observing:
It is common to use dollar quoting to avoid explicit escaping of single quote marks within stored procedure source code.
Everything within such block should be highlighted as a string literal, whereas opening $$ are black and closing $$ are red and everything in between is highlighted as if pl/PGSQL code. I have a dynamic SQL statements and it is very confusing as I would expect seeing those as strings (green?).

Steps to reproduce, if exist:
CREATE OR REPLACE FUNCTION blah()
 RETURNS text
 LANGUAGE plpgsql
 LEAKPROOF
AS $function$
BEGIN
  return format($$That's funny
    select 1;
  $$);
END
$function$
;

GFM isn't better … oh well … would be nice though.
Edit: Apparently pgAdmin is not aware of that either. I'll leave it here as a low-priority feature request :-)

This is a very bad workaround if you use PostgreSQL only and plain $$ for multiline strings.
diff --git a/plugins/org.jkiss.dbeaver.ui.editors.sql/src/org/jkiss/dbeaver/ui/editors/sql/syntax/SQLPartitionScanner.java b/plugins/org.jkiss.dbeaver.ui.editors.sql/src/org/jkiss/dbeaver/ui/editors/sql/syntax/SQLPartitionScanner.java
index 92ec6863d..7de329ca0 100644
--- a/plugins/org.jkiss.dbeaver.ui.editors.sql/src/org/jkiss/dbeaver/ui/editors/sql/syntax/SQLPartitionScanner.java
+++ b/plugins/org.jkiss.dbeaver.ui.editors.sql/src/org/jkiss/dbeaver/ui/editors/sql/syntax/SQLPartitionScanner.java
@@ -118,6 +118,7 @@ public class SQLPartitionScanner extends RuleBasedPartitionScanner {

     private void initRules(SQLDialect dialect)
     {
+        rules.add(new MultiLineRule("$$", "$$", sqlStringToken));
         rules.add(new MultiLineRule(SQLConstants.STR_QUOTE_SINGLE, SQLConstants.STR_QUOTE_SINGLE, sqlStringToken, dialect.getStringEscapeCharacter()));
         rules.add(new MultiLineRule(SQLConstants.STR_QUOTE_DOUBLE, SQLConstants.STR_QUOTE_DOUBLE, sqlQuotedToken, dialect.getStringEscapeCharacter()));

Feedback wanted. I sort of cheat by excluding $function$. Technically it is also a string but we do want to see stuff highlighted. Proper recognition would require yet another fancy rule (AS prefix?). I think it is okay to get by with exceptions. I'm not sure whether other PG versions return different quotation tag for body with pg_get_functiondef.

I'm somewhat at lost with this one before submitting a PR. Where shall I place rule addition? I mean do we want dialects to  have extra(rules) function to add whatever specific to dialects? But then we bring/import Eclipse Platform Text rules into dialects. For now, I check dialect name, but this is somewhat not reliable. I could check for PostgreDialect instance but it would bring those namespaces in as not everyone is using PG.

Bump? Can we get this in? I am willing to clean things up. I just need some feedback as I'm not that much into Java...

Sorry, I missed this ticket.
There is #6608 where we tracking this problem (it was created later than this but still).
I will check your commit and will write some feedback. Thanks for contributing !

@mlt please see comments in #6608.
We can't always treat dollar quoted string as strings. Sometimes we need to treat them as SQL statements.
Anyhow, I hope now the problem is solved.


Ensure key / values are shared between resumed sessions

Motivation:
When a session is resumed we need to also ensure we preserve the values that were put into the internal storage of the session.
Modifications:
Ensure we preserve the values / keys of the internal storage on resumption
Result:
Correctly implement session caching and reuse

I don't think this change updates the last accessed time on session reuse. The OpenSslSessionCache updates the last accessed time of the NativeSslSession but this is not carried over to the DefaultSslSession used in ReferenceCountedOpenSslEngine.
The OpenSslEngineTest.assertSessionReusedForEngine() method should include assertions for the last accessed time being more recent than the creation time.

@chrisvest got it... Will do some more changes

This will need to be ported to quic as well


Add a search option

Google-like
datafetcher search -name tweets OR twitter -tags sentiment english

dafter search image deep-learning

to search datasets that have the tags image and deep-learning


Adding Bigtable docs.

Docs serving (for now at least, as of 12:45pm Pacific on Feb. 26): https://blog.bossylobster.com/gcloud-python/latest/bigtable-usage.html

@jgeewax PTAL as well

Does it make sense to stuff the autogen stuff under a ref folder?

Finished a pass, please let me know when you're ready for another round.

@jonparrott Just pushed a new commit PTAL. I did a full pass to make sure things still made sense (since the API has changed over time) and found a few changes to make here and there (many are stylistic).

This LGTM once @jgeewax's comments are addressed.
(Also don't forget to squash)

Thanks

Dangling questions to be resolved when #1548 is. Merging now.

I see this one is merged -- are the docs up to date on both gh-pages and gcloud-python.readthedocs.org? (http://gcloud-python.readthedocs.org/en/latest/happybase-package.html doesn't look like the demo)

/cc @dhermes

AFK right now but will try to sleuth. Looks like the docs build is broken somehow

@jgeewax confirmed the build failure: https://readthedocs.org/projects/gcloud-python/builds/3798705/
For example:
/home/docs/checkouts/readthedocs.org/user_builds/gcloud-python/checkouts/latest/docs/happybase-table.rst:4: WARNING: autodoc: failed to import module u'gcloud.bigtable.happybase.table'; the following exception was raised:
Traceback (most recent call last):
  File "/home/docs/checkouts/readthedocs.org/user_builds/gcloud-python/envs/latest/local/lib/python2.7/site-packages/sphinx/ext/autodoc.py", line 385, in import_object
    __import__(self.modname)
  File "/home/docs/checkouts/readthedocs.org/user_builds/gcloud-python/checkouts/latest/gcloud/bigtable/__init__.py", line 18, in <module>
    from gcloud.bigtable.client import Client
  File "/home/docs/checkouts/readthedocs.org/user_builds/gcloud-python/checkouts/latest/gcloud/bigtable/client.py", line 32, in <module>
    from grpc.beta import implementations
ImportError: No module named grpc.beta

I forgot I had a hack in place and will do something similar (but closer to the recommended way of fixing bad imports).


CLI or HTTP API?

Hello there!
Thanks for providing an awesome tool. We @ https://transport.data.gouv.fr are very interested in using/contributing to this repository (and we already did! #39)
We'd like to know if it's possible to use this tool as a CLI tool or an HTTP API.
It seems like we can use it as a CLI by doing
cd gbfs-validator
npm install
node cli.js https://transport.data.gouv.fr/gbfs/strasbourg/gbfs.json

which gives a JSON report
There is also an HTTP API available at https://gbfs-validator.netlify.app/.netlify/functions/validator
curl -X POST -d '{"url": "https://transport.data.gouv.fr/gbfs/strasbourg/gbfs.json"}' https://gbfs-validator.netlify.app/.netlify/functions/validator

Is this safe to use? Should this be documented?

Hello @AntoineAugusti
Yes both version are available.
The main entry point was the website https://gbfs-validator.netlify.app/ (+ the api/lambda function). The cli was written to validate in batch the systems.csv file.
But, the api/lambda function nor the exposed JSON report are not intended to be used outside of this project.
Breaking changes can be introduced later.
After this warning, i think you can still use it as you want :)
The validator is currently hosted on my personnal Netlify (free plan, limited to 125k requests/month) account and we are in the way to transfer it to MobilityData.
On the long-term roadmap, we planned to export a report (less breaking changes).

Thanks a lot @PierrickP, I will keep that in mind! I think we'll use the HTTP API for now and we'll keep in mind that it can disappear or break.
We plan to make less than 3k HTTP requests per month and I will add a User-Agent: <email_address> to identify our requests. Thanks a lot for allowing us to use the Lambda outside this project.


Add TiDB support

Add TiDB support to Druid. There are couple additional changes to code path that suppose to support MariaDB.
This PR closes: #4591

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


[BUG] DefaultAzureCredential doesn't work with IIS environment variables

Describe the bug
EnvironmentCredential doesn't get the necessary values if they are stored in IIS environment variables.  It only works when we create OS level environment variables.  This is when using DefaultAzureCredential to authenticate against KeyVault.
Expected behavior
IIS environment variables and OS env vars would both work
Actual behavior (include Exception or Stack Trace)
Unhandled Exception: Azure.Identity.CredentialUnavailableException: DefaultAzureCredential failed to retrieve a token from the included credentials.
- EnvironmentCredential authentication unavailable. Environment variables are not fully configured.
- ManagedIdentityCredential authentication unavailable. The requested identity has not been assigned to this resource.
Status: 400 (Bad Request)
Content:
{"error":"invalid_request","error_description":"Identity not found"}
Headers:
Server: IMDS/150.870.65.511
Date: Tue, 08 Jun 2021 13:56:20 GMT
Content-Type: application/json; charset=utf-8
Content-Length: 68

To Reproduce
Steps to reproduce the behavior (include a code snippet, screenshot, or any additional information that might help us reproduce the issue)
Aspnet Core 2.1 application, connecting to KeyVault in program.cs like:
var secretClient = new SecretClient(new Uri($"https://{builtConfig["KeyVault:VaultName"]}.vault.azure.net/"), new DefaultAzureCredential());
config.AddAzureKeyVault(secretClient, new PrefixKeyVaultSecretManager(prefix));

Environment variables added in IIS to the app pool, saved in applicationHost.config:

Application crashes at startup because the KeyVault call is unauthenticated
Switch the environment variables to the OS, restart IIS and it all works.

Environment:

Name and version of the Library package used: Azure.Identity 1.3.0, Azure.Extensions.AspNetCore.Configuration.Secrets 1.0.2
Hosting platform or OS and .NET runtime version:
Host (useful for support):
Version: 2.1.18
Commit:  ba3b934aa4

.NET Core SDKs installed:
No SDKs were found.
.NET Core runtimes installed:
Microsoft.AspNetCore.All 2.1.18 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
Microsoft.AspNetCore.App 2.1.18 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.NETCore.App 2.1.18 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]

IDE and version : Visual Studio 16.9.2

Thank you for your feedback.  Tagging and routing to the team members best able to assist.

Hi @AndyMcKenna From what I can tell from the IIS documentation, those variables should be passed to the worker process, which would make them available to Azure.Identity's call to Environment.GetEnvironmentVariable
However, looking at your screen shot, I noticed that one of the variables has the wrong casing for the first letter. Does it work if you change that to "AZURE_TENANT_ID"?

I tried fixing the lowercase "a" and that didn't work and then I noticed it's spelled TENTANT there  🤦♂.  Corrected that, reset the app pool and then everything starts up successfully.


Add STES Rocketry

Hi there!
Thanks for your contribution!
I checked out the website, and saw you are accepting sponsorships, but what is the process for making a donation? Is there a Paypal account?

Yes, we do have a PayPal account. I've updated it in the website. You may find it under the sponsor bubble in Get involved.

Great!
Please add the paypal link (https://www.paypal.com/paypalme/stesrocketry?country.x=IN&locale.x=en_GB) in the 'how to donate' column and I will merge in the pull request.

Made the update. Please review.

This issue has not been resolved, so I am going to close this pull request.
If you are able to get a working donation link for your team working, I would, be happy to have you listed. Please make another pull request when ready. 🚀


menuflyout background incorrect after live color mode change

Which version of the app?
WinUI 3 Gallery
Description

start 'winui 3 gallery'
select 'menus and toolbars' -> 'menubar'
under 'a simple menubar' click any 'menubaritem', for example 'help', -> the 'menuflyout' will show 'about'
start 'windows settings'
select 'personalization' -> 'colors'
under 'choose your mode' change the value, for example from 'light' to 'dark' -> the color mode will change accordingly
go back to 'winui 3 gallery'
repeat 3., under 'a simple menubar' click any 'menubaritem', for example 'help', -> the 'menuflyout' will show 'about' with an incorrect background color (note: the 'menuflyoutitems have changed as expected)

Screenshots

step 3. in light mode

step 8. after switching from light mode to dark mode

step 3. in dark mode

step 8. after switching from dark mode to light mode

Windows version
Windows 11 22H2 (22621)
Additional context
No response

Any update on if this will be fixed in an upcomming release, or if works been done to correct the default theme?

I wonder if there is a workaround for WinUI Gallery while the issue is being investigated on the WASDK side. Could the theme be set on flyout.loaded?

I'm not sure if we can implement a way to solve this using the current API of MenuBar. There is no API that would expose the flyout as it is being created dynamically inside the MenuBarItem control. Guess this is a WinUI 3 issue then @karkarl?


Provide full support for "delimited" IMPORT format

Background: https://airtable.com/tblD3oZPLJgGhCmch/viw1DKmbKhg2MIECH/recrfAbhSAqoYUmjI
Description:
Team:
Github Tracking Issue:

it seems like nothing further needs to be added to documentation.


compilation with GCC-9 on OSX Catalina

Compilation of the v0.3.7 and latest develop branch (commit eb2eddf241dd5e46e0064cd311b6a63229897d21) is not successful on OSX -10.15.1 (Catalina). XCode 11.2.
GNU Fortran (MacPorts gcc9 9.2.0_1) 9.2.0
One important selected option:   "INTERFACE64 = 1", was set in Makefile.rule. Everything else was by default.

Backtrace for this error:
#0  0x10ce130d2
#1  0x10ce13870
#2  0x7fff63e83b1c
#3  0x10ced2900
#4  0x10cee2077
#5  0x10cdfe52e
#6  0x10ce059ce
make[1]: *** [level1] Segmentation fault: 11
make: *** [tests] Error 2

Any warnings earlier in the build, particularly around building of the sgemm kernel, or any chance you could rebuild with the additional option DEBUG=1  (which should hopefully provide a better backtrace) ?
I do not see any problems with gcc 9.1 on Linux (did not get around to installing 9.2 yet)

Please find attached the compilation log and the warnings and errors during compilation.
The compilation looks OK, but the verification of the compiled library fails.
As example, running the sblat2 test:
OPENBLAS_NUM_THREADS=1 OMP_NUM_THREADS=1 ./sblat2 < ./sblat2.dat
produces the following error:
At line 124 of file sblat2.f (unit = 6, file = 'stdout')
Fortran runtime error: Bad STATUS parameter in OPEN statement
The line 124 of the sblat2.f is:
124  OPEN( NOUT, FILE = SUMMRY, STATUS = 'NEW' )
This line of Fortran looks ok for me... Removing the "STATUS = 'NEW' " option propagates the error to a later position, and finally ends with a general segmentation fault.
Also, the folliwing error is there:
OPENBLAS_NUM_THREADS=1 OMP_NUM_THREADS=1 ./sblat1
Program received signal SIGSEGV: Segmentation fault - invalid memory reference.
Backtrace for this error:
#0  0x10ba270d2
#1  0x10ba27870
#2  0x7fff63e83b1c
#3  0x10bae6900
#4  0x10baf6077
#5  0x10ba066f1
#6  0x10ba06959
zsh: segmentation fault  OPENBLAS_NUM_THREADS=1 OMP_NUM_THREADS=1 ./sblat1
compilation.log
compilation.err.txt

STATUS='NEW' is valid fortran, it will generate a runtime error when the file already exists (SUMMRY is SBLAT2.SUMM here, all the BLAS2 and 3 tests produce corresponding .SUMM files). Strange that the backtrace for the sblat1 crash would not contain function names with DEBUG=1, which adds the -g option to the compiler flags. Could be all steps are inside libc or some other system library for which no debugging information is available though.

Not reproducible on Linux with gcc 9.2.0 either (and valgrind does not find any signs of illegal memory accesses there). Travis CI shows no suspicious build warnings but unfortunately does not appear to offer Haswell cpu features to its osx instances. A dynamic_arch build with nehalem for the build host target runs all tests without fault, though some of the results are intermixed with runtime warnings about IEEE fp signals for division-by-zero. (I beiieve these are caused by generic test code in LAPACK that deliberately performs a division by zero to see if the system handles it in a standard-conforming way.)

The latest upgrade of OSX- Catalina discontinues all 32-bit applications. Many of the previously installed software had released special updates/releases for OSX to comply with this drastic change. I wonder if the reported issue is somehow connected with this change of the new OSX.

Thanks for the pointer, but I do not see how this could be related. (If anything one might argue that it could make INTERFACE64 a requirement if it also implied changing all int to long). BTW does a build without the INTERFACE64=1 work on your system ?

No. Leaving everything in the Makefile.rule by default gives a similar result.

Could it be picking up a wrong dynamic library at runtime ? (It is a long shot, but the sblat1 binary is linked dynamically against libgfortran.so etc). Another thing to try would be specfiying a TARGET like SANDYBRIDGE or NEHALEM to check if the problem is linked to specific gemm kernel features, e.g. AVX2 (I am now trying to build for Sandybridge on Travis in the hope that the virtual
hardware is capable of running Sandybridge code)

No, not AVX512 (that would be "SkylakeX" target, i.e. recent Xeon cpu. Yours appears to be correctly autodetected as having AVX2 (Haswell, Kaby Lake or similar refreshes of the original Haswell architecture). I assume you are building on actual hardware, or is the build running in a virtual machine ?

No. I build it on my laptop.
Here are the libraries which are linked dynamically to sblat1 and sblat2 tests:
sblat1:
/usr/local/lib/libgfortran.3.dylib (compatibility version 4.0.0, current version 4.0.0)
/usr/lib/libSystem.B.dylib (compatibility version 1.0.0, current version 1281.0.0)
/usr/local/lib/libgcc_s.1.dylib (compatibility version 1.0.0, current version 1.0.0)
/usr/local/lib/libquadmath.0.dylib (compatibility version 1.0.0, current version 1.0.0)
Other test programs have the same dependencies.

Hmm, I wonder about the libgfortran.3.dylib - normally gcc 9.x comes with libgfortran version 5 (at least on Linux, but I do not see why OSX would be different). Do you have a libgfortran.5.dylib somewhere on your system as well ?

TARGET=SANDYBRIDGE build completed on Travis just now, all tests passing without any warning (not even the "IEEE... is signalling" seen from the dynamic_arch build).

Hmm, I wonder about the libgfortran.3.dylib - normally GCC 9.x comes with libgfortran version 5 (at least on Linux, but I do not see why OSX would be different). Do you have a libgfortran.5.dylib somewhere on your system as well?

Oh, I think the problem is solved now. It was my problem.
The libraries on my system were indeed wrongly linked since I had an old gfortran installation (which I had completely forgotten about).
After the libgfortran.5.dylib which came with gcc9 was correctly linked, everything worked fine.
I was confused by the fact that other Fortran codes compiled and worked fine after the system upgrade.
The issue is solved and may be closed. Thank you for the support and for your time.


Provide sources through SourceBrowser

It would be really awesome if ASP.NET Core would also be available through Kirill's SourceBrowser, Roslyn, CoreFx, and .NET Framework are already available. I guess this one is kinda of missing online ;)
I've been learning ASP.NET Core, and going through the excellent Fundamentals section, and being able to go through the sources makes it really easy to dive into the details, without having to clone every repo, and opening it in VS.

@natemcmaster you did something with this a while ago. Is it up anywhere?

I created an unofficial source browser site last year for the 1.0.0 release. I haven't updated it since, but here you go:
https://github.com/natemcmaster/AspNetSourceBrowser
https://aspnetsource.azurewebsites.net/

Awesome! That's really great, although, it's not completely up to date, it is really handy as reference.
Feel free to close the issue, if there are no plans to make this official or permanent (but I hope you guys do ;)

We considered making it official but never got around to actually doing that. I'll use my internal Microsoft email powers to check again on doing something official.

Looks like https://source.dot.net/ already exists for .NET Core. Until something official exists, https://aspnetsource.azurewebsites.net/ is the best place for ASP.NET Core.

Btw, you can follow https://github.com/dotnet/source-indexer/issues/5 for progress on that.

Yeah, I found https://source.dot.net/ through Kirill's readme.md (which now contains yours as well 😉 ), though, I noticed indeed that ASP.NET Core wasn't added. Thanks for making the issue on source-indexer 👍


[C#] Change generated On{SignalName} to EmitSignal{SignalName}

Change the generated On{SignalName}, which is introduced in #68233 , to EmitSignal{SignalName}
Editor already generates _on_{signal_name} for signal callbacks.
Having On{SignalName} introduces confusion and will likely break user code, since Godot users are already familiar with on signal name for signal callback functions.
Using EmitSignal{SignalName} conforms to Godot's name of raising signals emit_signal.
The terms and names are consistent between defining signals (using [Signal] attribute) and emitting signals (using EmitSignalName methods). Also it kinda looks great in the intellisense list, like:

EmitSignal (Godot's original emit_signal)
EmitSignalPressed
EmitSignalChanged

Although I'm not entirely sure of the Emit{SignalName} naming scheme, but I do agree that we should not use the On{SignalName}, as I mentioned in the original PR that On{SignalName} is used for referring Callee, not the Caller in guidelines.
Reference: .Net Guidlines.

According to the guidelines it should be On.... It's confusingly worded there, but on the events page is an example.
However, I think it's still worth considering if all the C# event guidelines should be followed as strictly. For me signals are not exactly the same as C# events. So using 'emit' instead of 'on' might be more in line with the Godot naming.

Oh... emit collides with Resource::emit_changed, so that's blocked.

We have to change On{SignalName} to [Something]{SignalName}.
We can't directly use Emit because Resource has a signal changed, and a function emit_changed in which also emits changed.
We can either use EmitSignal{SignalName} or Raise{SignalName}

Or we just tread that emit_signal as a special, already implemented case and do not generate the corresponding wrapper method of it.

Or we just tread that emit_signal as a special, already implemented case and do not generate the corresponding wrapper method of it.

I don't think it's appropriate. Resource::emit_changed has extra logic to notify ResourceLoader. We need to generate another method to just do emit_signal("changed")

EmitSignal{SignalName} seems a good choice.

@raulsntos Can you check this PR? EmitSignal{SignalName} is more suitable for me than On{SignalName}, because I already use On{SignalName} as the signal listening method in my personal project

This PR could resolve the compatibility issue mentioned in #97781, I honestly don't know why no one has reviewed this PR yet


TCOTA-518 TCOTA-538 implement student view of multiple attempts (back…

…end)
Added new step (view_assignment), backing service methods, and unit tests. Refactored assessment-specific methods from AssignmentService to AssessmentService, where they should live.

PR comments addressed!


OAUTH not working

I followed the instructions and used by HA cloud url in the correct place during step 4, but I receive an AUTH error when attempting to add the home assistant integration.
TRAKT
OAUTH ERROR
The requested redirect uri is malformed or doesn't match client redirect URI.

TRAKT
OAUTH ERROR
The requested redirect uri is malformed or doesn't match client redirect URI.

I will see if I can reproduce this error

I am sorry, I can't reproduce the error. In Trakt, in your api app, you are sure that your redirect URI is something like http://localhost:8123/auth/external/callback ?

Unfortunately it still doesn't work. I retested with every different combo for the URI. I tested with my nabu casa link, with and without the port, with my internal ip, with and without the port, and as http://homeassistant, with and without the port. I also tested as http and https. I tried them in an  incognito tab and on a different browser too. All of these returned the OAUTH error. I also retested today with the most current version of the component and current version of home assistant. No luck. If it matters, I used to have the sensors.trakt component installed and connected, but removed it after finding this one.

The steps should be the same as the other trakt integration. I can't reproduce the error, so I can't resolve it for now. If you had success with the other integration then you can just use it instead. I let this issue open in case someone else has the same issue. If you fix the issue, don't hesitate to close the PR.

Not working for me. I've tried many many variations of the uri and nothing.
http://<IP_ADDRESS>:8123/auth/external/callback
no luck.

Anything else we could try?
I did a fresh install of Homeassistant, Fresh Trakt useraccount, fresh API followed all steps. (using without cloud redirect_uri) Same result.
I'm using HA in a docker container.

I have to find the reason why you can't use the addon, it is still difficult since I can't reproduce it. Do you have some screens of the error ?

Hopefully this helps! 😬

Thanks, can you give me the redirect URI on https://trakt.tv/oauth/applications/ to be sure that's it is OK ?

Good call. Is there something in the documentation I missed on the callback URL?

I had this same issue, until I realized it was redirecting through "my home assistant". Here's what I had to do to fix this:

go to https://my.home-assistant.io/ and enter the correct path to my instance (for me this was http://<IP_ADDRESS>:8123/)
instead of entering my local address (http://<IP_ADDRESS>:8123/) in the TRAKT application page I entered this: https://my.home-assistant.io/redirect/oauth
added the integration
accepted in TRAKT
then accepted in the my home assistant page that showed up
profit

I had this same issue, until I realized it was redirecting through "my home assistant". Here's what I had to do to fix this:

go to https://my.home-assistant.io/ and enter the correct path to my instance (for me this was http://<IP_ADDRESS>:8123/)
instead of entering my local address (http://<IP_ADDRESS>:8123/) in the TRAKT application page I entered this: https://my.home-assistant.io/redirect/oauth
added the integration
accepted in TRAKT
then accepted in the my home assistant page that showed up
profit

I wonder if @dylandoamaral does not have the (default) "my" integration enabled? See: https://www.home-assistant.io/integrations/my/

Flawless! This works....
adding https://my.home-assistant.io/redirect/oauth as the redirect I don't understand it but it works. Thank you.

I had this same issue, until I realized it was redirecting through "my home assistant". Here's what I had to do to fix this:

go to https://my.home-assistant.io/ and enter the correct path to my instance (for me this was http://<IP_ADDRESS>:8123/)
instead of entering my local address (http://<IP_ADDRESS>:8123/) in the TRAKT application page I entered this: https://my.home-assistant.io/redirect/oauth
added the integration
accepted in TRAKT
then accepted in the my home assistant page that showed up
profit

I wonder if @dylandoamaral does not have the (default) "my" integration enabled? See: https://www.home-assistant.io/integrations/my/

Thank you for your workaround, I let the issue open to see if someone else has this problem even with your workaround. I will then add it to the README.
I don't think I have the "my" integration in both my development environment and my production one but maybe I am missing something. However in my production one I have a DNS routing maybe that's why it is working.

Thank you for your workaround, I let the issue open to see if someone else has this problem even with your workaround. I will then add it to the README.
I don't think I have the "my" integration in both my development environment and my production one but maybe I am missing something. However in my production one I have a DNS routing maybe that's why it is working.

I don't think my solution is really a workaround, because most installations (and definitely the new ones) will have the "my" integration enabled (https://analytics.home-assistant.io/#integrations shows it's enabled for 97.8% of installations that allow statistics to be collected).
Maybe you could test it out yourself with the "my" integration added @dylandoamaral ?

Thank you @robloh using https://my.home-assistant.io/ worked. I had never set this up before, but following your steps and using my local ip worked.
@dylandoamaral this solution resolved the issue for me.

YES THIS WORKS. I had same issue as the others.

It won't help since it actually works without it. I need to know if it fix the problem for people having this issue.

Thanks! The workaround works :) I was really close to trying this.. since the oauth2 error was complaining about the client url not matching the redirected one... of course it won't match! Kudos to you Rob!

I think I have enough reason to put this workaround in the readme, thank you guys !


Please allow all the params in _make_stripe_checkout_params be set in settings

Hello,
Great work here! Small request, please allow all of the parameters sent to stripe checkout session be set in settings!
Thank you

What are some of the parameters you would like to set from Django settings (https://stripe.com/docs/api/checkout/sessions/create)?

success_url path?
cancel_url path?

And the following I think would need to become an argument of the function, which are passed from the API request:

payment_method_types?
mode?

Everything under _make_stripe_checkout_params should be checked against settings imo. For me I'll be using cancel/success url and payment method types.
Btw the trial period when using the days variable actually adds 1 less day.


commitment.json not formatted for API 2

Bug description: The JSON format of the current commitment file provided by the ARR website is not formatted for API 2, employed by all commitment venues.
Suggested fix: In collaboration with the OpenReview team, I created a PR with the suggested revision.

Merged - thanks for raising and resolving this!


possibility of stream iterator implementation?

I am trying to use SDfat streams with a serializer (ThorsSerializer), which uses stream iterators to parse out strings during deserialization.
specifically here:  https://github.com/Loki-Astari/ThorsSerializer/blob/master/src/Serialize/JsonManualLexer.cpp#L157
I suppose this would also be tied to a full implementation of stdlib functions, or using a full distribution of stdlib (newlib on ARM) instead of the minimal implementation in SdFat.
With Teensy 4 and more powerful MCUs it shouldn't be as much of a problem to use full stdlib, but that is a different issue.

I can't help you. I get too many requests like this.


Tests fail to build on Windows

   Compiling glob v0.3.0 (C:\crater\workdir)
error[E0432]: unresolved import `std::ffi::AsOsStr`
    --> src\lib.rs:1081:17
     |
1081 |             use std::ffi::AsOsStr;
     |                 ^^^^^^^^^^-------
     |                 |         |
     |                 |         help: a similar name exists in the module: `OsStr`
     |                 no `AsOsStr` in `ffi`

error[E0624]: method `prefix` is private
    --> src\lib.rs:1086:33
     |
1086 |                 .and_then(|p| p.prefix().map(|p| p.join("*")))
     |                                 ^^^^^^

error: aborting due to 2 previous errors

Oops... Thanks for the report @retep998!
I've reached out to see if we've got an Azure Pipelines plan we can use here in the nursery so we can check Windows in CI too.

Alrighty, we've got a Pipelines project created so I'll go set it up so we get CI for Windows running.

CI is now set up for Windows so if anybody would like to take a shot at getting these tests to compile on Windows they should be checked properly now going forward.

What is that prefix function even supposed to do? It isn't anywhere in the documentation of PathBuf or Path.

What is that prefix function even supposed to do?
The one in the Rust source code has a prefix() -> Prefix definition, but Prefix does not contain join either.


chat-textfield disabled

the button of the chat-textfield should be marked as disabled if the textfield does not contain any content after trimming and allow-empty-submit parameter is set to false
the empty submit is already working if the parameter is set, just the ui represenation is not implemented yet

fixed in pr #1576


Why doesn't it support link custom network?

Why doesn't it support link custom network?
I want to use the app to link my test network

see https://github.com/polkawallet-io/polkawallet-flutter/issues/46


Build(deps-dev): Bump babel-loader from 9.1.2 to 9.1.3 (#4960)

Pull request opened by the merge tool on behalf of #4960

Merged in 517c4308e8d399c8dbb78895f9622d27c489d89c


Bad value for type long when using PostgreSQL database

Hi, I am using your library but I have problems with extracting JSONBs from database. The error I am getting is
Bad value for type long: (some json string)

I am using PostgreSQL 12.3 version.
I have been debuging my app and discovered that it is calling org.hibernate.type.descriptor.sql.BlobTypeDescriptor::getExtractor() method in which it returns implementation of BasicExtractor class.  Then it calls doExtract() method from BasicExtractor class in order to wrap blob from database:
return javaTypeDescriptor.wrap( rs.getBlob( name ), options );

however rs.getBlob() is trying to call toLong(fixedString()) method. But I have no idea why would it call toLong when it is string.
Here are the classes:
@Data
@Entity
@Table(name = "merchant_stores")
@DiscriminatorValue("MERCHANT_STORE")
@EntityListeners(AuditingEntityListener.class)
@TypeDef(name = "jsonb", typeClass = JsonBinaryType.class)
@NamedEntityGraphs({
    //
})
public class MerchantStore {
    @Id
    @GeneratedValue(strategy = GenerationType.IDENTITY)
    private Long id;

    // ...

    @Type(type = "jsonb")
    @Column(name = "business_hours")
    private List<BusinessHours> businessHours;

    // ...
}

@Data
public class BusinessHours implements Serializable {
    @JsonInclude
    private String openTime;

    @JsonInclude
    private String closeTime;
}

In order to investigate it, you will need to provide a replicating test case. Use any of the existing test cases as the base for your test. I'll take a look at it once it's done.

I have created fresh new project, put your library in there, created table with jsonb in PostgreSQL database, and it works, but for some reason in my current project it does not. Further debuggings show that it thinkgs that it is JsonBlobType instead of JsonBinaryType... Somehow my spring boot project ignores @TypeDefs....
I don't know how would I create test case, when it just does not work for my project, while it works in fresh new....

Then, it's definitely not a problem with this library. It's either a Spring Boot or a Hibernate issue, so you need to open the issue for those projects.

I figured it out, for some reason I had this dependency in my pom.xml faile:
<dependency>
    <groupId>com.fasterxml.jackson.module</groupId>
    <artifactId>jackson-module-jaxb-annotations</artifactId>
</dependency>

I can't remember why I put it there, but once I removed it (I also see that your library is also using this dependency) it started working...

I don't see why that dependency would cause that issue. Again, maybe it's a Hibernate issue.


php 8 compatibility

I am using php version 8.1.3 with example from documents on mysql
// Create the Connection URI
// See more: https://github.com/byjg/anydataset#connection-based-on-uri
$uriString = 'mysql://' . $dbUser . ':' . $dbPass . '@' . $dbHost . ':' . $dbPort . '/' . $dbName;
$connectionUri = new \ByJG\Util\Uri($uriString);

// Create the Migration instance
$migration = new \ByJG\DbMigration\Migration($connectionUri, './resources', false);

// Register the Database or Databases can handle that URI:
$migration->registerDatabase('mysql', \ByJG\DbMigration\Database\MySqlDatabase::class);
$migration->registerDatabase('maria', \ByJG\DbMigration\Database\MySqlDatabase::class);

$migration->reset();

I got these warnings:

PHP Deprecated:  parse_str(): Passing null to parameter #1 ($string) of type string is deprecated in /var/www/html/vendor/byjg/uri/src/Uri.php on line 118
Deprecated: http_build_query(): Passing null to parameter #2 ($numeric_prefix) of type string is deprecated in /var/www/html/vendor/byjg/uri/src/Uri.php on line 125

I tested this on 7.4 & 7.3 with no warnings

I created a PR for the project URI, where the error is thrown.
Could you validate it?
Thanks.

Those warnings are fixed, however this new one appears
PHP Deprecated:  file_exists(): Passing null to parameter #1 ($filename) of type string is deprecated in /var/www/html/vendor/byjg/migration/src/Migration.php on line 187
issue appears when migrating up or reset (with up)

All right. I merged the URI project, so you can restore your composer JSON and the next update will get the correct one.
I'll take a look at this error.

require __DIR__ . '/vendor/autoload.php';

$dbHost = 'mysql'; // name of mysql container
$dbName = 'project';
$dbPort = 3306;
$dbUser = 'user';
$dbPass = 'pass';

$uriString = 'mysql://' . $dbUser . ':' . $dbPass . '@' . $dbHost . ':' . $dbPort . '/' . $dbName;
$connectionUri = new \ByJG\Util\Uri($uriString);

$migration = new \ByJG\DbMigration\Migration($connectionUri, './resources', false);
$migration->registerDatabase('mysql', \ByJG\DbMigration\Database\MySqlDatabase::class);
$migration->registerDatabase('maria', \ByJG\DbMigration\Database\MySqlDatabase::class);

$migration->reset();

Found the issue and fixed it in branch 4.2.1.
You can change your composer to use 4.2.1.x-dev
Let me know if it fixes, so I can create a new release with this.

seems to be fixed

OK. PR merged.
Release 4.2.2 created.
I am closing this issue.


Communications guidelines

I added a communications guidelines / how to use slack page.
Added link to this page on the let's talk section of the homepage. I also removed the private message link, since it is now redundant with the old page. But the "widgets" are not centered for some reason.
Added links to the #Communications sections in the syllabuses.
Let me know if anybody wants any changes on the proposed structure or content.  Or feel free to edit directly!

This is great content, thank you! My only comment at this time is about the public link for joining Slack. For longevity, I'm starting to think the channel should be open to everyone, but then make a private channel for each iteration. I'll bring up the topic next Monday in our meeting.
I have an idea: I've been wanting to promote R groups like RLadies. To do this, I think it would be nice to have a third icon, labelled "community", with a subcaption reading something like "Connect with groups outside of STAT 545". The logo could be something like Font Awesome's users (using it should only involve plugging in icon_pack: "fas" and icon: "users" in the widget md file).
Instead of the section being labelled "Let's Talk", perhaps it can be called "Connect".
On the "Community" page, some groups could be listed off and linked. Here's content I can think of -- and would be enough to start with, I think. Please adjust as you see fit!

Interested in engaging with the R community? Here are some options. Are you part of an R group and don't see your group here? Contact us!

R Ladies: world-wide groups promoting gender diversity in the R community, holding in-person meetups. Open to everyone.
UBC's R study group: UBC-based group of passionate useRs who organize meetups on campus to share R knowledge.
Tidy Tuesday: An online group that posts regular data projects and showcases responses.
Vancouver R Users Group: A meetup.com group dedicated to teaching and sharing R.
rstudio::conf(): Annual global conference about R and data science.


Tooltip and Popover made seo-friendly

Hi,
Would it be possible to make both the tooltip and popover components seo friendly in the 4.0 version?
Content/Code ratio is one ranking factor for Google and wasting so much content in popovers or tooltips that are not parsed or read by the Google bot is a pity.
Thanks for considering it.
Fabio

dupe of https://github.com/twbs/bootstrap/issues/20364


Chrome may be installed into %AppLocalData% folder

https://stackoverflow.com/a/40674915/21950762
https://github.com/hasansezertasan/chrome-version/blob/0c4acf6faceb0114de9c09d0a91adf87a48e6132/chrome_version/core.py#L25-L35

If you navigate to "%ProgramFiles%\Google\Chrome\Application" in the Explorer, the new directory will look something like this: "C:\Program Files\Google\Chrome\Application".
Which is already supported (check out your permalink highlights).
But when I navigate to "%LocalAppData%\Google\Chrome\Application", I get an error, that such a directory doesn't exist.
Is this the new directory that the Chrome application will be installed on, in the next versions?


temperature and humidity module doesn't stream humidity right

when I plug in the kittenbot temp + humidity module, only temperature shows up in the web UI initially. double clicking on 'humidity' in the device tree fixes it.

Does this still repo? I'll need a trace, I can't find the hardware module.


.setDisplayMode Overlay

Would it be possible to make the pass produce the shadows only, rather than doing the renderPass together?  Essentially this would be like using .setDisplayMode("AO") but instead of white/black debug output we have transparent/black output onto the previous pass... you could call it something like .setDisplayMode("Overlay")
So in a project we could then have something like this:

renderPass
Some other passes and effects
N8AO ❤
Some other passes and effects

By having the option to separate it this way, we would have more flexibility with the final image.
But there is more!  By having separate passes, N8AO could use a different scene, camera or camera.layers for it's pass.  This would allow us to use different geometry/objects for N8AO, or make some objects include/exclude N8AO (e.g. if a static object has prebaked shading, it might be desirable for that to not use N8AO).
This would make N8AO the ultimate SSAO solution for everybodies three.js projects!

N8PostPass already does this... the reason N8AOPass doesn't is because it needs the depth texture in order to work... and RenderPass doesn't export that. I'm not sure how to solve this.

Okay thanks, that's great!  I was a bit confused by the postprocessing library, as I had assumed it had been integrated into Three.js (since it had EffectComposer, RenderPass, etc.)  I now realise it's different... silly me!  Again, thanks.

Yeah... I'll add a feature to pass your own depth/scene texture in the next update.


HA call restart service fail

systemctl status:
/usr/bin/python3 /usr/local/bin/hass --config /opt/homeassistant
Home Assistant release with the issue:
0.73 upgrade to 0.81.6
Last working Home Assistant release (if known):
0.73
Operating environment (Hass.io/Docker/Windows/etc.):
ubuntu 17

Component/platform:
restart service

Description of problem:
after called the restart service , the HA will not start , it must reboot in the ubuntu.
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
none

Traceback (if applicable):
root@HAserver:~# systemctl status -l<EMAIL_ADDRESS>鈼�<EMAIL_ADDRESS>- Home Assistant
   Loaded: loaded<EMAIL_ADDRESS>enabled; vendor preset: enabled)
   Active: active (running) since Thu 2018-11-22 00:14:47 CST; 11min ago
 Main PID: 783 (hass)
    Tasks: 7 (limit: 9830)
   CGroup<EMAIL_ADDRESS>           鈹斺攢783 /usr/bin/python3 /usr/local/bin/hass --config /opt/homeassistant

Nov 22 00:18:04 HAserver hass[783]: 2018-11-22 00:18:04 DEBUG (SyncWorker_0) [pyhap.accessory_driver] Stopping mDNS advertising
Nov 22 00:18:09 HAserver hass[783]: 2018-11-22 00:18:09 DEBUG (MainThread) [homeassistant.core] Bus:Handling <Event homeassistant_close[L]>
Nov 22 00:18:09 HAserver hass[783]: 2018-11-22 00:18:09 DEBUG (SyncWorker_0) [pyhap.accessory_driver] Stopping HAP server
Nov 22 00:18:10 HAserver hass[783]: 2018-11-22 00:18:10 INFO (SyncWorker_0) [pyhap.hap_server] Stopping HAP server
Nov 22 00:18:10 HAserver hass[783]: 2018-11-22 00:18:10 DEBUG (SyncWorker_0) [pyhap.accessory_driver] AccessoryDriver stopped successfully
Nov 22 00:18:10 HAserver hass[783]: 2018-11-22 00:18:10 DEBUG (SyncWorker_6) [pyhap.accessory_driver] Shutdown executers
Nov 22 00:18:10 HAserver hass[783]: 2018-11-22 00:18:10 INFO (SyncWorker_6) [pyhap.accessory_driver] Closed the event loop
Nov 22 00:18:15 HAserver hass[783]: 2018-11-22 00:18:15 INFO (SyncWorker_0) [homeassistant.config] Upgrading configuration directory from 0.81.6 to 0.73.1
Nov 22 00:18:15 HAserver hass[783]: 2018-11-22 00:18:15 INFO (SyncWorker_0) [homeassistant.config] Migrating old system configuration files to new locations
Nov 22 00:18:15 HAserver hass[783]: Config directory: /opt/homeassistant

Additional information:

Issue still present on raspberry HASS 0.87.1:
Feb 17 19:33:57 raspberrypi hass[11894]: 2019-02-17 19:33:57 INFO (Thread-31) [pyhap.hap_server] Stopping HAP server Feb 17 19:33:57 raspberrypi hass[11894]: 2019-02-17 19:33:57 INFO (Thread-15) [pyhap.accessory_driver] Closed the event loop

Could this be linked to https://github.com/home-assistant/home-assistant/issues/12508
Cheers

Please reopen if this is still an issue, thanks!


[Usage] TypeError: LlavaLlamaForCausalLM.forward() got an unexpected keyword argument 'cache_position'

Describe the issue
Issue:
Model Worker is showing this error when i chat with it.
TypeError: LlavaLlamaForCausalLM.forward() got an unexpected keyword argument 'cache_position'
Log:
2024-03-04 08:41:32 | ERROR | stderr | Exception in thread Thread-3 (generate):
2024-03-04 08:41:32 | ERROR | stderr | Traceback (most recent call last):
2024-03-04 08:41:32 | ERROR | stderr |   File "/home/ec2-user/anaconda3/envs/llava/lib/python3.10/threading.py", line 1016, in _bootstrap_inner
2024-03-04 08:41:32 | ERROR | stderr |     self.run()
2024-03-04 08:41:32 | ERROR | stderr |   File "/home/ec2-user/anaconda3/envs/llava/lib/python3.10/threading.py", line 953, in run
2024-03-04 08:41:32 | ERROR | stderr |     self._target(*self._args, **self._kwargs)
2024-03-04 08:41:32 | ERROR | stderr |   File "/home/ec2-user/anaconda3/envs/llava/lib/python3.10/site-packages/torch/utils/_contextlib.py", line 115, in decorate_context
2024-03-04 08:41:32 | ERROR | stderr |     return func(*args, **kwargs)
2024-03-04 08:41:32 | ERROR | stderr |   File "/home/ec2-user/LLaVA/llava/model/language_model/llava_llama.py", line 138, in generate
2024-03-04 08:41:32 | ERROR | stderr |     return super().generate(
2024-03-04 08:41:32 | ERROR | stderr |   File "/home/ec2-user/anaconda3/envs/llava/lib/python3.10/site-packages/torch/utils/_contextlib.py", line 115, in decorate_context
2024-03-04 08:41:32 | ERROR | stderr |     return func(*args, **kwargs)
2024-03-04 08:41:32 | ERROR | stderr |   File "/home/ec2-user/anaconda3/envs/llava/lib/python3.10/site-packages/transformers/generation/utils.py", line 1592, in generate
2024-03-04 08:41:32 | ERROR | stderr |     return self.sample(
2024-03-04 08:41:32 | ERROR | stderr |   File "/home/ec2-user/anaconda3/envs/llava/lib/python3.10/site-packages/transformers/generation/utils.py", line 2696, in sample
2024-03-04 08:41:32 | ERROR | stderr |     outputs = self(
2024-03-04 08:41:32 | ERROR | stderr |   File "/home/ec2-user/anaconda3/envs/llava/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1518, in _wrapped_call_impl
2024-03-04 08:41:32 | ERROR | stderr |     return self._call_impl(*args, **kwargs)
2024-03-04 08:41:32 | ERROR | stderr |   File "/home/ec2-user/anaconda3/envs/llava/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1527, in _call_impl
2024-03-04 08:41:32 | ERROR | stderr |     return forward_call(*args, **kwargs)
2024-03-04 08:41:32 | ERROR | stderr |   File "/home/ec2-user/anaconda3/envs/llava/lib/python3.10/site-packages/accelerate/hooks.py", line 165, in new_forward
2024-03-04 08:41:32 | ERROR | stderr |     output = old_forward(*args, **kwargs)
2024-03-04 08:41:32 | ERROR | stderr | TypeError: LlavaLlamaForCausalLM.forward() got an unexpected keyword argument 'cache_position'
Screenshots:

I encountered the same error while running "eval_mode".

eval_model(args)
You are using a model of type llava to instantiate a model of type llava_llama. This is not supported for all configurations of models and can yield errors.
Loading checkpoint shards: 100%|███████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████| 2/2 [00:04<00:00,  2.27s/it]
/root/anaconda3/envs/llava/lib/python3.10/site-packages/transformers/generation/configuration_utils.py:410: UserWarning: do_sample is set to False. However, temperature is set to 0 -- this flag is only used in sample-based generation modes. You should set do_sample=True or unset temperature.
warnings.warn(
Traceback (most recent call last):
File "", line 1, in
File "/home/humaodi/code/LLaVA/llava/eval/run_llava.py", line 115, in eval_model
output_ids = model.generate(
File "/root/anaconda3/envs/llava/lib/python3.10/site-packages/torch/utils/_contextlib.py", line 115, in decorate_context
return func(*args, **kwargs)
File "/home/humaodi/code/LLaVA/llava/model/language_model/llava_llama.py", line 137, in generate
return super().generate(
File "/root/anaconda3/envs/llava/lib/python3.10/site-packages/torch/utils/_contextlib.py", line 115, in decorate_context
return func(*args, **kwargs)
File "/root/anaconda3/envs/llava/lib/python3.10/site-packages/transformers/generation/utils.py", line 1544, in generate
return self.greedy_search(
File "/root/anaconda3/envs/llava/lib/python3.10/site-packages/transformers/generation/utils.py", line 2404, in greedy_search
outputs = self(
File "/root/anaconda3/envs/llava/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1518, in _wrapped_call_impl
return self._call_impl(*args, **kwargs)
File "/root/anaconda3/envs/llava/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1527, in _call_impl
return forward_call(*args, **kwargs)
TypeError: LlavaLlamaForCausalLM.forward() got an unexpected keyword argument 'cache_position'

I encountered the same error too. Any ideas?

Got this error as well. Not been able to fix yet. Tracking this Issue.

I had the same issue. Fixed it by ensuring the transformers version to be same as the one mentioned in pyproject.toml, i.e. transformers==4.37.2

I had the same issue. Fixed it by ensuring the transformers version to be same as the one mentioned in pyproject.toml, i.e. transformers==4.37.2

Problem solved. Thanks!

Problem solved. Thanks!

this was because transformers 4.38.0 added static cache. So have to use any version below.

您的来信已收到，祝您每天有个好心情。

https://github.com/haotian-liu/LLaVA/issues/1218#issuecomment-1977497811
Thank you so much, my problem was solved via degrading transformers==4.37.2.The transformers package interfaces changes frequently, which makes me so confused that I have to spend too much time to debug the meaningless bugs.

I had the same issue. Fixed it by ensuring the transformers version to be same as the one mentioned in pyproject.toml, i.e. transformers==4.37.2

TY <3

I had the same issue. Fixed it by ensuring the transformers version to be same as the one mentioned in pyproject.toml, i.e. transformers==4.37.2

thank you!!!!!!

new models use transformers>4.39 is there a way to actually fix this ?

您的来信已收到，祝您每天有个好心情。

TypeError: LlavaLlamaForCausalLM.forward() got an unexpected keyword argument 'cache_position'
my transformers version is 4.37.2，also have this problem.

I still got this error after installing transformers==4.37.2
TypeError: llama_model_forward_4_36() got an unexpected keyword argument 'cache_position'

Hey, adding cache_position=None to the forward method also works. Check here

您的来信已收到，祝您每天有个好心情。


Add swap file to Python.gitignore

Reasons for making this change:
Many a times a user adds and commits the temporary os generated files like the ones with *.swp extension which becomes a little irritating. I'm not sure if anyone really wants to commit these, just like the pyc files. These are the files generated by vim, so the users shall be benefited.
Links to documentation supporting these rule changes:
https://unix.stackexchange.com/questions/27923/what-causes-swap-files-to-be-created - Link to a solution on stack overflow.

This rule is part of https://github.com/github/gitignore/blob/master/Global/Vim.gitignore ; see https://github.com/github/gitignore#folder-structure


LSP: Buffer decorations before sending them to the IDE.

This is on PR #93.
On a larger document, the proof states are updated multiple times, and each such update causes the decorations to be sent to the IDE. It gets overloaded sometimes. The updates can be sent with some delay to avoid sending many intermediate updates.
Also, maybe line ranges can be considered when sending the decorations. I'm not sure if there is enough information for that.

It was resolved in PR #93.


Corrected mistake in table of symbols

Was "GlobalNavButton" - the correct enum is "GlobalNavigationButton".

@mijacobs : PR passed acceptance checklist. Please evaluate further and approve/merge if appropriate. Thanks!


properly stop MessageDispatcher if Subscriber fails

When PollingSubscriberConnection fails, it calls notifyFailed.

Stopping connections is implemented with stopAsync.
The problem is with the connection that originally failed.
Since it has already entered the FAILED state, it cannot be stopped
by stopAsync.
Consequently, this leaks a MessageDispatcher.
This commit makes the connection explicitly stop itself before notifying
the Subscriber that it has failed.
cc @davidtorres

Changes Unknown when pulling b42d9f42a58bcb12f3fc353febcc1a4e9d4b4c26 on pongad:subscriber-stop into ** on GoogleCloudPlatform:master**.

LGTM


Multiple students in different courses is not handled currently

The pilot courses don't seems to have students in multiple courses but the user table that is created as such  you cannot add a same user with different course. User table PK needs to be based of user_id and course_id.

Test Plan: This changes Primary Key(PK) for the user table. Making a new column as PK should not impact the views at all

Make sure Files/assignment/grades personalized and non-personalized view is loading correctly.
Testing this is more or less like a regression testing.

As far as I can tell this is working.  I see the correct view of the screens when I'm logged in as myself (non-personalized) and when I masquerade as a student in SI 110.
The only view I can't test is the grade distribution view - we don't have a student from the SI 664 or HMP 654 to test with in dev.


404 not found event if the routes are declared and executed correctly.

Issue #909 introduced a side effect. In my case, some APIs do not need to return a body. An http status 200 is enough. To cope with that, I set the flag externalContainer to true. My routes just return null in that case. Then I just decided to upgrade to 2.7.0. Bad idea! All those APIs return a 404 even if they have executed correctly. Logs say : The requested route [{}] has not been mapped in Spark for {}.
Is there a different way to ignore a null body other than returning an empty string?
Thank you.

Yes, sure. At the end it is the same, I have to return an empty string. And it seems the only issue.
The thing is, the behavior has changed completely.
The error message says 404, the route has not been mapped in Spark. However, the route is correctly mapped and executed correctly.
Some applications are full rest applications. They do not serve any page. And they do not need to return a body.
I would like to propose a fix, but as I read on the Sparkjava page, 2.7.x is probably the last release before the 3.0.0. And I do not see any branch for that. (I hope it is not the end, sniff)

I experience similar issue. I mapped:
post("/abc", ...);
put("/abc", ...);
get("/abc", ...);

POST and GET works fine, but PUT returns 404.
However the execution is done correctly, I can see the modified data again via GET.
In logs:
... The requested route [/abc] has not been mapped in Spark for Accept: [*/*]

Hello, I want to try to solve this problem


Remove some submodules in favour of opam pins

Get rid of three submodules, replacing them with opam pins.
Submodules slow down the checkout, and are often annoying when switching branches. Replace them with opam pins.

We had async_kernel and rpc_parallel pinned because we had local changes to those libraries.
If we're using stock versions of those libraries, we don't need to pin them, just as we don't pin most other OPAM packages.
Are we discarding those local changes?

If we're using stock versions of those libraries, we don't need to pin them, just as we don't pin most other OPAM packages.
Are we discarding those local changes?

They're being pinned from our github repos rather than submodules: https://github.com/MinaProtocol/mina/pull/11220/files#diff-4e59d97a11c5f0a15ddcb8dac12467e33c0a2730c176efe8969358b7fd566a28R376.
It makes them more annoying to update, but should reduce friction for everyone else when there are updates (since it's just an opam switch import with no submodule pins).

They're being pinned from our github repos rather than submodules: #11220 (files).

Ah, thanks for clarifying.

No changes in 14 days, @balsoft ? Does this mean that you are not working on this PR? Perhaps it should be marked as a draft, if so.

Companion: #11388


fix clickable nodes style in mermaid

Hi @matcornic,
Thanks again for this awesome theme!
As per the referenced https://github.com/matcornic/hugo-theme-learn/issues/168, we can create clickable nodes in mermaid as per the demo site, but the cursor is not being styled correctly. This sorts that out.

add CSS ruleset for clickable nodes
replace JS with latest minified from mermaidjs.github.io

Any idea why the styling changes on the 1st example in the exampleSite after applying this patch?

Removed update to mermaid.js from this PR


Use OpenSSL ClientHello callback

Would it be possible to implement this using https://www.openssl.org/docs/manmaster/man3/SSL_CTX_set_client_hello_cb.html from recently released version of OpenSSL instead of patching it?
That should simplify maintenance quite a bit.

Originally implementation it is, see https://github.com/phuslu/nginx-ssl-fingerprint/tree/9b4ad1a4c9364a075113f1206a9db25cdffeda78
Also we upstream the PR https://github.com/openssl/openssl/pull/16910 and got merged.
But for the real production env, we need to be more performant, so here we are.

Thanks for clarification!


[tool_crash] FileSystemException: Cannot create link, OS Error: Función incorrecta., errno = 1

Command
flutter pub get

Steps to Reproduce

...
...
...

Logs
FileSystemException: Cannot create link, OS Error: Función incorrecta.
, errno = 1
#0      _Link.throwIfError (dart:io/link.dart:254:7)
#1      _Link.createSync (dart:io/link.dart:182:5)
#2      ForwardingLink.createSync (package:file/src/forwarding/forwarding_link.dart:20:16)
#3      ForwardingLink.createSync (package:file/src/forwarding/forwarding_link.dart:20:16)
#4      _createPlatformPluginSymlinks (package:flutter_tools/src/flutter_plugins.dart:1057:12)
#5      createPluginSymlinks (package:flutter_tools/src/flutter_plugins.dart:987:5)
#6      refreshPluginsList (package:flutter_tools/src/flutter_plugins.dart:1089:5)
<asynchronous suspension>
#7      FlutterProject.ensureReadyForPlatformSpecificTooling (package:flutter_tools/src/project.dart:357:5)
<asynchronous suspension>
#8      PackagesGetCommand.runCommand (package:flutter_tools/src/commands/packages.dart:179:5)
<asynchronous suspension>
#9      FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:1257:27)
<asynchronous suspension>
#10     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
<asynchronous suspension>
#11     CommandRunner.runCommand (package:args/command_runner.dart:209:13)
<asynchronous suspension>
#12     FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:283:9)
<asynchronous suspension>
#13     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
<asynchronous suspension>
#14     FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:229:5)
<asynchronous suspension>
#15     run.<anonymous closure>.<anonymous closure> (package:flutter_tools/runner.dart:64:9)

[✓] Flutter (Channel stable, 3.7.8, on Microsoft Windows [Versi¢n 10.0.22621.1105], locale es-MX)
    • Flutter version 3.7.8 on channel stable at E:\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 90c64ed42b (6 days ago), 2023-03-21 11:27:08 -0500
    • Engine revision 9aa7816315
    • Dart version 2.19.5
    • DevTools version 2.20.1

[✗] Windows Version (Unable to confirm if installed Windows version is 10 or greater)

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.2)
    • Android SDK at E:\android-sdk-windows-1.6_r1\android-sdk-windows-1.6_r1
    • Platform android-33, build-tools 33.0.2
    • Java binary at: C:\Program Files\Java\jdk-11.0.17\bin\java
    • Java version Java(TM) SE Runtime Environment 18.9 (build 11.0.17+10-LTS-269)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at C:\Program Files (x86)\Google\Chrome\Application\chrome.exe

[✓] Visual Studio - develop for Windows (Visual Studio Build Tools 2022 17.4.4)
    • Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2022\BuildTools
    • Visual Studio Build Tools 2022 version 17.4.33213.308
    • Windows 10 SDK version 10.0.22000.0

[✓] Android Studio (version 2022.1)
    • Android Studio at E:\Android Studio
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.15+0-b2043.56-9505619)

[!] Android Studio
    • Android Studio at E:\android-sdk-windows-1.6_r1\android-sdk-windows-1.6_r1
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    ✗ android-studio-dir = E:\android-sdk-windows-1.6_r1\android-sdk-windows-1.6_r1
    ✗ Unable to find bundled Java version.
    • Try updating or re-installing Android Studio.
    • Consider removing your android-studio-dir setting by running:
      flutter config --android-studio-dir=

[✓] Connected device (4 available)
    • CPH2127 (mobile)  • b3eb9168 • android-arm64  • Android 12 (API 31)
    • Windows (desktop) • windows  • windows-x64    • Microsoft Windows [Versi¢n 10.0.22621.1105]
    • Chrome (web)      • chrome   • web-javascript • Google Chrome 111.0.5563.111
    • Edge (web)        • edge     • web-javascript • Microsoft Edge 111.0.1661.54

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 2 categories.

Flutter Application Metadata
Type: app
Version: 1.0.0+1
Material: true
Android X: false
Module: false
Plugin: false
Android package: null
iOS bundle identifier: null
Creation channel: stable
Creation framework version: 90c64ed42ba53a52d18f0cb3b17666c8662ed2a0
Plugins
facebook_auth_desktop-0.0.9
firebase_auth-4.3.0
firebase_auth_web-5.2.10
firebase_core-2.8.0
firebase_core_web-2.2.2
flutter_facebook_auth-5.0.8
flutter_facebook_auth_web-4.1.1
flutter_native_splash-2.2.19
flutter_plugin_android_lifecycle-2.0.9
flutter_secure_storage-8.0.0
flutter_secure_storage_linux-1.1.3
flutter_secure_storage_macos-3.0.0
flutter_secure_storage_web-1.1.1
flutter_secure_storage_windows-2.0.0
google_sign_in-6.0.2
google_sign_in_android-6.1.8
google_sign_in_ios-5.6.1
google_sign_in_web-0.11.0+2
image_cropper-3.0.1
image_cropper_for_web-1.0.3
image_picker-0.8.7
image_picker_android-0.8.6+6
image_picker_for_web-2.1.12
image_picker_ios-0.8.7+1

Closing as duplicate of https://github.com/flutter/flutter/issues/66224


Cannot delete the wifi network configuration saved by Wi-Fi provisioning service

Describe the question
Cannot delete the wifi network configuration saved by Wi-Fi provisioning service (remove the saved wifi AP configuration)?
System information

Version: 202007.00 release

Question
I have configured a wifi network by following the wifi provisioning service steps, and then demo starts work. But I do not find how to delete the wifi network I saved.
After reboot the board, I tried to delete the saved wifi network by swiping the saved wifi item to the left to delete wifi information by repeating the wifi provisioning steps on my cellphone application.
But the error raised,
16 17628 [XXXXX] [ERROR][BLE_WIFI_PROV][lu] Failed to process the delete network request. Already a request is being processed
So, my question is how can I delete the saved wifi network configuration?  Whether the saved wifi netwoork can only be cleared by restoring factory settings?
And I noted that there is a change 893f09ee40f0f6ec46d0763dcb4e4c577b1405ba, the BLE connection will be removed once the wifi AP connected. So, in the wifi provisioning sequence, after the wifi AP is set, the target board will disconnect from the cellphone. As a result, the saved wifi network information cannot be displayed on the cellphone application. However, before this change, in the wifi provisioning phase, the saved network information will be displayed in the cellphone application. Is this behavior normal?
Thank you!

@lylezhu2012
Demo sample provided, optimizes power usage for the device, so it disables BLE network once the WIFI is connected and application proceeds on WiFi. You can choose to have BLE  always enabled by removing lines 135-141 and lines 165-171 from iot_demo_freertos.c
The saved network can be deleted by sliding the connected network towards left. The error above seems to occur because a list network request or an save network request is ongoing, as the WiFi driver can only process one request. You can retry from the app to send the request again.
Also note that the demos completes , and it shuts down all the networks as part of cleanup. You may want to wait in the demo if you need to try all the features from app.

Hi @ravibhagavandas,

@lylezhu2012
Demo sample provided, optimizes power usage for the device, so it disables BLE network once the WIFI is connected and application proceeds on WiFi. You can choose to have BLE always enabled by removing lines 135-141 and lines 165-171 from iot_demo_freertos.c

Could you add a configuration for the changes? So, we can configure the feature in application layer.

The saved network can be deleted by sliding the connected network towards left. The error above seems to occur because a list network request or an save network request is ongoing, as the WiFi driver can only process one request. You can retry from the app to send the request again.
Also note that the demos completes , and it shuts down all the networks as part of cleanup. You may want to wait in the demo if you need to try all the features from app.

@lylezhu2012
Thanks for your suggestion. This seems to be more an application defined behavior which can vary across each applications. We will look into how best we can incorporate this into our demo sample.

@lylezhu2012
I am closing this issue. Please feel free to reopen this if you have any further questions.


callback architecture

rewrite gush commands in callback form to allow other commands be written outside of the phar into a common folder and allow them to have their own dependencies. In other words, Gush be like drupal and the customized commands be like modules. Though drupal is probably a bad example. cc @sstok @stephpy @pierredup any comments?

closed we will rewrite gush with console https://github.com/puli/cli from @webmozart


Updating caption

This PR updates the copy of my slide because it was out of date
This PR resolves #3
ToDo:

[ ] merge this
[ ] remove the branch

Changes previously approved.  Closing PR


When using the new version of spring cloud gateway mvc, an exception occurs when the request address contains spaces

Describe the bug
Exception occurs when the request address contains spaces: Invalid character ' ' for QUERY_PARAM in "2023-12-11 00:00:00"

Before this, I was using the webflux gateway and this problem did not occur.
Details
spring boot 3.2.0
spring cloud 2023.0.0
spring cloud starter gateway mvc 4.1.0
request url: http://localhost:8080/enterprise-service/api/v3/enterprise?start_created_time=2023-12-11 00:00:00&end_created_time=2023-12-12 00:00:00
I hope your team can help us analyze the possible causes and provide some solutions as soon as possible. Thank you very much!

In the future, please paste text rather than screenshots.
How are you accessing that URL? Through a browser it automatically URL encodes it.

@spencergibb
I tried using the browser and Apifox but got the same error.
URL： http://localhost:8080/enterprise-service/api/v3/enterprise?start_created_time=2023-12-11 00%3A00%3A00&end_created_time=2023-12-12 00%3A00%3A00

Replacing spaces   with a + things go through. I'm just not sure it's the responsibility of the gateway to do that.
Maybe @rstoyanchev might have an opinion?

Hi, we're facing issues with all encoded characters in query parameters. the norwegian 'æåå', among others. I think this may be the same issue.

@spencergibb  @rstoyanchev
When I use Webflux-based spring-cloud-gateway, I have no problem including space characters in my requests, Therefore, I think the logic of spring-cloud-gateway-mvc processing should be consistent with the behavior of spring-cloud-gateway-webflux

I believe this may be related to https://github.com/spring-cloud/spring-cloud-gateway/issues/3082 in that for some requests FormFilter will create a request body containing url decoded parameters. If those decoded parameteres are used then in https://github.com/spring-cloud/spring-cloud-gateway/blob/fb3b782afc40db9c220667ff8cc52d4968075099/spring-cloud-gateway-server-mvc/src/main/java/org/springframework/cloud/gateway/server/mvc/handler/ProxyExchangeHandlerFunction.java#L69-L77 in replaceQueryParams then HierarchicalUriComponents.verify ends in exception.

@tomgag
There has been no effective official reply so far. How did you solve this problem based on the current version?
Please help provide some solution ideas, thank you very much!

@tomgag
There has been no effective official reply so far. How did you solve this problem based on the current version?
Please help provide some solution ideas, thank you very much!

By holding our change. We own both sides, so might rewrite this to a POST.

@tomgag
There has been no effective official reply so far. How did you solve this problem based on the current version?
Please help provide some solution ideas, thank you very much!

By holding our change. We own both sides, so might rewrite this to a POST.

To rewrite the GET request into POST, we need to do a lot of adaptation work, and it does not conform to the style of restful api, so we hope there are other better ways to handle it.

@spencergibb @rstoyanchev @spring-cloud-issues
I think the core cause of this issue is the inconsistent behavior of the queryParam passed in the image below:

I hope the official can fix this problem as soon as possible. There has been no progress for three months.

Same issue with accentuated letters.
java.lang.IllegalArgumentException: Invalid character 'é' for QUERY_PARAM in "Veszprém_HU" 	at org.springframework.web.util.HierarchicalUriComponents.verifyUriComponent(HierarchicalUriComponents.java:422) ~[spring-web-6.1.3.jar:6.1.3] 	at org.springframework.web.util.HierarchicalUriComponents.lambda$verify$4(HierarchicalUriComponents.java:390) ~[spring-web-6.1.3.jar:6.1.3] 	at org.springframework.util.UnmodifiableMultiValueMap.lambda$forEach$0(UnmodifiableMultiValueMap.java:114) ~[spring-core-6.1.3.jar:6.1.3]


Fixed code formatting in demo readme

Coverage increased (+0.01%) to 72.465% when pulling 8614ae4e67c334cef71d98c29009ff64e4c7de49 on glikson:patch-1 into f1052d233531740ad375d74e3487943908936021 on cmu-db:master.


A page for List of Publishers(HTML)

In this web page, the user will able to see a list of publishers name.

Commit: 5a4a82858a031f66366523581fbfef79a4824386


Create printing module

This PR will create macros we can use to easily print things and control how things are printed, it is in draft state :)

It also introduces a new config item + CLI argument for api_mode which when turned on does not print to the terminal.
My thinking is that people would want to turn human checker off, but still see the output being printed
This will also make our API very friendly, as currently it prints to the terminal which is a bit cringe 😬

A big ToDo is that we should include all of our prints in verbose mode too

ToDo: I want to get across how fast "84 times" is. I can write some code that calculates how many decodings / second it did, and then we can compare to.. Ciphey? CyberChef?
Also funny bug, the timeout is in u32 which does not support floating point numbers. That means when Ares decodes something in less than a second it has to round up to 1 second 😂

Force merging because there is no longer any blocking comments and:

Anyway, since this PR touches a lot I am keen to merge it now and fix the smaller details later. It will block other people since almost every aspect of the program is touched 🙈


Development Plan

@fondsonlu you can just create a PR and link to this issue once you've finished converting our google docs to latex


ENH: stats: use Boost implementation of skewnorm cdf/ppf

@mckib2 Boost's skewnorm distribution accepts location and scale. What is the best way to handle that in _info.py? a is really the third argument in Boost's implementation.

What is the best way to handle that in _info.py? a is really the third argument in Boost's implementation.

The codegen as is assumes (incorrectly) that the positional arguments will align from boost -> scipy.  This doesn't seem like too much of an issue though, as you can simply reorder the arguments in the rv_continuous/discrete methods when they get called

The other arguments aren't used, so I just gave them names that never get used in the code.


Fixed race between compact (gc, populate) and head append causing unknown symbol error.

Fixes https://github.com/prometheus/prometheus/issues/7373
Hopefully improved version of https://github.com/prometheus/prometheus/pull/7556
Why https://github.com/prometheus/prometheus/pull/7556 was not working is explained here: https://github.com/prometheus/prometheus/pull/7559
@codesome feel free to use this branch to test yourself things. It would be nice to add some test, but it's kind of impossible unless we rewrite things allowing better race mocks.
Signed-off-by: Bartlomiej Plotka<EMAIL_ADDRESS>

Changes make sense to me, but could use another eye as I haven't worked with this part of the code in a long time.

Thanks for all your efforts here.
FYI: It would be great to get this in for the upcoming 2.20 RC.

Addressed comments, let's take final look (:

Merged cc @beorn7

\o/ Thanks everyone!


Behavior of config variable ignore_closed_lid is inverted

Current behavior is when ignore_closed_lid is true, the lid state is checked and when it's false, the lid state is ignored. The expected behavior is when an "ignore" variable is true, the thing gets ignored.
FIX:
change line 30 of pam.py to if not config.getboolean("core", "ignore_closed_lid"):
OR
change all occurrences of ignore_closed_lid to sense_closed_lid

Yeah this was discussed at the time of implementing, and is caused by the prior naming of ignore_ssh. Both of them were named from the perspective of PAM, so whether PAM should ignore (skip) the module if the lid is closed. I agree it's not a very intuitive way to name them, i'm open for suggestions. Personally, i don't think "sense_closed_lid" would be ideal either, as it doesn't just sense the state, it stops execution

Another option would be to rename it to ignore_when_lid_closed or cancel_when_lid_closed


[Support] Update API version used by Azure CLI for Microsoft.Support from 2020-04-01 to 2024-04-01

Preconditions

[X] No need to upgrade Python SDK or the Python SDK is ready.

Related command

Old command group -> az support tickets
New command group -> az support in-subscription tickets and az support no-subscription tickets

Old command group -> az support tickets communications
New command group -> az support in-subscription communication and az support no-subscription communication

Resource Provider
Microsoft.Support
Description of Feature or Work Requested
Update API version used by AZCLI.
Minimum API Version Required
2024-04-01
Swagger PR link / SDK link
https://github.com/Azure/azure-rest-api-specs/pull/28068
Request Example
No response
Target Date
ASAP
PM Contact
anaghranjan
Engineer Contact
bhshah
Additional context
This is a breaking change. We need to update our API version for Azure CLI to version 2024-04-01. This provides added functionality and the breaking changes listed below
List of breaking changes:

Old command groups mentioned above will be replaced with new command groups with added functionality to pick subscription account or no subscription account for tickets and communication operations.
Parameter "filters" used for list operations will be replaced with "filter"
Parameter "quota-change-regions" and "quota-change-payload" for tickets create operation will now be replaced as "quota-change-requests[].region" and  "quota-change-requests[].payload" respectively
Parameter "partner_tenant_id" for tickets create operation will now be removed

Thank you for opening this issue, we will look into it.

Hi @yonzhan, can we assign the breaking change label as well? I did not see that option while creating the issue, so I picked the feature request template