Project Feedback!

:+1: Nice work overall. A few notes after checking out the code:

Consider working more on UI polish
Good to see you were able to format the timestamp cleanly (consider using the short format - 8m, 7h, etc.)
Nice touch with the character count being displayed while the user composes a tweet
Nice to see you used SwipeRefreshLayout library for easy timeline refreshes
Good to see you added the new tweet directly to the home timeline after it was successfully posted to Twitter.
Consider adding local persistence of tweets by using DBFlow (not file) in the Tweet and User. See the persistence guide and this other guide for more details.
Consider using Parceler instead of directly using Parcelable to reduce boilerplate and conveniently read/write objects to parcel.
Consider adding progress bars to your app whenever a network request goes out

Here's a detailed Project 3 Feedback Guide here which covers the most common issues with this submitted project. Read through the feedback guide point-by-point to determine how you could improve your submission.
Let us know if you have any other thoughts or questions about this assignment. Hopefully by now you feel pretty comfortable with all the major pieces to basic Android apps (Views, Controllers, ActionBar, Navigation, Models, Authentication, API Communication, Persistence, et al) and see how they all fit together. We are close now to a turning point in the course where you should be hitting a "critical mass" towards your knowledge of Android.

:+1: Nice work. A few notes after checking out the code:

Nicely done architecting the fragments (base class and extension classes) such that you put as much code and logic as possible in the base class and then used abstract methods to define only the specific behavior where they differ
Good to see you properly communicated the user to the user timeline fragment from the profile activity.
Nice to see you used the ViewPager to properly setup the tabs
Considering this is 4th assignment and extension on top of last week's assignment, more UI polish would have been good.
You have some code duplication in TwitterClient in the methods for getHomeTimeline, getMentionsTimeline, getUserTimeline. In few other places, long methods can be broken down into smaller method. That makes the code more readable and maintainable.
Consider better organizing your source files
Try to implement more optional features so that you will get to play with more widgets and understand complexities involved

Here's a detailed Project 4 Feedback Guide here which covers the most common issues with this submitted project. Read through the feedback guide point-by-point to determine how you might be able to improve your submission.
This week (Week 5), we are going to cover the last major piece to the Android puzzle and that is using the hardware and SDK components such as the camera, photo gallery, location, maps, etc. After that, Week 6 and week 7 we will be covering a few important intermediate topics such as more about styling and animation as well as testing.
Following the bootcamp, we are going to have a public demo day to celebrate the progress you've all made with our next batch of Android students and multiple companies attending to see the group projects that you all have built. We are going to help however we can over the next few weeks to get the team project apps in shape for that.


Add instruction to fetch latest azure podvm image tag

Changes can be found here
https://deploy-preview-20--frolicking-manatee-96c0c8.netlify.app/docs/cloud-api-adaptor/azure/azure-advanced/


AttributeError: 'HorizontalAdjacency' object has no attribute 'reset'

Hi,
I came across this error when I run the example after a fresh installation from source:
...
WARNING:root:Instance identifier 'root_handle' should be unique for adding to this registry mapping! Existing root_handle: None
2023-03-03 12:31:54 [35,904ms] [Error] [omni.physx.plugin] Particles feature is only supported on GPU. Please enable GPU dynamics flag in Property/Scene of physics scene!
2023-03-03 12:31:54 [35,904ms] [Error] [omni.physx.plugin] Particles feature is only supported on GPU. Please enable GPU dynamics flag in Property/Scene of physics scene!
2023-03-03 12:31:54 [35,904ms] [Error] [omni.physx.plugin] Particles feature is only supported on GPU. Please enable GPU dynamics flag in Property/Scene of physics scene!
2023-03-03 12:31:54 [35,904ms] [Error] [omni.physx.plugin] Particles feature is only supported on GPU. Please enable GPU dynamics flag in Property/Scene of physics scene!
2023-03-03 12:31:54 [35,951ms] [Warning] [omni.isaac.dynamic_control.plugin] Failed to find articulation at '/World/DustSystem/Dust_template'
2023-03-03 12:31:54 [35,952ms] [Warning] [omni.isaac.dynamic_control.plugin] Failed to find articulation at '/World/StainSystem/Stain_template'
2023-03-03 12:31:54 [35,952ms] [Warning] [omni.isaac.dynamic_control.plugin] Failed to find articulation at '/World/ceilings_ayfjzo_0'
Traceback (most recent call last):
  File "/home/user/anaconda3/envs/omnigibson/lib/python3.7/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/home/user/anaconda3/envs/omnigibson/lib/python3.7/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/data/omniverse/lib/isaac_sim-2022.2.0/OmniGibson/omnigibson/examples/scenes/scene_selector.py", line 81, in <module>
    main()
  File "/data/omniverse/lib/isaac_sim-2022.2.0/OmniGibson/omnigibson/examples/scenes/scene_selector.py", line 58, in main
    env = og.Environment(configs=cfg)
  File "/data/omniverse/lib/isaac_sim-2022.2.0/OmniGibson/omnigibson/utils/python_utils.py", line 89, in wrapper
    func(*values.args, **values.kwargs)
  File "/data/omniverse/lib/isaac_sim-2022.2.0/OmniGibson/omnigibson/envs/env_base.py", line 70, in __init__
    self.load()
  File "/data/omniverse/lib/isaac_sim-2022.2.0/OmniGibson/omnigibson/envs/env_base.py", line 276, in load
    self._load_scene()
  File "/data/omniverse/lib/isaac_sim-2022.2.0/OmniGibson/omnigibson/envs/env_base.py", line 198, in _load_scene
    og.sim.import_scene(scene)
  File "/data/omniverse/lib/isaac_sim-2022.2.0/OmniGibson/omnigibson/simulator.py", line 300, in import_scene
    self.play()
  File "/data/omniverse/lib/isaac_sim-2022.2.0/OmniGibson/omnigibson/simulator.py", line 506, in play
    obj.initialize()
  File "/data/omniverse/lib/isaac_sim-2022.2.0/OmniGibson/omnigibson/prims/prim_base.py", line 79, in initialize
    self._initialize()
  File "/data/omniverse/lib/isaac_sim-2022.2.0/OmniGibson/omnigibson/objects/dataset_object.py", line 206, in _initialize
    super()._initialize()
  File "/data/omniverse/lib/isaac_sim-2022.2.0/OmniGibson/omnigibson/objects/stateful_object.py", line 142, in _initialize
    state.initialize(self._simulator)
  File "/data/omniverse/lib/isaac_sim-2022.2.0/OmniGibson/omnigibson/object_states/object_state_base.py", line 89, in initialize
    self.reset()
AttributeError: 'HorizontalAdjacency' object has no attribute 'reset'
Exception ignored in: <function _make_registry.<locals>._Registry.__del__ at 0x7f76f66dab00>
Traceback (most recent call last):
  File "/data/omniverse/lib/isaac_sim-2022.2.0/kit/extscore/omni.kit.viewport.registry/omni/kit/viewport/registry/registry.py", line 103, in __del__
  File "/data/omniverse/lib/isaac_sim-2022.2.0/kit/extscore/omni.kit.viewport.registry/omni/kit/viewport/registry/registry.py", line 98, in destroy
TypeError: 'NoneType' object is not callable
Exception ignored in: <function _make_registry.<locals>._Registry.__del__ at 0x7f76f66dab00>
Traceback (most recent call last):
  File "/data/omniverse/lib/isaac_sim-2022.2.0/kit/extscore/omni.kit.viewport.registry/omni/kit/viewport/registry/registry.py", line 103, in __del__
  File "/data/omniverse/lib/isaac_sim-2022.2.0/kit/extscore/omni.kit.viewport.registry/omni/kit/viewport/registry/registry.py", line 98, in destroy
TypeError: 'NoneType' object is not callable
Exception ignored in: <function SettingChangeSubscription.__del__ at 0x7f78f4b95f80>
Traceback (most recent call last):
  File "/data/omniverse/lib/isaac_sim-2022.2.0/kit/kernel/py/omni/kit/app/_impl/__init__.py", line 114, in __del__
AttributeError: 'NoneType' object has no attribute 'get_settings'
Exception ignored in: <function RegisteredActions.__del__ at 0x7f6a549f6e60>
Traceback (most recent call last):
  File "/data/omniverse/lib/isaac_sim-2022.2.0/extscache/omni.kit.viewport.menubar.lighting-104.0.8/omni/kit/viewport/menubar/lighting/actions.py", line 345, in __del__
  File "/data/omniverse/lib/isaac_sim-2022.2.0/extscache/omni.kit.viewport.menubar.lighting-104.0.8/omni/kit/viewport/menubar/lighting/actions.py", line 350, in destroy
TypeError: 'NoneType' object is not callable
2023-03-03 12:31:55 [36,421ms] [Warning] [carb.audio.context] 1 contexts were leaked

I got the similar errors when install using the docker. It seems part of the code is buggy. Any ideas on this? Thanks!

It solves this problem. Thanks!


[SPARK-23626][CORE] DAGScheduler blocked due to JobSubmitted event

What changes were proposed in this pull request?
Forcing partition evaluation in callsite thread before sending org.apache.spark.scheduler.JobSubmitted event to org.apache.spark.scheduler.DAGScheduler#eventProcessLoop can help in mitigation against job submission event blocking the DAGScheduler thread
Why are the changes needed?
DAGScheduler becomes a bottleneck in cluster when multiple JobSubmitted events has to be processed as DAGSchedulerEventProcessLoop is single threaded and it will block other tasks in queue like TaskCompletion.
The JobSubmitted event is time consuming depending on the nature of the job (Example: calculating parent stage dependencies, shuffle dependencies, partitions) and thus it blocks all the events to be processed.
Similarly in my cluster some jobs partition calculation is time consuming (Similar to stack at SPARK-2647) hence it slows down the spark DAGSchedulerEventProcessLoop which results in user jobs to slowdown, even if its tasks are finished within seconds, as TaskCompletion Events are processed at a slower rate due to blockage.
Refer: http://apache-spark-developers-list.1001551.n3.nabble.com/Spark-Scheduler-Spark-DAGScheduler-scheduling-performance-hindered-on-JobSubmitted-Event-td23562.html
I see multiple JIRA referring to this behavior
https://issues.apache.org/jira/browse/SPARK-2647
https://issues.apache.org/jira/browse/SPARK-4961
Does this PR introduce any user-facing change?
No
How was this patch tested?
Added UT to reproduce and evaluate fix.

gentle ping @squito @dongjoon-hyun @vanzin @srowen


fix: MacOS error when listCaptureDevices was called

