Remove duplicate test data, make relperm test more forgiving

The test suite was passing on CI but failing on my local machine. This fixes.

Codecov Report

Merging #170 (2a45be4) into master (ca609fc) will decrease coverage by 0.02%.
The diff coverage is 84.37%.

:exclamation: Current head 2a45be4 differs from pull request most recent head f68a77b. Consider uploading reports for the commit f68a77b to get more accurate results

@@            Coverage Diff             @@
##           master     #170      +/-   ##
==========================================
- Coverage   43.69%   43.67%   -0.03%
==========================================
  Files          51       51
  Lines       23632    23627       -5
==========================================
- Hits        10326    10319       -7
- Misses      13306    13308       +2

Impacted Files
Coverage Δ

resqpy/lines.py
57.38% <50.00%> (ø)

resqpy/well.py
47.85% <78.94%> (-0.03%)
:arrow_down:

resqpy/model.py
58.22% <100.00%> (-0.06%)
:arrow_down:

resqpy/organize.py
57.79% <100.00%> (-0.10%)
:arrow_down:

resqpy/property.py
41.49% <100.00%> (ø)

resqpy/time_series.py
53.92% <100.00%> (ø)

resqpy/crs.py
67.78% <0.00%> (-0.84%)
:arrow_down:

Continue to review full report at Codecov.

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


compile error

/home/ros2/Micro-XRCE-DDS-Agent/include/micrortps/agent/datawriter/DataWriter.hpp:62:21: error: cannot declare field ‘eprosima::micrortps::DataWriter::topic_type_’ to be of abstract type ‘eprosima::micrortps::TopicPubSubType’
TopicPubSubType topic_type_;

Hi @baozeng90
Can you provide us with more information on your compilation environment?:

Which version of Micro XRCE-DDS Agent are you trying to compile?
Against which version of Fast RTPS are you compiling?
Which compilation command are you launching?

I am having the same issue. I believe this is due to using FastRTPS 1.7.0 with Micro-XRCE-DDS on master. The "Agent" repo link needs to be updated to master for micro xrce to compile with FastRTPS 1.7.0.

I am having the same issue. I believe this is due to using FastRTPS 1.7.0 with Micro-XRCE-DDS on master. The "Agent" repo link needs to be updated to master for micro xrce to compile with FastRTPS 1.7.0.

yes,I'm using FastRTPS 1.7.0 with Micro-XRCE-DDS,if use FastRTPS in Micro-XRCE-DDS/agent/thirdparty/fastrtps,it si OK!So I think The  versions should be the same(ROS2 and Micro-XRCE-DDS's FastRTPS).

Yes, this is an issue due to the use of Fast RTPS 1.7.0.
I have merged a PR into Micro XRCE-DDS Agent fixing these incompatibilities with Fast RTPS 1.7.0.
PR: https://github.com/eProsima/Micro-XRCE-DDS-Agent/pull/42
If you update your Micro XRCE-DDS Agent master branch, it should not cause these compilation errors.

Thankyou


Sensor component for Fronius inverters, storages, and smart meters

Description:
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#4339
Example entry for configuration.yaml (if applicable):
- platform: fronius
  host: <IP_ADDRESS>
  device: 1
  type: inverter

- platform: fronius
  host: <IP_ADDRESS>
  type: power_flow

- platform: template
  sensors:
    electricity_inverter1_power_netto:
      unit_of_measurement: 'W'
      value_template: >-
        {% if states.sensor.fronius_inverter_1723110610_1.attributes.power_ac is defined %}
          {{ states.sensor.fronius_inverter_1723110610_1.attributes.power_ac | float | round(2) }}
        {% else %}
          0
        {% endif %}
    electricity_autonomy:
      unit_of_measurement: '%'
      value_template: >-
        {% if states.sensor.fronius_power_flow_1723110611.attributes.relative_autonomy is defined %}
          {{ states.sensor.fronius_power_flow_1723110611.attributes.relative_autonomy | float | round(2) }}
        {% else %}
          0
        {% endif %}

Checklist:
If user exposed functionality or configuration variables are added/changed:

[X] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[WiP] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[X] New dependencies have been added to the REQUIREMENTS variable (example).
[X] New dependencies are only imported inside functions that use them (example).
[X] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[X] New files were added to .coveragerc.

If the code does not interact with devices:

[-] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[-] Tests have been added to verify that the new code works.

@fabaff @balloob thanks a lot for your comments.
I've updated the sensor component according to your notes - and I hope I've done it properly regarding to the standards.

