Permissions to Kits not working

Some time ago, I've encountered a bug on my server, where player's where unable to take their kits. They had no permission to take them, but admins had. I've checked the permission file - everything was fine (To take a kit x I've added the permission kit.x). Then I've checked the dev.rocketmod.net Kits repository, where everybody was screaming that the plugin was broken.
Alright, then I've searched the sources myself and found the potential bug location. The code seems OK to me, but I have a clue why it isn't working.
I think it's this method, which is (what I think) legacy code and should be replaced with a proper AllowedCaller implementation.
Maybe I'm completely wrong, too. I'll test in a day or two, and create a pull request if it works. If not, I hope you will get more success with your research.
Best regards,
nickkoro

i have the same problem, let me know if u fixed it

We don't update these anymore (in fact I'd personally recommend using OpenMod instead), if you do manage to fix this, feel free to PR.

hello can any 1 help me with this problem?
OverflowException: Value was either too large or too small for an Int32.
at System.Number.ParseInt32 (System.String s, System.Globalization.NumberStyles style, System.Globalization.NumberFormatInfo info) [0x00057] in <9577ac7a62ef43179789031239ba8798>:0
at System.Int32.Parse (System.String s, System.Globalization.NumberStyles style, System.IFormatProvider provider) [0x0000e] in <9577ac7a62ef43179789031239ba8798>:0
at System.Xml.XmlConvert.ToInt32 (System.String s) [0x00007] in <1d98d70bb7d8453b80c25aa561fdecd1>:0
at Microsoft.Xml.Serialization.GeneratedAssembly.XmlSerializationReaderKitsConfiguration.Read6_NullableOfInt32 (System.Boolean checkType) [0x00027] in <48040cf479854ce499b8aff7dcbfdd30>:0
at Microsoft.Xml.Serialization.GeneratedAssembly.XmlSerializationReaderKitsConfiguration.Read7_Kit (System.Boolean isNullable, System.Boolean checkType) [0x0043d] in <48040cf479854ce499b8aff7dcbfdd30>:0
at Microsoft.Xml.Serialization.GeneratedAssembly.XmlSerializationReaderKitsConfiguration.Read8_KitsConfiguration (System.Boolean isNullable, System.Boolean checkType) [0x00235] in <48040cf479854ce499b8aff7dcbfdd30>:0
at Microsoft.Xml.Serialization.GeneratedAssembly.XmlSerializationReaderKitsConfiguration.Read9_KitsConfiguration () [0x00050] in <48040cf479854ce499b8aff7dcbfdd30>:0
at (wrapper managed-to-native) System.Reflection.MonoMethod.InternalInvoke(System.Reflection.MonoMethod,object,object[],System.Exception&)
at System.Reflection.MonoMethod.Invoke (System.Object obj, System.Reflection.BindingFlags invokeAttr, System.Reflection.Binder binder, System.Object[] parameters, System.Globalization.CultureInfo culture) [0x00032] in <9577ac7a62ef43179789031239ba8798>:0
Rethrow as InvalidOperationException: There is an error in XML document (19, 7).
at System.Xml.Serialization.XmlSerializer.Deserialize (System.Xml.XmlReader xmlReader, System.String encodingStyle, System.Xml.Serialization.XmlDeserializationEvents events) [0x0011b] in <1d98d70bb7d8453b80c25aa561fdecd1>:0
at System.Xml.Serialization.XmlSerializer.Deserialize (System.Xml.XmlReader xmlReader, System.String encodingStyle) [0x00000] in <1d98d70bb7d8453b80c25aa561fdecd1>:0
at System.Xml.Serialization.XmlSerializer.Deserialize (System.IO.TextReader textReader) [0x0001c] in <1d98d70bb7d8453b80c25aa561fdecd1>:0
at Rocket.Core.Assets.XMLFileAsset1[T].Load (Rocket.API.AssetLoaded1[T] callback) [0x00026] in :0
Rethrow as Exception: Failed to deserialize XMLFileAsset: Plugins/Kits/Kits.configuration.xml
at Rocket.Core.Assets.XMLFileAsset1[T].Load (Rocket.API.AssetLoaded1[T] callback) [0x00073] in :0
at Rocket.Core.Assets.XMLFileAsset1[T]..ctor (System.String file, System.Type[] extraTypes, fr34kyn01535.Kits.KitsConfiguration defaultInstance) [0x0002a] in <de2b2fb669424d2dae1c4b01e1864fe9>:0 at Rocket.Core.Plugins.RocketPlugin1[RocketPluginConfiguration]..ctor () [0x000ad] in :0
at fr34kyn01535.Kits.Kits..ctor () [0x00000] in <427e31678a3d487281f1fd370abd75ac>:0
UnityEngine.GameObject:Internal_AddComponentWithType(Type)
UnityEngine.GameObject:AddComponent(Type)
UnityEngine.GameObject:.ctor(String, Type[])
Rocket.Core.Plugins.RocketPluginManager:loadPlugins()
Rocket.Core.Plugins.RocketPluginManager:Start()
(Filename: <9577ac7a62ef43179789031239ba8798> Line: 0)
NullReferenceException: Object reference not set to an instance of an object
at Rocket.Core.Plugins.RocketPlugin`1[RocketPluginConfiguration].LoadPlugin () [0x00012] in :0
at Rocket.Core.Plugins.RocketPlugin.OnEnable () [0x00000] in :0
UnityEngine.GameObject:Internal_AddComponentWithType(Type)
UnityEngine.GameObject:AddComponent(Type)
UnityEngine.GameObject:.ctor(String, Type[])
Rocket.Core.Plugins.RocketPluginManager:loadPlugins()
Rocket.Core.Plugins.RocketPluginManager:Start()
(Filename:  Line: 0)

hello can any 1 help me with this problem?
OverflowException: Value was either too large or too small for an Int32.
at System.Number.ParseInt32 (System.String s, System.Globalization.NumberStyles style, System.Globalization.NumberFormatInfo info) [0x00057] in <9577ac7a62ef43179789031239ba8798>:0
at System.Int32.Parse (System.String s, System.Globalization.NumberStyles style, System.IFormatProvider provider) [0x0000e] in <9577ac7a62ef43179789031239ba8798>:0
at System.Xml.XmlConvert.ToInt32 (System.String s) [0x00007] in <1d98d70bb7d8453b80c25aa561fdecd1>:0
at Microsoft.Xml.Serialization.GeneratedAssembly.XmlSerializationReaderKitsConfiguration.Read6_NullableOfInt32 (System.Boolean checkType) [0x00027] in <48040cf479854ce499b8aff7dcbfdd30>:0
at Microsoft.Xml.Serialization.GeneratedAssembly.XmlSerializationReaderKitsConfiguration.Read7_Kit (System.Boolean isNullable, System.Boolean checkType) [0x0043d] in <48040cf479854ce499b8aff7dcbfdd30>:0
at Microsoft.Xml.Serialization.GeneratedAssembly.XmlSerializationReaderKitsConfiguration.Read8_KitsConfiguration (System.Boolean isNullable, System.Boolean checkType) [0x00235] in <48040cf479854ce499b8aff7dcbfdd30>:0
at Microsoft.Xml.Serialization.GeneratedAssembly.XmlSerializationReaderKitsConfiguration.Read9_KitsConfiguration () [0x00050] in <48040cf479854ce499b8aff7dcbfdd30>:0
at (wrapper managed-to-native) System.Reflection.MonoMethod.InternalInvoke(System.Reflection.MonoMethod,object,object[],System.Exception&)
at System.Reflection.MonoMethod.Invoke (System.Object obj, System.Reflection.BindingFlags invokeAttr, System.Reflection.Binder binder, System.Object[] parameters, System.Globalization.CultureInfo culture) [0x00032] in <9577ac7a62ef43179789031239ba8798>:0
Rethrow as InvalidOperationException: There is an error in XML document (19, 7).
at System.Xml.Serialization.XmlSerializer.Deserialize (System.Xml.XmlReader xmlReader, System.String encodingStyle, System.Xml.Serialization.XmlDeserializationEvents events) [0x0011b] in <1d98d70bb7d8453b80c25aa561fdecd1>:0
at System.Xml.Serialization.XmlSerializer.Deserialize (System.Xml.XmlReader xmlReader, System.String encodingStyle) [0x00000] in <1d98d70bb7d8453b80c25aa561fdecd1>:0
at System.Xml.Serialization.XmlSerializer.Deserialize (System.IO.TextReader textReader) [0x0001c] in <1d98d70bb7d8453b80c25aa561fdecd1>:0
at Rocket.Core.Assets.XMLFileAsset1[T].Load (Rocket.API.AssetLoaded1[T] callback) [0x00026] in :0
Rethrow as Exception: Failed to deserialize XMLFileAsset: Plugins/Kits/Kits.configuration.xml
at Rocket.Core.Assets.XMLFileAsset1[T].Load (Rocket.API.AssetLoaded1[T] callback) [0x00073] in :0
at Rocket.Core.Assets.XMLFileAsset1[T]..ctor (System.String file, System.Type[] extraTypes, fr34kyn01535.Kits.KitsConfiguration defaultInstance) [0x0002a] in <de2b2fb669424d2dae1c4b01e1864fe9>:0 at Rocket.Core.Plugins.RocketPlugin1[RocketPluginConfiguration]..ctor () [0x000ad] in :0
at fr34kyn01535.Kits.Kits..ctor () [0x00000] in <427e31678a3d487281f1fd370abd75ac>:0
UnityEngine.GameObject:Internal_AddComponentWithType(Type)
UnityEngine.GameObject:AddComponent(Type)
UnityEngine.GameObject:.ctor(String, Type[])
Rocket.Core.Plugins.RocketPluginManager:loadPlugins()
Rocket.Core.Plugins.RocketPluginManager:Start()
(Filename: <9577ac7a62ef43179789031239ba8798> Line: 0)
NullReferenceException: Object reference not set to an instance of an object
at Rocket.Core.Plugins.RocketPlugin`1[RocketPluginConfiguration].LoadPlugin () [0x00012] in :0
at Rocket.Core.Plugins.RocketPlugin.OnEnable () [0x00000] in :0
UnityEngine.GameObject:Internal_AddComponentWithType(Type)
UnityEngine.GameObject:AddComponent(Type)
UnityEngine.GameObject:.ctor(String, Type[])
Rocket.Core.Plugins.RocketPluginManager:loadPlugins()
Rocket.Core.Plugins.RocketPluginManager:Start()
(Filename:  Line: 0)


Fix broken ci

Forced CI/CD pipeline to use gradle version 7.3.2 during build process.

Hi @dashrath-chauhan. I noticed that the base branch of this PR is set to "main." I've made an adjustment to change the base branch to "develop".
Thank you for updating the Gradle version. However, since we've encountered some Github Actions workflow errors after the Gradle version update, I would suggest fixing them before merging this PR so we are sure that the bug is addressed.

Hi @yash-puligundla, other tests/errors are fixed now, and ready to merge.


Fixed #936 (https://github.com/elastic/puppet-elasticsearch/issues/936)

Fixes: ttps://github.com/elastic/puppet-elasticsearch/issues/936
I'm not at all sure if expanding the glob pattern is the way to go, but at least for me it fixed the issue.

Hey @gellweiler, thanks for the fix! I had the fix nearly read to go in another branch with a few additional checks for failure conditions, so your fix along with some additional ones I've made are here: #941
Once that passes CI and gets merged in I can cut a release to fix the issue. Cheers!


Migrate workload with placement

Ref:
https://github.com/open-cluster-management-io/ocm/issues/60
https://github.com/open-cluster-management-io/ocm/issues/61

/assign @qiujian16 @mikeshng