Description
[ERROR:flutter/runtime/dart_vm_initializer.cc(41)] Unhandled Exception: Unsupported operation: Operation 'toDartString' not allowed on a 'nullptr'.
#0      Utf8Pointer._ensureNotNullptr (package:ffi/src/utf8.dart:61:7)
#1      Utf8Pointer.toDartString (package:ffi/src/utf8.dart:41:5)
#2      RecorderFfi.listCaptureDevices (package:flutter_recorder/src/bindings/recorder_io.dart:152:33)
#3      Recorder.listCaptureDevices (package:flutter_recorder/src/flutter_recorder.dart:190:33)
Steps To Reproduce
 Recorder.instance.listCaptureDevices();

Seems that the OS doesn't give a name to your input device! Anyway, I fixed this in this repo. Please try it!
Feel free to reopen if the problem persists.


Script fails when LibYAML is not installed

The script always tries to use yaml.CLoader, which leads to an AttributeError if LibYAML is not installed.

Fixed in v1.0.1


Fixing Ticking Entity

When I try to join my friends server I get kicked for ticking entity.
This is the error code
Neruina Caught Exception
java.lang.NullPointerException: Cannot read field "use_vanilla_magic_damage_type" because "net.spell_power.SpellPowerMod.attributesConfig.value" is null
When I go into the attributes file in the spell_power folder this is what I see
{
"use_vanilla_magic_damage_type": true,
"base_spell_critical_chance_percentage": 5.0,
"base_spell_critical_damage_percentage": 50.0
}
So I was wondering why Neruina was kicking me from the server?

Report to spell power and include your full log. This is Neruina functioning as expected so I'm closing this


Will crispy-bootstrap5 depend on django-crispy-forms in the future?

In the usage area it says that django-crispy-forms must also be installed. Is it planned in the future that only one app needs to be installed if you want to use crispy-bootstrap5?
Will the crispy-bootstrap5 project still be maintained?

Hey @tildebox
Yes, both packages will be required and maintained going forward.
Crispy-forms has a long standing aim to unpackage the template packs (e.g
crispy-bootstrap5) from Crispy-forms  core (form helper et al).
Thanks!

Thank you for your reply.
Is crispy-bootstrap5 ready for production?

Is crispy-bootstrap5 ready for production?

It's my list to bump the status when I come to add django 4.1 support. I
The package has been around for a while now and seems stable enough.

django-crispy-forms has an extensive docs (readthedocs.io).

Where can I find something like that for crispy-bootstrap5?

This is mainly just a template pack so features are rather limited. The main docs are for the core package. The small number of additional features we'll document in the readme.
Do you think something else is required?


amazon-ec2-utils: Fix shebang not being patched

Description of changes
Fixes #228878
Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[X] Tested, as applicable:

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

[X] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[X] Tested basic functionality of all binary files (usually in ./result/bin/)
[X] Fits CONTRIBUTING.md.

That works for me in my test on EC2!
# ec2-metadata
ami-id: ami-047cc9c74e042f8c9
ami-launch-index: 0
ami-manifest-path: (unknown)
ancestor-ami-ids: not available
block-device-mapping:
         ami: xvda
...

@kranzes thank you! Could this be added to nixos-22.11 as well?

@Kranzes thank you! Could this be added to nixos-22.11 as well?

No point imo, 23.05 is released at the end of this month and it will have it.

22.11 is EOL in like a month but I'll backport anyway.


"Standard libraries" page mentions outdated package

The "Standard libraries" page says:

The Microsoft.Quantum.Development.Kit NuGet package installed during installation and validation provides the Q# compiler, and parts of the standard library that are implemented by the target machines. The Microsoft.Quantum.Standard package provides the portion of the Q# standard libraries that are portable across target machines.

However, the Microsoft.Quantum.Development.Kit package has been deprecated for a long time (replaced by Microsoft.Quantum.Sdk) and has recently been removed.
I'd like to update this paragraph, but I'm not sure what the intent of this page is. Is it intended to provide a breakdown of all the different standard library packages? There are several of those, like Microsoft.Quantum.QSharp.Core, Microsoft.Quantum.QSharp.Foundation, Microsoft.Quantum.Standard, as well as the target-specific decompositions like Microsoft.Quantum.Type{1,2,3}.Core.
But the Microsoft.Quantum.Sdk package should automatically handle dependencies for the needed standard library packages. Is it enough to just mention that Microsoft.Quantum.Sdk automatically includes the standard library without describing all of the package details?

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

ID: 444f8056-d62d-c359-8c63-cd35798e94ff
Version Independent ID: 0f3bb81a-fc74-7f11-3a53-a0a7770eb341
Content: Introduction to Microsoft Q# standard libraries - Azure Quantum
Content Source: articles/user-guide/libraries/standard/index.md
Service: azure-quantum
Sub-service: qsharp-guide
GitHub Login: @QuantumWriter
Microsoft Alias: martinro

Work item created in Azure DevOps for the content team's backlog: #1857980


AIO script improvements

Added --lvm option to allow the AIO automatic deployment script to run on LVM images. Also changes to python virtual environment dependencies and creation for Rocky9 and Ubuntu Jammy.

From in person conversation - It would probably make more sense to invert this and make the LVM specific steps default, then make a --non-lvm flag that skips them if required

Added --non-lvm option.


Add two parameters

Added $AppendLog Parameter to append to existing logfile
Added $ImportFunctions Parameter to get back loaded user-defined functions

Alrighty!  Merged some upstream changes back in, cleaned up a few things, and tests seem to be passing - thanks for this!


Change 'app:create' to 'app:install'

Make it clearer to the CLI user that an existing app is expected as input to the command.
Fixes #87

I agree with Adam install is confusing here, especially since it's a command we might want to provide at some point to install an existing app directly from the CLI.

But this doesn't create a new app. I called "run app:create", supplying a non-existent directory, and nothing was created.
"Install" seems as close to what it does as I know--it takes an existing (already created app) and "installs" it into the vault.

It’s really about creating a new instance of an app in the vault, as a developer you’re creating the record, generating an app ID, setting name, version and in the future description, app icon etc. As for linking to the contents of the app, right now you are only able to link to a existing contents but in future you’ll be able to create a new template app like you were expecting,  generated from a bootstrap script and already be configured to work with SDK, Morpheus UI etc.

So the intent is to create an app in the future?
I feel like this still conflates two separate things, even if in the future it creates an app (i.e. its contents) for you. Suppose we make "app:create" do all the things you've outlined, what should a cli-user call if they've used create-react-app or some other external tool to supply the contents and want to add it to the vault?
What I'd like to see is a set of simple commands that are called by a higher-level command. For example, suppose we have some simple commands, "app:generate" and "app:link", and these get called by a higher-level command "app:create".
As it stands, I still think "app:create" sets expectations that are unmet until it does the contents-creation part.

I'm not sure app:link works either, as I mentioned above it's really about a developer creating an app in the daemon and generating an app ID. It's also consistent with the underlying daemon api if you look at the code, so I think it could be bit misleading for a developer following the underlying code and seeing methods around app creation and then at the edge it's called app:link.
I feel it's similar if you went to iTunes or Google play to deploy your app, you'd first create a new app, which would generate an app ID, and then upload the contents. Maybe the linking contents part is a bit confusing but it was mainly to streamline the process to a single step.

I think I'm going to throw in the towel at this point. Please consider that as a new user of the cli "this was confusing to me." Perhaps other developers' feedback (if given) can shed more light.


Is there is a way to specify the recognize mode for android sdk?

As subject.
There is an introduction on this link:
https://docs.microsoft.com/en-us/azure/cognitive-services/speech/concepts#recognition-modes
There are three modes of recognition: interactive, conversation, and dictation. The recognition mode adjusts speech recognition based on how the users are likely to speak. Choose the appropriate recognition mode for your application.
These modes are applicable when you directly use the REST or WebSocket protocol. The client libraries use different parameters to specify recognition mode. For more information, see the client library of your choice.
But I cannot find a way to specify recognition mode, anyone can help? Thanks a lot!
This issue is for a: (mark with an x)
- [ ] bug report -> please search issues before submitting
- [x ] feature request
- [ ] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce

Any log messages given by the failure

Expected/desired behavior

OS and Version?

Windows 7, 8 or 10. Linux (which distribution). Other.

Versions

Mention any other details that might be useful

Thanks! We'll be in touch soon.

if you use StartContinousRecognition we use automatically conversation mode.
if you just use Recognize we use interactive mode.
you can set it by yourself if you specify the URL including the parameters once for creation of the recognizer. do you need more fine-grained control?
thx
Wolfgang

closing this one without additional feedback.
for specific issues please open a new request ...


using alternative HTTP clients

tl;dr are you open to a PR which replaces the HTTPoison dep with Req?
The "my app uses a different HTTP client to one of my deps, and leads to bloat" issue is obviously a tricky one. Andrea Leopardi has the best summary of the current state of play for Elixir HTTP clients.
In particular, httpoison is a bit old-school these days - many cool kids are using Req (my fav) or Tesla.
My specific question in this issue: is this lib using httpoison for a specific reason? (There used to be some multipart form-based reasons to use it over e.g. Req, although I don't think that's still the case.) The Leopardi blog post linked above says that other HTTP client libs have superseded httpoison these days. If not, are you open to a PR which replaces the HTTPoison dep with Req? Client API would remain the same; winners of this change would be those who are already using Req in their apps, and losers would be those who are currently using  httppoison (in my estimation the former is a bigger group than the latter, although I don't have stats on that). Although no-one's code would break, the "victims of bloat" problem would just be re-distributed.
Anyway, sorry about the long issue - just wanted to check first before I did any work. If you don't want it that's fine too :)

Even Jose seems to be thinking along these lines for Phoenix.


Add config option to spawn players randomly per HCS on initial login

doesn't offer spawn-location benefits until redstone and a (probably) short walk-about
might put you closer to edges of initial HCS and working villages and temples

might put you closer to edges of initial HCS and working villages and temples


feat(clerk-js): Show text for organization roles when not an admin

Type of change

[ ] 🐛 Bug fix
[x] 🌟 New feature
[ ] 🔨 Breaking change
[ ] 📖 Refactoring / dependency upgrade / documentation
[ ] other:

Packages affected

[x] @clerk/clerk-js
[ ] @clerk/clerk-react
[ ] @clerk/nextjs
[ ] @clerk/remix
[ ] @clerk/types
[ ] @clerk/themes
[ ] @clerk/clerk-expo
[ ] @clerk/backend-core
[ ] @clerk/clerk-sdk-node
[ ] @clerk/edge
[ ] build/tooling/chore

Description

[ ] npm test runs as expected.
[ ] npm run build runs as expected.

