[BUG] New user DM doesn't appear on the list

New user Direct Message not found

Your Rocket.Chat.Android app version: 1.1.0
Your Rocket.Chat server version: 0.60.3
Device model (or emulator) you're running with: LG V20
Steps to reproduce:

Existing user can login to server as usual, read through channels and DMs fine.
On separate device (PC browser) create new account and login to server.
With New user send Direct Message to Existing user.
Existing user receives push notification that DM has been received in notification bar.
Existing user clicks notification bar to be taken to DM.
Rocket chat opens to default channel, DM and New user do not appear on the DM list. New user can be seen in default channel, click on New user name returns "User does not exist"
Close Native app, clear Cache and app Data. Re-sign in with Native app and the DM and New user will appear on unread list.

Issue is not valid anymore in version 2.0.0.

Thanks for closing this issue, I had forgotten about it.
The problem has been resolved in 2.0.0


Fix blocking command error

For contributor use:

[ ] Targeted PR against master branch
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work
[ ] Code follows the standards mentioned here
[ ] Updated relevant documentation
[ ] Re-reviewed Files changed in the Github PR explorer
[ ] Added appropriate labels

Codecov Report
Attention: Patch coverage is 0% with 6 lines in your changes are missing coverage. Please review.

Project coverage is 39.27%.  Consider uploading reports for the commit 95bee6c to get more accurate results

Files
Patch %
Lines

internal/super/setup.go
0.00%
6 Missing :warning:

Additional details and impacted files
@@                     Coverage Diff                      @@
##           feature/interactive-init    #1573      +/-   ##
============================================================
- Coverage                     39.33%   39.27%   -0.06%
============================================================
  Files                            52       52
  Lines                          3427     3432       +5
============================================================
  Hits                           1348     1348
- Misses                         1912     1917       +5
  Partials                        167      167

Flag
Coverage Δ