Hello @mikeshng , in this scenario I use the ArgoCD pull model as an example, to show how we can migrate
the Applications among clusters by Placement taint & toleration. Could you help take a review from ArgoCD integration's view, to see if it makes sense, and any comments?

@mikeshng would you like to take a look at this one?

@mikeshng thanks for the review, have accepted all the comments and synced changes to zh/ folder.


Tagging an untagged domain through review adds 2-3 same tags

Trying to tag a domain in untagged domains review (e.g. want to tag #notspam) shows:

The domain has one tag, though:

Part of the general JS fustercluck. Reload your page when this starts happening and it usually stops. Otherwise [status-declined] because I've tried to fix JS before and it's never worked.


insar_tops_burst fails with Failed to connect to download.agisoft.com

hyp3_isce2.insar_tops_burst, and all INSAR_ISCE_BURST jobs submitted through HyP3 since 2024-06-27T00:00:00Z, are failing with the following stacktrace during DEM preparation:
Traceback (most recent call last):
  File "<frozen runpy>", line 198, in _run_module_as_main
  File "<frozen runpy>", line 88, in _run_code
  File "/opt/conda/envs/hyp3-isce2/lib/python3.11/site-packages/hyp3_isce2/__main__.py", line 51, in <module>
    main()
  File "/opt/conda/envs/hyp3-isce2/lib/python3.11/site-packages/hyp3_isce2/__main__.py", line 47, in main
    sys.exit(process_entry_point.load()())
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/conda/envs/hyp3-isce2/lib/python3.11/site-packages/hyp3_isce2/insar_tops_burst.py", line 530, in main
    insar_tops_burst(
  File "/opt/conda/envs/hyp3-isce2/lib/python3.11/site-packages/hyp3_isce2/insar_tops_burst.py", line 116, in insar_tops_burst
    dem_path = download_dem_for_isce2(dem_roi, dem_name='glo_30', dem_dir=dem_dir, buffer=0, resample_20m=False)
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/conda/envs/hyp3-isce2/lib/python3.11/site-packages/hyp3_isce2/dem.py", line 112, in download_dem_for_isce2
    dem_array, dem_profile = dem_stitcher.stitch_dem(
                             ^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/conda/envs/hyp3-isce2/lib/python3.11/site-packages/dem_stitcher/stitcher.py", line 358, in stitch_dem
    dem_arr, dem_profile = merge_and_transform_dem_tiles(
                           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/conda/envs/hyp3-isce2/lib/python3.11/site-packages/dem_stitcher/stitcher.py", line 197, in merge_and_transform_dem_tiles
    dem_arr = remove_geoid(
              ^^^^^^^^^^^^^
  File "/opt/conda/envs/hyp3-isce2/lib/python3.11/site-packages/dem_stitcher/geoid.py", line 72, in remove_geoid
    geoid_arr, geoid_profile = read_geoid(geoid_name, extent=list(extent), res_buffer=res_buffer)
                               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/conda/envs/hyp3-isce2/lib/python3.11/site-packages/dem_stitcher/geoid.py", line 40, in read_geoid
    geoid_arr, geoid_profile = read_raster_from_window(geoid_path, extent, extent_crs, res_buffer=res_buffer)
                               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/conda/envs/hyp3-isce2/lib/python3.11/site-packages/dem_stitcher/rio_window.py", line 158, in read_raster_from_window
    with rasterio.open(raster_path) as ds:
         ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/conda/envs/hyp3-isce2/lib/python3.11/site-packages/rasterio/env.py", line 451, in wrapper
    return f(*args, **kwds)
           ^^^^^^^^^^^^^^^^
  File "/opt/conda/envs/hyp3-isce2/lib/python3.11/site-packages/rasterio/__init__.py", line 304, in open
    dataset = DatasetReader(path, driver=driver, sharing=sharing, **kwargs)
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "rasterio/_base.pyx", line 310, in rasterio._base.DatasetBase.__init__
rasterio.errors.RasterioIOError: CURL error: Failed to connect to download.agisoft.com port 80 after 129539 ms: Couldn't connect to server

Complete log file: b887ed80-d679-4ab1-ad1e-f7ffb9180908.log
hyp3_isce2 relies on dem-stitcher, which retrieves Geoid models from agisoft.com. The URLs for these data files recently changed, causing processing to fail. The underlying issue is being tracked at https://github.com/ACCESS-Cloud-Based-InSAR/dem-stitcher/issues/88

@ZGHHGZ thanks for the report! I've updated the issue to fully describe the situation.
We hope to have a fix in place next week, but in the meantime INSAR_ISCE_BURST jobs submitted to HyP3 will continue to fail. INSAR_GAMMA jobs are unaffected and should process as normal.


Update parse VS parseBlock documentation

I originally had confusion on .parse returning an array and not a string of ALL the html. Hope this helps someone in the future.

Thanks @groteworld for making it more awesome. :heart:


Map message type to MQTT topic

Per discussed in Discord.
As I want to send commands from Node Red to a specific IoT device. Since I do not have a Wolverine envelope there's no way Wolverine knows what kind of data that I'm sending to it, thus it cannot execute.
Your proposal: to map a certain topic to a message type, we could send messages to that specific topic and Wolverine knows how to execute.

Turns out is was already there, but not documented.

Turns out is was already there, but not documented.

Could you help me out with the syntax?


Improve the order of the menu items

Before

After

I feel like this is a better order. It's also makes it the same as the Kap menu in the editor and also makes it easier to reach "Preferences…" in the cropper:

We have two more items coming for that menu. One is Clear recordings or something similar that deletes our temp files, and then a Enable audio recording which is a radio shortcut to the preferences toggle. Where do you think those should be in that list?

Maybe after "Export History"?

Clearing recordings may not need to be in the list, but rather grouped alongside options to reset preferences and plugin configurations near the bottom of the Preferences window itself?


Validate output conversion from Rust to R?

While we now can use use_try_from = true for type-checking, TryFrom is implemented one-way (R to Rust), if I understand correctly
I'm wondering what should happen in this case.
rextendr::rust_function(r"(
fn f() -> i64 {
  i64::MAX
}
)",
  patch.crates_io = list(`extendr-api` = list(git = "https://github.com/extendr/extendr")),
)
#> ℹ build directory: '/tmp/RtmpR1VmHe/file161681b33c70a'
#> ✓ Writing '/tmp/RtmpR1VmHe/file161681b33c70a/target/extendr_wrappers.R'.

f()
#> [1] -1

Created on 2021-07-23 by the reprex package (v2.0.0)

This is definitely incorrect.
It looks like it is truncating the i64 value to i32.
The result should be 263 as it is the closest
f64 value to 263-1. But we may also refuse to
convert as 2**63-1 is not representable and generate
a NA.

The conversion to r objects is done via the ToNumericValue trait.
The i64 and u64 implementations should return REALSXP instead of INTSXP.
Definitely a bug.

impl_integer_tvv!(i8);
impl_integer_tvv!(i16);
impl_integer_tvv!(i32);
impl_integer_tvv!(i64); <- should be to_real_tvv!(i64)
impl_integer_tvv!(u8);9
impl_integer_tvv!(u16);
impl_integer_tvv!(u32);
impl_integer_tvv!(u64); <- should be to_real_tvv!(U64)

Thanks for taking a look, I got it. I guess u32 is also the case.

Yes, good thinking.
We do do usize already as a real.


[YUNIKORN-2589] Convert AllocationID to AllocationKey

What is this PR for?
Remove references to AllocationID, and replace with AllocationKey where appropriate as the REST API no longer produces objects with AllocationID fields.
What type of PR is it?

[ ] - Bug Fix
[ ] - Improvement
[ ] - Feature
[ ] - Documentation
[ ] - Hot Fix
[ ] - Refactoring

Todos

[x] - Task

What is the Jira issue?
https://issues.apache.org/jira/browse/YUNIKORN-2589
How should this be tested?
Screenshots (if appropriate)
Questions:

[ ] - The licenses files need update.
[ ] - There is breaking changes for older versions.
[ ] - It needs documentation.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 38.77%. Comparing base (a7a8362) to head (64bd150).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #190   +/-   ##
=======================================
  Coverage   38.77%   38.77%
=======================================
  Files           2        2
  Lines          49       49
=======================================
  Hits           19       19
  Misses         27       27
  Partials        3        3

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


Make moveit_config compatible to moveit_configs_builder

As mentioned in #917 our current config is not compatible to the moveit_configs_builder.
This PR aims at making our config compatible with it.
The current state is a working version which allows loading the description from the robot_state_publisher's topic and can be used in the moveit_configs_builder
Things still need to be done

[x] Lazy starting of move_group. Currently, it crashes when the description topic isn't yet published
[ ] Add upstream issue about the crash when no description topic is provided
[x] Add moveit_servo

Things that significantly change in this PR:

the tf_prefix has been removed. It didn't work correctly, anyway. Since there are joints inside config files such as the joint_limits.yaml and they to my knowledge cannot handle argument substitution during loading inside the configs builder, this doesn't work.

Things suggested in https://github.com/UniversalRobots/Universal_Robots_ROS2_Driver/issues/917#issuecomment-2100517142 I did not implement

I did not add different IK solutions, as it seems not to be possible to pass them to the configs builder dynamically. As this is merely an example repo, I'd rather not add this. Users can still define their own when they implement their own launchfiles with the configs_builder in them.

Unorthodox perhaps, but what about not using the config builder at all and creating a more ROS-1-like MoveIt config pkg? Tyler made a nice example showing how to do this, right around the time I was also looking into it.
See ROSCON 2023 - Retro ROS 2 Launch.

Thank you for the feedback @gavanderhoorn!
We have considered for quite a while now whether we want to migrate our config package or not. Personally, I am also not the greatest fan of the way the configs builder hides the required config files away. Though once looking at its source code it became less hidden for me, so a better documentation could potentially improve things here. In my opinion there are a couple of arguments that justify making this step:

configs-builder seems to be the standard used by MoveIt! nowadays. Of course, that doesn't mean that we'll have to follow that and we do a couple of things differently than a standard moveit_config package would, but it might help new users to get familiar with the structure more easily. And I expect that package to be used by less experienced MoveIt! users anyway, as more experienced users would probably create their own moveit_config package rightaway.
Reusing this to write your own launchfiles launching your custom nodes should be much more convenient, especially if you are used to using the configs builder. The main point about this PR is changing the structure, not necessarily using the configs builder in our own launchfiles.  That's also (from my understanding) what the OP in #917 was expecting.
Tyler's package I think is also from a time where the configs builder didn't exist, yet. The launchfiles become much less cluttered once the description is re-used from the description topic.
Using one big config file for everything isn't necessarily reducing complexity in my opinion.

I decided against an upstream issue regarding the description topic as it seems intentional to have a default of 10 seconds timeout.


Refactor how you choose weapons for default loadouts

Currently you are able to drag a singular item into the side bar.
It should be that when you drag an item into the sidebar, the item will stay where it is in the mainContainer but duplicate itself in the side bar. The item in the mainContainer will then be greyed out which represents that it has been included in the side bar selection.
It should also allow you to simply click the item to add it to the side bar selection instead of dragging it.
When an item is added to the side bar, add a div that represents a remove, in the form of an (x) icon; in the top right.

Weapons are also to receive a light level number.

Closing this for now as default loadouts will not be a part of synergy for a while yet.


Install dependencies for http.git only if the plugin is installed.

Supposing you don't want to use the git plugin and want create a custom build without any plugin:
docker build . --build-arg plugins= --tag test/caddy --no-cache
Against the default compilation:
docker build . --build-arg 'plugins=git' --tag test/caddy-git --no-cache
Final result will be:
REPOSITORY                     TAG                 IMAGE ID            CREATED             SIZE
test/caddy                     latest              4af2c986d74f        5 minutes ago       23.9MB
test/caddy-git                 latest              dd9f4e366df0        5 minutes ago       41.9MB

This will help to reduce the final image size.
I hope you like it.
Cheers.

You should add RUN apk add --no-cache ca-certificates, I think it is required for https and caddy.
Installing openssh-client installs it but now that openssh-client is becoming optional, we should install that.
Also, I'd be interested to know if the space gain is still similar.

I forgot about it, I saw so many dockerfiles that I assumed that ours have it too 😝
test/caddy                     latest              18e889092d59        2 minutes ago       24.4MB
Even after the ca-certificates, it is still pretty good.

Any news on this?


Simple DNS A record dicsovery

Having assumed an overlay network, one can simply use default ports and lookup cluster members as DNS A record, without having to use SRV. This would make etcd extremely easy to deploy (and scale) on Weave.

I don't follow. Can you provide an example.

As an alternative to current SRV discovery...

Well, why can't you do this today by specifying (resolvable-by-DNS) hostnames instead of IP addresses?

Sure, but what I am talking about is A records with multiple answers, like
the SRV that is currently supported.
On Mon, 9 Nov 2015 05:06 Jonathan Boulle<EMAIL_ADDRESS>wrote:

Well, why can't you do this today by specifying (resolvable-by-DNS)
hostnames instead of IP addresses?
—
Reply to this email directly or view it on GitHub
https://github.com/coreos/etcd/issues/3832#issuecomment-155056690.

@errordeveloper Why not just SRV? DNS A record is not designed for the purpose you want to achieve here I think.

Well, SRV is only needed when port number has to be discovered, if default
ports are okay to use (like on Weave Net), there is no need to use SRV.
On Thu, 21 Apr 2016, 19:29 Xiang Li<EMAIL_ADDRESS>wrote:

@errordeveloper https://github.com/errordeveloper Why not just SRV? DNS
A record is not designed for the purpose you want to achieve here I think.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/coreos/etcd/issues/3832#issuecomment-213054332

My question is why SRV is difficult for you but A record is not? More steps?

I do not see a clear need here. We do not want to add endless discovery mechanisms into core etcd.
We are considering adding etcd init and etcd dicsovery subcommand. So users can easily add whatever discovery mechanism they want into etcd subcommands.

@errordeveloper https://github.com/coreos/etcd/issues/5208


Adopt new wallet core Ethereum proto change

https://github.com/trustwallet/wallet-core/pull/1210

Fixed in https://github.com/trustwallet/TrustSDK-iOS/pull/73

Fixed in https://github.com/trustwallet/TrustSDK-iOS/pull/73


Use constants from libc where they exist

Hi. I'm writing a wrapper for libnftnl in Rust to be able to control the Linux firewall. To do that I need to talk netlink. Since libnftnl uses libmnl for that natively I could just write a wrapper for libmnl as well. But the more pure-rust I can use the better was my thought. And it also feels like there are already too many libraries duplicating the NlFamily enum. So I looked for pure Rust netlink libraries.
After looking at yours for a while I see that many of the constants you declare are already in libc. So I thought I could try to make it use them instead. I don't know if you want this change or not, but it reduces duplication at least. We could quite likely get the remaining constants merged into libc as well, I have had constants from for example <linux/netfilter/*> merged before.
I had to do a little trick with the macro there to get rid of unnecessary unsafe warnings since there are now both safe and unsafe usages of that macro. But it turned out quite OK. Can be simplified again if all constants become safe as well.

@faern This seems to make a lot of sense for now. I will work on getting the rest merged into libc. Given that I'm already using libc for socket code, this would require no additional deps so I'm inclined to eliminate unnecessary C deps as well. I will see if I can get netlink constants merged into libc soon. If it is merged quickly, I will make some edits and switch them all over to libc. If it takes them a bit to get to my PR, I'll review and merge then switch over to libc once my PR is accepted. Thanks for the suggestion and PR!

Nice! Awesome crate by the way. Would be really nice to have comprehensive Netlink support in Pure rust without too many dependencies.

@faern rust-lang/libc#921 is now open. After looking at your proposed changes, I have a few more changes that I'd like to make on top of yours so I think I will merge and push some more changes then put out the next release on crates.io. I'll keep checking on that issue and will ping you once that process gets moving so we can hopefully get that C dependency out in future releases. Thanks again!


volana failed and exited for some error

./volana
volana » whoami
kali
volana » cat /etc/shadow
cat: /etc/shadow: Permission denied
2022/10/11 01:16:43 Run command failed with exit status 1

how to prevent volana from exited if error occur?

Should be ok with the last version of
https://github.com/ariary/volana/blob/main/pkg/command/command.go
Try to recompile it

IMO, these tools are not magic and are just some little tools to help you hide your post exploitation. The goal is to pass under the radar of several detection mechanisms and also to make forensic complicated
The interesting section is https://github.com/ariary/volana#hide-from to see when volana can be effecient


Add missing permissions for<EMAIL_ADDRESS>to deployer service account

/kind bug
What this PR does / why we need it:
With https://github.com/gardener/ci-infra/pull/717 the deployer service account of the trusted cluster requires additional permissions for metrics.k8s.io.
Before they have been granted, the deploy job failed.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:

/remove-lifecycle stale


Title of the LoA not at the correct height

Description
The position of the title of the LoA is a bit too low (in comparison to the titles of ToC and LoF).
How to reproduce
Steps to reproduce the behavior:

You can see the difference when reading the PDF: starter-recommended.pdf
(The view should be adjusted to the size of a complete page.)

Screenshots

This is apparently also the case for Sperrvermerk, Anhang # and Selbstständigkeitserklärung.


Chrome Canary 33 requires a hardcoded snippet

I'm running Chrome Canary 33.0.1736.2. Which has introduced an interesting bug. Until this version (maybe a few versions previous), the following snippet worked for kicking off livereload:
<script>
(function(d,t){
var g=d.createElement(t),s=d.getElementsByTagName(t)[0];
g.src='//' + window.location.hostname + ':' + (parseInt(window.location.port, 10) + 2) + '/livereload.js';
s.parentNode.insertBefore(g,s)}(document,'script')
);
</script>

No longer. Now, only the following works:
  <script src="//localhost:8082/livereload.js"></script>

is it still an issue now?

Hello @te1 @joeybaker
Like @jrobeson asked, is it still an issue ?
Closing for now, feel free to reopen if it's still an issue for you.


Rails 5 compatibility (rails/test_unit/railtie and ActionController::TestRequest.create)

Hi there, Happy New Year to y'all!
Together with the suggestion in https://github.com/drapergem/draper/issues/697 of adding gem 'activemodel-serializers-xml', git: 'https://github.com/rails/activemodel-serializers-xml' to my Gemfile these to commits/changes made draper work on rails v5.0.0.beta1.
I suppose this being merged would also fix https://github.com/drapergem/draper/issues/698 and https://github.com/drapergem/draper/issues/681, both of which I sadly just discovered... (poor me ;))
PS:
I tried to set this up to actually run the test suite against the Rails beta on TravisCI, but did not quiet succeed. Have a look at https://github.com/janraasch/draper/commit/92b1b1e8dac19508689eb19052764634a76ddee1 to see where I left off. The main remaining problems are the incompatibilities of required gems in mongoid and minitest-rails.

Hello!
I've been trying to fix build for Rails 5 too.
Here where I'm stuck, mongoid dependency.
https://github.com/drapergem/draper/compare/master...Mehonoshin:fix-specs

Hi, thanks for the PR! In #752 I've put together all the changes necessary to get Draper working with Rails 5. It would be appreciated if you could test it out in your app and report any issues you find :octocat:

Thank you, @seanlinsley. Will do.


Create complex.c

I have successfully submitted my code

Create pull request for README.md file with link to your solution .


Fix RST error in README.rst

This will make the PyPI page for sh look pretty again.
This includes a test to make it harder to accidentally cause this kind of error again.

@Flimm so much nicer, good work.  Anonymous rst hyperlinks..learned something new today.  Also thank you for the test :+1:


Counting concurrently

Given a simple Python script that calls ioctl() with garbage (so that it sticks out):
import fcntl

while 1:
    try:fcntl.ioctl(76542,765432)
    except Exception: pass

And a bpftrace script using every available means to count those events:
tracepoint:syscalls:sys_exit_ioctl
    /args->ret < 0/
{
    @runscalar = @runscalar + 1;
    @runcpu[cpu] = count();
    @runcount = count();
}

interval:s:1
{
    printf("\n\n\n");
    printf("cpu %d\n", cpu);
    print(@runscalar);
    print(@runcount);
    print(@runcpu);
    clear(@runscalar);
    clear(@runcount);
    clear(@runcpu);
}

On an 8 thread machine, we see that:

With a single thread, all counts appear to more or less agree - roughly 500k calls/sec.
With 2 threads, runscalar and runcount mostly agree that roughly 800k calls/sec are happening, however runcount regularly deviates by 2x or more, possibly due to clear() racing with another CPU in the process of updating the value
With 2 threads, runcpu appears to suffer from the same problem -- occasionally twice as many events are reported (seemingly unrelated to interval delay)
With 2 threads, the problem appears to snowball until it is reporting millions of extra events over many reporting intervals
With 3 threads, runscalar indicates 900k calls/sec, and seems to be far off the expected 1.5m calls/sec
With 3 threads after after 2 consecutive intervals that fired on time, runcpu can be seen with one bucket having double as many events as reported by the others

Example of all counts looking suspect:
ms: 1000, cpu 0
@runscalar: 1006754
@runcount: 1221697
@runcpu[6]: 425505
@runcpu[2]: 818408
@runcpu[0]: 1947663

ms: 999, cpu 0
@runscalar: 934129
@runcount: 1597779
@runcpu[6]: 393040
@runcpu[2]: 804994
@runcpu[0]: 1945031

I looked under the hood and saw per-cpu hash tables are used for all variables, although I didn't get as far as understanding how those tables are polled, summed and cleared. The expectation would be that print() could safely read and sum these per-cpu variables, but clear() and zero() might have a much harder time avoiding concurrent work modifications made by other CPUs.
I have no idea which, if any of these counts are accurate.  It looks like in any case, clear() is racing with the running probe, which has a temporary copy of the value loaded for incrementing?
At least in my mind, it seems impossible to provide a running counter like this with concurrent updates without the ability to somehow snapshot a table over reporting intervals, and calculate the difference from the previous snapshot, unless some other magic is going on under the hood. Otherwise there will always be a race, and overreporting due to multiple unsynchronized writers trying to increment or clear the buckets involved.
Is a variable copy feature something that might worth be looking at? Which of these mechanisms is safe and recommended? Thanks

In the process of hunting for another variant, I found yet more strange behaviour:
BEGIN {
    $ii = 0;
    unroll(8) {
        @count[$ii] = 0;
        $ii = $ii + 1;
    }
}

tracepoint:syscalls:sys_enter_ioctl
    /args->fd == 76542/
{
    @count[cpu] = @count[cpu] + 1;
}

interval:s:1
{
    $acc = 0;
    $i = 0;
    unroll(8) {
        $acc = $acc + @count[$i];
        $i = $i + 1;
    };
    print(@count);
    printf("%ld\n", $acc);

    //printf("%d\n", $acc - @last_acc);
    @last_acc = $acc;
}

$acc has a garbage value unless the print(@count) is moved above the unroll().

Given that this is 6 years old, I'm sure all the context is gone and perhaps bugs that were there are now fixed or obsolete. However, I think this can be closed as my read is that this is a result of UX issues over anything else (@jordalgo).
My take is that there are two fundamental issues here:

First, obviously the two probes are fundamentally racy and clobbering each other. This is why you have the runscalar and runcount variability and why it is difficult to reason about what is correct.
Second, the much more fundamental issue is that maps are printed by reference and the scalar values are copied by value. So they may be printed at any moment, before or after the delete and will change (unlike the scalar values which are copied when the print actually fires). This is why I think you see the map values increasing as they are printed, as the probes are still running.

The man page states:

Note that maps are printed by reference while scalar values are copied. This means that updating and printing maps in a fast loop will likely result in bogus map values as the map will be updated before userspace gets the time to dump and print it.

But I can definitely see how this would be a foot gun and lead to surprising behavior.

Just to add to what @amscanne said. There is also this issue which discusses the topic of async vs sync map printing and this, now merged, feature to iterate over per-cpu maps to get the total  value: https://github.com/bpftrace/bpftrace/pull/3189
I think we can close this issue in favor of this one:

https://github.com/bpftrace/bpftrace/issues/3028


RESTORE DATABASE broken

PHP Driver version or file name
ExtensionVer => 5.3.0
SQL Server version
SQL Server 14.0.2002.14
Client operating system
Debian 9
PHP version
PHP 7.2.14
Microsoft ODBC Driver version
ODBC Driver for Microsoft SQL Server <IP_ADDRESS>
Problem description
RESTORE DATABASE sqlsrv_query returns false
and "Processed XXX pages for database..." error
Expected behavior and actual behavior
I expected the database to be restored into a functional state, which has worked in PHP 5.6 using the mssql driver.
What actually happens is sqlsrv_query returns false and sqlsrv_errors(SQLSRV_ERR_ERRORS) gives "Processed XXX pages for database..." and the database is left in Restoring... state
I have set sqlsrv_configure("WarningsReturnAsErrors",0) so I dont see why I get this as an error.
Seems to be a known issue from 2010
https://blogs.msdn.microsoft.com/brian_swan/2010/04/13/working-down-the-stack-enabling-odbc-tracing-with-the-sql-server-driver-for-php/
https://blogs.msdn.microsoft.com/brian_swan/2010/07/01/restoring-a-sql-server-database-from-php/
which then could be fixed by running a
USE TestDB
query after the RESTORE DATABASE query.
But that doesnt work for me and sqlsrv_query didnt return false in his blog post either so guess the issue has gotten worse since 2010.
Repro code or steps to reproduce
Call sqlsrv_query with $sql RESTORE DATABASE TestDB FROM DISK='C:\Backups\testdb.bak'

@pakomp I can reproduce the behaviour you see, except that USE TestDB works if I call it immediately after attempting the RESTORE query (if I try reconnecting or call it from a different script, then it fails).
Have you tried to restore the database using sqlcmd?

Running the code from the blog the output I get is
RESTORE DATABASE TestDB FROM DISK = '//10.164.XXX.YYY/BAK/sql.bak'  WITH RECOVERY Array ( [0] => Array ( [0] => 01000 [SQLSTATE] =>  01000 [1] => 4035 [code] => 4035 [2] => [Microsoft][ODBC Driver 17 for  SQL Server][SQL Server]Processed 1046528 pages for database 'TestDB', file  'Data' on file 1. [message] => [Microsoft][ODBC Driver 17 for SQL  Server][SQL Server]Processed 1046528 pages for database 'TestDB', file  'Data' on file 1. ) )  USE TestDB Array ( [0] => Array ( [0] => 42000 [SQLSTATE] => 42000 [1] => 927  [code] => 927 [2] => [Microsoft][ODBC Driver 17 for SQL Server][SQL  Server]Database 'TestDB' cannot be opened. It is in the middle of a restore.  [message] => [Microsoft][ODBC Driver 17 for SQL Server][SQL Server]Database  'TestDB' cannot be opened. It is in the middle of a restore. ) )
Using sqlcmd works, but I cant pass the correct login password to that since it doesnt have anyway to handle characters that needs escaping, so I cant use that as a workaround.

Thank you @pakomp. We will investigate this issue and get back to you.

@pakomp please elaborate what you meant by not being to escape characters using sqlcmd. FYI, you might find this question on stackoverflow useful

The character I seem to have problem with is using " in the password, without it I seem to be able to supply the password as both -P  or -P ''.
But when I add the " to the password I just get
Sqlcmd: Error: Microsoft ODBC Driver 17 for SQL Server : Login failed for user
however I try.
Feels like that stackoverflow question is about how windows handles escaping, which doesnt help me much when using bash on linux.

@pakomp, I forgot to mention last time, as @david-puglielli pointed out earlier,

USE TestDB works if I call it immediately after attempting the RESTORE query

That is, the workaround USE TestDB still works if it is called immediately after the RESTORE query (do not disconnect in between).
Meanwhile, we will continue investigating the root cause.

@pakomp to use sqlcmd with a password with special characters, you can try running sqlcmd like this:
sqlcmd -S <yourserver> -U <userid>
This will prompt you for your password, which you should enter directly. There is no need to escape any character.
 Therefore, the client should process messages like "[Microsoft][ODBC Driver 17 for SQL Server][SQL Server]Processed  pages for database 'test_database', file 'test_database_log' on file 1."
The larger the database, the more of such messages you will have to process. Otherwise, the server will stop in the middle of the task. When using the ODBC driver, we should keep invoking SQLMoreResults() and reading diag recs until no more are found until the operation completes successfully.
When using PHP, we could do something like this:
    //Restore DB.
    $sql = "RESTORE DATABASE $database FROM DISK = '$backup_file' WITH RECOVERY";
    echo "Query: $sql" . PHP_EOL;
    $stmt = sqlsrv_query($conn, $sql);
    if ($stmt === false) {
        die(print_r(sqlsrv_errors()));
    } else {
        echo "Database being restored from $backup_file..." . PHP_EOL;
        print_r(sqlsrv_errors());
    }

    while ($next_result = sqlsrv_next_result($stmt)) {
        print_r(sqlsrv_errors());
    }

Therefore, when it's successfully finished, you should expect to receive a message like this one:
 [Microsoft][ODBC Driver 17 for SQL Server][SQL Server]RESTORE DATABASE successfully processed 299 pages in 0.089 seconds (26.246 MB/sec).
Here are some references for your interest:
https://stackoverflow.com/questions/278359/sql-server-backup-database-over-odbc
http://otl.sourceforge.net/otl4_ex688.htm

I was running
sqlsrv_configure("WarningsReturnAsErrors",0);
to late in my code, after moving it up before the sqlsrv_connect call then
$stmt = sqlsrv_query($conn, $sql);
stopped returning false, and I could call sqlsrv_next_result to get the db to restore.
Why are all messages treated like warnings?

hi @pakomp
By default, WarningsReturnAsErrors is true, and the user has to deliberately turn it off by setting it to false.
The RESTORE DATABASE operation actually returns SQL_SUCCESS_WITH_INFO, and by calling the SQLGetDiagRec Function it retrieves any error, warning, and/or status information.
In this case, the messages are returned as warnings. If you are concerned about setting WarningsReturnAsErrors to false, you can also turn on logging, like this:
sqlsrv_configure("WarningsReturnAsErrors", 0);
sqlsrv_configure("LogSeverity", SQLSRV_LOG_SEVERITY_ALL);
sqlsrv_configure("LogSubsystems", SQLSRV_LOG_SYSTEM_CONN | SQLSRV_LOG_SYSTEM_STMT);

When you give this a try, you will see that the messages returned are basically informational messages with SQLSTATE = 01000

@pakomp I believe this issue is resolved. I'm closing this issue, but please feel free to reopen.


Error when using in WSL

exists: The network path was not found.: "//wsl%24/Ubuntu/home/stan/projects/personal/WeakAuras2\"
stack traceback:
	[C]: in function 'bee.filesystem.exists'
	task:27: in function <task:25>
	(...tail calls...)
	...ions/sumneko.lua-0.15.4/server/script/glob\gitignore.lua:149: in method 'scan'
	task:37: in function <task:7>
	[C]: in function 'xpcall'
	task:49: in local 'f'
	[string "ID             = 3..."]:23: in function <[string "ID             = 3..."]:10>
	[C]: in function 'xpcall'
	[string "ID             = 3..."]:28: in main chunk
	[C]: in ?
stack traceback:
	.../extensions/sumneko.lua-0.15.4/server/script/service.lua:991: in method 'listen'
	...an\.vscode\extensions\sumneko.lua-0.15.4\server\main.lua:23: in main chunk
	[C]: in ?
create_directories: The network path was not found.: "\\wsl%24\Ubuntu\home\stan\projects\personal\WeakAuras2\.vscode"
stack traceback:
	[C]: in function 'bee.filesystem.create_directories'
	...0.15.4/server/script/method\workspace\executeCommand.lua:77: in field '?'
	...0.15.4/server/script/method\workspace\executeCommand.lua:256: in function 'method.workspace.executeCommand'
	[C]: in function 'xpcall'
	.../extensions/sumneko.lua-0.15.4/server/script/service.lua:58: in method '_callMethod'
	.../extensions/sumneko.lua-0.15.4/server/script/service.lua:105: in method '_doProto'
	.../extensions/sumneko.lua-0.15.4/server/script/service.lua:723: in method '_loadProto'
	.../extensions/sumneko.lua-0.15.4/server/script/service.lua:969: in method 'onTick'
	.../extensions/sumneko.lua-0.15.4/server/script/service.lua:992: in method 'listen'
	...an\.vscode\extensions\sumneko.lua-0.15.4\server\main.lua:23: in main chunk
	[C]: in ?
stack traceback:
	.../extensions/sumneko.lua-0.15.4/server/script/service.lua:105: in method '_doProto'
	.../extensions/sumneko.lua-0.15.4/server/script/service.lua:723: in method '_loadProto'
	.../extensions/sumneko.lua-0.15.4/server/script/service.lua:969: in method 'onTick'
	.../extensions/sumneko.lua-0.15.4/server/script/service.lua:992: in method 'listen'
	...an\.vscode\extensions\sumneko.lua-0.15.4\server\main.lua:23: in main chunk
	[C]: in ?
[Error - 23:59:13] Request workspace/executeCommand failed.
  Message: {
	arguments = {
		[1] = {
			action = "add",
			key = {
				[1] = "diagnostics",
				[2] = "globals",
			},
			value = "GetSpellInfo",
		},
	},
	command = "config",
}
create_directories: The network path was not found.: "\\wsl%24\Ubuntu\home\stan\projects\personal\WeakAuras2\.vscode"
stack traceback:
	[C]: in function 'bee.filesystem.create_directories'
	...0.15.4/server/script/method\workspace\executeCommand.lua:77: in field '?'
	...0.15.4/server/script/method\workspace\executeCommand.lua:256: in function 'method.workspace.executeCommand'
	[C]: in function 'xpcall'
	.../extensions/sumneko.lua-0.15.4/server/script/service.lua:58: in method '_callMethod'
	.../extensions/sumneko.lua-0.15.4/server/script/service.lua:105: in method '_doProto'
	.../extensions/sumneko.lua-0.15.4/server/script/service.lua:723: in method '_loadProto'
	.../extensions/sumneko.lua-0.15.4/server/script/service.lua:969: in method 'onTick'
	.../extensions/sumneko.lua-0.15.4/server/script/service.lua:992: in method 'listen'
	...an\.vscode\extensions\sumneko.lua-0.15.4\server\main.lua:23: in main chunk
	[C]: in ?
  Code: -32603

Should be fixed.


Update help in app (it's correct in the README).

Other fields have an example of a regexp (such as the skip phrase)
or call it out in the Help.
We were excited to realize we could use this feature after an upgrade.

Thanks @jibsheet !


Boolean event payload value

In nmos/json_fields.h, in addition to payload_number_value, it seems like it would be useful to also have:
const web::json::field_as_bool payload_boolean_value{ U("value") };

Thanks, yes. Done.


Swagger Documentation for Response Models

Informations

Type
Version

Question
2.x

Description
The swagger documentation that is generated contains all the endpoints and example request models (defined by decorators on models).
Does the library currently generate the same documentation for response models to inform users of the API what will be returned to them?
If so, how is this done? I currently have the same decorators on the response objects.
If not, is this a feature that could be added?
Thanks.

Hi @Ahagan ,
@Returns do that :)
@Returns(Model, {description: "Description"}) //OR
@Returns(Model)
 async myMethod(): Promise<Model>  {

 }

more details : https://romakita.github.io/ts-express-decorators/#/api/swagger/returns

Thanks :)
On 25 Oct 2017, at 22:11, Romain Lenzotti <notifications<EMAIL_ADDRESS>wrote:
Hi @Ahaganhttps://github.com/ahagan ,
@returnshttps://github.com/returns do that :)
@Returns(Model, {description: "Description"}) //OR
@Returns(Model)
async myMethod(): Promise  {
}
more details : https://romakita.github.io/ts-express-decorators/#/api/swagger/returns
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Romakita/ts-express-decorators/issues/153#issuecomment-339472845, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AEcJaNKY6EAwHrvCtLTEjQ-LxYbyJfWsks5sv6P9gaJpZM4QGEEO.


Upgrade interop cats

This fixes the hanging tests in my testing. We ran into a similar issue while upgrading Tapir that required a fix to interop-cats:
- https://github.com/softwaremill/tapir/pull/2043
- https://github.com/zio/interop-cats/issues/533

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.3 out of 4 committers have signed the CLA.:white_check_mark: justcoon:white_check_mark: pjfanning:white_check_mark: toxicafunk:x: Eric RodriguezEric Rodriguez seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Can someone approve a build? If it fails I'd like to be able to at least debug what's going on.


budget allocation with no control vars

Discussed in https://github.com/pymc-labs/pymc-marketing/discussions/1028

Originally posted by bella0715 September 12, 2024
Here's how I built the model.
mmm = DelayedSaturatedMMM(
    model_config = mmm_config,
    sampler_config = sampler_config,
    date_column = date_var,
    channel_columns = spend_vars,
    control_columns = None,
    adstock_max_lag=8,
    yearly_seasonality=1,
)

When I try to run the below code to do the budget allocation, I get an error 'UnboundLocalError: cannot access local variable '_controls' where it is not associated with a value'. How do I run the budget allocation without control_columns?
response = mmm.allocate_budget_to_maximize_response(
    budget=total_budget,
    num_days=8,
    time_granularity="weekly",
    budget_bounds=budget_bounds,
)
```</div>

It seems like part of the problem:
https://github.com/pymc-labs/pymc-marketing/blob/936270958f79fd3cce0488ae2301f2d8f3e2a35f/pymc_marketing/mmm/mmm.py#L2074-L2077

Curious about what would be an optimal solution from the experts. In the meanwhile, sharing my workaround for this issue where I added a few new methods to the MMM class, extended others, and used historical control variables plus random noise using the same approach as in _create_synth_dataset.

Find last year's start_date_comparison based on last_date https://github.com/pymc-labs/pymc-marketing/blob/d5fa54348816404668182f34d0bee72ca20bbc25/pymc_marketing/mmm/mmm.py#L2079:

from dateutil.relativedelta import relativedelta

start_date_comparison = last_date - relativedelta(years=1)

Get comparison df from train dataset X

    from datetime import datetime

    def get_comparison_df(
            self,
            start_date_comparison: datetime = None,
            num_periods: int = None,
    ) -> tuple[pd.DataFrame]:

        end_date = start_date_comparison + pd.DateOffset(weeks=num_periods)

        date_filter_str = (f"{self.date_column} >= {start_date_comparison.strftime('%Y%m%d')} and "
                           f"{self.date_column} <= {end_date.strftime('%Y%m%d')}")

        df_train_comparison = self.X.query(date_filter_str).copy()

        return df_train_comparison

Added **kwargs to allocate_budget_to_maximize_response and the _create_synth_dataset call in allocate_budget_to_maximize_response to support new params without too many modifications to the original methods.

def allocate_budget_to_maximize_response(
    self,
    ...
    **kwargs,
) -> az.InferenceData:
    ...
        synth_dataset = self._create_synth_dataset(
            df=self.X,
            date_column=self.date_column,
            allocation_strategy=self.optimal_allocation_dict,
            channels=self.channel_columns,
            controls=self.control_columns,
            target_col=self.output_var,
            time_granularity=time_granularity,
            time_length=num_periods,
            lag=self.adstock.l_max,
            noise_level=noise_level,
            **kwargs,
        )
    ...
    return ...

Added start_date_comparison and df_train_comparison to method _create_synth_dataset, and added the workaround logic:

    def _create_synth_dataset(
        self,
        ...
        start_date_comparison: datetime | str | None = None,
        df_train_comparison: pd.DataFrame | None = None,
    ) -> pd.DataFrame:
        """Create a synthetic dataset based on the given allocation strategy (Budget) and time granularity.

        Parameters
        ----------
        ...
        start_date_comparison : datetime | str | None
            A date from the synthetic dataset will be created.
        df_train_comparison : pd.DataFrame | None
            A dataframe from a previous year from the train dataset
        """
        ...

        if start_date_comparison is not None:
            last_date = pd.to_datetime(start_date_comparison).tz_localize(None)
        else:
            last_date = pd.to_datetime(df[date_column]).max()  # ln:2079

        ...

        new_rows = [
            ...
        ]  # ln:2108

        # Add historical control variables plus random noise
        if df_train_comparison is not None:

            synth_dataset = pd.DataFrame(new_rows)

            for control in self.control_columns:
                synth_dataset[control] = [value + np.random.normal(0, noise_level * value)
                                          for value in df_train_comparison[control].to_list()]

            return synth_dataset

        else:
            return pd.DataFrame(new_rows)  # ln: 2110

Hope this is not too convoluted and self-explanatory.
Happy to hear your thoughts and optimal solution.

Seems like a pretty good workaround. Would you want to make a PR for this @AlfredoJF?
Seems like a simple edit around these lines might do the trick:
https://github.com/pymc-labs/pymc-marketing/blob/af946dfa8687a65018ddd4d708f434a7f32f30ab/pymc_marketing/mmm/mmm.py#L2091-L2094


[Improvement]: Remove requiring single quotes around cases in data providers

Description
ATM selectively running tests requires single quotes around cases apart from the quotes as in the below example at https://ballerina.io/learn/test-ballerina-code/define-data-driven-tests/#execute-specific-data-sets
$ bal test --tests fruitsDataProviderTest#"'banana'"
Before SL U3 due to a technical limitation in the Java implementation, we had to use single quotes as well. But, with the Ballerina based test engine, we need to look at the possibility of supporting it without the single quotes.
The single quote check can be removed here.
https://github.com/ballerina-platform/ballerina-lang/blob/master/misc/testerina/modules/testerina-core/src/main/ballerina/execute.bal#L299
Verify after adding fix:

[ ] Need to test for all possible cases
[ ] Keep supporting the single quotes to avoid breaking changes
[ ] Test https://github.com/manuranga/nballerina with the change

Describe your problem(s)
No response
Describe your solution(s)
No response
Related area
-> Test Framework
Related issue(s) (optional)
No response
Suggested label(s) (optional)
No response
Suggested assignee(s) (optional)
No response

With following change, it worked without single quotes. We need to do the verifications mentioned in the issue.
--- a/misc/testerina/modules/testerina-core/src/main/ballerina/execute.bal
+++ b/misc/testerina/modules/testerina-core/src/main/ballerina/execute.bal
@@ -299,7 +299,7 @@ function skipDataDrivenTest(TestFunction testFunction, string suffix, TestType t
                 if (subFilter.startsWith(SINGLE_QUOTE) && subFilter.endsWith(SINGLE_QUOTE)) {
                     updatedSubFilter = subFilter.substring(1, subFilter.length() - 1);
                 } else {
-                    continue;
+                    updatedSubFilter = subFilter;
                 }
             }

Tasks to verify the fix
[ ] Test edge cases
[ ] Test nballerina
[ ] Update the website with new version
[ ] Manually test with a sample repo


Add detailed exception for Claims check

I tried to create an issue on your new repo without success.
Could it be possible to add exception like AudienceException or ExpiryException so ti will be easier to handle them in the code later?
Thanks

Hi,
This is a nice feature but now this library only receives security fixes and will be abandonned by the end of 2019 (see #297).
I will not add it here, but it is already incorporated in my local repository of the next project. This new project will be marked as stable in December.

Done in https://travis-ci.org/web-token


Export transaction records as CSV format

Description
Export all the transaction records of one account as CSV file.
Context
As requested by community members, it would be good to have an export function in our explorer to support download all the transaction records for certain account in a zipped CSV file format.
For example, all Tx for this account https://explorer.harmony.one/#/address/one1ycus2qrrdem5dx6g09mzdcqvkng8az9fq6hh36
can be downloaded via a link to csv file.
One design to avoid abuse of the system, user has to sign-in using one wallet to his/her own account in explorer.
Users can only request download of his/her own transaction records in csv file format. The CSV file generation can be handled in the background from a queue system. After the csv file is generated, user will have the link to download the file.
The harmony explorer frontend repo is here. https://github.com/harmony-one/harmony-dashboard
If you need access to the backend repo, please contact the team.
Acceptance Criteria

[ ] A feature design doc
[ ] support sign-in to explorer using harmony ONE wallet
[ ] csv download link only available to users who signed in to the explorer
[ ] csv file generation and add link to the frontend to specific user, following harmony explorer UI theme

Reward
USD $2,500 equivalent amount of Harmony ONE token.

well done
hey guys. i have finished the develop work. can you have a check here(https://harmony-dashboard.vercel.app). let me know if there is any issue for it

@LuttyYang In general, it looks pretty good. I tried it and it works now with metamask. A few comments.

please submit a PR to our repo
please put the button to download at the right bottom corner of the transaction list, similar to what etherscan have
the output file name with the account number in the filename
cache the generated zip file to make it less load on server, maybe cache for one hour

Example on etherscan,

Completely accepted your feedback.It looks like this now:

PR is here: https://github.com/harmony-one/harmony-dashboard/pull/147/files

this bounty is done.


Better Multi Monitor Handling

As it currently stands, the script perfectly supports multi-monitor setups for all operations except when switching the first / last client from one monitor's focused workspace to another monitor's currently open but unfocused workspace.

Fixed with 7072b4f967a0391f344e6b342f8f2ada81f78050


Extra tooltip appears when hovering over an item/spell

Hello!
I've noticed that when hovering over an item, spell, ability etc. etc. there's a tooltip with its name under the cursor. The thing is, it sometimes overlaps the information card (if it's enabled to appear near the cursor) or some other UI elements. Sometimes it works fine. For example, hovering over DEX it shows Dexterity in the tooltip, but there are places where it's completely unnecessary. I believe there is no need to duplicate the item's or ability's name when I'm already looking at it with the cursor placed there.
Would be great if there was a way to disable/configure this tooltip.

This tooltip is "exclusive" too Tidy 5E. It does not appear on default 3.0 character sheets or legacy ones:

I am using the latest FVTT stable build, D&D 5E system and Tidy 5E sheets. Also tested with disabling all the other modules, except for Tidy 5E

Thank you for reporting this. It will take a little longer to overhaul the tooltips here because of ongoing projects, but I will make sure that the Item Name tooltip does not pop up when the item card is in floating mode.
The tooltip exists to ensure that even the item name, when truncated with ellipses (...), will still tell the player the item name. I can definitely see how that would be irrelevant when using floating item cards.

My apologies for the wait. I am going to remove the small tooltip from the item/spell names. There are other things which have small tooltips, like the classic control icons, and those will stay.

Will this work for you?

This is now released in Tidy v4.3.0.


Able to build in /spawn world

If you fly right to the world border, you can place blocks off of the world edge and make a platform

Fixed players being able to escape from the spawn into this flat world. Thank you for reporting this :)


[Java] Unable to find Serilizer.java interface as present in java graph guide

Search before asking

[X] I had searched in the issues and found no similar issues.

Version
Fury - 0.4.1
Component(s)
Java
Minimal reproduce step
https://github.com/apache/incubator-fury/blob/ea9602f9425002d633569381c5dc7986706759bb/docs/guide/java_object_graph_guide.md#implement-a-customized-serializer
What did you expect to see?
Expect to implement customized serializer
What did you see instead?
Unable to find Serilizer interface as shown in
https://github.com/apache/incubator-fury/blob/ea9602f9425002d633569381c5dc7986706759bb/docs/guide/java_object_graph_guide.md#implement-a-customized-serializer
Anything Else?
No response
Are you willing to submit a PR?

[ ] I'm willing to submit a PR!

Hi @aarvy-9 , thanks for reporting this issue. org.apache.fury.serializer.Serializer is an abstract class,  this is an error in uour doc. Could you try class FooSerializer extends Serializer<Foo>?

Sure...the class we want to serialize constantly evolves and I want to make sure all developers register newly added class with fury. Apart from Serializer abstract class, is there any interface that we can implement?

You can configure Fury with FuryBuilder#requireClassRegistration(true) and register every class by Fury#register manually


Question : Can't manage to import npm packages

Hi,
I am having a lot of trouble loading npm packages from 3rd party libraries on the client side.
It does not seem to be an issue with Meteor itself, which usually manage to import packages (though it sometimes give me hard times), but with the interaction between Typescript/Angular/Meteor.
I saw thousands of Stack Overflow questions on the subject, each time too specific or solving the problem with a hack... Import works fine on well designed package, but most of the time developpers build there export for 'classical' use, not for Meteor.
Could you please provide an example of import that works at least on client side, for example for plotly.js, d3 or crossfilter2 ? That would help begginners a lot, and I'd like to improve the tutorial or the doc with some examples.
Edit :  to be precise, for the moment, I manually drop the minified js file of the library in the client/compatibility folder, and add a declare var theObjectCreatedByMyPackage:any (for example declare var crossfilter : any or declare var Plotly : any) so that the typescript compiler is happy

Hi @eric-burel, take d3 for example, you can search by using typings search d3.
There is a source showing NPM, so you can install by using typings install d3 --save.
If the source only has dt, you need install by typings install dt~XXX --save.
When you install, if you meet

Attempted to compile "XXX" as an external module, but it looks like a global module. You'll need to enable the global option to continue.

You need add --global, like typings install dt~XXX --global --save

Thanks for this precision, I did not now about typings. Does the fact that I lacked a typings package can explain a failure when importing ? However, import faker from 'faker' still won't work after the typings package install.
Importing npm modules always works with a "vanilla" Meteor, so I guess the issue is related to typescript, or even to Meteor.
Can you please try to import faker in a file located in the imports folder in a small angular-meteor project for example ? That could help me to find out if the mistake is in my own code or not...

The reason that you need to do import * as faker from 'faker' due to the fact that the faker doesn't export anything by default. Or in other words, import foo from 'foo' only works in TypeScript only if there is modules.default = ... inside that NPM.
On the your another points. I don't agree that using Meteor in that case differs from the "classical use". Meteor works with NPM now, so everything that matters stay the same.
We presume that you understand the concept of the TypeScript definition files, if not you can read about them on Internet or in our tutorial
At the same time, we provide some initial information (including README in this package) on the typings management and about the tooling we recommend to use: here https://github.com/Urigo/angular2-meteor#typings, here https://github.com/barbatus/typescript#typings and here https://github.com/meteor-typings/meteor.
I believe most people find it enough to begin with.


Disable test-gen when using -DskipTests

This is a fix to make the test-gen-plugin compatible with Jitpack (and the automatic publication of artifacts).
Basically when passing -DskipTests the entire test infrastructure generation is disabled, this way Jitpack should be able to automatically publish preview releases.

Superseeded by #93


URLSearchParams Error

Hey Guys, please help after upgrading to latest version of whatwg-fetch v3.3.1
I am seeing this error
TypeError: Cannot use 'in' operator to search for 'URLSearchParams' in false
at /home/karan/Desktop/repos/idwapp/node_modules/whatwg-fetch/dist/fetch.umd.js:10:37
at /home/karan/Desktop/repos/idwapp/node_modules/whatwg-fetch/dist/fetch.umd.js:2:66
at Object. (/home/karan/Desktop/repos/idwapp/node_modules/whatwg-fetch/dist/fetch.umd.js:5:2)
at Module._compile (internal/modules/cjs/loader.js:1158:30)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:1178:10)
at Module.load (internal/modules/cjs/loader.js:1002:32)
at Function.Module._load (internal/modules/cjs/loader.js:901:14)
at Module.require (internal/modules/cjs/loader.js:1044:19)
at require (internal/modules/cjs/helpers.js:77:18)
at Object. (/home/karan/Desktop/repos/idwapp/functions/index.js:1:143379)
error: Forever detected script exited with code: 1
error: Script restart attempt #9
/home/karan/Desktop/repos/idwapp/node_modules/whatwg-fetch/dist/fetch.umd.js:10
searchParams: 'URLSearchParams' in global,

@karanmartian Can you run this on the page and tell us what it returns please?
(typeof self !== 'undefined' && self) || (typeof global !== 'undefined' && global)

@JakeChampion its returning false, in fetch.umd.js
var global = (typeof self !== 'undefined' && self) || (typeof global !== 'undefined' && global);
console.log(global); --> false
var support = {
searchParams: 'URLSearchParams' in global,
iterable: 'Symbol' in global && 'iterator' in Symbol,
......

Is that from the self or from the global?

global

Oh that is interesting, is this error from a NodeJS application?
Do you know what is modifying the global variable named global?

React JS and Node JS yes
I have no idea what is modifying it. Can another library influence it?

Yes, it is a global variable which is meant to point to the global object.
We can add a work around in this codebase but really, anything which is making the global variable global not point to the global object will be causing issues in lots of packages.

@JakeChampion could you please add the workaround? I dont seem to have another option at this time and I have been stuck at v2.0.4 for a long time now.

This should fix it for you if you are on NodeJS 12 or higher -- https://github.com/github/fetch/pull/822

@JakeChampion Tysm will there be a new release with this fix?

Sure thing, in the meantime you can find the culprit modifying the global self variable with this code.
If you include this in your entry file before any other code and then open the developer tools you will find the piece of code which is modifying the self variable.
Object.defineProperty(window, 'self', {
  configurable: true,
  enumerable: false,
  set: function(value) {
    // The delete command removes the setter.
    delete window.self;
    window.self = value;
	debugger;
    console.log("self global variable was modified!");
  }
});

published in 3.4.0 👍

tysm @JakeChampion really appreciate it a lot☺


Bigger Reactors Uranium Ore Missing Item Tags

No problem in processing
But some tags missing in item tags
This mod's features contains 'Fix Tags'
So... It looks good to fix

4c210a5fdcf1bd5c9067462aa6d89b6edf53d808


Please add this package to the JCenter() repo on Bintray.

Please add the bintray project to the JCenter common repo.
https://jfrog.com/blog/increase-your-maven-packages-exposure-by-adding-it-to-jcenter/
Without this, we have to add a custom repo just for this project; which complicates transitive deps. I'd like to be able to build and ship projectes to JCenter which depend upon this project.
For those wondering how to get it to link, you need this repo at the moment:
"https://dl.bintray.com/wasmer/wasmer-jni"

Request sent to JCenter. Waiting for a reply.

Reply (for the record):

Hi hywan,
Thank you for submitting this request,
JCenter hosts java applications that follow maven convention. In addition to the .jar and .pom file, your version should include a sources jar, and optionally a javadoc jar.
Your files should be under a maven path layout. (See https://bintray.com/docs/usermanual/uploads/uploads_includingyourpackagesinjcenter.html)
Once those files are added, we'll be glad to include your package in JCenter.


Circleci project setup

This adds mac and linux autobuilding on push.
Windows autobuilding doesn't work because you have set paths to SDL that the script cannot access in its runner - so those
will need to be built manually.


Fix all auto-fixable problems not fixing everything

VSCode Version: 1.15.0
OS Version: Windows 10

When I try to fix all issues with f.ex quotemark, no-inferrable-types and semicolon it fixes everything but the quotemarks.
It doesn't matter which problem I am starting the command from. Even from the quotemarks it doesn't fix them.
I also see now it's the same with the missing whitespace rule (didn't notice since I have this being formatted on save)
If quotemark is the only issue the option works.

Sorry, but I don't know whether this is a VS Code or an extension issue.
What language/extension are you using?

Aha I see. Figgured out it's coming from the TSLint extension. So maybe the issue should be logged there instead?

I'll move the issue to TSLint then.

This issue was moved to Microsoft/vscode-tslint#239


[DOC] Add engineering section to the docs

Commits descriptions
Supporting examples
Clear distinction between various types of commits
rationale behind the code commit process

Resloved by https://github.com/apache/systemds/pull/1231


[REQUEST] Low Health red vignette

When the player is on low health (>15%), a red vignette should appear around the edges of the screen, slowly getting higher in intensity as health depletes. This vignette can flash slightly, mimicing emergency lights on the ship. When health reaches above 15%, the vignette should not be visible.
The vignette will be accompanied with alarm sounds and blaring.
Describe alternatives you've considered
There could also be a small text box that appears, stating that you're low on health and might crash any second. For further information or uncertainties, contact Ebony.

Task Estimation (time): 3 (complete guess)
Does this issue address a risk? Not directly
Risk: Have not done something like this in Unity before (the vignette thing, I've done event triggers before!)
Risk Resolution: If it takes more than a websearch and some swearing I'll ask for help!

Actual time taken: ~3h.  Huh.  It felt like a lot longer than that! Also right on, I managed to guess correctly!
Testing procedure: (pick one) Playtest
Test Pass Condition(s):  SO MUCH PLAYTEST, as it's an in-combat cosmetic change.  Basically, does it work As Expected, or no? If yes, pass! Yay!
Risks: It may Mysteriously Break, as my screen shake code seems to have.  :/
Risk Resolution: I will yell about it until we figure out why.  (The why may just be Unity being a butt, at which point all you can do is fix it, siiiiigh.)


When using wac as a library, it interacts

I am using wac (specifically, the wac_resolver crate) as a library in another program.  As part of this, I call RegistryPackageResolver::resolve().  I am using the wa.dev registry for dev-test purposes, and when I try to resolve a package for the first time I get this:
? Package `fermyon:spin` is not in `warg.wa.dev` registry.
Registry recommends using `fermyon.wa.dev` registry for packages in `fermyon` namespace.
Accept recommendation y/N

These prompts come from warg_client, and I'd like to be able to opt out of them, e.g. in CI scenarios.
It seems like warg_client has a cli-interactive feature, which is on by default.  Would it be possible for wac to surface a similar feature?  (I'm happy to send a PR for this if you're open to it.)

Hmm, although it looks like Warg's behaviour if not allowed to interact might be to fail so whee.

Thanks @calvinrp.  I don't really want to rely on users installing and configuring warg - but if wkg is going to solve this then that's cool, I probably need to wait for wac to adopt wkg before I can ship my feature anyway!  Closing!

We are getting closer on wkg updates. But I wanted to understand the change here that will help.
To confirm, you'd like wac configurable so that it doesn't ask for user input while running in a CI/CD process?

@calvinrp My specific context is that I am using wac_resolver as a library and I want my program to (have the option to) not interact.  So yes to being able to configure the wac crates, and no opinion on the wac CLI tool.  Hopefully that clarifies - I'm happy to go into more detail if that would be useful.  Thanks!

@itowlson release 0.6.0 should fix, can you confirm?

That seems to be working fine.  Thank you @calvinrp!


pysnmp.smi.error.SmiError: MIB object ObjectIdentity('<IP_ADDRESS>.<IP_ADDRESS>.<IP_ADDRESS>.1.2.100') is not OBJECT-TYPE (MIB not loaded?)

I am attempting to use an SNMP set command to copy config files on a Cisco device.
I am attempting to use the OID for Cisco Config Copy MIB.
The first OID is <IP_ADDRESS>.<IP_ADDRESS>.<IP_ADDRESS>.1.2.x.
I simply cannot get it to work.
from pysnmp.hlapi import *

for (errorIndication,
     errorStatus,
     errorIndex,
     varBinds) in setCmd(SnmpEngine(),
                          CommunityData('private'),
                          UdpTransportTarget(('<IP_ADDRESS>', 161)),
                          ContextData(),
                          ObjectType(ObjectIdentity('<IP_ADDRESS>.<IP_ADDRESS>.<IP_ADDRESS>.1.2.100'), 2),
                          lookupMib=False):

    if errorIndication:
        print(errorIndication)
        break
    elif errorStatus:
        print('%s at %s' % (errorStatus.prettyPrint(),
                            errorIndex and varBinds[int(errorIndex) - 1][0] or '?'))
        break
    else:
        for varBind in varBinds:
            print(' = '.join([x.prettyPrint() for x in varBind]))

Traceback (most recent call last):
File "C:\Project Files\snmptest.py", line 11, in
lookupMib=False):
File "C:\Python27\lib\site-packages\pysnmp\hlapi\asyncore\sync\cmdgen.py", line 210, in setCmd
lookupMib=options.get('lookupMib', True)))
File "C:\Python27\lib\site-packages\pysnmp\hlapi\asyncore\cmdgen.py", line 235, in setCmd
contextData.contextName, vbProcessor.makeVarBinds(snmpEngine, varBinds),
File "C:\Python27\lib\site-packages\pysnmp\hlapi\varbinds.py", line 36, in makeVarBinds
__varBinds.append(varBind.resolveWithMib(mibViewController))
File "C:\Python27\lib\site-packages\pysnmp\smi\rfc1902.py", line 845, in resolveWithMib
raise SmiError('MIB object %r is not OBJECT-TYPE (MIB not loaded?)' % (self.__args[0],))
pysnmp.smi.error.SmiError: MIB object ObjectIdentity('<IP_ADDRESS>.<IP_ADDRESS>.<IP_ADDRESS>.1.2.100') is not OBJECT-TYPE (MIB not loa
ded?)
If I'm using OID why do I need an MIB? Why does it not have this MIB anyways if its on http://mibs.snmplabs.com/asn1/CISCO-CONFIG-COPY-MIB, and how do I fix?

If I'm using OID why do I need an MIB?

That's because you are passing a value (2) along with the OID. The value is untyped (to SNMP). To send a well formed packet, pysnmp needs to figure out SNMP type of the value you are sending. The only way for pysnmp to handle this is to look up the MIB. But pysnmp can't [yet] figure what MIB it should load to get hold of MIB object identified by that OID (e.g. OID->MIB mapping is not implemented).
To fix that you simply need to give it SNMP type for your value:
ObjectType(ObjectIdentity('<IP_ADDRESS>.<IP_ADDRESS>.<IP_ADDRESS>.1.2.100'), Integer32(2))

Why does it not have this MIB anyways if its on http://mibs.snmplabs.com/asn1/CISCO-CONFIG-COPY-MIB, and how do I fix?

We need someone to implement OID->MIB map so that pysnmp could download and use proper MIB in some cases. I'd happily merge such kind of PR. ;-)

Help ful

Best to copy mib_file_name.py to "\venv\Lib\site-packages\pysnmp\smi\mibs"


Better support for paths and filenames

String variables whose values are files and paths are common in justfiles.
It might be nice to provide better support for this. Some ideas:

path(string) constructor for building paths. Should maybe call out that this will perform the conversion according to the environment, like path_from_windows(), path_from_unix(), and path_from_local()
A / operator for joining path components. Don't need to worry if one part already has a /
Abstract over / and \ separated paths so it works on both windows and unix
Provide some useful functions that operate on paths, such as converting to an absolute path, popping off a path component, and the like
A . variable which is the directory that contains the justfile
A justfile variable which is the path to the justfile
Perhaps also a file(string) function which throws an error if there is a path component

Moving this to the wiki and closing:
https://github.com/casey/just/wiki/Feature-Ideas


How can I use Piper TTS voices with Google Talkback?

Hi.
I am totally blind, and I am enjoying the Sonata TTS voices for the NVDA Screen Reader.
I would really like to have Piper TTS voices on my Google Android Phone so I might enjoy them with my Google Talkback Screen Reader as well.
Is that possible?

Ya know, I was also asking the same question as you, but more generally. E.G. Using Piper TTS as my engine for GPS apps and other such things. I'm using Jieshwo screen reader, which oh, surprise surprise, I'm the spanish translator.

My favorite voice is the Librits_R voice and Speaker #1851.
I would like to have this voice as the default voice for my Google Pixel 7
pro Phone as well as my Windows PC.
I amtotally blind and I am running windows 11 and I have NVDA as my Screen
Reader, and I would like to have the same voice across both platforms.
It would really be nice if these great Piper Neural TTS voices could be
SAPI5 compliant as well.
On Mon, May 27, 2024 at 10:59 PM Luis Carlos @.***>
wrote:

Ya know, I was also asking the same question as you, but more generally.
E.G. Using Piper TTS as my engine for GPS apps and other such things. I'm
using Jieshwo screen reader, which oh, surprise surprise, I'm the spanish
translator.
—
Reply to this email directly, view it on GitHub
https://github.com/rhasspy/piper/issues/510#issuecomment-2134266963, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AKJ77NR6G43V33CEMONUMT3ZEPXJLAVCNFSM6AAAAABILQILR6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMZUGI3DMOJWGM
.
You are receiving this because you authored the thread.Message ID:
@.***>

Totally blind since birth? What a connection we have! Me, also totally blind, but here's the difference: My old dell laptop running Windows 11 insider, my new HP laptop running Windows 10, my 12 year-old desktop Dell pc running Windows 11... I believe that's... umm... truely something...
You've catched me about Piper tts supporting Sapi5 protocol! That way I want to play my games or do other stuff, even using Narrator with piper neural voices. Have 160 or so voices installed, including mine.

I'm working on this - in our VoiceBroker project - at least making it a SAPI engine in Windows. Its tough but watch this space

Oh boy!
I would really like to be able to have Piper TTS voices for such apps
like:  Kindle for PC, Balabolka, Narrator, and other SAPI5 software that
uses TTS.
As one who is totally blind from birth, I can say it's a real shame that we
blind people have to be content with TTS voices that were developed back in
2006 and earlier.
It's really not fair that Microsoft should make it so tough.
I'll be looking forward to a link once you get it working.
On Sat, Jun 1, 2024 at 10:29 AM will wade @.***> wrote:

I'm working on this - in our VoiceBroker project - at least making it a
SAPI engine in Windows. Its tough but watch this space
—
Reply to this email directly, view it on GitHub
https://github.com/rhasspy/piper/issues/510#issuecomment-2143468375, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AKJ77NUIBJIVAECKWQTCX5TZFHLGPAVCNFSM6AAAAABILQILR6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBTGQ3DQMZXGU
.
You are receiving this because you authored the thread.Message ID:
@.***>

Hi.
I downloaded the Piper speech engine that I wanted to my Android phone, but
I can't figure out how to chuse the speaker I want.
Right now, I have speaker #0987, and I want #1851.
How do I change that setting?
Otherwise the engine works great?
Thank you so much.
On Tue, Jun 11, 2024 at 3:15 AM Fangjun Kuang @.***>
wrote:

I suggest that you try
https://k2-fsa.github.io/sherpa/onnx/tts/apk-engine.html
We support all Piper TTS models and provide Android APKs for them. You can
use it to replace your Android System TTS engine.
—
Reply to this email directly, view it on GitHub
https://github.com/rhasspy/piper/issues/510#issuecomment-2159967686, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AKJ77NS5S6Z6BF37P7KH3FTZG2P2LAVCNFSM6AAAAABILQILR6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNJZHE3DONRYGY
.
You are receiving this because you authored the thread.Message ID:
@.***>

Could you us which model/apk you have downloaded?
You can change speakers only for models that have multiple speakers.

I downloaded the Librits_R US english voice.
I am totally blind, and I am running the NVDA Screen reader on my Windows
PC.
I have the Sonata TTS addon, and I am running the Librits_R Piper voice
model, and I really like Speaker ID #1851, and that's the voice I want on
my Phone.
I no about multiple speakers, I just don't know how to pick #1851 for my
phone.
It's pretty straight forward for NVDA, al you do is pick it.
How do I pick it for my Phone?
On Tue, Jun 11, 2024 at 10:01 AM Fangjun Kuang @.***>
wrote:

Could you us which model/apk you have downloaded?
You can change speakers only for models that have multiple speakers.
—
Reply to this email directly, view it on GitHub
https://github.com/rhasspy/piper/issues/510#issuecomment-2160845890, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AKJ77NSJY3BBP73LAV7WZ4DZG37LLAVCNFSM6AAAAABILQILR6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNRQHA2DKOBZGA
.
You are receiving this because you authored the thread.Message ID:
@.***>

According to https://huggingface.co/rhasspy/piper-voices/blob/main/en/en_US/libritts_r/medium/en_US-libritts_r-medium.onnx.json#L849
speaker ID #1851 is 351.
Please set speaker ID to 351

For the following sample sentence
This is a sample sentence, generated using the speaker ID three hundred and fifty one, which is mapped to the original speaker ID eighteen fifty one.

The generated wav is posted below
(It was generated using https://huggingface.co/spaces/k2-fsa/text-to-speech)
https://github.com/rhasspy/piper/assets/5284924/edba5885-d499-45e2-9dbb-f40d79023bc2

That's it!
Thank you so much.
Now, is there a way to save that setting for the future?
Now that I know how to do it, I can set the speaker ID without sighted
assistance, but I find that if I restart my Phone, and my Talkback screen
reader comes up, that it defaults to the 903 setting and I would have to
reset my 351 speaker ID again.
Is there any way to save that setting so I won't have to reset it again?
Not that I restart my phone that often.
It's just that I would be appriciative if there was a way to save new
speaker ID settings.
Again, thank you so much.
All best,
Mike
On Tue, Jun 11, 2024 at 9:48 PM Fangjun Kuang @.***>
wrote:

For the following sample sentence
This is a sample sentence, generated using the speaker ID three hundred and fifty one, which is mapped to the original speaker ID eighteen fifty one.
The generated wav is posted below
(It was generated using
https://huggingface.co/spaces/k2-fsa/text-to-speech)
https://github.com/rhasspy/piper/assets/5284924/edba5885-d499-45e2-9dbb-f40d79023bc2
—
Reply to this email directly, view it on GitHub
https://github.com/rhasspy/piper/issues/510#issuecomment-2161927379, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AKJ77NWY25ILVB4ACXKQEEDZG6SGRAVCNFSM6AAAAABILQILR6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNRRHEZDOMZXHE
.
You are receiving this because you authored the thread.Message ID:
@.***>

Is there any way to save that setting so I won't have to reset it again?

Yes. Please wait for our next release.

Will I be able to get to it using the link that you pointed me too?
On Tue, Jun 11, 2024 at 10:18 PM Fangjun Kuang @.***>
wrote:

Is there any way to save that setting so I won't have to reset it again?
Yes. Please wait for our next release.
—
Reply to this email directly, view it on GitHub
https://github.com/rhasspy/piper/issues/510#issuecomment-2161976464, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AKJ77NQLRKRUHOJ5M4FREMDZG6VWLAVCNFSM6AAAAABILQILR6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNRRHE3TMNBWGQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

https://k2-fsa.github.io/sherpa/onnx/tts/apk-engine.html

Yes. I will let you know when it is ready.

That would be much appreciated.
Thank you.
On Tue, Jun 11, 2024 at 10:22 PM Fangjun Kuang @.***>
wrote:

https://k2-fsa.github.io/sherpa/onnx/tts/apk-engine.html
Yes. I will let you know when it is ready.
—
Reply to this email directly, view it on GitHub
https://github.com/rhasspy/piper/issues/510#issuecomment-2161980166, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AKJ77NTEVZPTPS7HEH43JNTZG6WGZAVCNFSM6AAAAABILQILR6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNRRHE4DAMJWGY
.
You are receiving this because you authored the thread.Message ID:
@.***>

Released. Please try version 1.9.28
@mikebayus

How do I update?
do I just click on the link as before?  And must I uninstall the old one?
On Tue, Jun 11, 2024 at 11:32 PM Fangjun Kuang @.***>
wrote:

Released. Please try version 1.9.28
@mikebayus https://github.com/mikebayus
—
Reply to this email directly, view it on GitHub
https://github.com/rhasspy/piper/issues/510#issuecomment-2162037218, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AKJ77NWXYOQ2VPQZXKORLPDZG66MTAVCNFSM6AAAAABILQILR6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNRSGAZTOMRRHA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Download the app
Install it. (It will automatically replace the previous installed one)

Does it work?

Yes!
It does work.
I got scared though, because the #903 speaker ID was still showen in
Parentheses.
But when I restarted my phone, Speaker ID #351 was still the voice that
came up when Talkback was launched.
I got your EMail just as I was about to get into bed last night, or rather
this morning, as it was just after midnight.
i am in Florida USA, and therefore I am on EDT.
As I am blind, and as I use talkback to hear what is on screen, I must
locate the right file by hearing one file name read at a time.
So it takes me some time to find the speech engine that I want.
So when I got up this morning, and after a good Breakfast, I got to work
and the whole thing  took me about an hour.
That's why I didn't answer you right away last night.
Thank you so much.
I think we can close my Piper issue now.
Best,
Mike
On Wed, Jun 12, 2024 at 12:56 AM Fangjun Kuang @.***>
wrote:

Does it work?
—
Reply to this email directly, view it on GitHub
https://github.com/rhasspy/piper/issues/510#issuecomment-2162104894, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AKJ77NTVFS4WLVE7CFJYCY3ZG7IHBAVCNFSM6AAAAABILQILR6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNRSGEYDIOBZGQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

So it takes me some time to find the speech engine that I want.

Sorry. I should have sent you the link to the model directly for downloading.
Great to hear that it solves your issue.

Next time we'll know.
Let's keep in touch..
On Wed, Jun 12, 2024 at 10:07 AM Fangjun Kuang @.***>
wrote:

So it takes me some time to find the speech engine that I want.
Sorry. I should have sent you the link to the model directly for
downloading.
Great to hear that it solves your issue.
—
Reply to this email directly, view it on GitHub
https://github.com/rhasspy/piper/issues/510#issuecomment-2163104483, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AKJ77NQ4K2TT3UX6CEYA7ULZHBIZHAVCNFSM6AAAAABILQILR6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNRTGEYDINBYGM
.
You are receiving this because you were mentioned.Message ID:
@.***>

Issue fixxed.


Give names to anonymous structures and unions

This PR is to make FFI binding easier by giving names to all anonymous structures and unions in yaml.h. Some tools, like the OCaml ctypes library, expect everything to be named. I feel it should be harmless for the upstream to adopt similar changes already made in https://github.com/avsm/ocaml-yaml.
PS: I am not attached to these names at all. I just picked some most verbose ones, hoping that they would not cause problems in existing programs.

No longer being able to maintain this PR.


Undefined method `dig' in read_crossref

Try to use bolognese with the DOI 10.1111/nph.14619:
$ bolognese https://doi.org/10.1111/nph.14619
/var/lib/gems/2.3.0/gems/bolognese-0.9.65/lib/bolognese/readers/crossref_reader.rb:156:in `read_crossref': undefined method `dig' for nil:NilClass (NoMethodError)
        from /var/lib/gems/2.3.0/gems/bolognese-0.9.65/lib/bolognese/metadata.rb:113:in `initialize'
        from /var/lib/gems/2.3.0/gems/bolognese-0.9.65/lib/bolognese/cli.rb:32:in `new'
        from /var/lib/gems/2.3.0/gems/bolognese-0.9.65/lib/bolognese/cli.rb:32:in `convert'
        from /var/lib/gems/2.3.0/gems/thor-0.20.0/lib/thor/command.rb:27:in `run'
        from /var/lib/gems/2.3.0/gems/thor-0.20.0/lib/thor/invocation.rb:126:in `invoke_command'
        from /var/lib/gems/2.3.0/gems/thor-0.20.0/lib/thor.rb:387:in `dispatch'
        from /var/lib/gems/2.3.0/gems/thor-0.20.0/lib/thor/base.rb:466:in `start'
        from /var/lib/gems/2.3.0/gems/bolognese-0.9.65/bin/bolognese:6:in `<top (required)>'
        from /usr/local/bin/bolognese:23:in `load'
        from /usr/local/bin/bolognese:23:in `<main>'

The error is probably caused by Ruby version < 2.3, as dig was added then: https://www.tiagoamaro.com.br/2016/08/27/ruby-2-3-dig/
I will add a requirement for Ruby 2.3, as this method is heavily used in bolognese and a big time saver.

@flange-ipb I have created a new version v.0.9.66 that should be available via RubyGems in a few minutes. But that version doesn't really help you until you upgrade to Ruby 2.3 or 2.4, assuming that this is the underlying issue.

I'm using ruby version 2.3.1pl12.
It seems journal_issue is simply null, because meta.dig("crossref", "journal", "journal_issue") in crossref_reader.rb:112 is either never called or it returns null. Also check CrossRef's metadata for 10.1111/nph.14619 - it has no issue field.

Thanks, will create a test for that particular DOI to reproduce and fix this particular parsing issue.

@flange-ipb this should been fixed in the newest version that I just released, v.0.9.67. And as you said, the missing issue field wasn't handled properly.


Running from source with singularity

Description of the bug
I'm cloned the pipeline and I am trying to run using singularity so I can modify in an HPC environment. I've been successful with this using Docker on my laptop.
$ nextflow -log fresh_run.log run ./main.nf -profile singularity -dump-hashes
N E X T F L O W  ~  version 22.10.7
Unknown configuration profile: 'singularity'
Also, running the test via 'nextflow run f-core/rnaseq ... -profile test,singularity' works fine in the HPC environment.
Command used and terminal output
No response
Relevant files
No response
System information
No response

I was able to get past this by adding an entry for singularity in the nextflow.profile:
  singularity {
    process.container = 'quay.io/nextflow/rnaseq-nf:v1.1'
    singularity.enabled = true
  }

Now I have other errors to work through.

Hi @T300M256 ! That's not the right container for the pipeline!
Did you try:
nextflow run nf-core/rnaseq -profile singularity <OTHER_PARAMETERS>

Also might be worth exporting NXF_SINGULARITY_CACHEDIR so you reuse the Singularity containers.

Yes, I can do that successfully. However, I was exploring running from source. In retrospect my questions might be best suited for nf-core Slacks. Thanks.


Updated README to reference Visual Studio 2015.

Fixed README to reference Visual Studio 2015.
Updated link to Visual Studio download page.

Ah yeah, good shout thanks!

All merged

BTW, development is done on the develop branch, and master only contains releases, per Contributing


Reduce PEP-8 errors

See https://github.com/Senzing/g2-python/issues/123

https://github.com/Senzing/g2setupconfig/tree/issue-13.jae.1 already is up-to-date.


Fix off-by-one error in TemporalData train/test split

Currently, the index starting the validation (and test) set is displaced by +1
This is because int((self.t <= val_time).sum()) already adds 1 to the index since the tensor is 0-indexed.
As a simplifying example, using just a train-val split (no test set) of ratio 0.75-0.25,
import numpy as np
t = [0,0,0,0,1]
val_time = np.quantile(t, 0.75)
val_idx = int((t <= val_time).sum()) + 1
t[:val_idx], t[val_idx:]
>>> ([0, 0, 0, 0, 1], [])

The desired split should be ([0, 0, 0, 0], [1]) instead, which we can rectify by removing the + 1 to val_idx and test_idx.

Yes, you are absolutely right. Thanks for spotting!


[AVM Module Issue]: Missing Traffic Selector Policies

Check for previous/existing GitHub issues

[X] I have checked for previous/existing GitHub issues

Issue Type?
Feature Request
Module Name
avm/res/network/connection
(Optional) Module Version
0.1.2
Description
Despite the flag being available for usePolicyBasedTrafficSelectors there is no option to configure the traffic selectors in the module
(Optional) Correlation Id
No response

@marshalexander99, thanks for submitting this issue for the avm/res/network/connection module!

[!IMPORTANT]
A member of the @Azure/avm-res-network-connection-module-owners-bicep or @Azure/avm-res-network-connection-module-contributors-bicep team will review it soon!

Hey @fabmas, please triage the issue when you have a moment 💪

The property TrafficSelectoryPolicies has been added to the module and the corresponding PR is the following: #3057


[Flink] Bump Flink to 1.17.2

Which Delta project/connector is this regarding?

[ ] Spark
[ ] Standalone
[x] Flink
[ ] Kernel
[ ] Other (fill in here)

Description

This bumps the Flink version to 1.17.2 and resolves #2155
How was this patch tested?

Does this PR introduce any user-facing changes?

/cc @scottsand-db


Rebranding UI Comparison Table

Description
Added a Comparison Table for the landing page.
Type of change

[x] New feature (non-breaking change which adds functionality)

Screenshots/Sandbox (if appropriate/relevant):

@hasparus I think some changesets are pushed by accident. Could you revert them? Thanks!

chore(dependencies): updated changesets for modified dependencies

I wanted to ask, why does the GitHub Action add them when I didn't modify package.json nor the lockfile?
chore(dependencies): updated changesets for modified dependencies


golint fixes and pool_test additions

Hi,
I've fixed a number of golint complaints, and added a basic pool_test.go
Tested in linux & windows.
One controversial change might be in pool_x.go there was always a (needless?) bar.SetWidth which didn't seem to add anything in normal terminals, and broke the test cases.

I'll close the request then as I don't think you want to pull any of this...


Fix unit tests on Windows

To address #120

We should always be using forward slashes for package paths so the use of "path" which constrains to forwards slashes looks good to me.
ref: https://groups.google.com/g/Golang-nuts/c/bmKNJ1dkGiw/m/vgPCu6-5BAAJ


Gitter for this repo is set to private

As the title says. Seems like this shouldn't be private since a link to it is in the project readme, and the project is no longer private.

Apologies that Gitter link was unintentionally there. For questions and discussion, please use [Stack Overflow|http://stackoverflow.com/questions/tagged/falcor]

I tried asking a question on Stack Overflow and I'm getting downvoted because "StackOverflow is not a support form". My question doesn't seem like it would be appropriate to open an issue about, and it may be deleted from Stack Overflow for being off-topic (which I've had happen before). So where do I go? Twitter?


coot/ghc warnings

Part 1 of #2215

network-mux: GHC warning flags
ouroboros-network-framework: GHC warning flags

ps. I didn't started at the very bottom just to get to the top quicker.

bors merge

bors merge