When not an Organization admin, show text instead of the role <Select/>.

Do we still need this PR?

We no longer need this PR, change will me merged with another one.

George confirmed that we no longer need this PR, closing.


Use "" not <> for internal/ includes

; g grep '<internal' | wc
99     204    4779
; g grep '"internal' | wc
685    1477   36615
So I fixed the first case.

merged.  it's not popularity contest, it's a beauty contest ;)


Cannot initialize a variable of type 'char *' with an rvalue of type 'int *'

In file "rknn_model_zoo/utils/image_utils.c"
line 626:
char* p_imcolor = &imcolor;

Error: Cannot initialize a variable of type 'char *' with an rvalue of type 'int *' --clang(init_conversion_failed)
How to solve the problem? Thx!

我也有这个问题  请问你解决了吗?


[eleventy-plugin-local-respimg] Feature request: Allow overriding loading="lazy"

Version: 0.3.1
Using lazy: true in the config adds loading="lazy" to all img tags. I would like to use this, but also be able to explicitly set loading="eager" on some images, which are displayed above the fold. However, loading="eager" always gets overwritten if it is set.
Individual overrides per image is already possible when specifying the "sizes" attribute.

Yup I'm happy to support this, and it seems pretty straight forward. If you'd like to submit a PR feel free to do so, otherwise when I get some cycles I'll work on it.


No matches found: .zcomp^(*.zwc)(.)

In a terminal, run zmanage info and paste the output below:
Zim commit ref:   cc34e3f
Zsh version:      zsh 5.3 (aarch64-unknown-linux-android)
System info:      Linux localhost 3.18.22+ #1 SMP PREEMPT Fri Dec 9 18:59:01 CST 2016 aarch64 Android

Description
zlogin tries to glob zcomp* and it fails with no occurrences.
/data/data/com.termux/files/home/.zlogin:24: no matches found: /data/data/com.termux/files/home/.zcomp^(*.zwc)(.)

Asynchronous compiling twice is also bad anyway:
https://github.com/Eriner/zim/blob/master/modules/completion/init.zsh#L24
https://github.com/Eriner/zim/blob/master/templates/zlogin#L24
Steps to reproduce
Maybe a race, maybe a false assumption. Reproduced on fresh termux install, and reproduced by an Arch user installing globally.

This seems to be different from the Arch case. It simply fails on termux, possibly due to zsh globbing fails on the Android permission model.

source .zlogin to remedy the problem, beyond that I'm not sure. Please post any details or additional information you discover.

Unless there's any new information that indicates this is an issue with zim specifically, I'm going to go ahead and close this issue.

@Eriner It seems like this issue only occurs when trying to source .zlogin if the cache hasn't been built, so during installation or directly after running zmanage clean-cache. Opening a new terminal as suggested in the installation instructions resolves it. I guess a catch could be added for that error to instruct the user to open a new terminal but I think it's fine as is.
For the record I'm also running arch but I think that's probably coincidental.

@Eriner This is the result of sourcing .zlogin before the cache has been built in init.zsh, so either during installation or directly after running zmanage clean-cache. Opening a new terminal as instructed fixes it. I guess the message printed could state explicitly that you must open a new terminal and the build-cache command won't work until you do.
For the record I'm also using arch linux but I think it's coincidental.


ci: added daily trivy scan

:tada: This PR is included in version 1.1.10 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.1.10 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Exclude files through wildcard patterns

Describe the feature
More aggressive exclude/ignore files pattern matching.
Babel plugin or link to the feature description
Related to https://github.com/swc-project/swc/issues/466 but it should cover this kind of pattern instead:
.swcrc
{
  exclude: ["**/__tests__/**"]
}

Folder structure path
src/auth/__tests__

The expected output: __tests__ folder will not be transformed and exported into the out dir folder.

It's already supported by swc. See: https://swc.rs/docs/configuring-swc/#exclude

It's already supported by swc. See: https://swc.rs/docs/configuring-swc/#exclude

@kdy1 it only supports through specifying extensions afaik, does it also support excluding folder through glob patterns?

@kdy1 it only supports through specifying extensions afaik, does it also support excluding folder through glob patterns?

Yes. As it's a regular expression, you can exclude a folder using it.

Yes. As it's a regular expression, you can exclude a folder using it.

One more question, how do you make --copy-files respect exclude option?

One more question, how do you make --copy-files respect exclude option?

It's currently not possible.
I patched @swc/cli to respect .swcrc. Updating cli to v0.1.34 will do the trick.
Note that it may take some time for the new version to be installable.

It's currently not possible.
I patched @swc/cli to respect .swcrc. Updating cli to v0.1.34 will do the trick.
Note that it may take some time for the new version to be installable.

One more question, how do you make --copy-files respect exclude option?

Did you manage to find a setting to exclude files while using --copy-files?  It unfortunately copies snapshot files.

I created my own copyFiles function:
const cpy = require('cpy');
const relativeOutDir = path.relative(srcPath, outDir);

await cpy(['**', `!**/*.{snap,ts,js,tsx,jsx}`], relativeOutDir, {
   cwd: srcPath,
   parents: true,
});


[WIP] Move from apache module mod_auth_kerb to mod_auth_gssapi

https://www.pivotaltracker.com/n/projects/1610127/stories/160659028
This PR is one of a set that will implement moving external authentication support from
the Apache module mod_auth_kerb to mod_auth_gssapi
Once all of the PRs are available I will remove the WIP flag

@miq-bot add_label wip

@miq-bot add_label enhancement

@miq-bot add_label hammer/yes

@gtanzillo and @abellotti I have made the update we agreed to yesterday. This should be good to go now.

https://bugzilla.redhat.com/show_bug.cgi?id=1636547 will be used to track this work.

Hammer backport details:
$ git log -1
commit 9fe5cc5a44d08890934058c8aee086c15dbb7d69
Author: Gregg Tanzillo<EMAIL_ADDRESS>Date:   Fri Oct 5 16:39:04 2018 -0400

    Merge pull request #282 from jvlcek/gssapi

    Move from apache module mod_auth_kerb to mod_auth_gssapi

    (cherry picked from commit 9ab682c2641fa2384b0d4d4836b23f6a203d4c5f)

    https://bugzilla.redhat.com/show_bug.cgi?id=1636547


Location-USERNAME.json not being updated

Expected Behavior
web/location-USERNAME.json file should be updated
Actual Behavior
File is not being written too
Catchable and Items are being updated fine
Your config.json (remove your credentials and any other private info)
{
    "auth_service": "google",
    "username": "######",
    "password": "######",
    "location": "######",
    "gmapkey": "######",
    "libencrypt_location": "/home/user/Pokemongo-Bot/",
    "tasks": [
      {
        "type": "HandleSoftBan"
      },
      {
        "type": "CollectLevelUpReward"
      },
      {
        "type": "IncubateEggs",
        "config": {
          "longer_eggs_first": true
        }
      },
      {
        "type": "TransferPokemon"
      },
      {
        "type": "EvolvePokemon",
        "config": {
            "evolve_all": "all",
            "first_evolve_by": "cp",
            "evolve_above_cp": 500,
            "evolve_above_iv": 0.8,
            "logic": "and",
            "evolve_speed": 20,
            "use_lucky_egg": true
        }
      },
      {
        "type": "RecycleItems",
        "config": {
            "item_filter": {
            "Pokeball":       { "keep" : 60 },
            "Greatball":      { "keep" : 75 },
   	    "Ultraball":	    { "keep" : 125 },
            "Potion":         { "keep" : 0 },
            "Super Potion":   { "keep" : 0 },
            "Hyper Potion":   { "keep" : 0 },
	    "Max Potion":     { "keep" : 0 },
            "Revive":         { "keep" : 0 },
            "Razz Berry":     { "keep" : 50 }
          }
        }
      },
      {
        "type": "CatchVisiblePokemon"
      },
      {
        "type": "CatchLuredPokemon"
      },
      {
        "type": "SpinFort"
      },
      {
        "type": "FollowPath",
        "config": {
          "path_mode": "loop",
          "path_file": "configs/path.######.json"
        }
      }
    ],
    "map_object_cache_time": 5,
    "forts": {
      "avoid_circles": true,
      "max_circle_size": 50
    },
    "websocket_server": false,
    "walk": 13.00,
    "action_wait_min": 1,
    "action_wait_max": 4,
    "debug": false,
    "test": false,
    "health_record": false,
    "location_cache": true,
    "distance_unit": "km",
    "reconnecting_timeout": 15,
    "evolve_captured": "Pidgey,Rattata,Weedle,Zubat",
    "catch_randomize_reticle_factor": 1.0,
    "catch_randomize_spin_factor": 1.0,
    "logging_color": true,
    "catch": {
      "any": {"catch_above_cp": 0, "catch_above_iv": 0, "logic": "or"},
      "// Example of always catching Rattata:": {},
      "// Rattata": { "always_catch" : true }
    },
    "release": {
      "any": { "keep_best_cp" : true, "keep_best_iv" : true, "logic": "or" },
      "// Example of always releasing Rattata:": {},
      "// Rattata": { "always_release" : true },
      "// Example of keeping stronger (based on CP) Pidgey:": {},
      "// Pidgey": { "keep_best_cp" : true },
      "// Example of keeping stronger (based on IV) Zubat:": {},
      "// Zubat": { "keep_best_iv" : true }
    },
    "vips" : {
         "Any pokemon put here directly force to use Berry & Best Ball to capture, to secure the capture rate!": {},
        "any": {"catch_above_cp": 1200, "catch_above_iv": 0.9, "logic": "or" },
        "Lapras": {},
        "Moltres": {},
        "Zapdos": {},
        "Articuno": {},
        "Charmander": {},
	"Bulbasaur": {},
	"Pikachu": {},
	"Squirtle":{},
	"Dratini": {},

        "// S-Tier pokemons (if pokemon can be evolved into tier, list the representative)": {},
        "Mewtwo": {},
        "Dragonite": {},
        "Snorlax": {},
        "// Mew evolves to Mewtwo": {},
        "Mew": {},
        "Arcanine": {},
        "Vaporeon": {},
        "Gyarados": {},
        "Exeggutor": {},
        "Muk": {},
        "Weezing": {},
        "Flareon": {}

    }
}

Steps to Reproduce
above config, starting the bot
Other Information
OS:Ubuntu
Git Commit: (run 'git log -n 1 --pretty=format:"%H"' and paste it here)
Python Version: (run 'python -V' and paste it here)

I can confirm the issue

Have you create et setup file PokemonGo-Bot/web/config/userdata.js ?

I have. the initial position of the trainer is loaded and displayed on the map -- just not update during the run.

