[Bug]: Padim doesn't support metrics AUPR

Describe the bug
When I change engine = Engine(pixel_metrics=["AUROC", "AUPR", "PRO"], image_metrics=["AUROC", "AUPR", "PRO"], task=TaskType.SEGMENTATION) to engine = Engine(pixel_metrics=["AUROC", "PRO"], image_metrics=["AUROC", "PRO"], task=TaskType.SEGMENTATION) or with default metrics Engine(task=TaskType.SEGMENTATION), it has no error reporting.
Dataset
MVTec
Model
PADiM
Steps to reproduce the behavior
Thanks for the great work!
I used API to train the code
from anomalib import TaskType
from anomalib.data import MVTec
from anomalib.engine import Engine
from anomalib.models import Padim

# configure logger
logging.basicConfig(level=logging.INFO, format='%(asctime)s - %(levelname)s - %(message)s')
logger = logging.getLogger(__name__)

datasets = ['screw', 'pill', 'capsule', 'carpet', 'grid', 'tile', 'wood', 'zipper', 'cable', 'toothbrush', 'transistor',
            'metal_nut', 'bottle', 'hazelnut', 'leather']

for dataset in datasets:
    logger.info(f"================== Processing dataset: {dataset} ==================")
    model = Padim()
    datamodule = MVTec(category=dataset, num_workers=0, train_batch_size=256,
                       eval_batch_size=256)
    # metrics is under "anomalib/metrics/"
    engine = Engine(pixel_metrics=["AUROC", "AUPR", "PRO"], image_metrics=["AUROC", "AUPR", "PRO"], task=TaskType.SEGMENTATION)

    logger.info(f"================== Start training for dataset: {dataset} ==================")
    engine.fit(model=model, datamodule=datamodule)

    logger.info(f"================== Start testing for dataset: {dataset} ==================")
    test_results = engine.test(
        model=model,
        datamodule=datamodule,
        ckpt_path=engine.trainer.checkpoint_callback.best_model_path,
    )

Then I got the error. Error message doesn't seem to be related.
What I see in the file under "src/anomalib/metrics/", It should support the following metrics:
    "AUROC",
    "AUPR",
    "AUPRO",
    "AnomalyScoreDistribution",
    "BinaryPrecisionRecallCurve",
    "F1AdaptiveThreshold",
    "F1Max",
    "F1Score",
    "ManualThreshold",
    "MinMax",
    "PRO"

Do I miss something?
OS information
OS information:

OS: [Ubuntu 22.04]
Python version: [3.10.14]
Anomalib version: [1.0.1]
PyTorch version: [2.1.2+cu121]
CUDA/cuDNN version: [12.4]
GPU models and configuration: [GeForce RTX 3090]

Expected behavior
The metrics should be supported.
Screenshots
No response
Pip/GitHub
pip
What version/branch did you use?
1.0.1
Configuration YAML
None, I used API.