unittests
39.27% <0.00%> (-0.06%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

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


feat: scoped[NS] command (was: localized)

This just moves the localized command from Mathlib.Mathport.Syntax into its own file and adds some documentation. Also renames localized to scoped per the Zulip discussion.

bors merge


Inheriting from list or dict types

Change Summary
Attributes of classes inheriting from List or Dict are not yielded as instances of the derived class when validating. This change aims to solve this.
Related issue number
No related issued has been filed.
Checklist

[x] Unit tests for the changes exist
[x] Tests pass on CI and coverage remains at 100%
[x] Documentation reflects the changes where applicable
[x] changes/<pull request or issue id>-<github username>.md file added describing change
(see changes/README.md for details)

as noted in the docs, please create issues to discuss such changes before submitting a PR. It avoids wasting your time if the feature request is refused.
Here I'm inclined to refuse this chance since it may have a significant effect on performance.
Instead you should use custom data types (potentially use if TYPE_CHECKING to trick mypy if you're having typing problems). If that doesn't work, please create an issue explaining the problem you're having


Always Failed to load h5frida plugin";

Running iPhone 13 Pro non jailbreak.

Placed all files mentioned in .app folder
When running testscript for frida I got: "Failed to load h5frida plugin".

Running iPhone 13 Pro non jailbreak.

Placed all files mentioned in .app folder

When running testscript for frida I got: "Failed to load h5frida plugin".

If your side-loading h5gg you must include h5frida.dylib and the associated test dylib's when your signing your ipa

Reference

I am using sideloadly and put h5gg.deb into injection. Then place all dylibs and files mentioned in the .app folder when app is on phone. I dont need to sign it at all it is done by Sideloadly. Where do I need to include h5frida.dylib? In the Sideloadly? If so what else needs to be placed in the .app and what needs to be injected directly via Sideloadly?

You need to

Open the app (the one inside Payload)
Copy and Paste h5frida-15.1.24.dylib into it
Copy and Paste frida-gadget-15.1.24.config, frida-gadget-15.1.24.dylib, hookme.test.dylib into it as well
Repackage your ipa
Sideload the ipa with Sideloadly

inject dylibs/framework -> H5GGv7.9通用注入版.deb
Suggest to turn ON “Enable File Sharing”


Looking for where the dependencies on the tools (foodcritic, berks, etc) are defined

Description
I'm attempting to depend on the chef-dk gem in order to install all of the tools in chef-dk via bundle/Gemfile. I'm doing this mainly for cookbook CI purposes (although there are some other minor benefits). If I'm using a SaaS CI service where I don't have a whole lot of control in installing chef-dk via the usual packaged means, if I specify chef-dk in the Gemfile, then a simple bundle install followed by bundle exec TOOLNAME would give me all the tools in chef-dk. I could explicitly add foodcritic, berkshelf, etc, etc to the Gemfile, but it'd be simpler to be able to just depend on chef-dk.
However, I'm noticing via the gemspec that these tools aren't brought in via gems dependency. What's in /opt/chefdk/bin doesn't match the gemspec. (/opt/chefdk/embedded/bin definitely looks different, but I half expected that.)
So my (possibly dumb) question is, where are these dependencies defined? Is there a easy Gemfile-based way to depend on chef-dk and get all the tools? Or are these tools brought in a more complicated process than just define a set of dependencies?
ChefDK Version
0.14.25 - but this isn't really relevant to a specific version.
Platform Version
Mac for dev, Ubuntu for everything else - but this isn't really relevant to a specific platform.
Replication Case
N/A
Stacktrace
N/A

This is neither recommended nor hugely supported (in the literal sense, there will be little support from the community and you'll have to be comfortable spelunking the code). The Berks team has specifically stated they won't promise gem-based installs will always work. As for how to pull in all the deps, https://github.com/poise/poise-boiler/blob/master/poise-boiler.gemspec#L37-L73 is what I use but I say again that this is not a good idea.
Which CI are you on? Travis supports installing the ChefDK packages even on their docker workers via the addons system.

There isn't a gem/bundler way to do this. We did recently change the way we install things such that all our ruby tools are dependencies in ChefDK's Gemfile, so that might be an option. To answer your other question, everything in the package gets installed via omnibus (see the omnibus dir in this repo). In any case, we're not likely to put all the deps in the gemspec because that applies just to the chef CLI tool.

Well, that makes sense. Thanks.
@coderanger, I'm using codeship. I'm going to look into how to install chefdk on it. hopefully it is possible and i just missed it the first time.


[Feature Request] Host Image Login button Service

Is possible host image cPanel Plesk button? https://github.com/Katamaze/WHMCS-Free-Action-Hooks/blob/master/README.md#cpanel--plesk-login-button-in-my-services
templates/{$template}/img/katamaze_autologin/{$kt_autologin[$service.id]->type}.png
Thanks

I didn't catch that :worried: Can you clarify that for me?

You have upload icons below?

True, icons are not included. I need to find a way to include them on Github. In the meantime I attached the images in the zip:

Download the zip
Unizp it
Upload katamaze_autologin directory in templates/{YOUR_TEMPLATE}/img/ path

Thanks @Kian987
Is the border blue around the image is made via css?

Yes, it's a simple Bootstrap Button with image inside.


New Mapping Task

Mission details

Map center GPS position.
Map height in feet.

CRUD an image map for the mission.

16:9 aspect ratio.
PNG or JPG format.
Up to 10MB in size.

Interop CLI for image map.
Judge specified quality field added to scoring:

HIGH at 100% points.
MEDIUM at 50% points.
INSUFFICIENT at 0% points.

Mapping scored at 10% of points, avoidance dropped from 20% to 10%.
Frontend for setting the quality score.

May be best to wait for Angular transition and rely on Django admin in interim.

What's the mapping task about, may I ask? Or do I need to wait for the official rules to be released? :smiley:


🐛 BUG: wrangler dev wasm not working

What version of Wrangler are you using?
2.0.25
What operating system are you using?
Linux
Describe the Bug
I have installed the wrangler2 using the following command:
npm --location=global i wrangler@latest

I got the latest version of wrangler
wrangler -v

⛅ wrangler 2.0.25
When I try to run my wasm module using wrangler dev I get the following error:
✘ [ERROR] No loader is configured for ".wasm" files: target/wasm32-wasi/release/hello-world.wasm

Yeah, it's broken. Same thing on Windows, and when trying to publish

To help us diagnose this further, would you be able to provide some more information about your environment? Specifically, the contents of your wrangler.toml would be really useful!

Hey 👋 !
We haven't heard from you about this so we are going to close this. If you have further clarification on this please feel free to reopen this, or open another issue with some more details

I hit the exact same issue when following the steps in this blog post
Seems like everything in that blog post just doesn't work:tm:
For instance also: https://hexly.examples.workers.dev/ 404's
fwiw here's my wrangler.toml
name = "cf-workers-test"
compatibility_date = "2022-09-30"

And how I got there:
> wrangler2 dev target/wasm32-wasi/release/hello_world.wasm

But also, did this ever work?
https://github.com/cloudflare/wrangler2/blob/main/packages/wrangler/src/entry.ts#L156

@cameron-robey @penalosa

It looks like one way to work around the problem is to set up a rollup build and treat the wasm bundle as an external. I found an example of that here: https://github.com/ssssota/svg2png-worker .
That said, it would be great if wasm worked out of the box.


Crash when note is deleted

Deleting first note after launching program causes app to crash.

CurrentNote is note reassigned after deleting currentNote.


validate html

To test that we are good boys

Almost there

Great work! I think we have a bit too many Dockerfiles now, so let's see if we can reduce that in a future PR. This is good to go, though. 👍

Reducing the amount can't wait.


Implement more robust path behavior

When .from_csv expects the base path and appends the respective participant ending by default (e.g. switch_input.csv).
It should also work when the ending is included in the path.
We should also support path objects.

Update the type hints with some custom PathLike union type


Created Branch

Codecov Report

Merging #1 into master will increase coverage by 0.26%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master       #1      +/-   ##
==========================================
+ Coverage   75.79%   76.05%   +0.26%
==========================================
  Files         188      188
  Lines        8792     8792
==========================================
+ Hits         6664     6687      +23
+ Misses       1538     1522      -16
+ Partials      590      583       -7

Impacted Files
Coverage Δ

proxy/shadowsocks/server.go
62.23% <0%> (-0.7%)
:arrow_down:

transport/internet/kcp/connection.go
75.28% <0%> (+0.56%)
:arrow_up:

proxy/socks/server.go
76.15% <0%> (+0.76%)
:arrow_up:

app/proxyman/mux/mux.go
70.29% <0%> (+0.83%)
:arrow_up:

proxy/freedom/freedom.go
77.9% <0%> (+1.16%)
:arrow_up:

proxy/vmess/outbound/outbound.go
75.51% <0%> (+2.04%)
:arrow_up:

transport/internet/headers/http/http.go
81.56% <0%> (+2.12%)
:arrow_up:

common/buf/copy.go
76.47% <0%> (+7.84%)
:arrow_up:

transport/internet/kcp/listener.go
84.54% <0%> (+8.18%)
:arrow_up:

Continue to review full report at Codecov.

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


re-training needed for new pdb files? evaluating the AF2-multimer prediction?

Hi,
I have pdb files of 2 molecules (an antibody and an antigen) predicted by Alphafold2-multimer.
Do I need to re-train RDE-PPI to run pointmut_analysis.py?
If I am not sure about the quality of the AF2-multimer prediction, could the results of doing a mutational sweep with pointmut_analysis.py give me an indication of the validity of the AF2-multimer pdb? What should I be looking for in the resulting numbers?
Thx in advance.

Hi,
Generally, there is no need to re-train the model for other PPI structures.
The model is expected to work well on experimentally determined structures. I am not sure whether it can give some indication of the quality of AF2-predicted structures. Perhaps the entropy might have some weak correlation with the quality of predicted structures? As entropy can be considered as some energy function, native-like structures should have lower entropy.
The pointmut_analysis.py script outputs a csv file pm_results.csv. The first column is the mutation name, the second column is ddG (lower is implies higher binding affinity), and the last column is the percentile ranking.

Thanks.

Thanks.
What's the difference between the mutations: section in the yml file and
the interests: section?
Where should I do the sweep of antibody amino acid changes? Would doing a
sweep on antigen amino-acid changes give any relevant information?
Out of an ensembl of pdbs, would the one with the minimal ddG for the full
antibody amino-acid sweep be the most "believable" prediction?
Thanks in advance.
On Mon, Mar 6, 2023 at 10:39 AM Shitong Luo @.***>
wrote:

Hi,
Generally, there is no need to re-train the model for other PPI structures.
The model is expected to work well on experimentally determined
structures. I am not sure whether it can give some indication of the
quality of AF2-predicted structures. Perhaps the entropy might have some
weak correlation with the quality of predicted structures? As entropy can
be considered as some energy function, native-like structures should have
lower entropy.
The pointmut_analysis.py script outputs a csv file pm_results.csv. The
first column is the mutation name, the second column is ddG (lower is
implies higher binding affinity), and the last column is the percentile
ranking.
[image: image]
https://user-images.githubusercontent.com/45375360/223086769-33379d02-e641-4eef-bcf6-373554103ad4.png
Thanks.
—
Reply to this email directly, view it on GitHub
https://github.com/luost26/RDE-PPI/issues/2#issuecomment-1455888348, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AABGSN54EVQOPR3BC7YMRMDW2W5FTANCNFSM6AAAAAAVQ5YEZI
.
You are receiving this because you authored the thread.Message ID:
@.***>

interest section is optional. It specifies which mutations of interest will be printed directly to the screen after finishing.

In principle, anywhere on the binding interface (both antibody and antigen) works.

I think it is reasonable but I am not sure about it.

Got it, thanks.
On Mon, Mar 6, 2023 at 11:22 AM Shitong Luo @.***>
wrote:

interest section is optional. It specifies which mutations of interest
will be printed directly to the screen after finishing.
2.
In principle, anywhere on the binding interface (both antibody and
antigen) works.
3.
I think it is reasonable but I am not sure about it.
—
Reply to this email directly, view it on GitHub
https://github.com/luost26/RDE-PPI/issues/2#issuecomment-1455952782, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AABGSN4WTS4MO25TUONWRT3W2XCGJANCNFSM6AAAAAAVQ5YEZI
.
You are receiving this because you authored the thread.Message ID:
@.***>


Error in events.js:187 in 2.275.0 runner

This has been working for the last two weeks. Now I'm getting this:
✗  kubectl -n github-runner get all
NAME                                    READY   STATUS             RESTARTS   AGE
pod/organizational-runner-bg7ks-726r5   1/2     CrashLoopBackOff   10         37m
pod/organizational-runner-bg7ks-dfkqd   1/2     Error              11         36m
pod/organizational-runner-bg7ks-n92sz   1/2     CrashLoopBackOff   10         36m
pod/organizational-runner-bg7ks-tlz97   1/2     CrashLoopBackOff   7          18m

✗  kubectl -n github-runner logs pod/organizational-runner-bg7ks-726r5 runner
...
Started listener process
Started running service

√ Connected to GitHub

2020-12-16 21:34:50Z: Listening for Jobs
Runner update in progress, do not shutdown runner.
Downloading 2.275.0 runner
Waiting for current job finish running.
Generate and execute update script.
Runner will exit shortly for update, should back online within 10 seconds.
renamed '/runner/_diag/SelfUpdate-20201216-211735.log' -> '/runner/_diag/SelfUpdate-20201216-211735.log.failed'
Runner listener exited with error code 4
Runner listener exit with undefined return code, re-launch runner in 5 seconds.
Starting Runner listener with startup type: service
Started listener process
events.js:187
      throw er; // Unhandled 'error' event
      ^
Error: spawn /runner/bin/Runner.Listener ENOENT
    at Process.ChildProcess._handle.onexit (internal/child_process.js:264:19)
    at onErrorNT (internal/child_process.js:456:16)
    at processTicksAndRejections (internal/process/task_queues.js:80:21)
Emitted 'error' event on ChildProcess instance at:
    at Process.ChildProcess._handle.onexit (internal/child_process.js:270:12)
    at onErrorNT (internal/child_process.js:456:16)
    at processTicksAndRejections (internal/process/task_queues.js:80:21) {
  errno: 'ENOENT',
  code: 'ENOENT',
  syscall: 'spawn /runner/bin/Runner.Listener',
  path: '/runner/bin/Runner.Listener',
  spawnargs: [ 'run', '--startuptype', 'service', '--once' ]
}

It looks like there is a hot fix in 2.275.1:
https://github.com/myoung34/docker-github-actions-runner/commit/b1dced691a0df0bc3fbad5d024152354f5df14d0

@mabushey we were having the same issue. Updated our custom runner image with 2.275.1 as per https://github.com/summerwind/actions-runner-controller/pull/239 and all good now 👍


Add message errors when the UI fails to fetch

Currently when the UI fails to fetch data this is handled silently. Instead, some sort of warning should be shown.

The error is shown in console, currently this warning has a low priority


how to convert a pytorch model to caffe2?

torch -> caffe -> caffe2 seems OK.  But how can I convert a pytorch model to caffe2? or is there is common way to deal with pytorch->torch ?

@qwliu You should look at the following links:
http://pytorch.org/tutorials/advanced/super_resolution_with_caffe2.html
http://pytorch.org/docs/master/onnx.html
Let me know if it's helpful.

@MatanHugi  It is really helpful.
Without tools like onnx, I'v chosen to transform my models to mxnet and apply its c++ api to deploy, which turned to be tremendously painful.
THANKS TO ONNX!


8787 and 18881 port permission required for service tag AzureMachineLearning

[Enter feedback here]
The table on the following site does not list the permission for port 8787, 18881 for the service tag AzureMachineLearning.
But it is actually required. Please add it.
Required public internet access

Outbound configuration

In the following document, an outbound connection to *.instances.azureml.ms is needed for ports 443, 8787, and 18881.
Azure Machine Learning hosts doc,

But, In the communication control of FQDNs using Azure Firewall application rules, rules can basically only be defined for port numbers 443, 80, and 3306.
Network rules and applications rules

Based on the above, since it is not possible to add ports 8787 and 18881 for outbound connections to *.instances.azure.ms on the Azure Firewall's application rules, please add ports 8787 and 18881 in the outbound connection to the service tag AzureMachineLearnig.

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

ID: 2eeeb5a8-3587-0932-0d80-6a47414c82c6
Version Independent ID: 34958145-3803-c078-c1e2-798169b329b2
Content: Configure inbound and outbound network traffic - Azure Machine Learning
Content Source: articles/machine-learning/how-to-access-azureml-behind-firewall.md
Service: machine-learning
Sub-service: enterprise-readiness
GitHub Login: @jhirono
Microsoft Alias: jhirono

@KazuOnuki Thanks for the question. We are investigating the issue and will update you shortly.

@jhirono Could you please review and update the document for 8787 and 18881 port permission.

@KazuOnuki We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Loading Spinner Added

A loading spinner has been implemented to replace the text "Loading..." and will now appear centered on the page.

Thank you @nabanichowdhury for your contribution. Merging this to main now.


Support SQL-on-FHIR v2 in data-pipes

We should be able to consume SQL-on-FHIR v2 JSON specs in the pipelines directly. The goal is to create flat views defined by this spec directly from the pipelines. Note this would be the last step of our SQL-on-FHIR v2 support road-map, which is:

Rely on FHIR-views to implement full v2 spec.
FHIR-views should then be able to generate JSON configs for the views and also read JSON files of the v2 specs.
Read v2 JSON configs directly in the transformation pipeline and apply them on the FHIR resources (i.e., this issue).

Note, modification/recreation of views should not require fetching all resources from the FHIR server again. IOW, we should have an efficient intermediate storage to be able to reload the FHIR resources and apply the v2 config on them.

This is done except for the incremental part which is being tracked under #916.


http://epicmax.co/vuestic is 404ing

Just a heads up, looks like the example site is down.

@jhendley25 we've killed our old website and vuestic page with it, will change all the links leading either to Vuestic Demo or it's GitHub repo


Supervisor/foreground mode

This is a small todo list for tasks that can be worked on for decoupling the runtime for containers from the actual daemon.  I'm trying to make this list from what we discussed the other day.  I think each of these tasks can be broken down into small PRs to save us from a mega merge.
I know there is alot missing from the list but it's a start, if you think of more just update the issue or comment.
ping @creack @vieux @alexlarsson @shykes
TODO:

[ ] Use fork/exec in dockerinit so that we are PID 1
[ ] move state out of the container struct
[ ] allocate IPs on container create and let them last the lifetime of the container (maybe)
[ ] write code to reap zombies if we are PID 1
[ ] decouple diff from the graph drivers

Has this gone anywhere? Is anyone/did anyone work on it?
I assume it's a prerequisite for #6851. Over a year on, is this still an accepted proposed design?


[SDPA-4967] - Added logic to limit nav depth to 3

Motivation and Context

JIRA issue: https://digital-engagement.atlassian.net/browse/SDPA-4967
The current IA has a depth of 5 but the nav design breaks after a depth of 3.
Changed

Added logic to limit nav depth to 3

Screenshots
Before:

After:

How Has This Been Tested?

Types of changes

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

Checklist

[ ] I've added relevant changes to the documentation.
[ ] I have added tests to cover my changes (if not applicable, please state why)
[ ] My change requires a template update for create-ripple-app.
[ ] I have added template update script for next release.

LGTM @ChrisCampbellCodes. Probably also need a corresponding fix on the backend to limit editors adding deeply nested menus in the first place. Relates to #1100

Thanks @dylankelly ! The backend menu structure still needs to go to 5 levels deep to meet the IA, this also allows the breadcrumbs etc to show 5 levels deep. I'll follow up with the BE team and ensure that the menu can only go to 5 levels deep


libusb: Pipe error

[x] I have read the FAQ.
[x] I have searched in existing issues.

Environment

OS: Windows 11
scrcpy version: 2.1
installation method: chocolatey
device model: Samsung A23 5G
Android version: 13

Describe the bug
The error occurs when I try to use OTG mode. Using the keyboard and mouse is fine when I use the ADB but trying to use scrcpy --otg or any other option (referring to --hid-mouse and --hid-keyboard) ends up with a console output like this:
C:\Users\Admin>scrcpy --otg
scrcpy 2.1 <https://github.com/Genymobile/scrcpy>
INFO: Killing adb server (if any)...
INFO: USB device:<PHONE_NUMBER>01 (04f2:b72b) Chicony Electronics Co.,Ltd. HD User Facing
WARN: On this platform, libusb does not have hotplug capability; device disconnection will not be detected properly
ERROR: REGISTER_HID: libusb error: Pipe error
WARN: Register HID keyboard failed

It worked, thanks.
Didn't completely do the fix because somehow I can't install Google's WinUsb drivers on my phone but the default ADB device driver works. Is there a way to do this without having to use USB debugging on android? Some apps restrict themselves if  developer mode is turned on.

Yes, --otg work with USB debugging turned off (except it's complicated on Windows because drivers may not detect the USB device if USB debugging is off, but on other platforms it works).


Kuroko does not read files with Windows \r\n newlines

I was trying out Kuroko 1.1.1 on Windows 10, and I noticed that in souce files I wrote with multiple lines I would get the error:
kuroko: could not read file 'hello.krk': No error
Upon changing my newlines to UNIX (\n), Kuroko could read the file perfectly.
Thus it is safe to assume that kuroko.exe was failing to parse files that use Windows (\r\n) newlines.

Odd, that message would suggest an issue with the fread call... is Windows giving a different size when I seek to the end of the file vs. the number of bytes it returns from the read because it’s doing newline conversion underneath? I don’t do much testing directly under Windows beyond the test suite, but I’ll see if removing the exact size check is enough to resolve this.

I'll check this again at the next stable release


Raise an exception when ClassDefBuilder attempts to add a duplicate glyph.

There is currently an assertion in ClassDefBuild.add that stops a glyph that is already present in a class from being added again. However, there is no error reported, and thus it can be quite difficult to debug.
This PR changes this assertion into a raised exception, while also adding the related tests.

hm, this reminded me of https://github.com/fonttools/fonttools/issues/2120
@dscorbett can you take a look?

hm, this reminded me of https://github.com/fonttools/fonttools/issues/2120
@dscorbett can you take a look?

It's unclear to me when exactly this can occur. Could you provide a feature file snippet that reproduces this error?

It's unclear to me when exactly this can occur. Could you provide a feature file snippet that reproduces this error?

I don’t remember the exact circumstances where I saw this, but I think it was in a contextual substitution rule that used a class containing the same glyph multiple times.

I don’t remember the exact circumstances where I saw this, but I think it was in a contextual substitution rule that used a class containing the same glyph multiple times.


There is not  features introduce

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

There are no features introduces

in the readme there is feature list. what kind of introduces you mean


Print Formatting

When the user hits Ctrl+P or ⌘+P it should re-format the page to make a print compatible layout.

@nywillb and when they type Ctrl+HEX it should hex the nsa...... (i am saying WHY IS THIS NECESSARY)

@willweinstein This is why it is nessicary:
PlanHub.pdf
PlanHub2.pdf

This probably will use a separate print stylesheet that does magical CSS stuff, rather than trying to detect when you press the print button.

Should be done with #275.
Closing it, but when are we going to pull and poke?
Paging @thatoddmailbox


Included specs can no longer be referenced in XcodeGen 2.33.0

I have a project structure similar to this:
├── Configuration
│   ├── project.yml
│   └── targets.yml
└── Sources
    └── MyApp.swift

project.yml
name: MyProject
include:
  - targets.yml
options:
  bundleIdPrefix: com.myapp

targets.yml
targets:
  MyApp:
    type: application
    platform: iOS
    deploymentTarget: "16.0"
    sources: [Sources]

In XcodeGen 2.32.0 and below, I could generate my Xcode project from the root directory with the following command:
xcodegen --spec ./Configuration/project.yml --project-root . --project .
Since XcodeGen 2.33.0, I get the following error message:
Parsing project spec failed: Error Domain=NSCocoaErrorDomain Code=260 "The file “targets.yml” couldn’t be opened because there is no such file." UserInfo={NSFilePath=targets.yml, NSUnderlyingError=0x11ee32fd0 {Error Domain=NSPOSIXErrorDomain Code=2 "No such file or directory"}}
I tried to change the path of targets.yml to Configuration/targets.yml, but this also generates an error message:
Spec validation error: Target "MyApp" has a missing source directory "Configuration/Sources"
Is this a bug, or do I have to update my configuration? I didn't read about any breaking changes in the latest release notes. Thank you!

Probably introduced in this commit where the project root is used (if it is given) instead of the current path.
From the changelog, it seems to be the intention that both the project root, as well as the current directory are considered for includes. So this seems like a defect.

cc @ma-oli

Any updates on this issue?

@FelixLisczyk If you've got a chance, could you try out https://github.com/yonaskolb/XcodeGen/pull/1262 and let me know if it fixes the issue.

Thanks, Craig! I've checked out the PR branch and built XcodeGen using make build. Unfortunately, the error message remains the same. I'll attach my sample project, which you can use to reproduce the issue. The build command is xcodegen --spec ./Configuration/project.yml --project-root . --project .
XcodeGen-TestProject.zip

This issue still occurs in the latest XcodeGen 2.34.0.

Probably introduced in this commit where the project root is used (if it is given) instead of the current path.

In my case 2.34.0 (and 2.33) fail even if -r is not specified:
$ mint run<EMAIL_ADDRESS>⚙  Generating plists...
⚙  Generating project...
⚙  Writing project...
Created project at /Users/eg/Workbench/TMBuddy/feature/XcodeGen-Test/TMBuddy.xcodeproj
$ mint run<EMAIL_ADDRESS>Parsing project spec failed: Error Domain=NSCocoaErrorDomain Code=260 "The file “Compact.yml” couldn’t be opened because there is no such file." UserInfo={NSFilePath=/Users/eg/Workbench/TMBuddy/feature/Compactness/Compact.yml, NSUnderlyingError=0x600003c9e520 {Error Domain=NSPOSIXErrorDomain Code=2 "No such file or directory"}}
$ mint run<EMAIL_ADDRESS>Parsing project spec failed: Error Domain=NSCocoaErrorDomain Code=260 "The file “Compact.yml” couldn’t be opened because there is no such file." UserInfo={NSFilePath=/Users/eg/Workbench/TMBuddy/feature/Compactness/Compact.yml, NSUnderlyingError=0x6000003982d0 {Error Domain=NSPOSIXErrorDomain Code=2 "No such file or directory"}}
$

@ma-oli are you able to look into this as I believe this PR  broke things

I was finally able to fix this issue by:

Referencing the include files with their path relative to the project root directory.
Adding relativePaths: false for each include entry.

Here is the updated project.yml file from my original post:
name: MyProject
include:
  - path: Configuration/targets.yml
    relativePaths: false
options:
  bundleIdPrefix: com.myapp

All paths inside targets.yml must also be relative to the project root directory.
I've tested this approach with the latest XcodeGen 2.35.0.


Barycentric Coordinates error?

Hi, this project is a great work with fine details.
But here is a little problem, when I tried the shade generation function, the loaded predefined barycoord is zero everywhere actually.
https://github.com/tranluan/Nonlinear_Face_3DMM/blob/1c0c7d388231e26460303134ad9b0f87357d010c/_3dmm_utils.py#L122-L130
Which means the shade generation function below is actually makes non-sense. The generated shade is the same every in the texture.
https://github.com/tranluan/Nonlinear_Face_3DMM/blob/1c0c7d388231e26460303134ad9b0f87357d010c/model_non_linear_3DMM.py#L158
So, it is designed like this or the file '3DMM_tri_2d_barycoord.dat'  is broken?

@tranluan Hi, I meet the same problem, can you check it ?


Revert "chore: Enable upgrade of knative,controller-runtime,k8s in go.mod"

Reverts aws/karpenter#2447

Broke logging as controller-runtime v0.11.x doesn't pass context through to Reconcile call


robyn_refresh from the demo script fails due to lambda_max column not set

Project Robyn
Describe issue
When I run the current demo.R script, it fails on the robyn_refresh stage due to lambda_max column not set in listInit$OutputCollect@resultHypParam. Error message below:
Warning message:
“Unknown or uninitialised column: `lambda_max`.”
Error in if (newLowB < getRange[1]) {: argument is of length zero
Traceback:

1. robyn_refresh(json_file = json_file, dt_input = dt_simulated_weekly,
 .     dt_holidays = dt_prophet_holidays, refresh_steps = 13, refresh_iters = 1000,
 .     refresh_trials = 1)
2. refresh_hyps(initBounds = Robyn$listInit$OutputCollect$hyper_updated,
 .     listOutputPrev, refresh_steps, rollingWindowLength = InputCollectRF$rollingWindowLength)

Provide reproducible example
The current demo.R script (2c4fa20) without any modifications
Environment & Robyn version
Make sure you're using the latest Robyn version before you post an issue.

Check and share Robyn version: packageVersion("Robyn"): <IP_ADDRESS>00
R version (Please, check and share: sessionInfo() or R.version$version.string)

Platform: x86_64-pc-linux-gnu (64-bit)
Running under: Debian GNU/Linux 10 (buster)

Matrix products: default
BLAS:   /usr/lib/x86_64-linux-gnu/openblas/libblas.so.3
LAPACK: /usr/lib/x86_64-linux-gnu/libopenblasp-r0.3.5.so

locale:
 [1] LC_CTYPE=C.UTF-8       LC_NUMERIC=C           LC_TIME=C.UTF-8
 [4] LC_COLLATE=C.UTF-8     LC_MONETARY=C.UTF-8    LC_MESSAGES=C.UTF-8
 [7] LC_PAPER=C.UTF-8       LC_NAME=C              LC_ADDRESS=C
[10] LC_TELEPHONE=C         LC_MEASUREMENT=C.UTF-8 LC_IDENTIFICATION=C

attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods   base

other attached packages:
[1] doRNG_1.8.6       rngtools_1.5.2    foreach_1.5.2     Robyn_<IP_ADDRESS>00

loaded via a namespace (and not attached):
 [1] nlme_3.1-162         bitops_1.0-7         matrixStats_0.63.0
 [4] lubridate_1.9.2      doParallel_1.0.17    RColorBrewer_1.1-3
 [7] httr_1.4.5           rprojroot_2.0.3      rstan_2.21.8
[10] repr_1.1.6           tools_4.2.3          utf8_1.2.3
[13] R6_2.5.1             rpart_4.1.19         mgcv_1.8-42
[16] colorspace_2.1-0     withr_2.5.0          tidyselect_1.2.0
[19] gridExtra_2.3        prettyunits_1.1.1    processx_3.8.1
[22] compiler_4.2.3       textshaping_0.3.6    glmnet_4.1-7
[25] cli_3.6.1            rvest_1.0.3          xml2_1.3.3
[28] labeling_0.4.2       scales_1.2.1         ggridges_0.5.4
[31] callr_3.7.3          rappdirs_0.3.3       systemfonts_1.0.4
[34] pbdZMQ_0.3-9         stringr_1.5.0        digest_0.6.31
[37] StanHeaders_2.21.0-7 extraDistr_1.9.1     base64enc_0.1-3
[40] pkgconfig_2.0.3      htmltools_0.5.5      fastmap_1.1.1
[43] rlang_1.1.0          shape_1.4.6          prophet_1.0
[46] generics_0.1.3       farver_2.1.1         jsonlite_1.8.4
[49] dplyr_1.1.2          zip_2.3.0            inline_0.3.19
[52] RCurl_1.98-1.12      magrittr_2.0.3       loo_2.6.0
[55] patchwork_1.1.2      Matrix_1.5-3         Rcpp_1.0.10
[58] IRkernel_1.3.2       munsell_0.5.0        fansi_1.0.4
[61] reticulate_1.28      lifecycle_1.0.3      stringi_1.7.12
[64] pROC_1.18.0          yaml_2.3.7           pkgbuild_1.4.0
[67] plyr_1.8.8           grid_4.2.3           parallel_4.2.3
[70] crayon_1.5.2         lattice_0.20-45      IRdisplay_1.1
[73] splines_4.2.3        lares_5.2.1          ps_1.7.5
[76] pillar_1.9.0         uuid_1.1-0           codetools_0.2-19
[79] stats4_4.2.3         glue_1.6.2           evaluate_0.20
[82] rpart.plot_3.1.1     RcppParallel_5.1.7   png_0.1-8
[85] vctrs_0.6.2          nloptr_2.0.3         gtable_0.3.3
[88] purrr_1.0.1          tidyr_1.3.0          ggplot2_3.4.2
[91] openxlsx_<IP_ADDRESS>     h2o_<IP_ADDRESS>         ragg_1.2.5
[94] survival_3.5-3       minpack.lm_1.2-3     tibble_3.2.1
[97] iterators_1.0.14     timechange_0.2.0     here_1.0.1

Hi @lukmaz
Thanks for reporting this bug. I've just reverted one of the changes we did a couple of weeks ago to fix this issue. I was able to replicate the error and after fixing it, it all looks back to normal. Please, do update to latest dev version, retry and let us know how it went. Appreciate it.

I confirm that after the fix the issue is resolved. Thanks Bernardo!


Discuss: Provide an option to copy in all questions?

We now have a lot of questions that use methods like name_fields() etc. Given how common it is to customize question prompts while leaving the fields alone, should we make that an option in the Weaver?
Currently since we have so many generic object questions, you can't just copy/paste the question as-is from the ql_baseline.yml file. You need to change the object names as well and remove the generic object specifier.
We could add a checkbox option that generates the question to go along with the object. We could also allow someone to edit that question in-place.

On Monday 7/19/21 we decided against this fix.

I had a new thought for how this might work: towards the end of the interview, let someone check a box for the screens they want to customize. It might be necessary to show a preview of the screen without customization so it makes sense to them though.
In general--would a preview of each screen in the reorder screens table be useful?
Might depend on work connected to https://github.com/SuffolkLITLab/docassemble-ALWeaver/issues/504

Thinking about this again a year later. I think we can just have a small confirmation screen that shows people the list of variables that will get a question from ql_baseline.yml and let them check a box to copy it in.
Of course "copy it in" is going to mean maintaining an extra copy of the questions, separate from ql_baseline.yml. So that isn't cost-free.


qdrant[patch]: run python 3.13 in CI

Integration tests pass with 3.13: https://github.com/langchain-ai/langchain/actions/runs/12055188348


Send metrics from Ops Agent main service

Description
Proof of concept on sending the agent/ops_agent/enabled_receivers metric from the Ops Agent main service (google-cloud-ops-agent). This implementation considers the future possibility of collecting and sending additional metrics obtained from the Ops Agent configuration.
The features implemented in this PR consist :

Send metrics collected from the Ops Agent configuration to the Cloud Monitoring API.
Register metrics to be sent every n minutes.
Register metrics to be sent at different time intervals and couple Monitoring API requests when possible.

Related issue
b/232815588
How has this been tested?
Verified the correct collection of the agent/ops_agent/enabled_receivers metric in the Metrics explorer. After corroborating the feasibility of this approach, further testing and implementation of integration tests will be completed.

Checklist:

Unit tests

[x] Unit tests will be implemented after design is approved.
[ ] Unit tests do not apply.
[ ] Unit tests have been added/modified and passed for this PR.

Integration tests

[x] Integration tests will be implemented after design is approved.
[ ] Integration tests do not apply.
[ ] Integration tests have been added/modified and passed for this PR.

Documentation

[ ] This PR introduces no user visible changes.
[x] This PR introduces user visible changes and the corresponding documentation change has been made.

Minor version bump

[ ] This PR introduces no new features.
[x] This PR introduces new features, and there is a separate PR to bump the minor version since the last release already.

Some initial questions/comments.

Adressed comments.


IPSet module

Created a module to support the ipset that could add the domain's ip to a list easily.
Purposes:

In my case, I can't access the facebook, twitter, youtube and thousands web site for some reason. VPN is a solution. But the internet too slow whether all traffics pass through the vpn.
So, I set up a transparent proxy to proxy the traffic which has been blocked only.
At the final step, I need to install a dns service which would work with ipset well to launch the system.
I did some research for this. Unfortunately, Unbound, My favorite dns service doesn't support ipset yet. So, I decided to implement it by my self and contribute the patch. It's good for me and the community.

# unbound.conf
server:
  ...
  local-zone: "facebook.com" ipset
  local-zone: "twitter.com" ipset
  local-zone: "instagram.com" ipset
  more social website

ipset:
  name-v4: "gfwlist"

# iptables
iptables -A PREROUTING -p tcp -m set --match-set gfwlist dst -j REDIRECT --to-ports 10800
iptables -A OUTPUT -p tcp -m set --match-set gfwlist dst -j REDIRECT --to-ports 10800

This patch could work with iptables rules to batch block the IPs.

# unbound.conf
server:
  ...
  local-zone: "facebook.com" ipset
  local-zone: "twitter.com" ipset
  local-zone: "instagram.com" ipset
  more social website

ipset:
  name-v4: "blacklist"
  name-v6: "blacklist6"

# iptables
iptables -A INPUT -m set --set blacklist src -j DROP
ip6tables -A INPUT -m set --set blacklist6 src -j DROP

Notes:

To enable this module the root privileges is required.
Please create a set with ipset command first. eg. ipset -N blacklist iphash

How to use:
./configure --enable-ipset
make && make install

Configuration:
# unbound.conf
server:
  ...
  local-zone: "example.com" ipset

ipset:
  name-v4: "blacklist"

Hi,
This pull request looks real good!  Could you spend some words to describe a way that it can be used.  In what situation would users use this feature?  Or how can the IPset get used (for example)?
From what I read it adds the IP addresses from the *.example.com A and AAAA lookups into the IPset. And that IPset can be used by the Firewall?  Is that correct?
Best regards, Wouter

Hi,
This pull request looks real good! Could you spend some words to describe a way that it can be used. In what situation would users use this feature? Or how can the IPset get used (for example)?
From what I read it adds the IP addresses from the *.example.com A and AAAA lookups into the IPset. And that IPset can be used by the Firewall? Is that correct?
Best regards, Wouter

Thank you for your reply. I have already updated the description for the PR. Please check and have fun.

Hi,
This patch can also be used in the Whitelist of a NAC (Network Access Controler). For example, in our project ALCASAR, we use this behavior with DNSMask. For the next version of ALCASAR, we've decided to replace 'Dnsmasq' with 'Unbound' (due to the better perfs). However, we must keep one instance of 'dnsmasq' (just behind 'unbound') in order to deal with the firewall and its ipset. With this module, we won't need dnsmasq anymore.

Thank you for the PR!  The documentation you write, I have put in doc/README.ipset.md, so that other people can reference it.

Many thanks for implementing this feature! Do you know the how many local-zone rules can Unbound handle?

There is no limit on it.  It uses memory to store them.  You can have as many as you like.  If you have a local-zone without local-data in it, it uses less memory.  It then also uses more time to read them in.  Consider using the include: "file" directive to organize the lists.

Thank you for this patch but I can't get this working.
I want to use this on OpenWRT.
So I modified the make file to enable ipset support and installed the libmnl headers.
Compiled fine.
Edited my unbound conf:
server:
...
module-config: "ipset validator iterator"
...
local-zone: "example.com" ipset
...
ipset:
name-v4: "test4"
name-v6: "test6"
Created the ipsets:
ipset create test4 iphash
ipset create test6 iphash
But nothing is added to the sets.
I tried to increase the verbosity but nothing relevant is showing up.
And is it possible to have multiple ipsets?

The previous commit should allow you to see in the logs (set verbosity high up for it) when an ipaddress is added to the set.

The output should look like this (for me in a test), with verbosity 2 or higher:
debug: ipset: add <IP_ADDRESS> to test4 for www.example.com.

i probably have the same issue as @schm0 although i get the log message as described nothing is appended to the ipset at all. I also found some issues in addition to that.

The confgparser.y does not allow local-zone statements of type ipset in views. It appears as it was just forgotten as i can see code added in that block too. I have a patch for this but its trivial. However. neither using this module globally nor in views works for me.

Second and in my opinion more serve is that it wont work if i have a local-zone set for "." in my view which is of type "refuse", "static" or similar. While this might seem obvious as there is no zone-data i don't know how to use this module to whitelist IP addresses in my firewall.

What i would like to do is having a special view for clients with restricted internet access. i.e. my TV set should not be able to access the internet by default but there are some cases where i want to allow it based on the response of unbound for i.e. "pool.ntp.org" as far as i can see the way this module is implemented does not allow such configuration at all. Its not a problem if you don't want to avoid responding with addresses that are not routable but doing so makes my TV set a lot less noisy with regards to connection attempts (that will fail repeatedly).

The documentation is missing that we need to add this module to the module-config line. If you already have setup another module before you probably know already about this but i was left confused at first. Moreover i recognized something strange. If i omit that line i can see in the log that 3 modules are loaded "subnet", "validator" and "iterator". However, adding the module-config line does not allow me to load the subnet module (whatever it does) and denies startup with this message:

fatal error: module_conf lists module 'subnet' but that module is not available.
But this is probably completely unrelated to this module and should be addressed in a separate ticket.

Additionally i would like to add that i would have a use-case for multiple ipset lists which currently is not possible either.

@mercora Thank you for your suggestion. Basically, I think to support the multiple ipset is the answer to solve the problem that you facing.
Unfortunately, This patch does not aim for that at the beginning.
I realized to support the multiple ipset is a useful feature for users.But I can't guarantee to you I can implement.

ok,sounds reasonable. Then lets find out why it is not working as is for now :)

Hi,
@k9982874 I hope you can see this comment here.
We would like to include the ipset module under the BSD license. It was contributed by you (Kevin Chou) but you did not mention the license.  Is it okay to include the ipset module under the 3-clause BSD license that we use for Unbound?
Best regards, Wouter

@wcawijngaards No worries. It's ok.

Thanks for the quick reply!  We try to keep our licensing as clear as possible.
Best regards, Wouter

感谢开发者的贡献，如果可以支持多ipset就更好了，因为有时需要区分不同网站走不同线路。

Hi all, are there plans for the ipset support in this module to be extended to allow multiple ipsets, for example to maintain a blacklist for some domains and a whitelist for others?

@wcawijngaards question from a noob - how does the update loop look like here? i mean what triggers queries for the domains specified in local-zone after initial startup query is done? i was trying to answer this question by looking at the code base, but failed miserably.

@wcawijngaards question from a noob - how does the update loop look like here? i mean what triggers queries for the domains specified in local-zone after initial startup query is done? i was trying to answer this question by looking at the code base, but failed miserably.

@paulgear No plans at the present for multiple ipsets.  We have contributed code.
After the lookup to the internet is complete for the query, the modules are done.  At that point the ipset.c:ipset_update() function is called with the just newly looked up query answer.  This scans the message for the names listed in the local-zone statements, and then adds them to the ipset in ipset.c:add_to_ipset().


String labels not functional with class_weight

String labels are accepted in y vector passed to fit function, but not in class_weight.
Environment info
Operating System: MacOS 10.14.5
Python version: 3.7
LightGBM version or commit hash: 2.2.3
Error message
ValueError: Class label positive not present.
Reproducible examples
import random
import numpy as np
import lightgbm

n = 10
class_weight = {'positive': 1, 'negative': 2}

X = np.random.rand(n, 1)
y = random.choices(list(class_weight.keys()), k=n)

classifier = lightgbm.LGBMClassifier(class_weight=class_weight)
classifier.fit(X, y)

@striajan Thanks a lot for catching this bug! I'll really appreciate if you check the fix from #2199.


Classname behavior

Should classname remove all the classes, or juste be added to it, or be ignored?
Let's take button, if you give a classname, at the moment it is overriding classname calculated before.

I suppose you are talking about the toggle button ? Haven't tried it yet, but I wonder the same question right now. Will test it soon using stories & jest ^^

Talking about https://github.com/thc-tools/react-material-component/blob/master/src/components/button/button.tsx

I guess you added theme.button for that, but shouldn't we ignore classname props then? And warn the user if defined?

damn, I forgot about the ...otherProps !
Yep, mistake on my end. I added it only for disabled state, but clearly haven't think it through.

Note that we still need the otherProps, for aria-, or data- html attributes at least

I know
On Wed 4 Jul 2018 at 09:34, Hartorn<EMAIL_ADDRESS>wrote:

Note that we still need the otherProps, for aria-, or data- html
attributes at least
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/thc-tools/react-material-component/issues/2#issuecomment-402390657,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABhG6cGdJG-cwBAjF5v7sOeHgs5DlKJ2ks5uDG_tgaJpZM4VCCvq
.

--
Cordialement,
Cédric BOULET KESSLER

Yep and also need to add filtering capabilities on those props to keep only html compliant ones

@Hartorn I updated code & TUs, can I have your input on it ?

@c3dr0x I think we can make some small improvement on button

I don't thing theme.button is useful, since we have className
I would change theme.icon by iconClassName (that way, we would have only primitive props, no object, so mutable/immutable probleme)
I would prefer  [theme.icon as any]: theme.icon !== undefined as theme.icon at least (same you done for className)
Component should be PureComponent, or render function (linked to primitive props usefulness instead of object)

So mainly, little things. Overall, looks clean and good !
About the test :

typo Can be have instead of Can have
to be reeeeeeeally complete, you could add test for default behavior (to avoid regression)
and an another one with all props (so no conflicting behavior)

Overall, this looks awesome copaing, with all the doc, test, storybook 👍 🥇
Pretty neat code !

The goal for theme is to have a behavior that is coherent throughout the lib. All classes that you can override are contained within a theme props, here you only have one or two entry but but for larger components it would be more important. And keep the className as you suggested only for the outermost element of the component, as a way to keep what react users are used to.
So, this might looks like a little overkill here for buttons but could be more clear for larger components.
Okay for the change for [theme.icon as any]: theme.icon !== undefined, its simpler, I agree ^^
About PureComponent/SFC I agree, I let classes for now because I wonder about using forwardingRefs as a default behavior to the outermost element of components. It might be overkill but I like the api quite much (https://reactjs.org/docs/forwarding-refs.html)
Okay for tests, duly noted :)
I'm glad that you like it !

I'm okay with forwarding ref, but it also work on function, so should be
okay to go for function, except if you want to use adapter and give it
react implementation.
On Fri, 6 Jul 2018 at 10:46, Cédric BOULET KESSLER<EMAIL_ADDRESS>wrote:

The goal for theme is to have a behavior that is coherent throughout the
lib. All classes that you can override are contained within a theme props,
here you only have one or two entry but but for larger components it would
be more important. And keep the className as you suggested only for the
outermost element of the component, as a way to keep what react users are
used to.
So, this might looks like a little overkill here for buttons but could be
more clear for larger components.
Okay for the change for [theme.icon as any]: theme.icon !== undefined,
its simpler, I agree ^^
About PureComponent/SFC I agree, I let classes for now because I wonder
about using forwardingRefs as a default behavior to the outermost element
of components. It might be overkill but I like the api quite much (
https://reactjs.org/docs/forwarding-refs.html)
Okay for tests, duly noted :)
I'm glad that you like it !
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/thc-tools/react-material-component/issues/2#issuecomment-402970158,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEuWfBmprUXYji-2oSxTKNacALEPioHMks5uDyQAgaJpZM4VCCvq
.


Switch networkx for graph-tool

See https://graph-tool.skewed.de/static/doc/index.html

(Very limited) start here: 7260e5025e3422cca7206fb2b348e5d0f451f01b. Slightly fiddly import code in network.py.

Working for main strain assigning functions in 5623a6710236a0535ec08eed0b1841641e1b3dc0.
Networkx version run on SPARC: https://microreact.org/project/9KY6djAvD (easy run took 1m56.113s on 24 threads)
Graph-tools version run on SPARC: https://microreact.org/project/yHmmBSjw3 (easy run took 1m54.125s on 24 threads)
Running detectably faster on only a few hundred genomes.
Only remaining networkx dependence is in the lineage_assignment.

Note that networks now store vertex labels as integers, not strings. Fiddly to remove and extend the network. Will need to check the behaviour with queries being added.

Great!
I think the integer indices may actually be a good thing, as I was beginning to think that this would be better linking with some of the other matrix code. We might need to change the code round a little, but if we keep a list (mapping int -> label in order) together with graph, which we map between at the start and end, it should be ok?

Yep, I think it is probably better, I just can't quite get my head around the way in which nodes are removed/added - e.g. after removal, the remaining are still consecutively labelled, so there needs to be some way of keeping track of what's what. It is possible to associate data with a vertex, but not sure how links are retained in e.g. shrinkage processes.
Another nice feature is that networks can be masked, e.g. for the visualisation code, rather than copied and pruned.
This version seems much faster with the refinement steps for large networks, I think it is definitely a worthwhile change.


Support Hints

Support an optional hint message for challenges
Update contrib.md to demonstrate adding hints
Added 2 hints: basic-any, intermediate-generic2

@100gle Thanks for your advice, I read the PicoCSS document and found out that there is really no need to write the styles myself, so I ended up with a button and a card from the pico. It did look better now. 😆

dark mode:

Thank you both! Will take a look today

Overall looks pretty good, two nits:

Can you also update requirements.txt? You can use pdm-autoexport or add it manually
This is up-to-you, but I find the shadow effect not fit into the existing style. Maybe we can remove it and use a slightly different background color (like light grey) for the hint?

Overall looks pretty good, two nits:

Can you also update requirements.txt? You can use pdm-autoexport or add it manually
This is up-to-you, but I find the shadow effect not fit into the existing style. Maybe we can remove it and use a slightly different background color (like light grey) for the hint?

requirements.txt has been updated, I've also edited Development.md to add a update dependencies section
box-shadow has been removed, the card now use #f4f4f4 as background(only in "light-mode")


RetryableEurekaHttpClient candidateHosts size may improve to avoid quarantineSet not be cleared

Assume we have a same url in two zone(some reasons, although it's not reasonable), simply the configuration file as flows:
eureka.client.region=region1
eureka.client.availability-zones.region1 = zone1,zone2
eureka.client.serviceUrl.zone1=http://localhost:8080/eureka
eureka.client.serviceUrl.zone2=http://localhost:8080/eureka
eureka.client.transport.retryableClientQuarantineRefreshPercentage=1

If eureka server is not available, then the url will be added to quarantineSet. Client will not be able to re-register to eureka server when eureka server restarted (Always throw throw exception "There is no known eureka server; cluster server list is empty").
I analyse the code of RetryableEurekaHttpClient in the eureka-client:
  private final Set<EurekaEndpoint> quarantineSet = new ConcurrentSkipListSet<>();

  ...

	 if (currentEndpoint != null) {
                quarantineSet.add(currentEndpoint);
         }
  ...

  private List<EurekaEndpoint> getHostCandidates() {
        List<EurekaEndpoint> candidateHosts = clusterResolver.getClusterEndpoints();
        quarantineSet.retainAll(candidateHosts);

        // If enough hosts are bad, we have no choice but start over again
        int threshold = (int) (candidateHosts.size() * transportConfig.getRetryableClientQuarantineRefreshPercentage());
        //Prevent threshold is too large
        if (threshold > candidateHosts.size()) {
            threshold = candidateHosts.size();
        }
        if (quarantineSet.isEmpty()) {
            // no-op
        } else if (quarantineSet.size() >= threshold) {
            logger.debug("Clearing quarantined list of size {}", quarantineSet.size());
            quarantineSet.clear();
        } else {
            List<EurekaEndpoint> remainingHosts = new ArrayList<>(candidateHosts.size());
            for (EurekaEndpoint endpoint : candidateHosts) {
                if (!quarantineSet.contains(endpoint)) {
                    remainingHosts.add(endpoint);
                }
            }
            candidateHosts = remainingHosts;
        }

        return candidateHosts;
  }

The problem is: threshold use candidateHosts.size() (Use List), but only quarantineSet.size() >= threshold the quarantineSet can be cleared.
Although two zones is diferent AwsEndpoint, they are the same EurekaEndpoint. At this time quarantineSet size is 1, but threshold is 2, quarantineSet.size() will always smaller than threshold. QuarantineSet will never be cleared, so client will not be able to re-register to eureka server when eureka server restarted.
My solution is translate candidateHosts list to set, and then use the candidateHostsSetSize:
    private List<EurekaEndpoint> getHostCandidates() {
        List<EurekaEndpoint> candidateHosts = clusterResolver.getClusterEndpoints();
        quarantineSet.retainAll(candidateHosts);

	 // Use Set to remove the same EurekaEndpoint in the List to prevent threshold always larger than quarantineSet size
        int candidateHostsSetSize = (new ConcurrentSkipListSet<>(candidateHosts)).size();
        // If enough hosts are bad, we have no choice but start over again
        int threshold = (int) (candidateHostsSetSize * transportConfig.getRetryableClientQuarantineRefreshPercentage());
        //Prevent threshold is too large
        if (threshold > candidateHostsSetSize) {
            threshold = candidateHostsSetSize;
        }
	if (quarantineSet.isEmpty()) {
            // no-op
        } else if (quarantineSet.size() >= threshold) {
            logger.debug("Clearing quarantined list of size {}", quarantineSet.size());
            quarantineSet.clear();
        }
   ...
  }

Assume we have a same url in two zone

I think this is not reasonable and does not make sense to "support" in this way or any other way except for maybe better validation.

Yes,  putting a same url in two zone is meaningless. However, in actual use, in order to be highly available, there may be an additional edge address in two zone (after all, there is no document to point out it is forbidden), but once we do this, it will be dangerous. Because if eureka server network is shortbale unavailable, it will not be able to recover since the quarantineSet cannot be cleared.
Therefore, I personally not agree it's an additional validation, after all, it is not reasonable to compare the size of Set with the size of List. Of course, I have considered other solutions, but the smallest and easiest way to change is to compare size with the same structure. So I hope to solve this issuse to avoid the risk, maybe there is a better way?

I think this is not reasonable and does not make sense to "support" in this way or any other way except for maybe better validation.

Yes, putting a same url in two zone is meaningless. However, in actual use, in order to be highly available, there may be an additional edge address in two zone (after all, there is no document to point out it is forbidden), but once we do this, it will be dangerous. Because if eureka server network is shortbale unavailable, it will not be able to recover since the quarantineSet cannot be cleared.
Therefore, I personally not agree it's an additional validation, after all, it is not reasonable to compare the size of Set with the size of List. Of course, I have considered other solutions, but the smallest and easiest way to change is to compare size with the same structure. So I hope to solve this issuse to avoid the risk, maybe there is a better way?

Yes, putting a same url in two zone is meaningless.

My point exactly. The url cannot be the same by definition, otherwise that's just the same zone, two urls which are the same point to the same thing.

in actual use, in order to be highly available, there may be an additional edge address in two zone (after all, there is no document to point out it is forbidden), but once we do this, it will be dangerous

Can you elaborate? Could you give an example maybe?
As for the document, there's no strict requirements doc for everything so some things are assumed where reasonable (like here).
I understand the bug you observed and understand the potential fix you proposed, the thing is that the system should not work around bad data and hide potential issues, this can manifest in other places and the system should fail early not continue pretending "it's fine" if two servers point to the same address. In this case I'd rather reject the configuration as fundamentally invalid.

Can you elaborate? Could you give an example maybe?
As for the document, there's no strict requirements doc for everything so some things are assumed where reasonable (like here).
I understand the bug you observed and understand the potential fix you proposed, the thing is that the system should not work around bad config and hide potential issues, this can manifest in other places and the system should fail early not continue pretending "it's fine" if two servers point to the same address. In this case I'd rather reject the configuration as fundamentally invalid.

Thanks for your reply and explain. It's much reasonable to check the config first and prevent the potential issues.
I gave a bad example, actually, we never config like this. This is a simple example to explain this issue.
I found this issue at a high-available exercise, and I spent few hours to find out why eureka servers alaways not available.
We use eureka with a large-scale nodes, to make the server more high available, we use multi-zone config, and we want use a smart DNS to resolve serviceUrls:
eureka:
 client:
   region: abc
   useDnsForFetchingServiceUrls: true
   eurekaServerPort: 10101
   eurekaServerDNSName: demo.com
   eurekaServerURLContext: eureka

and the DNS txt is:
txt.abc.demo.com -> aregion.abc.demo.com bregion.abc.demo.com
txt.aregion.abc.demo.com to <IP_ADDRESS> <IP_ADDRESS>
txt.bregion.abc.demo.com to <IP_ADDRESS> <IP_ADDRESS>

It works fine, but when bregion is down, the DNS will dynamic to resolve bregion address to aregion address.The problem comes... then I thought could we fix this.  Now I think we shouldn't let the Smart DNS to resolve urls dynamically, just let eureka client do by itself.
Thanks for your time to discuss this issue with me.


Add Golang 13 Roman to Integer solution

File(s) added: 13-Roman-To-Integer.go
Language(s) Used: Golang
Submission URL: https://leetcode.com/submissions/detail/755735144/

Thanks, @josuebrunel!


[Cosmos] Add caching for APR calculation to avoid making extra requests

Summary of Bug
minting/inflation and staking/pool endpoint being called quite often, I assume they were used to calculate APR for cosmos blockchains.

Also staking/validators?status=bonded does not change often


create a new react app exception raised by following destructions at https://reactjs.org/docs/create-a-new-react-app.html

Running Node.js v17.2.0, npm/npx comes up as 8.2.0.  Attempting to setup react app with destructions at: https://reactjs.org/docs/create-a-new-react-app.html
Under the heading "Create React App".
npx create-react-app my-app
cd my-app
npm start

Running in Windoze(aka Windows) 10.  Fresh first-time install of create-react-app function itself (so that process also pulled in react 17.0.2).  The result when I try running the freshly created app is an exception!  So I understand what the exception is trying to tell me but why is setting up all dependencies and their paths not covered by the react app setup?     This is just the npx create-react-app version of "hello world" right?   So should it not pull everything in it needs and set everything up?
Full insult at: https://pastebin.com/czGkpX1K.   Not to mention the apparently contradictory information at the end of the app creation https://pastebin.com/m9mr2EYP, and yes the same error is raised by darn yarn start as dang blasted npm start.😆
So, currently it appears the instructions may work for Node >= 14.0.0   <17.2.0 and npm >= 5.6 <8.2.0.
So, currently it appears the instructions should illuminate on yarn start option to remove confusion.

Thanks

rm -rf node_modules package-lock.json
Try this once time after again install all node modules and start your app. I also got the same error and Using the above my app worked properly.

We can’t help without a reproducing case unfortunately.


Colour values with different range

What is the recommended way to get e.g. RGB values from a colour with a 0-255 range? Do I have to do something like [int(i * 255) for i in col.rgb]? It seems like a shortcut for this very common case would be useful.

Actually, I guess it should use rgb_clamped instead of rgb to ensure the resulting values are within range. But even that is not quite right because only values exactly equal to 1.0 will map to 255, whereas it should really be values in the range 255/256 to 1.
So would this be more correct: [min(255, int(i * 256)) for i in col.rgb_clamped]?

After a bit more looking around, I found another way: col.to('rgb').color_object.get_upscaled_value_tuple(), but it doesn't clamp the values to the valid range:
In [1]: col = spectra.lab(80, -250, 80)
In [2]: col.to('rgb').color_object.get_upscaled_value_tuple()
Out[2]: (0, 261, 0)

I think a simple, documented, and correct way of doing this would be very useful.


packageVerificationCode 'sha1:' prefix

to make the resulting spdx file slightly more human-readable
See also https://spdx.github.io/spdx-spec/v2.3/package-information/

I'm not sure packageverificationcode uses prefixes? It looks like it always has to be SHA1. I do think after reading that spec document that I'm using the wrong field for dependencies. Perhaps it should actually be packageChecksum?

Mmm yeah maybe it is packageChecksum that allows prefixes - thanks
I'm not sure when we would want to use packageChecksum vs packgaeVerificationCode but I think at least that we won't want to merge this commit


Peer dependency warnings when using @elyra/canvas with React 17

Ref: 3605
@elyra/canvas now supports React 17 in addition to React 16 and React 18, thank you: #1559.
 There is an issue to add React 17 support but the package is clearly unmaintained, with no releases since June 2020.
react-contextmenu, which supports only React 16, has had no releases since May 2020 and is explicitly no longer unmaintained. React 17/18 support isn't coming here.
<EMAIL_ADDRESS>which supports only React 16. Unlike the other two, react-resize-detector is fully maintained and more recent versions support React 16, 17 and 18, so the solution here would simply be to upgrade. Releases are listed here and the breaking changes look fairly manageable.

@s100 FYI - we are currently working on this. It may take a few more days to get everything done.

Pleased to hear it, looking forward to taking the new version for a spin when it appears. :)

The react-contextmenu module has been removed from Elyra Canvas and the react-resize-detector module has been upgraded. We are handling the code mirror problem in a separate issue. I will post here when that one is finished.

@s100     The two changes for   react-contextmenu and react-resize-detector are included in Elyra Canvas v12.37.0.  Unfortunately the person working on the Code Mirror changes will be away for a couple of weeks to so this won't get completed until then.

Confirmed, I see that react-contextmenu is no longer a dependency of<EMAIL_ADDRESS>and you've upgraded to react-resize-detector@9 which allows React 17.
In the meantime, I also observe that since I opened this issue<EMAIL_ADDRESS>has been published, which adds support for React 17. This was unexpected as the project had been dormant for over three years, but quite welcome.
As a result of this, I no longer see any peer dependency warnings when installing @elyra/canvas alongside React 17. Thank you!


Problems with international stocks

Hello!
I would like to obtain time series of the 31 trackers of the Mexican Stock Market. The name of the symbols on yahoo finance always end with '.MX', for example, ALSEA.MX:
https://finance.yahoo.com/quote/ALSEA.MX?p=ALSEA.MX&.tsrc=fin-srch
I was able to download the time series in the past but recently I have been unable to do so because I get an invalid API call message.
Heres the code that I am using:
from alpha_vantage.timeseries import TimeSeries
ts=TimeSeries(key='1234,output_format='pandas')
ts.get_daily(symbol='ALSEA.MX',outputsize='full')
Thank you!

@hugo-am, since this issue is more about the underlying API rather than this Python library, please kindly direct your question to<EMAIL_ADDRESS>Thank you!


[Asana] ... doesn t allow creating a new task

Extension
https://www.raycast.com/raycast/asana
Description
Error:
Error: Error while refreshing tokens: 400 (Bad Request)
{"error":"invalid_grant"}

    at withAccessToken(r3)
Dr:index.js:11:2490
    at ray-navigation-stack
Lo:index.js:11:2088

Steps To Reproduce
Create a new task doesnt work
Current Behaviour
No response
Expected Behaviour
No response

It should be fixed in the latest release of the extension


HoldTillRename: Rename knative-sandbox to knative-extension

This is an automated PR to replace knative-sandbox strings to knative-extension. Do not merge until productivity renames knative-sandbox org to knative-extension.

many tests will pass once knative-extension will exist
Not sure that all
/retest


New place to chat?

https://blog.rust-lang.org/2019/04/26/Mozilla-IRC-Sunset-and-the-Rust-Channel.html

I'm not the right person to drive the new place to chat, but I'll absolutely support reflecting such change on the are-we-learning-yet site.

I see now the site has a Discord link.  But since it goes to the deprecated(??) Rust related programming server, and there's now an official Rust Discord server, finding a channel there seems like a good start.  I couldn't immediately find an issue queue for managing the Discord server though, so if anyone can help get that rolling, that'd be cool.

A unofficial/applicant Working Group focused around the machine learning domain has been put together over the last few weeks! The group's repository can be found at github.com/rust-ml, along with a Zulip chat channel here. @anowell During our most recent meeting, this site came up as a commonly-seen resource for people looking into machine learning in Rust. @anowell Would you be open to accepting PRs from members of the Rust-ML group (and, of course, you and anyone is welcome to participate!)? Alternatively, if you have other things on your plate at the moment, would you potentially be open to adding a few other people as maintainers to help keep things up-to-date as the ecosystem evolves? I personally think this site has a lot of value, and would really enjoy seeing it continue to be a good resource for Rustaceans interested in getting into ML.

yes. I've been falling behind on PRs, the weekly cron to update the numbers has been broken for a bit, and catching up some random weekend has been on my todo list for at least a month, not counting the random r/rust links I've been meaning to update w.r.t. content. Let me make that change immediately so it doesn't fall through the cracks too.
I may also be open to entertaining the notion of transferring this repo to the rust-ml org. That would basically just leave me the bottleneck on the domain name.

Current rust-ml members all have collab invitations immediately. Happy to continue discussing ways to improve the collaboration to make this more of a community project, but hopefully that immediately allows filling in the gaps where I've dropped the ball. (and also gives me a bit of extra motivation to fix the CI issue)

That's awesome, thank you! Depending on how you feel about things moving forward, I don't think that there would be any problem on our end with taking ownership of the repository, but is totally up to you--I don't think there's any push in that regard one way or another on this end. We're really focusing on trying to work with the existing ecosystem and help out where/when we can. There's no push to take ownership of things at an organizational level, although it's something that we're willing to do if that's the action that would be most helpful.
I'll let people know on the chat, and hopefully we'll have some things to add to it soon! If there's anything that comes up  the meantime, just let me know on here or feel free to pull up the Zulip chat.

@anowell This might be a bit of a dumb question, but I don't have much experience running web-based applications that aren't local. Is there a particular way that I should be trying to re-generate the actual site data every time that a new pull request is merged? My local branch shows the updated versions when I run it on my own machine, but the actual arewelearningyet.com site doesn't seem to show the updates.

@quietlychris Not at all a dumb question. I just added an explanation of publishing at the bottom of the README. Basically, I set CI long ago to auto-publish changes long ago. However, sometime in December (yikes, is that really 6 months ago already?), the Travis job started failing. It's been on my mind regularly via weekly email reminders that it's failing (which I do see). Somehow it took the extra human nudge (thanks 😉) to actually go fix it in 5218214e (Basically, Travis CI seems to have made a change affecting either the default ruby or bundler; I just upgraded ruby and bundler and it started working again. While I was at it, I upgraded Jekyll too).

Closing this now that all "chat" links on the site have been replaced with the Zulip Chat in commits: a6e5ecb and 882efab


Missing Requirement for Install

When installing CeleryAnalytics you need to manually pip install django-ninja as it is not defined in the requirements.
Running initial migration for CeleryAnalytics without manually pip install django-ninja results in the following error:
Traceback (most recent call last):
  File "/home/allianceserver/myauth/manage.py", line 22, in <module>
    execute_from_command_line(sys.argv)
  File "/home/allianceserver/venv/auth/lib/python3.10/site-packages/django/core/management/__init__.py", line 446, in execute_from_command_line
    utility.execute()
  File "/home/allianceserver/venv/auth/lib/python3.10/site-packages/django/core/management/__init__.py", line 440, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "/home/allianceserver/venv/auth/lib/python3.10/site-packages/django/core/management/base.py", line 414, in run_from_argv
    self.execute(*args, **cmd_options)
  File "/home/allianceserver/venv/auth/lib/python3.10/site-packages/django/core/management/base.py", line 460, in execute
    output = self.handle(*args, **options)
  File "/home/allianceserver/venv/auth/lib/python3.10/site-packages/django/core/management/base.py", line 98, in wrapped
    res = handle_func(*args, **kwargs)
  File "/home/allianceserver/venv/auth/lib/python3.10/site-packages/django/core/management/commands/migrate.py", line 91, in handle
    self.check(databases=[database])
  File "/home/allianceserver/venv/auth/lib/python3.10/site-packages/django/core/management/base.py", line 487, in check
    all_issues = checks.run_checks(
  File "/home/allianceserver/venv/auth/lib/python3.10/site-packages/django/core/checks/registry.py", line 88, in run_checks
    new_errors = check(app_configs=app_configs, databases=databases)
  File "/home/allianceserver/venv/auth/lib/python3.10/site-packages/django/core/checks/urls.py", line 42, in check_url_namespaces_unique
    all_namespaces = _load_all_namespaces(resolver)
  File "/home/allianceserver/venv/auth/lib/python3.10/site-packages/django/core/checks/urls.py", line 61, in _load_all_namespaces
    url_patterns = getattr(resolver, "url_patterns", [])
  File "/home/allianceserver/venv/auth/lib/python3.10/site-packages/django/utils/functional.py", line 49, in __get__
    res = instance.__dict__[self.name] = self.func(instance)
  File "/home/allianceserver/venv/auth/lib/python3.10/site-packages/django/urls/resolvers.py", line 696, in url_patterns
    patterns = getattr(self.urlconf_module, "urlpatterns", self.urlconf_module)
  File "/home/allianceserver/venv/auth/lib/python3.10/site-packages/django/utils/functional.py", line 49, in __get__
    res = instance.__dict__[self.name] = self.func(instance)
  File "/home/allianceserver/venv/auth/lib/python3.10/site-packages/django/urls/resolvers.py", line 689, in urlconf_module
    return import_module(self.urlconf_name)
  File "/usr/lib/python3.10/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1050, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1027, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1006, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 688, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 883, in exec_module
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "/home/allianceserver/myauth/myauth/urls.py", line 2, in <module>
    from allianceauth import urls
  File "/home/allianceserver/venv/auth/lib/python3.10/site-packages/allianceauth/urls.py", line 12, in <module>
    import allianceauth.services.urls
  File "/home/allianceserver/venv/auth/lib/python3.10/site-packages/allianceauth/services/urls.py", line 17, in <module>
    services = get_hooks('services_hook')
  File "/home/allianceserver/venv/auth/lib/python3.10/site-packages/allianceauth/hooks.py", line 123, in get_hooks
    register_all_hooks()
  File "/home/allianceserver/venv/auth/lib/python3.10/site-packages/allianceauth/hooks.py", line 112, in register_all_hooks
    hooks = list(get_app_submodules('auth_hooks'))
  File "/home/allianceserver/venv/auth/lib/python3.10/site-packages/allianceauth/hooks.py", line 101, in get_app_submodules
    yield name, import_module(f'{name}.{module_name}')
  File "/usr/lib/python3.10/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "/home/allianceserver/venv/auth/lib/python3.10/site-packages/celeryanalytics/auth_hooks.py", line 1, in <module>
    from . import urls
  File "/home/allianceserver/venv/auth/lib/python3.10/site-packages/celeryanalytics/urls.py", line 5, in <module>
    from .api import api
  File "/home/allianceserver/venv/auth/lib/python3.10/site-packages/celeryanalytics/api.py", line 6, in <module>
    from ninja import Field, Form, NinjaAPI
ModuleNotFoundError: No module named 'ninja'

fixed in 0.0.6


win10  with spring metrics No datapoints found.

prometheus-2.17.1.windows-amd64 with
micrometer-registry-prometheus 1.3.6

It makes more sense to ask questions like this on the prometheus-users mailing list rather than in a GitHub issue. On the mailing list, more people are available to potentially respond to your question, and the whole community can benefit from the answers provided.


[AutoComplete] Fix issue with popover positioning on iOS

Popover uses a helper iOSHelpers.js to make specific calculations for iOS. However, it seems to be useless with last Mobile Safari versions. Worse yet, it generates a bug with calculations:

See #5544 for more details.

[x] PR has tests / docs demo, and is linted.
[x] Commit and PR titles begin with [ComponentName], and are in imperative form: "[Component] Fix leaky abstraction".
[x] Description explains the issue / use-case resolved, and auto-closes the related issue(s) (http://tr.im/vFqem).

Are you saying iOS Safari fixed the issue? 😄 Is keeping the support for the older version needed? I guess not

Yep– iOS 10.3 was the magic version!  It seems to have roughly 3/4 of the iOS share.


default to minor for prod deploys

now when you deploy to production from master, it assumes that you want a minor release with the specific sha you are on.

I think this is likely too out of date now.


ci(ui_test): Automate Googlepay test for Adyen, Bluesnap and Stripe connectors

Type of Change

[x] Enhancement
[x] CI/CD

Description

This PR aims at automating Googlepay tests in UI Tests for connectors: Adyen_UK, Bluesnap and Stripe.
Main problem with Google Pay UI automated testing is Captchas.
Google detects automated bots / scripts when you try to login and throws in a captcha the moment Next button is pressed after entering in the email_id by elongated the content with Captcha (It is in fact hard for even humans to understand the text written on it).
This hides the Passwd element and hence the UI test fails as it cannot login to the Google account anymore by timing out.
Now, we're injecting the browser data (that contains the login data) into the browser and hence it does not ask for login now. So, no captcha now (Yes, I cannot guarantee, if tomorrow Google throws in captcha in other places).
Motivation and Context

Reduces manual testing burden.
How did you test it?

See Connector UI tests below in CI checks.
Checklist

[x] I reviewed the submitted code
[x] I added unit tests for my changes where possible

Converting this PR to draft as this is not working as expected.

Closing this PR for this: https://github.com/juspay/hyperswitch/pull/2178


Core Panic if RMT ISR executes during Preferences save (IDFGH-3453)

I have been experiencing issues with core panics ever since I started using the RMT peripheral and it's standard driver.
Every time I do a stack trace, it has been a Preferences save that was interrupted by the RMT ISR.
A sample stack trace:

ESP32 IDF Version: v3.2.3-14-gd3e562907

Hi,@rljonesau
Please try to comment this line: https://github.com/espressif/esp-idf/blob/055943e29346e77c50589c61a8a26101a8b35d7b/components/driver/rmt.c#L593
I think this issue is caused by accessing flash during cache disabling.
thanks !!

OK sounds quite plausible.
I will need to learn how to use the IDF with Platform IO, it is mainly an Arduino framework app.
But yes I do see that RMT RX BUFFER FULL message appearing at random times, due to noise from the 433MHz receiver I'm using.
Thank you for your reposnse.

Hello, I have finally worked out how to recompile the IDF for Arduino, under PlatformIO.
Yes, commenting out that line solves the cache disabled error, avoiding the core panic.
I note the latest IDF logging has a new method ESP_DRAM_LOGE (et al) which sounds like it would fix the problem, and retain the error reporting. I tried using that, but the IDF release to match up with arduino-esp does not support the function. :-(
Should I push a Pull Request along with a description of the problem so at least the problem is eventually dealt with at some point in the future?


Convert everything to symbol keys

Our main YAML file, article_scrape_patterns is currently written with string keys.
We could convert everything to symbols to reduce the complexity of switching between the two (e.g. https://github.com/richardwu/news_scraper/blob/5168d782a84eac9cd9773aea46af67f340dcd38a/lib/news_scraper/trainer/presets_selector.rb#L49).
Thoughts? @jules2689

thats the data type array, I dont think you can define symbols in arrays, things without quotes in yaml are strings (the quotes in the data types array are unnecessary)

Apparently you can
.

oh weird, i guess I never thought of that syntax

it only removed a .to_sym and changes up the yml file... so its a small change i wouldnt have any opinions on. Might make things easier in the future?

we could also make it all strings - dunno why there are .to_syms everywhere but we seem to be able to go either way if we want

I like symbols more because they show up more prominently in my syntax highlight 😄

Also hash rockets blow


Not work in 2023 version

Description of the bug & expected behaviour (required)
After updating to version 2023, the Pokemon bar is not displayed.
However, if I change the theme and then change it back, it can be displayed normally.
I've already raised this issue with JetBrains officially, but they suggested I contact the plugin author.
I really like this plugin and hope that it can be supported in the 2023 version.
Environment Info (required)

OS: Window
JetBrains Product: Rider
JetBrains Product Version: 2023
Plugin Version: 2.1.1

Steps to reproduce (optional)

open IDE
not display

Additional context or information (optional)
If I change the theme and then change it back, it can be displayed normally, not display when restart IDE.

This appears to be the first report of #89, which I plan to look into soon


Add support for more netmotion sourcetypes

Netmotion documention has several good examples of syslog events not supported by sc4s
https://help.netmotionsoftware.com/support/docs/Diagnostics/410/help/DiagnosticsHelp.htm#page/NetMotion%20Diagnostics[…]porting.14.11.html#ww1002396
https://help.netmotionsoftware.com/support/docs/Diagnostics/410/help/DiagnosticsHelp.htm#page/NetMotion%20Diagnostics[…]porting.14.12.html#ww1003744
https://help.netmotionsoftware.com/support/docs/Diagnostics/410/help/DiagnosticsHelp.htm#page/NetMotion%20Diagnostics[…]porting.14.09.html#ww1001667
The events look very unique, and should be an easy addition

:tada: This issue has been resolved in version 2.8.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Event handlers block processing of new events

When the library calls an event handler, it blocks the processing of new events received from Z-Wave JS Server until the event handler returns.
So if what is implemented in the event handler is time consuming it can cause big delays in the whole application.
Moreover if the event handler itself synchronously calls another ZWave JS .NET method which expects a response from the Z-Wave JS server it creates a deadlock.
To workaround this issue all my event handlers look like this:
private void ExclusionStartedEventHandler()
  {
      //need a Task.Run here because event processing in ZWaveJS.NET is delayed by any event handler
      Task.Run(() =>
      {
....
      });
  }

I don't mind to keep doing this, but I wonder if there is any case where we actually want this blocking behavior?

Nice catch!
Should be easy enough to move the trigger calls to a task!
I'll review it this weekend, and will commit the changes to the branch I'm working on, that I mentioned in the controller node question.

Fixed in recent commit