Delete the PokemonGo-Bot/web/location-USERNAME.json file, he will be re-created automatically.
Sometimes that fix the problem.

@vicch Nice one. But still don't understand why they disabled that handy feature.

I heard it's because a new web UI based on websocket is being developed, as a result, old file-exporting method is being cut down.

Is there an alternative way in the mean time?

Just comment the "pass" and uncomment the rest of the function temporarily. When the new update comes out for the map, just git stash && git pull .

After commenting pass and uncommenting
while True: self.web_update_queue.get() self.update_web_location()
I receive this in terminal
<IP_ADDRESS> - - [11/Aug/2016 01:15:13] code 400, message Bad HTTP/0.9 request type ('\x16\x03\x01\x00\xba\x01\x00\x00\xb6\x03\x03z\x01\xcd\xff\xecV\xe9\xb38f\xb4BH\xf9y') <IP_ADDRESS> - - [11/Aug/2016 01:15:13] "??z???V?8f?BH?y                                                      ?+????&?z??v??+?/̨̩?" 400 -
OSX: 10.11.6
Branch: Dev

I'v removed the pass, uncommented the next lines and recompiled with 'python -m compileall'. And it works.
Ubuntu 16.04
Branch: Dev

I removed pass and uncommented the two lines and it works without recompiling anything

fixed


Prometheus-exporter feature shouldn't enable the prometheus crate for producing metrics

Currently, the prometheus-exporter feature enables the prometheus feature, which not only pulls in the prometheus dependency (which we want), but it also overrides the default metrics library used to produce metrics so prometheus is used instead of opentelemetry.
When this mistake is corrected, the tests fail, indicating that there is a bug with how we're producing metrics when using opentelemetry.