Logs
/home/jinyao/anaconda3/envs/IADB/bin/python /home/jinyao/PycharmProjects/IADBE/train_test_with_padim.py
2024-05-08 12:01:25,544 - INFO - ================== Processing dataset: screw ==================
2024-05-08 12:01:25,544 - INFO - Initializing Padim model.
2024-05-08 12:01:25,746 - INFO - Loading pretrained weights from url (https://download.pytorch.org/models/resnet18-5c106cde.pth)
2024-05-08 12:01:25,821 - INFO - ================== Start training for dataset: screw ==================
GPU available: True (cuda), used: True
TPU available: False, using: 0 TPU cores
IPU available: False, using: 0 IPUs
HPU available: False, using: 0 HPUs
`Trainer(val_check_interval=1.0)` was configured so validation will run at the end of the training epoch..
2024-05-08 12:01:25,920 - INFO - Found the dataset.
You are using a CUDA device ('NVIDIA GeForce RTX 3090') that has Tensor Cores. To properly utilize them, you should set `torch.set_float32_matmul_precision('medium' | 'high')` which will trade-off precision for performance. For more details, read https://pytorch.org/docs/stable/generated/torch.set_float32_matmul_precision.html#torch.set_float32_matmul_precision
Traceback (most recent call last):
  File "/home/jinyao/PycharmProjects/IADBE/train_test_with_padim.py", line 24, in <module>
    engine.fit(model=model, datamodule=datamodule)
  File "/home/jinyao/anaconda3/envs/IADB/lib/python3.10/site-packages/anomalib/engine/engine.py", line 518, in fit
    self.trainer.fit(model, train_dataloaders, val_dataloaders, datamodule, ckpt_path)
  File "/home/jinyao/anaconda3/envs/IADB/lib/python3.10/site-packages/lightning/pytorch/trainer/trainer.py", line 544, in fit
    call._call_and_handle_interrupt(
  File "/home/jinyao/anaconda3/envs/IADB/lib/python3.10/site-packages/lightning/pytorch/trainer/call.py", line 44, in _call_and_handle_interrupt
    return trainer_fn(*args, **kwargs)
  File "/home/jinyao/anaconda3/envs/IADB/lib/python3.10/site-packages/lightning/pytorch/trainer/trainer.py", line 580, in _fit_impl
    self._run(model, ckpt_path=ckpt_path)
  File "/home/jinyao/anaconda3/envs/IADB/lib/python3.10/site-packages/lightning/pytorch/trainer/trainer.py", line 950, in _run
    call._call_setup_hook(self)  # allow user to setup lightning_module in accelerator environment
  File "/home/jinyao/anaconda3/envs/IADB/lib/python3.10/site-packages/lightning/pytorch/trainer/call.py", line 93, in _call_setup_hook
    _call_callback_hooks(trainer, "setup", stage=fn)
  File "/home/jinyao/anaconda3/envs/IADB/lib/python3.10/site-packages/lightning/pytorch/trainer/call.py", line 208, in _call_callback_hooks
    fn(trainer, trainer.lightning_module, *args, **kwargs)
  File "/home/jinyao/anaconda3/envs/IADB/lib/python3.10/site-packages/anomalib/callbacks/metrics.py", line 92, in setup
    pl_module.image_metrics = create_metric_collection(image_metric_names, "image_")
  File "/home/jinyao/anaconda3/envs/IADB/lib/python3.10/site-packages/anomalib/metrics/__init__.py", line 191, in create_metric_collection
    return metric_collection_from_names(metrics, prefix)
  File "/home/jinyao/anaconda3/envs/IADB/lib/python3.10/site-packages/anomalib/metrics/__init__.py", line 60, in metric_collection_from_names
    metrics.add_metrics(metric_cls())
TypeError: PrecisionRecallCurve.__new__() missing 1 required positional argument: 'task'

Process finished with exit code 1

Code of Conduct

[x] I agree to follow this project's Code of Conduct

Please install the latest version of anomalib from source: https://github.com/openvinotoolkit/anomalib?tab=readme-ov-file#-installation
AUPR was updated to be a child class of BinaryPrecisionRecallCurve instead of PrecisionRecallCurve which needs a task argument lately
https://github.com/openvinotoolkit/anomalib/blob/f846d7a719a57af34833b0ad9760ec915e14fdbf/src/anomalib/metrics/aupr.py#L16

@alexriedel1, thanks for the help!
@cjy513203427 if you want to use the pip version, we will try to release the new version on the 30th of May. Meanwhile you could use the source installation as @alexriedel1 mentioned

Please install the latest version of anomalib from source: https://github.com/openvinotoolkit/anomalib?tab=readme-ov-file#-installation
AUPR was lately updated to be a child class of BinaryPrecisionRecallCurve instead of PrecisionRecallCurve which needed a task argument
https://github.com/openvinotoolkit/anomalib/blob/f846d7a719a57af34833b0ad9760ec915e14fdbf/src/anomalib/metrics/aupr.py#L16

Thanks for the quick reply. Could you tell me how can I add a task argument for metrics if I use source mode?

you should not need task anymore as the base class is binary precision recall curve


Autocomplete expansion jump inserts $1 and jumps past insert point

I'm guessing this is just something in my config somewhere, trying to narrow it down, any idea why this would be happening?
If not, that's fine, I'll keep searching until I have a minimal reproducible example 👍 .
Can someone confirm that this does have something to do with this specific extension and not something completely different?

Same issue.

@chemzqm you're the best 🔥 .
I recently switched to full lua config, set everything up from zero, lua only plugins, native language server client + nvim-cmp + null-ls, all the goodies... but after using it a few days, concluded there is still too much missing compared to coc.nvim, also for Rust, so I came back 😄 .
Hats off to you and all other contributors 🙇 .


background image resizing issue

As i am resizing the tab size bg image getting shrinked and it's messing up UI. I would like to work on this issue.

background visible only for half of the page.

I have worked little on this now it's looking like this. I would like to work on this bit more and make all background responsive and snappy. please assign me for this.

Sure @snahashisk Go ahead and make a PR for the same ,and tell me if you need any assistance in this.


noobaa: don't set NodeAffinity for standalone

Signed-off-by: Jose A. Rivera<EMAIL_ADDRESS>

/cherry-pick release-4.9

Verified that this works on a personal OCP cluster in AWS.


Move switch element classes out of parent selector

all of the switch component element classes are unnecessarily nested in .pf-c-switch.

:tada: This issue has been resolved in version 2.59.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


How to integrate stripe in the project and how to get those .env varaiables

watch some tutorial on youtube for stripe setup


⚠ Bofrost.lu has degraded performance

In 9f20c74, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1123 ms

Resolved: Bofrost.lu performance has improved in 1ee0c65 after 6 minutes.


Text updates to Load Testing

I made additional wording updates to the load testing section based on PR review comments that I missed previously.

@icchan do you have permissions to merge or do you want me to merge?


(POTENTIAL ISSUE - LOOPS) Create FK in SyncronizationBus as separate set of operations

Basically create schema first, indexes and then apply FK creation. Might help to solve issue with loops. However has to double checked with SQLite as it can mount FK only as moment of table creation.

Example: two tables cross reference each other (not sure it's good idea to make db like that anyway, SQLServer will die with error on DB level when cross reference found). Currently such situations can be solved by disabling constrain creation.

Example: two tables cross reference each other (not sure it's good idea to make db like that anyway, SQLServer will die with error on DB level when cross reference found). Currently such situations can be solved by disabling constrain creation.

actually it's dup ticket

actually it's dup ticket


How do i make my audio go into 1 channel?

when i try to run the latest version of alvr, it errors out saying cannot have 2 channels of audio at once. How do i make it so it goes in 1 channel? My Computer headset is G635 Gaming headset, and the vr headset is the quest 2.

ALVR currently only supports stereo sound (dual channel, left and right). The G635 has surround sound, so ALVR can't really "talk" to it. Do you have another audio device you could output to? If not, you may be able to use VB-Cable, and have steamVR output to / ALVR read from it.


Add optional night and specular texture url source destination

Add planet and globe option night, specular, etc url parameters. If you want to change night texture, use option such as nightMaskUrl, specularMaskUrl , etc.
FYI @GuittonHubert @pavletto

Removed to https://github.com/openglobus/openglobus/issues/718


Import ComponentClass and Component

Both ComponentClass and Component should be imported from react. react-redux doesn't contain those two.

Thanks for the fix 👍


iPhone 14 Series

new phones are:
iPhone 14 -> iPhone14,7
iPhone 14 Plus -> iPhone14,8
iPhone 14 Pro	->  iPhone15,2
iPhone 14 Pro Max -> iPhone15,3

I'll make a PR.
Also:  there perhaps should be a new indicator, similar to has hasNotch for the new 'not notch'; perhaps hasDigitalIsland

PR -> Updates for iPhone 14 Family #81


Use retries in publishing events (Polly exponential)

Not implemented due to moving to a better approach: Outbox pattern


Add page where to show technologies/services/tools built on Monero

We don't have a place where to show technologies/services/tools built on Monero. We could have a dedicated page (the structure would be similar to the sponsorships page) for these projects.
Could be a new entry under the "Community" tab. So "Community" -> "Built on Monero".
Projects built on Monero:
Townforge: Game based on a slightly edited version of Monero, developed by Monero dev moneromooo. Even if the underlying technology is a fork of Monero, it shows the potentiality of the technology.
Primo: "Primo (Private Monero payments) is a protocol and set of software which enables a party to require payment for service.
Users will mine Monero and submit shares to the party like to a pool. The party (eg, a web site) uses such mining as payment,
and supplies the service based on those payments"
RPC-pay: Not sure about this one. It's more a stratch because it's a Monero feature not something built on Monero, but this it's so underused and unknown to most that might make sense to add it to such list.
Haveno: Exchange built on and powered by Monero multisig (WIP, so would be added later on. Also, disclaimer: I'm part of the core team of Haveno)
Anything else built on Monero (excluding wallets, for obvious reasons) that we would like to add?

Maybe we can add open source Monero only market too: https://github.com/Janoher/MoneroMarket

Is primo worth listing? seems dead, but still interesting use of the technology none the less


Replace chalk with piccolor

User story
As a developer I want to use latest tools, minimize the dependency graph.
chalk has a couple op disadvantages:

No dual module support
large bundlesize
many dependencies
slow compared to other alternatives

Benchmarks of alternative options:
https://github.com/webdiscus/ansis#benchmark
Acceptance criteria
Use https://github.com/alexeyraspopov/picocolors
Implementation details
Relevant steps for our codebase:

Replacing chalk

Replace package name in import:

- import chalk from 'chalk'
+ import pico from 'picocolors'

Replace variable:

- chalk.red(text)
+ pico.red(text)

Replace chains to nested calls:

- chalk.red.bold(text)
+ pico.red(pico.bold(text))

@BioPhoton
you can replace chalk with ansis.
Ansis is very small  and fast.
Ansis has all features of chalk and supports named import so your code will be more clean & clear:
- import chalk from 'chalk'
+ import ansis, {red, bold, underline, etc...} from 'ansis'

Replace variable:
- chalk.red(text)
+ ansis.red(text)
// or
+ red(text)

Replace text:
- chalk.red('text')
+ ansis.red('text')
// or
+ red`text`

Replace chains:
- chalk.red.bold(text)
+ ansis.red.bold(text)
// or
+ red.bold(text)
// or
+ red`${bold(text)}`

Thanks @webdiscus!! I will check it out right now.
What is the difference to picocolor?

@BioPhoton
The difference to picocolor you can see in  the features comparison.
Picocolor is very primitive and doesn't supports runtimes like Bun, Deno, Next.JS.

Convince, but your link is broken.

Convince, but your link is broken.

sorry, the link above is fixed

Description is updated, labeled correctly and moved into ready for refinement :)

there is some problem (missed colors) with pico.

I forgot to update the title 🙈
We switched to ansis. The title was the only description I forgot to update 🤷♂
Sry for that ✌


Missing (?) Trait Implementations

Sorry about that, pressed enter too many times.
Anyway, to the trait implementations. Would it be logical for Vector1-Vector4 to implement One?
Also, would it be a good idea for Vector3 to have the function to_homogeneous(), like Point3, only with the result being vec4(v.x, v.y, v.z, Zero::zero())?

Not sure that One really makes sense for vectors... seeing as 1 is the multiplicative identity. VectorN * VectorN -> VectorN doesn't really make sense, and hence isn't defined for vectors, and therefore there is no sensible definition for 1.

The homogeneous conversions are definitely gaps in the library though!


Add config file to let users modify config easily.

#52
now can change config by modify the file config

Can I set up a phone with your qr code organization

Yes


Share API Gateway does not work

Hi, I was trying to use an shared API with alias plugin, during the deployment there are no errors, the problem is that it does not create the stage variables SERVERLESS_STAGE and SERVERLESS_ALIAS. Thanks for any help.

Hi @anrocox , there is a general design problem of using aliases (the alias plugin) with shared API gateways. I elaborated on the topic in #79 and pointed out, why it is not a trivial problem to solve (if it can be solved at all).
Although there might be workarounds that somehow lead to a working system, I believe it to be not very stable in regards of stage and stack isolation. Any thoughts welcome in the referenced task 😄


 It's more important to keep test results in master green in the meantime. (I would do this initial PR to skip the tests, but I'm not sure which tests are affected and on which OSes they need to skip.)

We should reenable these tests and reinvestigate -- it's quite possible these have been fixed.

This can't be a P1, since we've had this bug for half a year, and we're no longer failing our own tests....
I did re-enable these tests, and they still fail. Which makes sense, since no fix on the CoreML side recently would affect test results on 10.13.
Ultimately, if we cannot safely export these models in a way that will work deterministically in CoreML in macOS 10.13 / iOS 11, we should document the dependency on a newer CoreML.

@hoytak Not sure anyone else has the necessary context; what precisely is the dependency on CoreML? In what cases should users expect these exported Core ML models to actually work?

If we can't get the exported models to work on 10.13, we should update their model spec version so they are limited to newer macOS releases (this will give a better user experience than just a documentation fix, as the exported model will complain in Xcode when targeting earlier macOS/iOS versions).

I've been working with Guihao on this one; the issue is with columns that have dictionary features and those with categorical lists.  The way these are unpacked in old versions of CoreML means they are unpacked into uninitialized memory.  He's verified all the tests pass now; is checking today I believe on 10.13.
@Jarvi-Izana -- is that a correct summary of the status?

@hoytak sorry for the late reply. I just finished running the pipeline with relevant xfail removed and it failed on mac_10_13 environment with python27.

Ok, I tried MoD with a macOS10.13 instance and it failed on python3 as well.

no code change is needed. We should keep triaging it.

We are not going to fix it due to the undefined behavior of CoreML prior to macOS version 10.14. These tests will be still marked as failure for versions prior to version 10.14.


Improve UI error handling

Is your feature request related to a problem? Please describe.
There is currently no user feedback whenever a feature breaks in the UI, which makes it unclear when something is working as expected or not. Detecting a crash requires one to inspect the console logs from the browser, which is not appropriate for all users.
Describe the solution you'd like
Include more error handling to inform the user whenever something went wrong, as it is already the case for some parts of the website. For example, trying to train a model without first uploading data creates a toaster (pop-up) saying "No file uploaded". I would like to implement something similar covering all errors.

Here are some examples of error handling with user feedback:

When uploading data and the error source has been identified:

Or a more generic one when the error source isn't known:


More verbose log on failure of container uninstall

Description
This PR adds a more verbose log on the failure of container uninstall.
The error message would look like this:
➜  dapr uninstall
ℹ  Removing Dapr from your cluster...
WARNING: could not delete run data file
❌  Error removing Dapr: uninstall failed
 could not remove dapr_placement container: Error: No such container: dapr_placement

 could not remove dapr_redis container: Error: No such container: dapr_redis

Issue reference
This PR will close: https://github.com/dapr/cli/issues/330
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[x] Code compiles correctly
[ ] Created/updated tests
[ ] Extended the documentation

This change is

Thanks for your contribution @ksashikumar :)


Fix invalid offset when loop option is used

Is it a bugfix ?

Yes
fix #617, #632, #635

Is it a new feature ?

No

Describe what you've done:

take loop property into account when calculating new offset in initState function
use initState.width and initState.height instead of window.width and window.height to calculate new offset in initState function
new offset is also saved to this.internals in initState function
refactor initState function a little bit

https://github.com/leecade/react-native-swiper/pull/646 claims to fix the same issues but fixes in that PR didn't help in my case.
How to test it ?

create swiper with several slides and loop={true}
go to the 1st slide
swipe back to go to the last slide
swipe forth to go to the 1st slide again
refresh page (I'm using RefreshControl component)

=>

swiper should display the 1st slide - not the last one
x offset of the slide should be calculated correctly on iOS

also I noticed (using console.log(this.state.offset) in renderScrollView function) that x offset is now correctly updated when swiping back and forth (it wasn't the case in 1.15.13 - sometimes x offset remained the same even after switching to another slide).
P.S. or else see linked issues for steps to reproduce the bug.

I found a new problem.When swiper only has a slide and loop is true, it will render empty.I tried to modify line 279 from 'const loopIndex = this.props.loop' to 'const loopIndex = this.props.loop && initState.total > 1' and fixed it.

@Devoson thanks, I'll try to reproduce this bug in few hours and will add your fix to PR if the bug is reproducible.

@Devoson sorry I didn't answer sooner - I can reproduce this bug but your fix it didn't help in my case. probably the problem is with my setup - I'll try to investigate this case further.

UPDATE: this bug was reproduced in my case exactly because of my incorrect setup. after fixing it swiper works as expected - including the case when it has one slide and loop is true.

@leecade Merge this.

Thank you.. this lib is unusable without this PR if you need to be damn sure the index you are on matches the slide shown!  👍
Had to fork and pull this in 👎


[User Documentation] Wiki Page (and/or something) for Usage with Different Linux Desktop Environments?

I was looking for a way to target the notification system for KDE Plasma, without using notify-send, when I came upon toastify.
Investigating how to use it with specific options (like KDE Plasma's "persistence" feature which keeps notifications in user-viewable history), I found that it was very difficult to identify the use of these features and providing a way to clarify this (and add it, if not currently accessible) would be very helpful.
In my example, I was expecting to be able to use these features:
neko-san@ARCH ~> toastify info
server information:
 ServerInformation { name: "Plasma", vendor: "KDE", version: "5.23.4", spec_version: "1.2" }

capabilities:
 ["body", "body-hyperlinks", "body-markup", "body-images", "icon-static", "actions", "persistence", "inline-reply", "x-kde-urls", "x-kde-origin-name", "x-kde-display-appname", "inhibitions"]

neko-san@ARCH ~>

but found that I either couldn't use them or that the knowledge necessary to use them was inaccessible to me.

Hi there!
Yes the spec is a bit unclear sometimes and the different desktop environments interpret it slightly inconsistently.
I've found that the timeout flag works relatively reliably.
So for a "persistent" notification you should be able to go timeout=0
here it says:

The timeout time in milliseconds since the display of the notification at which the notification should automatically close.
If -1, the notification's expiration time is dependent on the notification server's settings, and may vary for the type of notification.
If 0, the notification never expires.

So in toastify terms you shoud set  --expire-time 0

I mean, that sort of works, yeah but that's also the equivalent to:
notify-send 'Task Complete' --app-name=wget --icon=media-downloader --urgency=critical
(urgency=critical)
This only makes the timer never expire instead of never logging it into Plasma's notification history; which isn't bad by itself, it does guarantee the notification is seen, but in a hypothetical scenario where there's "too many" notifications: this method would fail because Plasma will just start throwing away notifications to make space for new ones (or the old ones - I forget) on-screen...
This, of course, is less likely to occur with direct average user usage but it's still feature I generally rely on so I don't forget things (sometimes I just want notifications out of the way but don't want to completely forget they existed, thus the notification history); which is part of the reason why I was looking away from using notify-send in the first place

Oh sorry, I think I mistook your original issue then. So what you are looking for is "how do I do things in general"?
So this tool basically just provides the means to create and send a notification to your desktop environment right, and it does so in a specified fashion. The specification is supposed to be linked here but I'm surprised to see that the link is actually currently dead 😨
Even the KDE page links there.
[The spec] is very generic, so what exactly happens visually is a bit up to things like GNOME and KDE themselves, and it may actually vary slightly from version to version.
Technically toastify does nothing different than what notify-send does, it sends an org.freedesktop.Notifications notification via dbus and hopes for the best.
There might be better documentation in the KDE or Gnome communities and if you find some, I'd love to link to it.
Sorry if I'm not very helpful here

Oof, okay
That's odd though; I'll try to see if I can ask in the KDE matrix chatroom about it (don't know if I'll get an answer though) and link them here so hopefully they can fill in that gap
Thanks for clarifying :ok_hand:


deactivate internal PDF Viewer

not quite an issue but is there a chance to disable the internal Chrome PDF viewer plugin programmatically ?

Sure:
Dim driver As New ChromeDriver

Dim pl_flash As New Dictionary
pl_flash.Add "name", "Adobe Flash Player"
pl_flash.Add "enabled", False

Dim pl_pdf As New Dictionary
pl_pdf.Add "name", "Chrome PDF Viewer"
pl_pdf.Add "enabled", False

Dim pl_list As New Dictionary
pl_list.Add "plugins_list", Array(pl_flash, pl_pdf, pl_native, pl_remote)

driver.SetPreference "plugins", pl_list
driver.Get "about:plugins"

thanks for the hint. I found the respective section in the preferences JSON file but did not now how to access it from within VBA. However, using the dictionary class to genreate the hierarchical structure is generally helpful.
I solved the issue another way and woud like to share my solution:
s_URL = objSelenium.url
objSelenium.get "chrome://plugins"
If<EMAIL_ADDRESS>PDF Viewer']]/div[@class='plugin-actions']/span/a[.='Deaktivieren']")) Then
<EMAIL_ADDRESS>PDF Viewer']]/div[@class='plugin-actions']/span/a[.='Deaktivieren']").Click
End If
objSelenium.get s_URL

Just found this one while writing the doc:
Dim driver As New ChromeDriver
driver.SetPreference "plugins.plugins_disabled", Array("Chrome PDF Viewer")
driver.Get "https://www.google.co.uk"
driver.Quit


(docs) yarn/npm upgrade installs rc9 instead of rc11

As I could not find information about this anywhere, it would be nice if the docs could list the following issue. Due to a bug/feature in SemVer check, version rc9 is regarded as a higher version then eg rc11. See https://jubianchi.github.io/semver-check/, put ^1.0.0-rc9 on the left and 1.0.0-rc11 on the right.
This means that if you run eg yarn upgrade it will by default install rc9. The funny thing is, if you run yarn upgrade nuxt@latest it does install rc11.
Both npm outdated as yarn outdated reflect this issue:
$ npm outdated
Package      Current       Wanted         Latest  Location
nuxt      1.0.0-rc11    1.0.0-rc9     1.0.0-rc11

$ yarn outdated
yarn outdated v1.1.0
Package Current    Wanted    Latest     Package Type URL
nuxt    1.0.0-rc11 1.0.0-rc9 1.0.0-rc11 dependencies https://github.com/nuxt/nuxt.js#readme
Done in 0.99s.

Relevant yarn issue: https://github.com/yarnpkg/yarn/issues/3560
I guess this was the same reason why in the past you switched from alpha2 to alpha.2?

1.0.0-rc11 < 1.0.0-rc9
1.0.0-rc.11 1.0.0-rc.9
Not having a dot makes them compare alphabetically.
This happens very often.
npm doesn't have epoch, so fixing this is awkward.
Next version has to be something like 1.0.0-rca.12 to get all of the -rc9 users to update.
Funnily enough, 1.0.1-rc.12 wouldn't satisfy ^1.0.0-rc11 either.

Next release will be 1.0.1-rc.12, thank you @pimlie @qm3ster for this precious information :)

That won't work :open_mouth:

@qm3ster what do you suggest instead? 1.0.0-rca.12?

As ugly it is, yeah. That correctly updates everyone and will be fixed once 1.0.0 is out.

Actually I would like to have 1 or 2 RC left before official 1.0 :/
What about 1.0.0-rca, then 1.0.0-rcb and finally 1.0.0
Ping @alexchopin @pi0

There's also -rc9[1-9] range :stuck_out_tongue:
But although I doubt there will be 8 more versions before 1.0, my vote is for a variant that compares the last group numerically, eg -rca.12 (-rca.1 would confuse people) because that's Semver™ and stands for Reeleeze CAndidate

Yes but I would like to show that these last rc are the last one:

Release Candidate A -> rca
Release Candidate B -> rcb

So we can tell everyone to upgrade to rca without having to give a number (even if it's not perfect Semver™). I don't think they will last for long anyway :)

There hasn't been a release in a while, but lots of commits every day. Is there some blocking issue preventing a release?


Hacktoberfest

Is your feature request related to a problem? Please describe.
When we are contributing to the project, we do not contribute to the hacktoberfest.
Describe the solution you'd like
Add the 'hacktoberfest' and 'hacktoberfest2021' topics to the repo.
Describe alternatives you've considered
No alternative

Done.
But if we start to get bots, I will remove it


Watching multiple refs results in a wrong TypeScript assertion in the callback

Version
3.0.5
Reproduction link
https://codesandbox.io/s/vue3-provide-inject-example-forked-f5tu1?file=/src/components/HelloWorld.vue
Steps to reproduce
If you hover over the returned values un the callback, you'll see a union type (string | number)[] instead of the correct [string, number] which can be inferred from the watched array
What is expected?
The array to be typed correctly
What is actually happening?
A union type is returned

By the way: casting the values as const works, but that seems like a bit of an escape hatch?
watch([foo, bar] as const, ([newFoo, newBar], [oldFoo, oldBar]) => {

By the way: casting the values as const works, but that seems like a bit of an escape hatch?
watch([foo, bar] as const, ([newFoo, newBar], [oldFoo, oldBar]) => {

This seems to work as intended with how TS unions arrays

This seems to work as intended with how TS unions arrays


I cant capture an image

the PeekabooCamera is just a plain camera and it doesn't provide a way to capture an image
val state = rememberPeekabooCameraState(onCapture = { /* Handle captured images */ })
PeekabooCamera(
state = state,
modifier = Modifier
.fillMaxSize()
.border(
1.dp,
Color.Red
),
permissionDeniedContent = {
// Custom UI content for permission denied scenario
},
)

@YoussefHachicha, you need to use PeekabooCameraState. There is a capture() method in there. Please, have a look at the sample app. You can find how to trigger photo capturing there.


Remove ol.format.BinaryFeature

I'm excited to have parsers for binary encodings, but this code is neither tested nor used currently.  And the addition of ArrayBuffer to the ol.format.Feature#readFeature methods is misleading (see also #3515).  Since this will forever be in our history, when we do have a specific implementation of a feature format for binary encodings, the author can decide if there is utility in this code.
There's more that can be removed after #3481 is in (specifically ol.format.FormatType.BINARY and related code in the feature loader).

Ok for me, but I think it would be nice that the removal of the binary format be done with one commit or consecutive commits, to make it easier for someone who wants to find back the old binary format code.

I am too slow apparently... see 7a8fa2c6b02678ca7e0e2da63638790d56f4506b of #3517 (WIP)
Plus one for the removal, though.

externs/vbarray.js can be removed as well

+1 on the removal as well

@tschaub see https://github.com/tschaub/ol3/compare/unbinary...fredj:unbinary for the externs/vbarray.js and ol.format.FormatType.BINARY removal.

@fredj, @tschaub waited for https://github.com/openlayers/ol3/pull/3481 for the removal of ol.format.FormatType.BINARY, to avoid conflicts.

@tschaub You can find a rebased branch here: https://github.com/openlayers/ol3/compare/master...fredj:unbinary

Thanks for the extra commits @fredj.  I think this is good to go now.  Anybody else?

I'm taking previous comments as a sign that this is good to merge.


add integration tests for a variety of 3rd party clients

Don't merge. The CI stuff can only be tested on CI so it may require some changes before working.
Adds tests for:

pymysql
python mysql
mysql ruby (the one written in ruby, included in rails)
go
dotnet mysqlconnector
mariadb for java
nodejs
php

Basically, it starts the example server and does a simple query. Because what we're checking here is that those clients are able to connect and query, that is enough.
All the tests follow the same structure to make it easier to develop new tests.
/integration
    /<test>
        Makefile

All tests must include a Makefile with the test task. Setting up dependencies and so on is the responsibility of each test.
Then, there is a /integration/run <test> script that launches the example server, runs the test and kills the example server.
For ease of use, they can be launched with make TEST=<test> integration from the root of the project.

Tests are passing now! :tada:


MI - INSTALL AND SETUP section (Enable SSL Tunneling)

Complete docs on enabling SSL tunneling [1].
[1] https://github.com/wso2/docs-ei/blob/master/en/micro-integrator/docs/setup/enabling_SSL_tunneling_thru_proxy_server.md

Copy editing completed.
On hold until TOML configs are finalized.

Done.
https://ei.docs.wso2.com/en/latest/micro-integrator/setup/enabling_SSL_tunneling_thru_proxy_server/


Writing EXIF/IPTC tags to jpeg

Although it is possible to read EXIF and IPTC from jpeg, there's no way to write data to files.
What about this feature? Does somebody want/need that?

Unfortunately GD is not able to deal with meta data at all and Imagick can retain existing data but not write new values. I think proper dealing with meta data is worth a dedicated package and I recommend using libraries like Exiftool.

https://github.com/romainneutron/PHPExiftool


"knowledge addtemporal" meaning

I'm so new to the topic, could I ask  about the meaning of "knowledge addtemporal", please? I have read explaining file already, but I'm still so confused about that? Could you briefly explain it please? Thank in advance!

In case it's not clear, I mean I don't know what exactly those following rows mean
"1 Region Month
2 Temperature Ws Rain RH
3 FFMC DMC DC
4 ISI BUI
5 FWI"
Thank you.

In our Tetrad application, you could load up this knowledge file and use it to guide a search, but it's being used here instead to encode what seems to be the case about the ground truth for the data. So it's interesting to see if you can make an algorithm that recovers these relationships without the crutch of knowing them in advance or can recover many of them in any case.

So, in other words, if you ran an algorithm, and it told you that Temperature causes Region, that would count against it if you take this knowledge to be correct.


Query to access third-party render-blocking script

Create an HTTP Archive query to find out what % of WordPress sites use third-party render-blocking script.

Initial draft for the query.
SELECT
  percentile,
  client,
  host,
  APPROX_QUANTILES(requests, 1000)[
OFFSET
  (percentile * 10)] AS num_requests
FROM (
  SELECT
    client,
    page,
  IF
    (NET.HOST(requests.url) IN (
      SELECT
        domain
      FROM
        `httparchive.almanac.third_parties`
      WHERE
        date = '2022-06-01'
        AND category != 'hosting' ), 'third party', 'first party') AS host,
    COUNT(0) AS requests
  FROM
    `httparchive.almanac.requests` AS requests
  JOIN
    `httparchive.pages.2022_10_01_*` AS pages
  ON
    pages.url = requests.page
  WHERE
    date = '2022-06-01'
    AND type = 'script'
    AND JSON_EXTRACT(pages.payload, '$._detected_apps.WordPress') IS NOT NULL
    AND CAST(JSON_EXTRACT( pages.payload, '$._renderBlockingJS') AS INT64) > 0
  GROUP BY
    client,
    page,
    host),
  UNNEST([10, 25, 50, 75, 90, 100]) AS percentile
GROUP BY
  percentile,
  client,
  host
ORDER BY
  client,
  percentile,
  host

cc. @felixarntz

@mukeshpanchal27 It looks like this query is based on https://almanac.httparchive.org/en/2022/javascript#requests? Please provide such sources for reference :)
I'm not familiar with the nature of the CMS Almanac-specific tables httparchive.almanac.*, so I'm not sure we can rely on those here. Also, please review the dates, as parts of your query are for October 2022, while others are for June 2022, so that would definitely give us incorrect data.
Last but not least, I see you formatted the query via the BigQuery formatter, which generally is good, however it sometimes formats some parts in a weird way, so in such cases we should manually fix this. For example, this is unnecessarily hard to read:
  APPROX_QUANTILES(requests, 1000)[
OFFSET
  (percentile * 10)] AS num_requests

Would be simpler as:
  APPROX_QUANTILES(requests, 1000)[OFFSET(percentile * 10)] AS num_requests

Feel free to make those changes and open a draft pull request. Better to review in there, not in the issue.

Thanks @felixarntz, for the comment.
The httparchive.almanac.third_parties has data till June 2022, so I have updated my query to use June 2022 data for httparchive.pages.2022_06_01_*
The query format was also corrected.

@felixarntz @mukeshpanchal27
The goal here is to study third-party scripts that are render-blocking.
Issue with initial query is:

httparchive.almanac.third_parties in combination with httparchive.almanac.requests do provide us with information on third-party scripts. Howevery it doesn't record render-blocking nature of the script
httparchive.pages.2022_06_01_* tell's the count of render-blocking script but can't tell if we the third-part script is one among them.
The current query gives a count of third-party scripts from sites having at least on render-blocking script, but it can't say if the third-party script is one among them.

We can get both the information from the lighthouse report but traversing it is a bit expensive.
I have come up with a draft query below:
CREATE TEMP FUNCTION
  getCountRenderBlockingTPJS(lighthouse STRING,
    pageUrl STRING)
  RETURNS INT64
  LANGUAGE js AS '''
try {
const report = JSON.parse(lighthouse, pageUrl);
const networkItems = report.audits["network-requests"].details.items
const fcp = report.audits["first-contentful-paint"].numericValue
let countRenderBlockingTPJS = 0
for(const item of networkItems ) {
  // network-items are arranges in order of start time.
  // render-blocking resources start before fcp
  // network can be very large, faster we break faster is the query
  if(item.startTime > fcp) break;
  if( item.resourceType == "Script" &&
      !item.url.includes(pageUrl) && // to determine third-party
      (item.priority == "VeryHigh" || item.priority == "High") //render-blocking
    )
  {
    countRenderBlockingTPJS++;
  }
}
return countRenderBlockingTPJS
} catch (e) {
  return -1;
}
''';
SELECT
  client,
  total_wp_sites,
  with_3pJS_render_blocking,
  (total_wp_sites-with_3pJS_render_blocking) AS without_3pJS_render_blocking,
  CONCAT(ROUND((total_wp_sites-with_3pJS_render_blocking)*100/total_wp_sites, 3),' %') AS opportunity
FROM (
  SELECT
    lh._TABLE_SUFFIX AS client,
    COUNT(lh.url) AS total_wp_sites,
    COUNTIF( getCountRenderBlockingTPJS(lh.report, lh.url) > 0 ) AS with_3pJS_render_blocking
  FROM
    `httparchive.technologies.2022_12_01_*` AS tech
  JOIN
    `httparchive.lighthouse.2022_12_01_*` AS lh
  ON
    tech.url = lh.url
  WHERE
    app = 'WordPress'
    AND category = 'CMS'
    AND lh._TABLE_SUFFIX = tech._TABLE_SUFFIX
  GROUP BY
    lh._TABLE_SUFFIX )


fix: cannot npm ci without lock file

So instead we run npm i

Codecov Report
Base: 36.93% // Head: 36.93% // No change to project coverage :thumbsup:

Coverage data is based on head (d86f823) compared to base (7e2c6be).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #342   +/-   ##
=======================================
  Coverage   36.93%   36.93%
=======================================
  Files          17       17
  Lines        4364     4364
  Branches      157      157
=======================================
  Hits         1612     1612
  Misses       2750     2750
  Partials        2        2

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Invalid unicode in ld+json output

What version of Hugo are you using (hugo version)?

$ hugo version
- hugo v0.74.3
- hugo v0.81.0+extended darwin/amd64
- hugo v0.82.0+extended darwin/amd64

Follow-up on:

https://github.com/gohugoio/hugo/issues/6524
https://github.com/gohugoio/hugo/issues/6695

I'm trying to render frontmatter data into ld+json:
<script type="application/ld+json">
  {
    "@context": "https://schema.org",
    "@type": "FAQPage",
    "mainEntity": [
      {{ range .Params.faqs }}
        {
          "@type": "Question",
          "name": "{{ .question | jsonify | safeJS }}",
          "acceptedAnswer": {
            "@type": "Answer",
            "text": "{{ .answer_schema | jsonify | safeJS }}"
          }
        },
      {{ end }}
    ]
  }
</script>

Output:
"name": "\u0022What is ...\u0022",
...
"name": "\u0022What\u0027s",

I tried the recommended solutions (| jsonify | safeJS), but nothing works.

Unless I'm missing something, this works fine.
content
+++
title = "Test"
date = 2021-04-04T22:55:13-07:00
draft = false

[[faqs]]
question = "What is"
answer_schema = "What's"

[[faqs]]
question = "What's the best song by Jackson Browne"
answer_schema = "Sky Blue & Black"
+++

template
<script type="application/ld+json">
  {
    "@context": "https://schema.org",
    "@type": "FAQPage",
    "mainEntity": [
      {{ range $k, $v := .Params.faqs }}
        {{ with $k }},{{ end }}
        {
          "@type": "Question",
          "name": "{{ .question }}",
          "acceptedAnswer": {
            "@type": "Answer",
            "text": "{{ .answer_schema }}"
          }
        }
      {{ end }}
    ]
  }
</script>

I think the issue with your original code was not an encoding problem, but instead an extraneous comma after the last item in the loop. Please let me know if I've misunderstood.
In the future please use the Hugo forum (https://discourse.gohugo.io) for questions and troubleshooting. We prefer to use GitHub for verified bugs and enhancements. Thanks.

Apologies, will do next time. The issue seems to be that the schema is in a partial. Here's a minimum reproduction of the issue:

https://gist.github.com/PatrickHeneise/18b792cc4d66b01cad6c277f477bc6b9

Do this:
git clone --single-branch -b hugo-github-issue-8386 https://github.com/jmooring/hugo-testing hugo-github-issue-8386
cd hugo-github-issue-8386
hugo server

Browse to http://localhost:1313/.
View source. Copy this part to your clipboard:
<script type="application/ld+json">
  {
    "@context": "https://schema.org",
    "@type": "FAQPage",
    "mainEntity": [

        {
          "@type": "Question",
          "name": "What\u0027s the best song by Jackson Browne",
          "acceptedAnswer": {
            "@type": "Answer",
            "text": "It\u0027s Sky Blue \u0026 Black"
          }
        }

    ]
  }
</script>

Go here (https://search.google.com/structured-data/testing-tool) and paste it as a code snippet.

Sorry about this. You're right, the unicode chars are interpreted the right way. Thanks a lot!


ZeroDivisionError

Thanks for your wonderful repo, but I got the following problem.
I use parameters with different epoch to detect the same dataset (epoch_50.pth, epoch_90.pth.....), for example, detection results of epoch_50.pth can get the right AP, but when I use the results of epoch_90.pth, main.py will return ZeroDivisionError(line 589). I am confused why could this happen, please help me.

Uhh that's weird... is it related to any file in specific? Can you debug it? If you prefer you can send me the files and I could have a look<EMAIL_ADDRESS>


MDCA Investigation Priority Score Retirement

The investigation priority score is retiring, published in message center under MC889532, it is expected to be fully decommissioned by the end of November.  This effectively renders the MDCA module inoperable.
Tenants where this has been decommissioned and are using the MDCA module will start to see an error like this:
{
    "Error": "Module processing failed, an unknown exception has occurred.",
    "InvocationId": "170d5892-08cd-4c18-9e2b-5aaf263fa877",
    "Traceback": [
        "Traceback (most recent call last):\n",
        "  File \"/home/site/wwwroot/modules/__init__.py\", line 19, in main\n    return_data = coordinator.initiate_module(module_name=module_name, req_body=req_body)\n",
        "  File \"/home/site/wwwroot/shared/coordinator.py\", line 21, in initiate_module\n    return_data = mdca.execute_mdca_module(req_body)\n",
        "  File \"/home/site/wwwroot/modules/mdca.py\", line 50, in execute_mdca_module\n    current_account['ThreatScore'] = 0 if mdcaresults['threatScore'] is None else mdcaresults['threatScore']\n",
        "KeyError: 'threatScore'\n"
    ]
}

So we likely need to retire the module but would like to do so in a way that won't create much impact for anyone who is calling it.  I propose we:

[x] Add a section to the FAQ in the Wiki
[ ] Change the module to always return successfully with minimal payload, adding a comment to the incident that they are using a deprecated module, include enough so the scoring module won't fail
[ ] Hide the MDCA action in the custom connector so it won't break existing deployments, but you won't be able to add it in new
[ ] Add warning to the scoring module if MDCA module is passed
[ ] Remove from test cases

@piaudonn am I missing anything?  Or in the alternative, is there another investment we should make into the module to save it?

I feel like this is the end for our integration...

@piaudonn yeah I think so too, I don't see anywhere else to go with it.  Do you agree with the steps I've listed in original post or do you think I'm missing anything that we need/should do?


NIFI-10108 Processor scheduling via parameter

Summary
NIFI-10108 enables Processors' Scheduling Period to be configured via parameters. A use case may be a generic flow (extracting data from a table) should run for table A every hour in environment A but for table B every day in another environment. Using parameters it can be easily achievable without modifying Processor configuration.
Tracking
Please complete the following tracking steps prior to pull request creation.
Issue Tracking

[x] Apache NiFi Jira issue created

Pull Request Tracking

[x] Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-00000
[x] Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-00000

Pull Request Formatting

[x] Pull Request based on current revision of the main branch
[x] Pull Request refers to a feature branch with one commit containing changes

Verification
Please indicate the verification steps performed prior to pull request creation.
Build

[x] Build completed using mvn clean install -P contrib-check

[x] JDK 8
[ ] JDK 11
[ ] JDK 17

Licensing

[ ] New dependencies are compatible with the Apache License 2.0 according to the License Policy
[ ] New dependencies are documented in applicable LICENSE and NOTICE files

Documentation

[ ] Documentation formatting appears as expected in rendered files

Hello @markap14, Can you please review?

Thanks @timeabarna will do.

Hello @markap14,
Thanks for your recommendations. I've refactored the code so most of the newly created methods have been removed. I've refactored the code in onParametersModified() method and moved the component referncing check from StandardParameterreferencingManager.class to AbstractComponentNode.class.
Can you please review the modifications?
Thanks in advance.

Thanks for updating @timeabarna. I think all of my concerns have been addressed. I did some testing and everything appears to be working as expected. I think there's some potential in the future for some improvements to the UI in order to allow auto-completion like we do in properties. But that can be done in a follow-on effort. Things look good here from my perspsective.
+1 will merge to main.


Alias Breadcrumb Link for non-menu content hierarchy

See #3537. Would like to demo today and would be great if we could add it still to 1.10.

Argh, misclicked the branch name.


darksidewalletd: set coinbase height correctly in fake blockchain tool

Can you please explain to me what coinbase height is and the advantage of setting it correctly?

TODO: After #186 merges there'll be a  TODO COINBASE-HEIGHT: restore this check after coinbase height is fixed comment in the code to address once this is done.
@LarryRuane Sorry I missed your comment, I think you already understand but the issue is just that the fake blocks generated by the fake-block-generating tool don't have the correct block heights encoded in their coinbase transaction because I lazily copypasted some random coinbase transaction to use in all of the blocks.


Add PGP library docker, scripts and endpoint

We don't have a dockerized PGP library so this needs to be added to https://github.com/SatoshiPortal/dockers
The PGP keypairs would be added in the configs of the Proxy. Possibly a good idea to put an additional password on modifying the keypairs since we want this to be resistant to an attacker take control of the app which would call the Cyphernode via API (don't want the attacker to be able to change the PGP key). and this is not a burdgen because you wouldn't be doing this too often anyway.
I think we should use https://github.com/openpgpjs/openpgpjs because it is so widely used and maintained, but other libraries can be used if they are good.
I think cleartext signing and verification + encrypt\decrypt string would be what we need.
TODO:

[ ] Dockerize OPENPGP.JS

[ ] Add PGP keypairs to proxy configs.

[ ] Add "clearsign" endpoint that  dispatches data to pgp docker to create a cleartext signature.

[ ] Add "detachedsign" endpoint that dispatches data to pgp docker to create a detached signature.

[ ]  Add "verify" endpoint for both detached and clearsign

https://github.com/openpgpjs/openpgpjs/blob/1bee091f2acd054b9cd78c6eca5c139061761662/src/cleartext.js

[ ]  In verified response, give smart info: fingerprint, errors

[ ] Add "encrypt" and "decrypt" endpoint that dispatches data to pgp docker to create a encrypted file or string

[ ] Add "encrypt" and "decrypt" endpoint that  dispatches data to pgp docker to create a encrypted file or string

BONUS
(privacy concerns)

[ ] When returned the fingerprint, add "keybase lookup" (e.g. for Bylls) https://keybase.io/_/api/1.0/user/discover.json?key_fingerprint=7B32CB5B143F0D26549E01AAA199584252141E1D
[ ] Look up on public key servers

Update: still pending. Contributions welcome

Self-assigned


#feat Major; Rocketpool Subgraph

I am not sure why build-checker fails... But this is the one.
https://thegraph.com/hosted-service/subgraph/benreilly19/rocketpool?version=pending
https://subgraphs.messari.io/subgraph?endpoint=https://api.thegraph.com/subgraphs/id/QmafQwPy2bua6UMnn92kGCSjEoMCAA98Jf4Vq9UZV9WoXK&tab=protocol

I am not sure why build-checker fails... But this is the one.

It's failing because your template file name and the name you put in the deployment.json don't match:
rocket-pool.template.yaml

@jaimehgb fixed build

@dmelotik corrected most if not all of these issues.

I am really not sure how to fix the occasional metrics dropoff. My only assumption would be that there are certain times in the chain when the getters fail for my smart contract calls.
If there is a way to work around that please let me know, but currently not sure how to attack this problem. @dmelotik
Other than that, all other issues either resolved or responded to 👍

If there is a way to work around that please let me know

@zdql It is prolly bc there are tiems where the pool is created, but the stakedOutputTokenAmount is never updated. I would just include that part in the creation of the poolSnapshot entities

Fixed all errors in latest commit, as well as adjusted to hopefully correct random 0 values in stakedoutput and other metrics for snapshots.
https://subgraphs.messari.io/subgraph?endpoint=https://api.thegraph.com/subgraphs/id/QmZ9Zasm4m92oXhxDBP4YCqrnsHN3sXbVNAC4w6jnF14oN&tab=protocol
@dmelotik

@dmelotik wow huh, looks like it might have made it worse.  That is bizarre, the pool should only get initialized once and from there the snapshots are set to the pool's staked output token amount. So I have no idea where this is coming from.

@dmelotik
Based on how hourly snapshots are getting initialized I don't see how it's possible we'd get these erroneous zero values. Can you take a look at the code here and see where this could be happening?

@zdql Does this seem like a more accurate call for staked RPL? https://github.com/rocket-pool/rocketpool/blob/master/contracts/contract/node/RocketNodeStaking.sol#L66
This seems like it would also fix the RPL spike

New deployment: https://thegraph.com/hosted-service/subgraph/zdql/rocket_pool_v2?selected=logs
RPL spike corrected:
https://subgraphs.messari.io/subgraph?endpoint=https://api.thegraph.com/subgraphs/id/QmPjQq8DfntV4McdLhN1Fe6JMoRPE1FPn2Brf3Sptsu6D3&tab=pool&poolId=0xae78736Cd615f374D3085123A210448E74Fc6393


Install on private k8s AWS private cluster

Hi folks!
We look for logging/metrics system for our system to replace the existing one based on AWS OpenSearch service.
Our K8S runs on a private cluster on AWS - no access to any public services/addresses, only AWS services are available.
We found SigNoz project very-very interesting option!
We installed the SigNoz on the cluster based on your helm charts. But we had at least two issues:

The location of the DB installation scripts can't be configured thru helm charts. It was hardcoded somewhere to http://github.com, which is not accessible from our cluster. We copied the scrips to our S3 bucker and changed the location in the service K8S service.
The second issue is that on the first login the Admin user should be created. But the "invite URL" should be sent (I see errors in the service log) via an external email service. As you can guess - the service also is not accessible. Finally, after some timeout we were able to login into the UI but the system does not feel well - sometimes we can't login, the "Settings" screen can't be opened, and some Admin functionality are not available.
Maybe there are additional cases when installation or SigNoz's services are trying to access public services?

Can SygNoz be installed on the private cluster? Is there any fixes/patches to resolve the above problems?
In any case, you have a great project!
Thanks in advance!
Dmitry

Hi @yukels , it should be possible to run SigNoz in a private cluster. Curious to see the problems and sort them out. Possible to join our Slack community. Slack offers better communication for sharing errors and logs.


Problem with 'npx create-react-app my-app'

C:\Users\Shubham Ghosh\Desktop\react0>npx create-react-app my-app
npm ERR! code ENOLOCAL
npm ERR! Could not install from "Ghosh\AppData\Roaming\npm-cache_npx\12260" as it does not contain a package.json file.
npm ERR!      C:\Users\Shubham Ghosh\AppData\Roaming\npm-cache_logs\2020-04-21T07_28_45_154Z-debug.log
Install for [ 'create-react-app@latest' ] failed with code 1

Solution: npm set cache "C:\Users\ReactJs" --global


[HOLD for payment 2024-08-02] [CRITICAL] [Advanced Approval Workflows] Create <AddApprovalWorkflow /> button

Coming from: #wave-control: advanced approval + workflows polish
Issue OwnerCurrent Issue Owner: @blazejkustra

Hey! I’m Błażej Kustra from Software Mansion, an expert agency, and I’d like to work on this issue!

Weekly Update

The PR was deployed quite a while ago.
@mollfpr were you ever compensated for your C+ review on the PR?

Next Steps

@tgolen Get payments cleared up and close this out

ETA

Closed by Wednesday, Sep 4

@tgolen I think we can close this. I already paid here: https://github.com/Expensify/App/issues/45947.


Mmca 4696 - Display only latest six months statements including current month

1 Test case is failing, need to fix it


Disconnect: Authentication Failed

https://user-images.githubusercontent.com/59307885/149797239-ddf721b9-21bb-4d8d-840b-28c64644d074.jpg
Bug
Unable to run Northstar.
Steps to reproduce

Install TF
Run Northstar through the command line parameter in Origin
Click on Northstar in the menu and get the above error.

Specifications

Northstar version: 1.5.1
Platform: Steam

I have had the exact same problem since 8 AM EDT on 20 March. Northstar work fine for TF2 on Saturday 19 March, having played for most of that day.
Gravis Maximus

I get the same error, but I got TF|2 off of origin.
Here's the console info that I got, but my ip is scribbled out

Northstar servers are currently shut down because of an exploit in the vanilla game https://github.com/R2Northstar/Northstar/issues/124#issuecomment-1073206536
For more info, you can join the official Discord server

As said in the issue @SamLam140330 linked to, we took down master server to investigate a few security vulnerabilities. So you not being able to connect is intentional.


Android toolchain - Could not determine java version

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce
[!] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
X Could not determine java version
Actual results

Logs

Logs

<!-- Paste your logs here -->

Flutter Doctor output

Doctor output

<!-- Paste your output here -->

Hi @shehanm95, do you have Java installed? Try typing java or javac in your command prompt to verify you do. These binaries should be present in your $PATH.
I also see you have Android Studio 4.1 installed - that's pretty old. Consider upgrading to the latest version.

I'm having the same issue.
I have java installed:
$ java -version
java version "1.8.0_391"
Java(TM) SE Runtime Environment (build 1.8.0_391-b13)
Java HotSpot(TM) 64-Bit Server VM (build 25.391-b13, mixed mode)
I do have an older macOS Monterey 12.7.1
Here is my flutter doctor response
$ flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
Could not parse java version from:
If there is a version please look for an existing bug
https://github.com/flutter/flutter/issues/ and if one does not exist file a new
issue.
[✓] Flutter (Channel stable, 3.13.9, on macOS 12.7.1 21G920 darwin-x64, locale
en-US)
[!] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
✗ Could not determine java version
[!] Xcode - develop for iOS and macOS (Xcode 14.0.1)
✗ CocoaPods not installed.
CocoaPods is used to retrieve the iOS and macOS platform side's plugin
code that responds to your plugin usage on the Dart side.
Without CocoaPods, plugins will not work on iOS or macOS.
For more info, see https://flutter.dev/platform-plugins
To install see
https://guides.cocoapods.org/using/getting-started.html#installation for
instructions.
[✓] Chrome - develop for the web
[!] Android Studio (not installed)
[✓] Connected device (2 available)
[✓] Network resources
! Doctor found issues in 3 categories.


[BYOC][TVMC] bugfix: disabled_pass -> disable_pass

This PR fixes typo in tvmc compile command line disabled_pass -> disable_pass
Change-Id: I22d24a2e219103485a6a1519ce6256a104103ebb

Thanks @d-smirnov @leandron


Move NodePipelineInterface and createFormatter out of the api namespace

Currently, NodePipelineInterface and createFormatter() are exported from api/mooltipage.ts. These are both internal and should be located somewhere else. They can be in a new file or merged into an existing one, as long as their new locations make sense and are not in the api folder.

This will be partly completed during #28 - NodePipelineInterface will be removed.

This is done and will be merged with #25.


max_grad_norm and use_clipped_value_loss

Hello! I was documenting your PPO code algo/ppo.py to improve my understanding of the algorithm, and I got confused on max_grad_norm and _use_clipped_value_loss.
If I am understanding this correctly, max_grad_norm is given to nn.utils.clip_grad_norm_() to set maximum gradient size, and _use_clipped_value_loss. However, I could not find relevent detail in the paper Proximal Policy Optimization Algorithm. If it was explicitly mentioned here, would you please point it out for me?
For L^VF, The paper seems to use the simple squared loss, equivalent to use_clipped_value_loss=False, but I could not find anything about the case when use_clipped_value_loss=True. Is this a trick not mentioned in the paper?
Thank you in advance for your help. Happy holidays!

They introduced the new loss in the implementation of PPO2:
https://github.com/openai/baselines/blob/master/baselines/ppo2/model.py#L63
Also see grad normalization here:
https://github.com/openai/baselines/blob/master/baselines/ppo2/model.py#L102

Thank you for the links! I see how they correspond to those parts of PPO2 in OpenAI Baselines.
It's unfortunate that these changes are not written in any paper. Guess I will have to read openai/baselines code as well.


Fix OpenGL _shadow_atlas_find_shadow error when light instance is freed

Fixes #84747

In file light_storage.cpp, when engine free a light instance, it didn't free the shadow atlas and set owner of it to RID().
https://github.com/godotengine/godot/blob/master/drivers/gles3/storage/light_storage.cpp#L366-L384
So the next time engine try to find a useable shadow atlas, it will have this issue
I think we can just ignore the case sli == null and return the shadow atlas when we meet it, since the light instance of that atlas already freed

Looks great! I left a small comment to clean up a line we don't need. Otherwise this looks great

Thank you, you was right. I removed that line.

Thanks! And congrats for your first merged Godot contribution :tada:

Cherry-picked for 4.2.2.


null refresh_token

3/5/2017,  same call that worked all week is returning json token data with a null refresh_token field.
https://login.eveonline.com/oauth/token
Response:
"{"access_token":"3Nx4ugW7S......","token_type":"Bearer","expires_in":1200,"refresh_token":null}"

ccpgames/esi-issues#274

I'm not understanding biow0lf..
This has been working all week and just today it stopped giving me a refresh token.  That says something.
I am authorizing this scope in the authorization request:
"esi-markets.structure_markets.v1"
The same scope is also setup in my application permissions for my account.
I see no scope called "public_data" in the published list of scopes on developers.eveonline.com
Can you please clarify ?  Thanks

@bbenham I just checked, if I pass scope 'esi-skills.read_skills.v1' then I receive token and refresh token. If I omit scope at all, backend doesn't return refresh token. Its not null, it just miss in response.

I can provide example rails app with omniauth eveonline gem for testing.

Strange.. it's sending me a refresh token today..  I'm not sure what to make of it.  Perhaps something temporary?  Anyway, seems to be working today. Thanks for helping Biow0lf.. much appreciate it.

I have the same issue. With no scopes i get always a null refresh_token


Fix importing wrong version of HTTPError exception

Fixes https://github.com/elbuo8/sendgrid-django/issues/74

What is blocking this?

Merging #77 into master will decrease coverage by 8.22%.

Is there a workaround?  Or did the person with the PR just abandon?
Sadly I am not skilled enough to contribute to solving the problem, but I
can say that if it doesn't actually report exceptions the library is
essentially unusable...
On Fri, Mar 16, 2018 at 12:45 PM, Andrii Soldatenko <
<EMAIL_ADDRESS>wrote:

Merging #77 https://github.com/elbuo8/sendgrid-django/pull/77 into
master will decrease coverage by 8.22%.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/elbuo8/sendgrid-django/pull/77#issuecomment-373824945,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAJ3ugBrwCD7rnWA1VDlb9kVSkHp1ES8ks5tfBZKgaJpZM4RcFRC
.

The problem with this is that the 3rd party library that's responsible for connecting with sendgrider is hard to mock. I will look into that further and in the meantime I can write to authors of this aforementioned library for help


import org.slf4j instead of parquet.org.slf4j

What is this PR for?
Import org.slf4j package instead or parquet.org.slf4j in ZeppelinR.java
Related to the error report http://apache-zeppelin-users-incubating-mailing-list.75479.x6.nabble.com/Zeppelin-0-6-Build-tp3000.html
What type of PR is it?
Bug Fix
Todos

[x] - Change package import

What is the Jira issue?
N/A
How should this be tested?
Screenshots (if appropriate)
Questions:

Does the licenses files need update? no
Is there breaking changes for older versions? no
Does this needs documentation? no

LGTM 👍

@prabhjyotsingh Thanks for taking a look.
CI failure is unrelated.
Merge into master if there're no more discussions


Possibility to set up alarms from calendar events.

An interesting feature—that does not exist in the official app—would be the possibility to set up alarms, instead of day reminders, from calendar events. The purpose would be to have the watch ringing few minutes before selected timed events (appointments, meetings, etc.). They would simply remind the user that he has something to do soon, and let him check his agenda for it.
The user may be able to get a selected number (n) of the watch alarms dedicated to this purpose. The app would be able to grab the n next events of the current day or of the next following 24 hours (eventually from a choice of selected calendars among the one available), grab their time, and setup alarms a few minutes before each of them (this number of minutes being configurable, typically from 5 to 15 minutes is useful). Old alarms would be wiped up or deactivated, if slots are still available.
This may be set up manually. For example, each morning, the user would be able to quickly set up ringing alarms for is few main appointments or meetings of the day. This may also be set up automatically: this process would be performed at each automatic time sync. This may finally be set up half-automatically, as an action; the user would simply have to push the down-right button of his watch to ask the app to grab the next meeting alarms and set them up on the watch.
Disclaimer: I have never even attempted to build an android app. These are suggestions from how I understand this may be working... I do not know how difficult it would be to have this feature.
Thank you

@rplasson Thanks for your suggestion. I look at every feature request very carefully and have implemented many suggestions. Unfortunately, I don't think this will work well for the following reasons:

Complicates the operation of the app. The user should be able to use all its alarms without worrying about some being overwritten.

Time updates are not reliable, even with the official app. For example, if the user leaves his phone away from the watch, these alarms will not be set reliably. Also, alarms on the watch repeat every day, so they have to be unset the next day. If the process fails, this will create confusion. Remember, alarms do not have titles, so users will be confused about which alarm is for which reminder.

In general, adds more work for the user. Most people just want to use their watch and use the app occasionally. Adding this feature will require the user to constantly be interacting with the app.

I had another suggestion in the past to automatically place the time of the event as part of the reminder title. However, we are dealing with a limited 20 characters for this title, so adding time will take valuable space.
In general, these are not smartwatches, and they have their limitations. We work with what we have, trying to make it easy to configure, but not changing their functionality.


fix example table url of material ui

Coverage remained the same at 87.937% when pulling a92fb8b6937e8e2e7f48104a9fe73fe9fc33fe91 on vaibhavmule:patch-1 into 91c184a8920573fc02b61534de3ce848785e8464 on hyojin:master.


Adding support for renamed redis rpush and publish commands.

This is the pull request that supports this new feature request:
https://github.com/logstash-plugins/logstash-output-redis/issues/16

@timezra thanks for your contribution! I agree with the idea of #16. Can you review and do step 2 of our contributing guide ?

You should now have my e-signature on record.
Thanks for all the great work on this plugin, Jordan!
---Tim---

Jenkins standing by to test this. If you aren't a maintainer, you can ignore this comment. Someone with commit access, please review this and clear it for Jenkins to run; then  say 'jenkins, test it'.

please jenkins, test this.

I thought I had signed the CLA, but apparently it did not go through. I re-signed the CLA, so this should pass the second check if you are willing to re-test this. Sorry about that.


Svelte Components

Adds Svelte components, which can be built with npm run build-svelte.
They are exported as .svelte components rather than compiled .js files. This is the recommended way to provide svelte components in a library, and are found via the svelte entry field in package.json.
If there's a strong argument for compiling them as .js files, I did do this initially using svelte's compiler API but I could not get them to work in SSR environments. I am sure there's a solution to this, but at that point I felt .svelte components may be best anyway.

@adamwathan and @sschoger , can this be merged in for svelte users please

I've been looking at adding typescript types so that the components use SVG attributes as their props. However, Svelte doesn't ship types for HTML elements, instead they're in their svelte2tsx package (which powers their language server). So in order to use those we'd have to add that as a dependency to @heroicons/svelte.
If we're OK with doing that for the sake of types, I can push my changes to this PR. I have these changes published in<EMAIL_ADDRESS>in the meantime.

Wow, thanks for doing this @mattjennings!!

If anybody need it to install now:
npm i https://github.com/mattjennings/heroicons.git\#@mattjennings/heroicons-svelte

or add to package.json:
"dependencies":
...
    "heroicons": "git+https://github.com/mattjennings/heroicons.git#@mattjennings/heroicons-svelte",
...

How are we going with this one?

Maintaining additional wrapper libraries isn't a priority right now I'm afraid, better for someone who wants this to exist to publish and maintain it themselves 👍🏻
Thanks either way I do appreciate the contribution ❤ we just don't have the resources to take on additional OSS maintenance.

No worries, I understand the decision completely.
I've published this under heroicons-svelte now and will keep it maintained as best I can.


feat (DDI-1259): Update popover position to be dynamic

:tada: This PR is included in version 4.0.0-alpha.144 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0-alpha.183 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 4.10.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.10.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.0.0-alpha.17 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0-alpha.21 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.2.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.3.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Commit #2 11/9/2016 fixes

Naruto D20 Modern
-Fixed Climb skill from saying /e rolls Bluff!
-Fixed Chakra Control skill fixed it from adding Balance as its source of bonuses
-Fixed Taijutsu skill not showing total amount
Still really new to Github and how it works, somehow committed extra stuff after syncing my clone repository? Sorry for those

-Fixed Climb skill from saying /e rolls Bluff!
-Fixed Chakra Control skill fixed it from adding Balance as its source of bonuses
-Fixed Taijutsu skill not showing total amount
-Fixed Chakra Pool not calculating properly
-Fixed Technique categories from all being read from the same entry

You have some deleted files from other sheets. The easiest, and most consistent, way to make sure your branch is up to date is to delete your branch after I have accepted your PR. Then when you're ready to make a new one, create a new Fork, add your changes, then open the PR. That way you are always working withe the newest code.


Update XDR validation + show error message on View XDR page

Endpoints > Transactions > Post Transaction was breaking when XDR was invalid. Updated validate.xdr method to fix it (we were expecting different format for the error).
XDR > View XDR > Added response error message to the error for clarity.

Preview is available here:http://laboratory-pr877.previews.kube001.services.stellar-ops.com/

Preview is available here:http://laboratory-pr877.previews.kube001.services.stellar-ops.com/


Parameter validation failure may print "&" instead of the actual type of the given variable

Describe the bug
It is sometimes the case that a parameter validation failure due to type mismatch in the parameters would print the type of the given argument is "&" instead of its actual type. Possibly an issue with the logger printing the type of the variable before the type has been resolved.
[error] [2023-05-12 14:18:26.409] [T-7872] [SHCore.cpp::1193] Parameter validation failed: Parameter Message not accepting this kind of variable: & (valid types: String) shard: Msg
[error] [2023-05-12 14:18:26.411] [T-7872] [SHCore.cpp::1239] Failed parameter index: 0 line: 17 shard: Msg

To Reproduce
(defwire example
  (Msg .undeclared))

(schedule main example)
(run main 1 1)

or
(defwire example
  3 (ToString) >= .test-value
  (Msg .test-value))

(schedule main example)
(run main 1 1)

Expected behavior
The error should clearly output the type of the given variable rather than a symbol "&".

It uses the same formatter as the docs (so & means ContextVar but it looks weird without any following type).
I should probably say &String, the formatter probably doesn't have any context to get the type from, which can maybe be added.


Pytree refactor

Just opening this right now so that we can get the GLM example rendered by read the docs. More work needs to be done here.
For the example in particular:

I'm using a dataset from DANDI, looking at head direction and 2d position.
I think this example is too complex to introduce the pytrees and more/better text is needed here.
Writing this made it clear that we can easily put together some simple helper functions that will make things easier, like doing the multiplication between bases and coefficients.
When this goes into the docs, the general "here's a pytree" should probably be a separate tutorial from "here's how to use them with GLMs".
I added some extra dependencies in pyproject.toml, but that's just so this example builds. We should think about how we want to handle this.

I addressed Guillaume's comments and added tests for pytrees. There weren't many, but I think they're sufficient. Let me know if there's anything else I should add to them.

Currently, one test is failing:

test_fit_init_params_type: when input is iter or a dict with keys p1,p2. The problem is that we expect params to be a 2-tuple (in order  to be compliant with sklearn, I thin), but this leads to some weirdness, as in these cases. We should talk through this


Hide relationship connectors in model tree window

This is somehow the same request as #646 but only to hide relations in the model window, not in the views.
As stated in #646, relations account for most of the size/height of this window and make it uncomfortable each time you want to search concepts using this tree view because you have to scroll A LOT...
It might be enough to let the "Relations" node closed by default (which is currently open by default) ?

You can hide any of the folder types in the Models Tree since Archi 4.10. See pages 16-17 of the User Guide.

All right ; I'm still on 4.9 that's why, will wait for the admin to install the latest version. Thanks !


fix: change filter pattern to satisfy new tags

Description

This pr modify the filter pattern of chglog to satisfy new tags.

In fork branch, like dynamiclink branch, we will use following version rule to release the tag in the feature. ^v\d+.\d+.\d[+-]\d+.\d+.\d+-dynamiclink.+$
For example, v1.0.0-0.7.0-dynamiclink1, v1.0.0+0.8.0-dynamiclink2

Closes #347
Types of changes

[x] Bug fix (changes which fixes an issue)
[ ] New feature (changes which adds functionality) (no needed)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected) (no needed)
[x] ETC (build, ci, docs, perf, refactor, style, test)

Checklist

[x] I followed the contributing guidelines.
[x] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes. (no needed)
[x] The PR title and commits are followed conventional commit form.

Looks like a little add-hock, but it is OK in this branch now.


ConcreteType should be a class

SuperType also may be an interface.

Released with version 8.1.0-beta.2


Test fixture renamed on stac-spec repo

One of the tests point to urls with files that no longer exist,
https://raw.githubusercontent.com/radiantearth/stac-spec/master/catalog-spec/examples/catalog.json is missing.
Looks like it should be
https://github.com/radiantearth/stac-spec/blob/master/catalog-spec/examples/basic.json
I think it would be better to just include the json in this project and mock the request.

Spoke with @jbants - the tags are required in the test fixtures to ensure correct versions are being tested, just like the local test fixtures, so will update these


Different port chosen through foreman vs yarn dev from terminal

Problem:
Foreman is not running the port I expected, i.e. the same port as when I run the same command outside of foreman
Steps to reproduce
From terminal
cd draw-front
yarn dev

Result
Your application is running here: http://localhost:8080
Through foreman
Procfile:
frontend: cd draw-front && yarn dev
foreman start

Expected outcome:
Same as when running yarn dev from terminal
Actual outcome:
Your application is running here: http://localhost:5101

I think that you can specify using -p what port you want to run.
foreman start -p 8080 for example.


Co-existing HDF5 (serial and parallel) + PKG_CHECK_MODULES issue

Upon compilation with the following modules loaded:
Currently Loaded Modulefiles:
  1) rocks-openmpi                     4) gcc/9.2.0                         7) apps/automake-1.16.1-gcc-7.3.0
  2) julia/1.6.3-gcc-9.2.0-ivybridge   5) apps/autogen-5.18.12-gcc-8.2.0    8) apps/libtool-2.4.6-gcc-7.3.0
  3) intel/2019.3

The following issues are encountered:

./configure: line 17380: syntax error near unexpected token `PKG_CFLAGS=""'
./configure: line 17380: `PKG_CFLAGS=""'

checking for matching HDF5 Fortran wrapper... /usr/bin/h5fc
./configure: line 17855: syntax error near unexpected token `HDF5,'
./configure: line 17855: `PKG_CHECK_MODULES(HDF5, hdf5 >= 1.8,'

Thanks,
Vijay

Thanks @v1j4y !
This bug was observed on the local curie cluster at LCPQ. Both issues have the same origin, which I will address as No.2 in the list below. However, on curie there are two unrelated bugs:

Issue with co-existing HDF5 installations (serial and parallel):

Bug: Several modules (e.g. HDF5) are installed using Spack package manager on curie. These modules can be loaded using module load hdf5/<hdf5-version> and some of them are correspond to parallel version of HDF5. However, there is also a default serial HDF5 installation (version 1.8.12) present in /usr/bin and /usr/lib. Thus, AX_LIB_HDF5() macro proceeds to search in the path first for serial installation (using h5cc compiler) and only if it is not found it looks for the parallel one (using h5pcc compiler). So even when parallel HDF5 module is loaded, it is not discovered due to existing installation in /usr/bin which is always seen first, no matter the order of PATH directories.
Solution: replace call to AX_LIB_HDF5() in configure.ac with AX_LIB_HDF5([parallel]) so that the search proceeds directly to use hpcc, which correspond to the loaded module and not to the default HDF5 installation. Since in this case the parallel version is used, HDF5 headers will try to #include "mpi.h". If you get an error pointing to mpi.h : no such file or directory, then you need to load the MPI too. On curie, do something like module load openmpi/3.1.2-gcc-7.3.0.

Issue with PKG_CHECK_MODULES macro from pkg.m4:

Bug: ./configure execution results in syntax error near unexpected token PKG_CFLAGS=""'  or syntax error near unexpected token HDF5,' related to PKG_CHECK_MODULES macro from configure.ac.
Solution: execute aclocal -I m4 in TREXIO root directory before running autogen.sh or autoreconf -i. More details and alternative solutions can be found in the answers to this question: https://stackoverflow.com/questions/36659867/pkg-check-modules-syntax-error-near-luajit .

Something unrelated, but also observed on curie:

Bug: in the output of ./configure execution there is checking for GNU libc compatible malloc... no line. After that make execution produces error undefined reference to 'rpl_malloc'. This issue comes from AC_FUNC_MALLOC built-in macro, which defines rpl_malloc if for some reason GLIBC-compatible malloc is not found/compiled. This is a cross-compilation bug and one possible workaround here is to patch configure execution like this ac_cv_func_malloc_0_nonnull=yes ./configure <configure_flags_if_any>.


DataTable: No way to set selectedRows to state

Hello, selectedRows in DataTable returns the selected rows from the checkboxes, however I don't believe there is any way (that I know of) to set this to the state of the React component (the external state managed outside of the DataTable).
This would be a very useful thing if we wanted to do something with those checkboxes outside of the DataTable components, for example, submit the results of those checkboxes to a REST API (which is what I'm trying to do).
I worked for several days to customize my DataTable and override some of the default behavior to get this to work, but seems like there should be native support for it.
Thanks

Hi,
One thing that I'm doing to get the selected rows from outside the DataTable is by using refs:
<DataTable
 ref={(e) => {this.dataTable = e}}

Then, in some click handler:
onSubmit = () => {
    const { rows  } = this.props;
    const selectedRowsIds = this.dataTable.getSelectedRows();
    // get the selected rows using the selectedRowsIds data.
}

About updating the state, I didn't found an event that is fired when a row was selected, thinking about updating the state in the render after receiving the selectedRows property seems crazy.

Hi :wave: @smchamberlin thank you for reporting and thanks @devniel for jumping in! In general changing component's state in this library tends to be done via props, as many of our components have prop -> state sync mechanism. AFAIK data table has some. That said, I'm wondering if you can try setting the new selected state via props. Thanks!

Thanks Akira,
I also see that this seems to be the same issue as:
https://github.com/IBM/carbon-components-react/issues/1285
which is still open and also has comments from you -- was there any resolution there ?
Also, if you look at the code for react-data-table-component (not the carbon version), they describe a simple way to do it with a onTableUpdate function which passes the state.selectedRows to the outside:
https://www.npmjs.com/package/react-data-table-component
Does the carbon version of react DataTable have something like this ?
I don't know what you mean by "set the new selected state via props".  Can you please give an example of that ?
Thanks.

@smchamberlin I thought the prop -> state sync mechanism covers the selection state, but seems that I was wrong because of #1247 - My apologies. CC @joshblack to see if he wants to give guidelines to contributors to move it forward.

@asudoh Is it currently possible to do something like this ? (Pasted from https://www.npmjs.com/package/react-data-table-component)
If not, one workaround might be to use the react-data-table-component and stylize it with CSS to look like carbon data component?
const handleChange = (state) => {
  // You can use setState or dispatch with something like Redux so we can use the retrieved data
  console.log('Selected Rows: ', state.selectedRows);
};

class MyComponent extends Component {
  render() {
      return (
        <DataTable
          title="Arnold Movies"
          columns={columns}
          data={data}
          selectableRows // add for checkbox selection
          onTableUpdate={handleChange}
        />
    )
  }
);

Seems not - My personal recommendation is getting a green light from legal to maintain a fork of this library and make changes you need. Contribution will be great and please get in sync with @joshblack to do so. I don't have a word on 3rd party library usage.

Hi @smchamberlin! 👋 Definitely agreed on all the points that you made in this issue 😄
For Data Table, a big part of the API over time will be formed by the newer features in React. Things like hooks and context updates will allow consumers to both read and write to state in a way that we can't express in the current API.
As a result, this has left us in a weird position since we want to deliver on features today while still preparing for the future. Definitely understand where your coming from, would love to hear what we could offer in the interim to help make things work out while we wait for some of these exciting changes 😄

@joshblack Sent a private email to your IBM account - I am unblocked for now as I made some modifications to track the checkpoint state myself, but I did this on top of the DataTable instead of modifying DataTable code directly, so I don't think I can contribute this back.

Just to leave a note before being closed - The requirement for initial selection (tracked in #1247) has been addressed. For reflecting after-creation-time selection changes, the team's current consensus is that application should use our presentational components to project application's state. Ones for data table are found as <Table*> at: https://github.com/carbon-design-system/carbon-components-react/blob/v7.1.0/src/components/DataTable/stories/default.js#L50-L107


Class hash calculation in devnet differs from starkli class-hash

CLI startup info is Predeployed accounts using class with hash: 0x4d07e40e93398ed3c76981e72dd1fd22557a78ce36c0515f679e27f0bb5bc5f which is old cario 0 contract not new cairo 1.

I can see 2 problems here, first, one is related to account_class_choice that was not set to cairo1, and the second is general class hash calculations.

As discussed on call today the problem is a slightly different artifact, not a class hash calculations algorithm.


development.yml ?

what's the purpose  of development.yml in the cloud ? it seems to deploys to "hosting" and is not triggered...

I wanted to introduce a dev and prod enviroment..
but I think we don't need it for now


Execute me. ClickHouseStatement batch insert bitmap data to DB will support in version 0.3.2??

Read the test code of version 0.3.2-test， it also mark like this:
// FIXME too bad batching is not supported
It has any other way to batch insert bitmap datas to db?
Thanks..

Yes, it will be supported in 0.3.2. Please refer to examples at here.

Tansks..


Propel findOne() always gets flagged as an empty assignment

This is likely to come down to a void return path mixed in with other non-void returns, ie the assignment could be empty.
It should instead treat the mixed void/non-void as an error and cast it to null.

In fact this is because of a void return from a property lookup, which in itself should never happen.

Fixed


Add files via upload

@MrSam7K First stage of verification passed, official domain is rackauskas.lt as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@MrSam7K Thank you for your patience while waiting for a review and processing your request. Pull request merged.


[CHANGE ME] Re-generated  to pick up changes in the API or client library generator.

This PR was generated using Autosynth. :rainbow:
Here's the log from Synthtool:
synthtool > Executing /tmpfs/src/git/autosynth/working_repo/synth.py.
synthtool > Ensuring dependencies.
synthtool > Pulling artman image.
latest: Pulling from googleapis/artman
Digest: sha256:66ca01f27ef7dc50fbfb7743b67028115a6a8acf43b2d82f9fc826de008adac4
Status: Image is up to date for googleapis/artman:latest
synthtool > Cloning googleapis.
synthtool > Running generator for google/spanner/artman_spanner.yaml.
synthtool > Generated code into /home/kbuilder/.cache/synthtool/googleapis/artman-genfiles/js/spanner-v1.
synthtool > Running generator for google/spanner/admin/database/artman_spanner_admin_database.yaml.
synthtool > Generated code into /home/kbuilder/.cache/synthtool/googleapis/artman-genfiles/js/spanner-admin-database-v1.
synthtool > Running generator for google/spanner/admin/instance/artman_spanner_admin_instance.yaml.
synthtool > Generated code into /home/kbuilder/.cache/synthtool/googleapis/artman-genfiles/js/spanner-admin-instance-v1.
.eslintignore
.eslintrc.yml
.github/ISSUE_TEMPLATE/bug_report.md
.github/ISSUE_TEMPLATE/feature_request.md
.github/ISSUE_TEMPLATE/support_request.md
.github/release-please.yml
.jsdoc.js
.kokoro/common.cfg
.kokoro/continuous/node10/common.cfg
.kokoro/continuous/node10/docs.cfg
.kokoro/continuous/node10/lint.cfg
.kokoro/continuous/node10/samples-test.cfg
.kokoro/continuous/node10/system-test.cfg
.kokoro/continuous/node10/test.cfg
.kokoro/continuous/node12/common.cfg
.kokoro/continuous/node12/test.cfg
.kokoro/continuous/node8/common.cfg
.kokoro/continuous/node8/test.cfg
.kokoro/docs.sh
.kokoro/lint.sh
.kokoro/presubmit/node10/common.cfg
.kokoro/presubmit/node10/docs.cfg
.kokoro/presubmit/node10/lint.cfg
.kokoro/presubmit/node10/samples-test.cfg
.kokoro/presubmit/node10/system-test.cfg
.kokoro/presubmit/node10/test.cfg
.kokoro/presubmit/node12/common.cfg
.kokoro/presubmit/node12/test.cfg
.kokoro/presubmit/node8/common.cfg
.kokoro/presubmit/node8/test.cfg
.kokoro/presubmit/windows/common.cfg
.kokoro/presubmit/windows/test.cfg
.kokoro/publish.sh
.kokoro/release/docs.cfg
.kokoro/release/docs.sh
.kokoro/release/publish.cfg
.kokoro/samples-test.sh
.kokoro/system-test.sh
.kokoro/test.bat
.kokoro/test.sh
.kokoro/trampoline.sh
.nycrc
.prettierignore
.prettierrc
CODE_OF_CONDUCT.md
CONTRIBUTING.md
LICENSE
README.md
codecov.yaml
renovate.json
samples/README.md
synthtool > Replaced "(const SpannerClient = require\\('\\./spanner_client'\\);)" in src/v1/index.js.
synthtool > Replaced '(module\\.exports\\.SpannerClient = SpannerClient;)' in src/v1/index.js.
synthtool > Replaced '../../package.json' in src/v1/database_admin_client.js.
synthtool > Replaced '../../package.json' in src/v1/instance_admin_client.js.
synthtool > Replaced '../../package.json' in src/v1/spanner_client.js.
synthtool > Replaced 'https:\\/\\/cloud\\.google\\.com[\\s\\*]*http:\\/\\/(.*)[\\s\\*]*\\)' in src/v1/doc/google/protobuf/doc_timestamp.js.
synthtool > No replacements made in **/doc/google/protobuf/doc_timestamp.js for pattern toISOString\], maybe replacement is not longer needed?
synthtool > Replaced '`\\[a-z\\]\\(https:\\/\\/cloud\\.google\\.com\\[-a-z0-9\\]\\*\\[a-z0-9\\]\\)\\?`' in src/v1/doc/google/spanner/v1/doc_spanner.js.
synthtool > Replaced '`\\[a-z\\]\\(https:\\/\\/cloud\\.google\\.com\\[-a-z0-9\\]\\*\\[a-z0-9\\]\\)\\?`' in src/v1/doc/google/spanner/admin/instance/v1/doc_spanner_instance_admin.js.
synthtool > Replaced '`\\(\\[a-z\\]\\(https:\\/\\/cloud\\.google\\.com\\[-a-z0-9\\]\\*\\[a-z0-9\\]\\)\\?\\)\\?`' in src/v1/doc/google/spanner/v1/doc_spanner.js.
synthtool > Replaced '`\\(\\[a-z\\]\\(https:\\/\\/cloud\\.google\\.com\\[-a-z0-9\\]\\*\\[a-z0-9\\]\\)\\?\\)\\?`' in src/v1/doc/google/spanner/admin/instance/v1/doc_spanner_instance_admin.js.
npm WARN deprecated<EMAIL_ADDRESS>This is a stub types definition. p-retry provides its own type definitions, so you do not need this installed.
npm WARN deprecated<EMAIL_ADDRESS>Please note that v5.0.1+ of superagent removes User-Agent header by default, therefore you may need to add it yourself (e.g. GitHub blocks requests without a User-Agent header).  This notice will go away with v5.0.2+ once it is released.

><EMAIL_ADDRESS>install /tmpfs/src/git/autosynth/working_repo/node_modules/grpc
> node-pre-gyp install --fallback-to-build --library=static_library

node-pre-gyp WARN Using needle for node-pre-gyp https download
[grpc] Success: "/tmpfs/src/git/autosynth/working_repo/node_modules/grpc/src/node/extension_binary/node-v57-linux-x64-glibc/grpc_node.node" is installed via remote

><EMAIL_ADDRESS>postinstall /tmpfs/src/git/autosynth/working_repo/node_modules/core-js
> node scripts/postinstall || echo "ignore"

Thank you for using core-js ( https://github.com/zloirock/core-js ) for polyfilling JavaScript standard library!

The project needs your help! Please consider supporting of core-js on Open Collective or Patreon:
> https://opencollective.com/core-js
> https://www.patreon.com/zloirock

Also, the author of core-js ( https://github.com/zloirock ) is looking for a good job -)

><EMAIL_ADDRESS>postinstall /tmpfs/src/git/autosynth/working_repo/node_modules/protobufjs
> node scripts/postinstall

><EMAIL_ADDRESS>prepare /tmpfs/src/git/autosynth/working_repo
> npm run compile

><EMAIL_ADDRESS>compile /tmpfs/src/git/autosynth/working_repo
> tsc -p . && cp -r src/v1 build/src && cp -r protos build && cp -r proto build && cp test/*.js build/test

npm notice created a lockfile as package-lock.json. You should commit this file.
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules/fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"linux","arch":"x64"})

added 1151 packages from 1402 contributors and audited 6703 packages in 36.735s
found 0 vulnerabilities

><EMAIL_ADDRESS>fix /tmpfs/src/git/autosynth/working_repo
> gts fix && eslint --fix '**/*.js'

installing<EMAIL_ADDRESS>installing<EMAIL_ADDRESS>synthtool > Cleaned up 2 temporary directories.
synthtool > Wrote metadata to synth.metadata.

Codecov Report

Merging #694 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master    #694   +/-   ##
======================================
  Coverage    99.1%   99.1%
======================================
  Files          12      12
  Lines        1348    1348
  Branches      215     215
======================================
  Hits         1336    1336
  Misses          4       4
  Partials        8       8

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 0ea0161...e19d5d4. Read the comment docs.


fix #1500

add exception fields processing

Codecov Report

Merging #1506 into master will decrease coverage by <.01%.
The diff coverage is 83.33%.

@@             Coverage Diff              @@
##             master    #1506      +/-   ##
============================================
- Coverage     82.79%   82.78%   -0.01%
+ Complexity     6026     6025       -1
============================================
  Files           136      136
  Lines         22043    22049       +6
  Branches       5034     5034
============================================
+ Hits          18250    18253       +3
  Misses         2552     2552
- Partials       1241     1244       +3

Impacted Files
Coverage Δ
Complexity Δ

...son/parser/deserializer/ThrowableDeserializer.java
92% <83.33%> (-0.56%)
32 <0> (ø)

...com/alibaba/fastjson/parser/JSONReaderScanner.java
72.61% <0%> (-1.28%)
44% <0%> (-1%)

...m/alibaba/fastjson/serializer/SerializeWriter.java
89.05% <0%> (-0.09%)
371% <0%> (-1%)

...ava/com/alibaba/fastjson/parser/JSONLexerBase.java
71.52% <0%> (+0.03%)
785% <0%> (+1%)
: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 d35a1a1...4fc5587. Read the comment docs.

ThrowableDeserializer是每个class一个的，可以在实例级别对Field做缓存么？从反射中获取field会很慢的

可以考虑，目前这么处理是因为：一般的逻辑是走不到新加的处理里面来的。