Why would you need to access the interval?The update method is called automatically by the Entity object in scan_interval intervals.                                                                                There should not be any need of triggering this yourself.                                                                                From<EMAIL_ADDRESS>February 19, 2018 08:03To<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Re: [home-assistant/home-assistant] Sensor component for Fronius inverters, storages, and smart meters (#11446)  @gbeine commented on this pull request.
In homeassistant/components/sensor/fronius.py:

+        device = DEFAULT_DEVICE

+    sensor = FroniusSensor(
+        fronius, name, config[CONF_TYPE], config[CONF_SCOPE], device)
+
+    async_add_devices([sensor])
+
+    @asyncio.coroutine
+    def async_fronius(event):
+        """Update all the fronius sensors."""
+        try:
+            yield from sensor.async_update()
+        except Exception:
+            _LOGGER.error("Update of sensor data failed.")
+
+    interval = config.get(CONF_SCAN_INTERVAL) or timedelta(seconds=10)
Ok, is this described somewhere?
How can I access the interval?
—You are receiving this because you commented.Reply to this email directly, view it on GitHub, or mute the thread.

Why do you recommend such a strange configuration?
- platform: template
  sensors:
    electricity_inverter1_power_netto:
      unit_of_measurement: 'W'
      value_template: >-
        {% if states.sensor.fronius_inverter_1723110610_1.attributes.power_ac is defined %}
          {{ states.sensor.fronius_inverter_1723110610_1.attributes.power_ac | float | round(2) }}
        {% else %}
          0
        {% endif %}
    electricity_autonomy:
      unit_of_measurement: '%'
      value_template: >-
        {% if states.sensor.fronius_power_flow_1723110611.attributes.relative_autonomy is defined %}
          {{ states.sensor.fronius_power_flow_1723110611.attributes.relative_autonomy | float | round(2) }}
        {% else %}
          0
        {% endif %}

If the pyfronius module does not return 0 instead of undefined on it's please open an issue over at https://github.com/nielstron/pyfronius

@nielstron I recommended this because I wrote this part of pyfronius.
It is a difference if a device returns undefined or 0.

This PR is gone stale. Please re-open it if you want to go on.


Update analysis options

Fixes #7166
Also cleans up from an aborted attempt at #5793

@danrubel

This change was only applied to one of the three .analysis_options files. @tvolkert, can you update the others to make sure they're all in sync (or that if they're different, it's intentional)?

This change was only applied to one of the three .analysis_options files

All the other .analysis_options files (e.g. this one) are just skeletons...

I meant the three that are mentioned at the top of the root one:
https://github.com/flutter/flutter/blob/master/.analysis_options


Creating container with new artifacts method fails with error: cannot find path

Creating a Docker Container with artifacts (BC15 or BC 16) fails with error
Cannot find path 'C:\NAVDVD\ServiceTier\program files\Microsoft Dynamics NAV' because it does not exist.

Action
Steps to reproduce:
New-BCContainer -accept_eula -containerName 'artifacttest' -artifactUrl (Get-BCArtifactUrl -type OnPrem -select Latest -country ch -version '15') -auth UserPassword -updateHosts

Result
NavContainerHelper is version <IP_ADDRESS>
NavContainerHelper is running as administrator
Host is Microsoft Windows 10 Enterprise - 1909
Docker Client Version is 19.03.8
Docker Server Version is 19.03.8
Removing C:\ProgramData\NavContainerHelper\Extensions\artifacttest
Using image mcr.microsoft.com/dynamicsnav:10.0.18363.778-generic
Creating Container artifacttest
Version: 15.7.43293.0-ch
Style: onprem
Generic Tag: <IP_ADDRESS>
Container OS Version: 10.0.18363.778 (1909)
Host OS Version: 10.0.18363.778 (1909)
Using locale de-CH
Using process isolation
Disabling the standard eventlog dump to container log every 2 seconds (use -dumpEventLog to enable)
Files in C:\ProgramData\NavContainerHelper\Extensions\artifacttest\my:
- AdditionalOutput.ps1
- MainLoop.ps1
- SetupVariables.ps1
- updatehosts.ps1
Creating container artifacttest from image mcr.microsoft.com/dynamicsnav:10.0.18363.778-generic
2d925a201f3dba83c1e16e309c593f9bee208cf0962e435fccbfc61d9b3054e5
Waiting for container artifacttest to be ready
Using artifactUrl https://bcartifacts.azureedge.net/onprem/15.7.43293.0/ch
Copying Platform Artifacts
Copying Application Artifacts
Copying Database
Copy Licensefile
Copying Installers
Copying Applications
Cannot find path 'C:\NAVDVD\ServiceTier\program files\Microsoft Dynamics NAV' because it does not exist.
at <ScriptBlock>, C:\Run\start.ps1: line 192
at <ScriptBlock>, <No file>: line 1Error
Using artifactUrl https://bcartifacts.azureedge.net/onprem/15.7.43293.0/ch
Copying Platform Artifacts
Copying Application Artifacts
Copying Database
Copy Licensefile
Copying Installers
Copying Applications
Cannot find path 'C:\NAVDVD\ServiceTier\program files\Microsoft Dynamics NAV' because it does not exist.
at <ScriptBlock>, C:\Run\start.ps1: line 192
at <ScriptBlock>, <No file>: line 1
Initialization of container artifacttest failed
In C:\Program Files\WindowsPowerShell\Modules\navcontainerhelper\<IP_ADDRESS>\ContainerHandling\Wait-NavContainerReady.ps1:44 Zeichen:17
+ ...             throw "Initialization of container $containerName failed" ...
+                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : OperationStopped: (Initialization ...facttest failed:String) [], RuntimeException
    + FullyQualifiedErrorId : Initialization of container artifacttest failed

Happens with all BC versions tested, 15 latest and 16 latest.
With BC 14 it does not fail, it succeed.
NAVContainerHelper Module version <IP_ADDRESS>.

Strange - copy/pasted your line and it works here.
Could you try to add -isolation hyperv (just for the test)

With parameter -isolation hyperv it fails too.
Complete command
New-BCContainer -accept_eula -containerName 'artifacttest16' -artifactUrl (Get-BCArtifactUrl -type OnPrem -select Latest -country ch -version '16') -auth UserPassword -updateHosts -isolation hyperv

Could you try to use Flush-ContainerHelperCache -cache bcartifacts and try again?

With Flush-ContainerHelperCache -cache bcartifacts before New-BCContainerit takes longer because artifacts are downloaded again (of course) but it does NOT break with the error mentioned above or any other error:
So the workaround to this issue is to use first
Flush-ContainerHelperCache -cache bcartifacts

-isolation hyperv is not required to solve this issue.
After that I was able to create multiple containers, e.g. a BC 16 container and a BC 15 container.

My assumption is that something went wrong during download of artifacts - and on the next run, it thinks that the artifacts are in the cache - and thus doesn't download - but instead tries to run with faulty artifacts.
Will have a look at the Download-Artifacts function to make it more resilient.


Allow Scripts to be based on the OS / Platform

Making this possible would allow to have a ci-runner on windows in the future, you just tell the CI project that you want to run something on a linux based runner, so only a runner that it on windows will execute that.
If the project has bother linux and windows runners then it'll use both.
This probably requires some rewriting of the CI internals as it would be necessary to have status badges for the different OS / Platform.
I'm also trying to make the runner work on windows currently, I'll make a pull request about that soon, but it would help if this would've been done before that...

Thank you for your interest in improving GitLab. We don't use the issue tracker for feature requests. Things that are wrong but are not a regression compared to older versions of GitLab are considered feature requests and not issues. Please use the feature request forum for this purpose or create a merge request implementing this feature. Have a look at the contribution guidelines for more information.


AttributeError: 'SplattingModel' object has no attribute 'gauss_params'

Getting the following error when trying out the ns-splatting command.
Traceback (most recent call last):
File "C:\Users\s_m_d\miniforge3\envs\nerfstudio\lib\runpy.py", line 196, in _run_module_as_main
return _run_code(code, main_globals, None,
File "C:\Users\s_m_d\miniforge3\envs\nerfstudio\lib\runpy.py", line 86, in run_code
exec(code, run_globals)
File "C:\Users\s_m_d\miniforge3\envs\nerfstudio\Scripts\ns-train.exe_main.py", line 7, in
File "C:\nerfstudio\nerfstudio\scripts\train.py", line 262, in entrypoint
main(
File "C:\nerfstudio\nerfstudio\scripts\train.py", line 247, in main
launch(
File "C:\nerfstudio\nerfstudio\scripts\train.py", line 189, in launch
main_func(local_rank=0, world_size=world_size, config=config)
File "C:\nerfstudio\nerfstudio\scripts\train.py", line 99, in train_loop
trainer.setup()
File "C:\nerfstudio\nerfstudio\engine\trainer.py", line 160, in setup
self.optimizers = self.setup_optimizers()
File "C:\nerfstudio\nerfstudio\engine\trainer.py", line 225, in setup_optimizers
param_groups = self.pipeline.get_param_groups()
File "C:\nerfstudio\nerfstudio\pipelines\base_pipeline.py", line 441, in get_param_groups
model_params = self.model.get_param_groups()
File "C:\nerfstudio\nerfstudio\models\splatfacto.py", line 633, in get_param_groups
gps = self.get_gaussian_param_groups()
File "C:\nerfstudio\nerfstudio\models\splatfacto.py", line 622, in get_gaussian_param_groups
return {
File "C:\nerfstudio\nerfstudio\models\splatfacto.py", line 623, in
name: [self.gauss_params[name]]
File "C:\Users\s_m_d\miniforge3\envs\nerfstudio\lib\site-packages\torch\nn\modules\module.py", line 1695, in getattr
raise AttributeError(f"'{type(self).name}' object has no attribute '{name}'")
AttributeError: 'SplattingModel' object has no attribute 'gauss_params'

Hi, thank you for reporting the error!
Could you tell me in more details about the command you are running? I update the code yesterday so that "splatting" method is called "nerfgs" now (to align with the naming in the paper).
Could you also mention your Python, nerfstudio, gsplat, and Torch version?

Hi! Got the same error with the new version, having re-installed everything. Had to install some additional dependencies that I've seen you've now added.
This is the command I'm running, and also get a warning straight after
(nerfstudio) B:>ns-train nerfgs --data Nerfs\NerfToGs\Wissahickon --max-num-iterations 1 --pipeline.model.ply-file-path Nerfs\nerfgs\nerfgs.ply nerfstudio-data --eval-mode filename
C:\Users\s_m_d\miniforge3\envs\nerfstudio\lib\site-packages\tyro_fields.py:343: UserWarning: The field ply_file_path is annotated with type <class 'str'>, but the default value None has type <class 'NoneType'>. We'll try to handle this gracefully, but it may cause unexpected behavior.
warnings.warn(
Am running python 3.10 through miniforge, Nerfstudio version 1.10 in a dedicated environment, latest dev version, Torch 2.1.2. Not sure about gsplat, that the same as splatfacto?

Getting the following error when trying out the ns-splatting command.
Traceback (most recent call last): File "C:\Users\s_m_d\miniforge3\envs\nerfstudio\lib\runpy.py", line 196, in _run_module_as_main return run_code(code, main_globals, None, File "C:\Users\s_m_d\miniforge3\envs\nerfstudio\lib\runpy.py", line 86, in run_code exec(code, run_globals) File "C:\Users\s_m_d\miniforge3\envs\nerfstudio\Scripts\ns-train.exe__main.py", line 7, in File "C:\nerfstudio\nerfstudio\scripts\train.py", line 262, in entrypoint main( File "C:\nerfstudio\nerfstudio\scripts\train.py", line 247, in main launch( File "C:\nerfstudio\nerfstudio\scripts\train.py", line 189, in launch main_func(local_rank=0, world_size=world_size, config=config) File "C:\nerfstudio\nerfstudio\scripts\train.py", line 99, in train_loop trainer.setup() File "C:\nerfstudio\nerfstudio\engine\trainer.py", line 160, in setup self.optimizers = self.setup_optimizers() File "C:\nerfstudio\nerfstudio\engine\trainer.py", line 225, in setup_optimizers param_groups = self.pipeline.get_param_groups() File "C:\nerfstudio\nerfstudio\pipelines\base_pipeline.py", line 441, in get_param_groups model_params = self.model.get_param_groups() File "C:\nerfstudio\nerfstudio\models\splatfacto.py", line 633, in get_param_groups gps = self.get_gaussian_param_groups() File "C:\nerfstudio\nerfstudio\models\splatfacto.py", line 622, in get_gaussian_param_groups return { File "C:\nerfstudio\nerfstudio\models\splatfacto.py", line 623, in name: [self.gauss_params[name]] File "C:\Users\s_m_d\miniforge3\envs\nerfstudio\lib\site-packages\torch\nn\modules\module.py", line 1695, in getattr raise AttributeError(f"'{type(self).name}' object has no attribute '{name}'") AttributeError: 'SplattingModel' object has no attribute 'gauss_params'

I just fixed this error. Could you try again by running
ns-train nerfgs --data DATA-PATH --max-num-iterations 1 --pipeline.model.ply-file-path exports/nerfgs/nerfgs.ply nerfstudio-data --eval-mode filename
I was using a very old version of nerfstudio. So I have to update the nerfgsModel for the newer nerfstudio versions.

The training works now, but the result from running nerfgs in the viser, as well as the output ply looks odd. Attaching file of it.

Also getting this warning:
C:\Users\s_m_d\miniforge3\envs\nerfstudio\lib\site-packages\torch\optim\lr_scheduler.py:136: UserWarning: Detected call of lr_scheduler.step() before optimizer.step(). In PyTorch 1.1.0 and later, you should call them in the opposite order: optimizer.step() before lr_scheduler.step().  Failure to do this will result in PyTorch skipping the first value of the learning rate schedule. See more details at https://pytorch.org/docs/stable/optim.html#how-to-adjust-learning-rate
warnings.warn("Detected call of lr_scheduler.step() before optimizer.step(). "

The training works now, but the result from running nerfgs in the viser, as well as the output ply looks odd. Attaching file of it.

Also getting this warning: C:\Users\s_m_d\miniforge3\envs\nerfstudio\lib\site-packages\torch\optim\lr_scheduler.py:136: UserWarning: Detected call of lr_scheduler.step() before optimizer.step(). In PyTorch 1.1.0 and later, you should call them in the opposite order: optimizer.step() before lr_scheduler.step(). Failure to do this will result in PyTorch skipping the first value of the learning rate schedule. See more details at https://pytorch.org/docs/stable/optim.html#how-to-adjust-learning-rate warnings.warn("Detected call of lr_scheduler.step() before optimizer.step(). "
And still getting this warning, don't know if it's relevant: C:\Users\s_m_d\miniforge3\envs\nerfstudio\lib\site-packages\tyro_fields.py:343: UserWarning: The field ply_file_path is annotated with type <class 'str'>, but the default value None has type <class 'NoneType'>. We'll try to handle this gracefully, but it may cause unexpected behavior. warnings.warn(

Hi, does it work when you use older version of nerfstudio?

Yes, I managed to get it to work that way!


[PHPUnit] Change @return array<> to @return Iterator<> on YieldDataProviderRector

Closes https://github.com/rectorphp/rector-phpunit/pull/414

All checks have passed 🎉 @TomasVotruba I am merging it ;)


Create default mesos_exporter console template

As a community it is probably worthwhile to invest into a common set of consoles for Mesos. Otherwise everyone has to re-invent the basics.
Ideas:

Plots showing aggregate CPU and memory usage of the different slaves. This will help with diagnosing uneven load distribution
Plots per slave detailing total memory or CPU usage: available vs requested by Mesos tasks vs actually used

Does anyone still have something worth sharing?

:+1: for sharing common sets of consoles for various exporters and jobs (perhaps standardize on a consoles directory in the respective exporter or instrumented job repo?).
As for mesos_exporter consoles, I don't have any, but someone who uses Mesos might have some...

That would be awesome, I don't have any consoles though. I'm using Grafana as frontend and have added the graphs there.


Update readme

Update readme to provide all the steps necessary to follow when running the app for the first time in a single section with pictures

Thumbs.db is a system file. Let's remove it.
// as a related note - it would be better to have JPEGs because of smaller file size, but it's NBD


NitroPDFIncNitroPDFPtyLtd.PrimoPDF version <IP_ADDRESS>

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.0 schema?

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

Microsoft Reviewers: Open in CodeFlow

This isn't right, why did it fail unattended setup?  It worked just fine when I did it locally.


Refactor cobra / use cobra.Args arguments

Uses the cobra.Args arguments for setting maximum / range / minimum
arguments required for some commands rather than using a custom checker.

Problem with standard checks is that you can't display a custom message that will guide user what he did wrong, or what arguments he should provide :-(

@kadel Problem is that our current output display messages aren't much different than what cobra displays. Unless we improve the UX a bit. I'll probably open up another issue to improve some of the descriptions (specifically, ocdev create).

OK, you are right, I looked closer on the messages, and they aren't that much help either :-(


Removed lowerInclusive, upperInclusive params from Bucketizer

Using splits instead of lowerInclusive, upperInclusive.  Added tests.
@yinxusen Please verify that this update looks good before merging it with your PR.  Thanks!

lgtm, I'll merge it.


Metric selection not possible without data

Previously, the metrics in the performance aspect configuration were only made available for selection when a data point was selected in the graph. But if no data is available, you cannot select data points and you never get to the metric selection. So without data, you cannot select metrics.

Comment out this.metricFinderService.loadData(from, now, appId, pageId, browserId); in edit-aspect-metrics.component.ts to reproduce the configuration without event result data.


support for actions in notifications

Hi there, thank you for writing this. I think it might be just what I'm looking for.
Do you know if pbns is expected to support actions on notifications?
The pushbullet feature list calls this "Mirrored notification action support", available for pro accounts only. But I'm not sure how these appear in the python API you're using here, or how dbus provides for actions.
I thought I better check before I try and debug things. Thank you!

Hi @katef! I actually stopped using Pushbullet a couple of years ago (or more, maybe?) so unfortunately I stopped working on this project. I should've marked this project as archived. From a cursory search, though, it looks like actions are not supported by pushbullet.py so they should be added to pushbullet.py before they can be passed by pbns.
Sorry I can't help you more, but I sadly won't work on this project in the foreseeable future.

Hi @katef! I actually stopped using Pushbullet a couple of years ago (or more, maybe?) so unfortunately I stopped working on this project. I should've marked this project as archived. From a cursory search, though, it looks like actions are not supported by pushbullet.py so they should be added to pushbullet.py before they can be passed by pbns.
Sorry I can't help you more, but I sadly won't work on this project in the foreseeable future.


Error No package metadata was found for openapi_python_client when running app.exe built from pyinstaller

Describe the bug
Error importlib.metadata.PackageNotFoundError: No package metadata was found for openapi_python_client when running app.exe built from pyinstaller.
I have an application that uses the openapi_python_client library, and I've built the application into an exe file using PyInstaller to include the entire library. However, when running the resulting exe file after building, I encounter the following error:

I have tried adjusting the __version__ in the __init__.py file of the openapi_python_client library from __version__ = version(__package__) to __version__ = "1.0", and the error is resolved.

Is there a way to adjust this part so that the application doesn't encounter this error when built with PyInstaller?
Desktop (please complete the following information):

OS: [windows 10]
Python Version: [3.10.11]
openapi-python-client version [v0.16.0]

problem resolved


Enhance transportation view (based on #84)

This aims to add the map transitions between food produced and transportation states. This PR is based on the changes introduced in #84. I created a new branch because produced-transport-transition got outdated with the various PRs we merged today and it was hard to fix the merge conflicts.
Changes added:

Toggle destination areas
Renamed layer ports-points to top-ports, which should be rendered in the world view exclusively
Added destination-ports layer to be rendered in the transportation view
Fixed lint/type issues

Screen capture:

@oliverroick this is ready for a first review, please feel free to add changes.

@oliver forgot to say we currently don't have maritime flows in the database, I added nearby ports to populate the map meanwhile.

@oliverroick thanks for the review, I cleaned up the response and fixed the types.


package is exist.but is say can't find function

package path
/Users/eleven/Desktop/mygo/src/github.com/golang/freetype
font.go also have this struct

please tell me what probable will cause this problem？thanks

did you do go get -u ./... yet?

i'v tried 。I can run it on windows.but can't run on osx


Exclude Nerves.Runtime from xref

This will prevent toolshed from building nerves_runtime and instead just exclude it from xref checks.

Thanks.


Config Format Detection

Description of Issue/Question
In #132 we're trying to detect the format of the config. I'm curious how someone might go about using "set" syntax w/o a line that begins with set. E.g. deactivating or deleting a bit of config w/o hacking in a superfluous command that starts with set first.
I'd be happy to submit a PR that made _detect_config_format more robust. Also was curious what you all think about explicitly exposing a format argument (that defaults to text) to the relevant methods that's passed to pyEZ's load()? Maybe in addition to a _detect_config_format refactor?

Hi @zachmoody, thanks for your thoughts!

E.g. deactivating or deleting a bit of config w/o hacking in a superfluous command that starts with set first.
I'd be happy to submit a PR that made _detect_config_format more robust.

You are correct. I have also thought about that, although I didn't raise and issue or submitted a PR to enhance it. If you are happy to help, please go ahead.

Also was curious what you all think about explicitly exposing a format argument (that defaults to text) to the relevant methods that's passed to pyEZ's load()? Maybe in addition to a _detect_config_format refactor?

This is one of the subjects long debated internally. For the time being we have agree on the solution above, but if you have a specific use case for this we'd like to hear more.
Cheers,
-Mircea

Awesome, I'll get started on a PR then. Out of the available "verbs" I thought these might make sense to include for completeness:

activate
annotate
copy
deactivate
delete
insert
protect
rename
unprotect

Any others you all might want to add or drop from that list?

This is one of the subjects long debated internally. For the time being we have agree on the solution above, but if you have a specific use case for this we'd like to hear more.

Sure, so for everything related to loading a config we've encountered text format has worked well, save this one scenario. When we're looking to deactivate a stanza w/o building out the child elements; junos will throw a warning (at least in vSRX) which is fatal to pyEZ. E.g. if we have:
system {
    login {
        user bob {
            uid 2000;
            class admin;
        }
    }
}

and I want to deactivate bob w/o knowing his uid or class. The docs say that I can just do this:
system {
    login {
        inactive:
        user bob;
    }
}

Which works, but not without a warning (warning: statement has no contents; ignored). So it seems we're relegated to using set syntax for this this, which is fine.
The whole idea of  passing a format named arg was just coming from a place of being (maybe overly) explicit. I think a refactored _detect_config_format will work just as well, assuming no new actions come out.   :)
While we're on the topic, does the same stance keep get_config() from having a format arg as well?


Error starting whatsapp-web.js

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
/usr/src/app/node_modules/puppeteer-core/lib/cjs/puppeteer/common/Connection.js:329
                error: new Errors_js_1.ProtocolError(),
                       ^

ProtocolError: Waiting for selector `[data-icon='search']` failed: Protocol error (Runtime.callFunctionOn): Target closed.
    at /usr/src/app/node_modules/puppeteer-core/lib/cjs/puppeteer/common/Connection.js:329:24
    at new Promise (<anonymous>)
    at CDPSessionImpl.send (/usr/src/app/node_modules/puppeteer-core/lib/cjs/puppeteer/common/Connection.js:325:16)
    at ExecutionContext._ExecutionContext_evaluate (/usr/src/app/node_modules/puppeteer-core/lib/cjs/puppeteer/common/ExecutionContext.js:211:46)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async JSHandle.evaluateHandle (/usr/src/app/node_modules/puppeteer-core/lib/cjs/puppeteer/common/JSHandle.js:97:16)
    at async WaitTask.rerun (/usr/src/app/node_modules/puppeteer-core/lib/cjs/puppeteer/common/WaitTask.js:114:28) {
  originalMessage: ''
}

Node.js v20.12.1

After restarting whatsapp-web.js, the error pops out.
Expected behavior
The whatsapp-web.js able to successfully start and log in.
Steps to Reproduce the Bug or Issue

Restart server node using the whatsapp-web.js library.
Error reproduced.

Relevant Code
No response
Browser Type
Chromium
WhatsApp Account Type
Standard
Does your WhatsApp account have multidevice enabled?
Yes, I am using Multi Device
Environment
OS: Docker + Alpine
Phone OS: Android
WhatsApp Web version: 2.2407.5
Node.js Version: v20.12.1
Additional context
No response

@pfnh26 Thank you very much for your support! Sorry for the quite late response. This problem should be fixed I think. I've updated the WhatsApp Web version to the latest stable 2.2412.5, and it fixed the problem for me. For now, I will still open this issue for a few days to let anyone test the fix.


[BUG]Long tilte video cant click save button.

Describe the bug
Cant click save button.
To Reproduce
Steps to reproduce the behavior:
download long tilte video
Expected behavior
Cant click save button.
Screenshots

Desktop (please complete the following information):

OS: windows11
EDGE
latest

The problem is caused by display: table in parent div
This maybe help the dev


Plugin game (ma sói, tictactoe, ...)

Plugin name: game
How's it work: role-play game on message platform

Thanks.

WIP

WIP


Stock information

Depending on the info retrieved from Yahoo Finance, when a stock in the ListView is clicked, a new view should be shown with the information retrieved from the API call.

Current information being retrieved include:
company name, change, change percentage, price, daily high/low, yearly high/low


kong/kong-gateway:latest and chart version

Hello friends,
I am curious why the latest 2.41.1 chart from 5 Sept ships with 3.6 while the kong/kong-gateway:latest is now at 3.8 :cat:
Thanks!

#1131 will bump the default kong gateway version 3.7.
You can use 3.8 by specifying it via values.yaml if you so desire.

https://github.com/Kong/charts/releases/tag/kong-2.42.0 already ships with 3.7 as default.
You can still set the 3.8 using values.yaml. Hope that helps.


Custom Mime Renders

Feature: Notebook Editor, Interactive Window, Python Editor cells
Description
I have created a couple non-public mime renders and would like to use them in vscode. Is it possible to include non-standard ones myself? If not, I propose we allow for this.
Microsoft Data Science for VS Code Engineering Team: @rchiodo, @IanMatthewHuff, @DavidKutu, @DonJayamanne, @greazer

Hey @mlucool . So this is not currently supported. But in the nearish future vscode is adding native notebook support into the core of vscode. I believe part of that plan would involve allowing for custom renderers provided by extensions. You can keep on eye on that development here:
https://github.com/microsoft/vscode/issues/91987

Thanks for your feedback. This issue will likely be addressed once VS Code adds native support for editing notebooks. Please see https://github.com/microsoft/vscode/issues/91987.


Job stuck in the Processing state (InvisibilityTimeout not working properly)

During some tests where our server is being restarted while jobs are being executed, we find out, that sometimes, jobs became stuck in the Processing state, and they never recover.
After we start the server, those jobs are never executed again no matter how much time we wait (we've waited for the invisibility timeout and nothing happens, we had left it for a few hours, and nothing happens as well)
The Hangfire UI seems to know, that the job is stuck/aborted as it shows the following message

The job was aborted – it is processed by server testpc:server1:33704:7daebff8-39b2-4899-9f5f-3dae35a39863 which is not in the active servers list for now. It will be retried automatically after invisibility timeout, but you can also re-queue or delete it manually.

After digging into this issue a little bit more, I've found out a weird behaviour that I think is causing this, and it's related to InvisibilityTimeout.
In our example we have an InvisibilityTimeout of 2min (it can be 30min, it won't make a difference).
We have some long-running jobs that may exceed our InvisibilityTimeout (for our usecase, we are OK with that)
When the InvisibilityTimeout is exceeded, the job CancelationToken is triggered and the worker that is running that job aborts that execution.
At the same time, a new worker will pick that job and run it again.
This is the expected behaviour (AFIAK).
The problem is that this only happens once, the worker that is now processing the long-running job, will no longer receive any kind of cancellation when the InvisibilityTimeout is exceeded.
The same happens, if the server is restarted at this point, the job will no longer be processed again, and will be stuck in "Processing".
This seems to be happening, because when the InvisibilityTimeout cancels the first execution, in the Worker.Execute(BackgroundProcessContext context) it will remove the job from the queue, despite the fact that the job is still being processed by another worker.
This happens in these lines:
In the Worker.Execute(BackgroundProcessContext context)
var state = PerformJob(context, connection, fetchedJob.JobId);
>>>> In this example state is null

if (state != null)
{
	// Ignore return value, because we should not do anything when the current state is not Processing.
	TryChangeState(context, connection, fetchedJob, state, new[] { ProcessingState.StateName }, CancellationToken.None, context.ShutdownToken);
}

>>>>> I'm not sure if the following assumption is correct, because in our case the job was not performed,
>>>>>it is still being executed (in another worker), so I don't think it should be removed from the queue

// Checkpoint #4. The job was performed, and it is in the one
// of the explicit states (Succeeded, Scheduled and so on).
// It should not be re-queued, but we still need to remove its
// processing information.

requeueOnException = false;
fetchedJob.RemoveFromQueue();

Steps to reproduce

Download this repro project: https://github.com/braca/hangfire-mongo-bug
Run the project and go to:  http://localhost:5000/swagger
POST /Test/LongJobtesting
Wait a few minutes
The job will be running for 1minute in one of the available workers
After 1min, the CancellationToken will be triggered (because of the InvisibilityTimeout)
A new worker will pick the job and execute it
This new worker will never receive any CancelationToken even if the job execution time exceeds the InvisibilityTimeout
Restart the server
The job will be stuck in Processing and it will never be picked by any worker
You can check the logs/app.logs and confirm this

Logs
16:50:37.231 [INF] ExecuteLongTaskAsync - Job: 61e992d0172392e01cb7be14 -  Worker: cc1e7f72-5408-42f6-9415-6288f083cbbd - Started
16:50:37.231 [INF] ExecuteLongTaskAsync - Job: 61e992d0172392e01cb7be14 -  Worker: cc1e7f72-5408-42f6-9415-6288f083cbbd - i = 0 - Execution time: 00:00:00.0025689
16:50:47.244 [INF] ExecuteLongTaskAsync - Job: 61e992d0172392e01cb7be14 -  Worker: cc1e7f72-5408-42f6-9415-6288f083cbbd - i = 1 - Execution time: 00:00:10.0158631
16:50:57.253 [INF] ExecuteLongTaskAsync - Job: 61e992d0172392e01cb7be14 -  Worker: cc1e7f72-5408-42f6-9415-6288f083cbbd - i = 2 - Execution time: 00:00:20.0250140
16:51:07.256 [INF] ExecuteLongTaskAsync - Job: 61e992d0172392e01cb7be14 -  Worker: cc1e7f72-5408-42f6-9415-6288f083cbbd - i = 3 - Execution time: 00:00:30.0278617
16:51:17.263 [INF] ExecuteLongTaskAsync - Job: 61e992d0172392e01cb7be14 -  Worker: cc1e7f72-5408-42f6-9415-6288f083cbbd - i = 4 - Execution time: 00:00:40.0343608
16:51:27.262 [INF] ExecuteLongTaskAsync - Job: 61e992d0172392e01cb7be14 -  Worker: cc1e7f72-5408-42f6-9415-6288f083cbbd - i = 5 - Execution time: 00:00:50.0336615
16:51:37.261 [INF] ExecuteLongTaskAsync - Job: 61e992d0172392e01cb7be14 -  Worker: 23f99319-21d4-4001-a383-18027f31abad - Started
16:51:37.261 [INF] ExecuteLongTaskAsync - Job: 61e992d0172392e01cb7be14 -  Worker: 23f99319-21d4-4001-a383-18027f31abad - i = 0 - Execution time: 00:00:00.0010493
16:51:37.278 [INF] ExecuteLongTaskAsync - Job: 61e992d0172392e01cb7be14 -  Worker: cc1e7f72-5408-42f6-9415-6288f083cbbd - i = 6 - Execution time: 00:01:00.0498272
16:51:37.320 [ERR] ExecuteLongTaskAsync - Job: 61e992d0172392e01cb7be14 -  Worker: cc1e7f72-5408-42f6-9415-6288f083cbbd - Token was cancelled - Execution time: 00:01:00.0917212
16:51:47.265 [INF] ExecuteLongTaskAsync - Job: 61e992d0172392e01cb7be14 -  Worker: 23f99319-21d4-4001-a383-18027f31abad - i = 1 - Execution time: 00:00:10.0047696
16:51:57.280 [INF] ExecuteLongTaskAsync - Job: 61e992d0172392e01cb7be14 -  Worker: 23f99319-21d4-4001-a383-18027f31abad - i = 2 - Execution time: 00:00:20.0198516
16:52:07.281 [INF] ExecuteLongTaskAsync - Job: 61e992d0172392e01cb7be14 -  Worker: 23f99319-21d4-4001-a383-18027f31abad - i = 3 - Execution time: 00:00:30.0211184
16:52:17.294 [INF] ExecuteLongTaskAsync - Job: 61e992d0172392e01cb7be14 -  Worker: 23f99319-21d4-4001-a383-18027f31abad - i = 4 - Execution time: 00:00:40.0335476
16:52:27.311 [INF] ExecuteLongTaskAsync - Job: 61e992d0172392e01cb7be14 -  Worker: 23f99319-21d4-4001-a383-18027f31abad - i = 5 - Execution time: 00:00:50.0509575
16:52:37.321 [INF] ExecuteLongTaskAsync - Job: 61e992d0172392e01cb7be14 -  Worker: 23f99319-21d4-4001-a383-18027f31abad - i = 6 - Execution time: 00:01:00.0607559
16:52:47.333 [INF] ExecuteLongTaskAsync - Job: 61e992d0172392e01cb7be14 -  Worker: 23f99319-21d4-4001-a383-18027f31abad - i = 7 - Execution time: 00:01:10.0725911
16:52:57.352 [INF] ExecuteLongTaskAsync - Job: 61e992d0172392e01cb7be14 -  Worker: 23f99319-21d4-4001-a383-18027f31abad - i = 8 - Execution time: 00:01:20.0913322
16:53:07.363 [INF] ExecuteLongTaskAsync - Job: 61e992d0172392e01cb7be14 -  Worker: 23f99319-21d4-4001-a383-18027f31abad - i = 9 - Execution time: 00:01:30.1025756
16:53:17.378 [INF] ExecuteLongTaskAsync - Job: 61e992d0172392e01cb7be14 -  Worker: 23f99319-21d4-4001-a383-18027f31abad - i = 10 - Execution time: 00:01:40.1181397

Hello. Have you tried setting the option InvisibilityTimeout = TimeSpan.FromMinutes(30) described in the readme? Does it solve your issue? I'd be interested to know.
NOTE: By default the parameter InvisibilityTimeout of the MongoStorageOptions is configure with the value null, making the job to stay in the status 'processing' in the case of the error in the application. To solved this issue, set the value for 30 minutes how in the SqlServerStorageOptions.
public void Configuration(IAppBuilder app)
{
    var migrationOptions = new MongoMigrationOptions
    {
        MigrationStrategy = new MigrateMongoMigrationStrategy(),
        BackupStrategy = new CollectionMongoBackupStrategy()
    };
    var storageOptions = new MongoStorageOptions
    {
        // ...
        MigrationOptions = migrationOptions,
        InvisibilityTimeout = TimeSpan.FromMinutes(30)
    };
    GlobalConfiguration.Configuration.UseMongoStorage("<connection string with database name>", storageOptions);

    app.UseHangfireServer();
    app.UseHangfireDashboard();
}

After digging into this issue a little bit more, I've find out a weird behaviour that I think is causing this, and it's related with InvisibilityTimeout.
In our example we have an InvisibilityTimeout of 2min (it can be 30min, it won't make a difference).
We have some long running jobs that may exceed our InvisibilityTimeout (for our usecase, we are OK with that)
When the InvisibilityTimeout is exceeded, the job CancelationToken is triggered and the worker that is running that job aborts that execution.
At the same time, a new worker will pick that job and run it again.
This is the expected behaviour (AFIAK).
The problem is that this only happens once, the worker that is now processing the long running job, will no longer receive any kind of cancelation when the InvisibilityTimeout is exceeded.
The same happens, if the server is restarted at this point, the job will no longer be processed again, and will be stucked in "Processing".
This seems to be happening, because when the InvisibilityTimeout cancels the first execution, in the Worker.Execute(BackgroundProcessContext context) it will remove the job from the queue, despite the fact that the job is still being processed by another worker.
This happens in these lines:
In the Worker.Execute(BackgroundProcessContext context)
	    var state = PerformJob(context, connection, fetchedJob.JobId);
                >>>> In this example state is null

                if (state != null)
                {
                    // Ignore return value, because we should not do anything when current state is not Processing.
                    TryChangeState(context, connection, fetchedJob, state, new[] { ProcessingState.StateName }, CancellationToken.None, context.ShutdownToken);
                }

                >>>>> I'm not sure if the following assumption is correct, because in our case the job was not performed,
                >>>>>it still being executed, so I don't think it should be removed from queue

                // Checkpoint #4. The job was performed, and it is in the one
                // of the explicit states (Succeeded, Scheduled and so on).
                // It should not be re-queued, but we still need to remove its
                // processing information.

                requeueOnException = false;
                fetchedJob.RemoveFromQueue();

**Steps to reproduce**
1. Download this repro project: https://github.com/braca/hangfire-mongo-bug
2. Run the project and go to:  http://localhost:5000/swagger
3. POST /Test/LongJobtesting
4. Wait a few minutes
4. The job will be running for 1minute in one of the available workers
5. After 1min, the CancellationToken will be triggered (because of the **InvisibilityTimeout**)
6. A new worker will pick the job and execute it
7. This new worker will never receive any CancelationToken even if it job execution time exceeds the **InvisibilityTimeout**
5. Restart the server
6. The job will be stucked in **Processing** and it will never be picked by any worker
7. You can check the **logs/app.logs** and confirm this

**Logs**

16:50:37.231 [INF] ExecuteLongTaskAsync - Job: 61e992d0172392e01cb7be14 -  Worker: cc1e7f72-5408-42f6-9415-6288f083cbbd - Started
16:50:37.231 [INF] ExecuteLongTaskAsync - Job: 61e992d0172392e01cb7be14 -  Worker: cc1e7f72-5408-42f6-9415-6288f083cbbd - i = 0 - Execution time: 00:00:00.0025689
16:50:47.244 [INF] ExecuteLongTaskAsync - Job: 61e992d0172392e01cb7be14 -  Worker: cc1e7f72-5408-42f6-9415-6288f083cbbd - i = 1 - Execution time: 00:00:10.0158631
16:50:57.253 [INF] ExecuteLongTaskAsync - Job: 61e992d0172392e01cb7be14 -  Worker: cc1e7f72-5408-42f6-9415-6288f083cbbd - i = 2 - Execution time: 00:00:20.0250140
16:51:07.256 [INF] ExecuteLongTaskAsync - Job: 61e992d0172392e01cb7be14 -  Worker: cc1e7f72-5408-42f6-9415-6288f083cbbd - i = 3 - Execution time: 00:00:30.0278617
16:51:17.263 [INF] ExecuteLongTaskAsync - Job: 61e992d0172392e01cb7be14 -  Worker: cc1e7f72-5408-42f6-9415-6288f083cbbd - i = 4 - Execution time: 00:00:40.0343608
16:51:27.262 [INF] ExecuteLongTaskAsync - Job: 61e992d0172392e01cb7be14 -  Worker: cc1e7f72-5408-42f6-9415-6288f083cbbd - i = 5 - Execution time: 00:00:50.0336615
16:51:37.261 [INF] ExecuteLongTaskAsync - Job: 61e992d0172392e01cb7be14 -  Worker: 23f99319-21d4-4001-a383-18027f31abad - Started
16:51:37.261 [INF] ExecuteLongTaskAsync - Job: 61e992d0172392e01cb7be14 -  Worker: 23f99319-21d4-4001-a383-18027f31abad - i = 0 - Execution time: 00:00:00.0010493
16:51:37.278 [INF] ExecuteLongTaskAsync - Job: 61e992d0172392e01cb7be14 -  Worker: cc1e7f72-5408-42f6-9415-6288f083cbbd - i = 6 - Execution time: 00:01:00.0498272
16:51:37.320 [ERR] ExecuteLongTaskAsync - Job: 61e992d0172392e01cb7be14 -  Worker: cc1e7f72-5408-42f6-9415-6288f083cbbd - Token was cancelled - Execution time: 00:01:00.0917212
16:51:47.265 [INF] ExecuteLongTaskAsync - Job: 61e992d0172392e01cb7be14 -  Worker: 23f99319-21d4-4001-a383-18027f31abad - i = 1 - Execution time: 00:00:10.0047696
16:51:57.280 [INF] ExecuteLongTaskAsync - Job: 61e992d0172392e01cb7be14 -  Worker: 23f99319-21d4-4001-a383-18027f31abad - i = 2 - Execution time: 00:00:20.0198516
16:52:07.281 [INF] ExecuteLongTaskAsync - Job: 61e992d0172392e01cb7be14 -  Worker: 23f99319-21d4-4001-a383-18027f31abad - i = 3 - Execution time: 00:00:30.0211184
16:52:17.294 [INF] ExecuteLongTaskAsync - Job: 61e992d0172392e01cb7be14 -  Worker: 23f99319-21d4-4001-a383-18027f31abad - i = 4 - Execution time: 00:00:40.0335476
16:52:27.311 [INF] ExecuteLongTaskAsync - Job: 61e992d0172392e01cb7be14 -  Worker: 23f99319-21d4-4001-a383-18027f31abad - i = 5 - Execution time: 00:00:50.0509575
16:52:37.321 [INF] ExecuteLongTaskAsync - Job: 61e992d0172392e01cb7be14 -  Worker: 23f99319-21d4-4001-a383-18027f31abad - i = 6 - Execution time: 00:01:00.0607559
16:52:47.333 [INF] ExecuteLongTaskAsync - Job: 61e992d0172392e01cb7be14 -  Worker: 23f99319-21d4-4001-a383-18027f31abad - i = 7 - Execution time: 00:01:10.0725911
16:52:57.352 [INF] ExecuteLongTaskAsync - Job: 61e992d0172392e01cb7be14 -  Worker: 23f99319-21d4-4001-a383-18027f31abad - i = 8 - Execution time: 00:01:20.0913322
16:53:07.363 [INF] ExecuteLongTaskAsync - Job: 61e992d0172392e01cb7be14 -  Worker: 23f99319-21d4-4001-a383-18027f31abad - i = 9 - Execution time: 00:01:30.1025756
16:53:17.378 [INF] ExecuteLongTaskAsync - Job: 61e992d0172392e01cb7be14 -  Worker: 23f99319-21d4-4001-a383-18027f31abad - i = 10 - Execution time: 00:01:40.1181397

@braca,
Thanks for the links. I will check this out ASAP.
Thanks!

@braca,
Its a bug, I have a fix for it. just need to verify it.
Thanks!

@gottscj any update on this?

Hi @braca,
I usually wait for the next release of hangfire, but im not sure when the next release is due, so I will create a release with this shortly.
thanks for waiting. :)

Is this bug fixed in Version 1.7.0?
I have this issue, too. And need to know an update of the package solve this problem, or not 😄

Hi @gottscj,
With the 1.7.0 release I cannot replicate our issue, it seems to be fixed, but I'll try to do more tests
Thanks :)

ℹ I changed the queue strategy from 'Watch' to 'Poll' and now it works.

@braca,
Thank you for validating the fix. And thank you for setting up the repo!

@Sacharkin,
Are you experiencing the same issue as this ticket? Thanks

Not 100 percent the same. My issue was to spam jobs into the queue (1000 jobs in one second) some of them (5-10) start to stuck in the process state.
Hangfire runs on multiple instances. With only one instance I can not reproduce the issue.

@gottscj
I think my issue is different, should I create a new task for it?
And after some tests, I was able to solve my issue.
Change the write and read concern for the mongo client to 'majority'.
It was about to have a mongo replicaset.

@Sacharkin,
Please create a new issue for this. thanks!

I have issue with OracleStorageOptions: long-running background job is retried in 30 mins. What am I supposed to do? HF is 1.7.28.

@alexandis,
This is the repo for hangfire support for mongo db. I, unfortunately don't have any insight into the storage you are using.
Good luck!

@gottscj
I think my issue is different, should I create a new task for it? And after some tests, I was able to solve my issue. Change the write and read concern for the mongo client to 'majority'. It was about to have a mongo replicaset.

thanks for your inspiration，in my case(MongoDB.Driver2.22.0),changing connection string from "readPreference=SecondaryPreferred"to "readPreference=PrimaryPreferred" solved the stuck problem.

@marshafan,
Thanks for sharing!

Hi @gottscj,
I hope this message finds you well.
I am currently encountering an issue with Hangfire Core version 1.7.31 and Hangfire.Mongo. In my AWS ECS deployment, when attempting to execute background jobs for long-running servers, I've observed that during testing forcibly stopping server or server scale-down, some jobs are reassigned to other servers, while others revert to the old server which is server actually terminated or going to terminate . The issue persists until a new server is started from AWS.
I'm seeking your expertise in resolving this matter. Could you please provide guidance on how we can address and solve this problem?
Your assistance is greatly appreciated!
Thanks in advance.

Hi @dwevedivaibhav,
Did you try to set the InvisibilityTimeout? It is default disabled, but if you set a value Hangfire.Mongo will requeue the job if job has been terminated.
Please be aware that if the job runs for longer than the timeout, hangfire.mongo will mistake it for a stale job and requeue it.
I hope this helps

Hi @gottscj,
Thank you for your prompt response.
In my scenario, there are multiple workflows running in parallel within a 24-hour timeframe. The execution time varies based on the load, with some workflows completing in 5 minutes and others taking up to 3 hours or more also. Given this variability, how can I set an invisible timeout in this situation?
Any leads or guidance you can provide would be greatly appreciated.

@dwevedivaibhav,
You would need to set the timeout to 3+ hours then. Else, if possible, you could consider breaking up the the jobs, so executing time is smaller. You could save state between the jobs?
I hope this helps.


Dispose when async workflow is cancelled

Fixes #1436

Hi @mrinaldi, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

This looks like a solid fix, I'm happy.

@KevinRansom That was fast :) TBH several additional reviewers would be welcome, especially since it's Async code.

We have a huge problem with the new msbuild and the compiler dependence on it.  As far as I can tell, and Omar has better details but it seems as if the compiler and fsi will need to be run from a VS cmd window for the msbuild magic to work.
I may have to knuckle down and figure out the new referencing magic I’ve been whining about for 18 months.
Kevin
From: Don Syme<EMAIL_ADDRESS>Sent: Friday, September 23, 2016 10:36 AM
To: Microsoft/visualfsharp<EMAIL_ADDRESS>Cc: Kevin Ransom<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [Microsoft/visualfsharp] Dispose when async workflow is cancelled (#1555)
@KevinRansomhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FKevinRansom&data=01|01|Kevin.Ransom%40microsoft.com|56220dc62e814748a45808d3e3d820e3|72f988bf86f141af91ab2d7cd011db47|1&sdata=6wl5VWamfDrFCkoE%2B11VsyC70hmC4qFK2VqKQNpbuE4%3D&reserved=0 That was fast :) TBH several additional reviewers would be welcome, especially since it's Async code.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2Fvisualfsharp%2Fpull%2F1555%23issuecomment-249255207&data=01|01|Kevin.Ransom%40microsoft.com|56220dc62e814748a45808d3e3d820e3|72f988bf86f141af91ab2d7cd011db47|1&sdata=UKPVpy6yyuRk7upKMx2Kg4fZO6hv2JZOysNJ7hTZ3Ns%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAE76Fv-XaujttjRAjIZil2HjCJcuU9IBks5qtA4RgaJpZM4KCS0R&data=01|01|Kevin.Ransom%40microsoft.com|56220dc62e814748a45808d3e3d820e3|72f988bf86f141af91ab2d7cd011db47|1&sdata=hfkcqgZ7gQ6h7AXVcThyXECDh89bc3lWwf0Z%2BogiUSk%3D&reserved=0.

LGTM :+1:


Expo React Native configuration questions

Hello! I have general questions about using this package with Expo/React Native.
I am using expo-av to record audio and then microsoft-cognitiveservices-speech-sdk to provide continuous recognition.
It doesn't work on mobile --> Is const audioConfig = AudioConfig.fromDefaultMicrophoneInput() compatible with react native at all or do I have to use the pushStream?
The library IS working as intended on Web only, but I am getting lots of warnings:
Require cycles are allowed, but can result in uninitialized values. Consider refactoring to remove the need for a cycle.

Require cycle: node_modules/microsoft-cognitiveservices-speech-sdk/distrib/lib/src/common.speech/Exports.js -> node_modules/microsoft-cognitiveservices-speech-sdk/distrib/lib/src/common.speech/VoiceServiceRecognizer.js -> node_modules/microsoft-cognitiveservices-speech-sdk/distrib/lib/src/common.speech/Exports.js

I wrote my code loosely based on this example
I have also added require('node-libs-react-native') in my metro.config.js
Have I perhaps done something terribly wrong or are these "Require cycles are allowed" warnings OK to ignore?
Here is a snippet of my code:
  const [initialized, setInitialized] = useState(false)
  const [recording, setRecording] = useState<Audio.Recording | null>(null)
  const [recognizer, setRecognizer] = useState<SpeechRecognizer | null>(null)

  useEffect(() => {
    async function requestPermissions() {
      const permissions = await Audio.requestPermissionsAsync()
      if (permissions.granted) {
        setInitialized(true)
      } else {
        console.log('Failed to get Permissions for Audio Session!')
      }
    }

    async function cleanupRecording() {
      if (recording) {
        try {
          const status = await recording.getStatusAsync()
          if (status.isRecording) {
            await recording.stopAndUnloadAsync()
            console.log('Recording stopped and unloaded')
          }
        } catch (error) {
          console.error('Error during recording cleanup:', error)
        }
      }
    }

    async function setupRecognizer() {
      const audioConfig = AudioConfig.fromDefaultMicrophoneInput()

      const newRecognizer = new SpeechRecognizer(speechConfig, audioConfig)
      newRecognizer.recognized = (_, event) => {
        if (event && event.result && event.result.text) {
          setQuestion(
            prevQuestion => prevQuestion + ' ' + event.result.text.trim(),
          )
        } else {
          console.log('No recognizable text found')
        }
      }
      setRecognizer(newRecognizer)
    }

    // 1. Request permissions
    if (!initialized) {
      console.log('initializing')
      requestPermissions()
      return
    }

    // 2. Setup Microsoft Speech Recognizer
    if (initialized && !recognizer) {
      console.log('Setting up Speech Recognizer')
      setupRecognizer()
    }

    return () => {
      recognizer?.close()
      cleanupRecording()
    }
  }, [initialized, recognizer])

  // ...return UI component here

@ckanissatran Thank you for using the JS Speech SDK and writing this issue up. To answer your questions:

For react-native microphone input, a pushStream must be used in the manner shown in the sample app. Note that mic input wasn't added to the sample until an enterprising community member demonstrated how the react-native-live-audio-stream library could be used to forward wav bytes to a pushStream.
The require cycles should not affect JS Speech SDK functionality.

One further comment, the JS Speech SDK does not officially React Native (or React). I wrote that sample as a courtesy proof of concept, but can't support it further currently.


ci: do not use sudo for additional runtime installation

📜 Description

💡 Motivation and Context

📢 Changelog

JS

iOS

Android

🤔 How Has This Been Tested?

📸 Screenshots (if appropriate):

📝 Checklist

[ ] CI successfully passed
[ ] I added new mocks and corresponding unit-tests if library API was changed

Sudo is required, so closing this PR


Update route to use new location of thumbnails

I was trying to lint, but always got error, so i cried myself to sleep. But now people "faces" show up!

Looks good to me, but please squash the commits 😊

Not sure if you know how to do this, so I looked up a tutorial: https://getaround.tech/git-rebase-edit-history/

Ok, trying it now!

Thanks a lot!!

Awesome! 👍


move audit- and error-log into database

Feature-request
Description
At the moment the audit- and error-logs for the users and the system are written into log-files.  In order to make this easier to handle, the logs should be moved into database. Beside this, it should be possible to request these logs via REST-API.

added new database-tables audit_log and error_log


[.NET][WPF] Inline Actions are missing when not using Xceed

Platform
What platform is your issue or question related to? (Delete other platforms).

.NET WPF (non-Xceed)

Issue
Inline actions appears to only have been implemented in the xceed version of the text input, not the default text input. Our visualizer uses xceed so it works fine in the visualizer:
            // Use the Xceed rich input controls
            Renderer.UseXceedElementRenderers();

If you comment that line of code out, however, inline actions no longer appear:

Json:
{
  "$schema": "http://adaptivecards.io/schemas/adaptive-card.json",
  "type": "AdaptiveCard",
  "version": "1.2",
  "body": [
    {
      "type": "TextBlock",
      "text": "Text input with an inline action"
    },
    {
      "type": "Input.Text",
      "id": "input1",
      "inlineAction": {
        "type": "Action.Submit",
        "iconUrl": "http://adaptivecards.io/content/cats/1.png",
        "title": "Reply"
      }
    },
    {
      "type": "TextBlock",
      "text": "Text input with an inline action with no icon",
      "wrap": true
    },
    {
      "type": "Input.Text",
      "id": "input2",
      "inlineAction": {
        "type": "Action.OpenUrl",
        "title": "Reply",
        "url": "http://adaptivecards.io"
      }
    },
    {
      "type": "TextBlock",
      "text": "Multiline Text Inputs with inline actions"
    },
    {
      "type": "Input.Text",
      "id": "input3",
      "isMultiline": true,
      "inlineAction": {
        "type": "Action.Submit",
        "iconUrl": "http://adaptivecards.io/content/cats/1.png",
        "title": "Reply"
      }
    },
    {
      "type": "Input.Text",
      "id": "input4",
      "isMultiline": true,
      "inlineAction": {
        "type": "Action.OpenUrl",
        "title": "Reply",
        "url": "http://adaptivecards.io"
      }
    }
  ]
}

@almedina-ms sending another one your way. Could you take a stab? Thanks!


[WIP] Slack notification

This PR adds a notification to the NRCan slack channel when the master branch fails CI.
Currently only added to the etl job while the exact message and mechanism is worked out.

It's unfortunate we have to do it at this level, but I don't currently see a better option.


Update 003 - CSS Preprocessors and Structuring CSS.md

Corrected some typos, added Twitter Handles

thanks!


Why do we not have a config file for options?

After i see the help of the tool , i wonder if we use a config file to save the cmds may be more clear and more usenable.

I would like the idea. It would allow sharing one config file across platforms, similar to a pyinstaller.spec file. Currently, one has to maintain two versions, a build.bat and another build.sh.

Here is an example of a cross platform, variant building, from my commercial test suite:
# nuitka-project: --include-package-data=data_inclusion
# nuitka-project: --embed-data-files-runtime-pattern=data_inclusion/*.txt

# nuitka-project-if: os.getenv("TEST_VARIANT", "standalone") == "standalone":
#   nuitka-project: --standalone
#   nuitka-project-if: os.getenv("TEST_VARIANT_QT", "pyside2") == "pyside2":
#     nuitka-project: --enable-plugin=pyside2
#     nuitka-project: --include-qt-plugins=sensible
#     nuitka-project: --embed-data-files-qt-resource-pattern=data_inclusion/res/*
#   nuitka-project-if: os.getenv("TEST_VARIANT_QT", "pyside2") == "pyside6":
#     nuitka-project: --enable-plugin=pyside6
#     nuitka-project: --include-qt-plugins=sensible
#     nuitka-project: --embed-data-files-qt-resource-pattern=data_inclusion/res/*
#   nuitka-project-if: os.getenv("TEST_VARIANT_QT", "pyside2") == "no-qt":
#     nuitka-project: --enable-plugin=no-qt
#     nuitka-project: --noinclude-data-files=data_inclusion/res

# nuitka-project-if: os.getenv("TEST_VARIANT", "standalone") == "accelerated":
#   nuitka-project: --follow-import-to=data_inclusion
#   nuitka-project-if: os.getenv("TEST_VARIANT_QT", "pyside2") == "pyside2":
#     nuitka-project: --qt-binding=PySide2
#   nuitka-project-if: os.getenv("TEST_VARIANT_QT", "pyside6") == "pyside6":
#     nuitka-project: --qt-binding=PySide6

# nuitka-project-if: os.getenv("TEST_VARIANT", "standalone") == "module":
#   nuitka-project: --module
#   nuitka-project: --follow-import-to=data_inclusion
#   nuitka-project: --module-name-choice=original

# The tests output to standard output, even for the Qt bindings, which
# otherwise without this option will recommend to disable console, but
# for this test, makes no sense.
# nuitka-project: --enable-console

Oh, I just re-read https://github.com/Nuitka/Nuitka/#nuitka-options-in-the-code, and when reading it the first time, I somehow got stuck on the "support for conditional options" part, and missed the obvious that all options can be placed in the code. That is pretty cool!

Yes, indeed. The else is not used in this example:
      # Compilation mode, support OS specific.
      # nuitka-project-if: {OS} in ("Windows", "Linux"):
      #    nuitka-project: --onefile
      # nuitka-project-if: {OS} not in ("Windows", "Linux"):
      #    nuitka-project: --standalone

Some variables are pre-defined and expanded. The bad thing maybe is that it is a DSL and not actual Python. It has served my needs when I did commercial deployments to be point that the different batch files only are about how to invoke Nuitka, and barely have a point.

Thank you for your help, the example can meet my demand.


[BUG] SeoToolkit.Redirects Unable to select a page that resides under Collection View

I can't select a page that resides under page with enabled collection view, because the Green Button "Submit" isn't showing and I get an error message in the console, see screenshot below:

The Error message:
$apply @ angular.js:19622 (anonymous) @ angular.js:29127 dispatch @ jquery.min.js?d=638276793750000000:2 v.handle @ jquery.min.js?d=638276793750000000:2 umbraco#/SeoToolkit/Redirects/list?mculture=en-US:1 An invalid form control with name='textbox' is not focusable. <input type="text" id="url" name="textbox" ng-model="model.value" class="umb-property-editor umb-textstring textstring ng-pristine ng-untouched ng-isolate-scope ng-valid-val-server ng-empty ng-invalid ng-invalid-required" val-server="value" ng-required="model.validation.mandatory" aria-required="true" aria-invalid="false" ng-trim="false" ng-change="change()" ng-readonly="readonly" required="required">
Installed packages:
<TargetFramework>net7.0</TargetFramework>
<PackageReference Include="Our.Umbraco.MultilanguageTextbox" Version="12.0.0" />
<PackageReference Include="SeoToolkit.Umbraco.MetaFields" Version="3.2.0" />
<PackageReference Include="SeoToolkit.Umbraco.Redirects" Version="3.2.0" />
<PackageReference Include="Umbraco.Cms" Version="12.0.1" />
<PackageReference Include="Microsoft.ICU.ICU4C.Runtime" Version="<IP_ADDRESS>" />
<PackageReference Include="Umbraco.TheStarterKit" Version="11.0.0" />
Steps to reproduce:

Install Umbraco.TheStarterKit
Install SeoToolkit.Umbraco.Redirects
Try to create a redirect to a blog page

Hi @grafsnikers,
Thanks for the report! I've taken a look and the error doesn't seem related to it (also not quite sure why the error is there), so I have created a different issue for that: https://github.com/patrickdemooij9/SeoToolkit.Umbraco/issues/221.
As for your issue, it has been fixed with this commit: https://github.com/patrickdemooij9/SeoToolkit.Umbraco/commit/4213d2ca58c7cd6283ab54b3c248612a1da55da4

Great, thanks a lot!

Fix is now available in the latest release: https://github.com/patrickdemooij9/SeoToolkit.Umbraco/releases/tag/3.3.0


Strapi v4

The plugin dosent seem to work with strapi v4, or i miss configured it.

Edit: found the solution, see: nohehf:patch-1


tracing-step feature for evm-runtime

Emitting RuntimeEvent::Step is costly under some circumstances, for example when the Memory layout is huge. This has a big impact in tracing performance.
This PR introduces a new tracing-step feature to gate this event.

This prevents to have a new node that support multiple ways of tracing, both needing and not needing to listen to this Step event.
A better idea would be to add a new function to the EventListener like ignore_step_event(&self) -> bool such that the EVM will not emit this costly event if not needed.
An even better approach would be to make the costly fields optional (stack/memory), and have functions care_about_stack/care_about_memory or something like that to avoid to clone those if not necessary. This would match the options supported by the raw tracing RPC and would improve their performances unless a user request a trace with all the data enabled.

Sounds like a good alternative to me @sorpaas what do you think? should I close this and go with @nanocryk proposal?


Add missing Thymeleaf expression language hints

The org.thymeleaf.expression package contains a variety of classes that can be used in Thymeleaf templates e.g. org.thymeleaf.expression.Arrays, which is accessed in a template via ${#arrays.xxx()}. This requires reflection config like the below example:
@TypeHint(types = {org.thymeleaf.expression.Arrays.class},
					access = AccessBits.LOAD_AND_CONSTRUCT_AND_PUBLIC_METHODS)}

On the basis that the project added IterStatusVar in https://github.com/spring-projects-experimental/spring-native/issues/679, I think the project could logically add something like the following for the 15-20 classes in the org.thymeleaf.expression package?
Happy to send a PR (including tests) if you think this makes sense.

Yes, I am not surprised we need to add some more. It is perhaps just a shame we will end up adding those that aren't being used to some apps, but I suspect working out which are required in a particular app is not easy.
Happy to take a PR, could perhaps enhance the existing sample to cover more kinds of expression.

PR provided to add the expression classes; I will look at improving the sample separately as having some issues getting the tests to pass on my machine even prior to the change.
I did think about whether I could create some kind of processor to scan through any templates found and identify which #xxx expressions were being used - but it felt like quite a bit of effort given the expression classes themselves look fairly small and straightforward.


Update README.md

Update the POSTHOG_HOST to reflect the new posthog cloud.

Or maybe do like the official documentation:
<ph_instance_address>
And add a list of address somewhere.

I will close this issue, but maybe we should add a list of addresses. And where we can get it.


fatal (exception reentered):

I got error fatal (exception reentered): with bullet and search_cop gem.
I don't know these gem related to error.
But, I was able to reproduce by simple project.
https://github.com/kntmrkm/search_cop_with_bullet
bundle exec rake db:create
bundle exec rake db:migrate
bundle exec rake db:seed

and, in rails console, execute below command.
User.take.comments.search_by_q('n')
Command will not finished.
On the view process. You will get fatal (exception reentered): error.
(above repo does not include view process)
After remove activerecord-precount.
The error will disappears.
thanks.

Additional info.
If remove bullet gem.
The error does not occurs.

@kntmrkm Could you try this? https://github.com/flyerhzm/bullet/pull/330
cc/ @kenchan0130

This is current problem.

With gem 'bullet', github: 'k0kubun/bullet', branch: 'module-prepend'.
Solved

ちなみに、ruby2.4.1だと以下のようなエラーになりました。
問題のエラーは解消されました。ありがとうございます。

Closing because https://github.com/flyerhzm/bullet/pull/330 is merged.

The second error may be bug for combination of Ruby 2.4.1 and ActiveSupport or in GC.


chore(main): release 0.1.5

:robot: I have created a release beep boop
0.1.5 (2023-07-27)
🐛 Bug Fixes

Correctly assign response headers from httpGraphQLResponse (#34) (26122aa)

🧹 Miscellaneous

deps: update all non-major dependencies (749b0fe)
deps: update all non-major dependencies (#31) (afee96b)
deps: update dependency @types/jest to v29.5.0 (f60493d)
deps: update dependency prettier to v3 (d9e32f3)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/apollo-server-integrations/apollo-server-integration-google-cloud-functions/releases/tag/v0.1.5 :sunflower:


Add commands for opening the game GUIs

In addition to the items, commands should be available in order to manually open the game GUIs.

Use /gamegui join/open when server is restarted.


Arbiter rewrite

I have created a basic zest-based arbiter which is capable of minting databox tokens for testing against the store (zestdb). It is light-weight in RAM compared to the node version (about 3MB compared to 44MB).
It is available here: https://github.com/jptmoore/arbiter
I have not yet implemented the existing arbiter functionality as it would be good at this point to discuss what (if anything) needs changing. I also did not find any documentation on the current API other than looking at the code itself.

This is great space saving! I think would be good to move over from the old arbiter, for easier integration as the platform gets used and we start using the arbiter for apps.

Cool,
Nice savings! API is documented in the readme https://github.com/me-box/core-arbiter
There are some more docs here and here. Thay looks reasonably up-to-date at first glance.
Most of the API is seems fit for purpose, but it's worth revisiting the path based tokens as the core-store is no longer a REST API.
@mor1 we should add this to the platform skype on Tuesday.

saw the notes. looks good. @jptmoore eta on supporting rest of api?

Should take a couple days to translate to zest.

This has now been integrated and will be in the next version


Error getting array usage: division by zero

Describe the bug
Error getting array usage: division by zero
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots

Unraid (please complete the following information):

7.0.0 RC1

Home Assistant (please complete the following information):

Home Assistant installation type (please complete the following information):

[ ] Docker / Container
[x] Virtual Machine
[ ] Raspberry Pi

Priority of Request

[ ] High
[x] Medium
[ ] Low

This has been addressed and will be available in the next release


FRASER 2.0 versions not available

Hello,
I am trying to update to a FRASER package which corresponds to FRASER 2.0.
According to GitHub anything above 1.9 would do and an R version of 4.0.0 or higher should enable installation with BiocManager.
Unfortunately, I am unable to install any version above 1.14.1.

available.packages(repos=BiocManager::repositories())["FRASER", "Version"]
[1] "1.14.1"
R.version
_
platform       x86_64-conda-linux-gnu
arch           x86_64
os             linux-gnu
system         x86_64, linux-gnu
status
major          4
minor          3.3
year           2024
month          02
day            29
svn rev        86002
language       R
version.string R version 4.3.3 (2024-02-29)
nickname       Angel Food Cake
packageDescription("FRASER")$Version
[1] "1.14.1"
BiocManager::install("FRASER")
Bioconductor version 3.18 (BiocManager 1.30.23), R 4.3.3 (2024-02-29)
Old packages: 'BiocFileCache', 'broom', 'bslib', 'cachem', 'curl',
'data.table', 'DBI', 'dbplyr', 'evaluate', 'farver', 'fastmap', 'fs',
'future', 'future.apply', 'GenomeInfoDb', 'ggplot2', 'gtable', 'hardhat',
'highr', 'htmltools', 'httpuv', 'KernSmooth', 'knitr', 'lattice',
'matrixStats', 'munsell', 'nlme', 'openssl', 'promises', 'ragg',
'RcppArmadillo', 'repr', 'rlang', 'rmarkdown', 'RSQLite', 'rstudioapi',
'seriation', 'shiny', 'stringi', 'survival', 'systemfonts', 'textshaping',
'tinytex', 'vegan', 'VGAM', 'xfun', 'xts'
Update all/some/none? [a/s/n]: n
Warning message:
package(s) not installed when version(s) same as or greater than current; use
force = TRUE to re-install: 'FRASER'

I tried the force installation or updating other packages but ended up with the same FRASER version.

Hi, can you please try to install the latest release from github?
devtools::install_github('gagneurlab/FRASER', ref='1.99.4', dependencies=TRUE) Should do the trick.
As to why installing from Bioconductor fails:
FRASER 2.0.0 is available on Bioconductor release of 3.19. I suspect that you are trying to install with a previous Bioconductor release (3.18) and that's why you end up installing 1.14.1.

@vyepez88 @ischeller should we update the readme to propose installing from github as default?


refactor: restructure project

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


Storybook > Elements >Card Detail >Selecting small mobile view radio button is stick on card text.

Describe the bug

Selecting small mobile view radio button is stick on card text.
Elements> Avatar >With Initial >Selecting dark theme Initial latter is not display.
Elements> Avatar >With Label>Selecting dark theme label is not display proper.

To Reproduce
Steps to reproduce the behavior:

Go to 'Elements'
Click on 'Card Detail'
Click on 'Card Detail'

Expected behavior
Selecting small mobile view space between card text and radio button.
Selecting dark theme Initial latter is display like WT.
Screenshots
Card Detail -

Avatar -

Avatar >With Label -

Desktop (please complete the following information):
-OS: [Windows 10 Enterprise]
-Browser [ edge]
-Version [ 125.0.2535.92]
Smartphone (please complete the following information):NA

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Completed -

@trivenikulkarni
Closed by Rameshwar in version 1.1.6, which is available in version 1.1.7

version 1.1.7


Use enum for indexType parameters in Public API

What
Use a enum instead of strings for index type parameters
Why
Users have been confused with the index type parameter and have been using their own, restricting this to an enum will make the API easier to understand.
How

Created IndexType enum
Change the parameter of type of indexType from String to IndexType

Tests
No additional tests have been added, however some tests were removed as they are no longer needed.
Reviewers
reviewer @tomblench
reviewer @alfinkel
Issues
Fixes: #246

+1

+1


Effect that ignore self does not work

Describe the bug
Aura is not applied to anything if 'ignore self?' is checked
To Reproduce
Steps to reproduce the behavior:

Setup an active aura with 'ignore self?' checked (like Spirit Gardians)
Use the spell

Expected behavior

Effect does not apply to the tokens when its supposed to
If 'Ignore self?' is unchecked, it works fine

Screenshots
Here is the aura tab:

And here is the result (unaffected tokens)

Now here's the result with 'Ignore self?' unchecked:

Environment:

Foundry Version 11.315
System dnd5e 2.4.0
Module Version 0.9.1
All Modules Disabled: No
Module List if no: see full list below
Active Modules:

Active-Auras v;
Active Token Effects v;
Automated Animations v;
Automated Evocations - Companion Manager v;
Babele v;
Better Roofs v;
Build-a-Bonus v;
Carousel Combat Tracker v;
Chris's Premades v;
Combat Booster: Turn Marker, Recent Actions and more v;
DFreds Convenient Effects v;
Dice So Nice! v;
Dice Tray v;
DnD5e Drag Ruler Integration v;
Drag Ruler v;
Dynamic effects using Active Effects (DAE) v;
Effect Macro v;
Filepicker + v;
Force Client Settings v;
fr-FR - DND5 System v;
Hover Distance v;
Image Context v;
Item Piles v;
JB2A - Jules and Ben's Animated Assets - Free Content v;
JB2A - Patreon Complete Collection v;
Let's Trade 5e v;
Library - Color Settings v;
Library - libWrapper v.0;
Library - socketlib v;
Macro Wheel v;
Midi QOL v1;
Midi SRD v;
Module Management+ v;
Monaco Macro Editor v;
Monk's TokenBar v;
Pin Cushion v;
PopOut! v;
Quickdraw v;
Quick Insert - Search Widget v;
Sequencer v;
SmallTime v;
Smart Target v;
Splatter v;
srd-heros-et-dragons v;
Taskbar v;
Template Macro v;
TheRipper93's Module Hub v;
Tidy5e Sheet v;
Times Up v;
Token Attacher v;
Token HUD Wildcard v;
Token Magic FX v;
Token Z v;
UI - Tidy UI - Game Settings v;
Warp Gate v;
World Scripter v;

Hmm, very odd, I'm sure this did work, as the code has been there for ages. Please try 0.9.2

It works with 0.9.2, thanks.


Update link for "MyPyramid Food Raw Data"

MyPyramid Food raw data can't be currently found at https://catalog.data.gov/dataset/mypyramid-food-raw-data-f9ed6
With a quick google search I found https://healthdata.gov/dataset/mypyramid-food-raw-data which seems to have the same exact data.

Hello everyone 👋
I hope this message finds you well. As you may know, the original repository for app ideas has not been updated in a while and is no longer actively maintained. However, I'm excited to let you know that a new repository has been created to continue this project (https://github.com/vincent0426/app-ideas).
The new repository aims to provide a comprehensive list of practical and achievable app ideas that developers of all levels can work on to improve their skills and build their portfolios. We welcome contributions from anyone who wants to add their own app ideas or improve the existing ones, and we are also looking for volunteers who can help maintain the repository and keep it up to date.
If you are interested in contributing or maintaining this project, please head over to the new repository (https://github.com/vincent0426/app-ideas) ⭐ and take a look at the existing issues and pull requests. Feel free to open a new issue if you have an app idea to suggest or if you have any questions about the project.
Thank you for your attention, and I hope to see you in the new repository soon! 💖

⚠ This message was generated by a robot 🤖, if this message reached you, consider leaving a star for this robot, repository link
⭐ https://github.com/aldairandre/github-bot ⭐


MAINT: modernization of qhull.pyx

The devdocs detailed roadmap contains a mandate for code modernization:

Cython’s old syntax for using NumPy arrays should be removed and replaced with Cython memoryviews.

This PR aims to do this in full for the Cython interface to Qhull. While I found no evidence of performance gains (see below) in a local test, there may be opportunities to release the GIL in more places now--the full complement of advantages over the old buffer syntax are highlighted in the docs.
For reviewers to consider:

this assumes that our tests are probing all attributes that are exposed from Qhull through to Python; any such attributes that aren't tested are prone to be returned as memoryviews instead of ndarrays proper; in-library usage of Qhull by some parts of interpolate exposed a few cases not covered by spatial tests but there are bound to be others
ways to avoid np.asarray and np.array calls are probably useful -- I think some may be hard to avoid, but maybe I have a few more than needed
some of my typing / casting changes may be too aggressive re: np.type vs type (but the tests don't complain in my hands)

asv continuous -e -b "time_convex_hull" 983c6d >& bench.txt
       before           after         ratio
     [f4f3c0ed]       [983c6d5f]
     <master>         <modernize_qhull_pyx>
+      69.0±0.8μs       84.5±0.7μs     1.22  spatial.ConvexHullBench.time_convex_hull(10, False)
+        74.0±1μs         90.3±1μs     1.22  spatial.ConvexHullBench.time_convex_hull(10, True)
+         155±3μs        172±0.9μs     1.11  spatial.ConvexHullBench.time_convex_hull(100, False)
+         163±1μs          181±3μs     1.11  spatial.ConvexHullBench.time_convex_hull(100, True)

I can see refguide is finding some untested code paths :)

The changes make the code 10 to 20 percent slower.  What is gained by making this change?  If it is just cosmetic (i.e. the newer syntax looks nicer), it doesn't seem worth it.

It's possible we're leaving some performance on the table by not specifying contiguous-access dimensions.
That is, we have cdef double[:, :] but might want to use cdef double[:, ::1] instead.

While doing this we need to be careful not to break existing code e.g. with joblib parallelization, see #9993

I'll put a "needs-work" label on this--hopefully I can get it up to a standard that satisfies reviewers eventually. It was tricky just doing the initial surgery / getting tests passing.

It was tricky just doing the initial surgery / getting tests passing.

And thanks for doing that!
I did some elementary tests, and in the simplest cases (e.g. simply summing a contiguous 1-d read-only array), the ndarray and memoryview versions had the same performance.  The relative performance in more complicated examples can depend on how error-checking is configured,  what kind of slicing is done on the arrays, and whether or not new arrays are created.  And probably other things--I don't have a complete grasp of all the issues.  Jake van der Plas's blogs from 2012 on the issue are interesting reading:

http://jakevdp.github.io/blog/2012/08/08/memoryview-benchmarks/
http://jakevdp.github.io/blog/2012/08/16/memoryview-benchmarks-2/

One thing I've lost time on here is getting traction to line profile the Cython code. There are plenty of stack overflow questions / answers about this, but the machinery is informal at best I think using i.e., @rkern line_profiler and some extra directives.  Some nice examples are available in Juypter context, but I don't think a transplant of the baked-in .pyx to a notebook cell is suitable in this case.
In particular, I think I'd need to pass CYTHON_TRACE=1 and CYTHON_TRACE_NOGIL=1 to gcc via define_macros, but many of our Cython files go through our custom cythonize.py, which doesn't even use define_macros directly. There's probably a way around this, but it is not obvious and feels awkward to get going.

This didn't quite work out -- maybe needs to be done in smaller chunks, but probably not a priority so closing my own PR here for now.


Fixed missing syntax block

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

Thanks for the contribution. I've merged your changes and they should be live soon.


Improve mascot announcements UX

This is the first pass at improving the UX of mascot announcements. There is more to be done after this!
In this PR:

Mascot no longer shows announcements automatically on hover
Instead, it moves slightly and shows a tooltip ("click to see what's new")
Announcements only shown upon clicking the mascot
Increase space given to app-header text on mobile screens

Demo

Coverage increased (+0.4%) to 33.996% when pulling 23ee4f8e7cd82416b29b5a671a5df999956fe1bb on thevoiceofzeke:styleUpdates into ec6fdcae1fe9bf1c104dab9009829bdaf6e85eb8 on UW-Madison-DoIT:master.

Coverage decreased (-0.07%) to 33.499% when pulling 71e97c45b13de73f50e08dc7610aec75fd710748 on thevoiceofzeke:styleUpdates into ec6fdcae1fe9bf1c104dab9009829bdaf6e85eb8 on UW-Madison-DoIT:master.

:+1:


Grading students

Problem Link
https://www.hackerrank.com/challenges/grading/problem
Coding platform
Hackerrank
Programming language

[ ] C
[ ] C++
[ ] Java
[ ] Python

I would like to solve this in python @Nivedita967

If anyone fails, I can also provide solution for this in C and Python

If anyone fails, I can also provide solution for this in C and Python

Python is done, you may go for C.

Can I do this in Java?

Can I do this in Java?

Sure @pri1311 Go for it.


await CreateAsync() when try to create a ServerError asynchronously

Fixes #1901
Add ability to set a non-json fixed response and additionally an Exception when creating a FixedReturnClient.
Introduced async version of ReturnConnectionStatus on InMemoryConnection in order to use the async request/response pipeline (which is where the original issue manifests)

LGTM


department data index

on the left hand side where a user can browse data and we have data listed by city category (infrastructure, management, etc) there is also the option of seeing it by department.  that function didn't work for me this morning.  for example, it said that DSD had "0" data. and when i clicked on DSD nothing came up.  it appeared to be the case for several other departments.

If you make a selection in either category or department, the other section changes based on your selection. For example, if I select the category of Public Safety, then Development Services Department would have 0 datasets because none of the datasets they released fall into the Public Safety category.
That most likely was the problem, but if it happens again, send me a screenshot so I can better troubleshoot.


Why SFML needs X11 for RenderTexture?

I want to use RenderTexture for creating image and save it to file, but when i try to do it without X11, i get error:
Failed to open X11 display; make sure the DISPLAY environment variable is set correctly
Aborted (core dumped)

My code:
#include<SFML/Graphics.hpp>

using namespace sf;

int main()
{
	RenderTexture drawtex;
	drawtex.create(800, 600);

	CircleShape shape(100.f);
	shape.setFillColor(Color::Red);

	drawtex.clear();
	drawtex.draw(shape);
	drawtex.display();

	drawtex.getTexture().copyToImage().saveToFile("image.png");

}

PS. Sorry for my english.

SFML isn't built to run without X11. It doesn't "lazy load" X11 as active dependency, because there are global handlers required for all features of SFML. There are some attempts being made (and hopefully merged at one point) with making SFML work without X11.
For questions about SFML you should be using the SFML forum or join our Discord server, as we try to keep the issue tracker for confirmed issues with SFML.


Ask for confirmation when removing all prescription items.

 Ask for confirmation while removing all listed drug items

Changes Unknown when pulling 7af7cd2ededdb1bae3118fa57d8e8e8084725c57 on ynurmahomed:confirmation into ** on esaude:master**.

Coverage increased (+0.1%) to 50.114% when pulling bdfc2840e7b2e3efe646d10f8cbf5e46a2d842a6 on ynurmahomed:confirmation into c54007ae71878f6af2a40c1497aa1b0d0233575d on esaude:master.

Coverage increased (+0.1%) to 50.114% when pulling b02fd3837130355b355f988c7457066532ae7dda on ynurmahomed:confirmation into c54007ae71878f6af2a40c1497aa1b0d0233575d on esaude:master.

Much better now. Just a small display problem in clinical presctiption:

I also like the idea of transforming this into a directive, now we can easily reuse :+1:

Coverage increased (+0.3%) to 50.114% when pulling a95d9de09145a7676b9cd3add0c268567e70b1a5 on ynurmahomed:confirmation into cf356be3feb75eabefa9d3166acb1a12b160d2c3 on esaude:master.


feat(content-pages): front matter validation, include front matter in metadata

Motivation
Complete a TODO.
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
Check test pages

Thanks, LGTM 👍


Popper/Blueprint based menus broken in react dev mode

This one manifesting itself strangely in two contexts that may be related

Blueprint based context menu  (used in DashContainer) is really slow to come up (e.g. 3 secs)
Popper based context menu (used in mobile menu button) is impossibly slow to close.

OK -- I figured out the source of this.  As part of our refactoring of menus for typescript we are now deep cloning our menu items.  This is bad news and causing some kind of circular dependency in dev mode! (yikes)

Fixed here in type script branch
https://github.com/xh/hoist-react/commit/8b26313ed1a65de3354ae5e7de0356f3bb4a8a25


User regions

resolves #445

Adds owner, public fields to a region and model_run. Defaults public to True and owner to None in both instances
Condenses the POST endpoint for Region creation to create a region not associated with model run and being bound to a user with the option to make it True or False
Adds a GET /region/details endpoint to get information about owners and public to the list.
Create Region Client Side code.  I think I could modify the existing Region button to have an option for 'Add Region' that allows a user to specify a region with name and the public setting.
User regions are filters near the top.

New Request:
Add the ability to export and download the Region Polygon in geoJSON format.


Janitor Monkey Email Tags

Janitor Monkey's emails just include instance IDs, which are relatively opaque and are too easily overlooked by users.
It would be much more useful if the Janitor Monkey notification emails included a list of all tags and their values for each instance, or a configurable list of tags.

This is a good suggestion and would be pretty straightforward to add if you want to do it yourself.
In BasicJanitorEmail, modify TABLE_COLUMNS to add a column for tags then in BasicJanitorEmailBuilder.getResourceRow, add logic to loop through and output tags.
At least for the first implementation, configurable tags is not necessary.  Although it probably makes sense to limit the output to the first 10 tags or similar just in case a resource has a million tags it wont completely destroy the email formatting.
This would actually add it for every resource type, not just instances, which makes sense to me.

@ebukoski This is as easy as it seems.  BasicJanitorEmailBuilder is using a resource object that was built from the contents of the SimpleDB data, not the resource object built by the crawler.
So either, we can 1) have BasicJanitorEmailBuilder use an AWSClient to re-query for the tags of the resources or we can 2) store a list of tags in SimpleDB.
Do you have a preference?

At first glance I would prefer to store in SimpleDB as the tags are readily available from the crawler.  But I am worried about the size involved and also the potential staleness of the tags as over time they might not reflect current tags/values on the resource.
But I'm not liking the idea of re-querying before sending the email.  It does easily solve both of those issues.  But now sending emails requires hundreds/thousands more AWS queries.  Yuck.
So maybe a property that defines the tags to include in the email and we store those in SimpleDB.  It would control the number of tags stored, avoid AWS queries at send email time, and while it doesn't prevent staleness it at least brings some visibility to which tags are being passed through the email.


List not working: File created but didn't work on VLC

Hi. I successfully  created the list. So far so good. But the list is not working in vlc.
I'm getting:
"Your input can't be opened: VLC is unable to open the MRL..."

It's generating URLs for viewstvn instead of viewms. The auth token is correct but it's trying to use it for the wrong site.

What do you mean? you mentioned two different things, the auth token and the server. Also I enter as a parameter viewms.

When you enter your information into the script it should write a URL for each channel that contains an auth token, server, the site name and a link for that channel. The script is doing that BUT it appears to be entering the site as "viewstvn" (StreamTVNow) even though you've entered "viewms" (MyStreams). So the auth token is correct but it's being used on the wrong site. There's an error in the script.

Hi guys,
Thanks for all the details.  I didn't full implement the handling of different hosts in the last update - I apologize for the inconvenience!  It's a quick fix and I'll have it done this morning.
Thanks @dbowles1975 for stepping up to help out!  Cheers 🍻


Curious why store is passed in here

I noticed in this method, the store is passed in as an argument, and isn't being used?
https://github.com/davezuko/react-redux-starter-kit/blob/master/src/routes/index.js#L12
Am I missing something?
Thanks! Still digging through your setup and really liking it.

Routes may need to access store, for example to dynamically load a reducer or to check authentication state in a Route.onEnter handler

@justingreenberg beat me to it.

Ah, so you could pass it in to a component as a prop?

Most common use case is to use it in route hooks (e.g. onEnter) in order to perform some action based on the store state (i.e. redirecting if not logged in).

Makes sense. Thanks!!

No problem :)


Don't override user's env var for AWS profile

This broke the --profile option, so tf -p <profile> no longer has any effect.
Do we want to just get rid of that option entirely?


Unable to save chart

Hi,
I am saving svg's using altair_saver in my code, and up until a few minutes ago, everything was fine. But now I am getting the following error message:

No enabled saver found that supports format='svg'.

Is there anything broken?

You'll have to make certain that altair_saver is installed, as well as either the selenium dependencies or the nodejs dependencies. See https://github.com/altair-viz/altair_saver#installation

Hey Jake,
I do have selenium installed, and altair_saver. Everything was going well until an hour ago for some reason. I had been using altair_saver since last weekend without issues.
Here's the error message I get:

File "/xxx/lib/python3.7/site-packages/altair_saver/_core.py", line 169, in save
Saver = _select_saver(method, mode=mode, fmt=fmt, fp=fp)
File "/xxx/lib/python3.7/site-packages/altair_saver/_core.py", line 69, in _select_saver
raise ValueError(f"No enabled saver found that supports format={fmt!r}")
ValueError: No enabled saver found that supports format='svg'

Something on your system must have changed (chromedriver, maybe?) WHat happens if you run this?
from selenium.webdriver import Chrome
driver = Chrome()

Indeed:

selenium.common.exceptions.SessionNotCreatedException: Message: session not created: This version of ChromeDriver only supports Chrome version 81

My chromedriver version is 81.0.4044.69... but my Chrome version is 81.0.4044.138. I guess the difference in subversion matters? [Going to try updating chromedriver to the latest version and see if that fixes it].

That fixed it! Thanks Jake and the team. I really enjoy using Altair :)