It turns out that there was a concurrency bug in how opentelemetry was starting the count for new metric label sets (https://github.com/open-telemetry/opentelemetry-rust/issues/1070), which is causing our tests to fail when the use the opentelemetry feature.

Replaced by https://github.com/autometrics-dev/autometrics-rs/pull/97


Fix missing ss command in ironic-pxe container

The ironic-pxe container requires ss command to check status of UDP port used by dnsmasq.

Probably good to add it in the base image like before.


renamings and additions in HoTT

Additions to the library of pointed types.
Changes in init.path to use the induction tactic instead of eq.rec, also added some missing theorems.
Major renamings as sent to lean-user:

is_hprop -> is_prop
is_hset -> is_set
(this included some C++ files)
hprop -> Prop
hset -> Set
Pointed -> pType
Currently if foo is a type, we currently use Foo for its corresponding pointed type.
New naming convention: use pfoo. So for example Susp will be renamed to psusp.
Similarly for pointed maps. E.g. Susp_functor will become psusp_functor

I fixed (the only) one broken test. It was funny why it broke: it broke because + was not overloaded anymore.

Adding two more commits.


fix spelling: "administrators"

Why:

Supports #23188
A follow-up to #23189, as I was asked to create new PRs for any one-off spelling errors I found there.

What's being changed (if available, include any code snippets, screenshots, or gifs):

Check off the following:

[x] I have reviewed my changes in staging (look for the "Automatically generated comment" and click the links in the "Preview" column to view your latest changes).
[x] For content changes, I have completed the self-review checklist.

👋 @SeanKilleen - many thanks, again, for this fix! Will get it merged once all the tests have passed ✨ ✅


Create NVD updater

The NVD updater needs to be run once a day.  Figure out an efficient way to do this.

It also needs to be run at system initialization.


Update for Makie v0.20

[X] Bump version to 0.3.2
[X] Fix CairoMakie raw-latex rendering
[ ] Update docs to deprecate inline latex rendering
[ ] Update the teximg recipe with converts to accept anything from a String to a TeXDocument, and comply with theming for the string (via e.g. used_attributes for specific input types)
[ ] Fix the README example
[ ] Advice for where things go wrong

Still a bunch of TODOs, but I'll get a working version out for now and fix those later!


RATIS-1180. Expose RaftServer status with Status Info

What changes were proposed in this pull request?
Right now we have isLeader() but it says false when it is leader and not ready. There is no way to figure out if it is leader and not ready from this API, we need to call isLeader and isLeaderReady for that.
This Jira exposes the ServerStatus with the following info.
NOT_LEADER, LEADER_AND_READY, and LEADER_AND_NOT_READY.
The purpose of Jira is to avoid multiple calls like isLeader and isLeaderReady, trying to achieve the same with a single API.
Expose ServerStatus API with the following info.
NOT_LEADER, LEADER_AND_READY, and LEADER_AND_NOT_READY.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/RATIS-1180
How was this patch tested?

@bharatviswa504 , just have filed https://issues.apache.org/jira/browse/HDDS-4517 and https://issues.apache.org/jira/browse/RATIS-1181 .

@bharatviswa504 , just have filed https://issues.apache.org/jira/browse/HDDS-4517 and https://issues.apache.org/jira/browse/RATIS-1181 .

Thank You @szetszwo . I will try to use the new APIs introduced once RATIS-1181 is over.

Closing this, it is taken care under subtasks of RATIS-1181.
Thank You @szetszwo for providing new APIs.


Cannot bind value of element and have onchange event fire.  Can only have either or.

<select id="cboLocations" bind="@uiLocation" onchange="@LoadForms">
<option value="A">A_</option>
<option value="B">B_</option>
<option value="C">C_</option>
<option value="D">D_</option>
</select>
@functions  {
string uiLocation =""
protected async void LoadForms()
{
// does an async call relying on the value of uiLocation variable
}
}

Yes, bind is currently always a two-way bind. If you want a more custom scenario, change LoadForms() to also set uiLocation to the new value, and also amend the <option> tags to get the current value, like so:
<option value="A" selected="@(uiLocation == "A")">A_</option>

Appreciate the hint but haven't figured this out yet.  How would I access the uiLocation if I can't bind it to the select html control?  Seems like a chicken/egg scenario if you can't bind a value and call an event trigger on the same element.  I find this to be rather limiting, but i assume I am just missing something.
If I can't have bind and have an onchange for a single dropdown then I need to figure out a way to call an event and reference the selected value of a dropdown within that event.  Similar to old school asp webforms where the controls were server side.  I haven't seen this ability yet but admittedly I'm still learning it.

Looking at your suggestion again, it looks like if I set the value of uiLocation, it would then set the selected value of the dropdown without me needing the bind keyword.  Clever.
My scenario is the opposite though.  I want to read the value of the select, and use it for a parameter in a event fired by the onchange of the select element.  I can't figure out how to read the value if it isn't bound and not resulting to some javascript interop.  If we still have not (not choose to) go to javascript than this project loses some of its steam IMO.

You don't need JS interop for this, but you do need to store uiLocation yourself.
The way I approached this (well, in my case, it was for radio buttons, not a drop-down, but same difference) was to create a component EnumRadioButtons that gets passed an enum type as a parameter. That way, it can populate the <option> tags itself using Enum.GetValues(), while holding the selected value as a .NET property.
There's no JS involved it (though there might be if I would use, say, visual effects from a JS library), but there's a fair bit of C# boilerplate involved still.

We are still evolving the "bind" APIs, so over time this might change depending on what people report, but right now there's a better way of achieving your goal.
Instead of using onchange and bind separately, just use bind on its own and then put logic into your property setter to take some action when it changes. This is much clearer than using bind and onchange together because it lets you be totally explicit about what order you want things to run in (e.g., whether you run your logic before the property value is updated, or after that).
This doesn't require resorting to any JS interop.
Example:
<select id="cboLocations"<EMAIL_ADDRESS>
@functions {
    string _uiLocation;
    string uiLocation
    {
        get => _uiLocation;
        set
        {
            // Run some logic before the value is updated here...
            _uiLocation = value;
            // ... or after value is updated here.
        }
    }
}

I made a workaround for this by creating a method that sets the value of the inputModel:
<input type="number" onchange=@((arg) => OnChange(arg, this.InputModel, inputModel => input.Number) />

@functions
{
    void OnChange<TObject, TValue>(UIChangeEventArgs uIChangeEventArgs, TObject target, Expression<Func<TObject, TValue>> propertyGetter)
    {
        if (uIChangeEventArgs?.Value != null)
        {
            var expression = (MemberExpression)propertyGetter.Body;
            var property = (PropertyInfo)expression.Member;
            property.SetValue(target, uIChangeEventArgs.Value);

            // Do your own stuff here
        }
    }
}

I made a workaround for this by creating a method that sets the value of the inputModel:
<input type="number" onchange=@((arg) => OnChange(arg, this.InputModel, inputModel => input.Number) />

@functions
{
    void OnChange<TObject, TValue>(UIChangeEventArgs uIChangeEventArgs, TObject target, Expression<Func<TObject, TValue>> propertyGetter)
    {
        if (uIChangeEventArgs?.Value != null)
        {
            var expression = (MemberExpression)propertyGetter.Body;
            var property = (PropertyInfo)expression.Member;
            property.SetValue(target, uIChangeEventArgs.Value);

            // Do your own stuff here
        }
    }
}

Seems like a function what I am needing... care to elaborate what packages/addons needed to be used for getting those params? If i try to use this i am getting something like that: The type or namespace name 'TValue' could not be found (are you missing a using directive or an assembly reference?)


/config/config/tmp/ fills up everytime a new version of CrashPlan client is release

I find out a new version of CrashPlan is released by getting notified my /var/lib/docker partition on the host has filled up.
/config/conf/tmp/ in crashplan container gets filled with all the failed download attempts (just cleared out 9gigs this morning)...
it contains 100s of files of the format c42..dl in that folder.
Do I need to tweak some permission on my file system somewhere so that it won't keep trying to download the new file?
it creates them pretty quickly as well.. roughly 2-3 a second in my case.

Hmmm.  I just checked docker\appdata\crashplan-pro\conf\tmp on my Synology.  conf\tmp only contains 4 files - all named:  c42.*.dl    They are timestamped 30 minutes apart starting at "2022-01-19 11:45" to "2022-01-19 13:15."  So, obviously is not a problem on my Synology.

I also had 10gb of files (200M every 15 minutes beginning yesterday) on my Synology.

CONFIRMED.
I found 34 GiB of c42.*.dl files. A new download every 15 minutes.
docker container exec crashplan-pro ls -lt /config/conf/tmp
total 35640072
-rw-r--r--    1 app      app      209742817 Jan 21 19:57 c42.12580293313160388707.dl
-rw-r--r--    1 app      app      209742817 Jan 21 19:37 c42.13946189431256022518.dl
-rw-r--r--    1 app      app      209742817 Jan 21 19:22 c42.120296281001130550.dl
-rw-r--r--    1 app      app      209742817 Jan 21 19:07 c42.14206040643177846202.dl
-rw-r--r--    1 app      app      209742817 Jan 21 18:52 c42.18165929816492718125.dl
-rw-r--r--    1 app      app      209742817 Jan 21 18:32 c42.1108601444248561847.dl
-rw-r--r--    1 app      app      209742817 Jan 21 18:17 c42.7936280995522326635.dl
-rw-r--r--    1 app      app      209742817 Jan 21 17:57 c42.5663719808675147507.dl
-rw-r--r--    1 app      app      209742817 Jan 21 17:42 c42.1570348590304214135.dl
-rw-r--r--    1 app      app      209742817 Jan 21 17:27 c42.17473428746789910232.dl
[...]

This issue has been also discussed in https://github.com/jlesage/docker-crashplan-pro/issues/336.
Not allowing writing to this tmp folder doesn't work, since this prevent the engine from starting.
But I finally found a way to prevent these downloads.  A new docker image with the fix should be available today, along with the update to CrashPlan version 8.8.2.

This should now be fixed in the new image.

Hi @jlesage.  I didn't see this issue before opening another one.  I have client version 8.8.2 and this is still happening - the downloads were coming in every 15 minutes.
I'll try updating my image and check again...

This is still an issue. Currently using container jlesage/crashplan-pro:v2.16.4

This is expected since you are using an outdated Docker image.
Please update your container with the latest Docker image.

But do you understand my position? Without petting/caring for my Docker server (running multiple services), the whole server stops working shortly after Crashplan releases a new version.

Yes I agree this is a good workaround when running a version with the bug.
But note that if you don't keep your container's image up-to-date, CrashPlan will eventually stop to backup your files. CrashPlan requires to be at the latest version.
Maybe you should have a look at Watchtower: https://github.com/containrrr/watchtower This great piece of software automatically updates your Docker containers, without any user intervention.

will this update from the crashpan download or does it check for when you, jlesage,  have issued a new version?

Watchtower checks for new Docker image versions.  So yes, it's when I publish a new Docker image that Watchtower will do its job.

will this update from the crashpan download or does it check for when you, jlesage, have issued a new version?

That's what Watchtower does (as proposed by jlesageà).
It will check continuously for a new container version, if found, pull the image, stop the running container, delete it, run a new container with the same parameters as the previous one.

But what will happen if Crashplan releases a new version right after jlesage went on holidays? :)

The latest docker image has a fix that blocks Crashplan from downloading new versions and filling up your partition. It’s a Crashplan bug but this latest docker image implemented a workaround for it. But you will still need to update to new docker images when new versions of crashplan are released.


🌱  Fix remaining wrong go versions in CI

Summary
This PR fixes the remaining places where CI was saying to use go release 1.22.
Related issue(s)
Fixes #

/cc @ezrasilvera
/cc @waltforme

/lgtm

/approve


Issue with Build-a-classifier model

Reference:
https://docs.microsoft.com/en-us/azure/cognitive-services/Custom-Vision-Service/getting-started-build-a-classifier
I was following tutorial for building classification model.
I have uploaded 5 images and tagged it as "Mask" but it is not allowing me to train the model.
Error:
Your project can't be trained just yet. Make sure you have at least 1 non-negative tag and 5 images for every tag.
Can you please explain what does non-negative tag means and how can I solve it out.

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

ID: 8198ccf7-c499-c84b-6d54-e48a8fc4a150
Version Independent ID: aae607bf-72be-59dd-32bc-be33039de820
Content: Quickstart Build a classifier - Custom Vision Service - Azure Cognitive Services
Content Source: articles/cognitive-services/Custom-Vision-Service/getting-started-build-a-classifier.md
Service: cognitive-services
Sub-service: custom-vision
GitHub Login: @PatrickFarley
Microsoft Alias: pafarley

@AmitavaHazra When a tag is created the service expects at least 5 images for each tag  and it is also necessary to tag some images as negative so the model can differentiate the two states. For example:
If your goal is to distinguish between an empty room and a room with people in it, we recommend tagging five or more images with people as a new class, People, and tagging five or more images without people as the Negative tag. This will help the model differentiate between the two states.

This is still very confusing to figure out through the UI how to resolve the issue for negative tags


View Linearity Injection Data

As an ECMPS user, I want to view QA test summary linearity injection data so that I can look at real-time data.
Acceptance Criteria
Given a user has selected Linearity Summary from the Test Type dropdown and a linearity injection data record is available
When the QA test summary linearity injection data is displayed
Then the data is displayed nested under the linearity summary table as a separate table (see list below for columns and order)
Injection Date
Injection Hour
Injection Minute
Measured Value
Reference Value
Reference Images:

Xd Prototype: https://xd.adobe.com/view/40aa3540-d85f-4b68-9bf2-63f694bee243-15f9/screen/e67af277-4135-4f37-bbfc-112eb383e0cc/

Test Passed


ATA-5400 Move generator from prism-sdk to this repo

This PR moves grpc clients generator to the pbandk repo.
Related PR is here

@pva701 please make sure the branch is up to date

@vlad107 plz wait and don't merge the PR, I figured out the small issue that a package of generated classes is io.iohk.atala.prism.protos but in the vault it should be io.iohk.atala.vault.protos or io.iohk.atala.prism.vault.protos I guess, so it's better make customisable. I will check how easy it to achieve.


Nginx regex annotation in Ingress is mandatory if Nginx version >= 0.34.1

Nginx bug report https://github.com/kubernetes/ingress-nginx/pull/5691  was fixed in image version >= 0.34.1 per https://github.com/kubernetes/ingress-nginx/releases/tag/controller-v0.34.1. If using these new images the Ingress yaml in the example won't work and everything will be routed to default backend with HTTP 404 because  regular expression in the path will be taken as literal. This annotation is mandatory now:
nginx.ingress.kubernetes.io/use-regex: "true"
Reference: https://kubernetes.github.io/ingress-nginx/user-guide/ingress-path-matching/

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

ID: ada58160-dc50-b977-d3bc-65f92406e16d
Version Independent ID: 5c596e04-fddb-effe-3b72-575f4df42d4e
Content: Create ingress with automatic TLS - Azure Kubernetes Service
Content Source: articles/aks/ingress-tls.md
Service: container-service
GitHub Login: @mlearned
Microsoft Alias: mlearned

Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

thank you @kennethgp for the feedback!
I have updated the article to address your feedback and the changes should be available soon.


[esbuild-plugin-html] Load style files without script

When using the HTML plugin and I import styles in JS via
import "path/to/style.css";

it results in an additional script tag added to the HTML:
<script type="module">
        (function() {
            function loadStyle(url) {
                var l = document.createElement('link');
                l.rel = 'stylesheet';
                l.href = url;
                document.head.appendChild(l);
            }
            loadStyle('style.css');
        }());
</script>

Not sure what the reason for this is over just including it via a <link> tag directly, perhaps to avoid render blocking?
Anyway I would like to have at least an option to configure it s.t. this would just add
<link rel="stylesheet" href="style.css"/>

because I want my styles to be render blocking (to avoid FOUC) and save some bytes.
The code responsible for this seems to be in collectScripts.js.
Happy to provide the PR for this change myself if you agree with this proposal.
Thanks for this plugin btw :)

Thanks for your reply @edoardocavazza. Specifying the entry point in the HTML file like suggested is in option, however that means I have to manage all my CSS imports in the root CSS file. With the other approach I only get the CSS I need when every UI component/JS module imports only the CSS it requires. That's why I prefer that way of importing CSS, it's one way to avoid bloat that I don't have to think about.

Our hope is that this issue will be fixed, so for now, we handle the CSS import, just as esbuild would, via javascript.

If you plan to drop that code eventually, I guess it doesn't make much sense to add options around that.

This is an intended behavior: if you import the CSS via JavaScript, the output will continue to load the CSS via JavaScript.

This is a good point, I just wish there was a way around that.

In the next days I will try to find out the esbuild position on this. If I can't find a plan to fix the issue, I think it would make sense to introduce an option to get the optimised result.


Absolute paths are translated incorrect from WSL to Win

Please fill out the below information:

Your Windows build number:  10.0.17134.765
wsl.conf:

[automount]
enabled = true
root = /
options = "metadata,umask=022,fmask=111,case=off"

What you're doing and what's happening:
$ notepad.exe /c/Users/carl-oskar/workspace/example/readme.txt

What's wrong / what should be happening instead:
Instead of notepad opening the file it shows an error message box stating that the path could not be found.
If using notepad++.exe instead it shows an error message box that says
"C:\c\Users\carl-oskar\workspace\example\readme.txt" cannot be opened:
Folder "C:\c\Users\carl-oskar\workspace\example" doesn't exist.

I tried with changing the mount root back to /mnt/ but it still failed but now with C:\mnt\c....
If using a relative path to open the file in notepad then the file opens as expected.

We don't automatically translate any command line arguments, but you can use wslpath to do this for you.
notepad.exe  $(wslpath -w /c/Users/carl-oskar/workspace/example/readme.txt)

Ah I see.
I noticed that the 'C:' is prefixed on the windows side, doing Win-R with
notepad++.exe /c/Users/carl-oskar/workspace/example/readme.txt
yielded the same problem with the path being "C:\c\Users..."
Unfortunately I was actually trying to do something like
GIT_SEQUENCE_EDITOR="/c/Program\\ Files/Notepad++/notepad++.exe" git rebase -i HEAD~3
but for another editor.
The "problem" line in the git-rebase--interactive script is
eval "$GIT_SEQUENCE_EDITOR" '"$@"'
Changing the line to
eval "$GIT_SEQUENCE_EDITOR" '"$(wslpath -w $@)"'
makes the file open as expected.
But I suspect that it is probably easier to get the author of the editor to adjust for wsl support than to get the git authors to adjust for this.
Thanks


[ERROR] unable to get GPU ammount

Tried -multi-gpu and -deviceid 0,1 using hiveos miner

I had this issue using dynexsolve custom miner on hive, but I have AMD cards, i solved it using srbminer instead

Anyone help me？RX580

This issue needs to be addressed with your pool operator / mining software developer

Marca esta opcion


Data model for lightweight mode on MvStorage

Schema for embedded database
Finding out if any box related data have been spent or not in query time puts huge pressure on DB
=> let's do that at indexing time so that queries are real-time !
Shared
headerIdsByHeight:  Map[Height, Set[HeaderId]] // more than one in case of a fork-in-progress
blockByHeaderId:    Map[HeaderId, Block] // arbitrary block data (depends on performance)

Unspent/NonEmpty
utxosByAddress:     Map[Address, Map[BoxId, Value]] // this would be a clone of Node's utxo state
addressByUtxo:      Map[BoxId, Address] // non-empty address by utxo

Spent
allBoxesByCustomAddress:     Map[Address, Set[BoxId]] // all boxes for a configured address by a dApp developer

There can be many of these indexes, please provide your suggestions and use-cases!
Facts to consider :

some bare minimum of data like UtxoState is going to be indexed for all data
arbitrary data, especially spent inputs (tens of millions of records) could be configurable for specific addresses
so that dApps developers can customize the explorer for their needs

Eventually the Http API will allow for retrieving anything that can be put together from these persistent Maps.

Shared
headerById:                       Map[HeaderId, Value] // Probably best to only keep n headers?
                                                       // Most of dApps will only need the last 10
                                                       // headers to be used as reduction context.
headerIdsByHeight:                Map[Height, Set[HeaderId]]

Unspent
boxById:                          Map[BoxId, Value]
boxIdsByContractHash:             Map[ContractHash, Set[BoxId]]
boxIdsByP2SContractTemplateHash:  Map[TemplateHash, Set[BoxId]] // In this map we can have a super node
                                                                // with P2PK contracts, so best to skip
                                                                // them on indexing.
boxIdsByR4Hash:                   Map[RegisterHash, Set[BoxId]] // Non-empty R4 register
boxIdsByR5Hash:                   Map[RegisterHash, Set[BoxId]] // Non-empty R5 register
boxIdsByR6Hash:                   Map[RegisterHash, Set[BoxId]] // Non-empty R6 register
boxIdsByR7Hash:                   Map[RegisterHash, Set[BoxId]] // Non-empty R7 register
boxIdsByR8Hash:                   Map[RegisterHash, Set[BoxId]] // Non-empty R8 register
boxIdsByR9Hash:                   Map[RegisterHash, Set[BoxId]] // Non-empty R9 register
boxIdsByTokenId:                  Map[TokenId, Set[BoxId]]
boxIdsByTransactionId:            Map[TransactionId, Set[BoxId]]

Spent
Spent boxes needs all Unspent maps plus the following:
mintingBoxIdsByTokenId:           Map[TokenId, Set[BoxId]] // EIP-4 only considers one minting box
                                                           // per token, but protocol allows multiple
                                                           // boxes in the same minting transaction,
                                                           // so best to follow the protocol.

Using hashes as contract and registers indexing keys
From storing efficiency point of view, it's better to use hashes instead of the content directly as indexing keys, BLAKE2b256 have 32 bytes against contracts and registers that can be as big as the maximum box size (4 KB) minus the required registers' size.

Average contract size: 121 bytes;
Average register size: 33 bytes, but it can grow with complex dApps like Paideia which stores entire boxes on its registers.

BLAKE hashing algorithm is know by its speed and security, and is extensively used on Ergo, however indexing times must be taken into consideration.

@anon-br Regarding the data model, I'm technologically limited to multimaps Map[PK, Map[SK, V]] ... Map[K, Set[V]] is not technologically possible as to data persistence. We are limited to working with key-value maps or nested key-value maps, simply because I use KV store.
https://github.com/pragmaxim/ergo-uexplorer/blob/master/modules/storage/src/main/scala/org/ergoplatform/uexplorer/storage/MvStorage.scala#L234-L240
Secondly, Map[RegisterHex, Set[BoxId]] is a map of cardinality 7, any low-cardinality Key cannot be used in KV store.
I could do something like Map[ContractHex, Map[BoxId, RegistersOrAssets]] for instance, but Set[BoxId] is not possible. Reason being is that you'd have to always serialize/deserialize a HashSet to binary on each asccess(R/W).

Copy/pasting some rest-endpoints from @arobsn ... as I keep uexplorer on Scala3, I'm currently spiking this on multiple tech stacks, started with Slick as I had experience with it, then doobie, ended up with : zio-protoquill, zio-http, zio-json which are all production ready or very close to production ready ... Eventually zio-protoquill could be replaced with zio-sql which is currently in development.
get  /boxes/{state}/tokens/{tokenId}/
get  /boxes/{state}/{boxId}/
get  /boxes/{state}/addresses/{address}/
get  /boxes/{state}/addresses/{address}/tokens/{tokenId}/
get  /boxes/{state}/contracts/{contractHex}/
get  /boxes/{state}/contracts/{contractHex}/tokens/{tokenId}/
get  /boxes/{state}/contracts/hashes/{contractHashHex}/
get  /boxes/{state}/contracts/hashes/{contractHashHex}/tokens/{tokenId}/
get  /boxes/{state}/contracts/templates/{contractTemplateHex}/
get  /boxes/{state}/contracts/templates/{contractTemplateHex}/tokens/{tokenId}/
get  /boxes/{state}/contracts/templates/hashes/{contractTemplateHashHex}/
get  /boxes/{state}/contracts/templates/hashes/{contractTemplateHashHex}/?R4=deadbeef&R5=cafe
get  /boxes/{state}/contracts/templates/hashes/{contractTemplateHashHex}/tokens/{tokenId}/
post /boxes/query/

// state = spent | unspent

get /tokens/{tokenId}/
get /tokens/{tokenId}/minting-box/

I keep uexplorer on Scala3, I'm currently spiking this on multiple tech stacks, started with Slick as I had experience with it, then Doobie, ended up with : zio-protoquill, zio-http, zio-json which are all production ready or very close to production ready ... Eventually zio-protoquill could be replaced with zio-sql which is currently in development.
There are 3 choices in the scala ecosystem when it comes to SQL : Typelevel stack and Zio stack ... My bet is on Zio as the Typelevel stack is not really united well. One needs to have at least 10 various dependencies to put a simple CRUD app together, whereas in Zio land, you are good to go with just : zio-protoquill, zio-http, zio-json. This pays off especially when using Scala3 as it is basically first-class citizen in Zio 2.0.


Give well de-selection its own highlight state

As a user I would like to be able to see which wells I am deselecting
Background
Currently when you hover over wells while holding shift, there is no highlight state
Acceptance criteria
-Give well deselection its own highlight so you can see what you are about to deselect
Design

Note: play around with dash sizes for 384 plate? Try it out

We have decided this is a good place to start and should proceed with dashed line


Add custom authority lookup

As part of the Instance Discovery changes in #1582 I removed the Instance Discovery call using the developer provided authority, in favor of the default authority. This PR reverts to using the provided authority as all other aliases are still included in the response. This is mainly to prevent a breaking change for custom domains such as login.windows-ppe.net that will not show up in the initial instance discovery call.

Coverage increased (+0.03%) to 79.063% when pulling e2bfb8b9e7c4bc9568f1b8c443319247a32f55db on custom-domain-msal-core into 7c1a302c29e36e2cdd48b755a6794edcbdd738fb on dev.


Error: babel version 4.4.3 is not currently supported

I just got this error. The reason is obvious. But could the error message be improved to include the supported versions so the user will be able to conclude that things will work if they upgrade their engine, rather than conclude that accord just doesn't have babel support?

Yeah, that shouldn't be too tough to add. You'd just have to pull the adapter's filename and add it to the error message. Would you be interested in adding a PR for this? If not I can take care of it later.


Next version bump ~v4.17.0

#804 (LOW) - Updating sage-lib workflow; does not impact kajabi-products
#807 (MEDIUM) - Anywhere there is a folder icon, verify it's rendering as a folder and not as a flag

[ ] View http://www.kajabi.test:3000/admin/products/1 for folder usage

#803 (LOW) Fixes label color on input focus and adds new disabled prop for Form Select component.
#809 (LOW) Visual updates to the SageIconList
#811 (LOW) No testing in kajabi-products is needed; Update with sage-lib's package.json file

Since I am the creator of this PR, I cannot approve this. Please consider this comment my approval ✅


LA-377 Update rsoprivatecloud/openstack-ops SHA

This change updates the SHA used to the HEAD of the openstack-ops
ocata-15.0 branch. A comment has been added to make clear that
openstack-ops does not mirror the RPCO branch process.

Will review it today.


option chain data

Is there an easy way to get option chain data for a ticker
cheers @JerBouma

Hi! Please have a look here: https://www.jeroenbouma.com/projects/financetoolkit/docs/options#get_option_chains. I did notice that the Yahoo Finance API can be a bit unstable so if the code doesn't work, let me know. I'll introduce a fix today.

thank you sir!
On Sun, Feb 4, 2024 at 9:08 PM Jeroen Bouma @.***>
wrote:

Hi! Please have a look here:
https://www.jeroenbouma.com/projects/financetoolkit/docs/options#get_option_chains.
I did notice that the Yahoo Finance API can be a bit unstable so if the
code doesn't work, let me know. I'll introduce a fix today.
—
Reply to this email directly, view it on GitHub
https://github.com/JerBouma/FinanceToolkit/issues/113#issuecomment-1925673655,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AARQUD2YJ37YIN3NLZ3WHNDYR5MZNAVCNFSM6AAAAABCYXIWPWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMRVGY3TGNRVGU
.
You are receiving this because you authored the thread.Message ID:
@.***>


reference CDI configuration details

Referenced CDI configuration details as requested in this review comment

Can you also update https://github.com/containerd/containerd/blob/main/docs/cri/config.md? There's an example config file where this comment would be useful too.

Thank you for the review. I'v updated the config.md as requested.


Nonlinear fix

This PR addresses issue #166.
As keyword arguments do not participate in methods dispatch, some nonlinear rules were simply ignored by ForneyLab.
Together with @semihakbayrak and @ivan-bocharov, we decided to switch the keyword argument variate to the positional argument.

Codecov Report

Merging #181 (13ec0f0) into master (5d9110e) will not change coverage.
The diff coverage is 37.50%.

@@           Coverage Diff           @@
##           master     #181   +/-   ##
=======================================
  Coverage   85.57%   85.57%
=======================================
  Files          83       83
  Lines        4061     4061
=======================================
  Hits         3475     3475
  Misses        586      586

Impacted Files
Coverage Δ

...c/engines/julia/update_rules/nonlinear_sampling.jl
58.06% <37.50%> (ø)

Continue to review full report at Codecov.

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


How can we use this plugin for manual deploys through cli?

First of all, thanks for a very nice and useful plugin.
This plugin works fine when connect to a github repository and installed through the netlify ui.
But, how can I use it deploy my site through the netlify cli?
Repository is not connected to github.
I would love an example or demonstration if possible.

hey @aditya-mitra, thanks so much for opening this issue! we had an issue open regarding plugin usage via the CLI for several weeks but happened to transfer it to an internal private repo yesterday, so unfortunately can't reference that for you here, BUT i can summarize! :D
basically, right now, the plugin will technically "work" with the CLI if you have it installed in your toml file or in the UI of your configured site.
here are the docs for our cli: https://docs.netlify.com/cli/get-started/
in your project directory using the plugin, assuming you've already run netlify login, you can run:
netlify init
this will prompt you to either create & configure a new site linked to your netlify account OR to link it to an existing site. sounds like you already have a site for this project from the UI flow, so select the "link to an existing site" option and navigate to your project's site.
from here, you can run netlify build or netlify deploy --build. this will run the plugin. however, if you have a custom publish or functions dir configured, the build will fail on the second run. this is because we haven't yet written logic to "clean" the plugin-generated files between builds/runs for these custom directories.
this is my number one priority at the moment as i've already promised it to @chrisbashton :). unfortunately, it's not super straightforward, so we're getting it done as quickly as we can.
ways around this right now:

this is painful, but you could find the plugin-generated files in your custom functions dir/publish dirs and delete them between netlify builds. you should be able to identify them via git status
use the next-on-netlify defaults for the time being -> see https://github.com/netlify/next-on-netlify#3-configure-netlify

i know this was all unnecessarily complicated so i can keep you updated when we have this fixed and supported within the plugin and then it should Just Work TM running netlify build. thank you for your patience!!!!!

hey @aditya-mitra, thanks so much for opening this issue! we had an issue open regarding plugin usage via the CLI for several weeks but happened to transfer it to an internal private repo yesterday, so unfortunately can't reference that for you here, BUT i can summarize! :D
basically, right now, the plugin will technically "work" with the CLI if you have it installed in your toml file or in the UI of your configured site.
here are the docs for our cli: https://docs.netlify.com/cli/get-started/
in your project directory using the plugin, assuming you've already run netlify login, you can run:
netlify init
this will prompt you to either create & configure a new site linked to your netlify account OR to link it to an existing site. sounds like you already have a site for this project from the UI flow, so select the "link to an existing site" option and navigate to your project's site.
from here, you can run netlify build or netlify deploy --build. this will run the plugin. however, if you have a custom publish or functions dir configured, the build will fail on the second run. this is because we haven't yet written logic to "clean" the plugin-generated files between builds/runs for these custom directories.
this is my number one priority at the moment as i've already promised it to @chrisbashton :). unfortunately, it's not super straightforward, so we're getting it done as quickly as we can.
ways around this right now:

this is painful, but you could find the plugin-generated files in your custom functions dir/publish dirs and delete them between netlify builds. you should be able to identify them via git status
use the next-on-netlify defaults for the time being -> see https://github.com/netlify/next-on-netlify#3-configure-netlify

i know this was all unnecessarily complicated so i can keep you updated when we have this fixed and supported within the plugin and then it should Just Work TM running netlify build. thank you for your patience!!!!!

hey all! i released the plugin with these changes so you should be able to run consecutive builds now with the CLI. we do want to improve this fairly fragile logic for right now and think it will require a lot more thought and attention than we can immediately give so hopefully this just gets people going with the CLI. please see the README for the recommended workflow until we can make it fully seamless and automatic. <3
cc @chrisbashton

hey all! i released the plugin with these changes so you should be able to run consecutive builds now with the CLI. we do want to improve this fairly fragile logic for right now and think it will require a lot more thought and attention than we can immediately give so hopefully this just gets people going with the CLI. please see the README for the recommended workflow until we can make it fully seamless and automatic. <3
cc @chrisbashton

Thanks for the upgrade.
I will try this plugin one more time

Thanks for the upgrade.
I will try this plugin one more time


Resolution of the program version 2.4.1

The resolucion of the program is low and the icons are pixalated, i install the driver for nvidia card in ubuntu 18.04 and continues the same. I download the version 2.4.3 app and works properly. How i can fixed the resolution of the program. No video, no other thing the resolution of the program in the screen.
thanks

This issue tracker is for bugs and features of the flathub.org website backend.
If you have any issue with an app distributed in flathub.org go to the app description page and click on the "Issues" button. If that button is not there, go to the website of the app and try to find a issue tracker or community forum
Here is an example for another app
Note: you forgot to mention the name of the application you are having problems with


Logs displaying in severity order instead of chronological

I love this new logger plugin - thanks for making something so easy to use!
One issue I've found is that the logs aren't displaying in the UI according to timestamp when I use different log levels (console.debug, console.error, etc.).  It looks like the logs are showing up in severity order first and then chronologically within each severity.  I was expecting that all logs would display chronologically across all levels - the way it is now makes it hard to understand what was happening during the invocation.

@manwaring Wanted to let you know that we've updated this and it should be fixed! Appreciate the report.


Baccarat Game Simulation with API Routing and Unit Testing

This PR introduces the Baccarat game simulation to our project. It includes the following updates:
UPDATES

Game Simulation

Implemented the core logic for simulating Baccarat games in game-server/games/baccarat/sim.py.
Added functionality to handle game outcomes, dealing of player and banker hands, and betting scenarios according to the rules of traditional baccarat.
Added the game-server/games/baccarat/helper_functions.py for utility functions related to game logic.

Routing

Implemented API endpoints for starting and managing Baccarat games in game-server/server/app.py

Unit Tests

Developed unit tests to validate the functionality of the game simulation in game-server/tests/test_baccarat.py.

Other Changes

Removed uvloop from requirements.txt
Added pandas in requirements.txt
Added a new directory game-server/tests to store tests related to game logic.

IMPACT:

Provides a complete Baccarat game simulation integrated into our project.
Enables interaction with the game through new API endpoints.
Ensures the correctness of the game simulation with comprehensive unit tests.

HOW TO RUN THE API:

Activate the virtual environment and run pip install -r requirements.txt
Navigate to the game-server directory cd game-server and run the ff uvicorn server.app:app
The API will be available at http://<IP_ADDRESS>:8000
Use the /baccarat endpoint to initiate a new Baccarat game:  http://<IP_ADDRESS>:8000/baccarat

HOW TO RUN THE UNIT TESTS:

Navigate to the game-server directory: cd game-server
Run the Tests: pytest tests/test_baccarat.py -v

ADDITIONAL NOTES

Please review the new game simulation logic and API endpoints for accuracy and functionality
Feedback on test coverage and additional edge cases is welcome.

Hi @gryAI  commiting as tests are passed and core functionality is working. will file separate issues for cosmetics.


Update minimum Python version to 3.7.2

Related to #6925

@malfet thanks for your input. Does that mean torch core and all other domain libraries need to update their Python lower bound?

@NicolasHug yes, but currently there is no good way to say, that we want to support say >=3.7.10, >=3.8.6, >=3.9.10 etc
See some discussion when similar OrderedDict dependency was introduced to the core: https://github.com/pytorch/pytorch/issues/74087


🐛fix: oak text editor font size - DASHV6-226

This fix editor base font size to avoid text increasing in editor to result in smaller text in component.
In addition, I also added some translations for all the content editor tooltips.

I still have a problem with a traduction key (Bold to be precise), in english no text is rendered and I have this error

It seems to come from junipero-utils/get function

The .get problem comes from poorly written code inside junipero-utils: https://github.com/p3ol/junipero/issues/598
I'll release a patch and update your PR once it is available on npm.


should propagation time include action

At the moment we wait for pods to be available (or policy to be applied) before computing the propagation time.
https://github.com/Kong/mesh-perf/blob/286a0cd324634f5e9a54c0bbc8c16a4cc9e5c392/test/k8s/simple_test.go#L214-L230
Should we include that time?
If we do we'll be sensitive of the time it takes to start a pod so platform dependent. However, this is a stat that's more useful to users than what we have today.
Another option is to watch for change to endpointSlice and count from there. This would sum up as "how much longer does it take for an endpoint to be added compared to core k8s" though that wouldn't even be exact...

Triage: regardless of the approach we pick using WaitNumPods is not accurate enough as it's using a loop with sleeps. Probably we should watch an endpointSlice and start the timer when the endpoint is added.


Gate parameter validation in Rust

What should we add?
Gates with parameters (e.g. RZGate) are not singleton and instantiation of the class is non-negligible overhead. However, recently I noticed that the Gate.params setter shows significant walltime in profiling due to the validation function written in Python
https://github.com/Qiskit/qiskit/blob/797bb285632a4b58c35c9e4cf5be44502b5bd78a/qiskit/circuit/gate.py#L241-L255
Since Rz can be heavily used in the parameterized circuits like twirling, it's worth considering implementing this in Rust to reduce instantiation overhead (or maybe add option to bypass).

In this circuit I have roughly ~10k Rz gates.

I'm not sure there is anything for rust to fix here. The validate_params method is purely a python space construct, it's basically protection against a user doing something like: RZGate("I am a valid angle") or any other random input type because python lets you put anything you want. Even when we move to standard gates being defined in rust with: https://github.com/Qiskit/qiskit/pull/12459 we still need to rely on Gate.validate_parameter when user input is being done in Python.
In rust this wouldn't even be a concern because the program wouldn't compile if you put an invalid type on the instruction. Although in the model from #12459 for backwards compatibility we have to allow any python object as a parameter type because a Python defined gate can contain any object as it's parameter (for example UnitaryGate can contain a numpy array).
In #12459 where I was able to skip validation was if the gate object is being created from rust, then we know it's already valid because it's been validated once before after being put on the circuit (although that validation was from validate_params). I think the only way you can avoid this overhead is after #12459 merges you can construct a CircuitInstruction manually with: CircuitInstruction(qiskit._accelerate.circuit.StandardGate.RZGate, [3.14159], qargs) and pass that directly to QuantumCircuit._append That would bypass all the validation, but then the burden is on you as the user to assert the parameters are valid because CircuitInstruction doesn't check anything.

I wonder if we can do something like pydantic where type validation is written in Rust (or maybe internally using pydantic). It would be great if the Gate constructor has an option to bypass validation, or factory method for expert users who doesn't need automatic validation.

Naoki: for your immediate purposes, can you try moving the line if isinstance(parameter, (int, float)) up to the top of the function, and changing it to if isinstance(parameter, (float, int))?  If that's enough for your immediate purposes (it'll save 2 isinstance checks per call for most cases), we can do that in the short term.

For fast-path constructors and whatnot, we'll likely grow a lot more of those a lot faster as we move more things to Rust - it's probably just going to get in the way trying to do it in Python space right now, since doing basically anything else with the circuit is likely to dominate the runtime over this.

Naoki: for your immediate purposes, can you try moving the line if isinstance(parameter, (int, float)) up to the top of the function, and changing it to if isinstance(parameter, (float, int))? If that's enough for your immediate purposes (it'll save 2 isinstance checks per call for most cases), we can do that in the short term.

That's neat! Since this is an extreme case of 10-100k gates with fixed parameters, the priority is not quite high. If you have fundamentally different solution with the Rust core that would be more appreciated.

The trick really is that just moving this function naively into Rust wouldn't help. The execution of the Python bytecode to implement the isinstance calls and the conditional branches will not be significantly slower than having the same code written in Rust.  If anything, it might be slightly slower, because we'd be calling the Python C API from an unfavoured position, rather than being baked into the full interpreter.
The only real way to speed this code up is to not do it at all.  The most reasonable path we have towards having an interface where that's a bit more possible (without introducing a lot of ways to shoot yourself in the foot) is to introduce alternative constructors or something for the standard gates, once we've got the Rust-space components in place.

For this setup:
import itertools
from qiskit.circuit import RZGate

%timeit [RZGate(0.0) for _ in itertools.repeat(None, 1_000_000)]

I find reorganising Gate.validate_parameter to check the float first can get about an 8% speedup on this microbenchmark, but in absolute terms that's still 100ms over one million gates.  It's pretty entirely just not a bottleneck; the 100ms is nearly unobservable as soon as you start adding the gates to a circuit.  Changing validate_parameter to be simply return parameter is not dissimilar to putting the float check first in this case (you get maybe a 1-2% additional speedup).
I think your benchmarking here might be overstating the cost of validate_parameter; cProfile will fundamentally have an oversized impact on small Python functions and on access to property fields, and does not trigger at all when inside Rust code.  I think what you're really seeing is that anything written in Python will appear to be much worse in cProfile than it is in practice, because you're mostly seeing the cProfile overhead.

cProfile overhead makes a lot of sense. I wondered why this simple Python setter took such long time (I should have measured pure walltime with timeit as you did).

The only real way to speed this code up is to not do it at all.

Yes. I hope Qiskit will provide an option to bypass unnecessary validations. I'm not sure which is the best way though -- writing everything in Rust vs adding expert's method in Python with no type safety.
Anyways let me close this comment because likely I overestimated the impact of validation.


BachLoggingProvider: There's no need to use async methods inside a dedicated thread

If there will be a real asynchronous operation, then its continuation will be on a thread pool thread. After that it won't return to the dedicated thread and can potentially use one threadpool thread for the rest of application lifetime.
_outputTask = Task.Factory.StartNew<Task>(
    ProcessLogQueue,
    null,
    TaskCreationOptions.LongRunning);

https://github.com/aspnet/Extensions/blob/master/src/Logging/Logging.AzureAppServices/src/BatchingLoggerProvider.cs
This certainly  will have continuation on a threadpool thread:
protected virtual Task IntervalAsync(TimeSpan interval, CancellationToken cancellationToken)
{
            return Task.Delay(interval, cancellationToken);
}

Describe the solution you'd like
Remove async methods. It's not a problem to block a dedicated thread. It is used only by only one task.
Describe alternatives you've considered
Or start the task without TaskCreationOptions.LongRunning (but that was probably was meant and desirable to run the logger on dedicated thread - so it is not a good option)
Additional context
https://github.com/andrewlock/NetEscapades.Extensions.Logging/issues/8

TaskCreationOptions.LongRunning doesn't start a dedicated thread, it would still run on thread. Using sync methods would just mean that we are blocking a thread waiting on an async code that still would use thread pool to complete.

Do you mean that now, CoreFx has a bug and uses a threadpool thread to execute a longrunning task on it?
In the full Net Framework it used a dedecated thread. I will try to find the code for the default TaskScheduler for CoreFx to check it.

Yep, you are right. https://github.com/dotnet/coreclr/blob/master/src/System.Private.CoreLib/src/System/Threading/Tasks/ThreadPoolTaskScheduler.cs#L42 . Let's change it to Task.Run then.

Want to send a PR?

Yes, it will work.  But in my own implementation, i will use a dedicated thread. It is more robust and reliable in this case.

What is PR?

A pool request.

No. I dont use it. I found it only because of trying to create my own LogProvider. I used it as example.
I did some optimisations
https://github.com/aspnet/Extensions/issues/572
The log provider writes XML formatted messages, but in the rest it is almost the same.

Giving it a thought - even if it will stay like it was - with TaskCreationOptions.LongRunning, it won't harm too much. The dedicated thread will be used until the first truly async task, then it will simply exits and dies. All the rest of the code will be on threadpool threads, And even that depends on the SynchronizationContext and the task scheduler associated with it.


Pagination deosn't work when i use findBySql

Hello i have dataprovider, where i use findBySql. The reason to use findBySql is that i have dynamically "left joins" in my query.
This is my query:

$prodQuery = Product::findBySql('
SELECT product.*

' WHERE ( '
.$whereString.
' AND (language=''.$lang->url.'')) GROUP BY product.id ');

In $joinString i have all joins to one table and in $whereString i have other conditions.
Here is my dataprovider

$dataProvider = new ActiveDataProvider([
'query' => $prodQuery,
'pagination' => [
'totalCount' => count($prodQuery->all()),
'pageSize' => $pageSize,
'route' => Yii::$app->getRequest()->getQueryParam('first_step'),
],
'sort' => [
'defaultOrder' => $orderArr,
],
]);

And i my view i use Listview. The problem is that pagination and sotring doesn't work. In my test site i have 4 products, when i set pageSize = 1, pagination has 4 pages, but in every page i have all 4 products.
Where is my error? Or how can i rewrite it, to have dynamically joins, because at the beginning i tried with Products::find()->joinWith()... but then i cannot have dynamically joins.
Thanks in advance.

you should either use SqlDataProvider instead or create the joins dynamically by calling join() method on the query.


Character Encoding issue

While working on PR #237 to solve #217 , I used  this symbol  "©" inside my html input file but one test failed because of  this character then I started thinking about a way to solve that issue.
We should find a way to add encoding utf8 to the InputStream at the core level not when doing tests. Now, we add encoding when doing tests, we should correct that.
https://stackoverflow.com/questions/5928046/spring-mvc-utf-8-encoding
https://stackoverflow.com/questions/5649329/utf-8-encoding-problem-in-spring-mvc
https://stackoverflow.com/questions/29434896/how-to-deal-with-java-encoding-problems-especially-xml
https://www.baeldung.com/java-char-encoding
DOES "multipart/form-data handle utf8 encoding ?
https://www.google.com/search?q=DOES+"multipart%2Fform-data+handle+utf8+encoding+%3F&rlz=1C1FCXM_pt-PTPT1032PT1032&oq=DOES+"multipart%2Fform-data+handle+utf8+encoding+%3F&aqs=chrome..69i57j33i22i29i30.19637j1j7&sourceid=chrome&ie=UTF-8
https://issues.redhat.com/browse/RESTEASY-390
https://stackoverflow.com/questions/546365/utf-8-text-is-garbled-when-form-is-posted-as-multipart-form-data
java define encoding of InputStream
https://www.google.com/search?q=java+define+encoding+of+InputStream&rlz=1C1FCXM_pt-PTPT1032PT1032&oq=java+define+encoding+of+InputStream&aqs=chrome..69i57j33i160l2j33i22i29i30l4.19441j1j7&sourceid=chrome&ie=UTF-8
https://stackoverflow.com/questions/3043710/java-inputstream-encoding-charset

set the charset of bytes java
https://www.google.com/search?q=set+the+charset+of+bytes+java&rlz=1C1FCXM_pt-PTPT1032PT1032&oq=set+the+charset+of+bytes+java&aqs=chrome..69i57j33i160j33i22i29i30.22275j1j7&sourceid=chrome&ie=UTF-8
https://stackoverflow.com/questions/88838/how-to-convert-strings-to-and-from-utf8-byte-arrays-in-java
//Convert from String to byte[]:

String s = "some text here";
byte[] b = s.getBytes(StandardCharsets.UTF_8);

//Convert from byte[] to String:

byte[] b = {(byte) 99, (byte)97, (byte)116};
String s = new String(b, StandardCharsets.US_ASCII);

This issue is also related to non ascii characters:
https://www.baeldung.com/java-char-encoding
https://stackoverflow.com/questions/41690641/non-ascii-value-symbols-not-getting-printed
https://docs.oracle.com/javase/8/docs/api/java/text/Normalizer.html
https://www.tabnine.com/code/java/classes/java.text.Normalizer
https://www.educative.io/answers/what-is-stringutilsisasciiprintable-in-java
https://stackoverflow.com/questions/30111273/how-do-i-remove-copyright-and-other-non-ascii-characters-from-my-java-string
https://github.com/google/guava/wiki/StringsExplained
https://stackoverflow.com/questions/54752377/handling-strings-with-special-characters-in-java


Encode FNC3 endcode

Hello! I was looking for assistance creating a 128 code barcode with a trailing fnc3. I was instructed to use a bar code generator using 128 code format (symbology, if I'm using that correct), and have BT_ADR############.fnc3 as the characters where the #s are the mac/serial of a unit. When I use that under a normal barcode generator it will not work. In doing some research it appears there's a way to encode that fnc3? Is there anyone that could help me create that bar code properly?

"\x60" is fnc3 https://en.wikipedia.org/wiki/Code_128
"BT_ADR############.\x60"

@lindell Thank you! I tried this and it didn't work properly. I'm getting some contradicting information. They company I'm working with said to use .fnc3 (hex30) which doesn't add up based on what you linked me. It appears fnc3 is /60, so I'm wondering what it really should be. I have the working barcode, maybe this would help figure out what exactly I need to enter into a random barcode generator to reproduce?

Hello again, I was wrong about \x60. That is the internal representation. You need to look at locking the code128 code to one of the modes (b for example). And then use the representation in the "code" column.
Like this:
JsBarcode("#code128", "BT_ADR1804EDF104FF.\xC9", {format: "code128b", width: 1});

This does not result in the exact same barcode as yours, but there is a lot of ways of generating code128 codes, like shifting between the different modes etc. I will not be able to help you in trying to find out exactly which modes and which switches your barcode example is using, but the snippet above should provide a valid code to your spec.

Thanks for the help @lindell I really appreciate it. I'm way into the programming of barcodes and I'm very unfamiliar. I cannot seem to figure out how to recreate this barcode but I learned how a lot works and that somehow I need to encode this fnc3 trailer.  Thanks again!


Use reconciliation context in kubeconfig provider

Description
Changes proposed in this pull request:

use reconciliation context in kubeconfig provider

Looks fine, I would like to additionally test that on KCP/dev.

I don't see any issues on KCP/dev.